[DOC] Links in documentation stop working after a few seconds

Documentation issue
Links within the documentation pages stop working after a few seconds.
Example page where this is happening:
https://refine.dev/docs/tutorial/getting-started/antd/generate-crud-pages/
Here's a loom video of the issue:
https://www.loom.com/share/3499962ba7ae4ecd92fe8d2525145e41?sid=38d036ba-4dee-4f13-bbdd-e922d626b8fc
Describe the thing to improve
Fix the bug where links become unclickable in the documentation
Describe the solution (optional)
No response

Hey @axertion, sorry for the issue! I've just sent a fix for the issue, turns out there's an overlay that captures the pointer events 🤦 It will be deployed in short time.

thank you @aliemir!


feat: added new src prop, for inserting SVG files into <Icon>

fixes #48

:tada: This PR is included in version 0.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


disable squashing for travis tests

@britter looks like it was about squashing logic that might have caused problems on travis, i've disabled it in this PR.

:+1: Happy it works now!


[FEAT] Submit Task For Mentee

When a mentee is assigned a task, he will submit that task. To submit a task, a mentee will provide the following

Task URL
Comments(Rich Text)

Is it a list of tasks or just one task at a time?

One task at a time. Not a list of tasks.

Would it be more flexible to assign more than one task at a time? Either a single task or multiple tasks

It's to submit na. You shouldn't be able to submit multiple tasks at once.


Add unit tests for the Data Structures and algorithms

Hi @salman-bhai, I think it would be important to have unit tests for each of the data structure and algorithm implementations.
This would help improve the quality of the implementations since it would ensure the implementations actually work in the respective languages.

Go ahead, @alexkiura . Send in a Pull Request!

Submitted one for the python Stack data structure.


[SPARK-36721][SQL] Simplify boolean equalities if one side is literal

What changes were proposed in this pull request?
This PR proposes to improve simplifications of EqualTo/EqualNullSafe binary comparators when one side is a boolean literal.
For example: EqualTo(predicate, TrueLiteral) => predicate, EqualNullSafe(predicate, TrueLiteral) => And(predicate, IsNotNull(predicate))
This PR helps pushing down the filter and reducing unnecessary IO.
Why are the changes needed?
The following query does not push down the filter in the current implementation
SELECT * FROM t WHERE (a AND b) = true

although the following equivalent query pushes down the filter as expected.
SELECT * FROM t WHERE (a AND b)

That is because the first query creates EqualTo(And(a, b), TrueLiteral) that is simply not in the form that we can push down. However, we should be able to get it simplified to And(a, b)
It is fair for Spark SQL users to expect (a AND b) = true performs the same as (a AND b).
Does this PR introduce any user-facing change?
No
How was this patch tested?
Added unit tests
build/sbt "testOnly *BooleanSimplificationSuite  -- -z SPARK-36721"

cc @cloud-fan @sunchao @viirya @dbtsai

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47982/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47982/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47982/

Test build #143471 has finished for PR 34055 at commit 578bdce.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143471/

Test build #143493 has started for PR 34055 at commit 2e44a5c.

Test build #143493 has finished for PR 34055 at commit 2e44a5c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48004/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48004/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143493/

@viirya it would be great if you could take another look when you get a chance. Thank you!

retest this please

Test build #143543 has started for PR 34055 at commit 2e44a5c.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48052/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48052/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48052/

Test build #143543 has finished for PR 34055 at commit 2e44a5c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143543/

Test build #143556 has started for PR 34055 at commit f4976bb.

case a EqualTo TrueLiteral if a.dataType.isInstanceOf[BooleanType]

This is not necessary. The analyzer guarantees that the inputs of binary comparisons are of the same type. We can just write case a EqualTo TrueLiteral

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48065/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48065/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48065/

case a EqualTo TrueLiteral if a.dataType.isInstanceOf[BooleanType]

This is not necessary. The analyzer guarantees that the inputs of binary comparisons are of the same type, and optimizer will never hit an unresolved plan. We can just write case a EqualTo TrueLiteral

Thanks, updated @cloud-fan

Test build #143568 has started for PR 34055 at commit 58c9f7c.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48077/

Test build #143556 has finished for PR 34055 at commit f4976bb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143556/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48077/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48077/

Test build #143568 has finished for PR 34055 at commit 58c9f7c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143568/

Thanks all! Merging to master.

+1, LGTM, too.


altair: 6.2.0 -> 6.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/altair/versions.
meta.description for altair is: A feature-rich GraphQL Client IDE
meta.homepage for altair is: https://github.com/imolorhe/altair
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.3.0 with grep in /nix/store/hgakaw09gpgcin90pkzkpydwwajfzgjc-altair-6.3.0
found 6.3.0 in filename of file in /nix/store/hgakaw09gpgcin90pkzkpydwwajfzgjc-altair-6.3.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
altair

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hgakaw09gpgcin90pkzkpydwwajfzgjc-altair-6.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A altair https://github.com/r-ryantm/nixpkgs/archive/72f2751cd47d9080faa783fea7a94f16496a12b0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/72f2751cd47d9080faa783fea7a94f16496a12b0#altair

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hgakaw09gpgcin90pkzkpydwwajfzgjc-altair-6.3.0
ls -la /nix/store/hgakaw09gpgcin90pkzkpydwwajfzgjc-altair-6.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

altair

Maintainer pings
cc @evalexpr for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/altair/versions.
meta.description for altair is: A feature-rich GraphQL Client IDE
meta.homepage for altair is: https://github.com/imolorhe/altair
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.3.1 with grep in /nix/store/q5bbng5vbaf4kj8l3wv956i71fb9jwaz-altair-6.3.1
found 6.3.1 in filename of file in /nix/store/q5bbng5vbaf4kj8l3wv956i71fb9jwaz-altair-6.3.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
altair

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q5bbng5vbaf4kj8l3wv956i71fb9jwaz-altair-6.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A altair https://github.com/r-ryantm/nixpkgs/archive/834815bfac7f2db1c04814a4c902119f399cba65.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/834815bfac7f2db1c04814a4c902119f399cba65#altair

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q5bbng5vbaf4kj8l3wv956i71fb9jwaz-altair-6.3.1
ls -la /nix/store/q5bbng5vbaf4kj8l3wv956i71fb9jwaz-altair-6.3.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

altair

Maintainer pings
cc @evalexpr for testing.


freeciv: 2.4.0 -> 2.5.0

Update freeciv version to 2.5.0.
Remove the gtk switch as it is expected to be present by default.

So it's intentional that the sdl version depends on gtk2 as well?

So it's intentional that now the sdl version depends on gtk2 as well?

Yes, I tried to build without the gtk client, but the configure script does not pass and there is no flag to prevent the lookup for gtk.

The configure setting --enable-fcmp=cli disables the GTK interface of an optional component. Both clients gui (GTK3 and SDL) still exist and are different: it seems worth it to keep them both.

Our expression uses gtk2, not gtk3. BTW, the 2.5 news claims there's a new Qt interface, but no need for us to add it now.

@vbgl , Thanks, I tested and it compiles fine, even with gtk disabled.
@vcunat , I noticed, and freeciv is able to compile either against gtk2 or gtk3, so this should be fine.
The Qt interface was already there in the version 2.4.0, and I did not tried it at the time.  I do not recall why, except that I packaged the SDL client because I found the interface nicer, and this is also the one used for the online version.

For myself, the SDL interface style is certainly preferable to the gtk2 one.


BLE failed to set new device name after reconnection (IDFGH-7995)

ENVIRONMENT:
<EMAIL_ADDRESS>-> framework-espidf @ 3.40300.0 (4.3.0)
BLE driver: Bluedroid (loaded dynamically)
DESCRIPTION
I need to change device name when advertising BLE and it is giving an error if the device was already advertising with other name. See full details below.

I have tried unregister esp_ble_gatts_app_unregister(gl_profile_tab[PROFILE_A_APP_ID].gatts_if); before registering again and no success
Disconection process before reconnection in same boot cylce:
    esp_ble_gap_stop_advertising();

    esp_bluedroid_disable();
    esp_bluedroid_deinit();

    esp_bt_controller_disable();
    esp_bt_controller_deinit();

    esp_ble_gatts_app_unregister(gl_profile_tab[PROFILE_A_APP_ID].gatts_if);

Instead of setting my new name, the "ESP32" name is set.
The name set is done inside handler:

Am I missing sth ?
Regards, Ramon.

The error code means that your argument is invalid, please check your argument not NULL and the length of argument no more than CONFIG_BT_MAX_DEVICE_NAME_LEN.
The device name in advertising is included in advertising data or scan response data. So if you want to change the device name when advertising, you don't need to reboot or stop advertising. Just follow the steps below:
/* set device name */
esp_ble_gap_set_device_name("device name");
/* if adv data include name */
esp_ble_gap_config_adv_data(&adv_data);
/* or if scan rsp data include name */
esp_ble_gap_config_adv_data(&scan_rsp_data);

Hi,
Thanks for your helpful answer.
You were right, I was passing a longer device name (37 bytes) due to a non-initialized buffer in my side.
Regards, Ramon.


GEODE-4411 Normalize AttributesFactory to match other factories

createRegionAttributes renamed to create and returns value
AttributesFactory renamed to RegionAttributesFactory
AttributesFactory constructor takes value of RegionAttributes

Yeah there was a lot of search and replace.  Good points for var and auto, we'll take another pass and try to nab some of those since that wasn't at the top of mind with the other changes.


Add missing locales for scrollBackToContent

This PR adds missing locales for the Scroll Back To Content button.
I'll leave this open, feel free to add your language's locale.

Btw, does anyone know why are we keeping two different locales(public/locales, src/locales)?

No clue. Those files were moved in https://github.com/excalidraw/excalidraw/pull/638/files#diff-a8f1efebab5ad72e17a5dfd80776c157, but it seems it kept the original files FSR. Let's remove them now.

I will then remove them.


How to create the Header File to work with LIB file(dilb.lib)

we can generate a LIB file with the help of CMake.
But if i want to import the LIB file into my VS project ,i must use a Header file to work with LIB file.
So,the question is how to create a header file ?

