Provide OpenSSL3 support

What would you like to be added:
OpenSSL3 should be integrated into Garden Linux. Thereby, all packages that rely on OpenSSL should also be rebuilt to ensure that these packages are using OpenSSL3 properly, as stated in the OpenSSL Wiki.
Before dealing with packages that depend on OpenSSL3, the OpenSSL binaries itself must be rebuild for Garden Linux. Luckily, OpenSSL3 is already available in debian/experimental. The source package could be reused, to create an equivalent binary package for Garden Linux.

[ ] Build OpenSSL3 package for Garden Linux

Next, there are dependent packages that must be adjusted.
As of now, the following packages depend on OpenSSL in Garden Linux and must be rebuilt:

tcpdump
openssh
python3.9
krb5
kmod
libfido2
cryptsetup
bind9

First, it must be ensured that these packages can be built properly against the new OpenSSL3 version. There were issues with krb5 (fixed in experimental) and libfido2 (fixed in testing) that have already been resolved.  Nevertheless, the other packages could have issues as well, so this needs to be verified.
In addition, there are already pipelines for some of these packages, as one can see here:
https://gitlab.com/gardenlinux
For all others, new build pipelines and project must be established:

[x] tcpdump
[x] openssh
[x] krb5
[x] kmod
[x] libfido2
[x] cryptsetup
[x] bind9

Moreover, the affected builds must ensure, that they are using OpenSSL3 binaries, so the build dependency could look like this for example:
Build-Depends: libssl-dev (>= 3)

This can be achieved, by providing patches for these packages.
By default, builds are using debian/bookworm for installing required packages during the build. One must ensure, that these builds have access to OpenSSL3.
Simply using the OpenSSL3 version from debian/experimental may cause unintended changes because other packages from experimental could be used during the build as well if no proper pinning is applied. For this reason, another solution must be found.
Why is this needed:
This is needed to fulfill #219

Hi everyone,
quick update. We have created package project for each affected Debian packages that must be built against OpenSSL3.
However, we're having problems to build the ARM packages for the following packages:

[ ] krb5
[ ] libfido2

We're analyzing the package build and find the issue. As soon as we found the cause an a possible solution, I would update this issue.

I was able to fix the libfido2 build. Next would be the krb5 one.

Quick update to this issue.
We were not able to built krb5 against OpenSSL3. Therefore, we've decided to remove krb5 from the Garden Linux Repo. As a result, the following packages must be adjusted and re-built as well:

[ ] audit
[ ] bind9
[ ] bind9-libs
[ ] cifs-utils
[ ] curl
[ ] inetutils
[ ] libkrb5-dev
[ ] libtirpc
[ ] nfs-utils
[ ] rsyslog

If these packages get adjusted, there is no need to keep krb5 in the Garden Linux Repo and therefore it doesn't need to be built against OpenSSL3 then.

Quick update; I was able to successfully cross compile krb5 on arm64. Therefore, we do not need to rebuild the mentioned packages without Kerberos support. However, @nanory and me still need to fix some other OpenSSL related packages to make them work on arm64 cross compile.

Hi everyone,
quick update to this issue. We were able to build all packages related to OpenSSL in the Garden Linux Repo against OpenSSL3.
Thereby, the following new package projects have been introduced to Garden Linux:

bind9
bind9-libs
tpm2-tss
tpm2-tools
sbsigntool
socat
rsyslog
python-cryptography
open-vm-tools
nghttp2
libssh2
libssh
libevent
ipmitool
curl
cryptsetup
libfido2
kmod
krb5
openssh
tcpdump

The packages python3.10 and cyrus-sasl are already part of Garden Linux and therefore were already built against OpenSSL3.
After merging the following PR, the new packages will be part of the Garden Linux Repo, so that all Garden Linux packages require OpenSSL3 (libssl3) from now on:

https://gitlab.com/gardenlinux/gardenlinux-package-build/-/merge_requests/72

Hi everyone,
I checked the Garden Linux Repo (dist: dev) and there are no dependencies for libssl1.1 anymore. The libssl1.1 package is still mirrored to the Garden Linux repo, but we will remove it once we have finished the following issue:

https://gitlab.com/gardenlinux/gardenlinux-package-build/-/issues/13

Nevertheless, we can declare this issue as done.


Adding to existing project

Any plans for instructions on adding this to an existing iOS project? I'm curious what the preferred way of adding this to a project would be. I'm mostly using the swift package manager for my dependencies.

The runtime is now up in CocoaPods; instructions for installing can be found on the Github and Help Center pages.


Please give an example of this

Please give an example of how to use this property.

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

ID: ea80097b-5d2e-8f56-8f92-55900361561d
Version Independent ID: 27ad6299-c31d-a079-aa19-48d2d4d70d86
Content: CustomAPIName Property (Page) - Business Central
Content Source: dev-itpro/developer/properties/devenv-customapiname-page-property.md
Service: dynamics365-business-central
GitHub Login: @solsen
Microsoft Alias: jswymer

Hi, thanks for your feedback. This property is not available in Business Central anymore, and this article will be deleted from the documentation since it is no longer relevant. Instead, use the EntityName and EntitySetName properties to set the name of an API endpoint. Thanks!


Scan QR Code does not work on Gnome

Yubico Authenticator version:  4.3.2
Operating system and version: Ubuntu 16.04
YubiKey model and version:  Yubikey 4.3.7
Bug description summary: QR code scan does not work

Steps to reproduce
Install gnome on ubuntu 16.04. Or on Arch Linux. Install the authenticator App
Both XOrg and Wayland doesn't scan QR codes from screen
Expected result
I scan scan QR codes from the screen
Actual results
QR Code was not found

i am getting this error on Manjaro/arch running i3 window manager.  I am unable to scan the qr code in another tiled window next to the yubico authenticator.  Is there a way to get this to work?


WIP refactor(ndt_scan_matcher): isolate align function into ndt package

Signed-off-by: kminoda<EMAIL_ADDRESS>Description
I propose to isloate align function in ndt package.
Current interface of ndt_ptr_ (defined in ndt) is a bit redundant. It would be easier to handle the object by making a new function as
result_pose = ndt_ptr_->align(initial_guess_pose)

I will make this pull request open after https://github.com/autowarefoundation/autoware.universe/pull/2024.

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

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

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

[ ] The PR follows the pull request guidelines.

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

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

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

@TakaHoribe
I want to include tf2_eigen in tier4_autoware_utils/geometry.hpp in this PR, but it seems that the toMsg namespace conflicts in behavior_velocity_planner (with this one: here).
Here's some solutions I have in mind:

change the name of the toMsg function in the behavior_velocity_planner
add a namespace before the function in behavior_velocity_planner (like behavior_velocity_planner::toMsg())
make the new functions in tier4_autoware_utils as a separate file

Do you have any preferences on these three? (or do you have any other better idea?)


[TACHYON-345] Format tachyon worker data directory only.

https://tachyon.atlassian.net/projects/TACHYON/issues/TACHYON-345
Previously we would attempt to format all of the folders in the storage directory path. This will cause null pointers if a folder cannot be read due to permissions.

@shimingfei Is there a case when not only the data directory would need to be formatted?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2912/
Test PASSed.

@calvinjia currently only the data directory needs to be formatted. since the user temp folder is in the data folder.
But considering the meaning of the parameter, I think it is better to format whole WORKER_DATA_FOLDER.


Crash on New Document/Insert

Hello,
I am running adminMongo on my domain.com/adminMongo
it works normally fine but when i add or insert a new document it crashes, i have checked in NPM for errors:
Here it what it shows on crash

TypeError: Cannot read property '0' of undefined
    at /root/developer/screens/adminMongo/routes/index.js:607:104
    at handleCallback (/root/developer/screens/adminMongo/node_modules/mongodb/lib/utils.js:96:12)
    at Object.cb (/root/developer/screens/adminMongo/node_modules/mongodb/lib/collection.js:1041:5)
    at /root/developer/screens/adminMongo/node_modules/mongodb-core/lib/connection/pool.js:446:18
    at authenticateStragglers (/root/developer/screens/adminMongo/node_modules/mongodb-core/lib/connection/pool.js:402:16)
    at null.messageHandler (/root/developer/screens/adminMongo/node_modules/mongodb-core/lib/connection/pool.js:424:5)
    at Socket.<anonymous> (/root/developer/screens/adminMongo/node_modules/mongodb-core/lib/connection/connection.js:294:22)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:146:16)

npm ERR! Linux 4.4.0-22-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node app.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node app.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the admin-mongo package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node app.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs admin-mongo
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls admin-mongo
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /root/developer/screens/adminMongo/npm-debug.log
root@wolf:~/developer/screens/adminMongo# npm start

><EMAIL_ADDRESS>start /root/developer/screens/adminMongo
>

npm starts work fine, i am running it behind a "screen" using "npm start"
Please help!
Thanks!

The error shouldn't be thrown and I can look at that but does your user have the correct permissions to perform those actions?

Yeah, it added the document in the db but crashes it as well.

Ok. Can you please provide the version of MongoDB server you are running?

Also doesn't appear you are using the latest version of adminMongo. There isn't a line 607 in the /routes/index.js file in the latest version.
Can you please update and try again?

Try delete or rename the folder and try again. Same as: https://github.com/mrvautin/adminMongo/issues/103

Will give it a try! Since this shouldn't even happen if the folder name is same as package!
npm install
npm start

Running the app on behind 'screen'
Direct works fine!

That is an NPM issue and occurs on all packages, not just adminMongo.

Yeah i can understand, anyways thanks ^_^


docs(readme): use gif preview image

I don't think it's a good idea. See https://github.com/netless-io/flat/pull/1263#issuecomment-1003300836.

I don't think it's a good idea. See #1263 (comment).

This proposal has recently been revived by our product manager(@Leooeloel ). 🙇♂
Let me explain why in a nutshell 💬:

We have received a lot of feedback from users that the preview images in the README do not clearly describe the functionality of the software and this is leading to a loss of users. 🙅
We don't have more time to complete the video in English and Chinese at the moment. ⌛
This is a temporary solution, it is not permanent. We will take the time to record the video after the Chinese New Year has passed. ⏲

Like https://github.com/microsoft/vscode, https://github.com/wulkano/Kap and https://github.com/OpenEmu/OpenEmu do. They never put a GIF into the readme.
You will make readme worse if you add a bad quality GIF to the readme.
And I don't think the loss of our users has anything to do with the GitHub repo, we should be improving our homepage, not here.

@Leooeloel

Out of instinct I would like to explain a few things, but out of my heart I agree with you.
Ok, please help me close this PR.
@LitoMore @BlackHole1


I am using the "redisson-spring-boot-starter" dependency to read the yaml file and encountered some problems

