Remove unnecessary OTHER_LDFLAGS from sample/skeleton Xcode projects

The extra -ObjC seems unnecessary at best. At worst, it makes interopability with systems that depend on setting OTHER_LDFLAGS (e.g. CocoaPods) annoying. Tracking down why your React Native app fails with a linker error is pretty annoying and near impossible for an Xcode novice.

Hmm, I guess I was wrong about the "unnecessary" part. It worked locally, but it seems the end-to-end tests invoke perf stuff which isn't otherwise referenced -- and so it's not packaged in the static library.
Hmm.
Is there another way to have the .xcodeproj default to -ObjC, or archieve the same result, without making it impossible for systems like CocoaPods to supply their own OTHER_LDFLAGS? My Xcode-fu isn't that strong, I'm afraid...

CocoaPods can specify their own ldflags since they let you run arbitrary Ruby in a post_install hook. So what you do is get programmatic access to the Xcode project file and add whatever build settings you want.

I'm not sure I'm explaining myself well here.
Let's say you'd like to use CocoaPods in your brand new React Native app project, let's call it MyAwesomeApp. Say you want to pull in all native dependencies, including the react-native iOS code, using CocoaPods. One reason why you might want this is because some of the react-native extension libraries you want to use contain native code which in turn depends on 3rd party frameworks, etc. One example here is the Facebook SDK which ships as a 3rd party iOS framework, and any react-native wrapper for it will depend on it as well as react-native headers.
Ok, so you're using CocoaPods to do all your native code dependencies. This means you can, nay, must delete references to React.xcodeproj, RCTImage.xcodeproj, etc. from MyAwesomeApp.xcodeproj and use the React pod instead. As a result, CocoaPods will build and link react-native (via React.podspec). It does the linking by defining a custom xcconfig file that sets OTHER_LDFLAGS to link to the static libraries it creates. However, because MyAwesomeApp.xcodeproj also defines OTHER_LDFLAGS by default, CocoaPods's value won't be seen by the linker and the build fails with a linker error.
It'd be nice to prevent that. It's super non-obvious for an Xcode n00b to find the "Other Linker Flags" setting in the Xcode project that was generated for them by react-native init and remove the setting.

You mention the post_install hook. I'm not sure the post_install hook has access to MyAwesomeApp.xcodeproj, although it's possible I missed something. It's also a one-time thing, so putting into the MyAwesomeApp/PodFile seems a bit overkill. Basically everybody would have to copy that hook into their PodFile and run it once to get rid of a default setting that won't apply to them...

Maybe the solution here is to have react-native init-cocoapods which generates a new project with a MyAwesomeApp.xcodeproj and PodFile that will work together out of the box...?

Ok I think I get it now. You might be able to specify "-ObjC" in the project's build settings, and specify "$(inherited)" in the target's build settings (which causes it to inherit from the project). Our setup matches what you've described and the target's build settings is a combination of flags from the parent project + cocoapods.

@ide thanks! I hadn't considered that. Trying that in b7242266785aa6a28093a58673e5d37cf43d8c54.

Yay looks like that worked!

awesome, glad it worked

@foghina would you mind taking a look at this pull request? It's been a while since the last commit was reviewed.

Hey @philikon! Thanks for making the pull request, but we are closing it due to inactivity (14 days with no activity) to make sure all pull requests are either being worked on or closed. If you want to get your proposed changes merged, please rebase your branch with master and send a new pull request :)
Does the comment above say this was resolved?

Trying that in b724226.
Yay looks like that worked!

Yes, I believe this was resolved :)


Windows Applications' Save As Dialogs don't Show the \Linux\ Folder

Version
Microsoft Windows [Version 10.0.22621.674]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
Visual Studio Code 1.72.2 with WSL extension
Notepad 11.2208.25.0
Repro Steps

 ctrl-n to create a new file in VS Code/Notepad
Add content to new file
ctrl-s to save new file

Expected Behavior

Application's save dialog should show the \Linux\Ubuntu-22.04 (\\wsl.localhost\Ubuntu-22.04) folder in the same way as Windows File Explorer does

Actual Behavior

Application's save dialog doesn't show the \Linux\Ubuntu-22.04 (\\wsl.localhost\Ubuntu-22.04) folder

Diagnostic Logs
No response

I am also looking for this feature, good issue description.

Workaround: Go to WSL in Windows, right click the folder of interest and select 'show more options' > 'Map network drive'. Now you can at least select the WSL instance folder as a network drive.

Still happening on a fresh Windows 11 install (23H2)


Filter Content with options

Hi, can I Filter Cart content by options??
using Eloquent (Where) for example.
I have a ['options' => ['type' => 'practice']]
and I want to get all items that have the type == practice.
Thanks you.

Cart::content() returns a Collection. So you can just use one of the methods available on that class to filter it down. 🙂


Starting User Services inside of LXD inside of Virtuozzo VPS failed

Required information

Distribution: Ubuntu 22.04
The output of "lxc info" or if that fails:

Kernel version: Linux 5.2.0 #1 SMP Tue Jan 25 12:49:12 MSK 2022 x86_64 x86_64 x86_64 GNU/Linux
LXC version: 5.12
LXD version: 5.12
Storage backend in use: dir

Issue description
User daemons don't start because the systemd user daemon does not start.
Everything else works except this (i.e. creating/starting/stopping/deleting containers, shell into the container, ssh into the container etc. etc.)
Steps to reproduce

install LXD from source inside of a Virtuozzo VPS
spawn a LXD container
in container, create a new user
in container, loginctl enable-lingering
in container, start a simple user daemon

Information to attach
root@container:~# systemctl start<EMAIL_ADDRESS>Job for<EMAIL_ADDRESS>failed because the control process exited with error code.
See "systemctl status<EMAIL_ADDRESS>and "journalctl -xeu<EMAIL_ADDRESS>for details.
root@container:~# systemctl status<EMAIL_ADDRESS>×<EMAIL_ADDRESS>- User Manager for UID 1001
     Loaded: loaded<EMAIL_ADDRESS>static)
    Drop-In<EMAIL_ADDRESS>             └─timeout.conf
     Active: failed (Result: exit-code) since Wed 2023-04-05 15:27:45 UTC; 4s ago
       Docs: man:user@.service(5)
    Process: 484 ExecStart=/lib/systemd/systemd --user (code=exited, status=1/FAILURE)
   Main PID: 484 (code=exited, status=1/FAILURE)
     CGroup<EMAIL_ADDRESS>
Apr 05 15:27:45 container systemd[1]: Starting User Manager for UID 1001...
Apr 05 15:27:45 container systemd[484]: pam_unix(systemd-user:session): session opened for user mynewuser(uid=1001) by (uid=0)
Apr 05 15:27:45 container systemd[484]: Failed to create<EMAIL_ADDRESS>control group: Permission denied
Apr 05 15:27:45 container systemd[484]: Failed to allocate manager object: Permission denied
Apr 05 15:27:45 container systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=1/FAILURE
Apr 05 15:27:45 container systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
Apr 05 15:27:45 container systemd[1]: Failed to start User Manager for UID 1001.

Can you test in a normal LXD setup, that is with LXD running on a physical system or VM?
I suspect it's Virtiozzo playing tricks with how the cgroup hierarchy is setup and not something we can do much about.

I wanted to move my setup from a Raspberry Pi to a VPS. What I could do is to reset the new host and recreate it with a KVM. The hoster seem to offer both. Do you think it will not exhibit the same issue?
Is there anything else I can run on the host or container to provide you more information (before I format the host)?

Having it run under KVM should avoid the issue.
I don't know that there's anything obvious we can capture to further debug this. OpenVZ/Virtuozzo kernels are known for having absolutely gigantic patchsets applied to them which can change a fair bit of how features like CGroups function, causing this kind of issue.
@mihalicyn has some past experience on that, but given this is a pretty unusual setup, I don't think it's worth the potentially massive amount of efforts to isolate what's wrong and trying to get the Virtuozzo folks to tweak their kernels.

Using KVM now, systemd user services work.

Good to hear!


Visibility for hand landmarks

The document says that there is a visibility value associated with each landmark in multi_hand_world_landmarks.
However, when I'm printing them out, they all seem to be 0.0 (which I'm pretty sure are the default values).
Is there a way to extract the visibility values for each hand landmark, or if my understanding is wrong?
I'm using the ready-to-go Python solution (Python 3.7.9) running on a Windows 10 desktop.

Hi @siyuan-peng, "multi_hand_world_landmarks" field  contains the hand landmarks on each detected hand in real-world 3D coordinates which is in meters with the origin at the hand's approximate geometric center. Could you please check with the latest version.

Hi @siyuan-peng, "multi_hand_world_landmarks" field contains the hand landmarks on each detected hand in real-world 3D coordinates which is in meters with the origin at the hand's approximate geometric center. Could you please check with the latest version.

The latest document still contains a visibility description.

@siyuan-peng thanks for the bug report. Indeed, it's a documentation bug - hand landmarks model doesn't output visibility. Documentation, should be fixed with the next release.


Fix incorrect attribute names for Glue Job Name constraint check

Description of changes:
PR #2816 added constraints for the Glue Job resource type, but with a wrong attribute name
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

hey @kddejong would be great, if you can review my PR, because it currently fails all AWS::Glue::Job resources, which set a value for Name 😄

Released in v0.79.6


Gui Idea

https://cdn.discordapp.com/attachments/697564089907675219/764985524548665354/unknown.png
Minimize half of the gui to be able to use it on both sides when kept open?

I'm not sure I understand what you're asking. Regardless, that screen is being redesigned, so it might not be possible to implement what you want.

The new inventory screen has only the left part, as the right part is made redundant by search improvements on the stations and depots tabs.

Ok, I'll see how the 0.4 will be. sorry for not responding i forgot about the post. as am not checking mail as often as the factorio page


Node 14 compatible

Main changes

chore: dependencies updated to the latest versions so we can use the library in node v14 environments

Other changes

chore: minor version upgraded
chore: .nvmrc file updated

Thanks @inigomarquinez. This is causing problems because the service won't boot up, and you get no errors (even using debug AFAIK)

Especially now that systemic is being bumped to Node 14


Cannot adjust other vertex after one vertex is deleted.

After enabling edit, i delete one vertex and then I cannot adjust other vertex.

I'd be great if you could provide a demo with the issue. Preferably on jsfiddle or codepen but you can also just send me a working demo. If I see the issue, I'll help and/or fix it asap.

