New Members receive the locale of the currently logged in user, instead of i18n::$default_locale

Affected Version
Observed in SS 4.2.1
Description
When a new Member is created, the Locale property is populated from i18n::get_locale(). This method returns the current locale if set, and falls back to the i18n::$default_locale config property.
When a user logs in to the CMS, the current locale is set according to that user's locale (in LeftAndMain::init()).
This leads to new users inheriting the locale of the currently logged in CMS user, instead of the i18n::$default_locale property.
I don't think that the locale of the currently logged in member should influence the locale of new Members, especially if a developer has provided a default locale through yml config. The default locale should trump the locale of the current user.
Steps to Reproduce

Note the locale of the currently logged in user (in Security section of CMS)
Set the default locale to something else through yml config and flush

SilverStripe\i18n\i18n:
  default_locale: 'en_GB'

Try to create a new Member in the CMS. The locale will be pre-selected to match the logged in user rather than the specified default.

Why is this a problem

I suspect CMS users don't pay a lot of attention to the locale field, so I think it's important that the default is respected as users may not correct a problem if present
The workflow of setting up a new site may lead to en_US being the de-facto default locale, despite configuration otherwise*

*I discovered this issue because the New Member form always had en_US selected, even though I knew I'd set the default locale to en_GB. Once I dug in to how Members receive a locale I realised that my default administrator Member received an en_US locale during installation, before I changed the default locale through yml config. If I hadn't paid attention all new Members I created after that would have gotten the en_US locale too. (Note: Possibly the installer wizard asks you what locale to use for your default admin, but I use a parent .env file so skip this process.)

Thanks @robbieaverill!

No worries! Fixed in #8423 in 4.2.x release line


cant upgrade from 0.5.8

Hello,
today i  tried to upgrade from 0.5.8 > 0.5.11
But I run into an Error. :(
I tried also to delete the Adapter and reinstll but same error.
Have somebody the same issue? Can somebody help me?
npm verion =  10.5.0
node verion = v21.7.3
python verion = 3.11.2 >>maybe this is the issue? I will try to upgrade
OS = Debian 12 on proxmox
Many thanks
BR
root@FHEM-SERVER:/opt/iobroker/node_modules# iobroker upgrade  "homekit-controller" --debug
This upgrade of "homekit-controller" will introduce the following changes:
-> 0.5.11:
Stability and compatibility improvements for BLE
-> 0.5.10:
Fix user interface
-> 0.5.9:
Prevent crash when a single value is returned as error status
send booleans always as 0/1 to make sure all devices handle it correctly
Update dependencies
Would you like to upgrade homekit-controller from @0.5.8 to @0.5.11 now? [(y)es, (n)o]: y
Update homekit-controller from @0.5.8 to @0.5.11
NPM version: 10.5.0
Installing<EMAIL_ADDRESS>(System call)
npm WARN deprecated<EMAIL_ADDRESS>Rimraf versions prior to v4 are no longer supported
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.

<EMAIL_ADDRESS>install
node-gyp-build

<EMAIL_ADDRESS>install
node-gyp-build

gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp info find Python using Python version 3.11.2 found at "/usr/bin/python3"
(node:322383) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)
gyp ERR! UNCAUGHT EXCEPTION
gyp ERR! stack TypeError: Cannot assign to read only property 'cflags' of object '#'
gyp ERR! stack     at createConfigFile (/opt/iobroker/node_modules/node-gyp/lib/configure.js:117:21)
gyp ERR! stack     at /opt/iobroker/node_modules/node-gyp/lib/configure.js:84:9
gyp ERR! stack     at FSReqCallback.oncomplete (node:fs:193:23)
gyp ERR! System Linux 6.1.0-28-amd64
gyp ERR! command "/usr/bin/node" "/opt/iobroker/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /opt/iobroker/node_modules/@stoprocent/noble
gyp ERR! node -v v21.7.3
gyp ERR! node-gyp -v v7.1.2
gyp ERR! Node-gyp failed to build your package.
gyp ERR! Try to update npm and/or node-gyp and if it does not help file an issue with the package author.
npm ERR! code 7
npm ERR! path /opt/iobroker/node_modules/@stoprocent/noble
npm ERR! command failed
npm ERR! command sh -c node-gyp-build
npm ERR! A complete log of this run can be found in: /home/iobroker/.npm/_logs/2024-12-13T21_35_28_464Z-debug-0.log
npm WARN deprecated<EMAIL_ADDRESS>Rimraf versions prior to v4 are no longer supported
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp info find Python using Python version 3.11.2 found at "/usr/bin/python3"
(node:322383) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)
gyp ERR! UNCAUGHT EXCEPTION
gyp ERR! stack TypeError: Cannot assign to read only property 'cflags' of object '#'
gyp ERR! stack     at createConfigFile (/opt/iobroker/node_modules/node-gyp/lib/configure.js:117:21)
gyp ERR! stack     at /opt/iobroker/node_modules/node-gyp/lib/configure.js:84:9
gyp ERR! stack     at FSReqCallback.oncomplete (node:fs:193:23)
gyp ERR! System Linux 6.1.0-28-amd64
gyp ERR! command "/usr/bin/node" "/opt/iobroker/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /opt/iobroker/node_modules/@stoprocent/noble
gyp ERR! node -v v21.7.3
gyp ERR! node-gyp -v v7.1.2
gyp ERR! Node-gyp failed to build your package.
gyp ERR! Try to update npm and/or node-gyp and if it does not help file an issue with the package author.
npm ERR! code 7
npm ERR! path /opt/iobroker/node_modules/@stoprocent/noble
npm ERR! command failed
npm ERR! command sh -c node-gyp-build
npm ERR! A complete log of this run can be found in: /home/iobroker/.npm/_logs/2024-12-13T21_35_28_464Z-debug-0.log
host.FHEM-SERVER Cannot install<EMAIL_ADDRESS>7
root@FHEM-SERVER:/opt/iobroker/node_modules#
From the LOG:
0 verbose cli /usr/bin/node /usr/bin/npm
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing npm:load:whichnode Completed in 6ms
4 timing config:load:defaults Completed in 8ms
5 timing config:load:file:/usr/lib/node_modules/npm/npmrc Completed in 3ms
6 timing config:load:builtin Completed in 4ms
7 timing config:load:cli Completed in 10ms
8 timing config:load:env Completed in 2ms
9 timing config:load:file:/opt/iobroker/node_modules/iobroker.info/.npmrc Completed in 0ms
10 timing config:load:project Completed in 20ms
11 timing config:load:file:/home/iobroker/.npmrc Completed in 1ms
12 timing config:load:user Completed in 1ms
13 timing config:load:file:/usr/etc/npmrc Completed in 0ms
14 timing config:load:global Completed in 1ms
15 timing config:load:setEnvs Completed in 2ms
16 timing config:load Completed in 50ms
17 timing npm:load:configload Completed in 51ms
18 timing config:load:flatten Completed in 10ms
19 timing npm:load:mkdirpcache Completed in 11ms
20 timing npm:load:mkdirplogs Completed in 3ms
21 verbose title npm
22 verbose argv "--version"
23 timing npm:load:setTitle Completed in 4ms
24 timing npm:load:display Completed in 4ms
25 verbose logfile logs-max:10 dir:/home/iobroker/.npm/_logs/2024-12-13T21_43_02_608Z-
26 verbose logfile /home/iobroker/.npm/_logs/2024-12-13T21_43_02_608Z-debug-0.log
27 timing npm:load:logFile Completed in 21ms
28 timing npm:load:timers Completed in 0ms
29 timing npm:load:configScope Completed in 1ms
30 timing npm:load Completed in 160ms
31 verbose exit 0
32 timing npm Completed in 183ms
33 info ok

phyton upgrade to 3.13 same issue
I also checked the system with the "iob nodejs-update " command, and it looks ok.
But i said "y" to switch to the recommended version 20.18.1 but this also did not fix the issue
Recommended nodejs-version is: 20.18.1
Checking your installation now. Please be patient!
Your current setup is:
/usr/bin/nodejs         v21.7.3
/usr/bin/node           v21.7.3
/usr/bin/npm            10.5.0
/usr/bin/npx            10.5.0
/usr/bin/corepack       0.25.2
We found these nodejs versions available for installation:
nodejs:
Installed: 21.7.3-1nodesource1
Candidate: 21.7.3-1nodesource1
Version table:
*** 21.7.3-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
100 /var/lib/dpkg/status
21.7.2-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.7.1-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.7.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.6.2-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.6.1-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.6.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.5.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.4.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.3.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.2.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.1.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
21.0.0-1nodesource1 600
500 https://deb.nodesource.com/node_21.x nodistro/main amd64 Packages
18.19.0+dfsg-6~deb12u2 500
500 https://ftp.debian.org/debian bookworm/main amd64 Packages
18.19.0+dfsg-6~deb12u1 500
500 https://security.debian.org/debian-security bookworm-security/main amd64 Packages
Nothing to do - Your installation is using the correct paths.
You are running nodejs v21.7.3. Do you want to install recommended version 20.18.1?
Press  to continue or any other key to quit

JIPIII,
after upgrading node-gyp "https://github.com/nodejs/node-gyp/releases" I was albe to upgrade the adapter.

Can be closed!


Update scalameta, semanticdb-scalac to 4.8.0

About this PR
📦 Updates

org.scalameta:scalameta
org.scalameta:semanticdb-scalac

from 4.5.13 to 4.8.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:semver-spec, commit-count:1

Superseded by #1774.


outline: automatically position the outline based on cursor position.

Right now, the outline display seems to be independent of the cursor position in the editor.

It would be great if there was a way to highlight the outline based on the cursor position.
For example:
Here, cursor is inside the foo function, and the foo function in the outline is highlighted:

I then move to the bar function and the bar function is highlighted in the outline:

There could be an option to have this behavior automatic or not, and a command+shortcut that would position the outline at the cursor position in the editor in case the user don't want to have it enabled all the time.
Note: the outline should be expanded to show current position if needed.

Oops, it's already implemented: I guess I should read the release notes more carefully :)


[FEATURE] Create a SOCI Index without a Container Runtime

Description
Today the soci create command can only create a SOCI index from a container image that is stored locally to the soci client in a containerd image store.
https://github.com/awslabs/soci-snapshotter/blob/db7df3ab2a840bc927417a84448263885b3e21ff/cmd/soci/commands/create.go#L63-L74
However the SOCI Index Builder is able to generate a SOCI Index by first downloading and extracting a container image to a local arbitrary directory on the host using the ORAS library. Then it imports the logic from this repository to create the SOCI Index. Showing that its possible to generate a SOCI Index without a container runtime.
For CI/CD use cases it is often challenging to run a container runtime in a build job without adding additional privileges to the runner. So there is appetite to generate Indexes without any dependency on a container runtime, this has been shown by @kichik who has taken the logic out of the SOCI Index Builder into a standalone library https://github.com/CloudSnorkel/standalone-soci-indexer.
This feature request is to be able to build indexes without a container runtime either with the soci CLI or with a standalone binary.
Describe the solution you'd like
No response
Describe any alternative solutions/features you've considered
No response
Any additional context or information about the feature request
No response

