Bug Fix - Support for Windows and Unix-base Systems End Of Line Characters

Clean Keys from End of Line Characters
This PR fixes a new bug that was detected when using docker to run our application in Windows. The bug was related to the parsing of our public key. When Running TI in docker, in a Windows environment, the key had \r\n characters do to the way Windows end of line characters work.
Solution
Normalize key from \r\n to \n and then clean the key from \n.
How to Test

run ./gradlew shadowJar
run docker compose up --build to spin up the containers
request an access token to our auth endpoint
make sure request is successful
stop containers by running docker compose down
run ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./gradlew e2e-execute.sh

Issue
Checklist

[ ] I have added tests to cover my changes
[ ] I have added logging where useful (with appropriate log level)
[ ] I have added JavaDocs where required
[ ] I have updated the documentation accordingly

Note: You may remove items that are not applicable

@jorg3lopez
Just a note for testing this PR on Mac:
⚠ I got this error message when running:
./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./gradlew e2e-execute.sh
Error
FAILURE: Build failed with an exception.
* What went wrong:
Task 'e2e-execute.sh' not found in root project 'trusted-intermediary' and its subprojects.

✅ But, if I run this ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && cd .. && ./gradlew e2e-execute.sh all the test pass.

@jorg3lopez
Just a note for testing this PR on Mac:
⚠ I got this error message when running: ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./gradlew e2e-execute.sh
Error
FAILURE: Build failed with an exception.
* What went wrong:
Task 'e2e-execute.sh' not found in root project 'trusted-intermediary' and its subprojects.

✅ But, if I run this ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && cd .. && ./gradlew e2e-execute.sh all the test pass.

In Windows, the first version fails, and I get a bash: ./gradlew: No such file or directory when I run the second.  This works ok for me, though: ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./e2e-execute.sh

Just a note for testing this PR on Mac:
⚠ I got this error message when running: ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./gradlew e2e-execute.sh
Error
FAILURE: Build failed with an exception.
* What went wrong:
Task 'e2e-execute.sh' not found in root project 'trusted-intermediary' and its subprojects.

✅ But, if I run this ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && cd .. && ./gradlew e2e-execute.sh all the test pass.

In Windows, the first version fails, and I get a bash: ./gradlew: No such file or directory when I run the second. This works ok for me, though: ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./e2e-execute.sh

Using the command that Joel used I no longer get directory errors, but I do get errors in MetadataTest. See below:
./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./e2e-execute.sh:
BUILD SUCCESSFUL in 23s
15 actionable tasks: 15 executed
~/Repos/FlexionRepos/trusted-intermediary/trusted-intermediary/app ~/Repos/FlexionRepos/trusted-intermediary/trusted-intermediary
Starting API
API starting at PID 32074
~/Repos/FlexionRepos/trusted-intermediary/trusted-intermediary
Waiting for API to respond
API is responding
Running the end-to-end tests
To honour the JVM settings for this build a single-use Daemon process will be forked. For more on this, please refer to https://docs.gradle.org/8.9/userguide/gradle_daemon.html#sec:disabling_the_daemon in the Gradle documentation.
Daemon will be stopped at the end of the build

> Task :e2e:test

ConsolidatedSummaryTest > a consolidated summary is returned from the ETOR consolidated summary endpoint FAILED
    org.spockframework.runtime.ConditionFailedWithExceptionError at ConsolidatedSummaryTest.groovy:35
        Caused by: groovy.lang.MissingPropertyException at ConsolidatedSummaryTest.groovy:35

MetadataTest > a metadata response is returned from the ETOR metadata endpoint FAILED
    java.lang.NullPointerException at MetadataTest.groovy:37

MetadataTest > a metadata response is returned from the ETOR metadata endpoint for results FAILED
    java.lang.NullPointerException at MetadataTest.groovy:81

MetadataTest > a 404 is returned when there is no metadata for a given ID FAILED
    org.spockframework.runtime.SpockComparisonFailure at MetadataTest.groovy:116

MetadataTest > linked id for the corresponding message is included when retrieving linked metadata FAILED
    java.lang.NullPointerException at MetadataTest.groovy:155

MetadataTest > a metadata order is not linked when the corresponding linking fields do not match FAILED
    java.lang.NullPointerException at MetadataTest.groovy:194

24 tests completed, 6 failed

> Task :e2e:test FAILED

@jorg3lopez
Just a note for testing this PR on Mac:
⚠ I got this error message when running: ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && ./gradlew e2e-execute.sh
Error
FAILURE: Build failed with an exception.
* What went wrong:
Task 'e2e-execute.sh' not found in root project 'trusted-intermediary' and its subprojects.

✅ But, if I run this ./gradlew :app:test && ./gradlew :shared:test && ./gradlew :etor:test && cd .. && ./gradlew e2e-execute.sh all the test pass.

Good catch, it should be ./e2e-execute.sh. I will edit the PR

All unsigned commits were put into one signed commit and then force pushed.

Completed test using docker.


MatlabReader gives Arithmetic overflow reading simple double matlab array

In F# 4.1 (visual f# tools 10.1), .net Standard 2.0, visual studio 15.7.1 community;
in FSI reading matlab file with a single matrix (17196 x 6) with max value 1.15 e+10 gives arithmetic overflow exception despite easily fitting in a double.
. .mat file loads fine in matlab (r2016a). Happens with some other matrices as well, but not all.
Is this a bug, or what am I doing wrong?
test data:
sp500.zip
[code]
// read the first matrix as double
let filePath  = System.IO.Path.Combine( __ SOURCE_DIRECTORY __,"../path/to/sp500.mat")
let m = MatlabReader.Read(filePath);
[/code]

I think the issue is MatlabReader appears to read .mat v7 files, but not v7.3 files (the default since 2006??)
... and the documentation says it reads level 5 files, so I guess it is my JRTFM.
It would be intuitive though that it should be able to read a file format that been around for 12 years now...

Yes, I'd love to support Matlab HDF5 files. Unfortunately HDF5 seems to be much more complicated than the old format, so we either need an implementation (which I don't have time to contribute myself) or an existing library we can use.

I understand - here someone nicely created an f# hdf5 lib:
https://rodhern.wordpress.com/2015/03/15/how-to-use-the-hdf5-type-provider/
https://rodhern.wordpress.com/2015/04/18/how-i-compile-the-hdf5-type-provider/
these leverage the dlls from the hdfgroup.org
It would be great if that was incorporated into matlab reader, but if not now hopefully this comment will help others...


Video sound starts playing when loading Sandcastle in IE and Edge

