python311Packages.rpy2: 3.5.14 -> 3.5.15

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.rpy2 is: Python interface to R
meta.homepage for python311Packages.rpy2 is: https://rpy2.github.io/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.5.15 with grep in /nix/store/llizi3x7nza17ygkb0g5gsb4d2rcy4zs-python3.11-rpy2-3.5.15
found 3.5.15 in filename of file in /nix/store/llizi3x7nza17ygkb0g5gsb4d2rcy4zs-python3.11-rpy2-3.5.15

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

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath
cantor
labplot
libsForQt5.cantor
plasma5Packages.cantor
python310Packages.rpy2
python311Packages.rpy2
sage
sageWithDoc

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/a80ea4445ab68fb69eccbd780651e09f403a571a#python311Packages.rpy2

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/llizi3x7nza17ygkb0g5gsb4d2rcy4zs-python3.11-rpy2-3.5.15
ls -la /nix/store/llizi3x7nza17ygkb0g5gsb4d2rcy4zs-python3.11-rpy2-3.5.15/bin

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

4 packages failed to build:

cantor
labplot
sage
sageWithDoc

2 packages built:

python310Packages.rpy2
python311Packages.rpy2

Maintainer pings
cc @joelmo for testing.

Already on master.


Creation of Azure storage from connection string failed for shared key credentials

the creation of a Azure file or blob storage instance failes if doing so via StorageFactory.Blobs.FromConnectionString
Reason:
The factory uses FluentStorage.ConnectionString.StorageConnectionString internally to parse the connection string. For each property value the method UrlDecode is called by the Parse method. This makes a lot of sense to be able to pass special characters in passwords.
The Azure account key is Base64 encoded, though. If it contains + characters, those get replaced by spaces, other characters might also get replaced falsely. Therefore the Azure account key must not be tampered with.
One workaround would be to store the account key urlencoded in the connection string in the application configuration - not very user friendly, its base64 already...
The StorageConnectionString on the other hand side has no knowledge if the secret key is urlencoded or base64 encoded... not sure how to solve this.
Maybe it would make sense to mention the url-encoding for passwords in the documentation.

I ran into this just now. Thanks for the tip to store the account key as URL encoded, that worked!

Which API needs to change? Can you describe the change required?

Hi @robinrodricks, absolutely. Let's take a look at the following screenshot for context:

Notice line 131, where every part of the connection string is always URL-decoded. This becomes problematic when using a key like the following:
aGi**********************************************************************f0e+CZ2aH+A*******xg==

During URL-decoding, the + characters are replaced with spaces. Later on, this value is passed to the Azure Blob Storage module, which tries to base64-decode it:

As @AloisMaierhofer pointed out, we can work around this by URL-encoding the key, which will then be decoded correctly by the StorageConnectionString class.
However, I’m unsure whether the StorageConnectionString should be responsible for URL-decoding at all. My first question would be: Should this responsibility lie with StorageConnectionString, or should it be up to the application's configuration to handle this?

It would appear that this issue is essentially a duplicate of #42.

@robinrodricks I think that's the ideal solution: it's user friendly and it's simple 👍🏻


[11.x] Allow adding array or string for web and api routes in bootstrap/app.php

This pull request proposes an enhancement to Laravel's bootstrap/app.php file, enabling developers to add routes in array format or as strings for both web and api route groups.
Motivation
The motivation behind this enhancement is to provide developers with greater flexibility and convenience when defining routes, particularly in scenarios where applications have different guards and need to create different routes for each guard rather than writing require/require_once in route file.
Use Cases
In projects where route management becomes complex due to the proliferation of routes or diverse authentication scenarios, this will simplifies the process by enabling developers to organize routes into multiple route files effortlessly. By allowing routes to be defined in array format, developers can easily create separate route files corresponding to different functionalities, user roles, or authentication guards. This approach promotes code organization and maintainability, as each route file can focus on a specific aspect of the application, making it easier to understand and manage.
Additional Context
I'm developing an ecommerce website with five different guards, and I want to include separate route files for each guard. Using require or require_once made my code look outdated, and I disliked it. Let's simplify by allowing arrays as well in the $web parameter of withRouting() method in Illuminate\Foundation\Application

Hey there - please re-submit with all unnecessary code formatting changes reverted.

Thank you


allow multiple space type filters on decomposedfs

The decomposedfs driver now evaluates multiple space type filters when listing storage spaces.

is that a thing that can be tested in unit tests?


"copy_local" is not respected for indirect dependencies

I have a following case:

paket.dependencies:copy_local: false
...

paket.references:some_package_1 copy_local: true

some_package_1 depends on some_package_2.

After running "paket install", in csproj file I have:
...
<Reference Include="some_package_1">
  <Private>True</Private>
  ...
</Reference>
...
<Reference Include="some_package_2">
  <Private>False</Private>
  ...
</Reference>
...

I expected that some_package_2 would also get "True".
Is there a way to propagate this option to indirect dependencies of referenced nuget package?

Mhm I think it's already inherited in the dependencies file. So you're
right we should inherit it also in the references file.
As a workaround add the dependent package also to the references file.
On Jun 3, 2015 10:55 AM, "Kalyanov Dmitry"<EMAIL_ADDRESS>wrote:

I have a following case:

paket.dependencies:

copy_local: false
...

paket.references:

some_package_1 copy_local: true

some_package_1 depends on some_package_2.

After running "paket install", in csproj file I have:
...

True
...

...

False
...

...
I expected that some_package_2 would also get "True".
Is there a way to propagate this option to indirect dependencies of
referenced nuget package?
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/856.


UI preserve interaction context

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Windows 7, vesion_2.1.4-stable_win64, not graphic related.
Issue description:
It is very inconvenient to lose the menu upon clicking outside of a modal.
Steps to reproduce:
Click elsewhere when using a menu modal.

if you set to exclusive, still same?

just tested, if you enabled exclusive on the modal window it doesn't close if you click outside of it, no issue here :)

Related: #6453

Thank you for the response and solution. Sorry for the misunderstanding.

I had believed it to be the same sort of issue as #6453


Ability to set EnableIPForwarding to false for a custom network using the bridge driver

Description of problem:
Ability to set EnableIPForwarding to false for a custom network using the bridge driver. Normally I've been having to set this for a docker host on the daemon config, but I want some isolated containers that will run without network access (though reachable by a proxy that does have outward network access). I was hoping that it was exposed like com.docker.network.bridge.enable_icc (which I've also used), but that was not so.
docker version:
Client:
 Version:      1.9.0
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   76d6bc9
 Built:        Tue Nov  3 19:20:09 UTC 2015
 OS/Arch:      darwin/amd64

Server:
 Version:      1.9.0
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   76d6bc9
 Built:        Tue Nov  3 19:20:09 UTC 2015
 OS/Arch:      linux/amd64

docker info:
Containers: 2
Images: 87
Server Version: 1.9.0
Storage Driver: aufs
 Root Dir: /mnt/sda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 91
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.12-boot2docker
Operating System: Boot2Docker 1.9.0 (TCL 6.4); master : 16e4a2a - Tue Nov  3 19:49:22 UTC 2015
CPUs: 1
Total Memory: 1.956 GiB
Name: default
ID: 6PME:KNOZ:7JOH:PRES:GF7Q:6N7B:2CZ3:HMR6:XHS6:OVNQ:B5BK:ZGA7
Debug mode (server): true
 File Descriptors: 20
 Goroutines: 36
 System Time: 2015-11-05T06:29:41.056042169Z
 EventsListeners: 0
 Init SHA1:
 Init Path: /usr/local/bin/docker
 Docker Root Dir: /mnt/sda1/var/lib/docker
Username: rgbkrk
Registry: https://index.docker.io/v1/
Labels:
 provider=virtualbox

uname -a:
Darwin lack 14.5.0 Darwin Kernel Version 14.5.0: Tue Sep  1 21:23:09 PDT 2015; root:xnu-2782.50.1~1/RELEASE_X86_64 x86_64

Environment details (AWS, VirtualBox, physical, etc.):
Virtualbox via docker-machine, just the default
How reproducible:
Very much
Steps to Reproduce:

Look at labels in code, notice lack of com.docker.network.bridge.enable_ip_forwarding

Additional info:
I filled out the template because I always suspect Gordon the turtle around the corner.

Probably worth detailing why I care about having this flag on a per-container basis.
For Jupyter/IPython we have hosted demos where users have full access to an analysis environment:

Any user that shows up gets access to an ephemeral container for them to try things out, play, learn, whatever they need to. This includes shell access into the container (as with any programming environment) and we explicitly disallow networking to the outside world (--ip-forward=false) as well as to other containers (--icc=false):

@oreillymedia would also love a feature like this, since we're also using containers to create and manage ad-hoc environments for running code on our new sites.

Being able to control network access at a container level would provide a much broader range of products we could offer via containers.

@rgbkrk @odewahn we have designed the bridge driver to keep some as driver level control and some as network level control. enable_ip_forwarding is a bridge driver level configuration configured at the docker daemon (via the usual daemon flags --ip-forward).
But, if you are looking to prevent external access and inter-container access, you can use the network create options -o com.docker.network.bridge.enable_icc=false -o com.docker.network.bridge.enable_ip_masquerade=false
This will address your requirement without having to disable ip-forwarding.
Example :
$ docker network create -o com.docker.network.bridge.enable_icc=false -o com.docker.network.bridge.enable_ip_masquerade=false isolated
5875a61469025d59cc8a3869b2d34683e44cdf4a430cf5709150566431ea9699

$ docker run -it --net=isolated busybox
/ # ping www.google.com
^C
/ # ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
^C
--- <IP_ADDRESS> ping statistics ---
2 packets transmitted, 0 packets received, 100% packet loss

Please let us know if this satisfies your requirement.

@rgbkrk @odewahn essentially, all the daemon configurations that is available for the docker0 default bridge is available for all the bridge networks using the appropriate driver configuration options.

Oh! It's enable_ip_masquerade. Thank you so much, this is exactly what we need. I'll change the title to this issue and close it.

Thanks again @mavenugo, we're set to go on Docker 1.9. This is stellar.

@rgbkrk  @odewahn :+1: please let us know once this is operational, I would love to give it a try in your running system