This question makes no sense.  The dlib download comes with all the header files.  You don't "make them".  You need to either learn to use visual studio and how C++ works, or, use cmake the way the examples show.  Read the instructions here http://dlib.net/compile.html.  They tell you how to compile the examples.  Then read the example's CMakeLists.txt file.   It's a tutorial.  It's very easy to use cmake to setup visual studio.

http://dlib.net/examples/CMakeLists.txt.html


feat: env vars should be readonly

Closes #108

Should we also Object.freeze something?

Personally I'm not a fan of Object.freeze.
Pros

Ensures that the readonly state persists into the runtime, making the object even safer.

Cons

Object.freeze is superfluous in TypeScript-only applications. And we don't ever expect that someone would be exporting their application's environment variables to some kind of JavaScript-only consumer, so the runtime safety is truly pointless.
Prevents live runtime debugging using the Chrome development console.

Overall it seems like the cons outweigh the pros. Also note that there is no performance benefit to doing this.

im not a fan of committing commented out code into a repository, can I remove this please?

im not a fan of committing commented out code into a repository, can I remove this please?

Nah I want it as a reference to why the behavior is the way it is if we come back to it in the future


Add attribute for cwd for docker_image

Imagine an use case like this:
git "#{Chef::Config[:file_cache_path]}/node-app" do
  repository 'https://gist.github.com/3b8ae2fd16f46acadea9.git'
  revision 'master'
  action :sync
end

docker_image "node-app" do
  tag "latest"
  source "#{Chef::Config[:file_cache_path]}/node-app/Dockerfile"
  action :build
end

Dockerfiles contains:
ADD app.js /var/www/app.js