You can test on your demo link.  Thanks! Will have it fixed by tomorrow!

hmm even the map isn't draggable anymore

ok @leekosal this is a leaflet issue and was fixed in 1.0.2. Update to leaflet 1.0.2 and it should be fixed. I'll update the demo asap.

thank you.

@leekosal did it work?

Yes it works now. Thank you.


Added support for actors

Adds support for actors.
Issues: https://github.com/krzysztofzablocki/Sourcery/issues/1077

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

  Generated by :no_entry_sign: Danger


chore: portalloc cleanup

Would be good to wrap up and land. :) @tvolk131 DM for a snappy response when ready.

Sorry, been swamped the past few days - will get to this tonight!


Manage camel-quarkus-catalog in camel-quarkus-bom to allow Camel K to

use Quarkus Platform BOMs #3347

This should have no impact at all on any tests. Let's merge without wasting energy.


Print warnings when unable to find git dir or get git sha

Print warnings when unable to find git dir or get git sha.
When git fails to find the git dir, or fails to get the git sha, no information about why is communicated. This makes it difficult to debug why a git revision is not available.
A message on the console stating what error occurred would probably help users correct the problem.
We can use the cargo:warning= directive to communicate errors getting the git revision.

This is already done.


Http server v2

from v1:

Added function to replace a string only if the content changed
Added helper to append a char to a buffer
Manage the response content directly (sol-buffer) instead of create strings and then set them.
The virtual response_cb only set the value. Before it was setting the key (path) making harder to use it further on groups.

fixes on https://github.com/solettaproject/soletta/pull/839

pushed with the fixes, thanks for reviewing it


Fix Scala Native env variables values to match sbt

https://github.com/com-lihaoyi/mill/pull/1253 used valueOf which
given a Java enum it matches the name of the classes and not
the string values that match the official Scala Native env variables
values. This PR uses instead .values which returns all the values
then matches based on the String value the enums define.

My last PR had a problem I didn't notice. Sorry for that. Good that it didn't make in a stable release.
It was matching SCALANATIVE_MODE=ReleaseFast instead of SCALANATIVE_MODE=release-fast as sbt does.
I'm not sure about the phrasing of the error message. If you have a better idea, let me know.


syscall: linux-ppc64x (incorrectly?) treats R4 as a syscall return value

All of the linux-ppc64x Syscall functions return the contents of R4 as syscall return r2.
#42178 previously ran into issues because system calls were getting inconsistent r2 values. The short-term workaround there was to ignore r2 for AllThreadsSyscall, with the assumption that some syscalls may use r2.
On further investigation, it looks like that is not true.
The Linux ppc64 system call ABI documentation lists only R3 as a return value, with R4 getting clobbered.
The system call entrypoint I find a bit difficult to follow, as it is unclear exactly what function pointers are loaded by __LOAD_HANDLER, but not seeing lots of assembly implementations, I assume these are the C system call implementations.
syscall_exit_prepare seems pretty clear that it only expects to use R3 for the return value.
Thus my conclusion is that R4 is never a syscall return value and our syscall implementations should change to unconditionally returning 0 for r2.
Anyone know if I missed something that should keep R4 in use?
cc @minux @cherrymui @aclements @AndrewGMorgan @laboger

I agree what you said. I don't know why R4 was used. I think we should remove it.

cc @pmur

R4 is clobbered upon return from a syscall. Only R3 and CR0SO convey status on PPC64 via SYSCALL. Storing a consistent value of 0 seems much more reasonable than what is done now.


Automaton matcher rewrite

A new implementation of the scope automaton aka graph trie. This should scale much better. It is still very much work in progress for the moment.
Here is a list of things that must still be done (some of these might be in their own PRs)
TODO

benchmark performance
support weighted patterns
ability to remove symbols from scope for conflicting cases (also find a test case to demonstrate this)
more general edge properties for many-to-one edges
of course, fix tests
HUGR support

NB For review it's probably best to ignore the diff and just read the new code.


Remove unnecessary notFoundError log

What type of PR is this?
/kind cleanup

What this PR does / why we need it
When lws not found, it's mostly because of deleted, if we do not tolerate the notFoundError here, we'll have plenty of
unnecessary logs.
Which issue(s) this PR fixes

Fixes #
Special notes for your reviewer
Does this PR introduce a user-facing change?

None

/lgtm
/approve


QVM_URL endpoint for QPU use

Includes instructions on setting QVM_URL=https://job.rigetti.com/beta for QPU use.

@willzeng @ncrubin please take a look.

@cbosborn We will want to quickly move to have everything on one endpoint for ease of use.  This works in the short term to provide clarity.


[6438] Refactor pending trns page to include trn response

Context
https://trello.com/c/5qVI6mgI/6438-add-body-and-error-message-to-pending-trns-report
Changes proposed in this pull request
Before:

After:

Refactors the pending trn page layout to list all the trainees using the govuk accordion
Includes the trn request response details into each row for more details

Guidance to review
If this works, we can apply to other layouts. The accordion allows us to add any type of information without layout constraints in each row if needed. We regards to filtering/sorting, we could simply have these as buttons in a sort of "toolbar" above the results which would then filter based on the query params 🤷.

 Add body and error message to Pending TRNS report

I can see why you've implemented it this way @darokel but I'm not sure this is the best long term solution. Ideally the accordion is used to toggle visibility on a subset of content but the lists of data here are quite long and each currently just shows a name with 3 buttons.
Off the top of my head it could be more elegant to use a similar pattern to the a trainee lists we currently have. The 'card' would currently include the name and actions with filtering in a sidebar and sorting above?
Or we could investigate a multiselect table rather than duplicating buttons for every instance.
The accordion you have does work functionally but doesn't feel as though it's the intended use case.

I can see why you've implemented it this way @darokel but I'm not sure this is the best long term solution. Ideally the accordion is used to toggle visibility on a subset of content but the lists of data here are quite long and each currently just shows a name with 3 buttons.
Off the top of my head it could be more elegant to use a similar pattern to the a trainee lists we currently have. The 'card' would currently include the name and actions with filtering in a sidebar and sorting above?
Or we could investigate a multiselect table rather than duplicating buttons for every instance.
The accordion you have does work functionally but doesn't feel as though it's the intended use case.

Yeah this slightly abuses the accordion. The current data in the review app isn't real so it's not just those 3 buttons. The original requirement is to show a details toggle thing which expands into quite a bit of diagnosis information about the trn request. Using this breaks the table markup completely so thought the accordion space could fill in any number of additional information DQT may want to see in the future.
The card rows could also work - I can give that a go

Decided the first solution is good enough as a temporary workaround for now. But a proper mock up will be needed to decide how to build these sort of pages in the support console


brm2_thicktarget port

Port SSW IDL code as is to python as a starting point to #21
There are some very obvious and some not so obvious improvements that can be made, units for example. But I think without some upfront design even the improved code would ultimate suffer from the same limitations so I don't know if this would be time well spent or not. The main issue is the spread of different logical functions mixed into single implented functions or procedures. I thinks the most beneficial route would be to try and separate these logical functions.
At a high level the task is given a array of photon energies and an electron distribution, low, high electron energy integration limits or cutoffs and a model of the electron plasma interaction calculate the photon flux at the given photon energies.
So looking at the current code and general task there seems to be three major components that can be generalised and abstracted to some degree:

generating electron flux distributions or integrals (e.g. power law, broken power law, kappa, ...)
model the electron plasma interaction (thin, thick, ....?)
integrating the election distribution and model from the current photon energy to high electron energy for each photon energy

current integration code can probably be replaced with scipy quadrature and or fixed_quad but ideally should leave open to other integration schemes

We may want to keep this code as is or at least a cleaned up version for reference and comparison to what we eventually end with?

I was looking at the roadmap and it suggested that we wait for Richard(me) to put my changes to the non-thermal brem code online.  I wouldn't wait for me to get to that. I have pressing RHESSI issues and I don't think it serves a purpose. Both Shane and Danny understand the primary code issues for the numpy version and I forget who is ahead on coding them. It's perfectly fine to build the new code without updating the old as the newer version won't produce different results. Maybe I can't find them here but the python non-thermal codes should be here in developmental branches even at this fetal stage well past embryonic.  And none of the IDL code came down from Mt Sinai with Moshe.

Yea so tox lets us test against different versions of python so python3.6  (py36) python3.7 (py37).

When is our next tag up, meet up, telecon? Is there a place where this is
posted?
On Tue, Mar 31, 2020 at 11:39 AM Laura Hayes<EMAIL_ADDRESS>wrote:

@hayesla approved this pull request.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sunpy/sunxspex/pull/23#pullrequestreview-384859919,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADMUMX2Y3MYY7UCDCPHIUOTRKIFBLANCNFSM4JPWDCMQ
.


ModuleNotFoundError

Instrument
No response
What happened?
The first time I tried to execute this code, I got the following error message.
I thought maybe the PATH wasn't going through, so I messed around with it, but now I'm not sure. What should I do?
Error traceback output
File "C:\Users\toshi\Desktop\jwst\eureka\test_data\JWST\JWST\run_eureka.py", line 2, in
import eureka.lib.plots
ModuleNotFoundError: No module named 'eureka.lib'
What operating system are you using?
Windows 10
What version of Python are you running?
Python 3.10
What Python packages do you have installed?
 Please follow very carefully the installation instructions on our readthedocs page, and let me know whether or not that resolves your issue

Thanks for pointing that out. I followed it and when I tried to install it, I got the following error
(omitted)......
Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [50 lines of output]

...(omitted)...
raise ValueError("path '%s' cannot be absolute" % pathname)
      ValueError: path '/home/runner/work/drizzle/drizzle/src/cdrizzleapi.c' cannot be absolute
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

I don't seem to be able to install drizzle on Windows, but I wonder if that is the cause of this error.

Ah, so I think the only way you'll be able to install on a Windows computer are by using the Windows Subsystem for Linux (you can read tutorials on how to set that up online) or by dual booting Linux on your machine (sometimes simpler than WSL, sometimes more complicated; again, there's many tutorials online). I've done both in the past, and have both currently working on my personal laptop. Sadly many packages don't natively support Windows


