linux_testing: 5.0.0-rc6 -> 5.0.0-rc7, drop interp trunc patch

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

@grahamcofborg eval

Separately handled in d607715ab3afef876a87c93acfe7c2f78085135e , closing.


The effect of Chrono doesn't work

Usingmake launch ARGS="--chrono" to start the server, the effect of running  manual_control_chrono.py looks the same as that of running manual_control.py and press Ctrl + o to enable Chrono. Is there any difference in effect?
CARLA version: 0.9.15
Platform/OS: Win10

I am also having trouble getting Chrono to work on windows.
I can see that chrono is built and installed in carla/Build/chrono-install and executing the binaries seems to work, but chrono will not enable in the manual_control_chrono.py when pressing Ctrl+o. I've checked that the relevant line world.player.enable_chrono_physics(...) is being executed, but it just doesn't do anything.
I've also tested this on Ubuntu 20.04 Linux with the exact same setup and it works out of the box. Chrono is correctly enabled when pressing Ctrl+o


Upgrade to pants 2.21.0

Looks like CI has passed, so merging this in!

I manually retriggered the CircleCI jobs, I guess that caused them to report status correctly.


Create a field for account type when adding social media account.

If we are going to allow users to toggle their account visibility based on casual or professional, then the small form at the bottom of the social media accounts page needs to have an additional dropdown that lets them choose between one of these two options.
Additionally, on this page, each row of accounts that the user can edit should also have a dropdown to change between casual or professional.

So if I understand this correctly, you'd like for there to be a dropdown on the page to generate the QR code where a user can specify a casual or professional preset of social media handles.
If that's the case, I would suggest not doing a drop down but maybe the buttons that Leah currently has on the screen. I think there can be some front-end logic where when the button is clocked, it will automatically toggle the specified fields.
I wouldn't be opposed to having drop downs for the page where you add social media accounts

So if I understand this correctly, you'd like for there to be a dropdown on the page to generate the QR code where a user can specify a casual or professional preset of social media handles.

No, the generate QR code page looks great with the buttons the way they are, I'm not suggesting any change there.

I wouldn't be opposed to having drop downs for the page where you add social media accounts

This is all I'm suggesting. It doesn't even need to be a dropdown, it could be radio buttons or whatever, just some way that a user can indicate the account type when they edit and create a new social media account. When data is sent to the backend to add or edit an account, the account type just needs to be sent as well.

This is all I'm suggesting. It doesn't even need to be a dropdown, it could be radio buttons or whatever, just some way that a user can indicate the account type when they edit and create a new social media account. When data is sent to the backend to add or edit an account, the account type just needs to be sent as well.

Gotcha, that makes sense. From what I recall of the page dimensions, there's a lot of content so I would advise against something like a radio button since it'll have to resize to the mobile UI.

I would advise against something like a radio button since it'll have to resize to the mobile UI.

That's up to you Mr. Frontend -_-


How to disabled the image assets plugin alone?

the image assets is binding to the photoshop host, and enabled by default.
while some times the plugin just crash down because of the assert function used in the code.
eg.
        assert.strictEqual(this.id, rawChange.id, "Document ID mismatch");
        assert.strictEqual(this.version, rawChange.version, "Version mismatch.");

when the plugin crash down, it breaks the generator runtime also, which makes my own generator plugin not working...
here is the exception log

Error trace: AssertionError: Document ID mismatch
    at Document._applyChange (C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Plug-Ins\Generator\assets.generate\lib\dom\document.js:740:16)
    at DocumentManager._processNextChange (C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Plug-Ins\Generator\assets.generate\lib\documentmanager.js:306:32)
    at DocumentManager.<anonymous> (C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Plug-Ins\Generator\assets.generate\lib\documentmanager.js:256:18)
    at _fulfilled (C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Generator-builtin\node_modules\q\q.js:787:54)
    at self.promiseDispatch.done (C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Generator-builtin\node_modules\q\q.js:816:30)
    at Promise.promise.promiseDispatch (C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Generator-builtin\node_modules\q\q.js:749:13)
    at C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Generator-builtin\node_modules\q\q.js:557:44
    at flush (C:\Program Files\Adobe\Adobe Photoshop CC 2017\Required\Generator-builtin\node_modules\q\q.js:108:17)
    at nextTickCallbackWith0Args (node.js:415:9)
    at process._tickDomainCallback (node.js:385:13)

Turn of the Enable Generator switch will shut down all generator plugins, that's not what i want.
So, is there some ways to disable the image assests plugin alone,
Or make my own generator plugin in some other process?
thanks a lot.
xiaoqiang  from china.

Any update on this?
assets-generator is causing a problems in adobe cc 2019; bundled generator-assets: v2.9.8
This happens at some point when I send JSX to generator to execute:
generator.evaluateJSXString()

[warning:generator-assets 00:11:51.579 document.js:848:38] Unhandled property in raw change: parentID 261
[warning:generator-assets 00:11:52.593 document.js:848:38] Unhandled property in raw change: parentID 195
[warning:generator-assets 00:14:19.947 document.js:848:38] Unhandled property in raw change: parentID 1163
[warning:generator-assets 00:14:20.758 document.js:848:38] Unhandled property in raw change: parentID 195
[warning:core 00:16:16.264 generator.js:133:30] Photoshop error { id: 206, body: 'Unknown JavaScript error' }
[error:app 00:16:16.266 app.js:350:25] Unknown JavaScript error


use paljs  select is empty object

use it https://paljs.com/plugins/select
import { makeSchema } from 'nexus'
import { paljs } from '@paljs/nexus'

export const schema = makeSchema({
  types: [types],
  plugins: [
    paljs(),
    nexusShield({
      defaultError: new ForbiddenError('Not allowed'),
      defaultRule: allow,
    }),
  ],
})

resolve(_parent, args, { prisma, select }) {
  console.log(' select: ', select)
}
// log
select:  { select: {} }

I don't really work with nexus these days and I really dont work with paljs, so happy to merge a PR that fixes the issue but I wont be working on this.

Closing since there is nothing actionable, feel free to submit a PR


Stop depending on geojson types v1.x

Creating this PR based on what has been discussed on issue #47 of terraformer-wkt-parser.

thx @julianrojas87. advertising support for both like i proposed in #326 wasn't the correct thing to do.


Loader goes down and then comes back up again

Hello,
First of all, awesome work with this!
Second, I have a small issue with the loader:
https://screencast.com/t/3EycFpLmuO
Basically as you can see, for some reason it goes down and then comes back up .. is there a way of restricting it to go that down?
And also, there is a small line at the end of the loading that does from right to left (I can take a video of that as well, since it's only present on mobile). This might not be from the directive, but I had to ask
Thank you

Hi @Scobee , thanks for opening up this issue and I will look into this. In the meantime, have you tried setting up the ion-refresh-position="55" to higher value? That should allow you to set a custom position where the refreshing state kicks off before it goes back up.
I would also like to see about the small line you are referring to.

Hello @omelsoft ,
Thanks for your reply
I've made a live video with the issues bc I've looked at the screencast and you can't really tell them from that
This is the link: https://drive.google.com/file/d/1JN7zxqvAQPIGJXwq6Vu69DPAmB0hPeyV/view
As you can see when you pull fast, the loader stops at 55px and reloads the content, but when you pull really slow, it acts weird, it goes down too much, and it also spins randomly when it starts showing up from under the header
Plus, you can also see the right line I was telling you about

i got same problem. I fixed that by set pullMax=60

Hi.
i got same problem but i fixed it with @luongtu996 solution. But idk how to fix the other problem. The other problem is, i can see some weird line, i circled it in the photo below.


Change default URL path to deepstream

Change the default URL path to 'deepstream' on both client and server to decouple us from engine.io in case we ever intend to switch
urlPath: /engine.io

Done in server, requires client change


Reduce spread operators for perf round 2: .parse()

After https://github.com/fabian-hiller/valibot/pull/46, we still have object spread operator when calling .parse(). I found this also has negative performance impact.
I'll post benchmarks and will try to update other files as well

I'll check the changes in 15 minutes and give you feedback. I'll commit a few changes to the main branch that I've made in a few minutes.

@fabian-hiller, thanks for handling the actual remaining changes 👍
Now Valibot is faster than Zod for valid data 🎉
This means probably we should see better score than Zod in https://github.com/moltar/typescript-runtime-type-benchmarks after we publish the new Valibot package since their benchmark seems to be checking only "valid data" case
vs. Previous Valibot
I ran naruaway/valibot-benchmarks using Node.js v20.5.0 and Bun v0.7.2 on my MacBook Air M2 to compare this PR (27278fe99be29dfa96fe950b126ad7fca54218d5) against the main branch (27278fe99be29dfa96fe950b126ad7fca54218d5).

Significant speed up (more than 100%) is observed for both Node.js and Bun.
the most notable case is 170% faster for Node.js :tada: for deep data set
This PR improved performance consistently across the all combinations

Valibot in this PR compared against Valibot `main`

Running benchmarks using nodejs
Showing the result when running runtime environment (e.g. Node.js) 100 times for each:
case: "deep", data: "valid"
 170% faster
  baseline: 130783 ops/s, +- 3% stddev
  target: 352828 ops/s, +- 3% stddev
case: "many_features", data: "invalid"
 13% faster
  baseline: 16468 ops/s, +- 2% stddev
  target: 18690 ops/s, +- 2% stddev
case: "many_features", data: "valid"
 38% faster
  baseline: 77907 ops/s, +- 1% stddev
  target: 107824 ops/s, +- 1% stddev
case: "optional_nullable", data: "valid"
 83% faster
  baseline: 207116 ops/s, +- 2% stddev
  target: 378485 ops/s, +- 3% stddev
case: "wide", data: "invalid"
 7% faster
  baseline: 3756 ops/s, +- 4% stddev
  target: 4013 ops/s, +- 4% stddev
case: "wide", data: "valid"
 80% faster
  baseline: 128989 ops/s, +- 4% stddev
  target: 232356 ops/s, +- 4% stddev
Running benchmarks using bun
Showing the result when running runtime environment (e.g. Node.js) 100 times for each:
case: "deep", data: "valid"
 69% faster
  baseline: 230037 ops/s, +- 5% stddev
  target: 388609 ops/s, +- 4% stddev
case: "many_features", data: "invalid"
 7% faster
  baseline: 81143 ops/s, +- 4% stddev
  target: 86581 ops/s, +- 3% stddev
case: "many_features", data: "valid"
 50% faster
  baseline: 264671 ops/s, +- 5% stddev
  target: 397774 ops/s, +- 5% stddev
case: "optional_nullable", data: "valid"
 112% faster
  baseline: 200650 ops/s, +- 7% stddev
  target: 425620 ops/s, +- 9% stddev
case: "wide", data: "invalid"
 10% faster
  baseline: 20165 ops/s, +- 4% stddev
  target: 22141 ops/s, +- 3% stddev
case: "wide", data: "valid"
 95% faster
  baseline: 147967 ops/s, +- 6% stddev
  target: 288962 ops/s, +- 5% stddev
out of 12 cases, 12 cases (100%) were faster

vs. Zod
I ran a similar benchmark as the above but to compare this PR against Zod

:tada: Now Valibot is faster than Zod for any test cases I prepared for valid data
Note that when the data is invalid, Valibot is consistently slower than Zod for any engine / schemas

Valibot in this PR compared against Zod v3.21.4

Running benchmarks using nodejs
Showing the result when running runtime environment (e.g. Node.js) 100 times for each:
case: "deep", data: "valid"
 110% faster
  baseline: 171819 ops/s, +- 2% stddev
  target: 361375 ops/s, +- 2% stddev
case: "many_features", data: "invalid"
 43% slower
  baseline: 32915 ops/s, +- 3% stddev
  target: 18823 ops/s, +- 2% stddev
case: "many_features", data: "valid"
 21% faster
  baseline: 87467 ops/s, +- 8% stddev
  target: 105528 ops/s, +- 5% stddev
case: "optional_nullable", data: "valid"
 88% faster
  baseline: 201040 ops/s, +- 6% stddev
  target: 378693 ops/s, +- 4% stddev
case: "wide", data: "invalid"
 82% slower
  baseline: 21994 ops/s, +- 6% stddev
  target: 4031 ops/s, +- 6% stddev
case: "wide", data: "valid"
 82% faster
  baseline: 130101 ops/s, +- 8% stddev
  target: 236298 ops/s, +- 5% stddev
Running benchmarks using bun
Showing the result when running runtime environment (e.g. Node.js) 100 times for each:
case: "deep", data: "valid"
 164% faster
  baseline: 148862 ops/s, +- 5% stddev
  target: 392686 ops/s, +- 6% stddev
case: "many_features", data: "invalid"
 57% slower
  baseline: 203677 ops/s, +- 2% stddev
  target: 86844 ops/s, +- 2% stddev
case: "many_features", data: "valid"
 115% faster
  baseline: 192470 ops/s, +- 1% stddev
  target: 413984 ops/s, +- 1% stddev
case: "optional_nullable", data: "valid"
 76% faster
  baseline: 249489 ops/s, +- 2% stddev
  target: 439901 ops/s, +- 6% stddev
case: "wide", data: "invalid"
 70% slower
  baseline: 74880 ops/s, +- 1% stddev
  target: 22697 ops/s, +- 1% stddev
case: "wide", data: "valid"
 57% faster
  baseline: 190875 ops/s, +- 2% stddev
  target: 298863 ops/s, +- 4% stddev
out of 12 cases, 8 cases (67%) were faster


edk2.py: Improve firmware volume (de)serialization

The last field of the firmware volume header is a variable size, run-length encoded blockmap, we cannot and shouldn't hardcode this.
The length of the firmware volume header can be variable and is described by the HeaderLength field.
Firmware volumes can have so called "extension headers", we don't understand these, thus it is likely better to error out then produce bad data when encountering them.

I added a few test cases to the repo which fail with this pull request applied. Would you mind to have a look? :)