fails with:
==> default: Step 12 : ADD app.js /var/www/app.js
==> default: STDERR: time="2015-04-03T22:24:13Z" level="info" msg="app.js: no such file or directory"
==> default: ---- End output of docker build  --tag="node-app:latest" - < /var/chef/cache/node-app/Dockerfile ----
==> default: Ran docker build  --tag="node-app:latest" - < /var/chef/cache/node-app/Dockerfile returned 1
==> default: [2015-04-03T22:24:13+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

I just ran into this same bug.  Eventually determined that it is due to the way docker build is being called:
==> default: Ran docker build  --tag="node-app:latest" - < /var/chef/cache/node-app/Dockerfile returned 1
is using stdin instead of a Dockerfile, which disables ADD, as documented here:
https://github.com/docker/docker/issues/841
and here
https://github.com/docker/docker/pull/845
.

Would it seem reasonable to cd to the directory and call docker build .? Will take a deeper look at this after 0.37.0 release.

Yes, this is the common workflow for building an image.

There is a native way to build containers using the chef lwrp, however it
currently has a bug.  This is a temporary workaround to just get things
working.
On Mon, Apr 13, 2015, 11:36 PM Giovanni Toraldo<EMAIL_ADDRESS>wrote:

Yes, this is the common workflow for building an image.
—
Reply to this email directly or view it on GitHub
https://github.com/bflad/chef-docker/issues/291#issuecomment-92648097.

Disregard that last message... Thought this was my private repo.
On Tue, Apr 14, 2015, 6:04 AM shaun<EMAIL_ADDRESS>wrote:

There is a native way to build containers using the chef lwrp, however it
currently has a bug.  This is a temporary workaround to just get things
working.
On Mon, Apr 13, 2015, 11:36 PM Giovanni Toraldo<EMAIL_ADDRESS>wrote:

Yes, this is the common workflow for building an image.
—
Reply to this email directly or view it on GitHub
https://github.com/bflad/chef-docker/issues/291#issuecomment-92648097.

ping ? It's an annoying issue, it's impossible to build an image with a context (using ADD) with the cookbook as it is now. If it is, what's the  good way to achieve this?

This is docker_image has been reimplemented in master and will be in the 1.0.0 release

Is this still an issue for anybody else? Running v2.11.1 and it seems COPY isn't locating a file in the same directory as the Dockerfile specified in the source property.
├── files
│   └── default
│       ├── db
│       │   ├── create-user.sql
│       │   └── Dockerfile
├── metadata.rb
├── recipes
│   └── default.rb

default.rb
directory "#{Chef::Config[:file_cache_path]}/docker/db" do
  owner 'root'
  recursive true
end

cookbook_file "db_dockerfile" do
  source "db/Dockerfile"
  path "#{Chef::Config[:file_cache_path]}/docker/db/Dockerfile"
  group "root"
  owner "root"
  mode 0440
end

cookbook_file "create_user_sql" do
  source "db/create-user.sql"
  path "#{Chef::Config[:file_cache_path]}/docker/db/create-user.sql"
  group "root"
  owner "root"
  mode 0440
end

docker_service 'default' do
  action [:create, :start]
end

docker_image 'db' do
  source "#{Chef::Config[:file_cache_path]}/docker/db/Dockerfile"
  action :build
end

Dockerfile
FROM mariadb

COPY create-user.sql /schema/000-create-user.sql

And this yields the following stacktrace:
       Compiling Cookbooks...
       Converging 5 resources
       Recipe: guacamole::default
         * directory[/tmp/kitchen/cache/docker/db] action create (up to date)
         * cookbook_file[db_dockerfile] action create (up to date)
         * cookbook_file[create_user_sql] action create (up to date)
         * docker_service[default] action create
           * docker_installation_script[default] action create
             * apt_package[curl] action install (up to date)
             * execute[install docker] action run (up to date)
       (up to date)
            (up to date)
         * docker_service[default] action start
           * docker_service_manager_systemd[default] action start
             * directory[/usr/lib/docker] action create (up to date)
             * template[/usr/lib/docker/docker-wait-ready] action create (up to date)
             * template[/lib/systemd/system/docker.service] action create (up to date)
             * template[/lib/systemd/system/docker.socket] action create (up to date)
             * template[/etc/systemd/system/docker.service] action create (up to date)
             * template[/etc/systemd/system/docker.socket] action create (up to date)
             * execute[systemctl daemon-reload] action nothing (skipped due to action :nothing)
             * service[docker] action enable (up to date)
             * service[docker] action start (up to date)
       (up to date)
            (up to date)
         * docker_image[db] action build

           ================================================================================
           Error executing action `build` on resource 'docker_image[db]'
           ================================================================================

           Docker::Error::UnexpectedResponseError
           --------------------------------------
           Couldn't find id: {"stream":"Step 1 : FROM mariadb\n"}
           {"stream":" ---\u003e 05a32751fb4d\n"}
           {"stream":"Step 2 : COPY create-user.sql /schema/000-create-user.sql\n"}
           {"errorDetail":{"message":"lstat create-user.sql: no such file or directory"},"error":"lstat create-user.sql: no such file or directory"}

           Cookbook Trace:
           ---------------
           /tmp/kitchen/cache/cookbooks/docker/libraries/helpers_image.rb:21:in `build_from_dockerfile'
           /tmp/kitchen/cache/cookbooks/docker/libraries/helpers_image.rb:50:in `build_image'
           /tmp/kitchen/cache/cookbooks/docker/libraries/docker_image.rb:42:in `block (2 levels) in <class:DockerImage>'
           /tmp/kitchen/cache/cookbooks/docker/libraries/docker_image.rb:41:in `block in <class:DockerImage>'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/provider.rb:132:in `instance_eval'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/provider.rb:132:in `compile_and_converge_action'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:78:in `run_action'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:106:in `block (2 levels) in converge'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:106:in `each'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:106:in `block in converge'
           /tmp/kitchen/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:105:in `converge'

           Resource Declaration:
           ---------------------
           # In /tmp/kitchen/cache/cookbooks/guacamole/recipes/default.rb

            32: docker_image 'db' do
            33:   source "#{Chef::Config[:file_cache_path]}/docker/db/Dockerfile"
            34:   action :build
            35: end

           Compiled Resource:
           ------------------
           # Declared in /tmp/kitchen/cache/cookbooks/guacamole/recipes/default.rb:32:in `from_file'

           docker_image("db") do
             action [:build]
             retries 0
             retry_delay 2
             default_guard_interpreter :default
             declared_type :docker_image
             cookbook_name "guacamole"
             recipe_name "default"
             source "/tmp/kitchen/cache/docker/db/Dockerfile"
             repo "db"
             tag "latest"
             rm true
             connection #<Docker::Connection:0x00000004318db0 @url="unix:///", @options={:socket=>"/var/run/docker.sock", :read_timeout=>120}>
           end

       Running handlers:
       [2016-12-06T18:11:17+00:00] ERROR: Running exception handlers
       Running handlers complete
       [2016-12-06T18:11:17+00:00] ERROR: Exception handlers complete
       Chef Client failed. 0 resources updated in 03 seconds
       [2016-12-06T18:11:17+00:00] FATAL: Stacktrace dumped to /tmp/kitchen/cache/chef-stacktrace.out
       [2016-12-06T18:11:17+00:00] ERROR: docker_image[db] (guacamole::default line 32) had an error: Docker::Error::UnexpectedResponseError: Couldn't find id: {"stream":"Step 1 : FROM mariadb\n"}
       {"stream":" ---\u003e 05a32751fb4d\n"}
       {"stream":"Step 2 : COPY create-user.sql /schema/000-create-user.sql\n"}
       {"errorDetail":{"message":"lstat create-user.sql: no such file or directory"},"error":"lstat create-user.sql: no such file or directory"}

       [2016-12-06T18:11:18+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 1 actions failed.
>>>>>>     Converge failed on instance <default-ubuntu-xenial64>.  Please see .kitchen/logs/default-ubuntu-xenial64.log for more details
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration

As a temporary workaround, compressing my COPY file and Dockerfile into a tar archive and building from that seems to work.

Confirm, have the same issue.


Current version of sass-loader doesn't support Vue

Version
3.11.0
Environment info
Environment Info:

  System:
    OS: Linux 4.19 Manjaro Linux undefined
    CPU: (4) x64 Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz
  Binaries:
    Node: 12.9.1 - /usr/bin/node
    Yarn: Not Found
    npm: 6.11.2 - /usr/bin/npm
  Browsers:
    Chrome: Not Found
    Firefox: 69.0.1
  npmGlobalPackages:
    @vue/cli: 3.11.0

Steps to reproduce
sass-loader 8.0.0 doesn't work with the current version of Vue.js and leads to a compiler error. 7.3.1 works fine. I thought it should be mentioned as the documentation refers the latest version.
What is expected?
Vue should run fine
What is actually happening?
Module build failed: TypeError: this.getResolve is not a function
at Object.loader (/PROJECT/node_modules/sass-loader/dist/index.js:52:26)

This has already been fixed in the 4.0.0-rc3, not sure if it can be backported without breaking changes.


#1476 fix. cleareLimit(), cleareOffset()

Can be reopened if fixed.

@elhigu

the same method name implemented twice
Stupid error(((

Oracle error:
  1 failing
  1) oracle | oracledb
       unions
         handles unions with a raw query:
     select * from "accounts" where "id" = :1 union select * from accounts where id = :2 - ORA-00942: table or view does not exist
  Error: ORA-00942: table or view does not exist

https://github.com/tgriesser/knex/pull/2869

@Fi1osof Ignore that error, same failure can be observed in master as well, there are known problems with Oracle.


corporate: Fix UnboundLocalError exception in sponsorship view.

We ran into a bug in production caused by two issues:

Some users came from orgs that didn't have a website and since
the URL field was required, they submitted invalid URLs.
We didn't properly respond to invalid form submissions, which
led to UnboundLocalError exceptions in another part of the
code.

This commit solves this by doing the following:

We now allow blank URLs and have a convenient placeholder text
label that tells users that they may leave the URL field blank.
This commit refactors the code such that invalid form submissions
result in an informative error message about what exactly went
wrong.

@timabbott FYI :)

Screenshot:

Merged, thanks @eeshangarg!


2 resources of snowflake_table_grant with same privileges deletes each other.

Provider Version
0.25.4
Terraform Version
0.15.4 and 0.13.0
Describe the bug
At first i worked with sets of permission to grant to different roles different privileges but some will obviously be the same, so i narrowed it down to this problem, incase granting a privilege i.e SELECT to 2 roles in different snowflake_table_grant they will be created but in the second apply they will get deleted
Expected behavior
Nothing should be changed.
Code samples and commands
snowflake_table_grant.bigbrain_dev_raw_grant_full will be updated in-place
  ~ resource "snowflake_table_grant" "bigbrain_dev_raw_grant_full" {
        database_name     = "bigbrain_dev"

        id                = "bigbrain_dev|raw||SELECT|false"
        on_future         = true
        privilege         = "SELECT"
      ~ roles             = [
          + "SFULL_BIGBRAIN_DEV",
        ]
        schema_name       = "raw"
        with_grant_option = false
    }

  snowflake_table_grant.bigbrain_dev_raw_grant_read_only will be updated in-place
  ~ resource "snowflake_table_grant" "bigbrain_dev_raw_grant_read_only" {
        database_name     = "bigbrain_dev"
        id                = "bigbrain_dev|raw||SELECT|false"
        on_future         = true
        privilege         = "SELECT"
      ~ roles             = [
          + "SR_BIGBRAIN_DEV",
        ]
        schema_name       = "raw"
        with_grant_option = false
    }

Plan: 0 to add, 2 to change, 0 to destroy.

After the second apply:
snowflake_table_grant.bigbrain_dev_raw_grant_full will be updated in-place
  ~ resource "snowflake_table_grant" "bigbrain_dev_raw_grant_full" {
        database_name     = "bigbrain_dev"
        id                = "bigbrain_dev|raw||SELECT|false"
        on_future         = true
        privilege         = "SELECT"
      ~ roles             = [
            "SFULL_BIGBRAIN_DEV",
          - "SR_BIGBRAIN_DEV",
        ]
        schema_name       = "raw"
        with_grant_option = false
    }

  snowflake_table_grant.bigbrain_dev_raw_grant_read_only will be updated in-place
  ~ resource "snowflake_table_grant" "bigbrain_dev_raw_grant_read_only" {
        database_name     = "bigbrain_dev"
        id                = "bigbrain_dev|raw||SELECT|false"
        on_future         = true
        privilege         = "SELECT"
      ~ roles             = [
          - "SFULL_BIGBRAIN_DEV",
            "SR_BIGBRAIN_DEV",
        ]
        schema_name       = "raw"
        with_grant_option = false
    }

Plan: 0 to add, 2 to change, 0 to destroy.

Could be related to the fact that the ID is the same?
Additional context
The use case here is to create for example 1 role with read privileges (SELECT) and another role with read write (SELECT and UPDATE)

The way you use grants is to define one privilege per resource (table/database/schema/etc) and grant it to a list of roles. If you want to create a role with read privileges and another role with read write privileges you would do:
resource snowflake_table_grant read_grant {
  database_name = "database"
  schema_name   = "schema"
  table_name    = "table"

  privilege = "SELECT"
  roles     = ["read_priv_role", "read_write_priv_role" ]
}

resource snowflake_table_grant write_grant {
  database_name = "database"
  schema_name   = "schema"
  table_name    = "table"

  privilege = "UPDATE"
  roles     = ["read_write_priv_role"]
}

Related to #210

We are closing this issue as part of a cleanup described in announcement. If you believe that the issue is still valid in v0.89.0, please open a new ticket.


Alias add_offense with add_match

Is your feature request related to a problem? Please describe.
With autocorrection in use for code transitions the idea that the starting code is an 'offense' may not be accurate. It may be more accurately phrased as a match of the rule we want to correct for.
These are similar to codemods from EmberJS and a newer idea to Rubocop with autocorrection features:
https://github.com/ember-codemods/ember-test-helpers-codemod
The idea that the starting code is an 'offense' does not feel correct in these circumstances.
Describe the solution you'd like
An alias for add_offense: add_match
This would be done to add clarity for code modifications while maintaining support for current rules using add_offense.
I can write the code fairly quickly for this but would want to ask the core team about ideas on this in general.

With autocorrection in use for code transitions the idea that the starting code is an 'offense' may not be accurate. It may be more accurately phrased as a match of the rule we want to correct for.

To some extent you're right. Typically there's some target node pattern and we're looking for a match, but this also updates that @offenses array, which I assume lead me to name it like this in the early days (although today I would have probably used register_offense). Here's the code of the method:
      def add_offense(node, location: :expression, message: nil, severity: nil)
        loc = find_location(node, location)

        return if duplicate_location?(loc)

        severity = find_severity(node, severity)
        message = find_message(node, message)

        status = enabled_line?(loc.line) ? correct(node) : :disabled

        @offenses << Offense.new(severity, loc, message, name, status)
        yield if block_given? && status != :disabled
      end

I wouldn't mind an alias that adds more clarity, but I'm not sure whether add/register_match won't confuse some people instead.

@bbatsov are you still considering this?

I had totally forgotten about it. :-) As no one else voiced support for this I guess we are probably fine without an alias, but I'm still open to the idea. It kind of fits the spirit of Ruby (even if I've often criticized its excessive usage of aliases in the Core library).
If we do this we'll also have to document the use case for which the alias was intended.

Since there hasn't been a push for it, or really a specific use case, since the original request 2 years ago, I'm going to close this for now. It can be brought back up of course!


Problem with module name 'moment'

I try to use the momentjs library with flow. I got a .d.ts file from https://github.com/borisyankov/DefinitelyTyped/blob/master/moment/moment.d.ts but couldn't convince flow to accept the moment module.
I debugged the problem and discovered that the module name 'moment' causes the problem.
I have created a strip-down version of the problem.
$ tree -a
.
├── .flowconfig
├── jssrc
│   └── Sample.js
└── lib
    └── moment.d.ts

$ cat .flowconfig
[ignore]

[include]
./jssrc/

[libs]
./lib/moment.d.ts

[options]

$ cat jssrc/Sample.js
/* @flow */

var moment = require('moment'); // REPLACE 'moment' with 'foobar' to get it working

$ cat lib/moment.d.ts
declare module moment { // replace moment with foobar to get it working
}

If I now run flow, I get
$ flow check

jssrc/Sample.js:3:14,30: moment.js
Required module not found

If I replace the two occurrences of moment with foobar, I get
$ flow check

Found 0 errors

Very surprising. I think I saw a similar bug elsewhere. Will look into it, thanks for reporting!

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

Any news?

This repros:
mkdir -p test
pushd test
printf "declare module foobar {\n  declare var foo: string;\n}\n" > lib.d.ts
printf "[libs]\nlib.d.ts\n" > .flowconfig
printf "// @flow\n\n((require('foobar').foo): number);\n" > index.js
flow check
# got a missing module error, rather than complaining that foo is a string, not a number
popd

The problem is that Flow only looks at file extensions .js and .json, not .ts, even if the entry in [libs] is an actual file.


v1.6.0 release PR

Release PR for v1.6.0
All changes in here have been approved previously. Only real changes are the version bumps in the last commit

nf-core linting fails because it cannot handle nf-schema yet. This is to be fixed in the future


fixes a bug where you couldn't specify the same project with different storage regions

e.g.
repositories = [{
  project = "my-project"
  clean_all = true
  storage_region = "eu"
 }, {
  project = "my-project"
  clean_all = true
  storage_region = "asia"
}]

data 'external' no more exists.
Closing PR.


Auto-update go-utils to latest version

Beep boop I am a bot
This is an automatically created PR to change keptn/go-utils to version 46192ab68374c26a3591d53a4bb6f914d9be7ff0.

Argh, the bot for creating a new PR for go-utils needs to have a signed commit :)


Add coverage with nyc

See #103.
This is, I think, the most straightforward way of adding test coverage. There are some things that can be done to improve it:

Move nyc and/or mocha configuration outside the test script command (mocha can use mocha.opts, while nyc can use the package.json or a .nycrc file).
Integrate with Coveralls.
Check that the coverage is above some threshold.

@fvictorio great ideas. But what's the benefit of moving the configuration outside the test script?

