feat: add delete button to edit list page

Description
This PR has the following:

a new organism delete list modal which can be used for deleting a list.
The existing delete list modal in the lists page is replaced by delete list modal organism  (works like before just some refactoring)
Added a delete list button at the bottom of the Edit list page which also uses the same delete list modal organism

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

[X] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[X] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Feat #2424
Mobile & Desktop Screenshots/Recordings

Steps to QA

Go to Lists page
Click on any list you want to delete
Go to edit list page
Scroll to the bottom of the page and you will find the delete list button.

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[X] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
NO
[optional] What gif best describes this PR or how it makes you feel?

Thanks for tackling this @sourabpramanik 👏 👏 ... Would you mind adding screen shot/recording of the before and after state of this change?

Thanks for tackling this @sourabpramanik 👏 👏 ... Would you mind adding screen shot/recording of the before and after state of this change?

@OgDev-01 actually before this PR the delete feature was not there in the Edit list page, but I have added the gif for the feature I have created. Will that be okay?

@nickytonline is there any other changes to be done? Please let me know. Thank you for your help.

There are some linting errors. Will resolve them and commit.

Testing on a large screen and this is how the button looks

We don't want the button spanning the full width of container

Understood will fix this in the next commit.

@OgDev-01 thanks for pointing out, the changes are done

You have some lint errors to fix locally @sourabpramanik

@nickytonline is there any path issue? I suspect the new changes are causing some issues. I will check

Aaahh!! such a dumb mistake by me. 😂😂😂

Great job @sourabpramanik 👏 👍 👌

Thanks @nickytonline and @OgDev-01


Install Murano?

It's clearly a good idea: Murano, an application catalog.

Yes: http://apps.openstack.org.

For now, other services are more helpful than Murano, such as Magnum and Tempest. I'm moving, again, to containers due to my poor resources... Indeed, Murano looks great, but I have to admit: it's a good feature, not a must-have one for my environment.
Sad, but true: I'm working lonely on my OpenStack private cloud, so I have to be time time-efficient ;)


Unable to retrieve full registry from Eureka server

Hello,
I am trying to use a  Eureka as a proof of concept to enable service discovery and directory. I managed to compile Eureka and get it running on a tomcat.
I used this eureka-js-client lib in order to register a service. That is working. Eureka interface shows my service :

I also get a successfull Heartbeat.
But I keep having this weird error: And i don't what to with it.
Error fetching registry Error: Unable to retrieve full registry from Eureka server
    at C:\_dev\nsi\referentiels\ref-medias\node_modules\eureka-js-client\lib\EurekaClient.js:439:16
    at C:\_dev\nsi\referentiels\ref-medias\node_modules\eureka-js-client\lib\EurekaClient.js:731:7
    at C:\_dev\nsi\referentiels\ref-medias\node_modules\eureka-js-client\node_modules\async\dist\async.js:473:16
    at next (C:\_dev\nsi\referentiels\ref-medias\node_modules\eureka-js-client\node_modules\async\dist\async.js:5315:29)
    at C:\_dev\nsi\referentiels\ref-medias\node_modules\eureka-js-client\node_modules\async\dist\async.js:958:16
    at Request._callback (C:\_dev\nsi\referentiels\ref-medias\node_modules\eureka-js-client\lib\EurekaClient.js:709:9)
    at Request.self.callback (C:\_dev\nsi\referentiels\ref-medias\node_modules\request\request.js:186:22)
    at emitTwo (events.js:125:13)
    at Request.emit (events.js:213:7)
    at Request.<anonymous> (C:\_dev\nsi\referentiels\ref-medias\node_modules\request\request.js:1163:10)
    at emitOne (events.js:115:13)
    at Request.emit (events.js:210:7)
    at IncomingMessage.<anonymous> (C:\_dev\nsi\referentiels\ref-medias\node_modules\request\request.js:1085:12)
    at Object.onceWrapper (events.js:314:30)
    at emitNone (events.js:110:20)
    at IncomingMessage.emit (events.js:207:7)
    at endReadableNT (_stream_readable.js:1045:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)

Here is the configuration for the registering:
const client = new Eureka({
    // application instance information
    instance: {
        app: 'ref-medias',
        hostName: 'localhost',
        ipAddr: '<IP_ADDRESS>',
        port: {
            '$': 3003,
            '@enabled': true,
        },
        vipAddress: 'v1/media/5',
        dataCenterInfo: {
            '@class': 'com.netflix.appinfo.InstanceInfo$DefaultDataCenterInfo',
            name: 'MyOwn',
        }
    },
    eureka: {
        // eureka server host / port
        host: '<IP_ADDRESS>',
        port: 8080,
    },
});

client.register(function(error){
    console.log('WITHIN REGISTER');
    console.dir(error, {depth:null,colors:true});
});

Any idea how to solve this ?
Regards,
Denis.

It would be interesting to hit: http://<IP_ADDRESS>:8080/eureka/v2/apps/ to see what it returns. It looks like that request may be failing.
curl -v http://<IP_ADDRESS>:8080/eureka/v2/apps/

Hey, after 15 minutes of Eureka server running, The error was gone, without any changes of my own. I had the chance afterward to hit this URL it was returning XML data about the services. I had to move on as my client didn't want me to spend too much time on this matter. But Eureka service directory will be use in the future (I don't know when) so When I will start again this service, I'll see if same error happens again, and keep you posted on the subject :)
Regards,
Denis.


enable Cynara unit testing

The upstream Cynara project has unit tests in the repo. They depend on components not readily available in OE-core, so we can only enable them if those dependencies are satisfied somehow, i.e. enable the tests conditionally.
In addition, a separate repo of tests was mentioned at some point. Not sure whether that's really correct. Need to check with Cynara devs.

I have asked to Alexandru Cornea from Security QA team to request access. Like the other question ... is where will these test reside.  Also is there is common unit test framework that we want strive for (if possible since upstream can mean anything)?  More or less about unit tests that are applicable to OE-core.

Each component may use its own unit testing framework. This gets bundled in OE as "ptests", see http://www.yoctoproject.org/docs/1.6.1/dev-manual/dev-manual.html#testing-packages-with-ptest
The problem with Cynara is that it depends on gmock, which is only in meta-oe, but not oe-core. In the cynara recipe, we must add a PACKAGECONFIG for enabling tests, off by default, and then the user of the recipe is responsible for adding a suitable layer with gmock.

Alex Cornea is looking into this now.

Because we had issues with running Cynara (turned out to be something else), I looked into this myself today. See https://github.com/01org/meta-intel-iot-security/pull/17

Thank you.

Closing as "obsolete" because Ostro OS is not using Cynara at the moment. Can be reopened if someone wants to work on it.


Improve test speed

Tests are too slow..

@alexcos20 do you want me to have a go at this? I think there are some quick wins by removing a lot of unnecessary sleep()

is this issue still relevant?

Nothing has been done about it and the tests are still pretty slow but I'd say it's low priority while we're not doing much active work on the subgraph. I'm happy to quickly remove the unnecessary await sleep() statements that are in a lot of the tests, which might speed it up a bit without requiring a lot of work

@mihaisc is this a) resolved &/or b) relevant to test functions for ormi testing at all ?

not relevant


Add PBCore, Mediatrace and image sequence CLI help

Here's my best guess at what these should look like..

thanks!


add GSoC-interested list

Very good idea, @GeorgLink.
In the case of GrimoireLab, we're going to ask students to create a repository with the staff they are producing for the evaluation. What do you think about requiring them to include a link to that repo in a fourth column in the table?

Fourth column with link to a repository is a good idea.

I've added that fourth column, and some text explaining it. Feel free to merge if you find it convenient.

Thanks @jgbarah

TODO:

Update wiki
Inform students


[RFC] Functional composition: chain

It'd be great if there was some way of connecting Omni code together with functional syntax, a la Faust

Perhaps, just as a way to chain things together:
require "Oscillators.omni"

chain:
    Sine * 100 -> Sine | Saw

This also requires to rework how to define pieces of omni code:
Oscillators.omni
process Sine:
    ins 1
    outs 1

    struct:
        phase

    init:
        phase = 0

    sample:
        freq_incr = in1 / samplerate
        out1 = sin(phase * twopi)
        phase = (phase + freq_incr) % 1

Can chain support defs?

What about like variable assignments?
chain:
    sine = Sine
    in1 -> [sine, sine]

How would it operate with outs ???

Feeback??
chain:
    [in1 -> Sine] <<


[Website] Update copy icon positioning

Resolves #24426

update positioning of copy icon in tables to the right top corner, since it is consitent with other copy icons positioning and it will not disturb in case of long content or overflow
make copy icon visible on mobile devices

before:

after:

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Verified, works as expected. Reviewed and tested by QA

@aromanenko-dev, could you kindly help with review of current PR?


Problem deleting document

Hi There,
https://groups.google.com/forum/#!topic/ektorp-discuss/uLKl4bFph8Y
My code for delete document looks like this :
    Example ex = null;
    List<Example> res = findByExampleId(id);
    Iterator<Example> iterator = res.iterator();
    while (iterator.hasNext()) {
        ex = (Example)iterator.next();
    }
    db.delete(ex);

But it always fetches the older revision of the file resulting in org.ektorp.UpdateConflictException.
How do I fetch the latest revision to ensure the delete happens successfully ?
Many Thanks !

answered on groups


Extensions with web target platform are associated with the nullExtension

VS Code for the Web is not loading some resources in the extension with web target platform
Eg: https://ms-toolsai.vscode-unpkg.net/ms-toolsai/jupyter/2023.4.1011131224/extension/out/extension.web.bundle.js?target%3Dweb.
This seems to work fine with a simple extension sandy081.999-pse published with web target platform. I do not know why it is breaking for above extension path.
When target platform web is introduced, we decided to go with the query parameter route considering that extensions will handle URIs correctly given the vscode.uri util. It seems we need to make changes how extension paths are handled to fix this.
CC @joaomoreno

When target platform web is introduced, we decided to go with the query parameter route considering that extensions will handle URIs correctly given the vscode.uri util.

I would suggest to reconsider this given that we didn't really think about this. We now discovered one issue, there is very likely more in our code base, and assuming all extensions use the uri utils is very bold

@sandy081
Here it is
Version 2023.4.1011131224 from the marketplace

For context:
@jrieken says

