Operator for Reaction Moments

Before submitting the issue

[X] I have checked for Compatibility issues
[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
Hi,
Is an operator for reaction moments (as for reaction force) available?
https://dpf.docs.pyansys.com/version/0.11/api/ansys.dpf.core.operators.result.reaction_force.html
Steps To Reproduce
Nothing
Which Operating System causes the issue?
Windows
Which DPF/Ansys version are you using?
Ansys 2024 R1
Which Python version causes the issue?
3.12
Installed packages
....

I also can't find any way to extract the reaction force in pydpf, but I can use pymapdl combined with APDL FSUM command and *GET command to extract the reaction force. Hopefully this will help you !

Hi @js4561207,
For the reaction force moment, have you tried result.raw_reaction_force?
Also, to extract the reaction force, you should be able to use result.reaction_force, does it not work?
Can you try to load your results in a Model and print the available results as shown in this example.

Results
This is pydpf results and I need to scope to nodes that are constrained.

If i try to scope to all nodes, the results are not correct. (Ie. the node 118341 don't have reaction forces in the xlsx file)
reaction.xlsx

This is pymapdl and apdl results, which extracts all reaction forces at all constrained nodes.

The results of these three are slightly different because of graphics control in MAPDL.
(Full graphics or Power graphics)
System and environments
Windows
python3.9.16
pyansys-dpf-core                        0.11.0
Ansys 2023R1

Hi,
my initial question is related to nodes with rotational DOFs (e.g. for shell elements) that are constrained and then to get the reaction moments. Using operator reaction_force will give the forces only ...

@GitHubUser120282
Using raw_reaction_forces.

BUT the results is still different with APDL's output

Hi @js4561207,
sorry for the late reply but despite our best efforts we could not reproduce the behavior you communicated above.
Would you happen to have a small sample result file for us to reproduce?


Escaping '__' coming from python methods

Python specials methods like constructors starts with '__' which is a bold tag for markdown, so this tag is now escaped.

Thanks

This commit has been added to the following release
https://github.com/codejamninja/sphinx-markdown-builder/releases/tag/0.5.3


[CAS] Cache symbol graph outputs

The symbol graph output from a module can contain an arbitrary number of files, depending on what extensions it contains, so cache a list of symbol graph files with their base name and contents so that they can be replayed.
rdar://140286819

@swift-ci please smoke test


Request for profile Create AbhiJain.json

sure

but I didn't see my profile in linkfree website?

but I didn't see my profile in linkfree website?

Hey it's there, you can check it out - https://linkfree.eddiehub.io/abhijain2003


Add Apple Silicon builds

@sawyerbfuller mentions in https://github.com/python-control/python-control/pull/670#issue-1048202627 that he can't get Slycot on his Mac M1. The files on conda-forge are over 1 year old. I guess the procedure described here needs to be executed:
https://conda-forge.org/blog/posts/2020-10-29-macos-arm64/

FWIW, I have slycot working on my Mac M1.  I'm pretty sure I just did a standard install (via conda) and it worked (presumably via Rosetta).

Tried it again with no luck. Standard 64-bit Anaconda (presumably running under Rosetta2). For now I'd be happy with just running under Rosetta2. After installing Anaconda using the graphical installer, the following happens:
(base) minster@Sawyers-MacBook-Pro-2 ~ % conda install -c conda-forge slycot control
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: /

(.. wait for awhile lots of conflicts reported, after which control and slycot still don't import)

@sawyerbfuller Slycot has now been rebuilt to include an ARM version.  Can you try it out and see if it is working now?

Hi @murrayrm thanks for letting me know. Installation of slycot seems to have worked using the installation shell script provided at the top of the page here: https://conda-forge.org/blog/posts/2020-10-29-macos-arm64/ and subsequent installation using conda-forge.
Related: control doesn't install now through conda-forge. Perhaps something isn't configured correctly?  (see below)
(base) minster@Sawyers-MacBook-Pro-2 ~ % conda install -c conda-forge control
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.

PackagesNotFoundError: The following packages are not available from current channels:

  - control

Current channels:

  - https://conda.anaconda.org/conda-forge/osx-arm64
  - https://conda.anaconda.org/conda-forge/noarch

To search for alternate channels that may provide the conda package you're
looking for, navigate to

    https://anaconda.org

and use the search bar at the top of the page.```

Ah the control-feedstock is missing the osx migrator PR.

https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/2366

..,which is actually dumb because control is pure python and should be a noarch package ...

@sawyerbfuller conda-forge now has a platform independent control package:
https://anaconda.org/conda-forge/control/files
Does that work for you?

Installs! There is a wrinkle:
Python 3.9.7 | packaged by conda-forge | (default, Sep 29 2021, 19:24:02)
[Clang 11.1.0 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import control
zsh: segmentation fault  python

Here is printout from the install procedure:
(base) minster@Sawyers-MacBook-Pro-2 ~ % conda install -c conda-forge control
Collecting package metadata (current_repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /Users/minster/miniforge3

  added / updated specs:
    - control

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    control-0.9.1              |     pyhd8ed1ab_2         262 KB  conda-forge
    scipy-1.7.3                |   py39h5060c3b_0        19.4 MB  conda-forge
    ------------------------------------------------------------
                                           Total:        19.6 MB

The following NEW packages will be INSTALLED:

  control            conda-forge/noarch::control-0.9.1-pyhd8ed1ab_2
  scipy              conda-forge/osx-arm64::scipy-1.7.3-py39h5060c3b_0

Does the segmentation fault also occur with import slycot or import scipy ?

Good point. Looks like it's slycot:
Python 3.9.7 | packaged by conda-forge | (default, Sep 29 2021, 19:24:02)
[Clang 11.1.0 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import scipy
>>> import matplotlib
>>> import numpy
>>> import slycot
zsh: segmentation fault  python

Installation of slycot seems to have worked using the installation shell script provided at the top of the page here:

I am not sure if I understand this statement. Did you compile it yourself or did you use the package from conda-forge?

@bnavigator my apologies for the incorrect wording.

installation of slycot using conda-forge completed. My understanding of this is that this was a binary package. No compilation
I had not yet tried to import it when I wrote that message, so it did not actually “work”

Hmm, seems like the conda-forge compiled slycot package is broken then.
For testing, I would recommend cleaning your (base) environment and check installations in a dedicated conda env first.
Are you able to step through a debugger or inspect the stack trace at the segmentation fault?
https://lldb.llvm.org/use/tutorial.html

I tried installing on my Mac (MacOS 12, M1 Macbook Air) and it worked OK, but I can't tell if I am running the arm64 version of the non-ARM version (with Rosetta2).  Specific sequence that worked:
conda create -n test python=3.9
conda activate test
conda install -c conda-forge control
python -c "import control; print(control.__version__)"
0.9.1

For me, there is a pretty long delay (15 seconds) before the version prints, which is often an indication that Rosetta is doing translation.  Second run of printing the version is instantaneous.

@murrayrm the delay would come because control imports slycot. The second call then uses the already translated slycot. Can you try without importing control first?
I guess you would see the used package in the name of the egg-info library. Check slycot.__path__ and look for the egg-info next to it.

arm64: slycot-<IP_ADDRESS>-py3.9-macosx-11.0-arm64.egg-info
x86_64: slycot-<IP_ADDRESS>-py3.9-macosx-10.9-x86_64.egg-info

Also (on my linux system):
[ben@skylab:~/tmp]% file slycot-mac_x86/_wrapper.cpython-39-darwin.so                                                            [0]
slycot-mac_x86/_wrapper.cpython-39-darwin.so: Mach-O 64-bit x86_64 bundle, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL>
[ben@skylab:~/tmp]% file slycot-mac_arm64/_wrapper.cpython-39-darwin.so                                                          [0]
slycot-mac_arm64/_wrapper.cpython-39-darwin.so: Mach-O 64-bit arm64 dynamically linked shared library, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|NO_REEXPORTED_DYLIBS>

Indeed, I am getting slycot-<IP_ADDRESS>-py3.9-macosx-10.9-x86_64.egg-info

@bnavigator Debugging slycot binary is probably beyond my abilities/time availability. I will keep fiddling to see if I can get things to work under Rosetta2 (no idea why that doesn't work for me)

Update: I am back to running anaconda using Rosetta2 (osx-64). When I created a new environment, slycot and control installed their own dependencies and ran successfully.
So it looks like the problem is a conflict between the default osx-64 anaconda distribution and slycot.
One question: right now control depends on installing slycot (see below) but given that slycot has more constrained depencies, and a lot of control can work without it, have we considered removing it as a dependency?
(base) minster@Sawyers-MacBook-Pro-2 ~ % conda search control --info
...
control 0.9.1 py38h50d1736_1
----------------------------
file name   : control-0.9.1-py38h50d1736_1.tar.bz2
name        : control
version     : 0.9.1
build       : py38h50d1736_1
build number: 1
size        : 542 KB
license     : BSD-3-Clause
subdir      : osx-64
url         : https://conda.anaconda.org/conda-forge/osx-64/control-0.9.1-py38h50d1736_1.tar.bz2
md5         : 85438141e5197f04a5525ed2fdc576cf
timestamp   : 2021-12-31 21:49:06 UTC
dependencies:
  - matplotlib-base
  - numpy
  - python >=3.8,<3.9.0a0
  - python_abi 3.8.* *_cp38
  - scipy
  - slycot
...
(and other versions continue)   ```

If/when we get this right, there should be no problem having control depend on slycot in terms of conda/conda-forge: they will both be available on all of the same platforms.  And there is some advantage, I think, in just having to do one install and getting all of the dependencies.
Note that there are still quite a few important things that won't work without slycot.  Many  operations won't work without slycot: state space realizations of a transfer function, any type of Hinf synthesis, balanced truncations.

I created a conda environment using arm64 using the following command
CONDA_SUBDIR=osx-arm64 conda create -n arm64 -c conda-forge python=3.9

When I install slycot using conda in that environment and try to import slycot, I get a segfault.
Output from otool -L _wrapper.cpython-39-darwin.so (@repagh request)
(arm64) murray@evaluna python % otool -L /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so
/Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so:
	@rpath/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so (compatibility version 0.0.0, current version 0.0.0)
	@rpath/liblapack.3.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libblas.3.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libpython3.9.dylib (compatibility version 3.9.0, current version 3.9.0)
	@rpath/libgfortran.5.dylib (compatibility version 6.0.0, current version 6.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.0.0)

Running with lldb I get the following output:
>>> import slycot
Process 13457 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x10)
    frame #0: 0x00000001041a0ce4 libpython3.9.dylib`PyModule_Create2 + 24
libpython3.9.dylib`PyModule_Create2:
->  0x1041a0ce4 <+24>: ldr    x9, [x9, #0x10]
    0x1041a0ce8 <+28>: ldr    x9, [x9, #0x358]
    0x1041a0cec <+32>: cbz    x9, 0x1041a0d00           ; <+52>
    0x1041a0cf0 <+36>: ldp    x29, x30, [sp, #0x20]
Target 0: (python) stopped.

Backtrace:
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x10)
  * frame #0: 0x00000001041a0ce4 libpython3.9.dylib`PyModule_Create2 + 24
    frame #1: 0x0000000103a69c14 _wrapper.cpython-39-darwin.so`PyInit__wrapper + 36
    frame #2: 0x000000010019f290 python`_imp_create_dynamic + 2376
    frame #3: 0x00000001000b3c20 python`cfunction_vectorcall_FASTCALL + 208
    frame #4: 0x000000010016bc0c python`_PyEval_EvalFrameDefault + 29264
    frame #5: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #6: 0x000000010005f9e0 python`_PyFunction_Vectorcall + 240
    frame #7: 0x000000010016edc8 python`call_function + 572
    frame #8: 0x000000010016b8e4 python`_PyEval_EvalFrameDefault + 28456
    frame #9: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #10: 0x000000010016edc8 python`call_function + 572
    frame #11: 0x000000010016b8c0 python`_PyEval_EvalFrameDefault + 28420
    frame #12: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #13: 0x000000010016edc8 python`call_function + 572
    frame #14: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #15: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #16: 0x000000010016edc8 python`call_function + 572
    frame #17: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #18: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #19: 0x000000010016edc8 python`call_function + 572
    frame #20: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #21: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #22: 0x0000000100060c84 python`object_vacall + 376
    frame #23: 0x0000000100060e74 python`_PyObject_CallMethodIdObjArgs + 132
    frame #24: 0x000000010019951c python`PyImport_ImportModuleLevelObject + 1564
    frame #25: 0x000000010015bef0 python`builtin___import__ + 124
    frame #26: 0x00000001000b32b0 python`cfunction_call + 80
    frame #27: 0x000000010005f7b0 python`_PyObject_Call + 156
    frame #28: 0x000000010016bc0c python`_PyEval_EvalFrameDefault + 29264
    frame #29: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #30: 0x000000010005f9e0 python`_PyFunction_Vectorcall + 240
    frame #31: 0x000000010016edc8 python`call_function + 572
    frame #32: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #33: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #34: 0x000000010005f9e0 python`_PyFunction_Vectorcall + 240
    frame #35: 0x0000000100060c84 python`object_vacall + 376
    frame #36: 0x0000000100060e74 python`_PyObject_CallMethodIdObjArgs + 132
    frame #37: 0x00000001001998d4 python`PyImport_ImportModuleLevelObject + 2516
    frame #38: 0x0000000100169f54 python`_PyEval_EvalFrameDefault + 21912
    frame #39: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #40: 0x000000010015eeec python`builtin_exec + 1032
    frame #41: 0x00000001000b3c20 python`cfunction_vectorcall_FASTCALL + 208
    frame #42: 0x000000010016bc0c python`_PyEval_EvalFrameDefault + 29264
    frame #43: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #44: 0x000000010005f9e0 python`_PyFunction_Vectorcall + 240
    frame #45: 0x000000010016edc8 python`call_function + 572
    frame #46: 0x000000010016b8e4 python`_PyEval_EvalFrameDefault + 28456
    frame #47: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #48: 0x000000010016edc8 python`call_function + 572
    frame #49: 0x000000010016b8c0 python`_PyEval_EvalFrameDefault + 28420
    frame #50: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #51: 0x000000010016edc8 python`call_function + 572
    frame #52: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #53: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #54: 0x000000010016edc8 python`call_function + 572
    frame #55: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #56: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #57: 0x0000000100060c84 python`object_vacall + 376
    frame #58: 0x0000000100060e74 python`_PyObject_CallMethodIdObjArgs + 132
    frame #59: 0x000000010019951c python`PyImport_ImportModuleLevelObject + 1564
    frame #60: 0x0000000100169f54 python`_PyEval_EvalFrameDefault + 21912
    frame #61: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #62: 0x000000010015eeec python`builtin_exec + 1032
    frame #63: 0x00000001000b3c20 python`cfunction_vectorcall_FASTCALL + 208
    frame #64: 0x000000010016bc0c python`_PyEval_EvalFrameDefault + 29264
    frame #65: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #66: 0x000000010005f9e0 python`_PyFunction_Vectorcall + 240
    frame #67: 0x000000010016edc8 python`call_function + 572
    frame #68: 0x000000010016b8e4 python`_PyEval_EvalFrameDefault + 28456
    frame #69: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #70: 0x000000010016edc8 python`call_function + 572
    frame #71: 0x000000010016b8c0 python`_PyEval_EvalFrameDefault + 28420
    frame #72: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #73: 0x000000010016edc8 python`call_function + 572
    frame #74: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #75: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #76: 0x000000010016edc8 python`call_function + 572
    frame #77: 0x000000010016b960 python`_PyEval_EvalFrameDefault + 28580
    frame #78: 0x000000010005fa60 python`function_code_fastcall + 116
    frame #79: 0x0000000100060c84 python`object_vacall + 376
    frame #80: 0x0000000100060e74 python`_PyObject_CallMethodIdObjArgs + 132
    frame #81: 0x000000010019951c python`PyImport_ImportModuleLevelObject + 1564
    frame #82: 0x0000000100169f54 python`_PyEval_EvalFrameDefault + 21912
    frame #83: 0x0000000100164374 python`_PyEval_EvalCode + 2968
    frame #84: 0x00000001001c9ccc python`PyRun_InteractiveOneObjectEx + 1432
    frame #85: 0x00000001001c6188 python`PyRun_InteractiveLoopFlags + 540
    frame #86: 0x00000001001c5ef4 python`PyRun_AnyFileExFlags + 72
    frame #87: 0x00000001001e9b14 python`Py_RunMain + 3316
    frame #88: 0x00000001001eab14 python`pymain_main + 1272
    frame #89: 0x000000010000573c python`main + 56
    frame #90: 0x00000001004490f4 dyld`start + 520

Walking up through the backtrace just shows assembly code, so not very useful.

See https://github.com/scikit-build/scikit-build/issues/589

Thanks @isuruf!
So does your fix in https://github.com/conda-forge/python-blosc-feedstock/pull/39 work? Can we do the same here?

@murrayrm, do you still get the segfault when you execute
install_name_tool -change @rpath/libpython3.9.dylib @rpath/libc++.1.dylib /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so

?

New error:
(arm64) murray@evaluna python % install_name_tool -change @rpath/libpython3.9.dylib @rpath/libc++.1.dylib /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so

/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/install_name_tool: warning: changes being made to the file will invalidate the code signature in: /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so
(arm64) murray@evaluna python % python
Python 3.9.9 | packaged by conda-forge | (main, Dec 20 2021, 02:41:06)
[Clang 11.1.0 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import slycot
zsh: killed     python

/usr/bin/codesign -s - -f /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so

Ran that:
(arm64) murray@evaluna python % /usr/bin/codesign -s - -f /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so
/Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so: replacing existing signature
(arm64) murray@evaluna python % python
Python 3.9.9 | packaged by conda-forge | (main, Dec 20 2021, 02:41:06)
[Clang 11.1.0 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import slycot
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/__init__.py", line 16, in <module>
    from .analysis import ab01nd, ab05md, ab05nd, ab07nd, ab08nd, ab08nz
  File "/Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/analysis.py", line 20, in <module>
    from . import _wrapper
ImportError: dlopen(/Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so, 0x0002): Symbol not found: _PyCapsule_Type
  Referenced from: /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so
  Expected in: /Users/murray/anaconda3/envs/arm64/lib/libc++.1.0.dylib
>>>

Okay, so the link is actually necessary. Could you try:

Reinstall slycot in the environement in order to restore the original link

Change the link to the same libpython3.9.dylib as the conda interpreter uses

(arm64) $ otool -L $(which python)
    ...
    XXX/libpython3.9.dylib (compatibility version 3.9.9, current version 3.9.9)
    ...

install_name_tool -change @rpath/libpython3.9.dylib XXX/libpython3.9.dylib /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so

I'm not getting a libpython3.9.dylib as a shared library for python:
(arm64) murray@evaluna python % otool -L $(which python)
/Users/murray/anaconda3/envs/arm64/bin/python:
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.0.0)
(arm64) murray@evaluna python % which python
/Users/murray/anaconda3/envs/arm64/bin/python

???

The latest build (from PR #47) appears to be working correctly (!).
@sawyerbfuller Can you give this is a try when you have a chance?
@isuruf Thank you for all of the help!

@murrayrm It seems I'm having trouble installing the osx-arm64 version of slycot (looks like I'm still getting osx-64 versions of related package like libblas, which I assume is not what I want):
(arm64) minster@Sawyers-MacBook-Pro-2 ~ % conda install conda-forge::slycot
Collecting package metadata (current_repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /Users/minster/opt/anaconda3/envs/arm64

  added / updated specs:
    - conda-forge::slycot

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    slycot-<IP_ADDRESS>             |   py39h10744cc_8         1.2 MB  conda-forge
    ------------------------------------------------------------
                                           Total:         1.2 MB

The following NEW packages will be INSTALLED:

  libblas            conda-forge/osx-64::libblas-3.9.0-12_osx64_openblas
  libcblas           conda-forge/osx-64::libcblas-3.9.0-12_osx64_openblas
  libcxx             conda-forge/osx-64::libcxx-12.0.1-habf9029_1
  libgfortran        conda-forge/osx-64::libgfortran-5.0.0-9_3_0_h6c81a4c_23
  libgfortran5       conda-forge/osx-64::libgfortran5-9.3.0-h6c81a4c_23
  liblapack          conda-forge/osx-64::liblapack-3.9.0-12_osx64_openblas
  libopenblas        conda-forge/osx-64::libopenblas-0.3.18-openmp_h3351f45_0
  llvm-openmp        conda-forge/osx-64::llvm-openmp-12.0.1-hda6cdc1_1
  numpy              conda-forge/osx-64::numpy-1.22.0-py39h9d9ce41_0
  slycot             conda-forge/osx-64::slycot-<IP_ADDRESS>-py39h10744cc_8
...

The commands leading up to this followed what you typed in above, and were executed on a standard anaconda installation. It looks like other other packages that were being installed are arm native:
minster@Sawyers-MacBook-Pro-2 ~ % CONDA_SUBDIR=osx-arm64 conda create -n arm64 -c conda-forge python=3.9
Collecting package metadata (current_repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /Users/minster/opt/anaconda3/envs/arm64

  added / updated specs:
    - python=3.9

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    bzip2-1.0.8                |       h3422bc3_4         148 KB  conda-forge
    ca-certificates-2021.10.8  |       h4653dfc_0         139 KB  conda-forge
    libffi-3.4.2               |       h3422bc3_5          38 KB  conda-forge
    libzlib-1.2.11             |    hee7b306_1013          55 KB  conda-forge
    ncurses-6.2                |       h9aa5885_4         915 KB  conda-forge
    openssl-3.0.0              |       h3422bc3_2         2.4 MB  conda-forge
    python-3.9.9               |h43b31ca_0_cpython        12.1 MB  conda-forge
    python_abi-3.9             |           2_cp39           4 KB  conda-forge
    readline-8.1               |       hedafd6a_0         259 KB  conda-forge
    setuptools-60.5.0          |   py39h2804cbe_0         1.1 MB  conda-forge
    sqlite-3.37.0              |       h72a2b83_0         1.6 MB  conda-forge
    tk-8.6.11                  |       he1e0b03_1         3.2 MB  conda-forge
    xz-5.2.5                   |       h642e427_1         222 KB  conda-forge
    zlib-1.2.11                |    hee7b306_1013          84 KB  conda-forge
    ------------------------------------------------------------
                                           Total:        22.2 MB

The following NEW packages will be INSTALLED:

  bzip2              conda-forge/osx-arm64::bzip2-1.0.8-h3422bc3_4
  ca-certificates    conda-forge/osx-arm64::ca-certificates-2021.10.8-h4653dfc_0
  libffi             conda-forge/osx-arm64::libffi-3.4.2-h3422bc3_5
  libzlib            conda-forge/osx-arm64::libzlib-1.2.11-hee7b306_1013
  ncurses            conda-forge/osx-arm64::ncurses-6.2-h9aa5885_4
  openssl            conda-forge/osx-arm64::openssl-3.0.0-h3422bc3_2
  pip                conda-forge/noarch::pip-21.3.1-pyhd8ed1ab_0
  python             conda-forge/osx-arm64::python-3.9.9-h43b31ca_0_cpython
  python_abi         conda-forge/osx-arm64::python_abi-3.9-2_cp39
  readline           conda-forge/osx-arm64::readline-8.1-hedafd6a_0
  setuptools         conda-forge/osx-arm64::setuptools-60.5.0-py39h2804cbe_0
  sqlite             conda-forge/osx-arm64::sqlite-3.37.0-h72a2b83_0
  tk                 conda-forge/osx-arm64::tk-8.6.11-he1e0b03_1
  tzdata             conda-forge/noarch::tzdata-2021e-he74cb21_0
  wheel              conda-forge/noarch::wheel-0.37.1-pyhd8ed1ab_0
  xz                 conda-forge/osx-arm64::xz-5.2.5-h642e427_1
  zlib               conda-forge/osx-arm64::zlib-1.2.11-hee7b306_1013
....
(test_slycot) minster@Sawyers-MacBook-Pro-2 ~ % conda activate arm64
(arm64) minster@Sawyers-MacBook-Pro-2 ~ % conda install slycot

@sawyerbfuller I had that problem as well.  You need to set the environment variable CONDA_SUBDIR to point to the right place:
CONDA_SUBDIR=osx-arm64 conda create -n arm64 -c conda-forge python=3.9
CONDA_SUBDIR=osx-arm64 conda install control

@murrayrm That seems to do the trick. I was able to install slycot and then and run some code from it.
That is, assuming after all that that the arm64 version of slycot was the one that was imported.

Here's the commands that I ran to convince myself I was running the ARM version:
(base) murray@evaluna recomm % conda activate arm64
(arm64) murray@evaluna recomm % python
Python 3.9.9 | packaged by conda-forge | (main, Dec 20 2021, 02:41:06)
[Clang 11.1.0 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import slycot
>>> slycot.__path__
['/Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot']
>>> quit()
(arm64) murray@evaluna recomm % ls /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot
__init__.py			math.py
__pycache__/			synthesis.py
_wrapper.cpython-39-darwin.so*	tests/
analysis.py			transform.py
examples.py			version.py
exceptions.py
(arm64) murray@evaluna recomm % file /Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so
/Users/murray/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so: Mach-O 64-bit dynamically linked shared library arm64

I think the arm64 at the end is the key: this says that you are on an M1 Mac.  Otherwise, you get x86_64.

Ok looks good, thanks! ```

import slycot
slycot.path
['/Users/minster/opt/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot']
exit()
(arm64) minster@Sawyers-MacBook-Pro-2 ~ % file /Users/minster/opt/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so
/Users/minster/opt/anaconda3/envs/arm64/lib/python3.9/site-packages/slycot/_wrapper.cpython-39-darwin.so: Mach-O 64-bit dynamically linked shared library arm64

Done.  Thanks @bnavigator, @repagh, and @sawyerbfuller for the comments and feedback and @isuruf for the final tricks to implement (in PR #47).


Remove apostrophe from filename

What's in this pull request?
Remove apostrophe from filename:
./test/SourceKit/Inputs/libIDE-mock-sdk/README.DON'T_MODIFY_FILES_DIRECTLY
No changes to contents of the file named above.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

How about "DON'T" to "DO_NOT"?

@tkremenek, OK.

@tkremenek, Done.

Thank you. Can you squash this into a single commit?

OK. Apologies, I've closed this Pull and open another one. I had a snafu in my previous fork.

Done. Please see pull #1873. Sorry about the multiple pulls!


Consider switching ModelFactory over to prototypes

I used Coffeescript specifically so I could do a sort of class structure where I have a base model class that all other models extend from. Essentially this simply uses the javascript prototype system. It makes for quite elegant code.

@ProLoser I did some refactor job for that service, but I'm not sure if this is what you ment. Feel free to comment.


Rewrite Vesting contract

Change Name, from vesting to token plan to sync with the dapp.
Change the meaning of start date. The reasoning is that current behaviour allowes the provider to deposit the tokens just before it vest, not assuring the claimer that they will get the tokens by that date.
Change expectedInitialDeposit for tokensPerPeriod to avoid rounding errors
Remove final withdraw which is unneeded
Derive the Vest state in a more resilient way without using Next


I'm finding it very annoying going back and forth between the Pallet editor, and the background colorizer

Could you please allow us to edit color pallets from any section that uses them?

same, would be nice to have palette editor more readily accessible.


Update version string to 2.1.0dev0

Summary

How to test

Checklist

[ ] I have added unit tests to cover my changes.
[ ] I have added integration tests to cover my changes.
[ ] I have added the description of my changes into CHANGELOG.
[ ] I have updated the documentation accordingly

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below).

# Copyright (C) 2024 Intel Corporation
#
# SPDX-License-Identifier: MIT

due to change of the release plan


Build(deps-dev): Bump eslint-plugin-react from 7.31.10 to 7.31.11 (#4482)

Pull request opened by the merge tool on behalf of #4482

Merged in 5f97f05ea0aef6d2d806b78db0c995a916eca1a0


/apps/legacy-api - WhaleApiClient caching

What would you like to be added:
Cache WhaleApiClient via network as key with a provider pattern? E.g. WhaleApiClientProvider.get(network).
Why is this needed:
Currently, the WhaleApiClient is created at every request. Not very performant in the long run. https://github.com/DeFiCh/jellyfish/pull/1011
/triage accepted
/assign @eli-lim
/area apps
/priority important-soon

We only need 3 networks:

playground via https://playground.defichain.com/v0/regtest
mainnet via https://ocean.defichain.com/v0/mainnet
testnet via https://ocean.defichain.com/v0/testnet


Prevent NullPointerException being thrown when constructing log message

https://github.com/apache/httpcomponents-client/pull/515 added additional logging information about the connection being released. However the log exists in a branch where the connection may be null, thus causing a NPE to be thrown under those circumstances.
This PR accounts for the potential for the connection to be null when constructing the log message.

@topbadger Thank you for reporting the problem. Corrected by 9e3d79bedeee494d24bdb03cc598bae5184a3abf.


Iterate over dynamic path in template

Hey,
i would like to use etcd + confd to dynamically build configurations.
my data structure in etcd looks like this, if converted to json:
{
  "stage1": {
    "activeVersion": {
      "RSysLogReceiver": "v2015.04.02-16"
    },
    "endpoints": {
      "RSysLogReceiver": {
        "v2015.04.02-14": {
          "inst1:port": {
            "IP": "IPaddr1",
            "PORT": "PORTnum"
          },
          "inst2:port": {
            "IP": "IPaddr2",
            "PORT": "PORTnum"
          }
        },
        "v2015.04.02-16": {
          "inst3:port": {
            "IP": "IPaddr3",
            "PORT": "PORTnum"
          },
          "inst4:port": {
            "IP": "IPaddr4",
            "PORT": "PORTnum"
          }
        }
      }
    }
  },
  "stage2": {}
}

And the commands i used to populate the data to etcd:
etcdctl set /stage1/activeVersion/RSysLogReceiver v2015.06.16-1
etcdctl set /stage1/endpoints/RSysLogReceiver/v2015.06.16-1/inst1:12345 '{"IP": "host1","PORT": "12345"}'
etcdctl set /stage1/endpoints/RSysLogReceiver/v2015.06.16-1/inst2:12345 '{"IP": "host2","PORT": "12345"}'
etcdctl set /stage1/endpoints/RSysLogReceiver/v2015.06.16-1/inst3:12345 '{"IP": "host3","PORT": "12345"}'

I would like to use "activeVersion" to deploy multiple version of one application at the same time. All applications will register themself with there version number as prefix in "endpoints". "activeVersion" will contain the production version string, that should be used.
To create a config via confd with this pattern. i would need a template that looks something like this:
{{range gets "/endpoints/RSysLogReceiver/$(getv "/activeVersion/RSysLogReceiver")/*"}}
{{$data := json .Value}}
  port: {{$data.PORT}}
  ip: {{$data.IP}}
{{end}}

My current working version, with a static "activeVersion" looks like this
{{range gets "/endpoints/RSysLogReceiver/v2015.06.16-1/*"}}
{{$data := json .Value}}
  port: {{$data.PORT}}
  ip: {{$data.IP}}
{{end}}

Could you please give me a hint, how i have to modify the template to actually work with a dynamic path?

Try {{range gets (printf "/endpoints/RSysLogReceiver/%s/*" (getv "/activeVersion/RSysLogReceiver"))}}. The idea is that the string passed to gets is a literal value, so any dynamic argument parsing needs to be done and returned as a string.

works, thank you!


HIVE-27230 : Backport of HIVE-22033 and HIVE-26522 to branch-3

JIRA link : https://issues.apache.org/jira/browse/HIVE-27230
This ticket was present in branch-3.1 from where Hive-3.1.3 was released, so it is necessary to backport these changes to branch-3

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@vihangk1 Can you please review and merge this

@vihangk1 Can you please review and merge this


Fixes 2327

Fixes "reply to this email address" when field type is changed in edit mode
Fixes #2327

thanks!


Benchmark Submission: andrewztan

This pull request represents a submission to the codesearchnet benchmark.
It is the main communication channel for you and the reviewers.
Wandb run results for review

@andrewztan your benchmark submission only includes results for python, which is an incomplete submission in this case, I'll go ahead and close this PR in the meantime.


make the referred filter name small case in payload_to_metadata_filter

Risk Level: loc, just doc fix

cc @KBaichoo

/docs


Set up CI with Azure Pipelines

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


How to use the pre-bundled version

I don't know exactly how to use the prebundled version, I tried the following so far
<!DOCTYPE html>

    <link rel="stylesheet" href="./node_modules/graphql-voyager/dist/voyager.css" />

Loading...

What else do I need to do, if I want to use only de CDNs

@sgaseretto I updated example to use only CDN links:
https://github.com/APIs-guru/graphql-voyager/blob/master/example/index.html
Can you please test it?

I just added my endpoint http://<IP_ADDRESS>:8000/graphql again to the new code and it still doesn't work. Probably there is something else I need to do, but I don't know what

@sgaseretto Can you open dev tools and make a screenshot of the console?

@sgaseretto If you use HTTP you probably need to remove this line:
credentials: 'include',

This are the errors I'm getting

@sgaseretto you are opening page from file-system, that's why // in the url is treated as file:///.
To fix the issue add http:// to script urls:

and same for other scripts

You were right! Now it seems I'm having a problem with my server

While I figure out what's happening (maybe something related to CORS, despite my backend accepts requests from any IP) how can I modify the bundled version so, instead of fetching, I can embed in it my introspection and visualize it? Maybe this is a dumb question but I'm new to javascript

@sgaseretto It's because you are opening HTML as file that's why browser can't set Origin header.
You should host it somewhere, e.g. run local http server python -m SimpleHTTPServer
http://2ality.com/2014/06/simple-http-server.html

@sgaseretto If you just want to use it locally, try our demo:
https://apis.guru/graphql-voyager/
You can select "Custom introspection" like here:

I'm closing this stale issue.


NIFI-12299 run integration-tests when assemblies are updated

Summary
NIFI-12299 run integration-tests when assemblies are updated
Run the integration-tests GitHub Actions workflow if:

pom.xml is updated (e.g. for wide-ranging dependency updates)
an assembly or bom module pom.xml is update (e.g. if new modules are added to the repository that may contain ITs)

Also, skip known failing ITs (achieve a more stable baseline from which IT fixes can start to be applied thorugh NIFI-12190
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

~[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy~
~[ ] New dependencies are documented in applicable LICENSE and NOTICE files~

Documentation

~[ ] Documentation formatting appears as expected in rendered files~

Dependency Check workflow currently failing (for all PRs) due to https://github.com/RetireJS/retire.js/issues/423

retire.js bug has now been fixed

retire.js bug has now been fixed

Thanks for the heads-up @eoftedal, I have triggered a re-run of the dependency-check workflow 🤞


Add The Method Training Material to Engineering Excellence / System Design & Architecture

I would like to add PowerPoint presentation, recoding and references to Righting Software https://rightingsoftware.org/ to Engineering Playbook under the folder Engineering Excellence / System Design & Architecture.
Please let me know which direction I should choose.

Thank you, Chad, let me work on this.


Flower

FYI - in latest run, Flower didn't start.  Probably need to pass -D to webserver - daemonize it so 2nd part of command runs

Thanks! yeah, you're right. I'll change the script

Found out that flower does not support SQS
So I'm moving to redis instead

FYI what actually needs to happen is something like this (used in my case where I'm putting the scheduler and webserver on one host).
#!/usr/bin/env bash
if [ "\$AIRFLOW_ROLE" == "SCHEDULER" ]
then exec airflow scheduler -n 10
elif [ "\$AIRFLOW_ROLE" == "WEBSERVER" ]; then
    exec airflow webserver -D
    exec airflow scheduler -D -n 10
elif [ "\$AIRFLOW_ROLE" == "WORKER" ]
then exec airflow worker
else echo "AIRFLOW_ROLE value unknown" && exit 1
fi

exec spawns things in a new (dedicated) process so it can't have bash helpers like &&, will need to be called twice.

Yes, i did that, but flower throws error wich led me to that issue saying flower doesnt support SQS

Actually, be careful using that approach.  This may not be an issue with Flower but Scheduler quits frequently and is managed by systemctl.  Putting two things under that same script will mean that systemctl will wait for all of them to exit before restarting either.
I had to rewrite this a lot for my use case (scheduler + webserver on same host) ultimate setting up a systemctl for each of them, with slightly different service environments.


CI test - Production build tests should run

Hi @HCasler,
You have proposed changes to files in these packages:

JobConfig

which require these tests: build.
@Mu2e/write have access to CI actions on main.
:hourglass: The following tests have been triggered for 0747ccc95f991e290cdc6cfdf70bf41fdd49493b: build  (Build queue has 2 jobs)
About FNALbuild. Code review on Mu2e/Offline.


VEDirect.cpp get method does a set not a get, command set is :8 and not :7

Thanks for creating the library I found this issue
size_t VEDirect::get(uint16_t id){
  ved_t txBuffer;
  *********************
  ved_setCommand(&txBuffer, VEDirect_kSetCommand); //Should be VEDirect_kGetCommand
  *********************
  ved_setId(&txBuffer, id);
  ved_enframe(&txBuffer);
  return serialPort.write(txBuffer.data, txBuffer.size);
}

Thanks for the feedback, I have corrected the error.


ARROW-12590: [C++][R] Update copies of Homebrew files to reflect recent updates

Building on #12157

@jonkeane looks like it works 👍 takes 2x longer on macos-11 though (more features?) maybe we can add (sc)caching later to speed that up.
A sample run: https://github.com/ursacomputing/crossbow/runs/7617843476?check_suite_focus=true
Do I understand configure correctly that tools/autobrew is only used if there is a local formula?

@github-actions crossbow submit r-binary-packages homebrew-r-autobrew

I moved fixing the python lint to https://issues.apache.org/jira/browse/ARROW-17282

I also pulled out 10.13 testing, will open separate JIRA for those.

@nealrichardson upstreaming the changes in dependencies and build flags can happen with the 9.0.0 release PR right?

@github-actions crossbow submit homebrew-r-autobrew r-binary-packages

@nealrichardson upstreaming the changes in dependencies and build flags can happen with the 9.0.0 release PR right?

Already in progress: https://github.com/autobrew/homebrew-core/pull/114 Please review and make sure we haven't missed anything.

@github-actions crossbow submit homebrew-r-autobrew

@github-actions crossbow submit homebrew-r-autobrew

@github-actions crossbow submit homebrew-r-autobrew

@neal this should be done now. I will add the 10.13 runners to the nightlies in another PR.

@github-actions crossbow submit homebrew-r-autobrew

@assignUser Hmm.  Here's the output:
fatal: reference is not a tree: 197f509f6eb39dd40df1e0ac3a4106769ac5f884

Could you confirm this?

@github-actions crossbow submit homebrew-r-autobrew

@kou the issue is with the runner, I need to modify the cleanup script. I'll start another run once I'm done with that.

@github-actions crossbow submit homebrew-r-autobrew

It seems to be working.


Bug: tmerc operation returning incorrect results

I've been testing things with a EPSG:27700 (BNG) to EPSG:3857 (webmerc) pipeline and the round trip result for the tmerc operation is incorrect.
# Full pipeline roundtrip
echo 544748. 258372. | cargo run --features binary --bin kp -- "enu:in | tmerc inv lat_0=49 lon_0=-2 k_0=0.9996012717 x_0=400000 y_0=-100000 ellps=airy | webmerc lat_0=0 lon_0=0 x_0=0 y_0=0 ellps=WGS84 | enu:out" --roundtrip

-190686. 5461518. 0.. NaN

# tmerc only roundtrip
echo 544748. 258372.| cargo run --features binary --bin kp -- "enu:in | tmerc inv lat_0=49 lon_0=-2 k_0=0.9996012717 x_0=400000 y_0=-100000 ellps=airy | gis:out" --roundtrip

-190686. 5461518. 0. NaN

# tmerc Cart -> Geo
echo 544748. 258372. | cargo run --features binary --bin kp -- "enu:in | tmerc inv lat_0=49 lon_0=-2 k_0=0.9996012717 x_0=400000 y_0=-100000 ellps=airy | gis:out

0. 52. 0. NaN  # What I expect

# tmerc Geo -> Cart - Incorrect result
echo 0.118 52. | cargo run --features binary --bin kp -- "gis:in | tmerc  lat_0=49 lon_0=-2 k_0=0.9996012717 x_0=400000 y_0=-100000 ellps=airy | enu:out"

354062. 5719890. 0. NaN # NOT what I expect. Should be 544748. 258372.

I've tested this with kp and with geodesy-wasm which implements the Minimal context internally.
Possibly useful pointers:

The following pipeline produces the correct result in the forward direction but incorrect in the inverse (same control coordinate as above):

tmerc inv lat_0=49 lon_0=-2 k_0=0.9996012717 x_0=400000 y_0=-100000 ellps=airy | webmerc lat_0=0 lon_0=0 x_0=0 y_0=0 ellps=WGS84

Evaluating the individual parts:

tmerc and webmerc correctly transform the equivalent coordinate to WGS84[gis] (0.118 52.).
tmerc does NOT transform that WGS84 coordinate to cartesian coords correctly while webmerc does.

The last point I think suggests the tmerc FWD operation is producing incorrect results?

Tested on latest main as of 18/08/2023.
I've truncated all results to integers for readability

Thanks for reporting this - I believe it is now corrected in de9e129

Awesome. Thanks for fixing. All working on my end now!

Awesome - thanks for the feedback, and sorry for taking 3 days to respond: I was mostly offline during the weekend, and did not see your bug report until monday morning

No a problem in the slightest 😄


chore: scalex-jenkinsx-rollout-app to 0.0.1

chore: Promote scalex-jenkinsx-rollout-app to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


How to log to console along with file output?

I want to see program output in VS's output window but can't figure out how to do that. I tried to set FLAGS_alsologtostderr but this doesn't work for me.  I'm using windows, so cannot use utils like tee

@xahon
debug the logging_unittest project as a test,
you can comment out the second line code of "//CaptureTestStderr();" that in the main fucntion.
hope that helps

Setting FLAGS_alsologtostderr to true should be sufficient:
https://github.com/google/glog/blob/4244cec1405fcd46e6efc915a35b74c150b1c600/src/logging.cc#L841-L846
Please make sure that the flag is indeed set correctly.


Support v1.Node

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
fixes #4

License
Apache 2.0

What's in this PR?
Adds support for matching v1.Node objects.
Also it fixes the circleci build for kubernetes 1.11 by ignoring preflight check for crictl in minikube.
Checklist

[x] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline
[x] User guide and development docs updated (if needed)

Thanks a bunch! Looking forward to using this!


Preds is Nan

Hello, I have runned baseline in my loacl, but found the output of model is nan. Have you encountered this problem? Is it possible that there is a operating system or environmental problem?
Before this line, print the preds get nan.
https://github.com/isaaccorley/dfc2022-baseline/blob/d01e78ee0efd2d286e67a28dff4b506a50b3e897/src/trainer.py#L19

What OS and config are you using?

What OS and config are you using?

OS: windows10
conda: 4.10.3
python: 3.7.11
and pip list:

config file is from: https://github.com/isaaccorley/dfc2022-baseline/blob/master/conf/dfc2022.yaml , I didn't change anything.
Is it possible that the problem is data folder? the structure of my data directory is:
data/
    labeled_train/
    unlabeled_train/
    val/

I don't think it's related to the data. I'm using the latest versions of pytorch lighting torch and torchvision with cuda 11. If I had to guess it's probably package related. I haven't tested this on windows.

I don't think it's related to the data. I'm using the latest versions of pytorch lighting torch and torchvision with cuda 11. If I had to guess it's probably package related. I haven't tested this on windows.

OK, I will update cuda from 10 to 11, I Will be back after this to show the results.

I'm using the latest versions of pytorch lighting torch and torchvision with cuda 11.

A question, what's your torch, torchvision pytorch lighting version?

Install the latest version of torch(1.9.0+cu111), torchvision(0.10.0+cu111), the problem has been fixed. Thank you.

Install the latest version of torch(1.9.0+cu111), torchvision(0.10.0+cu111), the problem has been fixed. Thank you.

hello I get some error on install torchgeo0.3.0,  it seemed that the torchgeo0.2.0 don't support DFC2022 dataset .so I clone the git and try to install the follows the error:


Flag deprecated items in the search results

It would be very helpful if the search results flagged deprecated items in red (or however you'd want to indicate this). Currently the only way to see that an item is deprecated is to click on it in the results list. For a schema that has accumulated many deprecated items, it can be frustrating trying to identify the best/current search result.

I'd be happy to implement this feature 😁


Switch language according to user preferences

Users now use only one language
There should be a command to switch languages en-ru-en like /lang  or an inliner like /lang en

implemented binary en-ru flag as '/lang' command


Extension Metadata

Hey all, it seems like in version 0.10.2, some additional extension metadata requirements were added https://github.com/duckdb/duckdb/pull/11515 (its buried in the release notes as a non breaking change here https://github.com/duckdb/duckdb/releases/tag/v0.10.2). Prior to this release, the rust bindings using duckdb_entrypoint could be used to build extensions. Any plans to add something emitting extension metadata to duckdb_entrypoint? The metadata format is detailed in the linked PR. What's the recommended workflow for attaching this metadata?

Generally speaking the easiest way to add that metadata is to use the extension template - otherwise the format is straightforward enough to implement yourself

Where is this extension template you speak of?

https://github.com/duckdb/extension-template

Otherwise something like this can be used: https://gist.github.com/Mause/888ad36325f462641a8e169f6183834a

I was hoping for something integrated with the cargo toolchain, a workflow like https://github.com/overdrivenpotato/rust-psp/tree/master
I guess I'll build it myself and maybe upstream it if there's interest.


Invalid self-signed-certificate

Bug Description
Our traefik revision includes a bug, where the self-signed certificates are do not include any IP Sans even when metallb is used to expose it.
To Reproduce

deploy bundle
try to complete flow with hydra perform, you will get an error saying:

Post "...": tls: failed to verify certificate: x509: certificate signed by unknown authority

Solution
If we bump traefik to the latest revision, this will be fix.

Fixed by https://github.com/canonical/iam-bundle/pull/58


throw big error on false negative

Has this been tested when used as part of the Val service? Isn't merging
this in a bit premature?
Sent from a mobile device, excuse my brevity. Kind regards, Thomas
Op do 18 apr 2024 15:07 schreef Evandro Alfieri @.***>:

Merged #169 https://github.com/buildingSMART/ifc-gherkin-rules/pull/169
into development.
—
Reply to this email directly, view it on GitHub
https://github.com/buildingSMART/ifc-gherkin-rules/pull/169#event-12523231598,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAILWV5UHSDIUZWOUSLPRLDY57AR5AVCNFSM6AAAAABGNGSMWKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGUZDGMRTGE2TSOA
.
You are receiving this because you are subscribed to this thread.Message
ID: <buildingSMART/ifc-gherkin-rules/pull/169/issue_event/12523231598@
github.com>

@evandroAlfieri @Ghesselink

Has this been tested when used as part of the Val service? Isn't merging
this in a bit premature?


ENTESB-17076: downgrade qpid library

I've tested both 0.55.0 and 0.55.0.redhat-00005 versions with CK8.

Thanks @nicolaferraro


Click tracking docs does not explain plain email behavior

Expected content
A description that eludes the actual click tracking link substitution behavior for plain emails.
After investigating issues we had with bad activation links being sent to new users, we found that plain e-mails were the source of the problem. If link tracking is enabled (link tracking in plain e-mails is included by default), all links in plain e-mails seem to substituted (it looks like your parser is picking up everything with https:// or http:// in plain e-mails). Including a clicktracking=off in the HTML has no effect on the generated plain e-mail, and the substitution is still made.
Furthermore, there is no description of the "enable click tracking in plain emails" toggle that you have made available in the UI. This luckily served as our fix, without having to turn click tracking off entirely.
Actual content
No mention that using clicktracking=off in the HTML has no effect on click tracking in plain e-mails. Right now, the wording makes the reader believe clicktracking=off also would have an effect on the plain e-mail.
No mention of the "enable click tracking in plain emails" on/off toggle in the UI
No mention of how link substitutions are done on plain e-mails
Link to page:
https://sendgrid.com/docs/ui/analytics-and-reporting/click-tracking-html-best-practices/
https://sendgrid.com/docs/ui/account-and-settings/tracking/#click-tracking
Why did this cause an issue for us?
For Sendgrid developers:
Your link replacement process for the plain e-mails seems to html encode the links, which, when we are using multiple query string parameters leaves the redirected link invalid. For instance, the verification link https://foo.bar.com/verify?id=foo&token=bar is substituted with a sendgrid click tracking link redirecting to the html encoded https://foo.bar.com/verify?id=foo&token=bar (which resolves to the invalid https://foo.bar.com/verify?id=foo&amp%3Btoken=bar). This means that anyone who opened the plain-version of our transactional e-mails, only got invalid verification links. Of course we could implement an html decoder in our router, but this behaviour on your side seems like a bug to us. Same with the clicktracking=off clauses in the HTML - this should also lead to non-substituted links in the plain e-mail.

Hi @brynek,
Thanks so much for your feedback! We've added this to our internal docs backlog.
Twilio SendGrid Docs Team

@brynek
I would recommend hitting up the support team with this issue. I attempted to replicate with my own test account and could not create your issue.
I used the following link for testing:
https://www.google.com/search?q=url+encode&oq=url+encode&aqs=chrome..69i57j0l4j69i60.2829j1j7&sourceid=chrome&ie=UTF-8
I sent it in an email that had only plain text and made sure that my plain text links would get encoded by using the Enable click tracking in plain text emails option, but could not reproduce your encoding issue. Is there a part I am missing?

@kylearoberts
Thank you for looking into this.
After further investigation, we realized it was the Handlebars HTML escaping that caused this (forgot to mention this was in the context of transactional templates).
An update just like the one you did to your transactional template demo recently fixed it. Easy to forget those pesky triple brackets. This specific issue is therefore resolved for us.
As a side note of that, https://sendgrid.com/docs/for-developers/sending-email/using-handlebars/#replacement-with-html could be more clear about this in our opinion, as multiple query parameters are rather common among password reset/verification e-mails etc. I'll open another issue for the docs team on this.
As the link substitutions worked in the HTML version while this issue was happening, it seems you are performing some sort of HTML decode on links in the HTML version, which are not performed on the plain version. Can you confirm this? As the HTML version worked we were unaware of this issue for quite a long time.
I have added an example of one HTML-based and one plain click-tracking link, picked up from the same production e-mail and the same verification link (merged in by Handlebars using only double brackets). As you see, the HTML-based has been decoded somewhere along the way, which previously lead us to believe this worked without the triple brackets in Handlebars. The utm parameters are supplied by Sendgrid afterward, so they work well in both versions. In e-mail clients, browsers will, of course, decode the & as part of their HTML rendering process. However, this resolving is not being done as part of the redirection process from click tracking links, which lead to the error we experienced with plain e-mails.
ClickTrackCheck.txt

Similar issue:
https://github.com/sendgrid/docs/issues/5611


[backend] sync db to smart contract

Sync db to smart contract using cron job. Maybe every 1 hour or less.

moved to backend repo.


Removed Learned Enough to Be Dangerous books

What does this PR do?
Remove Resource(s)
Learn Enough Git to be Dangerous
Learn Enough Command Line to Be Dangerous
Checklist:

[x] Not a duplicate
[ ] Included author(s) if appropriate
[x] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

Thanks, and welcome to free-programming-books!


Update aws-java-sdk-core to 1.12.364

Updates com.amazonaws:aws-java-sdk-core from 1.12.336 to 1.12.364.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #160.


Adds Bingus and Space Cat spawners

Changelog:

Adds spawners for Bingus and space cats
Adds Bingus to the Cat Spawner
Adds rare chance(5%) for generic cat spawner to spawn space cat

Closing. Merged upstream


RaspberryPi 3 dependencies on install

Quick install question.  I'm attempting to install a full node on a raspberry pi 3.  As shown here .
I'm getting the following dependency errors.  (I tried with dpkg and apt-get.)
The following packages have unmet dependencies:
x42-core:armv7l : Depends: libnotify4:armv7l but it is not installable
Depends: libappindicator1:armv7l but it is not installable
Depends: libxtst6:armv7l but it is not installable
Depends: libnss3:armv7l but it is not installable
Depends: libxss1:armv7l but it is not installable
E: Unable to correct problems, you have held broken packages.
Any recommendations?

What is the OS you're running on that PI?

It was raspbian.  I believe it has something to do with electron having problems creating a .deb package for armv7l.   When I tried grabbing the source, I can run the UI with 'npm start mainnet'.  However when I try to create an installer package with 'npm run package:linuxarm' it builds up until the creation of the .deb, and then errors out with this [issue] (https://github.com/electron-userland/electron-builder/issues/3777).  Which, as you can see, isn't a high priority for electron.  And I didn't dig much deeper to see what it would take to fix or find a workaround.
I have personally moved on, as I'm still able to interface with the x42 node through the swagger api.


Issue58 wrong solver

Motivations:

As reported in issue #58 and elsewhere the homemade RK4 solver seemed to show some defects
As I investigated I found other defects than the ones we had thought of (see details below)
This PR fixes the homemade solver and also improves the structure of _solvers.py for single-sourcing, clarity and performance

Main changes:

The homemade RK4 is fixed, but it still has limitations (see below)
All solvers are a bit faster
A new get_available_solvers() routine is implemented
Unit tests and non-regression tests are updated and passing

Detailed changes:
get_available_solvers()
Dislays the available solvers, with a brief description
Can print and / or return a dict / list / str

homemade solver fixing
There were 2 main defects in the solver:

It incremented each variable independently (this is what @Dalus mentioned in a discussion)
The computation of the 4 coefficients was wrong

Details:

could have been fixed as proposed in the  discussion, but that would have necessitated a loop
Meanwhile, it can be seen that scipy solves this issue by forcing the user to define a single function (that produces a single vector containing all derivatives of all ode functions). This is elegant because in addition to ensuring that the coupling is properly taken into account, it allows to treat the whole set of equation as one, allows for vectorized computation and results in cleaner code.
It is easy to see that in the case in which the ode did not depend on itself the 4 weights were identical, which is wrong (purple square on image below).
This is the function returned by _solvers.get_func_dydt(), which is now common to scipy and homemade solver.
So I modified the homemade RK4 to allow to use the exact same single derivative function as the scipy solver.
The other advantage is that if any difference persists between the homemade and scipy solver, we are sure it comes from something else than the common derivative function.

a difference remained... which allowed me to focus on the core of the solver and realize that it was wrong.
The expressions of the weights (often named k1, k2, k3, k4, and that have the dimension of dy_on_dt ) were wrong, a multiplication by the time step was missing (purple arrows and circles on image below)
I renamed the weights as dy1_on_dt instead of dy1 because it makes it clear they must be multiplied by dt before being added to y (I had trusted the the notation dy1 and it took my attention off guard).

Before:

After:

Remaining discrepancies
Despite these corrections, there are still non-negligible differences between the results of the homemake RK4 and scipy's RK4.
My investigations show that the discrepancy tends to disappear when the time step is reduced (the homemade slowly converges to the scipy solution).

Unless someone has another idea, my opinion at this point is that this is due to the limitation of the homemade RK4 solver:

it has a fixed time step
it does not control the error on the g and does not try to adjust its strategy according to it

=> I think our homemade RK4 is not advanced enough and the error diverges slowly, especially when there are oscillations because they cause large time-derivatives variations (curvature) that a naive RK4 does not approximate well.
The scipy solver is more advanced, with an automatically-set time step that probably decreases when necessary.
=> On the long term, it might be useful to implement a more advanced RK4, taking for example the scipy implementation and references.
Use of buffers
Now most of the code is common for all solvers, so I tried to optimize it a bit by using temporary buffers and passing as many arguments as possible by reference, especially in the common single derivative function that is common to all solvers.
The code is now cleaner, simpler to understand and to maintain and more efficient.
Compared with the previous version, on the same model (GK) and preset (default):

the homemade solver is a tiny bit slower but that is not relevant because it does more things now (it handles coupling). So the performance cannot be compared directly
the scipy solvers, which have not changed, make use of the buffers in the common function and we get a 20 % faster result

Before:

After:

Computation of statevar
In the previous version:

the statevar were computed during the solver's job,m time step after time step for the homemade solver
were not cmputed by the scipy solver
Now they are computed the same way for all solvers: after the solver has finished computing the ode, using a vetorized (faster) syntax.

N.B.: of course they are also computed during the solver's job, but the result is not stored, it just overwrites a buffer for performance, which is why we need to recompute them afterwards
Unit tests

Non-regression unit tests have been updated and are passing

Caution
In models, it is important that users always define function such that they return output of the same size as the input
For example a function taking arrays should return an array of the same size, not an int or a float
This was one of the reasons I had difficulties debugging, the following (finally found in _model_GK.py) should be avoided:

And this should be used instead:

Issues:

Unless I miss something, this PR fixes issue #58 and all afferent discussions
However I think we should open a new issue to address the long-term implementation of a more advanced RK4. Since this is a significant task in itself (which probably requires some bibliography), I can't offer to do it myself as I will be less available starting next week. Advice from a professional computer scientist might be useful.

Hello @Didou09! Thanks for opening this PR.

In the file _plots.py:

Line 244:45: E226 missing whitespace around arithmetic operator
Line 244:54: E226 missing whitespace around arithmetic operator
Line 248:29: E226 missing whitespace around arithmetic operator
Line 248:35: E226 missing whitespace around arithmetic operator
Line 248:38: E226 missing whitespace around arithmetic operator
Line 252:36: E226 missing whitespace around arithmetic operator
Line 277:47: E226 missing whitespace around arithmetic operator
Line 277:56: E226 missing whitespace around arithmetic operator

In the file models/_model_GK.py:

Line 67:35: E226 missing whitespace around arithmetic operator

In the file utilities/_class_checks.py:

Line 301:25: E126 continuation line over-indented for hanging indent

In the file utilities/_solvers.py:

Line 107:17: E126 continuation line over-indented for hanging indent
Line 111:13: E121 continuation line under-indented for hanging indent
Line 120:26: W291 trailing whitespace
Line 146:37: W291 trailing whitespace
Line 359:1: E303 too many blank lines (3)
Line 366:33: E226 missing whitespace around arithmetic operator
Line 366:56: E226 missing whitespace around arithmetic operator
Line 367:33: E226 missing whitespace around arithmetic operator
Line 367:56: E226 missing whitespace around arithmetic operator
Line 368:48: E226 missing whitespace around arithmetic operator
Line 369:26: E226 missing whitespace around arithmetic operator
Line 369:40: E226 missing whitespace around arithmetic operator
Line 369:68: E226 missing whitespace around arithmetic operator

Do see the Hitchhiker's guide to code style

Excellent really !
Get solvers
Maybe it'll be worth adding more information as the different solvers we will have different abilities ( ODE, PDE, DDE, parallel systems adaptative timestep...), unnecessary for the moment but worth keeping in mind.
The fix
Funny thing is that the first version I coded of the model was with the buffer and on vector to be iterated, scipy style :). I moved from this structure because the code was "fast enough" with less efficient architecture and I didn't knew how to handle it well, yours is working great.
The error in the timestep being at a different place is also my fault, at one moment I didn't knew how to deal with statevar transformed into differential equation, hence this unusual formulation you found in my previous file. I looks like the buffer you created is much more efficient than my old one
The dispatch section is my old "results" dictionary, but with the new structure which is great. It will also allow us to store less iterations if needed (an improvement we can do once into heavy files)
Remaining discrepancy
The best way to look at the convergence is to create an array of dt value then look at the error with the reference, if the system has a slope of 4 then we're fine.
I don't know how scipy estimate errors but a way to be efficient in such system is to look at dydt/y , and decide that we base timestep on this value (maximum evolution of n% per iteration ) , I wrote the theory in a previous issue I can find it back if necessary. I think it is not too important for now the system is still pretty regular. But better solver could be handy someday !


rsync --exclude-from=<(git ls-files -o --directory | awk '{print "/"$0}') is not working under windows

As already described in https://github.com/firecow/gitlab-ci-local/issues/1026.
It seems that the rsync command which is executed by gcl is not 100% working under windows gitbash.
I can run "gcl --list":
parsing and downloads finished in 4.04 s.
json schema validated in 482 ms
name                                           description  stage                  when        allow_failure  needs
threat_modeling                                             pre-build              manual      true
gitleaks                                                    pre-build              manual      true
trivy-fs                                                    pre-build              manual      true
commit-log                                                  pre-build              manual      true
build-develop                                               build                  on_success  false          []
sbom-fs                                                     post-build             on_success  true
authenticate-aws-tooling-ecr                                package-ecr            manual      true           []
package-ecr-jib-aws-tooling-ecr                             package-ecr            manual      true
sbom-ecr-jib-aws-tooling-ecr                                package-ecr            manual      true           [package-ecr-jib-aws-tooling-ecr]
scan-ecr-jib-aws-tooling-ecr                                package-ecr            manual      true           [package-ecr-jib-aws-tooling-ecr]
rollout-eks-dev                                             rollout-eks:dev        manual      true
unrollout-eks-dev                                           rollout-eks:dev        manual      false
scan-eks-dev                                                rollout-eks:dev        manual      false

But I can't run a build job "gcl --stage build":
parsing and downloads finished in 2.04 s.
json schema validated in 356 ms
Error: Command failed with exit code 11: rsync -a --delete-excluded --delete --exclude-from=<(git ls-files -o --directory | awk '{print "/"$0}') --exclude .gitlab-ci-local/ ./ .gitlab-ci-local/builds/.docker/
rsync: [client] failed to open exclude file /proc/1737/fd/63: No such file or directory (2)
rsync error: error in file IO (code 11) at exclude.c(1481) [client=3.3.0]
    at makeError (C:\ProgramData\nvm\v20.17.0\node_modules\gitlab-ci-local\node_modules\execa\lib\error.js:60:11)
    at handlePromise (C:\ProgramData\nvm\v20.17.0\node_modules\gitlab-ci-local\node_modules\execa\index.js:118:26)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Function.rsyncTrackedFiles (C:\ProgramData\nvm\v20.17.0\node_modules\gitlab-ci-local\src\utils.ts:306:9)
    at handler (C:\ProgramData\nvm\v20.17.0\node_modules\gitlab-ci-local\src\handler.ts:89:9)
    at Object.handler (C:\ProgramData\nvm\v20.17.0\node_modules\gitlab-ci-local\src\index.ts:37:21)

It would may be better to redirect the result of git ls-files -o --directory | awk '{print "/"$0}' into a temp file and use this as input for rsync "--exclude-from"?
I'm running gcl 4.54.0 under windows gitbash.
$bash --version
#GNU bash, version 5.2.37(1)-release (x86_64-pc-msys)

Just and idea...
https://github.com/firecow/gitlab-ci-local/blob/32176c68302d4c0834249ce6f083f82ff2894132/src/utils.ts#L306
What about replace
--exclude-from=<(git ls-files -o --directory | awk '{print "/"$0}')
with something like this:
--exclude-from=$(temp_file=$(mktemp) && git ls-files -o --directory | awk '{print "/"$0}' >> "$temp_file" && echo "$temp_file")
the only problem is that this temp_file is not cleaned up automatically 👎

i don't have a windows gitbash environment to test,
https://github.com/firecow/gitlab-ci-local/pull/1380
can you test to see if this fixes your issue ?

hmm, will need to clone the draft and build it locally from your machine...


Added Bower for client dependency management

This allows for better management of the font and icons used by the application UI. The pre-start command I added to the package.json will automatically install the bower dependencies when $ npm start is run. Otherwise $ bower install will also do this. Bower will also decrease the amount of files commited to git and will make it easier to add libraries in the future.

i don't really want to add an additional package manager other than npm sorry. thanks for taking the time to create this pr though.


support shaded version for subscribe-sdk

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


login should return 400 on E_VALIDATION error instead of 500

While trying to integrate waterlock-local-auth with angular recently, I've noticed two things happening on login waterline validation failures:

res.serverError() is removing the response body in production mode, preventing angular from seeing which fields failed to validate
res.serverError() is returning a 500 Server Error when we should really be returning 400 Bad Request

This commit ensures 400 + error body is always returned on E_VALIDATION errors for login, and defaults to res.serverError(err) for anything else.
Thoughts?

We good on this? Am I OK to merge?

Up to you man I haven't had a chance to test it out.

I'll give it a spin on my commute back :)
On Mon, Jun 1, 2015 at 10:20 AM, David<EMAIL_ADDRESS>wrote:

Up to you man I haven't had a chance to test it out.
—
Reply to this email directly or view it on GitHub
https://github.com/waterlock/waterlock-local-auth/pull/33#issuecomment-107378324
.

--
--------------------
w://
t: 07508 215 459
--------------------


Fix regression and warn about deprecated autocommands when they are used

Regression around the "No matching autocommand" message occured at ba7b0f3 and 6feb403.
Instead of just suppressing the error message again, I've implemented the ideal behavior that will warn about deprecated autocommands only when they are used.
By the way, it seems that Jetpack* autocommands are called multiple times because ++once is missing for internal autocommands that call Jetpack* autocommands.
Is this the intended behavior? I think the next patch should be also applied (after merging this PR).
diff --git a/autoload/jetpack.vim b/autoload/jetpack.vim
index 4e42c73..8a335e0 100644
--- a/autoload/jetpack.vim
+++ b/autoload/jetpack.vim
@@ -466,13 +466,13 @@ function! jetpack#end() abort
         endif
       endfor
       let event = substitute(substitute(pkg.name, '\W\+', '_', 'g'), '\(^\|_\)\(.\)', '\u\2', 'g')
-      execute printf('autocmd Jetpack SourcePre **/pack/jetpack/opt/%s/* doautocmd User Jetpack%sPre', pkg.name, event)
-      execute printf('autocmd Jetpack SourcePost **/pack/jetpack/opt/%s/* doautocmd User Jetpack%sPost', pkg.name, event)
+      execute printf('autocmd Jetpack SourcePre **/pack/jetpack/opt/%s/* ++once ++nested doautocmd User Jetpack%sPre', pkg.name, event)
+      execute printf('autocmd Jetpack SourcePost **/pack/jetpack/opt/%s/* ++once ++nested doautocmd User Jetpack%sPost', pkg.name, event)
       execute printf('autocmd Jetpack User Jetpack%sPre :', event)
       execute printf('autocmd Jetpack User Jetpack%sPost :', event)
       " Deprecated autocommand (show a message if used)
-      execute printf('autocmd Jetpack SourcePost **/pack/jetpack/opt/%s/* doautocmd User Jetpack%s', pkg.name, event)
-      execute printf('autocmd Jetpack User Jetpack%sPost ++once call s:warn_deprecated_autocmd(%s)', event, string(event))
+      execute printf('autocmd Jetpack SourcePost **/pack/jetpack/opt/%s/* ++once ++nested doautocmd User Jetpack%s', pkg.name, event)
+      execute printf('autocmd Jetpack User Jetpack%sPost call s:warn_deprecated_autocmd(%s)', event, string(event))
     elseif isdirectory(s:path(s:optdir, pkg.name))
       execute 'silent! packadd! ' . pkg.name
     endif

Thank you so much. Please fix line reviewdog commented.
Futher, Please apply your patch to this pull request. Cheers.

Done!
(Actually I didn't delete the ++once at line 475. It's better to be there.)

Thank you for the fantastic improvement. Cheers 🍻


Update melior to llvm 18

Disclaimer: since tblgen looks unmaintained (last commit is from 8 months, i made a PR there updating to LLVM 18 but no answer so far), i made a fork (https://github.com/edg-l/tblgen-rs-alt) and this PR uses it, its published on crates.io too, if you want i can give you ownership over it.
This pr updates melior to llvm 18. This is a MVP, it doesn't add all the new llvm 18 features, just gets it working.

I dont know why the dialect_build CI fails, i guess its cuz it uses the old llvm verson from main?

I dont know why the dialect_build CI fails, i guess its cuz it uses the old llvm verson from main?

We can ignore it for now.

Thank you for the changes!

if you want i can give you ownership over it.

I think we don't need to do this for now. 👍


Publish settings and others

Fixes https://github.com/47deg/freestyle/issues/1
Temporarily, Snapshots artifacts are being published at https://oss.sonatype.org/content/repositories/snapshots/com/fortysevendeg/freestyle_2.11/, since we do not have permissions to write at io.freestyle domain.
Please @raulraja could you review? Thanks!

@juanpedromoreno right now there are no artifacts in sonatype, container folders are published for scala and sjs but not any files inside. Could there be anything wrong with our current publish settings?


Health route no longer at /health when using depth=1

When starting chartmuseum 0.7.0 with the env variable DEPTH set to 1 the /health route is no longer found at the context root, but at /some-string/health.
As a quick fix for my deployment I adjusted the kubernetes liveness and readiness probes to go to /depth1/health instead of /health in my local copy of the chartmuseum chart.
I assume the health route should remain at the context root regardless of the configured depth.

@fabzo good find, yes it definitely should

@fabzo this hsould be fixed on master. Can you please try with the "latest" docker image tag and let us know if this is working for you? Thanks!

Thanks for the fast response. I will test it tomorrow and report back.

Took me a while to get back to it. I reverted the liveness and readiness probes to the /health route and deployed it with the latest image.
Looks good to me:
{   "M": "[75] Request served",   "path": "/health",   "method": "GET",   "statusCode": 200, ... }
Thanks for the fix :-)

@fabzo great thanks for verifying!


💭 perspective Transform (@shopify/react-native-skia) not work on "frameProcessor" event

Question
Use processTransform3d in the code, rotate is work, but perspective not work.
`
      import { Matrix4, PaintStyle, perspective, processTransform3d, processTransformProps, rotateX, scale, Skia,SkImage, TileMode } from '@shopify/react-native-skia';
      import { useEffect } from 'react';
      import { StyleSheet, Text } from 'react-native';

      import {
          Camera,
          useCameraDevice,
          useCameraPermission,
          useSkiaFrameProcessor,
          useFrameProcessor
      } from 'react-native-vision-camera';

      export function CameraRealtimeDetection() {
          const device = useCameraDevice('back');
          const { hasPermission, requestPermission } = useCameraPermission();

          useEffect(() => {
              requestPermission();
          }, [requestPermission]);

          const frameProcessor2 = useSkiaFrameProcessor((frame) => {
              'worklet';

              var matrix = processTransform3d([
                  { rotateZ: 0.5 },       // work √
                  { perspective: 500 },   // no work ×
              ])

              frame.concat(matrix)

              frame.render()

          }, []);

          if (!hasPermission) {
              return <Text>No permission</Text>;
          }

          if (device == null) {
              return <Text>No device</Text>;
          }

          return (
              <Camera
                  style={[{height:300,width:200}]}
                  device={device}
                  isActive={true}
                  frameProcessor={frameProcessor2}
              />
          );
      }

`
What I tried
No response
VisionCamera Version
4.5.3
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Not enough details, no demo, and wrong issue template.


Can't assign type to masked input

Why can't we assign a type to the masked input?

Didn't realize, the type and class attributes are just undocumented.
My mistake.


config: Add a check if checkup image field is empty

Currently, if the kubelet cannot pull the required image - we wait for the required timeout before failing.
Add a check if the image field exists but empty, in order to not wait in vain for timeout expiration.
Signed-off-by: Orel Misan<EMAIL_ADDRESS>

Change: Answered a review comment.


Assignment status_change_log cites incorrect creator

The log always cites the assigned user as the creator of the assignment, which is not always true.  Sometimes an admin will create the assignments.
The current user's AuthToken (and thus their User object) can be obtained from thread-local storage.
https://github.com/TireSwingSoftware/openassign-server/blob/ddad31093c3820192ac5ce3f6ce6491acdb4b5e1/pr_services/pr_models.py#L1199

I dont see the auth token being stored in thread local storage anywhere. I see in middleware.py the request is being stored. I checked all uses of middleware.get_current_request and did not find anything. Is it possible this functionality (storing auth token) doesn't exist and if so, should I add that as part of the commit?

It looks like I just imagined that the AuthToken object was in thread local storage.  I think it's a good idea to have it there, so go for it.  The first place you might want to consider for placing it there is in pr_services.rpc.service.ShimInvoke._run(), which is where the AuthToken object is first retrieved for every request.


Mobilenet Pretrained model for 4  class custom dataset displays/predicts always 1 (same) class name for any input image

I will try to elaborate my prob. Currently I'm trying transfer learning using a pretrained mobilenet model that was initally trained for 1000 classes. My custom data has just 4 classes.
train_batches = ImageDataGenerator(preprocessing_function=tf.keras.applications.mobilenet.preprocess_input).flow_from_directory(directory=train_path, target_size=(224,224), batch_size=10)
valid_batches = ImageDataGenerator(preprocessing_function=tf.keras.applications.mobilenet.preprocess_input).flow_from_directory(directory=valid_path, target_size=(224,224), batch_size=10)
test_batches = ImageDataGenerator(preprocessing_function=tf.keras.applications.mobilenet.preprocess_input).flow_from_directory(directory=test_path, target_size=(224,224), batch_size=10, shuffle=False) horizontal_flip = True) #for randomly flipping half of the image
Initial model summary:
.....
......
**conv_pw_13_relu (ReLU) (None, 7, 7, 1024) 0
global_average_pooling2d (Gl (None, 1024) 0
reshape_1 (Reshape) (None, 1, 1, 1024) 0
dropout (Dropout) (None, 1, 1, 1024) 0
conv_preds (Conv2D) (None, 1, 1, 1000) 1025000
reshape_2 (Reshape) (None, 1000) 0
predictions (Activation) (None, 1000) 0
Total params: 4,253,864
Trainable params: 4,231,976
Non-trainable params: 21,888**
Modified Model summ
.......
**conv_dw_13_bn (BatchNormaliz (None, 7, 7, 1024) 4096
conv_dw_13_relu (ReLU) (None, 7, 7, 1024) 0
conv_pw_13 (Conv2D) (None, 7, 7, 1024) 1048576
conv_pw_13_bn (BatchNormaliz (None, 7, 7, 1024) 4096
conv_pw_13_relu (ReLU) (None, 7, 7, 1024) 0
global_average_pooling2d (Gl (None, 1024) 0**
dense (Dense) (None, 4) 4100
Total params: 3,232,964
Trainable params: 1,867,780
Non-trainable params: 1,365,184
#Train the model
model.compile(optimizer=Adam(lr=0.0001), loss='categorical_crossentropy', metrics=['accuracy'])
Epoch 1/5
227/227 - 136s - loss: 0.0492 - accuracy: 0.9859 - val_loss: 0.1093 - val_accuracy: 0.9600
Epoch 2/5
227/227 - 134s - loss: 0.0030 - accuracy: 0.9996 - val_loss: 0.1655 - val_accuracy: 0.9600
Epoch 3/5
227/227 - 142s - loss: 0.0026 - accuracy: 1.0000 - val_loss: 0.1175 - val_accuracy: 0.9600
Epoch 4/5
227/227 - 139s - loss: 0.0015 - accuracy: 0.9996 - val_loss: 0.1045 - val_accuracy: 0.9600
Epoch 5/5
227/227 - 140s - loss: 6.2960e-04 - accuracy: 1.0000 - val_loss: 0.1643 - val_accuracy: 0.9600
label_names = sorted(train_batches.class_indices.items(), key=lambda pair:pair[1])
label_names = np.array([key.title() for key, value in label_names])
label_names
Output: array(['Class 1', 'Class 2', 'Class 3', 'Class 4'], dtype='<U7')
img_path = image.load_img('./test/Class 2/img.jpg',target_size=(224,224))
test_image = image.img_to_array(img_path)
test_image = np.expand_dims(test_image, axis=0)
test_image = np.array(test_image)/127.5-1
plt.imshow(test_image.squeeze())
result = model.predict(test_image)
result1 = result[0]
for i in range (4):
if result1[i] == 1.:
break;
p=model.predict(test_image)
print(p.shape)
predict_class = label_names[np.argmax(p[0][0], axis=-1)]
plt.imshow(test_image.squeeze())
plt.title(predict_class)
plt.show()
Problem: While predicting, the model shows always one class name(Class 1) for any type image input. I'm clueless as to how to fix this.. I assume mistake should be with preprocessing the test image.
Thanks in advance. Pease help

i think while training you are using mobilenet_preprocess_input function but while inference you are not using the same preprocessing function thats where every image got classified as one class ....
to fix this  you can use below code
pred_dir  is the place of folder where your images present
def prepare_image(file):
img = image.load_img(file, target_size=(224, 224))
img_array = image.img_to_array(img)
img_array_expanded_dims = np.expand_dims(img_array, axis=0)
return tf.keras.applications.mobilenet.preprocess_input(img_array_expanded_dims)
#modelmobile1  is your trained model
for item in os.listdir(pred_dir):
imagepath = os.path.join(pred_dir,item)
preprocessed_image = prepare_image(imagepath)
predictions = modelmobile1.predict(preprocessed_image)
result.append([item,np.argmax(predictions,axis=1)])
Hope this helps you....


chore: bump postgres version to 15.5

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Bump postgres versiont to 15.5(because 16 is not  yet supported by aws yet) and 11.x have reached end of life
How did you test this code?

Locally tested with postgres 15.5

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

Comparison is base (ce95ac9) 95.95% compared to head (742ca86) 17.86%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3280       +/-   ##
===========================================
- Coverage   95.95%   17.86%   -78.10%
===========================================
  Files        1068      988       -80
  Lines       32721    28666     -4055
===========================================
- Hits        31399     5122    -26277
- Misses       1322    23544    +22222

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


GeoportalSearch in MapPanel doesn't show all results

search for langseth AND "Seismic navigation" AND MGL1111 at http://catalog-dev.iedadata.org:8080/geoportal/#  (note this is dev, likely to change content...) in the searchPanel, I get three results, and all three are listed.
If I do the same search using the GeoPortalSearch widget in the Map panel, it report three hits, but only two display. The first record shown in the searchPanel doesn't show up in the GeoportalSearch results list.
If I upload a test record with the text 'zzyzzx', and search for 'zzyzzx', I the record shows up in the search panel but not the GeoportalSearch on the map panel. I suspect its the same problem

Geoportal 2.6.2 included a new search widget, suggest to upgrade and try if the same issue still there, did a quick search on a test instance, result seems to be consistent.


Fix the last test coverage

Final piece in getting 100% coverage on tri.* on github!

Current coverage is 100.00%

Merging #1 into master will increase coverage by +0.22% as of 6088370

@@            master      #1   diff @@
======================================
  Files            2       2
  Stmts          470     470
  Branches         0       0
  Methods          0       0
======================================
+ Hit            469     470     +1
  Partial          0       0
+ Missed           1       0     -1

Review entire Coverage Diff as of 6088370

Powered by Codecov. Updated on successful CI builds.


Local observations list implemented

Here's the problem I'm trying to solve: there are many (over 4K in my case) observation directories with all sorts of artifacts from observations. Many of them are useless junk (e.g. because SDR or antenna was disconnected). The goal here is to develop automated code that would:

[x] list all locally saved observations and would show their status
[x] allow deleting useless observations
[ ] determine if an observation was uploaded or not
[ ] allow deleting uploaded observations

In the current form, it's not possible to determine if an observation was uploaded or upload it later. The reason is that some crucial metadata (AOS, TCA) parameters are not recorded during observation storage on local disk.
Also, not possible to implement deleting local copy of an reservation that's been uploaded.


Updated Java TLS bug note

Fixes https://cockroachlabs.atlassian.net/browse/DOC-1596.

This change is


Different colors for combinations of Debuffs?

I posted on CurseForge but wanted to post here for visibility.
Is there any way to do 1 color for a specific debuff, but a different color if 2 debuffs exist?
For instance: Enhancement Shamans want Flame Shock on targets and then want to Lava Lash targets to cause the debuff Lashing Flames so they deal more damage. So I'm attempting to have:
1 color for just flame shock
1 color for just lashing flame
1 color for both flame shock and lashing flame on same target.

Interesting concept. I might see if I can put together something for that.

This is now supported.


Add wheels for Windows

The project publishes wheels for macOS and Linux. We must add wheels for Windows too, as requested in https://github.com/wasmerio/python-ext-wasm/issues/24.
cc @xmclark

If I understood correctly, now with pyo3-pack build should be pretty straightforward to build wheels for Windows

Yes. The only required step is to migrate from CircleCI to Azure Pipelines.

@Hywan Is here also any update? Wheels for Windows should really be needed as it is currently very hard to use this on Windows.

I've to migrate from CircleCI to Azure. I can focus on that next weeks.

@Hywan Have you done anything?

We need to configure Azure CI properly, and that will be fast after that. Working on it.

As #100 is now merged, does this mean that Windows will be supported in next release?

Correct!

Let's wait for the next release :-).


Fixes #3509 and adds a simple test program

Tested on Desktop and Android platforms.
The test program is not very efficient, since the btOverlappingPairCache, btCollisionAlgorithm, btPersistentManifold and btManifoldPoint classes do not use pooling.

Thanks!


No assistant found with id 'asst_4Gvk7WJuvcSqPuzUciiGSw8j'

Hey!
After you setup the project and try to run it locally, you get the message (server logs):

Error: 404 No assistant found with id 'asst_4Gvk7WJuvcSqPuzUciiGSw8j'.
at async runAssistant (./src/app/actions/run-assistant.ts:27:17)

Should we create our own Assistant in https://platform.openai.com/ ?

Hi!
Replace the assistant here: https://github.com/felipetodev/wanderlust/blob/main/src/app/actions/run-assistant.ts#L7
and then add your functions like:

Correct, I can't access that with my OpenAI key. Can you share the assistant instructions and function descriptions?


Scan source files for SPDX license ID's

This is a proposed enhancement based on a discussion thread in the license-maven-plugin issue https://github.com/mojohaus/license-maven-plugin/issues/323
The source files could be scanned for SPDX license identifiers declared in the source.  This could be a simple regex applied to the source files which are already being opened and processed for checksums.  It would not take much additional compute time if the regex is efficient.
Any files with an SPDX license identifier would use that license ID for the declared license for that file.

resolved by PR #19


Sample cleanup job & task

What does this PR do and why?
Created a new job and task which accepts the argument days_old and permanently deletes samples older than n.
Fixes #427.
Screenshots or screen recordings
N/A
How to set up and validate locally

Login to irida-next.
Find a sample and delete it.
Verify the sample was soft-deleted.
Open a terminal.
Run the command rake samples:cleanup or rake samples:cleanup[n] where n is days old.
Verify the sample and its attachments and workflow executions are no longer in the database.

To verify a sample was soft-deleted:

Open a rails console.
Run the command Sample.only_deleted.

To verify a sample was removed from the database:

Open a rails console.
Run the command Sample.only.where(id:?).

Note: If you open a rails console and try running any of the following queries, WHERE "samples"."deleted_at" IS NULL is always appended to the database query. Use Sample.only as a workaround.

[x] I have evaluated the PR acceptance checklist for this PR.

This looks good, could you please add this task to the good_job cron config. It can be found in config/application.rb.

6b13d79c4a53acf41e3f2f2669592c40647fa0a7