xxx

Can you describe the issue?


Default salt setup for azure is broken

So, perusing the logs, I get the following error in /var/log/minion on the salt master:
This master address: 'kubernetes-master' was previously resolvable but now fails to resolve! The previously resolved ip addr will continue to be used

I set the hostname kubernetes-master to point to <IP_ADDRESS> in the hosts file (as im on the master) and then I get the following:
2014-12-01 07:46:58,289 [salt.minion      ][DEBUG   ] Attempting to authenticate with the Salt Master at <IP_ADDRESS>
2014-12-01 07:46:58,290 [salt.crypt       ][INFO    ] Generating keys: /etc/salt/pki/minion
2014-12-01 07:46:58,773 [salt.crypt       ][INFO    ] Generating keys: /etc/salt/pki/minion
2014-12-01 07:47:06,106 [salt.crypt       ][DEBUG   ] Decrypting the current master AES key
2014-12-01 07:47:06,114 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:06,449 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:06,896 [salt.loader      ][DEBUG   ] Loaded localemod as virtual locale
2014-12-01 07:47:06,905 [salt.loader      ][DEBUG   ] Loaded groupadd as virtual group
2014-12-01 07:47:06,908 [salt.loader      ][DEBUG   ] Loaded grub_legacy as virtual grub
2014-12-01 07:47:06,911 [salt.loader      ][DEBUG   ] Loaded linux_sysctl as virtual sysctl
2014-12-01 07:47:06,920 [salt.loader      ][DEBUG   ] Loaded debian_ip as virtual ip
2014-12-01 07:47:06,929 [salt.loader      ][DEBUG   ] Loaded parted as virtual partition
2014-12-01 07:47:06,941 [salt.loader      ][DEBUG   ] Loaded aptpkg as virtual pkg
2014-12-01 07:47:06,951 [salt.loader      ][DEBUG   ] Loaded sysmod as virtual sys
2014-12-01 07:47:06,960 [salt.loader      ][DEBUG   ] Loaded zcbuildout as virtual buildout
2014-12-01 07:47:06,963 [salt.loader      ][DEBUG   ] Loaded upstart as virtual service
2014-12-01 07:47:06,965 [salt.loader      ][DEBUG   ] Loaded useradd as virtual user
2014-12-01 07:47:06,974 [salt.loader      ][DEBUG   ] Loaded dpkg as virtual lowpkg
2014-12-01 07:47:06,975 [salt.loader      ][DEBUG   ] Loaded cmdmod as virtual cmd
2014-12-01 07:47:06,997 [salt.loader      ][DEBUG   ] Loaded virtualenv_mod as virtual virtualenv
2014-12-01 07:47:06,999 [salt.loader      ][DEBUG   ] Loaded djangomod as virtual django
2014-12-01 07:47:07,040 [salt.loader      ][DEBUG   ] Loaded syslog_return as virtual syslog
2014-12-01 07:47:07,049 [salt.loader      ][DEBUG   ] Loaded couchdb_return as virtual couchdb
2014-12-01 07:47:07,049 [salt.loader      ][DEBUG   ] Loaded carbon_return as virtual carbon
2014-12-01 07:47:07,050 [salt.loader      ][DEBUG   ] Loaded sqlite3_return as virtual sqlite3
2014-12-01 07:47:07,132 [salt.loader      ][DEBUG   ] Loaded saltmod as virtual salt
2014-12-01 07:47:07,150 [salt.loader      ][DEBUG   ] Loaded zcbuildout as virtual buildout
2014-12-01 07:47:07,156 [salt.loader      ][DEBUG   ] Loaded virtualenv_mod as virtual virtualenv
2014-12-01 07:47:07,190 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:07,471 [salt.crypt       ][DEBUG   ] Decrypting the current master AES key
2014-12-01 07:47:07,473 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:07,768 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:08,076 [salt.crypt       ][DEBUG   ] Decrypting the current master AES key
2014-12-01 07:47:08,079 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:08,389 [salt.config      ][DEBUG   ] Reading configuration from /etc/salt/minion
2014-12-01 07:47:08,508 [salt.config      ][DEBUG   ] Including configuration from '/etc/salt/minion.d/grains.conf'
2014-12-01 07:47:08,519 [salt.config      ][DEBUG   ] Reading configuration from /etc/salt/minion.d/grains.conf
2014-12-01 07:47:08,524 [salt.config      ][DEBUG   ] Including configuration from '/etc/salt/minion.d/log-level-debug.conf'
2014-12-01 07:47:08,524 [salt.config      ][DEBUG   ] Reading configuration from /etc/salt/minion.d/log-level-debug.conf
2014-12-01 07:47:08,525 [salt.config      ][DEBUG   ] Including configuration from '/etc/salt/minion.d/master.conf'
2014-12-01 07:47:08,525 [salt.config      ][DEBUG   ] Reading configuration from /etc/salt/minion.d/master.conf
2014-12-01 07:47:08,798 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:08,805 [salt.crypt       ][CRITICAL] The Salt Master has rejected this minion's public key!
To repair this issue, delete the public key for this minion on the Salt Master and restart this minion.
Or restart the Salt Master in open mode to clean out the keys. The Salt Minion will now exit.
2014-12-01 07:47:13,814 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2014-12-01 07:47:13,820 [salt.crypt       ][CRITICAL] The Salt Master has rejected this minion's public key!
To repair this issue, delete the public key for this minion on the Salt Master and restart this minion.
Or restart the Salt Master in open mode to clean out the keys. The Salt Minion will now exit.

So since #2982 is closed, does this mean that this issue is fixed?

@nterry is this working now?

@brendanburns to me this sounds dated and can be assumed no longer applicable.
@nterry if you needs Kubernetes on Azure, you might like to checkout new guide with Weave and CoreOS, which works fine and it doesn't use Salt.

Please re-open or file a new issue if you're still having problems.


fix: handle existing scheduler more gracefully (ENG-2223)

Sometimes the task for the existing scheduler takes a bit of time to terminate / cleanup, which can leave the scheduler not running. This adds additional retries and some waiting to give it time to take over as the active scheduler

Tested in my sandbox:
[2023-06-23 20:44:19,261][PID:12][INFO][redash.tasks.schedule] Current jobs: refresh_queries, cleanup_query_results, sync_user_details, remove_ghost_locks, empty_schedules, send_aggregated_errors, refresh_schemas, version_check
--
[2023-06-23 20:44:19,261][PID:12][INFO][redash.tasks.schedule] No jobs to schedule
[2023-06-23 20:44:19,262][PID:12][INFO][rq_scheduler.scheduler] Registering birth
[2023-06-23 20:44:19,264][PID:12][INFO][redash.cli.rq] Waiting for existing RQ scheduler...
[2023-06-23 20:44:24,268][PID:12][INFO][rq_scheduler.scheduler] Registering birth
[2023-06-23 20:44:24,270][PID:12][INFO][redash.cli.rq] Waiting for existing RQ scheduler...
[2023-06-23 20:44:29,272][PID:12][INFO][rq_scheduler.scheduler] Registering birth
[2023-06-23 20:44:29,274][PID:12][INFO][redash.cli.rq] Waiting for existing RQ scheduler...
2023-06-23 20:44:32,278 INFO success: scheduler-0 entered RUNNING state, process has stayed up for > than 60 seconds (startsecs)
[2023-06-23 20:44:34,279][PID:12][INFO][rq_scheduler.scheduler] Registering birth
2023/06/23 20:45:01 [scheduler_healthcheck] Received TICK_60 event from supervisor
2023/06/23 20:45:01 [scheduler_healthcheck] Performing `RQ Scheduler Healthcheck` check for process name scheduler-0
2023/06/23 20:45:01 [scheduler_healthcheck] SchedulerHealthcheck: Scheduler healthcheck: Periodic jobs ok? True (8/8 jobs scheduled). ==> Is healthy? True


[WIN 11] Dark mode full integration

Description of the new feature / enhancement
Top bar of PowerToys Settings is white.
Would be nice, darker, matching the W11 settings app.
Scenario when this would be used?
Always
Supporting information
No response

Duplicate of #732

Thanks for your suggestion! This has been raised in #732. To centralize the discussion thread, please go to that thread for discussion.


Flutter 3.7 can't compile

Before reporting a bug, please test the beta branch!
Bug description
Running all the versions specified in the beta branch but it does not seem to run with Flutter 3.7 because of the dependency versions it uses

Could not build the precompiled application for the device.
Error (Xcode):
../../.pub-cache/hosted/pub.dev/graphql-5.1.1/lib/src/links/websocket_link/websocket_client
.dart:470:7: Error: The non-abstract class 'GraphQLWebSocketChannel' is missing
implementations for these members:

Specs
Using what's in defined in beta readme

Got past this issue by not specifying the version and letting it be set to the highest resolvable which was:
version: "0.14.1-alpha+1672756470474"

Actually I will reopen this and feel free to close/create a new more relevant topic.
But in general this has resulted in very specific versions for packages to get it to work correctly
dependencies:
  flutter:
    sdk: flutter
  artemis: '>=7.0.0 <8.0.0' # only if you're using ArtemisClient!
  json_annotation: 4.7.0
  equatable: 2.0.5
  gql: 0.14.1-alpha+1672756470474
  graphql: 5.1.3

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_lints: 2.0.1
  build_runner: 2.3.3
  json_serializable: 6.5.4

I would suggest to enable us to use the latest versions without locking us to lower versions of the packages.

This was mainly an issue with how Artemis is using an old version of gql. So was hard to get a later version of graphql. Hence how the latest one that is used is 0.14.1-alpha+1672756470474.
Left it open as I am not sure what the approach is here but it's very hard to keep other dependencies updated when Artemis is "pinned" to old versions.


Fix syntax

Depends on nbuilding/N-lang#60

Actually it probably doesn't but whatever

Actually it probably doesn't but whatever


Maestral can't start due to upstream changes in python-keyring

Describe the bug

Possible regression caused by upstream changes to python-keyring, which python-keyring-alt depends on. Tracked by
this issue. Hopefully should be a speedy fix as there's already a PR addressing it.
To Reproduce

simply run maestral start
Expected behaviour

Error free start
System:

Maestral version: 1.6.3
Python version: 3.10.6
OS: EndeavorOS
Desktop environment: i3
PyQt version (for Linux GUI): 6.3.1

Additional context

Relevant error from the python traceback
ImportError: cannot import name 'properties' from 'keyring.util' (/usr/lib/python3.10/site-packages/keyring/util/__init__.py)

Quick and dirty solution until python-keyrings-alt (or your distro equivalent) gets updated to 4.2.0:
sudo pip install keyrings.alt --upgrade

Tested on Arch.

