Ionic2 & how to access to seekTo() function?

Hi!
I'd like to create back & skip 30 seconds buttons so I need to access the seekTo(pos function.
I think this used to be accessible in the Ionic1 version but I cannot seem to find a way to access it at the current version.
Any suggestions?
Thanks!

i.e. of my current function:
    back30s($event) {
        var pos = this.currentTrack.progress;
        pos = pos - 30
        if (pos >= 0) {
            this.audioProvider.seekTo(pos); // wat do?
            // this.audioProvider.current.seekTo(pos); // why u no work?
        }
    }

PS: These are the only functions I can access:

Anybody has any idea? I'm trying to use this for an upcoming cross-platform app but I cannot figure out these tiny details for the life of me. Any help would be greatly appreciated 😃  Thanks!

Hi @tbergeron !
The seekTo() method belongs to the track prototype. So first you would need to get the current track from the AudioProvider and then you get access to the seekTo method:
let track = this.audioProvider.tracks[this.audioProvider.current];
track.seekTo(pos)

@arielfaur this worked wonders! thanks!
whenever you can, would you mind looking at this question too please? https://github.com/arielfaur/ionic-audio/issues/106#issuecomment-291692181 this is also a blocking issue for me thanks a whole lot for the help I greatly appreciate it :D


delete or modify existing contex rules

Hi I need to modify some of the existing context rule (change max_scope) etc. Is there a way to do this? Do I need to create a blank a new ConText and add my rules one by one?
Thanks

Probably the best way to go about things like this would be to copy the provided context rule json into your project and make modifications to this file locally.
https://github.com/medspacy/medspacy/blob/master/resources/context_rules.json
Then, when creating the context component in your pipeline add "rules" to your config dictionary.
nlp.add_pipe("medspacy_context", config={"rules":"path/to/your/context_rules.json", ...  })

ContextRules themselves should not be modified individually because the underlying spaCy Matchers that run the rules do not allow rule modification after creation.
This is definitely an area our documentation is missing and we are planning to do another documentation pass in the near future to add more explanations for dealing with issues like this.

Thanks, that's what I ended up doing anyways.


MINOR: [Java] Bump org.apache.derby:derby from <IP_ADDRESS> to <IP_ADDRESS> in /java

Rationale for this change
Bump to latest version that supports Java 11. See Apache Derby support matrix here https://db.apache.org/derby/derby_downloads.html
What changes are included in this PR?

Bump derby to <IP_ADDRESS>

Are these changes tested?
CI
Are there any user-facing changes?
No

@github-actions crossbow submit -g java

@github-actions crossbow submit -g java

Seems like we may need to file an issue for this C++ compile error.

Merging, since the failure is unrelated.


Jumpbox and forum widget links don't work

Links in the Jump To menu are broken.
Links in the Forums List widget are broken.

This bug was fixed in version 1.5.4.


Mention that ImageMagick must be installed in "Animated GIF support"

See #2505

Merged in 5eef0f0a2459755cec9324a52e5b95cdafe4c5f8 - thanks!


fix tests

Sorry I didn't see this @a-tarasyuk. It wasn't linked to #44638.

@sandersn Never mind, I saw your PR and closed it in favor of yours. I wanted to stay with the original test name, with a case-sensitive fix.


OS and libs upgrades

I will get these tested once merged.

I believe we did have some issues in the past where the Tech5 libraries weren't happy when we fiddled with the version.

That's great to know, thanks!


Activate only on command or Azurite file presence

Closes https://github.com/Azure/Azurite/issues/2467
Note
Optionally this could additionally add onStartupFinished and that will at least minimise the performance impact of Azurite on vscode startup, however if you're going to do that, no point in having any of the other activation events and they should all be removed.
Potentially considered a breaking change because a "new" workspace won't activate the extension and people may be used to seeing the blob/table/etc. links in the bottom status tray and they won't show up until you run an azurite command. Once run however and the azurite files are present in the workspace, it will auto-start from then on for the project.

@XiaoningLiu
Would you please help to review this PR?

@JustinGrote
Thanks for the contribution!
However, this breaking is not acceptable to Azurite.
We would expect a PR that make Azurite load faster in VSC, but not expect PR that won't load Azurite by default in VSC.

I will adjust it to OnStartupCompleted then, but this does not make you a good vscode citizen loading your extension in every workspace even when its not necessary, especially with your heavy startup times.

@blueww please re-review. Info about OnStartupFinished
https://code.visualstudio.com/api/references/activation-events#onStartupFinished

@blueww please re-review. Info about OnStartupFinished https://code.visualstudio.com/api/references/activation-events#onStartupFinished

Thanks @JustinGrote  for the sharing!
I see onStartupFinished will be activated some time after VS Code starts up. This is similar to the * activation event, but it will not slow down VS Code startup.
So I can understand "onStartupFinished" instead of "" can avoid slow down VS Code startup.
However, I see many other event are removed besides than "", will there be any side effect?

/azp run

@blueww please re-review. Info about OnStartupFinished https://code.visualstudio.com/api/references/activation-events#onStartupFinished

Thanks @JustinGrote  for the sharing!
I see onStartupFinished will be activated some time after VS Code starts up. This is similar to the * activation event, but it will not slow down VS Code startup.
So I can understand "onStartupFinished" instead of "" can avoid slow down VS Code startup.
However, I see many other event are removed besides than "", will there be any side effect?

No side effect, the other events are redundant if you are always activating on startup. They are what you should be using to minimize usage, but you have chosen to always run on startup even if it is not relevant to the workspace.

@XiaoningLiu , @EmmaZhu
Would you please help to review?

/azp run

/azp run

@JustinGrote
We have approved the PR, but the test failure blocks PR merge.
Would you please rebase/merge you PR with the latest change on main, so we can include the test fix in PR validation.

I have rebased as requested. FYI in the future, you have write access to pull request branches and can perform this rebase yourself without needing to wait on me, thanks!


Unable to exec into task as root

Description
I'm trying to exec into a running container as root to debug an issue, however I am not able to achieve a full-fledged root user as part of the filesystem is in read-only mode. I used to be able to achieve this with:
docker exec -it -u root <CONTAINER_ID> bash
however this does not work as expected with containerd..
Steps to reproduce the issue

ctr -n k8s.io task exec -t --exec-id 9999 --user root <CONTAINER_ID> bash
mkdir /root/test

Describe the results you received and expected
Error received:
mkdir: cannot create directory ‘/root/test’: No such file or directory
I am expecting to gain full root access to the filesystem and be able to create files and directories as normal root user should have.
What version of containerd are you using?
1.6.19
Any other relevant information
No response
Show configuration if it is related to CRI plugin.
No response

In the container, /root is exist?

In the container, /root is exist?

Yes and I can read its contents

hmm. But the error shows mkdir: cannot create directory ‘/root/test’: No such file or directory. Would you mind providing reproduce steps for it (including image)?  thanks

hmm. But the error shows mkdir: cannot create directory ‘/root/test’: No such file or directory. Would you mind providing reproduce steps for it (including image)? thanks

here is a screenshot of commands that did not succeed..

however it is possible to read even though when trying to append this file i faced an error:
bash: .bashrc: No such file or directory

@sagi-shimoni sorry, I can't reproduce the issue your report. Please provide the image you used or we can reproduce on it. thanks

@sagi-shimoni sorry, I can't reproduce the issue your report. Please provide the image you used or we can reproduce on it. thanks

hi, sorry the image is in a private repo... but you can use this Dockefile to build it and reproduce the issue
FROM ubuntu:22.04

RUN useradd -m ubuntu

USER ubuntu:ubuntu

Also, please see below the response we received from AWS Team regarding this issue:
Hello team,
Greetings for the day :)
Thank you for reaching out to AWS Premium Support. My name is Daeeun who is a support engineer for Containerization and based on Seattle. It is my pleasure to be able to assist you with your case today!
I understand that you've reacched out to us about the issue regarding accessing the container using ctr, which once access the container using "$ ctr -n k8s.io task exec", you would not be able to run mkdir command for creating directory named "test" under the /root directory due to the reason of "cannot create directory ‘/root/test’: No such file or directory", which you already reached out to containerd team from their github.
I went ahead to check if the issue is on the kubernetes part or containerd part by creating multiple instances on my replication environment, and tested the followings:

Create and run "$ docker exec" an NGINX container using Docker, and run "$ mkdir /root/test/"

/root/test/ has successfully created

Create and run "$ ctr task exec" an NGINX container using ctr with ContainerD from one of EKS worker node that is actively registered to one of cluster, and run "$ mkdir /root/test/"

Resulted "mkdir: cannot create directory '/root/test': No such file or directory"

Create a independent instance that is not registered as EKS worker node, install containerd and ctr, create and run "$ ctr task exec" an NGINX container, and run "$ mkdir /root/test/"

Resulted "mkdir: cannot create directory '/root/test': No such file or directory"

From the independent instance, tried to create directory using "$ mkdir" command under different directory then "/root"
-the "/test" directory was successfully created under the "/etc" and "/var", yet other directories returned the same response as "No such file or directory"
=====================================================================================
By replicating the issue, I was able to conclude that no matter if the instance is registered as EKS worker node or not, using containerd's "$ ctr task exec" to logging in to the container, there is limitation to write permission under several directories except "/etc/" and "/var/" instead of lossing writing permission to filesystem within containers. As I've checked that the issue is replicatable under the non-EKS-worker-node environment, it seems like that this might required attention from the containerd team. Therefore, I would like to request you to update the GitHub issue with the information that we were able to find, as containerd is an open-source project that is independent from AWS.