If you go to Sandcastle by entering an address (like http://localhost:8080/Apps/Sandcastle/index.html?src=Hello World.html&label=Showcases) rather than through the link on http://localhost:8080/, the sound from the Video sandcastle starts to play even when its not on that demo.
This only happens in IE and Edge.

@lilleyse I have tried the https://www.cesiumjs.org/Cesium/Apps/Sandcastle/?src=Hello World.html&label=Showcases with the following browser versions and it seems that the issue has gone.
Edge: 40.15063.0.0
IE: 11.540.15063.0

Thanks @bampakoa I confirmed that the issue is gone too.

@lilleyse I noticed that if you refresh the page, then the audio starts 😕

It seems ok for me, I'll try on another computer later.

Seems like this issue is still open with Edge 41.16299.15.0 and InPrivate mode (Ctrl + P)


Paloma js false throwing js error

Using js false when rendering a page will throw this error.

Uncaught TypeError: Cannot read property 'controller' of null

(With turbolinks) Using js false when rendering a page will execute the previously executed Paloma action/controller.

solved as of version 4.1.2


Added lanyards and badges. Default pricing

See email from Mary.

@inglesp did I commit this to the wrong branch?

Yes, I think so -- at least, the pull request should be made against master.


[HUDI-6108] Alter table should not include metadata fields in the dat…

…a schema
Change Logs
The write schema should never include metadata fields.
Impact
none
Risk level (write none, low medium or high below)
none
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

The failed test: https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=16489&view=logs&j=3b6e910d-b98f-5de6-b9cb-1e5ff571f5de&t=30b5aae4-0ea0-5566-42d0-febf71a7061a&l=652759 should be be caused by this patch, would merge it soon ~


two JWT schemes: Exception doesnt parse needed auth config

I have 2 types of users and auth systems respectfully: client lgged in by phone number and unique device id, and operator, who work on orders, received from clients. Operators should log in with email/password or maybe username/password. More classical. How can I define two JWT schemes? Or I should write my own implementation for one of schemes? (or copy from adonis auth middleware?

Yo. Actually it is kind of working. I just copy-pasted jwt scheme in auth config. And used it like auth.authenticator('sms').attempt(phone, uuid).
All tests pass excluding one, checking for error when attempt is failed. (eg wrong phone provided.
Look at my code: config/auth.js
authenticator: 'jwt', (default authenticator)
...
sms: {
    serializer: 'lucid',
    model: 'App/Models/Client',
    scheme: 'jwt',
    uid: 'phone',
    password: 'uuid',
    options: {
      secret: 'self::app.appKey',
    },
  },

  jwt: {
    serializer: 'lucid',
    model: 'App/Models/User',
    scheme: 'jwt',
    uid: 'email',
    password: 'password',
    options: {
      secret: 'self::app.appKey',
    },
  },

And in controller:
  async login({ request, response, auth }) {
    const { phone, uuid } = request.all()
    return auth.authenticator('sms').attempt(phone, uuid)
  }

Despite I stated to use my sms authenticator, error handler just takes dafault config and generates error from its uid/password fields. I think it is error.
https://github.com/adonisjs/adonis-auth/blob/046610960bddfa20a6f256922fd458a0bda5e3d0/src/Exceptions/index.js#L44

Yup seems like a bug 🐛

Mind checking it from the develop branch of github and see if it reports with right errors?

Thanks. Will try a bit later, maybe weekend. Now quite busy.


feat!(core): simplify and improve ergonomics of LogFilter

Description

filter block range now defaults to latest block,
addresses are no longer appendable, but are replaced on each "address" setter call (same as topics)
topics are now stored as array of hashes rather than Topic sealed class
additional helper methods to enable calling filter setters with lists of values

Solution

#92  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArtificialPB and the rest of your teammates on  Graphite


[TOPI] Numpy consistency: always broadcast binary ops

So far we have mixed support for broadcasting ops in topi. For example broadcast_add, broadcast_sub are broadcasting ops, while ops like left_shift, right_shift are somewhat restricted. We would like to improve numpy consistency and brings full support to broadcasting to binary ops as numpy did. Here are a few items that I would like to propose to make this happen.
This is a non-backward compatible change in API.

[x] remove all broadcast_xxx API as we broadcast implicitly by default

broadcast_add -> add
broadcast_sub -> subtract
broadcast_mul -> multiply
broadcast_div -> divide.
Hopefully most things are fine as operator overloading works as intended.

[x] Support simple marcro to define broadcast ops.
[ ] Redirect broadcast ops to cpp implementations
[ ] Upgrade all the existing binary ops to broadcast ops.

Note that these changes are only happening at TOPI level, for now. The nnvm operators will be kept as the same name.

cc @tmoreau89 @kevinthesun


The exe file in EXE folder can't run.

当我在win11系统上双击EXE文件夹下的exe文件，我遇到了错误，首先是缺少一些dll文件，在我下载dll文件之后，我遇到了这样的错误：应用程序无法正常启动（0x000007b）。请单击“确定”关闭应用程序
When I double-click the EXE file under the exe folder on win11 system, I encounter errors, first of all some dll files are missing, after I download the dll file, I encounter this error: the application does not start properly (0x000007b). Click OK to close the application

well, now the problem was addressed. I need to edit the start file under the EXE folder to specify the path of .obj file.


Signalrclient's portfile.cmake is outdated. Now cpprestsdk's version information is 2.10.0

The cpprest_2_9.lib should be updated to cpprest_2_10.lib:
vcpkg_configure_cmake(
SOURCE_PATH ${SOURCE_PATH}
OPTIONS_DEBUG
-DCPPREST_SO=${CURRENT_INSTALLED_DIR}/debug/lib/cpprest_2_9.lib
OPTIONS_RELEASE
-DCPPREST_SO=${CURRENT_INSTALLED_DIR}/lib/cpprest_2_9.lib
OPTIONS
-DCPPREST_INCLUDE_DIR=${CURRENT_INSTALLED_DIR}/include
-DDISABLE_TESTS=ON
)

Fixed in master. Thanks for reporting this issue!


Give repositories a priority order

Repositories should be able to have a priority order to get cloned. This should allows users to override a widget.
Repositories will have to get cloned by the reverse way of the priority order to override the widgets.
The priority order should be defined at the repository creation and edition. It could be updated by performing a drag & drop on the list of repositories in the repositories page.
Issue type
[] Bug
[] Improvment
[] Documentation
[x] Feature

Currently the repositories priority order isn't clear, when we add two (or more) repositories, we don't know which one is taken into account first, and they're not always displayed in the same order we added them
This should be fixed to something like the creation date and displayed the same way in the UI (and optionally being modified from the UI)
This should be made clear for the users so they don't have interrogations about the priority order anymore


Increased memory use when using bin/jobs to start Solid Queue?

I'm working on a project that was using Solid Queue v0.7.0 and using bundle exec rake solid_queue:start to start the process. When I updated to v1.0.0 last week, I also changed the command to start Solid Queue to use bin/jobs per current documentation.
I noticed that the memory use of the associated Heroku app was greatly increased. I wasn't sure if this was due to updating the gem or changing the start command, so I tried changing back the command from bin/jobs to the Rake task and have noticed a decrease in memory use, per screenshot below.

I read in the discussion on #343 that using bin/jobs is supposed to help with memory use thanks to eager loading, so I'm a bit puzzled by this situation.

Huh, that's indeed puzzling! May I ask what's your Solid Queue configuration? And whether you have config.rake_eager_load set to true by any chance?

Here's my Solid Queue configuration:
# config/queue.yml
default: &default
  dispatchers:
    - polling_interval: 1
      batch_size: 500
  workers:
    - queues: [ login, background, default ]
      threads: 3
      processes: 1
      polling_interval: 0.1

development:
 <<: *default

test:
 <<: *default

staging:
 <<: *default

production:
 <<: *default

# config/recurring.yml
one:
  class: JobOne
  schedule: every 5 minutes
two:
  class: JobTwo
  schedule: every 75 minutes
solid_queue_cleanup:
  command: "SolidQueue::Job.clear_finished_in_batches"
  schedule: every day at 3 am
  queue: background

I don't have config.rake_eager_load set at all so it defaults to false I believe.

Thanks so much for sending that.

I don't have config.rake_eager_load set at all so it defaults to false I believe.

Yes, that's right. I'm quite puzzled in any case, about this behaviour... But I wonder if, in your case, eager loading is causing that extra memory to be used in the beginning 😕  Would you be up to do a quick test if you can do that without running into trouble with your Heroku instance? That'd be setting
config.rake_eager_load = true

and then checking to see the memory usage invoking Solid Queue with bundle exec rake solid_queue:start.

Here's what I'm seeing after setting config.rake_eager_load = true an hour ago:

It looks like it's using a bit more memory than without eager loading, but it's still not as much as using bin/jobs.
(On the graph there's a brief period where we're using bin/jobs again due to an automatic deployment.)


Revamp upgrade

Need to find a way to upgrade live clusters with minimum disruption, e.g. no message loss.

@brunomcustodio you were supposed to create this ticket and add the info we discuss.

@pires I was waiting for rework to be merged in order to do that 😇 Also, I now need some input about pires/kubernetes-nats-cluster#9.
I will fully describe the problem once I can get my hands on my notebook (and also run some more tests).

Ping me when the issue in the other repo is done.

Hi @bmcstdio  and @pires,
Is it safe to upgrade a nats-operator image nowadays?
I've just forked this repo and modified it a little bit to add a second container running nats-exporter for Prometheus.
Now I need to update the operator image, but I'm not sure about data consistency during the process.
Can I update its image and delete every nats pod, one by one, until cluster get fully synced again?

Currently only NATS server is supported and it is stateless, so there would be a few disconnects during the process but in general apps using NATS would have retry logic anyway so it should be safe to upgrade as clients would eventually reconnect to a node in the cluster.

There's no guarantee messages will not get lost, though.

Thx @wallyqs and @pires!
Are you guys interested in adding a prometheus-exporter container within nats-pod managed by the operator?
I forked this repo to add this feature, but perhaps it may be useful for more people. I don't know.

@igorvpcleao that sounds great, feel free to submit the PR and will take a look 👍

Fixed by #105.


Readability improvements for delivery-and-retry.md

Addresses minor grammar nits; improves readability; ensures consistency of word usage (e.g. the dash between "dead" and "letter" or "time", "to", and "live").

@sixlettervariables : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add Linux arm64 support

Adds support for Linux arm64 in the wasm-pack npm package.
Fixes: #1169
Make sure these boxes are checked! 📦✅

[x] You have the latest version of rustfmt installed

$ rustup component add rustfmt

[x] You ran cargo fmt on the code base before submitting
[x] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

Hmm. Doesn't seem like the test failures are related to the PR changes.

Hope this gets merged soon

Hi @drager 👋
Any estimations on when this fix gets released? 👼  Thanks in advance 🙏


arch-riscv: adding stats to show completed page walks

This commit adds statistics showing completed page walks for 4KB and 2MB pages. This will add to stats.txt the variables num_4kb_walks, num_2mb_walks and the corresponding values. This is done based on the level of page table walk traversed specific to Sv39 Virtual Memory System.

Hi @aryanitish,
None of the commits in this pull request contains a Change-ID, which we require for any changes made to gem5.   Thank you.


unknown unstable option: gcc-ld error in checks

checks fail with this error, failing the checks of every pr which makes us unable to see if the pr itself is correct, holding us back from merging prs

This was fixed in https://github.com/twilight-rs/http-proxy/pull/79.


Added jupyternotebook and dataset for ML experimentation demo

Added ML experimentation demo for structured data

can you also add an entry to the root README in the repo too please

@rmoff - TODO: Add link to the blog once it is up on the website.


Do not use the singleton of AbortedStreamException

Motivation:
AbortedStreamException.get() returns the singleton or new instance depending on the Flags.verboseExceptionSampler()'s decision.
However, if there's a problem in certain circumstances, it is very hard to find the cause with the singleton.
Especially, this one. https://github.com/openzipkin/zipkin/issues/3197#event-3908775954
Modification:

Changed to created AbortedStreamException instances whenvere get() is called.

Result:

You now get the stack trace of AbortedStreamExcepetion every time.

Had a chat with the team and this can cause performance degradation. So let me close this. 🤣


Added example on how to use RandomNumberGenerator

I don't know how to merge this PR with squashed commits into #30010. I deleted the fork which was used to create #30010 and and re-forked the project to create this PR (because I didn't know what was going on tbh). But I just want to update the PR of #30010. Any pointers on how to achieve this @akien-mga ?

Since you deleted the fork used to make #30010, it can no longer be updated, so we'll have to continue here :)

Sweet, thanks a bunch!

It's almost good to merge, there's just one issue left, which is that the commit log is "Updated based on suggestions" when it should likely be "Added example on how to use RandomNumberGenerator". You can change it with git commit --amend.

Thanks! I amended it and then force pushed, but is that the correct way of updating the commit log?

Yes, that's all good now :)

Thanks!

Cherry-picked for 3.1.2.


How to write version info to _version.py file?

Thank you developers for making life easier with this package. No nonsense about future dev/ post string in the final name while creating the package. This is what I was exactly looking for.
However, I wonder if I can write the version info in a separate file, so that version same version info is available in both development and installed mode. I am just looking at the option write_to = "package-name_version.py" being offered by setuptools_scm package. And solution or idea to make it possible is welcome.
Thank you.

Hi, thank you ☺.
One of ideas this package is based on -  make no changes in the repo while calculating a version number. Writing generated version to a file requires either adding all the changes of this file into commit history (which causes merge conflicts), or adding this file to gitignore (no conflicts, but this requires to generate the file manually in a local repo).
That's why there is no such option in this project, and I hope it never be.
If you really need this, you can generate version number and export it to a file using bash redirect:
python setup.py --version > ./VERSION

But probably you need to look at this on another angle, for example to store version number in the file and then use it as a source for a package version:

https://setuptools-git-versioning.readthedocs.io/en/stable/schemas/file/version_file.html
https://setuptools-git-versioning.readthedocs.io/en/stable/options/count_commits.html

Logical and fair enough.  I think this should be part of the main documentation in your site as it clears done fundemental question.
The problem I am facing here is that redirecting the version number at the time of GitHub release process makes the version to change from x.x.x to x.x.x.post. Is this also intended by the package? This causes the pypi build process to fail due to naming convention. Here is the simple release action workflow i am using https://github.com/wxguy/wrfplot/blob/main/.github/workflows/release-pip.yaml.

This is probably caused by the fact that file path you're saving current version to is not in the .gitignore. This leads to creating uncommitted changes in the local copy of a repo in Github runner, and thus setuptools-git-versioning generates not a release version but a dirty version:
https://setuptools-git-versioning.readthedocs.io/en/stable/schemas/tag/dirty_version.html


slow loading times when accessing bigger chats

After playing around with the snapshot linked in the older version of the article and the newest version we noticed that the performance with "bigger" chats (already starting at about 20 messages) is a lot worse with the current version. When clicking on a chat to open the view freezes. The more messages, the longer until it then slides to the actual conversation view (already a few full seconds for more than 20 messages). There are also other things where the app laggs for some time, such as when adding people to chats or just navigating the app. Looking at the console output, that mostly occurs when there is a Disconnection and Reconnection for no apparent reason:
2022-02-27 18:23:00.413438+0100 RChat[10194:2895762] Sync: Connection[3]: Disconnected
2022-02-27 18:23:01.012754+0100 RChat[10194:2895762] Sync: Connection[4]: Session[4]: client_reset_config = false, Realm exists = true, client reset = false
2022-02-27 18:23:01.039700+0100 RChat[10194:2895762] Sync: Connected to endpoint 'xxxxx:443' (from 'xxxxx:49665')

With the linked commit the loading time was almost instant.
Could this be due to the new way of passing the .environment(...)?
.environment(\.realmConfiguration, app.currentUser!.configuration(partitionValue: "all-users=all-the-users"))

Thanks for Your help!

Update: After finding a post of someone experiencing a similar issue I rolled back from 10.21.1 to 10.21.0 which completely fixed the issue. Loading times in this version are almost instantaneous.


(BOLT-717) Add BoltSpec::Run public helper

This adds a new set of BoltSpec spec helpers that can be used to run
tasks, commands, and plans from tests without handling cli flags or
running in a separate process.

CLA signed by all contributors.

Merge when passing.


collect2: error: ld returned 1 exit status Makefile:1079: recipe for target 'logging_unittest' faile

collect2: error: ld returned 1 exit status Makefile:1079: recipe for target 'logging_unittest' faile

Isn't this a duplicate of #240?


ASR TTS Merge into one apk

I can't even download TTS.
https://github.com/k2-fsa/sherpa-onnx/releases --Only ASR ! Where's TTS?

I can't even download TTS.
https://github.com/k2-fsa/sherpa-onnx/releases --Only ASR ! Where's TTS?

Please read our README.

Please see
https://k2-fsa.github.io/sherpa/onnx/tts/apk-engine.html

The title is different from the text on this issue, as pointed by others TTS is in a different apk, also different for each language and each voice at the moment.
Google have choose to bring TTS and STT/ASR in one apk, since (probably) use similar or same voice database assets.
This maybe will be interesting in the future, but depend on the developers view, if merge TTS and STT/ASR will lead to a more complex code for gain about only 70/100MiB of free space... probably the developers should choose the best choice for their perspective. Honestly I will love to have all in one app, but compared to other issue I don't think that is a priority. Consider that STT/ASR have not implemented Recognition Service as I cannot see it on voice input option, and don't support ACTION_RECOGNIZE_SPEECH since I cannot use it from my keyboard.

The code is shared but the models are different.
Also, you can install asr apk, tts apk, and speaker identification apk simultaneously on your phone.

Also, you can install asr apk, tts apk, and speaker identification apk simultaneously on your phone.

Sure, but in that case there is a minimum of 3 differebt app that will be update to stores #520 (fdroid, google play, xiaomi, huawei, samsung, amazon, ecc.).
Only one app is simpler only for that, but will complex code maintain and development.
Also, as proposed by @mablue on #569 maybe the icons will be different TTS, STT and identification with similar sherpa logo but still different.
I can do that, but I need some confirmations:
Licence of the original logo(apache? Creative commons?);
Would be you (and other developers) like that idea?
Also if there is 3 different apps, maybe would be a good idea to separate the repository (but still on the official k2-fsa), since in the future will be more issues about an app specific bug and not all sherpa onnx code.
What do you (and others) think aboit that?

Sherpa tts download page is not accesable for blinds.
Also them need a sherpa-onnx telegram group to connect directly with developers

I think first problem fixed by @jing332 client for sherpa but I cant use it. Its still not work. I dont know why but it have not voice in all phones!! But its very good in ui and managing voices and langs! Its advance but still not like tts server https://github.com/jing332/SherpaOnnxTtsEngineAndroid

This is a free software, writed by community of people around the world (and maybe xiaomi if I'm not wrong), you cannot pretend nothing, especially to have fully functional app in zero days.

And ASR functionality for persian still not available

You have opened #559, have you tained a persian model?

I'm will learn icefall ...I didnt start reading icefall.
I haven't started learning icefall yet. But I am interested. Ganjur site is a good source for Farsi texts and sounds

I'm will learn icefall ...I didnt start reading icefall. I haven't started learning icefall yet. But I am interested. Ganjoor site is a good source for Farsi texts and sounds

@mablue this is totally off-topic, but I think is simpler to you use an already used dataset, such as common voice, that have persian language and is supported by icefall

@csukuangfj we are going offtopic, but you can respond about the logo question? Since merge all in one apk does not have reason (and so this issue can be closed), what do you think about make different logos fot TTS, ASR/STT and speaker identification?

what do you think about make different logos fot TTS, ASR/STT and speaker identification

Yes, that sounds good to me. Would you like to contribute?

Also, as proposed by @mablue on #569 maybe the icons will be different TTS, STT and identification with similar sherpa logo but still different. I can do that, but I need some confirmations: Licence of the original logo(apache? Creative commons?); Would be you (and other developers) like that idea?
Also if there is 3 different apps, maybe would be a good idea to separate the repository (but still on the official k2-fsa), since in the future will be more issues about an app specific bug and not all sherpa onnx code.
@csukuangfj What do you (and others) think about that?

@csukuangfj probably you have missed the text 4 days ago, yes, I would like to contribute, but please confirm licence of the original logo (since I have to modify it mixing with other image).

please confirm licence of the original logo

The logo is from us and we are publishing all of our work with Apache 2.0 license.

Since merge all in one apk does not have reason

Why? We generating 20 gigabits of repeating binary and etc in each apk file!
The reson that nobody can do anything to this project is that there is not any all in one small just java binary client. And everything is Scattered. Me and many Developers waiting to have just one client and many models to generate what we want with one clinet. If we cant make all in one apk for all  tts models it will be a failed project. Cuz no one understands what and where should start the job of updating and etc.
I think we should continue to @jing332 project.
I love to do someting but internet in iran is a trash 🗑

@mablue
merge ASR/STT, TTS and speaker identification is TOTALLY different to have all the languages in a TTS apk.
Please read carefully this issue

@mablue merge ASR/STT, TTS and speaker identification is TOTALLY different to have all the languages in a TTS apk. Please read carefully this issue

If we cant merge all just merge tts. Tts can save lifes. Blinds are using it all around the world. Some people in iran have not google tts in his language cuze of senctions.

follow the document，asr and tts merge into one apk is no difficult thing

follow the document，asr and tts merge into onine apk is no difficult thing。 the one app will keep two or more model files In assert directory.

It's not a hard job yes. some thing like this:
https://github.com/jing332/SherpaOnnxTtsEngineAndroid
It's just merge of all tts in one onnx model loader!
But the problem is switching between models while reading a (persian-english) mixed text


y-axis tooltips wrongly positioned

Copying from https://github.com/apexcharts/apexcharts.js/issues/2556
I'm assuming the author closed my previous issue requesting to re-open that issue because it didn't follow the template? If they had left a comment instead of just closing it, I wouldn't have to assume. Here I've copy pasted the original person's issue, since the author apparently does not entertain requests to re-open existing issues.  I felt it was reasonable to assume you'd prefer that over just filing duplicate issues. Without communication, I'm not quite sure if you just don't want to hear about the bugs or something.

Bug report
Codepen
CodePen example of issue: https://codepen.io/benben_ik/pen/WNjmWOb
Explanation

What is the behavior you expect?
When crosshairs are enabled for grouped charts: Only the Y-axis tooltip of the chart a user is hovering is shown on a grouped chart. I do not expect to see Y-axis tooltips of both the chart I'm hovering and Y-axis tooltips of other grouped charts that do not belong to the chart I'm hovering.
What is happening instead?
Instructions (see codepen): Create two or more grouped charts and enable crosshairs and Y-Axis tooltips on each chart. This results in all Y-Axis showing all Y-Axis tooltips of all grouped charts on each Y-Axis when hovering one of the charts. This overlap makes Y-Axis tooltips unreadable and confusing. It seems that the Y-axis crosshair keeps on drawing outside its frame.
Example:

What error message are you getting?
There is no error message, there is just unexpected behaviour.

Thank you for your time.

I'm assuming the author closed my previous issue requesting to re-open that issue because it didn't follow the template?

You would be correct. You may have had that thought due to the instructions in the issue template.
If an issue needs to be reopened, please request it on the issue itself. Do not recreate another issue just pointing to the original. That would have been preferred. I will reopen the older issue, but in the future please continue to conversation on the original issue and do not recreate old issues. I'm closing this one to avoid the duplicate.


Documentation errors

Describe the bug
These are not code bugs but documentation errors -- which I consider bugs

[ ] inverse() : "compose(a,inverse(a)) = a" should be "compose(a,inverse(a)) = identity"

[ ] local_coordinates() :
"Implementation is simply to_tangent(between(this, b)). Tangent space perturbation that conceptually represents “this - a”."
should be
"Implementation is simply to_tangent(between(this, b)). Tangent space perturbation that conceptually represents “this - b”."
also, I think it should be
"Implementation is simply to_tangent(between(this, b)). Tangent space perturbation that conceptually represents “b - this”."
also, between() does not appear listed in the list of available methods, so the doc above is confusing.

To Reproduce
Not relevant
Expected behavior
Not relevant
Screenshots

SymForce webpage documentation on Aug 26, 2022

Thank you for pointing this out @joansola!


Router with nested component - documentation incomplete?

UI5 developer guide mentions, it is possible to nest components with each component having its own (sub)routes.
https://openui5.hana.ondemand.com/#docs/guide/2c66fb3eb775426087401117cfa6a94e.html
I think it is a great feature, helping to modularize bigger applications. Or it were great, if I knew how to use it.
The documentation mentions, the child route should be declared with parent attribute and nothing else.
What we have is the following:

parent component

child1
child2

In the parent we define two routes that lead to two views. One view has component container and embeds component child1, the other has container with child2.
The path /child1 opens the view with child1 component.
Child1 has its own routes that refer to parent. We expect the path /child1/view1 to trigger the /view1 route in child1 component and display the respective view.
This does not work, as the router in the parent component tries to handle the complete path and it obviously does not have such a path defined.
I am wondering, how it should know that there are some paths defined in the child components.
Should we somehow refer to the child components in manifest of the parent component? Is is correct to use component container as described above?
I saw similar question on Stack Overflow (https://stackoverflow.com/questions/39754764/sapui5-routing-between-different-components). Since it has not been answered there for over half a year and I could not find anything else on this topic, I suppose the knowledge, how to achieve this is generally missing. Therefore, I am asking my questions here, in hope the topic could be cleared by some UI5 insiders.

Hi rasmk,
thanks for your explanation. I have an imagination of what the problem could be, but it is hard to say without the corresponding code. Could you please share your configuration, or at least a simpified version of it?
I have prepared a jsbin, which shows the example in our documentation. You may just modify it to have an equivalent to your own config: https://jsbin.com/zifugihiho/edit?html,output
Best regards
Benedikt

Has anyone managed to find any further information on this issue? I'm trying to implement the same sort of thing but getting nowhere at the moment. I can't seem to find any practical examples.
I've create a repository where I'm attempting to have a play around but so far no had any joy.
(https://github.com/bawder/ui5nestedcomponents)

Is there a proper way to navigate back to the parent component?
For the moment I am using event Bus between to send an event to parent so that it can handle the routing.

I have a (sort of) working demo here if you are still interested in this.

I also struggled with this scenario, and eventually came up with my own alternative:
https://blogs.sap.com/2019/08/23/ui5-nested-routing-with-reusable-components/
What freaks me out, is that the standard supported way does not work propertly, and even requires you to reference the parent route in the subcomponent! A subcomponent must be parent-agnostic! That's just bad software architecture.

I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards, Diana

Hi @FiddleBe and @rasmk ,
UI5 developed a new feature called "Routing in Nested Components". See https://openui5.hana.ondemand.com/topic/fb19f501b16e4e4991eb6a017770945b. This feature should provide the power you need for your requirements.
If you have any further comments or improvement proposals, don't hesitate to get in touch with us by opening a new issue.
All the best,
Florian


Warn on function returning unknown with no return statement

TypeScript Version:  3.1.0-dev.20180721
Code
function f(): unknown { ""; }
function g(): string | undefined { ""; }

Expected behavior:
Both f and g are errors.
Actual behavior:
Only g is an error.

This issue seems to not be fixed yet:
function empty(): unknown { // Error
}
function returnNothing(): unknown { // no error
    return;
}
let anon: () => unknown = () => {}; // no error

Expected behavior:
Defining a function of type () => unknown without a return statement in the body should not result in an error, since void is assignable to unknown.
Actual behavior:

Error: A function whose declared type is neither 'void' nor 'any' must return a value.

Playground Link: playground link

Adding another example that shows what I feel is also weird behavior ... if you simply add a return statement in any code path, the error goes away, even though for all the other code paths nothing has actually changed.
function errors(a: boolean): unknown {
    if (a) {
    }
}

function doesNotError(a: boolean): unknown {
    if (a) {
        return;
    }
}

Playground Link

@RyanCavanaugh  I want to pick this up but it is not really clear to me what the behavior should be.
The original suggestion was to have an error on functions returning unknown.
In @chharvey 's example he mentions the he actually expects to not have an error when creating a function that returns unknown since void is assignable to unknown.

Defining a function of type () => unknown without a return statement in the body should not result in an error, since void is assignable to unknown.

The one bit of inconsistency here IMO is that  let anon: () => unknown = () => {}; is not raise an error. Maybe we could issue an error in this case as well although I am not sure that this is in the scope of this bug.
@jthomerson 's example behaves exactly as a function returning undefined does
function errors(a: boolean): undefined {
    if (a) {
    }
}

function doesNotError(a: boolean): undefined {
    if (a) {
        return;
    }
}

So I don't think changing  anything for unknown is in the scope of this issue. Maybe we should consider forcing a return statement on all branches, even if the return type contains undefined although that would be a breaking change and a different discussion that was probably already had.

Is there any workaround for this? Something such as excluding void from unknown in some way?

Excluding void from unknown is a bad idea IMO, since unknown is supposed to be the “top type” or “universal type” — it is a supertype of every other type. That includes void. If you’re looking for way to annotate a function that returns a non-undefined value, try Object for any value including primitives, or object for any non-primitive value.
function returnSomeValue(): Object {
   return 42;             // returning a primitive is fine
   return {};             // returning a POJO is fine
   return new Array();    // returning a class instance is fine
   return;                // ERROR: `undefined` is not assignable to `Object`
   // no return statement // ERROR: must return a value
}
function returnSomeNonPrimitiveValue(): object {
   return 42;             // ERROR: primitive is not assignable to `object`
   return {};             // returning a POJO is fine
   return new Array();    // returning a class instance is fine
   return;                // ERROR: `undefined` is not assignable to `Object`
   // no return statement // ERROR: must return a value
}

(POJO = plain old javascript object)

I think the desirable behavior would be that you're allowed to implicitly return from a function if the annotated return type is assignable from undefined, and that noImplicitReturns controls whether or not you're allowed to implicitly return

These are unified now


Cannot create custom expressions

Describe the bug
When you create a custom expression then reload the skript, skript cancels the reloading of the file and throws a huge error in console
To reproduce

Create a skript file
use the code listed below:

expression glowing %item%:
	return type: item
	get:
		set {_item} to expr-1
		if {_item} is a fishing rod:
			enchant {_item} with infinity 1
		else:
			enchant {_item} with lure 1
		return {_item} with hide enchants item flag

reload the file

Expected behavior
Its meant to give items the shiny/glowing effect.
Error
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! [Skript] Severe Error:
[08:50:43 ERROR]: #!#! Exception occurred in Skript's main command
[08:50:43 ERROR]: #!#! Used command: /sk reload /main/api.sk
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! Something went horribly wrong with Skript.
[08:50:43 ERROR]: #!#! This issue is NOT your fault! You probably can't fix it yourself, either.
[08:50:43 ERROR]: #!#! It looks like you are using some plugin(s) that alter how Skript works (addons).
[08:50:43 ERROR]: #!#! Here is full list of them:
[08:50:43 ERROR]: #!#! skript-placeholders v1.6.0 (https://github.com/APickledWalrus/skript-placeholders) skript-citizens v1.0.0 skript-reflect v2.4 (https://github.com/SkriptLang/skript-reflect) skript-gui v1.3 (https://github.com/APickledWalrus/skript-gui) DiSky v4.12.2-beta1 SkBee v3.4.1 (https://github.com/ShaneBeee/SkBee) SkLogs v1.1.1 (https://github.com/EquipableMC/SkLogs)
[08:50:43 ERROR]: #!#! We could not identify which of those are specially related, so this might also be Skript issue.
[08:50:43 ERROR]: #!#! You should try disabling those plugins one by one, trying to find which one causes it.
[08:50:43 ERROR]: #!#! If the error doesn't disappear even after disabling all listed plugins, it is probably Skript issue.
[08:50:43 ERROR]: #!#! In that case, you will be given instruction on how should you report it.
[08:50:43 ERROR]: #!#! On the other hand, if the error disappears when disabling some plugin, report it to author of that plugin.
[08:50:43 ERROR]: #!#! Only if the author tells you to do so, report it to Skript's issue tracker.
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! Stack trace:
[08:50:43 ERROR]: #!#! java.lang.NullPointerException: Cannot invoke "java.util.Map.remove(Object)" because "syntaxes" is null
[08:50:43 ERROR]: #!#!     at skript-reflect.jar//org.skriptlang.reflect.syntax.CustomSyntaxStructure.lambda$unload$1(CustomSyntaxStructure.java:158)
[08:50:43 ERROR]: #!#!     at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
[08:50:43 ERROR]: #!#!     at skript-reflect.jar//org.skriptlang.reflect.syntax.CustomSyntaxStructure.unload(CustomSyntaxStructure.java:153)
[08:50:43 ERROR]: #!#!     at Skript (5).jar//ch.njol.skript.ScriptLoader.unloadScripts(ScriptLoader.java:830)
[08:50:43 ERROR]: #!#!     at Skript (5).jar//ch.njol.skript.ScriptLoader.unloadScript(ScriptLoader.java:864)
[08:50:43 ERROR]: #!#!     at Skript (5).jar//ch.njol.skript.SkriptCommand.onCommand(SkriptCommand.java:189)
[08:50:43 ERROR]: #!#!     at org.bukkit.command.PluginCommand.execute(PluginCommand.java:45)
[08:50:43 ERROR]: #!#!     at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:155)
[08:50:43 ERROR]: #!#!     at org.bukkit.craftbukkit.v1_19_R3.CraftServer.dispatchCommand(CraftServer.java:929)
[08:50:43 ERROR]: #!#!     at org.bukkit.craftbukkit.v1_19_R3.command.BukkitCommandWrapper.run(BukkitCommandWrapper.java:64)
[08:50:43 ERROR]: #!#!     at com.mojang.brigadier.CommandDispatcher.execute(CommandDispatcher.java:264)
[08:50:43 ERROR]: #!#!     at net.minecraft.commands.CommandDispatcher.performCommand(CommandDispatcher.java:322)
[08:50:43 ERROR]: #!#!     at net.minecraft.commands.CommandDispatcher.a(CommandDispatcher.java:306)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.network.PlayerConnection.a(PlayerConnection.java:2297)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.network.PlayerConnection.lambda$handleChatCommand$20(PlayerConnection.java:2257)
[08:50:43 ERROR]: #!#!     at net.minecraft.util.thread.IAsyncTaskHandler.b(IAsyncTaskHandler.java:59)
[08:50:43 ERROR]: #!#!     at java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1768)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.TickTask.run(TickTask.java:18)
[08:50:43 ERROR]: #!#!     at net.minecraft.util.thread.IAsyncTaskHandler.d(IAsyncTaskHandler.java:153)
[08:50:43 ERROR]: #!#!     at net.minecraft.util.thread.IAsyncTaskHandlerReentrant.d(IAsyncTaskHandlerReentrant.java:24)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.MinecraftServer.b(MinecraftServer.java:1342)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.MinecraftServer.d(MinecraftServer.java:197)
[08:50:43 ERROR]: #!#!     at net.minecraft.util.thread.IAsyncTaskHandler.x(IAsyncTaskHandler.java:126)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.MinecraftServer.bi(MinecraftServer.java:1319)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.MinecraftServer.x(MinecraftServer.java:1312)
[08:50:43 ERROR]: #!#!     at net.minecraft.util.thread.IAsyncTaskHandler.c(IAsyncTaskHandler.java:136)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.MinecraftServer.i_(MinecraftServer.java:1290)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:1178)
[08:50:43 ERROR]: #!#!     at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:320)
[08:50:43 ERROR]: #!#!     at java.base/java.lang.Thread.run(Thread.java:840)
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! Version Information:
[08:50:43 ERROR]: #!#!   Skript: 2.8.3 (latest)
[08:50:43 ERROR]: #!#!     Flavor: skriptlang-github
[08:50:43 ERROR]: #!#!     Date: 16:23:04.254419600
[08:50:43 ERROR]: #!#!   Bukkit: 1.19.4-R0.1-SNAPSHOT
[08:50:43 ERROR]: #!#!   Minecraft: 1.19.4
[08:50:43 ERROR]: #!#!   Java: 17.0.10 (OpenJDK 64-Bit Server VM 17.0.10+7)
[08:50:43 ERROR]: #!#!   OS: Linux amd64 5.15.0-91-generic
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! Server platform: Paper
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! Current node: null
[08:50:43 ERROR]: #!#! Current item: null
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! Thread: Server thread
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! Language: english
[08:50:43 ERROR]: #!#! Link parse mode: DISABLED
[08:50:43 ERROR]: #!#!
[08:50:43 ERROR]: #!#! End of Error.
[08:50:43 ERROR]: #!#!
Server information

skript-reflect: 2.4
Skript: 2.8.3
Bukkit 1.19.4-R0.1-SNAPSHOT
Minecraft: 1.19.4
Java: 17.0.10 (OpenJDK 64-Bit Server VM 17.0.10+7)
OS: Linux amd64 5.15.0-91-generic

Additional context
Skript info:
[08:58:21 INFO]: [Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[08:58:21 INFO]: [Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[08:58:21 INFO]: [Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[08:58:21 INFO]: [Skript] Server Version: git-Paper-550 (MC: 1.19.4)
[08:58:21 INFO]: [Skript] Skript Version: 2.8.3 (skriptlang-github)
[08:58:21 INFO]: [Skript] Installed Skript Addons:
[08:58:21 INFO]: [Skript]  - skript-placeholders v1.6.0 (https://github.com/APickledWalrus/skript-placeholders)
[08:58:21 INFO]: [Skript]  - skript-gui v1.3 (https://github.com/APickledWalrus/skript-gui)
[08:58:21 INFO]: [Skript]  - DiSky v4.12.2-beta1
[08:58:21 INFO]: [Skript]  - skript-reflect v2.4 (https://github.com/SkriptLang/skript-reflect)
[08:58:21 INFO]: [Skript]  - skript-citizens v1.0.0
[08:58:21 INFO]: [Skript]  - SkBee v3.4.1 (https://github.com/ShaneBeee/SkBee)
[08:58:21 INFO]: [Skript]  - SkLogs v1.1.1 (https://github.com/EquipableMC/SkLogs)
[08:58:21 INFO]: [Skript] Installed dependencies:
[08:58:21 INFO]: [Skript]  - Vault v1.7.3-b131

I was wondering if there is a nightly build I can use that fixes this until a new version is released?

Are you sure it's not a duplicate of #86? I assume it may be a duplicate of an addon's expression. Try to change glowing to gloowing and see if it works.

Are you sure it's not a duplicate of #86? I assume it may be a duplicate of an addon's expression. Try to change glowing to gloowing and see if it works.

No addons have glowing that I have installed. Skbee I know doesn't have it, skript-citizens is a self build of nylhus (modified to support 1.19+)

Are you sure it's not a duplicate of #86? I assume it may be a duplicate of an addon's expression. Try to change glowing to gloowing and see if it works.

No addons have glowing that I have installed. Skbee I know doesn't have it, skript-citizens is a self build of nylhus's skript-citizens addon plugin (it's modified to support 1.19+)

Still try to change glowing to something unique. Maybe one of your other scripts have the same custom expression?

Are you sure it's not a duplicate of #86? I assume it may be a duplicate of an addon's expression. Try to change glowing to gloowing and see if it works.

No addons have glowing that I have installed. Skbee I know doesn't have it, skript-citizens is a self build of nylhus's skript-citizens addon plugin (it's modified to support 1.19+)

Still try to change glowing to something unique. Maybe one of your other scripts have the same custom expression?

None of them do, I only have 3 skript files. One for a scoreboard, one for my discord bot, and one for functions and custom expressions (where that glowing expression is located at)

I'll still try glooowing right now though ^^

weird, it no longer errors

it happens when you have the same custom syntax declared more than once (even in another script). If you can reproduce the error without that condition, please leave a comment with the scripts to reproduce


Tax Agent functionality

Is there any Tax Agent functionality available in the new sandbox?

Hi @andyws
At the moment, you are able to simulate Tax Agent authorisation and subscription check failures using the Gov-Test-Scenario headers AGENT_NOT_AUTHORIZED and AGENT_NOT_SUBSCRIBED.
We do not currently implement any other agents functionality.
Thanks,
Jacob

Hi @GlassAndOneHalf,  I'm curious because I've been following the 3/4 Agent projects on Github (where no question is ever answered), attended presentations to Agents and received a recent presentation from SDS.  It would appear that somewhere someone has access to some Agency functionality ... I wonder where?


RFD - Programmatic Loki access

Status
Draft 🚧 / Open for comments 💬/ Accepted ✅ /Implemented 🚀/ Obsolete 🗃

Author(s)
@tylergraff

Date Created
2024-09-06

Date Last updated
dd-MM-YYY

Decision deadline
dd-MM-YYY

Title
Documentation Demonstrating Loki Programmatic Access
Summary
I propose adding documentation to this repo which demonstrates how to programmatically access Loki logs from within Nebari (e.g. from a Nebari jupyter notebook). I have linked a PR in the comments which adds this documentation via an example Python program.
Shout-out to @aktech for creating the implementation; I only cleaned it up a bit and re-validated that it works.

User benefit
Users can use this example to create their own log analysis and introspection capabilities. These capabilities can help Nebari app creators and users debug their own apps, and can also help Nebari administrators and advanced users debug Nebari itself.
Release notes headline could be: Added example demonstrating programmatic introspection of Nebari logs.

Design Proposal
I propose updating nebari-docs to include a section on Programmatic Log Access, along with a working Python example implementation (user must set their own Nebari URL and Grafana token). In the comments I have linked my PR which adds this documentation.

Alternatives or approaches considered (if any)

Best practices

User impact
Documentation change only, no user-facing functionality changes.

Unresolved questions

PR implementing the above proposal:
https://github.com/nebari-dev/nebari-docs/pull/518

Hi @tylergraff, I think adding these docs are reasonable.  RFDs are put in the https://github.com/nebari-dev/governance repo.  BTW, RFDs are typically only for major changes that would affect a majority of end users as described in https://www.nebari.dev/docs/community/decision-making#major-updates---request-for-discussion-rfd-process
Updating docs usually doesn't require an RFD.  We don't want to make contributors go through the hassle of an RFD for smaller changes.  Thanks for opening the issue!


Assign product id to micro:bit

[x] assign product id (@pelikhan)
[ ] take picture (@mmoskal )
[ ] add to device catalogue (@jamesadevine)

product id: 0x32f6253d

Added to https://github.com/microsoft/pxt-jacdac

I can't add the micro:bit via the device registration page. Should I just do it manually?

Manually is fine.

Added to device catalog.

@mmoskal might want to take a picture with the standardized lightning


vs-code-prettier-eslint does not work with yarn 3 PnP

If vs-code-prettier-eslint is used with projects managed by yarn 3 in PnP mode, the extension does not work. Here is the output:
Error: Cannot find module 'eslint'
Require stack:
- c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js
- c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\loader.js
- c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\bootstrap-amd.js
- c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\bootstrap-fork.js
-
Error: Cannot find module 'eslint'
Require stack:
- c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js
- c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\loader.js
- c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\bootstrap-amd.js
- c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\bootstrap-fork.js
-
	at Module._resolveFilename (node:internal/modules/cjs/loader:987:15)
	at i._resolveFilename (node:electron/js2c/utility_init:17:840)
	at p.resolve (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\loader.js:4:761)
	at n4 (c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:204:36)
	at Lge (c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:180:599)
	at k0 (c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:204:6948)
	at c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:204:7802
	at Generator.next (<anonymous>)
	at c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:1:796
	at new Promise (<anonymous>)
	at M0 (c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:1:616)
	at gye (c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:204:7597)
	at Object.provideDocumentRangeFormattingEdits (c:\Users\dev\.vscode\extensions\rvest.vs-code-prettier-eslint-5.0.4\dist\extension.js:205:74)
	at F.provideDocumentRangeFormattingEdits (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:96:39981)
	at c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:96:61976
	at se.s (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:96:56458)
	at se.$provideDocumentRangeFormattingEdits (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:96:61963)
	at u.N (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:104:11208)
	at u.M (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:104:10926)
	at u.H (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:104:9981)
	at u.G (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:104:9000)
	at c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:104:7788
	at d.invoke (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:145)
	at w.deliver (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:2029)
	at g.fire (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:1667)
	at h.fire (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:72:14314)
	at c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:120:15804
	at d.invoke (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:145)
	at w.deliver (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:2029)
	at g.fire (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:1667)
	at h.fire (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:72:14314)
	at MessagePortMain.<anonymous> (c:\Users\dev\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:120:13960)
	at MessagePortMain.emit (node:events:526:28)
	at MessagePortMain._internalPort.emit (node:electron/js2c/utility_init:5:364)

Versions:

VS Code Prettier ESLint extension 5.0.4
Visual Studio Code: 1.74.0
Node: 18.12.1
Package Manager: yarn 3.3.0
prettier: 2.8.1
eslint: 8.29.0

System Specifications:

OS: Windows 10.0.19045 x64 Enterprise
Processor: Intel i7 6700HQ @ 2.6GHz
RAM Size: 32 GB

@yarick123 I'll take a look. I'm pretty sure I'll have to bundle ESLint and Prettier with the extension and I'd rather not do that. Would installing them globally work?

@yarick123 I think this may do the trick. Run the following inside your project directory:
yarn dlx @yarnpkg/pnpify --sdk vscode
See https://github.com/prettier/prettier-vscode/issues/1502#issuecomment-728376262 for more details

``@idahogurl in yarn 3 the command should be yarn dlx @yarnpkg/sdks vscode, executing the command you sugesst produces the following output:
> yarn dlx @yarnpkg/pnpify --sdk vscode
...
Usage Error: The 'pnpify --sdk ...' command has been moved to the '@yarnpkg/sdks' package - use 'yarn dlx @yarnpkg/sdks ...' instead

And yes, I executed this command before creating the issue :)

@yarick123 I'll take a look. I'm pretty sure I'll have to bundle ESLint and Prettier with the extension and I'd rather not do that. Would installing them globally work?

Unfortunately installing prettier and eslint globally does not help either. The error output was also not changed.

Hmmm... the problem seems to go away after installing esbenp.prettier-vscode plugin

I saw that extension is officially supported by Yarn PnP (see https://yarnpkg.com/getting-started/editor-sdks#tools-currently-supported) I looked through their issues and code and tried to figure out what they did to get it functioning. I found that ticket I linked earlier so I thought that is all they did was run that command.

@yarick123 Are you sure it's my extension that is formatting the document and not the esbenp.prettier-vscode extension?

@yarick123 Also, is your project using Yarn workspaces? Or is it a single project?

@idahogurl

@yarick123 Are you sure it's my extension that is formatting the document and not the esbenp.prettier-vscode extension?
after installing esbenp.prettier-vscode I am not sure, if your plugin does something. Only one thing I am sure is, that the exception is not there any more :)

@yarick123 Also, is your project using Yarn workspaces? Or is it a single project?
It is a single project


linux/glibc tagged ptr supports for arm64 control addition.

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

Above all waiting if the aux vec flag will pass during CI then would make sense to uncomment the rest.

@bors r+

:pushpin: Commit 10ba417405710f12e7bd742517c2833a4b00e4f6 has been approved by JohnTitor

:hourglass: Testing commit 10ba417405710f12e7bd742517c2833a4b00e4f6 with merge cd559916d0b13de75a8731d3bba1b9429a92e690...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit 76ce6319461e2abfabc1503640121579e1315a0e has been approved by JohnTitor

:hourglass: Testing commit 76ce6319461e2abfabc1503640121579e1315a0e with merge f1d3afabdbc9958d210a58ad50595aeeb067e88c...

:broken_heart: Test failed - checks-actions

 error[E0432]: unresolved imports `libc::PR_GET_TAGGED_ADDR_CTRL`, `libc::PR_SET_TAGGED_ADDR_CTRL`, `libc::PR_TAGGED_ADDR_ENABLE`
    --> /checkout/target/aarch64-unknown-linux-musl/debug/build/libc-test-d622df2af2fc66ce/out/semver.rs:4059:5
     |
4059 |     PR_GET_TAGGED_ADDR_CTRL,
     |     ^^^^^^^^^^^^^^^^^^^^^^^ no `PR_GET_TAGGED_ADDR_CTRL` in the root
4060 |     PR_SET_TAGGED_ADDR_CTRL,
     |     ^^^^^^^^^^^^^^^^^^^^^^^ no `PR_SET_TAGGED_ADDR_CTRL` in the root
4061 |     PR_TAGGED_ADDR_ENABLE,
     |     ^^^^^^^^^^^^^^^^^^^^^ no `PR_TAGGED_ADDR_ENABLE` in the root

The semver check fails on aarch64-unknown-linux-musl.

@bors r+

:pushpin: Commit a7df4defb3cbc197af50f455ba035f504fbd2e98 has been approved by JohnTitor

:hourglass: Testing commit a7df4defb3cbc197af50f455ba035f504fbd2e98 with merge ce3d1a2b86d4856376ea7025838569145eb9dd26...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing ce3d1a2b86d4856376ea7025838569145eb9dd26 to master...


[Bug]: Sweep with Bayes search fails with "Need at least one searchable parameter" error

Describe the bug
Issue Description:
I am encountering an issue when attempting to run a sweep using the Bayes search strategy. The error message suggests that no searchable parameter is defined, although the configuration includes both the metric and the parameter search space. Here's the error and the steps leading to it:
The following is the error message

400 response executing GraphQL.
{"errors":[{"message":"Invalid sweep config: Need at least one searchable parameter for bayes search.","path":["upsertSweep"]}],"data":{"upsertSweep":null}}
wandb: ERROR Error while calling W&B API: Invalid sweep config: Need at least one searchable parameter for bayes search. (<Response [400]>)
wandb: ERROR Invalid sweep config: Need at least one searchable parameter for bayes search.
Traceback (most recent call last):
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/lib/retry.py", line 131, in call
result = self._call_fn(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 345, in execute
return self.client.execute(*args, **kwargs)  # type: ignore
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/vendor/gql-0.2.0/wandb_gql/client.py", line 52, in execute
result = self._get_result(document, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/vendor/gql-0.2.0/wandb_gql/client.py", line 60, in _get_result
return self.transport.execute(document, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/lib/gql_request.py", line 59, in execute
request.raise_for_status()
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://api.wandb.ai/graphql
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/users/home/bkhalil/Repos/uqdd/uqdd/models/model_parser.py", line 258, in
query_dictf"{args.model}_hyperparam"
File "/home/bkhalil/Repos/uqdd/uqdd/models/evidential.py", line 228, in run_evidential_hyperparam
sweep_id = wandb.sweep(sweep_config, project=wandb_project_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/wandb_sweep.py", line 86, in sweep
sweep_id, warnings = api.upsert_sweep(sweep, prior_runs=prior_runs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/apis/internal.py", line 133, in upsert_sweep
return self.api.upsert_sweep(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/apis/normalize.py", line 73, in wrapper
raise err
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/apis/normalize.py", line 41, in wrapper
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 3216, in upsert_sweep
raise e
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 3210, in upsert_sweep
response = self.gql(
^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 317, in gql
ret = self._retry_gql(
^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/lib/retry.py", line 147, in call
retry_timedelta_triggered = check_retry_fn(e)
^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/util.py", line 881, in no_retry_4xx
raise UsageError(body["errors"][0]["message"])
wandb.errors.UsageError: Invalid sweep config: Need at least one searchable parameter for bayes search.
Traceback (most recent call last):
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/lib/retry.py", line 131, in call
result = self._call_fn(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 345, in execute
return self.client.execute(*args, **kwargs)  # type: ignore
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/vendor/gql-0.2.0/wandb_gql/client.py", line 52, in execute
result = self._get_result(document, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/vendor/gql-0.2.0/wandb_gql/client.py", line 60, in _get_result
return self.transport.execute(document, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/lib/gql_request.py", line 59, in execute
request.raise_for_status()
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://api.wandb.ai/graphql
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/users/home/bkhalil/Repos/uqdd/uqdd/models/model_parser.py", line 258, in
query_dictf"{args.model}_hyperparam"
File "/home/bkhalil/Repos/uqdd/uqdd/models/evidential.py", line 228, in run_evidential_hyperparam
sweep_id = wandb.sweep(sweep_config, project=wandb_project_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/wandb_sweep.py", line 86, in sweep
sweep_id, warnings = api.upsert_sweep(sweep, prior_runs=prior_runs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/apis/internal.py", line 133, in upsert_sweep
return self.api.upsert_sweep(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/apis/normalize.py", line 73, in wrapper
raise err
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/apis/normalize.py", line 41, in wrapper
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 3216, in upsert_sweep
raise e
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 3210, in upsert_sweep
response = self.gql(
^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/internal/internal_api.py", line 317, in gql
ret = self._retry_gql(
^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/sdk/lib/retry.py", line 147, in call
retry_timedelta_triggered = check_retry_fn(e)
^^^^^^^^^^^^^^^^^
File "/home/bkhalil/miniconda3/envs/uqdd-118/lib/python3.11/site-packages/wandb/util.py", line 881, in no_retry_4xx
raise UsageError(body["errors"][0]["message"])
wandb.errors.UsageError: Invalid sweep config: Need at least one searchable parameter for bayes search.
2024-09-06 13:19:42,869 - wandb.wandb_agent - INFO - Cleaning up finished run: p8xnkfd5
`

What I tried:

Verified that the metric is correctly specified in the sweep configuration.
Ensured that the configuration includes valid parameters for Bayes optimization.
Confirmed during debugging that the metric is part of the sweep_config.

reproduction steps:
The error is triggered when running the following command for a previously started sweep:
wandb agent -e bolak92 -p $wandb_project --count $run_count $sweep_id
The sweep config is a json that I load during the python run and it looks like this:
{ "program": "model_parser.py",   "entity": "bolak92",   "method": "bayes",   "metric": {     "goal": "minimize",     "name": "val/loss"   },   "parameters": {...}   }
Environment:

wandb version: 0.17.9
Python version: 3.11.8
OS: Linux

Hi @bolak92, thank you for reaching out and sorry to hear you are experiencing this issue.
Would you be able to share some additional information to help us investigate this:

the content of the parameters": {...} dictionary
the URL for the sweep in question
What version of the wandb SDK you are on

Thanks!

Hi Thanks for the prompt response.

The parameters dictionary is as follows:

{
"program": "model_parser.py",
"entity": "bolak92",
"method": "bayes",
"metric": {
"goal": "minimize",
"name": "val/loss"
},
"parameters": {
"sweep": {
"value": true
},
"model": {
"value": "evidential"
},
"lamb": {
"values": [1.0,0.1,0.01,0.001]
},
"aleatoric": {
"value": true
},
"chem_layers": {
"values": [
[1024,512,256],
[1024,512],
[1024,256],
[1024],
[512],
[256]
]
},
"prot_layers": {
"values": [
[1536, 1024, 512, 256],
[1024,512,256],
[512,256,128],
[512,128],
[512,256],
[512],
[256]
]
},
"regressor_layers": {
"values": [
[256,128],
[128,64],
[256],
[128],
[64]
]
},
"dropout": {
"values": [0.1,0.2]
},
"batch_size": {
"values": [128,256,512]
},
"epochs": {
"value": 3000
},
"early_stop": {
"value": 100
},
"loss": {
"value": "evidential_regression"
},
"loss_reduction": {
"value": "mean"
},
"optimizer": {
"values": ["adagrad","adamw","adam","sgd"]
},
"lr": {
"values": [1e-2,1e-3,5e-4]
},
"weight_decay": {
"values": [0,1e-2,1e-4]
},
"lr_scheduler": {
"value": null
},
"max_norm": {
"values": [null,50.0,100.0]
}
}
}

The URL is
https://wandb.ai/bolak92/evidential/sweeps/asgvot1p/overview

My wandb version is 0.17.9

I solved the problem.
For the record, the issue was in my own script.
when I was calling the agent again with wandb agent it was keeping an argument from the config --sweep_count which I have defined as the condition in the script whether to run hyperparameter optimization vs normal model run. because it wasn't popped out of the kwargs dictionary that I was feeding to the config dictionary definition, it was kept with the downstream runs as well.
that being said, it was trying to rerun the sweep hpo script on top of the previous sweep hpo script.
Moving forward, I think a better error message can be very useful in pointing better to this specific issue.
I realized the problem by looking quite close into the commands that are being run through the script with wandb agent

Thanks for confirming you managed to get this fixed on your end!


Add email assertion

That's something I'm using recently on my projects. But if it doesn't fit the needs of this project, feel free to close. =)

Hm, I wish it was this simple to verify an email. Have a quick look at this repo and we will see that emails are really hard to validate properly: https://github.com/egulias/EmailValidator

It depends on how strict you want to validate it. But you right, this is just a simple validation.  Even EmailValidator can't guarantee that an email is real.
Reference: https://github.com/beberlei/assert/blob/master/lib/Assert/Assertion.php#L1529

I'd say that for a 'simple' assertion library, this is a good enough assertion.
 If that is my use case i'd search for a dedicated validator, like the one @Nyholm linked

Thank you for your contribution @KennedyTedesco

@BackEndTea You're welcome! 😊


Feature: Make it available in python pip

The only option to make that happen seems to be forking as there is not much development going on.
I could help because at the end I'm interested in including altdns into the Fedora Security Lab and for this are certain requirement needed like a setup.py file.

Hi Fabian,
I’m happy to maintain this. Please give me the weekend to resolve or
comment on any issues.
Cheers
On Fri, 17 May 2019 at 5:09 pm, Fabian Affolter<EMAIL_ADDRESS>wrote:

The only option to make that happen seems to be forking as there is not
much development going on.
I could help because at the end I'm interested in including altdns into
the Fedora Security Lab and for this are certain requirement needed like a
setup.py file.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/infosec-au/altdns/issues/18?email_source=notifications&email_token=ABH7YUGQ2VY2XWAZEZGG3FDPVZK2RA5CNFSM4DQPICR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVT6BKY#issuecomment-493346987,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABH7YUBKKEGEFGRAGDCAU6TPVZK2RANCNFSM4DQPICRQ
.

Issue resolved.
pip install py-altdns
Cheers


Fix color picker full mode

This PR fixes a bug where using the "Full" color picker mode doesn't update the color of fixtures or UI color settings.

coverage: 32.068%. remained the same
when pulling 0a1d3322fdb562ecacf76612a3b57f3659f13293 on lilyCalla:full-color-fix
into 3150b445eae811cd84ca76462ba33c98f1f6c074 on mcallegari:master.

Thanks :+1:


[BUG] CertificateClient.StartDeleteCertificateAsync throws ArgumentNullException

Describe the bug
Azure.Security.KeyVault.Certificates.CertificateClient.StartDeleteCertificateAsync throws ArgumentNullException whenever certificate without soft delete is successfully deleted.
Expected behavior
No exceptions thrown.
Actual behavior (include Exception or Stack Trace)
Stacktrace:
System.ArgumentNullException: Value cannot be null.
Parameter name: uriString
   at System.Uri..ctor(String uriString)
   at Azure.Security.KeyVault.Certificates.DeleteCertificateOperation..ctor(KeyVaultPipeline pipeline, Response`1 response)
   at Azure.Security.KeyVault.Certificates.CertificateClient.StartDeleteCertificateAsync(String certificateName, CancellationToken cancellationToken)

To Reproduce

Create Azure Key Vault without soft delete enabled.
Create App Registration for your app and add it as Access Policy to Azure Key Vault.
Create C# project (I used Visual Studio 2017, .NET Core 2.2).
3.1. Ensure C# version is minimum 7.1.
3.2 Install NuGet packages:
3.2.1 Azure.Security.KeyVault.Certificates 4.0.1
3.2.2 Azure.Identity 1.1.1
Run snippet, similar to https://gist.github.com/shevchenkobn/32a5cfcf56d12aa06edbfa50c6dfab76

Environment:

Azure.Security.KeyVault.Certificates 4.0.1; Azure.Identity 1.1.1
Windows 10.0.17763 .NET Core 2.2.106
Visual Studio 15.9.17

Thank you for the report. Are you able to work around this for now? We are working toward a new preview and can include a fix there (to be released later this spring).
I also highly recommend enabling soft delete - now the default. If someone deletes a Key Vault, your secrets, keys, and certificates are gone forward unless you have other backups. Soft delete allows you to recover it.
Still, this is something we'll fix (also in Keys and Secrets, which likely have the same problem). Just wanted to recommend that.

Thank you for your reply.
Workaround
I have a workaround for now. It is included in the code gist I shared:
try-catch with a particular stack trace, very dirty hack.
Soft delete feature
I tried it initially, but couldn't purge the certificate even though app
registration had purge privilege. It seems only recourse group owner can
purge them, however I didn't drill into this. It is unclear whether my app
needs soft delete anyway.
On Mon, Apr 13, 2020, 8:18 PM Heath Stewart<EMAIL_ADDRESS>wrote:

Thank you for the report. Are you able to work around this for now? We are
working toward a new preview and can include a fix there (to be released
later this spring).
I also highly recommend enabling soft delete - now the default. If someone
deletes a Key Vault, your secrets, keys, and certificates are gone forward
unless you have other backups. Soft delete allows you to recover it.
Still, this is something we'll fix (also in Keys and Secrets, which likely
have the same problem). Just wanted to recommend that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-net/issues/11258#issuecomment-612996800,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF23D62CMDCFTV6ZATLMK33RMNCNNANCNFSM4MGRO6JQ
.

Thanks for confirming. We'll include this in the next preview going out since a simple workaround is available.
Interesting about the purge. All our tests use that with a service principal that has all permissions, so it should work. If ever you enable it again, we'd be happy to help figure out why you couldn't. While your app may not need to delete and purge (FWIW, purge happens after a period of time automatically) itself, soft delete also enables the entire Key Vault to be recovered. For example, there was an incident some time back where a vault was deleted that did not have soft delete enabled and we had to re-create a bunch of secrets for tests. We enabled soft delete on all our Key Vaults after that.


DPL-568 Bioscan - Labware creator to create Lysate plate

User story
As PSD we will need a new Labware creator to create the Lysate plate in the Insect Lysate Prep pipeline. This will need to determine the randomised locations of where the two controls will be positioned and create those Control samples. The Beckman robot method will then call an API (separate story) to fetch those control well locations and perform the transfers.
Who are the primary contacts for this story
Scott T
Emma B
Who is the nominated tester for UAT
TBC
Acceptance criteria
To be considered successful the solution must allow:

[x] labware creator makes the LBSN-96 Lysate plate (with addition of controls) from a stock LILYS-96 Stock plate source
[x] labware creator needs to create the 2 control samples, using parameters stored in Limber config on the plate purpose
[x] control locations are to be randomised on the plate (subject to the Beckman limitation described below)
[x] the bioscan integration test should run using the creator

Dependencies
This story is blocked by the following dependencies:
Development and SATs for the Beckman robot (completed early April 2023)
References
This story has a non-blocking relationship with:
Integration test - https://github.com/sanger/General-Backlog-Items/issues/233
Limber pipeline configuration - https://github.com/sanger/sequencescape/issues/3703
Sequencescape pipeline configuration - https://github.com/sanger/sequencescape/issues/3703
Limber bed verification for Stock plate location - https://github.com/sanger/limber/issues/1198
Limber bed verification for Stock to Lysate plate - https://github.com/sanger/limber/issues/1199
Additional context
This is the part of the pipeline where the stock insect plate is transferred into the lysate plate and a positive and negative control are added.
An insect sample can be lost in order to randomise the control placements. Typically we expect the H12 location for the positive control to NOT have an insect sample in it. [This part obsolete, Beckman can cope with both controls being randomised other than restriction mentioned below]
See diagram here
From Scott 27th March re: Beckman limitiation:
Beckman are able to handle the randomisation of both positive and negative control coordinates with one exception:
Where positive_control= H1 AND negative_control = G1.
Any other combination works (including either one of these conditions being true)

Lot pf problems with this story around the use of the Sequencescape version 2 JSON API. Eventually got it working but had to make a few changes to the JSON API relationships in Limber. Mocking those interactions for the tests was also not straight forward.

At UAT Scott has mentioned an additional restriction, that neither PCR control should be placed in H12, as H12 may be used to hold the negative Lysate control. Will confirm requirements and write up as separate stories.


Can I use it?

Dear Developer,
can I use this code for my own application please?
Kind regards,
Thorin

Yes of course! That's why I put it here :-)
Op zo 28 aug. 2022 20:53 schreef thorinEichenschild007 <
@.***>:

Dear Developer,
can I use this code for my own application please?
Kind regards,
Thorin
—
Reply to this email directly, view it on GitHub
https://github.com/KiezelPay/Fitbit_Realistic_HRM/issues/3, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRMFWJUKQCHW7I6A5VY2W3V3OYSLANCNFSM573QNQRA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


pin name can't have a '

Library name and version
Azure.Maps.Rendering 1.0.0-beta.2
Describe the bug
If the name passed to PushpinPosition has a ' in it, such as "Dave's Home", you get the following:
{"pins":["Invalid format for location value ''Dave's home'-105.13008 39.98274'. Expected a floating-point longitude between -180 and 180."]}
Expected behavior
It can handle names with a '
Actual behavior
It throws an exception
Reproduction Steps
example here
Environment
Windows 11 on a desktop. App is Blazor server .NET 8

Please let me know how to escape a ' and if there's any other characters it chokes over.

Also, if this SDK is open source, please point me to it. I'll fix the 2 bugs I found - should only take 1/2 hour or so.
thanks - dave

Hi @DavidThielen,
We're looking into this issue and will get back to you soon. Thank you very much for reporting this issue!

I could reproduce this as described. After investigation, I found that the problem is on the server side. A direct call to the REST API yields the same result.
curl --location 'https://atlas.microsoft.com/map/static/png?api-version=2022-08-01&bbox=13.228%2C52.4559%2C13.5794%2C52.629&language=&pins=default%7CcoFF00FF%7Csc0.8%7Clc008000%7Cls10%7C%7C%27Dave%27s%20home%2712.56%2022.56' \
--header 'Accept: "application/json, image/jpeg, image/png, image/pbf, application/vnd.mapbox-vector-tile"' \
--header 'x-ms-client-request-id: REQUEST-ID' \
--header 'x-ms-return-client-request-id: true' \
--header 'subscription-key: KEY'

returns:
400 Bad Request with message "{
    "pins": [
        "Invalid format for location value ''Dave's home'12.56 22.56'. Expected a floating-point longitude between -180 and 180."
    ]
}"

It's possible to work around this by replacing ' with \u2019.

This was fixed on the server side.

It's fixed and now it's working. @DavidThielen, please let us know if you still encounter any problem. I'll close this issue now. Thank you again for reporting this issue!


Adjust in code

I made adjustments to the code
Fixed when inserting a user that doesn't exist

Codecov Report

Merging #12 (3be6066) into master (3c5d815) will increase coverage by 25.48%.
The diff coverage is 40.74%.

@@             Coverage Diff             @@
##           master      #12       +/-   ##
===========================================
+ Coverage   20.87%   46.36%   +25.48%
===========================================
  Files           4        5        +1
  Lines          91      110       +19
===========================================
+ Hits           19       51       +32
+ Misses         72       59       -13

Impacted Files
Coverage Δ

profile_python/App.py
0.00% <0.00%> (ø)

profile_python/__main__.py
0.00% <0.00%> (ø)

profile_python/profile.py
55.17% <78.57%> (+55.17%)
:arrow_up:

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 3c5d815...3be6066. Read the comment docs.


Datto.CloudContinuityAgent version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree [company="Datto Inc."]

@microsoft-github-policy-service agree company="Datto Inc."

@ffrodriguez The installer stops at about 66%, while processing PlatformVerifier.x64.exe. Does the installer have a dependency?

@ffrodriguez The installer stops at about 66%, while processing PlatformVerifier.x64.exe. Does the installer have a dependency?

It does but the bundle should be handling it.
PlatformVerifier.x64.exe depends on VC++ runtime and dlls in the code all of which should be installed before the verifier is run.

@stephengillie just wanted to follow up on whether I can provide any additional information to help with the approval.

/azp run

Hi @ffrodriguez,
The installer forces a reboot, and unfortunately, we do not accept packages that reboot during installation. The attended installer ends with 2 buttons: "Reboot" and "Cancel". Unattended install initiates a reboot at the end of install. Is there a flag or other way to have this package install without rebooting?
Additionally, the installer hash provided in this PR no longer matches the downloaded installer's hash. If the provided hash does not match the installer's hash, the install will be aborted. This package URI looks like it might be a main download location that could be frequently updated. Is there a URI with versioned paths, or other download location that might retain the same hash long-term?

@stephengillie

The installer forces a reboot, and unfortunately, we do not accept packages that reboot during installation. The attended installer ends with 2 buttons: "Reboot" and "Cancel". Unattended install initiates a reboot at the end of install. Is there a flag or other way to have this package install without rebooting?

Yes, the installer supports /norestart for the unattended install.

Additionally, the installer hash provided in this PR no longer matches the downloaded installer's hash. If the provided hash does not match the installer's hash, the install will be aborted. This package URI looks like it might be a main download location that could be frequently updated.

Yeah, after releasing it to a an initial target, we found an issue that warranted rolling back making this specific installer.

Is there a URI with versioned paths, or other download location that might retain the same hash long-term?

We do, but, sans this issue where we had to rollback this specific <IP_ADDRESS> version, the goal was get this installer's manifest approved and then use wingetcreate in a CI/CD pipeline to simply submit a version update as new versions are rolled out, with the intention of having winget just have the "latest version".
That said, is the above workflow not supported by/for winget?

@ffrodriguez

Yes, the installer supports /norestart for the unattended install.

Validation was originally blocked by the mandatory restart. Please add this install parameter to the manifest for validation to proceed.

Yeah, after releasing it to a an initial target, we found an issue that warranted rolling back making this specific installer.

Validation was subsequently blocked by the hash mismatch. Please also update the hash in the manifest for validation to proceed.

We do, but, sans this issue where we had to rollback this specific <IP_ADDRESS> version, the goal was get this installer's manifest approved and then use wingetcreate in a CI/CD pipeline to simply submit a version update as new versions are rolled out, with the intention of having winget just have the "latest version".

This is an ideal workflow in general. The only concern is that if the package at this URI changes, such as being replaced with an updated version, then this would present a hash mismatch again and Winget would no longer install this version. The hash is a security feature, to ensure that the installer being downloaded and executed is the same package as originally submitted.

Close with reason: Stale PR;


Use GetCurrentGlobalParams where applicable

#1179  👈
feature/proof-of-stake

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lazynina and the rest of your teammates on  Graphite


UUID

Main
Gets us to a UUID place.

another issue is that with these long ids the editor gets messed up, so we probably want to add some text-overflow there:

a lot of the tests are failing here


[WIP] project: remove basemodels

Description
Get rid of human-protocol-basemodels duplicating hmt-basemodels repository.
Summary of changes
Removing occurrences of human-protocol-basemodels across the project.
How test the changes
make generate-json from packages/sdk/python/human-protocol-basemodels should still work as intended, as it will generate json schema from hmt-basemodels, if it is installed in python that is used by schema generating script.
Related issues
Keywords for linking issues

#187
#205
Operational checklist

[X] All new functionality is covered by tests
[ ] Any related documentation has been changed or added

TODO:

[ ] create README.md in human-protocol-basemodels and explain how to use human-protocol-sdk dependencies to generate JSON schema

Note to reviewers
This is my first contribution to this repository. To my best knowledge, this change doesn't brake existing tests and forces this project to use the existing library hmt-basemodels instead of its copied version. When looking at #205 change, it seems to me that we are using JSON schema to generate typescript schemas. JSON schema could be obtained from hmt-basemodels instead of human-protocol-basemodels. Do this project and its derivatives have some additional requirements that cannot be covered by hmt-basemodels?

@pared-imi , As far as I know, we will archive hmt-basemodels repo from hCaptcha, and manage basemodels from monorepo moving forward. cc, @posix4e

hopefully yes

@leric7 @posix4e So that means that we will have to maintain 2 copies of basemodels? Or will humanprotocl depend on monorepo? (Or other way around?)

less copies is always good

I understood from an internal conversation that the basemodels diverged somewhere around the 1.1.1 version. I am closing this PR for now. I will introduce my changes in a separate PR.


RestoreRequests returns XML Malformed when RetrievalTier = GlacierJobTier.Expedited

Describe the bug
When I tried to run the following code, I got the following error: The XML you provided was not well-formed or did not validate against our published schema.
The code follows:
var restorationRequest = new RestoreObjectRequest
{
     BucketName = settings.Bucket,
     Days = days,
     Key = filename,
     RetrievalTier = GlacierJobTier.Expedited,
     Tier = GlacierJobTier.Expedited,
};
_ = await client.RestoreObjectAsync(restorationRequest);

I know that It may be too little information, but the operation goes wrong just when I set RetrivialTier or Tier. If I try to make the same request just with BucketName, Days and Key, it works fine.
I was looking in others SDKs for S3 and I found similar issues. Like this one for Java: https://stackoverflow.com/questions/52166563/set-data-retrieval-option-on-s3-bucket-for-glacier-storage-class. And this one for ruby: https://github.com/boto/boto3/issues/2656.
I dont know what the main problem is, but I think that at least should indicate a more specific error message.
I think maybe is a bug. Idk.
Expected Behavior
It should at least show a more specific error message.
Current Behavior
The XML you provided was not well-formed or did not validate against our published schema
Reproduction Steps
Try to RestoreObjectAsync like in this code:
var restorationRequest = new RestoreObjectRequest
{
     BucketName = settings.Bucket,
     Days = days,
     Key = filename,
     RetrievalTier = GlacierJobTier.Expedited,
     Tier = GlacierJobTier.Expedited,
};
_ = await client.RestoreObjectAsync(restorationRequest);

Possible Solution
No response
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
<IP_ADDRESS>
Targeted .NET Platform
.NET 7.0
Operating System and version
Windows 11

Needs to be reproduced.


i18n(zh-cn): update markdown-content.mdx

Description (required)

update markdown-content.mdx
Related issues & labels (optional)

https://github.com/withastro/docs/pull/7218

Closes #

Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
guides/markdown-content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.