Broken on Ubuntu 20.04 due to GLIBC version,  both gnu and musl packages contain the same binary

TL;DR read title.
I've tried downloading both gnu and musl packages in both tar.gz and zip format, but they all contain the same binary:
$ uname -a
Linux x1e 5.15.0-105-generic #115~20.04.1-Ubuntu SMP Mon Apr 15 17:33:04 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

### pnpm-shell-completion_x86_64-unknown-linux-musl.tar.gz

$ mkdir musl && cd musl
$ wget https://github.com/g-plane/pnpm-shell-completion/releases/download/v0.5.3/pnpm-shell-completion_x86_64-unknown-linux-musl.tar.gz
$ tar xf pnpm-shell-completion_x86_64-unknown-linux-musl.tar.gz --get pnpm-shell-completion

$ ./pnpm-shell-completion
./pnpm-shell-completion: /lib/x86_64-linux-gnu/libc.so.6: version 'GLIBC_2.33' not found (required by ./pnpm-shell-completion)
./pnpm-shell-completion: /lib/x86_64-linux-gnu/libc.so.6: version 'GLIBC_2.32' not found (required by ./pnpm-shell-completion)
./pnpm-shell-completion: /lib/x86_64-linux-gnu/libc.so.6: version 'GLIBC_2.34' not found (required by ./pnpm-shell-completion)

$ cd ..

### pnpm-shell-completion_x86_64-unknown-linux-gnu.tar.gz

$ mkdir gnu && cd gnu
$ wget https://github.com/g-plane/pnpm-shell-completion/releases/download/v0.5.3/pnpm-shell-completion_x86_64-unknown-linux-gnu.tar.gz
$ tar xf pnpm-shell-completion_x86_64-unknown-linux-gnu.tar.gz --get pnpm-shell-completion

$ ./pnpm-shell-completion
./pnpm-shell-completion: /lib/x86_64-linux-gnu/libc.so.6: version 'GLIBC_2.33' not found (required by ./pnpm-shell-completion)
./pnpm-shell-completion: /lib/x86_64-linux-gnu/libc.so.6: version 'GLIBC_2.32' not found (required by ./pnpm-shell-completion)
./pnpm-shell-completion: /lib/x86_64-linux-gnu/libc.so.6: version 'GLIBC_2.34' not found (required by ./pnpm-shell-completion)

$ cd ..

$ diff -s gnu/pnpm-shell-completion musl/pnpm-shell-completion --color=always
Files gnu/pnpm-shell-completion and musl/pnpm-shell-completion are identical

I tried couple of older releases as well, the same issue. Building from source worked.
Workaround for others with the issue (requires Rust)
cargo install --git https://github.com/g-plane/pnpm-shell-completion.git

Try v0.5.4.

Fix confirmed. Thanks for the prompt response!


system problems

Can this program run under the windows system ?A sincere hope to get a reply

Yes @duan147.
You can simply install anaconda and follow on installing the required packages as prompted.


Changing Telemetry to be More Category-based

As written in title; causes less empty telemetry events.

You don't quite have these GDPR annotations correct.
Here's an example of one surrounding cloning in the GitHub PR extension https://github.com/microsoft/vscode/blob/main/extensions/git/src/commands.ts#L504-L508
As you see the the event name is the top level key and then it contains all the properties inside with their classifications and purposes. My suggestion was around you having an event called preview and then something like method as the property showing what type of preview they were in. Let me know if that helps clarify.
Additionally, you don't need your own setting to turn on and off telemetry as it's linked to the VS code setting. Having one is up to your extension if you wish to keep it.

Ah ok, that helps a lot! Btw, I think I linked it to the vscode setting telemetry.enableTelemetry, so it's not my own setting. Should I still link it, or does it happen automatically?

This looks better :). You could clean up further by condensing things like tasks.terminal.startFromExtension and tasks.terminal.start together but that's just me being a neat freak. Nice job 👍

Thanks! tasks.terminal.start will actually register whenever tasks.terminal.startFromExtension is run (but vice versa is not true), so that's why I made them separate events instead of condensing them into one.


no-sock implies no-http

So we can simplify the (only done sometimes... why?) tests for OPENSSL_NO_SOCK with OPENSSL_NO_HTTP and only test for the latter.
Checklist
Thank you for your contribution. Please answer the following:

[X] I acknowledge that I am authorized to submit this code under
the terms of the Apache License
[ ] All new public APIs are documented
[ ] All new public functions have tests
[ ] All new public functions have fuzzing tests

It always struck me as funny that you can disable sockets in a secure sockets layer library.

Yeah, but Intel's UEFI source, for example, uses just the crypto lib. Or something like that.  But yeah it's a bit silly. We might need to separate the network/crypto part a bit better.


ICOS retrieve error - found via doc build

So I just encountered this error whilst building the docs, in Retrieving_remote_data.rst we use the ipython:: Sphinx directive which runs the lines as it goes. I get this error which looks like somethings changed since we outsourced the metadata maybe?
File ~/Documents/Devel/openghg/openghg/standardise/meta/_metadata.py:84, in sync_surface_metadata(metadata, attributes, keys_to_add, update_mismatch)
     82 if str(value).lower() != str(attr_value).lower():
     83     if not update_mismatch:
---> 84         raise AttrMismatchError(
     85             f"Metadata mismatch for '{key}', metadata: {value} - attributes: {attr_value}"
     86         )
     87     else:
     88         logger.warning(
     89             f"Metadata mismatch for '{key}', metadata: {value} - attributes: {attr_value}\n"
     90             f"Updating metadata to use attribute value of {key} = {attr_value}"
     91         )

AttrMismatchError: Metadata mismatch for 'station_long_name', metadata: WAO - attributes: Weybourne Observatory, UK

I'd say this is part of a larger conversation we keep coming back to about how we want to handle these mismatches. From discussion on the code retreat Alice opened up Issue #613 for discussion around this as well.

This no longer happens as the site has been changed to SCO.


Remove unneeded parameter key

The parameter was not used inside the method and only had one call-site
Please ignore if you are planning to use the parameter in some way.
All tests are passing

Thank you!
Any idea what the intention behind the parameter was in the first place?

Thanks for the contribution! I wouldn't remember at this point. It could've been introduced accidentally, or it could be something vestigial and lost its purpose after some refactoring. The main thing is that it's now cleaned up 🙂


Add MSYS2 as a way of installing gfortran on Windows

At https://fortran-lang.org/learn/os_setup/install_gfortran , I think it's fine to keep equation.com as the 1st method to install gfortran on Windows, since it is simple and fast to install. I suggest adding MSYS2 as method #4. I just tried it on Windows 10, and it works fine after following the instructions. Once installed, gfortran --version says GNU Fortran (Rev9, Built by MSYS2 project) 10.2.0 . It takes longer to install than the equation.com compiler binary but comes with more stuff, such as python. A list of MSYS2 packages is here.

I'll certainly let my students try to use MSY2 in my Fortran programming course next month, so we'll get some data points on how well this setup strategy works. I have already written a short introduction here which I could contribute here.

I am using gfortran on MSYS2 for many years, to test gtk-fortran under Windows. It includes CMake, pkg-config, and all the classical tools you need in a UNIX-like environment.
It is slower than on a native Linux environment because many things must be "translated" to make programs work under Windows, but it is very usable.


Add kubernetes-replicator for AdGuard Home

Use https://github.com/mittwald/kubernetes-replicator to replicate from a secret to a configmap so I don't have to manually update the admin password after bootstrap.

Closing as decommissioned AdGuard Home


How long train 61 frames?

Thank you for your excellent work!
I want to train the 61 frames config with 8 A100, and i have used the map cache. The time of 1 epoch is 15-20 days with 1 batchsize(2 batchsize out of memory), is it normal? How long did you train the 61 frames config? What‘s your batchsize and epoch? Thank you for your answer!

We train about 20 days in total for 80k steps. Config is just the one we used. You should have all the hyper-parameters set.

Thank you, i have 15400 steps in one epoch, and i have 8 gpus, it means total step is 15400*8, is it right?

No, when counting steps, we do not multiply with the number of processes. The number indicates effective update steps.

Thank you, i find your batchsize is 1 as well in the config, the picture is my parameters. I also use the advanced_12Hz_infos pkl(download online), but my time is so long in 8 A100 gpus.

20-30s/it is fine.


Upgrade to xgboost v1.3.1

Interesting things:

v1.4.x didn't work for me and I couldn't figure out why.  I think it had issues somewhere in the JNI layer.  Thus this upgade only goes to the v1.3.x series.
xgboost v1.3.3 is identical to v1.3.1 for jvm packages -- so this is the latest version of 1.3.x series for jvm packages.
I had to reduce the floating point precision comparison in the tests.  I think 1e-6 is all we should normally expect though?

ugh, apparently the xgboost tests are flakey 😭 .  Was passing on my dev env, but failed in travis.  But by repeated runs in dev env, it sometimes passes and sometimes fails.
Let me dig in a bit.


Test K8s versions 1.18, 1.19, and 1.20 in E2E

Use the new ability of the shared E2E GHA to configure the Kubernetes
version. Add jobs that cover the latest available patch versions
(limited by kind images) from the three most recent minor versions.
Run one E2E job with all-default configuration per K8s version. Also
test the latest K8s version with features that are primarily implemented
in this repo, like the cable driver (or ovn) and using globalnet or not.
Signed-off-by: Daniel Farrell<EMAIL_ADDRESS>

A couple of failing jobs, collected logs and re-running:
E2E (1.20.2, globalnet):
https://pastebin.com/apYwpA5w
E2E (libreswan, operator, 1.17.17, ovn):
https://pastebin.com/p9CENskb

So this adds another 6 jobs that will run for every PR. Perhaps we should consider running these as a periodic job?

I think we should gate on testing our supported K8s versions. Otherwise we're just going to do more of the anti-pattern of periodic breaking, someone finally noticing, maybe some slow fix.
To a lesser extent, I think we should have multi-K8s coverage for major matrix features in at least one place. Ideally that coverage would also gate out breakages by being close to where the feature is implemented (as I think would happen here).
I think we should add periodic jobs that cover features implemented here (globalnet, cable drivers, OVN) for all supported minor versions (adding 1.18 and 1.19 dimensions).