leaf index

how can I get the leaf index in a distribution environment?

Thanks for your reply~But when I use the function predictLeaf, I found the size of the index array is different from n_estimator, Specifically, for binary classification, the size of the index array is equal to 2 * n_estimator, for multi-class classification, it is equal to classNum * n_estimator + 2,  do you know why? looking forward to your reply!

The strict shape of predict leaf output is actually 4 dim, but java/scala don't have a canonical n-dim array implementation like numpy on Python, as a result xgboost4j returns concatenated dimensions.


ci: Add unit and functional tests using Dart beta

Fixes #1235
- What I did
Created new workflow to run unit tests and functional tests using Dart Beta channel
- How I did it

Copied at_client_sdk workflow
Stripped out unnecessary run condition
s/stable/beta/ for Dart SDK in unit tests and functional tests
Removed all other tests

- How to verify it
Action will run against this PR
- Description for the changelog
ci: Add unit and functional tests using Dart beta

Per https://github.com/atsign-foundation/at_server/pull/1786#issuecomment-1932299102

@gkc I've updated the required tests to use the (stable) versions, so it would be good to get this merged ASAP


Adding Remote in HTTP Incoming Request log

Description
This would add Remote to incoming request log for format style http.
Motivation and Context
https://github.com/zalando/logbook/issues/598
Types of changes

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

Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

It seems getting proper remote is missing for jax-rs
https://github.com/zalando/logbook/blob/master/logbook-jaxrs/src/main/java/org/zalando/logbook/jaxrs/RemoteRequest.java#L40

Please suggest me if this not the way to add remote.

:+1:

Looks good! Thanks for the contribution! :tada:

I'm currently releasing 2.0.0-RC.4, it should be available in central shorty.


d4m doesn't run on Nobara 37

Title, i was able to run it previously but i tried it today and it doesn't open.

