Luanch

pkg install wget openssl-tool proot -y && hash -r && wget https://raw.githubusercontent.com/EXALAB/AnLinux-Resources/master/Scripts/Installer/Ubuntu/ubuntu.sh && bash ubuntu.sh

I don't know what this was meant to communicate. Please do not spam this repository.


[Unexpected Behavior] InputHandler(event) callback doesn't provide realtime data

Build Used
OutFox Alpha 4.3-pre14
Steps to Reproduce

Create modfile with the following included:

local function InputHandler(event)
    if not event.PlayerNumber return end
    lua.ReportScriptError(tostring(event.DeviceInput.level))
end

Test modfile with analog input

Expected Behavior
Error log reports current, realtime analog input level.
Actual Behavior
Error log reports initial level, then reports level of 1 for repeat until input release.
Tested with Akai Professional MPD218 via MIDI input, observed sending realtime analog input level in ScreenTestInput.

This bug was revealed to me after receiving a test build that fixed issue #80 to correct MIDI behavior.

we have added proper analog support in lua, I could add another .analoglevel to the input handler but I consider it unesseasry, also I wouldnt change the way .level works because it will break backward compatibility

Definitely important to keep compatibility, but I'm not partial to any specific implementation. having a .analog or similar key-value would work for me. Not sure my feedback on how it's implemented is completely necessary, but I'm glad I'm given the opportunity at giving my input.

Thanks for replying, we'll revisit midi when the linux driver is added.


Add API to (auto) switch output device on lwjgl3

This pr adds support for three main things (only for lwjgl3):

Auto switching the Output device if e.g. a headphone gets disconnected. This is mainly relevant on windows, because before that the audio just died.
It reevaluates the default device, if a new device gets connected, and therefor maybe switches to it (the os decides)
It adds a API to specifically select a output device. This API supports swapping too, so if the device gets disconnected and reconected, it will be choosen again.

I also added a Test for this.

This is now ready to get reviewed, since libGDX got updated to lwjgl3 3.3.2.


No file or variants found for asset: app/app.zip.