Ok, we then need to make changes how extension paths are handled. Today /foo/bar/baz/hello.txt?something isn’t considered a child of /foo/bar/?something and therefore all those extensions run as “null extensions”. that doesn’t just affect proposed API but also logging, telemetry, storage, mementos etc pp (edited)

We construct a trie data structure for extension path lookups. For that it is just a constructor option and it ignores query and fragment. However, there might more issues because I assume this is reflected all the way into the Extension.extensionUri API and I don’t know if all extensions are expecting a query part (edited)

Two paths forward:

We push towards a fix in VSCode, by having the trie not bail out on query params. But as @jrieken mentioned, we might find other places in the product that do not play well with query params in URIs, further down the flow.
We allow using the path component to specify platforms in vscode-unpkg.

By now it's rather clear that the latter is the lowest hanging and lowest right change.


Script running multiple instances of application

We're using GSM to manage Ark Survival Ascended servers. Starting this week we've noticed that the powershell script is running to check if the server is running, and then starting a new instance of the server despite the server already running.
We're running 5 server instances from the same VPS, only one is having this issue. I suspected this might be due to the scheduled job running too regularly and so increased the time in task scheduler but this has not solved the problem.

If you run many instances from the same folder, don't do that. It will prevent proper updates.


Reposition Add to Cart button to an always visible position in Mobile view

What is the motivation for adding/enhancing this feature?
When using a mobile to browse the app, on the product view, it would be easier to purchase an item if the Add to Cart is visible. This way the user won't have to look for the Add to Cart button, it will be accessible.
What are the acceptance criteria

[  ] Add to cart button pops up from on the bottom of the screen when the user scrolls under the title.

Which Release Cycle state this refers to? Info for the developer.
This is an idea needs design and implementation. It is an easy good first request.

[x] This is a normal feature request. This should be available on https://test.storefrontcloud.io and then after tests this can be added to next Vue Storefront version. In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[ ] (Pick this option only if you're sure) This is an important improvement request for current Release Candidate version on https://next.storefrontcloud.io and should be placed in next RC version. In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[ ] (Pick this option only if you're sure) This is a critical improvement request for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version. In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Additional information
This is a quick and not detailed mockup

I will take care of this issue.

i will have a look on this issue at hackathon #17 in brunswick


validate type specific options

@Nyholm trying to do this validation thing

Cool. I like it. =)

fixed to use $this, and added the missing tests for the new configuration (and plugin configuration in general)

Cool. Thanks!


Panicked at 'byte index 1 is out of bounds on invalid query