It's easier to read and modify, editing the flags in the script can be cumbersome and error-prone. But right now it's not really necessary.


plugin name bug

Actual Behavior
See https://github.com/OpenFunction/builder/blob/main/cmd/go/of_functions_framework/main.go#L318-L318
if plg.IsDir() && strings.HasPrefix(plg.Name(), "plugin-")

https://github.com/OpenFunction/builder/blob/main/cmd/go/of_functions_framework/main.go#L329-L329
Since the above condition exists, this condition is always True.
if strings.Contains(plg.Name(), "-")

@tpiperatgod We cannot use - to split the plugins?
https://github.com/OpenFunction/builder/blob/main/cmd/go/of_functions_framework/main.go#L330

Is it possible to output errors to the log, and kill the entire process? instead of ignoring it.

plgFiles, err := ioutil.ReadDir(plgAbsPath)
if err != nil {
   continue
}

https://github.com/OpenFunction/builder/blob/main/cmd/go/of_functions_framework/main.go#L320-L323

This is indeed a bug, and the second if statement needs to be removed.

@tpiperatgod We cannot use - to split the plugins? https://github.com/OpenFunction/builder/blob/main/cmd/go/of_functions_framework/main.go#L330

Here we can skip the if statement and split the plugin's name directly.

Is it possible to output errors to the log, and kill the entire process? instead of ignoring it.
plgFiles, err := ioutil.ReadDir(plgAbsPath)
if err != nil {
   continue
}

https://github.com/OpenFunction/builder/blob/main/cmd/go/of_functions_framework/main.go#L320-L323

I don't think it's necessary to abort for failing to look up the plugin, but a log should be added...

We need to recreate openfunction/builder-go:v2-1.16 ?

Yes, I will update the default functions-framework-go version to v0.2.3 at the same time


Bitswap tracking

The bitswap want list should be indexed on the CID's Multihash not the CID.  A block can have multiple CIDs but only one Multihash.

GO and JS do not do this.  Let's not do this.


Put fabric8 mvn tooling in fabric8 profile

Fixes #30

Current coverage is 32.11% (diff: 100%)

Merging #35 into master will not change coverage

@@             master        #35   diff @@
==========================================
  Files            16         16
  Lines           218        218
  Methods           0          0
  Messages          0          0
  Branches         20         20
==========================================
  Hits             70         70
  Misses          135        135
  Partials         13         13

Powered by Codecov. Last update ebc7e08...ea971ca

thanks !
lgtm

[merge]


Add HTTPUnit bot - very common bot

This completes the PR found here: https://github.com/WhichBrowser/Parser-PHP/pull/107

Coverage remained the same at 99.952% when pulling e4c4fcdc6de22bca635360d7190ff6fa8bd20577 on ayumi-cloud:HTTPUnit into 880b9fa797401d14b28956442944c3daa70240ff on WhichBrowser:master.

@NielsLeenheer Please can you merge this pr.

Fixed in 8f8acf4575aa2558a5307f4cf8bed6ae133cd13b

Fixed in 8f8acf4575aa2558a5307f4cf8bed6ae133cd13b


error when starting dev server on v2.7.11

Describe the bug
Just upgraded to v2.7.11, on previous 2.7.10 all starts fine
Reproduction
need reproduction
System Info
System:
    OS: macOS 12.0
    CPU: (8) arm64 Apple M1
    Memory: 79.16 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.8.0 - ~/.nvm/versions/node/v16.8.0/bin/node
    Yarn: 3.1.0 - ~/.nvm/versions/node/v16.8.0/bin/yarn
    npm: 7.21.0 - ~/.nvm/versions/node/v16.8.0/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Browsers:
    Chrome: 97.0.4692.71
    Firefox: 88.0.1
    Safari: 15.0
  npmPackages:
    @vitejs/plugin-react: ^1.1.4 => 1.1.4
    vite: 2.7.11 => 2.7.11

Used Package Manager
yarn
Logs
error when starting dev server:
TypeError: Cannot read property 'replace' of undefined
    at normalizeId$1 (/node_modules/vite/dist/node/chunks/dep-48c3e8a4.js:3879:34)
    at optimizeDeps (/node_modules/vite/dist/node/chunks/dep-48c3e8a4.js:56534:34)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async runOptimize (/vite/dist/node/chunks/dep-48c3e8a4.js:60540:48)
    at async Http2SecureServer.httpServer.listen (/node_modules/vite/dist/node/chunks/dep-48c3e8a4.js:60556:21)

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Same here, i've started an open source project here, simply run npm run dev, that started slidev, a third part dependency that uses vite (latest version, so v2.7.11), and has this error.
System:
OS: macOS 12.1
CPU: (8) arm64 Apple M1
Memory: 8.00 GB
Shell: 5.8 - /bin/zsh

Yup. Just bumped into this issue now, was trying out vite for first time. Needed to install commonjs plugin.
And by that example, i needed to add plugin in optimizeDeps.esbuildOptions.plugins. But was getting the error.
error when starting dev server:
TypeError: Cannot read property 'replace' of undefined
    at normalizeId$1 (/Users/akhilmhdh/Akhi/one-platform/packages/feedback-spa/node_modules/vite/dist/node/chunks/dep-48c3e8a4.js:3879:34)
    at optimizeDeps (/Users/akhilmhdh/Akhi/one-platform/packages/feedback-spa/node_modules/vite/dist/node/chunks/dep-48c3e8a4.js:56534:34)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async runOptimize (/Users/akhilmhdh/Akhi/one-platform/packages/feedback-spa/node_modules/vite/dist/node/chunks/dep-48c3e8a4.js:60540:48)
    at async Server.httpServer.listen (/Users/akhilmhdh/Akhi/one-platform/packages/feedback-spa/node_modules/vite/dist/node/chunks/dep-48c3e8a4.js:60556:21)

Then i just applied without any plugin and noticed that, its still firing only when plugins is used inside esbuildOptions.
Here is my config file
import { defineConfig } from 'vite';
import react from '@vitejs/plugin-react';
import legacy from '@vitejs/plugin-legacy';
import { viteCommonjs, esbuildCommonjs } from '@originjs/vite-plugin-commonjs';

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [
    viteCommonjs(),
    legacy({
      targets: ['defaults', 'not IE 11'],
    }),
    react(),
  ],
  optimizeDeps: {
    esbuildOptions: {
      plugins: [esbuildCommonjs(['@patternfly/react-styles'])],
    },
  },
});

Downgraded to v2.7.0, and walla works perfectly. So some issue with v2.7.2 i think so.
To reproduce

Create a react app using vite cli
Just create an empty array inside optimizeDeps.esbuildOptions.plugins

Has same issue here.
After some investigation, it caused by a bug in mergeConfigRecursively when merging array values, which has been fixed in #6499
Workaround can be used before new version released:
optimizeDeps: {
  ...
  include: []    // add this
}

@recih change is in 2.7.12


fix(sec): upgrade golang.org/x/sys to 0.1.0

What happened？
There are 1 security vulnerabilities found in golang.org/x/sys v0.0.0-20220715151400-c0bba94af5f8

CVE-2022-29526

What did I do？
Upgrade golang.org/x/sys from v0.0.0-20220715151400-c0bba94af5f8 to 0.1.0 for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
How can we automate the detection of these types of issues?
By using the GitHub Actions configurations provided by murphysec, we can conduct automatic code security checks in our CI pipeline.
The specification of the pull request
PR Specification from OSCS

This is still relevant but seems like the build failed due to the Go version per the CI error.


[Feature request] Make editing options more accessible

Editing is such operations as Copy, Paste, Undo, Redo, Delete, Cut etc.
Editing with touch/pen input and/or software keyboard is hard, because all those options are only available either through context menu or through shortcuts.
I'd like to see Editing options accessible for touch/pen or similar input as always visible options.
I'd suggest using Command Bar for this: https://docs.microsoft.com/en-us/windows/uwp/design/controls-and-patterns/app-bars
Additionally that feature could be optional or even better automatically detect the input type and switch context.

100% agree on this: when pen input detected, command bar should be shown for quick access. It is nice to have but I will make it low piority for now until I solve other high piority issues.


[Feature] Pad search in lobby.

Added, waiting for proper CSS to be applied

Css is done, will be pushed soon.

The search results can be a bit funny.

are you talking about the search bar?

@uricorn sadly, those pads have special characters in their slugs and cannot be hidden because they cannot be targeted by jQuery. We will be changing the policy so these pads will have their keys revoked (probably)

@DerpgonCz, I meant that the results aren't always what you expect.
For instance, I search for my pad "AnimeMusic.me and Japanese Music", when I search "Anime" it finds it. When I search "Jap" or "Music" it doesn't show.

Hmm, strange, I will check it when I get back home

Looks like it's a case sensitivity issue to me.  If you type your search in the exact case of the room name, it works.  It should be case insensitive, obviously.

@ImGrumpy Yeah, I just arrived at the PC, it is fixed


Swap availability for #if check on Float16.

Xcode 12 on Catalina has Swift 5.3 but the macOS 10.15 SDK, which breaks the conditionals around Float16. Since the type is unconditionally unavailable on macOS, we can simply #if-out any Float16 extensions instead.

@lorentey yeah, step 2 here is definitely to figure out how to avoid this mess in the future =)


INTENG-3466 Fix for html tags in Dynamic links

URL encoded Base64 encoded link data that is generated when creating Dynamic links.
Exposed urlEncodedString from BNCEncodingUtils

Result of Integration 1
Duration: 7 minutes and 20 seconds
Result: Build failed 1 test out of 179

Result of Integration 2
Duration: 6 minutes and 46 seconds
Result: Build failed 1 test out of 184

looks good to me. Tests are failing though.

@ahmednawar I just fixed the SDK tests. waiting for Buildasaur

Result of Integration 3
Duration: 6 minutes and 42 seconds
Result: Perfect build! All 184 tests passed. :+1:
Test Coverage: 60%


Sending an order to a machine operator from a storage wormhole when far away from the storage accessor crashes the game