Issue seems to be resolved after updating to python-keyring 23.9.3-1

Excellent, thanks for the update!


fix: Eloquent model toJson fix for existing errors.

The Problem
Related to this issue:
https://github.com/laravel/framework/issues/46368
When casting an eloquent model to JSON using the toJson method and passing in the JSON_THROW_ON_ERROR option, if there was a previous, unrelated JSON error, an erroneous exception would be thrown.
The problem related to this line here:
public function toJson($options = 0)
{
    $json = json_encode($this->jsonSerialize(), $options);

    // This line will always trigger if JSON_THROW_ON_ERROR is passed
    // as an option, and an existing json_encode or json_decode error exists
    if (json_last_error() !== JSON_ERROR_NONE) {
        throw JsonEncodingException::forModel($this, json_last_error_msg());
    }

    return $json;
}

The situation transpires when someone has a previous JSON error, but passes the JSON_THROW_ON_ERROR flag to the toJson function:
// ... earlier on

$array = @json_decode('{some badly formed, but non essesntial, JSON'});

// now json_last_error() is not JSON_ERROR_NONE
// ... and then later in the code ...

$widget = new Widget(); // An Eloquent Model
$widget->name = 'bob';

try {
    $json = $widget->toJson(JSON_THROW_ON_ERROR);
} catch (JsonEncodingException $e) {
    var_dump($e->getMessage()); // Syntax error
} catch (\JsonException $e) {
}

This happened to me in developing using the @js directive from Livewire, which uses the JSON_THROW_ON_ERROR flag as a required option.
I think because PHP supports both ways of handling JSON errors, it should be handled both ways by Laravel as well.
Why fix this?
A good example is the @js directive in Livewire. This will always fail if there has been any json error without the use of JSON_THROW_ON_ERROR.
Considering that handling json errors in both ways is still supported, Laravel should support this as well.
Proposed solution
The solution is fairly simple - we see if JSON_THROW_ON_ERROR was passed as an option, and only throw on error in that case:
public function toJson($options = 0)
{
    $json = json_encode($this->jsonSerialize(), $options);
    if (! ($options & JSON_THROW_ON_ERROR)) {
        if (json_last_error() !== JSON_ERROR_NONE) {
            throw JsonEncodingException::forModel($this, json_last_error_msg());
        }
    }

    return $json;
}

Tests
Included in this PR are 7 tests appended to DatabaseEloquentModelTest.php to test various toJson parameters and outputs, including exception cases.
Integration tests are hard to write, because this is such a core feature.
Related
This contrived example demonstrates the problem well:
$object = @json_decode('[invalid json]');

try {
    $json = json_encode("\xB1\x31", JSON_THROW_ON_ERROR); // an invalid UTF-8 sequence
} catch (JsonException $e) {
    var_dump($e->getCode()); // 5
    var_dump($e->getMessage()); // Malformed UTF-8 characters, possibly incorrectly encoded
}

var_dump(json_last_error()); // 4
var_dump(json_last_error_msg()); // Syntax error

Just to note, this is no longer a breaking change (unlike my previous PR).
For future versions, though, I would consider getting rid of JsonEncodingException and just replacing it with JsonException.

No plans to change this right now.

This is a shame - it renders the @js directive in livewire virtually useless!


Wallet Connection Termed Balance Still Showing

The connection between dapp and wallet has been terminated, but the balance of the wallet is still displayed.

Thanks for being awesome!


Perform CentCom-ification

Replace most of "CentCom" variants with canonical version
Fixes #9807.
Does not touch some stuff not seen by players.
Changelog
:cl:

fix: It's CentCom. Not Centcom, centcom, CentComm, Centcomm or centcomm.

lgtm


Configure Github Actions CI

Set CI rules

Hi @felipecesr , I can work on this issue. If you would provide me an insight on what are the steps needed that would be helpful.


How do I reset all aspect of modal wizard on show

when the modal popup shows, if the user clicks somewhere outside of the modal window, the modal window goes away. if  the user shows the modal again, it remembers the last data entry and the active tab it was on. What do I need to do so the modal doesnt remember anything and acts as if this is the first time user has clicked on it

@aashrafCL thank you for using our product. We just did the design over Bootstrap Wizard: http://vinceg.github.io/twitter-bootstrap-wizard/# . Please check their documentation on how to reset or jump to the first step. Hope this helps.
Best,
Alex


Add KnownLayout trait

In its initial form, the KnownLayout trait encodes type layout information slightly more complex than can be gleaned from any arbitrary T: ?Sized. This allows it to support not just sized and slice types, but also "custom DSTs" (those with fixed-size fields followed by a trailing slice type). This is the first step to supporting various operations on arbitrary custom DSTs.
Makes progress on #29

There's a bit of different logic to handle Layouts now that could use a glance before you approve again.


Fix for 12.17.15 -> 15.10.X upgrade

Description
[Please describe what this change achieves]
Fix for resolving 12.17.15 upgrade 15.10.X
https://buildkite.com/chef/chef-umbrella-main-chef-server-full/builds/485#0191995c-c932-455c-a1bf-f5a0a798d6e2
Issues Resolved
https://buildkite.com/chef/chef-umbrella-main-chef-server-full/builds/485#0191995c-c932-455c-a1bf-f5a0a798d6e2
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/main/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

Adhoc: https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/6683


Fix for escape fields when they are created in tsbd.Point

I receive panic error when I'm trying to write message with commas and internal quotas.
# curl -G 'http://localhost:8086/query' --data-urlencode "q=CREATE DATABASE mmm"
{"results":[{}]}

# curl -XPOST 'http://localhost:8086/write?db=mmm' --data-binary<EMAIL_ADDRESS>curl: (52) Empty reply from server

# cat t.g
{
  "database" : "mmm",
  "retentionPolicy" : "",
  "points" :  [ {
    "measurement" : "http_status",
    "tags" : {
      "service" : "identity-service",
      "hostname" : "host1",
      "url" : "http://localhost:35774/"
    },
    "timestamp" : "2015-06-16T09:28:26.000Z",
    "fields" : {
      "value_meta" : "{Hello\"{,}\" World}",
      "value" : 1.0
    }
  } ],
  "tags" : { }
}

The problem is that with creation tsbd.Point:
func NewPoint(name string, tags Tags, fields Fields, time time.Time) Point {
        return &point{
                key:    makeKey([]byte(name), tags),
                time:   time,
                fields: fields.MarshalBinary(),
        }
}

In function MarshalBinary is error. Because there are added quotas on begin and end of field.
Problem is that we receive string:
"{Hello"{,}" World}"

instead of expected by function scanFieldValue
"{Hello\"{\\,}\" World}"

The biggest problem occurs later because with wrong string function scanFieldValue splits string in comma and we receive:
panic: unsupported value type during encode fields: <nil>

goroutine 29 [running]:
github.com/influxdb/influxdb/tsdb.(*FieldCodec).EncodeFields(0xc2081084b0, 0xc208146e70, 0x0, 0x0, 0x0, 0x0, 0x0)

@jwilder

I rebased to your master and added new commit to fix what is escaped and in which order.
I create new pull request https://github.com/influxdb/influxdb/pull/3073


Unable to install Hide Private Properties from AssetLib, no files available to install

Hello, I hope it's a fine day for you. I was about to download the small "Hide Private Properties" addon from the AssetLib. It may be what I need. However, it is not possible to do so. Due of the way all these addons are exposed, no files are available to install.

It is not something particularly terrible for me, as I can just take inspiration from the plugin's source to create something more, but I just felt like it was worth pointing out to the creator.

This only affects the in editor asset lib.

https://github.com/godotengine/godot/issues/81311

Will be fixed in Godot.


Increae recent show in welcome page

I constantly works with lot of different projects, which span multiple workspace. #228987 will help.
However, I still have a lot frequent use workspace pushed out of the recent list in the welcome page, and I have to click "More...".

Option to increase number of recent shown in welcome page
Ability to "star"/"lock" recent workspace so they always show up on top of the recent list.

Did you try https://github.com/microsoft/vscode/issues/229264#issuecomment-2365125960?

@gjsjohnmurray Yes, that increase the list from 6 to 10 only. The issue is for people with multiple active workspace, and at the same time have to go through a lot of temporary workspace, the recent list get "polluted" fast. So both length and "star" are useful.

On Windows I pin VS Code to the task bar and use the native feature there to pin favourite workspaces. Don't know whether Mac has a similar capability.

/duplicate of #229264


Added publicPath support to modern-web-webpack-plugin

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution. I think the change is ok. What problem are you solving with this?

quick question why a separate parameter and not part of the options object?

quick question why a separate parameter and not part of the options object?

Not really into the webpack conventions so I was in doubt if I should create option.publicPath or access compiler.options.output.publicPath. What would you suggest @daKmoR?

Thanks for your contribution. I think the change is ok. What problem are you solving with this?

We're actually integrating the open-wc stuff into an existing project which requires different paths when the website is actually served.

Ah, that makes perfect sense to add in.
@daKmoR the separate parameter is because it's not a user provided option, it's an option retreived inside the plugin from the webpack config.
Good to go, you just need to run npm run format from the repository global to format your files.

@LarsDenBakker I'm unable to do so, if I checout the project run npm install and then npm run format I get: ESLint couldn't find the plugin "eslint-plugin-html"..
After adding that one: Error: Cannot find module 'babel-eslint'
Not sure what you are expecting, maybe you can run it your self?

@paales I will do it :hugs:
can you sign the CLA then I will make sure to release it?

@daKmoR Signed :)

thank you for your contribution ❤
PS: sorry for the delay 🙈  better late then never right 😅 🤣


Update MatchApi.java

Correcting the IncludeTimeline "if".

Nice find, thanks!

Thanks for catching this.


Grid: dot in field-value not working

If you define a column with a dot in the field-value then the column values will not shown:
columns: [
{ field: "recid", caption: "Id", size: "50px" },
{ field: "users.id", caption: "Users Id", size: "50px" },
...
],
records: [
{ recid: 1, "users.id": 1, ...},
...
]

I am also facing the same issues when tried to load data into columns having dot in the name..

escape it. Dot in field names are considered nested objects. So, if you define column as users.id, then it should have this structure
records: [
   { recid: 1, users: { id: 1} },
...
]


'waiting' message while waiting for events

Sometimes it takes google a long time to deliver the events (or maybe another issue?).  In the meantime, the calendar should somehow indicate that the events are not yet populated.

The latest commit (5e45868848703e3d745e28a5f7bfc22595483c93) resolves this.  It can be further customized if anyone requests specific behavior for the status indicator.
Currently, the behavior is that "Loading" appears when the page loads.  Immediately after the events are populated, and only in the case where there exists at least 1 event, the "Loading" sign disappears.