As for the workaround to access the container with the writing access, I tested if "$ kubectl exec" allows to create "/root/test" directory, and confirmed that "$ kubectl exec" is allow you to create folders under any directories that "crt task exec" returned "No such file or directory". Therefore, I would like to suggest to use "$ kubectl exec" as a workaround if it is required to use writing permission within the containers inside of pods for any Kubernetes clusters. Please find more information about accessing containers within pods using kubectl from Kubernetes document [1],
If there are any other questions or concerns that I can assist on AWS side, please feel free to reach out to me!
Have a great day :)
Reference:
[1] Kubernetes - Get a Shell to a Running Container
https://kubernetes.io/docs/tasks/debug/debug-application/get-shell-running-container
We value your feedback. Please share your experience by rating this and other correspondences in the AWS Support Center. You can rate a correspondence by selecting the stars in the top right corner of the correspondence.
Best regards,
Daeeun Y.
Amazon Web Services
To share your experience or contact us again about this case, please return to the AWS Support Center using the following URL: https://console.aws.amazon.com/support/home#/case/?displayId=13749835891&language=en
Note, this e-mail was sent from an address that cannot accept incoming e-mails. To respond to this case, please follow the link above to respond from your AWS Support Center.
===============================================================
Don’t miss messages from AWS Support when you need help! Update your contact information:
https://console.aws.amazon.com/billing/home#/account
If you receive an error message when visiting the contact information page, visit:
https://aws.amazon.com/premiumsupport/knowledge-center/iam-billing-access/
AWS Support:
https://aws.amazon.com/premiumsupport/knowledge-center/
AWS Documentation:
https://docs.aws.amazon.com/
AWS Cost Management:
https://aws.amazon.com/aws-cost-management/
AWS Training:
http://aws.amazon.com/training/
AWS Managed Services:
https://aws.amazon.com/managed-services/
Amazon Web Services, Inc. is a subsidiary of Amazon.com, Inc. Amazon.com is a registered trademark of Amazon.com.
This message was produced and distributed by Amazon Web Services, Inc. or its affiliates 410 Terry Ave. North, Seattle, WA 98109.
© [2023], Amazon Web Services, Inc. or its affiliates. All rights reserved. Read our Privacy Notice.

Also, please see below the response we received from AWS Team regarding this issue

@sagi-shimoni sorry, I can't reproduce the issue your report. Please provide the image you used or we can reproduce on it. thanks

hi, sorry the image is in a private repo... but you can use this Dockefile to build it and reproduce the issue
FROM ubuntu:22.04

RUN useradd -m ubuntu

USER ubuntu:ubuntu

@fuweid hi , were you able to reproduce using this Dockerfile ?

I was able to locally repro this issue on v1.6.19. The caveat is to have exec with --user
e.g.
ctr t exec --tty --exec-id 9999 --user root test bash

The --user option seemed break the user permissions. Before executing the exec, mkdir was fine in the init task. After the exec, mkdir failed for both init and exec tasks.
That said, the issue seems to be fixed in v1.6.21 by #8357.

thank you for the update!
closing this issue


Add wrapper function around states

We need a function that takes a vector of states and outputs faceted plots.
This should generalize the Ohio example and guide us toward a single state to do the rest of the analysis with. It will, hopefully, match our intuition on what's common.

@seabbs -- have you already done this one?

Yes though not with a function - I was very lazy.


feat: gotouch should

Is your feature request related to a problem? Please describe.
When templating other Go template files (e.g. Helm templates) using the same delimiter can be annoying because you've to escape it every time like {{ "{{" }}.
Describe the solution you'd like
It would be great to be able to change the delimiter e.g. per template in the YAML config.

Great idea. @baez90. We will add delimeters to project structure and the value should be in , for example, << >> format, seperated by space. But this field will not be mandatory.

@baez90  Can you try if it works or not? I also wrote integration test to check it. Can you also inform us. Thanks for your contribution.


Add class to line ending modal

Add line-ending to class list of the line ending list view, to distinguish it from other modals for styling purposes.

:+1: Good idea.


No way to install OpenJFX (JavaFX for OpenJDK) and use it for development

Issue description
Subj. Currently there is no way to install it via nix-env (or I have not found it)
Steps to reproduce

nix-env -i openjdk idea-community
Run idea-community, open a JavaFX project
Set project SDK path to ~/.nix-profile/lib/openjdk
Observe that idea fails to build the project or highlight javafx code

Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)

16.03.973.8bf31d7 (Emu)

Nix version: (run nix-env --version)

nix-env (Nix) 1.11.2

Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)

nixos-16.03: 16.03.973.8bf31d7
nixos-unstable: 16.09pre85639.453086a

Is there any work in progress, any news, any update, any hope, any ideas on this issue?

At the first sight, it seems like someone has to read and follow these instructions: https://wiki.openjdk.java.net/display/OpenJFX/Building+OpenJFX and obtain an OpenJFX package parameterized with a specific JDK derivation.
Personally, I'm not going to have time for this during next 4 days. In addition, I may not be able to test it properly because I don't use OpenJFX in any of my current projects.

There are at least two obstacles here:

Building Gradle projects is not implemented yet: #17342
OpenJFX is installed by copying binaries to OpenJDK directory: in our case, it implies that one will have to merge OpenJFX and OpenJDK folders somehow (symlinks?).

I packaged OpenJFX privately one time because I wanted to try Cryptomator, which depends on it.
https://github.com/eqyiel/nixos-config/blob/7f061191e906102588ff4d537f0b33e0c30d45eb/pkgs/openjfx/default.nix
I gave up on Cryptomator so I don't have any motivation to work on this, but you would be welcome to use it as a starting point.
You can tell OpenJDK how to find it like this: https://github.com/eqyiel/nixos-config/blob/7f061191e906102588ff4d537f0b33e0c30d45eb/pkgs/cryptomator/default.nix#L26-L55

@eqyiel, I'm trying to use OpenJFX for Java development. When I run javac Main.java, OpenJFX is not found.
Do I understand correctly that the second snippet you linked to, can be used to make OpenJFX available for other Nix packages? But how do I tell OpenJDK (aka javac) in command-line where OpenJFX can be found?

@dimatomp said:

OpenJFX is installed by copying binaries to OpenJDK directory: in our case, it implies that one will have to merge OpenJFX and OpenJDK folders somehow (symlinks?).

Can't we solve this by copying the JAR files to $out/share/java as it is described in the nixpkgs manual?

@mekeor We can give it a try. I'm not sure if IntelliJ IDEA will recognize such an installation since IDEA does not currently require an OpenJFX dependency.

These are the JAR files after the build-phase:
./apps/samples/Ensemble8/lib/lucene-grouping-3.2.0.jar
./apps/samples/Ensemble8/lib/lucene-core-3.2.0.jar
./tests/system/build/libs/systemTests.jar
./modules/swing/build/libs/swing.jar
./modules/base/build/libs/base.jar
./modules/media/build/libs/media.jar
./modules/builders/build/libs/builders.jar
./modules/controls/build/libs/controls.jar
./modules/graphics/build/libs/graphics.jar
./modules/fxml/build/libs/fxml.jar
./modules/extensions/build/libs/extensions.jar
./modules/web/build/libs/web.jar
./modules/fxpackager/build/libs/ant-javafx.jar
./modules/fxpackager/build/libs/packager.jar
./modules/swt/build/libs/swt.jar
./modules/jmx/build/libs/jmx.jar
./build/sdk/lib/ant-javafx.jar
./build/sdk/lib/packager.jar
./build/sdk/lib/javafx-mx.jar
./build/sdk/rt/lib/jfxswt.jar
./build/sdk/rt/lib/ext/jfxrt.jar
./build/libs/stringtemplate-3.2.jar
./build/libs/swt-debug.jar
./build/libs/ant-1.8.2.jar
./build/libs/antlr-runtime-3.1.3.jar
./build/libs/antlr-3.1.3.jar
./buildSrc/build/libs/buildSrc.jar