see https://github.com/klikli-dev/occultism/issues/268#issuecomment-873016756
https://pastebin.com/Rsay7pgH

I've updated and tried again. Crash still happens on 1.5.3
https://pastebin.com/NE3Y6j1A

It's a different crash now, which weirdly didn't happen for me.
I'll try to reproduce it though!

It could be related to the overkill distance I tested it from (10,000,000 blocks away from spawn)

Will be handled in #272


ci: fix ssj resources

Time spent on this PR:
Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Resolves #
What is the new behavior?
SSJ resources should pass

This change is

Should the requirement to run test be relaxed ?
@ClementWalter
Not sure this is useful to run those for this PR.


Make it usable in main process

Hi there,
This PR is for making it usable in main process, the current implementation is unable to work because --user-data-dir= is not exist in main process.
My changes are in resolveUserAppDataPath function:
function resolveUserAppDataPath() {
+  const userDataPath = app?.getPath("userData");
+  if (userDataPath) {
+    return userDataPath;
+  }

  const arg = process.argv.find((arg) => arg.startsWith("--user-data-dir="));
  if (!arg) {
    throw new Error(
      "Unable to find --user-data-dir with valid path in process.argv"
    );
  }

  ...
}

P.S The code in the diff view is a bit messy because we have lint conflict, let me know if you accept the idea and I will proceed to cleanup my code and help setup ESLint as well!
Thanks!

Thanks for your PR. I already described how use this in Main
https://github.com/cawa-93/electron-nano-store#in-main
It may be a little tricky but I would like to avoid a confusing mix of contexts. At least for now

Thanks for your response. I've tried but seems like there has a bug:
Here is my code:
defineStore<StoreData>(resolveStoreFilepath('test', app.getPath('userData')))

Here is the error:

Here is the bug:
https://github.com/cawa-93/electron-nano-store/blob/main/src/index.ts#L42
There is using basename(storeName) to compare with storeName. In my case, basename(storeName) is test.nano-store.json while storeName is the full path.

Also I'm wondering what's your thoughts behind this design? From my point of view, my instincts idea is using the same API for different secnarios.
Thanks and happy holiday :)

You probably import defineStore from wrong package.

Eh, here is my full of code:
import {app} from 'electron';
import {defineStore, resolveStoreFilepath} from 'electron-nano-store';
import {basename} from 'path';
import type {StoreData} from './type';

const storeName = resolveStoreFilepath('test', app.getPath('userData'));
console.log(storeName); // /Users/kexin.li/Library/Application Support/queryqueen/test.nano-store.json

console.log(basename(storeName)); // test.nano-store.json

defineStore<StoreData>(storeName);

// basename(storeName) !== storeName

I'm using electron-nano-store v0.3.0

In preload script, the usage is:
import {defineStore, resolveStoreFilepath} from 'electron-nano-store';
import {basename} from 'path';

defineStore<StoreData>('store')

// basename('store') === 'store'

Yeap. Wrong one. Check the docs again. In Main you should use directly fs-nano-store.

Understood, thanks.


Database initialization

Hi!
Could you please add in the readme instruction on how to initialize the database ?
Seems that

bin/console doctrine:migrations:migrate
is not enough