I would like to change it so the server caches the events.  When a request comes, it sends the most up-to-date events along with the time-of-update.  The server will routinely update the events (maybe every 5 minutes) and when somebody calls the page, it can also trigger the server to update the events, so that the next time they reload, they hopefully will get the newest stuff.
On the user side, "Loading" will be replaced by "Last updated --datetime--"

This is now good, with the exception that I want the status to be truncated when the date is today.  For example, it need not say "Tuesday July 3 2:00 PM" when that is today.  Rather, in that case, it can just say "2:00 PM".

Issue persists.  Example:

I go to page
It says "loading", but doesn't load even after 15 seconds
I reload page twice
It gets evens and says "last update 1 month ago" or similar
I reload again
It gets events and says "last updated 1 minute ago"

Now we have a daemon that periodically fetches events from google and stores them in a database.  The server simply fetches the events from the database.


Possible to add gaps?

Hi!
Is it possible to add gaps within a contig so to split the contig into two sequences (coordinates given by the user) that will be part of the same scaffold?
Thanks for the help
Valentina

Please take a look at the brand new instruction for masking sequences!
https://github.com/vgl-hub/gfastats/tree/main/instructions#mask
The logic for this one is less trivial than you'd think, because gaps typically would connect segments, not be embedded in them. Therefore I introduced the idea of 'inner gaps', that is a gap that is within a segment, so the GFA segment is literally split in two, before and after the gap, as part of a path. You can check out how this looks like on this test file:
gfastats testFiles/random1.fasta -k testFiles/random1.mask.sak -ogfa2
Please try it out and let me know if it works for your case and does what is expected.
All the best


General support

I've been trying to get someone from support to help me for literally years on this product.  But I never even get a reply.  Who do I call, email, or whatever to get some support?  I love this product!  It's so much better than anything else I've tried for my purposes, even the paid solutions.  PLEASE...  I think my issues could be resolved if I can just find someone to talk too.
Many thanks,
Troy
502.386.0185
<EMAIL_ADDRESS><EMAIL_ADDRESS>Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 13054e70-b770-f803-4596-6d0ab3fe865c
Version Independent ID: 3e474a79-1129-fdf6-bbd4-a12e0e7ebe27
Content: Change the listening port in Remote Desktop
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/change-listening-port.md
Service: windows-server
Sub-service: remote
GitHub Login: @robinharwood
Microsoft Alias: wscontent

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Update examples to the latest published package versions

