Packaging issues

Packaging issues
I just updated the nixos retdec package from version 3.0 to 3.2. That was not a pleasant experience, which is probably why it remained outdated for so long. I don't know if distribution packaging is a priority for this project, but in case it is I will outline the problems here.
Retdec-support is huge
Only 127M compressed but 4G uncompressed. That takes up space on our mirror server but, more importantly, on every users computer. We currently remove the PE static code patterns (3G) by default and only install these when explicitly requested by the user. I've opened https://github.com/avast-tl/retdec-support/issues/3 for that.
Dependencies are fetched at build-time
This is by far the biggest issue. NixOS and many other distros separate the build in two distinct phases: one to fetch the sources, one to build. The build phase is executed in a sandbox that does not have internet access. There are various reasons for that, including:

reproducibility: If the build process could be influenced by the availability and content of files on the internet, there is no way to guarantee two different builds will yield the same result of even succeed at all.
security: Some of these files could be replaced by malicious ones.
mirroring: We are able to mirror all the source files in a content-addressable manner on our build servers.

Since the retdec build process fetches its dependencies at build time, a lot of patching is required. We have to fetch all the dependencies ourselves and then patch every CMakeLists file to use those local versions instead. Since that essentially duplicates parts of the build system, it makes a lot of manual adjustments during upgrades necessary.
I've started discussing this at https://github.com/avast-tl/retdec/issues/279.
Dependency versions are very specific
All dependencies are fetched by commit-hash. That makes it very difficult for distros since they usually try to ship one version of every package and share those dependencies. It is generally impossible to judge the true intentions behind the build hash:
Is it "we need at least this version since we depend on feature x" or "newer versions break the api" or "we need exactly version 2.3"?
This and the lack of a central file that collects the necessary dependency versions makes it painful to adjust the dependency versions on an upgrade.
Dependencies are forked
Capstone and yara are forked. Can the changes not be upstreamed?
Dependencies are built from source
While a distro would ideally build all the dependencies themselves and then build retdec based on those pre-compiled dependencies, retdec instead insists on building everything itself. The exception to this is openssl, which is only built if it cannot be found on the system. That should be the case for all dependencies.

Thank you for your reply, I'm glad to see that you take the issues seriously even though they are not top priority. TL/DR: solving (3) would go a long way and (2) and (4) basically wouldn't matter anymore from a packaging perspective.