I receive the following exception when creating packages
SQLSTATE[42P01]: Undefined table: 7 ERROR: relation "messenger_messages" does not exist LINE 1: INSERT INTO messenger_messages (body, headers, queue_name, c... ^

Seems i was missing  bin/console messenger:setup-transports

This command is in both installation variants: Ansible and Docker, but I added it to README too :wink:  in #78


Cannot call method 'replace' of null

Hi William is your actvmile still working? Mine started giving an error after the time zone fix. Error is
Uncaught TypeError: Cannot call method 'replace' of null
at line 97 of js
return input.replace(/"/g, '"').replace(/'/g, '');
Any ideas?
From
http://www.dailymile.com/people/engmex

OK thanks.  Interestingly engough I can repost some older runs, but not new ones.  Seems like the information is not being passed from the Motoactv page to the js.  I've tried various browsers to no effect.

Today I did a 15km run and Actvmile worked perfectly.  Strange some runs sync ok and some don't and give  the error I posted above.  If you want access to my Motoactv account to check it out let me know.
Regards


sf/sfdx command to retrieve all package names

Is your feature request related to a problem? Please describe.
As part of a deployment pipeline, I use sf to download one or more change sets (up to 100 change sets at a time). The sf command will error when a change set doesn't exist, or the name is not passed correctly (due to special characters or spaces). The only way to confirm that a change set exists is to run the command "sf project retrieve start --package-name placeholder --json" and check the status code returned. This can take several minutes to run, depending on the size of a single change set. Downloading multiple large change sets (50 or more) can take up to an hour.
There also isn't an option to retrieve a list of all the change sets in an environment.
What are you trying to do
As part of a deployment pipeline the command "sf project retrieve start --package-name placeholder1 --package-name placeholder2 --package-name placeholder3...--package-name placeholder100 --json" will fail but only show the error for the first missing or misspelled change set breaking the pipeline and taking time to manually resolve. The only way to confirm all the change sets is to run the command independently for each change set, confirm that status, and add the failed change set to a list.

"sf project retrieve start --package-name placeholder1--json"
"sf project retrieve start --package-name placeholder2--json"
"sf project retrieve start --package-name placeholder3--json"...

"sf project retrieve start --package-name placeholder100--json"

Describe the solution you'd like
I would like two commands:

returns a list of all the change sets in an environment.
return if a change set or list of change sets exists.

Describe alternatives you've considered
The only way to confirm all the change sets exist is to run the command independently for each change set, confirm that status, and add the failed change set to a list.

"sf project retrieve start --package-name placeholder1--json"
"sf project retrieve start --package-name placeholder2--json"
"sf project retrieve start --package-name placeholder3--json"...
"sf project retrieve start --package-name placeholder100--json"

Additional context

I'd also like this feature; we do something similar on several of my projects.


cannot convert lambda expression to type 'EventCallback' because it is not a delegate type

currently preview9 EventCallbacks for onclick handlers do not work as expected:

@foreach (var project in _projects)
{

 SelectProject(project))">


Request: Always sort absolute imports before relative imports to comply with eslint-plugin-import by default

Hi!
First of all thanks for this module, as you know it provides a solid foundation for the Visual Studio Code extension.
I wanted to ask whether you would be open to adjusting the eslint (default) style for the parser to be in accordance with the default configuration of the popular eslint-config-airbnb. The idea is that absolute imports always come before relative imports. Right now, the first order of the default sort style looks at the import method (unnamed import, default import, etc.) instead of the import origin.
Example:
Current default eslint sorting:
import './foo';
import A from 'A';

Target default eslint sorting:
import A from 'A';
import './foo';

What do you think? It seems that import-sort-style-renke is already quite close to the wished behavior, but I think it might be worth putting it into the default style.

Hi @peterjuras
is this compatible with the default sort-imports rule from ESLint? Also, is this related to #17?

Hey,
the default sort-imports rule from ESLint and the import/first rule with absolute-first option from eslint-plugin-import are incompatible with each other, so i made https://github.com/alexanderbast/import-sort-style-absolute, which is compatible with both the StandardJS and Airbnb configs. :)

I had the same problem with the airbnb preset.
"style": "eslint" would list my import for global styled components at the top.
import './global-styles';

import { Route, Switch } from 'react-router';

import App from './containers/App';
import { BrowserRouter } from 'react-router-dom';
import MuiThemeProvider from 'material-ui/styles/MuiThemeProvider';
import React from 'react';
import injectTapEventPlugin from 'react-tap-event-plugin';
import { render } from 'react-dom';
...

and linting then fails with
...
   3:31  error  Absolute imports should come before relative imports  import/first
   5:30  error  Absolute imports should come before relative imports  import/first
...

@alexanderbast your sort style works perfect and fixes that; Thx a lot

I am closing this for now. I will have another look at the eslint style in the future, but custom plugins will probably be out of scope.

I'm not sure I understand what the resolution to this issue is? Is the cause a plugin to eslint? The automatic sort was working for me, but I just tried using react-boilerplate and now when I save files I get the linting error. I'm going to have to disable the sort-imports until I figure out a solution


TestRun.appendOutput() formats output in "Test: Show Output" badly if there are no carriage returns

Appending test output with online newlines does not render correctly when running the "Test: Show Output" command:
run.appendOutput("this is a\ntest1\ntest2\ntest3");

This is because it's rendered in the Terminal, and the Terminal requires \r to reset the position on each new line.
It feels like that Test: Show Output uses the terminal is an implementation detail, and callers of TestRun.appendOutput should not need to know about this. However, if its deemed by design, the docs for TestRun.appendOutput should call this out because it's very easy to be tripped up by this (for example, you test on Windows where the output of the test process includes \r\n then find your Mac/Linux users are all getting skewy output because the test process only output \n there).

Talking to Daniel, this is just a property of terminals, but I have documented this behavior alongside other information about appendOutput.


exported the keep_alive_while_idle config

Signed-off-by: Yu Juncen<EMAIL_ADDRESS>This is the patch for #42.

@davidli2010 PTAL


RHINENG-7061: add intersystems field to system_profile

RHINENG-7061
Inventory-schema PR at https://github.com/RedHatInsights/inventory-schemas/pull/122

The local test looks good with:
--------------
Rules Executed
--------------
[META] src.puptoo.process.profile.system_profile
------------------------------------------------

    iris                   : {'iris_running_instances': [{'instance_name': 'IRIS',
                                                          'product': 'IRIS',
                                                          'version': '2023.1'},
                                                         {'instance_name': 'IRIS2',
                                                          'product': 'IRIS',
                                                          'version': '2023.1'}],
                              'iris_system': True}

Get the confirmation of the field name changes in Jira :

Use "intersystems" instead of "iris" as field name.

Updated the code accordingly: (Amended the commit message to include the "intersystems" word.)

The local test looks good with:
    intersystems           : {'is_intersystems': True,
                              'running_instances': [{'instance_name': 'IRIS',
                                                     'product': 'IRIS',
                                                     'version': '2023.1'},
                                                    {'instance_name': 'IRIS2',
                                                     'product': 'IRIS',
                                                     'version': '2023.1'}]}

inventory-schema local test also passed, against the latest code in https://github.com/RedHatInsights/inventory-schemas/pull/122 .

Updated the code accordingly after the field name changes confirmation in Jira .
And local test looks fine: details at https://github.com/RedHatInsights/insights-puptoo/pull/316#issuecomment-1907878622
Hi @r14chandra , could you kindly help review the changes?

Hi @xiangce , could you also have a look at this update?


Multiple reports

Added the option to allow multiple reports.
Sample behat.yml:
        format:
            - html
            - clover
        output:
            html:
                target: public/coverage-behat
            clover:
                target: clover/behat.xml

I've left the original report option as is, so no breaking changes.

Hi,
Thank you for the PR. There are a few problems:

The code update is very messy in ReportService.php. The nested if/then/else is unclean and there should be a more clean way to handle this.
The code style is very different from the current coding style that project is using. I have made #25 so we have code style checks as part of the Travis.
The travis build for your update contains errors (the tests fail). The tests must be fixed.


[EXPORT] externalId of categories are not used when selected

When I use the categoryBy parameter with the value externalId for exports, the csv then contains the slugs of the default language and not the externalIds

This is fixed here https://github.com/sphereio/sphere-node-product-csv-sync/pull/145

Merged..


License for the repository

HI
This is an excellent work and I would like to carry out some experiments (non-commercial of course) using this repository. Could you please provide the license for the same.
Thanks

Added https://github.com/hongzimao/deeprm/blob/master/LICENSE

Thank you very much for your quick action. It will surely be a great help.


backwards roll

Allow the wheel to be rolled in reverse. The surveyed data should be incrementally shortened to reflect this.
Design questions:

What happens when a user rolls beyond the start of a surveyed feature that has not been completed? Is it deleted?
What happens when a user rolls beyond the end of a surveyed feature that has been completed?

https://github.com/sharedstreets/curb-wheel/tree/backroll


Models: Access to an undefined property

I find myself doing this a lot. What's the recommended approach to not having to ignore these?

There are a couple of ways to discover PHPStan these properties.
e.g. The @property tag in top of the class declaration.
Could you describe one? Where it comes from?

Please see  #175
and https://github.com/nunomaduro/larastan/issues/384#issuecomment-666344851

I'll got with the @property for the classes. Thank you.

You're welcome.

Normally you shouldn't need @property
Please put some code examples where these errors happens.
For Nova, you should put @mixin YourModel to your Nova resource class.

For Nova, I didn't like the @mixin, but type hinting works.
- URL::make('Stripe Id', fn () => "https://dashboard.stripe.com/subscriptions/{$this->stripe_id}")
+ URL::make('Stripe Id', fn (Model $subscription) => "https://dashboard.stripe.com/subscriptions/{$subscription->stripe_id}")
    ->textAlign('left')
    ->sortable()
    ->showOnPreview()
    ->displayUsing(fn (Model $subscription) => $subscription->stripe_id),

However for Eloquent Model classes, I need to add @property for relationships:
namespace App\Models;

/**
 * @property \App\Models\Company $company
 * @property \Illuminate\Database\Eloquent\Collection $licenses
 * @property \Illuminate\Database\Eloquent\Collection $subscriptions
 */
class Customer extends Model
{
    public function company()
    {
        return $this->belongsTo(Company::class);
    }

    public function subscriptions()
    {
        return $this->hasMany(Subscription::class);
    }

    public function licenses()
    {
        return $this->hasManyThrough(License::class, Subscription::class);
    }
}

@tillkruss Try this.
    public function subscriptions() : HasMany
    {
        return $this->hasMany(Subscription::class);
    }

See our tests: https://github.com/nunomaduro/larastan/blob/22e68b3f257eb10886c89e05a04f23aa1f480835/tests/Application/app/Group.php#L16

Yes, for relationships to be recognized you'd need to add return typehints for your relationships.
If you don't want to add them at least you should update your PHPDocs to include the generic types for the collection: @property \Illuminate\Database\Eloquent\Collection<int, \App\Models\Subscription> $subscriptions


Exclude certain resources

Can we use this: http://www.cocos2d-x.org/wiki/How_to_exclude_files_in_resources_directory_when_building_for_android
?
Specifically the iOS app included resolution files 1x, 2x and 3x, but since the Google Play Store only supports 50 MB app size, we consider deleting the 3x folder from the res folder. It looks like build-cfg.json from the method above would work, can it be used still or is there another way?

I added this to makefile all/release/distribution for now: find ./assets/Assets/res -name 3x | xargs rm -fr

It's entirely up to you what resolutions/devices you want to support. If your project does become too big, you can submit multiple .apk's as per this link in the Android README, meaning you don't necessarily need to delete all the highest res images. In addition, Android handles different resolution assets differently than iOS by putting the files for one intended resolution in one folder, while iOS looks for images with either no suffix, "@2x", or "@3x", depending on the size of the screen of the device the app is running on. However, we're getting out of the realm of what RapidGame is intended for. The template project that's provided is well within the size requirements for both app stores and is meant to be an example project that runs on all possible devices, meaning both phones, hi-res phones, and tablets.
Btw, all the command does is delete files with "3x" in them.

Ok I guess http://www.cocos2d-x.org/wiki/How_to_exclude_files_in_resources_directory_when_building_for_android cannot be used with RapidGame because it uses another way of building the app?

just FYI - The thing about 3x, 2x, 1x etc is not my question, just a side note to explain why I needed to exclude certain assets. The res folder includes sprite atlases (not individual sprite files) for each scene. The res includes 3x, 2x and 1x version of each atlas. The 3x was used for iPhone 6 Plus, the 2x for most other, 1x for iPad mini - this is for iOS of course - I designed the game for iOS in mind. I am using the same assets to speed up development. For high DPI Android screens I use 3x, for medium DPI 2x and so on. I don't use a hardcoded design size, rather each device uses its own native screen size, which is why I could not use the default setup of RapidGame. Of course I could have mistaken something but with my current setup should not have any screen ratio problems.

Hey! First off, sorry if my last comment sounded a bit negative. I had some other stuff going on that had me in a really bad mood.
Anyway, for your particular case, the link I sent you before should help you with your problem. You don't have to build a single .apk for your app. You can submit multiple .apk's associated as one app to the Play Store. So, if the higher res files are pushing your app over the 50 meg mark, you can for example exclude the highest res files from the first build and edit AndroidManifest.xml to reflect this fact, meaning this .apk will support phones and tablets with X resolution and below. Then, you can do the same thing but only build the .apk for high res devices and edit the same manifest appropriately.
And you are correct--the link you had to remove certain files from a build will not work with RapidGame. However, depending on your code and the API calls you make, you might be able to get away with simply moving the files you don't want included in the assets to a temporary directory before you run make distribution, so that only the files you want in that build are included. I forget exactly how different resolutions are handled in Android/Android NDK/Cocos2dx (haven't had a lot of free time recently), but you may want to get a better understanding of how it works, because it may turn out you have to do nothing more than that. Otherwise, you may need to refactor your code to better handle resolutions. But it is possible and a solution can probably be found and implemented within a reasonable amount of time.
So as long as you follow the rules of submitting multiples .apk's, it's just a matter of making at least 2 .apk's that both work well with their respective assets.

Ok got it. I will consider sending several asks. Thanks!


Name schema for plugins and extensions

Since most (if not all) of the projects within the node ecosystem are using this technique, I think we should adapt it in favour of making the experience consistent for all users:
hyper-<name>

What I'd like to do is modify the existing "plugins" array within the config file. This way, the user only needs to enter the name of the plugin and the string will automatically get prefixed behind the curtains and downloaded from npm:
plugins: [
  'links',
  'blink',
  'cwd'
]

Results in these packages getting installed:

hyper-links
hyper-blink
hyper-cwd

Completely agree with your analysis, and I would strongly prefer technique 1 :)
The only problem that has to be faced is about the existing plugins, since npm doesn't allow renaming published packages.

@lordgiotto Cool!
What we would do in this case is re-publish all existing packages under the new, correct name. And within HyperTerm itself, the old ones will simply continue to work until the whole community made the switch in their config.
How? I'd say checking if the plugin/theme name starts with "hyper", it simply won't be prefixed automatically it gets installed.

What about internal/private packages, not hosted on the official npm registry or on a npm registry at all. npm supports installing from git or arbitrary tarballs.
If there is auto-prepending of a hypterterm- prefix, there should also be a check for the other possibilities, I'd say.

@dak0rn Agreed! We'll only auto-prefix it if it's a normal package hosted on npm. So if it starts with a @ or contains a /, we'll just pass it through to npm.

@leo Maybe you should also add path.sep to the list, as the dependency can also be a file path.

@dak0rn All locals plugins should be located in ~/.hyperterm_plugins/local/. If you don't want to put your directory there, you can link it.

Sorry, I might have been a bit unclear. If I got the config correctly, then plugins is the list of npm dependencies that are installed into the very folder you mention. What I wanted so say is that npm is able to install from file locations, e.g. from a tarball on a shared network device, and if you want to support both prefixing and all possible npm install paths in plugins, you should add support for file paths. Therefore, path.sep because there's this fancy backslashy operating system.

@dak0rn Ahhh! Yea, I think that's actually a great idea. 😊

