dependency not found in repo

tried 1.0.0 and 1.0.0-grails3, actually checked the links and there is no folder for this plugin there.
| Error Error initializing classpath: Could not find org.grails.plugins:simple-captcha:1.0.0-grails3.
Searched in the following locations:
    file:/home/pablo/.m2/repository/org/grails/plugins/simple-captcha/1.0.0-grails3/simple-captcha-1.0.0-grails3.pom
    file:/home/pablo/.m2/repository/org/grails/plugins/simple-captcha/1.0.0-grails3/simple-captcha-1.0.0-grails3.jar
    https://repo.grails.org/grails/core/org/grails/plugins/simple-captcha/1.0.0-grails3/simple-captcha-1.0.0-grails3.pom
    https://repo.grails.org/grails/core/org/grails/plugins/simple-captcha/1.0.0-grails3/simple-captcha-1.0.0-grails3.jar
    https://jitpack.io/org/grails/plugins/simple-captcha/1.0.0-grails3/simple-captcha-1.0.0-grails3.pom
    https://jitpack.io/org/grails/plugins/simple-captcha/1.0.0-grails3/simple-captcha-1.0.0-grails3.jar

| Error Error initializing classpath: Could not find org.grails.plugins:simple-captcha:1.0.0.
Searched in the following locations:
    file:/home/pablo/.m2/repository/org/grails/plugins/simple-captcha/1.0.0/simple-captcha-1.0.0.pom
    file:/home/pablo/.m2/repository/org/grails/plugins/simple-captcha/1.0.0/simple-captcha-1.0.0.jar
    https://repo.grails.org/grails/core/org/grails/plugins/simple-captcha/1.0.0/simple-captcha-1.0.0.pom
    https://repo.grails.org/grails/core/org/grails/plugins/simple-captcha/1.0.0/simple-captcha-1.0.0.jar
    https://jitpack.io/org/grails/plugins/simple-captcha/1.0.0/simple-captcha-1.0.0.pom
    https://jitpack.io/org/grails/plugins/simple-captcha/1.0.0/simple-captcha-1.0.0.jar

You can get it here, repository changed.
https://mvnrepository.com/artifact/org.grails.plugins/cache-headers/2.0.2
and the repository is https://dl.bintray.com/grails/plugins/

You can get it here:
compile 'domurtag.plugins:grails-simple-captcha:1.0.0-grails3'


Add stylua config and format

Edit: I hope you don't mind multiple changes in one pull request, seems I don't really understand the github cli all too well. I'll try add any comments about changes to their respective commits.
This adds a stylua config, .stylua.toml, found in and copied from the neovim/neovim repository. Afaik stylua is the informal standard for formatting lua in the context of neovim and the stylua config used for the neovim repo itself seems like a good default. This then brings the formatting of texpresso.vim in line with other neovim plugins.

Thanks, I am not familiar with lua/neovim ecosystem, this helps a lot.
I am worried about the impact of replacing nvim_get_hl_by_name by  nvim_get_hl on version compatibility. Do you know what is the story usually for managing these kinds of API migration?
It seems that nvim_get_hl have been introduced recently (neovim 0.9.5) while I know some linux distributions are still using old versions (I think the current plugin is compatible with 0.7.x, but I cannot guarantee that).

Good point! I hadn't even considered the version requirement, not so surprisingly as I've been compiling neovim from the main branch for I believe almost a decade by know.
Afaik api migrations such as these are mostly handled with an if-else conditional on a version check.
But for the audience of texpresso I don't think requiring a recent version of neovim should be a problem.
Texpresso itself already requires the know how to compile it, so I would think compiling a newer version of neovim shouldn't be too much of a challenge. If that's not possible, one can still get the latest neovim release as a binary or appimage, released by neovim itself.
On macOS it shouldn't be a problem at all, because the version of neovim available via brew and macports is the latest available.
I personally don't think the advantage of keeping up compatibility with older versions outweighs that of the increased development load, code fully up to date using the latest api's and other improvements which might be available in the latest version of neovim.
Neovim plugins seem to move quite fast anyhow, for example at times even requiring unreleased versions of neovim for some feature (although this shouldn't be relevant for texpresso.vim).
Although I prefer the advantages of a recent neovim version over compatibility I think there aren't any conclusive arguments for one or the other and as such the choice is up to you. If you choose compatibility I can revert the change and will revisit whenever the function is fully deprecated to add the required if-else.
Are my comments above on some of the code in this pull request visible to you? If you could answer those I could move forward with some further, more substantial, improvements.

One of the more substantial changes I wanted to make is to convert vim.fn.jobstart to vim.system.
The reasons for preferring vim.system over vim.fn.jobstart are:

doesn't incur the cost of converting vimscript values to lua and vice versa
documentation states that it should be used unless the pty option is required (which the current code does not)
vim.system is asynchronous, vim.fn.jobstart might be but it isn't mentioned in its documentation.

There's more detail in the pull request adding vim.system https://github.com/neovim/neovim/pull/23827.
If the performance benefits of vim.system are noticeable this might make for a more substantive case for dropping compatibility with older versions of neovim. I believe vim.system is available from version 0.9, although I couldn't really find out from which point it is supported exactly. It is mentioned in the online documentation which I'd assume is for the latest stable version but I couldn't confirm this.
I'll try and make the changes locally to see if it has any benefits, I'll have to try and understand the logging system better however as it requires changes since the code currently isn't compatible with asynchronous execution. The log buffer isn't common in neovim plugins afaik so it might be better to take a different approach, this is what some of the question/comments on the code were about.

Thanks for the suggestions, and sorry for the late answer.
About compatibility: The current plugin works with Neovim 0.7.2 which is shipped with Ubuntu 23.10. I don't want to break compatibility because I don't think it is reasonable to expect someone to build a custom version of their editor just to try out an experimental software, so I prefer to stay to keep things as compatible as possible.
Neovim 0.6.x is not supported because of the script uses nvim_create_autocmd which was not available. This is a bit sad because Ubuntu 22.04 LTS still ships this version of Neovim. In private communications I heard that there even are some users on Ubuntu 20.04 LTS but with Emacs so the old version was less of a problem (... well old Emacs are quite buggy when it comes to buffer synchronization, that's another story).
So I would like to retain as much compatibility as possible with older LTS distributions.
That being said, the improvements you are suggesting seem very nice. Apparently, vim.system is very recent: according to https://github.com/R-nvim/R.nvim/issues/36, it is not available in 0.9.5. It will take some time to become widely available, so two code paths are needed. Maybe we could have two versions of texpresso.lua:

a compatible one, aiming at good backward compatibility,
an efficient one, following the recommended and latest neovim practices.

To minimize maintenance burden, development could happen only on the efficient one, and new features or bugfixes could be backported only when justified. My vision is that it is fine to require a newer version for newer features, but it is not fine to break an otherwise functional installation with a minor update.
About logging (and error report, e.g. the use of :copen and a quick fix list): I hacked something that was good enough for my workflow, but I have no idea how to properly integrate such features in neovim. I welcome any improvements, but it seems I can't see the comments you are referring too.

Couldn't blame you for a late answer because here's mine :) But in all seriousness, definitely no worries, I'm already really grateful for the great program you've made for everyone to use and everything else is an added bonus.
I think you're right in that the best approach would be to create two versions of the plugin and to backport anything relevant. I think the easiest way to do so is to simply create another branch for the version using the latest api's, there's precedent for this as other plugins have taken this approach before, for example nvim-treesitter.

About compatibility: The current plugin works with Neovim 0.7.2 which is shipped with Ubuntu 23.10. I don't want to break compatibility because I don't think it is reasonable to expect someone to build a custom version of their editor just to try out an experimental software, so I prefer to keep things as compatible as possible.

There's a small misunderstanding here however, there's no need to build anything as the neovim project releases builds of the latest stable & dev version, including appimages (which are more akin to how you'd install an app on macOS). So that has the added benefit that even if users are on an older distro they could use the version of the plugin with the latest api's, without having to build anything.

About logging (and error report, e.g. the use of :copen and a quick fix list): I hacked something that was good enough for my workflow, but I have no idea how to properly integrate such features in neovim.

A better approach to this might be to try and integrate texpresso with vimtex. I'd say vimtex is the, so to speak, industry standard for latex editing using (neo)vim. I've not had time to look at this integration properly, or really at all in any detail, but if it is is possible texpresso.vim might not be needed at all as vimtex would be taking care of all its functionality. Even better, its main developer, lervag, is very helpful and responsive.
Vimtex already supports tectonic so it might not even be much work to get the diagnostics part working, I'm not quite sure how launching texpresso and the communication between neovim and texpresso would work however.
I'll first try and figure out whether this integration is possible asap. If it's not I'll split this PR up into the formatting part, which should be applicable to both the current and eventual newer version of this plugin, and make any further improvements then.

I welcome any improvements, but it seems I can't see the comments you are referring too.

I still have those questions but they'll only be relevant if the integration isn't possible so I'll ask and copy them to the comments by then.

Couldn't blame you for a late answer because here's mine :) But in all seriousness, definitely no worries, I'm already really grateful for the great program you've made for everyone to use and everything else is an added bonus.

Thanks, that's nice to hear :).

A better approach to this might be to try and integrate texpresso with vimtex. I'd say vimtex is the, so to speak, industry standard for latex editing using (neo)vim. I've not had time to look at this integration properly, or really at all in any detail, but if it is is possible texpresso.vim might not be needed at all as vimtex would be taking care of all its functionality. Even better, its main developer, lervag, is very helpful and responsive.
Vimtex already supports tectonic so it might not even be much work to get the diagnostics part working, I'm not quite sure how launching texpresso and the communication between neovim and texpresso would work however.
I'll first try and figure out whether this integration is possible asap. If it's not I'll split this PR up into the formatting part, which should be applicable to both the current and eventual newer version of this plugin, and make any further improvements then.

Integration with Vimtex would be ideal indeed. Thanks for looking into this.

I still have those questions but they'll only be relevant if the integration isn't possible so I'll ask and copy them to the comments by then.

I am ready to answer those questions ;).

I've been experimenting with integrating texpresso into vimtex.
At the moment, only live editing works and it only works on vanilla vim.
The code can be found here: https://github.com/tsung-ju/vimtex/blob/texpresso/autoload/vimtex/compiler/texpresso.vim
Feel free to use the code if you find it useful!


Author section

Hi,
I have a problem with my api rest, when i'm try to get the user list, I have an error "You don't have permission to get the list" but in your issue #30 you say no need auth with api v2.
Thanks

Getting users does not require any permission : http://demo.wp-api.org/wp-json/wp/v2/users
Please share your API url.

