Count provider occurrences in executed query results

Problem

We do not currently have any visibility into the number of times a provider shows up in search queries or how many queries they show up in period.
When considering the concerns @AetherUnbound brought up in this Make post about iNaturalist, it became clear that we need to have visibility into these metrics so that we know how dramatic changes to existing providers and the addition of new providers is actually reflected in real user searches.
Description

In the search_controller.search method, before returning the search results, create a tally of the providers for each result present in the page being returned:
provider_occurrences = defaultdict(int)
for result in results:
    provider_occurences[result["provider"]] += 1

for provider, occurrences in provider_occurrences:
    redis.incr(f"provider_occurrences:{week_timestamp}:{provider}", occurrences)
    redis.incr(f"provider_appeared_in_searches:{week_timestamp}:{provider}", 1)

This will allow us to see (a) how many searches each provider actually shows up in over the course of a week and (b) how many results from each provider were shown in search. If we can collect this data before iNaturalist data is fully available in the API, then we'll be able to directly see the effect of adding iNaturalist, both in the count of how many iNaturalist results start to show up and in and potential decreases in result counts for other providers.
Alternatives

Store the entire list of result ids returned for all queries in a time series. This would allow us to do more different kinds of analysis long term but would take way longer for us to get usable results from for the purposes of measuring the impact of iNaturalist on the actual user query results.

This may be obvious / already assumed, but it would be great to save the search terms and maybe the page number of the results here as well. Yes? And where would the info get stored? I could imagine stashing docs on s3 with that info as a quick solution, or setting up a new postgres table? I'm totally ignorant about the API, so please forgive me if these questions are silly in any way.

My plan was to store the tallies naively in Redis as described in the issue description, mostly because it is fast and simple to implement. Both seem important to me to avoid delaying turning on the iNaturalist DAG for very long.
I thought I remembered an issue existing in this repository to store queries and their results in Redis, but I can't find it now. It might have been #19, but I recall one with more details about what to store.
That being said, I'm happy to store whatever the folks with more data analysis skills than me think would be useful. I can definitely see how raw result position and page number would be useful information. If iNaturalist is dominating pages 10 – 200 maybe we care less as long as the first 3 or 4 pages are still as diverse.
@rwidom, can you suggest a data model for storing this in Postgres? Should we store individual searches with queries as a JSON blob and the results as a Postgres array of IDs, along with the page number? Page size and result position can be derived from the result's position in the array plus the length of the array multiplied by the page count. It also occurred to me that we could store each search result as a separate row with the overall position with a foreign key to the query stored in a separate table. Not sure which is more useful or easy to query.
One more complication, even if we store the individual searches, the API is cached pretty heavily in Cloudflare and repeated queries for the same terms will just used the cached response, so we wouldn't necessarily see the total frequency at which people are seeing iNaturalist, just how often it shows in queries overall. i.e., assume that for any unique query, there'd likely only be a single entry in a table storing the executed queries.
My primary goal though is to find a solution that we can implement quickly and easily throw out or iterate on. A table of queries and results would grow pretty quickly and be hard to change in the future, so I'd consider it something worth spending more time on up-front to get right.
If you can share more details about how you envision using S3 for this for a fast solution, I'd be grateful. I hadn't considered how we could use it for this.

Thank you for jumping on this @sarayourfriend, and for your thoughts @rwidom! I do think storing the preliminary information suggested in the issue description is a great start, and will at least give us a sense for how frequently iNaturalist shows up before & after the ingestion occurs. For now, I think this is sufficient - we could even avoid adding these statistics beyond page 3 or 4, as you mention @sarayourfriend. I do think having more information about our searches & provider diversity would be helpful, but for ensuring that iNaturalist doesn't flood all search results after we ingest it I believe that storing this information in Redis and checking it before/after ingestion will give us the impression we need!
I think we might want an RFC similar to the frontend event tracking for deciding which metrics and how to store the data related to searches themselves, and I don't think we'd want to block the iNaturalist ingestion for that effort.

I like the suggestion to restrict this data gathering to the top few pages. That will probably target the most relevant cases for this particular study.
Agreed to not block on a wider search metrics project.
Thanks for the input!

I've reverted the Postgres approach in the PR and switched it back to the Redis approach.
I'll add unit tests and then undraft the PR so we can get this in ASAP and avoid too many delays to iNaturalist getting into the API fully.


How to read SCB output file?

Hi, how can we open and see the output SCB file? Any software that we need to install? Thanks.

Sorry for the delayed response:
There are three options:

I have an informal set of python tools I use when working with Menge. They include the ability to read scb files (and perform various types of analysis). I'm still documenting them. I was thinking of simply pushing them up into a repository that Menge users can pull from.
It would be simple enough for me to create a utility in Menge that would create an ascii csv file from an scb file. Then you could consume it any way you like.
I could modify the output such that you can choose between an .scb file and an .sca file (the ascii equivalent). I'd always intended to have .sca as an output, but when modeling 10-30K agents, ascii files get ridiculously large very quickly.

Preferences?

Hi @MengeCrowdSim , thanks for your reply. Since I would like to try Menge with large crowd in the near future (probably more than 30k), I will pass Option 3 first.
For Options 1 and 2, it seems like that Option 2 is a straightforward approach to read output files. So I'd like to choose Option 2 if it doesn't cause you a lot of work, because I guess it will fulfil all my needs now.
For Option 1, if you could send me some sample tools that I can simply try them by myself without documents, that is good to me. Otherwise, I would be happy to wait for the new repository, and pull it in the future.
Thanks a lot!

Fastest is to pull this repository (I just set it up):
https://github.com/curds01/MengeUtils/tree/master
If you message me with an email address, I can give you access to the documentation that exists and help you use it. There are a number of very useful utilities (such as cropping, sampling, visualizing, etc., scb files).

Hi @curds01 , I'll pull the repository and try it, and my email address is
<EMAIL_ADDRESS>Thanks for your kind help~


Tags should be published with the exact version number instead of latest

Currently there is a bug in the publish shell script.
It publishes the image as latest event for tags.

Fixed


feat(tutorial): complete step 3

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Hey I'm not sure why this step is not passing the CI tests here. My site looks exactly like the preview in the tutorial and it passed all the CI checks when I did yarn CI-check. Any help and feedback would be greatly appreciated. Thanks!


Improve handleArcs function in translate/Main.hs

This was designed to include OR-causality handling, and has been tested and working, but was done quickly and without much thought on efficiency.
A TODO has been included in the code, but this issue serves to document it better.
What needs to be done:

[ ] Get rid of explicit recursion. Possibly by using groupBy
[ ] Generally improve performance.

I suggest to have a look at function groupSortOn in Data.List.Extra library: https://hackage.haskell.org/package/extra-1.5/docs/Data-List-Extra.html

groupSortOn requires that the type we're testing for is of type Ord.

groupOn might work. I'm trying that now

@snowleopard: OK groupOn doesn't work either. It groups them individually by snd but not all where snd are grouped together.
groupSortOn would probably do the same

For example:
arcs = [([A+, B+], C+), ([A-], C-), ([B-], C-), ([D+], C+), ([E+], C+), ([D-,E-], C-)]
This is: orGate a b c <> andGate d e c
When using groupOn or groupSortOn, the result is:
[A+,B+] C+
[A-] C-
[B-] C-
[D+] C+
[E+] C+
[D-,E-] C-

It doesn't combine them like I hoped. I was hoping for:
[[A+, B+], [D+], [E+]] C+
[[A-], [B-], [D-, E-]] C-

groupSortOn would probably do the same

@jrbeaumont No, actually it will sort them and then group, which is what we need. It does require a more complex constraint Ord instead of just Eq, but I think this is not a big deal for us. Essentially we ask for a to be not only comparable by ==, but also by <, so I suggest to try it.

When using groupOn or groupSortOn, the result is:

Could you show the code?

@snowleopard The latest version is in my repo: [https://github.com/jrbeaumont/concepts]

@snowleopard: Currently this doesn't print anything usable by a .g parser, just what the list contains for debugging.

Just an experiment:
Prelude Data.List.Extra> let arcs = [([1,2], 'a'), ([3], 'b'), ([4,5], 'a')]
Prelude Data.List.Extra> let grouped = groupSortOn snd arcs
Prelude Data.List.Extra> grouped
[[([1,2],'a'),([4,5],'a')],[([3],'b')]]

So it looks like groupSortOn does sort elements. There is still a bit of work to be done though :)

By the way, without Ord constraint we wouldn't really be able to improve efficiency of the code anyway.

Yeah I found an issue.
Not lines 36-44 in translate/Main.hs
This is what I've had to do to include Ord for DynSignal, as well as Ord for Transition a in order for this to work.
Are you aware of a better way?
I understand the importance of Ord but I couldn't find the best way to do it. The whole thing works with this.

Mmm, what about data DynSignal = Signal Int deriving (Eq, Ord)? :)
Otherwise, I'd implement it as:
instance Ord DynSignal where
    compare (Signal x) (Signal y) = compare x y

Mmm, what about data DynSignal = Signal Int deriving (Eq, Ord)? :)
I have tried that, and it just throws errors in the handleArcs function, needing Ord x => included, but I cannot work out what to put there, as there isn't an a or something associated with DynSignal.

So I added my own instance. compare x y is the same, but a better than (<=) ... I'll pop that in :)

Fixed in #51


AVG Detection

Should be fixed by b48cd1d
Will be included in version <IP_ADDRESS>
Please re-open this issue if you still see the problem in the new version


mesa: 22.1.4 -> 22.1.5

Description of changes

Bump changes: https://docs.mesa3d.org/relnotes/22.1.5.html

Moved patches (that were already using SRI hashes), I've adopted what the wiki calls the preferred way.

Added the fix for https://gitlab.freedesktop.org/mesa/mesa/-/issues/6519 as a patch (for darwin only).

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Are you sure that the darwin fixes have been included in 22.1.5?  I haven't noticed them in the release notes.

Are you sure that the darwin fixes have been included in 22.1.5? I haven't noticed them in the release notes.

I somehow looked the wrong tag, reverting this...

SRI: I like to simply paste the hash published on https homepages (example), but I don't really mind.

SRI: I like to simply paste the hash published on https homepages (example), but I don't really mind.

Indeed, that's easier. Reverted...
Instead of keeping the darwin friends outdated, I pulled the fix as a patch, seems fine right? Will ask a colleague to build it...

@ofborg build mesa

That most likely won't work, because the staging rebuilds are normally very large.  (and ofborg tries to merge with newest version of the target branch)

That most likely won't work, because the staging rebuilds are normally very large. (and ofborg tries to merge with newest version of the target branch)