I think we should gate on testing our supported K8s versions. Otherwise we're just going to do more of the anti-pattern of periodic breaking, someone finally noticing, maybe some slow fix. This is a big lesson of OpenStack's CI.
I also think we should have gating latest-K8s coverage for major matrix features in at least one place. Ideally that coverage would gate out breakages by running on PRs close to where the feature is implemented (as I think would happen here). If we move our default K8s version to be the latest one, these jobs would go away.
I think we should add periodic jobs that cover features implemented here (globalnet, cable drivers, OVN) for all supported minor versions (adding 1.18 and 1.19 dimensions).

I hear ya.  Ideally we would gate on everything but of course we have resource limitations. We already see sporadic job timeouts and keep increasing the timeout. I'm afraid adding more PR gating jobs will make that worse.

I think these Globalnet with 1.21 failures are consistent:

1.20 ;-)

We already see sporadic job timeouts and keep increasing the timeout. I'm afraid adding more PR gating jobs will make that worse.

@tpantelis Running more jobs will make us hit rare failures more, but I don't think adding more jobs will cause more flakeyness. I don't think the fix is to run fewer jobs, but instead to continue fix flakeyness (as you've done a ton of, thanks).

It seems this Globalnet/1.20 failure is consistent. Here's a second example of the logs:
https://pastebin.com/6gL0tcFV
The OVN errors seem to be flaky failures, as they appear in different jobs each workflow run.

I guess this shouldn't be a draft, as the failures for Globlanet and 1.20 aren't introduced here, only revealed. We can mark those jobs as not-required for now.

Ah nice, the Globalnet+1.20 job passed after the most recent rebase.

Ah nice, the Globalnet+1.20 job passed after the most recent rebase.

It might be a fluke 😄

I think we need to find a way of limiting the amount of jobs we add to PRs, we don't have infinite resources.
Periodic, although can be problematic should be a path.
What if we test the maximum & minimum versions on all PRs, and all of them in periodic as a compromise solution?

I think we need to find a way of limiting the amount of jobs we add to PRs, we don't have infinite resources.

Are we hitting resource limits? I haven't seen so.

Periodic, although can be problematic should be a path.

Strongly disagree. All of our CI lessons from history, and all of the discussions of this community, show us the importance of gating on PRs and not following the anti-pattern of waiting for periodic jobs to break.

What if we test the maximum & minimum versions on all PRs, and all of them in periodic as a compromise solution.

If I understand you, this would leave us with no breakage-preventing coverage for cable drivers, globalnet, or the CNI. As I discussed above and for a big chunk of the call Monday, covering the major matrix features implemented in each repo is the right strategy IMHO. That means more jobs here (but fewer elsewhere), because more things are implemented here.

What if we test the maximum & minimum versions on all PRs, and all of them in periodic as a compromise solution?

Actually I think this would result in more jobs than we currently have after we making default=latest K8s version.

Ah nice, the Globalnet+1.20 job passed after the most recent rebase.

It might be a fluke

It passed again. :sweat_smile: :partying_face:

@mangelajo @mkolesnik @tpantelis How about https://github.com/submariner-io/submariner/pull/1242 as an alternative? It only adds one job but meets all of our coverage needs by jumping to 1.20 as the default and dropping coverage for 1.17 and 1.18 (not required as I understand it).

@mangelajo @mkolesnik @tpantelis How about #1242 as an alternative? It only adds one job but meets all of our coverage needs by jumping to 1.20 as the default and dropping coverage for 1.17 and 1.18 (not required as I understand it).

We build against the 1.17 version so perhaps we should test against that instead of 1.19. So full coverage for 1.20, sanity check for 1.17 and skip in between.

So full coverage for 1.20, sanity check for 1.17 and skip in between.

That makes sense to me, I'll update https://github.com/submariner-io/submariner/pull/1242.

Closed in favor of #1242.


pg: Add ScopeName -> Vec<GroupName> mapping

Introduces an index field to the PgState struct to map ScopeName to Vec<GroupName>.
If this isn't what was in mind, I'd need another pointer :)
Would also be open to refactoring the index's value and the modules Vec<...>-method returns to HashSets if that makes more sense.
--
Relating to https://github.com/slawlor/ractor/pull/177#discussion_r1363016547

This is where having another index might be helpful, i.e. Scope -> Group name in a separate mapping. But that's an optimization that can be added in the future.

and https://github.com/slawlor/ractor/issues/184#issuecomment-1846273238

I meant another global index linking the scopes to groups. So you could efficiently lookup the groups from a given scope. This would remove some scan operations but it is a minimal operation.

Sorry I messed up my Git history here. Will reopen a hopefully cleaner PR...


[SPARK-5949] HighlyCompressedMapStatus needs more classes registered w/ kryo

https://issues.apache.org/jira/browse/SPARK-5949

Test build #28235 has started for   PR 4877 at commit 97beaf8.

This patch merges cleanly.

lgtm otherwise

Test build #28235 has finished for   PR 4877 at commit 97beaf8.

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

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

Test build #28239 has started for   PR 4877 at commit f9a0b7c.

This patch merges cleanly.

Test build #28240 has started for   PR 4877 at commit 09610c6.

This patch merges cleanly.

Test build #28241 has started for   PR 4877 at commit a5cb744.

This patch merges cleanly.

ok I think I fixed the style issues

Test build #28242 has started for   PR 4877 at commit 5f6bb6d.

This patch merges cleanly.

i'd really like to automate the style checker so it can catch more stuff like this ... :)

Test build #28240 has finished for   PR 4877 at commit 09610c6.

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

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

Test build #28243 has started for   PR 4877 at commit 7e13316.

This patch merges cleanly.

Test build #28239 has finished for   PR 4877 at commit f9a0b7c.

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

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

Test build #28241 has finished for   PR 4877 at commit a5cb744.

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

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

Test build #28242 has finished for   PR 4877 at commit 5f6bb6d.

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

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

Test build #28243 has finished for   PR 4877 at commit 7e13316.

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

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

Thanks. Merging in.


[WIP] bump docker-stacks tag to e7367

Get scipy stack back in, which includes sympy.
The previous build inherited from all-spark before pyspark was re-based on scipy.

[x] waiting for test build

Had to pin activemodel for ruby since we have ruby 2.1, which activemodel 5.0 doesn't support, but it's all working now.

LGTM if the build completed for you.

Build succeeded. Pushing...


Added James Cook University

Added James Cook University on line 573

Thank you @jdkeypath

Hi Ryan, thanks for making such an awesome and comprehensive list.
Will you still update the list on this page? http://101.datascience.community/2012/04/09/colleges-with-data-science-degrees/ it would be great to add it to that too :)

Hmm, I thought it would be there. I will look into this and it should be up later today. @jdkeypath


chore: workaround for php-cs-fixer 3.63

Description
PHP CS Fixer 3.63.1 7th Gear by Fabien Potencier, Dariusz Ruminski and contributors.
PHP runtime: 8.1.29
Running analysis on 4 cores with 10 files per process.
Parallel runner is an experimental feature and may be unstable, use it at your own risk. Feedback highly appreciated!
Loaded config CodeIgniter4 Coding Standards from ".php-cs-fixer.user-guide.php".
    0/1593 [░░░░░░░░░░░░░░░░░░░░░░░░░░░░]   0%  160/1593 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░]  10%  319/1593 [▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░]  20%  478/1593 [▓▓▓▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░]  30%  638/1593 [▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░░░░░░░░░]  40%  797/1593 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░░░░░░]  50%  956/1593 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░░░░]  60% 1116/1593 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░░░░░]  70% 1275/1593 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░░░░]  80% 1434/1593 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░░]  90% 1593/1593 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

   1) /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/uri/009.php (fully_qualified_strict_types)
      ---------- begin diff ----------
--- /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/uri/009.php
+++ /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/uri/009.php
@@ -1,5 +1,5 @@
 <?php

-$uri = new \CodeIgniter\HTTP\URI('***example.com:21/some/path');
+$uri = new URI('***example.com:21/some/path');

 echo $uri->getAuthority();  // user@example.com:21

      ----------- end diff -----------

   2) /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/publisher/002.php (fully_qualified_strict_types)
      ---------- begin diff ----------
--- /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/publisher/002.php
+++ /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/publisher/002.php
@@ -1,3 +1,3 @@
 <?php

-$publisher = new \CodeIgniter\Publisher\Publisher();
+$publisher = new Publisher();

      ----------- end diff -----------

   3) /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/publisher/001.php (fully_qualified_strict_types)
      ---------- begin diff ----------
--- /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/publisher/001.php
+++ /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/libraries/publisher/001.php
@@ -1,3 +1,3 @@
 <?php

-$publisher = new \CodeIgniter\Publisher\Publisher();
+$publisher = new Publisher();

      ----------- end diff -----------

   4) /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/concepts/factories/009.php (fully_qualified_strict_types)
      ---------- begin diff ----------
--- /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/concepts/factories/009.php
+++ /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/concepts/factories/009.php
@@ -3,4 +3,4 @@
 $user = model('UserModel');

 // The code above is the same as the code below.
-$user = \CodeIgniter\Config\Factories::models('UserModel');
+$user = Factories::models('UserModel');

      ----------- end diff -----------

   5) /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/concepts/factories/008.php (fully_qualified_strict_types)
      ---------- begin diff ----------
--- /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/concepts/factories/008.php
+++ /home/runner/work/CodeIgniter4/CodeIgniter4/user_guide_src/source/concepts/factories/008.php
@@ -3,4 +3,4 @@
 $appConfig = config('App');

 // The code above is the same as the code below.
-$appConfig = \CodeIgniter\Config\Factories::config('App');
+$appConfig = Factories::config('App');

      ----------- end diff -----------

Found 5 of 1593 files that can be fixed in 1.913 seconds, 30.00 MB memory used
Script utils/vendor/bin/php-cs-fixer check --ansi --verbose --diff --config=.php-cs-fixer.user-guide.php handling the cs event returned with error code 8
Error: Process completed with exit code 8.