MacType not working in Nightly Chatterino & window outline issue

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
The window outline in the Nightly version of Chatterino is more pronounced in both the unfocused and focused states. I think the outline in the Stable version of Chatterino looked better, as it remained consistent in both states. Especially, the left window border is noticeably more visible when the window is focused in the Nightly version. (See screenshot.)
Also, the window in the Nightly version must be resized carefully to have the chat line separator appear correctly, whereas this issue does not occur in the Stable version.
Lastly, MacType doesn't work in the Nightly version, though it works fine in the Stable version. When using the MacType Process Manager, I can see that MacType is injecting but the font doesn't change.
(I'd like to quickly mention that I really appreciate all the Chatterino contributors and thank you for all the work being done on Chatterino. )
Screenshots
https://github.com/user-attachments/assets/dfac0ece-9d5a-4222-b0d7-571302bd1a28

OS and Chatterino Version
Chatterino Nightly 2.5.1 (commit e1aded05) built on 2024-11-30 with Qt 6.7.1, MSVC 194234433, Crashpad Running on Windows 11 Version 24H2, kernel: 10.0.26120

The border thing is a duplicate of https://github.com/Chatterino/chatterino2/issues/5645.

Lastly, MacType doesn't work in the Nightly version, though it works fine in the Stable version. When using the MacType Process Manager, I can see that MacType is injecting but the font doesn't change.

Not our issue imo. We use Qt's font rendering, which uses GDI+ (Qt <6.8) and for fonts that require emulation uses DirectWrite. You can change the fontengine by passing arguments or environment variables.

Could you please write simple instructions on how to change the font engine to the older one currently used in Stable?

At least in 6.7, the font engine should be the same (assuming stable runs on something like 6.5). You can change the fontengine like this:
chatterino.exe -platform windows:fontengine=freetype

Or replace freetype with gdi (6.8) or directwrite. With freetype, you might be able to change its behavior using FREETYPE_PROPERTIES, but I'm not sure if that works here. I don't know what exactly MacType does, so I can't really help there. Either way, this is an issue with MacType not being able to inject its stuff into Qt.
Alternatively, you can pass nodirectwrite (chatterino.exe -platform windows:nodirectwrite to disable DirectWrite.


Remove extra slash in project mount

Old code was adding extra slash
 "-v" "/Users/runner/work/my-rust-project://Users/runner/work/my-rust-project:Z"

I remember there being a specific reason for this. Ill try and dig it up unless someone else knows why.


change fprintf to use unsigned format specifier

Thanks! How'd you find this? By code audit I assume? Don't think sid could ever get large enough to cause a bug there.
Still good to have it be correct. I rewrote the commit message to be clear as to which system it touched.

@dormando I usually look at all the pull requests, and see if I can spot anything that may have been missed.

awww :) That's super nice. You should drop +1's somewhere so I can tell someone's listening.
Development speed will be a bit faster for a little while at least, fwiw.

No problem. This is probably my favorite piece of software :) I will!
That's good to hear. I'm still learning the internals and I hope to provide more contributions as time goes on.

If you're interested in contributing feel free to stay in contact (via e-mail/IRC/etc) and ask questions. I can also parcel out small tasks and specific reviews. would be great to get a few people up to speed while I have time.

@dormando absolutely. i would be interested in learning about what work needs to be done and what needs to be reviewed. My email can be found in the commits I believe. Or If you want me to email you first, I can, I just don't see your mail address.
-Ryan


CANNOT OVERRIDE INHERITED MEMBER

Hello, Dev.
Please, I am currently working on a c# project and I was getting these errors:
Compilation error (line 42, col 22): 'SimpleGoal.MarkCompleted()': cannot override inherited member 'Goal.MarkCompleted()' because it is not marked virtual, abstract, or override
Compilation error (line 90, col 22): 'ChecklistGoal.MarkCompleted()': cannot override inherited member 'Goal.MarkCompleted()' because it is not marked virtual, abstract, or override
Compilation error (line 167, col 23): 'GoalTracker.NegativeGoal.MarkCompleted()': cannot override inherited member 'Goal.MarkCompleted()' because it is not marked virtual, abstract, or override
Thank you in anticipation as you help me solve this problem.
Here is the code that I am working on:
using System;
using System.Collections.Generic;
// Base class for all goals
public class Goal
{
// Private attributes
private string name;
private int points;
private bool completed;
// Constructor
public Goal(string name, int points)
{
this.name = name;
this.points = points;
this.completed = false;
}
// Mark this goal as completed and return the points earned
public int MarkCompleted()
{
	completed = true;
	return points;
}

// Get a string description of this goal
public virtual string GetDescription()
{
	return name + " (" + points + " points) [" + (completed ? "X" : " ") + "]";
}

}
// Subclass for simple goals that can be marked complete
public class SimpleGoal : Goal
{
// Constructor
public SimpleGoal(string name, int points) : base(name, points)
{
}
// Override MarkCompleted method to simply call the parent implementation
public override int MarkCompleted()
{
	return base.MarkCompleted();
}

}
// Subclass for eternal goals that are never complete but each recording earns points
public class EternalGoal : Goal
{
// Private attributes
private int pointsPerRecording;
// Constructor
public EternalGoal(string name, int pointsPerRecording) : base(name, 0)
{
this.pointsPerRecording = pointsPerRecording;
}
// Record this goal and return the total points earned
public int RecordGoal()
{
	int pointsEarned = base.MarkCompleted();
	pointsEarned += pointsPerRecording;
	return pointsEarned;
}

// Override GetDescription to show the total points earned so far
public override string GetDescription()
{
	return name + " (" + pointsPerRecording + " points per recording, " + base.GetDescription() + ": " + pointsPerRecording * Convert.ToInt32(completed) + ")";
}

}
// Subclass for checklist goals that must be completed a certain number of times
public class ChecklistGoal : Goal
{
// Private attributes
private int pointsPerCompletion;
private int targetCount;
private int completedCount;
// Constructor
public ChecklistGoal(string name, int pointsPerCompletion, int targetCount) : base(name, 0)
{
this.pointsPerCompletion = pointsPerCompletion;
this.targetCount = targetCount;
this.completedCount = 0;
}
// Mark this goal as completed and return the points earned
public override int MarkCompleted()
{
	int pointsEarned = base.MarkCompleted();
	completedCount++;
	if (completedCount == targetCount)
	{
		pointsEarned += pointsPerCompletion * targetCount;
	}
	else
	{
		pointsEarned += pointsPerCompletion;
	}

	return pointsEarned;
}

// Override GetDescription to show the completion status
public override string GetDescription()
{
	return name + " (" + pointsPerCompletion + " points per completion, " + base.GetDescription() + ": Completed " + completedCount + "/" + targetCount + " times)";
}

}
// Class for tracking the user's goals and score
public class GoalTracker
{
// Private attributes
private List goals;
private int score;
// Constructor
public GoalTracker()
{
goals = new List();
score = 0;
}
// Add a new goal to the list of goals
public void AddGoal(Goal goal)
{
	goals.Add(goal);
}

// Mark a goal as completed
public class LargeGoal : Goal
{
	private int progress;
	private int target;
	public LargeGoal(string name, int points, int target) : base(name, points)
	{
		this.target = target;
		this.progress = 0;
	}

	public void AddProgress(int amount)
	{
		this.progress += amount;
		if (this.progress >= this.target)
		{
			this.completed = true;
			this.points += 500;
		}
	}

	public override string GetDescription()
	{
		string status = this.completed ? "Completed" : "Incomplete";
		return this.name + ": " + status + ", Progress: " + this.progress + "/" + this.target;
	}
}

public class NegativeGoal : Goal
{
	public NegativeGoal(string name, int points) : base(name, points)
	{
		this.points *= -1;
	}

	public override int MarkCompleted()
	{
		this.completed = true;
		return this.points;
	}

	public override string GetDescription()
	{
		return this.name + ": " + (this.completed ? "Completed" : "Incomplete") + ", " + this.points + " points";
	}
}

public class GoalTracker1
{
	private List<Goal> goals;
	private int score;
	public GoalTracker1()
	{
		this.goals = new List<Goal>();
		this.score = 0;
	}

	public void AddGoal(Goal goal)
	{
		this.goals.Add(goal);
	}

	public void MarkGoalCompleted(string goalName)
	{
		foreach (Goal goal in this.goals)
		{
			if (goal.GetName() == goalName)
			{
				this.score += goal.MarkCompleted();
				return;
			}
		}
	}

	public void RecordGoal(string goalName)
	{
		foreach (Goal goal in this.goals)
		{
			if (goal.GetName() == goalName)
			{
				this.score += goal.RecordGoal();
				return;
			}
		}
	}

	public List<string> GetGoalList()
	{
		List<string> goalList = new List<string>();
		foreach (Goal goal in this.goals)
		{
			goalList.Add(goal.GetDescription());
		}

		return goalList;
	}

	public int GetScore()
	{
		return this.score;
	}

	public void SaveData(string fileName)
	{
		using (StreamWriter writer = new StreamWriter(fileName))
		{
			writer.WriteLine(this.score);
			foreach (Goal goal in this.goals)
			{
				writer.WriteLine(goal.Serialize());
			}
		}
	}

	public void LoadData(string fileName)
	{
		this.goals.Clear();
		using (StreamReader reader = new StreamReader(fileName))
		{
			this.score = int.Parse(reader.ReadLine());
			string line;
			while ((line = reader.ReadLine()) != null)
			{
				Goal goal = Goal.Deserialize(line);
				this.goals.Add(goal);
			}
		}
	}
}

}

@byui-webster StackOverflow or a similar Q&A site is a better place to ask for help. This repo is specifically for feedback related to .NET documentation.


Cannot cancel my Azure subscription

I am trying vainly to cancel an Azure subscription. I have been redirected here from the Azure portal for some reason. I have also been trying to open a support ticket, for the same purpose but everything on the relevant bits of the portal is either greyed out or the logic is circular, so I end up back where I started. Please call me on +447774111303 or email me on<EMAIL_ADDRESS>Thanks
Richard Edwards
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: e76ce485-5a06-add0-76a3-b823fb6c3c7a
Version Independent ID: af78deae-d409-6261-a817-b504965d1988
Content: Cancel your Azure subscription - Microsoft Cost Management
Content Source: articles/cost-management-billing/manage/cancel-azure-subscription.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@RichardHEdwards
Thanks for your feedback! We will investigate and update as appropriate.