Thank you! :)


Any plans to add a SAX parser ?

Or there is any way that the library can be used for parsing in streaming apps?

Unfortunately, pugixml is not made with streaming use case in mind. I think there are some pugixml users that do manual framing with string searches and parse the resulting buffers into fresh documents (say, expecting a stream of  nodes, you could find  with strstr and some manual buffer stitching). I don't necessarily recommend this approach - it's pragmatic but not robust to arbitrary changes in the input format :(
I've thought about adding a SAX mode, but my current understanding is that it's close to a complete rewrite - while some parsing techniques that are used can be adapted, a streaming parser that can have a guaranteed minimum working set (given some sensible requirements wrt document structure, like having a limit on the tag name length etc.) will have to do certain things like chunking PCDATA contents which seems like it renders all pugixml parsing code invalid.
I'll keep this issue open for now to see if others have comments and/or to gauge interest but the strengths of pugixml as I see them right now pretty much make SAX parser a separate effort which is not planned.

@zeux thanks for your reply. It's clear that a SAX mode will make a difference. right now I don't see to many libraries with good performances with SAX. I think the only option you have is expat which I don't think it's maintain any longer.

I have implemented a simple SAX parser for pugixml & rapidxml at here:
https://github.com/halx99/xmldrv3

This SAX parser is nice but unfortunately since it reuses the same parsing core/approach it requires the entire document buffer to be present up front, and to be mutable. What I had in mind when writing my original comment to this issue is a very different structure that would make it possible to e.g. parse a multi-gigabyte XML file in a small (several kilobytes) buffer, unless the XML in question has large text nodes - although even then it might make sense to segment these to guarantee the limited memory footprint.

Hello @zeux  I followed your advice and implemented in top of pugixml a small class with a minimum overhead that breaks the stream into independent stanzas: https://github.com/silviucpp/erlxml/blob/master/c_src/xmlstreamparser.cc . Once a complete stanza is found will be sent to pugixml. This seems to work pretty nice with our ejabberd servers and also parsing performance was improved a lot from 112.71 MB/sec with one of the most mature erlang parser based on expat to 558.53 MB/sec with this one based on pugixml. Of courser there are some limitations we don't care right now about like :

not supporting cdata
not supporting comments with special xml charachters inside
not supporting doctype

Thanks for your advice,
Silviu

Since I currently see SAX parsing effort as a completely separate codebase without much opportunity for code sharing, I'll close this issue with wontfix. Please feel free to continue the discussion or share more links to code that can be used to assist pugixml in parsing data in a streamable fashion.


Use validate_axis.

Codecov Report

Merging #1038 into master will decrease coverage by 1.41%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1038      +/-   ##
==========================================
- Coverage    95.1%   93.69%   -1.42%
==========================================
  Files          34       34
  Lines        6758     6768      +10
==========================================
- Hits         6427     6341      -86
- Misses        331      427      +96

Impacted Files
Coverage Δ

databricks/koalas/series.py
96.41% <100%> (ø)
:arrow_up:

databricks/koalas/base.py
94.96% <100%> (+0.06%)
:arrow_up:

databricks/koalas/utils.py
98.17% <100%> (+0.04%)
:arrow_up:

databricks/koalas/namespace.py
87.34% <100%> (+0.03%)
:arrow_up:

databricks/koalas/frame.py
96.55% <100%> (+0.06%)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
80.85% <0%> (-6.39%)
:arrow_down:

databricks/conftest.py
94% <0%> (-4%)
: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 0baa4f0...703f32c. Read the comment docs.


Allow using quoted strings in docker command

I want to run a command like /bin/sh -c 'some script here', but the command line is splitted into array ["/bin/sh", "-c", "'some", "script", "here'"] instead of ["/bin/sh", "-c", "some script here"]. Please fix parsing command line to allow escaping spaces (shell-like syntax would be the best).

Where do you want to run it? In dockerCloud this command will be overwritten in any case by launcher (or maybe entrypoint).

I'm trying to use ancient docker-freebsd (fork of docker 1.7; yes, it's a crap) with SSH launcher. I'm not sure that I will be able to get java working in it, but the problem I'm trying to solve currently is that I cannot run a script. It is not overridden, but it is splitted as I described and unusable.

Could you describe what at all do you want achieve?

I need FreeBSD inside a container. There are several problems, in particular DNS does not work until /etc/resolv.conf is fixed. This must be done after running the container because docker will overwrite it otherwise. So I'm trying to run sh -c 'echo "nameserver <IP_ADDRESS>" > /etc/resolv.conf && /usr/sbin/sshd -D'. The command specified in CMD doesn't run for some reason, so I use "Docker Command" in "Create Container Settings".

launchers overriding cmd/entrypoint with their own commands.
nameserver should be in create options afair. And better use JNLPLauncher...

--dns=IP_ADDRESS... | Sets the IP addresses added as nameserver lines to the container's/etc/resolv.conf file

https://docs.docker.com/engine/userguide/networking/default_network/configure-dns/

https://github.com/KostyaSha/yet-another-docker-plugin/blob/master/yet-another-docker-plugin/src/main/resources/com/github/kostyasha/yad/commons/DockerCreateContainer/help-dnsString.html

Thank you, but --dns does not work in this case.
Shell,  Shell for when RUN, CMD, and ENTRYPOINT uses a shell
do you know how implement right parsing?

Sorry, I'm not a Java coder, but some answers to very similar questions at SO suggest using regexes:

Split string on spaces in Java, except if between quotes
Tokenizing a String but ignoring delimiters within quotes