Terminal Output
Type conversion already registered from type QFuture<QString> to type QFuture<void>
Traceback (most recent call last):
  File "/app/lib/python3.9/site-packages/urllib3/connection.py", line 174, in _new_conn
    conn = connection.create_connection(
  File "/app/lib/python3.9/site-packages/urllib3/util/connection.py", line 72, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/lib/python3.9/socket.py", line 954, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -2] Name or service not known

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(
  File "/app/lib/python3.9/site-packages/urllib3/connectionpool.py", line 386, in _make_request
    self._validate_conn(conn)
  File "/app/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1042, in _validate_conn
    conn.connect()
  File "/app/lib/python3.9/site-packages/urllib3/connection.py", line 358, in connect
    self.sock = conn = self._new_conn()
  File "/app/lib/python3.9/site-packages/urllib3/connection.py", line 186, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f7e92757190>: Failed to establish a new connection: [Errno -2] Name or service not known

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/lib/python3.9/site-packages/requests/adapters.py", line 489, in send
    resp = conn.urlopen(
  File "/app/lib/python3.9/site-packages/urllib3/connectionpool.py", line 787, in urlopen
    retries = retries.increment(
  File "/app/lib/python3.9/site-packages/urllib3/util/retry.py", line 592, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='divamodarchive.xyz', port=443): Max retries exceeded with url: /favicon.ico (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f7e92757190>: Failed to establish a new connection: [Errno -2] Name or service not known'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/app/lib/python3.9/site-packages/d4m/__main__.py", line 4, in <module>
    import d4m.gui
  File "/app/lib/python3.9/site-packages/d4m/gui.py", line 33, in <module>
    "divamodarchive": d4m.api.download_favicon("divamodarchive"),
  File "/app/lib/python3.9/site-packages/d4m/api.py", line 77, in download_favicon
    return SUPPORTED_APIS[origin].download_favicon()
  File "/app/lib/python3.9/site-packages/d4m/dma.py", line 98, in download_favicon
    r = requests.get("https://divamodarchive.xyz/favicon.ico")
  File "/app/lib/python3.9/site-packages/requests/api.py", line 73, in get
    return request("get", url, params=params, **kwargs)
  File "/app/lib/python3.9/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/app/lib/python3.9/site-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/app/lib/python3.9/site-packages/requests/sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "/app/lib/python3.9/site-packages/requests/adapters.py", line 565, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='divamodarchive.xyz', port=443): Max retries exceeded with url: /favicon.ico (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f7e92757190>: Failed to establish a new connection: [Errno -2] Name or service not known'))

Getting the same result on Arch, seems to be a recent change in some of the API's used


Generic xDS resource manager

Title: Generic xDS resource manager
Description:
Currently components that use xDS resources directly interact with the subscription messages, and in part need to adhere to the xDS-protocol and the transport-layer. This could lead to inconsistencies in the ways that different xDS services behave (for example, #13009, #16350, #12061).
We propose xDS-Manager, that will be the entry point for xDS-resource subscriptions that will:

Decouple the xDS-transport layer from the xDS-data layer, and ensure the correct usage of the xDS-protocol.
Serve as a single component that manages the connections with external config sources.
Allow internal components (including extensions) to fetch resources using xDS - including arbitrary resource types over ADS.
Ease the introduction of new xDS types and ensure they adhere to the xDS-protocol.
Manage the fetched resources lifetime.

The design doc can be found here.
This proposal may address or improve the following issues (among others):

#13009 - ensuring that CDS updates are using the latest EDS resources.
#11547 - blob discovery.
#8400 - resource patching (partial support).

reopening an removing stale.

Current status: need to estimate the memory requirements when adding the cache layer for all xDS resources.

I would not expect this approach to significantly increase memory usage.  Keep in mind that Envoy must already be storing most of this data, since it needs it for the admin interface.  I think it's just currently storing the data in a decentralized way (in each data plane component) instead of storing it only once in the xDS transport protocol cache.  Given that, I would expect this approach would actually reduce memory usage a bit, because problems like #13009 mean that it is currently storing multiple copies of the EDS data, and that would be eliminated with this centralized cache.

The stress test would be something like an inline Wasm binary. I agree that a central config store should help with memory long term, but may add overhead initially.


VCellMTMD plugin has a zlib built into it

Ran across this while doing some housekeeping on database plugins VCelMTMD has the "info-zip" implementation of zlib as source code files in our tree. Anyone know why? Any disagree it should use zlib that rest of VisIt uses and we should remove the info-zip sources from our tree
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2670
Status: Rejected
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: VCellMTMD plugin has a zlib built into it
Assigned to: Mark Miller
Category: -
Target version: 2.13.0
Author: Mark Miller
Start: 08/23/2016
Due date:
% Done: 0%
Estimated time:
Created: 08/23/2016 03:23 pm
Updated: 03/17/2017 01:46 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.10.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Ran across this while doing some housekeeping on database plugins VCelMTMD has the "info-zip" implementation of zlib as source code files in our tree. Anyone know why? Any disagree it should use zlib that rest of VisIt uses and we should remove the info-zip sources from our tree
Comments:
There was some question as to whether this plugin still needs to be supported.My read is that our plugin is likely woefully out of date.I do see some VisIt generated images on the vcell.org site but have no idea how recent they are.I have sent email to the support contact for vcell.org. We'll see if they have any info. We've decided best action is to simply remove the plugin, see #2787

2670_biomodel.vcml.xml.zip


new synonym for greasy coat

Need to add "oily coat" as an exact synonym of existing term greasy coat MP:0003849.

"oily coat" has been added as an exact synonym.


Fix recursive resource local to scene (array/dictionary properties on resources)

Related to #71243
Fixes #71406, #77380.
Any resource that contains arrays or dictionaries with local to scene resources will now be duplicated and configured correctly. This also includes built-in nodes, such as when marking a mesh's surface materials as local to scene.
A side effect of this change is that any array and dictionary property is now duplicated for local-to-scene instances.
What isn't handled in this PR is the case where a scene's node has an exported array/dictionary property containing local to scene resources. That code has a lot more special cases that I don't fully understand yet, but the solution would look largely similar.

NOTE: Using resources as dictionary keys does not work with the inspector (they get saved as null into the scene file), so this code branch was not tested.

Seems to assign the wrong local scene in the https://github.com/godotengine/godot/issues/71243 MRP.

Output Log
pre-modified first ---------
simple:
true	1	LocalToSceneTests1:<Node#26474447287>
array:
true	2	<Object#null>
true	3	<Object#null>
dict:
true	4	<Object#null>
sub:
true	5	LocalToSceneTests1:<Node#26474447287>
sub array:
true	6	LocalToSceneTests2:<Node#26575110566>
check first ---------
simple:
true	1	LocalToSceneTests1:<Node#26474447287>
array:
true	4	<Object#null>
true	6	<Object#null>
dict:
true	8	<Object#null>
sub:
true	5	LocalToSceneTests1:<Node#26474447287>
sub array:
true	12	LocalToSceneTests2:<Node#26575110566>
check second ---------
simple:
true	2	LocalToSceneTests2:<Node#26575110566>
array:
true	4	<Object#null>
true	6	<Object#null>
dict:
true	8	<Object#null>
sub:
true	10	LocalToSceneTests2:<Node#26575110566>
sub array:
true	12	LocalToSceneTests2:<Node#26575110566>

I would like to push this if possible.
I'm building a game that relies heavily on resources to decouple systems and I have a lot of resources in arrays and dictionaries.
With this merged in I can at least create a workaround by only having a Local to Scene resource hold my arrays of resources. Not 100% optimal, but it would let me do what I want to do.
Also a little annoying to debug this issue when you first discover the Local to Scene checkbox.

Fixes the MRP in https://github.com/godotengine/godot/issues/71406.

Tested, it doesn't. The textures are still all the same.
Maybe something changed since then; there were a couple of PRs touching local to scene.

Applied review feedback, which cleaned up the code by a lot! :D
Also tested both issue's MRPs again and they behave as expected.

Seems like it fixes #71243 too, not sure why you didn't link it.

Seems like it fixes #71243 too, not sure why you didn't link it.

It doesn't fix the case where an array or a dictionary with sub-resources is exported directly on a Node (as opposed to being a property on a Resource).
To fix it, we would need changes in the PackedScene loading code, which looks scary to me since it also needs to handle scene inheritance among other things.

Should be good to merge now, but could be worth a rebase to make sure everything still builds fine.

Thanks!


start transition of CI according to apache infra reqs

Description
This PR transitions from travis to gha. it is trying to follow https://docs.github.com/en/actions/migrating-to-github-actions/migrating-from-travis-ci-to-github-actions
An alternative approach would be to use: https://github.com/marketplace/actions/run-travis-yml (parallel PR?)

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

so far only in the original branch on the PR repo. It can be replicated to your own.

@potiuk @rohityadavcloud I moved this to an apache branch so all that have interest can add to it.

@assignUser Can you have a brief review of this? Maybe^H^H^H^H^HProbably I am missing something obvious...

thanlks @assignUser , not please the @rohityadavcloud has taken this further by now in #6956, maybe you can give us some advice there?

closing in favour of #6956

closing in favour of #6956


New Zowe Repository - CLI FTP Plugin

The CLI Squad has another plugin they would like to donate to Zowe - the Zowe FTP plugin. This plugin began as an innovation effort by a member of the CLI Squad, and exposes the following functionality:

Delete USS files or MVS datasets
Download USS file, MVS dataset, or Job Output
List USS files or MVS datasets
Rename USS files or MVS datasets
Submit Jobs from Dataset Member
Upload USS files or MVS datasets
View USS files, MVS datasets, or Job Output

The source code is staged here: https://github.com/MarkAckert/zowe-ftp-plugin-staging .
@zowe/zlc @jmertic Please review and comment here if you have questions or concerns.

Next step is for @jmertic to clarify legal issues around binary distributions and will provide an update on 2019-04-23

Competed

@MarkAckert to verify dependencies on 3rd party open source.

Plugin that donated into Zowe makes use of the zos-node-accessor package from IBM open source: https://github.com/IBM/zos-node-accessor...


bin/setup - No env file?

= Copying sample files ==
.env
cp: .env.example: No such file or directory
--
Is that file perhaps missing from repo?

You need to add your own .env file with your api_keys. I added an example file for documentation


chore: add description to keymaps

Thanks for the plugin!
Some plugins like folke/which-key.nvim use the "desc" field on the opts table when calling vim.keymap.set to show information about the keymap.

Thanks for the contribution!


[TEVA-2269] Remove back links when not in stepped process

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-2269

Review app deployed to https://teaching-vacancies-review-pr-3337.london.cloudapps.digital


dry run data migration from legacy to new system

Steps that need to happen for migration from redmine to lexbox

change dns for domain names to lexbox prod in maintenance mode
take a dump of the redmine db and import into a local mysql db
execute /api/migrate/migrateData in swagger with an instance connected to that local mysql db.
export local postgres db
import dump to prod postgres db
copy repos from redmine to lexbox (need some help on the best way to do this)
take lexbox prod out of maintenance mode

can be done later:

execute /api/project/updateAllRepoCommitDates api. Might take a while and need tweaking
execute /api/project/autoUpdateUnknownProjectTypes tracked in #199

things to figure out

[x] how exactly the transfer repos from redmine to lexbox
[ ] #223

new plan

migrate all sql data at once
proxy s&r traffic to redmine, use the secret key that LF Merge is using for auth to redmine
migrate repo data one at a time from redmine to avoid data corruption issues
in case of proxy issues we can rollback DNS changes

Need to determine specifically how we're going to migrate data. Do we use SSH? or pull the data out via a hg clone? or something else I'm not sure. I'd like this to happen automatically at the push of a button, not a manual task that a dev has to do.

Recommended rsync options: -vaxHAXP

-v is "verbose" and lets you see the file currently in process in rsync's stdout
-a is "archive" and basically enables a bunch of other flags that say "copy permissions, modified dates, descend recursively, copy symlinks as syminks"
-x is "one file system" and says "don't cross filesystem boundaries" (in case there's a mount point underneath what you're copying)
-H is "hard links" and preserves hard links (in addition to the symlinks included in -a)
-A is "ACLs", so if the source filesystem used Access Control Lists, those will be saved
-X is "xattrs" and preserves extended attributes (used for file metadata and other things)
-P is "show progress and keep partial files": it shows a progress indicator during file transfer (so you can see how much time remains on that one file) and if the transfer is interrupted partway through, the partially-transferred file is kept. This will allow the next rsync to resume from the point where the transfer was interrupted, and eventually finish even in a flaky network environment.

If you don't need verbosity, -v can be dropped and -P can be replaced with --partial, which keeps partial files but doesn't show progress.
An optional -z can be added to compress files before transfer, which saves bandwidth at the expense of more CPU usage. In an AWS-to-AWS environment, we'd want to leave -z off.

migration was a success! This looks like a good plan at the moment.


Correction for raw affiliation Compsys team, LIP, Computer Science Laboratory, UMR 5668 CNRS-ENS Lyon-UCB Lyon-Inria, France

Correction needed for raw affiliation Compsys team, LIP, Computer Science Laboratory, UMR 5668 CNRS-ENS Lyon-UCB Lyon-Inria, France
raw_affiliation_name: Compsys team, LIP, Computer Science Laboratory, UMR 5668 CNRS-ENS Lyon-UCB Lyon-Inria, France
new_rors: 02kvxyf05;029brtt94;02feahw73;04msnz457
previous_rors: 02kvxyf05;029brtt94;02feahw73
works_examples: W2120533159
contact: 43bad7b4208912693c8fbbd5d3b85fe4:77ab09549df4b5f233a32d7ccbec @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Not respecting --steps when using --init-image

Every time I use a seed image with --init-image Imaginairy runs with 15 steps.  As you can see below, it only runs 15 steps even though it knows I asked for 40.
Is there a way to force it to use more steps in this scenario?
❯ imagine --init-image="/Users/natelandau/Desktop/sailor.jpeg" --steps=40 "sunny day"
🤖🧠 imaginAIry received 1 prompt(s) and will repeat them 1 times to create 1 images.
Running in CPU mode. it's gonna be slooooooow.
Generating 🖼  1/1: "sunny day" 512x512px seed:852196928 prompt-strength:7.5 steps:40 sampler-type:k_dpmpp_2m
Loading model /Users/natelandau/.cache/huggingface/transformers/24bd254e54b30e83bcbc15efae29f0e onto cpu backend...
    Loaded input 🖼  of size (1024, 768) from /Users/natelandau/Desktop/sailor.jpeg
100%|████████████████████████████████████████████████████████████████████████████| 15/15 [00:38<00:00,  2.58s/it]

that is strange.

okay yeah I remember. while i agree it's unexpected it's because the --init-image-strength defaults to 0.6 which means it skips the first 60% of the generation process.
40 steps * (1 - 0.6) ~= 15


backend user metadata

[x] setting user metadata feature doesn't change,
[ ] can't save metadata with symbols or umlauts just like {  or ö. - Unknown base64 encoding at char: {
[ ] AuthService: Error on deleting some keys - Errors.Metadata.NotFound (META-ncnw3)
[ ] MgmtService: Error on saving -  User could not be found (COMMAND-4M0fs)

I can't reproduce the second error. I encoded Hallöle with base 64 and it worked fine


runtime error with httprequest

modules/httprequest/tests/simple_get.wx on macOS:
Wonkey Runtime Error: SIGNAL: Memory access violation
GC state=IDLE

On macOS, need to install wget(brew install wget) before.

On macOS, need to install wget(brew install wget) before.


Thesauri section

There is currently a sub-section about Thesauri in the meta-data section. Since none of those thesauri are actually on meta-data (they are domain knowledge) they should probably be in a separate section.

I was struggling with where to put the content on semantic web specifications for metadata. Finally it was moved to the Harvester component now, where sources for bot metadata and knowledge harvesting are described. Can the Thesauri chapter remain now as it is? Does it make more sense now?


Move out of Heroku

We should finally do this.
This will allow us to drop Play framework and use something decent: #78 #112
Also it will allow us to use encryption without paying $20 every month: #73

We may try to use http://theforeman.org/

The application itself is already moved and available at http://codingteam.org.ru:9000/

Don't forget to close all Heroku-specific tasks when closing this.

Fuckin' done!

Fuckin' done!


Add CORs errors

Is your feature request related to a problem? Please describe.
Yes it is related to a problem all modern browsers will block requests to this API
Describe the solution you'd like
Add CORs headers allowing all origins allow-origin: *
Describe alternatives you've considered
N/A
Additional context
N/A

On it

done


Fix a bug of running node selector predicate on kubelet introduced by #5...

...715.
part of #5207, and fixed #5810
cc/ @davidopp @bgrant0607

LGTM
Will merge once Shippable or Travis finishes.

Shippable passed so I'm merging.


not compatible on WeChat Video chatting

I make a fork on my account，add mft for MPEG to NV12，because none of my notebooks can use this sample。
https://github.com/comor86/Windows-Camera
It works for most apps，but not on WeChat PC version(https://windows.weixin.qq.com/)。
When I using the virtual camera for video chat，I can't switch to the physical camera，and after closing the video chat，the Camera Led still lighting, unless I exit the WeChat app.
I make a small chang on SetStreamState()
        winrt::slim_lock_guard lock(m_Lock);

        RETURN_IF_FAILED(_CheckShutdownRequiresLock());
        DEBUG_MSG("[%d] SetStreamState %d", m_dwStreamId, state);

        wil::com_ptr_nothrow<IMFMediaStream2> spStream2;
        RETURN_IF_FAILED(m_spDevStream->QueryInterface(IID_PPV_ARGS(&spStream2)));
        RETURN_IF_FAILED(spStream2->SetStreamState(state));

        if (state == 0)
        {
            std::thread([&]() {
                std::this_thread::sleep_for(std::chrono::milliseconds(3));
                if (m_parent != NULL)
                {
                    m_parent->Shutdown();
                }
                });
        }

        return S_OK;

And it seems work, but cause other problem.
Wish a better dealing for this, thanks!

If no change added, below is the log. Up is normal close, down is WeChat unnormal close.

Hi @comor86, sorry for the delay in response, I was out for vacation and am back today. Indeed I would not advise that you shutdown the virtual camera upon stream stop :\ as this will lead to headaches when virtual camera user (i.e. an app on behalf of their consumer) stops then want to start again.
Looks like WeChat is keeping a hold onto the camera and not properly disposing of their camera handle when not using the camera which effectively locks the virtual camera from being used somewhere else (and the physical camera it wraps as well). If you use the API released in newer SDK, i.e.:

asking for the MF_VIRTUALCAMERA_PROVIDE_ASSOCIATED_CAMERA_SOURCES https://github.com/microsoft/Windows-Camera/blob/1d1cee6f4f26ef1a4883a5d22ad30db35ff96849/Samples/VirtualCamera/VirtualCameraMediaSource/VirtualCameraMediaSourceActivate.cpp#L322
then using the MF_VIRTUALCAMERA_ASSOCIATED_CAMERA_SOURCES to retrieve the IMDMediaSource of the target physical camera you wish to wrap https://github.com/microsoft/Windows-Camera/blob/1d1cee6f4f26ef1a4883a5d22ad30db35ff96849/Samples/VirtualCamera/VirtualCameraMediaSource/VirtualCameraMediaSourceActivate.cpp#L27

then the problem should be avoided. if not using this API then you would need to terminate the IMFMediaSource of the wrapped camera upon stopping the stream, and recreating it upon starting.. but this strategy would not guarantee success of recreating the device source of the wrapped camera if another app is holding onto it concurrently.

I wonder what this mean " If you use the API released in newer SDK"? I'm using SDK 10.0.22000.0.
According to the sample code, there is a comment about MF_VIRTUALCAMERA_PROVIDE_ASSOCIATED_CAMERA_SOURCES.
// The below 2 GUIDs are defined in Windows build 22621 and above only,
// if targetting SDK for Windows 22621 or higher this would need to be commented out.
// GUIDs for retrieving the device source instead of recreating it from within the vcam
DEFINE_GUID(MF_VIRTUALCAMERA_PROVIDE_ASSOCIATED_CAMERA_SOURCES,
    0xF0273718, 0x4A4D, 0x4AC5, 0xA1, 0x5D, 0x30, 0x5E, 0xB5, 0xE9, 0x06, 0x67);

DEFINE_GUID(MF_VIRTUALCAMERA_ASSOCIATED_CAMERA_SOURCES,
    0x1BB79E7C, 0x5D83, 0x438C, 0x94, 0xD8, 0xE5, 0xF0, 0xDF, 0x6D, 0x32, 0x79);

Does it mean that I can't use this two features on Windows 11 21H2, and could use them on 22H2? I have tried your suggestion and found that if (SUCCEEDED(m_spActivateAttributes->GetUnknown(MF_VIRTUALCAMERA_ASSOCIATED_CAMERA_SOURCES, IID_PPV_ARGS(&spCollection)))) didn't return S_OK, but RETURN_IF_FAILED(m_spActivateAttributes->SetUINT32(MF_VIRTUALCAMERA_PROVIDE_ASSOCIATED_CAMERA_SOURCES, 1)); actually called and return S_OK, so the problem still there.

There are 2 different topics to disambiguate:

the SDK version your project targets (i.e. in the vcxproj that defines the minimum Windows version required to run this compiled binary)
the OS version at runtime of the virtual camera consumer (an app/vcam built using 22000 SDK may run on Windows version 22000+).

This header defines GUID that are also defined in the 22621 SDK to opportunistically leverage functionalities available in Windows 22621 without changing the project target SDK version. That's why this comments says that if you change your project target to 22621, then you have to comment those GUID definition since they will conflict with the SDK defining them as well in a required header.
If this code runs on a machine updated to Windows 22621, then the line
if (SUCCEEDED(m_spActivateAttributes->GetUnknown(MF_VIRTUALCAMERA_ASSOCIATED_CAMERA_SOURCES, IID_PPV_ARGS(&spCollection))))

should return true and avoid the problem you exposed with WeChat. If running on Windows version 22000, then you can gracefully do something else meanwhile to accommodate the app not disposing properly of the camera when stopped and not in use.
If you can run exclusively on Windows 22621 and above then the problem would be avoided. If not, then you could put in place the aforementioned mitigation of disposing the of the IMFMediaSource on Stop, and recreating it upon start using again MFCreateDeviceSource() (albeit with its set of caveats)

closing, please reopen if you encounter an issue still in your implementation


apugli:modify_enchantment_level crashes game if apoli:enchantment entity condition is present

This power crashes the came if an entity condition apoli:enchantment is put inside the condition field
Example Power:
{
    "type": "apugli:modify_enchantment_level",
    "enchantment": "minecraft:fire_aspect",
    "condition": {
        "type": "apoli:enchantment",
        "enchantment": "minecraft:protection",
        "calculation": "max",
        "comparison": "==",
        "compare_to": 0
    },
    "modifiers": [
        {
            "operation": "add_base_early",
            "value": 1
        }
    ]
}

Adding on, apoli:enchantment Item Condition also seems to crash if added via apoli:equipped_item Entity Condition

Going to look into this rn

Okay so the issue lies in the game just running the same bit of code over and over due to the condition being met and unmet repeatedly.

I've fixed this by letting the power cache the modifier value of this power and compare it to the current power, so yeah, it should now work!

Will be implemented in 2.0.0

This will be fixed in 2.0.0, thank Jarva for the help.

Fixed in v2.0.0


VecMap::as_slice causes segmentation fault

I am seeing a segmentation fault error calling VecMap::as_slice().
I think the memory layout of struct Slot<K, V> will not be the same as (K, V) unless you use #repr(packed).
pub fn as_slice(&self) -> &[(K, V)] {
    // SAFETY: `&[Slot<K, V>]` and `&[(K, V)]` have the same memory layout.
    unsafe { &*(self.base.as_slice() as *const [Slot<K, V>] as *const [(K, V)]) }
}

I am happy to make a pull request to fix this and add some tests if you like?

A repr(packed) may not solve the issue. The tuple layout does not seems to be packed from this doc (https://doc.rust-lang.org/reference/type-layout.html#tuple-layout). Maybe an easier way is to replace Slot by something easier to transmute to pair.
struct Slot<K, V>{key:K,val:V}

#[repr(packed)]
struct SlotP<K, V>{key:K,val:V}

struct SlotT<K, V>(K,V);
struct SlotW<K, V>((K,V));

static_assertions::assert_eq_size!(Slot<i64, String>, (i64, String));
static_assertions::assert_eq_size!(SlotP<i64, String>, (i64, String));
static_assertions::assert_eq_size!(SlotT<i64, String>, (i64, String));
static_assertions::assert_eq_size!(SlotW<i64, String>, (i64, String));

static_assertions::assert_eq_size!(Slot<i64, u8>, (i64, u8));
// static_assertions::assert_eq_size!(SlotP<i64, u8>, (i64, u8));  <-- fail
static_assertions::assert_eq_size!(SlotT<i64, u8>, (i64, u8));
static_assertions::assert_eq_size!(SlotW<i64, u8>, (i64, u8));

static_assertions::assert_eq_align!(Slot<i64, u8>, (i64, u8));
// static_assertions::assert_eq_align!(SlotP<i64, u8>, (i64, u8));  <-- fail
static_assertions::assert_eq_align!(SlotT<i64, u8>, (i64, u8));
static_assertions::assert_eq_align!(SlotW<i64, u8>, (i64, u8));

Yikes, this is bad. Can you provide a reproducer for this segfault? I suspect this might be caused by field reordering, e.g. when tuples like (u8, i64) are involved.
Changing Slot to SlotT will probably not help as this is also subject to field reordering AFAIK, but something like SlotW might fix the issue.
In any case this calls for better test coverage for as_slice with various tuple combinations. Maybe miri could have detected this, but I never used it until now. Maybe I should...
If you to work on a fix please go ahead. I will not be able to work on this until the new year :).

The safest bet should be to just use a transparent wrapper around a tuple to ensure the layout is always the same as (K, V):
#[repr(transparent)]
struct Slot<K, V>((K, V));

@extremeandy thank you for reporting this! I just released vecmap-rs 0.1.14 which fixes the unsoundness. Merry christmas 😉.


[BEAM-9136]Add licenses for dependencies

Design proposal: https://s.apache.org/eauq6
This PR includes

Added scripts to pull licenses for Python, Java and Go.
Added licenses pulled for Python, Java and Go.
Copied licenses to docker images.

Not done yet.

A script to check missing license files daily.
(maybe)Release script or manual to check missing license files.
R: @tvalentyn
Cc: @alanmyrvold

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

For example, this PR includes licences for google-cloud-videointelligence. I suspect this license got pulled because it was installed in the environment on your machine, when you ran these scripts. I think we should strengthen the script to not include extra dependencies.
Ah, looks like we recently picked up this dependency: https://github.com/apache/beam/commit/af19496435483d690864e85fa98ed15b86cece7b#diff-e9d0ab71f74dc10309a29b697ee99330
I think the rest of my comment still applies (that the script fetches licenses for all dependencies currently installed).

I'll second the comment that we should include the list of license files in the beam repository itself, but just add the script to the container build process to install them into the container.
In terms of checking, this script would fail if it was not able to figure out the license for a dependency. This way whoever adds the new dependency would be responsible for making sure we were able to get its license in order to get the PR green. (We should be building containers as part of our precommits.)

In terms of checking, this script would fail if it was not able to figure out the license for a dependency. This way whoever adds the new dependency would be responsible for making sure we were able to get its license in order to get the PR green. (We should be building containers as part of our precommits.)

I was thinking to run a daily job to 1). check dependencies and missing/additional license file, 2) create a PR and send out for code review automatically. The PR will be sent to two or three person from a group of approvers without identifying which PRs introduced/removed dependencies, because identifying them is cost expensive.
Is daily check good enough?

In general, it's preferable to avoid checking in script-created files, preferring instead to create them on demand at build time. (There are exceptions, e.g. bootstrapping and/or difficult to obtain build tools, but I don't think that applies here.)
As another principle, it's preferable, where possible, to enforce correctness of the repository state before things get committed/merged (e.g. at pre-commit time), rather than have a separate, asynchronous, external process to try to fix things up after the fact.
Perhaps it's worth expanding the design doc on how we could do this?

In general, it's preferable to avoid checking in script-created files, preferring instead to create them on demand at build time. (There are exceptions, e.g. bootstrapping and/or difficult to obtain build tools, but I don't think that applies here.)
As another principle, it's preferable, where possible, to enforce correctness of the repository state before things get committed/merged (e.g. at pre-commit time), rather than have a separate, asynchronous, external process to try to fix things up after the fact.
Perhaps it's worth expanding the design doc on how we could do this?

Yep, I will add it to the doc and send out for review when it is ready. Thanks for explanation!

@tvalentyn , I fixed all your comments about Python. PTAL when you have time.

R: @alanmyrvold , do you have time to take a look for Go and Java?

I'll second the comment that we should not include the list of license files in the beam repository itself, but just add the script to the container build process to install them into the container.

I agree. It would seem better for all languages to add the license retrieval to the container build script.

I made following changes with Python scripts.

License files will not be checked in. They will be generated from scratch every time when docker images built.
Updated DockerFile to run the script.

Please let me know if I need to squash two python script commits if that makes it easier to review.
I will work on Go and Java in separate commits.

Run Seed Job

Run Python DockerBuild PreCommit

Run Python DockerBuild PreCommit

Run Python DockerBuild PreCommit

Run Seed Job

Run Python DockerBuild PreCommit

Run Python DockerBuild PreCommit

I wonder if it would make sense to have separate, more focused PRs for each of Python, Go, and Java.

I wonder if it would make sense to have separate, more focused PRs for each of Python, Go, and Java.

yep, will create separate PRs for each language.

Run Python DockerBuild PreCommit

I changed this PR to Python only.
New PRs will be created for Java and Go.

A link to logs from failed PythonDocker test: log

kindly ping @tvalentyn

Run Seed Job

Run Python DockerBuild PreCommit

Run Python DockerBuild PreCommit

Run Python DockerBuild PreCommit

Run Python DockerBuild PreCommit

Run Seed Job

LGTM for last commit as well. Please squash commits before or during the merge. Thanks!

Run Python DockerBuild PreCommit

CommunityMetrics job fails with dockerpycreds.errors.InitializationError: docker-credential-gcloud not installed or not available in PATH, which doesn't seem related to this PR.

CommunityMetrics job fails with dockerpycreds.errors.InitializationError: docker-credential-gcloud not installed or not available in PATH, which doesn't seem related to this PR.

cc: @yifanzou

LGTM for last commit as well. Please squash commits before or during the merge. Thanks!

Thank you Valentyn, your review helped a lot!


Syndication: Activity Pub social networks

Like Mastodon, GNU Social... Any of those will suffice, as they can communicate between each other.

https://botwiki.org/resource/tutorial/how-to-make-a-mastodon-bot-the-definitive-guide/
For the mastodon account we could setup a bot that feed im from the rss feed, so social network folx woudln't have to feed it?


Update uk_ua.json

YET AGAIN
Fixes #
Proposed Changes

so much requests lol

also you don't have to make a new branch each time you update, just update the branch you're using

Oh ok ok


Support Wallet

It should have support for wallet. Error handling and checking of status.

2022-09-29 12:57:15
{"action":"debug","data":{"string":"ae59c34e-b2e8-45b4-85f6-b5f38346d968: app was spawned with pid 2997"},"fiber":26686600,"timestamp":"2022-09-29T10:57:15.595434Z"}


Some cleanup of structure, tests, CI

fixed CI to test on Julia 1.9 and 1.10 only.
Got rid of sub-modules, there seems no point given how small the project is and it just complicated importing and exporting of functions
simplified test infrastructure

@tinatorabi  -- ready to take a look at this. There is more cleanup to be done, which we can do in separate PRs though.

Given this seems to be working, I suggest to add documentation  in the README only for now and then we tag v0.0.1 asap. Once tagged we can use version control for running final tests for the paper and then integrate this into ACEfit ?


Investigate performance of menubar.updateOverflowAction

Follow up from https://github.com/microsoft/vscode/issues/165441
When profiling window resize on Windows, this method appears as somewhat slow:

This is not entirely surprising, given it checks on container.offsetWidth.
Maybe we can do a bit of debouncing or scheduling on the next animation frame? Editor tabs have a similar problem where we need the container width, but we do debouncing there:
https://github.com/microsoft/vscode/blob/7b70ddcd7647b531970ecc32b14bda66c90be6c9/src/vs/workbench/browser/parts/editor/tabsTitleControl.ts#L1471
Profile:
CPU-20221223T064049.zip
//cc @jrieken

Maybe we can do a bit of debouncing or scheduling on the next animation frame?

The debouncing will help in the sense of running this less often but we could also attempt to buy more into the measure/modify pattern so that forced layouts happen less likely (see https://web.dev/avoid-large-complex-layouts-and-layout-thrashing/#avoid-forced-synchronous-layouts)


Open Ephys file format

Added support for Open Ephys file format
OpenEphys.py from https://github.com/open-ephys/analysis-tools.git, with minor changes
openephysio.py
Tested with Python 3.5.2 and 2.7

Thank you very much for this PR.
I also wanted to add openephys in neo.
Neo is io is moving slowly toward rawio for some format that need chunk acces and speed.
Format integrated in neo.rawio have automatically neo.io.
tridesclous and ephyviewer are already based on this rawio low level API.
I think having openephys in rawio will be important for spike sorting on long datasets.
I will have a look on thsi io but after would you mind if I try to port this Io to rawio ?

No problem, port it to whatever you find more appropriate. Happy to contribute.

@Palaeocortex:
files must not be upload directly in the neo git.
They should be uploaded here at gin gnode
See doc.
Best.

Hi @Palaeocortex
I am back on this.
I plan to rewrite this directly at neo.rawio level
So user of tridesclous could also use this IO.
I will do a lot od copy/paste from your code.
I will open a separate PR.
Could you send me the file for testing ?
When the new implementaion will be done could you help me by rewriwing it ?
best
Samuel

I got your file.
It was in your branch.

hi @Palaeocortex
I have started look at the dataset you provide in your PR.
I have read the the format doc
And I have some questions:

Where does the xml files come from ?
Are they part of open ephys ?
Why does some files have the following naming 100_CH1 and some other 100_CH1_2 ?
Does the _2 have a meaning of a new Segment ?
If so it is not present in your code.

Thank you for helping.
best

Hi Samuel! Sorry for uploading data in the wrong place, I forgot to correct.
The xml files are part of OpenEphys and carry information about GUI settings and channel info.
The _2 says that the measurement has been interrupted and restarted.  100_CH1 is the first measuring session, then recording stopped and started again. The second session is called 100_CH1_2, the third 100_CH1_3 and so on. My code checks the timestamps and puts the snips together, adding zeros inbetween, for the duration of the brake. The example file shows one such interruption. I can happen if the experimenter wants to interrupt or when occasionaly the OpenEphys GUI breaks and needs to be restarted.

Hi Cristian,
Thank you for clarification.
I did not get that from code nor it is explain in format documentation.
I will change this behavior to be more neo compatible with creating several neo.Segment
This will avoid to create to much zeros in between "session" (segment)
Best,

@Palaeocortex
If you have time please have a look to #580
For the moment the multi segment is not taken in account but it won't be difficult.

I'll check the code in #580. Maybe both behaviors should be implemented? In my case, it was desirable to have a single segment because data belongs to a single experiment with important timing. With several neo Segments I'd have to stick them together manually and add zeros. With software crashes you lack any other choice.

Yep. But guleing with zeros could be done outside this IO.
It is  generic for all IO with multi segment support.
It is more tools that could be( or not) in neo but I don't think to add this option  specifically to this IO
The new way of neo is :  load from disk only the necessary chunks to save memory bandwith.
So the fragmentation is kept as close as possible from the datset.

@Palaeocortex
Hi.
I implementing teh multi segment stuff in openephys io.
I experience some problem with the datset you provide.
For a given segment all channel files do not have the lenght.
How is possible ?
And worst than than the filesize is a not multiple of 1024 as it should be.
How is it possible. There is no garanty of atomic buffer writing at acquisition in OE ?

superseded by #580

Hi! Yes, the file length is not consistent and this occurs often, especially when the OpenEphys GUI freezes. I'm still loading the data because just one data point is missing (if I remember correctly), that means an interval of 1/sampling rate. It is not the official behavior of the OpenEphys GUI though, at least from what I understand from the documentation. I made the consistency checks in my code a bit more flexible to deal with this minor events.


[Machine Learning] How to build a GlassBox Model using InterpretML Python

Proposed title of article
[Machine Learning] How to build a GlassBox Model using InterpretML Python
Proposed article introduction
Glass box models are models that are transparent to the user. In a glass box model, all the features and parameters are known to the user and they know the criteria used by the model to reach its prediction results and conclusion. This gives full transparency and one can know how the model works and functions behind the scene.
Over the years, many machine learning engineers have been building black-box models. Black-box models, the users do not know the internal workings and the functionality of the model. Users cannot understand these models and they end up accepting the model blindly.  Developers hide the actual risk of using these models, Glassbox model is the better alternative that gives you exposure to the model.
In this tutorial, we will use InterpretML to build a glass box model. It will build a model that will be able to explain itself and give a more transparent view to the users. This will improve quality products, machine learning processes, and research. Examples of glass box models are decision tree models and linear models.
Key takeaways

Black box models vs Glass box models.
Example of the Glass box model.
How to work with InterpretML Python
Building a transparent.
Analysing the model results.

Article quality
This tutorial will compare two concepts of building machine learning models: Black box models and Glass box models. It will look into the traditional black-box models and some of their limitations that led to the introduction of glass box models. We will dive into a glass box model and build a custom transparent model from scratch.
Finally, we will analyze the prediction results made by the model so that we can have a better understanding of the model.
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Great topic 🚀 , make sure it matches the following:

Adds value beyond what is in the official docs and what is covered in other blog sites.
To avoid potential article closure, it should not overlap with existing EngEd articles or incoming topic suggestions.

Please reference any relevant EngEd articles in yours and build a unique project - Approved.


Update create-from-builder component to use getPreferredVersion

First of a small handful of these to update in catalog.


kyber/v3 and XORKeyStream

Hopefully, I have done that correctly.
I took the branch from the pull request (that updates to kyber/v3) and fixed the XORkeyStream. Additionally, I had to update some test that seemed to be broken. I think it is due to updating the kyber library. It just needed a few changes.
Two changes:
-crypto.go: uses a function from kyber library which in its current version have a proper XORkeyStream in place.
-Nonetheless, I fixed the error in XORKeyStream from util.go, if it wants to be used instead of the one in kyber.

Can you merge master back into your branch and resolve conflicts? Then I'll merge

I haven't merged into my master branch because I have modified the fork quite a lot to do other things I've been testing that I think they are not necessary for the protocol itself. Therefore, the branch pr/4 holds an older version of what i have now.
It would be great if you could just merge that branch. I took a look to the conflict and it is just 1, in which "my" crypto_test.go does not have the tests provided by a previous merge from Ne0nd0g. I think you can just merge altogether.

I meant merge this master. Git and GitHub don't allow merging a conflict. I will have to do the conflict resolution and push myself when I get a chance, no prob.

I have some time to look at this now! Will merge and maybe make a minor syntax adjustment or two...thanks!


HOLD: Testing Travis runs tests on subsequent pushes to PR branches

Testing that changes work as expected from #4680
This breaks FE tests and then fixes them to make sure the tests run in Travis on a subsequent push to this branch.
Additions

Removals

Changes

Testing

Screenshots
Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer 8, 9, 10, and 11
[ ] Edge
[ ] iOS Safari
[ ] Chrome for Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

@chosak successssssss pushes to PRs still trigger the tests https://travis-ci.org/cfpb/cfgov-refresh/pull_requests note this PR shows up twice 👍

and merging master in using "Update branch" also triggered the tests as expected. 💯  ok closing this fake PR now, thank u krang


fix: Cat Photo app 61 - give description better readability

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #48871

Seems like the issue has already been closed by another contributor.
Have a nice day and happy coding


Integers bigger 1M are being converted into exponential form

Output of helm version:
version.BuildInfo{Version:"v3.12.1", GitCommit:"f32a527a060157990e2aa86bf45010dfb3cc8b8d", GitTreeState:"clean", GoVersion:"go1.20.4"}

Steps to reproduce:
1) values.yaml:
someVarOne: 999999
someVarTwo: 1000000

2) templates/configmap.yaml:
apiVersion: v1
kind: ConfigMap
metadata:
  name: whatever
data:
  ONE: {{ .Values.someVarOne }}
  TWO: {{ .Values.someVarTwo }}
  TWO_Q: {{ .Values.someVarTwo | quote }}

3) render that
% helm template .
---
# Source: whatever/templates/configmap.yaml
apiVersion: v1
kind: ConfigMap
metadata:
  name: whatever