Related to https://github.com/awslabs/soci-snapshotter/issues/760


Trashcan icon and Cog Icon

https://d.pr/i/8991Sg
I'm not sure if this is a bug, but why does the "trashcan" icon and "cog" icon alternate when the Padding is on or off?
The trashcan icon behaves as expected (deletes padding) though I'd prefer to just keep it there the whole time. The "cog" icon however doesn't seem to do anything?

This is kind of a duplicate of #14, where this issue is discussed. Maybe better to upvote this.

Yeah, the cog doesn't do anything at the moment.
I'll add it back when I get around to adding the 'Advanced padding options'.
I have also changed the behaviour so that the trash-can appears all the time in the latest v0.3 build. 👍


Errors with "make px4_sitl gazebo" --> recipe for target 'px4_sitl' failed.

Hello!
Here are my stats:
Ubuntu version: 18.04
Gazebo version: 9.13.1
PX release: v1.9.0
ROS melodic
When I try and run:
make clean
make px4_sitl gazebo
I get the following error:
error.log
Please let me know how I can fix this/what I may be doing wrong. Thank you for your support!

Same here !

@MKSwaminathan Could you maybe try the latest master and see if the issue is still there? I don't think PX4 v1.9.0 was ever tested against gazebo version 9.13.1

Hello!
Here are my stats:
Ubuntu version: 18.04
Gazebo version: 9.13.1
PX release: v1.9.0
ROS melodic
When I try and run:
make clean
make px4_sitl gazebo
I get the following error:
error.log
Please let me know how I can fix this/what I may be doing wrong. Thank you for your support!

I try this and work fine for me.
sudo apt-get install libprotobuf-dev libprotoc-dev protobuf-compiler libeigen3-dev libxml2-utils python-rospkg python-jinja2
sudo apt-get install libgstreamer-plugins-base1.0-dev gstreamer1.0-plugins-bad gstreamer1.0-plugins-base gstreamer1.0-plugins-good gstreamer1.0-plugins-ugly -y
Maybe you'll need to install more lib than that.

Thank you both for all your time and help!
@Jaeyoung-Lim I got the same issue building the latest master, but I think this was a machine specific issue (although I'm curious as to why).
@gelardrc I tried installing those and realized that already have those dependencies installed, but here's how I finally fixed the issue:
When I ran a standalone build of the Firmware/Tools/sitl_gazebo, the error log revealed that it was a linker issue, and it was trying to link dynamic and static libraries. Here is an explanation of why this is the case. I globally appended the -fPIC flag
export CXXFLAGS='$CXXFLAGS -fPIC'
and then
./configure --enable-shared
I then rebuilt px4 by making a build dir, running cmake etc. Somehow that fixed everything.


bug？

"react": "16.0.0-beta.5",
"react-native": "0.49.3",

same error
"react": "16.0.0-alpha.12", "react-native": "0.48.3", "react-native-keyboard-input": "^3.0.25",


Feature Request: Can we add features with the ability to chat with AI with an option to view embedding results

This feature request would be helpful if we can extend the functionality to chat with embedding where:

User clicks on LLM response
It expands on the right nav to detail out which embedding results the LLM response was derived from
Embedding responses include both metadatas as well as text. There can be multiple embedding responses.

I’m not sure I follow. Currently there is no use of vector embeddings, or did you mean something else? I think before this, use of vector embeddings will have to come first.
if you meant embedded information, like metadata as you mentioned, could you give more clarity about that

Let's say I were to use app using openai api where it prompts the model to search vector embeddings and find an answer. I would like an option to view those embedding results. Those embedding results will be in a form of pandas dataframe for example.


chore: remove unused deps

What this PR does / why we need it:
Remove unused dependencies from the project.

:tada: This PR is included in version 1.39.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Compatibility with symfony 3.2.x/3.3.x

Hi
can you change the composer.json to raise the symfony compatibility, please?
current:
"symfony/framework-bundle": "^2.0.0 || ^3.0.0",
new:
"symfony/framework-bundle": "^2.0 || ^3.0",
I'm using 3.2.x and the ZurbInkBundle works with it, but I had to change the above row in the composer.json!
Regards
Stefan

Hi,
there is a new release (2.2.5) which supports symfony 3.2.x.

thanks a lot!


Video on screensaver plays for a second and then stops

I experienced the same issue and this happens even in preview mode

@markiv @raghvendra-iiita-work Which versions are you running? Are you running the latest beta?

I got the same issue, I'm using 1.2 Beta 5 (but it was already in 1.2 Beta 1)

Was happening on the version i had installed, updated to the beta 5 and still happens… :( I have the cache disabled but i'm gonna try set it on an external drive and download the files and see if it still freezes…

I'm also having this issue. Tried 1.2 Beta 5, but it is there also.

I "fixed" it by manually caching all videos.

I am using latest stable. I tried using beta version also, faced the same issue.

@raghvendra-iiita-work Did you try caching the videos?

Same issue here.
@alistairmcmillan how ?

Download and install the latest version.
Open System Preferences
Click Desktop & Screensaver
Select Aerial and click "Screen Saver Options..."
Select the Cache pane and tick the "Cache Aerials As They Play" box.
Finally click "Download Now"

@Colir @raghvendra-iiita-work @carlosefonseca @angeloski @123marvin123 @rmarkiv Did you get the screensaver running?

I am experiencing this issue with the latest release and even after caching all the videos. I am on OSX 10.11.3 using latest build.

I believe I'm seeing the same problem with beta5 on 10.11.6 with all files cached.
I have a hot corner set up and when I activate it the screensaver exits most of the time. When it does play it's usually not for very long.

There's better caching option in 1.4 now. Also, you may experience slowdowns because of iCloud Photos activity, please reopen if you still have issues with 1.4


Extended Highlighting Manager

Hi Guys,
I am done with the ThemeableHighlightingManager. I have integrated it here: Dirkster99/Aehnlich and it works pretty good :-)
So, I am wondering if anyone is interested in integrating this into a later version of AvalonEdit itself.
Even if not, should there not at least a reviewed list of known AvalonEdit extensions and sample projects (?) so people have an easier time finding these things(?):

dirkster99/AvalonEditHighlightingThemes
siegfriedpammer/AvalonEditSamples

I hope I find the time to write another CodeProject article about the HighlightingManager extension but apart from that I really think this is the coolest WPF open source text editor there is. Thanks a lot for your work :-)

i've added it to the readme

How about an extensions & separate articles wiki article? We have something along the lines in ILSpy https://github.com/icsharpcode/ILSpy/wiki/Plugins - the Wiki has the added advantage of "editable by everyone"

i look to move it


Non-root namespace declarations not supported on XML writing

These are allowed by badgerfish and generated by our reader, but not handled by our writer

@fugu13 I can take this on now if you can explain what the bug is


scheduler crash

Version: 1.5.9
Reproducible: yes
Config: refer to attachment
Logs: refer to attachment
scheduler-log.txt
snmp-values.yaml.txt

Hey!
Can you provide more details about the condition when it happened?
Did it happen after you added more lines to the inventory? Have you had any working version before on the same environment?
Have you configured anything new when it crashed?
Does it happen every time you reinstall SC4SNMP?

When it happened.
5~10 mins after the sc4snmp started.
Did it happen after you added more lines to the inventory? Have you had any working version before on the same environment?
Yes, and the last working version have less lines in inventory and only 1 worker pod.
Have you configured anything new when it crashed?
Yes, worker pod number changed to 2 (pervious working version with 1 worker)
Does it happen every time you reinstall SC4SNMP?
Yes, it is reproducible.


Update disk-encryption.md

When a key is expired, VMs will not shutdown automatically. This has been tested and checked with ADE SMEs.

@roygara
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@roygara - Correct, this article isn't for ADE. Reason for mentioning ADE is that they take car of both in my team (ADE and SSE with CMK).
I have tested this in my lab as well. When I encrypt the disk with SSE with CMK and when the key expires, VM doesn't shutdown automatically as mentioned in the document. Rather it goes into failed state when we deallocate and start the VM from Portal. Thank you.

@braghavendraknl
This article has been moved into a new repo, azure-compute-docs, so this PR can't be merged.
To resubmit your changes, find the article in the new repository and make the changes there (or copy/paste the raw markdown from your changed file) and open a new PR.
Thanks for contributing!


Python environment types not detected

Testing #17070

This seems to happen with the latest main.

Not able to repro this.

Is it only the windows store one that's not being detected correctly for you?


WIP: HiGHS Callbacks

Reference issue

What does this implement/fix?

adds callbacks that are run every iteration of HiGHS IPX

Additional information

based on mckib2:2022-10-13-highs-update

We discussed that when I tried to update the submodule, git said that it couldn't find the commit. I changed into the scipy/_lib/highs directory, added your fork of HiGHS as a remote, and fetched. I returned to the main SciPy directory and did git submodule update --init successfully. I tried to build and got:
Traceback (most recent call last):
  File "C:\Users\mhaberla\Desktop\scipy\scipy\setup.py", line 33, in <module>
    from numpy.distutils.core import setup
  File "C:\Users\mhaberla\Anaconda3\envs\pythran\lib\site-packages\numpy\__init__.py", line 140, in <module>
    from . import core
  File "C:\Users\mhaberla\Anaconda3\envs\pythran\lib\site-packages\numpy\core\__init__.py", line 100, in <module>
    from . import _add_newdocs_scalars
  File "C:\Users\mhaberla\Anaconda3\envs\pythran\lib\site-packages\numpy\core\_add_newdocs_scalars.py", line 9, in <module>
    import platform
  File "C:\Users\mhaberla\Anaconda3\envs\pythran\lib\platform.py", line 119, in <module>
    import subprocess
  File "C:\Users\mhaberla\Anaconda3\envs\pythran\lib\subprocess.py", line 48, in <module>
    import signal
  File "C:\Users\mhaberla\Desktop\scipy\scipy\signal\__init__.py", line 309, in <module>
    from . import _sigtools, windows
  File "C:\Users\mhaberla\Desktop\scipy\scipy\signal\windows\__init__.py", line 42, in <module>
    from ._windows import *
  File "C:\Users\mhaberla\Desktop\scipy\scipy\signal\windows\_windows.py", line 7, in <module>
    from scipy import linalg, special, fft as sp_fft
  File "C:\Users\mhaberla\Desktop\scipy\scipy\__init__.py", line 65, in <module>
    from numpy import __version__ as __numpy_version__