I personally would prefer something like this, in C-like pseudocode:
char quot = '\0';
for (char *p = s; *p != '\0'; p++) {
    switch (*p) {
    case '\'':
    case '"':
        if (quot == '\0') {
            quot = *p; // open quote
        } else if (quot == *p) {
            quot = '\0'; // close quote
        } else {
            append_to_token(*p); // quoted quote
        }
        break;
    case '\\':
        p++;
        append_to_token(*p); // escaped character
        break;
    case ' ':
    case '\t':
        if (quot) {
            append_to_token(*p); // quoted space
        } else {
            end_token();
        }
        break;
    default:
        append_to_token(*p);
        break;
}
end_token();

ok, will pick something.
PS My apple laptop gpu is crapped again...

https://github.com/KostyaSha/yet-another-docker-plugin/pull/178/files#diff-8f38882cbf864d4a172bd377b63443a8R318

@mikhirev please try newest version.


Add Properties for activation and changed the planet sorting

Can you review the percentage calculation for the planet activation?
Now we can log the activations like this:
Console.WriteLine("\nPersonality Gate Activations:");
foreach (var (planet, activation) in chart.PersonalityActivation)
{
    Console.WriteLine($"\t{planet} {activation}");
}

Console.WriteLine("\nDesign Gate Activations:");
foreach (var (planet, activation) in chart.DesignActivation)
{
    Console.WriteLine($"\t{planet} {activation}");
}

Personality Gate Activations:
        Sun <IP_ADDRESS>.3 C26% T56% B82%
        Earth <IP_ADDRESS>.3 C26% T56% B82%
        NorthNode <IP_ADDRESS>.4 C45% T71% B53%
        SouthNode <IP_ADDRESS>.4 C45% T71% B53%
        Moon <IP_ADDRESS>.4 C61% T66% B31%
        Mercury <IP_ADDRESS>.4 C77% T63% B17%
        Uranus <IP_ADDRESS>.4 C29% T72% B60%
        Venus <IP_ADDRESS>.1 C19% T16% B81%
        Mars <IP_ADDRESS>.1 C36% T15% B73%
        Neptune <IP_ADDRESS>.2 C6% T38% B91%
        Saturn <IP_ADDRESS>.3 C26% T55% B76%
        Jupiter <IP_ADDRESS>.1 C1% T6% B31%
        Pluto <IP_ADDRESS>.3 C42% T52% B61%

Design Gate Activations:
        Sun <IP_ADDRESS>.2 C6% T36% B82%
        Earth <IP_ADDRESS>.2 C6% T36% B82%
        NorthNode <IP_ADDRESS>.5 C31% T86% B29%
        SouthNode <IP_ADDRESS>.5 C31% T86% B29%
        Moon <IP_ADDRESS>.1 C68% T8% B40%
        Mercury <IP_ADDRESS>.2 C22% T30% B49%
        Uranus <IP_ADDRESS>.4 C27% T61% B4%
        Venus <IP_ADDRESS>.1 C1% T5% B25%
        Mars <IP_ADDRESS>.4 C77% T61% B6%
        Neptune <IP_ADDRESS>.4 C95% T69% B45%
        Saturn <IP_ADDRESS>.4 C95% T69% B45%
        Jupiter <IP_ADDRESS>.2 C72% T31% B57%
        Pluto <IP_ADDRESS>.1 C19% T11% B57%

For example this line here:
ColorPercentage = (((x % DegreePerLine) % DegreePerColor) / DegreePerColor) * 100.0
You can skip the line part and just write:
ColorPercentage = ((x % DegreePerColor) / DegreePerColor) * 100.0
This should be the same result, but it's a bit more straight forward to read.


How to cancel the waveform processing before done?

I have added that package to my flutter project.
But when move before and next page, I met a issue for percent change one.
The percent value is return the previous and current ones.
Could you help me for that issue?

There is already a pull request to support simultaneous requests which would help prevent the wire crossing you're experiencing, but also a way to cancel a processing job is a good idea.
I'll note it as a new feature idea.

I am re-opening this issue because I thought it was a good idea!


Unable to upgrade JAVA JDK in my project due to "NewRelic Sdk". As it is STOP logging the traces in account DashBoard when upgrade JDK from 17 to 22.

Description
I am unable to upgrade JAVA JDK (from 17 to 22) in my project due to NewRelic Sdk. As in this SDK it STOP working/logging the traces in account DashBoard.
TIP

Task :app:newrelicTransformClassesForDebug
[InvocationDispatcher] Class[unknown] ignored: JDK not supported
[InvocationDispatcher] Class[unknown] ignored: JDK not supported
[InvocationDispatcher] Class[unknown] ignored: JDK not supported

Configure project :app
The New Relic plugin may not be compatible with Android Gradle plugin version 8.5.1.
AGP versions 7.0.0 - 8.4 are officially supported.
Set property 'newrelic.warning.agp=false' to disable this warning.
Set property 'newrelic.halt-on-warning=true' to treat warnings as fatal errors.
AGPBI: {"kind":"warning","text":"Accessing value buildConfigFields in variant debug has no effect as the feature buildConfig is disabled.","sources":[{}]}
AGPBI: {"kind":"warning","text":"Accessing value buildConfigFields in variant release has no effect as the feature buildConfig is disabled.","sources":[{}]}

Steps to Reproduce

Demo integration sample can be found here

Expected Behavior

It should keep working with existing task.
It should allow other application to upgrade there JAVA JDK versions.

Relevant Logs / Console output

Your Environment

NewRelic Android agent '7.4.1' (agent details)
using kotlin  version "2.0.0" (i.e. org.jetbrains.kotlin.android & org.jetbrains.kotlin.plugin.compose)
AndroidGradlePlugin version "8.5.1"
JAVA JDK version 22 ( it only working till JDK 17 )

Additional context

@cthomas-newrelic @ywang-nr
This should be a quick fix considering the Past Pull-Request. But not sure about complexity if some BOLCER are here.

@PratyeshSingh We have work lined up for verifying upgrade agent to JAVA21+, no ETA at the moment but will keep this issue posted with updates.

@ndesai-newrelic any update on this ?

We will have a release coming in the next few weeks. Thanks.

v7.6.0 is released. You can find the detailed release notes here.


sqlcmd not found when running MSSQL database create statements in AWS AMIs

Description:
MSSQL client utility in AMIs are not working when running sql commands from AWS user.
Related Issues:
#590

Who is the AWS user?

Output of "whoami" command says it is the "root" user. This was fixed by adding symbolic links to sqlcmd bin directory.


comparing TrueClass fails

What Ruby, Rails and RSpec versions are you using?
Ruby version: 2.6.5
Rails version: <IP_ADDRESS>
RSpec version: 3.9.2
Observed behavior
Expectation is failing when comparing values. I have a spec that is failing even though the value is as expected. This only happens when the name param is set to '' or ' '. When printing the response to the console it prints true but expect(response).to be_truthy fails. I've tried expect(response).to eq(true) as well as expect(response).to be(true)
Can you provide an example app?
'Carmen Jones', ' '].each do |name|
          it "return true for common_name: #{name}" do
            data = { subject_dn: {
              country_name: 'US',
              state_or_providence: 'CA',
              locality_name: 'Pasenda',
              organization_name: 'Random Corporation',
              organizational_unit_name: 'Information Technology Department',
              postal_code: '91001',
              street_address: '3000 Pasadena Blv',
              given_name: 'Marcus',
              sur_name: 'Meredith',
              common_name: name
            } }
            response = described_class.new(JSON.dump(data)).call
            puts "response -> #{response.class}:#{response}"
            expect(response).to be_truthy
          end
        end

Hopefully you figured it out but be_truthy generally means "not nil or false" in Ruby. So any other value will pass be_truthy whilst eq(true) means it must return true when compared using == etc and be true means it must actually be the value true.
Theres not enough information here to help further so hopefully you sorted it.


[Dialog] The Save changes button has the wrong accessible name and the close button has no accessible name

Bug report
Current Behavior
The close button has no other content than just the SVG. This means for screen readers this will be announced as "button" with no context of what it does.
In the frontend for the Save Changes button:
<button aria-label="Close" type="button" class="c-hYXuTU c-hYXuTU-eqKpNk-variant-green">Save changes</button>
The aria-label overrides the content inside.
In the frontend for the close button:
<button type="button" class="c-kLfaJB">
    <svg width="15" height="15" viewBox="0 0 15 15" fill="none" xmlns="http://www.w3.org/2000/svg">
        <path d="M11.7816 4.03157C12.0062 3.80702 12.0062 3.44295 11.7816 3.2184C11.5571 2.99385 11.193 2.99385 10.9685 3.2184L7.50005 6.68682L4.03164 3.2184C3.80708 2.99385 3.44301 2.99385 3.21846 3.2184C2.99391 3.44295 2.99391 3.80702 3.21846 4.03157L6.68688 7.49999L3.21846 10.9684C2.99391 11.193 2.99391 11.557 3.21846 11.7816C3.44301 12.0061 3.80708 12.0061 4.03164 11.7816L7.50005 8.31316L10.9685 11.7816C11.193 12.0061 11.5571 12.0061 11.7816 11.7816C12.0062 11.557 12.0062 11.193 11.7816 10.9684L8.31322 7.49999L11.7816 4.03157Z" fill="currentColor" fill-rule="evenodd" clip-rule="evenodd">
      </path>
   </svg>
</button>

Expected behavior
The "Save changes" button should have no aria-label, the text inside is perfect on it's own and should not be overwritten.
The "Close" button must have an accessible name, so a screen reader can announce the purpose of the button and voice recognition software can target the button more easily.
Reproducible example
Dialog on radix-ui.com
VoiceOver recording:
https://user-images.githubusercontent.com/2351479/173365056-e67ba2d6-aa60-4526-b38f-86dc526e713d.mov
Suggested solution
In the frontend for the Save Changes button, remove the aria-label:
<button type="button" class="c-hYXuTU c-hYXuTU-eqKpNk-variant-green">Save changes</button>
In the frontend for the close button, add the aria-label "Close"
<button type="button" class="c-kLfaJB" aria-label="close">
    <svg [...]</svg>
</button>

Or even better: Add visual text, then also people who don't know what the X means benefit too:
<button type="button" class="c-kLfaJB" aria-label="close">
     Close
    <svg [...]</svg>
</button>

Your environment
MacOs Monterey 21.1 with Safari 15.2 and VoiceOver.
Tested software on Dialog on radix-ui.com

@benoitgrelard Thank you :-D

Hey @rianrietveld,
Good catch! The close label had been applied to the save button by mistake instead of the close button.
It's fixed now in #421.
You can preview the change here: https://radix-website-git-dialog-example-accessibility-fix.modulz-deploys.com/docs/primitives/components/dialog


Packages overriding inherited lints in Cargo.toml + adding lints

Problem
Hi,
With the release of 1.74.0 and Lint configuration through Cargo I was interrested to have lints configure for my workspace and to override some of them in specific crates.
I was met with an error in rust-analyzer that stopped working with error :

cannot override workspace.lints in lints, either remove the overrides or lints.workspace = true and manually specify the lints"

I was surprised as my intuition was that it should have work, especially with the level system for each lint. Nothing in the Cargo Book mention that it would be impossible to override lints if inherited from the workspace (I looked in the manifest format and in workspace reference)
After researching a long time in various places, I finally found in the RFC 3389 (here, precisely) the confirmation that this is an hard error:

Currently, it is a hard error to mix workspace = true and lints. We could open this up in the future for the package to override lints from the workspace.

(By the way, after testing, it's not only overriding lints that is problematic, also adding new lints to the inherited lints in a crate is an hard error)
While it is still possible in lib.rs and/or in main.rs to add attributes #![allow(...lints...)] to override lints, I think it should still be added, as with the level system, it seems clear who override who. For me the main reason I prefered to put my "global/crate wide" lints in the cargo.toml manifest, was to get a clear view of wich crate in my workspace override what lint without touching to lib.rs/main.rs
For example we could have a workspace where in general, we wish to deny unsafe code but we could have one very low level crate that override it because we absolutly need it.
Example:
# Cargo.toml of workspace

[workspace.lints.rust]
unsafe_code = { level = "deny", priority = -1 }

[workspace.lints.clippy]
all = { level = "deny", priority = -1 }
cargo = { level = "deny", priority = -1 }
pedantic = { level = "deny", priority = -1 }
too_many_lines = "allow"

# Cargo.toml of a member crate

[lints]
workspace = true

[workspace.lints.rust]
unsafe_code = "allow"

Proposed Solution
I see 2 solutions:

We can override and/or add lints in crates that inherited from workspace as I explained above.
We leave the lints in cargo.toml as it is now but we document in the Cargo Book (here  and here) the interdiction to add or/and override if  lints.workspace = true. And we explain that the idiomatic way to do is to use the attributes #![allow(...lints...), deny(....), ....] in lib.rs/main.rs

I personally prefer solution 1) as it is more intuitive and the level system permits this.
Notes
No response

Huh, for some reason I thought we had an issue for this but I'm not seeing it.

There's a few lints where being able to do this is fairly critical for some use cases.
If I have written a wrapper for some third party/std lib function 'foo' to hook in some extra logic/validation and I want to ensure that all of my code uses the wrapper and not the original version of the function the 'disallowed_' lints are useful, but I need to be able to suppress the warning/error in the implementation of the wrapper, otherwise I'll forever be stuck with 1 error/warning that wont go away.

@pcone is there a reason you can't just put an #[allow(...)] on that one use?
We actually have a similar situation in Cargo where std::env::var access is banned except for our wrapper (and some odd cases) and that is what we do.

@pcone is there a reason you can't just put an #[allow(...)] on that one use?
We actually have a similar situation in Cargo where std::env::var access is banned except for our wrapper (and some odd cases) and that is what we do.

Ah you're quite right. This was a case of me not reading the docs. I assumed 'warn' and 'forbid' were the same as 'warn' and 'error' in other languages - I didn't catch that there's actually warn, force-warn, deny and forbid and that you're not allowed to override force-warn and forbid. I went down a rabbit hole of searching and came to this issue which made me think it was related.