data:
  ONE: 999999
  TWO: 1e+06
  TWO_Q: "1e+06"

Expected behavior:
% helm template .
---
# Source: whatever/templates/configmap.yaml
apiVersion: v1
kind: ConfigMap
metadata:
  name: whatever
data:
  ONE: 999999
  TWO: 1000000
  TWO_Q: "1000000"

Some part of the yaml parsing seems to be converting the seven-digit value as a float:
https://helm-playground.com/#t=LTBQG9wAgOgNQIYBsCuBTAzjDB7AtmogE4AqA7jlAD5QAORAlgHYAuAZlAEQCkJnUAXwERo8ZOiy4CxcpRr1m7Ltwz8hoIA&v=LQhQGcHsFsFMDUCGAnA8gO1gLgAQE4DCIYEUAVAd0lwEIaBLdAFxwEYAGTroA
template:
--
{{ .Values.someVarTwo | printf "%T" }}
{{ .Values.someVarTwo | printf "%s" }}

values.yaml
---
someVarTwo: !!int 1000000

output:
--
float64
%!s(float64=1e+06)

(unsure which part)

I would suggest, that unless you plan on doing numeric operations on these values, simply represent them as a string:
someVarOne: "999999"
someVarTwo: "1000000"

(and if you do plan on doing numeric operations, use casts/printf to format the numbers as expected)
Helm itself, I don't think will be converting int -> float here. Rather, one of the libraries/frameworks that Helm uses to parse yaml. First, it would need to be identified where/what this conversion is happening. Then it would need to be understood if this conversion is the expected behavior. Then finally an issue raised upstream (presumably), a fix issued, then Helm would need to incorporate that fix. If you want to take on that investigation, I can point to the code where Helm looks to parse/handle values.