https://stadion-actu.fr/wp-json/wp/v2/users

I have detected the problem, it's wp security plugins the problem. I'm search where is the option. Thk :)


broker emitting noisy warnings with GCS backing store

Problem
There are a large number of warnings frequently being logged by brokers assigned to journals with stores on GCS. There is a warning from each journal every few minutes, but across all journals, this adds up to a lot of noise.
That said, this issue does not prevent the broker from performing its duty, so it is merely noisy, not broken.
Proposals
The hypothesis is that the containing directory is included in the iteration across the objects since it matches the prefix in the query.

Look for a way to exclude the directory from the results.
Recognize the directory do not attempt to parse its name as a fragment file.
Suppress the warning when the object is the directory.

Example logs
Apr 2 11:59:45 hoping-pike-gazette-ffb6f6479-qclbr gazette warning {"bucket":"XXXX","err":"wrong Fragment format: YYYY","level":"warning","msg":"parsing fragment","name":"ZZZZ/YYYY/","time":"2019-04-02T15:59:45Z"}

What's odd about this is that something else is creating "directory"-like empty objects in the fragment store. gazette itself doesn't do this. Still, handling it is reasonable, as there are legitimate reasons for wanting these (eg, it's being mounted / interacted with as a GCS FUSE file-system, for example).

Fixed.


[14.0.x] ISPN-14527 Meta model may not reflect some schema changes

https://issues.redhat.com/browse/ISPN-14527
https://issues.redhat.com/browse/JDG-5997
Backport of #10659

Merged, thanks

Thanks Tristan!


How to resize the input blob and how to define a new output layer with a pretrained Network?

Hi,
i am using this https://github.com/leonardvandriel/caffe2_models pretrained ResNet-Model and i am trying to get it to work with bigger pictures (FullHD), but i couldn't find anyway to resize the input blob like i did in caffe(1):
net.blobs['data'].reshape(1,3,width,height)
I already tried #614 and #956 but i couldn't get it to work. I am using the newest version of caffe2 and python (2.7.12).
The next thing is i would like to cut the network before the fully connected layers, in caffe i would just remove the layers in the .prototxt, but here  i don't know how to do it.
Thanks

@MarviB16 Hi, I'm working on removing layers, can you please tell me how to do thai? Thanks a lot!

related issue: after changing the input dimensions the model is failing at fc layers
How do we remove the fc layers or change dimensions of fc layers?


SampleClass should return a Class statement instead of a constructor

We should refactor the constructor statement such that it is a child of a class statement.

Solved by #158


Error running through Docker

when I use the command: "docker run -d -p 9117:9117 apache_exporter", all I get is this error: standard_init_linux.go:185 exec user process caused "exec format error.
This is my Dockerfile:
FROM quay.io/prometheus/busybod:latest
COPY apache_exporter.go /bin/apache_exporter
ENTRYPOINT ["/bin/apache_exporter"]
EXPOSE          9117

I imagine the answer at https://github.com/prometheus/busybox/issues/11 solved your issue. Closing this issue.


Menu alignment issue in IE

Create Profile page the "Get started" button and Terms of service are not aligned in IE. Maybe they should be left justified.

@skylenewman not sure I understand what you mean. I am using http://netrenderer.com/index.php to test in IE and according to this tool's IE9 test, the "By clicking 'Get started!'" text is left aligned below the "Get Started!" button, which is how it's supposed to be.

This was a Wunderlist item from Joe.


tpl function fails when called within a named template that uses a dict

Output of helm version:
version.BuildInfo{Version:"v3.5.2", GitCommit:"167aac70832d3a384f65f9745335e9fb40169dc2", GitTreeState:"dirty", GoVersion:"go1.15.7"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.11", GitCommit:"c6a2f08fc4378c5381dd948d9ad9d1080e3e6b33", GitTreeState:"clean", BuildDate:"2021-05-12T12:27:07Z", GoVersion:"go1.15.12", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.12-gke.2100", GitCommit:"82b41aad731f2dda24cc5ca99e09328163474782", GitTreeState:"clean", BuildDate:"2021-07-16T09:23:20Z", GoVersion:"go1.15.13b5", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.): GKE
tpl fails when being used inside a named template that uses a dict to pass in values. Here is my use case:
values.yaml:
sso:
  oidc:
    host: foo
    realm: bar
  token_url: "https://{{ .Values.sso.oidc.host }}/auth/realms/{{ .Values.sso.oidc.realm }}/protocol/openid-connect/token"
  auth_url: "https://{{ .Values.sso.oidc.host }}/auth/realms/{{ .Values.sso.oidc.realm }}/protocol/openid-connect/auth"

template.yaml:
...
{{- $enabled := true }}
{{- $defaults := include "my.named.template" (dict "root" .  "enabled" $enabled) }}

{{- define "my.named.template" -}}
...
    auth:
      signout_redirect_url: {{ .root.Values.monitoring.sso.grafana.signout_redirect_url | default (tpl "https://{{ .root.Values.sso.oidc.host }}/auth/realms/{{ .root.Values.sso.oidc.realm }}/protocol/openid-connect/logout" .) }}

Running helm install test ./chart --dry-run --debug generates the following error:
Error: template: templates/monitoring/values.yaml:6:17: executing "templates/monitoring/values.yaml" at <include "my.named.template" (dict "root" . "enabled" $enabled)>: error calling include: template: templates/monitoring/values.yaml:73:99: executing "my.named.template" at <tpl "https://{{ .root.Values.sso.oidc.host }}/auth/realms/{{ .root.Values.sso.oidc.realm }}/protocol/openid-connect/logout" .>: error calling tpl: cannot retrieve Template.Basepath from values inside tpl function: https://{{ .root.Values.sso.oidc.host }}/auth/realms/{{ .root.Values.sso.oidc.realm }}/protocol/openid-connect/logout: "BasePath" is not a value

Calling .root.Values.xxx works as expected for accessing globally scoped values but not within tpl.
The behavior is similar to https://github.com/helm/helm/issues/5979, but I'm not using range anywhere and none of the workarounds provided in the issue work for my use case.

@danehans In tpl, try {{ root.Vaules.xxx }}

@yxxhero when using root.Values:
Error: parse error at (<SNIP>.yaml): function "root" not defined

I assume you mean .root and not root. When using .root.Values:
error calling include: template: <SNIP>.yaml: executing "my.named.template" at <.root.Values>: nil pointer evaluating interface {}.Values helm.go:81

@danehans  oh, sorry. I didn't see the problem clearly.  Can you show the whole my.named.template.

try default (tpl "https://{{ .Values.sso.oidc.host }}/auth/realms/{{ .Values.sso.oidc.realm }}/protocol/openid-connect/logout" .root)

@yxxhero thanks for the help. Unfortunately the above did not work. I have a few templates that use tpl that are experiencing this issue. Here are the details. PTAL and let me know if you have any suggestions.

Hello. I'm having the same issue using Helm 3.10.3.
My named template looks like...
{{/*
    @param .string      The template to convert
*/}}
{{- define "mystuff.tpl" -}}
{{ tpl $.string $ }}
{{- end -}}

{{ include "mystuff.tpl" (dict "string" .Values.someString) }}

Result:
[ERROR] templates/: template: templates/3_deployments/test.yaml:103:3: executing "templates/3_deployments/test.yaml" at <include "mystuff.tpl" (dict "string" .Values.someString)>: error calling include: template: templates/_helpers.tpl:51:3: executing "mystuff.tpl" at <tpl $.string $>: error calling tpl: cannot retrieve Template.Basepath from values inside tpl function: Hello World: "BasePath" is not a value

This is also easily repro'able using the following as well.
{{- define "mystuff.tpl" -}}
{{ tpl "Some Test String" $ }}
{{- end -}}

{{ include "mystuff.tpl" (dict "string" "Hello World!") }}

The output is the same...
[ERROR] templates/: template: templates/3_deployments/test.yaml:103:3: executing "templates/3_deployments/test.yaml" at <include "mystuff.tpl" (dict "string" "Hello World!")>: error calling include: template: templates/_helpers.tpl:48:3: executing "mystuff.tpl" at <tpl "Some Test String" $>: error calling tpl: cannot retrieve Template.Basepath from values inside tpl function: Some Test String: "BasePath" is not a value

Try
{{- define "mystuff.tpl" -}}
{{ tpl "Some Test String/{{ .string }}" . }}
{{- end -}}

and
{{ include "mystuff.tpl" (merge $ (dict "string" "Hello World!")) }}

I think the real solution would have been:
{{- $enabled := true }}
{{- $defaults := include "my.named.template" (merge $ (dict "enabled" $enabled)) }}

{{- define "my.named.template" -}}

...
    auth:
      signout_redirect_url: {{ .Values.monitoring.sso.grafana.signout_redirect_url | default (tpl "https://{{ .Values.sso.oidc.host }}/auth/realms/{{ .Values.sso.oidc.realm }}/protocol/openid-connect/logout" .) }}


Reformat examples as part of the build

This adds a step to the build process to reformat the example JSON files. The main reason is that this allows us to quickly edit all the examples. For example, #444 removes the format field simply by adding data.delete("format") to the rake task.
This will also prevent errors with duplicated keys in the JSON (converting it to a Ruby hash will solve them).
PR prompted by https://github.com/alphagov/govuk-content-schemas/pull/444#discussion_r90045687.
cc @andrewgarner

👍
After my comment yesterday I was also thinking about whether we could incorporate something like https://github.com/PagerDuty/jsonlint

With this I think we catch most errors. The examples are validated against the schemas, so any syntax errors are picked up already.

:100: 👍

Can something similar be applied to details.json and links.json?
A PR made it through today that has incorrect indentation which I think this would have prevented:
https://github.com/alphagov/govuk-content-schemas/pull/447#pullrequestreview-10723190

Sure, I don't see any reason not to do that. Seems useful!


mWeb - Reimburse expenses - Rate amount flashes

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
Login with any account
Go to Settings -> Workspaces -> Open any WS -> Reimburse expenses
Focused rate input field

Expected Result:
Rate amount doe snot flash
Actual Result:
Rate amount flashes
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/234954822-00538473-b2fe-425a-a62e-45839fe5f868.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @arielgreen (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Need to reassign this

Triggered auto assignment to @kevinksullivan (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Reviewed on android chrome and confirmed the amount flashes. Passing on!

Job added to Upwork: https://www.upwork.com/jobs/~019635b5c9f7089d36

Current assignee @kevinksullivan is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia (External)

Triggered auto assignment to @hayata-suenaga (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Asked for help in the open source channel

Is this still occurring? I think it's fixed by https://github.com/Expensify/App/pull/18266

@thesahindia would it be okay for you to try reproduction again on staging?

Sounds good


DTS related question

Hi Seb,
Thanks for all the great work you've done. I'm wondering about the device tree source file that you've written. The FPGA configuration that I'm using will not enable some of the peripherals in Platform Designer, as you've enabled for the DE0-NANO. Will Linux crash if this is the case, that your dts file contains support for peripherals that the Platform Designer app has not enabled in my FPGA project?
Will I have to edit your device three source file? Or can I inherit it as it is?
I'm using EMAC1 (HPSI/O Set 0),  SD/MMC (HPS I/O set 0, 4-bit data),  and I2C0 (HPS I/O Set 1). The above noted peripherals I've enabled in Platform Designer. Also, I've configured the AXI Bridges as you guide in this repo describes.
All the best

The Linux will only crash in case it tries to access non-accessible addresses. I enbaled whithin my Linux Distro the Kernal Debug Mesages. The Kernel will show in case a driver is not loadebile a Error Msg. This will be the case if the Linux tries to load a driver that was not enabable inside Quartus Prime Platform Designer. Based of a bug in SoC EDS 18.1 I had this isssue during developing often.

Thanks for your reply. Then it seems like the Kernel will output error messages, however this will not result in Linux crash as long as the drivers are not accessing non-accessible addresses. I can then proceed with using your DTS file. Many thanks robseb.

Yes, you can also follow my rsyocto guide to create a own version. This guide here is a older version with EDS 18.1. The Quartus Prime FPGA side is vailed until today. The Linux side chnagned.

Ok cool. My intention is to use the DTS file from rsyocto release 1.041, located inside folder for DE0 Nano platform. Then I've followed the guide in https://github.com/robseb/meta-intelfpga for building the Yocto powered Linux distribution. Then I'll intend to apply the rs-tools layer to leverage HPS-FPGA comms. Then I'll try to but it all together with https://github.com/robseb/socfpgaPlatformGenerator for generating the SD card image.


Uncaught TypeError: Cannot set property 'innerHTML' of null at qunit:2638

I've just updated from 1.15 to 1.16 and one of my tests fail with:
Uncaught TypeError: Cannot set property 'innerHTML' of null at qunit.js:2638
At line 2638 in qunit.js I see:
if ( tests ) {
  id( "qunit-testresult" ).innerHTML = html;
}

Until now I've used markup template like:

Wicket Ajax tests

Looking at http://qunitjs.com/ I see that now the template should be just:

This fixed my test !
Problem 1) I didn't see a HEADS UP about this change in a minor release
Problem 2) Now the header says "All tests". Is there a way to use "Wicket Ajax tests" as before ?
Problem 3) The qunit-header is no more a link. qunit-banner contains a link but it is not clickable because it doesn't have text child element:

Problem 3) The qunit-header is no more a link. qunit-banner contains a link but it is not clickable because it doesn't have text child element:

Turns out this was actually broken in 1.15.0 as well, so I'm guessing you actually upgraded from 1.14.0 to 1.16.0...?

PR #716 submitted for review.

Turns out this was actually broken in 1.15.0 as well, so I'm guessing you actually upgraded from 1.14.0 to 1.16.0...?

No, I upgraded from 1.15.0 to 1.16.0, but I haven't noticed before.

Ah, gotcha.


In MSSQL, own schemas not seen, only default ones

I created the following adapter:
CREATE OR REPLACE JAVA ADAPTER SCRIPT adapter.jdbc_adapter_mssql AS
  // This is the class implementing the callback method of the adapter script
  %scriptclass com.exasol.adapter.jdbc.JdbcAdapter;

  // This will add the adapter jar to the classpath so that it can be used inside the adapter script
  // Replace the names of the bucketfs and the bucket with the ones you used.
  %jar /buckets/bucketfs1/bucket_adapter/virtualschema-jdbc-adapter-dist-1.6.0.jar;

  // You have to add all files of the data source jdbc driver here (e.g. Hive JDBC driver files)
  %jar /buckets/bucketfs1/bucket_adapter/mssql-jdbc-7.2.1.jre8.jar;
/

Then I created a connection using SCHEMA_NAME = 'dbo'. It works fine.
However, when I set SCHEMA_NAME = 'export', I get to following error message:
SQL Error [22002]: VM error:
com.exasol.adapter.AdapterException: Schema export does not exist. Available schemas: db_accessadmin, db_backupoperator, db_datareader, db_datawriter, db_ddladmin, db_denydatareader, db_denydatawriter, db_owner, db_securityadmin, dbo, guest, INFORMATION_SCHEMA, sys
Stack trace:
com.exasol.adapter.jdbc.JdbcMetadataReader.findSchema(JdbcMetadataReader.java:221)
com.exasol.adapter.jdbc.JdbcMetadataReader.readRemoteMetadata(JdbcMetadataReader.java:46)
com.exasol.adapter.jdbc.JdbcAdapter.readMetadata(JdbcAdapter.java:135)
com.exasol.adapter.jdbc.JdbcAdapter.readMetadata(JdbcAdapter.java:125)
com.exasol.adapter.jdbc.JdbcAdapter.handleCreateVirtualSchema(JdbcAdapter.java:118)
com.exasol.adapter.jdbc.JdbcAdapter.adapterCall(JdbcAdapter.java:44)

It looks like the adapter can only access the default MSSQL schemas like dbo, guest, sys, db_..., but not other created schemas.

The available schemas are retrieved with DatabaseMetaData.getSchemas() this should work out of the box. Maybe it is a matter of metadata visibility. Have you tried selecting sys.schemas with that same user you are using in the virtual schema? Maybe some GRANTs are missing.

Privileges in MSSQL are OK because I can access the schemas and their tables with IMPORT using the same connection object (therefore the same user).
That's why I suppose that it is a Virtual-Schema-specific issue.

I also have the impression that there is a datatype issue. A simple SELECT * FROM ... gives this error:
Query execution failed

Reason:
SQL Error [42636]: ETL-3003: [Column=229 Row=16] [String data right truncation. String length exceeds limit of 21 characters] (Session:<PHONE_NUMBER>131976898)

Could it depends on some issues with the character set recognition?
The DDL I get from MSSQL with Dbeaver has lines like this:
col0 nvarchar(40) COLLATE Latin1_General_CI_AS
The DDL I get from the table in the virtual scheme for the same column has this:
col0 VARCHAR(20) UTF8
IMPORT in a new physical table created by myself has no issues.
So it should be really an issue within the adapter.

The available schemas are retrieved with DatabaseMetaData.getSchemas() this should work out of the box. Maybe it is a matter of metadata visibility. Have you tried selecting sys.schemas with that same user you are using in the virtual schema? Maybe some GRANTs are missing.

With SELECT * FROM sys.schemas; I don't get the wished shemas. So you're right, I have to ask for metadata visibility.

So since this is a permission problem in the MS SQLServer you are using, I will close this ticket.


add original_request tag

add original_request tag

Pull Request Test Coverage Report for Build 57

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+35.06%) to 87.234%

Totals

Change from base Build 55:
35.06%

Covered Lines:
41

Relevant Lines:
47

💛  - Coveralls


Exception on genesis failure

For issue #34

Pull Request Test Coverage Report for Build 297

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.035%

Totals

Change from base Build 289:
0.0%

Covered Lines:
134

Relevant Lines:
141

💛  - Coveralls


fix: show correct SAML Frontend URL on edit

Changes
Ensures that the current stored Frontend URL is shown when editing, instead of using the current URL.
Also includes a bonus fix for a typo.
How did you test this code?
Ran the FE against production and tested create / edit.

Let's convet this to done


Obscured text long-press menu controls

Internal: b/140830033
As per #24100, long pressing obscured text fields shows menu controls that allow copying it, which makes it indirectly readable.
Because of the code freeze, #24100 focuses on temporarily disabling menu controls by default for obscured text fields, but for it to match the current experience, it should still appear but with the controls that can be used to access the data filtered out. For example, on Android, you may find the controls Select All, Paste, and Auto Fill there, because those ones do not leak the data already inside the field.
This is a request to implement such filters to the long-press menu controls based on the obscurity of the controlling text field.

This would be awesome! We need it :)

b/140830033

This appears fixed to me.  I can't copy from an obscured text field.  I think this was deliberately fixed in a PR awhile back, but I can't find the specific PR.
Anyway, if anyone is still able to copy text out of an obscured field, let me know and we can reopen this.


Actually remove event listeners in KeyControlMethod.destroy()

KeyControlMethod.destroy() calls addEventListener(), which is a peculiar course of action for a destructor, and perhaps due to a copy-and-paste mishap.
This is (mostly) of no consequence, but does lead to errors being logged when the window loses focus after the Marzipano instance has been destroyed, as the window.onblur handler for each of that instance’s KeyControlMethods remains in place.
This PR thus corrects three calls to addEventListener() in KeyControlMethod.destroy() to removeEventListener().

CLA submitted.

Thanks for the fix!


Define format to describe multiple tracers on Animals and Samples Sheet