[QUESTION] Exception is not thrown as expected when the session is killed in SQL database

Question
A follow up issue of #1846: https://github.com/microsoft/mssql-jdbc/issues/1846
Hi @lilgreenbird, I have investigated the issue with the Databricks side. From the current findings, the issue may still persist with the newly released 11.2.0 version: https://github.com/microsoft/mssql-jdbc/releases/tag/v11.2.0
We have tried to only use Scala codes, which don’t involve Spark and just tries to call the library in the same way how Spark calls it. . After killing the session and giving only part of the records, but the error doesn’t show up. It could be the scenario when the error doesn't show up.
%scala
import java.sql.DriverManager
import java.sql.Connection
import java.util.Properties;
import java.sql.{Connection, PreparedStatement, ResultSet}
val jdbcHostname = "xinrandatabseserver.database.windows.net"
val jdbcDatabase = "xinranSQLDatabase"
val jdbcPort = "1433"
val driver = "com.microsoft.sqlserver.jdbc.SQLServerDriver"
val token = "xxxxxxxxx"
val jdbcUrl = "jdbc:sqlserver://%s:%s;databaseName=%s;encrypt=true;trustServerCertificate=false;hostNameInCertificate=*.database.windows.net".format(jdbcHostname, jdbcPort, jdbcDatabase)+ ";accessToken="+token
val info:Properties = new Properties();
info.setProperty("accesstoken", token);
// make the connection
Class.forName(driver)
connection = DriverManager.getConnection(jdbcUrl,info )
val sqlText = "SELECT 1 FROM (select UNITS from payment_balance_new) emp_alias"
val stmt = connection.prepareStatement(sqlText,
ResultSet.TYPE_FORWARD_ONLY, ResultSet.CONCUR_READ_ONLY)
stmt.setFetchSize(0)
stmt.setQueryTimeout(0)
val resultSet = stmt.executeQuery()
var count = 0
while ( resultSet.next() ) {
// println("__________________________"+resultSet.getString(1))
count = count + 1
}
print(count)
// the actual result should be 2673440
//assert(count == 2673440)

Could you help further look into it? Thanks in advance!
Relevant Issues and Pull Requests
https://github.com/microsoft/mssql-jdbc/issues/1846
https://github.com/microsoft/mssql-jdbc/pull/1857

Hi @DorisTao77,
We're following up on this issue through the email you sent, expect updates on that thread.

Hi @Jeffery-Wasty , thanks for your reply!
I have just updated on the email thread.
The used java code example from my side to programmatical kill the session, the returned results would show clearly how many records were returned, the number is less than the actual records:
java codes.txt
We have had a further tests from our side. We may have observed that the performance may be different on the Ubuntu machine when they have different cores, even we are using the exact same codes and same Java version.
•	When the Ubuntu machine only has only 1 core, the error throws as expected when the session is killed, same as the behavior on Windows:

•	While the Ubuntu machine has more than 1 core, for example 4 cores, the error could not show up when the session is killed for most of the time:

While the behavior seems different for the Ubuntu on Window, although it has multiple cores:

As Databricks clusters would use Ubuntu VMs with multiple cores for parallelism, the tests seem to be able to explain the reason behind the issue.
Could you help check from your side about the different behavior on Ubuntu VM with different number of cores, and the possible reason behind? Thanks in advance!

could you please give this a try I made a change it's just a guess since I'm unable to repro the issue you are seeing.
driver.zip

Hi, I have downloaded and retired with this new version, but the results are the same.

Hi, I have shared the test details in the internal email thread, you can also try to login the shared VM to check more details, thanks in advance!

thanks @DorisTao77 I believe I've found the issue now here are the jars built with the change in PR #1943 please give this a try
driver.zip

Hi @lilgreenbird , thanks so much for your hard work!
I have done tests from my side with this newly provided library(Version 12.1.0) on the multi-core Ubuntu VM, the failure (Done Status 258) would show up every time during the tens of tries.

Have also done the same tests on the Databricks side with Spark, the behavior is the same:

I believe the issue should have been resolved with this latest jar. May I also confirm which driver version is expected to have this fix? Thanks again!

Closed by #1943


Refactor Bazel unit test CI

Unify workflows for Unix and Windows and use more efficient caching for workflow artifacts. Harden the workflow with fixed action hashes.
Reduces Linux workflow runtime to ~1 min for best-case cache hits.

This change is

FYI cache reuse runtimes are in the workflows here (the ones that take ~50 seconds :smiling_imp:) https://github.com/aaronmondal/turbo-cache/pull/3


Expose API to use siddhi manger level datasources at the store extension levels

Description:
Siddhi Manager has API to set SQL data sources, however, these are not exposed to store extensions.
In the case of rdbms:query() and rdbms:cud(), functions, even though these functions expose the functionality to run queries against a data source, these work only in OSGi environment, However if the siddhi manager exposes the data sources, rdbms:query() functions can be used in vanilla siddhi also.
Affected Siddhi Version:
v5.1.x

Datasources are set to SiddhiContext at the SiddhiManager.
Can't we get the SiddhiContext from the store and get the relevant data sources from there?


Add solution for leetcode problem 25 Reverse Nodes in k-Group

Added solution for Leetcode problem 25 - Reverse Nodes in k-Group. Link - https://leetcode.com/problems/reverse-nodes-in-k-group/description/
Please review

Follow guidelines


Merge pull request #34 from Cloudanix/gcp-function

Gcp function
Made the changes as suggested in the Notion Card

Closing it. Ajinkya would merge this code into the gcp-function branch


Only accept the item you click on the first time you click on a category widget

According to @saleiva, when all items of a category widget are selected and you click on an item, instead of rejecting the one you clicked, we should only accept that one (or reject all others). I guess this should be the behaviour the first time an item is clicked.

Yep, also the auxiliar controls are missing from the actual version (buttons for selectiing/unselecting all items)

yep, you are right @saleiva

https://github.com/CartoDB/cartodb.js/issues/750

Done


(#7) Add timeout option

Thank you. I added ourselves to Codeship (I wasn't there either) and added a commit making the syntax uglier, for Ruby 1.9.3 compatibility. I'm not sure whether that's necessary but it's in the README.


Use Docker BuildKit in Esti

This may improve build times by being better able to use existing Docker caches.

Now happening :-)


Communication with web site

is there a way or can it be done in future?
for example:
specifically load torrent or magnet though webtorrent?
get back from webtorrent local link to video file on user machine, so it can be playd through video player on site ?
so basically it be nice to have a way to communicate with webtorrent from site, so it can be used as service on user machine

You can already paste magnet links from anywhere into WebTorrent Desktop
You can also set WebTorrent Desktop as your default magnet link and torrent file handler, after which you can just click on a magnet link and it should open in the app

I know about this, and this is not my question/suggestion.
I don't want webtorrent to be default torrent client, its not for that, at this point.
I need so a way of communicate with webtorrent from outside. Pragmatically


Nans near end of training run

Thank you for your nice implementation of YOLOv4 in TF2. I am using the tiny version on a custom dataset and sometimes I encounter nans during training. Do you have any ideas as to why this is happening? Is it possible for there to be a division by zero in the loss function?

what is yolo version?
python3 -m pip show yolov4

can you share your dataset sample?

Version 2.0.3.
I'm using a custom tf data pipeline, so I cannot easily share. However, I'm not using any data augmentation and the nans only appear after several epochs, which leads me to believe my data loader is not the problem (i.e., the model sees all the samples several times before the nans appear).

iou, obj, cls loss are all nan?

I turned loss verbose off, but here is the model.fit verbose:
Epoch 15/20
188/188 [==============================] - 226s 1s/step - loss: 2.7143 - output_1_loss: 2.3952 - output_2_loss: 0.0162 - val_loss: 8.0879 - val_output_1_loss: 7.7695 - val_output_2_loss: 0.0173
Epoch 16/20
188/188 [==============================] - 222s 1s/step - loss: nan - output_1_loss: nan - output_2_loss: 0.0114 - val_loss: nan - val_output_1_loss: nan - val_output_2_loss: 0.0000e+00
I will turn loss verbose back on and see if I can get more information.

I have modified a lot of parts where nan may occur.
and I am working on yolov4 v3.0.0 now and reviewing the math part again.
Please leave a comment if you come up with a reason for nan other than the math part.

Ok that's great. Hopefully the issue is resolved in v3.0.0. Do you know when you will be releasing it?

To improve the training speed, I'm converting some of the dataset related codes to c++.
There are a few other works left, but it will probably take about a week.

Looks like the issue is related to conf_loss:
step before nan:
grid: 30*30 iou_loss: 2.21239805 conf_loss: 0.192396224 prob_loss: 0.0061624 total_loss 2.41095662
grid: 15*15 iou_loss: 0 conf_loss: 0.0158854965 prob_loss: 0 total_loss 0.0158854965

nan step:
grid: 30*30 iou_loss: 2.03567219 conf_loss: nan prob_loss: 0.00672992691 total_loss nan
grid: 15*15 iou_loss: 0 conf_loss: 0.0159363803 prob_loss: 0 total_loss 0.0159363803

step after nan:
grid: 30*30 iou_loss: 34.6766624 conf_loss: nan prob_loss: 89.1754837 total_loss nan
grid: 15*15 iou_loss: 0 conf_loss: 0 prob_loss: -0 total_loss 0

Note that all my bounding boxes are small (12x12 px using input size of 480x480), so I think that's why the iou loss for 15*15 grid is always 0.

Looking at the loss function now... if pred_conf + 1e-9 = 1, you will run into backend.log(0)... Do you think that could be it?

It can be.
K.epsilon and np.finfo(np.float32).eps are about 1e-7.
v3.0 use 1e-7. for more safe, actually use K.binary_crossentropy. K is backend.
change 1e-9 to 1e-6 , 1e-7 or backend.epsilon()

yolov4 v3.0.0 is released.
https://wiki.loliot.net/docs/lang/python/libraries/yolov4/python-yolov4-training

Thanks. Changing eps to 1e-7 in v2.0.3 worked for me.


[5.3]lumen use queue with redis

Laravel Version: 5.3.#
PHP Version:7.0.1
Database Driver & Version:

Description:
I want use queue in my lumen 5.3 project.
Steps To Reproduce:

composer require illuminate/Redis 5.3.*
the error is {"message":"Type error: Argument 1 passed to Illuminate\\Queue\\Connectors\\RedisConnector::__construct() must be an instance of Illuminate\\Redis\\Database, instance of Redis given, called in \/home\/wwwroot\/KMS_api\/vendor\/illuminate\/queue\/QueueServiceProvider.php on line 217","status_code":500}
then I add $app->register(Illuminate\Redis\Database::class); to app.php
the error is Type error: Argument 1 passed to Illuminate\Redis\Database::__construct() must be of the type array, object given, called in /home/wwwroot/KMS_api/vendor/laravel/lumen-framework/src/Application.php on line 164
if I run php artisan queue:table or  php artisan queue:failed-table
the error is  Command "queue:table" is not defined.

What can I do？I am so desperate ah!

Please use Lumen repo


Content-static map.blade

Hi:
Please check the text for a small description and a legend to the colours.

                The Farm Platform provides three farming systems in farmlets, each consisting of five component catchments comprising approx.
                21 ha in total per farmlet. Each farmlet is managed using alternative approaches to livestock production from grassland and
                measurements on water, air and soil are also recorded.

                 Legend to RED

                Legend to GREEN

                Legend to BLUE

Hi Nath..
Needs updating to something like this now:-
The Farm Platform provides 4 farming systems comprising of 3 farmlets and one indoor beef finishing system. Each farmlet  consists of five component catchments comprising approx. 21 ha in total per farmlet. 2 farmlets (Green and Blue) explore pasture based ruminant production systems, 1 farmlet (Red) allows the investigation of production of crops for the human food chain. The indoor beef finishing unit represents typical business as usual commercial beef production.

[ ] Get the ArcGIS map updated


My chessboard ended up like this

My chessboard ended up with this distortion how do I fix this?

Looks like you have geometry on top on each other
Either than or graphical corruption.
Try: Removing doubles from the Chess Board

did it and nothing I will try to re do the chess board again.
On Wed, Jul 12, 2017 at 12:37 PM, Michael Bridges<EMAIL_ADDRESS>wrote:

Looks like you have geometry on top on each other
Either than or graphical corruption.
Try: Removing doubles from the Chess Board
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CompleteBlenderCreator/04-Low-Poly-Chess-Set/issues/1#issuecomment-314874153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZOIKRfBuaAWrL3VA_gc2gScVTuN7PPYks5sNSBlgaJpZM4OVKx2
.

I [image: Inline image 1]also discovered that I assign new material on
separate faces without affecting the entire mesh.I always un parent each
face.I looked over the bowling pin stripe creation over and over and could
not find the solutio. Please help.I chose a purple color to better
exemplify this dilemma I have been having even before I took the course.
On Wed, Jul 12, 2017 at 12:37 PM, Michael Bridges<EMAIL_ADDRESS>wrote:

Looks like you have geometry on top on each other
Either than or graphical corruption.
Try: Removing doubles from the Chess Board
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CompleteBlenderCreator/04-Low-Poly-Chess-Set/issues/1#issuecomment-314874153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZOIKRfBuaAWrL3VA_gc2gScVTuN7PPYks5sNSBlgaJpZM4OVKx2
.


Blocking progress view

Create a blocking overlay with direct access to a progress view.

@zmears thanks for contribution! Please check my comments.


OverPack animation doesn't render if the object is at the bottom of the page

I have an issue with the (badly named) <OverPack> component which, if i understand, triggers an animation when the element is scrolled upon.
However, i have an animation to trigger but it's on the last section of my page, so it will never reach the middle of the viewport, but rather stay at the bottom of the page.
So, i don't want to trigger the animation as soon as the element comes into view (when it is fully visible in the viewport)
How should i do that ?
Is there a parameter that i missed ? There should be one to giving us the percentage of the viewport to consider to trigger an animation. In my case, it would be the full viewport's height : 0..100%
<Col className="call_to_action" span={24} lg={10}>
	<OverPack inViewport={[0, 100]}>
		<TweenOne animation={ type: 'from', x: 200, opacity: 0 } >
			<Link className="button" to={link}>{call_to_action}</Link>
		</TweenOne>
	</OverPack>
</Col>

You can set playScale = "0.1" and col minHeight

https://github.com/ant-motion/ant-motion-dva-cli-example/blob/2.0/src/routes/Home/Footer.jsx#L16

@jljsj33 Thhxx you really much ! :)
What is the meaning and values of playScale ? It seems to be defined on [0, 1] ?
Could you point me to the documentation of this component ?
I've only found this previous example :
https://github.com/ant-design/ant-motion/blob/master/components/scroll-anim/demo/overPack.md