That's why I was thinking about adding these lines to the installPhase like this:
    installPhase = ''
      mkdir -p $out/share/java/openjfx

      cp -v  build/sdk/lib/*.jar        $out/share/java
      cp -v  build/sdk/rt/lib/*.jar     $out/share/java
      cp -rv build/sdk/rt/lib/ext/*.jar $out/share/openjfx

      cp -rv build/sdk $out
    '';

but the javac and java commands still can't find OpenJFX after installing this package.

Do you have an idea what I'm doing wrong? You can check status of the WIP package here: https://github.com/mekeor/config/blob/master/nix/overlays/openjfx.nix

@mekeor yeah, it won't work with javac like this, the only reason that package I linked could find it is becuase of this patch: https://github.com/eqyiel/nixos-config/blob/7f061191e906102588ff4d537f0b33e0c30d45eb/pkgs/cryptomator/default.nix#L26-L55
AFAICT, the proper thing to do is overlay the result of building OpenJFX over an OpenJDK installation, which should be possible by overriding the OpenJDK derivation to copy the files into the right place: https://wiki.openjdk.java.net/display/OpenJFX/Building+OpenJFX#BuildingOpenJFX-Overlay-JDK8
I might take another look at this next weekend if no one has gotten around to it before then.

Should javafx work with oraclejdk? I have it installed with
  programs.java.enable = true;
  programs.java.package = pkgs.oraclejdk8;

I can see the javafx files in place but I'm still getting this error.
Caught: java.lang.NoClassDefFoundError: javafx/application/Application
java.lang.NoClassDefFoundError: javafx/application/Application

I installed idea-community which comes with its own JDK which includes a functioning JavaFX.

Yes I noticed that working but when I try to run my code using a groovy script (main use case) it doesn't work. Since the oracle jdk comes with javafx shouldn't that work? It seems like even if openjdk is overlayed with openfx it may not work.

After #63574 openjfx works for me, I think we can close this issue


[9.x] fix foreach loop in Str::is method

There is a typo in foreach loop for the pattern variable which modifies the pattern value probably unintentionally. It's a small fix for that.

If there is a bug send a failing test with your fix.


Add support for multi POTD

Occasionally, the POTD has multiple images, like on July 25 2022: https://en.wikipedia.org/wiki/Template:POTD/2022-07-25

Also today, which is an even wonkier case: https://en.wikipedia.org/wiki/Template:POTD/2022-08-28


[Go Live][21 Dec]

Try to enable zone pivots using YAML config file.

Docs Build status updates of commit 86e29d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

msteams-platform/zone-pivot-groups.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


placeholder not showing for Select element when using defaultValues

When using the useForm hook and set the "defaultValues" key the placeholder of the Select element does not show. When I remove the defaultValues key, it shows the placeholder.
Code:
const form = useForm<Partial<z.infer<typeof formDataSchema>>>({
        resolver: zodResolver(formDataSchema),
        defaultValues: {
            ...values
        },
    })

<FormField
                    control={form.control}
                    name="someField"
                    render={({ field }) => (
                        <FormItem>
                            <FormLabel>Some label</FormLabel>
                            <Select onValueChange={field.onChange} defaultValue={field.value}>
                                <FormControl>
                                    <SelectTrigger className="w-full">
                                        <SelectValue placeholder="Choose some option" />
                                    </SelectTrigger>
                                </FormControl>
                                <SelectContent>
                                    <SelectItem value="some value">some value</SelectItem>
                                </SelectContent>
                            </Select>
                            <FormMessage />
                        </FormItem>
                    )}
                />

What exactly is your field.value is it undefined or empty string ?

Add an s to the placeholder i.e placeholders="Enter your name"

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


[Tempest] Remove all images from logs

In addtion to deleting the files with img extension [0] from the logs folder, delete also qcow2 files.
[0] https://github.com/openstack-k8s-operators/tcib/pull/197

recheck
Looks like the ~/.ssh/id_cifmw key got moved to a different place?
 msg: The 'file' lookup had an issue accessing the file '~/.ssh/id_cifw'. file not found, use -vvvvv to see paths searched

recheck
The job was fixed with this PR -> https://github.com/openstack-k8s-operators/ci-framework/commit/7f04e0872db3cef2584f2c2789d89904c8fe205a

/cherry-pick 18.0-fr1


feat(comp:time-piper): code optimizing

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added/updated or not needed
[ ] Docs and demo have been added/updated or not needed

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Component style update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

What is the new behavior?
Other information

@jackson-yyy 有空 review 一下，或者跟我约个时间一起看。


AttributeError: 'NoneType' object has no attribute 'view'

Thanks for the great implementation.
I tried to run python omniglot_train.py --metric adacos, but the error occurred when validation started.
Traceback (most recent call last):
  File "mnist_train.py", line 228, in <module>
    main()
  File "mnist_train.py", line 204, in main
    val_log = validate(args, test_loader, model, metric_fc, criterion)
  File "mnist_train.py", line 98, in validate
    output = metric_fc(feature)
  File "/home/cvrg/.pyenv/versions/torch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/cvrg/Documents/pytorch-adacos/metrics.py", line 28, in forward
    one_hot.scatter_(1, label.view(-1, 1).long(), 1)
AttributeError: 'NoneType' object has no attribute 'view'

Could you tell me how to validate the model?

I have fixed.
Please pull and run again!

Hi, @4uiiurz1.
Thank you for fixing it.
I really appreciate your quick response.
Best Regard,


Expose max total num tokens from Runtime & Engine API

Motivation
Exposing max total num tokens, as requested in #1900.
Modifications
Usage: Accessed through get_max_total_num_tokens in Runtime or Engine API.
I'll add examples to Native APIs doc if this looks good.
Checklist

[x] Format your code according to the Contributor Guide.
[x] Add unit tests as outlined in the Contributor Guide.
[ ] Update documentation as needed, including docstrings or example tutorials.

Updated documentation in Native APIs.


Update about.md

That small mistake confused at first. I proposed the output is 14, not 7.

good catch!

Hehehe...

@hepranmarunduri Could you please check if this issue also needs to be fixed in the introduction.md of the concept and the introduction.md of the cars-assemble exercise?

I could.
I may be confused where to find those files, but I will find it and I will fix it.

@hepranmarunduri Could you please include the other two fixes here in this PR so they will be merged together? (You can do that by editing the files in the patch-1 branch.)


Remove dangling rpath and add /opt/vespa/jdk symlink

FYI @aressem

The travis build is green - not sure why the color isn't able to update.


rover lp destroy command line not parsing -c correctly

When I tried this command with an incorrect -c parameter, the command tried to run anyway, gave incorrect results.
rover lp destroy -c $HOME/caf_config -s $HOME/git/caf-terraform-landingzones -l level0
I left off /demo/level0 from the -c parameter.

The implementation is naïve and doesn't check for the existence of the folder PLUS doesn't check if the folder contains any .tfvar files, those check would probably fix this.
Of course there's no way to stop a user pointing it a folder filled with totally incorrect .tfvars


Kindred skin bug

Kindred skin is not change

The wolf has to update. Use w and it will most likely fix itself

I will look into this issue. #149


Update to new graph api, #679

Update to new graph api
Summary
Update to the new graph api.  Carry api through security principal (both in oidcUserService flow and AADAuthenticationFilter flow).
Issue Type

Update Feature (old graph to new graph)

Starter Names

active directory spring boot starter

Additional Information

Ref this PR to #679

@wandrille-hubert Thanks for the contribution, some test case failed, could you check?

@sophiaso fixed those failing tests.  by the way did you want me to update the endpoints.cn.aadMembershipRestUri and cn.aadGraphApiUri to new graph api (microsoftgraph)?

@wandrille-hubert
Thanks a lot for your contribution. It looks your PR is kind of Breaking Change, right ? I mean if I would like to leverage old graph API, it will works well. Somehow one day the old graph will be expired, but it is better to keep both works well, as well as some EOL for old one.
How do u think of it ?

@Incarnation-p-lee yes I agree that this is a breaking change as it is swapping the old api for the new.  Would we want to set a flag in the settings to let users choose which api to go with/or would we want to grab both api tokens?  seems like only getting one api token might be more favorable.

@wandrille-hubert
I may prefer somewhat like flag to reach this, we can make it to EOL once old graph expires.
@sophiaso How do u think of it ?

@wandrille-hubert Over the two options, prefer to  set a flag in the settings to let users choose which api to go with, thanks.

@wandrille-hubert It seems like the Codacy quality review failed. Since new graph API and new config introduced, we'd better update the document and sample projects too.

@saragluna for the sample projects, did we want to update the current ones to use the new graph api, or create new samples that utilize the graph api and keep the current ones as is?

@wandrille-hubert  Creating new samples that utilize the graph APIs and keeping the current ones are a better choice. And you also need to fix the Codacy Quality Review issues related to your PR.

@neuqlz ok will get some new samples created.
also the codacy quality review, will take care of that once we resolve the converstaion on .../src/main/java/com/microsoft/azure/spring/autoconfigure/aad/AADAuthenticationProperties.java
as that is the reason for that quality to fail.

@wandrille-hubert And I also think there need some guides which teach users how to use the version2 graphAPIs. Could you add a section which guides users how to use the version2 graphAPIs into the doc(azure-spring-boot-starters/azure-active-directory-spring-boot-starter/README.md) ? Thanks a lot for your contribution.


how to perform secondary anno based on pre-annotation by "Label Tasks As Displayed"?

can anyone help me with image object detction bbox annotation issue? dont know how to perform secondary anno based on pre-annotations.
i specific my pre-annotations and import data through json format as official doc says:

but when i start labeling with "Label Tasks As Displayed", it syas "No More Tasks Left in Queue":

i mean actually i already have partial annotations, and now i want to adjust some bboxes, just correct them
is there any option to set the pre-annos to be incomplete?

Can you share your labeling configuration and example files, including the pre-annotations, to help us reproduce your issue?

Hey @dao027 — good question! I'm happy to help out and lend a hand here. :)
Looking at your screenshot it looks as if they are showing that it does appear they are labeled and at this moment there is no way that I know of to set the preannotations as incomplete.
Further context would be great to think about a possible solution

just great
the issue board is full of people trying to improve upon existing annotations and it isn't a function anyone thought of
just wtf


chore: toolkit repo migration

This PR is a successful merge of the Toolkit repo into the SDK repo which was performed by following these steps: https://contentful.atlassian.net/wiki/spaces/PROD/pages/4413883469/2023-12-11+Migration+to+Single+Repo

A question on CI providers and one minor change

Seems like our PRs will fail due to a requirement of having Circle. I've instead modified the job so that it doesn't really do anything and all real actions are handled by GHA.

We can revisit removing or modifying Circle once the changes hit main


Add code that removes duplicates when parsing groups.plist

(You might want to make public.kern2.B be ["B", "A", "B", "A", "C"] or so to ensure that order is kept, otherwise just making a set would pass as well)

Looks good. I think you can send this upstream now.


Publish as a .NET tool?

I would like to manage the component-detection configuration for my repository from my local tools configuration file.
AB#2123785

This makes sense. Not sure when it'll get prioritized from our end, but will update here if the work is taken.


Formally required props on ManagedRefreshContextModel are not strictl…

…y required.
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[X] Caught up with develop branch as of last change.
[ ] Added CHANGELOG entry, or determined not required.
[X] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[X] Updated doc comments / prop-types, or determined not required.
[X] Reviewed and tested on Mobile, or determined not required.
[X] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Looked at with Lee. We're not 100% about this. Need to look closer at the app code that would require this change.


Add ExaGrid key, CVE-2016-1561

Looks good to me, go ahead and land it.

Just got Metasploit contributors access to this repo, landing.


a bug in _remove_group

Sorry, the last code that I pulled has error in line 157.
0-1 0
-1 1-1 #0 empty 1 black -1 white
-1-1 0
in this case, 1 will be removed,then replaced by 0. if self.liberty_counts[gx][gy] += 1 is used, then the white group in the lower left will count the liberty twice, because two stones in that group is adjacent to the stone 1, and I add the liberty twice.

Good catch, thanks! There is a helper function get_groups_around that may have been useful to avoid this, but it's fine as-is


POE overlay Not Work

Describe the bug
The overlay poe doesn't work with the game,
when I open the game the overlay closes "/
I went back to version 0.6.29
which is working normally for me

Unfortunately the author took this great addon and pretty much ruined it with Overwolf.  Fortunately there is an overwolf free fork available. Just search reddit.
Overwolf leeches user data and has an overall history of scummy practices.

I agree with you friend, it works much better before
I'll look for the fork link, thank you very much!

You're using Vulkan? It's not fully supported yet but will be coming soon(hopefully this week)
Try changing to dx11 and see if that works


Shader proc_macro fails to compile if vulkano is nested

So  the shader() proc_macro works fine if you use vulkano directly but if you use a crate that has vulkano as a dependency it will not compile if this macro is used.
This is due to the macro creating a mod like this:
mod vs {
  use vulkano
  //etc
}

This use vulkano points to vulkano as an external crate. So it has to be listed directly in your Cargo.toml.
We can fix this be changing it to:
mod vs {
  use crate::vulkano
  //etc
}

The putting use vulkano or use my_crate::vulkano in the root of the crate calling this proc_macro (ie. lib.rs).
However this will be a breaking change for any projects that are currently using vulkano as a direct dependency and do not have this use in there root.
What are peoples the thoughts on this problem? It would be nice to be able to use this macro without needing to have vulkano as a direct dependency.

I dont understand?
https://github.com/rukai/vulkano-text has vulkano as a dependency and it works fine?

This only affects a project that uses the proc_macro directly but doesn't depend on vulkano directly.
So if you used vulkano_text as a dependancy but also tried to use the shader macro in your code it wouldn't compile unless you also have vulkano as a dependancy

Ok, I think I see the issue.
You want to reexport the vulkano shader macro from nannou?
I don't see how your suggestion allows users to call the macro without including vulkano as a dependency.

In that case we'd do use nannou::vulkano.
I'm not sure if this is a great solution though. I think the real issue is not having the equivalent of $crate for proc_macro like there is for regular macros.

I think this is the same issue that https://github.com/Lokathor/bytemuck/issues/93 has. So we probably can't fix it, until Rust itself comes with a feature for it.


Fix: File comments don't have line number information

Addresses #263. This is a simple fix, not sure whether the missing line number information for "file" comments should be handled in another place.

Looks good, will merge this and ship it with the next release. Who knows why Gitlab has so many different types of comments 🙄


After I went online and packaged and uploaded the package, I was prompted with an error. The log indicated that the port was occupied, and I changed a lot of free ports. This error is still displayed. Why?

Below is a list of service status and feedback information.

E：It is very likely that your application will try to listen to this port twice before and after. For example, tarsphp will listen to 10002 when it is started, and it will also try to listen to this port when hyperf is initialized.
C：很有可能你的应用前后两次尝试监听这个端口，比如tarsphp启动的时候监听了10002，hyperf初始化的时候也在尝试监听这个端口。

How can I check which ports are monitored by tarsphp?

如何查看tarsphp监听了哪些端口？

It is very likely that your application will try to listen to this port twice before and after. For example, tarsphp will listen to 10002 when it is started, and it will also try to listen to this port when hyperf is initialized.

tarsphp只监听你配置的应用端口。

After replacing the port, it is found that the current status is off when it is just started. After two minutes, I refresh the page and the service status becomes active. Does this phenomenon indicate that the service is normal? When you click Restart or Stop, the error is reported again, and it still prompts that the port is occupied. why?

更换了端口后，发现刚刚启动的时候，当前状态显示off。两分钟后，我刷新页面，服务状态变成active，这个现象说明服务正常了吗？当点击重启或者停止，再次报错，日志中依然提示端口被占用。这是什么原因？

Observe how long it took to start the service and fail to start it. It may have been started for a long time, and the heartbeat report was reported normally. For example, tarsphp starts the relevant registration first, and then starts the first detection and reporting of the heartbeat ten seconds later. In these 10 seconds, the hyperf is still initializing, and the hyperf hangs after the heartbeat is triggered. It is recommended to track the startup process code.

看刚启动服务到启动失败全程经历了多久，有可能中间启动了很久，心跳上报正常上报了。比如tarsphp先启动执行相关注册，然后十秒以后开始第一次检测和上报心跳，在这10秒里hyperf还在初始化中，心跳触发之后hyperf才挂掉，建议跟踪一下启动过程的代码。


51 legend color charts

Added siteProps to Layout.tsx which is a map pushed down to charts and legends. These are used to link colors legends and charts.

Front-end tests fail ... ?


[updater][v2] Updater causes critical "app may be damaged" state on macOS

Hello! Our users are experiencing a pretty critical error when their app auto-updates, it gets into this critical issue where the app cannot be opened:

Here are the logs:
[2024-03-29][18:28:02][app_lib::updater][INFO] Update available: 0.2312.1
[2024-03-29][18:28:02][reqwest::connect][DEBUG] starting new connection: https://storage.googleapis.com/
[2024-03-29][18:28:02][reqwest::connect][DEBUG] proxy(http://<IP_ADDRESS>:9091) intercepts 'https://storage.googleapis.com/'
[2024-03-29][18:28:04][app_lib::updater][ERROR] Error downloading and installing update: invalid gzip header

(our updates are stored on Google Cloud)
Re-installing the app from the DMG fixes the issue, but this is pretty problematic for the user. What's also odd is that the hosted update tar file unzips perfectly fine when done manually and I can run the app in that zip as well. So I think it is something with the updater logic
The updater should probably decide not to install the update if it's invalid or has some issue downloading. Let me know if I can provide any more logs to help!

After some investigation, this looks to be due to a malformed manifest file on our end. Closing for now, I'll re-open if this crops up again!


BUG: Imorting ztf_functions.py fails when there no access permission to the s3 bucket

ztf_functions.py accesses a remote bucket during import. This is probably not a good idea as it prone to failure when the permission to the bucket is not setup correctly.

This problem has already been captured in https://github.com/nasa-fornax/fornax-demo-notebooks/issues/311. Note, the ZTF access issue cannot be remedied yet as it is not yet have an ODR place.
I would go ahead and close this as a duplicate, but if you have an idea for a suitable workaround, feel free to reopen (I don't think it's any particularly useful to skip ZTF and yet create the light curves when there is no access to the bucket as in practice that dataset is heavily used in all the follow-up notebooks).

OK, that makes a lot of sense, reopening it for tracking the python script importability rather than the bucket issue.

@bsipocz is it ok if I fix this? This module uses multiprocessing internally which complicates this a bit, but I think I see how to lazy load that variable in an efficient way.

Sure! I was only planning to move the quoted line into a function body, so it's not touched at import time, but anything would work really.


Paginate links in IE

Paginate links doesn't work in IE.

"Object doesn't support property or method 'find'"

+1

Since no one answered this before, this component uses the find() method on the Array object, which is not supported by IE natively. There is a polyfill available you can pull in yourself or pull in some exisiting polyfills that can help cover more cases.

#78 Can fix the problem


On-device model generation

@jaustin asked me to make this PR to show you the changes we made to ml-machine for using it with on-device models! This isn't intended to be merged, and it would be quite difficult to do so because we based our work on a commit before the removal of ml5.
(This pr should be based from a branch from commit 3284df8, as far as I can tell -- @jaustin mentioned he would create a branch for me to use on that commit, and I'll edit this PR to point to that branch once it's done -- until then it's probably best to look at the individual commits to see our changes.)
This project uses ml4f to compile the trained model to ARM bytecode, then writes the generated binary to a specific memory address in a precompiled firmware file (the source of which can be found at JuliaScythe/microbit-gesture-ml), which passes the same data the frontend would normally receive to the model, and displays the output on the microbit display.

@JuliaScythe thanks so much for making this PR (and the work you and your team did that his demonstrates!)
For the clarity of those that aren't familiar with the work you did, is it worth changing the PR title to something like "generate a model to run on the device" (as I parsed this as suggesting the model was actually generated on the device, at first)
For making a cleaner PR, does oxford-project-base look like a branch at the commit you need?

I have just tested it. This is amazing! Thanks @JuliaScythe!
We will take a look at the code and see if we can make a version with the new ML-building implementation.
@JuliaScythe do you know of any issues with using a newer version of tensorflow? (ml5 is basically just a wrapper of an old tensorflow version)

This is amazing work, thank you!
I am amazed at how quickly this demo was realized and I'm curious as to it's potential. This is definitely something we will be using as a reference in the future.
The rest of the comment will be notes on the implications this change may bring, feel free to skip

I have just tested it. This is amazing! Thanks @JuliaScythe!
We will take a look at the code and see if we can make a version with the new ML-building implementation.
@JuliaScythe do you know of any issues with using a newer version of tensorflow? (ml5 is basically just a wrapper of an old tensorflow version)

I think ml4f uses the TF.js model format. I do not believe the switch to Tensorflow from ml5 to be an issue we will have to revert or deal with, although my knowledge is limited. I am investigating, will update

@r59q
To begin with, I think we should see this as an exploratory process. So not worry too much about how this feature will blend with the rest of the tool. Step one should be to test if we can make a version that is good and stable enough for a classroom activity. Step two would then be to test this version of a tool with teachers and students. So in the first phase, we should have this feature on its own branch and make a version that is dedicated to run the model on the micro:bit.
I agree, that It will become messy with all the different options for running and creating logic around the model. So we may consider to see this repo more as toolkit with which you can configure and build different versions of the tool. But that is a discussion for later.
Right now,  I think the focus should be to make a version that we can test in a classrooms to get some feedback and experiences with this feature.

I bumped tfjs in ml4f to v4.10 - it didn't require any changes and still seems to pass all the tests. Great work!

Im closing this, we will note it as a reference :)


[Bug] Install failed

Run the command:
npm install --global starring
Result:
 ENOENT: no such file or directory, chmod '/Users/Users/.nvm/versions/node/v6.3.1/lib/node_modules/starring/dist/cli.js'

@eeeeeLin Let me know if it works now.

@ritz078 Sorry for the late response 😜
It works correctly right now  👍


Roi management

This branch will be able to load the regions of interest extracted by clinicadl extract.
Must be merged after #119

Hello @14thibea! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file clinicadl/clinicadl/tools/deep_learning/data.py:

Line 442:76: E241 multiple spaces after ','

Hello @14thibea! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file clinicadl/clinicadl/tools/deep_learning/data.py:

Line 442:76: E241 multiple spaces after ','


[Meta] Create and standardize pages on "Context & Propagation" across languages

As of today only JavaScript, Erlang & PHP have pages on propagation:

PHP
JavaScript
Erlang

Javascript additionally has a page on Context:

JavaScript

The following languages have a section on Context and/or Propagation in their manual instrumentation docs:

Go
C++
Java
Ruby

I would like to create consistency for this across languages:

Context Propagation is an important enough topic to have a dedicated page below each language (like PHP, JS, Erlang today)
In the manual instrumentation doc (and others if applicable) the subsection should still exist but link out to that dedicated page
(Open Question): Do we want to merge the JS pages on Context/Propagation or do we want to have 2 dedicated pages? Or do we want to have a parent page?

As a counter-suggestion: We usually have quite a bit of pages under "Automatic": Why not doing the same for "Manual" and have the Propagation doc there? Manual instrumentation docs for languages like Java have grown to a huge size and are a bit unwieldy.

As long as we say "Manual" and "Automatic" instrumentation this would give people the feeling that this is only relevant for their manual instrumentation (see https://github.com/open-telemetry/opentelemetry.io/issues/3228 for this discussion as well). But I see your point, our menu is growing quickly and might require some care & love in 2024 as well

FWIW I'm fine with splaying things out right now and condensing later. I think we're just going to face the ugly reality that it's complex and there'd gonna be a lot of topics to lay out.


update to nppplugintemplate version 4.2 for N++ 8.3

(https://github.com/npp-plugins/plugintemplate/commit/e699e2dd94866f72eab05d6ba9f84b79538014f9) for N++8.3

appveyor.yml for VS2022
version 2.3

Thanks @chcg 👌


Edit

👷 Deploy Preview for adoring-leavitt-e31a11 processing.

Name
Link

🔨 Latest commit
8d7525a7f52a59afc1c52683c72024392d41ad44

🔍 Latest deploy log
https://app.netlify.com/sites/adoring-leavitt-e31a11/deploys/62ab1a6a78d229000905b786


Bump default dbcache

Cherry picks of https://github.com/bitcoin/bitcoin/pull/8273 amd https://github.com/bitcoin/bitcoin/pull/8407

@dagurval Were you going to remove the second commit?

Done

utACK


No image selected

When there is no image selected for a device model, an URL is display in the device model detail

and nothing is displayed in the device list for those devices

Maybe it should display somethin else in those two locations.

Update
The problem is still here with the new UI :


Bug: Player crawl when spamming jump and crouch

Hello, I've found that when a player is in an area where they are surrounded by blocks and they spam the jump key and the crouch key, the game makes them crawl on top of the block they are facing.
https://github.com/user-attachments/assets/6fda9d35-7524-4ba7-bfb1-de6d952f4a78

I forgot to update Minestom


McpeFun.com no longer available.

The domain McpeFun.com is no longer available. I will not be hosting it in that domain anymore, please use puremc.pw instead.

0k/

It's already working.

Changed.


Add next direction in iterator

Normally next direction is forward, but if current Node is final but also has children, the next direction. This may not be possible, since if new node is added, the direction can change.

Not possible


Enhance the network logging

Moved from:

https://github.com/vegaprotocol/devops-infra/issues/1433

Network logs: We are losing important logs when network topology changes.
The three cases we’ve seen with system tests are:
Restarting from a checkpoint: we only ever get the logs for the new network and the logs from before the restart are lost.
When a node is added and removed: the logs for that node never appear in the network logs at the end of the test, so we lost that information
When a node dies mid run and vegacapsule automatically starts it again (up for debate whether that is the ideal): we sometimes get the logs from the original task, and sometimes don’t
See this slack thread for more context:
https://vegaprotocol.slack.com/archives/C03GXJT3JLT/p1656584457886179
Comment in transferred issue:
@jgsbennett

https://jenkins.ops.vega.xyz/blue/rest/organizations/jenkins/pipelines/common/pipelines/system-tests/runs/4722/nodes/277/steps/286/log/?start=0
This Jenkins job is a lovely example of a case where we have no data node logs, and therefore poor @ettec is unable to work out what happened without attempting to reproduce locally.

There is linked bug: https://github.com/vegaprotocol/vegacapsule/issues/186

For @peterbarrow 's network, i hacked this by printing logs with the fluent bit. Here is a configuration:
[INPUT]
    Tag              cli
    Name             tail
    Path             /local/vega/.wallet/state/wallet-cli/logs/*.log
    Read_from_Head   true

[INPUT]
    Tag              service
    Name             tail
    Path             /local/vega/.wallet/state/wallet-cli/logs/*.log
    Read_from_Head   true

[OUTPUT]
    Name             stdout
    Match            *

    task "wallet-logs" {
      driver = "docker"
      user = "root"

      template {
        data = local.fluentbit_config

        destination = "fluentbit.conf"
        perms = "755"
      }

      config {
        image = "cr.fluentbit.io/fluent/fluent-bit:1.9"
        // command = "ganache-cli"
        args = [
          "--config", "/fluentbit.conf"
        ]
        volumes = [
          "/home/vega/.wallet:/local/vega/.wallet:ro",
          "fluentbit.conf:/fluentbit.conf"
        ]
        auth_soft_fail = true
      }

      // resources {
      //   cpu    = 1000
      //   memory = 100
      //   memory_max = 12288
      // }
    }

Latest issue with logs -- take this example on last nights overnight tests:
https://jenkins.ops.vega.xyz/blue/organizations/jenkins/common%2Fsystem-tests-wrapper/detail/system-tests-wrapper/1895/pipeline
In the output from the system-tests themselves we can see that the network reached epoch 209:
MainProcess utils.py:pause_until_new_epoch:99 ----- Reached target epoch 206

but in the logs collected for validator 0 they just randomly stop mid block, and exist only up to ~ epoch 90:
https://jenkins.ops.vega.xyz/job/common/job/system-tests-wrapper/1895/artifact/testnet/logs/testnet-nodeset-validators-0-validator/vega-validator-0.stdout.0.log
This does not seem to be a one off thing and all logs on all runs I've checks just stop at block-number ~1600. Its also not related to node coming up or down, or because of checkpoints because if you spin up a network locally and just leave it to do nothing for 15mins, the logger side-car thing just stops picking up new lines.

I believe all the issue above should be already solved.

I can confirm that the way the logs are now capture fix all the issues here, so I'm happy for this to be closed.


is this project dead ?

There's one active maintainer with a recent upgrade.
https://github.com/ui-router/ng2/graphs/contributors
Feel free to find issues and create pull requests if you need updates. Otherwise you can be patient and wait.

This project is still active, and I'm committed to maintaining it.  However I have limited resources at the moment, and I am also responsible for maintaining numerous other projects.
I'd like to add new maintainers.  Let me know if anyone is interested.

@christopherthielen I interested. Currently I'm working in some issues.

@gcca  excellent, thank you.  I'm in the middle of moving out of state, which is why my time is currently so limited.  I'd like to release a new ui-router-core (5.0.0) ASAP once the remaining issues in the milestone are fixed.  Then we can release ui-router-ng2 beta.5 compatible with ng2, and then beta.6 compatible with ng 4+.
I can invite you to the slack we used for ui-router dev communication

Great! That will help me to collaborate.

@gcca =If you send me your email address, I'll send an invite
email me github at sandgnat dot com

@christopherthielen I would like to help with the project if it's possible.

Please send me your email addresses @gcca @wawyed and I'll invite you to the slack.

I'm @christhielen on twitter

Cool, email sent.


Required asterisk not showing in reactive forms

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Form fields show required asterisk when validator is set through reactive form
What is the current behavior?
form fields do not show the required asterisk when set through a reactive form model
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
https://stackblitz.com/edit/angular-p3sswf?file=src%2Fapp%2Fapp.component.ts
What is the use-case or motivation for changing an existing behavior?
I would like to be able to have full control over when the field should be required or not (during creation of the fields), without having to add helper functions for the required parameter in the template.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 6.0.7, material 6.3.2
Is there anything else we should know?
If I add a required attribute to the field itself it works.
The actual validation on required does work, focusing and defocusing shows the red error outline. I also noticed from inspecting the elements the classes actually get added (ng-star-inserted), it just doesn't show up:
Not working:

Working (with required attribute in template):

@Tommatheussen, please check #2574

@revov yep, same issue, seems I didn't search thorough enough, closing this one


feat: allow to define a typed body

Description
To better use the request body in Serinus, the framework should allow to define a typed body.
Requirements

[ ] Define the interface for a typed body.
[ ] Update Tests.
[ ] Update Documentation.

The proposal is the following:

Add an option to the command generate for the models
Remove the config.yaml file and use the pubspec.yaml to manage all the serinus things
Add a ModelsProvider to ensure the models are provided correctly
Integrate json_serializable, dart_mappable, freezed and allow to define an own serialization strategy.


Finances tab in iOS to show all defi related tokens #2029

Closes #2029

Other than the EIP55 comment, all looks good. Can you help to squash the commit I pushed into yours if there's no problem with it?

Done


Fix links that assumed branch "dev" exists (which doesn't)

I noticed that a lot of links in the readme actually lead to 404 pages because branch dev is missing (d742bea01e2d3179e142d708dc5be18ed2784432).
This just changes "dev" to "master", not sure if that's correct or desirable so feel free to close this again if not correct or necessary.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: MSeifert04 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@SteveSandersonMS - could you also take a look and then merge?

Thank you for the fast review 👍


goffice: 0.10.46 -> 0.10.47

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/goffice/versions.
meta.description for goffice is: "A Glib/GTK set of document centric objects and utilities"
meta.homepage for goffice is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.10.47 with grep in /nix/store/gp7p8swckpdfpiir7aw8b2h3hgig7655-goffice-0.10.47

directory tree listing: https://gist.github.com/cd488ec9670ffaeaf1a7e1b414d81253

du listing: https://gist.github.com/cc243f746dd3df7de6f93953a2eb7b69

Rebuild report (if merged into master) (click to expand)

15 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
abiword
gnumeric
goffice
nip2

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gp7p8swckpdfpiir7aw8b2h3hgig7655-goffice-0.10.47 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A goffice https://github.com/r-ryantm/nixpkgs/archive/65bd987b129b33d5a05f6f9a07cba8ee7b82868b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gp7p8swckpdfpiir7aw8b2h3hgig7655-goffice-0.10.47
ls -la /nix/store/gp7p8swckpdfpiir7aw8b2h3hgig7655-goffice-0.10.47/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

4 packages built:
- abiword
- gnumeric
- goffice
- nip2

Maintainer pings

Changelog at https://gitlab.gnome.org/GNOME/goffice/-/blob/3c5c021179debed46d05665f447ee1330ea655f4/NEWS looks good and the bot has tested all dependees.


Discussion on using multiple JVMs for series 0.8

This thread is a discussion thread for anyone to give input on the design.  The topic being how to support multiple JVMs either serially (start/shutdown, repeat) or in parallel.  This is topic has previously been requested in #438
Introduction
With the release of the 0.7 series,  I am pressing forward to start work on JPype 0.8 series.   Just like the development of 0.7, the work starts with the designing core to remove limitations that prevented JPype to be a fully integrated bridge solution.   The single largest limitation JPype has currently is lack of support for multiple JVMs.  This limitation comes in two ways.  The inner core module and the python model both contain a huge number of global variables.  These globals are all designed to be use once with no thought to ever cleaning up after the party.  Second, the API itself is oriented towards a single JVM being used.   To remove this limitation means changing both of those situations.
The inner core restrictions are currently being removed currently.  Prior to the rewrite for 0.7, changing out the global situation would have been a near impossibility.  But now that the host resources are untangled sufficiently there is no reason not to try and resolve the global situation.  The procedure is a three prong attack: move as much of the logic from the C++ layer to Java,  move all memory management of the C++ layer to Java, and move everything that remains into an object under Python control. This is good sound design regardless of whether we ever support multiple JVMs as we now are cleaning up properly rather than waiting for the house to burn down.
The second half would be how to make the API support a second JVM.  There are two separate ways this can be implemented with two different sets of issues.
Series of JVMs
This one seems like the simplest.  We have a start and a shutdown command, so it seems like nothing in the API is affected.  Not so fast, all of the resources global resources that we create are ties to the first JVM started.  And these resources are registered globally within the Python module system.
To successfully shutdown and restarted we would need to place all of the Python bridge classes in a neutral state such that they can be stood up again when the next JVM comes along.  This would require two things to happen.

As part of shutdown, every Java class wrapper from that JVM would need to be visited and the __javaname__ would need to be populated.  Further the __javaclass__ and the mro table would need to be cleared such that class tree can be garbage collected.  Thus after this stage the only wrapper classes left are the ones that the user actually held on to.
Once the new JVM is started we need to reactivate the prior resources, either by locating them aggressively or waiting for them to be used in a lazy fashion, Then we monkey patching them back to an working state.

Clearly this is a heavy lift.  The act of visiting, patching and copying would all increase the overhead of the JPype for this one feature.  Lazy would be less cost, but much harder to implement.
Multiple JVMs
Initially this one seems much harder.  Our whole API is focused on providing easy access to one JVM.  Thus providing for two or more seems like it would require devastating changes to the API.  Fortunately, this one is not as bad as it appears.  Only a few calls are required to be tied to the JVM instance.  Those are the JVM controls and the Factory methods.  All of the primitives are JVM neutral and thus they can live outside of any JVM.
To make this option possible we would need to introduce the concept of the JVM itself.  Internally I refer to these as the context.  This context would then need to passed to each of the JPype method that requires a context.
  jvm = jpype.createJVMContext()
  jpype.startJVM( jvm=jvm)
  MyClass = jpype.JClass('org.place.MyClass', jvm=jvm)
  myObj = MyClass()
  myObj.doSomething()
  jpype.shutdownJVM(jvm=jvm)

We can simplify this a bit by shadowing a copy of our API inside the context.  This would just be a cheap forwarding to our regular API.
  jvm = jpype.createJVMContext()
  jvm.startJVM( )
  MyClass = jvm.JClass('org.place.MyClass')
  myObj = MyClass()
  myObj.doSomething()
  jvm.shutdownJVM()

Now for the issues:
The first is purely technical, the JVMs would need to share namespace because we would have two or more classes with the same name in the Python class system.  We may be able to hack them apart such that they have different names, but I am not sure how to accomplish this.
The second is the limitations of mixing the API.  The type conversion system would not recognize items created in one JVM as being Java resources to another.  Thus the only resources that can be shared directly are primitives.  We could expand this slightly by creating a bypass for strings.  Then with additional work we can make Serializable objects pull themselves across the JVM boundaries.
The good news is as this doesn't involve dealing with monkey patching of existing classes, it is a much lighter lift.
Discussion
I am willing to a shot at either of these assuming that I don't have to break the API to do it.  User feed back on which of these options are useful and some use case examples of how you would actually make use of such an API would be great so that I can vector my approach towards something someone wants to test and use would be great.

I worked through the issues one the API, but unfortunately I could not find any current JVM implementations that actually allow starting, stopping, or running a sub JVM using the JNI interface.  To make this work one would need to use IPC to communicate with another process which would proxy to the normal JNI interface.
The overall lift on this would not be too extraordinary because most of it is just adding an IPC layer which does the push.  The sticking points are registerNative, arrays, and threads.   Last we would need to figure out how to build and install one additional shared library (the fake JNI front end) and a new executable which is outside the scope of my current efforts.
I am going to drop this work for now in favor of other objectives for 0.8.


Update to commons-beanutils:1.9.4 without disabling the protection

This pull request attempts to update to commons-beanutils:1.9.4 without disabling the additional protection it offers around the class attribute.
It seems in my (fairly limited so far) testing that the problem is st:include's class attribute. So this pull request renames the attribute to clazz (and keeps the existing setter for compatibility).
This PR additionally proposes a workaround that applies when Jelly files are being parsed and the resulting tags are created: For the st:include tag, when the class attribute would be set, instead set clazz. This attempts to take care of all existing code that sets the class attribute for this tag.
I am unsure how best to go about ensuring this doesn't cause a lot of problems. While this fix looks fairly safe, it may also be incomplete. In the short term, #209 seems like the safer approach.
CC @basil

Who's able to merge this PR @daniel-beck, @oleg-nenashev? It has enough approvals for a few days already. The permission updater repo doesn't mention any maintainer: https://github.com/jenkins-infra/repository-permissions-updater/blob/master/permissions/component-stapler.yml

The permission updater repo doesn't mention any maintainer

That's explained in the linked file in a comment.
@jglick Please hold off releasing for another week until we've merged in changes from the upcoming security update.

Why can't we just release and hold-off updating this into Jenkins Core?
We do not understand the rationale here. Thanks!

Because there's a privately staged new release of Stapler. That release you're asking for would never be in a core update anyway (and if it were it'd be a pretty big regression over next week's core release for absolutely no reason).


Skip appropriate AuTests if Netcat is not present

Some AuTests require Netcat as a client, but do not get skipped if it is missing. This updates the tests so that they get skipped if Netcat is not installed.

We may not want to merge this because it would cause CI to pass without indicating an issue if the nc command were missing, and we need those tests to run.

The AuTest ja3_fingerprint failed.

[approve ci autest]

I've repurposed the PR to have the CMake build and autest.sh check for netcat and fail if it is missing, rather than skipping tests based on its presence.


Fix minor typos

Overview
Fix minor typos in docs
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Summary by CodeRabbit

Documentation

Corrected typographical errors in various documentation files.
Clarified descriptions in the DAClient implementation process.
Improved explanations of the GetAppHash method and State Fraud Proofs.
Fixed grammatical issues in the documentation to enhance readability.

Bug Fixes

Addressed a spelling mistake within the BlockExecutor module documentation.

External PRs should focus on improving documentation or have code changes from open issues.


Potentially confusing colData augmentation

Just noticed that the estimateDiversity and other related functions are directly adding new columns to colData, even if the colData already has elements with the identical names.
Not knowing this, it took a while to realize that one error message that I got was related to this. Potentially confusing and error prone.
We could either overwrite the past values that had the same name, or provide warning or error if new elements with identical names are being added to colData.
Any comments / recommendations, how does this compare to other standard practices with SE objects?
library(mia)
data(esophagus)
tse <- esophagus

esophagus <- estimateDiversity(esophagus)
cd <- colData(esophagus)
print(colnames(cd))

print("---------")

esophagus <- estimateDiversity(esophagus)
cd <- colData(esophagus)
print(colnames(cd))

I don't see the problem. The user can always choose a different name for the values to be added to and if he/she/they uses the default values and in the special users context they don't apply, it is not our fault.
For example: If the user save values to the colData named shannon and runs estimateDiversity and expect the original value to be present, he ran estimateDiversity without reading the man page. One could argue a warning might be in order, but its still a layer 8 problem.
runMDS and all other functions I know of, like addPerCellQC, behave the same way.

Not knowing this, it took a while to realize that one error message that I got was related to this. Potentially confusing and error > prone.

I would like to see the context of this error message.

The error message I mentioned came when I tried to pick a field by its name, and got unexpected values. Then, investigating this I noticed that the colData had several columns with the same name and picking a field by its name returned one that had the same name but different contents.
If the general idea is to help users by implementing the functionality so that unexpected errors do not happen easily by mistake, it would make sense to generate a warning at least.

Then it would be good to add this check into .add_values_to_colData. That you way you catch it for all functions

Yes, it came up while digging that one. I can add this in that PR.

I am now adding a warning in the other PR. However, continuing discussion in this issue because I would like to ask if it is completely out of question to add an argument that would allow the user to override the previous values..
Like: tse <- estimateDiversity(tse, replace = TRUE)

The values need to be overwritten always. The user has the option to use another name.
However, if the name already exits a warning is issued. Thats it.

Just checking - if the name already exists, and a warning is issued + new field with the identical name added, then the values will not become overwritten. What does it then mean in this context that the values need to be overwritten always?

No. the values gets replaced. I mean you can spell the problem out in the warning, if you want to but it comes down to read the manual

What does it then mean in this context that the values need to be overwritten always?

That the current behaviour is correct. The value will be replace. The only addition, is that the user gets warned about it.

The previous values are not replaced. A new field is added instead.
With "value is replaced" I get that the values in the previously created field would get overwritten. This does not happen in the present implementation. Do you mean that it does happen, or that we should change the implementation so that it would happen this way?
Do we understand the terms in a different way, or do we have a different take on of how the function behaves currently?

I messed up. It should be replaced ... arg... Thats was on piece of paper ToDo List.. I will have a look at the PR and think about it.

Ok great! :-) I was confused for a reason.


Analyze brands

Data-plane change for new classifier.
Note the openapiportal tool would not validate ("Sorry, an error occured or you've encountered an unsupported feature").  I ran autorest --model-validator and --azure-validator from the cli instead.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

Is this good to go?


nativemem - Native memory profiler

Native memory allocations profiling and memory leak reporting, via GOT patching.
There is a new argument in the profiler nativemem, and jfrconv gains native memory tracking and memory leak reporting.
Usage
New argument --nativemem [N]/nativemem[=N] is used to enable the MallocTracer engine.
It can profile malloc, calloc, realloc and free invocations that are dynamically linked to the target.
When N is specified, only one allocation every N bytes is sampled.
Profiling with nativemem
java -agentpath:"$ASPROF_LIB/libasyncProfiler.so=start,nativemem,file=/dev/stderr" <app>

A flame graph can be generated directly as the profiler output option, which includes all sampled malloc calls. Alternatively, use jfrconv to process a jfr file and chart only the memory leaks:
Memory leak analysis via jfrconv
java -agentpath:"$ASPROF_LIB/libasyncProfiler.so=start,event=cpu,nativemem,file=malloc.jfr" <app>

jfrconv --total --nativemem --leak malloc.jfr malloc-leak.html

# without leak analysis, includes all collected samples:
jfrconv --total --nativemem malloc.jfr malloc.html

Generated flame graphs unit is in bytes if total is specified, otherwise in number of invocations.
asprof
If the profiler is attached via asprof, nativemem profiling can be started and stopped multiple times for the same process as necessary.
asprof start -e nativemem --total <app>
asprof stop <app>
...

How has this been tested?

New tests added that call malloc functions with and without multi-threading.

Sample outputs
All allocations
From a test application.

Memory leaks only
Profile of the same application, processed via jfrconv.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Exciting to see memory profiling as part of async profiler! :clap:  I mentioned a few of the gotchas we encountered while implementing something similar in case this can be of interest.


add stellamarisdavao.txt

Hi @philipto
This is related to this previous  PR you close this because you mention that there is a sign of fraud, but you open the wrong url. The correct url of the school is https://www.stellamarisdavao.edu.ph/
which is this

And has its certificate is here

@twocngdagz First stage of verification passed, official domain is stellamarisdavao.edu.ph as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@philipto
The official domain is  https://www.stellamarisdavao.edu.ph/ your missing the 'www' in you link.

@twocngdagz  Thank you for the clarifications. I apologize for the misunderstanding. Pull request merged.

@philipto
Thank you


Make Table  use-case example  overflow scrollable

Is your feature request related to a problem? Please describe.
When you select all the columns for the view.
It appears but the table is not scrollable  the overflow is hidden which blocks other information
Describe the solution you'd like
Add an overflow auto or scrollable to the components table to nativly manage it without the user managing it himself
Describe alternatives you've considered
table should be scrollable
Screenshots or Videos

do you mean when the column popover is open, the table cannot be scrolled?
please provide a minimal reproducible environment

when we  display all columns  in the popover
the table  cannot be scrolled  to view all columns

So when you choose more columns to display, the width of the table is increasing and you mean horizontal scrolling is not available in your case right? Still, please provide a minimal reproducible environment.

EXACTLY

assuming you're talking about this use case example, I'm able to scroll after adding all columns.

closing - not reproducible + inactivity


Add tox.ini to MANIFEST.in

And check for other missing files.

tox just doesn't work from sdist given versioneer, but examples should be included.


proposal: golang.org/x/text/unicode/ucd package for Unicode Character Database data

I'd like to write a self-contained utility program to print what Unicode codepoints are covered by a font file. I have codepoint values (e.g. '\u0102') and I'd like to print e.g. "LATIN CAPITAL LETTER A WITH BREVE".
I'd therefore like a golang.org/x/text/unicode/ucd package for Unicode Character Database data (http://unicode.org/ucd/).
There is an existing golang.org/x/text/internal/ucd package, but it's (1) internal, and (2) currently about parsing UCD files instead of exposing pre-generated tables.
There is also an existing golang.org/x/text/unicode/cldr package (CLDR, not UCD), which is not internal, and it is also about parsing. However, its package doc says "This package is inteded to be used by the table generation tools for the various internationalization-related packages." The git log for that directory ends with "The cldr package is quite low-level and probably would only for internal use if it hadn't predated the internal directory convention." Maybe we should make it internal, although that is a separate concern.
Back to a "package ucd", a strawman proposal for its API would be:
fmt.Println(ucd.Name('\u2603'))           // Prints "SNOWMAN".
fmt.Println(ucd.Block('\u2603'))          // Prints "Miscellaneous Symbols".
fmt.Println(ucd.HanDescription('\u4E2D')) // Prints "central; center, middle; in the midst of; hit (target); attain".

Other possible package names are "chardb" and "codepoints", so it would be "chardb.Name" or "codepoints.Block", but for now, I'll paint the bikeshed "ucd", consistent with the "cldr" abbreviation.
Canonical sources:
http://www.unicode.org/Public/UCD/latest/ucd/NamesList.txt
http://www.unicode.org/Public/UCD/latest/ucd/Unihan.zip and its Unihan_Readings.txt

Seems fine to me.

SGTM.
Regarding the naming, though: is your intention to expose all data in the ucd package? What about the derived and test data or data that is already exposed in other packages? If not, I am not very keen on the name ucd, as it would be akin to a Unicode equivalent of calling a package "util". Personally I would split data into functional packages, such as one for names/chardb, emoji, indic, hangul, vertical, etc. to complement the packages that already expose UCD data (norm, width, cases, bidi, etc.) The names package alone would be huge, let alone a unihan package.
Also, although the cldr package should probably be internal, it is conceivable that the ucd parser would be exposed.

Is ucd going to hard-code the tables as data or is it going to expect to find them at run time?

@mpvl, yeah, not all of UCD. Perhaps the bikeshed should be x/unicode/names, x/unicode/blocks, x/unicode/unihan (unihan.Description), etc. loosely based on the filenames in http://www.unicode.org/Public/UCD/latest/ucd/

@rsc hard-code the tables as (go generate'd) data.

@mpvl, looking at some Lookup functions in the existing golang.org/x/text sub-repo:
width/width.go
74:func Lookup(b []byte) (p Properties, size int) {
81:func LookupString(s string) (p Properties, size int) {
87:func LookupRune(r rune) Properties {
unicode/bidi/prop.go
61:func LookupRune(r rune) (p Properties, size int) {
85:func Lookup(s []byte) (p Properties, sz int) {
148:func LookupString(s string) (p Properties, sz int) {
(I note that width.LookupRune and bidi.LookupRune are inconsistent wrt returning a size... but that's a side conversation).

Anyway, for a concrete proposal:
A new package, golang.org/x/text/charnames. It contains one function, func Name(rune) string.
For example, charnames.Name('\u0201') returns "LATIN SMALL LETTER A WITH DOUBLE GRAVE".

@nigeltao To be consistent with Go, why not runenames or runename?  The latter reeds a bit better if the intended API is to be of the form runename.Lookup(b).
Also this package is quite low-level and I would therefore not put it at top level. The most obvious place is golang.org/x/text/unicode/runename. An alternative could be golang.org/x/text/runes/runename (but I think I like the first better). Other low-level packages, like bidi, norm, and rangetables are already under x/text/unicode.
The LookupRunes should probably be made consistent, indeed.

I went with char because the C in UCD stands for Character. But I don't feel strongly either way, and I'm happy to roll with rune.
I went for names plural to be consistent with the existing golang.org/x/image/colornames package.
It's not entirely obvious to me when something is x/foo and when it's x/unicode/foo (why is looking up rune names low level and not high level??), but again, I don't feel strongly either way, and I'm happy to roll with x/unicode/runenames.
As for the function name being Lookup, that seems inconsistent with the Lookup functions in other packages, which take a []byte argument and return a size.
Calling the function Name instead of LookupRune is shorter, but again, I don't feel strongly either way, so if you prefer LookupRune, let's roll with that.
(I'm not sure if it's a good idea, but while I think of it, another possibility is to rename Lookup and LookupRune to LookupBytes and Lookup across all of the golang.org/x/text packages).
Anyway, new new proposal:
package import path is golang.org/x/text/unicode/runenames
It has one function:
func LookupRune(r rune) (name string)

Admittedly, calling it Name will be consistent with language/display. In fact, this package defines a Namer interface, although it might be too contrived to make it fit in this case.
I just didn't like the stutter of runename.Name(). But I'm fine with Name, if you don't think this is an issue.
I'm also fine with runenames if you feel strongly about this and for the sake of consistency. I guess runenames.Name is less bad then runename.Name, if you go with that.
The high vs low-level is indeed somewhat arbitrary. Common vs. uncommon usage is another part of it. Or pure Unicode or not. Package width should probably go in the unicode directory too. Package cases and collate also mix in CLDR data.
What would you rename all the Lookup methods too?

(Side-track Conversation Number 1: For high vs low-level, uses UCD/CLDR or doesn't distinctions, another idea is just flatten everything, so x/text/unicode/foo becomes x/text/foo for foo in {bidi,norm}, with the presumption that anything under x/text imples Unicode, other than x/text/encoding/bar which converts to and from Unicode. Separately, x/text/unicode/cldr would become an internal package. Maybe the same for x/text/unicode/rangetable but I don't have a good sense for who uses it.)
(Side-track Conversation Number 2: To clarify renaming Lookups, I meant: s/Lookup/LookupBytes/ and s/LookupRune/Lookup/.)
As for the actual topic, a new new new proposal: golang.org/x/text/unicode/runenames and func Name(rune) string. We could take the "unicode" out of the import path pending Side-track Conversation Number 1.

ST 1) I don't think it is a good idea to give up unicode now. There is a lot more to add and a flattened hierarchy would become quite large. Also, if that approach is taken, it may be good to group all shaping- and rendering-related packages into a single directory (bidi, arabshaping, vertical, etc.), but I'm not sure such a reorganization would be better.
ST 2) I think such a renaming would be gratuitous and even undesirable. The primary focus of text is centered around bytes, second strings, then runes. It is often iffy to deal with text in units of runes, so this should be discouraged as much as possible, e.g. by making the text APIs feel like they are byte-sequence-centered foremost.  Also, the current naming is consistent with the core utf8 package.
My preference of naming is, from most-preferred to least:
golang.org/x/text/unicode/runenames
golang.org/x/text/runes/runenames, or  // similar to language/display
golang.org/x/text/runes/names, or         // similar to language/display
golang.org/x/text/runenames
If you feel strongly about either you can go with that.  I'm okay with using the Name signature as you proposed in this case, as it is a very rune-centered package. But generally speaking lookups should be optimized for bytes in API and implementation. So I would consider this to be an exception.

@mpvl my reading of your latest comment is that we both agree with the new new new proposal. To repeat: golang.org/x/text/unicode/runenames and func Name(rune) string.

@nigeltao: ack


CLOUD VISION API WITH FLUTTER.

I want to use CLOUD VISION API with Flutter to detect landmarks with node.js as backend, But can't find any tutorial online to do so.
Anyone can help???

Hi @sameep-s
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/ .


Added implementations for get_token_largest_accounts() and get_token_supply()

I have added implementations for the get_token_largest_accounts() and get_token_supply() methods in the synchronous RPC client as well as their respective _args methods in rpc.core.

Would you mind writing a couple integration test for these?

I've gone ahead and fixed the formatting issues and this latest commit is passing with Black. I also wrote integration tests for both methods.
I created a stubbed_token_addr() method for testing as both of the new functions require a known valid token address to return results instead of an error.

Thank you, was not aware of the constant. I went ahead and made all of the suggested changes in this new commit and added an import for WRAPPED_SOL_MINT in test_http_client.py

@whdev1 Would you mind adding the async methods and tests too? If you're not familiar with async stuff in Python, you can look at async_api.py and test_async_http_client.py and just mimic those. Will just be a matter of copy-pasting and changing a few characters (Python forces a bit of duplication when it comes to sync/async stuff)

@kevinheavey Definitely. I just added a new commit with the async implementations for both. I've also changed the title of the PR to reflect this.

Thank you, just fixed both of those issues.

lgtm, @michaelhly does it look good to you?


Shipping docker images with github releases and github actions

If docker is the main form of deployment there should a docker img built and deployed with every release. There's likely a github action that can do this but I'm really only familiar with GoReleaser  which is Go specific.

Thanks @TimothyStiles this is a good suggestion.
Yes, you are correct.  We do build and deploy with every release.  We currently only push the Docker image to AWS ECR.  We should also push the Docker image to Docker Hub, or perhaps replace AWS ECR with Docker Hub entirely.


Add new 202111 month-avg variable groups

biology growth rates
grazing
light
mortality
vvl grid

Tested on salish to create 2007-01-01 to 2023-12-31 climatologies for grazing/mortality & biology growth rates fields.


Handle instance permission set in k8s runner resource.

This closes #236

To be merged after https://github.com/iterative/terraform-provider-iterative/pull/683


Fix code style warnings in communication module

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[ x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
azdev style communication PASSED
[ x] Have you run python scripts/ci/test_index.py -q locally?
azdev test communication PASSED

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

communication

Published again PR 4959.


RuntimeError: CUDA error: device-side assert triggered

Hello,
I'm training the code on docker using pytorch/pytorch:1.10.0-cuda11.3-cudnn8-devel as base image with python 3.8. Setup worked fine until I tried to train the code, then these error came out:
/opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [33,0,0], thread: [57,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [16,0,0], thread: [52,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [18,0,0], thread: [0,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [16,0,0], thread: [93,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [17,0,0], thread: [75,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [17,0,0], thread: [87,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [17,0,0], thread: [99,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [17,0,0], thread: [110,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [17,0,0], thread: [126,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/cuda/IndexKernel.cu:91: operator(): block: [18,0,0], thread: [92,0,0] Assertionindex >= -sizes[i] && index < sizes[i] && "index out of bounds"failed. epochs:   0%|                                                                                | 0/6 [00:13<?, ?it/s] Traceback (most recent call last):   File "Models/SFD/tools/train.py", line 212, in <module>     main()   File "Models/SFD/tools/train.py", line 167, in main     train_model(   File "/workspace/Models/SFD/tools/train_utils/train_utils.py", line 86, in train_model     accumulated_iter = train_one_epoch(   File "/workspace/Models/SFD/tools/train_utils/train_utils.py", line 38, in train_one_epoch     loss, tb_dict, disp_dict = model_func(model, batch)   File "/workspace/OpenPCDet/pcdet/models/__init__.py", line 44, in model_func     ret_dict, tb_dict, disp_dict = model(batch_dict)   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl     return forward_call(*input, **kwargs)   File "/workspace/Models/SFD/pcdet_extensions/models/detectors/sfd.py", line 11, in forward     batch_dict = cur_module(batch_dict)   File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl     return forward_call(*input, **kwargs)   File "/workspace/Models/SFD/pcdet_extensions/models/roi_heads/sfd_head.py", line 595, in forward     self.roicrop3d_gpu(batch_dict, self.model_cfg.ROI_POINT_CROP.POOL_EXTRA_WIDTH)   File "/workspace/Models/SFD/pcdet_extensions/models/roi_heads/sfd_head.py", line 554, in roicrop3d_gpu     image[total_pts_features[:,7].long(), total_pts_features[:,6].long()] = global_index.to(device=total_pts_features.device) RuntimeError: CUDA error: device-side assert triggered
It seems like there is an index error from the function roicrop3d_gpu. Could you please verify this? I got stuck for a few days already. Thank you!

hello，have you  solved the problem?  and how?

hello，have you solved the problem? and how?

hello，have you solved the problem? and how?

I have solved the problem according to the method in this webpage. https://github.com/LittlePey/SFD/issues/23
May I ask how you solved it?

hello，have you solved the problem? and how?

I have solved the problem according to the method in this webpage https://github.com/LittlePey/SFD/issues/23. May I ask how you solved it? @vacant-ztz


proposal: support instrumentation

It would be useful if the cert manager had support for metrics.
Some useful data to keep track of:

number of certs issued/renewed
time to acquire a cert
number of errors

I use prometheus to monitor our k8s services, but some people use other monitoring solutions. I'd like to add metrics support to this projects, using the github.com/go-kit/kit/metrics library with initial support for prometheus and expvar endpoints.
Would you be interested in this? Any thoughts/concerns?

As long as things keep working if there is no metrics endpoint available, yes, this would be absolutely very interesting. It could also be mixed with #13 and #12 as it'd probably touch the same code


EOS-19534 Consul Hare Agent to be started by Hare in Mini-Provisioner Init Phase

Solution: Modified code to skip consul stop during shutdown, also note that consul will be restarted during bootstrap

run vm-deployment

test vm-deployment

test this please


usage of omvll.StringEncOptStack() causes application crash

Hi,
I'm trying the release version 1.1.0 from https://github.com/open-obfuscator/o-mvll/releases/download/1.1.0/omvll_ndk_r26d.tar.gz
Enabling "stack string obfuscation" via omvll.StringEncOptStack() causes application crash.
If I use omvll.StringEncOptGlobal()  instead, everything works fine.
Romain told me that it might be caused by the latest changes in the code.
Could you please take a look at it?

Please make sure that this conforms with what you're trying to do. As from the documentation, when StringEncOptStack is enabled, the decoded string lives in a stack-allocated buffer. This implies that the lifetime of the buffer is tied to the lifetime of the function where the string is used. We plan to upstream an option which decodes strings on-the-fly and doesn't use neither stack-allocated buffers nor global constructors some time soon.