FEATURE DESCRIPTION
Feature Inspiration
Experiments sometimes use multiple tracer compounds per animal. The Animal and Sample data sheet needs to be able to accomodate this information. Right now, researchers use various forms of shorthand (see #389)  to encode this, but the format is not well defined and can be ambiguous. For example, when specifying labeling for a 6 amino acid tracer combination, labeled with both Carbon and Nitrogen, the researcher simply specified six numbers (e.g. <IP_ADDRESS>.3.5). It's not clear which is associated with which amino acid nor which element(s) are labeled in each amino acid.
Feature Description

Describe a format that is expressive enough to capture multiple tracers and multiple labeled elements per tracer

Current example - check the AnimalMultiTracersMultiElements sheet within the book, but dual label (C, N) of 3 amino acids was paired with only 3 labeled ion count (Nitrogen perhaps all assumed to be 1)
Proposal
Tracer
There with be a single Tracer Infusate column that encodes/replaces the existing template Tracer Compound,Tracer Labeled Element &Tracer Label Atom Count. Infusion Rate remains unchanged.

Individual tracer compounds will be formatted: compound_name-[weight element count,weight element count]
examples:

valine-[13C5,15N1]
leucine-[13C6,15N1]
isoleucine-[13C6,15N1]

If a tracer is not fully labeled for a given element, the positions of the that element must be included: compound_name-[position,position-weight element count]
example:

L-Leucine-[1,2-13C2]

Mixtures of compounds will be formatted: tracer_mix_name {tracer; tracer}. tracer_mix_name is optional
Examples:

BCAAs {isoleucine-[13C6,15N1];leucine-[13C6,15N1];valine-[13C5,15N1]}
isoleucine-[13C6,15N1];leucine-[13C6,15N1];valine-[13C5,15N1]

Tracer Concentration
The Tracer Concentration Tracer Concentrations column remains (pluralized), and if it is a complex mixture, it contains a semi-colon delimited list of floats, in the same order and cardinality as the list as semi-colon delimited tracer encodings in the  proposed Tracer column.
Alternatives Considered
See questions and comments, below...
Comment
Before finalizing the format, it must be reviewed with researchers to ensure they find it workable.
NOTE: Updated template is here (will eventually replace old template) - https://docs.google.com/spreadsheets/d/1ywj9BvwX76ygRc5iXHwmzBcg1CWc5eFjLGOC6NtSHo8/edit?usp=sharing

ISSUE OWNER SECTION
Assumptions

regex pattern the bracket-encoded tracer encoding for approx. (13C)(\d+) and (15N)(\d+) for tracer element and count.
store intact/encoded individual tracer in tracer.name
store alphabetically sorted encoded tracer list in tracer_group.name
store optional mix name ("6EAAs") in tracer_group.preferred_name
lists of tracer encodings and concentration are semi-colon delimited
the compound name in a tracer encoding must exist in either compound table or compound_synonym

Requirements

Cocktails of isotope labeled tracers (multiple compounds)
Multiple isotopes per tracer (13C, 15N)

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected/Changed Components

Sample Animal template, potentially
implies changes to parsing and loading samples/animals

DESIGN
GUI Change description
Describe changes the user will see.
Code Change Description (Pseudocode optional)
Describe code changes planned for the feature.
Tests
A test should be planned for each requirement (above), where possible.

Here is the current proposal from the lab: https://github.com/Princeton-LSI-ResearchComputing/tracebase/pull/389#discussion_r819952659:

Tracer group specification format:

tracer compound = BCAAs {valine, leucine, isoleucine}
tracer labeled element = C, C, C (each tracer compound has carbon labeled)
tracer labeled element count = 5, 6, 6 (valine has 5, leucine 6, isoleucine 6)
tracer concentration = 80, 100, 60 (each tracer compound has its own concentration

Are any infused tracers not 100% labeled for the targeted element(s)?

Yes, sometimes only a subset of atoms in the tracer are labeled.  Whole-body PPP is an example where only two of the carbons in glucose are labeled.

Are there any existing naming conventions that could be used to communicate the isotope element and counts?

None are reliably adhered to in the lab right now, but I think the best option is L-Glutamine-[13C5,15N2] because this is like the format used by most suppliers of the tracer.  See example from cambridge isotopes

Given the above, would need to be able to communicate a list of tracers

Yes and I can see how this would get a little complicated.  For example:  BCAAs {valine-[13C5,15N1], leucine-[13C6,15N1], isoleucine-[13C6,15N1]} (perhaps using semicolon instead of comma).
An additional problem with the named tracer list is that BCAAs could mean various combinations of labeled elements.  We would have to keep track on the tracebase side or come up with a standardized way to track this, or perhaps we can ignore the user supplied name of the tracer group and instead derive our own?

@lparsons @hepcat72 @fkang-pu @mneinast One thing to be considered (since this template proposal predated the model proposal discussed this week, including Infusate) is whether Tracer and Tracer Concentration should remain labeled so [and everyone knows what is truly inside those columns and how they are mapped], or whether they might be better labeled Infusate and Infusate Concentrations (or similar).

I think that part of the effort to allow multiple tracers correctly separates Tracer from being a 1:1 with infusate to a M:1, which means that "infusate" as a term now represents the mixture.  And each tracer has its own concentration in that solution.  I.e., we're not doing multiple infusions into the mouse.  We're doing 1 infusion and the infusate includes N tracers, each at a specific concentration.  I in fact think it would be confusion to apply the term infusate to each tracer and each tracer's concentrations, because that, to me, does not truly represent the relationships.  But I think it's good that we consider potential user confusion, so I would like to know if the researchers apply the term "infusate" to the mixture or each individual tracer. If there are differing opinions, we would definitely need something to address it.

I think that part of the effort to allow multiple tracers correctly separates Tracer from being a 1:1 with infusate to a M:1, which means that "infusate" as a term now represents the mixture. And each tracer has its own concentration in that solution.

Exactly, the software (TraceBase) is now making a clear distinction in how it uses those terms. It's probably good to be as consistent about those as we can, esp. when we are communicating ideas. However, it's important that we understand that researchers may not be as precise with the language (and may not even define things similarly), so when we intake data, we should be as forgiving and relaxed as is reasonable. That said, I think making the default column headers Infusate and Infusate Concentrations makes a lot of sense. We have the flexibility to map those to different headings if the researcher uses other terms. As long as the data is in the proper format, we can interpret the headings.

Zoom discussion with @lparsons  and @hepcat72  decided the the columns should be Infusate  and Tracer Concentrations [rather than the original proposal Tracer and Tracer Concentration ].  This final decision was made to indicate the concept of multiple tracers (and corresponding concentrations) in the infusion process/cocktail.

Wait, I thought "Infusate" was to be for "Tracer Group".  I wouldn't think that the "Tracer" column should be named "Infusate". But like I said, I'm not married to my suggestion. I just think we shouldn't use infusate differently in different contexts. @lparsons @jcmatese

This has been decided now, and the issue for implementation is #407.

In our last meeting on this, remember a discussion on using commas to delimit 2 different kinds of data.  I'm guessing that it was the position delimiting and the label-delimiting, but since the given examples in the proposals above don't include that case, I thought I'd double-check that it wasn't in regard to the tracer-delimiting (which the format shows using semicolons).
So, to extend one of the examples, it this correct?:
BCAAs {isoleucine-[1,2-13C6,15N1];leucine-[13C6,15N1];valine-[3,4,5-13C5,15N1]}
@jcmatese

So, to extend one of the examples, it this correct?:
BCAAs {isoleucine-[1,2-13C6,15N1];leucine-[13C6,15N1];valine-[3,4,5-13C5,15N1]}

I thought that would be
BCAAs {isoleucine-[1,2-13C2,15N1];leucine-[13C6,15N1];valine-[3,4,5-13C3,15N1]}
i.e. if the ArrayField is 1,2 (length==2) then it should be 13C2 or if 3,4,5 then 13C3

I thought we decided in the last meeting about this that positions were optional - that you didn't have to have all positions. Didn't Michael chime in on that in that in-person meeting?
But I was asking about the semicolon delimiter. I thought you wanted it to be a comma based on a chemical db site's standard.

No, I would prefer it to be left as semi-colon, because semi-colons are NOT generally used in chemical names, where as commas are fairly commonplace.  Unless someone has a better delimiter that the lab members prefer...
I don't feel too strongly about the isotope-count postscript. [if absent, assume fully labeled based on formula, else respect the integerArrayField count ].  Whatever the lab members want, because although it is explicit, it is kind of redundant, and I would not claim it necessarily is widespread/common, though I have seen it for fully labeled molecules
Examples
L-Leucine-13C6,15N - postscript on carbon, but not the nitrogen (1 is assumed, by convention).  Synonym lacks postscripts
L-Alanine-13C3,15N- postscript on carbon, but not the nitrogen  (1 is assumed, by convention)
Ideally, we wold try to closely approximate the nomenclature from whatever catalog they are ordering from, but I never saw that spreadsheet or a suggestion of a preferred vendor.

Since I'm working on #430, I'm going to assume that the discrepancy between the description of the concentrations column in the revised template:

[...] Multiple tracers in a single infusion should have concentrations specified as comma separated lists.

...and the description in the issue:

Tracer Concentration
The [...] Tracer Concentrations column [...] contains a semi-colon delimited list of floats, in the same order and cardinality as the list as semi-colon delimited tracer encodings in the  proposed Tracer column.

Not sure which is the more recent update to that column, but I will go with semicolon.  Thus, I will modify the description of that column in the template.


Bug report context/kube-config support

Please provide a description of this PR:
Currently istioctl bug-report has an issue where certain commands do not honor the --kubeconfig and --context flags. This PR adds support for them for the missing commands.
https://github.com/istio/istio/issues/35574

/retest


Create and build a proper crate for the SDK

It should work seamlessly for both the native and riscv targets, and be easy to compile.
The C SDK could possibly be removed completely, as linking with cxlib/speculos binaries turns out to be a huge headache; for native compilation, the implementation of most of the bolos calls could be emulated with native Rust libraries, instead.

Hi, I fully agree, and have reached the same conclusion while digging in the repo architecture 🤯
Could you please assign me this issue ? 🙏


Fixed video duration when avVideoStream->nb_frames > 0.

When avVideoStream->nb_frames > 0, it overrides the stream duration.   See for example this movie:
https://mega.nz/file/3HgDUApI#eIzVWfblyQxeG8eVTxR5Yy_BE5USHskGbRSXKCgRW7Y
The streams are 2000 or so frames long, but the movie is shortened to 1428 frames (the value of avVideoStream->nb_frames).

One thing, thou.  Seeking is broken in tlRender in that movie, even with this patch.

Does this also apply to audio streams?

Yes, I believe it does as ffprobe reports it for any stream.  At least that was what mrViewer did.
However, I am having second doubts about this PR.  I verified it and found ffplay does not use nb_frames anywhere in its code.  ffplay (which seeks this movie file correctly) seems to always rely on the duration set in AVFormatContext for its seeks and when the duration is not set (ie. when it is 0), it relies on seeking by bytes.

OK.  The movie is indeed 2818 frames long, but under a timebase of 60 not an fps of 30.  What's wrong is the speed (30.30fps) vs. timebase (60) that it is encoded.  Here's the output of av_dump_format() in ffplay:
Stream #0:10x2: Video: h264 (Main) (avc1 / 0x31637661), yuv420p(tv, bt709, progressive), 404x720 [SAR 1:1 DAR 101:180], 7299 kb/s, 30.30 fps, 60 tbr, 600 tbn (default)
Here's a summary of the meanings of fps, tbr, tbn:
https://superuser.com/questions/1362410/what-is-fps-tbr-tbn-tbc-in-ffmpeg/1759811#1759811
tlRender is incorrectly choosing the tbr as the fps.  In code, this would be:
            const double tbr = av_q2d(avVideoStream->r_frame_rate);
            const double fps = av_q2d(avVideoStream->avg_frame_rate);

It is unclear to me how we can get the timebase be 60 while the fps is 30 currently.  Setting speed (fps) to avg_frame_rate is the probably the best compromise for now.

Does that movie file have a variable frame rate? That's not something I had been planning on supporting.

Yes, I believe you are right.  It has a VFR, as it has duplicated frames, which showed when I converted the movie to another format.


struct field name generation rules

This looked interesting, so I took it for a spin. This is for v0.2.1
Just from the behavior, it seems like

long names are required (edit: I now see in the docs that it's required), and
if a name is specified (not the short or long flag name) it doesn't have any effect on the struct field name.

So, for example, at https://github.com/matklad/xflags/blob/092d70de0777cee9502f3a062a2025258d52313b/examples/longer.rs#L34
The field name of the generated struct is from the long-flag-name only https://github.com/matklad/xflags/blob/092d70de0777cee9502f3a062a2025258d52313b/examples/longer.rs#L71
I was expecting that the struct name could any of name, long flag, short flag (in that order).
Let me know what you expect (and if I just missed something). I took a quick peek at the code, it looks like it'd fun to submit a PR

For --switches, --long name determins the name of struct's field, and (optional) value-name only affects the generated help
For positional arguments, (mandatory) value-name affects both the help and the fields name.
In theory, in-place generation should answer such questions, but claryfing PR to the docs would be welcome!

Ok, thanks for the clarification. I'll go back and take a look at the docs again.


Clicking outside combobox selects the last focused item

🐛 Bug Report
Due to a recent change in @react-stately/combobox, combobox calls onSelectionChange when clicking outside.
NOTE: This is a different issue than #2181.
🤔 Expected Behavior
Clicking outside should not select last focused option.
😯 Current Behavior
Depending on the implementation of the combobox's popover, clicking outside may select a value.
In react-spectrum's implementation, it doesn't happen, because of how Popover is implemented. But here is an example copied from react-aria docs: https://codesandbox.io/s/react-stately-combobox-issue-v54cui?file=/src/App.js
The only difference with the example in the documentation is that the Popover is made modal (by not passing isNonModal to Popover).
💁 Context and Possible Solution
This seems to be a regression caused 168ca5a. Previous to this change close function returned from useComboboxState was coming from triggerState, and with this change, it's overwritten to commit:
https://github.com/adobe/react-spectrum/blob/73d8be4dd7ff1713084cfe3f602f11aef1cecb07/packages/%40react-stately/combobox/src/useComboBoxState.ts#L304-L308
I'd guess the intention of this change was to not just close the menu, but to make sure the necessary cleanup is done as well, whenever the combobox menu is closed for whatever reason (such as outside click). But in that case it should be commitSelection or commitCustomValue being called instead of commit.
💻 Code Sample
https://react-spectrum.adobe.com/react-aria/usePopover.html#example
🌍 Your Environment

Software
Version(s)

react-aria
3.22.0

react-stately
3.20.0 (no issue with 3.18.0)

browser
Chrome

Operating System
MacOS

Curious that the setting for isNonModal changes the behavior, still digging why. That option is necessary for the ComboBox so that the input isn't aria hidden when the popover is open

Mind expanding on why you are removing isNonModal from your implementation of ComboBox?

Curious that the setting for isNonModal changes the behavior, still digging why

I've traced it through:
When isNonModal is false, in  useOverlay is called with isDismissable: false:
https://github.com/adobe/react-spectrum/blob/dc035b2f5878c6883d92567e089209d105d8c04b/packages/%40react-aria/overlays/src/usePopover.ts#L81
When isDismissable is false, useInteractOutside is called with onInteractOutside set to null.
https://github.com/adobe/react-spectrum/blob/dc035b2f5878c6883d92567e089209d105d8c04b/packages/%40react-aria/overlays/src/useOverlay.ts#L123
That in turn means in useInteractOutside, the pointerDown handler on document won't call onInteractStart callback passed from useOverlay:
https://github.com/adobe/react-spectrum/blob/dc035b2f5878c6883d92567e089209d105d8c04b/packages/%40react-aria/interactions/src/useInteractOutside.ts#L49-L56
And that's important because in onInteractStart callback in useOverlay, preventDefault and stopPropagation is called on the pointer down event:
https://github.com/adobe/react-spectrum/blob/dc035b2f5878c6883d92567e089209d105d8c04b/packages/%40react-aria/overlays/src/useOverlay.ts#L94-L101
Which means the onBlur handler in useCombobox won't get called:
https://github.com/adobe/react-spectrum/blob/dc035b2f5878c6883d92567e089209d105d8c04b/packages/%40react-aria/combobox/src/useComboBox.ts#L144
Which otherwise would call setFocused In useComboboxState, with false:
https://github.com/adobe/react-spectrum/blob/dc035b2f5878c6883d92567e089209d105d8c04b/packages/%40react-stately/combobox/src/useComboBoxState.ts#L287-L302
And that would call commitSelection, which clears the input and closes the menu.

Thank you for the detailed tracing! From a quick pass of the logic, I believe this change makes sense. It would also mean that we then wouldn't commit the focused value when using the visually hidden dismiss buttons, but that feels like the correct behavior.

Yeah, good point with the visually hidden dismiss button. I guess I can open a PR tomorrow.


Use new version of provisio plugin

This includes a new version of the Takari Archiver which supports
hardlinks in archives.

The RPM, using the tarball with hardlinks, is also reduced in size.

The user experience is that the tarball, RPM, and on disk size are reduced in size (hardlinks are preserved when unpacking).


Issue#181 - Add shiny forms

Hi! I recently got into Pokemon and found your website super helpful in learning all the types. I've also been trying to learn Typescript so I wanted to help out. Anyways, this PR is looking to solve #181 and the way it works is that you just click on the sprite and it will toggle between shiny/normal. For Pokemon without shiny sprites, I believe that it will just display a red circle.
I also updated the json files and uploaded the images into the public folder. I'm not completely sure if this is the right way but I'm guessing you ran these scripts in this order:

npm run scrape-pokeapi
Running scrape-gen9.js on console
npm run merge-data
npm run fetch-images

Let me know if you see anything that needs to be changed and feel free to do whatever with it.
e.g. after clicking on the picture...

Hmm yeah, that's a good idea! I can try to make that change and a clickable link should be ok.
I was originally thinking of the icon button in the corner using MUI but I got kind of stuck on the implementation.

Added the toggle link to bottom right!

Feel free to merge it whenever you think it's good! Thanks again and thank you for making such a great website!

I noticed an issue with small viewports (phone size), so I'm gonna grab this and try a new layout for the links section. I'll add you on the PR I make :)
Just want to ask again, do you want to be in the credits? I'd love to add you, for your help.


Alburgess/fix comments

What type of PR is this?

What this PR does / why we need it: Cleaning up lint errors. Comments and trailing white spaces fixed here
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

34 of 36 (94.44%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 79.334%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/agent/utils.go
13
15
86.67%

Files with Coverage Reduction
New Missed Lines
%

pkg/agent/datamodel/sig_config.go
1
59.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2000

Relevant Lines:
2521

💛  - Coveralls

lots of fixes, thanks! :)
mind adjusting the title and rebasing/squashing commits with a message following this format (it's why the lint check currently fails)? https://www.conventionalcommits.org/en/v1.0.0/
lgtm otherwise :)

not sure if you did these manually or with CLI, golangci-lint run --fix or similar may help with some of the formatting errors.
if there are rules that make sense to adjust, feel free as well -- I just took a basic sample from google that passed eyeball inspection

alexeldeib
I am making the changes pretty manually. I am using the golangci-lint run so that I don't have to run my azure actions over and over, but I am tackling the whole output list.

(I've been working on 111 next and I just run golangci-lint run mocks.go ---disable-all -E lll which is pretty useful!)


[HANDS-ON BUG] Runtime Error when accessing leaderboard on completion of hands-on

I have attempted the Lunar Lander several times and tried to check if it was accepted on the leaderboard for certification later on. I am having this run time error as shown in the picuture since I started on the first of November.

Thank you

I noticed it's the same as issue #441 so will probably be the same fix needed

Can confirm it's crashed for me too.


bugfix: 插件配置版本匹配不符合预期的问题 (fixed #357)

Codecov Report

Merging #361 (ea18944) into V2.1.360-rc (4268dd0) will decrease coverage by 0.00%.
The diff coverage is 70.00%.

@@               Coverage Diff               @@
##           V2.1.360-rc     #361      +/-   ##
===============================================
- Coverage        62.75%   62.75%   -0.01%
===============================================
  Files              246      246
  Lines            19957    19963       +6
===============================================
+ Hits             12524    12527       +3
- Misses            7433     7436       +3

Impacted Files
Coverage Δ

apps/backend/plugin/views.py
34.66% <0.00%> (ø)

apps/backend/plugin/tools.py
92.92% <77.77%> (-0.77%)
:arrow_down:

apps/node_man/handlers/job.py
86.07% <0.00%> (-0.26%)
:arrow_down:

Continue to review full report at Codecov.

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


Tracking oauth2 server support

Just a bug tracking oauth2 server support. Not sure if it is in scope.

Integrating with oauth is something I'd like to account for on this lib, but only after getting some architectural design issues and basic interactions worked out. I'll leave this issue open as reminder :+1:

My plan of progress is to use the examples at https://github.com/kivra/oauth2 as a base.
I like how their sense of security is based on Solaris's RBAC.


Adding replication support for dedicated clusters

@guywi-ms
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Bugfix formaction dict class update

Description
The actions property of a View can contain a dictionary with a formaction. In Python, if you set a dictionary as an attribute to a class, and you update its value, the class dict is updated, not only the instance itself. Changed the code to no longer update the actions directly.
Type of change

Bug fix (non-breaking change which fixes an issue)

Coverage remained the same at 73.475% when pulling 80782dbf863426facb242310a34d2e6bd0c5b229 on bugfix/formaction_dict_class_update into 62a2bda8b17161be03ac370289da29d5f9993e1f on develop.


Templates: Update Cooking template to 9:16

Use this Figma doc to apply 9:16 updates to the existing Cooking template:
https://www.figma.com/file/RTzSw47oi6pEZXEfP2JoBo/Templates?node-id=766%3A61

Image missing:

Okay, I re-generated the template json and double checked that all the images are properly rigged and are part of the repo soooo hopefully they should all show up:

One thing to note, if you've created a story from a template by clicking on "Use Template", that story will not be updated with updates to the source template.  To view the latest versions of the templates be sure to go to "Explore Templates".

What just came to mind:
Because of file size, we currently load all template assets from our GitHub pages „CDN“. That only uses what‘s in master though...
So if a PR adds new template images, those can‘t be seen on the QA site because they‘re not on master.
I initially thought of this as a rather rare case, but it seems like I should expedite work on this.
I‘ll tackle this ASAP.

What just came to mind:
Because of file size, we currently load all template assets from our GitHub pages „CDN“. That only uses what‘s in master though...
So if a PR adds new template images, those can‘t be seen on the QA site because they‘re not on master.
I initially thought of this as a rather rare case, but it seems like I should expedite work on this.
I‘ll tackle this ASAP.

That's totally what's happening, nice catch!

image not appearing:

Probably because of https://github.com/google/web-stories-wp/issues/2472#issuecomment-650440917
If the template is otherwise fine, and works locally, I would suggest marking this as QA-passed.


Radius vs server radius

I was wondering why the radius for the app is a lot smaller than the pokemon radius I am seeing on my server.
Is there any way to increase the radius so I can see more pokemons on my map?

pinch zoom? It zooms like any other iOS app.

@MysticalOS This does not matter here, it's not about seeing a large portion of the map. It's about getting to see more pokemon in a large area of the map. I know how to zoom...

@stephan-v I added this functionality for pogom server


stats: hidding a chart from blocksout/stats with ENV var

Is it possible to hide one of the charts with an environment variable? For example: New transactions number chart.

If I set
STATS_CHARTS__LINE_CHARTS__AVERAGE_TXN_FEE__ENABLED: "false"

which when enabled returns:
2024-09-02T12:03:50.789844Z ERROR stats_server: service failed with error: unknown field `line_charts`, expected one of `counters`, `lines`, `template_values`
Error: unknown field `line_charts`, expected one of `counters`, `lines`, `template_values`

and from reading the code, this should be working??? charts.rs lines 279
      (
                "STATS_CHARTS__LINE_CHARTS__AVERAGE_TXN_FEE__ENABLED",
                "false",
            ),
        ]
        .map(|(s1, s2)| (s1.to_owned(), s2.to_owned()))
        .into();

        let expected_counter = ChartSettingsOverwrite {
            enabled: Some(true),

and then as per logs, I could change it to:
STATS_CHARTS__LINES__AVERAGE_TXN_FEE__ENABLED: false

2024-09-02T12:12:54.999829Z ERROR stats_server: service failed with error: unknown field `enabled`, expected one of `title`, `order`, `charts`
Error: unknown field `enabled`, expected one of `title`, `order`, `charts`

Hi, I see what's the problem.

and from reading the code, this should be working

The code in current main is an updated version (soon we'll release v2.0.0, it'll be the same). However, you seem to run the latest release (v1.6.1) that has a somewhat different structure of envs. So the code I linked should be relevant for the release.
Now, regarding the STATS_CHARTS__LINES__AVERAGE_TXN_FEE__ENABLED.
In that version the charts were placed inside categories (you can see it in the config). So to control settings for the chart you have to do:
STATS_CHARTS__LINES__TRANSACTIONS__CHARTS__AVERAGE_TXN_FEE__ENABLED: false
Hope it helps 😃

Ok, let me try.
Do you think it makes sense to document this better?

Do you think it makes sense to document this better?

documentation for envs is coincidentally in development exactly now 🙃

Does that mean you can't configure New transactions chart?

Why not? It's just going to be NEW_TXNS instead of AVERAGE_TXN_FEE

Oh I thought since they don't have enabled field that it wouldn't be possible. But I guess this property is optional.
Nevertheless, I'm running 1.6.1 and I've set:
STATS_CHARTS__LINES__TRANSACTIONS__CHARTS__AVERAGE_TXN_FEE__ENABLED: false

2024-09-02T13:19:44.004188Z ERROR stats_server: service failed with error: unknown field `enabled`, expected one of `title`, `order`, `charts`
Error: unknown field `enabled`, expected one of `title`, `order`, `charts`

Does 1.6.1 keep the structure for schedules?
  STATS_UPDATE_GROUPS__SCHEDULES__TOTAL_BLOCKS: "0 0 */1 * * * *"
  STATS_UPDATE_GROUPS__SCHEDULES__ACTIVE_ACCOUNTS: "0 2 */1 * * * *"
  STATS_UPDATE_GROUPS__SCHEDULES__COMPLETED_TXNS: "0 5 */1 * * * *"
  STATS_UPDATE_GROUPS__SCHEDULES__TOTAL_ADDRESSES: "0 7 */1 * * * *"
  STATS_UPDATE_GROUPS__SCHEDULES__NEW_TXNS: "0 10 */1 * * * *"
  STATS_UPDATE_GROUPS__SCHEDULES__NEW_CONTRACTS: "0 15 */1 * * * *"
  STATS_UPDATE_GROUPS__SCHEDULES__NEW_VERIFIED_CONTRACTS: "0 30 */1 * * * *"

and STATS_CHARTS__TEMPLATE_VALUES__NATIVE_COIN_SYMBOL?

Alright, I think my config map was somehow stale and I got the wrong error logs.
STATS_CHARTS__LINES__TRANSACTIONS__CHARTS__AVERAGE_TXN_FEE__ENABLED

STATS_CHARTS__LINES__TRANSACTIONS__CHARTS__NEW_TXNS__ENABLED

Is working for 1.6.1!

you can check the changes here for now


Better Discord Wont Install Properly

Better Discord has not been installing properly. In the installer it will have its usual installment. Then it gets to the end of the installment, it gets stuck on downloading the .zip file. When I go to the folder location the .zip file is there but its empty. So I closed the installer and tried to open Discord. That's when i got this error
https://imgur.com/GVIWpH3 . If someone could please help me with this. I have tried all sorts of different ways to install it. I don't know if this is important or not but my labtop is running windows 8 (yes i know, its really outdated).

Choose BandagedBD (if stuck then move installer to desktop and run as admin) https://github.com/rauenzi/BetterDiscordApp/releases

Thank you, its working perfectly now!

not working in my PC

Mine's stuck on "attempting to kill discord"


Add searchable list of all CLA signers

Also included is a nasty script to sort & date list as new names are
added.
Signed-off-by: Emily Shaffer<EMAIL_ADDRESS>

Can you squash the changes down to one commit and add your SoB tag? Looks okay otherwise. @bradbishop are you okay with this explicit public record of those under the CLA?
Just to clarify I'm not after squashed series in general, but in this case I don't think the changes need to be in separate commits.

@bradbishop are you okay with this explicit public record of those under the CLA

I'm not able to come up with any reasons to hide this.

I'm not able to come up with any reasons to hide this.
@krtaylor any opinions here?

I'm not able to come up with any reasons to hide this.

The only thing that gave me pause was that the PDFs weren't publicly accessible, but i don't think it's worth hiding either. The list can be approximated by identifying people who have their patches merged.

The entire list of members that have signed the CLA is usually available to others that have registered, at least with other projects I've been involved with. But since it is required, it is true that everyone going forward that has merged a patch has signed the CLA, as was said. Email is already easily obtained from gerrit so I'd say there is no reason to restrict this info.


MSID: 96789 Version: 1 DOI: 10.1101/2<IP_ADDRESS>0325

MSID: 96789
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>0325
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=96789
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0325

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/96789v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>0325
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/96789/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Emails need fixing:


Commands And BlackScrolls

Hello, I currently have the latest version and this is happening.  When I try to remove an enchantment from the item with a black scroll, it just acts as if you were clicking any other item onto the sword, just replaces the sword's spot.  Also when I try to do /ce add and /ce remove, it says "An internal error occured while attempting to perform this command".  Please Help, Thanks!  Here is my config:  http://pastebin.com/0n3jms3y

You probably didn't update your enchantment.yml

@KingRaptor345 Yes what @Yukun99 said.
If you want to test if this is true backup your Enchantments.yml file and then delete it and reload. Then try and add a enchantment. If it works will problems then you didn't update the file. If you didn't edit the Enchantments.yml file then you can just use the new 1.


Expecto "Watch mode" gives "Seq has not enough elements"

Hi there,
Trying to fix a bug in Hopac, so I'm introducing Expecto to it and then selecting "Start watch mode":

Then, after this, calling the "Expecto: Run" command, gives this output:

    56 Warning(s)
    0 Error(s)

Time Elapsed 00:00:11.67
[09:13:38 INF] EXPECTO? Running tests...
[09:13:38 ERR] samples/should fail failed in 00:00:00.0080000.
I should fail because the subject is false.. Actual value was false but had expected it to be true.

[09:13:38 INF] EXPECTO! 2 tests run in 00:00:00.0694001 – 1 passed, 0 ignored, 1 failed, 0 errored. ( ರ Ĺ̯ ರೃ )

Unhandled Exception:
Mono.CompilerServices.SymbolWriter.MonoSymbolFileException: Symbol file `/Users/h/dev/logibit/Hopac/Tests/Hopac.Tests/bin/Debug/Hopac.Tests.exe.mdb' does not match assembly
  at Microsoft.FSharp.Control.AsyncBuilderImpl.commit[a] (Microsoft.FSharp.Control.AsyncBuilderImpl+Result`1[T] res) [0x00026] in <53ed7cac904cf4daa7450383ac7ced53>:0
  at Microsoft.FSharp.Control.CancellationTokenOps.RunSynchronously[a] (System.Threading.CancellationToken token, Microsoft.FSharp.Control.FSharpAsync`1[T] computation, Microsoft.FSharp.Core.FSharpOption`1[T] timeout) [0x000a3] in <53ed7cac904cf4daa7450383ac7ced53>:0
  at Microsoft.FSharp.Control.FSharpAsync.RunSynchronously[T] (Microsoft.FSharp.Control.FSharpAsync`1[T] computation, Microsoft.FSharp.Core.FSharpOption`1[T] timeout, Microsoft.FSharp.Core.FSharpOption`1[T] cancellationToken) [0x00076] in <53ed7cac904cf4daa7450383ac7ced53>:0
  at Expecto.Tests.runTestsWithArgs (Expecto.Impl+ExpectoConfig config, System.String[] args, Expecto.Test tests) [0x0017a] in <58d63d180c1a9511a7450383183dd658>:0
  at Expecto.Tests.runTestsInAssembly (Expecto.Impl+ExpectoConfig config, System.String[] args) [0x00088] in <58d63d180c1a9511a7450383183dd658>:0
  at Hopac.Tests.main (System.String[] argv) [0x00006] in <58d8bba1a534e9e7a7450383a1bbd858>:0
[ERROR] FATAL UNHANDLED EXCEPTION: Mono.CompilerServices.SymbolWriter.MonoSymbolFileException: Symbol file `/Users/h/dev/logibit/Hopac/Tests/Hopac.Tests/bin/Debug/Hopac.Tests.exe.mdb' does not match assembly
  at Microsoft.FSharp.Control.AsyncBuilderImpl.commit[a] (Microsoft.FSharp.Control.AsyncBuilderImpl+Result`1[T] res) [0x00026] in <53ed7cac904cf4daa7450383ac7ced53>:0
  at Microsoft.FSharp.Control.CancellationTokenOps.RunSynchronously[a] (System.Threading.CancellationToken token, Microsoft.FSharp.Control.FSharpAsync`1[T] computation, Microsoft.FSharp.Core.FSharpOption`1[T] timeout) [0x000a3] in <53ed7cac904cf4daa7450383ac7ced53>:0
  at Microsoft.FSharp.Control.FSharpAsync.RunSynchronously[T] (Microsoft.FSharp.Control.FSharpAsync`1[T] computation, Microsoft.FSharp.Core.FSharpOption`1[T] timeout, Microsoft.FSharp.Core.FSharpOption`1[T] cancellationToken) [0x00076] in <53ed7cac904cf4daa7450383ac7ced53>:0
  at Expecto.Tests.runTestsWithArgs (Expecto.Impl+ExpectoConfig config, System.String[] args, Expecto.Test tests) [0x0017a] in <58d63d180c1a9511a7450383183dd658>:0
  at Expecto.Tests.runTestsInAssembly (Expecto.Impl+ExpectoConfig config, System.String[] args) [0x00088] in <58d63d180c1a9511a7450383183dd658>:0
  at Hopac.Tests.main (System.String[] argv) [0x00006] in <58d8bba1a534e9e7a7450383a1bbd858>:0

Now, every time I focus/unfocus the editor I get the "Seq has not enough elements":

@jbevain – do you recognise Mono.CompilerServices.SymbolWriter.MonoSymbolFileException: Symbol file /Users/h/dev/logibit/Hopac/Tests/Hopac.Tests/bin/Debug/Hopac.Tests.exe.mdb' does not match assembly`? It's been freshly compiled.

There are 3 things here:

We should try to find underlying issue which makes Mono.Cecil crash
We should catch this in Expecto so it just gives empty paths instead of crashing test run on --summary-location
I should handle better things here in Ionide, so it doesn't go crazy when expecto fails for some reason.

Apparently Cecil is being asked to read an assembly and an associated mdb. the associated mdb has a different unique identifier from the assembly.
This could suggest that the assembly has been modified but not the mdb, or that the mdb is out of sync with the assembly.

In 2.27.4 I've added some additional error handling so it won't print out random error messages.


Unable to use this with Google-Maps-iOS-Utils

hi there, i seems to be having problem integrating PXGoogleDirections into my current project where Google-Maps-iOS-Utils is already installed
is there a workaround to this problem?

Hi @jhhomme, what do you mean by "having problem"? What are the error messages you're getting?

Hey @jhhomme, any update on this issue? Could you please give me more info  regarding the problem you're having? Thanks!


oracle: Object does not wrap anything with requested interface

Caused by: com.baomidou.mybatisplus.exceptions.MybatisPlusException: Error: GlobalConfigUtils setMetaData Fail !  Cause:java.sql.SQLException: 对象不使用请求的接口包装任何内容
when i attempt to monitor oracle, but i get the exception.
this is my maven config:
    `<dependency>
        <groupId>com.nepxion</groupId>
        <artifactId>discovery-plugin-strategy-starter-gateway-opentracing</artifactId>
        <version>${discovery.version}</version>
        <exclusions>
            <exclusion>
                <groupId>io.opentracing.contrib</groupId>
                <artifactId>opentracing-spring-cloud-jdbc-starter</artifactId>
            </exclusion>
        </exclusions>
    </dependency>`

if i exclusive jdbc-starter, i can run my project

I think I've created a fix for this in PR #275 . Give it a try when that is merged?

Relates to #210 which I think is solved https://github.com/opentracing-contrib/java-spring-cloud/issues/210#issuecomment-630991291


Paging events not firing as expected

Hello,
Not having any luck with the ldap paging against Active Directory.  I think I've fought through the paging/paged documentation mismatch (Issue 321 appears to have been closed too soon) but can't seem to get ldapjs to fire off all of the paging events, or handle ranged/paged results correctly.
Here's my code.  It works _perfectly_ when I have a group with 20 members.  But if I request a group with 5200 or so members (and it should start paging) I get a single searchEntry event called with a range of data, then one "page" event (with empty results), and then an "end" event.  Pages 1500-2999, 3000-4499, 4500-5200 do not appear to be coming in.
values = [];

var client = ldap.createClient({
    url: myServer
});

client.on('connect', function (){

    client.bind(myAccount, myPass, function(err) {

        var opts = {
          filter: ('(cn=' + myGroupName + ')'),
          scope: 'sub',
          attributes: ['dn', 'sn', 'cn','member'],
          paged: { pageSize: 250 }
        };

        client.search(myContainer, opts, function(err, ldapresponse) {

            ldapresponse.on('searchEntry', function(entry) {
                console.log("searchEntry event");

                if (Array.isArray(entry.object["member"])){
                    values = entry.object["member"]
                }
                else {
                    values.push(entry.object["member"]);
                }

                console.log("found: " + values.length + " members in group");

            });

            ldapresponse.on('page', function(entry) {
                console.log("page event");
                console.log('entry: ' + JSON.stringify(entry));
            });

            ldapresponse.on('end', function(result) {
                console.log('end event');
                res.setHeader('Content-Type', 'application/json')
                res.send(JSON.stringify(values));
            });

            ldapresponse.on('error', function(resErr) {
                console.log("error " + resErr);
            });
        });
    });

});

When I query a 5200 member group, I get a single searchEntry event with an object like this:
entry: {"dn":"CN=MyGroupName,DC=domain,DC=com","controls":[],"cn":"MyGroupName"
,"member":[],"member;range=0-1499":["USERDN1","USERDN2","USERDN3","USERDN4", ...... ]}
It's a ranged object so assuming there's some paging to be done...  But the next event to fire is the page event, which seems to have empty data, then the "end" event and it's all done.
Can someone point out what I'm doing wrong here?  I see that the activedirectory library, which leverages ldapjs, is able to handle paged results, but looks like it's doing this at a lower-layer and using a PagedResultControl rather than depending on these "page" events.
I can't find any examples in this repo or the interwebs that has working paging code, I'm assuming I'm doing something wrong, but I've tried "paging" and "paged" and it's just not working.
Thanks for having a look!

I'm losing hope as the sample ldapjs-search app doesn't appear to be able to handles paging either:
node .\ldapjs-search -b "dc=mydomain,dc=com" -s "sub" -D "CN=myaccount,DC=mydomain,DC=com" -w "mypass" -i -u "ldap://server.mydomain.com" -g 200 "(samaccountname=mygroup)" member
This too returns just the range 0-1499 and that's it.  This happens with or without the -g (paging) option.
So if the sample code can't seem to page, pretty sure I'm hosed.

Is this project still alive?  No commits since December 2015, and my report of a search result paging not functioning (even using the provided demo application) hasn't even gained a comment like "weird?!?" in 22 days.

@sinfloodmusic, Seems that the owners doesn't care about this project anymore.
About your Issue, take a look a the Issue #362. It seem you ran into the very same problem as me few months ago.

@Xotl, Thanks.  It appears that this project has been abandoned - which is a total bummer.
I did run into your issue, with the documentation not matching.  That burned at least an hour or two - scratching my head wondering what was going on.
I did eventually resolve that, and the issue I have now (paging) is even encountered by the example code they provide, so there appears to be an underlying paging bug when it comes to "range" returns.  Not insurmountable, I did find that the activedirectory library just uses some lower layer controls to get around it, but it's very hard to read.  I have a workaround in the meantime, when I get time I guess I'll just do what they are doing.

Any alternate for node-ldapjs ?

I haven't found one - but if you do find one please post here.  My workaround just shells out to powershell, runs the AD cmdlets which write to stdout, then my nodejs picks it up from there.  It's quite a bit slower, but at the moment I don't need the speed that "native" will get me.  That may change as my requirements change.  The bummer is that sticks me to running this on a Windows box and I'd rather run this on linux.

Not sure if you have read their source code. but I have found out that they have made a type error. the option attribute you need to set is actually 'paged' not 'paging'. The page event will fire after you have set the paged attribute in the options.

So first you need to use paged instead of paging in the options.
Secondly you need to pass a callback in the on page event. Something like the following.
ldapresponse.on('page', function(entry, cb) {
if(cb) cb();
});
the cb basically tells it to carry on. however you need to check if cb because for the last batch there will be no cb available because it is the end and no more pages will come after that.

@terryxsq I did not have any luck with your suggestion as cb is undefined the first time the function is called.

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


[mempool+json rpc] get rid of unnecessary dependencies

it includes:

mempool. Removing prost, storage-service, chrono
json-rpc. Removing reqwest, prometheus, libra-temppath

:hourglass: Testing commit 34f77fc45384bfa8f1628602b894042cd3b39e41 with merge edf1af939fd05966d3d24aaae44138e03a86c79c...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: metajack
Pushing edf1af939fd05966d3d24aaae44138e03a86c79c to master...


Don't replace links starting with http(s)

-/// [fmt]: https://docs.rs/tracing-subscriber/latest/tracing_subscriber/fmt/index.html
+/// [fmt]: https:::::docs.rs::tracing-subscriber::latest::tracing_subscriber::fmt

Oh wait, this is a bug I introduced myself in https://github.com/poliorcetics/cargo-intraconv/pull/4.

Wait no, this was present even before. I have a fix.

Done in the most recent version, thanks again though !


Q: Server event() not working - Dashboard Event Creator works

Hi there
I followed the instructions here https://docs.beyondco.de/laravel-websockets/1.0/getting-started/installation.html and read a lot on StackOverflow and in different issues here, but I am not able to get this working.
I configured my boostrap.js
import Echo from 'laravel-echo';

window.Pusher = require('pusher-js');
window.Echo = new Echo({
	broadcaster: 'pusher',
	key: process.env.MIX_PUSHER_APP_KEY,
	cluster: process.env.MIX_PUSHER_APP_CLUSTER,
	encrypted: false,
	wsHost: window.location.hostname,
	wsPort: 6001,
	disableStats: true,
	enabledTransports: ['ws', 'wss'],
});

Uncommented both Broadcast Providers in the config.php.
I created an example event and firing it event(new UserEvent($request->user()));.
In the event I return new Channel('Test').
I broadcastAs return 'user.updated';.
I listen for
Echo.channel('Test')
	.listen('.user.updated', (e) => {
		console.log(e);
	});

but nothing happens.
If I check the logs I see [2019-06-25 14:35:05] local.INFO: Broadcasting [user.updated] on channels [Test] with payload: but it never ends up on the client side, no console output.
If I go to the dashboard and use the Event Creator with channel Test and event user.updated I receive the console out.
So, it works in general, but not if I call event(), even if the logs say its Ok.
I searched and looked and am really missing what this could be.
Any hint on where to look, what to configure would be greatly appreciated. Is there something else I have to register, to do? It seems the event is fired but never catched by anything regarding the websocket server.
Thanka a lot in advance

Nevermind my stupidness, I overlooked one single line: BROADCAST_DRIVER=pusher
Sorry sorry sorry, please delete my nonsense ^^
Thanks


doc/storage: explain how optimized instance transfer works

Signed-off-by: Ruth Fuchss<EMAIL_ADDRESS>

Documentation preview available at: https://linuxcontainers.org/lxd/docs/pr.11323/

Thanks for the explanation @tomponline !
I gave it another go (after a lot of writing and rewriting ... this is a really tricky concept to explain!).

Thanks! Updated.


Dispose text controllers

In the examples, the textControllers in the modal sheets are not disposed. How to proceed properly ? Thank you.

Hi @outrelouxhe. So the text controller is part of the main content and provided as singleChildContent. Can't you make your singleChildContent a stateful widget and dispose them in the widget's dispose method?

If we scoped the text controller to the singleChildContent then we can't access the value in the nav bar widget events. I'm disposing after I pop the sheet. Seems to work.

@chris-herring One possible solution is moving the TextEditingController to a higher scope which can be done via the decorator field of the modal sheet.
You can also provide a way for the singleChildContent to notify the navigation bar or the parent widget about changes without directly accessing the controller but using the decorator field.
An example can be found here.
Before updating the TextEditingController, you can check if it's mounted or if it has been disposed of. This can prevent errors related to updating a disposed object.
if (textController.hasListeners) {
// Update the controller
}

Thank you @ulusoyca this is a good solution.


DataFrame conversion tutorial

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[x] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

What is this?
A new tutorial with ultra-simple examples of DataFrame conversion functions using Narwhals.
Motivation
I maintain a library which uses Polars internally. I want to allow my users to supply any DataFrame format, but I don't want to re-write my library's code, and not all the methods I use are already implemented in Narwhals.
For me, the most attractive application of Narwhals is to use it to "ingest" any DataFrame, and convert them to Polars for internal use. This is the lowest barrier to entry approach to Narwhals, and my guess is that many users would like this.
Given the likely popularity of this workflow, I think it deserves its own page and tutorial on the website.

thanks @vincentarelbundock ! just left a comment

Your suggested code doesn't work on my computer. I posted a comment above.

it would also be nice to have the code be runnable in the docs, so that it stays up to date - check other docs pages (e.g. docs/how_it_works.md) for examples of how to do that

I added this to the markdown chunks, but I wasn't sure if the session option was necessary, or what I should insert there.
```python exec="1" source="above"
2 + 3
```

thanks for updating - the snippet you posted works for me:
In [1]: import narwhals as nw
   ...: from narwhals.typing import IntoDataFrame
   ...: import duckdb
   ...: import polars as pl
   ...:
   ...: df = pl.DataFrame(
   ...:   {
   ...:     "A": [1, 2, 3, 4, 5],
   ...:     "fruits": ["banana", "banana", "apple", "apple", "banana"],
   ...:     "B": [5, 4, 3, 2, 1],
   ...:     "cars": ["beetle", "audi", "beetle", "beetle", "beetle"],
   ...:   }
   ...: )
   ...: df_duckdb = duckdb.sql("SELECT * FROM df")
   ...:
   ...: def df_to_polars(df: IntoDataFrame) -> pl.DataFrame:
   ...:   return nw.from_arrow(nw.from_native(df), native_namespace=pl)
   ...:
   ...: df_to_polars(df_duckdb).to_native()
Out[1]:
shape: (5, 4)
┌─────┬────────┬─────┬────────┐
│ A   ┆ fruits ┆ B   ┆ cars   │
│ --- ┆ ---    ┆ --- ┆ ---    │
│ i64 ┆ str    ┆ i64 ┆ str    │
╞═════╪════════╪═════╪════════╡
│ 1   ┆ banana ┆ 5   ┆ beetle │
│ 2   ┆ banana ┆ 4   ┆ audi   │
│ 3   ┆ apple  ┆ 3   ┆ beetle │
│ 4   ┆ apple  ┆ 2   ┆ beetle │
│ 5   ┆ banana ┆ 1   ┆ beetle │
└─────┴────────┴─────┴────────┘

could you show the output of nw.show_versions() please?

Sorry for the imprecise report. The df_to_polars() works when I call it the first time. But for some reason, it doesn't work when I call it a second time on the same DuckDB data frame.
In contrast, the function I had originally proposed seems to work for multiple calls.
:shrug:
import narwhals as nw
from narwhals.typing import IntoDataFrame
import duckdb
import polars as pl

df = pl.DataFrame(
  {
    "A": [1, 2, 3, 4, 5],
    "fruits": ["banana", "banana", "apple", "apple", "banana"],
    "B": [5, 4, 3, 2, 1],
    "cars": ["beetle", "audi", "beetle", "beetle", "beetle"],
  }
)
df_duckdb = duckdb.sql("SELECT * FROM df")

def df_to_polars(df: IntoDataFrame) -> pl.DataFrame:
  return nw.from_arrow(nw.from_native(df), native_namespace=pl).to_native()

df_to_polars(df_duckdb)

| A   | fruits   | B   | cars     |
|-----|----------|-----|----------|
| i64 | str      | i64 | str      |
| 1   | "banana" | 5   | "beetle" |
| 2   | "banana" | 4   | "audi"   |
| 3   | "apple"  | 3   | "beetle" |
| 4   | "apple"  | 2   | "beetle" |
| 5   | "banana" | 1   | "beetle" |

df_to_polars(df_duckdb)

InvalidInputException: Invalid Input Error: There is no query result
---------------------------------------------------------------------------
InvalidInputException                     Traceback (most recent call last)
Cell In[2], line 1
----> 1 df_to_polars(df_duckdb)

Cell In[1], line 17, in df_to_polars(df)
     16 def df_to_polars(df: IntoDataFrame) -> pl.DataFrame:
---> 17   return nw.from_arrow(nw.from_native(df), native_namespace=pl).to_native()

File ~/Downloads/narwhals/narwhals/functions.py:473, in from_arrow(native_frame, native_namespace)
    468 implementation = Implementation.from_native_namespace(native_namespace)
    470 if implementation is Implementation.POLARS and parse_version(
    471     native_namespace.__version__
    472 ) >= (1, 3):
--> 473     native_frame = native_namespace.DataFrame(native_frame)
    474 elif implementation in {
    475     Implementation.PANDAS,
    476     Implementation.MODIN,
   (...)
    480     # These don't (yet?) support the PyCapsule Interface for import
    481     # so we go via PyArrow
    482     try:

File ~/.virtualenvs/global/lib/python3.12/site-packages/polars/dataframe/frame.py:430, in DataFrame.__init__(self, data, schema, schema_overrides, strict, orient, infer_schema_length, nan_to_null)
    426 elif hasattr(data, "__arrow_c_stream__"):
    427     # This uses the fact that PySeries.from_arrow_c_stream will create a
    428     # struct-typed Series. Then we unpack that to a DataFrame.
    429     tmp_col_name = ""
--> 430     s = wrap_s(PySeries.from_arrow_c_stream(data))
    431     self._df = s.to_frame(tmp_col_name).unnest(tmp_col_name)._df
    433 else:

File ~/Downloads/narwhals/narwhals/dataframe.py:376, in DataFrame.__arrow_c_stream__(self, requested_schema)
    374 native_frame = self._compliant_frame._native_frame
    375 if hasattr(native_frame, "__arrow_c_stream__"):
--> 376     return native_frame.__arrow_c_stream__(requested_schema=requested_schema)
    377 try:
    378     import pyarrow as pa  # ignore-banned-import

InvalidInputException: Invalid Input Error: There is no query result

nw.show_versions()

System:
    python: 3.12.4 (main, Jun  9 2024, 13:30:56) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
executable: /Users/vincent/.virtualenvs/global/bin/python3
   machine: macOS-14.4-arm64-arm-64bit

Python dependencies:
     narwhals: 1.10.0
       pandas: 2.2.2
       polars: 1.7.0
         cudf:
        modin:
      pyarrow: 17.0.0
        numpy: 2.0.2

Right, thanks for confirming! there's indeed no guarantee that __arrow_c_stream__ can be called repeatedly, and this does need documenting, so thanks for highlighting this 🙏
tbh this pushes me (back) to preferring the name it from_pycapsule rather than from_arrow, else people expect to be able to call it multiple times just as if they were say passing a pyarrow dataframe to it 🤔
Anyway, I think we need to document both ways:

nw.from_arrow if you just need to convert once, and this should be more most efficient way
going via pyarrow, which may be less efficient but can be called liberally

Sounds good. I edited the file with a short discussion of these issues and the two alternative df_to_polars() functions.

Edits look great, thanks for agreeing to merge this!
Unfortunately, it looks liek the code doesn't execute properly on the site: https://narwhals-dev.github.io/narwhals/basics/dataframe_conversion/

thanks for spotting this! i think we can just add duckdb to the docs requirements, we'll need to expand duckdb support anyway, there's a lot of demand for that


IPython Console is not adding horizontal scrollbar when output is too wide

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Spyder console scrollbar hides part of the code or letters.
What steps reproduce the problem?

Write a print statement long enough.
Print the code to the console.

What is the expected output? What do you see instead?
 I've seen this before but we haven't had time to address it.
For now, please resize the all panes to the right of the interface by placing the mouse on the vertical separator. I attached a screenshot below showing where to put the mouse to do that:

Unfortunately in macOS that separator is almost invisible, but it's there.

Thanks for addressing my problem. It partially fixed the issue, but when I'm printing long pieces of code it's quite hard to resize the window in order to see everything. Thank you anyways!

I'll leave this one open so we don't forget to address it in the future.


EES-574 Updated godog to v0.10

What does this PR do?
A lot of changes and deprecations in the new version - updating library to be more or less up to date.

+1
this would've saved me a lot of time... last week :D


Tsc should get no parameters and leverage tsconfig.json

According to https://github.com/Microsoft/vscode-languageserver-node-example/blob/master/server/package.json#L17, tsc is compiling in the current running directory, but in this way it looks like changes are not propagated back to the client project.
I made some tests and removing the -p flags make it work correctly.

Not sure I understand but in this way it looks like changes are not propagated back to the client project.
tsc -p . === tsc. I added -p . to be more explicit.

I tried it one more time and it does not work with the original command. I'm not sure whether it's my machine or something else!

I just tried this today and it does work for me without any problems.
The changes are propagated back to the client directory via the outDir setting in the tsconfig.json.

This example is deprecated. Use the language server samples in this repository instead: https://github.com/Microsoft/vscode-extension-samples