OK. I found it :
https://motion.ant.design/api/scroll-anim#Element


Add responsive-layout feature

A lot of pages have different min-width set on the body tag (e.g. the profile page with 1280px), this feature reduces the overall min-width to 768px and adjusts some layouts below 1080px to allow this.
768px has been chosen because in a lot of cases everything "just works". Going further below that would require a lot more additional layouting.
I opted to namespace the styles with the rgh-responsive-layout class which is added to the body via JS, so the feature can be easily disabled.
It's disabled by default on pages that are already responsive (containing the page-responsive class). It should therefore be future-proof in case of GitHub updating more pages to be responsive by default.
Examples

Profile page of other user

Profile page of own user

Issue page

Repo page

Sorry but this doesn’t belong to Refined GitHub. Making a site responsive is a huge endeavor and GitHub is already working on it.
There’s probably already some userstyle for it


Caching not working for in-house/local Terraform Providers

Describe the bug
We have a Terraform Provider which is created/maintained in-house, and the installation of it is to just install it from a tarball, directly into ~/.terraform.d/plugins/ (and is also probably relevant for the local development of Terragrunt Providers too)
I'm finding that with the new caching implementation (eg: TERRAGRUNT_PROVIDER_CACHE=1), it keeps trying to lookup the provider's defined URL (which doesn't actually go anywhere) even if it already exists, irrespective of all the settings i've tried so far.
I've tried various combinations of things like:
TF_PLUGIN_CACHE_MAY_BREAK_DEPENDENCY_LOCK_FILE=1
TF_PLUGIN_CACHE_DIR=/home/myuser/.terraform.d/plugins
TERRAGRUNT_PROVIDER_CACHE_DIR=/home/myuser/.terraform.d/plugins

(domain/provider in below examples amended for privacy reasons)
Example with TERRAGRUNT_PROVIDER_CACHE_REGISTRY_NAMES=terraform.example.net:
│ Error: Failed to query available provider packages
│
│ Could not retrieve the list of available versions for provider
│ terraform.example.net/myprovider/specialthing: could not query provider registry for
│ terraform.example.net/myprovider/sepcialthing: the request failed after 2 attempts,
│ please try again later: 503 Service Unavailable returned from
│ <IP_ADDRESS>:59825

and without it:
│ Error: Failed to query available provider packages
│
│ Could not retrieve the list of available versions for provider
│ terraform.example.net/myprovider/specialthing: could not connect to
│ terraform.example.net: Failed to request discovery document: Get
│ "https://terraform.example.net/.well-known/terraform.json": dial tcp: lookup
│ terraform.example.net on <IP_ADDRESS>:53: no such host

This is different to the behaviour when TERRAGRUNT_PROVIDER_CACHE=1 is not set:
$ terragrunt init

Initializing the backend...

Initializing provider plugins...
- Reusing previous version of terraform.example.net/myprovider/specialthing from the dependency lock file
- Installing terraform.example.net/myprovider/specialthing v3.0.0...
- Installed terraform.example.net/myprovider/specialthing v3.0.0 (unauthenticated)

Terraform has been successfully initialized!

Steps To Reproduce

Create a TF module with a custom provider, which expects to be pre-installed, and does not have a valid URI - eg:

terraform {
  required_providers {
    specialthing = {
      source  = "terraform.example.net/myprovider/specialthing"
      version = ">= 0.0.1"
    }
  }
}

provider "specialthing" {}

resource "specialthing" "this" {
    name = "myname"
}

Manually place the custom provider into ~/.terraform.d/plugins
Run Terragrunt with TERRAGRUNT_PROVIDER_CACHE=1 set

Expected behavior
Support for pre-existing/manually deployed Terraform modules should be supported; as it is without the Terragrunt Provider Cache enabled.
Versions

Terragrunt version: 0.62.1
OpenTofu/Terraform version: 1.3.9
Environment details (Ubuntu 20.04, Windows 10, etc.): Ubuntu 22.04

Additional context
Discovered when migrating from Terraform cache on 0.55.3 (with the single-threaded slow init processing) to 0.63.1 to make use of the improved Terragrunt caching mechanism.

I promise I did look, but didn't see this 😅
https://github.com/gruntwork-io/terragrunt/issues/3244
Closing as duplicate.


date string coerced to datetime shouldn't infer timezone

Fix https://github.com/pydantic/pydantic/issues/8762

To fix lint: merge and rebase on this: https://github.com/pydantic/pydantic-core/pull/1191


OpenCV doesn't seem to like bitonal files

Can you please help me understand why I can't get imread to open simple bitonal files like this one?
https://app.box.com/s/y2h71ydjb7uim1um1gwqjvsuqz2cpmf2
This is an example of what I get. It worked in 3.0 rc but with 3.0.0 it no longer works.
Unhandled exception at 0x00007FF992BB9A1D (opencv_world300d.dll) in OpenCv_FastHough.exe: 0xC0000005: Access violation reading location 0xFFFFFFFFFFFFFFFF.

Could you add the line of code? Also did you check latest master branch?

I used the fast_hough_transform.cpp distributed in the opencv_contrib folder as a test jig.  So this is the first few lines out of the function that loads the image.
static bool parseArgs(int argc, const char **argv,
    Mat &img,
    int &houghDepth,
    int &houghAngleRange,
    int &houghOperator,
    int &houghSkew)
{
    if (argc > 6)
    {
        cout << "Too many arguments" << endl;
        return false;
    }

    const char *filename = argc >= 2 ? argv[1]
        : "../../../samples/data/building.jpg";
    img = imread(filename, 0);
    if (img.empty())
    {
        cout << "Failed to load image from '" << filename << "'" << endl;
        return false;
    }

I have not tried the latest branch yet.  I only tried opencv 3.0rc and 3.0.0. I just did a git pull and I'm getting my dependencies in order now to compile opencv. I'll report back when I get the latest branch compiled.

I've compiled the latest branch and it still fails.  I couldn't compile opencv_world so I just compiled the individual libraries to link against them.
This is the error.
Unhandled exception at 0x00007FF8349BE98D (opencv_imgcodecs300d.dll) in OpenCv_FastHough.exe: 0xC0000005: Access violation reading location 0xFFFFFFFFFFFFFFFF.

And the code breaks into the UNROLL8 line of tif_getimage.c.
/*
 * 1-bit bilevel => colormap/RGB
 */
DECLAREContigPutFunc(put1bitbwtile)
{
    uint32** BWmap = img->BWmap;

    (void) x; (void) y;
    fromskew /= 8;
    while (h-- > 0) {
    uint32* bw;
    UNROLL8(w, bw = BWmap[*pp++], *cp++ = *bw++);
    cp += toskew;
    pp += fromskew;
    }
}

Fix from #5357 should be ported to master branch

I can confirm that these changes fix my problem with the bitonal image.  Thanks.  Looking forward to the merge for this fix.  Thank you!


[WIP] Fix randomized relu

Description
Fixes #14447 Fixes #12894

Request the right resource manager & use latest random number generator
Add missing unit test for rrelu operator

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes
Comments

Thanks for your contributions @stu1130.
Can you look into the CI-failures on this ?
@mxnet-label-bot Add [operator, pr-work-in-progress]

@stu1130 Could you please look into the CI failure and push towards review/merge? Thanks!


Mek Generators + BC = Crash

newest Forge Version (<IP_ADDRESS>4)
newest Mek. Versions (<IP_ADDRESS>7)
newest BC Verison (7.99.6)
REPORT

Please use the search functionality.
Duplicate of #4513

oops, sorry.

@ProxyPlayerHD you are able to close your own issue btw

oh i didn't knew that.


Use your library in ObjC

Hi,
Is there a way to use your library in ObjC ?
Thanks

You should be able to, but you'll have to set it up manually. Here is a rough guide

Download the Regift.swift file from this repo.
Drag it into your Obj-C project.
Make sure your build settings are set as per these instructions.

Let me know how that goes—I'll reopen if it doesn't work :)

I'm having the same problem,
on step 3, i've added :
#import "myProjectName-Bridging-Header.h" #import "myProjectName-Swift.h"
and @objc on the swift class.
Regift keywork appears in the autocompletion
i dont know how to get access to methods though

@Louis-Fournier what errors/issues specifically are you seeing?
Also just added cocoa pods for this project. Not sure if that helps in this scenario.

no problems left. i was just adding complementary infos to make it work.


How to disable turbolinks in Rails 5?

I need to disable turbolinks in Rails 5 in a specific page.  Adding the fowling in the div tag “data-no-turbolink” is not working.
Any suggestions?

The tag for disabling Turbolinks has changed in Turbolinks 5.

Thanks @nateberkopec
I tried adding <%= link_to episode.title, episode, :"data-no-turbolink" => false %> but on show view my script for my video is not loading.
Am I missing something else?

Change "data-no-turbolink" to "data-turbolinks": <%= link_to episode.title, episode, :"data-turbolinks" => false %> (or <%= link_to episode.title, episode, data: { turbolinks: false } %> ).

Thanks @javan,
That was really helpful. Any idea how to the   data: { turbolinks: false } to a loop?
<%= link_to episode do  data: { turbolinks: false }%>

Thanks for the help @javan 👍


HTTPS in capital showing error

When HTTPS in capital is entered it is showing error.

Whereas in browser

If it is then changed to "https" then url editText error is not set to null.

Should I go for it?

I wouldn't have thought anyone would have typed HTTPS, but seems reasonable to force-lower-case the server URL when they enter it; it's a quick change and not likely to cause problems.


Add Resolve Comment feature

Started on #1056, if anyone wants to take it over just go right ahead
New menu item on comments:

Collapsed comment when matched "Comment resolved by @username":

Expanded:

Closed #1056.

Things to do:

Turn "resolve" button into "unresolve" button when comment is already resolved
"edited by" could be replaced with "resolved by"
I'm not sure if there are any cases where the user can't edit the comment by the actions dropdown exists
This only works with thread comments, not code comments
I don't watch the dom properly for updates, navigating around sometimes creates additional "resolve" buttons to be added

Looking forward to seeing this!
It should probably also work when there's no dropdown

Sorry, I really want to get back to this. I just have a lot of stuff to work on right now

This PR is running late!

@ramlmn Your comment is not contributing anything to the discussion.

Your comment is not contributing anything to the discussion ;)