Describe the bug
@syheliel reports the following errors:
1. arrow-datafusion/datafusion/physical-expr/src/[planner.rs:54](http://planner.rs:54/)
      if &variable_names[0][0..2] == "@@" {
2. arrow-datafusion/datafusion/physical-expr/src/[planner.rs:1740](http://planner.rs:1740/)
      if &var_names[0][0..1] == "@" {

Which will panic if the  length of variable names is less than 2 characters.
These bugs can be fixed by using get(0..2) instead of [0..2] for a slice.
To Reproduce
Here is one reproducer:
❯ SELECT  N | "".   LT;
thread 'main' panicked at 'byte index 1 is out of bounds of ``', library/core/src/str/mod.rs:107:9
stack backtrace:
   0: rust_begin_unwind
             at /rustc/4b91a6ea7258a947e59c6522cd5898e7c0a6a88f/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/4b91a6ea7258a947e59c6522cd5898e7c0a6a88f/library/core/src/panicking.rs:142:14
   2: core::str::slice_error_fail_rt
             at /rustc/4b91a6ea7258a947e59c6522cd5898e7c0a6a88f/library/core/src/str/mod.rs:107:9
   3: core::ops::function::FnOnce::call_once
             at /rustc/4b91a6ea7258a947e59c6522cd5898e7c0a6a88f/library/core/src/ops/function.rs:248:5
   4: core::intrinsics::const_eval_select
             at /rustc/4b91a6ea7258a947e59c6522cd5898e7c0a6a88f/library/core/src/intrinsics.rs:2376:5
   5: core::str::slice_error_fail
...

Expected behavior
A valid error message rather than panic
Additional context
This is similar to https://github.com/apache/arrow-datafusion/issues/3137 / fixed https://github.com/apache/arrow-datafusion/pull/3162

This is probably a good first issue as the change is identified and there is a test case


📜 mockbuild: Reduce log output

Running mock with verbose enabled made sense a while back when we were
still unsure if things would work, but it's generating a ton of logs now
that eventually clogs Jenkins' disk.
Disable the verbose flag to reduce mock's log output.
Signed-off-by: Major Hayden<EMAIL_ADDRESS>

The mock build went through fine so I stopped the rest of the job so we could free up resources in Jenkins sooner.


consul systemd hangs at 'create symlink at /etc/service/consul to /etc/sv/consul' on Centos70

I am trying to get consul to work on Centos70, which worked in Vagrant, so I packer'd up a centos7 ami and launched with recipe[consul] and custom json
    "consul": {
      "install_method": "binary",
      "init_style": "systemd"
    }

things seemed ok, but it just hangs endlessly
    * template[/etc/sv/consul/log/config] action create
      - create new file /etc/sv/consul/log/config
      - update content in file /etc/sv/consul/log/config from none to e3b0c4
      (no diff)
      - change mode from '' to '0644'
      - restore selinux security context
    * link[/etc/init.d/consul] action create
      - create symlink at /etc/init.d/consul to /sbin/sv
    - configure service runit_service[consul]
    * link[/etc/service/consul] action create
      - create symlink at /etc/service/consul to /etc/sv/consul

does anyone have a working combination of centos66 or centos7 + consul working?

The 1.0 release will support systemd/upstart/sysv through poise-service.

I am closing this out because of impeding #126 landing on master. The expectation here is that you'll be able to use the native pid managers as well as others such as supervisord/runit.


CLIMATE-707 - Add initial conda env based Easy-OCW scripts

Note https://github.com/apache/climate/pull/262 for info on the broken tests

Merged build triggered. Test Failed.

Merged build started. Test Failed.

Merged build finished. Test Failed.


Interop for io.estatico.newtype

Was inspired by https://github.com/zio/zio-json/issues/852 issue
What do you think about it? If it's ok, I will add documentation for interop

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Grigorii Berezin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@fsvehla hello, could you pls help me with publishLocal job? It does not work with * as I can see

@GrigoriiBerezin
CI is failing, can you please take a look


Update Sparkline README.md

📜 Documentation
Setting up env for superset-ui and realized the example wasn't working str8 of the box so I fixed inside my storyboard and wanted to run the changes back into master
@williaster Let me know what you think?

LGTM, thanks @hughhhh  🙌 stoked for superset-ui! 👀

Coverage decreased (-0.05%) to 84.404% when pulling ad4651771945adf1040b83f1887ebd9fc88b2c7e on hughhhh:update-docs into f5b59a1720ff069dd47f29ca614a3a695487653a on williaster:master.


So a good way to attack would be a "standalone" Pods. On the other hand there are Operators like pgng? being a controller for the pods themself not using sts, deploy etc.

Hello @erkules.
We're working on that, it should be available in the next release 0.2.0.

The feature is now available in the [v0.2.0](https://github.com/falcosecurity/falco-talon/releases/tag/v0.2.0


Acceptance Test (or integration test) for initial sync logic

Test SNAP / Checkpoint sync logic, progress and completion
First - is this sufficiently covered by hive tests?
If not, we should add acceptance or integration tests for this in Besu.

See RunnerTest - tests full and fast sync with one peer
ClusterAcceptanceTest - sets up a cluster

can we use Kurtosis for this?


Add ability to hide topic instead of content.

Tried to keep as simple as possible, I think I hit all of the requirements for your surrounding tools.
Changed only one unit test, otherwise tested simply with:
python3 -mflashcards ../examples/example.yaml -I
Tox hiccups:

'check' fails because setup.py:31 is too long (Not an addition I made)
I have no interpreter for python3.3, python3.4, python 3.5
'report' fails: coverage combine --append returns No data to combine

Not sure what this is about, coverage report works just fine.

Codecov Report

Merging #2 into master will decrease coverage by 0.87%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
- Coverage   67.11%   66.24%   -0.88%
==========================================
  Files           5        5
  Lines         149      157       +8
  Branches       14       15       +1
==========================================
+ Hits          100      104       +4
- Misses         46       49       +3
- Partials        3        4       +1

Impacted Files
Coverage Δ

src/flashcards/cli.py
0% <0%> (ø)
:arrow_up:

src/flashcards/__init__.py
100% <100%> (ø)
:arrow_up:

tests/test_flashcards.py
94.87% <100%> (+0.13%)
:arrow_up:

src/flashcards/flashcards.py
66.66% <65%> (-1.08%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks a lot, I'll try it later today

Hey looks good, can you rebase on top of master?

thank you!! :tada:

v.0.3.0 released! :100:


Tapo P110 result error

After a recent auto firmware update this python module has stopped working.
Device: Tapo P110
Firmware Version: 1.3.0 Build 230905 Rel.152200
Python 3.9
PyP100==0.1.2
Python Error:
python3 test_plug.py
{'error_code': 1003}
Traceback (most recent call last):
File "test_plug.py", line 32, in
control_smart_plug("off")
File "test_plug.py", line 16, in control_smart_plug
plug.handshake() #Creates the cookies required for further methods
File "env/lib/python3.9/site-packages/PyP100/PyP100.py", line 128, in handshake
encryptedKey = r.json()["result"]["key"]
KeyError: 'result'
Device works in the Tapo app. But python gives the error above.

Seems to be similar issue with Tapo T100 motion sensor!

PyP100 is not my library

Has anybody a solution for this? As I have the same issue/error

I have a work around for now until the library is fixed. Install the real home assistant docker image, pair the Tapo with it, use the home assistant api to control the plug.

I found the solution meanwhile, by using the updated PyP100-package from https://github.com/almottier/TapoP100 instead of the original one of https://github.com/fishbigger/TapoP100

PyP100 is not my library (plugp100 Is) and this integration is based on my library, so this issue is not related to this repository

Yes you said that. Thanks.

Hello, I still have the same problem, any solution?

Same issue, now 1 year later......


Require Node.js 18 (and release a new major version)

The EOL of Node.js 16 is approaching (in September 2023), and Node.js 14 is almost EOL (end of April 2023), so it might be reasonable to skip both of those and require Node.js 18 for JSKOS Server. For this, we should release a new major version, i.e. version 2.0.0.
I'm not sure if there are other breaking changes that we could include in this release. @nichtich?

[x] #193 could be considered a breaking change, and it's also easy to implement, so it should be part of v2.
[x] I would like to move to ES modules, see #159.

[x] Followed by finally updating all dependencies to their latest versions. (#140)

[x] #194 is necessary for a new feature in Cocoda, so we should clarify and implement it (should also be fairly easy).
[x] We should check if JSKOS Server is affected by this issue: https://github.com/gbv/login-server/issues/98
[x] #199

To-Dos before release

[ ] Do some manual integration testing with Cocoda, Login Server, and maybe BARTOC
[ ] Prepare release notes
[ ] Import mismatch vocabulary into our main instances
[ ] Question: Can we adjust our automatic updates that they won't do major updates without confirmation?

To-Dos after release

[ ] Make necessary adjustments in cocoda-sdk and release a new major version
[ ] Check for required adjustments in other projects (BARTOC in particular)
[ ] Check whether everything works in our release instances

Suggestion for release notes (note that links don't work until the release is out):
Breaking Changes

Node.js 18 is now required. (#195)
The Docker image was moved from coliconc/login-server to ghcr.io/gbv/login-server. See the Docker README for more information about Docker usage.
The GET /data endpoint was completely adjusted. It now returns items of any kind by their URI. See documentation for more information. (#193)
Concept write access was moved to the new /concepts endpoint, i.e. POST/PUT/PATCH/DELETE /data is replaced with POST/PUT/PATCH/DELETE /concepts. (#193)
All old concept endpoints (/narrower, /ancestors, /search, /suggest) are now available under the /concepts endpoint, e.g. /concepts/narrower. The old concept endpoints are still available as deprecated aliases, but will be removed in the next major release. (#193)
All PATCH endpoints now merge data only on the top level. To remove top-level properties via PATCH, set them to null. (#199)

Further Changes

Add GET /concepts endpoint that implements previous behavior of GET /data for concepts only. (#193)
Negative assessment annotations can now include a mismatch tag for mappings. See documentation for further details. (#194)

Support for this will be added to Cocoda in an upcoming release.

The properties parameter was updated. See documentation for further details.
Fix issues with MongoDB connection in Node.js 18.
Code base was refactored to ESM. (#159)
Update all dependencies to their latest version. (#140)

v2.0.0 is now released: https://github.com/gbv/jskos-server/releases/tag/v2.0.0
I noticed a small issue that I fixed in v2.0.1: https://github.com/gbv/jskos-server/releases/tag/v2.0.1
This release also caused Cocoda to stop working until I updated it because it increased the API version to 2.0 as well, and Cocoda was configured to only support JSKOS API 1.0. This also means that none if the Cocoda versions in our archive work anymore which is a bit sad. Any ideas @nichtich? Related: #201 (One workaround could be to rebuild all Cocoda versions in our archive to skip the JSKOS API version check...)

One workaround could be to rebuild all Cocoda versions in our archive to skip the JSKOS API version check...

This is probably a one-liner fix + rebuild of all old version for the archive, but currently not a priority.

Make necessary adjustments in cocoda-sdk and release a new major version

I had this on the to-do list here, but since it's not related to jskos-server itself, I created a separate issue over in cocoda-sdk.
With this, I'm finally closing this issue.


Fix specs due to RelatonIetf

As title (see Travis logs).

fixed it, but I don't know how to restart AppVeyor.


Using an existing Log Analytics workspace

[Enter feedback here]
Hi,
Can we use an existing Log Analytics Workspace to capture B2C logs? This document only explains the scenario for new workspaces, which is hardly the case for existing systems.
Thanks

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

ID: 24e34c89-9a70-b5b2-3b38-bc81660f83c1
Version Independent ID: 77fa8d3c-56dd-294f-13ae-ac6ba44018a7
Content: Monitor Azure AD B2C with Azure Monitor - Azure AD B2C
Content Source: articles/active-directory-b2c/azure-monitor.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@zaalion
Thank you for pointing this out!
If you have an existing Log Analytics workspace you can use it when setting up Azure AD B2C with Azure Monitor.
Since I already had existing resources, I skipped steps 1 thru 3.3 (for testing, I didn't need to create custom auth and delegation in Azure Lighthouse), and started at step 4 - Select your subscription to enable Diagnostic settings within my B2C tenant.

I've created a PR to update the documentation and it's currently waiting for the author to review and sign off.
I will now close out this issue. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!


Converting a SavedModel Tensorflow Format to Luxonis Blob format

Issue Type
Support
OS
Windows, Other
OS architecture
armv7, armv6
Programming Language
Python
Framework
TensorFlow
Model name and Weights/Checkpoints URL
This is a custom trained model on edgeimpulse.com which provides a Tensorflow SavedModel as well as a Tensorflow Lite Model at the end of the training. The issue is converting a Tensorflow SavedModel to a Luxonis Blob model by first Freezing the Tensorflow Saved Model and then trying to use https://blobconverter.luxonis.com/.
Description
These are my files:
SavedModel file:
SavedModel.zip
Tflite file:
Tflite_file.zip
Sorry for opening this issue since this is beyond the work you do but since I saw your work on the Luxonis discord on helping convert Tensorflow SavedModels to Luxonis Blob format, I just wanted to reach out as a last resort. I've tried many different alternatives:

Freezing a Tensorflow Saved Model. The method successfully freezes the SavedModel, however it fails while using https://blobconverter.luxonis.com/ to convert the Frozen format to a blob format.

Code to freeze the SavedModel:
import tensorflow as tf

# Load the saved model
loaded = tf.saved_model.load("/content/saved_model/")

# Extract the graph
graph = tf.function(loaded.signatures[tf.saved_model.DEFAULT_SERVING_SIGNATURE_DEF_KEY]).get_concrete_function(tf.TensorSpec(shape=[1, 320, 320, 3], dtype=tf.float32))
frozen_graph = graph.graph.as_graph_def()

# Save the frozen graph
with tf.io.gfile.GFile("/content/frozen_model.pb", "wb") as f:
    f.write(frozen_graph.SerializeToString())

Error I get with blobconverter:
[ ERROR ]  Cannot infer shapes or values for node "StatefulPartitionedCall".
[ ERROR ]  Expected DataType for argument 'dtype' not None.
[ ERROR ]
[ ERROR ]  It can happen due to bug in custom shape infer function <function tf_native_tf_node_infer at 0x7f99c67e1af0>.
[ ERROR ]  Or because the node inputs have incorrect values/shapes.
[ ERROR ]  Or because input shapes are incorrect (embedded to the model or passed via --input_shape).
[ ERROR ]  Run Model Optimizer with --log_level=DEBUG for more information.
[ ERROR ]  Exception occurred during running replacer "REPLACEMENT_ID" (<class 'openvino.tools.mo.middle.PartialInfer.PartialInfer'>): Stopped shape/value propagation at "StatefulPartitionedCall" node.
 For more information please refer to Model Optimizer FAQ, question #38. (https://docs.openvino.ai/latest/openvino_docs_MO_DG_prepare_model_Model_Optimizer_FAQ.html?question=38#question-38)

Using a SavedModel -> IR Representation (OpenVINO) -> Blob conversion
I used the standard OpenVINO instructions on the site to convert the SavedModel into a .xml and a .bin file.

This is apparently what I used and it converted it successfully to .xml and .bin files, however the .xml file was just 2 Kb and the .bin file was 0 Kb
Output while conversion:
[ INFO ] The model was converted to IR v11, the latest model format that corresponds to the source DL framework input/output format. While IR v11 is backwards compatible with OpenVINO Inference Engine API v1.0, please use API v2.0 (as of 2022.1) to take advantage of the latest improvements in IR v11.
Find more information about API v2.0 and IR v11 at https://docs.openvino.ai/latest/openvino_2_0_transition_guide.html
[ SUCCESS ] Generated IR version 11 model.
[ SUCCESS ] XML file: /content/saved_model.xml
[ SUCCESS ] BIN file: /content/saved_model.bin

Nevertheless, I tried to convert to a blob file and it converted it to a 1Kb file and when I tried to run it, the NN size was displayed to be 3x320 which was absurd since it was trained on 320x320 images [1,320,320,3] format.
I reran the conversion again using:
!mo --input_shape [1,320,320,3] --saved_model_dir /content/saved_model/ --layout "ncwh->nhwc"

However, this time when I converted it to blob and used it while running, it gave me an error stating the bounding boxes contain x=1,y=0,w=0,h=0.

This time I tried a .tflite to .onnx approach using https://github.com/zhenhuaw-me/tflite2onnx which wasn't successful in converting it to onnx in the first stage itself. ( A .tflite model is also included on the EdgeImpulse dashboard so I thought of trying this out)

Code:
import tflite2onnx

tflite_path = '/content/trained.tflite'
onnx_path = '/content/model.onnx'

tflite2onnx.convert(tflite_path, onnx_path)

Error:
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
[<ipython-input-37-c193f27d3323>](https://localhost:8080/#) in <module>
      4 onnx_path = '/content/model.onnx'
      5
----> 6 tflite2onnx.convert(tflite_path, onnx_path)

4 frames
[/usr/local/lib/python3.8/dist-packages/tflite2onnx/op/common.py](https://localhost:8080/#) in create(self, index)
    152             if opcode in tflite.BUILTIN_OPCODE2NAME:
    153                 name = tflite.opcode2name(opcode)
--> 154                 raise NotImplementedError("Unsupported TFLite OP: {} {}!".format(opcode, name))
    155             else:
    156                 raise ValueError("Opcode {} is not a TFLite builtin operator!".format(opcode))

NotImplementedError: Unsupported TFLite OP: 83 PACK!

Next, I tried using tf2onnx (https://github.com/onnx/tensorflow-onnx) using a SavedModel file
Input:
python -m tf2onnx.convert --saved-model /content/saved_model/ --output /content/model.onnx

Output:
2023-01-19 17:18:58,166 - WARNING - '--tag' not specified for saved_model. Using --tag serve
2023-01-19 17:19:12,548 - INFO - Signatures found in model: [serving_default].
2023-01-19 17:19:12,548 - WARNING - '--signature_def' not specified, using first signature: serving_default
2023-01-19 17:19:12,550 - INFO - Output names: ['output_0', 'output_1', 'output_2', 'output_3']
2023-01-19 17:19:15,690 - INFO - Using tensorflow=2.9.2, onnx=1.13.0, tf2onnx=1.13.0/2c1db5
2023-01-19 17:19:15,690 - INFO - Using opset <onnx, 13>
2023-01-19 17:19:15,694 - INFO - Computed 0 values for constant folding
2023-01-19 17:19:15,700 - INFO - Optimizing ONNX model
2023-01-19 17:19:15,715 - INFO - After optimization: Const -3 (4->1), Identity -1 (4->3)
2023-01-19 17:19:15,716 - INFO -
2023-01-19 17:19:15,716 - INFO - Successfully converted TensorFlow model /content/saved_model/ to ONNX
2023-01-19 17:19:15,716 - INFO - Model inputs: ['input']
2023-01-19 17:19:15,716 - INFO - Model outputs: ['output_0', 'output_1', 'output_2', 'output_3']
2023-01-19 17:19:15,716 - INFO - ONNX model is saved at /content/model.onnx

But after that, while converting it to blob using blobconverter, this is the issue:
[ ERROR ]  Numbers of inputs and mean/scale values do not match.
 For more information please refer to Model Optimizer FAQ, question #61. (https://docs.openvino.ai/latest/openvino_docs_MO_DG_prepare_model_Model_Optimizer_FAQ.html?question=61#question-61)

I'm completely stuck and have tried everything I could and hoping there is some workaround. Thanks a lot for your open-source contributions! Any workaround on converting this to blob would be really helpful since I'm completing the project under a narrow time constraint.
Thanks!
Dhruv Sheth
Relevant Log Output
No response
URL or source code for simple inference testing code
SavedModel file:
SavedModel.zip
Tflite file:
Tflite_file.zip

First, I have all the tools currently available to convert models between mainstream frameworks.

https://github.com/onnx/tensorflow-onnx (ONNX official tool)
https://github.com/PINTO0309/onnx2tf
https://github.com/PINTO0309/tflite2tensorflow
https://github.com/PINTO0309/openvino2tensorflow

Since each tool has its own characteristics, it is difficult to say that this tool is the best, but tensorflow-onnx or  tflite2tensorflow is probably the most suitable for your application.
Your model contains a special OP TFLite_Detection_PostProcess. This is a very special OP that can only be interpreted by the TensorFlow Lite runtime. I expect that your model was generated using the TensorFlow Object Detection API or something similar.

Therefore, it is necessary to replace this special OP with a general OP and then perform a conversion that optimizes it for OpenVINO and Myriad. tflite2tensorflow and openvino2tensorflow have that capability. As a result of continued enhancements without regard to the name of the tool, a single tool is able to convert models between multiple frameworks.
First, I rarely use saved_model. I use the tfltie model instead. This is because the structure of the model is determined to be in a nicely optimized state. tensorflow-onnx does not optimize the model well enough.
https://github.com/PINTO0309/tflite2tensorflow
xhost +local: && \
docker run --gpus all -it --rm \
-v `pwd`:/home/user/workdir \
-v /tmp/.X11-unix/:/tmp/.X11-unix:rw \
--device /dev/video0:/dev/video0:mwr \
--net=host \
-e XDG_RUNTIME_DIR=$XDG_RUNTIME_DIR \
-e DISPLAY=$DISPLAY \
--privileged \
ghcr.io/pinto0309/tflite2tensorflow:latest

tflite2tensorflow \
--model_path trained.tflite \
--flatc_path ../flatc \
--schema_path ../schema.fbs \
--output_pb \
--optimizing_for_openvino_and_myriad

tflite2tensorflow \
--model_path trained.tflite \
--flatc_path ../flatc \
--schema_path ../schema.fbs \
--output_no_quant_float32_tflite \
--output_onnx \
--onnx_opset 11 \
--output_openvino_and_myriad

or

or
http://blobconverter.luxonis.com/

We can see that the level of optimization of the model's structure is different from other tools.

Thanks a lot @PINTO0309 for the explanation! I'll try out tflite2tensorflow and openvino2tensorflow today! In the meantime, if you do happen to have the converted blob, could you attach it in the discussion here? Would be really helpful in trying it out on OAK-D before going through the conversion process. Sorry for the trouble and thanks for the prompt response! Really appreciate all your help!
Best,
Dhruv

https://s3.ap-northeast-2.wasabisys.com/temp-models/ PINTO_model_zoo_323/saved_model.zip

@PINTO0309 sorry for bothering you again. There seems to be some sort of loss during conversion. The detections are erroneous after converting it to blob format. It's not like the accuracy is low, the detections are very random across the frame and the model is not able to detect the object. Any clue on what's happening here?
For clarification, I'm converting the .onnx generated model to .blob using the blob converter since the .blob generated directly through the script doesn't work with oak-d.

Additionally, I tried to follow the code below:
import argparse
import tensorflow as tf
import sys, subprocess
#import tf2onnx

parser = argparse.ArgumentParser(description='TFLite to ONNX')
parser.add_argument('--tflite-file', type=str, required=True)
parser.add_argument('--out-file', type=str, required=True)

args = parser.parse_args()

### TF Lite
# Load TFLite model and get input/output details
interpreter = tf.lite.Interpreter(model_path=args.tflite_file)
interpreter.allocate_tensors()
input_details = interpreter.get_input_details()
output_details = interpreter.get_output_details()

# Get input and output names
input_name = input_details[0]['name']
output_name = output_details[0]['name']

# convert to ONNX
print ('Creating ONNX model...')
CMD = "python3 -u -m tf2onnx.convert --verbose --tflite {0} --opset {1} --inputs-as-nchw {2} --inputs {2} --outputs {3} --output {4}".format(args.tflite_file, 12, input_name, output_name, args.out_file)
subprocess.run(CMD, shell=True, check=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE)

This gives me an onnx model too which I convert to (.blob) through https://blobconverter.luxonis.com/ but this generates too many false positives too.

I set the model optimizer params as none and myriadx params as none too because the default params gives me a model which gives the error:
[18443010F188940F00] [<IP_ADDRESS>] [5.907] [NeuralNetwork(0)] [error] Input tensor 'mean' (1) exceeds available data range. Data size (270000B), tensor offset (270016), size (1B) - skipping inference

@PINTO0309 Any idea if the params are wrong or if I should be using different params. Input size is 300x300
Thanks and sorry for the trouble


Cannot find ViewPropTypes.style

what can i do to slove this Q?

React version: 15.3.2
React native version: 0.37.0

You can change ViewPropTypes to View.propTypes and also remove the import

downgrade react-native-scrollable-tab-view. Latest versions not compatible with react-native older than 0.44 😞

@skv-headless which version doesn't have this issue for older RN versions? 0.7.0?

@Jiepeixuan & @dozel I had had the same issue.
I solve this by downgrade the react-native-scrollable-tab-view version.
React version: 15.4.0
React native version: 0.38.0
React-native-scrollable-tab-view: 0.6.0
this may be help


Idea: using .env variable for auto generate commands

The idea
As the CLI is already reading out the DATOCMS_API_TOKEN, would it be an option to use DATOCMS_ENVIRONMENT to use if available for the --autogenerate command?
Bonus points to use a third env variable DATOCMS_ENVIRONMENT_SOURCE from which the user can tell the CLI the fork has been created.
The potential way of working
So the CLI would run something like this automagically when env vars are available:
npx datocms migration:new someMigrationFileName --autogenerate=$DATOCMS_ENVIRONMENT:$DATOCMS_ENVIRONMENT_SOURCE

Reasoning behind the idea
When developing in larger teams it might happen that you create new features and new DatoCMS environments async and create new environment on a frequent basis (especially with continuous deployments).
When one developer brings their changes to main and deploy it to a new environment, the other developers still might need to generate their migration based on an older environment.
If they in this case forget to set the source environment, they will end up generating a migration, removing all new fields/models/blocks that have been brought to primary environment.
This feature would make that process slightly less error prone ✌🏻

In case anyone else sees this, there is a workaround on the forum: https://community.datocms.com/t/datocms-cli-env-variable-usage-for-environment-names/5304/2?u=roger


Remove sidebar flickering on login screen

Where is this happening? Do we have a screenshot/video showing the issue or steps to reproduce?

Update:
I am closing this issue for the following reasons:

The "issue" has not been experienced by any other collaborators in several run tests.
Not enough report information.
No response back since May 1


Consider allowing catch to work with non-exact request types

Currently when declaring a retry with a catch, the catch verb must include the exact request type. eg:
//ftl:verb
//ftl:retry 1 1s catch catchExample
func Example(ctx context.Context, req ExampleRequest) error {...}

//ftl:verb
func CatchExample(ctx context.Context, req builtin.CatchRequest[ExampleRequest]) error {...}

This prevents these use cases:

Defining a single catch verb that can be used for multiple retryable verbs (assuming they don't have the exact some request type)
Defining a catch in FSM's retry directive, as each transition has different event types

It could be useful to have a catch verb with takes builtin.CatchRequest[map[string]Any] or just builtin.CatchRequest[Any].
Something to be careful about:

A failing catch verb  currently retries forever
If a request can't be unmarshalled into the request type at runtime we could easily end up repeatedly failing forever...

I think allowing Any makes sense.

decision: Allow Any or exact match


More specific error message as part of CAPOCI events when there are no IPs available to be attached on the new node.

What would you like to be added:
we would like to have support for specific error messages to be available in CAPOCI events when there are no IPs available to be attached to the instance being provisioned. We want the message to come from compute workrequest instead of a generic message.
Why is this needed:
A message similar to "No more available IP in CIDR <IP_ADDRESS>/24" coming from the compute workrequest would give a better idea about the issue than what we are getting right now as part of CAPOCI events, ie. "oke-7c6a84abe44f43f5a7bafd13c32d8f3d-qstlg.17a3b3bdfd981416: Instance  has invalid lifecycle state TERMINATING"

A similar case is there when cluster create fails because there are no IPs available for NLB to be created. It would be better if the message we get in the CAPOCI events could be enhanced and could come from network load balancer workrequest error saying something like "No more available IP in CIDR <IP_ADDRESS>/28", instead of "Failed to create cluster due to an Unknown error" which we are getting currently.


Add example for BSC p2p

Describe the feature
It'd be nice to have a manual p2p example for bsc
TODO

add BSC chain chain spec: https://github.com/bnb-chain/bsc/blob/master/params/config.go

Add an example, very similar to the polygon example:
https://github.com/paradigmxyz/reth/blob/1c46e5ae7d21a0696bc299fefc172d5a6cb993f6/examples/polygon-p2p/src/main.rs#L32-L32
https://github.com/paradigmxyz/reth/blob/1c46e5ae7d21a0696bc299fefc172d5a6cb993f6/examples/polygon-p2p/src/chain_cfg.rs#L10-L10
Additional context
No response

Can I take this one?

assigned
would be great if we could connect and get some txs we can print or something

Hey @Quertyy, what's the status on this? Feel free to open a draft PR if you get stuck/need feedback on the direction 😄


Update client file structure

Based on the group by file type example from: https://reactjs.org/docs/faq-structure.html

Remove yarn.lock since we're using npm
Add web3modal to npm dependencies to fix missing dependency in App.js

Looks good man

Looks good man


keyboardShouldPersistTaps=true is deprecated

https://facebook.github.io/react-native/docs/scrollview.html
keyboardShouldPersistTaps = true is deprecated use always instead. :)

this is awesome package, I just install it. It's working well but still got the warning message keyboardShouldPersistTaps = true is deprecated use always instead.

@d-a-n  : Please merge this PR


Support special latest version in apt plugin

Is your feature improvement request related to a problem? Please describe.
When installing software via the tedge-apt-plugin, I mostly want to install the latest version of the software. In the Cumulocity Device Management application, the UI forces the user to select a version to be able to install the software package.
This means that the user always has to go through the list of software version to check which is the latest version and this is tedious and erroneous. It also makes any kind of automation much more difficult.
Describe the solution you'd like
Allow a special value latest to be used for the version value and it will instruct the tedge-apt-plugin to install the latest available version (equivalent to apt-get install mypackage instead of apt-get install mypackage=1.0.0)
Describe alternatives you've considered
Allow an empty version field when creation the software operation, however this would require a change in the Cumulocity Device Management UI and an empty version value is not as logical for the users to understand what it represents.
Additional context

An example showing the user being able to select a special latest version from the software version list.

Yes, this seems like what I was wanting

This functionality exists already as you can just skip the version prefix in the expected version format <version>::<type> as in ::apt which is interpreted by the apt plugin as the latest version. But still, I agree that something explicit like latest would make it clearer.

how to test
This PR allows the user to install the latest version of package from the c8y UI, by specifying latest in the version name.

[ ] Go to Management -> Software repository -> Click "Add software"

[ ] Configure as below:

NOTE the last field is not empty, but contains an empty space ' '

[ ] Click add software

[ ] Go to your device -> Software -> Install software -> Rolldice (with latest) version -> Apply changes


create headless mode for workflow runs

Currently workflow runs require an API machine & token to run.
Remove this requirement, and allow runs to happen without API.

https://github.com/niaid/image_portal_workflows/pull/134


⚠ Jenkins Plugins Artifactory has degraded performance

In 6850d47, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1055 ms

Resolved: Jenkins Plugins Artifactory performance has improved in eb506d1 after 12 minutes.


Add Münster, Germany

Would love to add Münster, germany.

This is live now! http://click-that-hood.com/muenster
Thanks for submitting…


chore(engine): adapt postgres backend connection URI

SUMMARY
postgres connection the URI scheme designator can be either postgresql:// or postgres://.
refer: https://www.postgresql.org/docs/10/libpq-connect.html#LIBPQ-CONNSTRING

Codecov Report

Merging #11233 into master will decrease coverage by 5.59%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #11233      +/-   ##
==========================================
- Coverage   65.52%   59.92%   -5.60%
==========================================
  Files         831      393     -438
  Lines       39295    24613   -14682
  Branches     3592        0    -3592
==========================================
- Hits        25747    14749   -10998
+ Misses      13439     9864    -3575
+ Partials      109        0     -109

Flag
Coverage Δ

#cypress
?

#javascript
?

#python
59.92% <100.00%> (-0.79%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/db_engine_specs/postgres.py
100.00% <100.00%> (ø)

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/db_engine_specs/hive.py
53.90% <0.00%> (-30.08%)
:arrow_down:

superset/db_engine_specs/presto.py
69.76% <0.00%> (-12.32%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/utils/celery.py
82.14% <0.00%> (-3.58%)
:arrow_down:

superset/examples/world_bank.py
97.10% <0.00%> (-2.90%)
:arrow_down:

superset/examples/birth_names.py
97.36% <0.00%> (-2.64%)
:arrow_down:

superset/views/database/mixins.py
80.70% <0.00%> (-1.76%)
:arrow_down:

... and 443 more

Continue to review full report at Codecov.

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

I will increase the UT to prevent the coverage rate decrease. Please wait for the merge. @dpgaspar

To simplify this somewhat more, we could make BaseEngineSpec.engine: Union[str, List[str]] and add logic for mapping all keys to the engine here if it is indeed a List[str]:
https://github.com/apache/incubator-superset/blob/88af85ac53b62cef9b95fb2f7a069bd460d19c29/superset/db_engine_specs/__init__.py#L51
I suspect there are other engines that accept multiple engine dialect names, so this way we wouldn't need to create child classes for all synonyms.
Let me know what you think!

@villebro, thanks for you remind.
I read the related logic carefully. Using BaseEngineSpec.engine: Union[str, List[str]] may not be a good way, for the following reasons:

There are already many places where database.spec.engine is used to get dialect name. If we modify the engine datetype, it means that many places need to be adjusted accordingly
SQLAlchemy dialect name received is a string datatype. ref: https://github.com/zzzeek/sqlalchemy/blob/34f8622b687f33dadff4f8e81532b3f4050246f4/lib/sqlalchemy/engine/interfaces.py#L38

I refer to SQLAlchemy "translate" method support "postgres" and "postgresql". https://github.com/zzzeek/sqlalchemy/blob/6eda3aa8c5f58e415a49ac8dc73eb8608f4be306/lib/sqlalchemy/dialects/init.py#L21

There are already many places where database.spec.engine is used to get dialect name. If we modify the engine datetype, it means that many places need to be adjusted accordingly

I may be wrong, but the db engine specs are mostly used to get values back without needing to check for what the engine itself is. Was there some place in the code that referenced the value of BaseEngineSpec.engine directly, other than db_engine_specs/__init__.py?

SQLAlchemy dialect name received is a string datatype. ref: https://github.com/zzzeek/sqlalchemy/blob/34f8622b687f33dadff4f8e81532b3f4050246f4/lib/sqlalchemy/engine/interfaces.py#L38

This shouldn't cause trouble, as we'll be mapping all values in the list to the engines dict in https://github.com/apache/incubator-superset/blob/8676c3ebd6f49fd8102e68d682939728184b5638/superset/db_engine_specs/init.py#L51

I refer to SQLAlchemy "translate" method support "postgres" and "postgresql". https://github.com/zzzeek/sqlalchemy/blob/6eda3aa8c5f58e415a49ac8dc73eb8608f4be306/lib/sqlalchemy/dialects/init.py#L21

Hmm, I wonder if we could leverage this to automatically populate the dialect aliases..

There are already many places where database.spec.engine is used to get dialect name. If we modify the engine datetype, it means that many places need to be adjusted accordingly

I may be wrong, but the db engine specs are mostly used to get values back without needing to check for what the engine itself is. Was there some place in the code that referenced the value of BaseEngineSpec.engine directly, other than db_engine_specs/__init__.py?

In superset.views.core have some spec.engine to get engine name, like:

L1951
L2087
L2091

SQLAlchemy dialect name received is a string datatype. ref: https://github.com/zzzeek/sqlalchemy/blob/34f8622b687f33dadff4f8e81532b3f4050246f4/lib/sqlalchemy/engine/interfaces.py#L38

This shouldn't cause trouble, as we'll be mapping all values in the list to the engines dict in
https://github.com/apache/incubator-superset/blob/8676c3ebd6f49fd8102e68d682939728184b5638/superset/db_engine_specs/__init__.py#L51

I refer to SQLAlchemy "translate" method support "postgres" and "postgresql". https://github.com/zzzeek/sqlalchemy/blob/6eda3aa8c5f58e415a49ac8dc73eb8608f4be306/lib/sqlalchemy/dialects/init.py#L21

Hmm, I wonder if we could leverage this to automatically populate the dialect aliases..

If we modify the engine datatype, these places may need to be adjusted

LGTM! Will merge once CI passes

Thank you very much for your patient


Cosmos item ID no longer autogenerated if left blank?

Under properties of an item: User-defined unique name in a logical partition. If the user doesn’t specify the ID, the system automatically generates one.
This behavior seems to have changed in .net and js - https://github.com/Azure/azure-cosmos-js/issues/241 and the call will throw an error if no id is given.

Dokumentinformasjon
⚠ Ikke rediger denne delen. Den kreves for koblingen docs.microsoft.com ➟ GitHub-problem.

ID: d7941d92-a0ea-0f78-12af-7dc8ccb2b1ea
Version Independent ID: d6cc05c7-b724-df22-116f-b8f464293a35
Content: Work with databases, containers, and items in Azure Cosmos DB
Content Source: articles/cosmos-db/databases-containers-items.md
Service: cosmos-db
GitHub Login: @markjbrown
Microsoft Alias: mjbrown

@realolap We are checking this with our doc author and will get back to you.
@markjbrown Please check the user feedback and make changes to document if needed,

@realolap Thanks for bringing to our attention. I updated the doc to remove that statement.  When using the SDKs, you can generate a random GUID and assign it to the id property.
We will close this issue if there aren't further questions.
#please-close


The interface is not displayed properly

Hello, the interface display is abnormal when using version 2.2.2. Win10 19042.630, resolution 4k, system zoom layout 150%.

Fixed via b366d5d


"Document is damaged" on Word App

Thanks for this great library. We use it in a one of our project and it works great appart for one thing.
When we generate the docx from an iPad (with Safari on the latest version of iOS), we get a blank page returned. If we open this in the Word App (published by Microsoft), we get a « Document is damaged » warning.
We’ve try with the online demo  http://evidenceprime.github.io/html-docx-js/test/sample.html as well, but it does seems that this issue appear here too.
Do you have any work around to have Word document that could work on iOS ?

We haven't tested if Word for iOS supports altchunks feature that is crucial for our solution to work. I will do some research and get back to you.
Thank you for bringing this to my attention!

Our investigation shows that the iOS app is really a wrapper around Word Online, so it has the same problems in handling altchunks. The bottom line is that unfortunately it won't work until Microsoft adds this feature to Word Online.


Concurrent eigenda blob dispersals

Make sure to rebase this on top of #2 changes, to get monitoring when testing on devnet.
Let's make sure to track progress of https://github.com/celo-org/optimism/pull/213 and merge our ideas. :)

Closing in favor of #5


Bug when serializing JsonExtensionData of a class with a readonly property

Hello,
Our service recently went down due to what we believe is a bug in Newtonsoft. We are using version 11.0.2. Here is what happened:
We have a class named NetworkState, which contains (among other things) a readonly property called HasIngressConfig, which looks like this:
        public bool HasIngressConfig => this.IngressConfig != null;

We recently added a JsonExtensionData property called AdditionalData, which looks like this:
        [JsonExtensionData(ReadData = true, WriteData = true)]
        public IDictionary<string, JToken> AdditionalData { get; set; } = new Dictionary<string, JToken>();

Upon deploying the change that adds the AdditionalData, the service started returning the following 500 error message:
Newtonsoft.Json.JsonSerializationException: Error setting value in extension data for type 'Microsoft.SeaBreeze.NetworkResourceManager.Model.Networks.NetworkProperties'. Path 'Properties.HasIngressConfig', line 1, position 2338. ---> System.ArgumentException: An item with the same key has already been added.\r\n   at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)\r\n   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)\r\n   at Void Add(System.String, Newtonsoft.Json.Linq.JToken)(Object , Object[] )\r\n   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetExtensionData(JsonObjectContract contract, JsonProperty member, JsonReader reader, String memberName, Object o)\r\n   --- End of inner exception stack trace ---\r\n   at System.Fabric.Store.TStore`5.<>c__DisplayClass329_0.<EnumerateComponentKeys>b__0(Task`1 t)\r\n   at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()\r\n   at System.Threading.Tasks.Task.Execute()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at Microsoft.ServiceFabric.Data.ReplicatedStore.DifferentialStore.KeyValueAsyncEnumerator`2.<MoveNextAsync>d__6.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at Microsoft.ServiceFabric.Data.Collections.SnapshotAsyncEnumerator`2.<MoveNextAsync>d__9.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()\r\n   at System.Fabric.Wrp.ReliableDictionaryDataAccess.IAsyncEnumerableExtensions.<ForeachAsync>d__2`1.MoveNext() in D:\\SeaBreeze\\src\\Common\\ReliableDictionaryDataAccess\\IAsyncEnumerableExtensions.cs:line 68\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()\r\n   at System.Fabric.Wrp.ReliableDictionaryDataAccess.PersistedStore`2.<GetReadOnlyEntriesAsync>d__14.MoveNext() in D:\\SeaBreeze\\src\\Common\\ReliableDictionaryDataAccess\\PersistedStore.cs:line 87\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()\r\n   at Microsoft.SeaBreeze.NetworkResourceManager.Operation.Networks.GetNetworksAdminOperation.<ExecuteAsync>d__14.MoveNext() in D:\\SeaBreeze\\src\\NetworkResourceManager\\Operation\\Networks\\GetNetworksAdminOperation.cs:line 84\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at Microsoft.ServiceFabric.Rp.Operation.OperationBaseWithoutInstrumentation`1.<RunAsync>d__49.MoveNext()

Reverting the change seems to have fixed the issue for us. Here's what happened:

NetworkProperties is serialized, including all of its properties.
NetworkProperties is deserialized, and since the serializer can’t find a place to store HasIngressConfig that property gets put in the AdditionalData dictionary.
NetworkState is serialized again, and now HasIngressConfig is serialized twice—once from the readonly property and once from the AdditionalData dictionary.
On deserialization, the serializer tries to put both properties in the AdditionalData dictionary under the same name. This throws the exception.

I would have expected at least one of the following things to be true:

Any type that's in my class definition shouldn't be added to the ExtensionData, even if it's readonly. Otherwise, it should at least fail silently or add just one of the extra properties.
If having JsonExtensionData and readonly properties in the same class is not supported, it should be documented.

Let me know if I missed anything or if anything is unclear in my scenario.

Just did a small test and confirmed the issue:
            ResourceId networkResourceId = new ResourceId(
                Guid.NewGuid().ToString(),
                "testRG",
                ResourceConstants.ProviderNamespace(false),
                new Tuple<string, string>(ResourceConstants.NetworkResourceType, "testNetwork"));

            var newClass = new NetworkState(networkResourceId)
            {
                Properties = new NetworkProperties()
                {
                    Description = "test Network",
                    AddressPrefix = "<IP_ADDRESS>/24",
                    IngressConfig = new IngressConfig()
                    {
                        QosLevel = IngressQosLevel.Bronze,
                        Layer4 = new Layer4IngressConfig[1] {
                            new Layer4IngressConfig()
                            {
                                Name = "testIngressConfig",
                                ApplicationName = "testApplication",
                                ServiceName = "testService",
                                EndpointName = "testEndpoint",
                                PublicPort  = 45
                            }
                        },
                        PublicIPAddress = "<IP_ADDRESS>"
                    }
                }
            };

            var serializationSettings = WrpJsonSerializer.GetDefaultPublicSerializationSettings();
            serializationSettings.DefaultValueHandling = Newtonsoft.Json.DefaultValueHandling.Ignore;
            string newClassSerialized = WrpJsonSerializer.SerializeObjectToString(newClass, serializationSettings);
            Console.WriteLine(newClassSerialized);

            NetworkState oldClass = WrpJsonSerializer.DeserializeObjectFromString<NetworkState>
                (newClassSerialized, serializationSettings);
            string oldClassSerialized = WrpJsonSerializer.SerializeObjectToString(newClass, serializationSettings);
            Console.WriteLine(oldClassSerialized);

            NetworkState oldClassRoundTrip = WrpJsonSerializer.DeserializeObjectFromString<NetworkState>(oldClassSerialized, serializationSettings);
            string oldClassRoundTripSerialized = WrpJsonSerializer.SerializeObjectToString(oldClassRoundTrip, serializationSettings);
            Console.WriteLine(oldClassRoundTripSerialized);

            NetworkState roundTripNewClass = WrpJsonSerializer.DeserializeObjectFromString<NetworkState>
                (oldClassRoundTripSerialized, serializationSettings);

            Assert.AreEqual(
                WrpJsonSerializer.SerializeObjectToString(newClass, serializationSettings),
                WrpJsonSerializer.SerializeObjectToString(roundTripNewClass, serializationSettings));

This prints out:
{
  "type": "Microsoft.ServiceFabricMesh/networks",
  "id": "/subscriptions/77c04e11-f332-4543-b9da-07c13d06153d/resourcegroups/testRG/providers/Microsoft.ServiceFabricMesh/networks/testNetwork",
  "name": "testNetwork",
  "tags": {},
  "asyncOperations": {},
  "changedAsyncOperations": [],
  "properties": {
    "description": "test Network",
    "addressPrefix": "<IP_ADDRESS>/24",
    "ingressConfig": {
      "publicIPAddress": "<IP_ADDRESS>",
      "layer4": [
        {
          "name": "testIngressConfig",
          "publicPort": 45,
          "applicationName": "testApplication",
          "serviceName": "testService",
          "endpointName": "testEndpoint"
        }
      ]
    },
    "hasIngressConfig": true
  },
  "storeKey": "/subscriptions/77c04e11-f332-4543-b9da-07c13d06153d/resourcegroups/testRG/providers/Microsoft.ServiceFabricMesh/networks/testNetwork",
  "referencingResourceIds": [],
  "dereferencedResourceIds": []
}
{
  "type": "Microsoft.ServiceFabricMesh/networks",
  "id": "/subscriptions/77c04e11-f332-4543-b9da-07c13d06153d/resourcegroups/testRG/providers/Microsoft.ServiceFabricMesh/networks/testNetwork",
  "name": "testNetwork",
  "tags": {},
  "asyncOperations": {},
  "changedAsyncOperations": [],
  "properties": {
    "description": "test Network",
    "addressPrefix": "<IP_ADDRESS>/24",
    "ingressConfig": {
      "publicIPAddress": "<IP_ADDRESS>",
      "layer4": [
        {
          "name": "testIngressConfig",
          "publicPort": 45,
          "applicationName": "testApplication",
          "serviceName": "testService",
          "endpointName": "testEndpoint"
        }
      ]
    },
    "hasIngressConfig": true
  },
  "storeKey": "/subscriptions/77c04e11-f332-4543-b9da-07c13d06153d/resourcegroups/testRG/providers/Microsoft.ServiceFabricMesh/networks/testNetwork",
  "referencingResourceIds": [],
  "dereferencedResourceIds": []
}
{
  "type": "Microsoft.ServiceFabricMesh/networks",
  "id": "/subscriptions/77c04e11-f332-4543-b9da-07c13d06153d/resourcegroups/testRG/providers/Microsoft.ServiceFabricMesh/networks/testNetwork",
  "name": "testNetwork",
  "tags": {},
  "asyncOperations": {},
  "changedAsyncOperations": [],
  "properties": {
    "description": "test Network",
    "addressPrefix": "<IP_ADDRESS>/24",
    "ingressConfig": {
      "publicIPAddress": "<IP_ADDRESS>",
      "layer4": [
        {
          "name": "testIngressConfig",
          "publicPort": 45,
          "applicationName": "testApplication",
          "serviceName": "testService",
          "endpointName": "testEndpoint"
        }
      ]
    },
    "hasIngressConfig": true,
    "hasIngressConfig": true
  },
  "storeKey": "/subscriptions/77c04e11-f332-4543-b9da-07c13d06153d/resourcegroups/testRG/providers/Microsoft.ServiceFabricMesh/networks/testNetwork",
  "referencingResourceIds": [],
  "dereferencedResourceIds": []
}

As you can see, the third print has "hasIngressConfig" twice. Adding JsonIgnore to that property fixes the issue.

This is already fixed here - https://github.com/JamesNK/Newtonsoft.Json/issues/1719

v12.0.3 still has this problem. Readonly properties are getting written to extension data on deserializing and then extension data overwrites the readonly property in the json on serialization.

Hi,
I can confirm it. The issue is still present. It duplicates read only properties in a second serialization:

public class SomeStructure
{
    public string Name { get; set; }

    public string NameFirstLetter => Name.First().ToString();

    [JsonExtensionData]
    public Dictionary<string, object> AdditionalData { get; set; }
}

[TestMethod]
public void DuplicateReadOnlyProperties()
{
    var structure = new SomeStructure()
    {
        Name = "Tobias"
    };

    var json = JsonConvert.SerializeObject(structure);

    var structureReloaded = JsonConvert.DeserializeObject<SomeStructure>(json);

    var json2 = JsonConvert.SerializeObject(structureReloaded);

    Assert.AreEqual(json, json2); // Fails
}

json: {"Name":"Tobias","NameFirstLetter":"T"}
json2: {"Name":"Tobias","NameFirstLetter":"T","NameFirstLetter":"T"}

Should this issue be re-opened or a newly one created?


Document Permissions required for storage writes of checkpoint files during Delta table Sink

On the Sink stage of a Delta Transformation, every 10th json files that is generated inside the delta log of the delta table, a "summary" file called checkpoint (in parquet) is written. The APPid that writes the checkpoint is different from the Appid (Service principal) that writes the rest of the files in Dataflow processes.
This different AppId corresponds to the ObjectId of the Datafactory where the dataflow is executed; this is not documented. You must give write permissions to the corresponding datafactory AppID over the storage affected otherwise you will receive an error such as below:
'Job failed due to reason: Operation failed: \'This request is not authorized to perform this operation using this permission.', 403, PUT,

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

ID: 344ed7ed-8489-abda-f74b-3c1cbacd5e0f
Version Independent ID: 053dfc5c-e1a9-3ecd-f40a-3e8eb9040dfd
Content: Delta format in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/format-delta.md
Service: data-factory
Sub-service: data-flows
GitHub Login: @kromerm
Microsoft Alias: makromer

Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.
@kromerm , @jonburchel - Could you please review the above feedback and help update the document as appropriate - Thank you

#please-close


Fix version checks

Codecov Report
Attention: 254 lines in your changes are missing coverage. Please review.

Comparison is base (a26557b) 89.38% compared to head (a419a95) 89.25%.
Report is 228 commits behind head on beta/v4.0.

Files
Patch %
Lines

integration/test_collection_openai.py
33.33%
38 Missing :warning:

weaviate/collections/batch/base.py
85.81%
21 Missing :warning:

weaviate/collections/filters.py
73.01%
17 Missing :warning:

weaviate/collections/data.py
78.57%
15 Missing :warning:

integration/test_collection_rerank.py
26.31%
14 Missing :warning:

weaviate/collections/batch/batch_wrapper.py
83.13%
14 Missing :warning:

weaviate/collections/batch/grpc_batch_objects.py
84.41%
12 Missing :warning:

weaviate/collections/config.py
66.66%
10 Missing :warning:

weaviate/client.py
84.48%
9 Missing :warning:

integration/test_collection_batch_delete.py
94.96%
8 Missing :warning:

... and 32 more

Additional details and impacted files
@@              Coverage Diff              @@
##           beta/v4.0     #748      +/-   ##
=============================================
- Coverage      89.38%   89.25%   -0.14%
=============================================
  Files            154      175      +21
  Lines          16747    18003    +1256
=============================================
+ Hits           14970    16068    +1098
- Misses          1777     1935     +158

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


Run make gen with latest image

Please provide a description of this PR:
Needed for https://github.com/istio/common-files/pull/1108#pullrequestreview-2455627724

@jacob-delgado: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
0ff96e959b14355bbe9d5b42bf56ec2cd50163f3
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


Remove Github API integration

APIs keep changing -  I don't see much value in keeping the API integration around. Instead, this tool will simplly become a pre-commit hook. It will have a nonzero exit status if FIXMEs or TODOs are found in the code.
That way, commits will be aborted and users are reminded to create issues wherever they like.

There’s also this alternative you could link to: https://github.com/sergeichestakov/todo-to-issue


Fix issue #105 do not override the original incomplete array type siz…

…e when it is completed by an initializer for a variable. Ensure that an array type is complete at the end of an array variable declaration.

Oups... the PR fixes the issue but brings a big regression.
When the type of the array is complete, patch_size_from_initializer() returns no type and the variable ends with a None type.
I made a 2nd attempt and the regression is fixed.

Oups... the PR fixes the issue but brings a big regression.
When the type of the array is complete, patch_size_from_initializer() returns no type and the variable ends with a None type.
I made a 2nd attempt and the regression is fixed.
My commit 6221ee8 is the 2nd version.
Not sure what I can do now.

I think this change is in real good shape! One thing to top it off is to run the black code formatter, and add the excellent failing test example from issue #105


code interpreter not working

I tested the example case:
The result indicates that the weight of a brick (B) is approximately 5.172 kg and the weight of a GPU (G) is approximately 2.414 kg

The output ends with "lets do that now", and it seems the next part of it is not executed/fetched.
(Also cool project 👍, haven't seen any other telegram capable api that allows use of this beta api yet).

thanks for kudos @andaag, checked, all good.
create a new assistant with "code interpreter", add it to tutors.yaml, select one (/tutor) before questions.

note: can't send files in chat yet. if code interpreter makes a file, you won't see it in chat
from logs: tools=[ToolCodeInterpreter(type='code_interpreter'), ToolRetrieval(type='retrieval')]

I had a few conversations going with it, and I realize what happened. If the assistant split the messages in two I think the responses from gptbot is mixed. Notice how answer comes before description on how you arrive at the conclusion.


handler (storage) thread-safety

HGH's design is not thread-safe since most of the use-cases are actors themselves - thus protected from concurrent access. However, not all and esp. under JRuby (with non-GIL concurrency and likely due threaded fibers) some parts tend to break unexpectedly when a handler update is executed by 2 threads.
Here's a failure from the older stack which is still relevant today (with an updated Adhearsion 2.6 stack) :
ERROR Adhearsion::Initializer: <NoMethodError> undefined method `find' for nil:NilClass
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/has-guarded-handlers-1.6.0/lib/has_guarded_handlers.rb:149:in `guarded?'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/has-guarded-handlers-1.6.0/lib/has_guarded_handlers.rb:92:in `trigger_handler'
org/jruby/RubyKernel.java:1254:in `catch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/has-guarded-handlers-1.6.0/lib/has_guarded_handlers.rb:91:in `trigger_handler'
org/jruby/RubyEnumerable.java:556:in `find'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/has-guarded-handlers-1.6.0/lib/has_guarded_handlers.rb:89:in `trigger_handler'
org/jruby/RubyKernel.java:1254:in `catch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/has-guarded-handlers-1.6.0/lib/has_guarded_handlers.rb:88:in `trigger_handler'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/punchblock-2.5.3/lib/punchblock/translator/asterisk/call.rb:159:in `process_ami_event'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/punchblock-2.5.3/lib/punchblock/translator/asterisk.rb:209:in `ami_dispatch_to_or_create_call'
org/jruby/RubyHash.java:1357:in `each_pair'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/punchblock-2.5.3/lib/punchblock/translator/asterisk.rb:207:in `ami_dispatch_to_or_create_call'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/punchblock-2.5.3/lib/punchblock/translator/asterisk.rb:90:in `handle_ami_event'
org/jruby/RubyKernel.java:1932:in `public_send'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/calls.rb:25:in `dispatch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/calls.rb:122:in `dispatch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/actor.rb:322:in `handle_message'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/actor.rb:416:in `task'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks.rb:55:in `initialize'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks.rb:47:in `initialize'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks/task_fiber.rb:13:in `create'

@handlers ||= Hash.new { |h, k| h[k] = Hash.new { |h, k| h[k] = [] } } the underlying issue is that when the same key is "missed" (under 2 threads) the default proc might execute twice ... with TS::Cache there's an atomic guarantee of fetch_or_store method - once set we're only retrieve the same value for the given key
p.s. ThreadSafe::Cache is part of the concurrent-ruby gem these days, if there's interest in getting the changes upstream it should be straightforward to migrate using Concurrent::Hash instead.

Thanks very much @kares.   I think this looks great 👍
@benlangfeld could you take a look when you get a chance?

Awesome!  Thank you both!


Update set-up-a-custom-blocked-urls-list-atp.md

Added a note to the document stating that only 500 URLs can be added to the block list.

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/2467

@JohanFreelancer9 Please copy review


Add Crypto Currenct Tracking Website

I want to add a frontend reactjs website that uses an API to fetch the latest data about crypto prices and showcases them in a visually appealing manner

@iamrahulmahato Can you assign this to me?

already someone added, Do not be demotivated and raise a new issue.

@iamrahulmahato Are you sure? I Cannot find it on the website link


 Enable that with --remote_coredump_enabled_for_unit_test flag
Fatal Python error: Segmentation fault

Current thread 0x00007f638a000000 (most recent call first):
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/interpreters/pxla.py", line 1285 in __call__
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/profiler.py", line 333 in wrapper
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/pjit.py", line 1648 in _pjit_call_impl_python
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/pjit.py", line 1694 in call_impl_cache_miss
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/pjit.py", line 1712 in _pjit_call_impl
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/core.py", line 949 in process_primitive
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/core.py", line 443 in bind_with_trace
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/core.py", line 2777 in bind
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/pjit.py", line 189 in _python_pjit_helper
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/pjit.py", line 331 in cache_miss
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/jax/_src/traceback_util.py", line 180 in reraise_with_filtered_traceback
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/experimental/users/sbodenstein/benchmarks/microbenchmarks/cudnn_attention.py", line 59 in matmul_benchmark
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/benchmark/bindings/python/google_benchmark/__init__.py", line 130 in _run_benchmarks
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/absl/app.py", line 404 in _run_main
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/py/absl/app.py", line 484 in run
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/third_party/benchmark/bindings/python/google_benchmark/__init__.py", line 134 in main
  File "/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/runfiles/google3/experimental/users/sbodenstein/benchmarks/microbenchmarks/cudnn_attention.py", line 68 in <module>
  File "<embedded module '_launcher'>", line 35 in _run_code_in_main
  File "<embedded module '_launcher'>", line 153 in run_filename_as_main

Extension modules: jax.jaxlib.cpu_feature_guard, numpy.core._multiarray_umath, numpy.core._multiarray_tests, numpy.linalg._umath_linalg, numpy.fft._pocketfft_internal, numpy.random._common, numpy.random.bit_generator, numpy.random._bounded_integers, numpy.random._mt19937, numpy.random.mtrand, numpy.random._philox, numpy.random._pcg64, numpy.random._sfc64, numpy.random._generator, zstandard.backend_c, google3.base.python.clif.cpp_flag, google3.base.python.sysinfo, google3.base.python.user_name, google3.base.python.clif.logging, google3.third_party.pybind11_abseil.ok_status_singleton, google3.base.python.clif.googleinit, google3.base.python.absl_faulthandler, google3.net.util.python._bnsutil, google3.third_party.upb.python._message, google3.monitoring.streamz.exporter._monitoring_streamz_exporter_pywrapmetricops, google3.util.task.python._util_task_python_pywrap__task, google3.base.python.clif.tracecontext_local, google3.perftools.tracing.public.python._tracecontext_util, google3.perftools.tracing.public.python.trace_url_builder, google3.learning.debug.error_log.python.logwriters_registered_check, google3.logs.writer.python._logs_writer_python_pywraplogwriter, google3.tech.env.client.python.envelope_client, google3.learning.deepmind.analytics.lamon.python.lamon, google3.learning.deepmind.python.gpu_monitor.clif.gpu_monitor (total: 34)
E0917 11:28:52.631628    9235 process_state.cc:805] RAW: Raising signal 11 with default behavior
/build/work/80eac4b44c5f7f6a8112b8791876b518bce1/google3/tools/test/remote-runtest.sh: line 263:  9235 Segmentation fault      "$@"
-- 2024-09-17 11:28:52 PDT Forge runner: Test killed by SIGSEGV while running on glcpl1.prod.google.com

System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.34
jaxlib: 0.4.34
numpy:  1.26.3
python: 3.11.8
jax.devices (1 total, 1 local): [CudaDevice(id=0)]
process_count: 1

$ nvidia-smi
Tue Sep 17 11:31:20 2024
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.129.03             Driver Version: 535.129.03   CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA H100 80GB HBM3          On  | 00000000:08:00.0 Off |                    0 |
| N/A   30C    P0             106W / 700W |  61309MiB / 81559MiB |      0%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+

+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
+---------------------------------------------------------------------------------------+

@kaixih

I just created a PR to fix/workaround this long-standing cuDNN limitation with dbias.
cuDNN only supports dbias when the batch size is 1 (here). For batch sizes greater than 1, the cuDNN SDPA API returns an all-zero dbias tensor (which isn't ideal, but that's the current behavior). When using vmap, the API only detects a singleton batch before the vmap is applied, causing it to mistakenly set has_dbias to True, which leads to this SegFault.
This PR resolves the issue by resetting has_dbias to False, and returns an all-zero dbias tensor as in the non-vmap version.
To summarize, the behavior is:
no vmap:
bias (1, ...) => has_dbias=True => OK
bias (B, ...) => has_dbias=False => OK, but dbias is all-zero

vmap:
bias (1, ...) => has_dbias=True => OK
bias (B, ...) => has_dbias=True => Segfault # which is fixed to be
bias (B, ...) => has_dbias=False => OK, but dbias is all-zero

Also, @Cjkkkk for comments on the dbias support from cudnn.

Having d_bias be zeroes when there is a batch dimension is definitely wrong behaviour: it seems like a silent failure that would be extremely difficult to debug for users when their training curves just don't look right. I think we should fail in this case rather until cuDNN supports this.

We actually had some internal discussions earlier. I think the dilemma is this: it seems that some models don't require d_bias but still want to benefit from cuDNN flash attention. If we simply throw an error when the bias has a batch size other than 1, they might complain. Ideally, if we could detect whether d_bias is needed, we could decide whether to error out or proceed. However, it seems that no such mechanism exists in JAX. Instead, we currently have silent all-zero biases when it's not supported (which is very bad, I know....). Do you think issuing a warning would help? Or should we just throw an error whenever cuDNN+bias is used with its batch size larger than 1?

I do. Wrong outputs aren't ok, because they are the kind of thing that makes people lose trust in a library.

Sure, I will essentially move this logic to the public API to throw an error for the cudnn exec path.

By the way, do you think we should apply this error-throwing behavior to the public API or the cuDNN API? Perhaps it should only be applied to the public API, allowing power users who are certain they don't have d_bias to use the private cuDNN API.

Private APIs (jax._src, etc.) can do whatever you like. If a private API breaks, you get to keep both pieces.

I have pushed a new change to the PR to check the validity of the bias shape. It will throw an error only when the bias is invalid and bprop is applied. Also, the original segfault issues is fixed in a separate PR. @sbodenstein can you take a look?


dotnet test fails when trying to execute Assembly.DefinedTypes or Assembly.GetTypes

From @derhally on January 1, 2018 15:1
Description
The following exception occurs when executing Assembly.GetTypes() or Assembly.DefinedTypes inside .net core sdk container
Failed   Test
Error Message:
 System.Reflection.ReflectionTypeLoadException : Unable to load one or more of the requested types. Retrieve the LoaderExceptions property for more information.
  ----> System.TypeLoadException : Could not load type 'Microsoft.VisualStudio.TestPlatform.ObjectModel.Navigation.DiaSourceClass' from assembly 'Microsoft.VisualStudio.TestPlatform.ObjectModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
Stack Trace:
   at System.Reflection.RuntimeModule.GetTypes(RuntimeModule module)
   at System.Reflection.RuntimeAssembly.get_DefinedTypes()
   at System.Linq.Enumerable.SelectManySingleSelectorIterator`2.MoveNext()
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source)
   at TestIssue.TestClass.Test() in /temp/TestIssue/TestIssue/Class1.cs:line 16
--TypeLoadException

Steps to reproduce

Copy the project to a container image of microsoft/dotnet:sdk
TestIssue.zip

Run dotnet build -c Release

Run dotnet test -c Release

Expected behavior
All tests pass
Actual behavior
Exception is thrown
System.Reflection.ReflectionTypeLoadException : Unable to load one or more of the requested types. Retrieve the LoaderExceptions property for more information.
  ----> System.TypeLoadException : Could not load type 'Microsoft.VisualStudio.TestPlatform.ObjectModel.Navigation.DiaSourceClass' from assembly 'Microsoft.VisualStudio.TestPlatform.ObjectModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.

Environment
Container using sdk image
Copied from original issue: Microsoft/testfx#341

@derhally Sorry for leave you hanging, I didn't get change to investigate this, Will get back to you soon.
Adding Microsoft.VisualStudio.TestPlatform.ObjectModel:15.5.0 package reference explicitly helps?

From @derhally on January 27, 2018 18:32
@smadala any thoughts or updates on this?

From @jmeline on January 29, 2018 18:36
I'm running into the same issue. I tried the TestIssue.zip and was met with the same error. I'm interested in any updates to this issue.
Microsoft.VisualStudio.TestPlatform.ObjectModel:15.5.0 doesn't exist in Nuget. The available version 14.0.0 and still complains about not having 15.0.0
.NET Command Line Tools (2.1.4)

Product Information:
 Version:            2.1.4
 Commit SHA-1 hash:  5e8add2190

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.12
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/2.1.4/

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.5
  Build    : 17373eb129b3b05aa18ece963f8795d65ef8ea54

From @derhally on January 29, 2018 22:16
@smadala did you mean to say "Microsoft.TestPlatform.ObjectModel" ?
I tried that and still get the same error
 System.Reflection.ReflectionTypeLoadException : Unable to load one or more of the requested types. Retrieve the LoaderExceptions property for more information.
  ----> System.TypeLoadException : Could not load type 'Microsoft.VisualStudio.TestPlatform.ObjectModel.Navigation.DiaSourceClass' from assembly 'Microsoft.VisualStudio.TestPlatform.ObjectModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
Stack Trace:
   at System.Reflection.RuntimeModule.GetTypes(RuntimeModule module)
   at System.Reflection.RuntimeAssembly.get_DefinedTypes()

@derhally @jmeline
Looks like issue due to code in Microsoft.VisualStudio.TestPlatform.ObjectModel.dll using ComImport. I have filed issue on corefx(https://github.com/dotnet/corefx/issues/26851).
Downgrading Microsoft.NET.Test.Sdk to 15.3.0 is one of available workaround.

@smadala Thanks.  Downgrading to Downgrading Microsoft.NET.Test.Sdk to 15.3.0 worked for me.

@derhally , we have identified the root cause for this issue, which is because we are using ComImport Attribute on certain classes in our ObjectModel dll. We need this to create certain Com Objects, & then use P/Invoke to get relevant data regarding Navigation of test case object.
It's fair that these object cannot be initialized on dotnet core in Unix, but we were expecting dotnet to allow users to atleast reflect upon types. We have filed a bug on dotnet, please monitor it for further details. I'll be closing this issue.

@derhally Updated workarounds here
/cc  @dukeofharen @david-x-chen @FrankSzendzielarz  You might facing same issue. From https://github.com/Microsoft/testfx/issues/203#issuecomment-354124177 https://github.com/Microsoft/testfx/issues/203#issuecomment-347289293
https://github.com/Microsoft/testfx/issues/203#issuecomment-360442789


Suggestion of changing the commands to install and update a plugin

I suggest run this command to install a plugin:
git clone -b master --single-branch --depth 1 {repo}
This command to update a plugin:
git fetch --depth 1 origin master && git reset --hard @{u}
Explanation:

We don't need to download history commits, so --single-branch --depth 1 is better.
The plugin folders are not supposed to be touched by user. Therefore, reset --hard is better than pull as sometimes pull doesn't work because of local changes by any reasons.

I also need "--depth 1", how to option for this

I agree. For example updating nerd-fonts takes forever (and A LOT of space) because that repo is huge AND has a lot of stale branches. --no-single-branch was the issue. I wonder why that's needed?
Btw, man git-clone states that --depth N imples --single-branch.


[client + contracts] Intro questing system

When players first launch the game, a series of quests should exist that guide the player through the mechanics.
Technically we do this by

Custom events in the Contracts tied to a wallet address. Once complete the events send resources/rewards to the entity.
Reflect these events in the client via some window structure.
Once events are emitted, the client will no longer show these as incomplete.

[ ] game tour - navigates around the map to shop key UI parts

@ponderingdemocritus can I work on this?