It already succeeded with aarch64-darwin, which is already a relief. :smile:
But yeah, you're right, for x86_64-darwin I'll have to wait LLVM.
For reference, without the patch, this same version fails with:
Undefined symbols for architecture x86_64:
  "_dri_kms_driver_extensions", referenced from:
      _swkmsCreateNewScreen in libdri.a(dri_util.c.o)
  "_galliumdrm_driver_extensions", referenced from:
      _dri2CreateNewScreen in libdri.a(dri_util.c.o)
ld: symbol(s) not found for architecture x86_64

I only see a failure on aarch64-darwin, though an unrelated one.

@ofborg build mesa

https://github.com/ofborg build mesa

@ofborg build mesa

Successful builds in both aarch64 and x86_64 for darwin, against unstable:

x86_64: https://logs.nix.ci/?key=nixos/nixpkgs.185699&attempt_id=7629ae3a-b87f-4e86-9fc7-343028f035ba
aarch64: https://logs.nix.ci/?key=nixos/nixpkgs.185699&attempt_id=deb5f7c8-fb0d-4a36-9db7-ae9cfb1678a2

I should have run the tests instead of package in ofborg...

@GrahamcOfBorg eval

@ofborg eval

@NixOS/ofborg eval

@ofborg eval

@vcunat what do you recommend me to deal with this? Keep the darwin patch (which builds, but I can't test), or revert and just push the bump?
PS: Meanwhile, the "No channel PR" won't re-trigger in the right target.

It sounds good to me this way.  I think we've never tested mesa for *-darwin beyond building stuff.

Hydra doesn't test much relevant things, and staging branch normally isn't built (immediately).  But getting further, into the the next staging-next iteration will create binaries and allow easier testing by people.  Either way, patch bumps shouldn't be problematic.

Let's skip to 22.1.6?
https://lists.freedesktop.org/archives/mesa-announce/2022-August/000683.html

22.1.6

In another, less noised PR.


Missing: global app menu

Right now, there's no common way to have a global application menu (instead of per application menu bar), like we have in X11.
So any environment like Unity or Mac OSX aren't possible in Wayland.
For KDE application, it seems to work, but not for Gnome based application like Firefox.

Yeah, this is something I miss dearly with many apps when run on Wayland. Global Menu just save spaces for me. It also provides some additional functionalities, like with Chromium browsers which exposes some things for quicker access at my top panel. Also, some things are outright broken because of lack of Global Menu support - I have to run OBS in xwayland every time I want to access remux menu and everything else not accessible directly from the app.
Details


Running k3s on an airgap environment

Hi all !
Is your feature request related to a problem? Please describe.
I'd like to be able to deploy small (= 1 node only) environment were node has no access to Internet. This is sometimes related as "airgap" environment.
Describe the solution you'd like
I need to be able to download everything needed for my cluster on one machine which might be connected to Internet and export it as files. In ideal conditions, it would be a Web-browser only machine (no other tools). Situation were I might have to install additional softwares are not critical.
Then, copy all the files (eg, via USB, DVD, other...) to an environment which is not connected to Internet (or filtered) and deploy.
Describe alternatives you've considered
Parsing several issues in k3s projects, I've found relevant information.
My last test was done with 0-2-0-rc4, which closes #141 , #99 and #92 (it will be documented in #167 ).
Steps :

run k3s server on a machine connected to Internet and see which images are loaded (k3 crictl images)
run docker pull on each previous image
then docker save -o k3s.required.images.tar Every image name:tag
copy k3s + k3s.required.images.tar on machine without Internet context
run k3s server (to create directories) then stop it
Copy k3s.required.images.tar into /var/lib/rancher/k3s/agent/images/
run k3s server
run k3s crictl images => OK, images are imported
run k3s kubectl get all --all-namespaces -o wide => Traefik does not work.
==> Traefik chart is required. Would it be possible to add chart into a new directory, as you did with images ?

Additional context
Not applicable
Thanks for your help :)

This is a duplicate of https://github.com/rancher/k3s/issues/166

This is definitely on the road map.  Image preloading already works (needs documentation).  But we need to also do offline helm charts.  So still working on that.

Thanks (and sorry I did not see #166)
One more thing : k3s (as most K8s do) complains about a lack of default route, which not always defined in isolated networks.
I'm not able to help in coding, but feel free to ask some support in documentation or testing phases :)

Thanks everyone !!!
My test result is: almost OK. Only problem is the default route that prevents k3s from starting, but there is a quirky workaround.
Here is my test scenario in case you need it for the documentation

Download k3s + k3s-airgap-images-amd64.tar in ~/
Remove my Ethernet wire
Configure network (to avoid default route problem) : sudo ip -c address add <IP_ADDRESS>/24 dev eno1 && sudo ip route add default via <IP_ADDRESS>
Create directory et move images sudo mkdir -p /var/lib/rancher/k3s/agent/images/ && sudo cp ./k3s-airgap-images-amd64.tar /var/lib/rancher/k3s/agent/images/
Run the server chmod +x ./k3s && sudo ./k3s server
Verify everything is working (try it for 2 minutes while everything is being deployed) : sudo ./k3s kubectl get all --all-namespaces -o wide . Note : pod/helm-install-traefik-zmzpc end in status Completed
Deploy a dummy container + service and verify service has an endpoint sudo ./k3s kubectl run dummy-test --image k8s.gcr.io/pause:3.1 --image-pull-policy=Never --expose --port 12000 && sleep 30 && sudo ./k3s kubectl get all -o wide && sudo ./k3s kubectl describe service/dummy-test. Service now has a non empty endpoint :)

Do I need to do more testing ?
Cheers,

Thank you @vdups! I will add some info to the docs re this feature and the default route.


dts: arm: st: Introduce pinctrl nodes and uart pinmux configuration

Introduce pinctrl nodes and uart pinctrl configuration in stm32 socs
A preambule for this change is the rework of the pinmux driver in each series in order
to achieve a model where pin could be configured with the call to a single array
of {pin, pinconfig} couples. This rework should be tansparent in term of existing pinmux
features.
Then a pinctrl yaml description and pinctrl binding are introduced to be used with a pinctrl node.
Pinmux configurations are introduced in -pinctrl.dtsi files. This is done only for u(s)art for now.
All uart pin configurations in existing c files are reproduced here. Once we'll be able to
generate code thanks to dts files, clean up should be done to remove duplicated information.

Based on discussion with @erwango - tested on nucleo_f401, nucleo_f334r8, disco_l4, nucleo_f103

@ydamigos or @mbolivar can you try this out on a 96b_carbon?

@galak I flashed the hello_world sample on 96b_carbon and it works fine. Should I try another sample?

samples/basic/blinky and samples/subsys/shell are working fine on 96b_carbon with this.

samples/subsys/shell is working fine also on olimexino_stm32 and stm32f3_disco

Updated according to comments.
Thanks @mbolivar, @ydamigos for testing!


Using GitCop

We want to use GitCop to make sure the commits have the following trailers:
License: MIT
Signed-off-by: user name <email@address>

We might also limit the subject of each commit to 80 characters.
What do you think?

Hm,  I think this page says max commit subject of 50 characters:
https://github.com/ipfs/community/blob/master/contribution-guidelines.md

Yeah we can decrease it to 50, but I think it will make too much noise for not much gain as 80 should still be readable for most people.
It might only be better for people looking at commits from smartphones, as today terminals are usually much wider than 80 characters and web browsers on recent desktop or laptops also have no problem with 80 characters.

ok 80 sounds good

Maybe the gitcop message should mention the commit hook directly too

@spikebike just had a hard time figuring out how to make this happen. maybe also have a couple of commands that guide people to amending their commits? (many people dont know how to use git rebase)

@jbenet ok I will improve the GitCop message and https://github.com/ipfs/community/blob/master/docs/commit-message.md soon.

@chriscool can this be closed?

Yes, let's close this.

Sorry but I think we should probably reopen this, as it is referenced in the GitCop message where we ask people to provide their feedback in this issue.