@sindresorhus Just random ping to get the discussion going!

@ramlmn In the past month, I've moved from Australia to the US, been sick for two weeks, had to go to the emergency room because I was having trouble breathing, and I started a new job. I'll get back to this PR when I have a chance, it's not hurting anyone sitting around in the meantime while I have more important things going on.

@jamiebuilds Wish you a full recovery. We can wait for you to your personal things "Resolved" first!

Apologies if I was being rude! All I wanted to know was the status of this PR as it has been more than a month (now we know why).
And as @yakov116 mentioned, this can wait till your personal things get "Resolved"!

Boom! https://blog.github.com/2018-04-18-new-tools-for-open-source-maintainers/


Added simple test for CL_DEVICE_PRINTF_BUFFER_SIZE.

Maybe this would be better structured as a test rather than as part of the init code?

I debated this as it is such a small test, wasn't sure if it warranted it or not. I'll spin it out to its own test case to make it more explicit - especially in the failure case.


Azure-keystore credentials not set

Hi,
It seems that the azure-keystore credentials are not set even though the 'elasticsearch-keystore add' commands have executed successfully.
[16042018-09:46:57] [configure_elasticsearch_yaml] Configure storage for Azure Cloud in keystore

I think this is caused by a missing call to elastic-keystore create before the add commands.
https://github.com/elastic/azure-marketplace/blob/acd6ae214bb4401fc4d52980a3522660c06a9eac/src/scripts/elasticsearch-ubuntu-install.sh#L704
https://www.elastic.co/guide/en/elasticsearch/reference/current/secure-settings.html

Hey @okke-formsma, I just ran a deployment to test this. The values were inserted into the elasticsearch-keystore correctly
root@data-0:/# /usr/share/elasticsearch/bin/elasticsearch-keystore list
azure.client.default.account
azure.client.default.key
bootstrap.password
keystore.seed

However, when trying to create a repository immediately, the following error is returned:
PUT _snapshot/backup_1
{
    "type": "azure"
}

----

{
  "error": {
    "root_cause": [
      {
        "type": "repository_verification_exception",
        "reason": "[backup_1] path  is not accessible on master node"
      }
    ],
    "type": "repository_verification_exception",
    "reason": "[backup_1] path  is not accessible on master node",
    "caused_by": {
      "type": "i_o_exception",
      "reason": "Can not write blob master.dat-temp",
      "caused_by": {
        "type": "storage_exception",
        "reason": "storage_exception: The specified container does not exist."
      }
    }
  },
  "status": 500
}

According to the azure-repository documentation, the container needs to be created in the storage account before creating a repository. The default container name is elasticsearch-snapshots. Once the container is created though, the repository creation request succeeds. Creating with Azure PowerShell
Set-AzureRmCurrentStorageAccount -ResourceGroupName "<resource group>" -Name "<storage account name>"

New-AzureStorageContainer -Name "elasticsearch-snapshots"

then
PUT _snapshot/backup_1
{
    "type": "azure"
}

----

{
  "acknowledged": true
}

and
PUT /_snapshot/backup_1/snapshot_1?wait_for_completion=true

----

{
  "snapshot": {
    "snapshot": "snapshot_1",
    "uuid": "hv5cp2e2TAqvC9RV29RsKQ",
    "version_id": 6020399,
    "version": "6.2.3",
    "indices": [
      ".watcher-history-7-2018.04.18",
      ".triggered_watches",
      ".security-6",
      ".monitoring-kibana-6-2018.04.18",
      ".monitoring-es-6-2018.04.18",
      ".watches",
      ".monitoring-alerts-6"
    ],
    "include_global_state": true,
    "state": "SUCCESS",
    "start_time": "2018-04-18T04:51:49.107Z",
    "start_time_in_millis":<PHONE_NUMBER>107,
    "end_time": "2018-04-18T04:51:51.516Z",
    "end_time_in_millis":<PHONE_NUMBER>516,
    "duration_in_millis": 2409,
    "failures": [],
    "shards": {
      "total": 7,
      "failed": 0,
      "successful": 7
    }
  }
}

Would you be able to try creating the container in the storage account, to see if this addresses the issue?

@russcam Thanks for investigating. I'll try a new deployment to investigate later today.

thanks @okke-formsma - you'll need to target master branch (commit e89baf2bd4fe9f8978aadcbae3eb3776435c598c) as it contains a fix for the oracle-java8-installer apt package

@russcam I ran with the latest commit but still have the same issue.
When is the elasticsearch-keystore create command executed? I can't find it in the azure-marketplace code anywhere. All manual commands I run require a keystore to be present before running 'add' commands.
Maybe the x-pack or kibana plugins create the keystore in your case? I'm running with those turned off.
root@el6stdata-0:/var/log# grep -R configure_elasticsearch_yaml .
./arm-install.log:[18042018-13:02:22] [configure_elasticsearch_yaml] Update configuration with data path list of /datadisks/disk1/elasticsearch/data
./arm-install.log:[18042018-13:02:22] [configure_elasticsearch_yaml] Update configuration with hosts configuration of ["el6stdata-0:9300"]
./arm-install.log:[18042018-13:02:22] [configure_elasticsearch_yaml] Configure master/client/data node type flags only master-0 only data-0
./arm-install.log:[18042018-13:02:22] [configure_elasticsearch_yaml] Configure node as master and data
./arm-install.log:[18042018-13:02:22] [configure_elasticsearch_yaml] Configure storage for Azure Cloud in keystore
./arm-install.log:[18042018-13:02:26] [configure_elasticsearch_yaml] Setting bootstrap.memory_lock: true

root@el6stdata-0:/etc/elasticsearch# echo "test" | /usr/share/elasticsearch/bin/elasticsearch-keystore add azure.client.default.account -xf
ERROR: Elasticsearch keystore not found. Use 'create' command to create one.

root@el6stdata-0:/etc/elasticsearch# ls /etc/elasticsearch/
elasticsearch.yml  elasticsearch.yml.bak  jvm.options  log4j2.properties

Actually we run a slightly modified version which installs 6.2.3 (6.2.2 has some issues with restoring backups from azure)

We do not install xpack or kibana.
Configuration template follows below, we install using the following command;
az group deployment create --resource-group staging-elastic6 --name staging-elastic6 --template-file ./azure-marketplace/src/mainTemplate.json<EMAIL_ADDRESS>
{
  "artifactsBaseUrl": {"value": "https://raw.githubusercontent.com/elastic/azure-marketplace/6.2.2/src"},

  "esVersion": {"value": "6.2.3"},
  "adminUsername": {"value": "snip"},
  "adminPassword": {"value": "snip"},
  "securityAdminPassword": {"value": "snip"},
  "securityReadPassword": {"value": "snip"},
  "securityKibanaPassword": {"value": "snip"},
  "securityBootstrapPassword": {"value": "snip"},
  "securityLogstashPassword": {"value": "snip"},

  "esClusterName": {"value": "staging-elastic6"},
  "vmHostNamePrefix": {"value": "el6st"},
  "vmSizeDataNodes": {"value": "Standard_A2"},
  "vmDataDiskCount": {"value": 1},
  "vmDataDiskSize": {"value": "Medium"},
  "vmDataNodeCount": {"value": 1},
  "dataNodesAreMasterEligible": {"value": "Yes"},

  "vNetNewOrExisting": {"value": "existing"},
  "vNetName": {"value": "snip"},
  "vNetExistingResourceGroup": {"value": "snip"},
  "vNetClusterSubnetName": {"value": "STAGING-ELASTIC6"},
  "vNetLoadBalancerIp": {"value": "snip"},
  "loadBalancerType": {"value": "internal"},

  "xpackPlugins": {"value": "No"},
  "kibana": {"value": "No"},
  "azureCloudPlugin": {"value": "Yes"},
  "azureCloudStorageAccountName": {"value": "snip"},
  "azureCloudStorageAccountKey": {"value": "snip"}
}

Running a test without installing X-Pack plugin now

Can replicate - will push a fix now

@okke-formsma I've opened #184. Would you mind trying it from the fix/create-keystore branch?

@russcam Thank you for fixing, I have tested and it works perfectly now.


FOXG1 syndrome/Disorder; Rett syndrome, congenital variant [Merge]