@kevmoo Hey, I mentioned in our chat room yesterday that I was doing this? I have a series of PRs now (starting with #193) where I have been adding updates and testing as I update all the samples and the dartfn templates. I did the work for #193-#196 yesterday and just submitted those, and now I'm updating the cloudevent sample and dartfn templates.


自定义筛选清空后 icon未清空高亮

自定义筛选清空后 icon未清空高亮
https://codesandbox.io/s/lively-glitter-w2f71u?file=/demo.js

i find the reason, i will create pr to solve it.

@liangjiax your code has error
setFilterKeys(value ? [value] : []);

the value is array, so it don't excute [], you should update code below
setFilterKeys(value.length ? [value] : []);
``

@liangjiax your code has error
setFilterKeys(value ? [value] : []);

the value is array, so it don't excute [], you should update code below
setFilterKeys(value.length ? [value] : []);
``

感谢感谢  我也发现是我的问题了


b:racer_experimental_completer is ignored (neovim)

I tried to put let b:racer_experimental_completer = 1 in ftplugin/rust.vim, but it seems to have no effect. Replacing b: with g: solves it though.
Expected behaviour:
b:racer_experimental_completer should be respected.

Why the feature is needed?
Please explain it.

This is the feature.
https://github.com/racer-rust/vim-racer/blob/master/README.md

Not really strictly needed for me, g: is suitable for my purpose, but expected to work. It would help to keep cleaner configs without cloggering global variables namespace though.

But the other variables have same problem.
I don't want to add the variables.

I'm sorry for the dumb question, but how do i get it working? I've installed the plugin and set that flag to true, but it doesnt happen automatically. Do i need to add something to vimrc?

Do i need to add something to vimrc?

let g:racer_experimental_completer = 1 in your vimrc.


Add button color prop and new error style button

title: "type_of_change(scope_of_work): ticket title"
reviewers: "sds-eng, sds-design"
Summary
Structural Element (Base, Gene, DNA, Chromosome or Cell)
Shortcut ticket: sh-XXXX
Copy ticket descirption here
Checklist

[ ] Default Story in Storybook
[ ] LivePreview Story in Storybook
[ ] Test Story in Storybook
[ ] Tests written
[ ] Variables from defaultTheme.ts used wherever possible
[ ] If updating an existing component, depreciate flag has been used where necessary
[ ] Chromatic build verified by @sds-design

So, I'm having some concerns around this approach where we are trying to generalize the code around colors.  I'm running into some conflicts between the secondary and error styles that can't seem to be resolved.  Basically, to get the error button to display correctly, I need to add this CSS to the base button:
      background-color: ${color[400]};
      border: ${border[400]};
      color: white;

Those attributes weren't set before.  I believe we were basically relying on passing color to MUI, and allowing their defined primary and secondary to set those attributes.  But when I set those attributes, it breaks the secondary button (it basically turns it into the primary button).
The other thing that breaks the secondary button is that we have to pass through the color prop to MUI as well, because we were relying on styling from that.
Due to these issues, I don't think we can generalize the base button code to support the new error type, and I think it may be easier and clearer to just add error button blocks in the switch statement in src/core/Button/index.tsx.
However, that leads to some other issues, and we'd have to check specifically for color in addition to sdsType and sdsStyle, and make sure it's done in the right order so it doesn't fall through to the wrong part of the switch statement.  I'm thinking that we actually should just add error as an sdsType option, but the switch statement can create it with the needed style

Hey Jerry! Thanks so much for your detailed investigation and writeup 🙏 Updating the button to support more color variants is definitely way more work than expected 😭
I happened to find a thread talking about the same problem and their solution seems to work for us too!
I opened a new draft PR to test it out 🙆♂
PTAL thank you!

Thanks for taking a look and putting the other PR together!  I'll check it out!

Real PR: https://github.com/chanzuckerberg/sci-components/pull/113


allow ability to catalog packages from /var/lib/dpkg/status.d/

Some debian-based variants (such as Google's Distroless images)
don't write a single file to /var/lib/dpkg/status, but rather write
a file per package to /var/lib/dpkg/status.d/
related to #44
Signed-off-by: Weston Steimel<EMAIL_ADDRESS>

Example output:
$ syft gcr.io/distroless/base:latest
 ✔ Loaded image
 ✔ Parsed image
 ✔ Cataloged image      [6 packages]
NAME        VERSION          TYPE
base-files  9.9+deb9u13      deb
libc6       2.24-11+deb9u4   deb
libssl1.1   1.1.0l-1~deb9u3  deb
netbase     5.4              deb
openssl     1.1.0l-1~deb9u3  deb
tzdata      2021a-0+deb9u1   deb

Hi @westonsteimel, thanks so much for this!
It'd be great to have at least one automated test that verifies this added code does what it's intended to. We'd be happy to help you with this, if you'd like. At a high-level, I'd suggest adding an integration test in ./test/integration/..., and we could even use gcr.io/distroless/base as the fixture image (although we'd want to pin to an exact image digest or tag rather than latest).

Hello @luhring!  Yes I did suspect having some sort of test would be desirable, I just wasn't quite able to wrap my head around how to implement one, so any help here would be greatly appreciated!  I'll try having a look at some of the existing integration tests when I have a chance this week to see if I can figure it out, but in the meantime if anyone has further suggestions or wants to have a go at it, I'd be perfectly happy to accept that.  Thanks!

Hi @westonsteimel, thanks, that makes sense. Let me dive into this again in the next few days and I'll get back to you with a recommendation — hopefully a good one! But no promises 😄

Sure, I did have a try with the existing "find dpkg packages" testcase in https://github.com/anchore/syft/pull/332/commits/6c80f9910f9a6cf26392c6d0ec2fb21293628e55.  I just added two new packages under /var/lib/dpkg/status.d/, so that verifies that syft detects packages from /var/lib/dpkg/status as well as any under /var/lib/dpkg/status.d/, which is really what I intended.


Inspect possibility to remove lifetimes from Remote and Querier

          Can we use `&'static dyn` here instead of adding a new lifetime to the structure? Let's consider removing the reference in the implementation of Querier for dyn Trait as well.

Originally posted by @kulikthebird in https://github.com/CosmWasm/sylvia/pull/367#discussion_r1592406573
#[derive(Serialize, Deserialize, Derivative)]
#[derivative(Clone, Debug, PartialEq)]
+ pub struct Remote<Contract> {
- pub struct Remote<'a, Contract> {
+   addr: std::borrow::Cow<'a, cosmwasm_std::Addr>,
-    addr: std::borrow::Cow<'static, cosmwasm_std::Addr>,
    #[serde(skip)]
    #[derivative(Debug = "ignore")]
    _phantom: std::marker::PhantomData<Contract>,
}

+ pub struct BoundQuerier<C: cosmwasm_std::CustomQuery, Contract> {
- pub struct BoundQuerier<'a, C: cosmwasm_std::CustomQuery, Contract> {
+    contract: &'static cosmwasm_std::Addr,
+    querier: &'static cosmwasm_std::QuerierWrapper<'static, C>,
-    contract: &'a cosmwasm_std::Addr,
-    querier: &'a cosmwasm_std::QuerierWrapper<'a, C>,
    _phantom: std::marker::PhantomData<Contract>,
}

I think that we could port https://github.com/CosmWasm/sylvia/pull/367#discussion_r1595321827 to the main branch to solve this issue


No transformation class/Fuction included in the package?

i want to resize and do many transformation with my image but i cant find any example related to it on the readMe file

okay, Thanks

Hi @unicodeveloper, how can I remove background with Cloudinary AI


Bump tendermint version to 0.2.40

Describe your changes and provide context
Testing performed to validate your change

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 62.61%. Comparing base (7ee257f) to head (d24fce1).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1448      +/-   ##
==========================================
- Coverage   62.64%   62.61%   -0.03%
==========================================
  Files         257      257
  Lines       16867    16867
==========================================
- Hits        10566    10562       -4
- Misses       5798     5801       +3
- Partials      503      504       +1

see 1 file with indirect coverage changes


Make into an installable PyPi package

For general use as a tool, users should be able to pip install pshtt and then be able to run the CLI right away, without cloning the repo and setting up developer dependencies.

@konklone I'm working on this now!


[spatial] Update Visium ingestor to use points instead of geometries for spots

[sc-49633]

The Visium spots will be left as a geometry dataframe.


add get_latest_release(), create Releases class in test_github.py

It's pretty common looking for the latest release only. It would be useful to have a fast method for it without looking for some identifier or getting all releases. Here it is.
I also think that tests for releases were in the incorrect class. Release tests which are increasing deserve its own class.

@marusinm Could you please rebase to the master?


Yor could use a check mode (see PR#364)

A while back, in #254, I suggested that a dry-run mode would yor more useful, because I wanted to:

write a test that calls yor --check or yor --dry-run and examine its exit code.

@hi-artem implemented the "don't actually change anything" side of the feature in #257, but it does not do anything useful with the exit status so we're left with parsing the CLI output.
I've submitted #364, which implements a check mode, which causes the output to be 0 if nothing changed and non-zero (via log.Error if anything changed (would have changed, if --dry-mode).
Feedback welcome.

hey @hartzell,
the problem with changing the exit code is that you won't know if a real error occurred in the code which can be confusing.
if going for it, it makes sense to do only when running dry mode and not in regular run.
as for naming I would prefer calling it validation instead of check

@hartzell what I meant is not to unite the flags. but when adding the new one you've suggested - make it turn on the dry mode as well, that way you will assure that this mode will have no side affects and only validate that there are no changes to be made

@omryMen -- done.

Closed by #364.


Some fonts are not displayed and do not prompt questions

Some fonts are not displayed and do not prompt questions
want to ask that it doesn't support mobile mode. It can't be scaled when accessing in mobile [mode](url)

wmp72test2.zip


Move commits and releases tab one down

Gogs version (or commit ref): <IP_ADDRESS>4
Git version: 2.1.4
Operating system: Deb 8
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Please move the tab "Commits" to the row of the branches.
I'm always wasting a lot of time looking fot the commits tab, because every git webui I used before have Branches and Commits on the same (bottom) line.
Current:

Expected:

Just for ref: (GitHub)

Just for ref: (GitLab)

Please lets discuss about this. I would really appreciate if this would be done, and would save me and other people working on my gogs some time.

Thanks, I think you should rename title to move down releases as well. I prefer go GitHub way.

@Unknwon  Thanks for feedback, great to hear that, I am also for moving releases aswell.

Just adding my voice to this, I agree that Commits should be moved down because currently there's a UI-bug (inconsistency) related to that.
 Though if you select another branch under Code and go to Commits the correct branch is selected :confused:
Moving it down to to be a sub-page of Code would have them actually linked, and I can file a real bug-report for this :laughing: (I will anyhow :wink:)
On the other hand, Releases shouldn't be moved down IMO, since they are not linked to Code in the same way commits are.

Commits should be moved yeah, Releases can, but it's no must.
Sure there is an relation:
Branches = multiple timelines of code
Commits = one timeline of code
Releases = point in an timeline of code

Note: Should add branches at the same time.

Related: https://github.com/gogits/gogs/issues/4154

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io/gogs/gogs.

@Unknwon
Thanks!

No problem.
Right now only two items look somewhat strange, but will be fine once we have branches and contributors.


Add requirement of dynamic volume provisioner?

Later at https://www.kubeflow.org/docs/started/k8s/kfctl-k8s-istio/ it is stated that "This Kubeflow deployment requires a default StorageClass with a dynamic volume provisioner."
I wonder if this should also be stated in the requirements page: https://www.kubeflow.org/docs/started/k8s/overview/#minimum-system-requirements
Or is this just specific to the Kubeflow Deployment with kfctl_k8s_istio ?

I am a little bit confused at the end of the https://www.kubeflow.org/docs/started/k8s/kfctl-k8s-istio/ page.
It seems to me that dynamic volume provisioner is after all optional. There seem to be instructions to manually manage PVs instead of dynamic volume provisioner.
Maybe the document could be clarified a bit to make it obvious if DVP is actually a requirement?

@sirkkalap Hi Petri - DVP is valuable for several workflows i.e. Do you expect to use Persistent Volumes for your data (in notebooks).   Where are you deploying i.e. GCP, AWS, other ?

/assign @Bobgy
(I think @jlewi has previously committed to the doc)
It probably makes sense to add to the minimum requirements/prerequisites that the KF deployment needs a default StorageClass with a dynamic volume provisioner (DVP) - at least in the beginning of docs/started/k8s/kfctl-k8s-istio/ (and possibly to https://www.kubeflow.org/docs/started/k8s/overview/#minimum-system-requirements) as @sirkkalap said.
WDYT?

I think there are a couple applications that will dynamically provision a volume. For example the jupyter web app will dynamically provision a PD to use as storage for your notebook server.
The jupyter web app supports the case where you don't have a dynamic provisioner.

@8bitmp3 I would suggest routing this issue to the OWNERs of kfctl-k8s-istio deployment. Is the OWNERs file up to date?

@jlewi The OWNERS should be good for getting started guides (docs/started/) - we've recently refreshed it. I don't think the kfctl-k8s-istio guide (docs/started/k8s/kfctl-k8s-istio/) contributors - @swiftdiaries, you and @sarahmaddox - are listed there though. Shall we ask @swiftdiaries to look into this issue?

https://github.com/kubeflow/website/tree/master/content/en/docs/started/k8s
Is missing an OWNERs file. We should figure out an appropriate OWNERs file and then route this issue to it.
I'm not an OWNER of this deployment platform.
I think OWNERs are @yanniszark @swiftdiaries

@jlewi OWNERS added.
@yanniszark @swiftdiaries can you take a look at this issue please? Much obliged

/assign @yanniszark @swiftdiaries

/unassign @Bobgy

@Bobgy How do you think we should proceed here? The issue is getting stale. Cheers

/assign @yanniszark @swiftdiaries
Friendly ping again.
@8bitmp3 don't worry, you don't need to make sure everything is fixed. The owners should follow up with these issues at their own schedule


Apply BackdropFilter for dropdown menu - "frosted glass" style

Hello,
I would like to apply a "frosted glass" effect to the dropdown menu, but I cannot find a way to customize the builder of it.
Is there a way to apply a BackdropFilter to the Container of the dropdown menu, achieving the kind of result shown in the screenshot below?

You can now do:
dropdownStyleData: DropdownStyleData(
  elevation: 0,
  decoration: BoxDecoration(
    color: Colors.grey.shade200.withOpacity(0.5),
  ),
  dropdownBuilder: (ctx, child) {
    return ClipRRect(
      clipBehavior: Clip.antiAlias,
      child: BackdropFilter(
        filter: ImageFilter.blur(sigmaX: 10.0, sigmaY: 10.0),
        child: child,
      ),
    );
  },
),


[10.x] Use container to build PendingDispatch

Problem
There is no way to swap in an implementation of PendingDispatch currently aside from duplicating the code and setting it as an alias. This is a challenge, as it means every time there's a tagged release on this repository, the developer needs to check for changes and if so, modify their PendingDispatch implementation for parity.
Why would I ever want to do that?
Working on a rewrite of an application that will have both a beta version and a legacy application in production running simultaneously.
Jobs dispatched from the beta application to the legacy application need to modify the PendingDispatch@__destruct() method so that they are dispatched to a separate Horizon database. (See https://github.com/laravel/horizon/pull/1271)
Solution
Build PendingDispatch from the container when needed in Dispatchable.
Alternatives

Add a protected static variable to Dispatcher that holds the FQCN for PendingDispatch and a static method to change it. Doesn't feel particularly clean to me and conflicts with how most binding is done in the framework.

Write a queue() method on things that need to dispatch to different Horizon Redis database
This would be easier if Dispatcher@pushCommandToQueue() were public.

Write a custom Dispatchable implementation
Again, entirely duplicating something that may change with a framework release.

I'm not sure I totally want to add in all these customization for a single application that is being rewritten.

I'm not sure I totally want to add in all these customization for a single application that is being rewritten.

I found a way to work around it. But I do think that there's a discussion to be had about what is built from the container and what isn't. You completely remove the user land's ability to tweak what's under the hood.
Which is nearly the equivalent of declaring a class as final.
@taylorotwell


Listen to all YouTube videos

There aren't a lot of videos yet, so I plan to watch them all. That will help me identify some evergreen content we can share more regularly instead of just once, and see if there are content gap opportunities. There are already ways to group content in playlist (like the four intros to API builder), fix 404 links in description and harmonizes the titles of the same series...

Things to do when I'll have access:

give same title structure for the API builder series;

add the series in an API builder playlist in the right order + the API Builder in 2 minutes videos with a mention/link in the description about the longer series;
update the description to link to other parts of the series, and have the same kind of information in all descriptions;


Screenshot give black image on iOS

The window.screenshot gives black image on iOS. There is multiples issues behind:

Because the method can be called anytime, it will be likely called on the back buffer. Unfortunatelly, it seems that if we don't retain the backing buffer, we'll get black pixels. 2 possibles approachs:

Retain the backbuffer by setting SDL_GL_RETAINED_BACKING to 1 (even if it's say deprecated, this is still used in the implementation of UIKit video for setting kEAGLDrawablePropertyRetainedBacking)
Force window to draw before calling glReadPixels by calling Window.dispatch('on_draw')

The glReadPixels use system width / height of the Window, but with retina screen, the real pixel size is not reflected. The good value to use is the one returned by self._gl_get_size().
The glReadPixels call use GL_RGB, it works on Desktop, but not on mobile, as the only available output format is GL_RGBA. The call will just fail.
Saving the image doesnt work if the SDL surface have stride, which is the case for the iPhone 6 retina screen.

Window have a method to save bytes into PNG, but limited to pure RGB image
ImageLoader have methods to save image, but the one in image_io breaks with RGB image

Maybe this issue could be split in multiple one once we know exactly what to do. I'll post a snippet on a code i did in user side to get a valid screenshot on iOS.

Don't you just <3 iOS :)
On Tue, Sep 1, 2015 at 1:23 PM, Mathieu Virbel<EMAIL_ADDRESS>wrote:

The window.screenshot gives black image on iOS. There is multiples issues
behind:

Because the method can be called anytime, it will be likely called on
the back buffer. Unfortunatelly, it seems that if we don't retain the
backing buffer, we'll get black pixels. 2 possibles approachs:
  Retain the backbuffer by setting SDL_GL_RETAINED_BACKING to 1 (even
  if it's say deprecated, this is still used in the implementation of UIKit
  video for setting kEAGLDrawablePropertyRetainedBacking
  -

  Force window to draw before calling glReadPixels by calling
  Window.dispatch('on_draw')
  -

The glReadPixels use system width / height of the Window, but with
retina screen, the real pixel size is not reflected. The good value to use
is the one returned by self._gl_get_size().
The glReadPixels call use GL_RGB, it works on Desktop, but not on
mobile, as the only available output format is GL_RGBA. The call will
just fail.
Saving the image doesnt work if the SDL surface have stride, which is
the case for the iPhone 6 retina screen.

Window have a method to save bytes into PNG, but limited to pure RGB
image

ImageLoader have methods to save image, but the one in image_io
breaks with RGB image

Maybe this issue could be split in different one once we know exactly what
to do. I'll post a snippet on a code i did in user side to get a valid
screenshot on iOS.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3607.

I think it might be the same for Android, but didn't tested :)

Here it is:
            if platform == "ios":
                from kivy.graphics.opengl import glReadPixels, GL_RGBA, GL_UNSIGNED_BYTE
                from kivy.core.image import ImageLoader
                # hacky, as Window don't have correct system size (why, i don't
                # know.)
                width, height = Window._win._get_gl_size()
                Window.dispatch("on_draw")
                data = glReadPixels(0, 0, width, height, GL_RGBA, GL_UNSIGNED_BYTE)
                # save using standard imageloader, but in rgba
                loader = [x for x in ImageLoader.loaders if x.can_save()][0]
                loader.save(screenshot_fn, width, height, "rgba", data, True)

Hopefully this is related to this issue, but unfortunately, it seems that the export_to_png() function is broken in ios as well.. I have been trying to get it to work but not having much luck, could you shed some light on it?
I have figured out that the problem point is it seems that the fbo.texture is still black even after calling the fbo.draw() function, while in windows fbo.texture is not black after calling that.  I'm really not sure where to go from there tho...
setting SDL_GL_RETAINED_BACKING=1 doesnt seem to do anything, neither does Window.dispatch("on_draw")... it's possible i'm doing something wrong in setting these tho, or it might be because this widget is not actually in the window.
There's actually 2 problems with the export_to_png function, the solid black output, and putting the 'with_stencilbuffer=True' option in the Fbo seems to freeze the app (but it freezes AFTER the export function has finished...).

Yes screenshort or export_to_png are broken, as the glReadPixels usage is broken on iOS. You can fix using the same approach that the one show here, or wait for the fix (that i need to merge in master...)

How do I implement this fix in the export_to_png function? I've tried putting in Window.dispatch("on_draw") but the fbo.texture is still blank.
I've tried putting the dispatch in various places, calling extra draw() functions all over the place, and nothing seems to make that texture actually render.

I just wrote code that used both screenshot and export_to_png on ios today, and it appears to work as expected (no black screens). Am I just getting lucky, or has this been resolved in the interim?

@tito, do you still experience the issue?

it still doesn't work for me.
https://github.com/ksgfan/drawing-app
Is there a problem in my code or on the iOS site?


Adds support for Debian Buster

Add debian-buster to supported os list
Add php-fpm7.3 environment meta
Add nginx template client_body_timeout avoids web-updater getting killed by nginx

@Sprinterfreak Thank you for submitting this! It wasn't quite what I needed in order to get Buster support on my setup. Can you take a look at https://github.com/aalaesar/install_nextcloud/pull/81 and see if that meets your needs, as well?

@aalaesar this should be closed. #81 already adds it.


[Core] evict documents with its elements, indices, content

elements and content are done, afaik;
what about query results?

Implemented, now has to be tested throughly


feat: create a new post

close #6

@ilyamochalov
I should have listened to your tip

TIP: don't "over-engineer" the system and do only what is stated in the task.

But TDD and CI for golang really took me some time, sorry for this.
Can you have a quick check?

@ilyamochalov
I should have listened to your tip

TIP: don't "over-engineer" the system and do only what is stated in the task.

But TDD and CI for golang really took me some time, sorry for this.
Can you have a quick check?

@jizusun Jizu, your progress looks fine. Please stick to the objectives of the task ;)

@jizusun Jizu, your progress looks fine. Please stick to the objectives of the task ;)

@jizusun Jizu, your progress looks fine. Please stick to the objectives of the task ;)

Thank you so much 😛
Can you help to approve it?

@jizusun Jizu, your progress looks fine. Please stick to the objectives of the task ;)

Thank you so much 😛
Can you help to approve it?


Feature/a1 sds 715 capture data

This PR contains Capturing Details Of EDC assetid,PolicyId and COntractNegotiationID

Merging PR to test local setup


Your Github Project Page Has An MIT button bower package says GPL ?

Different licensing details.

I'll update to MIT.

Updated with 1da0e2b1a580988fdc6ff46991ac5d70fb9a9c0a. Closes #6.


Null Point Exception after saving changes during edit settings of the Topic

Pre-conditions:
Kafka has a Topic
Steps:

Navigate to the Topic
Click “Edit settings”
Click “Submit”

Expected:
Successfully saving of the setting changes
Actual:
The app is freezing. Null point exception in the console.
java.lang.NullPointerException: null
   at org.apache.commons.pool2.impl.GenericKeyedObjectPool.returnObject(GenericKeyedObjectPool.java:470) ~[commons-pool2-2.2.jar!/:2.2]
   at com.provectus.kafka.ui.util.JmxClusterUtil.getJmxMetrics(JmxClusterUtil.java:47) ~[classes!/:?]
   at com.provectus.kafka.ui.service.KafkaService.lambda$getJmxMetric$53(KafkaService.java:413) ~[classes!/:?]
   at java.util.Optional.map(Optional.java:258) ~[?:?]
   at com.provectus.kafka.ui.service.KafkaService.getJmxMetric(KafkaService.java:413) ~[classes!/:?]
   at com.provectus.kafka.ui.service.KafkaService.lambda$fillBrokerMetrics$55(KafkaService.java:424) ~[classes!/:?]
   ...
ERROR com.provectus.kafka.ui.util.JmxClusterUtil - Cannot invalidate object in pool, service:jmx:rmi:///jndi/rmi://kafka1:9998/jmxrmi

@antipova0803, thank you for creating issue. We'll try to  find a root cause and fix it soon. Look's like we are triyng to connect jmx port after topic settings update without success.

Exception is not connected with Null Pointer Exception. There is not request to BE after Submit button click (you can see it if you open UI console).
@workshur can you please check what is going on UI ?


Random polybar-msg error

The script will randomly give an error saying "polybar-msg: No process with pid  ######" where the "######" will be the correct pid of the bar.
For example

but

Reloading polybar fixes the problem temporarily.
If music is playing, skipping to the next song or previous song fixes it temporarily
There is nothing in particular that seems to be causing it.
This only happens maybe 2-3 times in an hour.

Do you use some kind of launch script to launch polybar? Can you please share details on how you launch the bars?

Great! Can you add one line to kill zscroll everytime? So killall zscroll right below killall -q polybar. Even though I get this error very rarely (particularly when something like pywal restarts the bar to update the colors but that too not everytime), it is not often as you stated.
My first hunch when I saw this error was that the zscroll instance is still running while the bar is killed and hence, I added that line in the launch script. Let me know if you still see this error as often as you said. We might be able to work out a permanent solution.

A possible solution i was thinking of could be to redirect stdout and stderr of the polybar-msg command to /dev/null
so instead of
polybar-msg -p "$(pgrep -f "polybar -c /home/ki/.config/polybar/config-1 bottommain")" hook next 1 >/dev/null
I can do
polybar-msg -p "$(pgrep -f "polybar -c /home/ki/.config/polybar/config-1 bottommain")" hook next 1 &>/dev/null
My bash skills are not great though so i may be wrong
I will try your solution for now, but if the error persists I will try this

Rightly spotted! Redirecting stderr to null would suppress the error output and allow the playerctl output to be shown. You can do that by replacing that line with below:
polybar-msg -p "$(pgrep -f "polybar -c /home/ki/.config/polybar/config-1 bottommain")" hook next 1 1>/dev/null 2>&1

Here 1>/dev/null redirects stdout to null since 1 is the file descriptor for stdout. 2>&1 redirects stderr to whatever stdout which was previously redirected to null.
I suggest you make the change and if it doesn't give you the same error as often as you said, feel free to open a PR. I'd be happy to merge it.

I have created a PR for the proposed fix. Please feel free to take a look at it


Issues with ViT

Hello,
There are several serious issues with the use of ViT.

Perhaps one of the easiest to fix is that currently, with the config file provided as an example, images are center-cropped to 384 (to fit the input size of ViT). However, the pre_size is left to default e.g. 256 for ImageNet or even less for other datasets.
This means that the preprocessor behavior will be to resize to 256, then pad to 384, leading to bad performances (~70% accuracy instead of ~84% expected). The config file should include : config.dataset.pre_size = 384 (or 400 for some slight center crop).

But a more serious issue is that the current implementation cannot possibly work because the current wrapper for ViT, ImageClassifierWithReturnFeature, does not handle the 'return_loss' kwargs of the parent class (ImageClassifier from mmcls). By default, 'return_loss' is set to True, which returns the loss, and must be set to False to get the output. The wrapper must either set return_loss to False, or directly compute the output using extract_feat() and head.layers(). Additionally, the wrapper does not yet support return_feature_list, which is required for certain methods (e.g. mahalanobis). This can be done by setting model.backbone.out_indices = range(len(self.backbone.layers))

With recent updates we are able to evaluate ViT (specifically ViT-B-16) on ImageNet (see this tutorial). Basically this time we wrap around the torchvision implementation of ViTs, which should solve the issues you mentioned above. Let us know if there are any other problems.

First off, thank you so much for this wonderful benchmark. It has really streamlined my research! Unfortunately, the tutorial link you posted in your previous comment gives a 404 page not found error. Could you please repost the tutorial ?

Hi @MaxTorop thank you for your kind words. The tutorial is here. We switched to using CIFAR-10 for the tutorial (otherwise the imagenet data need to be downloaded which is too cumbersome for Colab). However evaluating on ImageNet is pretty much the same. Please also see example evaluation script here.


an wierd "a" infront of github tags

tested different Fonts.

Please try #30 to use a different symbol for the branch indicator.


Import pouchdb-authentication

Hi, I am using this library (^0.5.5) with Ionic 2 beta and now I am migrating to rc1.
In Ionic 2 beta I had this code working:
var PouchDB = require("pouchdb");
PouchDB.plugin(require('pouchdb-authentication'));

Now in rc1 I have this:
import PouchDB from 'pouchdb';
const remoteDB = new PouchDB(remoteDbHost)

How can I import and use this plugin now?
Thanks.

In an Ionic 2 RC0 project, I tried the following code :
import Pouchdb from 'pouchdb'
import PouchdbAuthentication from 'pouchdb-authentication'
Pouchdb.plugin(PouchdbAuthentication)
I have a build success but the browser fails by giving this error :
request.js:3 : Uncaught ReferenceError: require is not defined(…)
Any idea of what is going on?

Ionic 2 is moving from Rollup to Webpack which should fix this issue.


Added Custom Headers Logic to Query

Proposed Changes

Fixes #9457
Added logic to include Custom Headers in Query Function as a part of new OTP based flow for Scheduling Appointments
Merged custom headers with the default headers
Updated Function Signature to use Extended query Options.

@ohcnetwork/care-fe-code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

Summary by CodeRabbit

New Features

Introduced the ability to specify custom headers for query requests, enhancing flexibility in request management.
Added support for dynamic inclusion of additional headers in the request system.

Bug Fixes

Improved the handling of authorization tokens in headers to ensure correct formatting.

@srijantrpth I've also updated to include improved type-safety for the useQuery's error type. Pushed it along with this since it was a very minor piece.
cc: @Jacobjeevan
Before

After

@rithviknishad Kindly review the changes and let me know if you have any suggestions on improving the solution

I think the santizie headers is an overkill. Since we are anyways not going to take user input for setting the header values.
cc: @Jacobjeevan

@rithviknishad  So should we remove the sanitizeHeaders?

@rithviknishad So should we remove the sanitizeHeaders?

yep

@rithviknishad So should we remove the sanitizeHeaders?

yep

Ok, I am implementing the overwriting in the headers.

@rithviknishad Something is wrong with cypress tests maybe there's a version mismatch

@rithviknishad Something is wrong with cypress tests due to version maybe due to the version upgrade which happened Today

@Jacobjeevan Please review the latest changes and let me know if any changes are required.


WebRTC IP Leak

Does webtorrent fix the WebRTC bug [revealed on February 2015] that allows STUN servers to see the real IP as opposed to VPN IP?
Unfortunately is doesn't seem to be discussed anywhere.

In the browser, you should be able to prevent IP leakage as described by https://ipleak.net/#docs

Mozilla Firefox: Type "about:config” in the address bar. Scroll down to “media.peerconnection.enabled”, double click to set it to false.
Google Chrome: Install Google official extension WebRTC Network Limiter.

So the bug still remains in WebTorrent itself then.
Also, the docs are only for browsers, not desktop apps using WebTorrent such as Playback. Does this mean that for applications using WebTorrent, you have to contact them to check/fix this?

This isn't an issue for WebTorrent to resolve. Browsers have already taking steps to mitigate the leak, like always using the default route (i.e. the VPN route) when connecting to STUN servers, and not revealing the local IP addresses unless the user gives camera and mic permissions (which WebTorrent doesn't request).
Given these browser fixes, I see nothing for WebTorrent to do. If you're using a VPN, only the VPN default route should be used (at least in Chrome; I'm less familiar with the steps Mozilla has taken).