I think, though, that this is sufficient to flip the condition, that is, detecting plugin names that have to be prefixed.
const needsPrefix = str => !! str.match(/^(?!hyperterm-|hyper-)[\w-]+(#.+)?$/i);
Check this regex101 for an example.

I think it is a bad idea to mask plugin name.
It reminds me loader from webpack v1 when loader in name was optional in config file.
It can have some (rare) name collision. It can be tricky to understand why a revolution plugin is not loaded because a hyper-revolution exists.
Too few pros to make a difference between a plugin name and what to add in configuration imho.

It's also worth noting that atom does not do this.  I don't think that the standard is followed well enough for it to be relied on.

I think we agreed on not doing this


:zap: upgrade tightenco/collect package to support laravel 8

Thank you for the pr, we will review and merge it asap


fix(lanelet2_extensions): fix transverse mercator

Description
Fix Transverse Mercator Projector bug

Tests performed
Added a unit test

Effects on system behavior
None

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@mitsudome-r Hi, could you review this PR?


Remove librarians from process

Librarians.md was created as a solution to giving IMEC space to work on the running code repos in github.com/solid
In practice, this has been solved by creating a team https://github.com/orgs/solid/teams/imec
who work on specific repositories https://github.com/orgs/solid/teams/imec/repositories
so librarians.md was never implemented nor defined.

Assuming that the intention of this pull is to remove librarians.md - I think that makes sense. For posterity i'd recommend changing the title of this pull to reflect that (e.g. "Remove librarians from process").

PR closure, even by the original submitter, without merge and without comment, especially after meaningful and/or approving comment/discussion on that PR, is baffling to later readers like myself.  Such closure should always be accompanied by some explanation thereof.  Adding it now would be better than not adding it at all.


message_edit: Don't hide spinner on save success.

Hiding the spinner confusingly flashes the Save button before the edit view closes. This just prevents that from happening, so that the sign of success is the rendered, edited message.
CZO discussion
Screenshots and screen captures:
Spinner remains until the message-edit form closes:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

I think this removed line should at least be replaced by a comment explaining both this failure mode and also expressing how if you starting editing this message again, the button will be rendered from scratch and thus not be stuck with a stale loading spinner. It's very standard practice to hide loading spinners in both the success and error code paths, and so we should explain variations from that pattern.


Redefinition errors - Can't compile - CMAKE and VS2013

What steps will reproduce the problem?
1. Using CMAKE 3.0.2 with VS 12 2013 generator with 'build_shared_libs' off and
'cmake_static_linker_flags' added to be the same as the
'cmake_exe_linker_flags.'
2. Attempt ALL_Build in visual studio.
3.

What is the expected output? What do you see instead?
Instead of a successful ALL_Build I get multiple redefinition errors
(attached). int8_t, cbrt, etc etc

What version of the product are you using? On what operating system?
Using boost 1.56, CMAKE 3.0.2, VS 12 2013, windows 7, carve 1.4

Please provide any additional information below.
VS ALL_Build output attached:

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 3 Nov 2014 at 12:07
Attachments:

[ALL_BUILD output.txt](https://storage.googleapis.com/google-code-attachments/carve/issue-77/comment-0/ALL_BUILD output.txt)

can someone respond to that one ? I'm having the same problem


Description update

Remove "open source" from app descriptions
Remove "application that"

          3 Warnings

:warning:
Please rebase to get rid of the merge commits in this Pull Request

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

102
Dupe
https://github.com/xzzz9097/Muse

320
503
https://gitlab.com/inkscape/inkscape

  Generated by :no_entry_sign: Danger


Fix typo in doc of Partitioning

Which issue does this PR close?

Fix typo in doc of Partitioning
Closes #11593 .
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

PTAL @alamb.


SAK-44867 - Calendar Summary JSF upgrade

JSF upgrade for calendar-summary-tool
Needed to bring over the InputColorTag, InputColorComponent, InputColorRenderer from the jsf1 area

Codacy is complaining about some stuff, but I just copied an existing file into a new location.  I wasn't really looking to make changes to it!


python311Packages.google-cloud-container: 2.28.0 -> 2.29.0

Changelog: https://github.com/googleapis/python-container/blob/v2.29.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 248883 run on x86_64-linux 1

4 packages built:

python310Packages.google-cloud-container
python310Packages.google-cloud-container.dist
python311Packages.google-cloud-container
python311Packages.google-cloud-container.dist


eclipse

I don't have instructions on how to import it into Eclipse. The project was created in the JetBrains IntelliJ IDE.
Try to follow these instructions: https://github.com/collab-uniba/socialcde4eclipse/wiki/How-to-import-a-GitHub-project-into-Eclipse
Other than that, you just have to make sure that you have the dependencies functioning properly.


Asyrp Integration

Hi,
first thank you for your amazing work, it is straightforward to get it working!
However, one question, in line https://github.com/inbarhub/DDPM_inversion/blob/6b7344824449f56c667e953b3956a82960e49475/ddm_inversion/inversion_utils.py#L220 you mention Asyrp which is probably a reference to https://github.com/kwonminki/Asyrp_official - why is Asyrp not mentioned in your paper and why is the flag not used? I would love to integrate Asyrp with your work, but I have a hard time making Asyrp work.
Best,
Sidney

Hi,
We've incorporated it, though it hasn't been included in this paper. Integrating Asyrp can be seamlessly accomplished by retrieving the h_space from the UNET along with the inversion process.


BlinkApp → Blink

When setState is called, BlinkApp will re-render its Component. → After setState returns, Blink Component is re-rendered.

Thank you for your PR. I think the current text is good enough and changing it to the new version doesn't really help with readability, so let's keep it the way it is.

§1: When setState is called, BlinkApp will re-render its Component.

setState is called on Blink component. §1 doesn't clarify this.
Since setState is called on Blink component

react re-renders the Blink component
It is not the BlinkApp which re-renders its Component. BlinkApp is a component itself.
Thus §1 is not precise enough.


New method for StateDIff

TITLE
Description
solves issues #808

Codecov Report

Merging #809 (a9ae222) into main (f62e42e) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #809      +/-   ##
==========================================
- Coverage   93.17%   93.08%   -0.09%
==========================================
  Files          52       52
  Lines       12127    12158      +31
==========================================
+ Hits        11299    11317      +18
- Misses        828      841      +13

Impacted Files
Coverage Δ

src/state/mod.rs
94.96% <0.00%> (-4.34%)
:arrow_down:

... and 4 files with indirect coverage changes


SOLR-17516:  LBHttpSolrClient: support HttpJdkSolrClient

This is a minimal PR to add Load Balancing functionality for the HttpJdkSolrClient.  Included is test refactoring, adding the *JdkClient as a random choice for the integration test.
The result is not ideal:  We have two layers of abstract classes and 3 implementations.  Of the 3 implementations 1 is deprecated and the other two share all code.  If we really are getting rid of the deprecated client for 10.0, we could easily consolidate in the future.  A generic LB Solr Client is likely best but users may find it an annoying API change that only yields marginal improvement.

Instead of subclasses of LB....SolrClient, ...Cloud...SolrClient, ...Http...ClusterStateProvider, we can have concrete impls that delegate to a SolrClient that is provided.  It's a vision anyway; hopefully achievable?
CC @janhoy @gerlowskija as well

Instead of subclasses of LB....SolrClient, ...Cloud...SolrClient, ...Http...ClusterStateProvider, we can have concrete impls that delegate to a SolrClient that is provided. It's a vision anyway; hopefully achievable?

Isn't this a bit much to aim for in 10.0? I'd rather get a CloudJdkSolrClient ready for 10.0 with our current class hierarchy to offer a no-deps soljr, than getting stuck in a large refactoring.

I am torn about what to do about the lengthy doc-comment which was copy/pasted from LBHttpSolrClient to LBHttp2SolrClient.  I suppose the right thing to do at this point is to copy/paste it once again for LBHttpJdkSolrClient, and not to have anything elaborate in the abstract class doc-comment.  This is yet another argument for having a generic LB Client and not subclasses, but I also feel this is beyond the scope of this PR.

this is beyond the scope of this PR

I'm not sure I agree; it's how the issue is approached.  Maybe there's a sunk-cost issue here

Users generally don't refer to the LB client nor ClusterStateProvider; we shouldn't constrain ourselves, especially in a major release.

Agree, wouldn't care too much about breaking these two APIs. Also, if there is will and energy to refactor things for 10.0, it is better for users to take the hit in v10 than doing yet another change for v11.

I opened a better PR #2828 with a generic client.  Please review/compare.

superseded by #2828.


Unity WebGl build error

Hello there,
i become this 2errors in Unity and i dont know what is the reason for that ? i tried to build on WebGl with Version Unity2022.1.23f1

dergraph@13.1.8/Editor/Generation/Targets/BuiltIn/Editor/ShaderPreprocessor.cs:284)
UnityEditor.Rendering.BuiltIn.ShaderPreprocessor.OnProcessShader (UnityEngine.Shader shader, UnityEditor.Rendering.ShaderSnippetData snippetData, System.Collections.Generic.IList`1[T] compilerDataList) (at Library/PackageCache/com.unity.shadergraph@13.1.8/Editor/Generation/Targets/BuiltIn/Editor/ShaderPreprocessor.cs:348)
UnityEditor.Build.BuildPipelineInterfaces.OnPreprocessShaders (UnityEngine.Shader shader, UnityEditor.Rendering.ShaderSnippetData snippet, UnityEditor.Rendering.ShaderCompilerData[] data) (at <4ce403b7cdf744109283ae55cdc24285>:0)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr, Boolean&)

Error building Player: ArgumentOutOfRangeException: Specified argument was out of the range of valid values.
Parameter name: subshaderIndex

would be nice if someone can help me with this issue ?

Hey, please use Unity LTS versions (2021.3 / 2022.3), seems you used 2022.1.
Were you able to solve your issue?

Hello there,
Yes i was able to solve this issue but it should be in the Plugin direct i think ?
Even i have to find out if i am in Runtime mode or not. I dont want to change the PBRGraph itself.
So now the volume and transmission in unity is upside down and in the webgl build correct.
I use Unity 2023.2.1f1.
Cheers
Marcus

Von: hybridherbst @.>
Gesendet: Montag, 1. Juli 2024 18:15
An: KhronosGroup/UnityGLTF @.>
Cc: MarcusGT076 @.>; State change @.>
Betreff: Re: [KhronosGroup/UnityGLTF] Unity WebGl build error (Issue #742)
Hey, please use Unity LTS versions (2021.3 / 2022.3), seems you used 2022.1.
Were you able to solve your issue?
—
Reply to this email directly, view it on GitHubhttps://github.com/KhronosGroup/UnityGLTF/issues/742#issuecomment-2200560396, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJMZIXISVQVIUK6Z76SYKATZKF6CDAVCNFSM6AAAAABJY2XOLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBQGU3DAMZZGY.
You are receiving this because you modified the open/close state.Message ID: @.***>

Hey, I'm not sure what you mean by this:

but it should be in the Plugin direct i think

And also this:

Even i have to find out if i am in Runtime mode or not. I dont want to change the PBRGraph itself.

Could you clarify what you mean / what you encountered?
We saw #746 and will take a look this week.

Hi 🙂
So the problem is that the what the camera sees in the volume and transmission object is different in Unity editor and a webgl build.
When you see through the object with volume and transmission, you see the objects behind the object upside down. So the UV's are flipped.
void SampleSceneColor_float(float2 uv, float lod, out float3 color) its this function in the PBRHelper.cginc File where i corrected the UV's.
But of course now the material is in the editor upside down and in the build its correct.
Cheers,
Marcus

Von: hybridherbst @.>
Gesendet: Dienstag, 2. Juli 2024 13:32
An: KhronosGroup/UnityGLTF @.>
Cc: MarcusGT076 @.>; State change @.>
Betreff: Re: [KhronosGroup/UnityGLTF] Unity WebGl build error (Issue #742)
Hey, I'm not sure what you mean by this:
but it should be in the Plugin direct i think
And also this:
Even i have to find out if i am in Runtime mode or not. I dont want to change the PBRGraph itself.
Could you clarify what you mean / what you encountered?
We saw #746https://github.com/KhronosGroup/UnityGLTF/issues/746 and will take a look this week.
—
Reply to this email directly, view it on GitHubhttps://github.com/KhronosGroup/UnityGLTF/issues/742#issuecomment-2202889179, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJMZIXNODNX7EUPKYI2XGITZKKFVZAVCNFSM6AAAAABJY2XOLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBSHA4DSMJXHE.
You are receiving this because you modified the open/close state.Message ID: @.***>

Hey, that relates to https://github.com/KhronosGroup/UnityGLTF/issues/746 and that is a separate issue.
We're in the issue titled "Unity WebGl build error" here, so my questions relate to what you had to change to not have a build error anymore.

Ahhhh so i get it now 🙂
Sorry for the misunderstanding.
So it was my fault in this case thats because i closed this one. It was a problem of some settings in Unity itself and some scripts of mine.
Cheers,
Marcus

Von: hybridherbst @.>
Gesendet: Dienstag, 2. Juli 2024 19:17
An: KhronosGroup/UnityGLTF @.>
Cc: MarcusGT076 @.>; State change @.>
Betreff: Re: [KhronosGroup/UnityGLTF] Unity WebGl build error (Issue #742)
Hey, that relates to #746https://github.com/KhronosGroup/UnityGLTF/issues/746 and that is a separate issue.
We're in the issue titled "Unity WebGl build error" here, so my questions relate to what you had to change to not have a build error anymore.
—
Reply to this email directly, view it on GitHubhttps://github.com/KhronosGroup/UnityGLTF/issues/742#issuecomment-2203882206, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJMZIXNP4CHA7KPOMW6SFXTZKLOA3AVCNFSM6AAAAABJY2XOLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBTHA4DEMRQGY.
You are receiving this because you modified the open/close state.


"Download of browser failed"

% browser-agent "what day is it"
Error: Download of browser failed
Caused by:
0: Failed to create archive file
1: No such file or directory (os error 2)
%

I'm having the same issue

@Marviel Solved it. you need to create the following folders in the root directory:
browser
user_data

Nice, seems like that would be an easy PR fix
On Mon, Mar 27, 2023, 1:39 AM Zouvier @.***> wrote:

@Marviel https://github.com/Marviel Solved it. you need to create the
following folders in the root directory:
browser
user_data
—
Reply to this email directly, view it on GitHub
https://github.com/m1guelpf/browser-agent/issues/3#issuecomment-1484526504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPRLLL5K3I3UGKCG7XQZM3W6ERYNANCNFSM6AAAAAAWIUFQMU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Shouldn't this be fixed by #1?

Haven't tested today -- possibly
On Mon, Mar 27, 2023, 1:51 PM Miguel Piedrafita @.***>
wrote:

Shouldn't this be fixed by #1
https://github.com/m1guelpf/browser-agent/pull/1?
—
Reply to this email directly, view it on GitHub
https://github.com/m1guelpf/browser-agent/issues/3#issuecomment-1485576451,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPRLLKEPK6SDEVMWHCUG3DW6HHTLANCNFSM6AAAAAAWIUFQMU
.
You are receiving this because you were mentioned.Message ID:
@.***>

It still fails even with the fix by #1

Should be fixed by #4


Update Rainbow Six Siege

Game Title
Rainbow Six Siege
Status of the Anti-Cheat
❔ Unconfirmed
Proof of Mention
https://store.steampowered.com/news/group/4145017/view/3104663180636096966
https://www.reddit.com/r/linux_gaming/comments/qo5e0b/rainbow_6_siege_working_on_linux_with_proton/
What happen
The game launch and don't give any BattlEye error. Multiplayer is playable.
Launched via Steam Beta with Proton Experimental + Proton Battleye Runtime

Is there a way to see if BattlEye is working correctly? We can't find how to see BattlEye's log files

The first link does not mention anything about r6 and the 2nd one does not show any kind of log that the anti cheat loaded correctly, also your proton logs does not mean anything any game that has the BattleEye anti-cheat will load the BattleEye dll.

The first link does not mention anything about r6 and the 2nd one does not show any kind of log that the anti cheat loaded correctly, also your proton logs does not mean anything any game that has the BattleEye anti-cheat will load the BattleEye dll.

ye, i'm asking how to log Battleye to check if is working or not. I can't find anything about this online.

The first link does not mention anything about r6 and the 2nd one does not show any kind of log that the anti cheat loaded correctly, also your proton logs does not mean anything any game that has the BattleEye anti-cheat will load the BattleEye dll.

ye, i'm asking how to log Battleye to check if is working or not. I can't find anything about this online.

I am not sure but if this is a bad implementation of the anti-cheat which would be unlikely for a massive game like this but still, you could get banned because of how r6 handles banning (in waves) i would not recommend actually playing the game as you could get flagged.

The first link does not mention anything about r6 and the 2nd one does not show any kind of log that the anti cheat loaded correctly, also your proton logs does not mean anything any game that has the BattleEye anti-cheat will load the BattleEye dll.

ye, i'm asking how to log Battleye to check if is working or not. I can't find anything about this online.

I am not sure but if this is a bad implementation of the anti-cheat which would be unlikely for a massive game like this but still, you could get banned because of how r6 handles banning (in waves) i would not recommend actually playing the game as you could get flagged.

That's true, I just did one match online to be sure it's working and trying to get some logs and I closed the game after it. I just want to find a way to confirm all is working fine or not.

Confirmed not implemented. No longer working. Just try for your self you will get kicked. No bans doe.


Increase amount of memory available for tests in self-hosted runners

Our self-hosted runners have a lot of memory allocated to the /var/lib/docker which is tmpfs mounted storage for docker engine.
This is done in order to speed up immensely any docker-related operations - such as building and creating images and running and deploying kubernetes instances.
The memory allocated was at the 85% of capacity ~ 52 GB. However it seems that with our setup when we run up to two full K8S clusters, we are peaking at ~40GB . We can safely allocate more memory for tests and other operations - which might speed up the speed of tests.

I've run a few builds with debug-ci-resources turned on and this one was the biggest tmpfs usage I could find:

https://github.com/apache/airflow/actions/runs/7003593564/job/19050346759#step:7:1589

I think we can safely give our tests ~10% memory.
Actually - I am not even  sure if it will change anything (I believe when tmpfs capacity is set, it will use only as much as it actually uses, it does not reserve it, it's just the maximum it will use.
So I am not even sure if we should merge it. I just wanted to start potentially a discussion if we want to do some optimisations there (and show @hussein-awala the nice resource debugging feature we have by assigning 'debug-ci-resources` label.
BTW. We are discussing with datadog a possibilit of them donating us CI Datadog monitoring solution that we could plug-in instead of those "do-it-yourself" monitoring we have :)

IMHO it should be ok to reduce it to 60% (31.2Gb), but let's start with 10% of extra memory for tests, then we can check if we need more memory

See above - K8S tests take way more of docker filesystem (this is also the reason why we cannot run more than 3 of them in parallel).  I also am not sure if that changes anything (it's more of an exercise here to show what we have) - I believe tmpfs will only take as memory as it actually uses, so I am not sure if we are gaining much by decreasing it :)

IMHO it should be ok to reduce it to 60% (31.2Gb), but let's start with 10% of extra memory for tests, then we can check if we need more memory

See above - K8S tests take way more of docker filesystem (this is also the reason why we cannot run more than 3 of them in parallel). I also am not sure if that changes anything (it's more of an exercise here to show what we have) - I believe tmpfs will only take as memory as it actually uses, so I am not sure if we are gaining much by decreasing the limits :)

BTW. This actually made me think that possibly we could see if we could have the k8s tests themselves optimized. We are using kind and deploying the clusters  in their "default" configuration, which is - I guess - kind of not really "development" setting but something in-between development and production (usually those  come with default settings that are suitable for low-production setting as well) . But our tests are not very demanding (except setting up the cluster and deploying Airflow we merely run a few pods and several DAGs). So maybe we could somehow optimize the settings of the kind clusters we have to take less of /var/lib/docker. Nothing really immediate but if we do - we could possibly run more of them in parallel.
I've done similar exercise in the past to decrease memory used by Postgres/MySQL (and I was able to save 50% of memory they used by using low settings that were enough for our tests).

Updated the description and marked it as a Draft for now. Just wanted to spark a possible discussion :)


Endpoint regeneration unit test benchmark

[ ] Mock datapath interactions required for endpoint generation
[ ] Add unit test and benchmark around endpoint generation

Endpoint regenerations are not really a problem anymore due to massive policy engine improvements and BPF templating.


Update IgBLAST

We are using 1.4.0, but 1.6.0 has been released recently.

Closing this because it is the same as #12