Thanks. Bills keep arriving, for a service I can’t use and can’t cancel! Your reply is most welcome, as ‘support’ have not responded to my emails at all. Yours RE Richard Edwards +447774111303 mobile+441189713339 landline Thee/Thy, ThouSent from Mail for Windows 10 From: AjayBathini-MSFTSent: 01 December 2023 16:15To: MicrosoftDocs/azure-docsCc: RichardHEdwards; MentionSubject: Re: [MicrosoftDocs/azure-docs] Cannot cancel my Azure subscription (Issue<EMAIL_ADDRESS>for your feedback! We will investigate and update as appropriate.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>
@RichardHEdwards - Hello , Thank you for reaching out to us and for being patient.
For us to troubleshoot further, we would like to put you in touch with a support engineer so you can receive 1:1 support for a deeper investigation on this.
We would need you to send us an email at<EMAIL_ADDRESS>by providing the details below.
Subject: ATTN:Monika

Subscription ID
A link to this thread.
Please email us these details and we will work with you closely to get this issue taken care of and also post any findings here in the thread so the community can benefit from it.
Thanks in advance. Looking forward for your email.

@RichardHEdwards We will be working with you offline. As there are no further documentation changes, we will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


Update rooms-licensing.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/6528
Added link

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dstrome PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@dstrome PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Client authentication with Android KeyStore

Hi there,
we are required to use the Android KeyStore for our private key. But we also need to use that key for a TLS connection as client authentication. As it is not possible to extract the whole private key from the Android KeyStore to provide it to openssl - is it possible to delegate the required crypto functions that openssl needs to the Android KeyStore?
https://developer.android.com/training/articles/keystore
Best regards

Engines (in 1.1.1) and providers (in 3.0) are the way to do that.


Improvements to standalone forms

There are a number of issues with the current standalone forms implementation. I'm strongly recommending pulling it from live until we've got these. This is a priority issue that is required for us to cease using the old submission application's standalone forms.
Required functionality for us to leave the old app:

[x] They have no styling. Have them take on the same styling as the fund/lab forms (considering this a bug)
[ ] Add the ability to restrict form access to specified roles in the form editor. We need forms that are only available to the team, applicants, reviewers (and later for approved applicants from specific funds).
[x] Require a recaptcha (or some other anti-spam) field for forms that are publically available.
[ ] File attachment field
[x] A basic interface to review submissions (maybe this exists, can't tell cause currently erroring on test submissions)
[ ] Field to define team lead
[ ] Field to define slack channel to send notifications
[ ] Notification to site-wide slack channel on all submissions
[ ] Field to define webhook endpoint to send submissions to
[x] Hide none wagtail forms from /admin/forms/
[ ] Ability to preview completed form before submitting

Related PR is https://github.com/OpenTechFund/opentech.fund/pull/948

Since this is behind Cloudflare I do not think a captcha is needed. And if there are spam submissions we can do better than annoying captchas.
What is the "Team lead" field for. Just to direct notification?
The "webhook" field should end up in a separate issue I think, as an enhancement. I assume the system should not only ping the url put send some payload as well?

@Parbhat Can you take on the following tasks?

Add a file upload field to the available fields when you build a form.
Make it possible to restrict access to a form by role/group.

@danblah Wagtails forms on live is being blocked by Cloudflare cache, we have a page rule that tells Cloudflare to cache everything on www.opentech.fund. (By default Cloudflare only cache static assets like css, js and images, not HTML. So the cache bypass rules for test/dev/live is not really needed I think).
To get a page rule to bypass cache for pages with a cookie you would need to upgrade to a Cloudflare biz account. Overkill, and I think Cloudflare gets enough money already.
Could we decide that all wagtail forms get "-form" as part of their slug? I have added a page rule now that bypass the cache for "https://www.opentech.fund/-form".
The https://www.opentech.fund/about/jobs/test-form/ form seems to work now. I have also deployed the template fixes so the form looks like other forms on the site.

@frjo Thanks for fixing templates. Will work on the other requirements. Need some feedback on the second point.

Make it possible to restrict access to a form by role/group.

Do we need a form like this test form to be visible only to a certain type of users and not public? What should be the desired workflow? We can make a page accessible to users in specific groups using Private pages feature https://docs.wagtail.io/en/v2.4/advanced_topics/privacy.html

@Parbhat Private pages sounds like it should work. Nice that it's built it to wagtail.
OTF want to set up forms that e.g  only staff should have access to and others that only reviewers should see. It can be request for expense reimbursement for staff, questioners for reviewers etc.
Please notice that I added a third feature to your list above, about hiding none wagtail forms from the /admin/forms/ page.

What should be the desired workflow?
@Parbhat, ideally the form author can select which group/role they want to give access to in the form creation/edit page (for example https://apply.opentech.fund/admin/pages/814/edit/).

What is the "Team lead" field for. Just to direct notification?

@frjo, that's right. this would enable a notification to the site-wide channel in the same format as others, for example:
@lead A new submission has been submitted for <name/link to form>

The "webhook" field should end up in a separate issue I think, as an enhancement. I assume the system should not only ping the url put send some payload as well?

@frjo, that's right. the object/payload should have a name/value for as many of the fields being set in the edit page as possible.

Add a file upload field to the available fields when you build a form.

@Parbhat and @frjo, want to be sure these are private files, only accessible to defined roles (or team only if that's easier right now).

Hide none wagtail forms from /admin/forms/

@Parbhat, plus one to that page only showing wagtail forms (hiding all the ones created from the apply app).

@frjo For the file field upload option, should we allow to upload only images or other types of files as well? Wagtail forms do not have file upload feature but we can create it. Apply site has a custom implementation of forms using stream field. Do we want to look into the options to use stream field forms here as well instead of wagtail forms?
Created PR #963 to hide apply forms from /admin/forms/

@frjo For the file field upload option, should we allow to upload only images or other types of files as well? Wagtail forms do not have file upload feature but we can create it. Apply site has a custom implementation of forms using stream field. Do we want to look into the options to use stream field forms here as well instead of wagtail forms?
Created PR #963 to hide apply forms from /admin/forms/

Whoops, accidentally closed.

@frjo For the file field upload option, should we allow to upload only images or other types of files as well?

.md, .rtf, .txt, and all office and libreoffice formats + pdf, jpg and png.

Thanks @danblah

PR for 1. Add a file upload field to the available fields when you build a form. - #976
We will probably need an interface to review form submissions. As the default form submissions interface (In Wagtail admin sidebar), only displays text data. Currently storing the file name of uploaded file with every submission

This is not the application forms, it's the standalone forms. Not mush used I believe.

@frjo Am i correct in assuming these stand-alone forms are no longer used/rarely used therefore the improvements listed in checkboxes above are very not urgent?
I'm also assuming that 'stand alone forms' are application forms that are not connected to a fund/lab and are kind of like 'flash grants' @danblah have I made the correct(ish)assumption here? if so I'll update the issue body with some context.

Q1 2022 Rebasing updates:

Cleared old Milestone
Removed priority needed label
Asked fredrik how critical this is since it's 'not much used' and appears to have not been worked on since 2020.
Asked Dan for clarified context
Recommend review in Q2 rebasing for priority level.
Size estimated at M
Removed from dev backlog board: https://github.com/HyphaApp/hypha/projects/2

This is part of the public side of Hypha. A part soon no one will be using. I'm closing it and recommend we close all issues realting to the public side.


Add .property property

Fixes #112

@juanazam @jeradg ready for code review

@san650: There's a typo in your commit. In addon/index.js:
import { proeprty }    from './queries/property';         export { property };

@san650: Probably means that we need to test that the exports from addon/index.js actually work.

I'll add a test for it


Does OLM support downgrade

Bug Report

What did you do?
A clear and concise description of the steps you took (or insert a code snippet).

OLM extends Kubernetes to provide a declarative way to install, manage, and upgrade operators and their dependencies in a cluster.
There is not mentioned that OLM downgrade support. I just wonder to know if we support downgrade or not.
What did you expect to see?
A clear and concise description of what you expected to happen (or insert a code snippet).
When I use helm to upgrade, I can use helm downgrade to rollback the previous version. But from the document and my testing, I did not find the way how to download by OLM.

@camilamacedo86 @dinhxuanvu I saw there is a document mentioned that the downgrade - https://github.com/operator-framework/operator-lifecycle-manager/blob/master/doc/design/dependency-resolution.md#dependency-resolution
Could you please elaborate more details? Thanks.
FYI @gyliu513

The downgrade that has been mentioned in that document is referring more to an attempted upgrade that is reverting, rather than explicitly supporting an arbitrary downgrade. Philosophically, OLM is designed to always upgrade to the next version. If an emergency fix is needed, that fix should be made in a new CSV to be made available to OLM.

@jpeeler Thanks for you reply. If we want to rollback the previous version(I do not like this new version), there is no way by using olm. You have to reinstall.

I am interested in this topic as well and I would like to know what prevents OLM from support downgrading. Yes, it's possible to do downgrade by rolling forward, however, I think that's a hack and not a proper solution.

At the moment, OLM doesn't support downgrading but we are looking into the feature. However, there isn't a concrete plan for it yet. At the moment, downgrading is just another from of upgrading as far as OLM is concerned.
There are a few issues that OLM needs to resolve in order to enable downgrading. One of them is CRD and its CR handling. As you know, OLM is managing the CRD but not the CR created from the API provided by CRD. In order to downgrade and ensure no data loss, OLM needs to handle the CR migration before it can downgrade a CRD (if there are changes between this version of CRD and the prior one). Plus, if there are another operator depending on a specific API/CRD that this operator owns, then downgrading can potentially break those operators if there are incompatible changes in previous version. That situation needs to be handled as well. Furthermore, depending on when the downgrading happens, it may not even be possible if previous version is no longer applicable anymore due to cluster/k8s version upgrade and some k8s APIs have been deprecated.
This is an interesting feature that can be very helpful but there isn't an immediate plan for it right now.

@mhtgrwl5 As far as I know, OLM doesn't have any plans on rollback feature. However, you can ask questions on this on #olm-dev on Kubernetes slack channel.

@dinhxuanvu Thanks. I will investigate it further.

any news on this topic? I just had the case were keycloak updated to an incompatible version of our keycloak plugin. Having an option to downgrade/revert or just force a specific version would be useful.

Maybe you could add a new parameter on each install plan that if set to true, will redeploy. cos I can still see earlier install plans


[newrelic-k8s-metrics-adapter] Review the docs for values that could have changed

[x] Take a look if this work should be fone with helm-docs or frigate.
[x] Review that the values.yaml is properly documented to enable the creation of the README.md file automatically.
[x] Review docs.newrelic.com for any values.yaml that should be changed in the integrations/kubernetes integration.

Docs PR https://github.com/newrelic/docs-website/pull/6623

https://github.com/newrelic/docs-website/pull/6623


GH-35559: [Java] Implementing JDBC Flight Stream Result Set asynchronous VectorSchemaRoot Producer

Rationale for this change

The change leads to a performance increase when there is more than 2 million rows of data:

What changes are included in this PR?

Implement a double buffer blocking queue for VectorSchemaRoots.
Produce VectorSchemaRoots Asynchronously.
Add reading "buffersize" driver property to initalise blocking queue with a custom value.
Use VectorLoader and VectorUnloader.

Are these changes tested?

Yes. There is no change in functionality, so the existing tests are adequate.
Are there any user-facing changes?

No.

Closes: #35559

Want to rebase this? Looks like it won't build until conflicts are fixed

Is 0.5% performance improvement noticeable or even statistically significant?

Is 0.5% performance improvement noticeable or even statistically significant?

@lidavidm the posted results were taken from an automated test that we've discovered may not be the best indicator of performance. A more recent manual test fetching 500k rows has shown that this change led to a 4% faster performance - 114,256 ms vs 118,706 ms.
The performance enhancement is iterative, we wanted to create this PR early to engage the community and get feedback as early as possible. More results are pending as we are working with a customer to see results in their lab. Our customer has a use case that leads to a fast consumer, they move millions of rows over the network back and forth, so there was a hypothesis that implementing double buffering would lead to a performance increase.

Going to recreate the pull request in a cleaner fashion. Apologies all.


cookie path rewrite support, with new rewriting syntax.

Adding support for cookie path rewriting, according to syntax discussed at pull request #32

@gil @drewzboto Have you guys thought about the ability to support rewriting the cookie's domain? We're currently using this to proxy to backend servers on another domain and would like to be able to re-write the domain.
I'm going to try and work on a PR to do this as well. Thoughts?


Fix NPE with DelayedCollectionInitializer

A NPE is thrown due to DelayedCollectionInitializer relying on the AbstractCollectionInitializer behavior for resolveKey.  The overridden method does not register the resolution listener since the instance hasn't yet been resolved but only resolves the key.

Thanks @Naros


Google translate popup every time LS is refreshed

Describe the bug
Every time I refresh my instance of LS I get an annoying Google translate popup asking me if I want to translate Arabic to English.  It also happens when I log out of LS and log back in.

I've only observed this when running in Chrome.  Tried running on Edge and Brave browsers a few times and didn't see the popup.  Just wondering if anyone else has this issue?

OS: Ubuntu 20.04.6
Label Studio Version: 1.10.0

Nevermind.  My data contains some acronyms. That's what was causing it.

Actually I spoke too soon.  I did a clean install of LS on another machine and the popup was there on first log in.
Toggling back and fourth between Arabic and English the only difference I could see was the apostrophe in the word doesn't.
So with the Arabic translation it's,

and with the English it's,

Not sure if this helps.

We're not able to reproduce this issue. Do you have any extensions installed that might be screen-reading the page and causing this to happen?


Add --illegal-access=permit to run the tests

Please file a documentation issue so that we do not forget to document this when Java 16 rolls around officially. This only impacts when Weld is in the mix right?

Please file a documentation issue so that we do not forget to document this when Java 16 rolls around officially. This only impacts when Weld is in the mix right?

hello guys
after lombok configuration in eclipse the same error appeard and i don't know how to fix it
ps : i'm using jdk16


Enhaced Rate Us Page

Hello Project Admin
I Enhanced Rate Us Page - Solved #600
Change Made :
=> Added Emojis Instead of Stars
=> Added Toastify Message With Selected Emoji
=>Added Hover And Lighting Effect
=>Selected Emoji Size is Increased
Kindly Please go through it  @Anishkagupta04
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/assets/161804629/d58c99bc-fe06-4013-8500-e8417ae51486

@sailaja-adapa Resolve conflicts

@varshith257 i resolved issue kindly please go through it


How to use usbip in Docker container?

Out of the box a privileged docker container reports 2 vhci_vcd host controllers:
Bus 002 Device 001: ID 1d6b:0003 Linux 6.11.9-orbstack-00279-g4cf512143f2e vhci_hcd USB/IP Virtual Host Controller
Bus 001 Device 001: ID 1d6b:0002 Linux 6.11.9-orbstack-00279-g4cf512143f2e vhci_hcd USB/IP Virtual Host Controller
'usbip attach' works and lsusb shows a new device:
Bus 002 Device 001: ID 1d6b:0003 Linux 6.11.9-orbstack-00279-g4cf512143f2e vhci_hcd USB/IP Virtual Host Controller
Bus 001 Device 013: ID 0403:6001 FTDI FT232R USB UART
Bus 001 Device 001: ID 1d6b:0002 Linux 6.11.9-orbstack-00279-g4cf512143f2e vhci_hcd USB/IP Virtual Host Controller
No /dev/bus/usb entry is created, though. Manually creating one with 'mknod /dev/bus/usb/001/013 c 189 12' works, but device can not be accessed:
openat(AT_FDCWD, "/dev/bus/usb/001/013", O_RDWR|O_CLOEXEC) = -1 EPERM (Operation not permitted)
Might be caused by AppArmor or cgroups, but from my understanding this should not happen in a privileged container?

I can't use usbip either. Failed to attach device, which reports "Import device error".


Get AIRR format data from 10x Genomics data

I want to convert 10x Genomics single cell data to AIRR format. Is it possible?

Hi, @amufaamo !
Why do you need to convert format  10x Genomics to format AIRR? If you want to load your data in Immunarch, you could use standart immunarch function repload. It does not ideally work with single-cell data. We are currently working on imroving single-cell support.
Best regards,
Maria Samokhina

Thank you so much for your reply.
I understood immunarch does not fully support single cell data.
2022年8月5日(金) 5:30 Maria Samokhina @.***>:

Hi, @amufaamo https://github.com/amufaamo !
Why do you need to convert format 10x Genomics to format AIRR? If you want
to load your data in Immunarch, you could use standart immunarch function
repload. It does not ideally work with single-cell data. We are currently
working on imroving single-cell support.
Best regards,
Maria Samokhina
—
Reply to this email directly, view it on GitHub
https://github.com/immunomind/immunarch/issues/273#issuecomment-1205736369,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK5RGC52KBTT2NIBMWC3LNLVXQR63ANCNFSM54UII75Q
.
You are receiving this because you were mentioned.Message ID:
@.***>


chore(deps): update dependency zipp to 3.15.0

This PR contains the following updates:

Package
Update
Change

zipp
minor
==3.6.0 -> ==3.15.0

Release Notes

jaraco/zipp
v3.15.0
Compare Source
=======

gh-102209: test_implied_dirs_performance now tests
measures the time complexity experimentally.

v3.14.0
Compare Source
=======

Minor cleanup in tests, including #93.

v3.13.0
Compare Source
=======

In tests, add a fallback when func_timeout isn't available.

v3.12.1
Compare Source
=======

gh-101566: In CompleteDirs, override ZipFile.getinfo
to supply a ZipInfo for implied dirs.

v3.12.0
Compare Source
=======

gh-101144: Honor encoding as positional parameter
to Path.open() and Path.read_text().

v3.11.0
Compare Source
=======

#85: Added support for new methods on Path:

match
glob and rglob
relative_to
is_symlink

v3.10.0
Compare Source
=======

zipp is now a package.

v3.9.1
Compare Source
======

Removed 'print' expression in test_pickle.

bpo-43651: Apply io.text_encoding on Python 3.10 and later.

v3.9.0
Compare Source
======

#81: Path objects are now pickleable if they've been
constructed from pickleable objects. Any restored objects
will re-construct the zip file with the original arguments.

v3.8.1
Compare Source
======
Refreshed packaging.
Enrolled with Tidelift.
v3.8.0
Compare Source
======
Removed compatibility code.
v3.7.0
Compare Source
======
Require Python 3.7 or later.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Closing stale automated PR. Reopen if this was done in error


RateLimitAutoConfiguration$Bucket4jJCacheConfiguration

APPLICATION FAILED TO START

Description:
Parameter 0 of method jCache4jHazelcastRateLimiter in com.marcosbarbero.cloud.autoconfigure.zuul.ratelimit.RateLimitAutoConfiguration$Bucket4jJCacheConfiguration required a bean of type 'javax.cache.Cache' that could not be found.
Action:
Consider defining a bean of type 'javax.cache.Cache' in your configuration.

If you are using jcache you need to provide the cache bean


Builds depending on milli through git fail

Describe the bug
Building a Rust crate depending on milli (through this git repository) fails because the root /Cargo.lock is not being respected for some reason? Figured the Cargo.lock would be respected, but it obviously isn't and is causing issues with deserr.
To Reproduce
Cargo.toml:
milli = { git = "https://github.com/meilisearch/meilisearch", branch="main", package = "milli" }
Fix
Would you mind cherry-picking the diffs for milli 0.39.1 and 0.40.0 (and any others I may have missed) so that the builds will succeed again?

Hello @GregoryConrad
Thanks for your issue, we will do this it (re-importing the milli changes) into main once v1.0.0 is released (in 10 days).
It's easier for us. Sorry for the issue in the meantime

Sounds good! Thanks for the update.


Error in building from source on M1 MacOS

Bazel build -s envoy doesn't work:
$uname -a
Darwin mycomputer 21.1.0 Darwin Kernel Version 21.1.0: Wed Oct 13 17:33:24 PDT 2021; root:xnu-8019.41.5~1/RELEASE_ARM64_T8101 arm64
$bazel build -s envoy
ERROR: Traceback (most recent call last):
File "/xyz/external/v8/bazel/defs.bzl", line 396, column 28, in _v8_target_cpu_transition_impl
v8_target_cpu = mapping[settings["//command_line_option:cpu"]]
Error: key "darwin_arm64" not found in dictionary
ERROR: /xyz/external/v8/BUILD.bazel:3082:10: Errors encountered while applying Starlark transition
ERROR: Analysis of target '//:envoy' failed; build aborted:

I'm building on M1 without issue. What version of Bazel are you running (you should be using 5.0.0)? Do you have the full version of XCode installed? I build with a bunch of extensions compiled out using bazel build -c opt //source/exe:envoy-static.stripped

I do see Applications->Xcode in the Finder.
The .bazelversion file in the envoy directory has 5.0.0.
➜  envoy git:(main) ✗ cat .bazelversion
5.0.0
I see a recent commit with V8 and bazel changes and the error appears
related:
ERROR: Traceback (most recent call last):
File "/private/var/tmp/../external/v8/bazel/defs.bzl", line 396, column 28,
in _v8_target_cpu_transition_impl
v8_target_cpu = mapping[settings["//command_line_option:cpu"]]
Error: key "darwin_arm64" not found in dictionary
My naive reading is that V8 defs.bazel doesn't have a case for
"darwin_arm64" and needs a patch.
Thank you for your prompt response.
On Fri, Feb 11, 2022 at 11:42 PM moderation @.***>
wrote:

I'm building on M1 without issue. What version of Bazel are you running
(you should be using 5.0.0)? Do you have the full version of XCode
installed? I build with a bunch of extensions compiled out using bazel
build -c opt //source/exe:envoy-static.stripped
—
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/issues/19916#issuecomment-1036990166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ7YFQGJYNX45M4SZLQOGE3U2XQLFANCNFSM5OFRKFZA
.
You are receiving this because you authored the thread.Message ID:
@.***>

The .bazelversion in the code repo is not the installed version of Bazel you have. What is the output of bazel version when you run that from the command line?
I don't use any of the WASM functionality that requires v8 so I compile that out of my version with ~/.bazelrc including build --define wasm=disabled

➜  envoy git:(main) ✗ bazel --version
bazel 5.0.0
➜  envoy git:(main) ✗ bazel build -c opt //source/exe:envoy-static.stripped --define wasm=disabled
Target //source/exe:envoy-static.stripped up-to-date:
bazel-bin/source/exe/envoy-static.stripped
INFO: Build completed successfully, 2 total actions
➜  envoy git:(main) ✗ bazel build -c opt //source/exe:envoy-static.stripped
INFO: Build option --define has changed, discarding analysis cache.
ERROR: Traceback (most recent call last):
File "/...external/v8/bazel/defs.bzl", line 396, column 28, in _v8_target_cpu_transition_impl
v8_target_cpu = mapping[settings["//command_line_option:cpu"]]
Error: key "darwin_arm64" not found in dictionary
ERROR: /...external/v8/BUILD.bazel:3082:10: Errors encountered while applying Starlark transition
So wasm=disabled clearly works and v8 seems to be the problem.
I have not tried building from the other repo that @htuch commented above.

I submitted a fix for this here https://github.com/envoyproxy/envoy/pull/20216


fix invalid assertion

Scope & Purpose
Fix an invalid assertion that is triggered during testing.

[ ] Bug-Fix for devel-branch (i.e. no need for backports?)
[x] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)

Testing & Verification
This change is a trivial rework / code cleanup without any test coverage.
https://<IP_ADDRESS>/view/PR/job/arangodb-matrix-pr/7160/

Tests blue except unrelated hot_backup failure (which we try to catch).


add E2EE?

E2EE prevents unintended users, including third parties, from reading or modifying data when only the intended readers should have this access and ability. E2EE is used especially when privacy is of the utmost concern.
example: https://gist.github.com/therightstuff/84ae710c3c301fe7979d7f0aa9c7c147

this apps is only running in browser, no server & no data transferred outside, all data only live in indexedDB. I still don't know where and why to implement E2EE for this apps.


Native histograms: do not remote write exemplar if native histogram series is not sent

What did you do?
Set up Prometheus with native histograms feature enabled, scraping classic histograms as well, but disable sending native histograms to a remote storage.
What did you expect to see?
No errors on remote write.
What did you see instead? Under which circumstances?
Remote (Mimir) reporting:
ts=2024-02-07T09:30:14.28217803Z caller=grpc_logging.go:66 level=warn method=/httpgrpc.HTTP/Handle duration=19.66365ms msg=gRPC err="rpc error: code = Code(400) desc = failed pushing to ingester ingester-zone-c-0: user=17727: the exemplar has been rejected because the related series has not been ingested yet (err-mimir-exemplar-series-missing). The affected exemplar is {traceID=\"715a8ef985d1ccba\"} with timestamp 2024-02-07T09:30:09.794Z for series cortex_request_duration_seconds{cluster=\"...\", container=\"query-scheduler\", instance=\"query-scheduler-5c68646cbf-gnw7m:query-scheduler:http-metrics\", job=\".../query-scheduler\", method=\"GET\", namespace=\"...\", pod=\"query-scheduler-5c68646cbf-gnw7m\", prometheus_job=\"migrate\", route=\"debug_pprof\", status_code=\"200\", ws=\"false\"}\n"

Note that the series cortex_request_duration_seconds is a native histogram series and I disabled writing native histogram series to the remote, so this is totally correct error from Mimir's point of view.
System information
Kubernetes, amd64
Prometheus version
prometheus, version 2.49.1 (branch: HEAD, revision: 43e14844a33b65e2a396e3944272af8b3a494071)
  build user:       root@6d5f4c649d25
  build date:       20240115-16:58:43
  go version:       go1.21.6
  platform:         linux/amd64
  tags:             netgo,builtinassets,stringlabels

Prometheus configuration file
global:
        scrape_interval: 15s
    remote_write:
        - headers:
            X-Scope-OrgId: "**********"
          send_exemplars: true
          send_native_histograms: false
          url: http://***********.svc.cluster.local./api/prom/push
    scrape_configs:
        - job_name: histograms
          kubernetes_sd_configs:
            - role: pod
          metric_relabel_configs:
            - action: keep
              regex: cortex_request_duration_seconds.*
              source_labels:
                - __name__
          relabel_configs:
            ....
          scrape_classic_histograms: true

Alertmanager version
No response
Alertmanager configuration file
No response
Logs
ts=2024-02-07T10:11:49.283Z caller=dedupe.go:112 component=remote level=error remote_name=cd87f5 url=http://*********.svc.cluster.local./api/prom/push msg="non-recoverable error" count=1986 exemplarCount=14 err="server returned HTTP status 400 Bad Request: failed pushing to ingester ingester-zone-c-0: user=*****: the exemplar has been rejected because the related series has not been ingested yet (err-mimir-exemplar-series-missing). The affected exemplar is {traceID=\"66a9f18bba5436a9\"} with timestamp 2024-02-07T10:11:33.574Z for series cortex_request_duration_seconds{cluster=\"*****\", container=\"store-gateway\", instance=\"store-gateway-zone-a-0:store-gateway:http-metrics\", job=\"******/store-gateway-zone-a\", method=\"GET\", namespace=\"********\", pod=\"store-gateway-zone-a-0\", prometheus_job=\"migrate\", route=\"debug_pprof\", status_code=\"200\", ws=\"false\"}"

Not high priority.

Also it seems like remote write 2.0 will solve this by sending series and exemplars bundled together. Ref: #13105 . Should be possible to close after testing with remote write 2.0.


PARQUET-439: Conform copyright headers to ASF requirements

+1 LGTM.

Merged to master. Thanks!

thanks!


Notebook Routing

It would be nice to have a notebook for each environment. Currently, the Routing environment is missing a notebook.

Made PR #56 for this with some initial commits.


Spell check script

closing this PR and raising the new PR:
https://github.com/fetchai/cosmpy/pull/212


Updates

solved new daily puzzles
minor updates

Codecov Report

Merging #7 (0b203a7) into main (5c5e929) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main        #7    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           37        40     +3
  Lines         1328      1502   +174
==========================================
+ Hits          1328      1502   +174

Impacted Files
Coverage Δ

advent_of_code/year2015/day15.py
100.00% <100.00%> (ø)

advent_of_code/year2015/day24.py
100.00% <100.00%> (ø)

advent_of_code/year2016/day10.py
100.00% <100.00%> (ø)

advent_of_code/year2016/day11.py
100.00% <100.00%> (ø)

advent_of_code/year2016/day9.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c5e929...0b203a7. Read the comment docs.


Optimize rewriting module

Fixes #2275.
Some suggestions:

[x] Remove SQLRewriteEngine.originalSQL, use sqlStatement.getLogicSQL()
[x] For EncryptRule with SQLRewriteEngine, use current DatabaseType instead of DatabaseType.MySQL
[x] SQLRewriteEngine.isRewriteDistinctLiteral => isMeedRewriteDistinctLiteral
[x] SQLRewriteEngine.isContainsAggregationDistinctToken can move to sqlStatement, and let token type as an input parameter
[x] SelectStatement.getFirstSelectItemStartIndex() should as a Token and Split append distinct from rewriteInitialLiteral
[x] Make sure responsibility of rewriter and engine, only rewriter can use token
[x] EncryptSQLRewriter & ShardingSQLRewriter can new instance out of loop
[x] Make unify with stopPosition & stopIndex
[x] sqlBuilder is global field with SQLRewriteEngine, may avoid pass from private method by input parameter
[x] Make accurate with javadoc on SQLRewriter.rewrite
[x] Avoid start a new line if not exceed 200 char one line, at line 107 on EncryptSQLRewriter
[x] Make input parameter as same sequence for differnet methods, like: EncryptSQLRewriter.encryptInsertOptimizeResultUnit and EncryptSQLRewriter.encryptInsertOptimizeResult's param: unit add Predicate if Optional can not be absent, at line 105 on EncryptSQLRewriter
[x] InsertParameterUnit => InsertParameterGroup?
[x] Decouple SQLBuilder & ParameterBuilder
[x] Remove SchemaPlaceholder and test cases

Pull Request Test Coverage Report for Build 8202

46 of 55 (83.64%)  changed or added relevant lines in 13 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 60.406%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/filler/common/dml/SelectItemsFiller.java
0
1
0.0%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/SQLRewriteEngine.java
15
16
93.75%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/PreparedStatementExecutorWrapper.java
0
1
0.0%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/StatementExecutorWrapper.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/filler/common/dml/SelectItemFiller.java
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/filler/common/dml/SelectItemFiller.java
1
0.0%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/builder/ParameterBuilder.java
6
77.5%

Totals

Change from base Build 8199:
-0.02%

Covered Lines:
8101

Relevant Lines:
13411

💛  - Coveralls


fix(doc): add streaming parameter to ChatOpenAI

Add the streaming parameter to ChatOpenAI in the doc.
For agentExecutor.stream() to work properly, the ChatOpenAI streaming parameter must be true.

Hey @bracesproul
I made a small change in the document and only the commit I changed should appear, but I also see my previously approved commits in the pull request. Will this cause problems in the repo?

This should only be necessary for the streamLog method unless I'm missing something?


Updated tests and eqc to OTP20

Removing locks across the board.  This is causing continuous issues with getting a stable build as we change.  We will not .gitignore the rebar.lock in the root riak repo.


Could you please tell me how well Macad3D supports winform?

Could you please tell me how well Macad3D supports winform?

Does anyone know, please?

The GUI of Macad3D is realized in WPF. Perhaps you can be a little more precise in your question about where you expect Winforms to be supported?

Excuse me, among the dozen Macad3D projects shown in the picture, which ones can be directly used by Winform to develop new software without being limited to WPF?

@dipts

I see. The WPF dependency is introduced in the interaction tier, where user controls (e.g. property panels) are defined. The core layer (Macad.Core) itself does not access WPF functionality. The Macad.Exchange module defines the settings dialogue, if these are not used, the module should also be usable under WinForms.

I see. The WPF dependency is introduced in the interaction tier (Macad.Interaction), where user controls (e.g. property panels) are defined. The core tier (Macad.Core) itself does not access WPF functionality. The Macad.Exchange module defines the settings dialogue, if these are not used, the module should also be usable under WinForms.

Thank you for your patient answer. Then the remaining items such as Macad.Occt that you did not mention are what Winform and WPF can use in general, right?

Macad.Occt contains the wrapper for the OCCT C++ library and belongs to the core tier. You can look at the project dependency of Macad.Core, everything that is referenced there is WPF free. Everything that is added to Macad.Interaction uses WPF.
Winforms has a control called ElementHost, which can be used to host WPF controls in Winforms. However, I don't know how well this works.
Another tip: If you build an application on Macad.Core, you have to create or derive an instance of CoreContext. That's all you basically need.

Macad.Occt contains the wrapper for the OCCT C++ library and belongs to the core tier. You can look at the project dependency of Macad.Core, everything that is referenced there is WPF free. Everything that is added to Macad.Interaction uses WPF.
Winforms has a control called ElementHost, which can be used to host WPF controls in Winforms. However, I don't know how well this works.
Another tip: If you build an application on Macad.Core, you have to create an instance of CoreContext. That's all you basically need.

Got it, thank you again for your patience!


Using AUTH in url is unfriendly

The current method of passing in any auth would be through the registry url string, which is very unfriendly. And the method is already deprecated https://nodejs.org/api/url.html#url_urlobject_auth
We should expose an api to setup registry properly
i.e .setRegistry(url, auth, header, etc)

In my case I can not pass auth via registry url because of special symbols

Not strictly against it, in which cases do you want to pass additional headers to the schema registry?
Can you please both check the PR and comment if it fixes the issues?
@mac2000 Can you please give me an example which doesn't work to check if it is still an issue with the new version?

Looking at your code it should not be a problem
In my case imagine that you have @ symbol in password, which makes impossible to curl<EMAIL_ADDRESS>


[sysrst_ctrl,dv] Updated sysrst_ctrl checklist

Signed-off-by: Madhuri Patel<EMAIL_ADDRESS>

please fix the conflict and also change this to V2S
https://github.com/lowRISC/opentitan/blob/1ba8b40b2cff38ae2fe105ecce9b5d17a2dab725/hw/ip/sysrst_ctrl/data/sysrst_ctrl.prj.hjson#L14

all the AIs are done. Moving this to V2S


Linted Dockerfiles

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This Pull Request updates the Dockerfile and Dockerfile.hdfs to pass "linting" via the hadolint Dockerfile linter.
The previous Dockerfiles had numerous issues with generally regarded "best practices" for Dockerfiles.
There is probably some optimization (of image size) and lockdown (for security issues) that can still be done, this sets the files on a solid footing for approaching that work.
Checklist

[ x ] I have written necessary rustdoc comments  -- (updated the docker folder readme)
[ n/a ] I have added necessary unit tests and integration tests
[ n/a ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ no ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ n/a ] All checks passed in ./risedev check (or alias, ./risedev c)
[ no ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ no ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ no ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note

Dockerfile cleanup

I've resolved the merge conflicts.  I'm testing the builds on my laptop now and the pipeline is re-running.

It took a couple of tweaks, but builds with the new image work on my laptop now, and all of the pipeline tests here are passing.  LMK if you have any questions!

Hi @rotten , thank you for your interest and contribution to RisingWave.
We leave Dockerfile.hdfs as a standalone docker file because we only use it to manually package RisingWave images that support HDFS, and will not use it for automated testing and CI, so that your changes should ignore this file.

You could also consider adding the hdfs customizations as a different build target in the same docker file since the two files are almost identical.

(BTW, why the pr-title-checker GHA is not triggered? 🤔)


Issue when install prometheus and alertmanager

Hi,
When I install the prometheus and alertmanagers fails because de download url needs a diferent path:
Example puppet output :
err: /Stage[main]/Prometheus::Alert_manager::Install/Staging::File[alert_manager-0.5.0-alpha.0.tar.gz]/Exec[/opt/staging/prometheus/alert_manager-0.5.0-alpha.0.tar.gz]/returns: change from notrun to 0 failed: curl  -f -L -o /opt/staging/prometheus/alert_manager-0.5.0-alpha.0.tar.gz https://github.com/prometheus/alertmanager/releases/download/0.5.0-alpha.0/alertmanager-0.5.0-alpha.0.linux-amd64.tar.gz returned 22 instead of one of [0]

Current line on init.pp & alertmanager.pp
$real_download_url    = pick($download_url, "${download_url_base}/download/${version}/${package_name}-${version}.${os}-${arch}.${download_extension}")

Patch to work fine ( put only v${version} )
$real_download_url    = pick($download_url, "${download_url_base}/download/v${version}/${package_name}-${version}.${os}-${arch}.${download_extension}")

Thanks Oscar

Fixed in other release


⚠ Bofrost.at has degraded performance

In d078451, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1280 ms

Resolved: Bofrost.at performance has improved in 6119b35 after 8 minutes.


Add replacement to structured fields

closes #5899

@jsangmeister Maybe combine the merge with an update PR for the backend


Generalize CXCancellation optimization pass to generic gate-inverse pairs

What is the expected enhancement?
The CXCancellation pass currently only looks for adjacent CX-CX gates (on the same qubits) as opportunities for optimization, but it should support a broader set of both self-inverse gates and generic gate-inverse pairs.
One option (similar to what was done for CXDirection in #5609 ) would be to include in CXCancellation additional gate-inverse rules for which it can search the circuit to find applications.  It would be better though to add an optional attribute to Instruction instances to denote which Instruction (if any) is know ahead of time to be an instruction's inverse.

This PR #6855 can be used to complete this feature request -- all that would be required would be to search for the correct inputs for the new pass prior to running. I unlinked the PR to leave this issue open after it's merged.


[AIP-126] Pilot great godoc documentation.

This PR is an attempt to make documentation for a rule awesome.
It uses godoc to do the heavy lifting.
This requires making a function that returns the rule.
If we decide to go with this route, we will also probably need to tell rules what their godoc link is, because they will not be able to infer it as best as I can tell (rats).

We should not merge this until we talk about whether this is the strategy we want.
If we do want this strategy, we should probably shore up our docs everywhere since people are likely view them frequently. I have sent #250 as a down payment on this.

The godoc looks nice! I think we can remove the Details section, as it can be merged into the function's comment. How did you generate the godoc page?

Concur re: Details. I will do that.
I just ran the local godoc server (localhost:6060 and all that).


Problem with text selection in clickable places

Bug, feature request, or proposal:
If I try to select some text where in a location that has the click event, after selecting the text and releasing the mouse click, Angular triggers the click event, where it opens another page
sorry my english is not my main language
What is the expected behavior?
I hope that when you have some text selected in this environment with the click event, the click event will not be triggered
What is the current behavior?
when you release the mouse click the event is triggered even with selected text
What are the steps to reproduce?
Just select some text in a div that contains the click event
What is the use-case or motivation for changing an existing behavior?
on my system, the user navigates through cards similar to google inbox email. These cards have texts that are important to copy, but when clicked it takes the user to a new page, I just wish that if there were any selected text the click event would not be triggered
Any suggestions on how I can handle this?

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.

but this is what I'm reporting, it's a bug in relation to the click event
@josephperrott

The description you gave does not seem to be related to Angular Material.  For reporting a bug, please create a reproduction case of an issue with Angular Material and the team can take a look and try to figure things out.

ok tanks @josephperrott


grev/grevi issue with rs2 vs imm

It starts off by listing both grev ... rs2 and grevi ... imm.
Then the first paragraph says "It takes in a single register value and an immediate ..." which conflicts with the above and suggests the last operand must be an immediate.
Then the second paragraph says "This operation iteratively checks each bit i in rs2 ..." which suggests that the last operand must be a register.

Thanks. Also added the missing log2() to the 2nd paragraph.


Add codegen for BuilderOrderEntry

Codecov Report

Merging #1170 (769a0a1) into main (eaddd96) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 769a0a1 differs from pull request most recent head 99d83d3. Consider uploading reports for the commit 99d83d3 to get more accurate results

@@           Coverage Diff           @@
##             main    #1170   +/-   ##
=======================================
  Coverage   70.08%   70.08%
=======================================
  Files         133      133
  Lines        6718     6718
=======================================
  Hits         4708     4708
  Misses       1579     1579
  Partials      431      431

Impacted Files
Coverage Δ

pkg/apis/build/v1alpha2/builder_types.go
25.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Conversion of more classes to Kotlin

Some guidelines (which we should probably write down in the CONTRIBUTIONS.md)
Required Properties
Properties which can be considered as a required part of an expression, should be non-nullable and be initialised to a ProblemNode. In this case we can represent parsing problems in the graph and still avoid too many null checks. For example in the MemberExpression:
var base: Expression = ProblemExpression("could not parse base expression")

There might be cases, where either one or the other property might be required (if a property can either be an Expression or a Declaration, in this case we need to resort of having both properties nullable.
equals and hashCode
hashCode needs to include all properties that are also compared in equals. For easier readability, we should use the Kotlin expression body feature:
override fun hashCode() = Objects.hash(super.hashCode(), constructor, arguments)

Discussion with regards to required properties: Since we are only constructing nodes with the node builder, we could also just make them required in a constructor. Would that make more sense? I think so?
cc @KuechA @konradweiss

Yes, I wanted to propose the same thing. Right now, we have hundreds of properties which are nullable and seem to be optional but in reality, they aren't.

Yes, I wanted to propose the same thing. Right now, we have hundreds of properties which are nullable and seem to be optional but in reality, they aren't.

Perfect. Any preference in dealing it with a constructor vs. initialising it with ProblemExpressions ? Probably we could sort of both, for example make the expressions nullable in the node builder (in case something does not get parsed) and the node builder can then create a problemnode/expression and hand it to the constructor. In this case we would only really create problem nodes if there is an actual problem.

Perfect. Any preference in dealing it with a constructor vs. initialising it with ProblemExpressions ? Probably we could sort of both, for example make the expressions nullable in the node builder (in case something does not get parsed) and the node builder can then create a problemnode/expression and hand it to the constructor. In this case we would only really create problem nodes if there is an actual problem.

I'm not sure if it makes a real difference but somehow, I'd prefer to make the decision about a ProblemNode in the NodeBuilder. Also, I wouldn't change the NodeBuilder's interface to keep the impact "small" (lol)

Perfect. Any preference in dealing it with a constructor vs. initialising it with ProblemExpressions ? Probably we could sort of both, for example make the expressions nullable in the node builder (in case something does not get parsed) and the node builder can then create a problemnode/expression and hand it to the constructor. In this case we would only really create problem nodes if there is an actual problem.

I'm not sure if it makes a real difference but somehow, I'd prefer to make the decision about a ProblemNode in the NodeBuilder. Also, I wouldn't change the NodeBuilder's interface to keep the impact "small" (lol)

Sounds good. We just have to see whether Neo4j OGM needs some kind of default empty constructor - but as far as I know this gets magically injected in the newer versions.

Perfect. Any preference in dealing it with a constructor vs. initialising it with ProblemExpressions ? Probably we could sort of both, for example make the expressions nullable in the node builder (in case something does not get parsed) and the node builder can then create a problemnode/expression and hand it to the constructor. In this case we would only really create problem nodes if there is an actual problem.

I'm not sure if it makes a real difference but somehow, I'd prefer to make the decision about a ProblemNode in the NodeBuilder. Also, I wouldn't change the NodeBuilder's interface to keep the impact "small" (lol)

Sounds good. We just have to see whether Neo4j OGM needs some kind of default empty constructor for nodes - but as far as I know this gets magically injected in the newer versions.

I do like the idea, I am sceptical on the amount of properties that may not be set though, however, with the refactoring we also get some clarity on which once are currently not set in specific cases and can make a decision on whether or not we want to change it.


Package request: feishin

Project description
full-featured Subsonic/Jellyfin compatible desktop music player
Metadata

homepage URL: http://feishin.vercel.app/
source URL: https://github.com/jeffvli/feishin
license: GPL-3.0
platforms: linux, darwin

Could this possibly be based on the sonixd package? I'd imagine they share some similar properties.

Indeed. Feishin is a complete rewrite of Sonixd by the same developer.
From: https://github.com/jeffvli/sonixd

NOTICE: Maintenance Mode
The application is undergoing a full rewrite under the name Feishin. Check it out here!
Sonixd has entered maintenance mode as of 0.15.4. Only breaking bugs will be fixed. The application will continue to be available for download and use, but no new features will be added. The application will be maintained until the rewrite is complete.

Solved with https://github.com/NixOS/nixpkgs/pull/219005 🥳


Update MAIN_Introduction.md

minor edits plus visual element :) (delete if not deemed relevant ;) )

Can you do this again without the images? Keep the images in the  /images folder, leave a note, and it makes them easier to transfer into Eliademy at the time :)

oh, right you are - sorry, forgot about the internal image depot :) Now ready to be pulled ...

Awesome, thanks! Nice work :)


[web:canvaskit] excessive text layout in the web engine on material 3 demo

Switching between light and dark modes in the material 3 sample on the web is super janky. I haven't looked at HTML, but it does happen in CanvasKit mode. Profiling the app I see way too much text layout. Theoretically, there shouldn't be any layout at all, since only colors are changing. So even the framework shouldn't be laying out text.
But let's allow the framework re-layout text for some reason. The second problem is that the web engine re-lays out text during the paint phase. I'm suspecting that something broke the heuristic that attempts to save memory by proactively deleting paragraphs and then restoring them on repaint.

This is not right. We should fix this.
Reproduction

git clone git@github.com:flutter/samples.git
cd samples/material_3_demo
flutter run -d chrome --profile
Toggle the "Brightness" switch and observe janky transition
Use Chrome DevTools' "Performance" tab to record a profile
Observe a lot of text layout present in the profile (similar to the picture above)

Revisions
Flutter: 9a721c456d4a6418c60f2512cbb54ee420b9c69b
Samples: 3e9bb78aec619065bbb29cfb1d20a80026c63818

I think this would benefit greatly from https://github.com/flutter/samples/issues/1694
In the components screen, the component sections are not being built lazily. Each section is one child of the list view, so a lot of text that is off screen is being laid out.

Making each component category card a SliverList per https://github.com/flutter/samples/issues/1694 is a good idea and would certainly reduce the number of widgets that get built and laid out. Unfortunately, at the moment it would be rather difficult to create the current demo visuals using Slivers. We'll need SliverDecoration  to create something like the Cards for each component category.
The source of the performance problem here isn't just the cost of overbuilding etc widgets. Animation performance is still poor even when size of the window is reduced enough so that only one card from each column is visible.

Based on experience with much more involved/complex theming apps than the Material 3 demo, I don't think it has any issue that needs much fixing. I noticed it contains what might be considered an unusual amount of RepaintBoundaries that one might not normally have felt the need to insert. Reading the discussion above I'm guessing some of them where added to try to fix the poor SKIA web build performance and its theme transitions.
The same very poor performance can be seen in other web apps too built with SKIA, plus larger SKIA web builds just crashes after a while now, where VM builds and web HTML builds remain stable.
This issue https://github.com/flutter/flutter/issues/122189 seems to be on the right track concerning the root cause.

Quick update: the "excessive text layout" in the engine that I claimed above is only partially responsible for the slow down. I have a PoC where I removed the old caching heuristic and shut-off unnecessary re-layouts, but that brought the worst frames (the brightness switch in the demo) from 150ms down to only 90ms on my M1 laptop. An improvement, but far from ideal (we should be able to layout way more text than what's on the UI in 90ms). The frame continues to be dominated by the necessary text layout though (see comments by @HansMuller above).
I think the full fix for this will be what @jason-simmons is doing for https://github.com/flutter/flutter/issues/122189 (i.e. I agree with @rydmike's hypothesis).
In addition to the engine side, framework-side optimizations such as those suggested by @Piinks above could help even more.

This SKIA PR https://github.com/TheRakeshPurohit/skia/commit/c5fd6f7497d9bea72d17965bb9f0cf818378c941 mentions the issue https://github.com/flutter/flutter/issues/122189 as a fix for it. Anyone has some insights on when we might see it in the engine in master? Would love to try it out, sounded promising.

Not sure about the timeline, but @jason-simmons is working on it

Unfortunately https://github.com/flutter/engine/pull/40617 doesn't appear to have had an effect on the light/dark animation's rendering performance.


Update (Dev) Deps

@mistakia thank you for the PR! ❤ Sorry it took a while to get back to it. It looks fine to merge, so I'll go ahead and do that.

Thank you for contributing @mistakia!


registerMethod implementation?

Hi I think this library is really great but I noticed there's no registerMethod implementation. Lots of p5 libraries rely on this feature, including mine, p5.play!
I think making q5 compatible with p5.play would be perfect because my library only needs the 2D p5 functions. p5js is bloated and I think my users would especially benefit from q5 on mobile.
Let me know if you're currently working on a registerMethod implementation. If not I will try doing it.

I also noticed the q5xjs npm package is outdated. I would really need it to be kept up to date. If you're busy with other projects and aren't interested in maintaining q5xjs anymore, I'd like to submit my candidacy to maintain it for you! You can see how much work I've put into p5.play on my website and I'd be a big advocate for q5.
https://p5play.org

I would also need anything added by p5.js addon libraries to exist in the global scope or in q5 instances, not (only) in a separate windows.addons object.

btw I really like your code style! Short and concise. I'm gonna go ahead and try making all the necessary improvements to q5xjs so that it can work with p5.play. If you like what I end up with you can add me as a contributor to this project, otherwise I will make my own repo for it.

Here's my progress so far, I've made a lot of big upgrades!
https://github.com/quinton-ashley/q5js

I added automatic global instance creation to q5.js so p5.js sketches could be fully compatible with q5.js without the user needing new Q5('global'); at the beginning of their sketch. The neat part is I managed to figure out how to also keep your functionality of a user initiated start to the global q5 instance so users can still use q5 functions outside of the setup function!
I did it by adding a variable to check if a global instance of Q5 has been created yet.
function Q5(scope) {
	if (scope == 'global') Q5._global = true;

I put the following code at the end of the q5js file. It creates a new global Q5 instance after all scripts are loaded, if one hasn't been created by the user already.
document.addEventListener('DOMContentLoaded', () => {
	if (!Q5._global) new Q5('global');
});

I'll make a pull request with my changes soon. :)

Check out my list of what's new in version 1.0 😄
https://github.com/quinton-ashley/q5.js/blob/39c3662c454519c932c29544384cd3bf42dd75f3/q5.js#L372

There's a bug with sprite.rotateTo in p5play when using q5 that I'm gonna try to solve. After that I'll submit a pull request.


configure tailwind

@brianmuks I thought we were going to use AntD

True, we are going ahead with AntD for components and Tailwind for styling . Hope that makes sense. These two compliment each other. What's your thought ?

@brianmuks I thought we were going to use AntD

True, we are going ahead with AntD for components and Tailwind for styling . Hope that makes sense. These two compliment each other. What's your thought ?

Sounds good.


[Script]: AI text generation

Aim
Utilizes the transformers library to perform text generation using the GPT-3.5 architecture. You'll need to install the transformers library first
Details
Utilizes the transformers library to perform text generation using the GPT-3.5 architecture. You'll need to install the transformers library first
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

I want to work on this issue, please assign it to me