This isn't an issue for WebTorrent to resolve. Browsers have already taking steps to mitigate the leak, like always using the default route (i.e. the VPN route) when connecting to STUN servers, and not revealing the local IP addresses unless the user gives camera and mic permissions (which WebTorrent doesn't request).
Given these browser fixes, I see nothing for WebTorrent to do. If you're using a VPN, only the VPN default route should be used (at least in Chrome; I'm less familiar with the steps Mozilla has taken).


Switch to XDG directories

https://www.haskell.org/ghcup/guide/#xdg-support
https://cabal.readthedocs.io/en/stable/config.html#directories
https://docs.haskellstack.org/en/stable/GUIDE/#setting-the-stack-root-location


[BUG] Action Is Now Unable To Parse NPM pURL Without a Namespace

Describe the bug
After updating this action, I cannot use it with my existing configuration file because the package deny list contains the pURL pkg:npm/crypto-js/ or pkg:npm/crypto-js. Neither one is parsable by the action. This has made the action unusable for our PR checks if I want to use a deny list.
To Reproduce
See my example below:

When the pURL is set like this, the job output looks like this.
When the pURL is set like this, the job output looks like this.

Expected behavior
I would expect whichever of these notations align with the pURL spec to be parsable and usable by the action.
Action version
@v4
Examples
See above.
Additional context
This was not always an issue. I was previously using the @v3 tag on this action, and the pURL pkg:npm/crypto-js/ worked.