ImportError: cannot import name '__version__' from partially initialized module 'numpy' (most likely due to a circular import) (C:\Users\mhaberla\Anaconda3\envs\pythran\lib\site-packages\numpy\__init__.py)


[iOS] Fix TextInput maxLength when insert characters at begin

Summary
Fixes #21639 , seems we tried to fix this before, please see related PR like D10392176, #18627, but they don't solve it totally.
Changelog
[iOS] [Fixed] - Fix TextInput maxLength when insert characters at begin
Test Plan
From the git log, we faced some issues, like Backspace event, we need to ensure no regression happened, issues like #21639 #18627 should be fixed.

@rigdern Adam, what do you think?

@shergin I'm not familiar with this code. I read over the PR but it's unclear to me what flaw in the code is causing the symptom and why this delta fixes it.

@shergin @rigdern Thanks for review, my opinion is _predictedText should always equal to the combination of backedTextInputView's original text and replaced text. The original if..else.. check may leads to inconformity between _predictedText and backedTextInputView's text.

@shergin @cpojer Hi, any opinion?

@zhongwuzw Can you share that the issue is you're seeing and how to repro it? Like @rigdern said, it's not clear to me what the issue is and how this fixes it - if you're seeing an inconsistency here, then it seems like predictedText already got out of sync with the backing textinput view. I think this change may make sense regardless, but I just want to take a look and see where things are going wrong.

@ejanzer Hi, I borrowed the repro from #21639 :

Description
[iOS only]
When entering 1 character at the beginning of the TextInput, we expect to have only 1 character inserted. However we experience that the character is inserted and the current value is duplicated.
This behaviour is reproducible when the prop maxLength is used and when length of the current value is above the half of the maxLength prop.
I have created an app from scratch using create-react-native-app, and I have modified App.js to include the following:
type Props = {};
export default class App extends Component<Props> {
  constructor(props) {
    super(props);
    this.state = { text: '10000' };
  }

  render() {
    return (
      <View style={styles.container}>
        <TextInput
          style={{height: 40, width:120, borderColor: 'gray', borderWidth: 1}}
          maxLength={10}
          onChangeText={(text) => this.setState({text})}
          value={this.state.text}/>
      </View>
    );
  }
}

Reproducible Demo

Based on repro example, when we insert 0 at the begin, the _predictedText should equal to 0 10000, but the old code, it equal to 0, leads to the issue.

Oh I see, the problem is if you set an initial value from JS then _predictedText never gets properly initialized, so it's nil when you get here. We could also fix this by explicitly checking if it's nil, but I think this change makes sense - from the code it looks like _predictedText is supposed to be used to detect when the backing textinput updated without textInputShouldChangeTextInRange getting called.
@zhongwuzw would you mind also cleaning up the other places in this file where we set _predictedText, since they're no longer needed?

@ejanzer Hey, I clean up the code, please continue to review it.


Migrating Cypress test from old version to latest version throws error Expected to find a global registry

Current behavior
I am trying to migrate my existing Cypress BDD tests from 8.7.0 version to latest version 10.10.0.  I have installed the required libraries and amended the settings. I was having an old node version 12 + version, now updated to node v18.4.0 and tried again. Couldn't find any pointers to resolve similar issue, so any advise would be really helpful.
Following is my automation folder structure:

** feature file location**

tests/cypress/e2e/ login/login.feature

** Step definition location**
Imported in the step definition file
 import { Given, When, Then, Before, After} from "@badeball/cypress-cucumber-preprocessor";

tests/cypress/stepDefinitions/login.cy.js

Added e2e.js file under tests/cypress/support folder:
// Import commands.js using ES2015 syntax:
import './commands'
Cypress.on('uncaught:exception', (err, runnable) => {
    // returning false here prevents Cypress from
    // failing the test
    return false
})

Added .cypress-cucumber-preprocessorrc.jsonfile in to the test root folder

{
  "stepDefinitions": [
    "[filepath].{js,ts}",
    "tests/cypress/stepDefinitions/**/*.{js,ts}",
    "tests/cypress/e2e/[filepath]/**/*.{js,ts}",
    "tests/cypress/e2e/[filepath].{js,ts}",
    "tests/cypress/support/step_definitions/**/*.{js,ts}"
  ]
}

Below is my cypress.config.js file:
const { defineConfig } = require("cypress");
const webpackPreprocessor = require('@cypress/webpack-preprocessor');
const addCucumberPreprocessorPlugin = require("@badeball/cypress-cucumber-preprocessor");
const dotenvPlugin = require("cypress-dotenv");

async function setupNodeEvents(on, config) {
    await addCucumberPreprocessorPlugin.addCucumberPreprocessorPlugin(on, config);

    const options = {
        webpackOptions: {
            module: {
                rules: [
                    {
                        test: /\.feature$/,
                        use: [
                            {
                                loader: '@badeball/cypress-cucumber-preprocessor/webpack',
                                options: config,
                            },
                        ],
                    },
                ],
            },
        },

    };

    on('file:preprocessor', webpackPreprocessor(options));

    return config;
}

module.exports = {
    default: defineConfig({
        e2e: {
            baseUrl: "https://book.test.co",
            specPattern: "tests/cypress/e2e/**/*.feature",
            supportFile: false,
            setupNodeEvents,
        },
    }),
    setupNodeEvents,
};

Error Details

Desired behavior
Hope to run the BDD feature file without errors

Test code to reproduce
Removed following items from the automation framework

**plugin/index.js ** file
**cypress.json ** file
uninstalled old cypress-cucumber-preprocessor from the project

Versions

Cypress version:  Cypress 10.10.0
Preprocessor version: @badeball/cypress-cucumber-preprocessor 11.4.0
Node version: v18.4.0
cypress/webpack-preprocessor: 5.14.0
OS: Ubuntu 20.04.3 LTS

Checklist