Mondo term (ID and Label)
MONDO:0035383 FOXG1 syndrome
Reason for deprecation
GeneReviews uses FOXG1 syndrome as their preferred name for this chapter: https://www.ncbi.nlm.nih.gov/books/NBK604176/  that points to OMIM 613454	RETT SYNDROME, CONGENITAL VARIANT (See Table B for OMIM entries)
But I see there is also FOXG1 disorder (MONDO:0100040) that is parent to Rhett syndrome, congenital variant (MONDO:0013270), so I don't know if that is a conceptual match to FOXG1 syndrome?
If UMLS is correct, then Orphanet 561854 and OMIM 613454 are the same concept on CUI C3150705.
Term to be merged with
???
Not sure, but I would seem that there it at least one too many Mondo concepts here among this trio of entries. Or maybe it's not a merge but a restructuring of FOXG1 syndrome under the Disorder term?
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Dear @kanems,
Thank you for reaching out to the Mondo Disease Ontology, part of the Monarch Initiative!
We have received your ticket and will review it as soon as possible.
If you have any additional information or updates relevant to your inquiry, please add them in a comment on this ticket. Your input is valuable in helping us understand and resolve the issue effectively.
Thank you for your patience and for your contributions to Mondo!
Sincerely,
The Mondo Team

@kanems Thank you for this issue!
There seem to be a lot (potentially 4) overlapping terms in Mondo related to FOXG1 disorder/syndrome:
MONDO:0035383 'FOXG1 syndrome'
MONDO:0100040 'FOXG1 disorder'  that has children:

MONDO:0035713 'FOXG1 syndrome due to intragenic alteration'
MONDO:0013270 'Rett syndrome, congenital variant'

I will bring this issue up in the medical experts call next week to inquire which term(s) should be merged together.

Discussed in domain experts call on 2024-11-19:
Recommendation is to merge terms:
MONDO:0035383 'FOXG1 syndrome'
MONDO:0100040 'FOXG1 disorder'
MONDO:0035713 'FOXG1 syndrome due to intragenic alteration'
MONDO:0013270 'Rett syndrome, congenital variant'
Notes:

MONDO:0016833 '14q12 microdeletion syndrome' is a distinct syndrome and should not be merged with above.
@ahamosh recommends that the name of any disease should not be 'GENE syndrome' because tomorrow there might be another variant found in that gene that is related to a completely different disease. Instead the name  should related to the phenotypic features of the disease, for example, FOXG1-related encephalopathy.


Fixed #1303 - API to remove stored session cookies & auth tokens

Ported iOS implementation: https://github.com/couchbase/couchbase-lite-ios/pull/1330

Hi @pasin,
Thank you for reviewing PR. I applied your feedbacks. Could you please re-review PR?
Thanks!


Update igls.py

correct a call for the table in def start()

Thanks for this fix.
Are you able to get IG live stream now ?