https://github.com/codeigniter4/CodeIgniter4/actions/runs/10569308633/job/29281890072?pr=9151
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

@paulbalandan I cannot reproduce these changes with a plain php-cs-fixer and the following files.
<?php

$uri = new \CodeIgniter\HTTP\URI('***example.com:21/some/path');

<?php

$finder = \PhpCsFixer\Finder::create()
    ->exclude('somedir')
    ->notPath('src/Symfony/Component/Translation/Tests/fixtures/resources.php')
    ->in(__DIR__);
$config = new \PhpCsFixer\Config();

return $config->setRules([
    '@PSR12' => true,
    'fully_qualified_strict_types' => [
        'import_symbols'                        => false,
        'leading_backslash_in_global_namespace' => true,
    ],
])->setFinder($finder);

$ tools/php-cs-fixer/vendor/bin/php-cs-fixer fix --diff
PHP CS Fixer 3.63.1 7th Gear by Fabien Potencier, Dariusz Ruminski and contributors.
PHP runtime: 8.2.20
Running analysis on 1 core sequentially.
You can enable parallel runner and speed up the analysis! Please see usage docs for more information.
Loaded config default from "/Users/kenji/work/php/php-cs-fixer-test/.php-cs-fixer.php".
Using cache file ".php-cs-fixer.cache".
 1/1 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

Fixed 0 of 1 files in 0.025 seconds, 14.00 MB memory used


Solid Query: Query errors are not caught by ErrorBoundary in the same component

Describe the bug
If the option throwOnError is true, Solid Query throws errors which can be caught with an ErrorBoundary. The issue is that the error doesn't get caught by the ErrorBoundary in the same component as where the query is defined, only by it's parent.
This leads to less readable code, because to properly catch errors, you need to .
It's also inconsistent with the behavior of SolidJS's Suspense and createResource.
Your minimal, reproducible example
https://stackblitz.com/edit/tanstack-query-urg3e7?file=src%2FExample.tsx
Steps to reproduce

Open the link
See that it shows "Parent ErrorBoundary was triggered 😢" while the expected result would be "Inner ErrorBoundary was triggered 🎉"

Expected behavior
If throwOnError is true, the ErrorBoundary defined within the same component is triggered (more specifically, the ErrorBoundary which is closes to the place where queryResult.data is accessed).
Tanstack Query adapter
solid-query 5.25.0

Taking a look at this this week!


Generated Migration Class Partial Class Division & Name

I have some minor gripes about the generated migration partial classes.
I think the Id property should be part of the main class and not in the 2nd partial. By not putting it in the main partial class, it makes it seem like it's a "second class" member not worthy of being seen by developers. I think it's important enough that it should not be relegated to a partial class. This will also facilitate easier renaming.
I'm not sure what the ProductVersion property is used for or if it is permanent. It seems like a temporary field during the development phase? If permanent, could it be renamed to better reflect that it refers to the EntityFramework product version, and not the end-developer's product version?
It seems like the main purpose of the partial class file is to separate the migration-level target model from the actual migration code. If this is so, it would be nice if the generated file name reflected that better by using a file name like *.TargetModel.cs or *.MetaData.cs instead of *.Designer.cs. The new project structure in VS 2015 supports file nesting in the explorer for any "partial" name variation, not just "designer". "Designer" has a bad connotation due to its association with WebForms, and might imply that it is automatically maintained by VS, which it is not.
I might suggest not using a secondary partial class file at all, and just putting all members in a single class and file. However, I understand that the migration-level target model is an odd case, because it doesn't have relevance for every repository type. If I were to write a migration class on my own from scratch, I wouldn't see the need to divide these elements into a separate file. Although unless I needed true multiple-repository compatibility, I probably wouldn't bother with the migration-level target model at all. And with the migration syntax and model building being as cumbersome as it is compared to something like FluentMigrator, I would much rather use the migration scaffolding.

Hey,
Generally speaking we split things up based on "this is your code, you are free to edit it" and then the "you shouldn't really mess with this unless you really know what you are doing" stuff goes in the designer file.
The Id of the migration is also included in the filename, so that is where most people see it. You have to be very careful changing it... it's similar to rewriting history in Git... ok if you really know what you are doing... disastrous if you don't :smile:.
We use .designer.cs because we don't just work with Visual Studio 2015 but also with 2012 and 2013. Also, from Package Manager Console we can control the nesting, but when running from a command line (such as dnx . ef ... in ASP.NET 5) we are just writing files to disk and VS will automatically nest them when named *.designer.cs.

And with the migration syntax and model building being as cumbersome as it is compared to something like FluentMigrator, I would much rather use the migration scaffolding.

Do you have specific examples of what you would like to see improved. We can be quite smart when we scaffold. But in EF7 we are also enabling writing things by hand, in which case you don't need to specify and of the target/source model stuff at all. Those are just things that EF can use smartly when it is responsible for generating migrations.
~Rowan


osx: Support gamepad input.

Fixes #3881
Vibration support is not optimal yet as it doesn't try to emulate the "weak" and "strong" motor strength,
but just takes the parameter with the highest value for the vibration gain.

Looks good from a quick glance, and it's well contained in its own source files anyway. Let's merge and see what OSX users experience with it :)

@Hinsbart Do you think I should cherry-pick this PR for 2.1.1?


GUI: broken table

Follow the 1st two steps in issue #1244, you'll find the table not able to read.

@shaomeng please see if this is still a problem or if fixed by Qt5

@shaomeng please see if this is still a problem or if fixed by Qt5

It's no longer a problem as of 1/12/2021.

It's no longer a problem as of 1/12/2021.


minikube 1.12.3

Updating package minikube to release v1.12.3.
Release info
📣😀 Please fill out our fast 5-question survey so that we can learn how & why you use minikube, and what improvements we should make. Thank you! 💃🎉
Release Notes
Version 1.12.3 - 2020-08-12
Features:

Make waiting for Host configurable via --wait-timeout flag #8948

Bug Fixes:

Ignore localhost proxy started with scheme. #8885
Improve error handling for validating memory limits #8959
Skip validations if --force is supplied #8969
Fix handling of parseIP error #8820

Improvements:

GCP Auth Addon: Exit with better error messages #8932
Add warning for ingress addon enabled with driver of none #8870
Update Japanese translation #8967
Fix for a few typos in polish translations #8950

Thank you to our contributors for this release!

Anders F Björklund
Andrej Guran
Chris Paika
Dean Coakley
Evgeny Shmarnev
Ling Samuel
Ma Xinjian
Marcin Niemira
Medya Ghazizadeh
Pablo Caderno
Priya Wadhwa
RA489
Sharif Elgamal
TAKAHASHI Shuuji
Thomas Strömberg
inductor
priyawadhwa
programistka

Installation
See Getting Started
ISO Checksum
c993e057d3ee7e1ea71b3c3dc2cba9de0e2f97bbe078ac8efd9f8d40db2ddfba

Would you mind resolving conflicts? Thanks!


Can anyone reproduce the results? (my results attached)

Does anyone successfully reproduce the results? I ran several times but the results are far away from the author's. My results are attached.

Hi @ShuoYang-1998,
I am very sure that it is just a matter of some hyper-parameter that is causing the discrepancy. The code tip is one that I used to run a lot of ablations towards the end of the paper and for the rebuttal. I just need to find that rouge hyper parameter that is causing the issue. One other point to note from your reported results is that ORE is better than your reproduced Faster-RCNN+FT in most of the cases.
As @salman-h-khan said, I had an international travel on Saturday. Unfortunately enough, I am tested positive for COVID today. I have developed minor complications and hence am admitted to a hospital now.  I am typing this message from the hospital room. Kindly give me some time to regain my health, if possible.
Thanks,
Joseph

wish you all good

Does anyone successfully reproduce the results? I ran several times but the results are far away from the author's. My results are attached.

Hi Yang,
I have a question about the experiments of EBMs.
Is the validation used for fitting EBMs distribution using known and unknown labels? I wonder if the open-set settings will degenerate into few-shot settings after validation because unknown classes (UUCs)  with ground-truth labels shouldn't appear in the process except for testing.

Does anyone successfully reproduce the results? I ran several times but the results are far away from the author's. My results are attached.

Hi,Yang
You have reproduced the results so quickly, and I would like you to give me some advic. I can't send messages to you email address, if it is convenient for you, could you give me some guidance.Thank you, looking forward to your reply.

Does anyone successfully reproduce the results? I ran several times but the results are far away from the author's. My results are attached.

Hi Yang,
I have a question about the experiments of EBMs.
Is the validation used for fitting EBMs distribution using known and unknown labels? I wonder if the open-set settings will degenerate into few-shot settings after validation because unknown classes (UUCs) with ground-truth labels shouldn't appear in the process except for testing.

The EBUI does use all known and unknown data to learn a distribution, but it doesn't access the labels.

Does anyone successfully reproduce the results? I ran several times but the results are far away from the author's. My results are attached.

Hi,Yang
You have reproduced the results so quickly, and I would like you to give me some advice. I can't send messages to you email address, if it is convenient for you, could you give me some guidance.Thank you, looking forward to your reply.

I have uploaded my run.sh in issue 18, please refer.

Does anyone successfully reproduce the results? I ran several times but the results are far away from the author's. My results are attached.

Hi Yang,
I have a question about the experiments of EBMs.
Is the validation used for fitting EBMs distribution using known and unknown labels? I wonder if the open-set settings will degenerate into few-shot settings after validation because unknown classes (UUCs) with ground-truth labels shouldn't appear in the process except for testing.

The EBUI does use all known and unknown data to learn a distribution, but it doesn't access the labels.

Hi, Yang,
Thank you for your reply. I have checked train_loop.py and modeling/roi_heads/roi_heads.py, and find EBUI should have used the unknown annotations, as shown in the followed codes. The gt labels of the unknown instances will be allocated to the region proposals in roi_heads.py and are saved to fit the unknown WB distribution.  Is it right?
   wb_unk = Fit_Weibull_3P(failures=unk, show_probability_plot=False, print_results=False)
     def compute_energy(self, predictions, proposals):         gt_classes = torch.cat([p.gt_classes for p in proposals])         logits = predictions[0]         data = (logits, gt_classes)         location = os.path.join(self.energy_save_path, shortuuid.uuid() + '.pkl')         torch.save(data, location)
