Add double-quote-string-fixer to pre-commit-config

Rationale / Begründung
I use double quotes in my projects nowadays, because black convinced me. It is hard to switch back mentally. This project uses mostly single quotes, but not consistently. There is hook called double-quote-string-fixer that replaces double quoted strings with single quoted strings. Should we add this to the pre-commit-config? That way, it gets fixed automatically.
What do you think?
Affected
devs
References / Verweise

https://pre-commit.com/hooks.html

we had some discussions about this on the hackathon 🤣
see: https://rdmo.slack.com/archives/C050KFAQ79D/p1691675566585859
For me, the single quotes also look a bit weird
but when Guido uses single quotes, then I'm also convinced

Fine, I see. So it is consensus to use single quotes.
Should I add the hooks to enforce them (on me)?

I am totally fine when other people use double quotes. I would say there is no need to enforce them.

Alright, my argument was for consistency in the project. No pre-commit-hook then, that is fine for me.


Docker checkpoint command not found

Hi,
Im using Ubuntu 16.04, Docker 1.12 and CRIU 2.10.
I am able to perform checkpoint/restart with native criu. But I cannot do the same with docker.
root@ubuntu:~# docker checkpoint --help
docker: 'checkpoint' is not a docker command.
See 'docker --help'.
Please find the complete details below.
root@ubuntu:~# docker info
Containers: 1
Running: 0
Paused: 0
Stopped: 1
Images: 1
Server Version: 1.12.6
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 3
Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: null bridge host overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.35
Operating System: Ubuntu 16.04.2 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 1.945 GiB
Name: ubuntu
ID: CL5I:NEY7:VCHY:YCX6:4A76:EKYR:2V36:2FLH:OOXA:BFG2:JODH:YGMR
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
<IP_ADDRESS>/8
root@ubuntu:~# docker -v
Docker version 1.12.6, build 78d1802, experimental
root@ubuntu:~# criu --version
Version: 2.10
GitID: v2.10-32-g65214e4
root@ubuntu:~# uname -a
Linux ubuntu 4.4.35 #1 SMP Sun Feb 12 22:03:51 PST 2017 x86_64 x86_64 x86_64 GNU/Linux

From code level checking,  the "checkpoint" command comes after version v1.13.0.
Also, give command information like applying in which version might helps in this regard, say in page like:
https://docs.docker.com/engine/reference/commandline/checkpoint/.

Thanks. I was able to upgrade to 1.13.
Also, I updated the following config file to enable experimental.
/etc/init/docker.conf
exec "$DOCKERD" $DOCKER_OPTS --raw-logs -H fd:// --experimental=true
However I got the following message.
root@ubuntu:~# docker checkpoint
only supported with experimental daemon
Am I missing something?

The /etc/init/docker.conf file is not used by the official binaries (and not used if your system is using systemd, which includes ubuntu 16.04)
The recommended way to set daemon options is to use a daemon.json configuration file, see https://docs.docker.com/engine/reference/commandline/dockerd/#/linux-configuration-file
To enable experimental options, create the configuration file /etc/docker/daemon.json, if it doesn't exist, and set the experimental option; if this is the only option you're setting, it should look like this;
{
  "experimental": true
}

after saving the file, restart the docker service (sudo systemctl restart docker)

@thaJeztah a bit unrelated, but: If I try to configure -H in daemon.json, I get