By the way the GitCop footer message is now:
We ask for a few features in the commit message for Open Source licensing hygiene and commit message clarity.
`git commit --amend` can often help you quickly improve the commit message.
[Guidelines](https://github.com/ipfs/community/blob/master/contribution-guidelines.md#commit-messages) and [a script](https://github.com/ipfs/community/blob/master/dev/tools/hooks/setup_commit_msg_hook.sh) are available to help in the long run.
Your feedback on GitCop is welcome on [this issue](https://github.com/ipfs/infrastructure/issues/23).

Tell me if you see something that can be improved in it.
But note that GitCop for now doesn't let us provide a customized error message for each different error that can trigger it.
The different errors that can trigger it are the following:

the subject is more than 80 characters long
the commit message doesn't contain:

License: MIT
Signed-off-by: user name <email@address>

GitCop should delete it's comments when someone PRs a fix to the commit and there is no longer an issue.

Yeah, that might be a good idea. I will suggest it.

The changes we requested have been taken care of by Gary, the GitCop guy, and I have changed the GitCop configuration for the go-ipfs repo.
Now the GitCop messages are attached to individual commits, so when the commit are fixed, the GitCop comment will be masked by default by GitHub.
Also as https://github.com/ipfs/community/pull/126 is merged, each specific GitCop error messages have been customized to have a link to a specific section in https://github.com/ipfs/community/blob/master/contribution-guidelines.md


Switch to use knative operator for installing in e2e tests

/kind feature
Describe the solution you'd like
Currently kserve e2e tests are using kustomize for installing Knative which is difficult to maintain and keep up with upstream.
We'd like to switch to use knative operator to manage the installation and configuration customization would be way easier instead of creating overlays.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

/assign


only launch one thruster interface

Problem
today both thruster interface and mcu interface are launched ona "real" launch. Both do the same job, but mcu interface is for thruster connected to a mcu and thruster interface is for thruster connected to an adafruit board on an raspberry pi. Funny thing is that Manta after its refit now is using the setup that thruster interface was made for.
Solution
Seperate the two nodes in launch, such that the correct node is launched depending on what drone is used.

The thruster_interface node currently does something that is needed for the gladlaks mcu; namely converting a thruster_forces vector into PWM signals. I discussed with Håvard and the best solution seems to let the mcu_interface take care of that conversion, and also send the pwm message to the mcu. This removes the need for a separate topic for the pwm message.
Since we are approaching the integration test so fast, the mcu_interface will most likely be designed for the gladlaks first. Once we have more time, we want to eliminate the thruster_interface entirely, and make the mcu_interface usable for any hardware. The hardware specifics will be put in the configuration files.

Thanks for the clarification - seems I got tricked by our own documentation.
The move to include the pwm conversion on the mcu interface sounds very reasonable to me as well.

Note: This is currently fixed, but with hardware-dependent files in the repo. I believe it could be a better idea to add everything to a central MCU repo, but this will likely happen after the pooltest

Closing as this has now been resolved


Hovering over a tile should show the measurements of the cell

Not needed, the properties window is more than adequate


Add server side signature key and certificate

Codecov Report

Merging #57 (9811b3e) into master (3397f77) will increase coverage by 1.17%.
The diff coverage is 100.00%.

:exclamation: Current head 9811b3e differs from pull request most recent head dfbe0ab. Consider uploading reports for the commit dfbe0ab to get more accurate results

@@             Coverage Diff              @@
##             master      #57      +/-   ##
============================================
+ Coverage     74.32%   75.50%   +1.17%
- Complexity      295      296       +1
============================================
  Files            39       39
  Lines          1589     1592       +3
  Branches        110      110
============================================
+ Hits           1181     1202      +21
+ Misses          360      342      -18
  Partials         48       48

Flag
Coverage Δ

java
75.50% <100.00%> (+1.17%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/main/java/com/aliyun/gmsse/crypto/Crypto.java
90.80% <100.00%> (+21.75%)
:arrow_up:


Passing in className as options parameter with the addMarker method

It will be great to allow className to be passed through into maplibreGL's addMarker method which will allow developers to fine tune the marker styles.
Details on available mark options on MapLibreGL can be found here - https://maplibre.org/maplibre-gl-js/docs/API/types/maplibregl.MarkerOptions/

Now in<EMAIL_ADDRESS>


Bump to PHPStan 2 and PHPParser 5

All checks have passed 🎉 @TomasVotruba I think it is ready.

Awesome :+1:


stdin

stdin don't seem to work. it just shows help.
echo "test" | jinja2

Hi, what do you expect the stdin to be? There are two arguments if you read the --help. The first is the template to render, the second is the data.
data can be accepted over stdin, but the template itself is read from disk.
So you can do soemthing like:
echo '{"foo": "bar"}' | jinja2 something.j2 -

The design decision I had here was "which is more common to accept over stdin", and in all of my use cases, it's the data, not the template. And unfortunately, there aren't two stdins. In theory I can make the API handle one or the other or something, but that'd be pretty awkward.

I see it from your POV.
my case. using docker compose where ENV vars are mostly used. Looking for something simple like..
export WOOT=123
echo 'what is {{WOOT}}' | jinja2 > out
or something like.
cat nginx_conf.jinja | jinja2 > /etc/nginx/nginx.conf

found my error thanks!


Add rust-version spec to Cargo.toml

The last version of the crate (0.1.27) started to trigger build failures with older rustc (1.59 in my case) due to usage of the API that were introduced in 1.66:
error[E0412]: cannot find type `RawFd` in `std::os::fd`
 --> /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/jobserver-0.1.27/src/error.rs:2:27
  |
2 | type RawFd = std::os::fd::RawFd;
  |                           ^^^^^ not found in `std::os::fd`
  |
help: consider importing this type alias
  |
1 | use std::os::unix::prelude::RawFd;
  |

error[E0603]: module `fd` is private
   --> /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/jobserver-0.1.27/src/error.rs:2:23
    |
2   | type RawFd = std::os::fd::RawFd;
    |                       ^^ private module
    |
note: the module `fd` is defined here

The std::os::fd is only available since rust 1.66: https://doc.rust-lang.org/stable/std/os/fd/index.html
Adding the rust-version spec to Cargo.toml will make sure that cargo downloads a lower version that's compatible when doing a build. To make sure it works properly you'd also need to yank the 0.1.27 after releasing the fix.

See #59.


[Croissant] Update data field in new splits recordset

@lhoestq @severo could you PTAL? Thanks!

Follow-up to https://github.com/huggingface/dataset-viewer/pull/3058

Thanks @lhoestq -- do you know how long it will take for this to be deployed more or less? I remember it took a few days for https://github.com/huggingface/dataset-viewer/pull/3058, any chance we could speed the process up a bit for this PR?

I'm aiming for a deploy later today - will ping you once it's done

deployed ! :)

Merciii!

Hi @lhoestq ! We were wondering how long it might take before all datasets' croissants include this change?
We noticed that some of them, like e.g. https://huggingface.co/api/datasets/ai4bharat/sangraha/croissant, seem to still use split_name instead of split_record_set_name/split_name in the split recordset's "data" field.


[DOC] describe index.query.max_nested_depth setting

What do you want to do?

[ ] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Completion of https://github.com/opensearch-project/OpenSearch/pull/11670

2.x code doc: #6347
3.0 code doc: #6715

What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

Thanks @mkhludnev
how I can put #6347 to 3.0 as well?

Thanks @mkhludnev how I can put #6347 to 3.0 as well?

This will get pushed to 3.0, but if you want additional changes, you can create a new PR.

@hdhalter

This will get pushed to 3.0, but if you want additional changes, you can create a new PR.
3.0 doc change #6715

I'm looking for a way to clone this issue, but I don't think it's possible in GitHub. In any event, after 2.13, I'll reopen this issue so we can use it for tracking for the 3.0 PR.


Added music composition mutations and tests

This PR can delete work.py too, if this code is duplicated

I'm not sure why the conflicts are not showing up on my end again, but I've made the changes required.

git fetch origin and git pull just show "Already up-to-date", with no conflicts. But this page shows conflicts in 5 files.

what branch are you on when you're running those commands? your add_composition one? You'll have to merge master into the branch to fix these conflicts. Try this:
git checkout master
git fetch origin
git merge origin/master
git checkout add_composition
git merge master

great, thanks. Will get this merged today

I've made some changes to this, please take a look at them:

Remove work.py and test_composition.py, these were unneeded now that we have the musiccomposition version
I updated the structure of mutation_update_music_composition to not duplicate as much code

Can you please look at the tests for musiccomposition and update the following things:

Make the test files look more "correct" - e.g. EXPECTED_MUSICCOMPOSITION.txt seems to have field values that look more like a Document or a Person. These should at least look like an example from our modelling doc
rename the expected text files to lower-case, and a better name structure, like I did with person
Add some tests for create_musiccomposition and update_musiccomposition to test that exceptions are raised with bad data (e.g an invalid language or a format without a / in it)
add tests to check that optional values are not added if they're not given - e.g. do 2 tests for update_musiccomposition - one with only the required fields, check that only those fields are listed in the mutation, and one with all fields set, and make sure that all those fields are written in the mutation text.

thanks!

Looks good, but I see 3 failing tests:
<tests.test_musiccomposition.TestMusicComposition testMethod=test_add_broad_match>
<tests.test_musiccomposition.TestMusicComposition testMethod=test_update_all>
<tests.test_musiccomposition.TestMusicComposition testMethod=test_update_name>
looks like they're all to do with loading the expected text file


[Phcode.dev Bot] Publishing Release v0.1.1 to Extension Store

Thank you for contributing to phcode.dev extension store.
Please track extension/theme publish status by clicking this link.
You may close this issue at any time.

Extension successfully published to store. You can close this issue at any time.


Update compatible versions list for bevy_replicon_renet2

I believe bevy_replicon_renet2 has been updated to support Bevy 0.15 and Replicon 0.29 as per the Cargo.toml, but the compatible version list in the README is not updated. I personally search for these lists when I am trying to choose a crate, so it would be beneficial to correct it.

Fixed in https://github.com/UkoeHB/renet2/commit/37482c636a48e901f0b0ad06a8166210941c8468


Set up accounts

Digital ocean
Zapier
Mongohub

@tbashor If we are going to use Github auth, we'll likely need to register a client for an ID and Secret Key. Can this be added to the list?


fix taskinstance.isSubprocess misjudgment

What is the purpose of the pull request
#1984 fix taskinstance.isSubprocess misjudgment
Brief change log

fix taskinstance.isSubprocess
add processInstance UT

Verify this pull request
This pull request is already covered by existing tests, such as:

api/service/ProcessInstanceServiceTest.

Codecov Report

Merging #1985 into dev will increase coverage by 1.09%.
The diff coverage is 72.96%.

@@             Coverage Diff              @@
##                dev    #1985      +/-   ##
============================================
+ Coverage     32.31%   33.41%   +1.09%
- Complexity     1420     1552     +132
============================================
  Files           294      311      +17
  Lines         15022    15402     +380
  Branches       1810     1864      +54
============================================
+ Hits           4855     5146     +291
- Misses         9688     9727      +39
- Partials        479      529      +50

Impacted Files
Coverage Δ
Complexity Δ

...pache/dolphinscheduler/common/enums/QueryType.java
0% <0%> (ø)
0 <0> (?)

...hinscheduler/common/utils/TaskParametersUtils.java
73.68% <0%> (-4.1%)
11 <0> (ø)

...lphinscheduler/server/worker/task/TaskManager.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...phinscheduler/server/worker/task/AbstractTask.java
11.94% <0%> (-0.37%)
2 <0> (ø)

...ommon/task/sqoop/sources/SourceMysqlParameter.java
100% <100%> (ø)
19 <19> (?)

...common/task/sqoop/sources/SourceHdfsParameter.java
100% <100%> (ø)
3 <3> (?)

...ommon/task/sqoop/targets/TargetMysqlParameter.java
100% <100%> (ø)
19 <19> (?)

...common/task/sqoop/targets/TargetHiveParameter.java
100% <100%> (ø)
17 <17> (?)

...common/task/sqoop/targets/TargetHdfsParameter.java
100% <100%> (ø)
13 <13> (?)

...apache/dolphinscheduler/common/enums/TaskType.java
80.95% <100%> (+0.95%)
2 <0> (ø)
:arrow_down:

... and 33 more

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 4930016...4ff1b67. Read the comment docs.

re-submit pr


Poor formatting of some winrt-rs generated code

Here is some code that was generated by https://github.com/microsoft/winrt-rs that I have copied into a standalone repro to illustrate how rustfmt fails to format some portions of the code.
https://github.com/kennykerr/rustfmt-repro
It may have something to do with the complexity of the generated code, but at some point the formatting just seems to give up. For example, main.rs lines 94-97 looks like this:
(this . vtable() . operation_completed)
(this, handler . into() . get_abi(), < super :: super
    :: foundation :: EventRegistrationToken as :: winrt
    :: AbiTransferable > :: set_abi(& mut result__)) .

Subsequent lines are formatted correctly, but it seems to give up on this expression and passes right on by. Here's another example from lines 587-589:
< super :: super :: foundation :: collections :: IMap :: <
:: winrt :: HString, :: winrt :: Object > as :: std ::
convert :: From < & Self >> :: from(self) . lookup(key,)

Some like 6194-6212 are really hard on the eyes:
pub struct abi_IDataProviderRequest where
{
    base__ : [usize ; 6], pub format_id : unsafe extern "system"
    fn(:: winrt :: NonNullRawComPtr < IDataProviderRequest >,
        result__ : * mut < :: winrt :: HString as :: winrt ::
        AbiTransferable > :: Abi,) -> :: winrt :: ErrorCode, pub
    deadline : unsafe extern "system"
    fn(:: winrt :: NonNullRawComPtr < IDataProviderRequest >,
        result__ : * mut < super :: super :: foundation :: DateTime
        as :: winrt :: AbiTransferable > :: Abi,) -> :: winrt ::
    ErrorCode, pub get_deferral : unsafe extern "system"
    fn(:: winrt :: NonNullRawComPtr < IDataProviderRequest >,
        result__ : * mut < DataProviderDeferral as :: winrt ::
        AbiTransferable > :: Abi,) -> :: winrt :: ErrorCode, pub
    set_data : unsafe extern "system"
    fn(:: winrt :: NonNullRawComPtr < IDataProviderRequest >,
        value : < :: winrt :: Object as :: winrt :: AbiTransferable
        > :: Abi,) -> :: winrt :: ErrorCode,
}

Meta

rustfmt version: rustfmt 1.4.17-stable (8a93416c 2020-07-20)
From where did you install rustfmt?: rustup

There's a couple things going on with the first snippet in your issue description, all related to these portions of the generated code exceeding max_width when formatted. Missing from the snippet in the OP (but viewable in the repro repo) is a trailing .and_then(|| result__) which makes makes this a chain with an exceptionally long root/parent. There's a known issue in rustfmt wherein if any single chain item exceeds the value of max_width, then rustfmt has to bail on formatting the entire chain and leaves the original formatting in place (#3863)
Shortened, full repro with relevant indentation.
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=6220215ef9ed981c658f02bb9a1afd66
pub mod windows {
pub mod application_model {
        pub mod data_transfer {
            #[repr(transparent)]
            #[derive(Default, Clone, PartialEq)]
            pub struct DataPackage {
                ptr: ::winrt::ComPtr<IDataPackage>,
            }
            impl DataPackage {

                pub fn operation_completed<
                    'a,
                    T0__: ::std::convert::Into<
                        ::winrt::Param<
                            'a,
                            super::super::foundation::TypedEventHandler<
                                DataPackage,
                                OperationCompletedEventArgs,
                            >,
                        >,
                    >,
                >(
                    &self,
                    handler: T0__,
                ) -> ::winrt::Result<super::super::foundation::EventRegistrationToken>
                {
                    let this = <Self as ::winrt::AbiTransferable>::get_abi(self)
                        .expect("The `this` pointer was null when calling method");
                    unsafe {
                        let mut result__: super::super::foundation::EventRegistrationToken =
                            ::std::mem::zeroed();
                        (this . vtable() . operation_completed)
                        (this, handler . into() . get_abi(), < super :: super
                         :: foundation :: EventRegistrationToken as :: winrt
                         :: AbiTransferable > :: set_abi(& mut result__)) .
                        and_then(|| result__)
                    }
                }
                }
                }
                }
}

Secondly, the combination of the indentation levels and the very long path expression within the tuple also causes a max_width related formatting failure, that causes rustfmt to bail. Minimal repro (blocks and indentation can be excluded with a lower max_width value):
fn foo() {
    {
        {
            (< super :: super
                :: foundation :: EventRegistrationToken as :: winrt
                :: AbiTransferable > :: set_abi(& mut result))
        }
    }
}

Also slightly related FWIW @kennykerr, I briefly chatted about this with @rylev in a different thread within this repo, but want to reiterate that I'd advise against the current rustfmt usage within in the winrt-rs build macro that users utilize.
In my experience generated code is typically configured to have rustfmt ignore the generated code when users run rustfmt (by adding #![rustfmt::skip] to the top of generated files) vs. trying to continue to format the generated code.
I say that because there will inevitably be users that don't have rustfmt on their system, either because they deliberately don't use/install the rustfmt component or because they weren't aware that rustfmt is not included in the default rustup profile. There's also several scenarios where rustfmt will exit with a non-zero exit code (invalid config file for example), even when just formatting the generated code from the winrt-rs build script.
I believe that the resulting panic and stacktrace users will see in these scenarios when running a cargo build will likely be a source of confusion, especially for users that are new to the Rust ecosystem.

Thanks @calebcartwright - that's very helpful to understand.
I agree that it shouldn't fail if rustfmt isn't present. Our needs for formatting are at best lossy. The code shouldn't be committed/checked-in and only exists (1) to provide a reasonable debugging experience and (2) to enable rust-analyzer to work.
So if these are known issues that's great. We may eventually replace rustfmt anyway, as we have serious challenges with the performance of the Rust toolchain when it comes to the generated code and need to save time wherever possible. We may end up updating our code generator to generate pre-formatted code instead of a TokenStream as the latter has a very poor default string representation and running rustfmt is quite time consuming.
Still, really appreciate the time and effort you've put into it. As a long-time C++ developer, it's so refreshing to have such a tool.

@calebcartwright can this be closed as a duplicate of https://github.com/rust-lang/rustfmt/issues/3863?

Yes, I think so. The context in this PR is itself a bit outdated, and I think fixing #3863 will mostly fix the issues we're seeing in windows-rs (modulo some performance concerns).


Creating a ClusterRoleTemplateBinding with a non-existent user breaks the Cluster Membership page

Setup

Rancher version: dev-2.8 (also on 2.7.7)
Rancher UI Extensions: None
Browser type & version: Firefox

Describe the bug
If you create a CRTB that does not have a user that exists, when you open the Cluster Membership page the list of members will not load. Doing the same for PRTBs does not cause an issue in the Project Membership page. Also, if you use a field other than userName, such as userPrincipalName it doesn't cause the UI to break.
To Reproduce
With public API in 2.8.0:

Create a Cluster Role Template in the Users & Authentication tab
Open a kubectl shell for your local cluster
Apply the following CRTB via kubectl apply -f crtb.yaml

apiVersion: management.cattle.io/v3
kind: ClusterRoleTemplateBinding
metadata:
  name: test-crtb
  namespace: local
clusterName: local
roleTemplateName: <clusterRoleTemplate.Name>
userName: fake-user

Alternatively, this can be done through the Norman API in 2.7.7:

Create a Cluster Role Template in the Users & Authentication tab
Use the following curl command:

curl -u "${CATTLE_ACCESS_KEY}:${CATTLE_SECRET_KEY}" \
-X POST \
-H 'Accept: application/json' \
-H 'Content-Type: application/json' \
"${RANCHER_URL}/v3/clusterroletemplatebindings" -d \
'{
    "clusterId": "local",
    "groupId": "",
    "groupPrincipalId": "",
    "name": "",
    "namespaceId": "",
    "roleTemplateId": "${ROLE_TEMPLATE}",
    "userId": "fake-user",
    "userPrincipalId": ""
}'

Result
Go to the "Cluster and Project Members" tab and see that the list of Cluster Memberships won't populate any members, not even those previously visible.
Expected Result
All the members should be listed, and if the user doesn't exist use the same image from PRTBs:

Screenshots
What the broken UI looks like:

Additional context
Issue in rancher/rancher with some additional context and more screenshots:
https://github.com/rancher/rancher/issues/42822

I have validated this issue on v2.8-01127d844339ea8ab498f334121e2956be610409-head, the issue got fixed and closing the issue.


Add content about setting up and configuring influx cli with your cloud account

I was trying to provide instructions to a cloud user about setting up the influx command line tools, and i couldn't find a good page to link them to in the cloud docs.
There's this: https://docs.influxdata.com/influxdb/cloud/reference/cli/ but it doesn't tell you anything about setting up and configuring it. Same with this page: https://docs.influxdata.com/influxdb/cloud/reference/cli/influx/ there's no real install instructions or guidance on how to use it with cloud.
I propose a dedicated page for setting up the influx cli tool to work with your cloud account. the outline would be as follows:

Intro
Download and install the influx cli tool
Configure the cli tool to connect to your cloud instance
Additional references and/or examples

Is this addressed as part of the new get started content?

Hi @russorat and @timhallinflux ,
We have this information documented in the following places:

Download and install CLI is in the Cloud Sign Up doc.
Configure CLI tool to connect to cloud is on the reference index page . This page is also linked from every CLI example page in the related section.

Does that cover the content you're looking for? Let me know what you think.
Thanks,
Nora

I think the answer is no.  We need to be a bit stronger in terms of using the CLI with Cloud.


A little questuon about ENAS

hi,i'm new in your project,and it is the first time i ask question on github,Please bear with me if there are any mistakes. I find something may be wrong in your code about ENAS.
when i finish a code about ENAS by imitating your "main_tensorflow.py"  and i run it and find some error.
Traceback (most recent call last):
File "C:/Users/JssI/Desktop/deep_architect-masterPZJ/ENAS.py", line 322, in  main()
File "C:/Users/JssI/Desktop/deep_architect-masterPZJ/ENAS.py", line 312, in main inputs, outputs, _ ,searcher_eval_token = searcher.sample()
File "C:\Users\JssI\Desktop\deep_architect-masterPZJ\dev\enas\searcher\enas_searcher.py", line 104, in sample  outputs, list(hs.values()))):
NameError: name 'hs' is not defined
so i click into the "enas_searcher.py" i find that there is one  variable is not declared which named "hs"
`
 def sample(self):
    arc = self._sample()
    idx = 0
    hyp_values = {}
    for i in range(1, self.num_layers + 1):
        hyp_values['op_' + str(i)] = arc[idx]
        idx += 1
        for j in range(i - 1):
            hyp_values['H.skip_' + str(j) + '_' + str(i) + '-0'] = arc[idx]
            idx += 1

    inputs, outputs = self.search_space_fn()
    vs = []
    for i, h in enumerate(
            unassigned_independent_hyperparameter_iterator(
                outputs, list(hs.values()))):
        if h.get_name() in hyp_values:
            v = h.vs[hyp_values[h.get_name()]]
            h.assign_value(v)
            vs.append(v)
        else:
            v = random_specify_hyperparameter(h)
            vs.append(v)
    return inputs, outputs, vs, {'arc': arc}

`
What you do is a really nice work  and  i want to use your code to finish my experiment  during my holiday, i will very very appreciate you if you could help me solve the problem. Thank u!

by the way ,  could u write a code to call the enas as a simple exemple ? Thank u!!!!

I can eventually look at this, as we want to have ENAS implemented. Unfortunately, the code in the dev folder is highly experimental. It is kept there mostly as a reference for when we get around making it part of the main folder. Currently, I can only provide guidance on how to implement ENAS in the current codebase (e.g., where to look to support the operations needed for ENAS). With my current availability, it will be one or two weeks until I can look at this myself (and I'm not making any hard promises right now).
That being said, if you decide to implement ENAS yourself in DeepArchitect, I can provide support.


Fix Header example on index.md

The example provided didn't actually work. It seems to have been extracted from an earlier example but not update with the changes to the item method.

Thank you.


Errors keep coming that say Keybase is offline, but it isn't.

I keep getting errors saying that I cannot browse files when Keybase is offline.  However, keybase is online and when I try to actually access files it works without issue.  The notifications are odd and getting annoying.
my log id: 6b49ea5edbee63faa399791c

Dup of #21089


ERROR: 'provided'

Hi,
I am not verse in php so I am having a hard time. I have configured the default.cfg with just the API keys but I get the following error right off the bat:
Obtaining settings from config file.
Welcome to Poloniex Lending Bot
2016-07-21 14:04:12 ERROR: 'provided'
Traceback (most recent call last):
File "lendingbot.py", line 457, in
refreshTotalLended()
File "lendingbot.py", line 230, in refreshTotalLended
for item in cryptoLended["provided"]:
KeyError: 'provided'
Thanks!

My bad, the API Key was not enabled for the IP address so that's why it was getting probably an empty result set that was causing the error. I would suggest validation of the communication first and a catch of empty result sets to avoid this. Clearly user fault but would be nice for user experience to catch those mistakes. Thanks!

Yeah, we really need to add guidelines that link to https://github.com/Mikadily/poloniexlendingbot/wiki/FAQ-(Trouble-shooting)


timeout when to auth in neovim

I install copilot in neovim with lazyvim.
But when I try to auth, I got error:
[Copilot] RPC[Error] code_name = InternalError, message = "Request signInInitiate failed with message: Request to <https://github.com/login/device/code> timed out after 30000ms"

what is this? Did I miss any config here?
Please help, thanks in advance.

i have the same problem, are you fix?


Leverage admiral icon service to generate icon when harbor working in VIC mode

No engineering effort required for VIC.


Avoid overwriting methods when generating 2-to-1 overloads on arbitrary types

Fixes an oversight in #197 by separating overloads on arbitrary types from methods defined on tracers only, thus avoiding duplicate overwriting of existing methods.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 95.45455% with 2 lines in your changes missing coverage. Please review.

Project coverage is 37.37%. Comparing base (9ce962f) to head (ce26ffd).

Files with missing lines
Patch %
Lines

src/overloads/utils.jl
60.00%
2 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (9ce962f) and HEAD (ce26ffd). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (9ce962f)
HEAD (ce26ffd)

7
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #202       +/-   ##
===========================================
- Coverage   91.05%   37.37%   -53.69%
===========================================
  Files          47       46        -1
  Lines        2080     1806      -274
===========================================
- Hits         1894      675     -1219
- Misses        186     1131      +945

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix logstash declaration

Indentation
Add missing 'ports'

@amouat let me use this opportunity and thank you for a really great book!
'Using Docker' is a treasure of practical information, I just love reading it.

Thanks!


Remove single-element groups from GPUParticles2D

Inspired by #48385
Changes this

To this

Actually Collision group also has 1 property, but the two I moved are used every time, so it makes sense to make them more accessible.

Just to confirm because I am unsure, groups are not serialized when saving scenes right? In other words, this won't impact save files or break compatibility right?

Yes.

Thanks!


Event Source: Interesting idea of "everything through event source"

Idea is based on this:

one pipe to the server
a lot of requests could be going to it
requests will be collected and processed into event source (table whatever)
client will pick up that data

Maybe a bit complicated as request/response cycle will be split, so maybe it's not to go idea. At least try it.
With this data can go from whatever source, not just query mutations.
Aha! This can be that 0.13 leight upgrade...

Nope.


Base type in custom theme items can be set to any nonsensical value, but not left empty

Godot version
v4.2.beta4.official [93cdacbb0]
System information
Any platform
Issue description
See the discussion at #84259.
As of Godot 4.2, a custom theme item is required to have a base type defined. However, this base type can be any bogus name, as long as the field isn't left empty things will work.
The solution would either be to revert behaviour back to how it was in 4.1 (which I personally prefer), or to require users to fill in an existing class name that's valid for the items that the override should be applied to. In other words, if I make a custom container that inherits from Container, the only thing it is allowed to override in the theme is Container. This would require a major UX rework, because custom theme items are not directly linked to classes in the project.
Reverting this back to 4.1 has my preference because there is clearly no actual need for this field being filled out, other than having other tabs automatically filled with items that can be overridden (so it's editor UX, not engine functionality, why this requirement was added). If I have a custom item that has nothing to override I see no reason why I would still need to override anything.
Steps to reproduce
N/A
Minimal reproduction project
Not needed, but the project in #84259 is a good fit if you do want to check it out in the editor.

Or at the very least, if nothing changes in the functionality, a warning should be given to the user when a theme item is invalid.


how to handle wrong intent

Rasa version :13.04
Python version: 3.6.5
Operating system (windows, osx, ...):  Linux
Issue:
Can you suggest about rasa chatbot, how to handle an unknown question which enters by the user which would be not trained and getting random intent with it.

I am also facing the similar problem and looking for solutions...Although a threshold is set. Some user input with partial common parts with trained intents will easily get a confidence score higher than the threshold, also higher than completely new inputs.
The problem does not only occur in rasa nlu, but it is also a common issue for intent classification. One approach is to add out-of-scope examples, and you may need to add a variety of examples for every unclassified intent you met.

Please ask these kind of questions in our forum


Microphone Plugin don't work with Ionic on iOS

Hi,
I'm currently trying to use Wavesurfer in my Ionic setup and everything seems to work fine until I added the Microphone Plugin and I'm having a hard time figuring out why this breaks the below setup when build with Ionic and Xcode.
Wavesurfer.js version(s):
"wavesurfer.js": "^5.2.0"
Browser and operating system version(s):

Works on Macbook Pro M1 (all browsers)
Don't work when build with Ionic 6 (Vue 3) and Capacitor V3

Xcode Version 13.2.1
iOS Version 15.2.1 (real device)

Code needed to reproduce the issue:
<template>

</template>

have you searched wavesurfer.js tickets for other Ionic issues that could be related?

have you searched wavesurfer.js tickets for other Ionic issues that could be related?

@thijstriemstra yes I looked for similar issues, but I did not find anything related to this in the context of Ionic. Correct me if I'm wrong.


[FLINK-23950] Revert [FLINK-23738][state] to re-enable ChangelogBackend

What is the purpose of the change
This reverts commits:

d6bccd13190f0dbb2505b4512a38ae71b3d78195
b4c385e41832f16e39d5cbe4fb69ead9bbe077b2
67dd4932133d74d88666433d4d8aa4a6cbc94c78

Verifying this change
This change is a trivial rework without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? no

LGTM.


Ability to generate licence

The ability to gather the dependency (transient as well) license files running the ./gradlew dependencyLicenseReport task.
Will generate an index.html in build/reports/dependency-license will license information.

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/670/


HDDS-4325. Incompatible return codes from Ozone getconf -confKey

What changes were proposed in this pull request?
Make ozone getconf -confKey return error (255) instead of success (0) for missing key, to be compatible with Ozone 1.0.
https://issues.apache.org/jira/browse/HDDS-4325
How was this patch tested?
Manually verified that output and exit codes with patch:
$ ozone getconf -confKey asdf; echo $?
Configuration asdf is missing.
255
$ ozone getconf -confKey ozone.om.address; echo $?
om
0

match 1.0.0:
$ ozone getconf -confKey asdf; echo $?
Configuration asdf is missing.
255
$ ozone getconf -confKey ozone.om.address; echo $?
om
0

Added check for exit code in acceptance test case for missing key.  Also added test case for existing key.
https://github.com/adoroszlai/hadoop-ozone/actions/runs/295406415

Thanks @adoroszlai for working on this. The changes looks good to me.
+1

Thanks @umamaheswararao for reviewing and merging it.


bug: Integration test failing on main

Meltano Version
N/A
Python Version
NA
Bug scope
Other
Operating System
N/A
Description
Example failures:

https://github.com/meltano/meltano/actions/runs/3405847831/jobs/5664120687
https://github.com/meltano/meltano/actions/runs/3402614761/jobs/5659656060

Note that the test failed in different ways in the above 2 linked examples.
This is blocking PRs from being merged. For example:

https://github.com/meltano/meltano/pull/6961

Since the above PR clearly does not introduce any issues, and the integration test is failing on main, I'm going to bypass the requirement that the integration test passes before merging that PR.
Code
No response

Looks like we're getting other failures on main too: https://github.com/meltano/meltano/actions/runs/3405886679/jobs/5664185706
I'm not sure if these are related. I have not yet looked into them.


chore(main): release 1.1.1

:robot: I have created a release beep boop
1.1.1 (2024-10-22)
Bug Fixes

get secret in wf (6e9397e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sanriodev/blog.blvckleg.dev/releases/tag/v1.1.1 :sunflower:


firefox_profile option is ignored

Hi,
I'm trying to modify few firefox profile settings and it looks like it had been supported by https://github.com/HashNuke/hound/pull/94 but after metadata support landed, this option(firefox_profile) is now completly ignored and instead, hardcoded profile with ua agent is used( https://github.com/HashNuke/hound/blob/428403884dcb85e3103d5b4e94e35c5864b46c37/lib/hound/browser.ex#L20). There's no support for additional_capabilities (https://github.com/HashNuke/hound/blob/78bd8a08d786e75c1178927e2310b293f779c683/lib/hound/session.ex#L35).
I'll try to submit a PR fixing this.

fixed via https://github.com/HashNuke/hound/pull/174

I published a new patch version with your fix, thank you!

Great, thank you!


removing unused old Mtp1 code, fixes #596

Addresses #596
Removes IndShockConsumerType.Ex_Mtp1_over_Ex_Ptp1 and IndShockConsumerType.Ex_mtp1
This code crashes when executed and is neither documented nor up to style standards.

[x] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[x] Updated documentation of features that add new functionality.
[ ] Update CHANGELOG.md with major/minor changes.

Codecov Report
Base: 73.28% // Head: 73.34% // Increases project coverage by +0.06% :tada:

Coverage data is based on head (26a172b) compared to base (8313ec5).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1212      +/-   ##
==========================================
+ Coverage   73.28%   73.34%   +0.06%
==========================================
  Files          74       74
  Lines       12143    12130      -13
==========================================
- Hits         8899     8897       -2
+ Misses       3244     3233      -11

Impacted Files
Coverage Δ

HARK/ConsumptionSaving/ConsIndShockModel.py
87.24% <ø> (+0.84%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Change made to enable opm new flags --skip-tls-verify and --use-http as --skip-tls flag is deprecated.

Description of the change:
User is not able to run the operator-sdk run bundle in local kind-registry as --skip-tls flag is deprecated. OPM introduced new flags --skip-tls-verify and --use-http. This fix will enable the newly introduced flags.
Motivation for the change:
Users will be able to execute run bundle in local registries without requiring any certifications.
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[x] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

will add test cases for the --use-http flag as well.


simgrid: 3.23 -> 3.24

Motivation for this change
Keep nixpkgs's SimGrid up-to-date – I am a SimGrid maintainer.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers

@GrahamcOfBorg build simgrid


Failed CF template in China

Hi,
Has anyone tried this template in China; it keeps failing when I deploy it. Any solutions, please?

Solved with this: https://github.com/MikeSecure/security-hub-export-cn


File has unexpected size

VSCode Version: 1.44.1
OS Version: Ubuntu 19.10

Issue:
When running sudo apt update (used a fresh install to ensure minimal errors), receiving the following response from terminal :
Failed to fetch http://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  File has unexpected size (171612 != 171467). Mirror sync in progress? [IP: <IP_ADDRESS> 80]    Hashes of expected file:     - Filesize:171467 [weak]     - SHA512:f7d4a72db6bf2c81806cd506353b68da7d57bc69b72a01695ac44fc18494a6c17987c48cfcb806bbdef5966372d8c2550a8f0e7d9ceb54c270827dc362763263     - SHA256:1bea87fa36b58bc034d153c00cebf03c46303c6ef175e97df86bc63c6503ce91     - SHA1:d4ca05c3b67cf15caf07bf9c17dc18403292ac6b [weak]     - MD5Sum:ab7bb7a48f70850c05a4f647d3db704d [weak]    Release file created at: Tue, 14 Apr 2020 17:11:00 +0000 E: Some index files failed to download. They have been ignored, or old ones used instead.
Steps to Reproduce:

Run sudo apt update

Does this issue occur when all extensions are disabled?: Yes/No

Closed as duplicate.


[Legacy] [master] Most likely missing type

https://github.com/Microsoft/ChakraCore/wiki/Build-Status#legacy-builds
Broken since http://dotnet-ci.cloudapp.net/job/Microsoft_ChakraCore/job/master/job/daily_dev12_x64_debug/143/
See build log:
http://dotnet-ci.cloudapp.net/job/Microsoft_ChakraCore/job/master/job/daily_dev12_x64_debug/143/consoleFull#-447890337a82fefab-f698-416f-8fca-58544c94cd4e
WScriptJsrt.cpp(728): error C4430: missing type specifier - int assumed. Note: C++ does not support default-int [d:\j\workspace\daily_dev12_x---55cdd258\bin\ch\ch.vcxproj]
10:58:18 WScriptJsrt.cpp(728): error C2146: syntax error : missing ',' before identifier 'size' [d:\j\workspace\daily_dev12_x---55cdd258\bin\ch\ch.vcxproj]
...

See http://dotnet-ci.cloudapp.net/job/Microsoft_ChakraCore/job/master/job/daily_dev12_x64_debug/143/changes for commits between the last good build and the current broken build.
To repro locally please follow instructions here:
https://github.com/Microsoft/ChakraCore/wiki/Jenkins-Repro-Steps#legacy-builds-windows-only
When submitting a fix, remember to request legacy tests:
https://github.com/Microsoft/ChakraCore/wiki/Jenkins-Build-Triggers#legacy-builds

Tagging authors of the commits between last known good and the first failure: @antonmes @obastemur @Fishrock123 @agarwal-sandeep @mrkmarron @LouisLaf @EdMaurer @xanlpz
Can you all verify whether the break was caused by your changes?

probably not

Assigning @ThomsonTan to assess this issue.

Should be fixed by #1584 ?


gatling 3.0.3 compatibility

Hi,
I am working with gatling 3.0.3, and i find your solution very useful, do you plan to have a new version compatible to gatling 3.0.3 or is there another way that will help me to implement this kind of UI scenarios.
Thank you

I'm no longer using this day-to-day (I've changed employers since I wrote it), but I'm happy to accept pull requests if you can get it working on Gatling 3.0.3.


Fill md and mdctspecapp gaps

From #929
md
0 opinions for 2020. Missing 143 documents
mdctspecapp
Between November 1st, 2019 and January 27, 2021, we have 0 documents. Missing 114 documents from 2020, and around 30 from 2019.

Commands to fill the gaps:
docker exec -it cl-django python /opt/courtlistener/manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.md --backscrape-start=2020 --backscrape-end=2020

docker exec -it cl-django python /opt/courtlistener/manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.md --backscrape-start=2019 --backscrape-end=2021


example "scope" ]s mapping seemingly does nothing

Describe the bug
Simply put, the ]s, [s, ]S, [S mappings do nothing for me (no movement), when attempting to navigate around simple C and Go files (haven't tried others yet). Mappings like ]m, [m, ]M, [M do work.
I'm not sure if this problem is PEBKAC or not, but having seen locals.scm for Go/C/..., I suspect it isn't. The README gives this example:
["]s"] = { query = "@scope", query_group = "locals", desc = "Next scope" },

But there is no movement (see below). I've verified that nmap ]s shows me an anonymous Lua function is bound.
To Reproduce
-- reproducer.lua
vim.opt.runtimepath:append("$HOME/.vim/bundle/nvim-treesitter")
vim.opt.runtimepath:append("$HOME/.vim/bundle/nvim-treesitter-textobjects")

local treesitter_parsers = {
  "c",
  "cpp",
  "go",
  "lua",
}
require('nvim-treesitter.configs').setup({
  ensure_installed = treesitter_parsers,
  sync_install = false,
  highlight = {
    enable = true,
    additional_vim_regex_highlighting = false,
  },
  textobjects = {
    move = {
      enable = true,
      set_jumps = true, -- whether to set jumps in the jumplist
      goto_next_start = {
        [']m'] =<EMAIL_ADDRESS>        ["]s"] = { query = "@scope", query_group = "locals", desc = "Next scope" },
      },
      goto_next_end = {
        [']M'] =<EMAIL_ADDRESS>        ["]S"] = { query = "@scope", query_group = "locals", desc = "Next scope" },
      },
      goto_previous_start = {
        ['[m'] =<EMAIL_ADDRESS>        ["[s"] = { query = "@scope", query_group = "locals", desc = "Next scope" },
      },
      goto_previous_end = {
        ['[M'] =<EMAIL_ADDRESS>        ["[S"] = { query = "@scope", query_group = "locals", desc = "Next scope" },
      },
    },
  },
})

$ nvim -u reproducer.lua example.go

Expected behavior
I expected some movement to happen.v
Output of :checkhealth nvim-treesitter

Installation ~

WARNING tree-sitter executable not found (parser generator, only needed for :TSInstallFromGrammar, not required for :TSInstall)
WARNING node executable not found (only needed for :TSInstallFromGrammar, not required for :TSInstall)
OK git executable found.
OK cc executable found. Selected from { vim.NIL, "cc", "gcc", "clang", "cl", "zig" }
Version: cc (Debian 13.2.0-10) 13.2.0
OK Neovim was compiled with tree-sitter runtime ABI version 14 (required >=13). Parsers must be compatible with runtime ABI.

OS Info:
{
machine = "x86_64",
release = "6.6.13-amd64",
sysname = "Linux",
version = "#1 SMP PREEMPT_DYNAMIC Debian 6.6.13"
} ~
Parser/Features         H L F I J

bash                ✓ ✓ ✓ . ✓
c                   ✓ ✓ ✓ ✓ ✓
cpp                 ✓ ✓ ✓ ✓ ✓
go                  ✓ ✓ ✓ ✓ ✓
lua                 ✓ ✓ ✓ ✓ ✓
markdown            ✓ . ✓ ✓ ✓
markdown_inline     ✓ . . . ✓
python              ✓ ✓ ✓ ✓ ✓
query               ✓ ✓ ✓ ✓ ✓
sql                 ✓ . . ✓ ✓
starlark            ✓ ✓ ✓ ✓ ✓
vim                 ✓ ✓ ✓ . ✓
vimdoc              ✓ . . . ✓

Legend: H[ighlight], L[ocals], F[olds], I[ndents], In[j]ections
+) multiple parsers found, only one will be used
x) errors found in the query, try to run :TSUpdate {lang} ~

Output of nvim --version
$ nvim --version
NVIM v0.10.0-dev-2913+g5371ed36b
Build type: RelWithDebInfo
LuaJIT 2.1.1710088188
Run "nvim -V1 -v" for more info

nvim-treesitter: 1ba86026
nvim-treesitter-textobjects: 6e9bb56

Try this```lua
["]s"] = { query =<EMAIL_ADDRESS>query_group = "locals", desc = "Next scope" },

That actually does something, which is much better than before!
It's not exactly what I want, for example with this C code I first move to the first letter of the if, then to the first brace { when I use ]s. I would prefer just moving to the first brace:
    int c = 3;

    if (ptr == MAP_FAILED) {
/* 1                                      2 */
      perror("mmap");
      return 0;
    }

But it's already a lot better than before. Thanks.

This happens because for statement, if statement, while statement, function definition, etc are also defined to make their own scope in https://github.com/nvim-treesitter/nvim-treesitter/blob/4770d9a1a77b0cc2b723c646c3dbe43a9133e5db/queries/c/locals.scm#L58-L67
The work around is to copy that file to ~/.config/nvim/queries/c/locals.scm and then comment out the blocks you don't want to jump to.

So the query names in nvim-treesitter seems to have been changed. I guess we need to update the README.


Solution for Issue 57: Clicking Pause and then Play was not working. …

…Play did nothing. Now, for Android it resumes at a place before but near where it Paused. In UWP it restarts at the beginning. In iOS it works as expected.

Congrats on your first ever merged PR!


LCFS - Reformat In-App Notifications tied to Email Subscriptions into AG Grid Index format for BCeID and IDIR users

Description: Redesign in-app notifications linked to email notification subscriptions in the AG Grid Index format. The updated design should incorporate versions for both BCeID and IDIR users, including button functionality as specified in the provided wireframes.
BCeID Wireframe link: [BCeID Wireframe]
IDIR Wireframe link: [IDIR Wireframe]
Purpose and Benefit to the User: This design will improve notification clarity and interaction by standardizing in-app notifications in AG Grid Index format. Both BCeID and IDIR users will benefit from a unified and accessible notification experience that aligns with the email subscription updates.
Acceptance Criteria:

[ ] In-app notifications are redesigned in the AG Grid Index format.
[ ] Separate BCeID and IDIR versions are implemented with consistent styling and functionality.
[ ] Button functionality reflects the actions specified in the wireframes (e.g., check all, mark as read, delete and reset).
[ ] Cards are visually cohesive, with accessible typography, icons, and button placements.

Development Checklist:

[ ] Review and apply AG Grid Index format guidelines to the notification layout.
[ ] Customize design for BCeID and IDIR versions to ensure visual consistency.
[ ] Implement button actions in accordance with wireframe specifications.
[ ] Link wireframe prototype for team feedback and adjust based on usability testing.
[ ] When the "Check all" button has been pressed, replace button with "Uncheck all".
[ ] When nothing is checked, put "delete" button into inactive state.

Notes:

Verify accessibility compliance.
Ensure integration with existing notification and email subscription settings.

@StuartGALL wireframe links don't seem to be working

@airinggov Thanks Al, updated wireframes are working again


regex_replace_chars is not inherited by context (follow-up on #55)

I believe it didn't fix the problem. You forgot regex_replace_chars_or_context and as result this line is always setting default regex_chars.

I think it's ok.
regex_replace_chars                    = "${var.regex_replace_chars != "" ? var.regex_replace_chars : local.regex_replace_chars_context_or_default}"

will take it from the var if provided, or from context_or_default.
we don't need four local vars for this (as we do for name, namesoace, etc.) since we are not replacing the chars in regex_replace_chars

But the var is always provided. It has a default value.
On Monday, April 8, 2019, Andriy Knysh wrote:

I think it's ok.
regex_replace_chars                    = "${var.regex_replace_chars != "" ? var.regex_replace_chars : local.regex_replace_chars_context_or_default}"
will take it from the var if provided, or from context_or_default.
we don't need four local vars for this (as we do for name, namesoace, etc.) since we are not replacing the chars in regex_replace_chars
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.￼

--
Sent from my Jolla

See example:
module "base_label" {
  source              = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.7.0"
  namespace           = "com.dotlover"
  environment         = "staging"
  regex_replace_chars = "/[^a-zA-Z0-9-.]/"
  delimiter           = "."
}

module "dns_label" {
  source  = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.7.0"
  context = "${module.base_label.context}"
  name    = "dns"
}

output "base" {
  value = "${module.base_label.name}"
}

output "dns" {
  value = "${module.dns_label.name}"
}

It will output com.dotlover.staging and comdotlover.staging.dns (notice the dot missing). Is this how you meant it to work?

@rfvermut  thanks, you are correct.
regex_replace_chars is different from all other attributes (namespace, stage, name) because it always has the default value. This needs to be fixed. There are two solutions here:

When using context, the user needs to set regex_replace_chars = "" for the it to work (not a pretty solution for the user)

Since it's not possible to detect whether the regex_replace_chars value was used from default or was provided, we can introduce an internal local var regex_replace_chars_default and set it to the current default. Then in the module check if the provided value in the variable is different from the internal default; if different, use the provided value from the input; if the same, check and use the context

I am closing this as I think it has been fixed by #99 if not before. In any case, it does not reflect the current design. Please open a new issue if you still have this problem.


Nothing works

karma.conf.js:
var sysPath = require("path");
var root = __dirname;
var srcRoot = sysPath.join(root, "src");
var testsRoot = sysPath.join(root, "test");

module.exports = function (config) {
  config.set({
    singleRun: true,
    browsers: [ "PhantomJS" ],
    frameworks: [ "mocha", "chai-sinon" ],
    reporters: [ "mocha" ],
    files: [
      "webpack.config.test.js"
    ],
    preprocessors: {
      "webpack.config.test.js": [ "webpack", "sourcemap" ]
    },
    webpack: {
      devtool: "inline-source-map",
      resolve: {
        root: srcRoot
      },
      module: {
        loaders: [{
            test: /\.js$/,
            include: [ srcRoot, testsRoot ],
            loader: "babel"
          }]
      }
    }
  });
};

webpack.config.test.js:
require("es6-promise").polyfill();
var ctx = require.context("./test", true, /\.js$/);
ctx.keys().forEach(ctx);

test/index.js:
import "node-jsdom";
import "react";
...more imports...
// do some tests

With webpack config above, it tries to include jsdom and other deps in bundle, which is obviously not needed.
So I tried to use externals option: externals: [ /^[^\.]/ ] // everything non-relative.
Result:
START:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  SyntaxError: Invalid flags supplied to RegExp constructor.
  at /path/to/webpack.config.test.js:47 <- webpack:/external "/path/to/webpack.config.test.js":1:0

With externals: [ "node-jsdom" ]:
START:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  ReferenceError: Can't find variable: node
  at /path/to/webpack.config.test.js:4766 <- webpack:/external "node-jsdom":1:0

Well, it's because it uses "var" for externals by default.
But, with externals: [ { "node-jsdom": "commonjs node-jsdom" } ]:
START:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  ReferenceError: Can't find variable: require
  at /path/to/webpack.config.test.js:4766 <- webpack:/external "node-jsdom":1:0

With { output: { libraryTarget: "commonjs" } } and externals: [ "node-jsdom" ]:
START:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  ReferenceError: Can't find variable: exports
  at /path/to/webpack.config.test.js:1 <- null:0:0

With { output: { libraryTarget: "commonjs2" } } and externals: [ "node-jsdom" ]:
START:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  ReferenceError: Can't find variable: module
  at /path/to/webpack.config.test.js:1 <- null:0:0

This was predictable.
With target: "node" and externals: [ { "node-jsdom": "commonjs node-jsdom" } ], suddenly:
WARNING in ./~/es6-promise/dist/es6-promise.js
Module not found: Error: Cannot resolve module 'vertx' in /path/to/node_modules/es6-promise/dist
resolve module vertx in /path/to/node_modules/es6-promise/dist

START:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  ReferenceError: Can't find variable: require
  at /path/to/webpack.config.test.js:1073 <- webpack:/external "node-jsdom":1:0

I'm out of ideas. How to use it properly??

@jeron-diovis Are you trying to test the webpack.config.test.js file?
In karma config you're supposed to put the test file path in the files array, not the webpack config.


Issue #10189 fixed.

Issue #10189 fixed.

@ashishkumar-webkul sir all test has been passed.

Checked and Found that the issue has been fixed and working fine. Please check the video for reference.
Video
https://github.com/user-attachments/assets/4f768bb5-81f9-4c48-8f53-b1b208de90be


some confusion.

Hi, I am a bit confused about your code, and I would greatly appreciate it if you could answer my questions.

For example,near 150 in the ope. py file . What is the purpose of the for loop in the above code? Why do we need to perform three layer operations on the output and put each result output into outputs? What is the difference between output and outputs. Only one layer can obtain processed num_objects=3 prototypes？
If it's convenient, I hope to receive your reply as soon as possible. Thank you

In the above image, the value 27 of the 0th dimension in "appearance" already includes the prototype of all num_objects? Because it is obtained from self. num_objects * self. kernel_dim * * 2.

So the shape of "all_prototypes" obtained from the ope. py function is (3,27,1,256), and what does 3 in the 0th dimension mean? num_objects? But isn't num_objects included in 27 in the first dimension? I don't understand.

Hi @Aurora-zgz, thank you for your question.
As you correctly noted, as single IterativeAdaptationLayer can handle an arbitrary number of objects. However, to achieve iterative adaptation, we stack a number (in this case 3) of such layers to create the IterativeAdaptationModule. The iteration you mention goes over layers and not over objects, so the line output = layer(output) just says that the input to the next layer is the output of the previous layer. You can see the exact same thing in the transformer.py file, lines 32-36. This is the same as packing your layers in the nn.Sequential object and not using the loop.
Why the outputs list? Basically, we just gather the output of every layer in this list. We need this because we apply auxiliary losses to intermediate layers (see more details in the paper). Therefore, the 0th dimension in all_prototypes refers to the number of layers in the IterativeAdaptationModule (it's probably a bit confusing because both the number of objects and adaptation layers is 3 but try changing one and you will see the difference).
Let me know if this helps.

In addition, I also found that when visualizing the loca_few_shot.pt generated by reproducing the code, the quantity counted each time is different, and the difference is significant. In contrast, the pre trained loca_few_shot.pt you provided does not have this issue and the quantity is relatively accurate.

@djukicn Regarding Val COCO and Test COCO, I would like to know how they were obtained. Is it possible to directly extract Val COCO and Test COCO from FSC147 for verification and testing? What is the specific experimental process like? I don't quite understand according to the description in the paper. I would greatly appreciate it if you could provide me with a timely answer.


Non-descript exception thrown for Sylvan.Data.Csv

A specific line in a file continues to throw this exception, I cannot gleam any more data or reason as to why it's throwing this exception.
I unfortunately cannot share the CSV line as it contains confidential information, but I was hoping someone might have an idea as to how I can debug further?
Exception of type 'Sylvan.Data.Csv.CsvFormatException' was thrown
at Sylvan.Data.Csv.CsvDataReader.ReadField(Int32 fieldIdx)
at Sylvan.Data.Csv.CsvDataReader.d__6.MoveNext()
at Sylvan.Data.Csv.CsvDataReader.d__8.MoveNext()
at SomeFileReader.d__3.MoveNext() in

I was able to locate the issue, in the 8th ordinal of my file, it was a quoted text field with a quote inside it, like so:
0,1,2,3,4,5,6,7,"this is the "example" of the quoted text field"

Those quotes would need to be escaped (doubled up) to make it a valid CSV file.

Indeed, do you have any suggestions as to how I could detect this specific issue in files, so I may fail specifically and gracefully?
The CsvFormatException didn't provide any specific failure.
Thanks for an awesome library!


Is maplibre-gl 3.x not supported?

I couldn't help but notice the peer dependency for maplibre-gl only goes up to 2.x. Is that intentional, or is it just out of date? Is it safe to upgrade our app to use maplibre-gl 3.x?

@ffxsam I'm using 3.6.2 with an overrides entry in package.json. It works for my simple requirements at the moment, so it may be fine for you as well.
It would be nice to know what is happening with this library as you haven't received any other response and maplibre-gl is at 4.1.2 now.


How to use @ApiParam for GET request with SpringMVC?

We know that SpringMVC can package parameters to an Object with an GET request.
I use swagger's @ApiParam for the Request Object, but it was an Request Body not normal query parameters.
public ResponseEntity query(@ApiParam ApiRequest request) { ...... }

This Request Object has many properties, I used @ApiModelProperty for the Object.

OK thanks

fixed by #671
The properties of the ModelAttribute need to be annotated as @ApiParam however, since they will later be normal query parameters in the doc.

Sorry, not particularly clear, as in the question ousui, I expected custom types to generate interface documentation as parameters, but now I can only generate documentation as body
eg：
public ResponseEntity query(@ApiParam ApiRequest request) { ...... }
expect:

but actual：

大概是请求对象内部属性需要 @ApiParam 注解。


API for pulling staking ledger data properly + supercharging

We should fix the existing endpoint #4825
In addition, we can make staking info available via the archive node. Details TBD.

As some additional background, I added a few fields to the dumped staking ledger and exposed via a GraphQL API for use in developing some staking pool tooling. What I found most useful was:

Ability to filter the query by delegate
Adding the timed/untimed calculation for a delegate so you know if it will be a supercharged reward. In the data I am testing with all accounts are now untimed so timed_in_epoch is false but I added an untimed_slot as I guess it is possible to become untimed during the epoch.

As an example the following request shows that there are no timed balances in the delegations to the following address:
{
  stakes(
    query: {delegate: "B62qrKMAUagWp1VJhvb1AA6y1G4P4tr3inYyPqDRq1VyprA1wmVthcV"}
  ) {
    public_key
    balance
    timing {
      cliff_amount
      cliff_time
      initial_minimum_balance
      vesting_increment
      vesting_period
      untimed_slot
      timed_in_epoch
    }
  }
}

returns:
{
  "data": {
    "stakes": [
      {
        "balance": 65949.4,
        "public_key": "B62qrAoKfgQx8oXMVQauRs9WYzXpCmYqQrLxivgc8qMYVkK9eH9GjjF",
        "timing": null
      },
      {
        "balance": 71771.5149999,
        "public_key": "B62qrKMAUagWp1VJhvb1AA6y1G4P4tr3inYyPqDRq1VyprA1wmVthcV",
        "timing": null
      },
      {
        "balance": 65958.8001,
        "public_key": "B62qoDaiV18ZGhLfb3j2SuDQEhjyM7c8sDppmvpAyE3cXjutPtHLBfA",
        "timing": {
          "cliff_amount": 12000,
          "cliff_time": 150,
          "initial_minimum_balance": 60000,
          "timed_epoch_end": null,
          "timed_in_epoch": false,
          "untimed_slot": 2070,
          "vesting_increment": 150,
          "vesting_period": 6
        }
      }
    ]
  }
}

Available via:
curl -d '{"query":"{\n  stakes(\n    query: {delegate: \"B62qrKMAUagWp1VJhvb1AA6y1G4P4tr3inYyPqDRq1VyprA1wmVthcV\"}\n  ) {\n    public_key\n    balance\n    timing {\n      cliff_amount\n      cliff_time\n      initial_minimum_balance\n      vesting_increment\n      vesting_period\n      untimed_slot\n      timed_in_epoch\n      timed_epoch_end\n    }\n  }\n}\n"}' -H 'Content-Type: application/json' https://graphql.minaexplorer.com

As some additional background, I added a few fields to the dumped staking ledger and exposed via a GraphQL API for use in developing some staking pool tooling. What I found most useful was:

Ability to filter the query by delegate
Adding the timed/untimed calculation for a delegate so you know if it will be a supercharged reward. In the data I am testing with all accounts are now untimed so timed_in_epoch is false but I added an untimed_slot as I guess it is possible to become untimed during the epoch.

As an example the following request shows that there are no timed balances in the delegations to the following address:
{
  stakes(
    query: {delegate: "B62qrKMAUagWp1VJhvb1AA6y1G4P4tr3inYyPqDRq1VyprA1wmVthcV"}
  ) {
    public_key
    balance
    timing {
      cliff_amount
      cliff_time
      initial_minimum_balance
      vesting_increment
      vesting_period
      untimed_slot
      timed_in_epoch
    }
  }
}

returns:
{
  "data": {
    "stakes": [
      {
        "balance": 65949.4,
        "public_key": "B62qrAoKfgQx8oXMVQauRs9WYzXpCmYqQrLxivgc8qMYVkK9eH9GjjF",
        "timing": null
      },
      {
        "balance": 71771.5149999,
        "public_key": "B62qrKMAUagWp1VJhvb1AA6y1G4P4tr3inYyPqDRq1VyprA1wmVthcV",
        "timing": null
      },
      {
        "balance": 65958.8001,
        "public_key": "B62qoDaiV18ZGhLfb3j2SuDQEhjyM7c8sDppmvpAyE3cXjutPtHLBfA",
        "timing": {
          "cliff_amount": 12000,
          "cliff_time": 150,
          "initial_minimum_balance": 60000,
          "timed_epoch_end": null,
          "timed_in_epoch": false,
          "untimed_slot": 2070,
          "vesting_increment": 150,
          "vesting_period": 6
        }
      }
    ]
  }
}

Available via:
curl -d '{"query":"{\n  stakes(\n    query: {delegate: \"B62qrKMAUagWp1VJhvb1AA6y1G4P4tr3inYyPqDRq1VyprA1wmVthcV\"}\n  ) {\n    public_key\n    balance\n    timing {\n      cliff_amount\n      cliff_time\n      initial_minimum_balance\n      vesting_increment\n      vesting_period\n      untimed_slot\n      timed_in_epoch\n      timed_epoch_end\n    }\n  }\n}\n"}' -H 'Content-Type: application/json' https://graphql.minaexplorer.com

@garethtdavies and others -- if we don't fix the existing endpoint for mainnet, but provide an example archive node query to get all this information for any arbitrary epoch in the past (so strictly more powerful than this graphql endpoint was able to do)

cc @garethtdavies -- latest design is reflected in the body of the issue. Unfortunately, there are some nuances in getting the data into the archive node properly.

For item (6), the ledger export subcommand uses RPC to grab the entire ledger, as a list of accounts, from the daemon. So the ledger is read from the db before sending it over RPC.
We could add something like ledger export-local, that connects directly to the db, and reads and converts to JSON entries lazily. The exported ledger wouldn't contain anything in masks or any other uncommitted entries. We could also make a separate app for this purpose.

https://docs.minaexplorer.com/minaexplorer/calculating-payments
states We will validate this on the testnet and collaborate on a script for all pool operators to use (waiting on this issue)
but the issue has been closed: https://github.com/MinaProtocol/mina/issues/7737
is this tool available now?
Also, @bkase#2492 & @garethtdavies#4963, Tezos has a very mature tool for calculating payments to delegations (the standard, every validator uses it). While the protocol and actual calculations obviously differ, it might be a good a idea to take a look at some of the options in this tool. Worth mentioning:
a) you can set different rates for different delegators (to for example, calculate with 0% fee the service for partners in the validator service, lower fees for larger delegators, etc. the use cases are plenty)
b) one does not need to keep an archive node, there are three back ends for calculations: Tezos RPC, tzstats API and TzKT API. Naturally these APIs are fed by an archival node, but they allow for higher level queries of the chain, which is useful for developers looking to build on top of the chain.
I can also send you some sample output of how this tool runs if you find this would be useful.


Suggestion for v3 ngrok container

I run ngrok (2.x at the moment) in docker on a Synology NAS.   I do not have easy command line access to docker as Synology has it's own web GUI for it.
So running the ngrok config upgrade can become a 'chicken & egg' problem for some people.
What might be useful is to be able to add an environment variable to the container that would instruct it to start, perform an upgrade of the config and either continue running with the upgraded config or stop (instead of just stopping with an error).     (A lot of us in simple environments also have containers set to auto-restart which was quite ugly when Watchtower upgraded my ngrok 2 to ngrok 3 at 4am a few days ago).
Between portainer and setting up SSH access, I can technically get to a docker command prompt to maybe get it to run on a command but I don't have any command line docker experience so it's not something I'm in a rush to do and I figure some other users may be in the same boat, so some kind of method to get the container to do the upgrade for us would be nice to have.
Just a thought, thanks for reading.

Thanks for bringing this up @LeeThompson. Can you help me understand how you manage your config? I'm not familiar with Watchtower. The config upgrade is generally pretty simple, so I'm wondering if pointing to the changelog would be more expedient? The number of changes for v2->v3 is pretty limited (most folks will get away with just adding version).

If it's that easy then yeah a changelog or simple how to manually update/prepare for v3 would probably be fine.
I'll try to go over our config...
My wife and I run a number of containers on our Synology (a dozen or so), I do small coding projects (mostly for myself) and my wife has technical interests too.  We use ngrok to make it easy to access some of the containers we run from the net over TLS, one to access our ebooks etc.  That kind of thing.
Watchtower is a container that performs container updates  which I like because updating a container to a new version is a little tedious, especially if you have many of them.    It checks to see if any of the container's tags have been updated in the registry, if so it pulls the new image, exports your existing container, deletes it (I do have data/configs mapped to the host volume), re-creates it using the new image and starts the updated container.   You can even schedule when it does this.
Of course, this can backfire if a container update is rolled back or requirements change or something in the configuration needs updated or an automatic upgrade script goes wrong.   I have been stung by this myself a few times, but overall it's a real time saver.

Ah makes sense! Take a gander at the changelog linked and let us know if you have any questions. You're also welcome to write into<EMAIL_ADDRESS>for one of those folks to help, but I'll leave this open until you confirm it was "easy enough".


[Feature/Extension] OBO Authenticator should inspect the issuer field and make sure it matches the current cluster

Description
OBO Authenticator should inspect the issuer field and make sure it matches the current cluster.

Closing this one due to the merge of #3117.


README is not explicit about depending on the Replicated SDK

Had a discussion today where it wasn't clear to everyone that this depends on the Replicated SDK. Need to document that dependency.

Fixed #11


Update: added binary versions of the command line interface for windows, macos and linux

Fixes #185
Updated four files

package.json
package.lock.json
.gitignore
.npmignore

@Issue-03 Thank you, excellent job! 🎉 👍 😄


[beta 2.027] new [Dragon Scale] Upgrade bugged

Dragon scale upgdrade what is it:
an upgrade that is unclocked by petting your dragon (new heavenly upgrade)
Expected Behaviour:
be caclulated as coockie upgrade (simply increase CPS)
Bug#1;

NOTE: bug 1 does not always happen
Bug#2;
Once bought it causes all Building (not upgrades)  PP calculations to go haywire and they get negative values
Temporary Fix for those expiriencing this bug
Game.UpgradesById[648].bought = 0;
this is will "unbuy" the upgrade (no coockie refunds) after witch i recomend putting in your vault
Other info
The Dev version of coockie monster 2.022.2 was used, the beta 2.026.1 crashes the game

Thanks for the heads up.

This bug persists in 2.028

I made a mistake in the original post purchusing [Dragon Scale] does also Break the math on PP caclucations for upgrades (thought it doesn't make them go negative)

Bug#2 is also caused by unlocking the Milkhelp lactose intolerance relief tables prestige upgrade. You can remove the upgrade using Game.PrestigeUpgrades[85].lose().

We will now use #317 to discuss any missing upgrades or items as a result of CookieMonster not being updated for CookeClicker 2.031.


[New Feature] New interactive panel for playground in docs

🚀 New interactive panel for playground in docs

Feature Description
The display of playground doesn't let users interact. All the colors and options are displayed rather than user selection.
Adding a interactive panel like in radix ui playground would make it much more better .

from https://www.radix-ui.com/themes/playground

It can be sidebar which can be toggled to open or closed

Not specifically like this.
https://github.com/user-attachments/assets/acf922b2-cf64-4bfb-8744-688f297f4d78
more like this and instead of using key to toggle maybe a button