Hi.
It's my pleasure. Your igls module is very usefull for python STREAM client.
I've never had this pb in the past because I always use 'silent=False' when creating a Table ... so I never catch this code error, before reading the module code deeply.
This morning, I've try 'silent=true' when creating a table, and do a 'client.start(myTable)' just after :
The start function is called, the with statement is done but IG never start the stream.
I've try with the epic price Offer/Bid for several epic such as EURUSD or DAX or GBPUSD and no-one start the stream.
If I create my table with 'silent=false' (as default) then no pb the stream start at the next tick ...
So there is still a pb but I'm not aware of LS technology so where ? I don't know.
Florian
----- Mail original -----
De: "femtotrader"<EMAIL_ADDRESS>À: "femtotrader/ig-markets-stream-api-python-library"<EMAIL_ADDRESS>Cc: "falex69"<EMAIL_ADDRESS>Envoyé: Vendredi 29 Mai 2015 09:16:55
Objet: Re: [ig-markets-stream-api-python-library] Update igls.py (#1)
Thanks for this fix.
Are you able to get IG live stream now ?
—
Reply to this email directly or view it on GitHub .

Thanks Florian but igls.py is not my work but
https://github.com/dw/py-lightstreamer/blob/master/lightstreamer.py
Unfortunately I don't know LS technology enough to help
I think we should make a more object orient approach for this IG stream API library.
Like I previously did for REST API https://github.com/femtotrader/ig-markets-rest-api-python-library/
Any idea about API design ?
Femto

Ok
Will you report the "correction" to the source or do I ?
No, when I've got question about the API I write a message to the IG community at http://labs.ig.com
----- Mail original -----
De: "femtotrader"<EMAIL_ADDRESS>À: "femtotrader/ig-markets-stream-api-python-library"<EMAIL_ADDRESS>Cc: "falex69"<EMAIL_ADDRESS>Envoyé: Vendredi 29 Mai 2015 10:53:12
Objet: Re: [ig-markets-stream-api-python-library] Update igls.py (#1)
Thanks Florian but igls.py is not my work but
https://github.com/dw/py-lightstreamer/blob/master/lightstreamer.py
Unfortunately I don't know LS technology enough to help
I think we should make a more object orient approach for this IG stream API library.
Like I previously did for REST API https://github.com/femtotrader/ig-markets-rest-api-python-library/
Any idea about API design ?
Femto
—
Reply to this email directly or view it on GitHub .

It was ever fixed upstream. I don't know why my version was buggy.


SVG files are out of date

A lot of the png files have been updated to Android N versions, but the SVG versions still display the old (eg. non-human) versions

This is a duplicate of #30


Update Atoms info from calculated results

Initial work aiming to address part of #206:

Ensure outputs from ase.io.write are consistent with singlepoint (i.e. results in info, they may or may not also be saved in calc.results)

Originally, I thought this would be implemented as a custom write function that also moved results to info, then called ase.io.write, but we want this even if we're not writing to file.
There's probably a better name than output_atoms, but the idea is a generalised version of what was previously done in single_point - move Atoms.calc.results to Atoms.info or Atoms.arrays, labelled with the MLIP architecture, and optionally delete the Atoms.calc.results value in the process.
To do:

[x] More testing (possibly directly on the utils function) to ensure this behaves as expected, but so far seems to work with minimal changes
[x] Extend to other calculations (currently only done for single_point and geom_opt, to give an idea of the changes required)
[x] Update all CLIs (e.g. dealing with invalidate_calc, either as new option or via write_kwargs, since we don't return an Atoms object)

Following discussions, recommend developing separate script to automate generation of different MLIP records for database. That way files will only be read/written once and extra data appended to info object as it goes.
In terms of that, current implementation is fine provided non-arch key is also added so ASE automatically recognises the last run's parameters (this may want to be disabled in aforementioned script) as well as the tagged key.
Also suggest adding arch to the info block under the same grounds.

With the last set of changes, this now:

Includes an "arch" info label with the same tag as the labelled properties ([arch]_energy) etc.
Defaults to both including both the info and original calc.results dicts

I don't especially like this, but losing the ability to use built in functions like Atoms.get_stress etc. by default seems really quite inconvenient at the moment

The current default of copying results to the info dict is still not obviously the right choice for me personally, but I think @alinelena disagrees.

Relatively minor change to swap unpacking write_kwargs when it's passed to output_structs to passing it as the dict, as otherwise there could be keyword clashes e.g. from write_results.


shared dataset cannot be access

Hi
i would like run experiments with the shared data. The box link in below is not valid anymore.
https://ibm.box.com/v/titan-dataset
Could you reshare the data with us again?
Thank you

Hi
i would like run experiments with the shared data. The box link in below is not valid anymore.
https://ibm.box.com/v/titan-dataset
Could you reshare the data with us again?
Thank you @jannisborn @ndeutschmann

Please recheck the link, it should be functional


Auto merge auto-update-weights-for-dev-581b3e60eca6cdced023aee1ecdb25740f03e7cb to dev

Codecov Report

Merging #829 (ba29b09) into dev (816c460) will increase coverage by 0.00%.
The diff coverage is 94.44%.

@@           Coverage Diff           @@
##              dev     #829   +/-   ##
=======================================
  Coverage   78.24%   78.25%
=======================================
  Files          51       51
  Lines       10371    10375    +4
=======================================
+ Hits         8115     8119    +4
  Misses       2256     2256

Impacted Files
Coverage Δ

pallets/creditcoin/src/weights.rs
92.42% <93.75%> (+0.23%)
:arrow_up:

pallets/difficulty/src/weights.rs
100.00% <100.00%> (ø)

pallets/rewards/src/weights.rs
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


InitializableStruct access control

First of all, thank you for maintaining SideMenu.
I have a question, is it possible to keep InitializableClass internal instead of public?
The problem is in Swift it's impossible to keep extension with protocol conformance internal:

You can’t provide an explicit access-level modifier for an extension if you’re using that extension to add protocol conformance. Instead, the protocol’s own access level is used to provide the default access level for each protocol requirement implementation within the extension.

As result, extension NSObject: InitializableClass {} is extending NSObject with InitializableClass everywhere in a project where SideMenu was imported. It interferes with the custom implementation of func with(_ block: (Self) -> Void) -> Self that pod client may have in their projects. We have such an implementation used in 95% of our files.
Thanks.

Fixed in 6.2.0.


[5.1] Availability to append scheduled command output

Made a few small tweaks to this but thanks :+1:


scrcpy can't run on windows server system

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS:  Windows server 2008R2 ,  Windows server  2012, Windows server  2016
scrcpy version: [ 1.16]
installation method: [Windows release]
device model:
Android version: [Android 7]

Describe the bug
无法启动此程序，因为计算机中丢失 MFPlat.DLL。尝试重新安装该程序以解决此问题。

https://answers.microsoft.com/en-us/windows/forum/apps_windows_10-winapps-appscat_music/windows-10-mfplatdll-missing/5bfdc7fa-7ca4-4f5e-a524-83455d8abb9c

https://answers.microsoft.com/en-us/windows/forum/apps_windows_10-winapps-appscat_music/windows-10-mfplatdll-missing/5bfdc7fa-7ca4-4f5e-a524-83455d8abb9c

It's run ok on windows 10 , but can't run on windows server system

It's run ok on windows 10 , but can't run on windows server system

OK but this is probably a missing Windows update or something. Google MFplat.dll.

OK but this is probably a missing Windows update or something. Google MFplat.dll.


Checkpoint file 'd40a34821081' not provided.

Hello,
Hope you are doing well, I need checkpoint of trained model. Checkpoint not exists in the repo.
Can you provide this ASAP.

If you want to use trained weights, please check this link :
https://drive.google.com/drive/folders/16qM55ZZj43nnuy0JOu-_TKC3FbpShKd8?usp=sharing
You will find the weights for both the DenseNet and FRCNN


Add action to apply updates in /docs to /versioned_docs/version-XYZ

Depends on #665.
With the introduction of having a versioned_docs entry to represent the latest version, we'll have increased overhead of making sure /docs and versioned_docs/version-<latest> ( version-2.7 in this case) are in sync. With this action, when a PR is made against main, the corresponding changes will be applied against versioned_docs/version-<latest> as well and committed to the same PR.

Oh -- we also need to keep the sidebars in sync.

Oh -- we also need to keep the sidebars in sync.

I thought about that case and poked around for a bit, but it wasn't obvious how Docusaurus does the sidebar js -> json conversion. But I don't think this should be a blocker since it's not a regular occurrence that a sidebar affecting change is made  (in which case there would be other things to update) and we'll still need to review such changes anyways.

PR updated.
As discussed, the case where /docs and versioned_docs/version-<XYZ> are both updated is out of scope for this PR. When both directories have updated files:

The same file(s) may be modified in both directories in which case manual review is needed.
Both directories have modified files, but files are only modified in one of the directories. This will can be a future improvement.


Updated authors in tests.

Pull Request Test Coverage Report for Build 527

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

Totals

Change from base Build 526:
0.0%

Covered Lines:
3024

Relevant Lines:
3498

💛  - Coveralls


[ISSUE #4513] elegant equals using apache-commons

What is the purpose of the change
an elegant equals using apache-commons
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #4514 (b29400e) into develop (d5b4d84) will decrease coverage by 0.21%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #4514      +/-   ##
=============================================
- Coverage      48.10%   47.89%   -0.22%
+ Complexity      5121     5092      -29
=============================================
  Files            649      649
  Lines          43026    43021       -5
  Branches        5626     5623       -3
=============================================
- Hits           20699    20606      -93
- Misses         19820    19903      +83
- Partials        2507     2512       +5

Impacted Files
Coverage Δ

...che/rocketmq/common/protocol/route/BrokerData.java
38.09% <100.00%> (+4.05%)
:arrow_up:

...va/org/apache/rocketmq/logging/inner/Appender.java
29.21% <0.00%> (-7.87%)
:arrow_down:

...tmq/namesrv/processor/DefaultRequestProcessor.java
66.04% <0.00%> (-7.48%)
:arrow_down:

...ache/rocketmq/broker/topic/TopicConfigManager.java
54.61% <0.00%> (-4.62%)
:arrow_down:

...org/apache/rocketmq/common/stats/StatsItemSet.java
41.79% <0.00%> (-4.48%)
:arrow_down:

...ache/rocketmq/common/stats/MomentStatsItemSet.java
39.13% <0.00%> (-4.35%)
:arrow_down:

...ketmq/common/protocol/body/ConsumerConnection.java
95.83% <0.00%> (-4.17%)
:arrow_down:

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
68.64% <0.00%> (-2.89%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (-2.57%)
:arrow_down:

...a/org/apache/rocketmq/filter/util/BloomFilter.java
60.43% <0.00%> (-2.20%)
:arrow_down:

... and 12 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 d5b4d84...b29400e. Read the comment docs.

Coverage decreased (-0.03%) to 51.93% when pulling b29400e0ee61999870833b841eaacadcc38997b5 on PansonPanson:develop into d5b4d8431c32e443e2fea3feaed391acecf951eb on apache:develop.

Coverage decreased (-0.03%) to 51.93% when pulling b29400e0ee61999870833b841eaacadcc38997b5 on PansonPanson:develop into d5b4d8431c32e443e2fea3feaed391acecf951eb on apache:develop.

Linked issue: #4513


Audi or clock

I’m hearing audible clicks if amp envelopes are turned up. Going to try to get a screen of current settings but if you have a basic patch with DCO1 and open the envelope all the way, then turn up the release and play Middle C, you can hear clicks when the oscilloscope is just about to stop vibrating.
IPad Air 1st gen latest iOS

I'm sorry that you're having a click with AudioKit Synth One, but this is the repo for AudioKIt's Analogy Synth X, a completely different app.  The code for AudioKit Synth One will be made public in early July.


remember to update submodule when creating next release

This was completed for knb-lter-nes.16.3 release. Leave open for next version?


follow-up on docs (representation/layout of keys)

I got two more ideas i could change, but i want to wait until after this pr is accepted.

in stead of using bullets in front of key/shortcut examples, I want to use a table, to match the appearance in the UI
I used "quotes" to show keyboard output (in contrast to kbd for physical input) but now that I think about it, this output is data just as well, so I think it would be better if I used data.

Originally posted by @Jay-o-Way in https://github.com/microsoft/PowerToys/issues/11793#issuecomment-908547517

@niels9001 The image in https://docs.microsoft.com/en-us/windows/powertoys/keyboard-manager#remap-shortcuts has text that is the opposite of what the text is trying to say. This is also true for the text in the actual UI. When/how can we update?


Feature/add components

Updated some components and added new components.

Codecov Report

Merging #16 (8d5916c) into develop (a9b1c9f) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #16      +/-   ##
===========================================
+ Coverage    99.70%   99.74%   +0.03%
===========================================
  Files           17       19       +2
  Lines         1379     1565     +186
  Branches        59       59
===========================================
+ Hits          1375     1561     +186
  Misses           4        4

Impacted Files
Coverage Δ

src/components/AppBars/AppBar.tsx
100.00% <100.00%> (ø)

src/components/Icons/Icon.tsx
100.00% <100.00%> (ø)

src/components/UI/Buttons/Button.tsx
100.00% <100.00%> (ø)

src/components/UI/Buttons/IconButton.tsx
100.00% <100.00%> (ø)

src/components/UI/Textboxes/Textbox.tsx
100.00% <100.00%> (ø)

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


Fix converter

What does this PR do?
This PR brings fixes to the converter component. Fixes included are:
Setting col config from core config

Previously, the logic to set col config from core config would look for exporter datadog in the exporter list. This is incorrect in two ways: the name is not necessarely datadog (can be datadog/anything), and there can be multiple datadog exporters. Now the updated logic checks for prefix datadog, and will also apply the logic if there are multiple datadog exporters.
Previously, the logic would not be applied for site and or key if those sections were there but the value was unset. If the sections are unset, this is reason more to apply the settings from the core agent. Now the updated logic will add the site and key from core configs even if the section is not present in the config.

Prometheus
Previously, the prometheus logic would check if prometheus receiver was in same pipeline as DD exporter. If so, it wouldn't add it. It would exit out at first DD exporter. Now, this logic gets added to all DD exporters.
Motivation
Describe how to test/QA your changes
API Key
Unset:
Set API key in agent config:
api_key:12345

but not in col config:
exporters:
  datadog:
    api:
      key:

Ensure API Key is added
Missing section:
Set API key in agent config:
api_key:12345

but not in col config:
exporters:
  datadog:

Ensure API Key is added
Empty string:
Set API key in agent config:
api_key:12345

but not in col config:
exporters:
  datadog:
    api:
      key: ""

Ensure API Key is added
Site
Do the same as above but with site
Overrides
Set api key and site in agent config, and set api key and site in col config to different values. Ensure the col config is not overriden.
Prometheus
Configure collector with multiple DD exporters and prometheus receiver, and ensure each DD exporter is in a pipeline with DD exporter :
    prometheus/user-defined:
      config:
        scrape_configs:
          - job_name: 'datadog-agent'
            scrape_interval: 10s
            static_configs:
              - targets: ['<IP_ADDRESS>:8888']

exporters:
    datadog:
      api:
        key: 12345
    datadog/2:
      api:
        key: 12345

        metrics:
            receivers: [nop, prometheus/user-defined]
            processors: [infraattributes/user-defined]
            exporters: [datadog]
        metrics/2:
            receivers: [prometheus/user-defined]
            processors: [infraattributes/user-defined]
            exporters: [datadog/2]

Ensure that no additional prom receivers/ pipelines are added to your config.
In order to validate the used config, you can go to localhost:7777 and look for the enhanced config.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


[Docker] Image takes ages to build

https://github.com/inventree/InvenTree/pull/6614 has broken docker image push
builds are stuck on "building wheel for grpcio"
Related issue: https://github.com/inventree/InvenTree/issues/6291

Edit: Docker push is not broken, but it does take a significant amount of time just to build this one package

After much research, this seems to be related specifically to building the alpine image on ARM.
The downloads page shows lots of other precompiled wheel options but not the ones we want.
References

https://github.com/grpc/grpc/issues/27940
https://github.com/grpc/grpc/issues/34998
https://github.com/grpc/grpc/issues/32044
https://github.com/grpc/grpc/issues/27512
https://github.com/grpc/grpc/issues/27512
https://github.com/home-assistant/core/issues/56669
https://github.com/home-assistant/core/issues/56669
https://github.com/home-assistant/core/issues/40148

Potential Solutions
A) Use pre-build grpcio library
Alpine ships the py3-grpcio package which comes with the 1.54.2  version of the library pre-installed. This may sidestep our need to build the library entirely.
B) Different Base Image
Alpine / musl is a big part of the "problem" here - using a different base image would make this a much more tractable problem.
C) Compile Wheel
Can we compile a wheel for grpcio ourselves for the aarch64 architecture?
D) Consider Default Support for grpcio
Currently we use this for tracing. I do not want to split up dependencies, this is not my ideal solution. But including it here as a point of consideration.

@matmair @wolflu05 this single library (grpcio) is the cause for our very long docker build times on GitHub Actions.

This was the main reason I implemented the matrix build as it seems there are problems with that solution I think c) would be the next best

I think for a optimal setup, we need to:

use matrix builds and merge the manifest so that we don't have the issue we recently had. See https://docs.docker.com/build/ci/github-actions/multi-platform/
build our own wheel and cache that
use the free m1 runners for arm building

regarding 3. - I tried to get docker running on mac and it seems to be difficult. To get the M1 runner we should run MacOS 14 which seems to have problems with colima.
Also see https://github.com/actions/runner-images/issues/17

Arm runners + docker seems to be a problem https://github.com/douglascamata/setup-docker-macos-action?tab=readme-ov-file#arm64-processors-m1-m2-m3-series-used-on-macos-14-images-are-unsupported; which kind of defeats the purpose

I think compiling and caching our own wheel would go most of the way and certainly be a drastic improvement to our build times. That should be our priority

Anyone have experience doing something like this in GitHub?

Ok, thanks for referencing these. I now understand the trouble you mean. Very said that there are no Linux arm runners for free available.

I think compiling and caching our own wheel would go most of the way and certainly be a drastic improvement to our build times. That should be our priority

Looking into it. Seems simpler than getting docker to run

Maybe we can use piwheels too - they are already building wheels https://www.piwheels.org/project/grpcio/

@matmair isn't that for armv7 only?

@SchrodingersGat I think this was thourougly solved by https://github.com/inventree/InvenTree/pull/8680 - can we close this?

This was partially solved. The following issues remain:

Cannot build the devcontainer (or docker image in general) on Apple M1 (hangs on grpcio)
Cannot run playwright tests in the devcontainer (Alpine incompatibility)


Correct newline issues

Correct the missing newline issues discussed in #60 by changing a few -}} (whitespace-stripping template tags) to }} (no stripping). I don’t believe this has any negative consequences but I’m new to helm so I’m certainly open to criticism!

Thanks for contributing! That's great fix.


Multiple validate not working

I've the following schema
validate(project_success?: [:time_table]) do |time_table|
  time_table.loan.project.success?
end

validate(payment_date_is_passed?: [:time_table]) do |time_table|
  Date.today >= time_table.payment_date
end

It seems that only the last validate is executed, and the first one is never executed.
Is that a normal behavior ?

@nicolas-besnard  this one has been fixed in this PR https://github.com/dry-rb/dry-validation/pull/324 cc @solnic