Thank you, but the place can be found easily. It's here: https://github.com/helm/helm/blob/a50206fed21cda67fdf8633ad3d7b38b0eb26da0/pkg/chart/loader/load.go#L105-L107
The guilty package is sigs.k8s.io/yaml, v1.3.0, released on Sep 23, 2021. It looks abandoned and I doubt it's going to be fixed.
The issue has somewhat bigger scope because we observe very questionable behavior. And this particular package is just an implementation. For example, that package can be replaced with another one.

simply represent them as a string:

I do. And very unhappy with that. Too many workarounds, too many implicit stuff. Not only in helm.

I just checked https://github.com/go-yaml/yaml as a drop-in replacement. Works fine but should be tested thoroughly because there are some minor differences (yes/no, on/off, octals).

Not stale

Not complete, still an issue as of Helm 3.14.3

Still an issue as of Helm 3.15.1

How are you fixed issue?

workaround:
      {{- if .Values.relay.processing.kafkaConfig.messageMaxBytes }}
      - name: "message.max.bytes"
        value: {{ int64 .Values.relay.processing.kafkaConfig.messageMaxBytes | quote }}
      {{- end }}


DDPG cannot work on the robotics environment.

Hi, I am doing a project on the robotics environment, which is in continuous space, so I am trying to use DDPG in the FetchReach-v1 environment:
python -m baselines.run --alg=ddpg --env=FetchReach-v1 --num_timesteps=1e6