Hi @AlexWilson-GIS, thanks for the report. I noticed in your config that you're using arrays for the values of deny_groups and deny_packages. I believe that's the problem, since we actually expect those to be comma-separated strings (see example)
Would you mind trying again with the comma-separated list form?

Thanks for the quick response. Unfortunately, it doesn't appear to have made a difference:
Attempt 1
Attempt 2

Oh wait, that's 1 string with all of them inside it 😅

Unfortunately, putting all of them in a string and delimiting them with commas as the example shows doesn't work either. It appears to expect an array of strings, like what I did in my previous attempt.

@AlexWilson-GIS
Have you tried this config (second from yout OG post) but change first value of deny_packages from pkg:npm/crypto-js/ to pkg:npm/crypto-js?
The first error you got (here) complained about deny_groups values not ending with /, while the second error (here) complained that you have deny_packages values ending with /.

Hey @AlexWilson-GIS
looks like your first config is missing a / for your groups - 'pkg:maven/log4j/' and not for the packages
and for your second config the leading / for packages is not needed since it would consider it a namespace

@jtomkiew-mng haha nice catch! Looks like we both were looking at this at the same time 👍

Thanks for your help everyone! The problem was the slashes - the deny_packages list doesn't want a slash, and the deny_groups one does.


Pass puppet_module to ERB

This allows to access the module name using
the @module variable.

@cmurphy this PR would be very useful to generate docs for example (with links to issues URL, etc.).

@raphink instead of adding a new instance variable to the template, could :puppet_module be added as a key in the file_configs variable? Then it would be wrapped in the @configs variable.

@cmurphy I wanted to avoid that as that might conflict with existing parameters in @configs.

@raphink if there's already a :puppet_module key in @configs then the user wouldn't make use of the @module variable anyway.

Updated the commit. Do you like this better @cmurphy ?

:+1:  thanks!

@cmurphy thanks. Could you cut a new release, please?

@raphink yes, will do this week.


Add initial log line for run_single_worker

@jrafanie Please review.
As we've seen many times, containers in slow environments can seem to take forever to start with the user staring at "Writing encryption key" and wondering if anything is happening.  At a very minimum, we write this log line before we load the Rails environment to give more feedback.

👍   I'll be testing this later today hopefully

Kasparov backport details:
$ git log -1
commit d3ce6c21159a7180b02cbd7b9a76ca6f1f20c572
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Thu Oct 29 09:38:49 2020 -0400

    Merge pull request #20730 from Fryguy/logging_in_run_single_worker

    Add initial log line for run_single_worker

    (cherry picked from commit 7edcd35646e027fa1d4af2b991feb8f2ee944abb)


Extract Storybook content into subdirectory

🚨 🏗 WIP: Work in progress 🚧 🚨
Closes https://github.com/primer/react/issues/1849 by extracting Storybook files and configuration to the stories/ directory.
Screenshots
Please provide before/after screenshots for any visual changes
Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

All tests are passing except for the checkStoriesForAxeViolations tests, which require the stories themselves.
Now that the root and stories/ directories have separate package.json manifests and node_modules, it appears that we may have two separate instances of React installed. Temporarily, this can be resolved by rm -r-ing the stories/node_modules directory, which wipes out the additional React package installation, but I'm not sure what a permanent fix looks like, yet.

👋  I know this isn't done yet, but I just wanted to mention that this approach would move us away from the file structure ADR that @jfuchs proposed last quarter. Not a dealbreaker but something to consider.

@colebemis I don't think we have a choice, unfortunately, unless we want to continue having Primer React ship types from Storybook in its build output. This happens just because those types are in our node_modules directory when we compile.
The only other solution I might try is to restrict the typeRoots in tsconfig.build.json, but it'll mean listing each build dependency's node_modules types path specifically.

Closed in favor of https://github.com/primer/react/pull/1855


ioredis consistently times out on first connect on non-cluster configuration

Redis instance is located locally. It seems to be a false positive as it reconnects successfully on the second try, and I don't encounter any redis errors when testing. However, we don't have this warning using normal redis package.
Here is the log dump:
  ioredis:redis status[<IP_ADDRESS>:6379]: [empty] -> connecting +0ms
  ioredis:connection error: Error: connect ETIMEDOUT +5s
  ioredis:redis queue command[0] -> get(something1) +15ms
  ioredis:redis queue command[0] -> get(something2) +15ms
  ioredis:redis queue command[0] -> get(something3) +15ms
  ioredis:redis status[<IP_ADDRESS>:6379]: connecting -> close +1ms
  ioredis:connection reconnect in 2ms +0ms
  ioredis:redis status[<IP_ADDRESS>:6379]: close -> reconnecting +0ms
  ioredis:redis status[<IP_ADDRESS>:6379]: reconnecting -> connecting +8ms
  ioredis:redis status[<IP_ADDRESS>:6379]: connecting -> connect +4ms
  ioredis:redis write command[0] -> info() +2ms
  ioredis:redis status[<IP_ADDRESS>:6379]: connect -> ready +7ms
  ioredis:connection send 3 commands in offline queue +0ms
  ioredis:redis write command[0] -> get(something1) +1ms
  ioredis:redis write command[0] -> get(something2) +0ms
  ioredis:redis write command[0] -> get(something3) +0ms

The code is also pretty normal:
const Redis = require('ioredis');

const client = new Redis({
  host: process.env.CACHE_HOST,
  port: process.env.CACHE_PORT,
});

Hmm...ioredis defaults the connection timeout to 3 seconds, which seems not enough for your case (+5s). You can custom this option:
new Redis({
  connectTimeout: 10000
})

That solves it, thanks.


student app ui-imporve

Jira issue(s)
https://selleolabs.atlassian.net/browse/LC-233
Overview
This PR provides student app improvements according to designs
Screenshots

@typeWolffo, but why? lessonIndex is not used anymore, and I forgot to remove it. @tomaszbilka was changing this part, and he preferred another way of problem-solving.


Fix issue #2

Move “Remove button” and “Remove field” from toggleInputField into
click events.
Replace selector with this.

Needs review. Breaks functionality.


Stabilize fixes for RET50{5-8}

Summary
This PR stabilizes the fixes introduced in https://github.com/astral-sh/ruff/pull/9595

We have some open issues about a panic for the RET505 autofix:

https://github.com/astral-sh/ruff/issues/10099

Does it still reproduce on main?