[x] I've read the FAQ.
[x] I've read Instructions for logging issues.
[x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

You need to provide me with a complete, but minimal and runnable example.

I'm having the same issue with the 'Expected to find a global registry' error. I created a minimal setup with cypress and cucumber. Maybe this might help: https://github.com/DenFin/minimal-cypress-cucumber-setup

@DenFin, in your case you can remove js from specPattern. You're now trying to run step definitions as if they're the test (they're not).

@badeball  In my case , I didn't have js in my specPattern. This is my settings: specPattern: "tests/cypress/e2e/**/*.feature",

I'm having the same issue with the 'Expected to find a global registry' error. I created a minimal setup with cypress and cucumber. Maybe this might help: https://github.com/DenFin/minimal-cypress-cucumber-setup
@DenFin  Have you found out any idea or work around on how to handle this issue ?  I am kind of stuck at the moment.

@scrumvisualize actually removing js from the specPattern solved the problem for me

@scrumvisualize have you found any solution?

Closing due to lack of reproducible example. Feel free to open up another issue if anything changes in this regard.


Possible improvement by not using Socket.gethostname

In certain circumstances, "event["message"] = Socket.gethostname" may cost huge delay and render the plugin unusable. I think it might caused by hostname resolving or reverse-resolving. In my logstash use case, I changed it into
event["message"] = event["host"] || @hostname || Socket.gethostname
@hostname is an arg I added myself.

+1.  Just lost a ton of time to this, and there's no visibility to easily find the cause.  If you hit a lookup failure, why not log it so we'd have some clue?
In our case, logstash had no available DNS, so adding an entry to /etc/hosts worked:
<IP_ADDRESS>  myhost.mydomain.com


Error because naming convention of a script

In our organisation we have a naming policy. This policy is blocking the deployment of Cloud Katana. It gets stuck at createClientApp and createServerApp. The error looks like this: "Resource '[randomlettersandnumbersscripts]' was disallowed by policy. Reasons: 'The naming convention for the resource you are deploying does not meet the requirements. Please look it op in the Cloud-Foundation Wiki or contact the Cloud Foundation Team.'. See error details for policy resource IDs." Where can I find this script so I can change the name?

I get this error from the deployment scripts for createclientapp and createserverapp


[WEB] Feature: Open Candidate Detail

Summary
This PR is intended to add search-candidate functionality to rojak-ui-web.

@pyk please add some labels for this

cc @adrianha @y68 @CodeRiderz

Not sure about this one, buddy.
When you open /candidates page and you click one of the candidate, it should bring you to <DefaultResult /> instead of creating new page result called <OpenKandidat  />.
The <ResultGateway> intended to be a wrapper to separate views between special keywords (pairings, medias, candidates ) and specific keyword like anies, ahok, ahok djarot, kompas or even an url for news. It's there, because we would have a different looks / UI for those special pages and the default one.
We should create another smart container in <DefaultResult />to manage whether a keyword is candidate, media, or news.
For example, if user type ahok in the search field, it will bring you to /search/ahok.
Then, this smart container inside <DefaultResult /> should give you a View for a candidate.
It's the same for Link(s) listed in /candidates page, it should bring you to /search/ahok once you click it.
This way, we don't need to create another view / wrapper if user type agus in the search field.
What do you think ?

any update?

@CodeRiderz

The <ResultGateway> intended to be a wrapper to separate views between special keywords (pairings, medias, candidates )

Unfortunately, this /candidate/:id thingy is now another search terms with special keywords:

Okay, I will accept this for now. Lets see if this will works with another features.
LGTM

Cool thanks

@rromadhoni ini udah aku bild juga di https://rojak.id ya 🙇


Dep offer tx

Why this should be merged
This PR extends existing types with upgradeVersion (see https://github.com/chain4travel/caminogo/pull/262):

Extends deposit offers with new fields.
Extends DepositTx with new fields in order to support new offers.

It also adds:

New transaction AddDepositOfferTx, which allows to add new deposit offers. This transaction must be signed by an address that has address state "offer creator".
Two new address state roles: offerAdmin (can assign offerCreator) and offerCreator (can create deposit offers).
New test cases for affected code, updated existing tests.

One of the new offer features is "owned offers". If a deposit offer "owner" field isn't empty, then in order to use that offer, DepositTx will need to provide additional credentials and specify "deposit creator" fields.
How this works
Upgraded types
Extended type methods will check either the value of UpgradeVersionID field or some of the new fields for a non-zero value. If conditions are met, new logic is executed.
Owned offers
"Owned offer" owner is defined by an address (could be multisig alias as well). This offer can be used for depositing only by addresses that will be "whitelisted by offer owner. For each whitelisted address, owner must construct permission message (offerID+allowedAddress), sign it with its key (or keys) and send this signature(s) to whitelisted address. This signature then must be used by deposit creator in DepositTx.
How this was tested
Unit-tests.

@peak3d Seems that you somehow reviewed older version, very old one :)


Some small Ideas to discuss

I have been thinking about some things I noticed (this is not an "issue" but I don't know of any other place for this):
Elevation maps generated by platec usually contain values between 0 and 10; although some maps go up to even 14.

Would it be appropriate/feasible/wanted to scale that range up?

Advantages I see:
Imagine the elevation map being multiplied by 2000: The range between highest and lowest points of a world would be at roughly 20000 - if you translate that to meters it is pretty much as on earth, with the occasional world straying a bit further from that.
Thinking back to the strange rivers I was asking for some sense of scale to decide how much a "drop" of water can achieve - using our real physical units as a metric would be a great start for that. Instead of having to combine two quite alien scales, we would have a frame of reference to help make certain decisions.
There is a function that sets elevation thresholds based on a fixed percentage of land that should be declared e.g. a 'mounain'.

In conjunction with 1., would it be desirable to remove the fixed percentage and instead use a fixed "height"?

Advantages:
Not every world has exactly three percent mountains, but instead it depends on the seed if there are mountains and hills.
I thought it would be great if worldengine could generate higher-detail heightmaps (a pull request will show up at some point). I saw (in the README maybe?) that some other projects respectively games made use of worldengines maps already. I myself initially found worldengine looking for a program to create nice heightmaps to be used in a toy-project I was working on in Unreal Engine 4.
So I dug into draw.py and the GDAL-export() and noticed that the pillow library seems to be exclusively used to write out the finished PNGs of a world - nowhere else. And I found no way to export 16 Bit grayscale images with it. (GDAL can do it, but it is a pain to install!)

Should pillow maybe replaced by a different image-writing library?

Advantages:
After taking a look at several libraries I found some that can handle many if not all PNG-standards and thus seem more useful than Pillow (at least when it comes to writing PNGs) - the one I tried out is this: PyPNG (https://pythonhosted.org/pypng/ - seemingly available for Python 2.3 and above)
With just a small bit of work I made worldengine export proper 16 Bit heightmaps and even took a look at one in Unreal Engine 4. :) Seeing the world go 3D is nice - maybe it will prove useful to others!
I think that all of these are technically easy to realize, 1. would probably be the most complicated (depending on how much code depends on the scale of the elevation map). The benefits are there, but since those still are significant changes, I wanted to ask what you guys think about them. (The PNG-stuff will be visible in my next pull request to get an idea of it, but it could be removed).

I think that we don't really need to change the scale. As you said we can just multiply the scale by 2000 to get a close estimate as to the real world scale (or alternately multiply by .0005 if you need to convert from real world to platec). Since I suspect the results will often be renormalized if they are being imported into anything where the height is really important (such as an Unreal Engine program) the value would get altered a bit anyway.
I didn't realize that mountains were being handled purely by elevation. That should probably be addressed as a separate issue. Mountains and hills are a lot more than just elevation. They are a change in elevation over a distance. The Tibetan Plateua has an average elevation in the range of 4500 m but it is not what we should classify as mountain.
As for changing to PyPNG, assuming it doesn't cause any problems with any of the other routines I would love to have it replace pillow so that we can get a 16 bit grayscale image. If you could also include an Unreal Engine utility to view a world that would be fantastic, IMO.

Rescaling is not a big problem, that is true. I just thought about it because when I looked at hydrology.py I had no idea of the relation of the numbers with respect to each other (amount of water versus volume it can fill in the world). Had it just been meters and the rain e.g. mm/m² it would be really easy to check if the hard-coded values make sense.
I will add a comment that mentions the factor of 2000, at least.
I am not sure if there is more to mountains, but the very first step is related only to elevation. And not even total elevation - every world has 3% mountains at that point. You seem to have an idea of how to handle it a little better, so if it currently is really that simplified, you should take a look at it. (generation.py, ~line 118 ;P )
As far as I know, Pillow is only imported in draw.py (clean job!) and only accessed in a single class. For my test PyPNG worked great, I already started thinking about using it for more than grayscale-images.
By the way: It was only when I looked at a 16 Bit grayscale image that I realized that you cannot notice any difference with the naked eye since monitors (and most programs) cannot differentiate more than 256 shades of gray. :P (But for exporting data without having to worry about GDAL it is great.)

Yeah. You won't notice a difference when you are simply looking at 8 bit grayscale over 16 bit grayscale but if you are feeding the results into something like Wilbur or Fractal Terrain you can definitely see a change in the final output. I would also guess that your Unreal Engine project would show you a difference.

:D Of course it is a huge difference, I think I understand the math behind it.^^

Scaling isn't a problem with 8-bit, it is actually best when visualizing in an image. However once you get to 16 or 32 bit.... things go wacky. The difference between points is so extreme that trying to visualize them in 3D leaves you with a spiky world.
I've done this with OpenMW (another project I'm apart of).

The choice of using a fixed percentage for mountains is not ideal, I agree with that. However it is pretty robust: it guarantees that the world would have a reasonable distribution of the different kinds of elevation (plain, hill, mountain) and it could be used for normalization later (if in my application the mountain elevation should be 5 or 5000 for some reason I can easily rescale worlds generated by WorldEngine). I think that using absolute values to determine what is a mountain and what is not would be more correct from a theoretical point of view but it could be not so easy to achieve: we have to think about values which make sense for very different maps (for size, for number of plates, for collisions which happen during the plate tectonics phase and so on). So it could be hard to pick the right thresholds and to get decent results under different circumstances.

Well, right now it is a fixed number. If somebody were to write a small module and give it a life test, that might still be worth a try.
I wouldn't mind finding a world without mountains, or one with a lot of them. That is to be expected when creating whole worlds. :)

This all does remind me of something I've been meaning to bring up. There should probably be ways to regenerate most PNGs based off the world file.
As an example we are discussing various ways to identify areas of mountain. To the best of my knowledge there's nothing in the world file that says 'area X is mountain'. There's certain specific data such as temperature range (which is an open number not corresponding to any real world figure), humidity (likewise and open number) and elevation. If someone wants to generate a map showing mountain ranges (such as ancient_map) or ocean they could run worldengine with the altered parameters and the -w option to load an existing world file instead of completely regenerating the world.
This may cause certain issues as I believe some things like erosion look at whether an area is mountains or not so their processes will end up reflected in the final output, even if an area is later removed from being a mountain (or areas are later upgraded to mountains). However I think that it will still have value as it will allow 'fine tuning' where these behaviors have very little impact (I go from 3% mountains to 2.5% mountains then the number of misidentified areas are going to be very minimal) as well as allowing people to tweak around values until they get a map that is close to what they want (lets see what this will look like with 15% mountains) and then do a 'final render' with the new parameters, once they feel they are dialed in.
In a similar vein it might be worthwhile to see if platec (and other routines) can be tweaked so that results are more uniform regardless of size. Right now one map with a 512x512 resolution and a second map with 1024x1024 will produce very different images, I believe, even if they use the same seed. If the noise patterns were made a bit more agnostic in regard to size (and other routines were to divide the x/y distances by something such as x_width/512) this would probably allow for the generation of things such as 512x256 maps until an interesting map is spotted at which point a 4096x2048 version could be generated.

I agree with you. Indeed the world file contains all the information generated for the world so it is possible to load it again. We just need to implement the operation to change it.

That last part sounds very intriguing. Platec generates completely different worlds when certain parameters change, a bit more predictability would be appreciated. It is probably hard to realize, though.
Regarding all those parameters: Maybe it would be a first step to place all constant numbers into a single file and access them as global variables (and in return make them easily editable to the user, all in one place and including a detailed description). I don't know if that is an idea that agrees with your coding-standards, though. But it could lead to generating the same world with e.g. different amounts of mountains. Which is not exactly what you suggested, but it is very similar and very easy to implement in comparison.

Yeah, placing all the constant values in a file would probably not be a bad idea. Then at the initialization of the program the values are all read and constants are set to either the value in the file or an 'override' if provided (perhaps with an additional option to read override values from a separate file since there will quite likely be a lot of them).
This way people will be able to generate less terrestrial worlds by altering ocean levels, river and mountain variables, and the variables controlling the distribution of biomes.

Nobody would try to alter the ocean_level-variable anyway.
https://github.com/Mindwerks/worldengine/issues/114
https://github.com/Mindwerks/worldengine/pull/134
;P (Yes, I still plan to fix this. It could probably be decently done by removing a single line of code from one of the commits, but I hope for a better solution to randomly appear. Soon.)

Maybe this is not the best place to discuss it, but I'll try since seemingly everybody participates here anyway:
Going back to the global variables mentioned above, @esampson has kind of opened a seam for that now: https://github.com/Mindwerks/worldengine/pull/162
Maybe this is a good opportunity to create a central file that stores such parameters (including elaborate explanation) so they can be changed all in one place. We might want to provide more than one default set, too, so a user can choose between "standard world" and "desert world" or something similar. Commandline overrides will still take precedent, of course.
Now my question is: What format should such a file have (.py, .ini, .json or something like that) and where should it be placed (suggestion: settings.ini right in worldengines root folder^^). What happens to the settings after worldengine has been properly installed - if it was a compiled Python file that might not work out properly. Maybe the file itself can be passed as a parameter? worldengine --settings=settings.py --seed=5000 world
I would hold off seriously working on this until the recent PRs are processed. This would have to touch a lot of files, I assume, so time might be best spent on discussing the concept for now.


ImportError after install

I'm running Python 3.10.0 on a Windows 11 machine. When I try and run the module, I get the following error -
> py -m pypykatz
Traceback (most recent call last):
  File "C:\Users\data expunged\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\data expunged\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\Users\data expunged\AppData\Local\Programs\Python\Python310\lib\site-packages\pypykatz\__main__.py", line 156, in <module>
    main()
  File "C:\Users\data expunged\AppData\Local\Programs\Python\Python310\lib\site-packages\pypykatz\__main__.py", line 17, in main
    from pypykatz.kerberos.cmdhelper import KerberosCMDHelper
  File "C:\Users\data expunged\AppData\Local\Programs\Python\Python310\lib\site-packages\pypykatz\kerberos\cmdhelper.py", line 19, in <module>
    from pypykatz.kerberos.kerberos import get_TGS, get_TGT, generate_targets, \
  File "C:\Users\data expunged\AppData\Local\Programs\Python\Python310\lib\site-packages\pypykatz\kerberos\kerberos.py", line 7, in <module>
    from minikerberos.security import KerberosUserEnum, APREPRoast, Kerberoast
ImportError: cannot import name 'KerberosUserEnum' from 'minikerberos.security' (C:\Users\data expunged\AppData\Local\Programs\Python\Python310\lib\site-packages\minikerberos-0.4.0-py3.10.egg\minikerberos\security.py)

bad import name? cuz I also couldn't find "KerberosUserEnum" anywhere in the minikerberos repo.
If absolutely necessary I wouldn't mind switching to my Windows 10 machine or upgrading Python but I'd rather not if possible.

The issue was caused that I published the new version of minikerberos package but not the new version of pypykatz.
It is fixed now

https://github.com/skelsec/pypykatz/issues/117


[BUG]Use "TAB" to indent invalid

"B" Use "TAB" to indent invalid

The second little question

After using "shift+enter", "A1" should be located below "A", no need to press "TAB"

@lll637637 Thank you for your issue! Your screenshots are great!
This issue is duplicate of #38 so I'm closing it. Please follow #38


Fix and re-enable tests  in policyengine

https://github.com/amyxzhang/policykit/tree/master/policykit/policyengine/tests these tests don't get run with python manage.py test because there is no __init__.py file in the tests directory. The tests don't work, since they don't run on with GitHub actions. This issue is for fixing the tests, and adding the init file so they run with each PR.

lets just delete these


Allow blue signatures

Administration might require signatures to be blue.

I talked to our secretary, they will hold a meeting in the next week and discuss this issue with accounting.

time sheets with digital signatures are printed and signed by a chair representative.


Support Python 3.9

Codecov Report
Attention: Patch coverage is 79.13043% with 24 lines in your changes missing coverage. Please review.

Project coverage is 81.66%. Comparing base (69d8fcd) to head (a7bdb24).

Files with missing lines
Patch %
Lines

tests/stack/async/refs/dunder.py
0.00%
9 Missing :warning:

tests/stack/async/refs/nested.py
0.00%
9 Missing :warning:

tests/stack/sync/refs/dunder.py
81.81%
1 Missing and 1 partial :warning:

tests/stack/sync/refs/nested.py
81.81%
1 Missing and 1 partial :warning:

tests/stack/async/runner.py
91.66%
0 Missing and 1 partial :warning:

tests/stack/sync/runner.py
91.66%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #57      +/-   ##
==========================================
+ Coverage   81.33%   81.66%   +0.32%
==========================================
  Files          39       39
  Lines        1404     1407       +3
  Branches      264      187      -77
==========================================
+ Hits         1142     1149       +7
+ Misses        234      223      -11
- Partials       28       35       +7

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

alright, something this I was just recently asked via issue:

https://github.com/simonsobs/apluggy/issues/56

Imho not nice to commit dead code (commented out code) to project.
What I would do myself, is create immediately pr that drops 3.9 support and let it be in Draft state until the time comes.


Switching the group hangs when extension is in the middle of loading

Repro steps:

Expand a tree item that requires a lengthy loading time

Switch the grouping (in this case I chose "location")

Expected: Selecting a new grouping halts the loading and immediately switches to the new view.
Actual: The async call is fully awaited and the view switches after it completes (in this case it was 10 or 15sec).

We might consider having a cancellation token for loading children so that certain commands can cancel this async call.


Investigate potential race conditions in KafkaSupervisor

In KafkaSupervisor  , the taskGroups is accessed by multiple threads and modified by main exec thread. Operations are performed on the taskGroups value object without null checks for eg taskGroups.get(groupId).partitionOffsets. Potential places causing race condition could be in KafkaSupervisor.checkpointTaskGroup() or KafkaSupervisor.verifyAndMergeCheckpoints().
Related issue #5900

I would add some more details. Here is a snippet of KafkaSupervisor.checkpointTaskGroup().
  private ListenableFuture<Map<Integer, Long>> checkpointTaskGroup(final int groupId, final boolean finalize)
  {
    final TaskGroup taskGroup = taskGroups.get(groupId);
...
    return Futures.transform(
        Futures.successfulAsList(pauseFutures), new Function<List<Map<Integer, Long>>, Map<Integer, Long>>()
        {
          @Nullable
          @Override
          public Map<Integer, Long> apply(List<Map<Integer, Long>> input)
          {
            // 3) Build a map of the highest offset read by any task in the group for each partition
            final Map<Integer, Long> endOffsets = new HashMap<>();
            for (int i = 0; i < input.size(); i++) {
              Map<Integer, Long> result = input.get(i);

              if (result == null || result.isEmpty()) { // kill tasks that didn't return a value
                String taskId = pauseTaskIds.get(i);
                log.warn("Task [%s] failed to respond to [pause] in a timely manner, killing task", taskId);
                killTask(taskId);
                taskGroup.tasks.remove(taskId);

              } else { // otherwise build a map of the highest offsets seen
...

So, taskGroup is gotten at the first line, and is being used in the future executed by workerExec. The potential race condition is, taskGroup might not be in taskGroups when the future is executed. This matters because actually taskGroups has some state context as well. (taskGroup instances are moved from taskGroups to pendingCompletionTaskGroups if they finish reading). Not sure this is intentional or not.


can I split off on_message function

hey, is it possible to split off some of the part of the logic in my on_message() so that I can have a common base for all my scrapers...
aka a part something like this
elif type(msg) is str:
    if msg == "start":
        self.logger.info("cmd: START")
        self.on_start()
    if msg == "remove"
        if msg == "stop":
            self.logger.info("cmd: STOP")
            self.logger.on_start(cancel=True)

into a separate file.
Sorry I think maybe something like this is possible, I'm just not sure what the import parts and stuff and whether it needs to be a class and how it interacts with pyspider

Yes, you can override this method in a subclass of BaseHandler, and import the subclass into each project, inherit each handler from the subclass.


installer: add case for ash shell on alpine linux

Should resolve #302
Since alpine might not come with an .ashrc or .profile by default, depending on the distro, it would still be up to the user to touch ~/.ashrc && touch ~/.profile if running this in a fresh CI environment.

Will re-open without the messy git history...


InteropService Clone() - list index out of range

Observed while syncing test net

[I 171120 13:40:57 EventHub:99] [SmartContract.Execution.Success][646058] [8947279d24cfcac050e14c84d748ec78aeea6dd4] [tx a0f11cf23d1c828d782beae8103b6089e036aacce776a55ccfb544994f1925ea] [b'\x00']
[E 171120 13:40:57 ExecutionEngine:800] COULD NOT EXECUTE OP: list assignment index out of range b'\xc4' SETITEM
[E 171120 13:40:57 ExecutionEngine:801] list assignment index out of range
Traceback (most recent call last):
File "/Users/erik/Documents/code/neo-python/neo/VM/ExecutionEngine.py", line 798, in StepInto
self.ExecuteOp(op, self.CurrentContext)
File "/Users/erik/Documents/code/neo-python/neo/VM/ExecutionEngine.py", line 710, in ExecuteOp
newItem = newItem.Clone()
File "/Users/erik/Documents/code/neo-python/neo/VM/InteropService.py", line 265, in Clone
newArray[i] = None
IndexError: list assignment index out of range
[I 171120 13:40:57 EventHub:99] [SmartContract.Execution.Fail][646061] [8947279d24cfcac050e14c84d748ec78aeea6dd4] [tx 18378c04113089f56ce36ac474a2a6da9da249db628e37ba4c23bec370834a5c] [[[<neo.Core.TX.InvocationTransaction.InvocationTransaction object at 0x107011978>, <neo.Core.TX.Transaction.TransactionOutput object at 0x10590f630>, b'/<\xaa!\xd8(\x94\x02\xf2\x96\tT\x9a2\xed\xa0aZ7\xb7', [<neo.Core.TX.Transaction.TransactionOutput object at 0x117dfa550>, <neo.Core.TX.Transaction.TransactionOutput object at 0x117dfaeb8>, <neo.Core.TX.Transaction.TransactionOutput object at 0x120613160>], b"\xd4m\xea\xaex\xecH\xd7\x84L\xe1P\xc0\xca\xcf$\x9d'G\x89",<PHONE_NUMBER>0, [<neo.Core.TX.Transaction.TransactionOutput object at 0x117dfa550>, <neo.Core.TX.Transaction.TransactionOutput object at 0x117dfaeb8>, <neo.Core.TX.Transaction.TransactionOutput object at 0x120613160>], 3, <neo.Core.TX.Transaction.TransactionOutput object at 0x120613160>, False, None, None], 10], None, 6]

should be fixed now in 7172b342c0349d9fd07673182dd9fd591af8f46a


Remove cookie_factory

Note that the code is almost exactly 1:1 with what we were doing before, just manually - not through cookie_factory
That means the code is ugly and inefficient and spaghetti.
It was before, too, you just couldn't tell as easily :D
I intentionally am not cleaning the code up as part of this PR, to make the diff easier to check if any bugs crept in.

LGTM (well except the byteorder dep, but thats a discussion for another time)

Benchmark results, because I got curious:
cargo build (debug)

Code
Mean [s]
Min [s]
Max [s]
Relative

Master
35.360 ± 0.390
35.048
35.797
1.72 ± 0.02

This PR
20.587 ± 0.087
20.505
20.678
1.00

cargo build --release

Code
Mean [s]
Min [s]
Max [s]
Relative

Master
90.870 ± 0.407
90.456
91.270
3.80 ± 0.02

This PR
23.932 ± 0.010
23.921
23.941
1.00

LGTM (well except the byteorder dep, but thats a discussion for another time)

It's what we used in ruffle and I figured easy enough to switch; but it looks like we're exclusively BE, so switching away from this should be trivial in a followup if we wish


multiple stackoverflow errors caused by toml_parse.rl

As of 03e8a3ab1d4d014e63a2befe8a48e74783a81521, there are several inputs that cause stackoverflow error, which occur in at least 2 places in toml_parse.rl: line 544 and line 627.


Returned non-zero exit status 254

Thanks for sharing this amazing tool! I am running like this:
kdb --db-type standard --threads 30 --cache-dir .

And I continue to get an error like this:
subprocess.CalledProcessError: Command 'ncbi-genome-download --section refseq \
--format fasta --assembly-level complete --retries 3 --parallel 30 --progress-bar plasmid' \
returned non-zero exit status 254.

In line 23 of kdb.py I tried moving plasmid to the end of the list, so human is next. But I get the same error. Any thoughts. And is the a better place to post this question? Thanks!!

@jarrodscott Thanks for creating the issue. I will take a look and let you know shortly.

Plasmid fails because it isn't one of the "SUPPORTED_TAXONOMIC_GROUPS" listed in the config.py file of the ncbi-genome-downloader tool, which kdb uses to retrieve data from NCBI.  Neither are "human" or UniVec_Core", so these will fail too if you're running kdb to create a "standard" kraken2 database.
One solution is to edit the installed config.py file by adding 'plasmid' and 'human' to the "SUPPORTED_TAXONOMIC_GROUPS" list.
UniVec_Core is a bit trickier due to the way those genomes are identified (see kraken2's "download_genomic_library.sh" script for details). However, getting the UniVec_Core library is quick with kraken2's "kraken2-build --download-library" command as there are only ~3K genomes in it.


Update features-windows.md

In a VM, if there an existing extension with failed provisioning state, any other new extension fails to install.

@pranathipanyam : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off Thank you @pranathipanyam!

@cynthn: I'm sorry - only the author of this article, @amjads1, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

I'm aligned on the sign-off received from @cynthn. Thanks!


CHIDO

CHIDO created by<EMAIL_ADDRESS>on 2019-05-03 16:45:37

<EMAIL_ADDRESS>replied, CHIDO means â€˜a well for waterâ€™.  This is in sentences written by Ulang and confirmed by Charl.

<EMAIL_ADDRESS>replied, from the Japanese word "ido"

<EMAIL_ADDRESS>replied, Alii..
Sorry, this is not related to "chido"
Jim, I'm wondering if you have the lyrics for Nanyo Sakura...

<EMAIL_ADDRESS>replied, Yes, I do:Â  this is from a few years ago now:Â Â https://ouchacha.wordpress.com/2017/03/16/nanyo-sakura/

<EMAIL_ADDRESS>replied, Thank you, Jim!


[libusb] Add disable-udev feature

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[ ] Only one version is added to each modified port's versions file.

Adds new feature for libusb port to allow building with --disable-udev flag. Default behavior still remains enabling udev (explicit --enable-udev flag is added in this case for clarity)

Please run:
vcpkg x-add-version libusb --overwrite-version
git add .
git commit -m "Update version database"

to add the new version.


Add in hidden_reward for safety/control/ethics research

Lab2D and Melting Pot should support outputting a hidden_reward as part of the step method to enable safety/control/ethics research.  The hidden_reward output will be basically the same as the hidden_reward supported in https://github.com/deepmind/ai-safety-gridworlds.
If there is a better way to do hidden_rewards than the approach implemented in the ai-saftey-gridworlds, please add suggestions below to be discussed.
Next step:  I think this means changes are needed to Lab2D, Melting Pot and also dm_env and will look into what specific changes are needed.

Hi, the reward returned by step is only used for simplified single player RL environments and for any other types of data I would suggest using additional observations. This will prevent the need for changing Lab2D and dm_env. You can see this being done with the the multiplayer example environment clean_up here: https://github.com/deepmind/lab2d/blob/f634c378d428dd19e8b154aa5b590d33f42438bf/dmlab2d/lib/game_scripts/levels/clean_up/avatar.lua#L182.

@charlesbeattie ,  👍  Thanks for the reference.  This looks good and I'll give it a try.   Will close this issue for now.


Link to the category slugs page from the docs instead

Oops, just noticed that I didn't change this link in the docs when we changed to specifying slugs for categories in Cargo.toml :-/

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit f76044c has been approved by alexcrichton

:hourglass: Testing commit f76044c with merge 2324c2b...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 2324c2bbaf7fc6ea9cbdd77c034ef1af769cb617 to master...


[cherry-pick] Update rocksdb: fix a segfault

Fix a potential segfault when we have a very large number of ranges on stack in https://github.com/pingcap/rust-rocksdb/commit/b127863a82fcf7784d5697261207df827f6c74a0.

/run-integration-tests tidb=release-1.1 pd=release-1.1 tidb-test=release-1.1

LGTM

/run-integration-tests tidb=release-1.1 pd=release-1.1 tidb-test=release-1.1

/run-all-tests

LGTM

/run-all-tests

Close this since we are going to merge master to release-1.1 soon.


dracut/30ignition: support coreos.no_persist_ip

When coreos.no_persist_ip is specified on the kcmdline delete
/etc/sysconfig/network-scripts that was created by the dracut
write-ifcfg.sh script before running Ignition. This allows the use
case of configuring only the initial initramfs networking via ip= to
allow fetching of the Ignition config, and specifying a more complex
real root configuration in the Ignition config.

So this was only merged into the spec2x branch...I guess because FCOS (spec3) doesn't have ifcfg files.  But it's going to require us to...hmm, add a new translator to NM config files or something.
Or I guess maybe we could just merge this in master and ignore that it doesn't do anything for FCOS yet.

Side note: The way I've been testing out changes to this is to cosa run and edit the kernel cmdline to say ip=on.  This tricks the code that skips over ip=dhcp.


[BUG]: Helm deployment of vxflexos driver is failing

Bug Description
Helm install of 2.10 powerflex driver is failing with below error. Problem is not documented in troubleshooting documentation. Workaround was to update the mountPath value.
Install csi-powerflex driver at <IP_ADDRESS> using standalone helm chart
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
W0321 17:55:40.254700 1018944 warnings.go:70] unknown field "spec.template.spec.affinity.nodeSelector"
Error: INSTALLATION FAILED: 1 error occurred:
DaemonSet.apps "vxflexos-node" is invalid: spec.template.spec.initContainers[0].volumeMounts[4].mountPath: Invalid value: "/host_opt_emc_path": must be unique
Logs
Error: INSTALLATION FAILED: 1 error occurred: DaemonSet.apps "vxflexos-node" is invalid: spec.template.spec.initContainers[0].volumeMounts[4].mountPath: Invalid value: "/host_opt_emc_path": must be unique
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
Use the CSM 1.10.0 helm chart to deploy CSM for PowerFlex
Expected Behavior
CSM for PowerFlex deploys successfully
CSM Driver(s)
PowerFlex v2.10.0
Installation Type
Helm
Container Storage Modules Enabled
No response
Container Orchestrator
Kubernetes
Operating System
N/A

link: 22080


FormData.fromMap is encoding request body automatically

Package
dio
Version
5.0.2
Output of flutter doctor -v
No response
Dart Version
2.19.2
Steps to Reproduce
Future<FormData> mapToFormData(
  Map<String, dynamic> formData,
) async {
  try {
    return FormData.fromMap(formData);
  } catch (e) {
    rethrow;
  }
}

Expected Result
The above code should return request body as:

item[0][product_size_id]: 14

Actual Result
Instead of the expected result the code is returning request body as:

items%5B0%5D%5Bproduct_size_id%5D: 14

Yes, working.

On 17 Mar 2023, at 15:11, Alex Li @.***> wrote:
Is the same request working on 5.0.1?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

So this might be a regression invented from #1704. cc @kuhnroyal

Can you please add your input formData map to the issue.

Thanks a lot!
On Fri, Mar 17, 2023, 20:49 Peter Leibiger @.***> wrote:

Closed #1741 https://github.com/cfug/dio/issues/1741 as completed via
#1747 https://github.com/cfug/dio/pull/1747.
—
Reply to this email directly, view it on GitHub
https://github.com/cfug/dio/issues/1741#event-8781505624, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXO4DXFKZI3XR5SVUNNXTSDW4SW43ANCNFSM6AAAAAAV5CFESY
.
You are receiving this because you commented.Message ID:
@.***>

Seems an issue still in place. With 5.0.1 on the backend server I had in POST vars:
{"a": 1, "b": 2, "c": 3}
With 5.0.3 the same looks like:
[{"a": 1, "b": 2, "c": 3}]
Seems POST map wrapped with the additional List somehow.

Please provide me with a reproducible sample, then I can fix this asap.

I don't see how this can be happening.
This test succeeds:
  test('post map', () async {
    final dio = Dio()
      ..options.baseUrl = EchoAdapter.mockBase
      ..httpClientAdapter = EchoAdapter();

    final response = await dio.post(
      '/post',
      data: {'a': 1, 'b': 2, 'c': 3},
    );
    expect(response.data, '{"a":1,"b":2,"c":3}');
    expect(response.statusCode, 200);
  });

Here is a real case for Flutter web application. I'm sending:
dio.FormData.fromMap({'_mpc_api': {'dest': '/', 'data': {'a': 1, 'b': 2, 'c': 3}}}, dio.ListFormat.multiCompatible);
In 5.0.1 request is:
----dio-boundary-4271291696
content-disposition: form-data; name=“_mpc_api[dest]”
/
----dio-boundary-4271291696
content-disposition: form-data; name=“_mpc_api[data][a]”
1
----dio-boundary-4271291696
content-disposition: form-data; name=“_mpc_api[data][b]”
2
----dio-boundary-4271291696
content-disposition: form-data; name=“_mpc_api[data][c]”
3
----dio-boundary-4271291696--

In 5.0.3
----dio-boundary-3699245676
content-disposition: form-data; name=“_mpc_api[dest]”
/
----dio-boundary-3699245676
content-disposition: form-data; name=“_mpc_api[data]”
{a: 1, b: 2, c: 3}
----dio-boundary-3699245676--

I can confirm that this issue is a web-only seems. Everything is fine on IOS, Android and tests. Something is going wrong here (check fields, MapEntry _mpc_api[data] converted to the string):

I don't know, looks like the value is a String. It must have gone wrong before this but there is no separate implementation for the browser for the encoding.

I changed more formdata tests to run on browser but they alls succeed. Please try to debug FormData.fromMap and encodeMap().

I've found the source of a problem: it's happens when some part of a data is presented as Map<dynamic, dynamic>. Here is a test for that case:
test('fromMap with dynamic Map inside', () async {
   Map dynamicData = {
     'a': 1,
     'b': 2,
     'c': 3,
   };

   Map<String, dynamic> request = {
     'api': {'dest': '/', 'data': dynamicData}
   };

   final fd = dio.FormData.fromMap(request);
   final data = await fd.readAsBytes();
   final result = utf8.decode(data, allowMalformed: true);
   expect(result, contains('name="api[dest]"'));
   expect(result, contains('name="api[data][a]"'));
   expect(result, contains('name="api[data][b]"'));
   expect(result, contains('name="api[data][c]"'));
 });

That makes sense, it was one thing that came to mind when I reviewed my last PR. I think the maps should always have a String key so we probably need to improve the typing. But you should be able to work around the problem.

Yes, this issue already fixed in our codebase.


Trojan:Script/Oneeva.A!ml

I tried to download latest release dovi_tool-1.4.7-x86_64-pc-windows-msvc.zip and MS Defender deleted it. Defender says it found Trojan:Script/Oneeva.A!ml.
Previous release 1.4.6 is OK.

My Defender deleted it also.
Severe
Detected: Trojan:Script/Wacatac.B!ml
Details: This program is dangerous and executes commands from an attacker.

Huh, that's weird.

On my Windows 10 machine, Defender says it's clean FWIW.
Ran it through VirusTotal and there's only one detection.

There's nothing I can do about this so I'll close.
I had another Windows Defender definitions update and it still comes up clean.

just downloaded the latest version (1.5.4) and defender also deleted it. Oddly 1.5.1 worked fine with no complaints.

yes, here too.  :(

Versions 1.5.1-1.5.3 are clean. Version 1.5.4 triggers Defender again.
@quietvoid are you using same machine for all release builds?

The binaries are built by GitHub.

I found many projects on GitHub have same issues and no explicit solution. We can just wait for the new release.

It's a false positive from Defender, just like many other projects, and we can’t expect the result of future releases. Perhaps you should add it to the whitelist.


WIP: Support DPDK/SPDK in EVE

Support DPDK/SPDK in EVE

Hey @temerkhanov -- it also breaks on aarch64
Step 37/65 : RUN ./configure --prefix=/usr/ --with-vhost --with-uring --with-shared &&     make && make install && make install DESTDIR=/out
 ---> Running in dd788178cf05
Using default SPDK env in /spdk/lib/env_dpdk
Using default DPDK in /spdk/dpdk/build
Configuring ISA-L (logfile: /spdk/isa-l/spdk-isal.log)...done.
Creating mk/config.mk...done.
Creating mk/cc.flags.mk...done.
Type 'make' to build.
ar: `u' modifier ignored since `D' is the default (see `U')
The Meson build system
Version: 0.54.2
Source dir: /spdk/dpdk
Build dir: /spdk/dpdk/build-tmp
Build type: native build
Program cat found: YES (/bin/cat)
Project name: DPDK
Project version: 20.08.0
Using 'CC' from environment with value: 'gcc'
Using 'CC' from environment with value: 'gcc'
C compiler for the host machine: gcc (gcc 9.3.0 "gcc (Alpine 9.3.0) 9.3.0")
C linker for the host machine: gcc ld.bfd 2.34
Host machine cpu family: aarch64
Host machine cpu: aarch64
Program pkg-config found: YES (/usr/bin/pkg-config)
Program gen-pmdinfo-cfile.sh found: YES (/spdk/dpdk/buildtools/gen-pmdinfo-cfile.sh)
Program list-dir-globs.py found: YES (/spdk/dpdk/buildtools/list-dir-globs.py)
Program check-symbols.sh found: YES (/spdk/dpdk/buildtools/check-symbols.sh)
Program options-ibverbs-static.sh found: YES (/spdk/dpdk/buildtools/options-ibverbs-static.sh)
Program binutils-avx512-check.sh found: YES (/spdk/dpdk/buildtools/binutils-avx512-check.sh)
Program python3 found: YES (/usr/bin/python3)
Program cat found: YES (/bin/cat)
Program ../buildtools/symlink-drivers-solibs.sh found: YES (/bin/sh /spdk/dpdk/config/../buildtools/symlink-drivers-solibs.sh)
Checking for size of "void *" : 8
Library m found: YES
Library numa found: NO
Found pkg-config: /usr/bin/pkg-config (1.7.2)
Did not find CMake 'cmake'
Found CMake: NO
Run-time dependency libbsd found: NO (tried pkgconfig and cmake)
pcap-config found: NO
Run-time dependency pcap found: NO (tried pkgconfig and config-tool)
Compiler for C supports arguments -Wextra: YES
config/meson.build:223: WARNING: Consider using the built-in warning_level option instead of using "-Wextra".
Compiler for C supports arguments -Wcast-qual: YES
Compiler for C supports arguments -Wdeprecated: YES
Compiler for C supports arguments -Wformat-nonliteral: YES
Compiler for C supports arguments -Wformat-security: YES
Compiler for C supports arguments -Wmissing-declarations: YES
Compiler for C supports arguments -Wmissing-prototypes: YES
Compiler for C supports arguments -Wnested-externs: YES
Compiler for C supports arguments -Wold-style-definition: YES
Compiler for C supports arguments -Wpointer-arith: YES
Compiler for C supports arguments -Wsign-compare: YES
Compiler for C supports arguments -Wstrict-prototypes: YES
Compiler for C supports arguments -Wundef: YES
Compiler for C supports arguments -Wwrite-strings: YES
Compiler for C supports arguments -Wno-address-of-packed-member -Waddress-of-packed-member: YES
Compiler for C supports arguments -Wno-packed-not-aligned -Wpacked-not-aligned: YES
Compiler for C supports arguments -Wno-missing-field-initializers -Wmissing-field-initializers: YES
Program /spdk/dpdk/config/arm/armv8_machine.py found: YES (/spdk/dpdk/config/arm/armv8_machine.py)

config/arm/meson.build:156:2: ERROR: Could not execute command "/spdk/dpdk/config/arm/armv8_machine.py".

@giggsoff Petr, do you have any interest to take over and continue this work or we close? Sadly this PR does not show any performance numbers (maybe you have some?), because such a work should clearly show benefits of introducing another backend storage, otherwise the value of the whole work is just meaningless.

No progress on these for quite a while. @eriknordmark could you please close this with the "stalled" label?

Closing as stalled


Modified reverseWithStringConcat and reverseWithStringBuilder to handle null or empty string inputs gracefully, returning an empty string in such cases.

…le null or empty string inputs

By submitting this pull request I confirm I've read and complied with the below requirements.

[ ] I have read the Contribution guidelines and I am confident that my PR reflects them.
[ ] I have followed the coding guidelines for this project.
[ ] My code follows the skeleton code structure.
[ ] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.

We added a check to see if string is null or empty (""). If either condition is met, the methods return an empty string ("") right away. This prevents potential NullPointerExceptions if string is null and avoids unnecessary processing for an empty string input, making the code more robust and efficient.


From brokerList to bootstrapServers in the Kafka scaler trigger

Hi all,
I can see from Git history that the brokerList was brokers in the Kafka scaler time ago.
What I would like to propose is changing it again to bootstrapServers which is the most common and recognized way to specify "bootstrap" servers on the first connection from a Kafka client.
Actually, it's really a "bootstrap" servers list because it doesn't need to contain ALL the brokers in the cluster but one or more from which the client gets metadata so that can connect to right brokers later (so it's for bootstrapping...).
The "brokerList" name could drive in the wrong direction with the meaning that you need to put ALL the brokers there, which is not true. This name is still used in the Kafka upstream project for the kafka-console-producer tool but it's going to be changed in order to align with the kafka-console-consumer which uses a "--bootstrap-servers" parameter instead.
Said that, how do you approach when changing something in the CRDs?
As far as I can see, triggers metadata is just a map so there is no validation at CRD level / API server level but just inside the scaler when it gets the metadata.
In this case, do you agree if I propose to change this trigger field in bootstrapServers?

Your proposal makes sense. My only concern is, that this could be potentionaly a breaking change.
We could support both ways for some time and mark brokerList as deprecated, (we could print warning if user uses this property).
We are planning to release v2.0 with some breaking changes in ScaledObject CRD so that could be a right time of getting rid of brokerList.

Yes, for this reason, I was asking how you are approaching changes like this in CRDs.
I will open a PR supporting both of them for a marking brokerList as deprecated for the time being.
On 2.0 release we'll get rid of it.

Reopening for docs https://github.com/kedacore/keda-docs/pull/81

Closing because the related KEDA docs PR was merged.


More samples in the documentation

First I want to thank you for opening this amazing tool, the documentation is very good too, I think it's the best from these tools that mimic Storybook.
I saw in Reddit that you are planning to release some sample projects and I think it would be of great value for the community.
I'm opening this issue so this can be tracked and also to say that I'm open to help with the docs if needed.

You're very welcome and thank you for your kind words!
We are currently working on a sample project to show how Monarch can work with:

The Bloc pattern
The Provider pattern
Internationalization
Custom themes
Different UI interactions, like Modals, nav bars, etc.

We'll update this issue once we publish the sample project.

Monarch samples project is ready! 🚀
https://github.com/Dropsource/monarch_samples

Awesome! Thank you for the samples


make sure members complete safety training for myhal room access and record who has done it, also collect student numbers

How are you gonna organize this information. I don't recommend using a Google sheet. Unless it's the original members list sheet found here. Needs to be recreated. Better method is to make sure everyone is in the Facebook group with all members and add them on GitHub tasks. We also need to make sure everyone's on GitHub.
https://docs.google.com/spreadsheets/d/13FgYLvUG2JUt1Uslx0IyqQIjBoeisvwhsGjtTIWwvBU/edit?usp=drivesdk


Wrong Symbol Printed

I have:

[X] searched open and closed issues for duplicates

Bug description
This may be something with the printer I’m using (Generic POS-5805DD), but instead of $, it prints the yen symbol, no matter which character set I’m trying to use.
Steps to reproduce
from escpos import *

p = printer.File("/dev/cu.BlueToothPrinter-HS_SPP")
p.charcode(code="USA")
p.set(font='b')
p.block_text('USAA: $22.93 came from your account ending 7991. Details: WAL-MART #0547. Go to usaa.com/account. Reply HELP for help.',42)
p.control('FF')
p.cut()

OK, kinda solved my own issue. Apparently it’s a font issue. The correct sign prints in Font A but not in Font B.


Error while running convert_to_records.py

Hey there.
I love your program, it's very, very nice.
But i am getting this error, maybe you can help me?
Askes-MBP:DareToAGE aske$ python convert_to_records_multiCPU.py --imdb --nworks 2 Traceback (most recent call last):   File "convert_to_records_multiCPU.py", line 28, in <module>     import cv2   File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/cv2/__init__.py", line 4, in <module>     from .cv2 import * ImportError: dlopen(/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/cv2/cv2.so, 2): Symbol not found: _clock_gettime   Referenced from: /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/cv2/.dylibs/libavutil.55.78.100.dylib (which was built for Mac OS X 10.12)   Expected in: /usr/lib/libSystem.B.dylib  in /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/cv2/.dylibs/libavutil.55.78.100.dylib Askes-MBP:DareToAGE aske$
Kind regards

There seems some errors about CV2, however, I didn't have ever tested under MAC, so I'm not clear about how to solve it.


Add OpenMM-torch interface

Adds a MD interface to OpenMM via OpenMM-torch
Contributor (creator of pull-request) checklist

[ ] Tests updated (for new features and bugfixes)?
[ ] Documentation updated (for new features)?
[ ] Issue referenced (for PRs that solve an issue)?

Reviewer checklist

[ ] CHANGELOG updated with public API or any other important changes?

📚 Download documentation preview for this pull-request

⚙ Download Python wheels for this pull-request (you can install these with pip)

Typical code to start a simulation would look like this:
platform = openmm.Platform.getPlatformByName('CUDA')
properties = {'Precision': 'double'}
simulation = openmm.app.Simulation(topology, system, integrator, platform, properties)

The device is handled by OpenMM torch (it moves the model to the desired device).
The dtype is more tricky, as it is not handled by OpenMM (I think they're trying to allow model developers to do things in mixed precision, so they won't call to(dtype)). At the moment, if the precision is wrong, the model will crash at runtime, and if the user matches the dtypes correctly, the model will run.
Let me know what you think the best solution is (leave as is, check at each step, etc)

Let me know what you think the best solution is (leave as is, check at each step, etc)

The way we are doing this in other engines is to always convert from the engine dtype to the model dtype for inputs, and the other way around for outputs. We could add a warning about mismatched dtype if we do the conversion, but I would rather allow the code to run instead of crashing.

Printing metadata doesn't work because ::print() is not registered it seems, I'll see if I can fix it


How can I use one button using Floaty

I want one floating action button(no child button)
If it clicked, do something probably go to write message page
Is it possible?

use the delegate function: emptyFloatySelected() to catch the tap on an floaty button with no items added. Make sure to set the delegate.
floaty.fabDelegate = self
and implement:

}```


Add dependency on reskins-bobs

With the latest changes to reskins-bobs, setting the dependency here produces the desired result for #4.

Already done that locally, I was waiting for your fix to test it.


No way to exit impersonation on smaller screens

Below a certain size, the footer disappears. The Copyright information moves to the bottom of the menu, but the exit impersonation link does not.

Need to copy the {% if is_granted('ROLE_PREVIOUS_ADMIN') %} statement in app/resources/views/base.html.twig (line 181 ish) and paste it in the top navbar, after the "Logout" li (line 139 ish) and add the hidden-lg hidden-md classes to the li inside of the if statement.
That should make the "Stop Impersonating" li appear in the drop-down menu on small screens, and leave it in the footer on large and medium screens.

Could I give this a try? I've only made two pull requests so far and this seems like a good issue to practice with and become familiar with contributions.

Absolutely @alaskaa! If you need help, let me know.

Just as a heads up, still working on this :) should get it finished in the next two days :)


negative feature importance

After running the C4.5 algorithm I get negative values for feature importance,  I find a bit confusing as feature importance is calculated essentially as: the total amount of entropy from the parent node that the current node split can explain or "organize" into more homogenic nodes. If I am getting a negative feature importance it means that the node split is essentially creating entropy or less homogeneity which makes no sense. On top of that, these features, with negative importances, are being used in the rules.py.
Additionally, there is evidence in the literature that "Every time a node is split on variable, the combined impurity for the two descendent nodes is less than the parent node." Unbiased Measurement of Feature Importance in Tree-Based Methods - Split Improvement
Any chance you could explain this behavior?
DISCLAIMER: I can't share the data, as it is private.

would you please share your dataset to reproduce same feature importance?


Bug with new apps

When a new app is posted to this repo,  the author doesn't add the amount of stars. When app data is synced, it gets added.
The problem is that a GitHub actions push can't trigger another GitHub action.
There are two ways to resolve this:

Push an empty commit after an app gets updated.
Build & update the website during the app update workflow.
I prefer 2 because it's automated and also works when other data gets updated.

@gr2m What do you prefer?

The problem is that a GitHub actions push can't trigger another GitHub action.

It can, we just need to use a different means of authentication that a secrets.GITHUB_TOKEN. This PR should make it work: https://github.com/probot/probot.github.io/pull/384

Can you give it a try if it works as expected?

It seems like the commit author needs to be set to probotbot too.

it looks like that is not possible, the usage of GITHUB_ACTOR is hardcoded:
https://github.com/davidolrik/push/blob/b88b2c57f3a4c23bf2abf091c319e77088dd0172/start.sh#L38
GITHUB_ACTOR will always be set to GitHub Actions. Let me replace that step with simple git commands instead


fix #7782: check title and excerpt separately for page_content

Description: check title and excerpt separately for page_content so that if title is empty but excerpt is present, the page_content will only contain the excerpt
Issue: #7782
Tag maintainer: @3coins @baskaryan
Twitter handle: wilsonleao

@wnleao
Thanks for adding this. LGTM!


Update to fully support v6 of the Discord API

Looks like v5 (which Discore currently operates on) will be deprecated soon: https://github.com/hammerandchisel/discord-api-docs/pull/293. Once the v6 documentation is complete, Discore should be updated as soon as possible.
Edit: v5 will be deprecated as of October 16, 2017, issues #16 and #17 have been created to address this issue, but neither will add full support for v6 (just enough to keep running).

Oh boy, wasn't aware of this at all. Do you think the update will be applied to both versions, 3 and 2?
If not, I will start making needed changes asap for my current projects!
Thanks in advance!

If needed, I can update 2.x as well, but it looks like this requires breaking changes for both 2.x and 3.x anyway...

Perhaps it is time for me to update to v3 anyway ;)

Okay so good news, after reviewing the changes made by version 6, I have determined that it is completely possible to patch both 2.x and 3.x without any breaking changes. Both patches should be released by the end of this weekend and have their own issues: #16 #17 .
The deprecation date for v6 was also pushed back a couple months (now October 16, 2017), which should give anyone using Discore plenty of time to update their nuget packages. Discore v3.0.0 and all releases below v2.4.1 will also be deprecated on this date.
However, since those patches won't be adding any functionality, they don't resolve this issue. For Discore to update fully to v6, 2 breaking changes would occur:

DiscordGuildChannelType would be removed.
DiscordChannelType would get completely new values.

And of course anything using those two enums would also have to change.
Right now I'm not entirely sure how to go about this. Following semantic versioning, this would bring us to Discore 4.0.0, but it feels wrong to already have another major release.
Any and all feedback is appreciated!

Just released v3.0.1 and v2.4.1 to bring internal support for v6.

I've decided that this issue will be solved across two releases:
v3.1.0: All additions that Gateway v6 brings that can be implemented without breaking changes.
v4.0.0 All breaking changes/additions that will let Discore more accurately represent v6. This release will also have other things, but I don't have a finished plan yet.
What to expect in v3.1.0:

DiscordUserStatus.Invisible
DiscordMessage.Type
DiscordGuild.ApplicationId

It's not much, but that's all that can be implemented without breaking stuff.

This is fixed on the 4.0.0 branch. :D


Navigation (Usage Question)

Hi,
How do you navigate back to the high-level "Project View" from within a single diagram? I do this by re-opening the project JSON file.
(I apologize, this doesn't seem like the right forum. Where is the best place to make questions about Threat Dragon usage?)
Best,
Michael

Hello @statueofmike - sorry for the slow response, on holiday in Devon with intermittent network.
Within the diagram you can return to the project view page by closing the diagram (the second button of the seven diagram buttons, marked with a cross). I agree it is not intuitive, and this is a question that has come up a couple of times now, so I will add a FAQ to :
https://owasp.org/www-project-threat-dragon/#div-faqs
Cheers,
Jon


[FORMATTING] Extra tabs added to multiline comments

Input data
Which SQL and options did you provide as input?
Select * From TABLE_NAME Where 1=1
/*
comment line1
comment line2
comment line3
*/

Expected Output
Select
  *
From
  TABLE_NAME
Where
  1 = 1
  /*
  comment line1
  comment line2
  comment line3
  */

Actual Output
Select
  *
From
  TABLE_NAME
Where
  1 = 1
  /*
                      comment line1
                      comment line2
                      comment line3
                      */

Usage

How are you calling / using the library? - regular
What SQL language(s) does this apply to? - SQL
Which SQL Formatter version are you using? - latest

Every time format is applied to the query string a new tab is appended to each line of the multiline comment (except the first line containing start of comment)

Thanks for reporting. Unfortunately I'm unable to reproduce the exact issue.
I did notice one problem. Given the following SQL as input:
SELECT
  1
  /*
  comment line
  */

the output will contain an extra space on the last line of the comment:
SELECT
  1
  /*
  comment line
   */

This is definitely a bug, and I think I know what's causing this. Will write a fix.
However, the problem you are describing seems to be different. Could you provide more details about your setup:

How exactly are you invoking SQL Formatter? Through JavaScript API? From command line? Through editor plugin?
Which configuration options are you using? For example, you talk about new tabs being appended... are you perhaps using the useTabs option?
Which exact SQL Formatter version are you using? You say you're running the latest version, do you mean 12.1.0?
Which operating system are you running on?
Anything else about your environment that might be useful... (NodeJS version, the exact command you're using to call the formatter).

One factor that might influence this issue is whether your input file uses Windows-style line endings (\r\n) or UNIX style (\n).

Given no further info, I'm closing this as: unable to reproduce.

I also encountered the same problem. When performing multiple formatting, spaces are added to the content of the comment each time.
before：
select
  *
from
  commonData
  /*
    comment line1
    comment line2
    comment line3
  */

after:
select
  *
from
  commonData
  /*
         comment line1
         comment line2
         comment line3
  */

version: 0.1.12

I guess you're using sql-prettier, which is a super-outdated fork of this library. I can't help with that.

OK, I tried to use the latest version of sql-formatter, and it seems that there is no such problem. However, some syntax in my editor will be recognized as illegal keywords. I would like to ask you, is there a more conservative version when upgrading from sql-prettier to sql-formatter?

Check that you've specified the SQL dialect that you're formatting: https://github.com/sql-formatter-org/sql-formatter?tab=readme-ov-file#parse-error-unexpected--at-line-
See supported dialects.
If you're still running into problems, please open a new issue.


Add AngularBuddies ng-2 room as community resource

you have write permission so merge at will

Cool
On Wed, Sep 23, 2015 at 12:37 AM PatrickJS<EMAIL_ADDRESS>wrote:

you have write permission so merge at will
—
Reply to this email directly or view it on GitHub
https://github.com/angular-class/awesome-angular2/pull/12#issuecomment-142498926
.


Compatibility with Annotations 2

Running this plugin having Doctrine Annotations 2 installed currently results in a crash: https://github.com/doctrine/DoctrineBundle/actions/runs/3742592139/jobs/6353730157
The call to AnnotationRegistry::registerLoader() is safeguarded with a method_exists() call now. Alternatively, we could also declare a conflict with Annotations < 1.10 and remove the call entirely.

The PHP-CS-Fixer failure looks unrelated. But I might need some help getting the high-deps builds green.

But I might need some help getting the high-deps builds green.

I think the only option currently is to suppress / baseline the UnusedPsalmSuppress issue. I started the discussion on the Psalm repo to see if anyone has any better ideas.


entgql: add WithRelaySpec to be able to disable Relay specification

The naming is based on this comment: https://github.com/ent/contrib/blob/1faab982b6648b7704a6cf41ff65d9cb7811a2be/entgql/schema.go#L45

Thank @tadeokondrak for your contribution, ❤


fix conditional depenency numba on python 3.11

Removes dependency numba on python 3.11

@conda-forge-admin, please rerender