it tells me:
TypeError: can only concatenate tuple (not "NoneType") to tuple

It seems that this problem is because the robotics environments have a dictionary-based observation space.
It would be much better if someone can fix this bug and make the baselines.run a really general method for all the gym environments.
Thanks in advance!

FetchReach-v1 is a GoalEnv which are designed to work with HER. The observations are a dictionary as the algorithm needs additional data to construction the reward for actual targets.

should be fixed in this PR: https://github.com/openai/baselines/pull/767, closing this issue

How can I train the agent with DDPG without her to compare the result? Didn't see that in her part.


Only 1 led working

the code seems to be only updating 1 led @ present - will break out my scope later to see if I can see any thing obvious with my setup

ID ten T error

ID ten T error


Fix Jenkins CI failures

This is a mixed bag to hopefully remove the spurious Jenkins CI failures.

Well, no matter how many times you restart it, something is always wrong with Jenkins. :disappointed:


AttributeError: module 'stweet' has no attribute 'GetUsersTask'

after i pip install -U stweet
i copy the example in readme, but it doesnt work
please help

Have you tried in docker container?
What's your OS? Python version?

no, I didn't try in docker container, i just try in command prompt
my os is win10

Could you paste code and output (I know that it's in readme, but I want to have it here)

Could you paste code and output (I know that it's in readme, but I want to have it here)

Please call:
import stweet as st
print(dir(st))

And send me output

And call in terminal
pip list

Check version of street

#96


Refactoring

Converted the old style var js variables into the newer const and let variables.
This should reduce the chance of bugs due to variable hoisting.
const is used when the value shouldn't change after it's intially set, and let is for the opposite case.
Converted the magic number Score Type to be a ScoreType enum so it's easier to understand what the value is throughout the code.
Made some changes so variables are defined closer to where they are used in the application, I'm not sure if this is as common in javascript/typescript as it is in .Net, I'm willing to move them back to the top of the block if needed.

I want to wait on #47 to be merged before merging this.


vmq-admin not working on an IPv6 cluster

I've setup a cluster with two nodes using ipv6
Environment
VerneMQ Version: 1.10.0
OS: Dockerized alpine 3.9.4
Cluster size: 2 nodes
VerneMQ configuration:
Just changed listeners to the proper IPv6 addresses and added "-proto_dist inet6_tcp" to vm.args for erlang distribution to listen to ipv6
Node1 vernemq.conf:
listener.tcp.default = 2f01::8:1883
listener.vmq.clustering = 2f01::8:44053
listener.http.default = 2f01::8:8888
listener.ws.default = 2f01::8:8080
Node1 vm.args:
+P 256000
-env ERL_MAX_ETS_TABLES 256000
-env ERL_CRASH_DUMP /erl_crash.dump
-env ERL_FULLSWEEP_AFTER 0
-env ERL_MAX_PORTS 65536
+A 64
-setcookie vmq
-name VerneMQ@2f01::8
+K true
+W w
-smp enable
+zdbbl 32768
-proto_dist inet6_tcp
Node2 vernemq.conf:
listener.tcp.default = 2f01::d:1883
listener.vmq.clustering = 2f01::d:44053
listener.http.default = 2f01::d:8888
listener.ws.default = 2f01::d:8080
Node2 vm.args:
+P 256000
-env ERL_MAX_ETS_TABLES 256000
-env ERL_CRASH_DUMP /erl_crash.dump
-env ERL_FULLSWEEP_AFTER 0
-env ERL_MAX_PORTS 65536
+A 64
-setcookie vmq
-name VerneMQ@2f01::d
+K true
+W w
-smp enable
+zdbbl 32768
-proto_dist inet6_tcp
-eval "vmq_server_cmd:node_join('VerneMQ@2f01::8')"
Expected behaviour
vmq-admin cluster show to work properly showing something like
+--------------------+-------+
|        Node        |Running|
+--------------------+-------+
|VerneMQ@2f01::8| true  |
|VerneMQ@2f01::d| true  |
+--------------------+-------+
And every other vmq-admin command to work
Actual behaviour
When running any vmq-admin command it just states that the node you're running it is not responding to pings:
node1# vmq-admin cluster show
Node 'VerneMQ@2f01::8' not responding to pings.
Anything you try with vmq-admin just fails with the same error
node1# vmq-admin bad-command
Node 'VerneMQ@2f01::8' not responding to pings.
Everything is listening either to the proper ipv6
node1# netstat -nalp |grep 126 |grep LISTEN
tcp        0      0 2f01::8:8080            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:44053           :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:8888            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1883            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1884            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1885            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1886            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1887            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1888            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1889            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1890            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1891            :::*                    LISTEN      126/beam.smp
tcp        0      0 2f01::8:1892            :::*                    LISTEN      126/beam.smp
tcp        0      0 :::9100                 :::*                    LISTEN      126/beam.smp
or in the case of port 4369 to all ipv4 and ipv6 addresses
node1# netstat -nalp |grep 205
tcp        0      0 <IP_ADDRESS>:4369            <IP_ADDRESS>:*               LISTEN      205/epmd
tcp        0      0 :::4369                 :::*                    LISTEN      205/epmd
The nodes linked properly on boot:
Logs on node1:
17:16:32.106 [info] successfully connected to cluster node 'VerneMQ@2f01::d'
17:16:32.267 [info] completed metadata exchange with 'VerneMQ@2f01::d'. repaired 0 missing local prefixes, 0 missing remote prefixes, and 2 keys
Logs on node2:
17:16:32.111 [info] successfully connected to cluster node 'VerneMQ@2f01::8'
In fact publishing and subscribing to any node works as it should. It's just the admin tool which fails

@joliver82 thanks, good catch. Need to see how to make the vmq-admin script and the RPCs it's doing understand IPv6.

Thanks @ioolkos If you need any further testing just ask. Just in case it helps you, each vmq-admin execution generates traffic to <IP_ADDRESS>:4369 (not ::1), the connection is established but I altough don't know what happens inside, I think the issue could be there...

Need something similar, so that CLI (vmq-admin) can work over IPv6
https://www.rabbitmq.com/networking.html#distribution-ipv6

@joliver82 could you test PR https://github.com/vernemq/vernemq/pull/1666?
ie. check if calls like this work:
sudo vmq-admin session show -kernel proto_dist inet6_tcp

Just tested it and I'm getting the same response (not responding to pings)

@joliver82 thanks for testing. okay... so this was the wrong approach :)

Update in PR: you can now add erlang.distribution.proto_dist = inet6_tcp to the vernemq.conf file. This is mainly to not have to use a separate vm.args file.
The RPC (ie vmq-admin) still does not work automatically though.

Update in PR: you can now add erlang.distribution.proto_dist = inet6_tcp to the vernemq.conf file. This is mainly to not have to use a separate vm.args file.
The RPC (ie vmq-admin) still does not work automatically though.

In theory this commit  https://github.com/vernemq/vernemq/pull/1666/commits/465277d36d2558a7086a3f6d6fa671caf4f51848 should pull the proto_dist config from the current environment (via env.sh) and add it to nodetool.
In practice it doesn't work, but I don't know why yet.

In theory this commit  https://github.com/vernemq/vernemq/pull/1666/commits/465277d36d2558a7086a3f6d6fa671caf4f51848 should pull the proto_dist config from the current environment (via env.sh) and add it to nodetool.
In practice it doesn't work, but I don't know why yet.

I'm facing a similar issue. When I change the name of the node to vernemq@ipv6_addr, the node fails to start. I added erlang.distribution.proto_dist = inet6_tcp  to the vernemq.conf file but it's not working. What I should do?

@AmnaSnene does the node log anything useful at boot?

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.


Pause of Node.js process before terminating it when running a query

I can't really pinpoint this issue and I definitely haven't been able to figure out what's causing it.
I experience a pause after the query execution and the before termination of the Node.js process.
I am running this code:
// app.js
console.time('start-app');
const marklogic = require('marklogic');
const settings = require('./settings');

const db = marklogic.createDatabaseClient(settings);
const qb = marklogic.queryBuilder;

db.documents.query(
  qb.where(
    qb.collection('reviews'),
    qb.term('film')
  )
).result()
.then(response => {
  console.time('db-response');
  const uris = response.map(r => r.uri)
  console.log(uris);
  console.timeEnd('db-response');
})
.catch(error => console.error(error));
process.on('exit', () => {
  console.info('exiting');
  console.timeEnd('start-app');
});

// settings.js
const host = 'localhost';
const port = 8000;
const user = '';
const password = '';
module.exports = {
  host, port, user, password
};

I then run the application via node app.js and this is how the output looks like:
$ node test.js
[ '/reviews/review14',
  '/reviews/review15',
  '/reviews/review17',
  '/reviews/review18',
  '/reviews/review20',
  '/reviews/review0',
  '/reviews/review1',
  '/reviews/review8' ]
db-response: 1.358ms
exiting
start-app: 6110.513ms

Note that when executing this code there's a pause after the db-response: 1.358ms line and the "exiting" line. I'm not sure what's causing this, whether that's the MarkLogic Node.js Client API or Node.js itself.

uuohhh, nice!
console.time('start-app');
const marklogic = require('marklogic');
const settings = require('./settings');

const db = marklogic.createDatabaseClient(settings);
const qb = marklogic.queryBuilder;

db.documents.query(
  qb.where(
    qb.collection('reviews'),
    qb.term('film')
  )
).result()
.then(response => {
  console.time('db-response');
  const uris = response.map(r => r.uri)
  console.log(uris);
  console.timeEnd('db-response');
  db.release();
})
.catch(error => console.error(error));
process.on('exit', () => {
  console.info('exiting');
  console.timeEnd('start-app');
});

Results in:
node test.js
[ '/reviews/review14',
  '/reviews/review15',
  '/reviews/review17',
  '/reviews/review18',
  '/reviews/review20',
  '/reviews/review23',
  '/reviews/review0',
  '/reviews/review1',
  '/reviews/review8' ]
db-response: 1.366ms
exiting
start-app: 114.039ms

I didn't know we had a db.release() - I did assume that it's automatic. Are there any implications to using it in the .then() statement as seen above?

The database client object does release all resources automatically when it is garbage collected.
Calling the release() function releases resources immediately without having to wait for garbage collection to determine that the database client is no longer used and it is safe to release resources.
The only concern about calling release() is releasing the database client prematurely.  If the database client is used by multiple event handlers and one of them calls release() while another is still making use of the database client, that could have unpredictable and unfortunate effects.
If the application doesn't know whether the database client is still in use, the application should leave it to the garbage collector (and not try to replace the v8 garbage collector in JavaScript code).


FTP Filesystem sends wrong FTP commands that leads to "550 Filename invalid" error

Laravel version: 9.26.1
PHP version: 8.0
Browser: Google Chrome 105.0.5195.128 (Official Build) (64-bit)

Description:
When attempting to upload something using the functions provided by the framework, a strange behavior will be seen from the vision of log file in the FTP Server.
Current behavior:
FTP Commands sent from the Laravel application:
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 230 Logged on
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> PASV
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 227 Entering Passive Mode (127,0,0,1,200,119)
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> NOOP
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 200 OK
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> CWD
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 250 Broken client detected, missing argument to CWD. "/" is current directory.
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> CWD test
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 250 CWD successful. "/test" is current directory.
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> NOOP
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 200 OK
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> CWD
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 250 Broken client detected, missing argument to CWD. "/test" is current directory.
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> TYPE I
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 200 Type set to I
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> STOR test/sample.jpg
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 550 Filename invalid
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> QUIT
(000024)07/07/1401 23:43:35 p.m - admin (<IP_ADDRESS>)> 221 Goodbye

Expected behavior:
FTP commands sent from native PHP:
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> 230 Logged on
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> CWD test
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> 250 CWD successful. "/test" is current directory.
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> TYPE I
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> 200 Type set to I
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> PORT 127,0,0,1,210,206
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> 200 Port command successful
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> STOR sample.jpg
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> 150 Opening data channel for file transfer.
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> 226 Transfer OK
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> QUIT
(000029)08/07/1401 00:26:24 a.m - admin (<IP_ADDRESS>)> 221 Goodbye

Here we have two major notes to improve + one critical error:

Why Laravel keeps using NOOP as it's not necessary because those commands are executed after each other without latency. (NOOP is used to keep the connection alive)
We use CWD just for changing the directory and not print the current directory. Actually this command is also used instead of PWD in addition to it's main goal.
BUT the reason that produced the "550 Filename invalid" error is that here Laravel has used a full path ("STOR test/sample.jpg") instead of a filename ("STOR sample.jpg").

Steps To Reproduce:

Create a FTP driver disk in config/filesystems.php.
Use Storage::disk('ftp')->putFileAs('path/to', new File('path/to/the/file'), 'new-name.jpg');

This leads to "ftp_fput(): Filename invalid" ErrorException which is because of the "550 Filename invalid" FTP error mentioned above.

Our filesystem is a wrapper around flysystem which makes using file based operations universal for all storages. If you claim that the FTP implementation behaves differently from other storages then I think it's probably related to flysystem itself and not laravel. Can you please try a support channel first and then if that doesn't help the flysystem issue tracker?

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC


schematics API reference

I see that there is a guide for Schematics, but I don't see any reference for Schematics API, so we can learn all about all Schematics functions.

Was looking for the same stuff on the official documentation, hopefully the Angular team will provide it soon (I hope). +1

any news?
@alan-agius4

There is no ETA for this yet.

Is there anything that can be done by the community to improve the docs? There are few tutorials with some API functions explained (like this one) so maybe extracting the description and using the type definitions would help with this task.

Hey, I am planning to write a blog post about some very common API Util functions you can use. Unfortunately those utils aren't documented yet but can basically be found here: https://github.com/angular/angular-cli/tree/master/packages/schematics/angular/utility
You can import the function from @schematics/angular/utility/file.
So for example handling changes on package.json will work like this:
```ts
import { Rule, SchematicContext, Tree } from '@angular-devkit/schematics/';
import {
  NodeDependency,
  NodeDependencyType
  getPackageJsonDependency,
  addPackageJsonDependency,
  removePackageJsonDependency,
} from '@schematics/angular/utility/dependencies';

export function playground(_options: any): Rule {
  return (tree: Tree, _context: SchematicContext) => {
    const dep: NodeDependency = {
      type: NodeDependencyType.Dev,
      name: 'my-package',
      version: '~0.3.4-beta.1',
      overwrite: true,
    };

    addPackageJsonDependency(tree, dep);
    console.log(getPackageJsonDependency(tree, 'my-package'))
    // { type: 'dependencies', name: 'typescript', version: '~3.9.2' }

    removePackageJsonDependency(tree, 'my-package');
    console.log(getPackageJsonDependency(tree, 'my-package'))
    // null

    return tree;
  };
}

I think it would be really helpful I the docs for the utils will be generated directly from the source in the future.
Probably a separate docs site would be great such as schematics.angular.io or something similar

@d-koppenhagen, those utils are not considered as part of the public API and might break without warning in any release.
There are plans to provide some utilities via public API but this is still in planning stages.

Okay that's good to know. Yes it would be really great as I've seen a lots of people authoring schematics with almost the same util functions which is I think not the best.

Hi @d-koppenhagen, thanks for your interest in helping with this.I do think however that this needs to be taken by someone on the team. The reason being is that entire API surface needs to be reviewed and decide which parts of it should be kept, modified or deprecated.

Agreed that schematics documentation is quite lacking right now and we are hoping to improve it. We have a near-future project adding common utilities to schematics and this would be a good opportunity to refactor the documentation and provide more comprehensive information. There will also be some non-trivial infrastructure requirements to generate API documentation from JSDoc comments and plumb them through to angular.io. This is already done for other Angular APIs, but it is more complicated for the CLI since we live in a different GitHub repository than angular.io.
This work is being tracked separately, so I don't think there's anything immediately actionable in this issue. I'm going to close it for the time being, but we hoping to improve this area in the near-future.


Window is plain white

Just testing out the api, window opens, renders and closes without error, but drawing anything to it fails. No errors in console, nothing.
Breakout:
public class Main extends Breakout {

    public Main(Identifier identifier) {
        super(identifier, new BreakoutWindow("Example Breakout", 480, 720));
    }

    @Override
    public void render() {
        super.render();

        MinecraftClient client = MinecraftClient.getInstance();
        MatrixStack stack = new MatrixStack();

        client.textRenderer.draw(stack, "bruh", 1, 1, 0);
    }
}

call:
public static void openGUI() {
        Identifier id = new Identifier("testmod", "br1");
        BreakoutAPIClient.openBreakout(id, new Main(id));

    }

am i doing something wrong?

okay apparently filling color with DrawableHelper.fill() works

aight then


Recent Logins

The recent logins information from sysinfo.cgi is sometimes incorrect
To reproduce

login to the CP and tick remember me
close the browser and change your ip to connect to the CP
relogin to the CP --- at this point you are not asked for a user name & password
from this the recent logins section shows you as logged in (correct) but shows the IP of when you hit the 'remember me' option and not your actual IP .. perhaps there should be a function to clean all obsolete logins and update the current one ?

Hi, Jim.
You can already view and disconnect current login session by going to Webmin/Webmin Users/View Login Sessions.
I would also recommend to bind session to an IP address, meaning when you login from another IP the session will not be valid. You can do it by going to Webmin/Webmin Configuration/Authentication and there tick Limit sessions to the same client IP address.
I doubt that Jamie when planning WebminCore missed the subject point. It seems to be made on purpose that session reflects the initial IP but not the recent one. In case you think otherwise, please file an issue to Webmin repo.


--active-role option broken for SSO

Since upgrading to a version with #467 in it, where the AWS ENV vars are not exported by default, I am unable to use the --active-role (or -ar) feature.
#-❯ assume -ar
[✘] one of the require environment variables was not found while loading an sso profile ['GRANTED_SSO_START_URL','GRANTED_SSO_REGION','GRANTED_SSO_ACCOUNT_ID','GRANTED_SSO_ROLE_NAME']

This is after I have done a successful assume where i can do other credential required things (such as aws s3 ls) and granted has put an AWS_PROFILE in my environment.
A snippet of my ~/.aws/confg looks like this:
[profile my-profile]
granted_sso_start_url      = https://my-sso.awsapps.com/start
granted_sso_region         = my-region
granted_sso_account_id     = my-account
granted_sso_role_name      = my-role
common_fate_generated_from = aws-sso
credential_process         = granted credential-process --profile my-profile

I'm not entirely sure if there is some config I'm missing, or if I'm supposed to run something extra with the regular assume to make it export the GRANTED_* variables that assume -ar wants?

Hi @tigris, with v0.20.0, you will be able to add the configuration DefaultExportAllEnvVar=true to your ~/.granted/config. This should default to exporting all environment variables. Let me know if that works for you.


Add async initialization of persistence layer

Closes #24

@bukinoshita Hey Bu, am I missing something? Can I help you here with more changes?

@bukinoshita Hey Bu, am I missing something? Can I help you here with more changes?

Sorry for the delay. I'm checking it rn

@bukinoshita Thank you :)


ERR max number of clients reached

Version
node_redis : 2.8.0

Platform
Node.js 6.3.0 on Mac 10.12.6

Description
When the redis client connection problems, such as network problems or passwords are wrong, the redis connection is not released, resulting in a period of time after the redis will be filled with the number of errors (ERR max number of clients reached).
You can deliberately configure the wrong redis password to reproduce the problem, in the redis-cli implementation of the 'CLIENT list' will find the connection is not over.

@WangDanpeng You'll have to show some code, but it sounds like your inappropriately creating clients.

I can not think of much that can be done to prevent this from happening. If a connection is lost, a new connection must be established. It will likely take time for Redis to find out that the client is no longer connected. I will try to think about it again if there is anything that could improve the situation.

With no activity from the original author in over a month, I'm going to go ahead and close this.


Add localization resources

I'll just start adding localization stuff to make this happen. The file I added is an altered version of the one found in the nodejs/iojs.org repo, which is changed to reflect the new localization repos and accommodate to the new file structure.
If I missed anything, let me know, since I still need to actually look at how the website itself works :smile:

Are you still working on this? If so, I think we should restart the list on the base translation file - I don't think we want/need all the io.js ones (correct me if I'm wrong!).

Do you want me to modify this once it is accepted, or do you want me to comment on it as-is? The former would probably be better in terms of modularity of issues.

I say we merge it, although we're going to have to update most of those links soon since the locale groups are going through name changes.

@mikeal Fair enough, I wasn't too sure on how to include those there.

You missed some repos for localization.
[iojs-ar](https://github.com/nodejs/iojs-ar)
[iojs-bg](https://github.com/nodejs/iojs-bg)
[nodejs-ko](https://github.com/nodejs/nodejs-ko)
[iojs-ms](https://github.com/nodejs/iojs-ms)
[nodejs-ta](https://github.com/nodejs/nodejs-ta)

LGTM, please add the groups @outsideris mentioned and we're good to go.

Real quick, what country exactly is nodejs-ta? I couldn't find anything.

@sup: Tamil: https://en.wikipedia.org/wiki/Tamil_language

Alright, it should be good.

Great, thanks!


RadialGauge animationTarget = "plate"  turning needle issue

Hi,
First let me thank you for this most powerful gauge tool kit! I use it to create a popular marine open source instrumentation software built on Angular. See https://github.com/mxtommy/kip
Lots of people have been running the software for years without any issue but following a recent upgrade from Angular 12 to 14, in combination with recent browser updates, the Radial gauge with animationTarget = "plate" has it's "normally" fixed needle start to turn just like if it was in animationTarget = "needle" - both faceplate and needle turn. I did trouble troubleshoot but I can't see why, it's random and generates no exceptions. It can sometimes be fixed with faceplate turning, then suddenly the needle starts moving also. It does so on different browsers.
In the image below the needle should fixed at 0 degrees
Chromium rotated by 2 to 3 degress

Chrome MacOS rotated by 195 degrees and moving constantly

Not sure but it looks like dependency issues or latest webstandard changes...?

您的邮件我已收到，谢谢！


Use descriptive PR names and Commit messages.

Dates are not good practice to use as Commit messages.
Include a brief description of what your changes consist of in that particular commit.
@dhairyaostwal

I'll keep in mind next time onwards. Thanks @akshatvg

I'll keep in mind next time onwards. Thanks @akshatvg


Add speechd module

This should allow us to add more modules like https://github.com/NixOS/nixpkgs/pull/32979 and set the default ones.

This would still be nice.

Still relevant.

Could be handy for #30760.

Feel free to take over my linked draft PR.

We should probably lay out a clear image of what we want this to do first. Right now the package seems to be doing what I would ordinarily expect a module to be handling. We could simply have a module which that starts the systemd service provided by the package, but I like to think we could do a little better than that. For example, we could be checking the config for certain values (pulse, alsa, pipewire pulse/alsa options come to mind) and determine within the module which flags should be passed (or settings set), instead of handing that burden to the user. If I set pipewire.pulse.enable = true it should be fairly obvious that I would want speechd to use pulse. And if for some reason I didn't, we could have speechd.audioMethod to override, not unlike how pinentry is handled in the gnupg module.
speechd.settings is an obvious addition. The syntax could be troublesome. If a dotconf format works out, dedicated modules for speech engines open up some possibilities: speechd supports generic modules, which we can use to extend support to some other engines such as coqui TTS and place these behind speechd.*. Beyond that, we could have speechd.modules where you could define custom modules.

package seems to be doing what I would ordinarily expect a module to be handling

That is because the options affect closure size. But for the backends/modules, the dependencies get mostly pulled in through the config files so if we only substitute the paths in the NixOS module, we will be able to have narrower closure and enable module without rebuilding speechd. This was the reason I wanted this in the first place.
We could have the NixOS module just copy files from the package (similarly to what fwupd does) and then patch them to correct paths there. Most people probably will not need to customize the settings beyond what modules are installed anyway. And if they really need to, they can always use environment.etc.


fix: cache only http resources

Closes #161

I did not verify this locally but assumed it should work.


MultimediaObjectDescriptor Naming

getObjectID() is misleading in that it returns _ for the ProtoWriter / Selector

This issue is unclear to me, could you please elaborate?

Sorry, github formatting ate the relevant parts (<> causes content to disappear...). getObjectID() returns <type>_<object> and not just <object> - unless the ObjectID is  <type>_<object> and i misunderstood our offline discussion as in #41

For all practical purposes, <type>_<object> is the 'objectID', even though there should never be two objects of different types which have the same <object> part.