1. Retdec-support is huge
* See [comment](https://github.com/avast-tl/retdec-support/issues/3#issuecomment-429791800).

* Some signatures are not very helpful for general RetDec use. They are mostly good for our regression tests, but it is unlikely they would hit many binaries in a real world. They are too specific for that. These are mostly gcc signatures for compilers we used to generate binaries in regression tests.

* The biggest signatures are however for Delphi and MSVC, which have a distinct versions used all over, and therefore a decent chance to hit in real binaries.

Since the non-compiled signatures are relatively small, would it be reasonable to work with those and compile them to a cache on the fly? That should significantly cut down on the space overhead but also only take additional time on first use.

* Instead of compiled YARA, we could distribute text YARA and add a script that would compile the rules. This script could be triggered when RetDec is being installed to system. Package would be smaller, but rules would be compiled only once and there would be no performance penalty.

I think distributing text YARA and building it during the regular build process would be better practice than shipping the compiled results in either case. But if all rules are compiled on first use that would be no real space improvement. As I suggested before, would it be possible to compile them as-needed?

* We could experiment with signature sizes - maybe they do not have to be so huge.

* Make it possible to selectively choose which signatures to install (or none at all). Exact mechanism is to be discussed - e.g. different distro packages?

Maybe. In that case there would need to be some way to notify the user when he gets sub-optimal results because of missing signatures so that he knows he has to install those.

2. Dependencies are fetched at build-time
* This is caused by CMake. How can we hack it?

* Related to #279.

* What are you allowed to do in the first phase when you are fetching the sources? Are you allowed (can you) run CMake's configure step? I.e. if we get all the resources in this step, would it be ok? We could script CMake to download everything that is needed and then configure itself as suggested in #279 to use these local instances instead of downloading them in the build step. If this is not possible, would some script that would do this be an acceptable solution? If the only thing you can do is `git clone`, then I'm not sure we can help.

Whatever the "fetch phase" produces has to be compared against a known hash. So basically the rule is that we can do anything that will be 100% binary reproducible. If the whole configure step would be binary reproducible it would be possible, although a bit of a hack.
I don't know much about cmake -- is it not possible to seperate fetching the dependencies from the configure phase?

* It would be possible to create a git meta-repository that would use git submodules as you need, and then configure (#279) build to use them. This repo would be used only to build the package for packaging. It would not be a part of the official organization. I'm not sure if you want to do something like this. I hope some of the above solutions could work instead,

That would have all the disadvantages of both solutions wouldn't it? I thought the difficulty in maintaining such a repository is the main reason for using cmake instead of submodules in the first place?

3. Dependency versions are very specific
* Analyze the dependencies we use and determine if we can use system packages. This should be possible in most of the cases.

That would really be the greatest help. If the build system would use traditional autoconf-style configure checks of the dependencies (dep x can be found and has feature y / is at least version z), it wouldn't really matter very much how dependencies are fetched. Mabe a ./configure --local would disable fetching altogether and instead just say
checking for dependency x version >= y... Not found

and then error. That would solve a lot.

* If not possible, devise a solution.

4. Dependencies are forked
* Capstone could/should be merged to upstream.

This would be great and should also ease the maintenance burdon on the retdec team in the long run.

* Our YARA fork is setting some custom limits that are hardcoded in upstream YARA. This cannot be merged in the current state. Possible solution is to make it more general - modify YARA so that these limits can be set at build, or even runtime, and try to merge that. Then YARA could use its default limits, and we could set our own. It is not guaranteed that such a pull request would be accepted.

The runtime solution would be perfect. The buildtime solution would improve matters but would still require having two seperate yara packages.

5. Dependencies are built from source
* I think this would be solved by solving points 3. and 4.

Yes.


WIP postgres: Overhaul configuration

By default, Postgres define the public schema as available for every
created database. Newly created databases can benefit of this as not
usual create custom schema on test databases.
Signed-off-by: Helio Chissini de Castro<EMAIL_ADDRESS>Please ensure that your pull request adheres to our contribution guidelines. Thank you!

@heliocastro You mentioned on Slack that the schema property is missing in the documentation somewhere, can you add another commit to this PR and fix this as well? This would be useful even if we have a default value for the schema.

@heliocastro, has this PR been superseded by https://github.com/oss-review-toolkit/ort/pull/3982? If so, can we close this one? Also, why did you not simply update this PR?

Closing this as #3982 has been merged.


[BUG] Update permission should change roles/groups/accounts

Describe the bug
A clear and concise description of what the bug is.
Update permission currently only updates the permission in the permissions collection. It needs to update roles/groups/accounts as well with the new data.
To Reproduce
Steps to reproduce the behavior:

Change a permission
Retrieve role containing permission
Permission is unchanged in role

Expected behavior
A clear and concise description of what you expected to happen.
Updating a permission should update the permission data everywhere in the system including all roles, groups, and accounts

Should updating the permission count as updating the roles, groups, accounts, and system accounts in terms of the date_updated fields?


[FEATURE REQUEST] Provide health check support

https://github.com/grpc/grpc/blob/master/doc/health-checking.md


v3-beta7: ionic cordova run android without adding platform first fails

What version of the CLI are you using? Output of ionic --version
3.0.0-beta7
Short description of the problem:
Calling ionic cordova run android without adding platform first fails:
> ionic cordova run android
✔ Installing the platform: cordova platform add android - done!
\ Installing the project plugins: cordova plugin add --save <plugin> { Error
    at FatalException.Exception (C:\Users\Jan\Documents\cliBeta7\node_modules\@ionic\cli-utils\dist\lib\errors.js:8:23)
    at FatalException (C:\Users\Jan\Documents\cliBeta7\node_modules\@ionic\cli-utils\dist\lib\errors.js:17:9)
    at Shell.<anonymous> (C:\Users\Jan\Documents\cliBeta7\node_modules\@ionic\cli-utils\dist\lib\shell.js:37:27)
    at Generator.throw (<anonymous>)
    at rejected (C:\Users\Jan\Documents\cliBeta7\node_modules\@ionic\cli-utils\dist\lib\shell.js:5:65)
    at process._tickCallback (internal/process/next_tick.js:109:7) name: 'Exception', exitCode: 127 }

What behavior are you expecting?
Error message that I should add a platform first OR automatically adds platform, then runs project on platform.
Steps to reproduce:

Install the v3-beta7 CLI
Start project, go into directory
Execute ionic cordova run android

Post the output of ionic info below please
  Your system information:
    Cordova CLI: Not installed
    Ionic Framework Version: 3.0.1
    Ionic CLI Version: 3.0.0-beta7
    ios-deploy version: Not installed
    ios-sim version: Not installed
    OS: Windows 10
    Node Version: v7.9.0
    Xcode version: Not installed

I am not able to reproduce this with the given steps. I see that it's erroring with a status code of 127, which generally means the command being executed doesn't exist. This is strange. Perhaps one of the plugins is running a command that doesn't exist on your computer?

I'm on Windows 10.
I redid all the steps, fresh beta7 install and then:

ionic start
Go to created directory
ionic cordova run android

Here is the complete output:
PS C:\Users\Jan\Documents> ionic version
3.0.0-beta7
PS C:\Users\Jan\Documents> ionic start
? What would you like to name your project: cliBeta7_3
? What starter would you like to use: blank
✔ Creating directory C:\Users\Jan\Documents\cliBeta7_3 - done!
✔ Downloading 'blank' starter template - done!
✔ Updating project dependencies to add required plugins - done!
✔ Creating configuration file for the new project - done!
✔ Executing: npm install within the newly created project directory - done!

♬ ♫ ♬ ♫  Your Ionic app is ready to go! ♬ ♫ ♬ ♫

Run your app in the browser (great for initial development):
  ionic serve

Run on a device or simulator:
  ionic cordova:run ios

Test and share your app on a device with the Ionic View app:
  http://view.ionic.io

? Link this app to your Ionic Dashboard to use tools like Ionic View?
\ Installing the project plugins: cordova plugin add --save <plugin> { Error
    at FatalException.Exception (C:\Users\Jan\Documents\cliBeta7_3\node_modules\@ionic\cli-utils\dist\lib\errors.js:8:23)
    at FatalException (C:\Users\Jan\Documents\cliBeta7_3\node_modules\@ionic\cli-utils\dist\lib\errors.js:17:9)
    at Shell.<anonymous> (C:\Users\Jan\Documents\cliBeta7_3\node_modules\@ionic\cli-utils\dist\lib\shell.js:37:27)
    at Generator.throw (<anonymous>)
    at rejected (C:\Users\Jan\Documents\cliBeta7_3\node_modules\@ionic\cli-utils\dist\lib\shell.js:5:65)
    at process._tickCallback (internal/process/next_tick.js:109:7) name: 'Exception', exitCode: 127 }

PS C:\Users\Jan\Documents\cliBeta7_3> ionic info

  Your system information:

    Cordova CLI: Not installed
    Ionic Framework Version: 3.0.1
    Ionic CLI Version: 3.0.0-beta7
    ios-deploy version: Not installed
    ios-sim version: Not installed
    OS: Windows 10
    Node Version: v7.9.0
    Xcode version: Not installed

Probably related to #2094

Yes, #2094 was caused by me not having Cordova installed (because I got a new node install for testing the CLI beta). There the error was just a bit short - but this one here it is really not helping at all because it doesn't even say that "Cordova is not installed". So this should probably handle missing Cordova in a different way.

@piotrowski Thank you. I will be working on improving the messaging here to make it more clear.

Closed in favor of https://github.com/driftyco/ionic-cli/issues/2103


tpl/tplimpl: Update schema template

Changes:

Remove trailing comma from list of keywords.
Improve keywords precedence:

Use "keywords" term page titles.
Use "keywords" from front matter if "keywords" is not a taxonomy.
Use "tags" term page titles.
Use term page titles from all taxonomies.

Enable schema for all page kinds, previously limited to kind = page.
Remove trailing slashes from void elements.
Improve readability.

Closes #7570
Co-authored by: 0urobor0s<EMAIL_ADDRESS>

@jmooring do you also plan to remove the useless trailing slash from the other internal templates? If not I could try doing it later.


Modal without fade class flickers

Using beta v2 and the latest Chrome in Windows 10, all modals have an ugly flicker for a half second when hiding/showing the modal if they do not have the fade class. I do not want the fade class, I'd like modals to show/hide instantly.
See this fiddle: https://jsfiddle.net/4rffwu28/
Notice how for a split second when hiding/showing the modal theres some weird flickering action going on. This also wasn't happening with beta v1. I'm not sure what changed in v2 to cause this.

Windows 10 Home
Version 1703
OS Build 15063.674
Chrome: Version 61.0.3163.100 (Official Build) (64-bit)
Chrome: Version 62.0.3202.75 (Official Build) (64-bit)
Unable to reproduce this issue using the supplied link. Modal shows and hides without flicker.


hi, do you have any plan for EF .NET 5.0?

when is ready for .NET 5.0?

when is ready for .NET 5.0?

Hi,
I meant Ef core 5 which is the current version right now, released this November 2020.

Hi @gurelsoycaner,
Do you have any problem on .NET 5?
Please share information, I'm looking for the library and not sure this one is okay or not.
Thank you.

maybe this weekend

hi,
any progress about moving to EF CORE 5?


[BUG] Key/value 503 when requeuing message

Is there an existing issue for this?

[X] I have searched the existing issues

What version of workers-rs are you using?
0.0.18
What version of wrangler are you using?
3.16.0
Describe the bug
I have a queue consumer that sends a message similar to the handled one on the same queue before the 30 second timeout occurs so it can pick up where the current run left off. Unfortunately this often leads to the following failure:
KV GET failed: 503 Service Temporarily Unavailable

I've added an extra 'requeue' queue separate from the initial queue to confirm that it's only 'requeued' events that have this issue and that's indeed the case.
Does this have anything to do with how a queue message is routed to a specific instance depending on it's origin? Retries with exponential backoffs don't help out either, so it seems rather that the instance it runs on has an issue than it's a rate limiting issue.
I see it in many of our applications, where we use a similar pattern of requeuing events before execution times out. I'm under the impression that this wasn't always the case, but then again my observability was lacking in the past and we've scaled up the message rates in the meantime as well. Any thoughts?
Steps To Reproduce

Create a worker that produces a message and sends it to queue A
Have the same worker act as a consumer and pick up the message from queue A, do something with key/value and then send the processed message onwards to queue B
Have the same worker now also consume messages from queue B and see it fail on KV access.

Hi @JorritSalverda - thanks for reporting.
It's surprising to me to see a KV error. Are you also reading to/writing from a KV namespace in the same queue consumer?
For now I'll try to repro this on my side.

Hi @rts-rob I do write to KV indeed, but not the same key. It already fails on the first read. Might be hard to repro, but happy to jump on a call if it helps at any point.

@JorritSalverda I attempted to repro in this repository but could not with either version 0.15 or 0.18. Can you take a quick look and see if I missed something?
https://github.com/rts-rob/workers-rs-issue-414

@rts-rob I see you send a message to queue B, but in my case things don't start failing until I consume from the queue B and interact with the key/value store.
Essentially the following happens within a single application:
cron trigger -> produce event -> queue A -> consume, interact with KV and requeue -> queue B -> consume and interact with KV

In the first consumer all goes fine, it isn't until the last consume action that KV actions start to return the 503. I first thought it might have to do with the volume of messages and rate limiting from KV's side, but then again it only happens in that last step, not in the first consumer.
Initially I only used one queue and requeued to itself, but then I couldn't distinguish whether the KV errors happened for all events or just for requeued ones. It's not exactly the same message I requeue, just the payload.

Aha - This is not an issue with workers-rs. You're running into a known issue with the Queues service itself.
In cases like this the combined "journey" of that request exceeds the subrequest limit. That's why it's erroring out down around the 10th subrequest.
We have a ticket open to track this internally but unfortunately cannot give you an ETA. I'll get the bug label added to this issue and leave it open until it's fixed. If you want to discuss further please ping me in the #queues-beta channel in our Discord server.

@rts-rob thanks for investigating and confirming that it's indeed a current limitation of queues. I'll look for a workaround - like counting the number of subrequests and stop requeuing in time - until the internal issue and limitation has been addressed.

Hi @JorritSalverda - we just rolled out a fix for this. Any chance you can try the previous behaviour and confirm whether it now works as expected?

Hi @rts-rob I can now easily requeue events more than 20 times, so indeed seems to fixed!


Export to pdf

Is there a way to export the data into a pdf format?

Sorry, currently it doesn't support export to PDF.


Constant Auto-refreshing

When I open the file browser in a larger project, it constantly refreshes. I have noticed when I am at the top of the list, the problem doesn't appear. When the selection line causes the view to scroll, the auto refresh starts back up. This appears even when { "dired_autorefresh": false } is in sublime's settings.

Same issue here.

Having the same issue. Mostly its the "expanding" a nested directory and then scrolling. Anyone have any insight?

Is there any estimate when a new version with this fix is going to be released? This auto refreshing issue is so annoying..

@ksarunas If you install it manually instead of Package Control, it contains the fix.

@zackshackleton thanks for a suggestion, don't know why I did not think of that. Works great now. Anyways, would be nice if this fix was on Package Control too!


change pause number color

Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Fixes #2731
Type of change
I make the new class for pause number  to change the color of it
and it is done according to the style pattern of the code
add class to app.scss
add variable of pause color to var.scss
add class to pause number of DashboardHome.vue
Please delete any options that are not relevant.

User interface (UI)
Other

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
[x] My changes generate no new warnings

@louislam please check and merge the updates

A screenshot of the change could be helpful to get and indication of what it looks like.

A screenshot of the change could be helpful to get and indication of what it looks like.
backend server is not working from my side so i can't share any screen shot

A screenshot of the change could be helpful to get and indication of what it looks like.

I would argue that unknown should be orange and paused should remain grey. Orange indicates some sort of issue that requires attention but is not urgent, or something that could become urgent if it is not seen to, a category that unknown seems to fit. Paused doesn't seem to fit this description and I think it is fairly common in web interfaces to have a paused / disabled feature greyed out.

I would argue that unknown should be orange and paused should remain grey. Orange indicates some sort of issue that requires attention but is not urgent, or something that could become urgent if it is not seen to, a category that unknown seems to fit. Paused doesn't seem to fit this description and I think it is fairly common in web interfaces to have a paused / disabled feature greyed out.

Okay I will do changes for unknown as you mention

Hmm then this isn't doing what the title says, and doesn't actually solve the original issue. 😂 IIRC Unknown only happens if the monitor has just started but doesn't have the first beat yet, which is also something normally users don't need to worry about.

Yeah, I am not sure a change is really needed here. It was more just if we wanted to have a different colour for each of the numbers, unknown should be the one that got changed not the paused monitors one, at least if the colour it was going to be changed to was orange.

I'm not fussed on the specific colour. I just want to be able to more easily see ones that are paused.

Need any changes @Computroniks @Justinzobel

This doesn't seem to change the pause colour at all, so it is irrelevant to me.

But @Computroniks need to change in unknown rather then pause so I will do in that way

But @Computroniks need to change in unknown rather then pause so I will do in that way

I think making the paused number orange was a bit strange and that if anything was to be orange it should be the unknown number. A different colour could be more appropriate for paused monitors

But @Computroniks need to change in unknown rather then pause so I will do in that way

I think making the paused number orange was a bit strange and that if anything was to be orange it should be the unknown number. A different colour could be more appropriate for paused monitors

Let me know which color did you want I will update it

Now I think it is fine @Computroniks @Justinzobel

It didn't solve the original issue 🤷♂

Let me know your requirement for solving original issue


[WIP] Support MRI 3.0

This is built on top of #79, so please merge that first.
Treat MRI 3 the same as MRI 2. This will work as long as the as-yet-unreleased version of debug_inspector is installed. I've assumed that's going to be 1.0.0. So until that's been released, this PR is WIP.

[ ] Merge #79
[ ] Release debug_inspector v1.0.0
[ ] Remove debug_inspector custom source from this Gemfile

Test run is here: https://github.com/BetterErrors/binding_of_caller/actions/runs/447503283


fix(popup): avatar circle on stacks transactions

Try out this version of Leather — download extension builds.

Addresses #4472
Bug description
The issue is coming from useCurrentStacksAccount hook. So for the account name (that "Account X" title) we’re using the useCurrentAccountDisplayName hook that simply uses useCurrentAccountIndex hook that in most cases just reads default current account index.
In case of the account circle, we use useCurrentStacksAccount hook, that utilizes 2 hooks to decide on the account index. One is useCurrentAccountIndex (that is most probably a default current account index) and the other one is useTransactionAccountIndex. Now useTransactionAccountIndex searches stxAddress in txPayload, and if finds one, tries to find an account with that stxAddress. It finds one in another account and returns that another account.
Now it finds stxAddress only in stacks related transactions, thus the issue only happens when we sign stacks transactions.
Now, to fix the issue with the account circle i would just use useCurrentAccountIndex together with useStacksAccounts to make the account circle consistent with the name. But the logic behind useCurrentStacksAccount still seems wrong to me 🤔 It is probably going to breed more bugs if we don't refactor it (decouple stx logic from non-stx logic)
UI test
Reproduced bug
https://github.com/leather-wallet/extension/assets/22010816/d6ccffff-59da-45c7-8804-0b8eda233bda
Fixed bug
https://github.com/leather-wallet/extension/assets/22010816/6ed42b17-33e4-4a86-98e4-801c20ac02e7

@pete-watters re-ran the job and it passes now. Probably related to: #4502

Thanks for taking this on @edgarkhanzadian, indeed you're highlighting some legitimate flaws in the design of the code. This relates to the differences in the implementation between the "legacy" JWT-based stacks requests, and the newer .request API the wallet exposes.
useCurrentStacksAccount was written in a way where, for each request that can change the user's default address, we'd have to add explicit clauses in this hook to check each one for a possible account index. This isn't very easy to maintain as it's easily forgotten and requires this hook to be aware of unrelated implementation details.
With the .request methods, we use URLSearchParams to pass generic request-agnostic info (e.g. account index on signPsbt) to the pop up, allowing requests to globally alter the app state
For now I think your current approach is okay, as we later aim to remove these APIs.

@kyranjamie oof that's tough, thank you for a very detailed explanation ! I'll keep an eye on the differences of those 2 approaches in the code.


Support Multipart Content

Allows sending multiple commands in a single request:

Each command will ultimately be handled independently and may succeed / fail in their own right.
Will need a response message to show status of each command handled.
Consider handling the commands in parallel or randomized order.
Leverage stream based handling status #31

Not going to be implemented. 1 command per request.


Add CustomSonyTVList

Summary
This blocklist blocks smart TV metadata, tracking, and ad traffic for Sony Android TV's; and the same for default applications installed on these devices.
Blocklist Homepage
https://github.com/twcau/AdblockRules
Blocklist Link
https://raw.githubusercontent.com/twcau/AdblockRules/master/CustomSonyTVList

Thanks I'll check it out and decide soon 😃

Added to the generator, should take affect in the next commit.


fix: apply theme's token to LoadingProvider component

Github issue #1100
Description
apply theme's token to LoadingProvider component
Current behavior (updates)
no change
New behavior

Please describe the behavior or changes this PR adds

Is this a breaking change (Yes/No):

Additional Information

@taroj1205
I ran pnpm changeset, please confirm.

@taroj1205
Corrected, please confirm.

Thanks!


Add MIT License

To encourage Tezos adoption, it would be helpful to add a license to this repo.
MIT license pasted from:
https://opensource.org/licenses/MIT

Thanks for the PR!


biblStruct XML: multiple editors, single  tag

Current (0.7.x) GROBID output for multiple authors results in multiple <author> tags, each with a <persName>. But multiple editors results in a single <editor> tag with multiple nested <persName>.
I'm not an expert at TEI-XML, but a quick skim of editor tag docs make it look like multiple <editor> tags would be the norm.
The relevant source code is https://github.com/kermitt2/grobid/blob/master/grobid-core/src/main/java/org/grobid/core/data/BiblioItem.java#L2382
Here is an example citation string:
Uhlířová, K., Drumbl, M. (forthcoming 2020): Actors and Law Making in International Environmental Law, in M. Fitzmaurice, M. Brus and P. Merkouris (eds.): The Research Handbook on International Environmental Law. Cheltenham: Edward Elgar Publishing. 50 p.
and TEI-XML output from processCitation (via https://cloud.science-miner.com/grobid/ just now):
<biblStruct >
    <monogr>
        <author>
            <persName>
                <forename type="first">K</forename>
                <surname>Uhlířová</surname>
            </persName>
        </author>
        <author>
            <persName>
                <forename type="first">M</forename>
                <surname>Drumbl</surname>
            </persName>
        </author>
        <title level="m">Actors and Law Making in International Environmental Law</title>
        <editor>
            <persName>
                <forename type="first">M</forename>
                <surname>Fitzmaurice</surname>
            </persName>
            <persName>
                <forename type="first">M</forename>
                <surname>Brus</surname>
            </persName>
            <persName>
                <forename type="first">P</forename>
                <surname>Merkouris</surname>
            </persName>
        </editor>
        <meeting>
            <address>
                <addrLine>Cheltenham</addrLine>
            </address>
        </meeting>
        <imprint>
            <publisher>Edward Elgar Publishing</publisher>
            <date type="published" when="2020" />
            <biblScope unit="volume">50</biblScope>
        </imprint>
    </monogr>
    <note>The Research Handbook on International Environmental Law</note>
</biblStruct>

I can prepare a patch if that is helpful.

Hi @bnewbold !
Yes indeed <author> and <editor> are not consistent and better to have both following the current <author> approach. I think the motivation to have one <author> per person is to group the person with her/his affiliation. As there is no affiliation normally coming with an editor, I have forgotten to separate the <editor>.
PR is super welcome as always, thanks !!


Failed to run fuzz tests (target: fuzz_insert_logical_table.rs )

          https://github.com/GreptimeTeam/greptimedb/actions/runs/9755006232/job/26922900846?pr=4240

Is this related to https://github.com/GreptimeTeam/greptimedb/pull/4243? @WenyXu
thread '<unnamed>' panicked at tests-fuzz/targets/fuzz_insert_logical_table.rs:307:35:
fuzz test must be succeed: 0: Failed to execute query: CREATE TABLE `CupidiTaTE`(
ts TIMESTAMP(3) TIME INDEX,
val DOUBLE,
`eT` STRING,
`VoLUPTaTe` STRING,
`ASPerNatur` STRING,
`ExPLiCabo` STRING,
PRIMARY KEY(`VoLUPTaTe`, `ASPerNatur`, `eT`, `ExPLiCabo`)
)
ENGINE=metric with ("on_physical_table" = "doLORuM");, at tests-fuzz/targets/fuzz_insert_logical_table.rs:256:14
1: Database(MySqlDatabaseError { code: Some("HY000"), number: 1815, message: "Region `4647154614272(1082, 0)` already exists" })

Originally posted by @evenyag in https://github.com/GreptimeTeam/greptimedb/issues/4240#issuecomment-2201919238

Metasrv logs:
...
common_procedure::local::runner: Failed to execute procedure metasrv-procedure::CreateLogicalTables-483316c2-e426-45f0-996d-27adac338c5e, retry: true err=0: Procedure exec failed
2024-07-02T04:43:01.450796819Z stdout F 1: Retry later
2024-07-02T04:43:01.450803552Z stdout F 2: Failed to request RegionServer my-greptimedb-datanode-0.my-greptimedb-datanode.my-greptimedb:4001, code: The operation was cancelled, at src/client/src/region.rs:207:31
2024-07-02T04:43:01.450809032Z stdout F 3: Timeout expired, at src/client/src/error.rs:186:23
2024-07-02T04:43:01.451059994Z stdout F 2024-07-02T04:43:01.450873Z  INFO LocalManager::submit_root_procedure: common_procedure::local::runner: Procedure metasrv-procedure::CreateLogicalTables-483316c2-e426-45f0-996d-27adac338c5e retry for the 1 times after 500 millis
2024-07-02T04:43:01.964294762Z stdout F 2024-07-02T04:43:01.964048Z ERROR LocalManager::submit_root_procedure: common_procedure::local::runner: Failed to execute procedure metasrv-procedure::CreateLogicalTables-483316c2-e426-45f0-996d-27adac338c5e, retry: false err=0: Failed to execute procedure due to external error
2024-07-02T04:43:01.964319048Z stdout F 1: Failed to operate on datanode: peer-0(my-greptimedb-datanode-0.my-greptimedb-datanode.my-greptimedb:4001), at src/common/meta/src/ddl/utils.rs:36:27
2024-07-02T04:43:01.964324909Z stdout F 2: External error, at src/client/src/region.rs:62:31
2024-07-02T04:43:01.96433066Z stdout F 3: Failed to request RegionServer my-greptimedb-datanode-0.my-greptimedb-datanode.my-greptimedb:4001, code: Some entity that we attempted to create already exists, at src/client/src/region.rs:207:31
2024-07-02T04:43:01.964335188Z stdout F 4: Region `4647154614272(1082, 0)` already exists, at src/client/src/error.rs:186:23
2024-07-02T04:43:01.964561995Z stdout F 2024-07-02T04:43:01.964427Z  INFO LocalManager::submit_root_procedure: common_procedure::local::runner: Procedure metasrv-procedure::CreateLogicalTables-483316c2-e426-45f0-996d-27adac338c5e retry for the 1 times after 500 millis
2024-07-02T04:43:02.478148929Z stdout F 2024-07-02T04:43:02.477952Z  INFO LocalManager::submit_root_procedure: common_procedure::local::runner: Runner metasrv-procedure::CreateLogicalTables-483316c2-e426-45f0-996d-27adac338c5e exits

The error is returned by
https://github.com/GreptimeTeam/greptimedb/blob/db5d1162f0f0087a17e4deed2b6ceeb483ac4bb7/src/metric-engine/src/engine/create.rs#L221-L224
However, we actually has a check before the region creation:
https://github.com/GreptimeTeam/greptimedb/blob/db5d1162f0f0087a17e4deed2b6ceeb483ac4bb7/src/metric-engine/src/engine/create.rs#L171-L178
It seems that there are two create requests executed at same time?
Create region A               | Create region A
Check exists                  | Check exists
Create region                 |
                              | Create region (Will fail)


Issue with module dependency

This module pulls in a dependency on<EMAIL_ADDRESS>in the file
https://github.com/romnn/testcontainers/blob/c0e2a685eb012c415e05810fbd15a2519d75e397/log.go#L7
where it uses the log package.
This version of the dependency is quite out of date and the log package in github.com/prometheus/common is actually now deprecated (see https://github.com/prometheus/common/pull/306, https://github.com/prometheus/common/releases/tag/v0.27.0)
This dependency causes some issues when using this module in a module which uses a more recent version of github.com/prometheus/common as go can't detect the correct module version of github.com/prometheus/common to use (due to the log package removal being a breaking change). Consumers end up with errors of the form:
> go mod tidy
....
        github.com/romnn/testcontainers imports
        github.com/prometheus/common/log: module github.com/prometheus/common@latest found (v0.32.1), but does not contain package github.com/prometheus/common/log

I looks like this dependency could be completely removed as it is only used to log an error. I wonder if the standard go log package would suffice here...
Happy to raise a PR if you think that this is a suitable solution.

Dear @bvwells, thanks for your contribution!
Excuse the late follow up, I have not been actively working on this. I just updated the dependencies and refactored the code, which should resolve this issue (only standard go logging is used).
Best, Roman


Back Arrow still an issue.

Back arrow is not always present, even though back is  possible. (I use the back button on my mouse. Works everytime, arrow present about half the time)
Will not start full screen although selected in properties to open full screen. Opens about %25 size window. This may be an issue on my side though.
Keep up the good work. Play Music Rainmeter skin also works and looks great with GPMDP.

Merging into #2673


Use redux in react component

Requesting project review on bookstore app
Project requirements

First of all - make sure that you keep state immutable in reducers!
Secondly, change the default state in your books reducer from an empty array to an array with a few books. One of the goals is to display them in your React components.
Import Redux Provider and your store in the main component of your React app.
Connect your app to the Redux store.
Make sure that list of books displays books from Redux store.

Use useSelector to consume the state.

In your React component responsible for adding new books, add functionality for "Add new" button click:

Import Redux useDispatch.
Set the data inputs in the local React state (set title and author  - NOTE: categories and comments are NOT part of this step).
Once your new book object is ready to be submitted to Redux store, dispatch a corresponding action (import Action Creator that you created before and use it here).

In your React component responsible for removing books, add a functionality for "Remove" button click:

Import Redux useDispatch.
Dispatch a corresponding action (import Action Creator that you created before and use it here).

In your React component responsible for categories, add a functionality for "Check status" button click:

Import Redux useDispatch.
Dispatch a corresponding action (import Action Creator that you created before and use it here).

Thank you @sja-thedude

Hey, your project is approved. I mistakenly used the Github Comment option instead of Approved. Please go ahead, merge and submit it on your dashboard. 👍


[Usbdev] usbdev_env

This is an incremental PR add new changes in Driver and Seq_item Class Add drive data packet task in driver class
Add get response from Device task in driver class
Add crc generate task in usb20_item
Add Generate specific payload for control requestes task in usb20_item class
@jdonjdon  please review this

@jdonjdon can you please details. why test are failing. Thanks

@jdonjdon can you please share details. why test are failing. Thanks

You have merge conflict in hw/dv/sv/usb20_agent/usb20_item.sv, please rebase it.

@jdonjdon review it again the suggestion are incorporated.


[Suggestion] Furnace Removal Based On Input

Hey there, I was curious, is it possible you could implement a script that allows the user to remove a furnace recipe based on the input, this would combat the issues with outputs from mods such as Tough As Nails that are awkward if not impossible to remove
(Tough As Nails pure water bucket cannot be removed because the item ID is just forge:bucketFilled, Some other mods would have this issue)
It would also allows users to remove recipes with multiple outputs, Eg:
Mod A implements a recipe using quarts to make silicon
Mod B implements a recipe using quartz to make coal
using a script along the lines of this would allow all recipes using quartz to be removed
furnace.removeInput(minecraft:quartz)
Thanks for the work done to the mod so far!            -UnlimatedStone9

wrong repo, this is for documentation.
Just use the input specific removal
https://crafttweaker.readthedocs.io/en/latest/#Vanilla/Recipes/Furnace/Recipes_Furnace/

I don't see any input specific removal scripts so I'm not quite sure what you mean by that

furnace.remove(IIngredient output, IIngredient input);
you would need to specify both input and output.


CuboidCollider nested rotation seems incorrect

Unfortunately rapiers default collision detection is failing my models so I constructed some CuboidColliders in place and used MeshCollider to disable collision on the model. Seems like the relative translation of the parent rotation is off with the CuboidColliders. Or maybe I'm doing this wrong?
https://codesandbox.io/s/rapier-physics-box-fan-experiment-collision-rotation-iptdes?file=/src/BoxFan/FanCollider.js:927-972
🙇♂

That looks really strange indeed. Have you checked to see if visually the collisions actually make sense? The <Debug /> component needs some work, and it might be interpreting the translations incorrectly.

Uh, yeah those colliders are all over the place...
https://codesandbox.io/s/rapier-physics-box-fan-experiment-collision-rotation-forked-f1yh5w?file=/src/BoxFan/FanCollider.js
This does not seem to be the case in any of the /demo examples, I'm not sure what's going on.

Ahh, the issues are coming from the rotation added to the Colliders, will investigate further.

Thanks for all the debugging help 😅 @ClayCooperLA
The rotations should now be predictable in 0.5.2!
https://codesandbox.io/s/rapier-physics-box-fan-experiment-collision-rotation-forked-l27no8?file=/src/BoxFan/FanCollider.js

Thank you so much! @wiledal! Working perfectly now!


Staging CLI script drafts.

@mimig1,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@mimig1: i'd prefer we ship full scripts as this is a public repo. can you escalate your request to carolz?

@erickson-doug - I'll put the one's that are waiting on eng work in a separate PR, and keep the working ones in this PR.


Installation without super user permission?

Is it possible to use these tools without uid 0 being set by the user?

Hello! Can you please clarify your question?
If you're referring to running Docker without being super user you can do so following these instructions: https://docs.docker.com/install/linux/linux-postinstall/#manage-docker-as-a-non-root-user

In my situation I used the sudo rule several times to install CMake3 to install Docker to run Docker I know in VFX companies sudo privileges are rarely used

Agreed, it's a tough one. I think we'll have to figure this one as we go. We might just have to accept that running the server requires sudo or a sudo-docker group, and hope that's OK enough for experimentation. The long term goal is to be able to take a custom trained model, then package and deploy it natively in Nuke.

Yeah I know a guy that has deployed three models in C++ as an OpenFX plugin ;)


Wallet Detail - Action Summary & Action Logs for stewards

Business Requirement
For Direct Payments GoodCollectives, Each payment must trace back to a specific activity performed by the steward. This should be accessible from both the steward's wallet profile and the collective profile. As NFTs sometimes trigger multiple payments for different wallets, a new design must be created to account for and link to the NFTs Stewards were paid from, not necessarily the NFTs they hold.
Profile Page
A stewards/donor profile page should show all pools it participated in, either by donation, or by performed an action in a pool (or both).
It shows an 'impact-profile' card displaying aggregated data for any action (donation/stewards-action).
Next to this card will be shown cards per-pool a user participated in.
The block/section that shows 'actions' should link to the steward's Action Log page.
Detailed description
In the Action Log for stewards:

The treatment of Actions in the Summary should be updated to reflect the latest designs
The Action Log page (shown per steward) should be updated to link each action to all of the following:

The NFT that triggered the payment

The IPFS proof from that NFT

The Payment Transaction (minting) hash

Design Reference:
[ ] WIP - https://www.figma.com/design/ihw1PxBvLxacTHnN2aj4lC/3.-Product?node-id=19677-19070&t=5UFEZcJpM3XMuCcA-1
[ ] Need to choose treatment for Actions button in Wallet Detail (see options below)

@decentralauren
Screenshots:

Figma link:
https://www.figma.com/design/ihw1PxBvLxacTHnN2aj4lC/3.-Product?node-id=23077-16481&t=EVMRRReMLpG3KnkB-1


Chained method indentation fails when ? operator is inline

I have observed that something like this:
    ContextBuilder::new("Rain", 1024, 768)
        .quit_on_escape(true)
        .build()?
        .run(GameState::new)

is indented instead like this:
    ContextBuilder::new("Rain", 1024, 768)
        .quit_on_escape(true)
        .build()?
    .run(GameState::new)

I may try again to fix it (my attempt #222 here didn't work, because I didn't fully grok the function) but for now here's a bug report.

I just noticed that this problem doesn't exist in Doom's setup, which has rustic pinned at commit f7d5ac0c930ae435421f3f5bc827e8c61ce73662, from December 2020. I haven't looked into its config and/or hacks yet so it may not be a regression per se. Interesting clue though.

Hmmm, I know that the doom-emacs people often apply fixes for packages to doom-emacs and not to the affected packages. I'll take a look.

BTW, By making this change in rustic-interaction.el I'm getting the correct behaviour in the case I've described:
   (let ((continue t))
     (while continue
       (let ((starting (point)))
-        (skip-chars-backward "[:space:]\n")
+        (skip-chars-backward "?[:space:]\n")
         (when (rustic-looking-back-str "*/")
           (backward-char))
         (when (rustic-in-str-or-cmnt)

but I have not spent any time doing anything else, so for all I know it might break something else.

I didn't add this code myself so I'm a little hesitant, as you mentioned we can't be sure if this doesn't introduce other bugs. I would have a better feeling applying this change if we had some tests.

This is now a rust-mode issue since rustic is no longer a fork of rust-mode and requires it as dependency.

This is now a rust-mode issue since rustic is no longer a fork of rust-mode and requires it as dependency.


Integrations and platforms could not be set up

Describe the bug
When starting HA I see the following notification:
`The following integrations and platforms could not be set up:
eufy_security.binary_sensor
eufy_security.sensor
eufy_security.switch
eufy_security.select
Please check your config and logs.`
In the logs i see the following:
`Logger: homeassistant.setup
Source: setup.py:316
First occurred: 18:19:02 (4 occurrences)
Last logged: 18:19:02
Unable to prepare setup for platform eufy_security.binary_sensor: Platform not found (cannot import name 'EntityCategory' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)).
Unable to prepare setup for platform eufy_security.sensor: Platform not found (cannot import name 'EntityCategory' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)).
Unable to prepare setup for platform eufy_security.switch: Platform not found (cannot import name 'EntityCategory' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)).
Unable to prepare setup for platform eufy_security.select: Platform not found (cannot import name 'EntityCategory' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)).`
To Reproduce
Start HA
Expected behavior
4 sensors to be loaded
**Hardware Information about Home Assistant
Home Assistant Core
Version core-2021.10.3
Operating System
Version 6.4
**Hardware Information about Cameras

Camera Model: T8210
Live Streaming Protocol: RTSP

Additional context
When installing eufy security I cannot use port 3000 as i get a message to say it is already in use - I have selected port 3001, and installation is allowed to proceed.
I have also uninstalled the intergration; the addon;  rtsp addon and webrtc integration, deleted the folder config\custom_components\eufy_security, and reinstalled - but with the same result

Your home assistant version is too old, please update to 2012.12.xx at least.

thanks! That has resolved the issue - great intergration too!


move demos out of / and make them compile

Will add this to CI in a separate PR

As per PR description, I want to add this to CI in a follow up, as those are somewhat orthogonal :P

Oops, I checked the files, not the PR description.

lol, guess I need to pick another branch name

Cloning into 'tb_tmp'...
fatal: 'demos' could be both a local file and a tracking branch.

Always read the source before the docs, yeah :D

https://github.com/tigerbeetledb/tigerbeetle/pull/380


Cable Model

Implement cable model

@empireu You have code for the cable rendering but you didn't do a pull request for it yet; do you want to push a branch so that we can review it? Thanks!


fx-22

Link
https://fx-22.com/5-tips-for-choosing-the-right-brokerage-account-in-2023/
Version
Manifest-v3
What browser(s) are you using?
Chrome browsers (includes brave, edge, opera, etc)
What platform are you using?
Computer (Windows, Linux, MacOS)
(Optional) Anything else?
No response

From where did you get redirected to this website, because I don't think it has anything to do with bypassing link shorteners.

This seems to be a link protector (used in pirated websites for indexing pirated videos or similar), I'd like to know where did you get this link from, so we can look futher on this issue.


Questions about ZegCLIP training

I am very happy to see your work on ZegCLIP, it is very interesting and very helpful for me. I'm having a little trouble with your code.
I used the pth file you provided for inference and got results consistent with the paper. However, I use the same docker environment and source code for training, but there is a certain deviation in the inference results obtained.When running Inductive setting under the VOC dataset, my experimental results differ from yours by less than 2 points. However, when running Inductive setting under the COCO dataset, there is a difference of as much as 7 points. The experimental results are shown in the attachment.
Can you help answer this question?

I appreciate your interest in our work.
Could you please confirm that the mask weight you used in
https://github.com/ZiqinZhou66/ZegCLIP/blob/aa63859603e6b243c1f8f3722bb4e4715aa0a422/configs/coco/vpt_seg_fully_vit-b_512x512_80k_12_100_multi.py#L71 is 20 or 100? Because I noticed that the parameter in the original config is 20, but I do use 100 for training so I uploaded a new config. It may affect the performance if used the old version.
Besides, a widely used trick in many previous works that slightly reduces the logits on seen classes is helpful in the inductive setting. I have set the parameter to 0.1. Did it also use in your inference? Or you may try to change the factor to see the difference.

Thank for your eager reply, this phenomenon is caused by the fact that I ignored the number of iterations of the model.
In the MMSeg, batchSize = GPUNum * samples_per_gpu. Your paper has mentioned that it is using 4 GPUs for training, and I ignored this condition. I was using only a single card, so the amount of training data was only 1/4 of yours. After I made up the full number of training sessions, the method performance was significantly improved.
However, it is still slightly less effective than the paper by 1-2 points. I think this is because the number of trainings increased exponentially and I didn't change the super parameters like learning rate accordingly.
Thank you once again!

Thank you for your feedback.
I may need to be more specific that making sure the batch size is 16 when reproducing the effect of our work.
Best of luck with your research.

@Qyizos Hi, I am trying to validate the results for cocostuff164k but i got very good results for 11 classes but for all the rest its zero. Can you guide what am i missing here. I run the code with only updated datapath and rest of repo code was same


Grafana Client dashboard does not show wireless cameras or Amazon devices

The existing Grafana dashboard for clients (https://grafana.com/grafana/dashboards/10418) has a great view to show cameras and Amazon devices but only if they are wired.
This makes sense looking at the SQL statement
"SELECT non_negative_derivative(sum("wired-rx_bytes"), 1s) AS "Rx", non_negative_derivative(sum("wired-tx_bytes"), 1s) AS "Tx" FROM "clients" WHERE "name" =~ /camera|cam$/ AND "site_name" =~ /^$Site$/ AND ("sw_name" =~ /^$Switch$/ OR "ap_name" =~ /^$AP$/) AND $timeFilter GROUP BY time($__interval), "name", "Mac""
However, it would be good if also wireless devices could be shown.
Is this possible?

You can modify the queries to show anything you want. :) Definitely possible.

You can modify the queries to show anything you want. :) Definitely possible.

Wow. Smart...

This should give you what you want, but I'm making assumptions on what you want since all you asked is if something is possible. I can't test this since I don't have any wireless cameras.

thanks @davidnewhall , much appreciated.
It seems that each network interface is reported separately, (e.g. Apple TV comes with wireless and wired interface)
Having two queries work indeed but each interface is shown separately and not aggregated (which might also be right, based on the use case)

Correct. Each interface is a separate device to your network. You can do more trickery to combine them in the influx query, but it's not something I've ever tried, and I find it sufficient to have them displayed independently. On another panel I add w (to the alias) for wireless devices and e for ethernet-connected devices.  This helps distinguish them.
Please open another issue if I can help further!


Set application manifest for targeting Windows 8.1 and Windows 10

In Windows 8.1 and Windows 10, the GetVersion and GetVersionEx functions have been deprecated. In Windows 10, the VerifyVersionInfo function has also been deprecated. While you can still call the deprecated functions, if your application does not specifically target Windows 8.1 or Windows 10, you will get Windows 8 version (<IP_ADDRESS>).
In order to target Windows 8.1 or Windows 10, I think update the application manifest in the source file is necessary.
Reference: https://msdn.microsoft.com/en-us/library/windows/desktop/dn481241(v=vs.85).aspx

Seems like a reasonable thing to do, but I worry about how this will work on the earliest support builds of master.  Also, I found https://support.microsoft.com/en-us/help/978637/an-application-runs-in-a-windows-vista-context-instead-of-in-a-windows.
Compatibility gurus, what do you think?

Maybe I am not a guru, but I used to compile a program running on all Windows versions (I mean, from 1.01 to 10). Windows 2003 and older will just ignore the manifest and run normally.
I've extracted KB978637 and it seems that this is a patch for the 64-bit Windows mechanism handling the issue of old, mixed 16/32-bit setup program launchers. It has nothing to do with us, unless we use MS-Setup (1997) or somehow integrate with a very old version of NetMeeting. I think that we're not going to do any of that.

@craigwi we should be perfectly safe with this PR. Microsoft documentation states that adding this manifest has no known compatibility issues with previous versions of Windows.
According to Microsoft:
"Manifesting the .exe for Windows 8.1 or Windows 10 will not have any impact when run on previous operating systems. You can also add this to your .rc file if you already have it defined.
Adding the trustInfo isn’t essential, but it is highly recommended. This will allow your .exe to always get the correct version, whether the operating system is Windows 10, Windows 8.1, or Windows 8."

You folks are great!  Thanks for the confirmation.


!strcmp(locale, "C"):Error:Assert failed:in file baseapi.cpp, line 209

Report in PHP cli mode! StrCmp (locale, "C"): Error: Assert failed: in file baseapi. cpp, line 209

vim ~/.bash_profile
export LC_ALL=C
source ~/.bash_profile


[Port Manager] Use Data-Plane Manager REST APIs instead of schema

This requires DPM to finalize APIs and check in web classes.

@chenpiaoping For your own tracking.
@haboy52581 Please update the issue when the DPM web classes are ready.

This issue has been unblocked by the latest contract merge in PR #234 @chenpiaoping


Initial steps towards a delete action for a song folder.

This seems to be already working as intended - however it needs further improvements:

[ ] extend by a confirmation dialog where the delete operation can also be cancelled
[ ] update the song table to indicate that the local files are not available anymore
[ ] the delete operation should work for all songs of a selection.

Added in 936e35e6196c55f846665a5228a27d02b1a4f00b.


Incorrect assembly information for ClosedCaptionProperties

The correct assembly for this class is Windows.Media.dll and not Windows.Media.ClosedCaptioning.dll as mentioned here.

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

ID: 892c130c-f2d2-18aa-20b2-0a75e591a2bf
Version Independent ID: 24be0079-0ca3-312a-f28c-ebe0ebb97758
Content: ClosedCaptionProperties Class (Windows.Media.ClosedCaptioning) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.media.closedcaptioning.closedcaptionproperties.yml&version=GBlive&_a=contents
Product: uwp
Git Login: drewbatgit
Microsoft Alias: drewbat

That's correct. There is no such Windows.Media.ClosedCaptioning.dll. Not in the 32 or 64 bit version of (1903 build 18362.267) Windows 10 may 2019.

We are no longer listing assembly information in API reference pages.


Build with dpnp but do not require dpnp in runtime

Goal: Enable dpnp support does not mean numba-dppy can not run without dpnp.
It will eliminate complex dpnp dependencies (i.e. MKL) in numba-dppy CI and other scenarios.

True for kernel decorator, but the new dpjit decorator will make dpnp support an explicit user-visible feature. Thus, dpnp needs to be a runtime dependency as dpex' evolves in a dpnp compiler.


Update migrate-from-launchdarkly.md

🚀 Deployed on https://667b2e1c7f821377bb75ad44--cozy-fox-0defec.netlify.app


swift-api-checker: teach the script to also diagnose the breakages between two previous dumps

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


Add changes to propagate queryGroupId across child requests and nodes

Description
This change add logic to propagate queryGroupId across child requests and nodes. We will add separate PRs to populate this field later on. This will be the foundational work (partially as we still have to populate this info from ThreadContext to Task headers) for achieving resource monitoring and resource usage based cancellation for QueryGroups .
Related Issues
https://github.com/opensearch-project/OpenSearch/issues/12342
Check List

[X] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@kaushalmahi12 - Can you update the changelog as well? The check is failing

@kaushalmahi12 - There seems to be some merge issue after latest commits. Also, need to address the failing DCO check.


[Event Hubs] checkOnInterval should resolve when the check function returns true failing in nightly runs

Event Hubs nightly test runs are failing with:

Error message:
expected 67500 to equal 7500
Stack trace:
AssertionError: expected 67500 to equal 7500
at Context. (/mnt/vss/_work/1/s/sdk/eventhub/event-hubs/test/internal/checkOnInterval.spec.ts:18:12)
+ expected - actual
-67500
+7500

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2608597&view=results

It succeeded when reran.

flaky test?

@jeremymeng good question. It shouldn't be because it is a unit test that utilizes sinon and I was surprised it failed. My guess is that nodejs was in a weird state for some reason. I am not familiar with sinon well enough to discern issues with it.


models/VP2420.yml: added

Thanks @filipleple , but please fix pre-commit.

@macpijan removed all comments and the trailing whitespace. Not sure why pre-commit fails now - locally it's satisfied, here it fails due to osfv_cli.py, which I'm not modifying

@filipleple If yo uwant to find out, you can easily run pre-commit locally: pre-commit run --all-files.
It is probably my fault here, as the check in master fails as well. But if you can fix it in this PR, I won't complain :)

@macpijan that was a good tip, all green now


Hyper 0.12 support

This MR upgrades the swagger crate to use Hyper 0.12
This creates large scale changes:

NewService has been renamed MakeService - as such, we rename all implementing types.

AddContextNewService -> AddContextMakeService
The MakeService implementation of AllowAllAuthenticator is split into MakeAllowAllAuthenticator
CompositeNewService -> CompositeMakeService.

Hyper Services don't have the request as a type parameter -  instead the body of the request / response are type parameters. As such we create a new type, ContextualPayload, which implements hyper::body::Payload, and allows the addition of some contextual data.

hyper::headers has been removed. The replacement https://docs.rs/headers/0.2.1/headers/ doesn't provide a common interface, so in the interest of reducing change, we use a migration crate - https://crates.io/crates/hyper_old_types

- `Into<HeaderValue>` is not implemented for useful things like Strings, Vecs of Strings and DateTime, so we add some utility functions in `swagger::header`.  We also add move common code for X Span ID here as well.

hyper::client::Service has been removed. We re-implement it in swagger::client, so we can implement Swagger middleware.

We upgrade hyper-tls to 0.2 (from 0.1). As per https://github.com/Metaswitch/swagger-rs/issues/51#issuecomment-486120194, we can't currently upgrade further (either hyper-tls, or native-tls, or openssl), due to native-tls limitations.
We remove any dependency on tokio-core, which is deprecated, as the codegen moves to using tokio directly as part of the hyper 0.12 work.
We take this opportuntity to move a bunch of static code in the codegen for multipart support into the crate. This is hidden behind a feature flag.
We remove the deprecated AddContext type.

Has this PR stalled? I would be happy to help to get it over the line.

bors try

bors try

bors try

bors r+


PageCache generating too many 'additional' files

I'm using PageCache on a website with approximately 10,000 pages and I received from the hosting server (shared) a warning that I was with 'high inodes usage' and that my account could be suspended.
Looking at the server I saw that in the cache folder in each subfolder there are the files containing the page content but there are also many 'additional' files containing only strings like

a: 2: {i: 0; i: 5; i: 1; N;}

In a folder, for example, there are 149 files, 22 with page html and 127 with these 'additional' files. There are 256 folders in the cache folder, so this turns out to be a big problem.

I do not understand the need of these additional files because at the time a page cache is generated, only the page content is saved in a file. Apparently these other files only appear in the next day (considering a clean cache folder), so I don't think they are necessary for the cache to work properly.
The only cache setting I am using:

Additional info

Q
A

Yii version
2.0.23

PHP version
7.1.17

Operating system
Linux

These smaller caches contain arrays such as:
[
    0 => 5,
    1 => null,
]

PageCache is not likely to generate these especially considering the fact that they appear next day.

@samdark You're right, I had forgotten that in my RateLimiter implementation I was saving some  information in cache. Now I have set a cronjob to delete these expired caches daily. Thank you.


Needs CORS support

Hi I tried using this on a demo website and Chrome blocked the images with the following message
Access to image at 'https://avatars.dicebear.com/api/bottts/Player2.svg' from origin 'https://localhost:5001' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

My understanding is that if your API added the header "Access-Control-Allow-Origin" with value "*" then Chrome would no longer block the requests

I cannot reproduce the problem. Maybe there was a temporary problem with our provider. Can you check this again?
curl -I -X GET https://avatars.dicebear.com/4.5/api/bottts/Player2.svg
HTTP/2 200
date: Sat, 16 Jan 2021 15:01:59 GMT
content-type: image/svg+xml
content-length: 5714
vary: Accept-Encoding
server: BunnyCDN-DE1-565
cdn-pullzone: 147563
cdn-uid: cd8b9392-b9a4-4962-9235-a51defc82e95
cdn-requestcountrycode: DE
access-control-allow-origin: *
access-control-allow-headers: Server, x-goog-meta-frames, Content-Length, Content-Type, Range, X-Requested-With, If-Modified-Since, If-None-Match
access-control-expose-headers: Server, x-goog-meta-frames, Content-Length, Content-Type, Range, X-Requested-With, If-Modified-Since, If-None-Match
cdn-edgestorageid: 632
cache-control: max-age=2592000
cdn-cachedat: 2021-01-16 16:00:33
cdn-requestpullsuccess: True
cdn-requestpullcode: 200
cdn-requestid: a3d39f16085aed588fa2073de7fa725f
cdn-cache: HIT

I cannot reproduce the problem. Maybe there was a temporary problem with our provider. Can you check this again?
curl -I -X GET https://avatars.dicebear.com/4.5/api/bottts/Player2.svg
HTTP/2 200
date: Sat, 16 Jan 2021 15:01:59 GMT
content-type: image/svg+xml
content-length: 5714
vary: Accept-Encoding
server: BunnyCDN-DE1-565
cdn-pullzone: 147563
cdn-uid: cd8b9392-b9a4-4962-9235-a51defc82e95
cdn-requestcountrycode: DE
access-control-allow-origin: *
access-control-allow-headers: Server, x-goog-meta-frames, Content-Length, Content-Type, Range, X-Requested-With, If-Modified-Since, If-None-Match
access-control-expose-headers: Server, x-goog-meta-frames, Content-Length, Content-Type, Range, X-Requested-With, If-Modified-Since, If-None-Match
cdn-edgestorageid: 632
cache-control: max-age=2592000
cdn-cachedat: 2021-01-16 16:00:33
cdn-requestpullsuccess: True
cdn-requestpullcode: 200
cdn-requestid: a3d39f16085aed588fa2073de7fa725f
cdn-cache: HIT

Oh, I see. There is a problem with the redirects. As a workaround you can use the versioned version of the API.
https://avatars.dicebear.com/docs/http-api#versioning

Oh, I see. There is a problem with the redirects. As a workaround you can use the versioned version of the API.
https://avatars.dicebear.com/docs/http-api#versioning

Can you test this again? The header is now also added to the redirect.

Can you test this again? The header is now also added to the redirect.

It works now. Thank you for the very speedy weekend fix

It works now. Thank you for the very speedy weekend fix


Calling Dart_InitializeApiDL in the C++ plugin from flutter

I tried to call Dart_InitializeApiDL in the c++ plugin by sending NativeAPI.initializeApiDLData using platform channel. But I got a segfault.

Hi @rogurotus, Thanks for filing the issue. Please share the output of flutter doctor -v, The complete error logs, minimal and complete sample code along with the steps to reproduce the issue in order to investigate this issue further.


Problem with if statement parsing with literal expression

10 IF 10 % 4 EQ 0 THEN PRINTL "AHH WHY"

Nodes are being added to a literal expression, which should not happen.
Error corrects itself if changed to
10 IF (10 % 4) EQ 0 THEN PRINTL "AHH WHY"

elif op == "%":   return left %  right
ZeroDivisionError: integer division or modulo by zero
Node Stack
type: PROGRAM             content:               line: -1    children: Node[1]
type: LINE                content:               line: 10    children: Node[5]
type: IF-DEF              content: IF            line: 10    children: Node[0]
type: BINARY_EXP          content:     line: 10    children: Node[3]
type: LITERAL_EXP         content:     line: 10   children: Node[3]
type: LITERAL             content: 10            line: 10    children: Node[0]
type: BINARY              content: %             line: 10    children: Node[0]
type: BINARY_EXP          content:     line: 10   children: Node[3]
type: LITERAL_EXP         content:     line: 10   children: Node[3]
type: LITERAL             content: 4             line: 10    children: Node[0]
type: BINARY              content: EQ            line: 10    children: Node[0]
type: LITERAL_EXP         content:     line: 10   children: Node[3]
type: LITERAL             content: 0             line: 10    children: Node[0]
type: THEN                content: THEN          line: 10    children: Node[0]
type: PRINT               content: PRINTL        line: 10    children: Node[0]
type: STRING_EXP          content:               line: 10    children: Node[3]
type: QUOTATION           content: "             line: 10    children: Node[0]
type: STRING              content: AHH WHY  line: 10    children: Node[0]
type: QUOTATION           content: "             line: 10    children: Node[0]

'10 IF 10 % 4 EQ 0 THEN PRINTL "DUDE WHY"',    # div by zero error
'15 LET X = 10%4 EQ 0 AND 0 EQ 0', # div by zero error
'20 LET Y = 10-0 * 10 / 2' # works fine


Tracking: Support barrier and checkpoint decoupling

Is your feature request related to a problem? Please describe.
We have implemented concurrent checkpoints, we can send more frequent checkpoint to ensure that the table is updated faster. But we have many small file in s3, and more frequent uploads to s3.  We can't  reduce the frequency of checkpoint to solving it, because our reading depends on the epoch of the checkpoint. So we want to support barrier and checkpoint decoupling and do some bench. And I write it in this doc. https://singularity-data.quip.com/QLOaAegyeApR/Support-barrier-checkpoint-decoupling
We want to write the data of the barrier in shard buffer. When we do checkpoint, We build all the data as a Sstable uploads to s3.
Pros

We can reduce the nums of the sst in l0.
We can reduce the number of uploads to s3.
We can reduce the system congestion caused by hash_join.
We can improve the source_throughput when we have hash_join
We can be more flexible in terms of configuring barriers and checkpoints.

Describe the solution you'd like
We need to do something for the different formations
Frontend:

We may need to read the data in shard buffer. So we need to dispatch to this node for reading. If we don't write to the shard buffer after the last checkpoint, we can schedule our reads as we like, even if we need to read new data
Our pin_epoch will return two epoch. We need to choose which epoch we need.

Stream:

Maybe we don't need a lot of modifications. We only add a state to distinguish between barrier and checkpoint. If it is a barrier, We will not sync data to S3 , but just update barrier_maxcommit_epoch.
If create/drop_mv, flush and create over, We need checkpoint.

Storage:

We need sync all data of barrier to s3.  If its size is too large, we will spend many time to upload, so we can overflow to s3 to support larger data.
We need to maintain two epochs for reads.

[ ] https://github.com/singularity-data/risingwave/pull/4442
[ ] Maintain two epochs for reads. pin_epoch will return two epoch
[ ] Dispatch to node for reading
[ ] A state to distinguish between barrier and checkpoint and open decoupling

Our pin_epoch will return two epoch. We need to choose which epoch we need.

Still doubtful about this. :( So do we want to support different checkpoint intervals for different executors?

Our pin_epoch will return two epoch. We need to choose which epoch we need.

Still doubtful about this. :( So do we want to support different checkpoint intervals for different executors?

No, our barrier and checkpoint are all global.


bug(chips): After adding new chip to a chip list, previous selected chip is lost.

Reproduction
Check my solution at: https://stackblitz.com/edit/multiple-selection-chips
Steps to reproduce:

Add a chip from the list
Select the chip from the step 1
Add another chip from the list
Check console log and you'll see that MatChip.selected is equal to false when it was previously marked as selected.

Expected Behavior
I would expect to maintain previous selection, to bypass this issue I had to iterate over the whole list of chips and selected the last selected chip.
If you check, in my solution I've created a method to bypass this: recalculateChipSelection
Actual Behavior
Previously selected chip is Unselected.
Environment

Angular: 10.1.0
CDK/Material: 10.2.0
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

This is happening because your ngFor is binding to getSelectedChallenges(), which is creating a new array instance every time, so the values in the selection model aren't seen as the same.  The SelectionModel from the cdk just uses reference equality when doing comparisons.


[ColorPicker] Accessibility: Narrator announces color changes in main…

… view
Summary of the Pull Request
What is this about:
As soon as the picked color changes, Narrator announces it.
What is include in the PR:
How does someone test / validate:
Quality Checklist

[ ] Linked issue: #12113
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

@niels9001 Recent commit changes spoken color to its friendly name instead of using number code, since it makes much more sense.

@niels9001 Recent commit changes spoken color to its friendly name instead of using number code, since it makes much more sense for a common use-case imo.

Yeah, good call!


Read password from stdin instead of .nipaprc

Hello together
Even if the webinterface is really great I'm kinda the CLI-Guy. I tested it and like it - besides that part about storing your password clear text in an file.
If you're using the LDAP-Backend you're most likeley having several systems you have access to. And you don't want to give the server admin(s) or some botnet the opportunity to steal your password and gain more systems.
Therefore I'd like to add a feature request: Instead of exiting with an error if "password" is not found at .nipaprc give the user a prompt (e.g. using "getpass") and only raise the exception if thats also not provided.

Fair enough. I'm certainly well aware not everyone fancies storing a password in a clear text (I don't either).. guess the long term solution would be some form of tokens instead so we can still store it for convenience but it doesn't grant access to anything else.
Would you also like to have a shot at implementing it? Should be a fairly small patch :)

Well it's not fancy looking but it's working. See pull request #839

Fixed through merge of #950.


[Unity] Add Global info

As we start to build more structure into the program, there is an increasing need to incorporate globally static object that are referred by the IR itself. So we introduce "GlobalInfo" here to represent these objects. Below is an example of a possible representation of a distributed program with global info. Pay attention to "mesh", which represents the global infos in the module. It contains a list of two device meshes , and they are accessed using syntax like "mesh[0]", which is a sugar for IRModule.global_infos["mesh"][0]
    @I.ir_module
    class TestModule:
        I.module_attrs({"device_num": 10})
        I.module_global_infos(
            {
                "mesh": [
                    R.device_mesh((2, 2), R.range(0, 4)),  # mesh[0]
                    R.device_mesh((1,), R.range(4, 5)),  # mesh[1]
                ]
            }
        )

        @T.prim_func
        def tir_func(
            x: T.Buffer((T.int64(128), T.int64(128)), "float32"),
            y: T.Buffer((T.int64(128), T.int64(128)), "float32"),
        ):
            T.func_attr({"tir.noalias": True})
            for i, j in T.grid(T.int64(128), T.int64(128)):
                with T.block():
                    vi, vj = T.axis.remap("SS", [i, j])
                    y[vi, vj] = x[vi, vj] + 1.0

        @R.function
        def foo(
            x: R.DTensor((128, 128), "float32", device_mesh="mesh[0]", placement="S[0], R"),
        ) -> R.DTensor((128, 128), "float32", device_mesh="mesh[0]", placement="S[0], R"):
            gv0 = R.dist.call_tir(
                tir_func,
                x,
                R.DTensor(
                    shape=(128, 128), dtype="float32", device_mesh="mesh[0]", placement="S[0], R"
                ),
            )
            return gv0

To summarize the PR, it

add a base class GlobalInfo
add a global_infos: Map[str, List[GlobalInfo]] field in IRModule
support parsing/printing for I.module_attrs and I.module_global_infos.\

cc: @tqchen, @junrushao

Hi @jinhongyii thanks for the work! I have a follow-up question, looks the global info here is very similar to metadata section, is there any difference?  I am wondering if we can merge them

Meta data are used to store ObjectRef fragment in the IR that are not printable. However, global infos should be printable


Future request: The ability to start and stop the download like the previous one

So if it misses one or gets stuck on getstream and could be redownloaded with out having to get and do the the album.
As, this one didn't download and didn't show up in the errors either, not sure what the errors are for as I have yet to see one in there.

Was implemented in <IP_ADDRESS> release.
Allow to cancel and retry download-task.
Thanks


Fix the integration tests

Problem
The integration tests fail on the main branch
status: Failure(Action #4: PrepareError: Error happened while deserializing the module.),

This issue seems to indicate that we're somehow using opcodes outside the wasm-core-1 specification
Source: https://github.com/near/nearcore/issues/8358#issuecomment-1383247423

https://github.com/near/nearcore/issues/8358#issuecomment-1721807289

I searched for ways to disable the sign-ext opcodes on the compiler and ran across this issue that seems to indicate that the option to disable sign-ext exists 👍 but that it's not working. 😢 https://github.com/rust-lang/rust/issues/109807
Thus, I will likely pursue the wasm-opt --signext-lowering approach.

This issue was "bandaid" fixed by https://github.com/RequestNetwork/near-contracts/pull/16 which downgrades to Rust 1.67 - thus avoiding the later versions where sign-ext opcodes are added by default.
This issue should remain open until we've found a long-term fix.


Reflect nullability of Error and endTime

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

Please follow the link to find more details on PR review process.

/azp run automation - sdk

Can one of the admins verify this patch?


pkg.go.dev: Go discovery site tracking issue

This is the tracking issue for the Go discovery site, previously announced in the Go Modules in 2019 blog post. The parent umbrella issue for all “godoc”-like projects is #26827.
The website will display documentation for Go packages with version support and provide features that help developers discover and evaluate Go packages or modules.
The discovery site is currently in development. We are working towards launching by the end of the year.
As mentioned in https://github.com/golang/gddo/issues/567#issuecomment-503322119, some early mockups of the discovery site were first shared in the "Finding Dependable Go Packages" talk (slides) at GothamGo 2019 (video). These can be seen starting at slide 92.
If you are interested in this topic, please subscribe to this issue. We will use it to post future updates. You can also find us in the #tools channel on the Gophers Slack.

The discovery site is now live at pkg.go.dev!
You can read about it on the latest Go blog post.
We hope that you will try it out, and we would love to hear your feedback! We are just getting started with building out pkg.go.dev, so some bugs are to be expected. Please file any issues and share your thoughts with us by clicking "Share Feedback" or "Report an Issue" in the footer of every page, or by emailing<EMAIL_ADDRESS>We look forward to hearing everyone’s feedback!

We would love to know if your organization is running an internal Go documentation service, to inform our open source strategy for pkg.go.dev.
If this is something that your organization is currently doing, please share your use case with us by sending an email to<EMAIL_ADDRESS>or filling out the Go Developer survey.

We recently updated pkg.go.dev/license-policy to include our list of accepted licenses, and to clarify that we use github.com/google/licensecheck for license detection. If you still have questions about your license, feel free to email<EMAIL_ADDRESS>We plan to respond to all issues in the new year, and to continue working with our legal team to resolve any license issues. Thanks for being patient with us as we work through them!

You can now redirect your own requests from godoc.org to pkg.go.dev, by visiting godoc.org?redirect=on.
This will set a cookie in your browser, so that any request you make to godoc.org will be  redirected to pkg.go.dev.
If you want to turn this feature off, you can do so by visiting godoc.org?redirect=off.
Thanks to @jamalc for this contribution!
You can follow Go issue 37099 for any updates on this redirect option.

Closing this tracking issue, as information on the project direction will be shared in the x/pkgsite README Roadmap.
Additionally, the following milestones are used to track work for 2020:

Milestone pkgsite/design-2020: https://github.com/golang/go/milestone/159
Milestone pkgsite/godoc.org-redirect: https://github.com/golang/go/milestone/157


Free Content Page UI

Is your feature request related to a problem? Please describe.
Users are finding it difficult to navigate and access free content on the website due to the current layout and user interface.
Describe the solution you'd like
I would like to see a redesign of the free content page UI to make it more user-friendly and intuitive. This could include organizing content into categories, improving search functionality, and providing clear navigation paths.

You're talking about complete different website which is a main one we're setup this project only which is on learn.csedge.courses

learn.csedge.courses have only one page (Blog). If I am wrong pls rectify me.
And I really want to contribute to this project. Could you pls tell me the procedure

@Japneet001 you mentioned free content page that was located in main domain csedge.courses
This project is on sub domain learn.csedge.courses
please specify the link which page your willing to work on

@CSEdgeOff I am willing to work on https://csedge.courses/Resources.html
But could you pls specify the links that are a part of learn.csedge.courses, so that I can contribute to that sub-domain

Please add Level also of the issue

@CSEdgeOff Please add Level also of this issue and assign this issue to me.


Request: Updated program Icon

The current program Icon looks pretty out of place and awfully fuzzy pinned to the taskbar.

Yeh, can't agree more. Would be grateful if someone could make a better icon, I myself, unfortunately, do not have time. So if you or you might know someone who can create one - I will be happy to update it.


OHIF tools not visible for microscopy

Hi Guys,
Currently, I'm able to load study in ohif viewer locally but not able to see any other tools like Annotate, Length, Pan for 'SM' modality. Is there any settings needed to enable those tools?
Please check the screenshot for your reference.

Hi -
The microscopy plugin uses a different underlying library (openlayers instead of cornerstone) so we can't enable the same tools directly.
But there is a proof of concept annotation example to generate structure reports that correspond the the ones generated for radiology images.  You can try it out here: https://microscopy.dcmjs.org/structured_report/index.html
Eventually we plan to further develop and push this functionality into OHIF proper.

Thanks for the quick response.
It would be great to know by when would this be added

It's hard to say, but I'm pretty sure there's nobody actively working on this at the moment so maybe a year or two at least.  In the next year or so we'll be working more on the microscopy display features before annotations.  Even the dicom standards for representing annotations on microscopy images are still being developed.  Not trying to be discouraging, just want to you know the state of things.  I'll close this but feel free to ask more questions as needed.  If you can help with implementation that would be most welcome.

Hello. I am facing the same problem. Any updates on development of OHIF tools for microscopy extension? Any better solution for displaying dicom-microscopy images?


No return log for android app

I'm trying to use this library on only-android (no multiplatform setup) app. I used plugin and de.jensklingenberg.cabret:cabret-log, both with 1.0.3 versions. I enabled cabret and useIR in kotlinOptions block like that:

Whenever i use @DebugLog annotation i only have the function entry log. The return log never arrives (i checked in custom logger with breakpoint - only entry logs arrive there)

I'm sorry, the issue happens only for functions that do not return result

I'm sorry, the issue happens only for functions that do not return result

Thank you for reporting this, i will take a look it

Thank you for reporting this, i will take a look it


Feat compat config file

PR Details

support tools.styledComponents for module-tools
support close generator d.ts process with output.disableTsChecker
fix port problem for plugin-docsite

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] I have updated changeset
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

建议：

一个 PR 只做一件事情，方便 review & merge。
PR 标题按照 commit message 规范书写，避免 squash 后出现格式问题


net: Move NET_TCP/UDP_HAVE_STACK to netconfig.h

Summary
Now (after https://github.com/apache/nuttx/pull/14634 and https://github.com/apache/nuttx/pull/14650) the HAVE_PFINET(6)_SOCKETS depends on NET_TCP/UDP_HAVE_STACK, which is previously defined in net/ folder and cannot be included. Considering many places use this check, maybe moving them to netconfig.h could be better.
Impact
Before: IPv4 + TCP, without ICMP -> HAVE_PFINET_SOCKETS is undefined, we cannot use TCP socket (error is EAFNOSUPPORT)
After: IPv4 + TCP, without ICMP -> HAVE_PFINET_SOCKETS is defined, we can use TCP socket
Testing
QEMU with CONFIG_NET_IPv4 & CONFIG_NET_TCP enabled but CONFIG_NET_ICMP_SOCKET disabled.

[Experimental Bot, please feedback here]
This PR does not fully meet the NuttX requirements. Here's why and how to fix it:
Missing Information:

Summary: While the summary explains the "why", it lacks clarity on the "what" and "how".  Specifically:

What functional part is changed?  Clearly state which files are modified (e.g., netconfig.h).
How does the change work? Instead of describing the before/after behavior, explain how moving the definitions to netconfig.h solves the problem.  For example, "This change moves the definitions of HAVE_PFINET(6)_SOCKETS to netconfig.h, making them accessible to code that previously couldn't include the necessary headers from the net/ directory."

Impact:  Several sections need more detail:

Is new feature added? Is existing feature changed?  Be explicit. This is likely an "existing feature changed" due to a bug fix.
Impact on user:  While the before/after behavior is described, clarify how a user might observe this change. For example, "Applications that previously failed to create TCP sockets when ICMP was disabled will now function correctly."
Impact on build:  If only header files are changed, the build process itself is unlikely to change significantly.  Clarify this.
Impact on hardware: Be explicit. Likely "NO".
Impact on documentation: If the behavior changed, documentation might need updating.  Even if no changes are needed, explicitly state "NO".
Impact on security: Be explicit. Likely "NO".
Impact on compatibility: This is the most important part needing clarification.  Does this change break anything?  Is it backwards compatible?  Explain thoroughly.
Anything else to consider: Be explicit. Likely "NO".

Testing:

Build Host(s):  Provide details about your build host OS, CPU architecture, and compiler version.
Target(s):  Specify the QEMU architecture (e.g., qemu-system-x86_64).  Also, provide the full NuttX configuration name (e.g., nsh).
Testing logs:  The logs are missing entirely. Include actual output demonstrating the failure before the change and the success after the change.  Show commands used and their output.  For example:

Before (without the patch):
$ make ARCH=sim BOARD=qemu-i486 CONFIG=nsh
... build output ...
$ nsh
nsh> netstat -a
... output showing no TCP socket available ...
nsh> telnet google.com 80
EAFNOSUPPORT

After (with the patch):
$ make ARCH=sim BOARD=qemu-i486 CONFIG=nsh
... build output ...
$ nsh
nsh> netstat -a
... output showing TCP socket available ...
nsh> telnet google.com 80
Connected to google.com.
Escape character is '^]'.

By providing the missing information and clarifying the existing details, the PR will better meet the NuttX requirements and be easier for reviewers to evaluate.