PS C:\Users\user\PycharmProjects\MSM> flet build windows -vv
Flutter bootstrap directory: C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq
Creating Flutter bootstrap project...OK
Customizing app icons and splash images...OK
Generating app icons...  ╔════════════════════════════════════════════════════════════════════════════╗
║ The Dart tool uses Google Analytics to report feature usage statistics     ║
║ and to send basic crash reports. This data is used to help improve the     ║
║ Dart platform and tools over time.                                         ║
║                                                                            ║
║ To disable reporting of analytics, run:                                    ║
║                                                                            ║
║   dart --disable-analytics                                                 ║
║                                                                            ║
╚════════════════════════════════════════════════════════════════════════════╝
Resolving dependencies in C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq... (2.8s)
Got dependencies in C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq.
Building package executable... (2.2s)
Built flutter_launcher_icons:flutter_launcher_icons.
════════════════════════════════════════════
FLUTTER LAUNCHER ICONS (v0.13.1)
════════════════════════════════════════════
• Creating default icons Android
• Overwriting the default Android launcher icon with a new icon
• Overwriting default iOS launcher icon with new icon
Creating Icons for Web...
Creating Icons for Windows...
Creating Icons for MacOS...
✓ Successfully generated launcher icons
OK
Packaging Python app...Building package executable... (1.3s)
Built serious_python:main.
Running package command
Creating asset directory: C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\app
Copying Python app from C:\Users\user\PycharmProjects\MSM to C:\Users\user\AppData\Local\Temp\serious_python_temp24caec34
Error: FileSystemException: Failed to decode data using encoding 'utf-8', path = 'C:\Users\user\AppData\Local\Temp\serious_python_temp24caec34\requirements.txt'
Deleting temp directory C:\Users\user\AppData\Local\Temp\serious_python_temp24caec34
OK
Building Windows app...['C:\Users\user\Desktop\flut\flutter\bin\flutter.BAT', 'build', 'windows']
Resolving dependencies...
fl_chart 0.65.0 (0.66.0 available)
flutter_lints 2.0.3 (3.0.1 available)
flutter_native_splash 2.3.7 (2.3.9 available)
js 0.6.7 (0.7.0 available)
lints 2.1.1 (3.0.0 available)
matcher 0.12.16 (0.12.16+1 available)
material_color_utilities 0.5.0 (0.8.0 available)
meta 1.10.0 (1.11.0 available)
package_info_plus 4.2.0 (5.0.1 available)
path 1.8.3 (1.9.0 available)
petitparser 5.4.0 (6.0.2 available)
test_api 0.6.1 (0.7.0 available)
toml 0.14.0 (0.15.0 available)
web 0.3.0 (0.4.0 available)
web_socket_channel 2.4.0 (2.4.3 available)
xml 6.3.0 (6.5.0 available)
Got dependencies!
16 packages have newer versions incompatible with dependency constraints.
Try flutter pub outdated for more information.
windows/flutter/CMakeLists.txt does not use FLUTTER_TARGET_PLATFORM, updating.
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppBuild.targets(541,5): warning MSB8029: �஬������ ��⠫�� ��� ��⠫�� ��室� �� ����� �ᯮ�������� �� �६����� ��⠫���, ��⮬� �� �� ����� ���� ��稭
�� �஡��� � ������筮� ᡮમ�. [C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\build\windows\x64\ZERO_CHECK.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppBuild.targets(541,5): warning MSB8029: �஬������ ��⠫�� ��� ��⠫�� ��室� �� ����� �ᯮ�������� �� �६����� ��⠫���, ��⮬� �� �� ����� ���� ��稭
�� �஡��� � ������筮� ᡮમ�. [C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\build\windows\x64\plugins\serious_python_windows\CopyPythonDLLs.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppBuild.targets(541,5): warning MSB8029: �஬������ ��⠫�� ��� ��⠫�� ��室� �� ����� �ᯮ�������� �� �६����� ��⠫���, ��⮬� �� �� ����� ���� ��稭
�� �஡��� � ������筮� ᡮમ�. [C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\build\windows\x64\flutter\flutter_assemble.vcxproj]
Error detected in pubspec.yaml:
No file or variants found for asset: app/app.zip.
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(254,5): error MSB8066: ���짮��⥫�᪠� ᡮઠ ��� "C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\build\windows
x64\CMakeFiles\97ccedc66f37bc442ea4e473dec1ca92\flutter_windows.dll.rule;C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\build\windows\x64\CMakeFiles\9736cc717a512a8e02eaa94d6095a866\flutter_assemble.rule;C:\Users\u
ser\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\windows\flutter\CMakeLists.txt" �����訫� ࠡ��� � ����� 1. [C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq\build\windows\x64\flutter\flutter_assemble.vcxproj]
Building Windows application...                                    61,3s
Build process failed.
Deleting Flutter bootstrap directory C:\Users\user\AppData\Local\Temp\flet_flutter_build_arrvgVFfgq
Error building Flet app - see the log of failed command above.
PS C:\Users\user\PycharmProjects\MSM>

OK, this part worries me:
Error: FileSystemException: Failed to decode data using encoding 'utf-8', path = 'C:\Users\user\AppData\Local\Temp\serious_python_temp24caec34\requirements.txt'

What do you have in requirements.txt?

anyio==4.2.0
arrow==1.3.0
binaryornot==0.4.4
certifi==2023.11.17
chardet==5.2.0
charset-normalizer==3.3.2
click==8.1.7
colorama==0.4.6
cookiecutter==2.5.0
et-xmlfile==1.1.0
exceptiongroup==1.2.0
flet==0.18.0
flet-core==0.18.0
flet-runtime==0.18.0
h11==0.14.0
httpcore==0.17.3
httpx==0.24.1
idna==3.6
Jinja2==3.1.2
markdown-it-py==3.0.0
MarkupSafe==2.1.3
mdurl==0.1.2
numpy==1.26.3
oauthlib==3.2.2
openpyxl==3.1.2
packaging==23.2
pandas==2.1.4
Pygments==2.17.2
pypng==0.20220715.0
python-dateutil==2.8.2
python-slugify==8.0.1
pytz==2023.3.post1
PyYAML==6.0.1
qrcode==7.4.2
repath==0.9.0
requests==2.31.0
rich==13.7.0
six==1.16.0
sniffio==1.3.0
text-unidecode==1.3
types-python-dateutil==<IP_ADDRESS>40106
typing_extensions==4.9.0
tzdata==2023.4
urllib3==2.1.0
watchdog==3.0.0
websocket-client==1.7.0
websockets==11.0.3

Alright, please see "no pip freeze" note in this section: https://flet.dev/docs/guides/python/packaging-app-for-distribution#project-structure
Also, save requirements.txt in UTF-8 encoding.

@FeodorFitsner I think, they were powershell (it is my assumption). Because powershell uses UTF-16 encoding when  Write-Output is used.

@taaaf11 that makes sense when you do pip freeze from PS! Let's wait for confirmation and then create a new issue for that.

@FeodorFitsner I think  I am getting the issue... I pip freeze > requirements.txt-ed from my virtual environment. Opened the file in python prompt. f.read() function gave a lot of \x-s, and f.encoding gave cp1252.

@taaaf11 yep, but in context of flet buid requirements.txt is opening from Dart ;)

@FeodorFitsner I fear replying, I don't know Dart 🤐

somehow the project was built  (but with errors during building) and now when i am trying to start an exe file. mt app just blinks on startup and then closes itself. What could be a problem with that?

@Slakeec Make your requirments.txt hand-picked, like, if you are using packages flet and flask, include the packages in requirements.txt like this (put in the versions):
flet==(some ver)
flask==(some ver)

Also, edit your second-last comment and put necessary code formatting like this:

Surround the error message text (the full output) with three backticks (this  `).

windows/flutter/CMakeLists.txt does not use FLUTTER_TARGET_PLATFORM, updating.
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppBuild.targets(541,5): warning

Please correct the inverted slash thing.
If it is / just use that one
But if it's \ just that, nothing else please

Please give the latest Flet version a try.


Add change_datatype to import/export actions

and make PJAs less repetitive. Also tests all PJA conversions.
xref https://github.com/galaxyproject/galaxy/pull/7480

Alright, this is done, and the workflow API tests for https://github.com/galaxyproject/galaxy/pull/7480 are passing locally with these changes.

@jmchilton can you take a look at this?

Released as 0.8.1, thanks @mvdbeek!


Human ID fix

This PR adds a salt to the DApp specific humanID to make it non-guessable for privacy.
To ensure that each user still only gets one DApp specific humanID, the salt is registered and locked in a smart contract with each ZkKYC.
For simplicity, we reuse the signature that makes up the holder commitment as salt.
For the unlikely case that some user looses access to his account and salt, there is a reset mechanism that works when all zkKYCs of the user are expired, revoked or the issuer has been delisted.

Fixed a type error found by the pipeline.

Also created a GitBook PR to change the documentation accordingly https://app.gitbook.com/o/IbmhhVJSM8rZ0aECe2R3/s/NMoORBGBxztthVlosoIF/~/changes/51/guardian-guide/create-and-issue-zkkyc
@anhdungle93

we also need to add the saltSignature to ageProofZkKYC circuit too.

Is the salt only recommended or required to be the holder commitment? Maybe we can just use the holder commitment directly to make the circuit simple (depending on if the improvement is worth it or not).

we also need to add the saltSignature to ageProofZkKYC circuit too.

Is the salt only recommended or required to be the holder commitment? Maybe we can just use the holder commitment directly to make the circuit simple (depending on if the improvement is worth it or not).

For zkKYC we require the salt to be the pre-image of the holder commitment for simplicity. It has all the properties we need and this way we do not need to modify ageProofZkKYC. The private input is already there and the imported zkKYC component handles the humanID logic.

we also need to add the saltSignature to ageProofZkKYC circuit too.

Is the salt only recommended or required to be the holder commitment? Maybe we can just use the holder commitment directly to make the circuit simple (depending on if the improvement is worth it or not).

For zkKYC we require the salt to be the pre-image of the holder commitment for simplicity. It has all the properties we need and this way we do not need to modify ageProofZkKYC. The private input is already there and the imported zkKYC component handles the humanID logic.

I see, the three inputs are indeed used in both subcircuits Ownership and HumanID. In the beginning I thought ZkKYC has three more inputs.


Area chart fill range

Checklist

[x] I am running the latest version of the card
[ ] I checked if there isn't another feature request opened with the same request
[ ] I checked that the feature is not already available in the dev branch

Is your feature request related to a problem? Please describe.
Area charts start their shading fill from 0. When the data range goes into negative, this may or may not be the desired outcome.
Describe the solution you'd like
An option to fill up from the bottom or up or down from the x-axis would be great!
An option to fill up from the bottom or up or down from the x-axis would be great!

That exists already: https://apexcharts.com/docs/options/plotoptions/area/
apex_config:
  plotOptions:
    area:
      fillTo: end


revise deprecated setoutput to setout in bug-report

Please provide a description of this PR:
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@howardjohn Thanks for the reminding. Just add the SetErr part.


Add preferred middle names

Identify preferred middle names and display those on the profiles.

@sbagg Strangely, not all of these users with middle names are showing up.  Can we figure out why?
You can get a list of the experts that have a middle name with:
select ?md
where {
  ?user vcard:hasName ?s;
        a ucdrp:person;
        .
  ?s vcard:middleName ?m.
OPTIONAL {?s vcard:familyName ?l. }
OPTIONAL {?s vcard:givenName ?f. }
  bind(replace(str(?user),'http://experts','https://rc.experts') as ?link)
  bind(concat('[',$f,' ',$m,' ',$l,'](',?link,')') as ?md)
}

Which results in
Ralph Curtis Aldredge
Stavros George Vougioukas
Barbara Sabine Linke
C. P. van Dam
Francis F. Assadian
Bryan M Jenkins
Gail Michelle Bornhorst
Juliana Maria Leite Nobrega De Moura Bell
Jeremy K Mason
Jeffrey Mason Earles
Justin R Merz
Jennifer S Mullin
Marina S Leite
Michael R Hill
Niels Gronbech Jensen
Peter Ako Larbi
Ricardo H R Castro
Scott J Mccormack
Stephen K Robinson
Vessela Y Ensberg

I still see differences between Firefox and Chrome browsers


Update to sigstore v0.10.0 to pickup cosign fix

Re-enable cosign signature validation by default.
The sigstore update also included some API changes which required changes in bpfman.
Fixes: #1241

@dave-tucker can you take a look at this change?  Sigstore changed a couple of APIs including lifetime and async changes.  My updates work, but I wanted to know if you had any concerns.  Thanks, Andre

LGTM but added a hold label until Billy's comment is addressed.


Microwave Generator should have the Hidden gBalloon popping upgrade earlier

This tower's main strength is that it can pop any gBalloon type except Hidden. It's upgrades should allow it to do what it does best better.

Resolved via commit 7a43a8b40ad22445dfc4d5ecb7c7aaf492393831.


error boundary class to functional

Notes for Reviewers
This PR fixes ##11493
in this pr i convert error boundary class component to functional  and use SistentErrorBoundary
and also convert mesherydatetime picker from class to function based component
@leecalcote @nebula-aac @Yashsharma1911 kindly review this
Signed commits

[x] Yes, I signed my commits.

in this pr i convert error boundary class component to functional and use SistentErrorBoundary
@Yashsharma1911 according to you i choosing subtask to work on UI Migration from MUI v4 to MUI v5 and Sistent #11493  this issue and contributing and i want to contribute more and participating in dev meetings and meet with community people and maintainers and also contribute in past in other layer5 repos as i want to contributing more on this issue through the LFX mentorship

@krishkumar84 let's discuss this PR during the Meshery dev meeting call, if you would. Please add it as an agenda item to the meeting minutes.

Thanks, @krishkumar84! Will you investigate why the build check is failing here? https://github.com/meshery/meshery/actions/runs/10351991000/job/28651657180?pr=11620

@leecalcote  sure i will look into that and fix !

@sudhanshutech should this be resurrected?


Optimization idea: store last time filter was searched

@probablycorey suggests an optimization where we would record each time a filter was fetched. Then if the user hits refresh, we can alter their query so it filters by updated: and only includes results that have been updated since they last searched.

This is live!


[tools/verible] enable coverage macros

Apparently the verible tool cannot handle macros like this
`define PRAGMA_COVERAGE_OFF
//VCS coverage off \   // pragma coverage off
This was filed to verible with tracking number 251695088.
These macros are in hw/ip/prim/rtl/prim_macros.svh, and can be deployed once the verible bug is fixed.

You may want to cross-file this issue in the verible repo:
https://github.com/chipsalliance/verible/issues

We cannot depend on this for M3 since this is an external dependency, and we have an adequate alternative.


Rename X11 to x11

As of #210, not sure if successful or needed.

No, the capital X is correct. It is libX11 not libx11

OK, lib and header is X11, but package is x11. Now I know. Sorry for confusion.


Fix download links in Go discovery sample gen

Current coverage is 80.47% (diff: 100%)

Merging #645 into master will not change coverage

@@             master       #645   diff @@
==========================================
  Files           274        274
  Lines         10319      10319
  Methods           0          0
  Messages          0          0
  Branches       1360       1360
==========================================
  Hits           8304       8304
  Misses         1667       1667
  Partials        348        348

Powered by Codecov. Last update fba24e1...5c18bf7

LGTM

To achieve coverage for the time being, issued PR to add dfareporting to our baselines (https://github.com/googleapis/toolkit/pull/649). Also started https://github.com/googleapis/toolkit/issues/648 to record obscure test cases like this.


[BUG] Cannot parse package marker expression in requirements.txt

setuptools version
setuptools==75.1.0
Python version
Python 3.11
OS
macOS
Additional environment information

pyenv 3.11.6
pipenv 2024.1.0 to generate requirements.txt
pbr 6.1.0

Description
After updating greenlet to 3.1.1  the marker expression is set to:
greenlet==3.1.1; python_version < '3.13' and platform_machine == 'aarch64' or (platform_machine == 'ppc64le' or (platform_machine == 'x86_64' or (platform_machine == 'amd64' or (platform_machine == 'AMD64' or (platform_machine == 'win32' or platform_machine == 'WIN32')))))
It's not the best to build the expression but it's still valid. To which I get the error in the output section.
If I change to manually include start and ending parenthesis, i.e:
greenlet==3.1.1; (python_version < '3.13' and platform_machine == 'aarch64' or (platform_machine == 'ppc64le' or (platform_machine == 'x86_64' or (platform_machine == 'amd64' or (platform_machine == 'AMD64' or (platform_machine == 'win32' or platform_machine == 'WIN32'))))))
I no longer have the issue.
Expected behavior
Expected setuptools to parse the marker expression
How to Reproduce

install / Update greenlet to 3.1.1 and generate requirements.txt
build a package that is using those same requirements.txt
pip wheel --no-deps -w dist .

Output
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [96 lines of output]
      /Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/setuptools/__init__.py:94: _DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
      !!

              ********************************************************************************
              Requirements should be satisfied by a PEP 517 installer.
              If you are using pip, you can try `pip install --use-pep517`.
              ********************************************************************************

      !!
        dist.fetch_build_eggs(dist.setup_requires)
      [pbr] Generating ChangeLog
      /Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/setuptools/command/develop.py:41: EasyInstallDeprecationWarning: easy_install command is deprecated.
      !!

              ********************************************************************************
              Please avoid running ``setup.py`` and ``easy_install``.
              Instead, use pypa/build, pypa/installer or other
              standards-based tools.

              See https://github.com/pypa/setuptools/issues/917 for details.
              ********************************************************************************

      !!
        easy_install.initialize_options(self)
      /Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/setuptools/_distutils/cmd.py:66: SetuptoolsDeprecationWarning: setup.py install is deprecated.
      !!

              ********************************************************************************
              Please avoid running ``setup.py`` directly.
              Instead, use pypa/build, pypa/installer or other
              standards-based tools.

              See https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html for details.
              ********************************************************************************

      !!
        self.initialize_options()
      Marker evaluation failed, see the following error.  For more information see: http://docs.openstack.org/pbr/latest/user/using.html#environment-markers
      Error parsing
      Traceback (most recent call last):
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/markers.py", line 199, in __init__
          self._markers = _normalize_extra_values(_parse_marker(marker))
                                                  ^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/_parser.py", line 255, in parse_marker
          return _parse_full_marker(Tokenizer(source, rules=DEFAULT_RULES))
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/_parser.py", line 259, in _parse_full_marker
          retval = _parse_marker(tokenizer)
                   ^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/_parser.py", line 268, in _parse_marker
          expression = [_parse_marker_atom(tokenizer)]
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/_parser.py", line 284, in _parse_marker_atom
          with tokenizer.enclosing_tokens(
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/contextlib.py", line 144, in __exit__
          next(self.gen)
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/_tokenizer.py", line 187, in enclosing_tokens
          self.raise_syntax_error(
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/_tokenizer.py", line 165, in raise_syntax_error
          raise ParserSyntaxError(
      packaging._tokenizer.ParserSyntaxError: Expected matching RIGHT_PARENTHESIS for LEFT_PARENTHESIS, after marker expression
          (python_version < '3.13' and platform_machine == 'aarch64' orplatform_machine == 'ppc64le' or (platform_machine == 'x86_64' or (platform_machine == 'amd64' or (platform_machine == 'AMD64' or (platform_machine == 'win32' or platform_machine == 'WIN32')))))
          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^

      The above exception was the direct cause of the following exception:

      Traceback (most recent call last):
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/pkg_resources/__init__.py", line 1652, in evaluate_marker
          marker = packaging.markers.Marker(text)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/packaging/markers.py", line 217, in __init__
          raise InvalidMarker(str(e)) from e
      packaging.markers.InvalidMarker: Expected matching RIGHT_PARENTHESIS for LEFT_PARENTHESIS, after marker expression
          (python_version < '3.13' and platform_machine == 'aarch64' orplatform_machine == 'ppc64le' or (platform_machine == 'x86_64' or (platform_machine == 'amd64' or (platform_machine == 'AMD64' or (platform_machine == 'win32' or platform_machine == 'WIN32')))))
          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^

      The above exception was the direct cause of the following exception:

      Traceback (most recent call last):
        File "/Users/joseguerra/hellofresh/uk-utils/uk-utils/.eggs/pbr-6.1.0-py3.11.egg/pbr/core.py", line 105, in pbr
          attrs = util.cfg_to_args(path, dist.script_args)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/hellofresh/uk-utils/uk-utils/.eggs/pbr-6.1.0-py3.11.egg/pbr/util.py", line 274, in cfg_to_args
          kwargs = setup_cfg_to_setup_kwargs(config, script_args)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/hellofresh/uk-utils/uk-utils/.eggs/pbr-6.1.0-py3.11.egg/pbr/util.py", line 464, in setup_cfg_to_setup_kwargs
          if pkg_resources.evaluate_marker('(%s)' % env_marker):
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/joseguerra/.pyenv/versions/3.11.6/lib/python3.11/site-packages/pkg_resources/__init__.py", line 1655, in evaluate_marker
          raise SyntaxError(e) from e
      SyntaxError: Expected matching RIGHT_PARENTHESIS for LEFT_PARENTHESIS, after marker expression
          (python_version < '3.13' and platform_machine == 'aarch64' orplatform_machine == 'ppc64le' or (platform_machine == 'x86_64' or (platform_machine == 'amd64' or (platform_machine == 'AMD64' or (platform_machine == 'win32' or platform_machine == 'WIN32')))))
          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
      error in setup command: Error parsing /Users/joseguerra/hellofresh/uk-utils/uk-utils/setup.cfg: SyntaxError: Expected matching RIGHT_PARENTHESIS for LEFT_PARENTHESIS, after marker expression
          (python_version < '3.13' and platform_machine == 'aarch64' orplatform_machine == 'ppc64le' or (platform_machine == 'x86_64' or (platform_machine == 'amd64' or (platform_machine == 'AMD64' or (platform_machine == 'win32' or platform_machine == 'WIN32')))))
          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

Hi @joseguerr could you please try to reduce the reproducer by removing any unnecessary 3rd party code? Please see https://stackoverflow.com/help/minimal-reproducible-example.
I suspect that the only thing you need to do is to ensure you have the latest version of packaging installed (or that packaging is not installed at all, so that setuptools uses its own vendored version). This is conclusion is reached via the following test:
cd /tmp
python3.12 -m venv .venv
.venv/bin/python -m pip install -U pip setuptools
.venv/bin/python -m pip uninstall packaging  # Let's make sure setuptools uses the bundled version of packaging
.venv/bin/python

Python 3.12.7 (main, Oct  1 2024, 08:52:11) [GCC 9.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import setuptools # => changes sys.path so that vendored packaging is exposed
>>> from packaging.requirements import Requirement
>>> req = "greenlet==3.1.1; python_version < '3.13' and platform_machine == 'aarch64' or (platform_machine == 'ppc64le' or (platform_machine == 'x86_64' or (platform_machine == 'amd64' or (platform_machine == 'AMD64' or (platform_machine == 'win32' or platform_machine == 'WIN32')))))"
>>> Requirement(req)
<Requirement('greenlet==3.1.1; python_version < "3.13" and platform_machine == "aarch64" or (platform_machine == "ppc64le" or (platform_machine == "x86_64" or (platform_machine == "amd64" or (platform_machine == "AMD64" or (platform_machine == "win32" or platform_machine == "WIN32")))))')>

Another part that is a bit worrying about your logs is the following:
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.

Here we see that a deprecated feature is used, the deprecated feature in question is the CLI capabilities setup.py[^1]. This seems to indicated that a pyproject.toml file is missing. Please make sure your project has one. You can find more info in https://setuptools.pypa.io/en/latest/build_meta.html.
[^1]: setup.py is a valid configuration file for setuptools, but using it as a CLI tool is deprecated and as such is no longer fully supported.

Hi @abravalheri thanks for getting back to me.
It seems using a new packaging version resolved the issue. I no longer have the warning as well.
Thanks for the help!


WIP Enable parallel execution for most tests [ECR-534]:

The results are not very promising — a "hot" mvn verify (= native libs are built, no native ITs) takes 5-8 s less (from 01:14 min to 01:06-01:09 min — but I haven't measured as many times as required to make statistically sound conclusions).
TODO:

TestkitExtension can't work with @Nested tests, see https://github.com/junit-team/junit5/issues/1925
MockitoExtension does not work when Mocks are injected (maybe in other scenarios
too, but we don't use it much to say for sure): https://github.com/mockito/mockito/issues/1630

Overview

See: https://jira.bf.local/browse/ECR-534
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

That's how a testkit failure looks like:
[ERROR] Tests run: 12, Failures: 0, Errors: 9, Skipped: 0, Time elapsed: 0.599 s <<< FAILURE! - in com.exonum.binding.qaservice.transactions.ErrorTxIntegrationTest
[ERROR] getValue{TestKit}  Time elapsed: 0.019 s  <<< ERROR!
org.junit.jupiter.api.extension.ParameterResolutionException:
  Discovered multiple competing ParameterResolvers for parameter
  [com.exonum.binding.testkit.TestKit arg0] in method [void com.exonum.binding.qaservice.QaServiceImplIntegrationTest.getValue(com.exonum.binding.testkit.TestKit)]:
   - com.exonum.binding.testkit.TestKitExtension@27ac2b5b,
   - com.exonum.binding.testkit.TestKitExtension@61db87a5,
   - com.exonum.binding.testkit.TestKitExtension@68e1cae4

Submitted junit-team/junit5#1925

More experiments, this time, with mvn install -P ci-build — which is executed from ./run_all_tests.sh. See results in the Jira: https://jira.bf.local/browse/ECR-534?focusedCommentId=48422&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-48422

Coverage remained the same at 85.398% when pulling 0d5260886839fb689c8db593105a5cd6249974e3 on dmitry-timofeev:run-tests-concurrently-ECR-534 into 8f0eaabb9ceb545845fc492b96f430e44fb219fa on exonum:master.

Please see the updated description — and review, it must be ready :upside_down_face:


Ember Data decorators: An element descriptors .kind property must be either method

Hi, I followed these instructions ^ I installed the polyfill. Now I'm importing the decorators from DS (import DS from 'ember-data'), but I'm seeing a familiar error:

TypeError: An element descriptor's .kind property must be either "method" or "field", but a decorator created an element descriptor with .kind "undefined"
"ember-decorators-polyfill": "1.0.4", <- this is my polyfil version in devDeps
Ember-data is 3.8

Here's my project branch with 1) models honoring new API and 2) package.json reflecting new deps:
https://github.com/NYCPlanning/labs-applicantmaps/tree/545-deprecation-warnings

I think this problem is downstream of another deprecation warning concerning removing /babel-transforms and upgrading ember-cli-babel to at least 7.7.3. Once I did that, the problem was resolved.
Is it worth noting that these things are connected?

Yes, we should probably mention in the README that you have to be using the stage 1 transforms, and that the recommended way to use the stage 1 transforms is to upgrade ember-cli-babel and drop the babel-transforms package.

note for googlers, it's the @ember-decorators/babel-transforms addon that you need to remove

I'm going to close this issue for now, let me know if you're still having problems! Also thanks @BryanCrotaz for that note 😄


When updating a document entry, existing unmapped data gets removed

Scenario:
You have a document with field A and B.
I then save away a document.
Then I modify my 'model' so that it has field A and C.
I retrieve the existing document, A will be filled, C will be empty, and then save it away again as-is (as I received it in the first place)
Result: It'll store A and an empty C field, but the contents of the B field will be have been removed.
Imho, there should be a way to make it so that existing 'unmapped' data can be kept when updating an entry. Either by default or optionally.
We're actually considering using this for a 'cloud' storage solution (of 'data', not files) but that would require YesSql to be able to update a record with new data without removing the unmapped data.
For example: one of the 'users' could be running on an 'old' model, without this ability any 'old model' user would be able to 'remove' unmapped field data.
It would kinda beat the cloud concept if we'd have to update all 'clients' at the same time to prevent this kind of loss.
AimOrchard here btw ;)

I think it should work with the new version. The json document is untouched.


Implement as a discord bot

It will be great

As a general bot, it will not work. Because ChatGPT throughput 50 requests per hour without premium. But I will make a script for a personal bot later.


feat: Add predefined Gaussian driven control

Changes proposed in this pull request:

Add predefined Gaussian driven control as specified in #92

Thanks, this would work. Just a note that I think even after finding A and B, we'll still need to rescale the output of Eq. 5 before comparing with the values of the segments using code similar to the implementation itself because the segments are scaled such that their discrete integral is equal to rabi_rotation, not such that they actually take on values of the ideal Gaussian (i.e. the Gaussian normalised such that its integral is rabi_rotation).


pk-mount ignores -version switch

The -h switch says
$ pk-mount -h 2>&1 | tail -4 | head -2
  -version
    	show version

however,
$ pk-mount -version
2018/09/20 18:06:18 no mount point given and recommended directory /pk doesn't exist; using temp directory /tmp/user/1234/pk-mount960811681

$ perkeepd --version
perkeepd version: master, 2018-09-14-ca7f2e57e3
Go version: go1.11 (linux/amd64)

https://perkeep-review.googlesource.com/c/perkeep/+/18686


RuntimeError

just run python inference_DiffTSR.py
RuntimeError: Error(s) in loading state_dict for Transformer:
size mismatch for embedding_word.lut.weight: copying a param with shape torch.Size([7937, 512]) from checkpoint, the shape in current model is torch.Size([23558, 512]).
size mismatch for generator_word.proj.weight: copying a param with shape torch.Size([7937, 1024]) from checkpoint, the shape in current model is torch.Size([23558, 1024]).
size mismatch for generator_word.proj.bias: copying a param with shape torch.Size([7937]) from checkpoint, the shape in current model is torch.Size([23558]).

Thank you for your attention and interest.
This error is quite unusual because, as long as the model configuration file hasn't been modified, a loading error shouldn't occur. Could you have made changes to some parts of the code? If you could provide more information, it would help me assess the situation.


completed the participation section of the website

sir completed the participation section of the page

The counter should gradually increase from 0 to corresponding no. when in view ,there's react library for this that we used in 7.0 ,try to find some similar library for vanilla js or implement that kind of counter on your own
also the participation section will be below hero section

ok sir

https://github.com/user-attachments/assets/623ad4bc-c22c-4e17-b3f4-1b09f5f6cb10
Sir I have added the animation on the values


Allow in to be Parsed as an Operator

Summary
In Enso's type signature syntax, in behaves more like an operator than a function. However, the parser treats it as a standard variable identifier, which means that it doesn't get interpreted as an operator. This causes problems with how the compiler needs to handle it, as well as problems in how it's parsed relative to other operators (around variable precedence).
Value
The context ascription will be handled properly by the parser.
Specification

[ ] Add the in context ascription operator as an operator in the parser.
[ ] Update the parser to support this new syntax.

Acceptance Criteria & Test Cases

No parser tests are broken.
We can correctly parse in as an operator.

Closed. It is hackily supported in the current parser, and will be supported properly in the new parser.


plugins markdown-it-container does not work

<ReactMarkdown
              source={'::: spoiler click me\n*content*\n:::\n'}
              plugins={[[require('markdown-it-container'), 'classname', {
                validate: name => name.trim().length,
                render: (tokens, idx) => {
                  if (tokens[idx].nesting === 1) {
                    return `<div class="${tokens[idx].info.trim()}">\n`;
                  } else {
                    return '</div>\n';
                  }
                }
              }]]}
            />

TypeError: Cannot read property 'ruler' of undefined,
please help me...

The plugins for react-markdown need to be remark based.
There are quite a few https://github.com/remarkjs/remark/blob/master/doc/plugins.md
If you're interested in creating your own plugin, there is more info here: https://unifiedjs.com/learn


Libra Merchant image url in README.md return 404

Could you please check Libra Merchant image URL in README.md
source link: https://raw.githubusercontent.com/kulapio/libra-merchant/master/src/assets/img/logo/app-logo2.jpg?token=AADQJCDTUGZOUL4MN4SUD7C5ICDPC

Maybe use this link instead.

Thanks done merged


update docs/homepage: limited to 1.19 only or also compatible with newer versions?

It should work for version >= 1.19.
As long as the protocol does not change in minecraft and in CodeConnection.
It shoud work until CodeConnection was fixed i hope


Add DD logging + monitoring for the ETH approver

Logs sent to DD, monitor set to send alert if no logs when it comes in ever 2 hours. need to verify everything is working on couple cycles and first issue


500 Internal Server Error

Maybe similar to #23, submitting a ~1400 line JSON document as a guest (Firefox on Windows) yields:

Http failure response for https://prod.json.systems/v1/graphs:inferGraph?key=AIzaSyDOzK6DXqsLvq57cxa5Y7o6_TkMZ1SS3bk: 500 Internal Server Error
Maybe the file itself is too large?

Trying the sample JSON works as expected, so it sounds like that's the answer - beyond some size limit.

It's broken now, when using the sample JSON.

Hi bryanhuntesl, it's working for me. What browser are you using? Is there any debug from the Developer Console you can share (network calls, etc)?

@jackwootton it's reproducing when Array validation is set to "Allow anything", despite on JSON size:
https://prnt.sc/1ud8iux

This should be resolved with the new version.


Node requirement changed as minor version

I know it's too late for such issue but what was the point of changing required node version from 4 to 8 as minor version update (0.4.20 to 0.4.21)?
https://github.com/Leonidas-from-XIV/node-xml2js/commit/f2a3f802945ce629e2d27bb44ea36418c845d356#diff-b9cfc7f2cdf78a7f4b91a753d10865a2
Please check comment under the commit!
Minor version supposed to be used for not breaking changes, e.g. improvements or bug fixes. Node dependency is definitely breaking modifications which requires a lot of impact for everybody who uses current library. In my case current library was used as sub-sub-sub dependency with soft version like ~0.4.5.

Looks like someone else noted the issue as well just now:
https://github.com/Leonidas-from-XIV/node-xml2js/commit/f2a3f802945ce629e2d27bb44ea36418c845d356
The problem is that there is not really a way forward or back, I can't roll back the release. I will most likely have to sacrifice this evening to revert compatibility to Node 4.

@Leonidas-from-XIV  I understand you've addressed such questions before: https://github.com/Leonidas-from-XIV/node-xml2js/commit/51ce146d8555276ff4869336943d29a2f05fdcac#comments
and since it's pre-1.0, I respect the author's decision either way.

The lesson for me is to stop doing minor releases, since it is very easy to break one of the 871851 dependencies even with an extensive test suite.

You carry a great burden, sir. 😉  And now the GH interface includes the "Used by" count at the top, so you're always reminded.

Just released 0.4.22, which works again on Node 4. I don't really think there is a way to pull 0.4.20 and 0.4.21, but oh well.


Problem with marker querystring parameter when 2.5 dev app includes LocalLayer widget

The "center" parameter is working fine, but it doesn't seem to like me feeding it the same information (x,y,wkid) as a "marker" value. The marker graphic itself is created, but something (I'm struggling to trap exactly what) errors immediately afterwards.

---UPDATE---
Commenting out line 212 of MapManager.js -
MapUrlParamsHandler.postProcessUrlParams(this.urlParams, map);

seems to "fix" this (not sure what else it might break, but it's got the "marker" parameter working again for me!)


Fix permissions on release job

https://github.com/tfausak/saturn/actions/runs/5564655719/jobs/10164411587#step:3:8

Error: Resource not accessible by integration

https://github.com/svenstaro/upload-release-action/tree/a724093295a00203030b903ce60468e4b24e8d39#permissions

If you are using granular permissions in your workflow, you will need to add the contents: write permission to the token

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions

When you add the permissions key within a specific job, all actions and run commands within that job that use the GITHUB_TOKEN gain the access rights you specify.

https://github.com/svenstaro/upload-release-action/issues/70


Missing Abbreviations

[Enter feedback here]
Hello - we are on a journey to rename many of our resources to align with the Microsoft recommendations.  We have found that the following resource types are missing abbreviation suggestions:
SSH Key - microsoft.compute/sshpublickeys
Solution - microsoft.operationsmanagement/solutions
Data collection endpoint - microsoft.insights/datacollectionendpoints
DNS forwarding ruleset - microsoft.network/dnsforwardingrulesets
IP Group - microsoft.network/ipgroups
Restore Point Collection - microsoft.compute/restorepointcollections
Please add these to the list.
Thank you,
Matt

Data collection   endpoint

Data collection   endpoint
DNS forwarding ruleset

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

ID: 8c22eb33-2fde-55b4-021d-e229c857e51b
Version Independent ID: 1f767675-7d50-6ea5-b798-1323cd4a63a1
Content: Abbreviation examples for Azure resources - Cloud Adoption Framework
Content Source: docs/ready/azure-best-practices/resource-abbreviations.md
Service: caf
Sub-service: caf-methodology-ready
GitHub Login: @stephen-sumner
Microsoft Alias: ssumner

Thank you @Brownmattc for your suggestions! Working on these for you.

Added all but Solution - microsoft.operationsmanagement/solutions. Needs more research

Thank you but I do not see the new resources listed on the page.  What is the expected timing of the public view updating?

There aren't there yet. Hopefully in the next few days.


Fixes win32 input for Danish keyboard layout

I've made a small change to the input handling to match that of neovim-qt which handles characters "[", "]", "@" and "\" properly on my Danish keyboard layout which neovide didn't.
Since I am largely unfamiliar with vim and its input model, I don't know if this breaks other layouts but I would assume not since neovim-qt uses this logic. In any case I thought I would open this PR in case you want to merge it.
The change is taken from input_win32.cpp lines 28-37 and is a minor change from what already existed in the codebase.

I just noticed this seems to mess up clipboard functionality, (as is messed up in neovim-qt currently aswell) I will try to fix this then reopen the PR

Actually, after more investigation believe the clipboard "issues" are present even in the master branch so I'll reopen the PR. I'm not sure they are issues really though, it may just be that I expect the app to paste the clipboard contents on CTRL+V but that might be easier to handle through vim mappings.

After some consideration, I think the best workaround for me is to fix the broken Alt+Gr keys in my init.vim like this:
" Fix broken Alt+Gr characters
inoremap <C-M-@> @
inoremap <C-M-£> £
inoremap <C-M-]> ]
inoremap <C-M-[> [
inoremap <C-M-\> \

Ultimately from what I can tell each editor has its own implementation that accommodates some set of keyboard layouts but probably not all. Therefore it's probably best to leave the implementation as is. Feel free to include the work-around somewhere in your docs if you want, but it doesn't seem like a frequent issue, maybe we aren't many Danish vim users or something 😃

Unfortunately I don't know what the "correct" solution is :( Its a hard problem which is made harder by the windowing system.


Parse start scripts better, like npm does

We do a braindead parsing of package and/or bosco-service start scripts here:
https://github.com/tes/bosco/blob/a7bc7c484face457de1b259d77f38b5d79a9b878/src/RunWrappers/Node.js#L84-L90
npm itself uses "sh -c" to run package scripts (non-windows):
https://github.com/npm/npm/blob/master/lib/utils/lifecycle.js#L198-L207
so they have full shell parsing available to them, including, I believe, env variable definition, pipes, redirects and all sorts of other things (even subshells?).
We need to do better than we do, at least freaking out if we see pipes/redirects/env var definitions.  They should be able to define a bosco-friendly start in their bosco-service.json, although I'm not 100% sure this is supported currently.
Unfortunately, https://github.com/substack/node-shell-quote which was the only obvious lib I found, has some rough edges.  I don't really trust any implementations of this sort of stuff, there's too many gotchas.

What's the example?
The run wrappers are all about starting / stopping long running processes - in this instance node via pm2 - not executing arbitrary commands like exec does, so I'm unclear of why we want the start command to be something arbitrarily complex that pm2 probably couldn't process anyway?

I think I'm mostly worried about start scripts that define env vars like:
"start": "MY_VAR=foo NODE_SOMETHING=53 node --node_arg server.js extra args 'one more arg'",

And that quote args with spaces in them.  I admit that this is mostly paranoia to make sure things work "as expected" down the road.

Ok - I like the 'be prepared' theme, but not sure even if we did solve the parsing that PM2 would know what to do with the above - it's pretty picky in what it will run.

We can pass in script, args, and nodeArgs, for sure with pm2.start(), and by looking at the code, we should be able to pass in env as well.  We'd just have to parse it all out.


Inconsistency in ./configure --help for --disable-dtrace option

At commit abcbb0c (as automatically downloaded by julia at d15f183), ./configure --help produces an ambiguous line
--disable-dtrace        enable DTrace USDT probes. [default=yes]

That's just a bug in the help text of https://github.com/JuliaLang/libuv/blob/julia-uv0.11.26/m4/dtrace.m4, which no longer exists upstream.

Thanks - presuming that the fix will get pulled down eventually, I will close this.


Missing scheme on downstream request

Actual Behavior
On the LeastConnection balancer the downstream host scheme property is ignored and not return when Lease is called.
Fixes
???
Proposed Changes

LeastConnection predicates (filters) take the HostAndPort.Scheme property into account

Hi Ruben!
Thanks for your interest in Ocelot!
What issue is this PR related to?
Of Not related,
Could you describe your user case in details please?

@ggnaegi @RaynaldM
Could you review this piece of art please? 😄

Fixed by #2111


Update #includes in exec/ and stdexec/ based on clangd symbol uses

A few of these were build failures in my config.
This also attempts to maintiain/improve include order consistency: the system libraries were typically included last in the .hpp files

/ok to test

wow, thanks!


Allow specification of target_commitish when creating a release

Currently it's not possible to pass this parameter when creating a release, so all releases by default will tag the current master branch.

Seeing that you have made commits to the project since the last comment, I wonder if this is something you could consider adding? :)

I'm not sure how to even get this to work, it keeps returning "422 Unprocessable Entity" entity for me.
Can you list your workflow where this could be handy? I'm interested.
Anyway, the change seems sane and conforms to the GH API so I"m merging it.

Hey @aktau. Thanks for merging it in. It seems like you need to either use the full commit SHA or a branch name as the target, you cannot just use part of a commit SHA.
I made a test repository at github-release-test with 5 empty commits, and a branch called up-to-commit-4 which only holds the first 4 commits. Then I ran the following commands which gave the result you can see on the releases page:
$ export GITHUB_TOKEN="[...]"
$ export GITHUB_USER="Tenzer"
$ export GITHUB_REPO="github-release-test"
$ github-release release --tag no-parameters
$ github-release release --tag commit-2 --target 03807be7b0494b818bd94fab3339cf8a2455aa98
$ github-release release --tag commit-4-branch --target up-to-commit-4

I hope that explains it, otherwise let me know.


[geos] Bump version to 3.11.0.

Describe the pull request

What does your PR fix?
Updates GEOS version from 3.10.2 to 3.11.0

Which triplets are supported/not supported? Have you updated the CI baseline?

All. Yes,  I updated the CI baseline.

Does your PR follow the maintainer guide?

Yes

Please get failure logs here.


how many iteration to get a resonable result

i followed this project and make some changes:
1/ use different conv layers for FPN
2/ use 1 scales for [2, 3, 4] feature pyramid layers and 3 scales for[5] layer
3/ use adam instead of monent SGD
and after runing ~25k iters, i get result like this:

here is the problem
1/ it seems that there are many image with person, so may this class dominate the training and let the net converge to a sub opeimal , and how to overcome this problem?
2/ i tried  diffetent proportion of 5 loss[rpn_box, rpn_cls, rcnn_box, rcnn_cls, mask] and cant find some rules . do any one have idea how to choose this proportion
3/ do anyone get result as good as the paper present?

@LovPe  how did you train this ?

Specifications of the machine

@shamanez  GTX 1080TI 11GB

@LovPe after the training, how did you do the test on an image?

I also want to how to test on my image,please help me,thanks.

@TMTING i'm sorry that i already give up following this project, you can find my latest project here:
https://github.com/LovPe/tf_rcnn ;
In this project, I re-implement the orignal project but i think it may help .
The test code is in the inference folder. Good luck
By the way, I strongly commend you to try a Keras Implement here:
https://github.com/matterport/Mask_RCNN


add tracks and remoteDetectionAlert schemas

Tracks were added in 19dfbb82ea583aae8dd3bb5cd5946260869f5d5e and alerts in 18db4584ebdfb353b69a9feca33edc44012ee3b0. I think we're all good here.


Feature/jdk8support

Add support for JDK8

Closing the PR apparently some other changes are needed to keep compatibility with JDK8.


update subscribe worker params

allow subscribe worker to accept email and name params but only forward the email onto the mailer.

We should make people login instead of just using their display_name.


microk8s nodes losing network connection

Summary
It is regularly reported that there are network issues on our cluster. When we start looking, we indeed see that a lot of nodes end up in the "not ready" status. Commands like: "microk8s status" takes a super long time to finish...
If we look in the logs we see things like this:

Because of this we have the feeling that there is something wrong with the APIserver or the microk8s backend.
We are currently applying the workaround below to fix this problem:
On cluster node --> kubectl drain ,
On broken node --> microk8s leave,
On cluster node --> microk8s remove-node ,
On cluster node --> microk8s add-node,
On broken node --> copy output of add-node command on cluster
Of course, we want to solve the problem at the root. Unfortunately, we don't find anything in the logs right away that clearly shows us what the problem is.
We expect the nodes to have no network loss. If they do, we would expect that we should be able to clearly find the reason in one of the logs.
microk8s inspect file:
inspection-report-20221027_104957.tar.gz

Hi @ThomasMinschart,
In the logs there are a number of entries similar to:
Oct 27 10:45:15 lsp-k8s-lom04 microk8s.daemon-apiserver[415706]: Trace[517650125]: ---"Transaction committed" 852ms (10:45:00.725)

This indicates the disk the k8s datastore is using is under contentions. This is what probably causes the instability and the API server to restart and appear unavailable. Here is what you should try:

Upgrade to a newer k8s release. 1.20 is too old.
Have three control plane nodes and as many worker noders as you want (joined with the --worker flag).
Ensure that (disk) intensive workloads are not scheduled in the control plane nodes.


[docs][base] FocusTrap basic demo not focusing back to the Box in Firefox

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://mui.com/base-ui/react-focus-trap/#basics
Steps:

Go to https://mui.com/base-ui/react-focus-trap/#basics in Firefox browser
Click on "Open" to open the Box.
Scroll down and click anywhere.
It is focused back to the code snippet rather than the demo.

Current behavior 😯
After following the steps, the focus in the Firefox browser returns to the code snippet of the demo instead of the demo box.
Expected behavior 🤔
After following the steps, the focus should return to the Box demo instead of the code snippet in the Firefox browser.
Context 🔦
Originally reported in #37330, it was mistakenly assumed that two separate issues were related to each other. However, they are unrelated. I created this one specifically addressing the Focus trap problem, while keeping the original issue for the Base UI Modal demo. Please note that the issue only occurs in the Firefox browser; it works fine in Google Chrome.
Your environment 🌎

npx @mui/envinfo
Firefox Browser version 115.0.1 (64-bit)

@ZeeshanTamboli I would definitely like to jump in and help in this bug if others don't have the bandwidth for it.

@divyammadhok Thanks. Feel free to work on it.

Hey @ZeeshanTamboli Please have a look at this PR, in the description I've highlighted 2 ways we can go with this, and have implemented solution #1 as well.

We are currently working on this @ZeeshanTamboli, we'll be pushing a PR shortly.

Reason for closing - https://github.com/mui/material-ui/pull/37891#issuecomment-1642001027


[TypeScript][Bot-Solutions] Update test folder

Purpose
What is the context of this pull request? Why is it being done?
Changes
Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)
Tests
Is this covered by existing tests or new ones? If no, why not?
Feature Plan
Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.
Checklist
General

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated the appropriate tests
[ ] I have updated related documentation

I checked if manifestServer is used, apparently before this changes it was the inicializer of the server but now it doesn't have much relevance because we eliminated all the files related to the manifest configuration and this is the last one in the test folder.


No 'decortion options'

Hi
You say "This example shows one of decoration options with all supported social networks" and follow it with a beautiful picture showing buttons and icons.
However, there is no documentation on this feature anywhere and I can see nothing in the code that would generate it. How do you activate this 'option' please?
Thank you.
Frankie

See after this picture. HTML code do this, copy and be happy!

No it doesn't. It creates it minus the beautiful icons I asked about.
See here: WHAT YOUR CODE ACTUALLY PRODUCES
Do you have any code that makes the picture icons you advertised?
Thank you.

You may download icons here: http://goodshare.ru/download/fontello-goodshare.zip


Add process to remove value in case in query

Add process to remove value in case in query.

Thanks for fixing issue!
But I think a little redundant this...
I implemented at https://github.com/knocknote/rapidash/pull/8 .
Please see it .

Resolve by #8


Icons missing from custom view title bars in latest Insiders

Issue Type: Bug
In the latest Insides build, the icons for commands in the view title bars are missing. Clicking where the icon was still activates the command, so it's an issue of the icons not being visible. It could be a regression due to #114438 which seems related.


Fonts for monochrome displays

Hi, I'm following up on the discussion about fonts here: #125. For use on monochrome LCDs we need pixel perfect fonts that don't produce artefacts in small sizes. @upbeat27 provided modified version of unscii-8-thin font, which can be converted to perfect 1bpp bitmap using online converter. I tried to edit some other fonts but with little success :D. I found, however, many bitmap fonts in .bdf format that can easily be converted to source code. Here's preview of fonts I got working so far:

Second and third ones are converted using online converter from vector fonts(unscii and synchronizer). The rest I converted from fonts available in u8g2 library, that has hundreds of bitmap fonts for graphic LCDs. Library is open source, but fonts are available under different licences (see here). Here is another font with about 360 symbols on 12x12 grid (source):

I think there could be some collection of such fonts available in littlevGL for people using low resolution display. Or maybe add support for bitmap fonts in online converter ? Anyway, here are converted fonts:
fonts.zip

Hi,
It looks very well, thank you very much!
I wonder how did you forced the converter to generate fix width letters. Now it should store the in the smallest possible place and trim every empty columns on the left ad right.  (for example in f_6x10.c)
I think it would be great to add at least one pixel font (e.g. in 7 or 8 px height) with ASCII and Symbol characters as built in font. It would make much easier to start the development  on monochrome displays. What do you think?

I actually made my own converter for .bdf fonts. First i tried to convert them into vector fonts and use your online converter but it didn't work. Fonts I used have glyphs with constant size (e.g. 6x10), that's why all characters are same width.
Yes, it would nice to have some font in library as starting point. Unscii is probably good candidate as it is public domain. The symbol font (siji) however, is GPL and as far as i understand, it's not compatible with MIT licence. So maybe if someone could find similar font with different licence it could be used.

Okay, if you find it useful I will share the php files in lv_utils repo.


[beta] Failed to send Enter press

Hello,
when I add auth sufficient pam_howdy.so into /etc/pam.d/polkit-1 and use some program requiring admin rights (e.g. pamac-gtk), the authentication window shows up, the IR camera diodes are flashing, it seems the face is recognized, but the authentication window writes an Failed to send Enter press, waiting for user to press it insted error and it is not possible to authenticate or send the Enter press via keyboard. Only cancel works.

Is there any way to send the Enter press which I don't know? Or is it possible to tweak something in config to make it work?

I've searched for similar issues already, and my issue has not been reported yet.
Linux distribution (if applicable): Manjaro Linux x86_64, GNOME 43.1, wayland
Howdy version (sudo howdy version): Howdy 3.0.0 BETA

PS: The auth sufficient pam_howdy.so works as expected for /etc/pam.d/sudo and /etc/pam.d/system-local-login. Thank you.

Thanks for looking at the beta, have you tried pressing any key and then enter? The dialog might not allow empty passwords

@boltgolt This issue is kind of sorted for me. Shall I close this issue or leave it open until it is properly handled with pam_howdy.so? Or is it impossible to implement for now?


Wolfssl code back

Summary
Add wolfSSL build files, allows wolfSSL to be cloned in and used quickly
Impact
Allows wolfSSL to be built into NuttX
Testing
Tested using wolfSSL tests in the NuttX simulator and on the STM32F746ZG, STM32H753 and STM32L552ZE

I've squashed the readme commits, removed the old zip and curl logic since we now clone directly from git and removed the platform specific checks in user_settings.h

yeah sorry, should have just made it simple, squashed again

@jpbland1 but I don't see the download/clone in Makefile.

ping @jpbland1

Finally got back to this, I didn't realize that getting the source code was supposed to be an automated step in the Makefile, I've also added the GPL allowed condition to the Kconfig, let me know if something else is wrong.

@jpbland1 please fix the below nxstyle warning:
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:1:1: error: Missing file header comment block
Warning: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:1:1: warning: #include outside of 'Included Files' section
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:31:1: error: Missing blank line after comment
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:32:0: error: Missing space after opening C comment
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:32:24: error: Missing space before closing C comment

and rebase to the last mainline.

@jpbland1 please fix the below nxstyle warning:
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:1:1: error: Missing file header comment block
Warning: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:1:1: warning: #include outside of 'Included Files' section
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:31:1: error: Missing blank line after comment
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:32:0: error: Missing space after opening C comment
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/crypto/wolfssl/user_settings.h:32:24: error: Missing space before closing C comment

and rebase to the last mainline.

@jpbland1 just add the missing sections separation in the code and squash it and I will merge your PR!


Experience page bug and dark mode :)

I have done all the changes as discussed in slack
@smaranjitghose Have a look.

I have done all the changes as discussed in slack
@smaranjitghose Have a look.

This looks 💖

In your next PR, try inserting the data using a expereince.js file where we have three variables, work, volunteering and mentoring for the corresponding sections


docs(examples): add InstantSearch example

This adds a demo for Autocomplete with InstantSearch.

See live →
Features

Recent searches. This add user journey data to the search.
Query Suggestions in current InstantSearch category. This acts as a department store.
Query Suggestions outside of the current InstantSearch category. This allows to broaden the scope of the current search.
Instant autocomplete results, debounced InstantSearch results. This avoids too many UI flashes on the screen. The search page results are still instant after interactions with buttons from the autocomplete or filters.
Keyboard and mouse accessibility. All suggestions are links that respect accessibility standards when hitting ⌘ Enter or clicking with ⌘.

Implementation

Virtual search box. We need to use connectSearchBox with an empty implementation for the query UI state to be understood by InstantSearch.
Rely on the default historyRouter to compute the search URL. All Autocomplete suggestions link to search URLs for accessibility features. Those are computed by the router plugged to InstantSearch.
Each interaction with Autocomplete calls InstantSearch's setUiState.

What's next
This demo will be the reference for a new guide on the Algolia website.

There seems to be a batching issue:

These two queries should be bundled together into a multi query but they aren't.

@sarahdayan There's 1 query coming from the Autocomplete suggestions and 1 query coming from the InstantSearch results.

@sarahdayan Firefox is likely not the culprit here. Since we use the same search client for Autocomplete and InstantSearch, we append both Autocomplete and InstantSearch user agents to the client.


how can I get a bigger image without blur ?

Hi, paulovap .
I'm using your project  in my free software.
https://sourceforge.net/projects/ccseer/
Now, I get a problem when I was trying to read a bigger image .
QPdfium::page()->data()->image(   scale   );
The issue is the parameter 'scale' .  When I set a  num bigger than 1 , the result is not very clear.
what should I do.

Hey @DreamerCorey,
when scale is 1 it means with will render the image the at the natural width and height of the pdf page. (you can check it QPdfiumPage::width and QPdfiumPage::height).
If you want to render the image in a higher quality you can increase the scale, let's say to 2. So the rendered image will have  QPdfiumPage::width()*2 width and QPdfiumPage::height()*2 height. If that does not happen, then there is a bug.

The code is ok, actually.
The problem is , the quality of image has changed  if you scaled it double size.

Is it possible to make available the pdf file that you are using so I can see what's wrong? Maybe there is an embedded bitmap in it?

![Uploading I9GN_E4]_G)T(LLPZGS%OU4.jpg…]()
I just tried a lot.
I might be wrong  about the the quality of scaled image . Sorry... :cry:
But can I make image more sharp ?

![Uploading I9GN_E4]_G)T(LLPZGS%OU4.jpg…]()

Ummm...
Thx for your help . :smile:

So, what was the problem?

It is different if you look at the pic.
I want make the pdf render more sharp  and more clear.

Hi @DreamerCorey ,
Just tested here rendering the first page with scale 1.0 and 3.0. It worked fine.

Ah.... I'll test it on Windows tomorrow.
Thanks, Thank you very much.

Heres the code that generated those two images.
auto m_pdfium  = new QPdfium("iso.pdf");
QImage img;
img = m_pdfium->page(0).data()->image(3);
img.save(QString("test_3.jpg"), "jpg", 100);
img = m_pdfium->page(0).data()->image(1);
img.save(QString("test_1.jpg"), "jpg", 100);

It turns out  it's the platform differences.  I tested the code on os x , that's why image is blur


EC2 discovery

Hi and thanks for open sourcing such a useful tool!
Not sure if this is the correct channel to ask this, but we are trying to set this up and seems like it fails to detect our nodes.
We are using EC2 discovery, am I right that  scrutineer assumes a different discovery mode?
Is there a way to explicitly specify hostname/port of an elasticSearch node instead of relying on discovery based on cluster name?
thanks in advance!

As ES's discovery methods have changed significantly over the years, I'm closing this issue.  Scrutineer now requires a least a set of seed nodes as part of initial configuration to be able to join the cluster.


Add ravenPresend and ravenMessage events

Hello,
As we already have ravenHandle event when handling an exception, ravenSuccess and ravenFailure events for post-submitting data, I propose to add

ravenMessage when capturing a message
ravenPresend just before sending data to Sentry server.

This improves raven flexibility for the developer as it allows him to tweak the data.

So, we're definitely going to change the event API and get away from DOM events. I'm cool with adding these events; it's just a question of whether it makes sense for us to merge this ahead of those API changes, and wait until the API is done.
It will likely look like:
Raven.on('send', function () { ... }); // or presend/beforesend
Raven.on('exception', function () { ... });
Raven.on('message', function () { ... });
Raven.on('success', function () { ... });
Raven.on('failure', function () { ... });

Wow, this is definitely a better API. When these changes are supposed to land in a production-ready version?

When these changes are supposed to land in a production-ready version?

See: #521, which is targeted against a 3.0 release. We'll probably release 3.0 within a week (hopefully). After which we can re-evaluate this PR.
As an aside, if you need to use presend today, you get almost the same thing out of Raven.setDataCallback.
Re: the message event – was there a reason you wanted to bind to message events? Or were you just filling out the API for consistency?

I've added a discussion about changing the events API: #524
@romainneutron – appreciate your thoughts there

Sorry for the late reply. This sounds good. I wanted to use message event to customize message based on some pattern before sending them to Sentry.
Your proposal looks good, can't wait to test this


[Dynamic Region] pitr checkpoint ts lag keep increasing after simulate tikv network partition for 50m and recover

Bug Report

What version of TiKV are you using?
./tikv-server -V
TiKV
Release Version:   7.3.0-alpha
Edition:           Community
Git Commit Hash:   bd11fb033e85512f0dd20cffaf1bcefc5e8c0bd6
Git Commit Branch: heads/refs/tags/v7.3.0-alpha
UTC Build Time:    2023-06-27 09:12:24
Rust Version:      rustc 1.67.0-nightly (96ddd32c4 2022-11-14)
Enable Features:   pprof-fp jemalloc mem-profiling portable sse test-engine-kv-rocksdb test-engine-raft-raft-engine cloud-aws cloud-gcp cloud-azure
Profile:           dist_release
2023-06-28T19:59:01.981+0800
What operating system and CPU are you using?
8c/32g
Steps to reproduce
1、start pitr task
2、run tpcc
3、simulate tikv network partition for 50m and recover
What did you expect?
pitr checkpoint ts lag is less than 5min
What did happened?
pitr checkpoint ts lag keep increasing after simulate tikv network partition for 50m and recover

/type bug
/severity critical
/assign 3pointer

Currently, we can see pitr checkpoint lag increased due to resolved ts lag increased. but the reason why resolved ts increased still no clue.


MOUSEINPUT is of the wrong type

Summary
In version 0.51, windows::Win32::UI::Input::KeyboardAndMouse::MOUSEINPUT.mouseData was of type i32. Starting in version 0.52, it is of type u32. According to the documentation, the type is DWORD. However it also states:

If dwFlags contains MOUSEEVENTF_WHEEL, then mouseData specifies the amount of wheel movement. A positive value indicates that the wheel was rotated forward, away from the user; a negative value indicates that the wheel was rotated backward, toward the user. One wheel click is defined as WHEEL_DELTA, which is 120.

(source).
I believe the type should get changed back to i32.
Crate manifest
No response
Crate code
No response

This is already fixed upstream, we just haven't brought in metadata yet due to some blocking metadata bugs. https://github.com/microsoft/win32metadata/issues/1689

Looks like the metadata has already been fixed and we're just waiting on another update. I'll close this issue as there isn't anything more to do here.

Ok, thank you


chore: fix some typos in comments

Motivation:
fix some typos in comments
Modification:
fix some typos in comments
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

@welfuture thanks a lot!


added mysql support and flask support

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

The default database (DynamoDB) was changed to MySQL.
A Flask service was introduced to we can run the script by invoking a URL.


Connect MPSTensor representation with PEPS optimization

The major change of this PR is a function
function batch_tensor_contraction(tensortype, networks::Vector{Vector{ITensor}}, vars...; kwargs...)

where tensortype can be MPSTensor, this will make all intermediate tensors the MPSTensor type, and contract with approximations.
In MPSTensor, we currently have interfaces for two contraction rules, general_mps and singleindex_mps:
general_mps: each MPS siteindex can be a list of several indices. We can think about how to implement general MPO-MPO multiplicaitions under this rule.
singleindex_mps: each MPS siteindex is restricted to only one index. This is the algorithm implemented in https://arxiv.org/pdf/1912.03014.pdf.
Most of the PEPS optimization code doesn't need to change. We only need to change
function batch_tensor_contraction(networks::Vector{Vector{ITensor}}, vars...)

to
function batch_tensor_contraction(MPSTensor, networks::Vector{Vector{ITensor}}, vars...; kwargs...)

to perform approximate contractions.

@mtfishman let me know if you have other comments on this.

Thanks for the improvements Linjian, sorry for the slow response. The design now looks good, feel free to merge.
What is the status of using this as part of doing PEPS optimization?

Thanks for the improvements Linjian, sorry for the slow response. The design now looks good, feel free to merge.
What is the status of using this as part of doing PEPS optimization?

We are still far from getting the large scale PEPS optimization working, since I haven't implemented the approximate contraction rules for MPSTensors yet.
Currently the contraction rules for MPSTensors are placeholders to get the code working. For a 3 x 3 PEPS the optimization works, as I have shown in the added test.

That's what I thought, just wanted to double check. Thanks!


pretrained models

Thanks for your awesome and creative work!
I read your paper and try to reproduction the performance. but via diffusion method, it will be convenient and helpful if you could provide the pretrained model, Thanks.

Hi!
Thanks for your interest in the project! I will provide the pretrained models + updated code in approx. 4-6 weeks, as well as an improved version of the paper and method, as I await reviews for the work.

Hi @Chromer163 The pretrained models and code to run them are now available


Cannot deal with UTF8 sender names

    ** (FunctionClauseError) no function clause matching in Mailman.Render.normalize_addresses/1
        lib/mailman/render.ex:110: Mailman.Render.normalize_addresses("Stian Håklev<EMAIL_ADDRESS>

Thanks for reporting - I should have some time soon to get it fixed.

This is not a bug, but wrong parameter, normalize_addresses expects a list, not a string:
iex(2)> Mailman.Render.normalize_addresses(["Stian Håklev<EMAIL_ADDRESS>["Stian Håklev<EMAIL_ADDRESS>Cheers


Don't Deploy Forks

This patch makes the deployment run only on the main Tobira repository,
not on any fork. Running the deployment on forks would fail anyway since
they don't have the repository secret configured. This would cause the
whole CI workflow to fail.

Though... lots of deploy-related steps are still executed right now for forks. Those are kind of useless then. Shouldn't those have an if as well?

Though... lots of deploy-related steps are still executed right now for forks. Those are kind of useless then. Shouldn't those have an if as well?

I didn't because I thought you could thus verify that those work before creating a pull request but I do not have a strong opinion on this.

I didn't because I thought you could thus verify that those work before creating a pull request but I do not have a strong opinion on this.

Ah okay, that makes sense. I just checked:

"Prepare binary and demo data for deployment" took 0s
"Prepare deploy key" took 2s
"install ansible postgres extension" took 38s

What do you think about adding the if to the ansible step to make CI finish faster? The ansible step is rather unlikely to fail due to code changes.

Ah okay, that makes sense. I just checked:

"Prepare binary and demo data for deployment" took 0s
"Prepare deploy key" took 2s
"install ansible postgres extension" took 38s

What do you think about adding the if to the ansible step to make CI finish faster? The ansible step is rather unlikely to fail due to code changes.

What do you think about adding the if to the ansible step to make CI finish faster? The ansible step is rather unlikely to fail due to code changes.

That sounds reasonable. On it…

What do you think about adding the if to the ansible step to make CI finish faster? The ansible step is rather unlikely to fail due to code changes.

That sounds reasonable. On it…

Done. See #143

Done. See #143


Feature: safe-react integration

This PR allows a safe-react integration project. Now the service returns 202 statuses

Thanks @apanizo! I will take that commits into my branch and merge it to develop


Unable to install on new RHEL 8 with pipenv

Hello,
I use pipenv to manage the dependency of my application.
when running pipenv install cvpysdk, I got the following error.
[evro@lpotladm001a gcloudadm]$ pipenv install cvpysdk
Loading .env environment variables...
Installing cvpysdk...
Error:  An error occurred while installing cvpysdk!
Error text: Looking in indexes: https://repo.gcloud.belgium.be/artifactory/api/pypi/common-pypi-central-remote/simple
Collecting cvpysdk
  Using cached https://repo.gcloud.belgium.be/artifactory/api/pypi/common-pypi-central-remote/packages/packages/d5/98/3f8e69fff5620f5be26fb8edd92f0325f3e41eabdc42d60bd0badc81dd2f/cvpysdk-11.28-py2.py3-none-any.whl (1.1 MB)
Collecting future
  Using cached https://repo.gcloud.belgium.be/artifactory/api/pypi/common-pypi-central-remote/packages/packages/45/0b/38b06fd9b92dc2b68d58b75f900e97884c45bedd2ff83203d933cf5851c9/future-0.18.2.tar.gz (829 kB)
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'error'

  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [10 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 11, in <module>
        File "/usr/lib64/python3.8/tokenize.py", line 29, in <module>
          import collections
        File "/usr/lib64/python3.8/collections/__init__.py", line 27, in <module>
          from reprlib import recursive_repr as _recursive_repr
        File "/tmp/pip-install-40o3kdbv/future_bdd6c6e71cac47f99c5f170093bbbf26/src/reprlib/__init__.py", line 7, in <module>
          raise ImportError('This package should not be accessible on Python 3. '
      ImportError: This package should not be accessible on Python 3. Either you are trying to run from the python-future src folder or your installation of python-future is corrupted.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

This is likely caused by a bug in cvpysdk. Report this to its maintainers.
✘ Installation Failed

Strangely enough, the command pip3 install cvpysdk succeeds.
Python and pipenv version used:
[evro@lpotladm001a gcloudadm]$ python --version
Python 3.8.12
[evro@lpotladm001a gcloudadm]$ pipenv --version
pipenv, version 2022.10.11

Any reason why this could fails with pipenv and not with pip?

Hi @montaropdf
This looks more like an environment issue regarding the future module. I was not able to recreate it on my RHEL 8
Can you please refer this similar issue while installing future dependency and see if the thread/resolution helps
Let us know if it didn't help

Hi @kevinjojeejoseph,
Indeed unsetting the PYTHONPATH did the trick during installation of the package. afterward, pipenv does not complain about the future module.
Thanks for the help.


Making attachments paths in email samples absolute

Description
There was an error running the samples in the communication email live test pipeline. This was because the samples were being run from the root directory rather than the samples directory. These samples were modified to use an absolute path for the attachments being read to avoid this issue.

/azp run python - azure-communication-email - tests


add a background image for UI

add a background image for UI
example: https://github.com/sebiboga/humans-txt/blob/main/assets/humans.png

@arashjafari can you add a background image in UI?
I added one here in this task as example

@sebiboga Yes, I will do


Add a row decoration to RenderTable

This will let us draw in-row bottom borders and row-wide backgrounds,
both of which are necessary for Material data tables.

@abarth

hm, actually, for the background I'm going to need per-row decorations. Let me make this a list instead...

ok PTAL, it's now a list rather than a single decoration.

LGTM


Reshade - black screen when d3d9 is set to native

When d3d9 is set to native in winecfg, ReShade main window is completely black

So I have to set d3d9 to builtin for ReShade_Setup_4.4.2.exe to be able to avoid the black screen.

I know that it may be a dull issue to report, but I reported it here any way for anyone having the same issue.

Have you installed the Microsoft d3dcompiler dlls?

Yes.

Doesnt look like it from the winecfg frog

It is a custom override for ReShade_Setup_4.4.2 only. It is not the general winecfg settings.

You need it for more than the setup.

Works fine for me here:

Hm, ok, the setup is a lil' janky at times, ngl

Why is this rendered in D3D9... :v

Ok I've improved this slightly but it relies on partial presentation which is a bit of a bitch to impl


scale with screen module is not accurate cause dpi is not accounted for

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Is this still an issue?


Release v2.6.0

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

Comparison is base (398bb08) 60.15% compared to head (b51aed9) 60.46%.

:exclamation: Current head b51aed9 differs from pull request most recent head b63b027. Consider uploading reports for the commit b63b027 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #346      +/-   ##
==========================================
+ Coverage   60.15%   60.46%   +0.30%
==========================================
  Files           3        3
  Lines         128      129       +1
  Branches       30       30
==========================================
+ Hits           77       78       +1
  Misses         46       46
  Partials        5        5

Files
Coverage Δ

src/common.ts
38.09% <100.00%> (+0.99%)
:arrow_up:

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


Brace initialization of Color

Use brace initialization of Color objects.
Remove explicit full alpha channels, and use named color constants where possible.

Hmm, missed a few, where the variable name came between Color and the initialization values.


[SPARK-22668][SQL] Ensure no global variables in arguments of method split by CodegenContext.splitExpressions()

What changes were proposed in this pull request?
Passing global variables to the split method is dangerous, as any mutating to it is ignored and may lead to unexpected behavior.
To prevent this, one approach is to make sure no expression would output global variables: Localizing lifetime of mutable states in expressions.
Another approach is, when calling ctx.splitExpression, make sure we don't use children's output as parameter names.
Approach 1 is actually hard to do, as we need to check all expressions and operators that support whole-stage codegen. Approach 2 is easier as the callers of ctx.splitExpressions are not too many.
Besides, approach 2 is more flexible, as children's output may be other stuff that can't be parameter name: literal, inlined statement(a + 1), etc.
close https://github.com/apache/spark/pull/19865
close https://github.com/apache/spark/pull/19938
How was this patch tested?
existing tests

cc @kiszk @viirya @mgaido91 @maropu @rednaxelafx @gatorsmile

Test build #85112 has started for PR 20021 at commit aadb838.

Does this PR just check whether the condition of approach 2 occurs or not? If approach 2 does not replace with a temporary variable, assertion may occur.
Am I wrong?

Test build #85114 has started for PR 20021 at commit 9d22975.

Yea, I actually manually checked all the caller side of ctx.splitExpressions and didn't find any issue, adding an assert helps me prove it.
I also reverted some changes that tried to localize global variables as they are not needed now.

Test build #85114 has finished for PR 20021 at commit 9d22975.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

I see. I agree that to add assert helps all of developers.
My question is what is a rule to decide whether a mutable state should be localized or not? I think that we have to ensure caller of ctx.splitExpression does not get any EvalCode.

Test build #85120 has started for PR 20021 at commit 18ec598.

My question is what is a rule to decide whether a mutable state should be localized or not

There is no rule as we don't need to localize global variables anymore.

Test build #85121 has started for PR 20021 at commit f1afb92.

Here is a fix that avoids to pass a global variable to ctx.splitExpressions. Before this PR, this error occurs. I understand that my fix is not localization, but one of approaches does not pass a global variable to successor steps like localization. My lesson learned is to pass a global variable accidentally happens thru deep call stack.
Would it be possible to share finding from your manual check at all the caller side of ctx.splitExpressions. Why can we ensure any global variables are not passed to ctx.splitExpressions? I think that it would be good to put the rational into the comment.

Test build #85112 has finished for PR 20021 at commit aadb838.

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

Merged build finished. Test PASSed.

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

Test build #85120 has finished for PR 20021 at commit 18ec598.

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

Merged build finished. Test PASSed.

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

Test build #85121 has finished for PR 20021 at commit f1afb92.

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

Merged build finished. Test PASSed.

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

Why can we ensure any global variables are not passed to ctx.splitExpressions?

You can also do this check. Look at the arguments parameter, all the caller sides are using local variables, so we can be sure that no global variables would appear as parameters in the split method.

Test build #85150 has started for PR 20021 at commit 3d44195.

I checked some call site. Here is one example that arguments has ev.value instead of local variable.

Test build #85150 has finished for PR 20021 at commit 3d44195.

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

Merged build finished. Test FAILed.

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

Here is another example. This is complicated.
result is passed to arguments of ctx.splitExpressions in genHashForStruct. The result may get ev.value in a certain call path that is started from here
As this example points out, while arguments in ctx.splitExpressions has local variable, a local variable may have ev.value in some calling context. It does not seem to be easy to ensure no global variable in arguments.
Thus, IMHO, it would be good to conbine localization and adding assert. Then, we declare a contract no global variable in arguments in comment.
WDYT?
// hash.scala, line 277
override def doGenCode(ctx, ev): ExprCode = {
  ...
    computeHash(childGen.value, child.dataType, ev.value, ctx)
  ...
}

// hash.scala, line 448
protected def computeHash(input, dataType, result, ctx): String =
  computeHashWithTailRec(input, dataType, result, ctx)

// hash.scala, line 414
private def computeHashWithTailRec(input, dataType, result, ctx): String = dataType match {
  ...
    case StructType(fields) => genHashForStruct(ctx, input, result, fields)
}

//hash.scala, line 402
protected def genHashForStruct(ctx, input, result, fields) {
  ...
  ctx.splitExpressions(
    ...
    arguments = Seq("InternalRow" -> input, hashResultType -> result),
    ...
}

Jenkins, retest this please

Test build #85164 has started for PR 20021 at commit 3d44195.

Test build #85164 has finished for PR 20021 at commit 3d44195.

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

Merged build finished. Test PASSed.

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

I checked some call sites. Here is one example that extraArguments has ev.value instead of local variable.

Hey, ev.value is not from children, it's the output of the current expression, which we can make sure it's local variable, e.g. https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/hash.scala#L296

Oh, you are right. I misunderstood. After our optimizations, output is also a part of arguments. Let me check others again.

Honestly, I liked very much doing the test only for testing and not throwing an exception in production. IMHO it is an overkill to throw an exception in production and in the remote case that we happen to forget one place where this check can throw the exception, but it is not an issue, as it is perfectly possible, this would also cause a regression.
Thus, honestly I am strongly against this solution.

Well, I proposed to check it only for tests at the beginning, but I don't have a strong preference now as the new approach I took can guarantee that no place would violate it, by looking at all the caller sides of ctx.splitExpressions.
Anyway only checking it in tests is safer, WDYT @viirya ?

Ok. I'm fine with only checking it in tests.

Test build #85240 has started for PR 20021 at commit 900f246.

Test build #85240 has finished for PR 20021 at commit 900f246.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

retest this please.

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

Test build #85245 has started for PR 20021 at commit 900f246.

LGTM

LGTM too, thanks!

Test build #85245 has finished for PR 20021 at commit 900f246.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #85262 has finished for PR 20021 at commit 900f246.

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

Merged build finished. Test PASSed.

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

thanks, merging to master!


[Mobile] Dark Overlay when using Search Option

The first picture shows the app in its normal state.
In the second image, the search function is active and the entire area is darkened.
Why?
The last picture shows how it looks in dark mode. You can't see anything anymore.
can you fix this please?

Clone this package and change it

This issue has been solved, you can now customize the dialog barrier color, the dialog widget and you even have a callback for showing custom dialog