unable to configure the Docker daemon with file /etc/docker/daemon.json: the following directives are specified both as a flag and in the configuration file: hosts: (from flag: [fd://], from file: [unix:///var/run/docker.sock <IP_ADDRESS>:2376 <IP_ADDRESS>:2376])

because /lib/systemd/system/docker.service sets -H: ExecStart=/usr/bin/dockerd -H fd://
How do I set -H in daemon.json?

@albers that's a known issue; the daemon.json does not allow you to set an option that's already been set as a flag. We're still considering options ("override" when set through daemon.json, "merge" options if set on both).
The fd:// option is used for socket-activation, and can only be set through systemd. For now, the best way to add an additional -H, is to use a systemd drop-in file (see https://docs.docker.com/engine/admin/systemd/#custom-docker-daemon-options), for example;
create a directory for your drop-in files;
mkdir -p /etc/systemd/system/docker.service.d

Create a drop-in file (e.g.  /etc/systemd/system/docker.service.d/docker-host.conf);
[Service]
ExecStart=
ExecStart=/usr/bin/dockerd -H fd:// -H tcp://<IP_ADDRESS>:2376 -H <IP_ADDRESS>:2376

Then reload the systemd configuration, and restart docker;
sudo systemctl daemon-reload
sudo systemctl restart docker

All other options can be set through daemon.json, because they're not set as a flag.

@thaJeztah Thanks very much for explaining. I created a PR that adds this information to the daemon.json docs.
Should fd:// be used instead of unix:///var/run/docker.sock or is it a different thing (not sure I understand the term "socket-activation" properly)?

Should fd:// be used instead of unix:///var/run/docker.sock or is it a different thing  (not sure I understand the term "socket-activation" properly)?

Socket activation is a systemd feature; it allows you to automatically start the docker service (daemon) when you connect to the /var/run/docker.sock socket (there's actually a separate service / systemd-unit that handles that, see docker.socket). When connecting to that socket, systemd detects that you're trying to connect, starts the dockerd daemon, and forwards the socket to the daemon. The fd:// socket is the socket connection that's managed by systemd. Given that it's a systemd-specific option, it's not possible to define fd:// in the daemon.json, and that's the reason it has to be added in the systemd unit-file. (hope I have all the terms correct, but that's the general working of the feature)
On distros that use RPM as a package manager there were some differences in the mechanics, and socket activation resulted in the "Live Restore" feature to not work. For that reason, socket activation is not used on those platforms.

@aarthipa I think your question should be answered with the comments in this issue, so I'll go ahead and close this issue, but let us know if you still have problems to enable the experimental features.

Thanks @thaJeztah!

Thanks @thaJeztah. Its works. The explanations were great.


Paper question

Can you briefly explain to me these parameters of Non local attention mechanism in terms of the information it gives, i mean what is information obtained by Phi , G and Théta .

Thank you .

Can you briefly explain to me these parameters of Non local attention mechanism in terms of the information it gives, i mean what is information obtained by Phi , G and Théta .  Thank you .

I drew a diagram that might make the process easier to understand.

First perform a 1×1 convolution with learnable weights on the features X and X_{NP} respectively, and then expand the obtained tensor into one column (vectorization), that is, the results obtained by Phi , G and Théta.

But why we need to learn   weights on the features X and Xnp and my second question in wich part you did correlation between channel-wise , because you mentionned in your paper you used spatial and channel correlation to highlight important features.

also sir cross correlation is calculated between query and key not query and values as you mentionned above no ?

But why we need to learn weights on the features X and Xnp and my second question in wich part you did correlation between channel-wise , because you mentionned in your paper you used spatial and channel correlation to highlight important features.

Hi. The cross-correlation between features X and Xnp  represents the correlation between the feature at each spatial location of the feature map and the features of other locations (including its multi-scale neighborhood fusion features). The schematic above shows only the two-dimensional case, in fact it should be an expansion of a three-dimensional tensor. So the correlation is the correlation of each space and channel location. In the upper left paragraph of the fifth page of the paper, we mentioned that according to the commutative law of multiplication, we first calculate the last two items instead of Q×K in self attention. That is, (QK)V=Q(KV) in this case (using the dot-product form).

But why we need to learn weights on the features X and Xnp and my second question in wich part you did correlation between channel-wise , because you mentionned in your paper you used spatial and channel correlation to highlight important features.

unfold like this

in your paper you u calcuated cross correlation between K and Query and then you multiply by V

 in your paper you u calcuated cross correlation between K and Query and then you multiply by V

 in your paper you u calcuated cross correlation between K and Query and then you multiply by V

Thank you. I am sorry for the inconsistency with the illustration. In the illustration, we have drawn it in the order of self attention. However, in the equations in the subsequent part of the paper and in our codes, it has explained that the last two items are calculated first and then the former item is calculated.
See this part of the code:

In fact, the final calculation result of the two schemes is the same because it conforms to the multiplicative associative law and does not affect the performance.


heroic: add icu to FHS env

Description of changes
Add package icu to the Heroic FHS env wrapper.
Fixes #327153.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Diff looks OK, but why?

Sorry, forgot to mention the issue in the description.  Just edited to add that.

Superseded by #335737


[Loc] emergency update to langpack source files 2-26-2024

Updates the version number to the proper value and also includes strings that were checked in from the last langpack (non new).
Will be port requested soon.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 41.757%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30827

Relevant Lines:
69082

💛  - Coveralls


Putting objects into the context

Any chance to enhance the tags to accept objects in the context?
Something like (similar to your introductionary example):
class User {
    final String name;
    final String email;
    final List<String> roles;

    // constructor
}

List<User> users = List();
// add some users
context.variables['users'] = users;
final template = Template.parse(context, Source.fromString(raw));
print(template.render(context));

This would greatly simplify my code.

To answer my own question: Flutter has no such concept as for example java has with reflection. The reason is that the compiler wants to strip all unnecessary code and this is not possible if code can be accessed indirectly.
So it is imho not possible to handle an object to a library in order to access the properties of the object "magically".
But there is an easy solution: You may already using json_serializable. If not take a look at it. It produces code to convert objects from and to json. In the above case the code for sending the users to context would be something similar to:
context.variables['users'] = users.map((e) => e.toJson()).toList();

There are some advantages too with json_serializable:

You could add some more properties in the map() method
You could modify properties for example to format the properties for user-friendly output
You can easily clone objects (e.toJson() -> fromJson() )
Transfer the objects to/from servers via json

The latter two are not related to liquid_dart.
I close this issue now.


Bump apko dep, drop proot requirement

apko v0.7.0 dropped the need for apk to be installed, and instead manages apks with a pure Go implementation 🎉
melange can pick up this updated dep and no longer need proot to protect its apk invocations
https://github.com/chainguard-dev/melange/pull/305 was my previous attempt, but there were some fs dep changes in 0.7.0 and it sounded like @deitch might have a better idea how to handle them.
Opening this so we can track this work and so we don't forget.

Done!


xilem_web: Apply props when the element type changes in replace_inner

Fixes #518

(Attributes such as src in <iframe src=".."> were not applied when changing the inner element (e.g. <div> -> <iframe>))
Thanks

When I say the context here, I meant of why the change fixes that. But thanks, yeah

Oh, ok, yeah it's basically, that the ElementProps are a virtual DOM version of the actual DOM representation and get updated in the modifier views. To actually change the DOM, DomNode::apply_props has to be run. This didn't happen when changing the inner element of an AnyView. The attributes are not applied in the type-erased version of the Pod (AnyPod), as... well the type of the props is erased, this happens before upcasting, or generally when erasing the type of the element.


In site script, how to detect if it is open in Beaker?

You can check if the DatArchive API is available

Not exact detecting if it is in Beaker. For example, if it is a NodeJS app, it will say DatArchive is available?
Althought, if it shows DatArchive API as available, but isn't in a app, considers it as Beaker.

Hi Daniell,
You should always check for APIs/features and not for specific browsers.
Same as when developing a regular web page.


Update Composer dependencies (2023-02-21-00-10)

Visual regression test passed!
View the visual regression test report


PR on RustCrptyo Blocker

Some pending PR is blocking progress on Decryption pipeline. Requesting @laudiacay for comment.

waiting on https://github.com/RustCrypto/traits/pull/1189


allow missing optional params

Fix #129
When a parameter is marked as required: false, sway should not throw an exception if it is missing.
Except for parameters that are in: path, those are still always required.

Thanks @noahdietz!


Error executing 's2i build' at Windows. Looks like newline \r is inserted into filename

When using version source-to-image-v1.1.7-226afa1-windows-amd64 at Window 7 then I get an error which looks like newline is not handled probably.
An extra \r is inserted into the filename.
Is this a known bug?
s2i build http://bitbucket-devtools.<IP_ADDRESS>.nip.io/scm/dem/demo-jenkins.git openshift/jenkins-2-centos7 demo-jenkins
Your branch is up-to-date with 'origin/master'.
---> Copying repository files ...
---> Installing Jenkins 7 plugins using /opt/openshift/plugins.txt ...
Creating initial locks...
Locking maven:2.17
Locking
Locking blueocean:1.2.4
Locking
mkdir: cannot create directory '/opt/openshift/plugins/\r.lock': File exists
Failed to install plugins.
Build failed
ERROR: An error occurred: non-zero (13) exit code from openshift/jenkins-2-centos7

A work-a-round is to remove all empty lines from the plugins.txt however after that the download fails as an extra \r is inserted in the download filename:
Downloading plugin: maven from https://updates.jenkins-ci.org/download/plugins/maven/2.17
/maven.hpi
Really looks like the plugs.txt have to have unix line-ending.

Yeah this doesn't sound like an s2i bug, s2i is just sending the file as it exists on disk (which it got from a git clone).
Where did you get your plugins.txt from?

I wrote the plugins.txt in Eclipse running at Windows.

I wrote the plugins.txt in Eclipse running at Windows.

well that's why it has windows newlines in it then...  use a tool to strip them, e.g. https://sourceforge.net/projects/dos2unix/
(not an endorsement of any particular tool).

You are right, it must be the script that handles the plugins.txt file here:
https://github.com/openshift/jenkins/tree/master/2/contrib/jenkins/install-plugins.sh
Please close this issue.


{humble,rolling}: zstd-vendor remove mismatched patches

prevent-building-zstd-with-ExternalPr.patch is mismatched in humble and rolling. And build passed without the patch.
The patch can be successfully applied on jazzy and iron.
Failure log:
CmdError('quilt --quiltrc /home/jason/workspace/raspberry/rpi-build/ tmp/work/cortexa72-poky-linux/zstd-vendor/0.15.11-1-r0/recipe-sysroot-native/ etc/quiltrc push', 0, 'stdout: Applying patch 0001-CMakeLists.txt-prevent- building-zstd-with-ExternalPr.patch
patching file CMakeLists.txt
Hunk #1 FAILED at 9.
1 out of 1 hunk FAILED -- rejects in file CMakeLists.txt patching file cmake_minimum_required_2.8.12.patch
patching file no_internal_headers.patch
Patch 0001-CMakeLists.txt-prevent-building-zstd-with-ExternalPr.patch does not
apply (enforce with -f)
Passed log:
-- Found Zstd, skipping build.

I chose to refresh the integration patch so that zstd-vendor uses the zstd library provided by OE rather than downloading it again and building it inside zstd-vendor.


I don't know what to do!

Describe your problem here.
i tried to compile the psych engine 0.7.3, it should be noted that i am new to this source thing, well,iI followed the steps in the guide, but it doesn't work, and i don't know what to do, if I have installed what it asks me there or I do not know

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x86/x32
Did you edit anything in this build? If so, mention or summarize your changes.
no, I'm just testing to gain experience

Did you installed visual studio? You need 2019 not 2022

Unistall and try reinstall visual studio from here
https://github.com/ShadowMario/FNF-PsychEngine/blob/main/setup/setup-msvc-win.bat

and install hxcpp

that worked! thank you!

no problem


⚠ ClamAV has degraded performance

In 55ecca5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: ClamAV performance has improved in d08f639.


/src/sass/ mis-match with /dist/scss/

The two sets of source files are slightly out of sync. Namely...

dd padding/text-indent in _base.scss;
margins and classnames for .wrapper-large and .wrapper-small in _grid.scss; and
padding on .radio and .checkbox in _forms.scss.

It looks like /dist/scss/ is the canonical version, but then what's the point of the /src/sass/ directory?

I've accepted a few pull requests and haven't recompiled a new patch release that would update the dist. I believe this is the cause of the discrepancies.


Tablib repo

Hello KR Team!
The Tablib repo would benefit from some ambassadorial attention.
Please could you take a look at some of the PRs, and this issue?
https://github.com/kennethreitz/tablib/issues/329
Thank you all for your good work!

@hugovk Thanks for creating the Issue here. We will take a look at it and I will merge those PRs that looks good to me and will request review from @kenneth-reitz and @timofurrer for other feature PRs and major Bug fixes.
Thanks

I'll close this now, thank you all for your help!


Failed to Connect - Compact P Kol Sol Polar XL HMI Air 9+

Hello!
I tried to connect a Compact P + Air 9 with the integration, but I only get the message "Failed to Connect".
I'm trying to connect with a Waveshare USB TO RS485 device, but I've tried a Waveshare RS485 TO ETH , unfortunately that didn't work either.
Nilan integration: 1.2.16 and 1.2.17
Best regards
home-assistant_modbus_2024-11-03T21-26-40.835Z.log

Usb to rs485 should be simple and bulletproof.

Check wiring. See wiki. A, B, Ground
Check Usb path.
Check Modbus ID


maint: Fix duplicate symbols when compiling with luajit

Codecov Report
Patch and project coverage have no change.

Comparison is base (be67b53) 89.23% compared to head (b7a6b89) 89.23%.

:exclamation: Current head b7a6b89 differs from pull request most recent head 3c83641. Consider uploading reports for the commit 3c83641 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #366   +/-   ##
=======================================
  Coverage   89.23%   89.23%
=======================================
  Files           6        6
  Lines         892      892
=======================================
  Hits          796      796
  Misses         96       96

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

Thanks for the PR!
Can you explain how you encountered the issue that requires it?  I haven't seen any problems with automated checks against luajit in master.  Also, if you could provide a NEWS.md entry with the, that would be great!
Cheers, Gary!

@gvvaughan Thanks, PR has been updated, I encountered this issue when compiling luajit and luaposix at the same time in one project.


Dependencies removal in favor of Commons repo

Is your feature request related to a problem? Please provide an existing Issue # , or describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you are proposing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

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

The build is failing due to:
Caused by: org.gradle.internal.resolve.ModuleVersionNotFoundException: Could not find org.opensearch:performanceanalyzer-commons:<IP_ADDRESS>-SNAPSHOT.

Searched in the following locations:
See https://docs.gradle.org/7.6.1/userguide/command_line_interface.html#sec:command_line_warnings
  - https://repo.maven.apache.org/maven2/org/opensearch/performanceanalyzer-commons/<IP_ADDRESS>-SNAPSHOT/maven-metadata.xml
  - https://repo.maven.apache.org/maven2/org/opensearch/performanceanalyzer-commons/<IP_ADDRESS>-SNAPSHOT/performanceanalyzer-commons-<IP_ADDRESS>-SNAPSHOT.pom

The performanceanalyzer-commons publish to Maven is tracked at https://github.com/opensearch-project/opensearch-build/issues/3547


⚠ Bofrost.ch has degraded performance

In 7d44b1f, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 2150 ms

Resolved: Bofrost.ch performance has improved in e01900d after 313 days, 8 hours, 47 minutes.


Changes to batch_config.yaml

More a question than an issue.....
Colorado participated in N3C Data Expansion, which requires including NOTE, NOTE_NLP, and VISIT_DETAIL tables as part of our payload. The extracted tables and associated SQL are described in the config/batch_config.yaml file. But Line #1 in this file has a all-caps "DO NOT CHANGE" warning. Is this file autogenerated such that my additions of the above three tables will be overwritten?
I'm working on a new "Colorado" branch for my changes.

You are right, if you need to have more tables exported, you need to add SQLs here and add those additional 3 cdm tables to this list - https://github.com/stanfordmed/n3c-spark-exporter/blob/main/src/n3c_spark_extractor/config/batch_config.yaml#L22
And you should be set is what I think. If you could create a PR, I can take a look and you can be unblocked.
Permanent solution - we have to move -
https://github.com/stanfordmed/n3c-spark-exporter/blob/main/src/n3c_spark_extractor/config/batch_config.yaml#L22
to
https://github.com/stanfordmed/n3c-spark-exporter/blob/main/config_example.yaml
so that we can have this change work for any list of tables. And there is no need to build the module for the table list change. We can make that change in next version.
Hope this helps.

I have created https://github.com/stanfordmed/n3c-spark-exporter/pull/6 for adding support for additional tables. Please see the README file for configuration change required.

This is fixed in 1.2.0


Remove user command

Have some command that allows a user to be removed from the project. Perhaps
remove-user <admin-key> <name>

Closed with #29.


Updated description of always-run vs standard Result Filters.

Fixes #14625.

@halter73  please review or recommend a review.

This looks like MVC to me. @rynowak, can you take a look? The changes look small.

I'd need to reread a bunch 'o code to remember whether the original "unless" was at all correct. @rynowak am I right you have more recent experience with the always-run filters?

@serpent5  nice worik


Fix hyperlinks

Fix hyperlinks in documentation to point to correct source.

/assign @sarahmaddox

Thanks @kaivalya97! I'm focusing on internal projects at the moment. I'll assign this to @janeman98 for review.
/unassign
/assign @janeman98

/ok-to-test

Thanks @kaivalya97 !
/lgtm
/approve


Use Parse Server with S3/Cloudfront for streaming video

I am about to start a new project that requires videos to be uploaded and streamed to clients, and I'm wondering if Parse Server can integrate with AWS's Cloudfront to stream videos to clients. I'm assuming I would simply upload the video using the S3 file adapter and then integrate it with Cloudfront somehow, but not really sure how that would all work. Using AWS is a much higher learning curve than Parse which I'm already very accustomed to, so I'm hoping this can work. Thanks.

You should create an other adapter, based on S3 that would return the cloud front URL instead of the S3 URL.
Or you could with your beforeSave hooks replace the URL in the Parse.File with the cloud front URL.

@dcdspace How did you solve this? For some reasons parse server does NOT ALWAYS store the full file path to the DB. When a key only contains 1 File, only the filename is added:
{
  "_id": "QTS1i60M7m",
  "username": "myusername",
  "email"<EMAIL_ADDRESS>  "profile_image": "79df34843495cf509a08c296394af833_QTS1i60M7m_user.jpg"
  ...
}

But as soon as I have multiple files in an array, the full file path is stored:
{
  "_id": "JDrPTXG9DJ",
  "images": [
    {
      "__type": "File",
      "name": "26934937a2eba37a28e8daaca0127c8a_image1.jpg",
      "url": "https://mybucket.s3.amazonaws.com/26934937a2eba37a28e8daaca0127c8a_image1.jpg"
    },
    {
      "__type": "File",
      "name": "818560028e8e378e6a82be06bf77ae45_image2.jpg",
      "url": "https://mybucket.s3.amazonaws.com/818560028e8e378e6a82be06bf77ae45_image2.jpg"
    }
  ]
  ...
}

So there's not really a baseURL that I can change in a hook.

I actually haven't started the Cloudfront streaming feature of my app yet, but if you've had any success I'd be happy to hear how. I like the idea mentioned above of using a beforeSave hook to simply swap out the S3 url with the Cloudfront, or make a custom file object and have a pointer to it in my object.
I'm following this tutorial: http://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/MigrateS3ToCloudFront.html

I created a pull request for the s3 adapter: https://github.com/parse-server-modules/parse-server-s3-adapter/pull/6
This adds another option to the S3 config to specify a custom base URL for your S3 ressource (e.g. the CloudFront Url):
filesAdapter: new S3Adapter(
  "AKIAJ7YVVK5LMUCZTQFA",
  "HeyacRj1GSn3gNOLIrjlM1SOVIBBm6d291REo0fy",
  "dongxii-tokyo",
  {
    bucketPrefix: s3BucketPrefix,
    directAccess: true,
    baseUrl: 'https://images.example.com' // This could be your CloudFront URL
  }
),

Awesome thanks!

@flovilmart Parseserver+heroku+mLab was the recommended setup during initial migration days and now S3 is being recommended as a solution to all file related issues. Is the former setup not recommended anymore ? many including me have paid setup and switching to s3 immediately wouldn't be plausible.

@indiandragon MongoDB doesn't store any files, just URLs to where the file is hosted. I think AWS S3 bucket was always one the recommended platforms for file storage.


Check if AssetPublisher is started from a main thread

Starting location tracking requires us to be on the main thread. Currently, we're only handling the happy path, but we should also handle the scenario in which the AssetPublisher is called from a background thread.
┆Issue is synchronized with this Jira Task by Unito

I've found an interesting article that explained why the whole app was freezing when I was trying to use runBlocking with the main dispatcher. With its help it should be easy to fix this issue :wink:
https://proandroiddev.com/how-to-deadlock-your-android-app-with-runblocking-8dab02c2624d


feat(ui): Inject Custom CSS

Checklist:

[x] My organization is added to USERS.md.

Fixes #1451
You can test this feature by executing the following commands.
$ kubectl -n argo create configmap custom-css --from-literal "custom.css=.nav-bar { background-color: #2196f3; }"
$ kubectl -n argo patch configmap workflow-controller-configmap -p '{"data":{"ui.cssurl":"./extra/custom.css"}}'
$ kubectl -n argo patch deployment argo-server -p '{"spec":{"template":{"spec":{"containers":[{"name":"argo-server","args":["server", "--secure=false", "--auth-mode=hybrid", "--extra-files-dir=/shared/app/custom"], "volumeMounts": [{"name": "custom-css", "mountPath": "/shared/app/custom"}]}], "volumes": [{"name":"custom-css", "configMap": {"name": "custom-css"}}]}}}}'

I updated the code so the server doesn't return 304 on removed extra files.

I suggest doing this different.

Add a new variable to get info called “env”. This can be “dev” , “e2e” or “prod”.
Use this to load a CSS file based on the name. Eg dev.css
Bundle CSS using those conventional names.
Add a new parameter to Argo server, “—env” or “ARGO_ENV”

Thank you for your suggestion. I have 2 questions for the env idea.
What env should we use for a locally running argo server?
We could have 2 argo servers with different instance ID in the production. Should we use different env names like prod_1 and prod_2?

By the way, as for the env-like name, I injected it by the following CSS.
    .nav-bar {
      background-color: #3f51b5;
    }
    .nav-bar__logo::before {
      content: 'stg';
      color: white;
      text-align: center;
      display: block;
    }

And got the following result.

However, the env idea is also good. I'd like to hear other people's thoughts.

Hmm... No other comments.
@jessesuen Do you have any thought about the compatibility with ArgoCD?

Is there any progress on this?
I'd like to apply CSS with values as well as Argo CD.

where @dtaniwaki


"del" (borrar) label on keyboard does not fit in button"

@newtonick commented
"I meant to add this to my PR 620 comment when I merged, but forgot to. During my testing I saw these issues. All isolated to spanish and all UI issues that could in part be related to the translation text. None of them I considered blocking, but I think there could still be value in sharing.

"del" (borrar) label on keyboard does not fit in button"

Moving to Translations Repo
https://github.com/SeedSigner/seedsigner-translations/issues/3


update approach in verwey_estimate function

Update verwey_estimate function to use the step before and after peak in order to calculate 2nd derivative zero crossing to get a more precise temperature estimate rather than the current crude .argmax() approach.

It would also be useful to implement plotting that shows both the fit to the derivative as well as the fit shown with the initial thermal demagnetization curve. As in Figure 3 of:

At present, we are just showing the derivative curve:

The secondary issue raised in the last comment of integrating the derivative curves to show them in M vs T space is addressed through cd1d4a8

With 46169d1359ef90e95602cbe4d8dd1ff340445563 and 9625d181aba24c35e5e9ad98980f2a84a2b67793 the second derivative zero crossing approach is now implemented.
As a first check, we can compare the Verwey estimate determined in the IRM Quantum Ferret software (116.2 K for specimen NED2-8c) as seen in the screenshot below:

To the output from the rmag.verwey_estimate() function  first with the plots with the fit made:

and can see that the Verwey transition temperature that is determined is the same (116.2 K). Here it is shown with the zero crossing plot which is an optional plot that can be generated from the rmag.verwey_estimate() function via the new rmag.zero_crossing() function:


added-graphql-logo

I have taken care of each and every constraints. I have added the logo of GraphQL.  Issue no. #828

Do you think this looks right?

Would you like to try and fix it, or shall I just apply the Hacktoberfest spam label now?

Sir, please check out the changes I have did in the 2nd commit. I have uploaded the correct svg file now. Please do check and merge it if it is correct.

Did you read https://github.com/edent/SuperTinyIcons/blob/master/CONTRIBUTING.md ?
Do you think this complies with the information in there?

Yes, I think everything is okay except creating a android version

Your logo is over 1,024 bytes. It doesn't look like the others. You've added the R logo into the root. For some reason, you've also added an HTML5 logo.
I don't know if you're trying to complete Hacktoberfest - but this isn't the way to do it.
Don't just add loads of AI generated stuff you don't understand. Read the guidelines. Ask questions. Start small.


Initial deep state through $deepStateRedirect

Hi,
I was wondering if its possible to set an initial deep state, for example:
$deepStateRedirect.setInitialState("state1", "state1.childstate11.childstate111");

When state1 is first entered it wil direct to state1.childstate11.childstate111.
Thanks,
jvisser

Definitely. This has been in my todo for some time.

Just tested this, works like a charm :)
Thanks!

sweet. I hadn't even pushed the new docs yet, hah!
http://christopherthielen.github.io/ui-router-extras/#/dsr

Not sure if this is a new issue or related to this one. I noticed a sticky leaf state with parameter not reloading when reentering (this worked with 0.0.11). When I turned on firebug I noticed an exception on every state change: 'state is undefined' happening on this line.
https://github.com/christopherthielen/ui-router-extras/blob/master/src/sticky.js#L440

Doh. Likely unrelated. Are you using the new modular build, or full ct-ui-router-extras.js?  Could you put this into a plunkr for me?

Also running into @jvisser's issue so I've created a plunkr below.
http://plnkr.co/edit/po3jqSI7swYrAVaUFwxj?p=preview
Some quick sleuthing tell me that the bug is somehow related to https://github.com/christopherthielen/ui-router-extras/blob/master/src/transition.js#L43 and how it's used a few dozen lines below, probably should be returning data instead...


Speech recognition floating icons

Your speech recognition icons wen minimizes the CLIPTextEncodeFlux start floating around.

Your speech recognition icons wen minimizes the CLIPTextEncodeFlux start floating around.

Hi, disable them in the options, I'll see when I have time.

Fixed!


[FR]: Template key for is heating?

Is your feature request related to a problem? Please describe.
Hello! Thank you again for this integration it's been working great.
Maybe I am missing something, and I'm not entirely sure what this would be called in HA so excuse my ignorance here.
Currently I don't see a way to indicate whether the heater is actively heating. As you can see in the below screenshot. The bottom wiser climate entity lights up when it's actively heating. This one did when the temperature set point was over the room temperature until I override the set_temperature command. That wasn't really ideal anyway.

Would this be possible at all?
If it's too much work then don't worry as it's just a nicety feature more than anything.
Describe the solution you'd like
I'd like a way to specify by a template whether the heater is actively heating or not to trigger the UI display on the climate cards.
Describe alternatives you've considered
It's not a dire need would just be nice. There may be another way already but I couldnt tell with the available documentation.
Additional context
This is because I currently use this template over another climate entity and do all the controls for that climate through this templated climate.

Hello, thank you for the very structured feature request. So in short you would like to have the template to indicate current HVAC status, correct? If so I should be able to implement it easily.

Hello, thank you for the very structured feature request. So in short you would like to have the template to indicate current HVAC action, correct? If so I should be able to implement it easily.

Yes that would be perfect! Thank you so much. I didn't realise that's what HA used so good to know as well.

Hello, thank you for the very structured feature request. So in short you would like to have the template to indicate current HVAC action, correct? If so I should be able to implement it easily.

Just found this hvac_action_template template key. Seems it's already implemented? I'll test it out and close this if so :)

Yep that works fine, closing :)

You see, that happens when developer doesn't remember what he already implemented. ;-) I am glad that you found it anyway. :-)


Prioritise loading reference to knockout and jQuery from global context

Fixes #35
If knockout and jQuery are defined on the global context already, then there is no need to call those libraries again. Doing so would most likely create conflicts anyway.

Thanks for contributing!

Great. Can you do an npm publish? Thanks

Yes, I can. I will do it soon.

Thanks @faulknercs


Known Issue Rollback article conflicts with ADMX policy documentation

Known issue rollback article indicates that you can deploy the policy via an Intune ADMX backed policy.
https://docs.microsoft.com/en-us/troubleshoot/windows-client/group-policy/use-group-policy-to-deploy-known-issue-rollback#2-create-a-custom-configuration-profile-in-microsoft-endpoint-manager
However KIR registry keys in the ADMX backed policies are located under HKLM:\System (SYSTEM\CurrentControlSet\Policies\Microsoft\FeatureManagement\Overrides)
As per the following article ingested policies are not allowed to write to locations under HKLM:\System https://docs.microsoft.com/en-us/windows/client-management/mdm/win32-and-centennial-app-policy-configuration
Deploying ADMX file to machines running 21H2 (19044.1926) results in the following error.

Can the Known Issue Rollback deployment article be updated please?

Thank you for your feedback. We are investigating this issue, and I will reply later.

@dmichine This solution only works after the July 26, 2022 CU is installed. I will add a note to the article. Thank you for the feedback.


Attributes

Change def device_state_attributes(self) to def extra_state_attributes(self) and they will show up in HA.
I added water_level and gust to the attributes and had to do the change above to make them (and water_temp) to show up.

Thank you for the tips. I will look into this the following days, it sure would be great to have more information than only wind speed.


ARROW-11044: [C++] Add "replace" kernel

Purpose a "replace" compute kernel which could fulfil ARROW-10641 - [C++] A "replace" or "map" kernel to replace values in array based on mapping (@jorisvandenbossche) ARROW-9430 - [C++/Python] Kernel for SetItem(BooleanArray, values) (@xhochy). The implementation started on the basis of "fill_null" kernel. But it takes an additional BooleanArray parameter which is used as a mask to trigger value replacement.
 Replace kernel implemented for most (if not all) types but missing null handling.

Note on ArrayData::GetValues(i) in the context of commit https://github.com/apache/arrow/pull/9024/commits/ab2d3d2e0c4b91e54b808a886364d4c0cfbb19ae:

ArrayData::GetValues(i) use "byte offset" even for buffers[2] which stores variable width values...
=> Should it rather use buffers[1] value offset ?
=> Or force absolute_offset to 0 for buffers[2] ?

On my case, I fixed the bug in my code by forcing absolute_offset to 0.

I wouldn't expect a "replace" operation to do this. Instead, this looks more like a "select" operation. @wesm What do you think?

I wouldn't expect a "replace" operation to do this. Instead, this looks more like a "select" operation. @wesm What do you think?

This might actually rather be the "setitem" kernel as proposed in https://issues.apache.org/jira/browse/ARROW-9430 ? (which is something we want as well)

This might actually rather be the "setitem" kernel as proposed in https://issues.apache.org/jira/browse/ARROW-9430 ? (which is something we want as well)

"setitem" is confusing to me (I would expect something where you pass indices and values to set at those indices). "select" is used in Numpy. Another possible is "cond".

"setitem" is confusing to me (I would expect something where you pass indices and values to set at those indices). "select" is used in Numpy. Another possible is "cond".

"setitem" is confusing to me (I would expect something where you pass indices and values to set at those indices).

That's https://issues.apache.org/jira/browse/ARROW-9431.
There could be a variant of "setitem" that takes integer indices, and one that takes a boolean mask?
But it's true that for a "setitem" kernel, you would expect the "values to set" (Replacement in the example in the top post) to be of the length of the number of values that get set, not of the same length as the original array.
So what this PR proposes, is actually more a "where" kernel (using numpy's terminology), for which we have https://issues.apache.org/jira/browse/ARROW-10640.
There is some discussion on that PR that this can be generalized to something like numpy's np.choose. And the np.select  mentioned by @pitrou is quite related to that as well (the exact semantics are a bit different between both: select uses boolean conditions to choose between the arrays, choose a single array of indices into the choice arrays)

"setitem" is confusing to me (I would expect something where you pass indices and values to set at those indices).

That's https://issues.apache.org/jira/browse/ARROW-9431.
There could be a variant of "setitem" that takes integer indices, and one that takes a boolean mask?
But it's true that for a "setitem" kernel, you would expect the "values to set" (Replacement in the example in the top post) to be of the length of the number of values that get set, not of the same length as the original array.
So what this PR proposes, is actually more a "where" kernel (using numpy's terminology), for which we have https://issues.apache.org/jira/browse/ARROW-10640.
There is some discussion on that PR that this can be generalized to something like numpy's np.choose. And the np.select  mentioned by @pitrou is quite related to that as well (the exact semantics are a bit different between both: select uses boolean conditions to choose between the arrays, choose a single array of indices into the choice arrays)

I'd say that personally, I find "select" more intuitively guessable than "where" (which is a weird name for a function). But I could also live with it if it's Numpy's choice. @michalursa

I'd say that personally, I find "select" more intuitively guessable than "where" (which is a weird name for a function). But I could also live with it if it's Numpy's choice. @michalursa

Agreed that "where" is an unintuitive name. For me it's fine to directly use the more general name from numpy, which would actually be both "choose" or "select" (I think the single boolean mask can be seen as a special case for both generalized functions: assuming that the single boolean mask from "where" is interpreted as 0/1 integer indices) for "choose", or is interpreted as a boolean mask + its inverse mask for "select").
Another option mentioned in the JIRA is "if_else" (term used in the R world) or "case_when" (inspired on SQL).

Agreed that "where" is an unintuitive name. For me it's fine to directly use the more general name from numpy, which would actually be both "choose" or "select" (I think the single boolean mask can be seen as a special case for both generalized functions: assuming that the single boolean mask from "where" is interpreted as 0/1 integer indices) for "choose", or is interpreted as a boolean mask + its inverse mask for "select").
Another option mentioned in the JIRA is "if_else" (term used in the R world) or "case_when" (inspired on SQL).

"if_else", "choose", "select" are all fine with me.

"if_else", "choose", "select" are all fine with me.

Wow, I was not expecting such brainstorming on the name of this kernel. My initial intent was to mimic pandas.Series.replace(to_replace, value), where in C++ to_replace could be an BooleanArray mask (same length as input array) which trigger value replacement => So this kernel could be used after any combination of Compare and Boolean operations to implement specific replacement logic.
This could be extended to another implementation where to_replace could be an IntegerArray (length <= as input array) of indexes to replace.
Here is how I use "replace" in combination with "is_nan" (ARROW-11043) to implement fillna():
template <typename value_type>
std::shared_ptr<DataFrame> DataFrame::fillna(value_type value) {
    auto outdf = std::make_shared<DataFrame>();
    ...
    ...
    for (int i = 0 ; i < this->table_->num_columns() ; ++i) {
        auto field = this->table_->schema()->field(i);
        auto chunked_array = this->table_->column(i);
        auto value_datum = arrow::compute::Cast(arrow::Datum(value), chunked_array->type()).ValueOrDie();
        auto nulls = arrow::compute::IsNull(chunked_array).ValueOrDie();
        auto nans = arrow::compute::IsNan(chunked_array).ValueOrDie().chunked_array();
        auto to_replace = arrow::compute::Or(nulls, nans).ValueOrDie();
        auto clean_chunked_array = arrow::compute::Replace(chunked_array, nans, value_datum).ValueOrDie().chunked_array();
        outdf->table_ = outdf->table_->AddColumn(i, field, clean_chunked_array).ValueOrDie();
    }

    return outdf;
}

Wow, I was not expecting such brainstorming on the name of this kernel. My initial intent was to mimic pandas.Series.replace(to_replace, value), where in C++ to_replace could be an BooleanArray mask (same length as input array) which trigger value replacement => So this kernel could be used after any combination of Compare and Boolean operations to implement specific replacement logic.
This could be extended to another implementation where to_replace could be an IntegerArray (length <= as input array) of indexes to replace.
Here is how I use "replace" in combination with "is_nan" (ARROW-11043) to implement fillna():
template <typename value_type>
std::shared_ptr<DataFrame> DataFrame::fillna(value_type value) {
    auto outdf = std::make_shared<DataFrame>();
    ...
    ...
    for (int i = 0 ; i < this->table_->num_columns() ; ++i) {
        auto field = this->table_->schema()->field(i);
        auto chunked_array = this->table_->column(i);
        auto value_datum = arrow::compute::Cast(arrow::Datum(value), chunked_array->type()).ValueOrDie();
        auto nulls = arrow::compute::IsNull(chunked_array).ValueOrDie();
        auto nans = arrow::compute::IsNan(chunked_array).ValueOrDie().chunked_array();
        auto to_replace = arrow::compute::Or(nulls, nans).ValueOrDie();
        auto clean_chunked_array = arrow::compute::Replace(chunked_array, nans, value_datum).ValueOrDie().chunked_array();
        outdf->table_ = outdf->table_->AddColumn(i, field, clean_chunked_array).ValueOrDie();
    }

    return outdf;
}

Wow, I was not expecting such brainstorming on the name of this kernel.

Sorry for the distraction :-) That said, for clarity it seems that "if_else" would be both explicit and non-misleading.
The signature could be if_else(cond, when_true, when_false) with the semantics cond ? when_true : when_false. Nulls would probably just pass through.
Do you want to update this PR?

@bu2 would you have time to update this based on the above comments? (https://github.com/apache/arrow/pull/9024#issuecomment-775323925)

Closing as superceded by #10410.


How can I use hyperpwn with rr-debugger

Layout is wrong when I run "rr replay" in hyper, I only see command and stack window.
And I use gef.

Is the output of gef correct if you don't use gdb in hyper?
Could you post a screenshot of the wrong layout? Do you try stepi after the layout is loaded?

Without hyper:

In hyper:

Could you try to run hyperinator load hyperpwn-gef before rr replay likes this:

(You may need to npm install -g hyperinator firstly)
And what's the version of hyper and gef do you use?

Great! I have no idea why it can't be used directly, but at least we have a way to solve it 😃


new: dev: increase coverage for rubix-presto module

Only test classes have been introduced, so testing on a Qubole cluster should not be required.

Pull Request Test Coverage Report for Build 84

0 of 0   changed or added relevant lines in 0 files are covered.
47 unchanged lines in 15 files lost coverage.
Overall coverage increased (+0.1%) to 24.875%

Files with Coverage Reduction
New Missed Lines
%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/FileDownloader.java
1
90.91%

rubix-spi/src/main/java/com/qubole/rubix/spi/DataTransferClientHelper.java
1
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/RemoteFetchRequestChain.java
1
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/LocalDataTransferServer.java
2
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/CachedReadRequestChain.java
2
84.0%

rubix-core/src/main/java/com/qubole/rubix/core/DirectReadRequestChain.java
2
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/BookKeeperServer.java
2
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/utils/ShellExec.java
2
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/CachingInputStream.java
3
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/FileDownloadRequestChain.java
3
0.0%

Totals

Change from base Build 81:
0.1%

Covered Lines:
548

Relevant Lines:
2203

💛  - Coveralls

Confirmed that the build passes when built using JDK 8

I'll let others comment on the change to Java8

can we test hive with java7 and presto with java8?


ignore consecutive duplicates in history

bash style
added breaking test, and fix
it's a matter of opinion of course, so i can understand if you don't want to have history behave this way

Awesome PR - thanks!
:+1:


InternalError: Can't find method com.glide.script.system.GlideSystemUtilScript.getScriptError(string,string)

Hi,
after upgrading from 4.11 to 4.12, on my PDI, I receive error:
"InternalError: Can't find method com.glide.script.system.GlideSystemUtilScript.getScriptError(string,string). (; line 498)"
when I try to run any code.
My PDI is on:
Build name: Sandiego
Build date: 03-17-2022_1811
Build tag: glide-sandiego-12-22-2021__patch1-03-02-2022
Any idea why I get this error?
Thanks!

Hi,
Yes, you are right. ServiceNow changed the Java method used for checking scripts with the Tokyo release which broke Xplore. This release fixes that issue and although I had thought it backwards compatible with pre-Tokyo versions, it turns out it is not.
The workaround for now is to revert the processor to 4.11 version until I can get a fix out. If you upgrade to Tokyo, you'll need 4.12 again.
Thanks


Removal of previously public API breaks downstream crate

It looks like https://github.com/dtolnay/syn/commit/957840eba2c7f11c95e0227760d78dc481a91de7 breaks enumset_derive as unfortunately that crate has a reference to syn::export.

xref: https://github.com/Lymia/enumset/issues/17

xref: https://github.com/Lymia/enumset/issues/17

syn::export has never been a public API. It's hidden and commented "not public API".
https://github.com/dtolnay/syn/blob/d58d7cbe55a3160045db000802ca4c097bf774a1/src/lib.rs#L789-L791

syn::export has never been a public API. It's hidden and commented "not public API".
https://github.com/dtolnay/syn/blob/d58d7cbe55a3160045db000802ca4c097bf774a1/src/lib.rs#L789-L791


Caching question

Hi guys,
thank you very much for your work! Awesome!
I want to use PINRemoteImage in my current project but need to solve the problem that images are updated on the server side using the same image url.
I looked in the code and PINRemoteImage is just checking locally if the image is inside the device cache. If that is true it returns the image and ignores perhaps newer images on the url. Is that true? Best would be if PINRemoteImage would use the caching headers like modified-since or just show the locally cached image as a placeholder and downloads the new image and updates the view.
Any ideas how I could use PINRemoteImage to also check for images if the user is online, the url is the same and the image exists locally already?
Thank you very much!
Best
Alex

Hey @alibasta sorry for the super late reply :(
PINRemoteImage doesn't really have a good solution for this currently. The only work around is to ignore the cache on a request. I'd be open to pull requests that added support for this though!


[serverless-1.30.0] patch func deploy task for serverless 1.30.0

patch for func-deploy task to use midstream specific client built for 1.30.0

@jrangelramos  shouldn't this be also applied to pac version of the file?

/override "On Cluster RT Test (1.20.x, ubuntu-latest)"
/override "On Cluster Test (1.20.x, ubuntu-latest)"

/override "On Cluster RT Test (1.20.5, ubuntu-latest)"
/override "On Cluster Test (1.20.5, ubuntu-latest)"

/approve
/lgtm

/approve
/lgtm
/override "On Cluster RT Test (1.20.5, ubuntu-latest)"
/override "On Cluster Test (1.20.5, ubuntu-latest)"

/unhold


manifest release: select which directories/assets to release

I'm using the manifest release to version multiple packages inside a single repository.
It's all working as expected, minus the ability to chose which files/directories gets bundles into the released asset.
Is it possible to configure the manifest to only package assets as a function of globs, i.e.:
{
    "packages": {
        "first-package": {
            "component": "first-package",
            "release-type": ....,
            "assets": [
                 "./**.go",
                 "/ROOT_README.md"
            ]
        },
        "second-package": {
            "component": "second-package",
            "release-type": ....,
        },
        "assets": [
             "./**.cs",
             "./**.yaml",
             "/ROOT_README.md"
        ]
}

/T

Sorry for the delay in getting to this.
By assets, do you mean the zip file on the GitHub release? If so, release-please doesn't do that -- that source download is built into every GitHub release by default. release-please also does not actually handle publication -- it defers to whatever automation you want to run after creating a GitHub release.


Update to coincident v2

Looking into updating to the latest v2 released a couple of days ago: https://github.com/WebReflection/coincident/releases/tag/v2.0.0
And see what effects it may have on https://github.com/jupyterlite/xeus/issues/102 too.

Pausing this for now since there are quite a few changes in coincident v2.
Moving in favor of https://github.com/jupyterlite/xeus/pull/108 for now to align the behavior with https://github.com/jupyterlite/pyodide-kernel.


Problems with setup and example

Hello, I got some problem with setup and test the test in Moveit RViz.
In RViz, I cannot see any model, and the terminal shows lots of warn and Error. Such like:
[rviz2-1] [ERROR] [1714078878.830273830] [moveit_background_processing.background_processing]: Exception caught while processing action 'loadRobotModel': parameter 'panda_arm.kinematics_solver_timeout' has invalid type: Wrong parameter type, parameter {panda_arm.kinematics_solver_timeout} is of type {double}, setting it to {string} is not allowed.
[rviz2-1] [WARN] [1714078875.051494350] [moveit_robot_model.robot_model]: Link panda_leftfinger has visual geometry but no collision geometry. Collision geometry will be left empty. Fix your URDF file by explicitly specifying collision geometry.
I will past the picture under. Since I do follow the tutorial to setup, is there any change I should do in the code? Hope you could help me to figure out.

add export LC_NUMERIC=en_US.UTF-8 into your .bashrc


Some of the SVGs not loading correctly

Hi
I am working in my office application and we are downloading svgs from our api and using it.
But the problem is that our svgs have multiple colors but these colors not loading. Instead it showing only black color in whole svg.
Please help me by giving the solution.

Please help me by giving me an example of an svg that causes it to fail.

Closing for now, will reopen if this turns out to be a bug.


Update/Complete JavaDoc

Our JavaDoc for the APIs is largely incomplete and some parts of it which do exist are outdated or otherwise incomplete/terse.
These need to be fleshed out as a first step in a broader documentation strategy.

Closed as https://github.com/Terracotta-OSS/terracotta-apis/pull/2


Menu cell with custom UI support

added the ability to support custom UI

Probably to update the gif

@ghvg1313 can you please fix the conflicts? thanks

@quentinR It's good to review now


Unable to use sidebar due to color of all the component of sidebar change to red.

System Details

OS: Arch (linux v6.10.2)
Thorium Version  124.0.6367.218 stable

Problem
Out of no where after installing thorium browser the sidebar color is red.
Screenshots

More context it only happening when I open sidebar with bookmarks options, others work more of less fine.

also happening with macos - ive experienced

me too

New update from thorium fix this issue


Physics debug model is located at incorrect position when RootScene's Offset is not 0

Release Type: Official Release
Version: <IP_ADDRESS> / <IP_ADDRESS>-beta01-0441
Platform(s): Windows
Describe the bug
To Reproduce
Steps to reproduce the behavior:

Create new game project.
Add a StaticColliderComponent to the Sphere entity and add a sphere collider to it.
Set MainScene.Offset to X=5 Y=0 Z=0.
Add DebugPhysicsShapes asset and add it to the Sphere entity.
Run the game and press Ctrl + Shift + P to enable debug rendering.

Expected behavior
The physics debug model is located at the same position as SphereColliderShape.
Screenshots

Log and callstacks
Additional context

Hello, is anyone currently working on this issue? If not, I'd like to try working on it.
Thanks

No one should be looking into this currently, if you want to give it a go please do! If you have any questions feel free to hop in Discord if no one is available here I find most are active in Discord these days.
@Aggror tagging you in case there is anything to say someone is looking into this like was done for the VR task. I assume that was just due to the bounty that it had though.

This one may actually be solved already, I was not able to reproduce the behavior.  I'll leave it to you all to determine.  :)

sweet, Ill test it tomorrow and see if we can close it. Always nice to clean these old issues.
If you want another issue to look into I think https://github.com/stride3d/stride/issues/2236 should be fairly simple to get started. Seems like it just initializes with a NaN value.

Thanks I'll check out that one.

@Eideren I think this issue can be closed. I cant seem to recreate it in the latest release.


[BUG] HandyControls v3.4.1 breaks the themes

Describe the bug
Hello. I upgraded from HandyControls v3.3.11 to v3.4.1 and the application theme breaks. (The forked version, but I don't know where to put the issue).

The theme does not fully changes when the ThemeManager.Current.ApplicationTheme property changes. It changes partially, leaving the UI in an inconsistent state.
Colors from each theme are the default ones provided by HandyControls and the overwritten values are not respected.

Configuration
In my App.xaml
<hc:ThemeResources UsingSystemTheme="False" RequestedTheme="Light">
    <hc:ThemeResources.ThemeDictionaries>
        <ResourceDictionary x:Key="Light" hc:ThemeDictionary.Key="Light" Source="/Dictionaries/ThemeLight.xaml"/>
        <ResourceDictionary x:Key="Dark" hc:ThemeDictionary.Key="Dark" Source="/Dictionaries/ThemeDark.xaml"/>
    </hc:ThemeResources.ThemeDictionaries>
</hc:ThemeResources>

In my Code behind to change the theme:
using HandyControl.Themes;

public void UpdateTheme(RegisCarTheme theme)
{
    _ = this.Dispatcher.BeginInvoke(() =>
    {
        switch (theme)
        {
            case RegisCarTheme.Automatic:
                ThemeManager.Current.ApplicationTheme = ThemeManager.GetSystemTheme();
                break;
            case RegisCarTheme.Light:
                ThemeManager.Current.ApplicationTheme = ApplicationTheme.Light;
                break;
            case RegisCarTheme.Dark:
                ThemeManager.Current.ApplicationTheme = ApplicationTheme.Dark;
                break;
        }
    });
}

In my overridden themes: "/Dictionaries/ThemeDark.xaml"
<ResourceDictionary
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:hc="https://handyorg.github.io/handycontrol">

    <SolidColorBrush x:Key="BackgroundBrush" Color="#27272a" />
    <!-- More colors overridden here. Omitted for brevity. -->

</ResourceDictionary>

Steps to reproduce the bug

Upgrade from HandyControls v3.3.11 to v3.4.1.
Create a setup similar to the described above. (Two buttons, that change the UI theme)
The theme will not be fully swapped leaving some colors from the old theme.
If any color is overwritten in custom Resource Dictionaries, they are not used.

Expected behavior
The theme should change acordingly. See below for screenshots.
Screenshots
In v3.3.11 (Correct behavior)

The theme changes automatically whenever the ThemeManager.Current.ApplicationTheme property changes.
Colors from each theme are used from the overwritten values (see the "App.xaml" code).

Condition
Light theme
Dark Theme

Always. This is the expected behavior

In v3.4.1 (Incorrect behavior)

The theme does not fully changes when the ThemeManager.Current.ApplicationTheme property changes. It changes partially, leaving the UI in an inconsistent state.
Colors from each theme are the default ones provided by HandyControls and the overwritten values are not respected.

Condition
Light theme
Dark Theme

When RequestedTheme="Light" in App.xaml

When RequestedTheme="Dark" in App.xaml

NuGet package version
HandyControls (Custom version) 3.4.0
IDE
Visual Studio 2022
Framework type
.Net 6.0
Windows version
May 2021 Update (19043)
Additional context
Related issue in forked HandyControls repo: https://github.com/ghost1372/HandyControls/issues/158

We haven't released 3.4 yet.


CONSTRAINTS OF Pattern not recognized

out-of-class constraint definitions are not fully recognized:
    !!@ name = "BefundJaehrlichkeit muss Hauptprozess>Wasser zugeordnet sein"
    CONSTRAINTS OF BefundJaehrlichkeit =
      MANDATORY CONSTRAINT BefundProzessquelle->Hauptprozess == #Sturz;
    END;

The term CONSTRAINTS matches only against the constant CONSTRAINT and so the ending 'S' is out of match.

Ref. #22


Poster needed for our upcoming HUG Gandhinagar event on 17th Feb ,24

Goal:
We need poster for the event for the upcoming HUG  Gandhinagar event.
Info:
Event name: Cloud Tech Takeover:  HUG Gandhinagar &  GDG Gandhinagar Join Forces!
Venue: Lucent Innovation
Time: 10.00am to 1.00pm
Date: 17th Feb 2024

Also add ,GDG Gandhinagar, Open Source Weekend  as community partner

Style Information:

Deadline:
13th Feb,24

GDG Gandhinagar Logo

hi there I had messaged you the canva editing link for posters also below is the editing link
Canva Editing link


Allow starting prometheus stack in clusterloader2

See https://github.com/kubernetes/kubernetes/issues/74213
The feature is flag-gated behind the --set-up-prometheus flag.
I will be gradually launching it, starting from the smallest tests/clusters.
It's probable that we will have to tune the cpu/memory limits/requests in order to make it work for the big clusters, but as the yaml as read via the template provider it shouldn't be hard to have memory/cpu as a function of # of nodes in the cluster.

/assign @wojtek-t
/assign @krzysied

/lgtm
/approve
/ok-to-test


Abnormal Driving Dataset in Autopilot Mode

Hi CARLA team,
I have a question about collecting the abnormal driving dataset.
My original plan is to collect a few datasets in the autopilot mode that doing everything perfectly and collect a few datasets that drive overspeed and doing unstable steering motion in the manual control mode.
I personally experimented in collecting datasets in the manual mode and run into the issue of performance limitation of pygame that prevents me from driving. The python client becomes extremely sluggish when saving images and the pygame cannot take prompt keyboard inputs anymore so that the car can only speed up to 2km/hr no matter how I press the keyboard.
From that experience, I figure another way to collect abnormal data. I can increase the range of the steer to (-0.65,0.65) to make car driving behavior differently a lot from the regular driving in the autopilot mode. As I expected, I can see the car drive in the opposite lane or sidewalk etc. However, after a few seconds, the car driving into the neighborhood and is stuck in a house.
1.My first question is that what is your suggestion or advice to collect abnormal data efficiently in either manual control or in autopilot mode with a larger range of noise? For the autopilot mode with larger range of noise solution, I expect car to driving longer so that I can collect a larger dataset like 3mins?
Another thing I try out to get the abnormal dataset in the autopilot mode is to increase the throttle to make car drive overspeed. I basically time the throttle value 2 times and send them back to the controller. What I observed now is that the car will speed up faster than before. In the 30km/h speed limit zone, it can drive up to 32 km/h sometimes but in 90km/h speed limit zone, it will still follow the rule of speed limit.
2. My second question is that is there any way to make car drive overspeed in the autopilot mode.
My takeaway for now is that due to the performance limitation of the pygame, the user cannot collect datasets in the manual mod as they wanted ( in my case is to collect datasets driving crazy and overspeed). I think adjust the input from the autopilot control is a more promising solution, but I am stuck in what is the best way to achieve this goal.
Many thanks in advance.
Best regards,
Taylor

Hello @tayloryang722
Thanks for the issue. I will try to answer your questions.

Pygame is slow: Indeed, it is very hard to drive with pygame. One
turnaround is setting the outputscreen resolution very small and reducing
the resolution of your screen so you can se it better. However this
does not solve the problem when there are several sensors.
We are currently working on that on issue #421.
Also considering using a joystick for driving. Driving on the keyboard is very
unpractical for any serious data collection. Check issue #88

The autopilot has limitations, it lacks flexibility since it always obeys the speed
limits of the zone. Also, its fixed policy does not allow for you to build a more robust
recovery from noise for instance. We are currently on a code sprint to have a robust
client side agent #428 that will hopefully enable you to easily adapt for your needs !
If it interests you , you can eventually also contribute to our code sprint. Consider joining our discord chat for a more instant discussion on that https://discordapp.com/invite/vNVHXfb

Cheers,
Felipe.

嗨卡拉团队，
我有一个关于收集异常驾驶数据集的问题。
我最初的计划是收集一些在自动驾驶模式下一切都完美的数据集，并收集一些在手动控制模式下超速行驶和转向不稳定的数据集。
我亲自尝试以手动模式收集数据集，并遇到了 pygame 性能限制的问题，导致我无法驾驶。 python客户端在保存图像时变得非常缓慢，并且pygame无法再及时接受键盘输入，因此无论我如何按下键盘，汽车只能加速到2公里/小时。
根据这次经验，我想出了另一种收集异常数据的方法。我可以将转向范围增加到（-0.65,0.65），以使汽车驾驶行为与自动驾驶模式下的常规驾驶有很大不同。正如我所料，我可以看到汽车在对面车道或人行道等处行驶。然而，几秒钟后，汽车驶入附近并被困在一所房子里。
1.我的第一个问题是，您对在手动控制或噪声范围较大的自动驾驶模式下有效收集异常数据有何建议或建议？对于具有更大范围噪声解决方案的自动驾驶模式，我希望汽车行驶更长时间，以便我可以收集更大的数据集（例如 3 分钟）？
我尝试在自动驾驶模式下获取异常数据集的另一件事是加大油门以使汽车行驶超速。我基本上对油门值进行两次计时并将它们发送回控制器。我现在观察到车子会比以前加速得更快。在30公里/小时限速区，有时可以行驶到32公里/小时，但在90公里/小时限速区，仍然会遵循限速规则。
2.我的第二个问题是有什么办法可以让汽车在自动驾驶模式下超速行驶。
我现在的结论是，由于 pygame 的性能限制，用户无法按照自己的意愿在手动模式中收集数据集（在我的例子中是收集驾驶疯狂和超速的数据集）。我认为调整自动驾驶仪控制的输入是一个更有前途的解决方案，但我陷入了实现这一目标的最佳方式。
提前谢谢了。
最好的问候， 泰勒

Hi Have you build the abnormal driving dataset? If so, could you please share a link with me? I would appreciate that.


NSFS | NC | get_bucket_acl fails on reading bucket.owner_account.id

Environment info

NooBaa Version: noobaa-core-5.15.0-20240213.el8.x86_64
Platform: rhel8

Actual behavior

Invoking get-bucket-acl fails with internal error here -

https://github.com/noobaa/noobaa-core/blob/daba777409ffe5e6803282c24ad2f74f098d6de3/src/endpoint/s3/ops/s3_get_bucket_acl.js#L11-L15

Seems like  owner_account field is expected to be an object with id and email subfields in the sdk level, but bucketspace_fs.read_bucket() returns owner_account as it is stored in the configuration which is a string representing just the id.

Expected behavior

Should not fail the request.
This should affect also

Steps to reproduce

Invoke get-bucket-acl.

More information - Screenshots / Logs / Other output
Command:
$ aws  --endpoint http://localhost:6001/ s3api get-bucket-acl --bucket test1
An error occurred (InternalError) when calling the GetBucketAcl operation (reached max retries: 4): We encountered an internal error. Please try again.

From the log:
Feb 22 17:06:02 host-01 node[441680]: [nsfs/441680] [ERROR] core.endpoint.s3.s3_rest:: S3 ERROR
  <?xml version="1.0" encoding="UTF-8"?>
  <Error>
    InternalError
    <Message>We encountered an internal error. Please try again.</Message>
    <Resource>/test1?acl</Resource>
    <RequestId>lsxf0p7u-ezp1ef-edt</RequestId>
  </Error>
GET /test1?acl  { ... }
TypeError: Cannot read properties of undefined (reading 'toString')
    at Object.get_bucket_acl [as handler] (/usr/local/noobaa-core/src/endpoint/s3/ops/s3_get_bucket_acl.js:13:37)
    at async handle_request (/usr/local/noobaa-core/src/endpoint/s3/s3_rest.js:150:19)
    at async Object.s3_rest [as handler] (/usr/local/noobaa-core/src/endpoint/s3/s3_rest.js:65:9)

#7862 contains a fix for this issue, closing


As a User, I want to be able to import and export my existing timetable from and to other timetabling apps...

User Story: As a User, I want to be able to import and export my existing timetable from-to other timetabling apps, such as NUSMODS, so that I can save time and energy.
Feature: importtimetable feature
Assigned to: Alexis
import:

[x] User can import (from NUSMODS) and see the timetable

export

[x] User can export timetable out of the app. (and consequently, can import it back in again, with no issues!)

Important comments:

I am considering using a 3rd party ics-parser class, but that might be more complicated in the short run; and also have to learn/debug it.

I ended up using a third party class "biweekly" to help read/write to the .ics filetype.
See PR #55


Could not find artifact ca.bkaw:paper-nms:jar:1.19.2-SNAPSHOT

Here is my pom: https://pastebin.com/KYGufBdA
If I want reload my pom.xml comes this error "Could not find artifact ca.bkaw:paper-nms:jar:1.19.2-SNAPSHOT"

Hello. You will need to run paper-nms:init. Follow the instructions in the readme, steps 4 and onwards.

Oh damn... I'm sorry, it works.
Thanks


Request: Make maintainer links mailto: instead of github handle

It is very inconvenient and confusing that the "Name " links are not actual mailto links but github handles.
Please fix that.

Personally I prefer the link to the github profile. But I understand that some might prefer other means of communication. I'll see we add this functionality in the future.

Maybe we can have both, a octocat that links to github and a letter symbol that is a mailto link?


net/http: Transport.CancelRequest is inherently racy

The implementation of http.Transport.CancelRequest looks for the supplied request struct in a map of in-flight requests, doing nothing if it doesn't exist. If I understand correctly, that means that this snippet is racy, in the sense that cancellation of the context may be lost if it is seen before RoundTrip gets around to sticking the request in the map:
func cancellableDo(ctx context.Context, req *http.Request) (err error) {
  // Propagate cancellation of the context to the HTTP request.
  go func() {
    <-context.Done()
    http.DefaultTransport.CancelRequest(req)
  }()

  // Call through.
  err = http.DefaultTransport.RoundTrip(req)
  return
}

This is made more likely to happen by RoundTripper implementations like oauth2.Transport, which may make network calls before calling through to RoundTrip. In the end I'm forced to cancel in a hacky loop to ensure that the HTTP package pays attention.
Is there anything that can be done to eliminate this race?

CC @bradfitz

This came up on the list again. https://groups.google.com/d/topic/golang-nuts/m-Ocj7YVahU/discussion
I'm starting to think that it's impossible to do cancellation without races with the current interface. We might have to make a new RoundTrip interface which takes a context object and store the cancellation on there, like what http://blog.golang.org/context describes.

As pointed out in that thread, the documentation is correct. You can't cancel it until it's been started.
This code is also "racy":
m := map[string]string{}
go func() { mu.Lock(); m["foo"] = "bar"; mu.Unlock() }()
go func() { mu.Lock(); delete("m, foo"); mu.Unlock() }()

... sometimes the map entry will be there and sometimes it won't, because you're not waiting to delete it until it's been added.
That doesn't mean there's a bug with maps or with delete.
We could add more words if that would help, but we can't start adding contexts to APIs for cancellation because the standard library a) doesn't have them, and b) is frozen.

Having the outcome of the map example be non-deterministic is fine, but for the Transport, the stakes are a bit higher. If you cancel a request before it has been started, RoundTrip might block indefinitely when the request does issue, leading to a leaked goroutine and a leaked net.Conn since you never end up calling resp.Body.Close() on the response.
Clearly this is a punt to go1.6 issue, but still think we need to address this. The current API makes it too easy to write code that is subtly wrong.

@bradfitz: Of course there's no bug, in the sense that the API does what is promises. But I'm saying I think it's a less useful API than it could be. Your example is not fair: the user can synchronize on the completion of an insertion into a map they own, but it's impossible for them to synchronize on RoundTrip inserting into its internal map. So it's impossible for them to ensure that a cancellation will not be lost.
You can't change the existing CancelRequest function because of the API freeze, I understand. But surely there is another solution that is backwards compatible, by adding a new method? context.Context would be the "best" thing to use if designing from scratch since the community has standardized on it, but that doesn't mean the standard library can't have its own bespoke version (e.g. pass in a cancellation channel).

@jacobsa, the use cases this was originally designed for was calling Transport.CancelRequest after Transport.RoundTrip has returned (and then you know it's in the in-flight map).  See #3672 for history.
And for those concerned about Transport.RoundTrip taking too long to return the headers can use Transport.ResponseHeaderTimeout. History is in #3362.
If you really want to know what phase of its lifecycle a Transport's *Request is in, perhaps that ties into #9194
In addition to lack of contexts, it's also unfortunate that CancelRequest doesn't return a value too.
But unless there's something concrete to do here, I'm tempted to close this bug. So please propose something if you have an idea.
Maybe Transport could keep track of the set of requests which have been canceled but not yet started (and then fail the request if it does come in later), and start a timer to remove that map entry in N seconds? But what is N?

@bradfitz: Thanks for the background. I see, it makes a lot more sense when you view it from the lens of wanting to cancel once you've already received the response headers and are still reading the body. I came at it from "wanting to cancel at any time".
I'm sure you're aware that setting a timeout up-front is less satisfying than being able to cancel at an arbitrary time, especially when you're trying to bridge a context.Context API with net/http. But it's still helpful of course, since it bounds your latency from "context is cancelled" to "I actually return from my function that calls into net/http". Still, it has the same trouble you mentioned: how do I choose the timeout?
Here are two concrete proposals:

Add a callback that lets me know when the request becomes cancellable, so I can synchronize on that. (As you said, this might tie in with #9194.)

Add a chan struct{} field to http.Request. Approximately the same semantics of Context.Done: closing it means the request is regarded is cancelled, nil means not cancellable in this manner.

I personally like (2). The cost is that it would require net/http to maintain a background goroutine with a list of cancellation channels for in-flight requests (if any had ever been made), calling reflect.Select to wait for one of them to be closed or for a request to be finished. But this is what users have to do today if they want to support cancellation in the context.Context style, and they would still need to do it if (1) were chosen.

I kinda like (2) also. But: what is its type?
a) <-chan struct{} // closed on cancelation
b) chan struct{} // closed on cancelation
And if (b), who can close it? You wouldn't want multiple goroutines calling close on it. One might panic then.
What about Transport.CancelRequest? It would reasonably want to cancel it also. Maybe? I suppose we don't want to allocate the channel by default (and Transport.CancelRequest can't assume it's already there, nor set it from another goroutine), so I CancelRequest will have to forever use its existing mechanism. That leads me to say (a), which is also the type returned by Context.Done in http://godoc.org/golang.org/x/net/context#Context

I think <-chan struct{} would be better, yeah.
Regarding Transport.CancelRequest: I think you could actually leave this as-is, documenting its downsides and maybe marking it as deprecated, and have it continue to do the hard work of cancellation. That is:

RoundTrip inserts into the map then tells the cancellation goroutine about the channel.
The cancellation goroutine calls CancelRequest when one of the channels it knows about is closed.

Odd to be using the deprecated API to implement the new one internally, but it's safe because you can synchronize on the insert into the map. Put another way: if the old API weren't already promised, it could just be an implementation detail.

@bradfitz: What do you think of my previous comment? If you're good with the idea, I'm happy to send a CL.

I don't care much about the implementation. What's easiest or most efficient. I think both the easiest and most efficient is selecting from the *Request.Cancel <-chan struct{} channel (nil or not) would be easiest, without a new map insert or goroutine or anything.
Please send a CL, ideally soon. The tree is increasingly frozen.

Oh duh, I was way overcomplicating it, you're right. Code review that just adds new select cases (no new goroutines) is here.

@bradfitz: Oh duh, I was way overcomplicating it, you're right. Code review that just adds new select cases (no new goroutines) is here.


How to validate Duplicate field with gojsonschema?

This is my json content:
{
"name" : "tony",
"name" : "jane"
}
There are two same fields, but Validate OK.

Per RFC 8259, the behavior of duplicate fields is undefined.
Per the latest JSON Schema draft, objects in the JSON Schema data model cannot have multiple keys, so the behavior for a JSON document with multiple keys is undefined.

Hi,
So, I can't use gojsonschema to detect multiple keys, am I right?

@Houlton you cannot use any conforming JSON Schema implementation to detect multiple keys.  Multiple keys are not an interoperable feature of JSON and therefore they are not covered by the specification.  All implementations should have this limitation, because the IETF JSON working group that produced RFC 8259 and its predecessors chose not to address duplicate keys in an interoperable way.  JSON Schema therefore does not have any reliable way to detect or otherwise interact with them.

Got it . Thank you very much for your explanation.


chore: release 1.2.205

:robot: I have created a release *beep* *boop*
1.2.205 (2023-03-02)
Features

bundles: allow usage of bundles with included secret-values as oci chart dependencies (469678c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v1.2.205 :sunflower:


Support Object Storage to store FE metadata

While support for storage and compute separation is great the FE metadata is still on local disk today which means there is additional operational overhead etc so storing FE metadata in an object will make star rocks more cloud native.

This would be a very welcome feature so the persistent storage layer for StarRocks instance could exist completely in cloud storage.
The FE data must be backed up and recovered separately from the table data adds complexity and potential mismatches between structure and data if the backups can't maintain consistency between FE and cloud storage table data.


[OCP-4.x][GCP] Default storageclass ps-ssd needs to be created

https://github.com/openshift-scale/scale-ci-deploy/blob/master/docs/ocp4_gcp.md#openshift_prometheus_storage_class and https://github.com/openshift-scale/scale-ci-deploy/blob/master/docs/ocp4_gcp.md#openshift_prometheus_storage_class use pd-ssd as the defaults, which is not created by default. For this to work, sc can be created as follows:
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: pd-ssd
provisioner: kubernetes.io/gce-pd
parameters:
  type: pd-ssd

closed by #75


[minimal-blog]: favicon not using path prefix

Hey LekoArts,
First of all thanks for your beautiful themes - I really like them!
In case one wants to deploy a theme via github pages, its necessary to work with the path prefix for repo pages (https://www.gatsbyjs.org/docs/path-prefix/). As far as I know the favicon for all themes are placed in the SEO component e.g.:
https://github.com/LekoArts/gatsby-themes/blob/95fbdda698b33fb42378ed6e36c91518cb52ef52/themes/gatsby-theme-minimal-blog/src/components/seo.tsx#L59-L61
Since the withPrefix helper (https://www.gatsbyjs.org/docs/path-prefix/#add-the-path-prefix-to-paths-using-withprefix) is not used here, it is not working for when a path prefix is configured.
I could think of two ways to fix it. Obviously the usage of the withPrefix helper can be added here or it might be also an option to completely remove it from the SEO component and let gatsby-plugin-manifest do the work.
What do you think?
Cheers,
Frederik

Thanks for the report, fixed it with withPrefix()

Thanks for the quick fix!


twrnc - Cannot use import statement outside a module

Hey!
I'm getting this error when trying to import twrnc (https://github.com/jaredh159/tailwind-react-native-classnames) in my solito app.
Server Error
SyntaxError: Cannot use import statement outside a module

I've installed both like this
npx create-solito-app@latest my-solito-app
cd packages/app
yarn add twrnc

But then when trying to import it in any of my components
import tw from 'twrnc'

function HelloWorld() {
  return (
    <View style={tw`bg-blue-100`}>
      <Text style={tw.style('text-md')}>Hello</Text>
    </View>
  )
}

I get the error
SyntaxError: Cannot use import statement outside a module
Any ideas on how to fix this?
Thanks!

you need to add it to next-transpile-modules in next confits

you need to add it to next-transpile-modules in next config.

Working. Thanks @nandorojo!

no problem! they should probably add it to their nextjs docs


Synch develop branch to main

This looks right.

@levisimons If you had newer changes on the server somehow then you can pull them in after this merge.


analyze changes of CSRF handling in Spring Security 6 and potential implications for DS 8

Problem description
DSpace 8 comes with an update of Spring Security (from version 5 to 6). Spring Security 6 contains some changes in the CSRF handling that are listed here: https://docs.spring.io/spring-security/reference/servlet/exploits/csrf.html#migrating-to-spring-security-6
The changes in Spring Security result in a change of the behaviour of the DSpace REST API.
In DSpace 7.6.2 (and below) the response of a GET request on (almost) every REST endpoint contains a valid DSPACE-XSRF-TOKEN header. The value of this header can be used to create a successful POST request on /api/authn/login.
Example Request (on a running DS 7.6.2 instance):
curl -sS -D - 'https://api-depositonce.tu-berlin.de/server/api/' -o /dev/null

Excerpt from HTTP Response:
Set-Cookie: DSPACE-XSRF-COOKIE=b088b74c-9717-4c91-8371-688bbd6f0ff8; Path=/server; Secure; HttpOnly; SameSite=None
DSPACE-XSRF-TOKEN: b088b74c-9717-4c91-8371-688bbd6f0ff8

In DSpace 8 the response of this GET request does not contain the CSRF token and the Set-Cookie header with DSPACE-XSRF-COOKIE:
Example Request:
curl -sS -D - 'https://demo.dspace.org/server/api' -o /dev/null

In DSpace 7.6.2 a new REST endpoint was introduced to receive a valid CSRF token from the DS backend. The reponse of a GET request on /api/security/csrf contains a valid CSRF token in the DSPACE-XSRF-TOKEN header (both in DS 7.6.2 and DS 8.0):
curl -sS -D - 'https://demo.dspace.org/server/api/security/csrf' -o /dev/null

Excerpt from HTTP Response:
dspace-xsrf-token: 5e35e246-c942-4dc9-977c-40a2d4cc561c
set-cookie: DSPACE-XSRF-COOKIE=5e35e246-c942-4dc9-977c-40a2d4cc561c; Path=/server; Secure; HttpOnly; SameSite=None
set-cookie: DSPACE-XSRF-COOKIE=5e35e246-c942-4dc9-977c-40a2d4cc561c; Path=/server; Secure; HttpOnly; SameSite=None

(note that the Set-Cookie header is duplicated - this issue is addressed in #9773)
How to proceed futher?

Analyze the change in the DSpace API behaviour in DS 8 in more detail
Adapt the way Spring Security is used to create / manage CSRF tokens and undo the change of the behaviour of the DS 8 API
If this is not possible: add a note to https://github.com/DSpace/RestContract (and the DS 8 release notes) that there was a breaking change in the CSRF handling in DSpace 8 (this means that the implementation of DSpace API clients with respect to CSRF token handling has to be changed slightly)

Some discussion of this issue:
https://stackoverflow.com/q/74811673/2916377
https://stackoverflow.com/q/74447118/2916377

Needs a volunteer to investigate.  Definitely seems to be a side effect of the upgrade from Spring Security 5 to 6. As @saschaszott notes, we should either find a way to restore the older behavior (if easily doable) or document this as a major change in the REST Contract & Release Notes for 8.x.


Fix parameterized testing

Google has updated orchestrator to support parameterized tests. If this works, then we should update the readme with instructions for end users that they should update the androidx test dependency to run parameterized tests.

https://github.com/android/android-test/releases/tag/androidx-test-1.3.0-beta02
https://github.com/android/android-test/issues/215

I find it really interesting that we've been running parametrised tests on flank recently without problems. Both in AS and on FTL using flank.

I confirmed this is currently broken in Flank.
https://github.com/Flank/flank/pull/786
https://github.com/bootstraponline/flank_bug

@bootstraponline Since ~3 months flank is using manual sharding (beta option), so correct way to compare gcloud with flank is:
gcloud beta firebase test android run \
  --type instrumentation \
  --app app-single-success-debug.apk \
  --test app-single-success-debug-androidTest.apk \
  --device model=NexusLowRes,version=28,locale=en,orientation=portrait \
  --test-targets-for-shard "class com.example.test_app.InstrumentedTest#test"

with this arguments gcloud gives error too.

But it works for
  --test-targets-for-shard "class com.example.test_app.InstrumentedTest"

So looks like we cannot shard tests methods from parametrized tests. I hope that is root of problem.

I have the same problem when running parametrized tests locally in IDE.
Please correct me if I'm wrong, as I haven't check in source code, but it
seems to me that Parametrized runner can't launch specific test method - it
can only 'work' with test class being launched.
On Mon, 8 Jun 2020 at 16:48, Jan Góral<EMAIL_ADDRESS>wrote:

But it works for
--test-targets-for-shard "class com.example.test_app.InstrumentedTest"
So looks like we cannot shard tests methods from parametrized tests. I
hope that is root of problem.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Flank/flank/issues/809#issuecomment-640677140, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSXSGPNPEBMTN3RT33LFQDRVT24NANCNFSM4NGNRFKQ
.

I have the same problem when running parametrized tests locally in IDE.
Please correct me if I'm wrong, as I haven't check in source code, but it
seems to me that Parametrized runner can't launch specific test method - it
can only 'work' with test class being launched.

That's right. Now, we know the source of problem, so it's time for implementation.

@bootstraponline I see next problem. To fix problem with parameterized tests we need to recognize parameterized classes. Unfortunately DexParser not returns any info about test classes.

@jan-gogo This is awesome progress. Let's document the current status of parameterized testing in Flank and update the readme with this information.

gcloud:
  app:  ./app-single-success-debug.apk
  test: ./app-single-success-debug-androidTest.apk
  use-orchestrator: false

flank:
  disable-sharding: true

Since ~3 months flank is using manual sharding (beta option), so correct way to compare gcloud with flank is:

@jan-gogo when sharding is disabled (disable-sharding: true) I don't think we should be using manual sharding. We should be able to get the same result as gcloud firebase test android run flank.yml:gcloud when sharding is disabled in Flank.

That's right. for disable-sharding: true flank gives success on output
AndroidArgs
    gcloud:
      results-bucket: test-lab-v9cn46bb990nx-kz69ymd4nm9aq
      results-dir: 2020-06-08_23-46-26.386000_brfl
      record-video: false
      timeout: 15m
      async: false
      client-details:
      network-profile: null
      results-history-name: null
      # Android gcloud
      app: gs://test-lab-v9cn46bb990nx-kz69ymd4nm9aq/2020-06-08_16:40:58.988069_fIhy/app-single-success-debug.apk
      test: gs://test-lab-v9cn46bb990nx-kz69ymd4nm9aq/2020-06-08_16:40:58.988069_fIhy/app-single-success-debug-androidTest.apk
      additional-apks:
      auto-google-login: false
      use-orchestrator: false
      directories-to-pull:
      other-files:
      performance-metrics: false
      num-uniform-shards: null
      test-runner-class: null
      test-targets:
      robo-directives:
      robo-script: null
      device:
        - model: NexusLowRes
          version: 28
          locale: en
          orientation: portrait
      num-flaky-test-attempts: 0

    flank:
      max-test-shards: 1
      shard-time: -1
      num-test-runs: 1
      smart-flank-gcs-path:
      smart-flank-disable-upload: false
      files-to-download:
      test-targets-always-run:
      disable-sharding: true
      project: flank-open-source
      local-result-dir: results
      full-junit-result: true
      # Android Flank Yml
      keep-file-path: false
      additional-app-test-apks:
      run-timeout: -1
      legacy-junit-result: false
      ignore-failed-tests: false
      output-style: single

RunTests
  1 test / 1 shard

  1 matrix ids created in 0m 4s
  https://console.developers.google.com/storage/browser/test-lab-v9cn46bb990nx-kz69ymd4nm9aq/2020-06-08_23-46-26.386000_brfl

Matrices webLink
  matrix-2u7m5d516tkva https://console.firebase.google.com/project/flank-open-source/testlab/histories/bh.da0c237aaa33732/matrices/8385546085876157767/executions/bs.c7bedd3fccf3419

  2m 23s Test executions status: FINISHED:1
  2m 23s matrix-2u7m5d516tkva FINISHED

FetchArtifacts
  .
  Updating matrix file

CostReport
  Virtual devices
    $0.02 for 1m

MatrixResultsReport
  1 / 1 (100.00%)
More details are available at [https://console.firebase.google.com/project/flank-open-source/testlab/histories/bh.da0c237aaa33732/matrices/8385546085876157767]
┌─────────┬────────────────────────┬────────────────────┐
│ OUTCOME │    TEST_AXIS_VALUE     │    TEST_DETAILS    │
├─────────┼────────────────────────┼────────────────────┤
│ success │  matrix-2u7m5d516tkva  │                    │
└─────────┴────────────────────────┴────────────────────┘
  Uploading JUnitReport.xml .

Process finished with exit code 0

also on current master.

We are tracking current progress here:
https://github.com/adamfilipow92/flank_parametrized_tests/tree/master/test_apk


Replace (regression-)plot

(How) Can I replace the node-plots from dtreeviz by a custom plot function from me?
So I opened a stackoverflow-question yesterday but it seems to be ill-posed (as I only got 10 views until now), so I decided to bother here directly.
Best regards

unfortunately there's no way for you to pass in a custom plot. You would have to alter the software.


Fix CIDR validator

https://regex101.com/r/kvnWQF/1
Signed-off-by: Luca Carrogu<EMAIL_ADDRESS>Please See Git Pull Request Instructions
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1794 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1794   +/-   ##
========================================
  Coverage    52.97%   52.97%
========================================
  Files           24       24
  Lines         3981     3981
========================================
  Hits          2109     2109
  Misses        1872     1872

Impacted Files
Coverage Δ

cli/pcluster/config/mappings.py
100.00% <ø> (ø)

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 c3bec08...3aef78f. Read the comment docs.


Detecting mouse button in onClick

After #51 one can detect which mouse button was used when dragging. However this is not the case for onMouseDown or onClick. How would one go about detecting mouse button in that case?

Well, there is already a way to get the button with customMouseConfig. The events onMouseDown and onClick are fired after that, so you already have access to the button (you must keep in your model).


update CIL framework to point to current master

Both this PR and the master seem to be in the same state. Travis is happy for all builds except Docker. The Docker errors look unrelated to this PR.

These are the same jobs that were failing earlier since your previous PR, which flipped the situation between the 3rd job failing, to 3 others failing.
I think the docker builds are the currently the only ones that try to build CIL (as @rijobro disabled the direct tests).
https://travis-ci.org/github/CCPPETMR/SIRF-SuperBuild/jobs/663922430#L1584
still says
CMake Error: The source directory "/opt/SIRF-SuperBuild/sources/CCPi-Framework" does not appear to contain CMakeLists.txt.

It's hard to say what's going on as the output of different dependencies are overlapping. But it looks like it's getting old CCPi-Framework commit (9bd69b7 fix MixedL21Norm (#417), this line).
Is it possible that Docker uses the tags differently? Or caches them? Otherwise, I don't really see where I could have gone wrong (desired commit was adf4163).

@rijobro please add this TAG for CCPi-Astra 8c2493710f5eac1316945585fe6333ab56f9a2a1

Done.
Presumably, it still doesn't solve the Docker issue though. @KrisThielemans , @casperdcl the Docker must be working on a different logic for checking out branches?

I vaguely recall that the docker image just gets updated (as opposed to created from scratch at every build), which would make a lot of sense. So we might be hitting something like #257

I vaguely recall Docker only updating once something was in master? I'll merge this and see if it solves that problem.

If you want to test docker in PRs you'll need to temporarily update https://github.com/CCPPETMR/SIRF-SuperBuild/blob/c5b52b21661d5997a76528124c317eadd5ce21ec/docker/user_sirf-ubuntu.sh#L6


Fix Travis CI build

Changed log

Merge PR #23 and #24.
Resolves issue #25.
Fix Travis CI build.
Upgrade the chromedriver to latest stable version.
And it also installs the latest stable google-chrome browser version.
Add Node.js 12 version test during Travis CI build.

@gauntface, if possible, could you take a look at this PR?
And hopefully this PR will be merged and release new version :).