I also find EBUI can work along, which means the unknown labels are not from ALU, but from gt.

Hi @JosephKJ
I also can not reproduce your result reported in your paper. I try to run your code several times, but failed to get the same results.
other people also meet the issue，i really doubt the authenticity of this work, so i think you should consider the issue seriously.

d unknown data to learn a distribution, but it doesn't access the labels.

Hi, Yang,
Do you mean the specific labels?
I think that for the open-set settings, we shouldn't use unknown samples for training. In this validation, we need to learn(fit) the distribution and save the parameters, which should be seen as training with some tricks.
For MINIST, we cannot set 0-6 with real labels and set the rest are labeled unknown. In OWOD, I am not sure whether using Known and unknown labeled samples to fit the distribution is OK? Do you think it is OK?

d unknown data to learn a distribution, but it doesn't access the labels.

Hi, Yang,
Do you mean the specific labels?
I think that for the open-set settings, we shouldn't use unknown samples for training. In this validation, we need to learn(fit) the distribution and save the parameters, which should be seen as training with some tricks.
For MINIST, we cannot set 0-6 with real labels and set the rest are labeled unknown. In OWOD, I am not sure whether using Known and unknown labeled samples to fit the distribution is OK? Do you think it is OK?

I also have the same concern, some other people also raise this questions in [https://github.com/JosephKJ/OWOD/issues/16] (url) and https://github.com/JosephKJ/OWOD/issues/8. But the author didn't response.

@JosephKJ @ShuoYang-1998 @wyman123 @Hrqingqing @salman-h-khan What is the order in which I should use the configuration files in the T1 ~ T4 experiments?Looking forward to your reply!!

Hope to get your detailed analysis of the configuration file!!

@ShuoYang-1998 @Hrqingqing @wyman123
when i run the command line
python3 tools/train_net.py --num-gpus 1 --config-file configs/OWOD/t1/t1_train.yaml SOLVER.IMS_PER_BATCH 4 SOLVER.BASE_LR 0.005
It showed that some folders don't exits! What should I do next step?

Closing this issue due to inactivity. @dyabel is able to reproduce mAP and A-OSE with the latest code. Kindly reopen for more discussions.

@ShuoYang-1998 Hi! friend, I also tried to reproduce the result but failed, I attach the result in #77 , if you successfully reproduced the result, could you help me, thank you!


Requesting Python and Lua icons

There should be Python and Lua icons.
They are used for .py files for Python and .lua files for Lua.
Here's some reference for both icons:
Python
Lua

@lainsce those links don't seem to work. may just be me though.

@iDefineHD @lainsce
I have fixed urls for you 😉

Ehehehe, Markdown minus Coffee equals problems. :coffee:

Nice! Keep on adding icons, because there sure are a million requests just waiting to be realized hehehe :+1:

@lainsce Thank you, I will 😉


[MSSQL] N+1 issue on reading messages

Each read action is followed by FilterExpired() that tries to get the max age out of stream metadata. There is an internal cache, but it's not getting much hits.
Proposed fix: put latest metadata on a stream itself (including MaxAge and MaxCount) and read them together with regular message read action.
Possible workaround: unseal the MsSqlStreamStore, so in consumer code - GetStreamMetadataInternal can be overridden to avoid the Sql query (so Max age won't be supported)

This is not a N+1 issue but potentially a 2N one. SSS as a lib, doesn't have a background savaging process. Instead it lazily cleans up streams that have MaxAge and\or MaxCount metadata applied. Checking this metadata as streams are read via ReadAll is where the performance issue arise.... Current implementation is the metadata is read from the stream's metadata stream. This is effectively a double read if there are a large number of streams with small number of messages in them.
Since MaxAge and MaxCount are core metadata concepts, an optimization where by they are stored in the Streams table would help a lot here. (Clean up is actually done as a background task). This however is a breaking schema change for MS SQL.


SetWorldCanvas removed in 3.0, causing old bug to happen again

I upgraded to 3.0 and it has caused a previously resolved bug to start occurring for me again.
Here is a link to the previous issue: https://github.com/thestonefox/VRTK/issues/679#issuecomment-255418427
SetWorldCanvas does not seem to exist in 3.0. Is there a new method of calling SetWorldCanvas or is it supposed to happen automatically now?

You no longer need to set canvases, as any VRTK canvas now should contain a VRTK_UICanvas script on them.
If you want the canvas to not be interactable then you can simply disable that script.

Oh, I see. I didn't notice that in the new documentation. Thank you.

Just while I have your attention. Is there a place you've documented how to re-write your code to work in 3.0?
For example, all the different 'grab attach mechanics' have separate scripts now, so I'm unsure how to set them on my Interactable Objects. Is this explained somewhere?

I've put it all in a video on the new youtube channel:
https://www.youtube.com/watch?v=tMz04CqAYjw&t=1s

Aha, I was looking at TheStoneFox channel. Thanks again.


Add configuration field to customize release PR name

Motivations
Not everyone likes the format chore: release v0.2.31.
Solution
Add a config field called pr_name. Similar to the git_tag_name field, the pr_name field should be a tera template.
The field {{ version }} should be available
Alternatives

Additional context

Requested in https://github.com/MarcoIeni/release-plz/issues/1738

Hi @MarcoIeni, I was looking to get into some OSS development in the Rust space and your project caught my attention. Is it ok if I'd try my luck starting with a contribution for this issue?

Sure, this is a good issue to get started I think :)
I assigned it to you, let me know if you have any questions 👍

I'm not really sure how should we handle things when multiple packages are released, I see in the current behavior, the package name and the version are not shown when multiple packages with different versions are released.
Should we allow an array of values for the package/versions? I think it could work, but also bring some clutter if improperly used.
What do you think?

Should we allow an array of values for the package/versions in the template?

For now let's not do this. Maybe in the future in another PR 👍
If multiple versions are present, you can set {{ version }} to be empty string


AttributeError

Traceback (most recent call last):
File "projects/ie.py", line 3, in
with StanfordOpenIE() as client:
File "/home/jaypal/.local/lib/python3.8/site-packages/openie/openie.py", line 37, in init
from stanfordnlp.server import CoreNLPClient
File "/home/jaypal/.local/lib/python3.8/site-packages/stanfordnlp/init.py", line 1, in
from stanfordnlp.pipeline.core import Pipeline
File "/home/jaypal/.local/lib/python3.8/site-packages/stanfordnlp/pipeline/core.py", line 7, in
import torch
File "/home/jaypal/.local/lib/python3.8/site-packages/torch/init.py", line 196, in
from torch._C import *
ImportError: /home/jaypal/.local/lib/python3.8/site-packages/torch/lib/libtorch_cuda.so: cannot read file data
Exception ignored in: <function StanfordOpenIE.del at 0x7f2d0548d940>
Traceback (most recent call last):
File "/home/jaypal/.local/lib/python3.8/site-packages/openie/openie.py", line 100, in del
self.client.stop()
AttributeError: 'StanfordOpenIE' object has no attribute 'client'
I am getting this error. I have successfully installed stanford_openie library. What to do to solve this error?

@CJPJ007 it seems like it's a pytorch error, independent of this library:
Try to solve it like this: https://github.com/pytorch/pytorch/issues/44191

I fixed the attribute error problem: https://github.com/philipperemy/Stanford-OpenIE-Python/pull/38.
The source of the problem is the installation of Pytorch. Please post your issue on https://github.com/stanfordnlp/CoreNLP as this issue is unrelated to the python wrapper of OpenIE.
Thank you!


Show warning when no camera is used

Objective
Show a warning if meshes get rendered without a camera.
Fixes #1432
Solution
Add a system to the extract render stage that queries for all cameras and all meshes. If only meshes and no camera was found, a warning gets emitted.
What I don't like about this solution, is that the console gets spammed by the warning. Please let me know what you think.

Changelog
Print a warning when trying to render meshes without a camera.

Hmm yeah I don't love that this is spinning and constantly checking and warning. Is there an initialization step where we could check this?

Not a fan of adding a system just for that, and I'm not convinced there isn't a use case for having meshes without a camera

@alice-i-cecile I will take a look and see if I can find something.
@mockersf Ok, that might be true. If others agree, then it might make sense to close the open issue.

Yep, if you can't find a good way to do this without a system we should just close the issue.

I agree that there might be use cases for having meshes without cameras, and I'm also not a fan of a system spinning to do this check each frame.
Feel free to open a new PR if you find something that works around these issues!


twitter and package-size badged added

It was my first contribution. Thanks. Maybe I will try this package in my next mini project.


PerKit Benchmarker  with PerfKit Explorer how to handle?

PerKit Benchmarker  result how to handle PerfKit Explorer? Examples?

https://github.com/GoogleCloudPlatform/PerfKitExplorer           another?


Opening  images fails on Fedora 35 and Centos 7

Hi, I was able to use TissUUmaps on Windows and MacOS, however on my two linux systems the program just stays blank after opening an image file. The program was installed in a fresh conda enviroment according to the instructions.
Debug output below:
❯ tissuumaps -d
INFO:root: * TissUUmaps version: <IP_ADDRESS>

DEBUG:root:Debug mode
INFO:root:Starting port detection
INFO:root:Ending port detection 5432
 * Serving Flask app 'tissuumaps' (lazy loading)
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
/home/wintermute/mambaforge/envs/tissuumaps_env/lib/python3.9/site-packages/werkzeug/serving.py:716: ResourceWarning: unclosed <socket.socket fd=70, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=0, laddr=('<IP_ADDRESS>', 0)>
  self.socket = socket.fromfd(fd, address_family, socket.SOCK_STREAM)
ResourceWarning: Enable tracemalloc to get the object allocation traceback
INFO:werkzeug: * Running on http://<IP_ADDRESS>:5432 (Press CTRL+C to quit)
DEBUG:root:Path from referrer: /mnt/data/shared
INFO:werkzeug:<IP_ADDRESS> - - [14/Jun/2022 16:16:14] "GET / HTTP/1.1" 200 -
DEBUG:root:Path from referrer: /mnt/data/shared
INFO:werkzeug:<IP_ADDRESS> - - [14/Jun/2022 16:16:15] "GET / HTTP/1.1" 200 -
DEBUG:root:Opening image: / http://<IP_ADDRESS>:5432/home/wintermute/Downloads/EASI_CK/CK_Base_1_stitched-1.tif PyQt5.QtCore.QUrl('http://<IP_ADDRESS>:5432/home/wintermute/Downloads/EASI_CK/CK_Base_1_stitched-1.tif')
INFO:werkzeug:<IP_ADDRESS> - - [14/Jun/2022 16:16:26] "GET /CK_Base_1_stitched-1.tif?path=home/wintermute/Downloads/EASI_CK HTTP/1.1" 200 -

Any Ideas?

Reading about Qtwebengine issues, I found this one: https://github.com/vlaci/openconnect-sso/issues/69
Could you test the workaround proposed there:

Try to type, before starting TissUUmaps:
$ export QTWEBENGINE_CHROMIUM_FLAGS="--no-sandbox"

Hi cavenel, thank you for your message.
Exporting that variable did not make any differences.
However, the browser link starts the image conversion process and the program behaves as expected.

Hi @zeratoss,
Thanks for the feedback. If everything loads fine in the browser, it is definitely a problem with the Qt5 gui. It is difficult to debug without reproducing it.
I will see if I can test on a Fedora 35 or CentOS 7 virtual machine, and come back to you.
For now, you can use the TissUUmaps server instead, via the command line $ tissuumaps_server /path/to/image/folder/ -p 5001, and open the image in the browser directly on http://127.0.01:5001.

Hi,
Could you test reinstalling TissUUmaps with pip directly from the last version on GitHub?
pip install https://github.com/TissUUmaps/TissUUmaps/archive/master.zip

It should add the "--no-sandbox" automatically, and fix a small bug that was blocking the flag from working.
Tested on Fedora 35 but not Centos 7.

I can confirm it works now on Fedora.
Still blank on Centos

When it is blank, can you try to click on "File > Close image" in the TissUUmaps menu?
I was not able to reproduce the blank screen in CentOS 7.

TissUUmaps 3.1 is now using PyQt6 instead of PyQt5, so this issue is not relevant anymore. Please open a new issue if you still have issues in Linux.


Feature request: vsts build definition create

vsts-cli/src/command_modules/vsts-cli-build/vsts/cli/build/commands.py

@docunid - We released the Azure DevOps extension to Azure CLI  and will not be investing in vsts cli. At present we are working on supporting YAML pipelines from CLI (#333) inclduing creation of a pipelines. Let me know if that works for you.

that'll work :)


Add attr resource type handling

Fix for #2117

Thanks for the PR! I'm on break for the holidays, but I'll loop back to this soon.

Thanks and sorry for delay. Merged!


usb: Handle custom request not only for interface zero

This commit reduce the limitation in the USB core stack.
Until now custom handler could only be served if the request
was directed to the first interface associated with the USB device.
This limitation allows USB audio class to properly respond to the
requests.
This is first step to properly handle set/get interface request by the USB audio class.
For other classes the limitation should be done in the custom_handler.
Signed-off-by: Emil Obalski<EMAIL_ADDRESS>

@anangl, @jfischer-phytec-iot, @carlescufi  Please revisit.

Last push - fixed checkpatch issues.


Generic message aggregation

This is the hotly-anticipated (by me) follow-up to my previous PR, #1186. Closes #1183 (does that macro thing in github still work?)
I've implemented the whole chain necessary to get multiple messages into a single comment on the PR and wrote a bunch of tests. At the moment only bitbucket cloud can use the new message aggregation stuff but I'll be adding it to the other request sources eventually (probably bitbucket server, then gitlab first, then github... I think I'll leave bitrise and any other funky ones to people who care more / have access to them)
Dogfooded it at work too - it works nicely :)

          2 Errors

:no_entry_sign:
Please include a CHANGELOG entry.
You can find it at CHANGELOG.md.

:no_entry_sign:
Tests have failed, see below for more information.

Tests:

File
Name

comments_helper_spec.rb
Danger::Helpers::CommentsHelper#generate_message_group_comment when template is bitbucket_server_message_group with one of each type of message spits out a beautiful comment

  Generated by :no_entry_sign: Danger

sorry for letting this fall behind and sit so long - I'll ensure it's up-to-snuff and all conflicts resolved in the coming days once the more urgent #1208 is done. Been dogfooding this feature at work without any complaints. Sorry to all the non-bitbucket users for breakage!

👍  no worries, I've also not taken the time to review it too. Not on you

I can edit that TODO when I make the semver changes


Update perf tests and results

Fixes #51.
I ran the perf tests again in order to verify #51. I'm happy not to see major perf regression.
@robertjpayne could you tell me how you observed the dramatic dwifft speed up?

Codecov Report

Merging #62 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #62   +/-   ##
=======================================
  Coverage   94.21%   94.21%
=======================================
  Files          17       17
  Lines        1399     1399
=======================================
  Hits         1318     1318
  Misses         81       81

Shamelessly merging this.

💅


Traverse History

ctrl+, and ctrl+. will move backward and forward through history.  Attempted to use ctrl+h and ctrl+l however ctrl+h deletes input to the left of the cursor.

I might prefer alt+p and alt+n instead of Ctrl+, and Ctrl+.

@amjith Can you test and see if that shortcut will work correctly?  I don't see a problem with that if it works.

I just had a conversation with @blainesch. I'd like to make an argument for going back to up/down arrow. Here's why:
People are used to the up/down key. Using alt-p/alt-n is a weird key combo that no one will use. Zazu ships with the fallback search so there is always a result. So the case where the up/down arrow breaks is an extremely rare case that doesn't warrant a change in the key combo.

@amjith I think the problem with the up/down arrow would be that down will take you to the first result if you have one.
Since you said 'go back' maybe you are suggesting that the user select the query before pressing the up/down?  If that is the case would that really be more intuitive?  Seems like it would still merit some documentation exploring and a ctrl+a to select the text in the input.

There I fixed it for you. #262
😝


It seems tiktok detects this extention and blocks access 😢

Just found this issue today, I tried both chrome and firefox, tiktok website works normal with this extension disabled, but after I enabled this extension, after a few refresh the whole page shows below error, which previously only happened once a while when download failed, looks like tiktok dev team has targeted this extension 😭😡

Access Denied
You don't have permission to access "http://www.tiktok.com/@xxxxxxxxxx" on this server.
Reference #18.2758d617.1722788127.313659d7

https://errors.edgesuite.net/18.2758d617.1722788127.313659d7

screenshot

Hi,
Hmm, this is strange. It works fine on my end. I can download and view the page normally with it enabled. Unfortunately, there's not much I can do without being able to reproduce this, but I'll definitely keep an eye on it going forward.
If anyone has any further input or ideas on why this could be happening, feel free to let me know and I'll gladly look into it.

@sixem ok very weird, turns out this particular user page caused this issue 🤯 I closed the tab and tried others all working fine, sorry to bother, closing now


Feature extremeprecip add aggregation

This PR adds a Select Aggregate Function Setting & some notes to the Extreme Precipitation tool

@mukhtyar or perhaps instead of hiding the setting for aggregate function we should disable it when the location boundary type is loca grid cell, and make sure that in its "learn more" that this has no effect when boundary is "none".

@clhenrick good idea about disabling the Select Spatial Aggregation setting for LOCA. Implemented in https://github.com/berkeley-gif/caladapt-website-2021/pull/197/commits/eb84d0fb06cefb5c7c8b544083e6c4bf4d263ecd

 Tools: Extreme Precipitation Events


Remove winfetch from extras bucket

To be merged in the case that ScoopInstaller/Main#2667 is accepted.

This can be merged now

/bb


Verbose logging of cache and queues

What type of PR is this?
/kind feature
What this PR does / why we need it:
Produces a dump of the cache into the logs at log level 5.
A dump of the cache and queues can be logged on demand using SIGUSR2, regardless of the log level.
Refactored some existing dump methods used for testing to use slices for better performance and simpler formatting.
Which issue(s) this PR fixes:

Fixes #1438 #1437
Special notes for your reviewer:
Does this PR introduce a user-facing change?
At log level 6, the usage of ClusterQueues and cohorts is included in logs.

The status of the internal cache and queues is also logged on demand when a SIGUSR2 is sent to kueue, regardless of the log level.

Btw, this is how it looks:
{"level":"info","ts":"2024-01-02T19:00:43.567480487Z","logger":"dumper","caller":"debugger/debugger.go:59","msg":"Started dump"}
{"level":"info","ts":"2024-01-02T19:00:43.56753785Z","logger":"dumper","caller":"cache/snapshot.go:67","msg":"Found ClusterQueue","clusterQueue":{"name":"cluster-queue"},"cohort":"<none>","resourceGroups":[{"CoveredResources":{"cpu":{},"memory":{}},"Flavors":[{"Name":"default-flavor","Resources":{"cpu":{"Nominal":9000,"BorrowingLimit":null},"memory":{"Nominal":38654705664,"BorrowingLimit":null}}}],"LabelKeys":null}],"usage":{"default-flavor":{"cpu":3000,"memory":629145600}},"workloads":["default/job-sample-job-rphvr-2168d"]}
{"level":"info","ts":"2024-01-02T19:00:43.567775516Z","logger":"dumper","caller":"queue/dumper.go:32","msg":"Found pending and inadmissible workloads in ClusterQueue","clusterQueue":{"name":"cluster-queue"},"pending":[],"inadmissible":[]}
{"level":"info","ts":"2024-01-02T19:00:43.567789748Z","logger":"dumper","caller":"debugger/debugger.go:63","msg":"Ended dump"}

I'm preparing another PR with instructions.

Btw, this is how the dump looks:
{"level":"info","ts":"2024-01-02T19:00:43.567480487Z","logger":"dumper","caller":"debugger/debugger.go:59","msg":"Started dump"}
{"level":"info","ts":"2024-01-02T19:00:43.56753785Z","logger":"dumper","caller":"cache/snapshot.go:67","msg":"Found ClusterQueue","clusterQueue":{"name":"cluster-queue"},"cohort":"<none>","resourceGroups":[{"CoveredResources":{"cpu":{},"memory":{}},"Flavors":[{"Name":"default-flavor","Resources":{"cpu":{"Nominal":9000,"BorrowingLimit":null},"memory":{"Nominal":38654705664,"BorrowingLimit":null}}}],"LabelKeys":null}],"usage":{"default-flavor":{"cpu":3000,"memory":629145600}},"workloads":["default/job-sample-job-rphvr-2168d"]}
{"level":"info","ts":"2024-01-02T19:00:43.567775516Z","logger":"dumper","caller":"queue/dumper.go:32","msg":"Found pending and inadmissible workloads in ClusterQueue","clusterQueue":{"name":"cluster-queue"},"pending":[],"inadmissible":[]}
{"level":"info","ts":"2024-01-02T19:00:43.567789748Z","logger":"dumper","caller":"debugger/debugger.go:63","msg":"Ended dump"}

I'm preparing another PR with instructions on how to produce it.

It's great to see. Thanks.


Performance of deck insertion is so slow that it's probably worth worrying about now

As much as premature optimization is the root of all evil.

We could probably separate the deckcycle scraping and the retrieval of
decklists into separate actions.
And we should definitely try to ignore decks that have been imported too
recently.
Do tappedout decklists give Last Modified headers?
On Fri., 28 Oct. 2016, 18:39 Thomas David Baker<EMAIL_ADDRESS>wrote:

As much as premature optimization is the root of all evil.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/issues/332, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAL2zlg4FQbe91x1fEKhQq8fANFx-tK_ks5q4aamgaJpZM4KjKVR
.

This is in a good enough state that I'm going to close it.
It's certainly true that a lot of what we do loading decks from the db, scraping and a bunch of other stuff is way less efficient than it could be but we'll worry about it when we see issues.


Armis Connector Task

Updated PR with following changes

Merged Main Branch (Ruby version was upgraded from 2.6 to 3.1) 1709a88
Fixed lint errors d7b2ab8

@annapoltieva Let me know if you have any feedback!

No changes are remaining from my end, you can merge!


Create List Roles

Title

What is the difference between this and listall?

This was created in April, when listall was not implemented.  This can be closed.


feat: karpenter-global-config migration

Fixes #N/A
Description
This PR begins the process of migrating settings from the karpenter-global-config configmap to CLI arguments and environment variables. The configmap is still supported, but values specified by CLI args and environment vars will take priority.
How was this change tested?
TODO - Implement tests
Note: Due to an interface change this will break Karpenter. Karpenter PR is in the works.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

51 of 77 (66.23%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 82.218%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/settings/settings.go
0
1
0.0%

pkg/operator/injection/fake/settings.go
0
3
0.0%

pkg/operator/injection/injection.go
3
7
42.86%

pkg/utils/env/env.go
5
10
50.0%

pkg/operator/options/options.go
43
56
76.79%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
9021

Relevant Lines:
10972

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

51 of 77 (66.23%)  changed or added relevant lines in 5 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.3%) to 82.182%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/settings/settings.go
0
1
0.0%

pkg/operator/injection/fake/settings.go
0
3
0.0%

pkg/operator/injection/injection.go
3
7
42.86%

pkg/utils/env/env.go
5
10
50.0%

pkg/operator/options/options.go
43
56
76.79%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/scheduling/topology.go
2
86.49%

pkg/controllers/provisioning/scheduling/topologygroup.go
2
96.75%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
9017

Relevant Lines:
10972

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

51 of 77 (66.23%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 82.218%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/settings/settings.go
0
1
0.0%

pkg/operator/injection/fake/settings.go
0
3
0.0%

pkg/operator/injection/injection.go
3
7
42.86%

pkg/utils/env/env.go
5
10
50.0%

pkg/operator/options/options.go
43
56
76.79%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
9021

Relevant Lines:
10972

💛  - Coveralls


Restrincting access from other azure services like Data Factory

It would be great if we could restrict the access from some Azure Services like Data Factory. For sure, the restriction should be for allowing access from only the the customer-owned Azure Services to the Azure SQL Database and Azure SQL Servers.

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

ID: 1c2b2800-b8b3-a8f3-a14f-8dc95eb133ba
Version Independent ID: d92ac8c6-7812-9654-6d72-fcb8b61b8423
Content: Azure SQL Database Security Overview
Content Source: articles/sql-database/sql-database-security-overview.md
Service: sql-database
GitHub Login: @aliceku
Microsoft Alias: aliceku

@ycedillos Thank you for the detailed product feedback. With this product feedback, the appropriate means is via User Voice. Please upvote and comment on the following entry: Allow Access to Azure Services should be restricted by service name. This channel is for documentation and if the functionality doesn't exist, we are unable to address documentation about the functionality.

@ycedillos Please let us know if you have any additional questions. We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


node-gyp build error

I run this command:node-gyp build but I got these errors:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | ia32
gyp info spawn make
gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
make: Entering directory /home/slim/test/build' make: *** No rule to make target Release/obj.target/example/example.o', needed by Release/obj.target/example.node'.  Stop. make: Leaving directory /home/slim/test/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1074:12)
gyp ERR! System Linux 3.13.0-32-generic
gyp ERR! command "node" "/usr/bin/node-gyp" "build"
gyp ERR! cwd /home/slim/test
gyp ERR! node -v v0.12.7
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok

@HmidiSlim unfortunately this info. is not enough for me to help you ...
What happens when you run npm install bleno? Anymore output? Have you installed the prerequisites? https://github.com/sandeepmistry/bleno#linux-ubuntu
Btw, I don't see bleno anywhere in the path for the snippet you provided ...

Closing due to lack of activity ...


Allow netconf save

Add a netconf save option for vr-sros containers
The netconf part works, but not 100% sure on the integration into clab yet, my local compiled containerlab executable cannot find any nodes any longer 😦
$ sudo ../../containerlab/container-lab save -t ring1.yaml
INFO[0000] Getting topology information from ring1.yaml file...
Error: no containers found

Not 100% sure how this is related... at least it compiles, which is a step up from "hello world" 🚀

Awesome, will have a look at it
Thank you for the contribution!

Hey @kellerza
I have added a few things to your PR:

used dns name instead of IP as clab creates /etc/hosts entries for each node
introduced a new map which will hold nodes credentials per node kind

@hellt this seems to be ok, initially tested against an existing deployment, but with the changes you mentioned was obviously breaking that.
New deployments works well.

moved the PR to #355 as some unexpected commits were added

Yeah, indeed, realised & fixed it.
Typically suggestions on how to improve & telling people to go fix it goes a long way to build a community.


Cannot change VM icon to Generic after creation

The VM icon cannot be changed to Generic after creation.
Configuration

UTM Version: 2.4.1
OS Version: 12.0.1
Intel or Apple Silicon? Apple Silicon

Reproduction Steps

Create a VM with an OS or Custom icon
Attempt to change it to Generic

Upon saving with Generic selected, it acts as a no-op.

I can reproduce this. Workaround: change the icon style to "Custom", without choosing an icon file. The VM will now show the Generic icon again (but the option saved is Custom).


Switch to EsBuild for TypeScript bundling and CommonJS support in this plugin

Currently, this plugin uses tsc to bundle TypeScript code into ES Modules. However, I want to switch to EsBuild for better performance and support for both CommonJS and ES Modules in the bundled output.

Resolved the issue. Closing this now.


bug: order quantity not available in CoA rendered in PDF

Description
What keywords did you search in schema-tools issues before filing this one ?
PDF
Is this a BUG REPORT or FEATURE REQUEST ?
BUG
What you expected to happen :
If Order/Quantity is not available it and Order/QuantityUnit should not be rendered on PDF is it results in NaN undefined. This will match the results of the HTML rendering.

I guess the schema should be fixed first, currently Order.Quantity and Order.QuantityUnit are required properties.
Also i don't see any conditional rendering in the HBS template, so you mean to implement it too ?

I guess the schema should be fixed first, currently Order.Quantity and Order.QuantityUnit are required properties.
Also i don't see any conditional rendering in the HBS template, so you mean to implement it too ?

You are right, it is required. As many example certificates do not include them we have to make it optional. You are also right on the HBS template. But let's address after the first round of review with user. In this I just add it to the list of known issues.


Use old reference number

When the -1 icon is used (old reference number), it should default to the last reference number used for the observation.

Also, the old reference numbers should be specific to the current observation (ex: Paradiopatra only shows ref numbers used with this observation; currently show ANY reference number used).

We are experiencing extreme lag times when requesting a new ID # or the pick list for previously used ones (both at sea and onshore). I'm hoping that when the fix listed above is done it might help?


PasswordBroker object creation exception

Laravel Version: 5.5.2
PHP Version: 7.1
Database Driver & Version: no database

Description:
I got this exception
Type error: Argument 2 passed to Illuminate\Auth\Passwords\PasswordBroker::__construct() must implement interface Illuminate\Contracts\Auth\UserProvider, null given, called in
  .../vendor/laravel/framework/src/Illuminate/Auth/Passwords/PasswordBrokerManager.php on line 75

This exception happens because on line there's code:
<?php
        return new PasswordBroker(
            $this->createTokenRepository($config),
            $this->app['auth']->createUserProvider($config['provider'] ?? null)
        );

that makes possible to pass null as the second argument to the PasswordBroker constructor.
This is impossible because of PasswordBroker constructor:
<?php
    public function __construct(TokenRepositoryInterface $tokens,
                                UserProvider $users)

Steps To Reproduce:

require barryvdh/laravel-ide-helper composer package
execute php artisan ide-helper:meta

This looks like a mis-configuration on your side, please make sure you have the correct configuration in your config.auth.passwords, looks like Laravel is unable to create the provider assigned.


High memory usage with large datasets

We are using your gem to generate fairly large XLSX files of about 400 000 rows with 5 columns and the memory usage is quite high 1.4 GB +/-.
I checked and on the original library by jmcnamara there is a constant_memory property which get's set on the worksheet. http://libxlsxwriter.github.io/working_with_memory.html
From the above page:
The optimization works by flushing each row after a subsequent row is written. In this way the largest amount of data held in memory for a worksheet is the amount of data required to hold a single row of data.
Since each new row flushes the previous row, data must be written in sequential row order when 'constant_memory' mode is on:
#include "xlsxwriter.h"
int main() {
    lxw_row_t row;
    lxw_col_t col;
    lxw_row_t max_row = 1000;
    lxw_col_t max_col = 50;
    /* Set the worksheet options. */
    lxw_workbook_options options;
    options.constant_memory = 1;
    /* Create a new workbook with options. */
    lxw_workbook  *workbook  = workbook_new_opt("constant_memory.xlsx", &options);
    lxw_worksheet *worksheet = workbook_add_worksheet(workbook, NULL);
    for (row = 0; row < max_row; row++) {
        for (col = 0; col < max_col; col++) {
            worksheet_write_number(worksheet, row, col, 123.45, NULL);
        }
    }
    return workbook_close(workbook);
}

I would really like to help implement this feature if you could let me know if you think it's possible and if you think it's something you would look at adding in to the gem?

Hi,
I've added "optimization" option to write_xlsx.
install:

unarchive zip file and you'll get gem file.
gem install --local /path/to/gemfile

how to activate "optimization" option:
workbook = WriteXLSX('huge.xlsx', optimization: true)

this version is not well tested, so be careful using this option.
write_xlsx-0.84.1.beta.gem.zip

Our use case is very simple. We are simply taking raw data and putting it in to an xlsx spreadsheet with no formatting or fancy calculations and in this case version 0.84.1 works perfectly.

thanx for your comment,
official release will be done later, because I have to write some tests to this feature.

@cxn03651 thanks for your work, I've tried the beta gem and the process creating rows is good but when you close the workbook, the memory grows until 1gb more or less. could it be reduced?
I've tried this snippet:
workbook = WriteXLSX.new('huge.xlsx', optimization: true)
worksheet = workbook.add_worksheet

40_000.times do |row|
  500.times do |col|
    worksheet.write(row, col, "Hi Excel!")
  end
end

workbook.close

unfortunately I don't know how to profile memory usage of ruby script.
WriteXLSX finishes its work ( xlsx file creation ) by Workbook#close
method. So after Workbook#close, how about using GC (Garbage Collection)
like this:
... some work
workbook.close

workbook = nil
GC.enable
GC.start

2016-06-02 15:30 GMT+09:00 Alfonso Uceda<EMAIL_ADDRESS>
@cxn03651 https://github.com/cxn03651 thanks for your work, I've tried
the beta gem and the process creating rows is good but when you close the
workbook, the memory grows until 1gb more or less. could it be reduced?
I've tried this snippet:
workbook = WriteXLSX.new('huge.xlsx', optimization: true)
worksheet = workbook.add_worksheet
40_000.times do |row|
500.times do |col|
worksheet.write(row, col, "Hi Excel!")
endend
workbook.close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cxn03651/write_xlsx/issues/24#issuecomment-223207002,
or mute the thread
https://github.com/notifications/unsubscribe/AAOcBQQ1snnB8bCb2aiR889wTohWlbMxks5qHnidgaJpZM4Ioarn
.


Align Fleet platform type with Fleet Backend product code for consistency

Pull Request Details
It aligns the Fleet Backend platform type string to Fleet Backend product code FLIJ.
Description
Fleet Backend was incorrectly using FL as a product code in its configuration, and this has been fixed so the Gradle plugin must be aligned.
Note that this is written as a breaking change, but Fleet Backend plugin is unreleased so I don't think it requires a major version bump.
Related Issue
FL-21994
Motivation and Context
See FL-21994.
How Has This Been Tested
Tested manually in https://github.com/vladsoroka/gradle-daemons-services with a includeBuild statement.
Types of changes

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

Checklist

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

Thanks, Titouan!


Adds function for Sony Ci endpoint /assets/[asset_id]

@mrharpo whoops, cut and paste error on the relevant commit message, if merging please squash and change it to something that isnt' a lie, like:
Adds function for Sony Ci endpoint /assets/[asset_id]/download


[Bug] Previous Courses show up in Recommended Courses and not continue courses

Previous courses which have been bought are showing up in recommended courses and not continue courses

[ ] @kh4l1d64 to raise a PR for the fix.

Khalid unable to replicate, Rohit to recheck after redeployment


MariaDB support?

Hello, I noticed the documentation only mentions mysql, but not mariadb - and the README only mentions mysql. Considering most distros run MariaDB by default it would only be fair to also mention MariaDB, and remove any compatibility doubt for users. Ok if I submit a PR?

Made PR https://github.com/ctdk/goiardi/pull/86 now, hope this is ok? Or was there any particular reason not to add MariaDB to docs?


Y menu becomes unavailable after death.

Version 1.0.4 - MP official server.
It seems that after one dies for the you lose the ability to open the y menu, have been able to reproduce multiple times with multiple people.

Solved in 1.0.5


[cxx-interop] Remove no-op casts.

I need to rebase this after some other stuff lands.

@swift-ci please smoke test.

@swift-ci please smoke test.

@swift-ci please smoke test.


buildkit: 0.12.5 -> 0.13.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/moby/buildkit/releases.
meta.description for buildkit is: Concurrent, cache-efficient, and Dockerfile-agnostic builder toolkit
meta.homepage for buildkit is: https://github.com/moby/buildkit
meta.changelog for buildkit is: https://github.com/moby/buildkit/releases/tag/v0.13.0
Updates performed

Golang 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 0.13.0 with grep in /nix/store/r6n9rm9vk2n21a153npsbr7nc89v4rzg-buildkit-0.13.0
found 0.13.0 in filename of file in /nix/store/r6n9rm9vk2n21a153npsbr7nc89v4rzg-buildkit-0.13.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
buildkit
nerdctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r6n9rm9vk2n21a153npsbr7nc89v4rzg-buildkit-0.13.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 buildkit https://github.com/r-ryantm/nixpkgs/archive/582be95706fbfdffa2582ffeb44197476085e74d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/582be95706fbfdffa2582ffeb44197476085e74d#buildkit

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r6n9rm9vk2n21a153npsbr7nc89v4rzg-buildkit-0.13.0
ls -la /nix/store/r6n9rm9vk2n21a153npsbr7nc89v4rzg-buildkit-0.13.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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

buildkit
nerdctl

Maintainer pings
cc @vdemeester @marsam @developer-guy for testing.

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

2 packages built:

buildkit
nerdctl


Ears (v1.4.5) doesnt work with NFC Modloader (v4.6.1)

Summary
Installing Ears and ModloaderNFC in the same Minecraft instance causes a crash to desktop
Error Log
Ears Issue
Steps to Reproduce

Install NFC v3.4.2. (MultiMC or Prism Launcher recommended for these steps, Installation Instructions)

Download Ears

Move ears-nfc-1.4.5.jar to .minecraft folder of NFC Instance.

Rename ears-nfc-1.4.5.jar to ears.jar.

Add -javaagent:ears.jar to Java Arguments.

Download ModloaderNFC v4.6.1

Install as a jarmod. (MultiMC: Open Instance Edit Window, click Version, click Add to Minecraft.jar, find ModLoaderNFC v4.6.1.zip and click Open.)

Launch the title.

Click on Singleplayer.

Create a New World, gamemode and seed do not matter.

Load into the world.

Observe Crash.

Results
Expected Result: Load into the world with both Ears and Modloader working
Actual Result: Title crashes

Fixed with new jarmod version of Ears for v3.4.2 and the merging of Ears into NFC with v3.5


limes: add "tags" to ProjectReport

We will likely have to solve the underlying problem differently.


docs: update readme

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:

thank you for your work @southliu


Worked in 2.1, but exception in 2.2

TypeError: Cannot read property 'onPageLoad' of undefined
  From: Task: <anonymous>
      at Context.initFn (..\index.js:18:12)
  From: Task: "before all" hook

var initFn = function() {
			browser.manage().deleteAllCookies();
			browser.get("/");
		};

		describe("task1", function() {
			before(initFn);
...
		});

+1
Fatal error: Cannot call method 'onPageLoad' of undefined

+1 I'm also getting this. Works on 2.1.0 but fails with 2.2.0. With more explicit promise handling I get a fuller stacktrace:
Cannot read property 'onPageLoad' of undefined
TypeError: Cannot read property 'onPageLoad' of undefined
    at --OMITTED ROOT--/node_modules/protractor/lib/protractor.js:606:25
    at [object Object].promise.ControlFlow.runInFrame_ (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:1877:20)
    at [object Object].promise.ControlFlow.runEventLoop_ (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:1755:8)
    at [object Object].<anonymous> (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:2056:12)
    at goog.async.run.processWorkQueue (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/goog/async/run.js:125:21)
    at runMicrotasksCallback (node.js:337:7)
    at process._tickCallback (node.js:355:11)
From: Task: <anonymous>
    at [object Object].Protractor.get (--OMITTED ROOT--/node_modules/protractor/lib/protractor.js:605:29)
    at --OMITTED ROOT--/jstest/ui/bootstrap.js:49:20
    at [object Object].promise.ControlFlow.runInFrame_ (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:1877:20)
    at [object Object].promise.Callback_.goog.defineClass.notify (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:2464:25)
    at [object Object].promise.Promise.notify_ (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:563:12)
    at Array.forEach (native)
    at Object.goog.array.forEach (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/goog/array/array.js:203:43)
    at [object Object].promise.Promise.notifyAll_ (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:552:16)
    at goog.async.run.processWorkQueue (--OMITTED ROOT--/node_modules/protractor/node_modules/selenium-webdriver/lib/goog/async/run.js:125:21)
    at runMicrotasksCallback (node.js:337:7)
  1) "before all" hook

where --OMITTED--/jstest/ui/bootstrap.js:49:20 is:
return browser.get('/noop');

+1

Getting this error as well:
TypeError: Cannot read property 'onPageLoad' of undefined
at /home/kev/node_modules/protractor-cucumber/node_modules/protractor/lib/protractor.js:608:25
at [object Object].promise.ControlFlow.runInFrame_ (/home/kev/node_modules/selenium-webdriver/lib/goog/../webdriver/promise.js:1857:20)
at [object Object].promise.ControlFlow.runEventLoop_ (/home/kev/node_modules/selenium-webdriver/lib/goog/../webdriver/promise.js:1729:8)
at [object Object].eval (/home/kev/node_modules/selenium-webdriver/lib/goog/../webdriver/promise.js:2043:12)
at goog.async.run.processWorkQueue (/home/kev/node_modules/selenium-webdriver/lib/goog/async/run.js:125:21)
at process._tickDomainCallback (node.js:399:9)
  From: Task: <anonymous>
    at [object Object].Protractor.get (/home/kev/node_modules/protractor-cucumber/node_modules/protractor/lib/protractor.js:605:29)
    at [object Object].Wanderlust.go (/home/kev/WebstormProjects/TestsForIreland/features/step_definitions/lib/Wanderlust.js:13:31)
    at World.<anonymous> (/home/kev/WebstormProjects/TestsForIreland/features/step_definitions/my_steps.js:29:27)
    at World.<anonymous> (/home/kev/WebstormProjects/TestsForIreland/features/step_definitions/my_steps.js:21:9)
    at doNTCallback0 (node.js:407:9)
    at process._tickDomainCallback (node.js:377:13)
    at Function.Module.runMain (module.js:477:11)
    at startup (node.js:117:18)
    at node.js:951:3

+1

I get this error when trying to return a promise to CucumberJS:
this.Given(/^I am on the Wanderlust website$/, function() {
    return this.browser.get("http://mywanderlust.co/");
});

which is supposed to support this feature:
https://github.com/cucumber/cucumber-js#promises

This looks like it was due to changes in the plugins. We've started assuming that instances of Protractor have a plugins_ property. This is a bug, and the fix will just be checking that it actually exists. See https://github.com/angular/protractor/blob/master/lib/protractor.js#L570

FYI - @sjelin we should make sure that the Protractor class does not rely on the setup done in runner.js.

1487e5abf69bc1540226502aacadc8b3b42b0092 didn't actually fix my problem. I was creating an instance of Runner, then calling .createBrowser() on it with no arguments, which was blindly replacing this.plugins_ with undefined. Turns out I just needed to call .createBrowser(Plugins({})). Would it maybe be better to allow createBrowser() to be called with no arguments?

@cletusw the Runner is not guaranteed as a public API, so we may make breaking changes to it between versions without specifically noting them as breaking changes.
That said - it looks like the internal documentation for the Runner did not get updated along with the plugins change, so we should at least fix that. https://github.com/angular/protractor/blob/master/lib/runner.js#L174 I also think it does make more sense for createBrowser to treat that parameter as optional.

@juliemr I agree, what we're doing is pretty hacky. We'll start looking for a more official way to do it. Thanks!


[Bug]: JS editor replaces new text with original text on occasions

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
There are times when after function renaming or as seen in the video attached, when there's an error that occurs during code addition and during processing the exact error isn't displayed. Instead, new text is replaced by the original text. User has to check browser console to look for errors.

Steps To Reproduce

Try function name refactor and when there's an issue saving the application, there are times when this renaming fails and the function name sis reverted to the origin name
Follow the steps as seen in the video and observe copy paste failing when function isn't updated. Also, error displayed in not what's displayed on browser console. (Code copied is from Async/await from this link)

Environment
Deploy Preview
Version
Cloud

Making this critical since promises is now on prod and any delay in saving application causes this issue to occur.

@ramsaptami please check if still reproducible - this might have been resolved in BE Coders cc @nidhi-nair

Haven't come across this recently. May have been fixed so I'll close it for now and reopen if it happens again


On Graph Neural Networks (GNNs)

From Algorithmic Simplicity:

[x] Why Does Diffusion Work Better than Auto-Regression? - YouTube
[ ] Transformer Neural Networks Derived from Scratch - YouTube
[ ] Why do Convolutional Neural Networks work so well? - YouTube

Are there oversmoothing issues in CNNs, similar to GNNs?

GNN articles:

[x] 18.Limitations of Graph Neural Networks | machine-learning-with-graphs – Weights & Biases
[ ] Graph Neural Network Series 5 — The Future of Graph Intelligence: Challenges and Developments in GNN | by Renda Zhang | Medium
[ ] Issues with graph neural networks: the cracks are where the light shines through | Oxford Protein Informatics Group
[ ] Graph Neural Network and Some of GNN Applications
[ ] Classic GNNs are Strong Baselines: Reassessing GNNs for Node Classification

From JNTUH GNN group:

[ ] Stanford CS224W: Machine Learning with Graphs | 2021 | Lecture 1.1 - Why Graphs - YouTube (Lecture 3 to start)
[ ] [1706.02216] Inductive Representation Learning on Large Graphs
[ ] [1706.03762] Attention Is All You Need
[ ] [1710.10903] Graph Attention Networks

Transformer Neural Networks Derived from Scratch - YouTube
Are there oversmoothing issues in CNNs, similar to GNNs?


Add styling for route_lines_relations to staging versions of main and rail OHM styles

Related to https://github.com/OpenHistoricalMap/issues/issues/439, @Rub21 added a new transport_lines_relations layer to staging vector tiles only.
We'd like to add that to the staging stylesheet to see if it "just works" or if it might cause issues with duplicate labels or geometries.
In theory, the styling can be identical for transport_lines_relations and transport_lines.
One area where we might see some change is around San Jose https://www.openhistoricalmap.org/relation/2703515#map=11/37.2243/-121.8727&layers=O&date=1924-03-21&daterange=1824-01-01,2024-12-31
The railroad lines here have no names, but the Relations have names, so I'm curious to know if we'll start seeing labels once the relations come in

@vknoppkewetzel , here  some examples:   3 relations the use same ways as menbers,
Relation 1: from 1880 to 1925-03-31 -> https://www.openhistoricalmap.org/relation/2798921
Relation 2: from 1925-04-01 to 1930-09-12 -> https://www.openhistoricalmap.org/relation/2798919
Relation 3: from 1930-09-13 to 1955 -> https://www.openhistoricalmap.org/relation/2798922

https://vtiles.staging.openhistoricalmap.org/#13.19/38.86617/-90.16727

Ok confirming That I see it showing up in the demo style (ie ohm-tegola-draft), so next for me it to just, make these have same styling as transport_lines sounds like ✅

I have added the route_lines  layer in staging, according to https://wiki.openstreetmap.org/wiki/Relation:route ,  here e.g https://vtiles.staging.openhistoricalmap.org/#10.15/70.0004/-42.1938
I  have also fixed the issues about the SQL issues that was breaking the areas where relations have wrong date.

Possible idea for bus routes:

I am not sure all the things we want to be styling for route lines, but thinking throught things and for main (which should showcase things generally)

we can generally differentiate route type (ie like the bus) in the main style.` So that could show bus vs tram vs  XYZ. If we had a transit specific style, we could break these down further into network, including operator, show direction.

styling - area where I should be testing: #13.13/51.04667/3.73034

the only real conclusion from reviewing transit centric maps is mostly that for main we have to make sure the route lines are not the main point. de-saturated or more general. a little more complexity with being on top of roads
so... shades of gray....?

I think in main if we label them, ideally ensuring the font is different from road labels. rail isn't labeled in main, so I think it would make sense not to label transit routes in main and to only label in a transit-specific style.
Pausing on my work here in favor of hours for the Expressions work to get done, which Tim will work on next week.

Some of the examples above have been deleted as part of some refactoring. If you need a new example, this railway=rail way is a member of six route=railway relations, such as the South Pacific Coast Railroad. route=train relations are also common, such as the Ferrovia Milano–Bologna.
Note that the individual ways remain tagged with railway=*, representing the physical infrastructure on the ground. route=railway relations track changes in the physical infrastructure’s identity (name, ownership, etc.), while route=train relations track changes in services that run over the tracks. You still need to render the transport_lines layer as usual; the route_lines layer is mostly useful for labels and icons. Dashed or offset lines might work too, but not as a replacement for the usual railway styling of transport_lines.
Similarly, road routes such as these Interstate route relations in the U.S. complement the underlying highway=motorway ways for the purpose of drawing route shields.


Add test coverage

Add test coverage for db interactions 4e5ff80f6a47f9b80da491bf2613dd4628f3baa6

Add test coverage for uploading files to aws s3 f115b2a13d9951c9b6de2af412d4bb6af5e555ff

Testing for aws s3 delete operation 0c5f5cc1892fdf0bcbee90d765b01fc742709c40


Move internal services to its own kustomization

This change will allow renovate to properly update the core release service kustomization file.

Does this mean that we're currently limited by a single URL resource per kusotmization YAML?

@yftacherzog with the current renovate.json config yes. We can always add specific configuration for special cases for for this one I think the seperating the directories is the better approach. In addition, I hope that the internal-services manifests won't by include in core konflux in the future.


WIP: Adding basic travis setup

Adding a basic example travis setup for Luke to review.
The idea is to run separate environment builds for every combination of Play and Scala required.
Separate builds can be setup for other interpreters or requirements.
Publishing can be ran as a single build shell script to invoke multiple commands.

Things still to do:

[ ] Luke to enable Travis CI integration for this repository via GitHub marketplace apps. Free for open source repos, just need admin account to tick the Travis checkbox.
[ ] Add encrypted Sonatype credentials using Travis encrypted variables.
[ ] Encrypt PGP signing key and add to repo.
[ ] Create a build script to allow switching test modes depending on environment configs for different interpreters.
[ ] Create a publish shell script to allow publishing the complete suite of submodules from a single script execution command.

I'm closing this down as it's stagnant and we're most likely going to use github actions instead of travis.