We have an open issue about a panic for the RET505 autofix:
* [Rule RET505 cause panic #10099](https://github.com/astral-sh/ruff/issues/10099)

Does it still reproduce on main?

Answering my own question: yes, it still repros on main. It's somewhat unusual code causing the panic, though, so maybe it's still okay to stabilise?

Yeah, the problem is with adjust_indent rather than the rule itself. I'll push a fix

Nice work with that fix 😄👍


Some errors/typos in wiki.

In the jumbling subsection for extensions to grip theory, the word "bandaged" has an unintentional $ before it.

Third point for physical $2^4$ scrambling rules is not complete.

Table of contents for rules page writes "24" instead of $"2^4"$.

Everything has been fixed besides the Table of Contents. I can't change that because that's just how mkdocs works I think.


Exam mode: Fix missing clone repository button for test runs

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Server

[ ] I followed the coding and design guidelines.
[ ] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[ ] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[ ] I implemented the changes with a good performance and prevented too many database calls.
[ ] I documented the Java code using JavaDoc style.

Client

[ ] I followed the coding and design guidelines and ensured that the layout is responsive.
[ ] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

Description

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to Course Administration
...

Exam Mode Testing

Prerequisites:

1 Instructor
2 Students
1 Exam with a Programming Exercise

Log in to Artemis
Participate in the exam as a student
Make sure that the UI of the programming exercise in the exam mode stays unchanged. You can use the exam mode documentation as reference.
...

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test

Test Coverage

Screenshots

Codecov Report

Merging #6212 (fdafb4b) into develop (2d48e2d) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6212      +/-   ##
=============================================
- Coverage      80.29%   80.28%   -0.01%
+ Complexity     12842    12839       -3
=============================================
  Files           2273     2273
  Lines          88065    88066       +1
  Branches       13048    13047       -1
=============================================
- Hits           70708    70700       -8
- Misses          9651     9659       +8
- Partials        7706     7707       +1

Impacted Files
Coverage Δ

...www1/artemis/repository/StudentExamRepository.java
96.49% <ø> (ø)

.../www1/artemis/service/exam/StudentExamService.java
86.07% <ø> (ø)

...ails/exercise-details-student-actions.component.ts
69.60% <0.00%> (-0.69%)
:arrow_down:

...editor-button/open-code-editor-button.component.ts
50.00% <0.00%> (-3.34%)
:arrow_down:

...ises/shared/participation/participation.service.ts
74.62% <50.00%> (-0.38%)
:arrow_down:

...s/clone-repo-button/clone-repo-button.component.ts
73.52% <100.00%> (+2.51%)
:arrow_up:

...e/scheduled/WeeklyEmailSummaryScheduleService.java
65.51% <0.00%> (-6.90%)
:arrow_down:

...ice/scheduled/ParticipantScoreScheduleService.java
84.84% <0.00%> (-2.43%)
:arrow_down:

...va/de/tum/in/www1/artemis/service/FileService.java
79.85% <0.00%> (-0.74%)
:arrow_down:

Flag
Coverage Δ

client
76.41% <50.00%> (+<0.01%)
:arrow_up:

server
84.59% <ø> (-0.03%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d48e2d...fdafb4b. Read the comment docs.


updated: Style of Add-Item button

resolved issue: #7
@Pranavnk15 updated the style of the "Add Item" button.

Thank you for your invaluable contributions to our open-source project! 🚀 If you like our project, please consider to start ⭐ the repo.


TypeError: Cannot call method 'push' of undefined

Ran into this one today after upgrading a project to 1.2.0 in an attempt to fix a bug caused by #1464.  Now I'm seeing a different issue, and I'm trying to figure out if it's a bug or if my understanding of expected functionality is incorrect.  Here's the scenario - say I have two classes, one of which extends the other, and a class method on the child class that I want to bind:
class Hello
end
class World extends Hello
@doSomething: =>
Trying to compile this throws the "Cannot call method 'push' of undefined" exception, though to be honest I can't quite figure out what that means.  If I change the doSomething function to not be bound, or the World class not to extend Hello, the code compiles just fine (though obviously it doesn't function as expected).
I can work around the #1464 issue by changing my code and using coffeescript 1.1.3, but the TypeError seems to be a new error in 1.2.0.  I didn't see any existing issues that seemed to be the same problem.  Sort of stumped on how to proceed - can anyone shed some light?

Thanks for reporting this. I've fixed it on master, and we'll have to cut a 1.2.1 release soon to get this fix out to folks.

Just curious if 1.2.1 is going to land anytime soon. (Makes my NPM-ing more convenient when I deploy.)

@randallb: See #2135. 1.3.0 will be here... soon. Feel free to add some "encouragement" over there.


Case-level input data with metadata

Attempt to solve #136
Functionality for attaching metadata to input data for a case being exported to the current case structure, under share/input.

[ ] Agree on concepts/patterns. Synchronize with #174
[ ] Expand tests

Closed, now irrelevant


Back button centered in iOS

navigation back button in centered in iOS, as pictured. Component has bare minimum of styling with flex:1, and nothing more, just to demonstrate.
"react-navigation": "^1.0.0-beta.15",
"react": "^16.0.0",
"react-native": "^0.49.3",

Are you sure you don't have any other styling in, for example, the Component in which HomeScreen is rendered?
This to me looks like something related to that. Are you following some tutorial? Because our intro doesn't have that issue, as far as I can see.

Confirmed user error. Thanks for the quick reply!


Fix ini vulnerability by updating next

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM , but wait  until #145 is approved
Also, DO NOT merge the changes in until we have gone through the entire vetting process from AWS Solutions

Merged as a part of v1.0.1 release.

Merged as a part of v1.0.1 release.


Noble Network Support

Adding support for the noble blockchain.
Don't know how to validate that NAMESPACE is correct, been using this for a while now and its been working just fine tho.

Thank you so much! Namespace is correct, it's just the binary name capitalised. I'll include in the next release

Pre-release image: ghcr.io/akash-network/cosmos-omnibus:v0.4.21-rc3-noble-v5.0.0

@PoolPirate any chance you can allow me to edit this PR so I can fix the conflicts? Sorry I should have merged this one first.

@tombeynon Don't have the option to enable that in the PR for some reason. Did resolve conflicts myself for this one, will see where I can enable that going forward

Ah well I see you mean to merge this for the next release.
Added you as a collab to the fork repo. Not the intended way but can't see that option to allow changes to the PR here.

I've come across that before, I don't think it's possible for organisation forks which is very annoying.. appreciate the invite though, that'll work great in the future.
I'll do another release with just this PR shortly, the release process has to be completed quickly once started as it breaks e.g. Akash Console. I wasn't sure you'd respond this quick 😅

Github broke so I had to cancel the action anyway


Build fails with minifyEnabled

After adding the Cash App Pay SDK to our project and building the project with minifyEnabled true we are getting the following error:
ERROR: R8: Missing class kotlinx.serialization.KSerializer (referenced from: kotlinx.datetime.serializers.InstantIso8601Serializer)
Missing class kotlinx.serialization.Serializable (referenced from: kotlinx.datetime.Instant)

Besides this SDK nothing else is using kotlinx datetime in our project. Now I'm not fully sure if this error comes from your SDK or maybe it's already an issue in kotlinx datetime. I couldn't really find anything on this particular error.
I'm assuming it is safe to ignore this error in the ProGuard rules, since kotlinx serialization is not used. Can you verify this? Or do you maybe know any other solutions?
Cash App Pay SDK version: 2.2.0

This PR addresses this issue: https://github.com/cashapp/cash-app-pay-android-sdk/pull/125


403 error

I have this log. I dont know why
Error: Error trying to create GitHub check for stylelint: Received status code 403
    at createCheck (/home/runner/work/_actions/wearerequired/lint-action/v1/src/github/api.js:64:9)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
/home/runner/work/_actions/wearerequired/lint-action/v1/src/index.js:13
	throw new Error(`Exiting because of unhandled promise rejection`);
	^

Error: Exiting because of unhandled promise rejection
    at process.<anonymous> (/home/runner/work/_actions/wearerequired/lint-action/v1/src/index.js:13:8)
    at process.emit (events.js:210:5)
    at processPromiseRejections (internal/process/promises.js:201:33)
    at processTicksAndRejections (internal/process/task_queues.js:94:32)

Hi, thanks for the report. Can you￼￼ please share a link to the repo where this error has happened?

this is my repo https://github.com/KsTGTeam/Beautify

and this log
Error: Error trying to create GitHub check for stylelint: Received status code 403
    at createCheck (/home/runner/work/_actions/wearerequired/lint-action/v1/src/github/api.js:64:9)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
/home/runner/work/_actions/wearerequired/lint-action/v1/src/index.js:13
	throw new Error(`Exiting because of unhandled promise rejection`);
	^

Error: Exiting because of unhandled promise rejection
    at process.<anonymous> (/home/runner/work/_actions/wearerequired/lint-action/v1/src/index.js:13:8)
    at process.emit (events.js:210:5)
    at processPromiseRejections (internal/process/promises.js:201:33)
    at processTicksAndRejections (internal/process/task_queues.js:94:32)

It looks like you're using a custom GitHub token which doesn't have the necessary permissions. Can you try using the default token ${{ secrets.github_token }}?

i will try

Hey @Ksengine, did you get this resolved? Looks like you still might be using a custom GitHub token, curious what you changed to correct the permissions error?


Reexports and docs

After #230 was merged, i realized there's probably a better solution - #[doc(inline)] - e.g.:
#[doc(inline)]
pub use rg3d_sound as sound;

This preserves the docs from the original crate (although it doesn't mention anywhere that it's a reexport but that's just a tiny downside).
Should i make a PR to use #[doc(inline)] instead?
This made me wonder though - should the other crates reexport subcrates such as core publicly? When searching, it results in for example Pool being found 4 times. Some items like Vector are even worse. I think using the main rg3d crate as a full game engine is the primary usecase. If people wanna use the other crates separately, they should be able to put for example nalgebra and rg3d-ui in their Cargo.toml and be able to use both together without having to go through rg3d_ui::core::algebra.
There's a middle ground - use a (default) feature to reexport core, nalgebra and others when subcrates are used standalone but don't reexport when used from the rg3d crate.

I think #[doc(inline)] is better then.  Re-export is fine, I don't think it is a big issue.

I'm closing this, since PR was merged.


Revert "[PIP-71][SQL]Pulsar SQL migrate SchemaHandle to presto decoder (#8422)"

This reverts commit #8422: a1107ad9e008ef2f1f028feb95a893e74face08e.
Motivation
As explained in #9470, the changes in this commit are now making the SQL integration tests to fail and blocking all changes to master. Reverting now to unblock, please re-submit the PR once the test is fixed.

@merlimat @jerrypeng The master broken issue is fixed by https://github.com/apache/pulsar/pull/9472 I think, we have verified on the local environment. If the master issue can be fixed, I think we don't need to revert it.

That error looks same as Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 #6501 . Integration tests works on my macbook , maybe this is not caused by #8422 ?