O-MVLL Python Configuration problem

O-MVLL Python Configuration
[CXX1405] error when building with cmake using /home/mac/AndroidStudioProjects/butterFly/app/src/main/cpp/CMakeLists.txt: -- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /home/mac/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - failed
-- Check for working CXX compiler: /home/mac/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++
-- Check for working CXX compiler: /home/mac/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ - broken
-- Configuring incomplete, errors occurred!
See also "/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/CMakeOutput.log".
See also "/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/CMakeError.log".
C++ build system [configure] failed while executing:
/home/mac/Android/Sdk/cmake/3.18.1/bin/cmake
-H/home/mac/AndroidStudioProjects/butterFly/app/src/main/cpp
-DCMAKE_SYSTEM_NAME=Android
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON
-DCMAKE_SYSTEM_VERSION=21
-DANDROID_PLATFORM=android-21
-DANDROID_ABI=arm64-v8a
-DCMAKE_ANDROID_ARCH_ABI=arm64-v8a
-DANDROID_NDK=/home/mac/Android/Sdk/ndk/25.0.8775105
-DCMAKE_ANDROID_NDK=/home/mac/Android/Sdk/ndk/25.0.8775105
-DCMAKE_TOOLCHAIN_FILE=/home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android.toolchain.cmake
-DCMAKE_MAKE_PROGRAM=/home/mac/Android/Sdk/cmake/3.18.1/bin/ninja
-DCMAKE_CXX_FLAGS=-fpass-plugin=/home/mac/Desktop/omvll_ndk_r25.so
-DCMAKE_LIBRARY_OUTPUT_DIRECTORY=/home/mac/AndroidStudioProjects/butterFly/app/build/intermediates/cxx/Debug/4z1w3t4m/obj/arm64-v8a
-DCMAKE_RUNTIME_OUTPUT_DIRECTORY=/home/mac/AndroidStudioProjects/butterFly/app/build/intermediates/cxx/Debug/4z1w3t4m/obj/arm64-v8a
-DCMAKE_BUILD_TYPE=Debug
-B/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a
-GNinja
from /home/mac/AndroidStudioProjects/butterFly/app
CMake Warning at /home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android-legacy.toolchain.cmake:415 (message):
An old version of CMake is being used that cannot automatically detect
compiler attributes.  Compiler identification is being bypassed.  Some
values may be wrong or missing.  Update to CMake 3.19 or newer to use
CMake's built-in compiler identification.
Call Stack (most recent call first):
/home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android.toolchain.cmake:54 (include)
/home/mac/Android/Sdk/cmake/3.18.1/share/cmake-3.18/Modules/CMakeDetermineSystem.cmake:93 (include)
CMakeLists.txt:10 (project)
CMake Warning at /home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android-legacy.toolchain.cmake:415 (message):
An old version of CMake is being used that cannot automatically detect
compiler attributes.  Compiler identification is being bypassed.  Some
values may be wrong or missing.  Update to CMake 3.19 or newer to use
CMake's built-in compiler identification.
Call Stack (most recent call first):
/home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android.toolchain.cmake:54 (include)
/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/3.18.1-g262b901/CMakeSystem.cmake:6 (include)
/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/CMakeTmp/CMakeLists.txt:2 (project)
CMake Warning at /home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android-legacy.toolchain.cmake:415 (message):
An old version of CMake is being used that cannot automatically detect
compiler attributes.  Compiler identification is being bypassed.  Some
values may be wrong or missing.  Update to CMake 3.19 or newer to use
CMake's built-in compiler identification.
Call Stack (most recent call first):
/home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android.toolchain.cmake:54 (include)
/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/3.18.1-g262b901/CMakeSystem.cmake:6 (include)
/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/CMakeTmp/CMakeLists.txt:2 (project)
CMake Warning at /home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android-legacy.toolchain.cmake:415 (message):
An old version of CMake is being used that cannot automatically detect
compiler attributes.  Compiler identification is being bypassed.  Some
values may be wrong or missing.  Update to CMake 3.19 or newer to use
CMake's built-in compiler identification.
Call Stack (most recent call first):
/home/mac/Android/Sdk/ndk/25.0.8775105/build/cmake/android.toolchain.cmake:54 (include)
/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/3.18.1-g262b901/CMakeSystem.cmake:6 (include)
/home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/CMakeTmp/CMakeLists.txt:2 (project)
CMake Error at /home/mac/Android/Sdk/cmake/3.18.1/share/cmake-3.18/Modules/CMakeTestCXXCompiler.cmake:59 (message):
The C++ compiler
"/home/mac/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++"

is not able to compile a simple test program.
It fails with the following output:
Change Dir: /home/mac/AndroidStudioProjects/butterFly/app/.cxx/Debug/4z1w3t4m/arm64-v8a/CMakeFiles/CMakeTmp

Run Build Command(s):/home/mac/Android/Sdk/cmake/3.18.1/bin/ninja cmTC_54545 && [1/2] Building CXX object CMakeFiles/cmTC_54545.dir/testCXXCompiler.cxx.o
FAILED: CMakeFiles/cmTC_54545.dir/testCXXCompiler.cxx.o
/home/mac/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=aarch64-none-linux-android21 --sysroot=/home/mac/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/sysroot   -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security  -fpass-plugin=/home/mac/Desktop/omvll_ndk_r25.so  -fPIE -MD -MT CMakeFiles/cmTC_54545.dir/testCXXCompiler.cxx.o -MF CMakeFiles/cmTC_54545.dir/testCXXCompiler.cxx.o.d -o CMakeFiles/cmTC_54545.dir/testCXXCompiler.cxx.o -c testCXXCompiler.cxx
error: ModuleNotFoundError: No module named 'omvll-config'
ninja: build stopped: subcommand failed.

explain:
I found this issue after I input the command "source omvll.env" that the document mentioned. These are the file's contents.
export NDK_VERSION=25.0.8775105
export LD_LIBRARY_PATH=${ANDROID_HOME}/ndk/${NDK_VERSION}/toolchains/llvm/prebuilt/linux-x86_64/lib64
export OMVLL_CONFIG=$(pwd)/app/omvll-config.py
export OMVLL_PYTHONPATH=/home/mac/Desktop/Python-3.10.7/Lib
I don't know why this issue happened?? can you help me?
Environment
Target: Android/
O-MVLL Version: OMVLL Version: 1.1.0-86087c7 / 14.0.6git (4c603efb0cca074e9238af8b4106c30add4418f6)
Compilation of O-MVLL: CI

I also had the same problem as you. Have you found a solution yet?


Extending Signal Extraction

Addresses and closes #157.
Now that we've implemented a first pass of channel-level synthetic data generation, it's time to look into more detailed signal extraction techniques.
Currently, we're naively summing across all channels. As #157 outlines, we should start by working on more nuanced signal extraction techniques that can lend us more information.

Yeah, that sounds like a good idea!
On Wed, Aug 12, 2020 at 11:37 PM alex-l-kong<EMAIL_ADDRESS>wrote:

@alex-l-kong commented on this pull request.
In segmentation/utils/signal_extraction_test.py
https://github.com/angelolab/segmentation/pull/160#discussion_r469728209
:

combined_channels = np.stack((base_channel, base_channel + 1, base_channel + 2), axis=-1)

combined_channels = xr.DataArray(combined_channels)

coords = np.stack((np.arange(4), np.arange(4)), axis=-1)

channel_counts = signal_extraction.positive_pixels_extraction(cell_coords=coords,

                                                             image_data=combined_channels)

assert np.all(channel_counts == [0, 4, 4])

+def test_center_weighting_extraction():

this function tests the functionality of center weighting extraction

where we add a weighting scheme with more confidence toward the center

before summing across each channel

Currently, the test that will be pushed is pretty clunky in that I
manually force the extension of signal 1 px beyond the nucleus and membrane
depending on which cells and representations are being looked at (cell 1 is
currently nuclear, cell 2 is currently membrane) in the test function. I
think it would be good (and looking back, what you might have actually been
getting at) to have a parameter in the synthetic channel generating
functions (ex. uncertainty_thickness) that will allow the user to generate
signal beyond the desired nuclear or membrane boundaries.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/angelolab/segmentation/pull/160#discussion_r469728209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADJB47LI5KGK3DJA5ZPV4HLSAOC2NANCNFSM4P2LWMVQ
.

Still a couple things that would need to be addressed which I put in TODOs (previous @ngreenwald and @ackagel comments). But overall, the code should work fine and should have flushed the most significant issues with the previous review out.

OK this should be good to go for now. I won't touch this branch until after Noah's done reorganizing and renaming 😂.


.NET Standard 2.0 Support

I know this has been mentioned in the past and that you were waiting on .NET Standard 2.0.  Is this something that you are ready to do?  I am willing to help get this going.
Thanks,

Awesome, I am willing to help in anyway I can.  I will pull down the alpha
and check it out.
On Mon, Oct 9, 2017, 4:23 PM Patrick Lioi<EMAIL_ADDRESS>wrote:

.NET Core support is quite far along now. We have an alpha published on
NuGet.org and a sample project demonstrating its successful usage.
https://github.com/fixie/fixie/wiki#net-core-support
That 2.x line of development is over a year in the making, so there are a
few rough edges to polish before calling it Done.
However, that's all specifically working with Netstandard 1.x / Netcoreapp
1.x. I haven't experimented to see what would happen if a solution
consumed the alpha while itself targeting Netcoreapp 2.x / Netstandard
2.x or used the dotnet 2.x command line tool version, so there may be
some lurking need for further work on that aspect. Related, I haven't
decided whether there's benefit to requiring consumers to be on
Netcoreapp 2.x as a minimum, or if it's just as well to leave it as is at
1.x.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fixie/fixie/issues/175#issuecomment-335275269, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABfthbMsT1sAu1dfD29hUgdkewyEcfIJks5sqoDegaJpZM4PxjGP
.

@plioi Do you need any help to get this out the door for .Net Core?

@seesharper I've been making a lot of improvements in the 2.x "develop" branch as a final push for a beta release.  I'm targeting a beta release by end of March 3, at which point the most useful help will be to pull it down, try it out, and report bugs as separate Issues.

https://github.com/fixie/fixie/releases/tag/v2.0.0-beta-0001 is ready for adventurous .NET Core users. :tada:


Python Scope

Description
The objective is to explain the scope of variables in a python program and their behavior within the scope.
Domain
Python_Domain
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Successfully merged PR.


Improve Readability

Signed-off-by: Ryan Russell<EMAIL_ADDRESS>

Thank you, Ryan, much appreciated


feat: adding resources for AWS Glue Blueprints, ML Transforms, Sessions, and Workflows

Commits were cherry-picked from upstream contributor to keep attribution however to meet requirements for semantic commit messages, it was rebased and reworded accordingly. Additionally the resources were refactored for libnuke format and standards.
Upstream PR https://github.com/rebuy-de/aws-nuke/pull/1218

@swhite-oreilly would you be willing to test this out if I cut builds for you? I don't currently have an account setup to build and test these 4 resources. What OS do you operate on most? Alternatively you can clone this and run goreleaser build --snapshot --clean --single-target and test it locally.

@ekristen apologies.  I missed this comment and just saw the merge notice.

:tada: This issue has been resolved in version 3.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Feature: add application scopes framework and imp health scope

This PR is mainly to add application scope framework and health scope
This PR lasts several weeks mostly because the spec is changing.
https://github.com/microsoft/hydra-spec/issues/130
https://github.com/microsoft/hydra-spec/issues/144
https://github.com/microsoft/hydra-spec/pull/133
https://github.com/microsoft/hydra-spec/pull/127
https://github.com/microsoft/hydra-spec/pull/177
https://github.com/microsoft/hydra-spec/pull/196
add application scopes framework

[x] 1. add framework for network/health
[x] 2. create health scope instance
[x] 3. check component scope overlap
[x] 4. add/remove Component to scope

add health scope

[x] 1. create a sub project to implement healthscope
[x] 2. periodically aggregate status of health scope
[x] 3. create a service to make health scope available for others
[x] 4. docs for how to use it.

This is same with #160 as we have changed repo, I have to make a new PR.
fixes #9

It seems the version of openssl installed by base image is 1.1.0 which didn't match the requirement of the updated lib.
I'm trying to figure out whether I should use a new base image

@technosophos @hongchaodeng  I think this PR is ready for review now.
I have reset all the commits and split them into four big commits. Each commit is a different part of code so you can review them one by one.

ref https://github.com/oam-dev/spec/pull/156

@hongchaodeng  thanks very much, all comments are fixed in the last commit

@wonderflow i'll review this today.

The rebase that needs to be done is really simple, so feel free to merge as soon as you have rebased. I am excited to see this new addition!

Thanks @technosophos , I've rebased.
I noticed changes from https://github.com/oam-dev/rudr/pull/375 has been overrided by #366. I rebased it back here.

CI has passed, merging


gnome3.adwaita-icon-theme: 3.36.0 -> 3.36.1

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/adwaita-icon-theme/versions.
meta.description for gnome3.adwaita-icon-theme is: ""
meta.homepage for gnome3.adwaita-icon-theme is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.36.1 with grep in /nix/store/4k841z2aprxpq1mr2axmaybl56jbxs8p-adwaita-icon-theme-3.36.1

directory tree listing: https://gist.github.com/932066d71b4edc79ea49cac21cb37a80

du listing: https://gist.github.com/d367af41f84116aef0d86b9596d84b9e

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

845 total rebuild path(s)

277 package rebuild(s)

277 x86_64-linux rebuild(s)
269 i686-linux rebuild(s)
34 x86_64-darwin rebuild(s)
265 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
abiword
adapta-gtk-theme
almanah
apostrophe
aravis
arc-icon-theme
arc-theme
astroid
audacious
avocode
baobab
bibata-cursors
bibata-cursors-translucent
bibata-extra-cursors
bluefish
blueman
brave
calls
chrome-gnome-shell
chromium
cinnamon.mint-y-icons
citrix_workspace
citrix_workspace_19_10_0
citrix_workspace_19_12_0
citrix_workspace_19_6_0
citrix_workspace_19_8_0
citrix_workspace_unwrapped
citrix_workspace_unwrapped_19_10_0
citrix_workspace_unwrapped_19_12_0
citrix_workspace_unwrapped_19_6_0
citrix_workspace_unwrapped_19_8_0
citrix_workspace_unwrapped_20_04_0
contrast
coq_8_10
coq_8_11
coyim
cozy
darktable
deepin.dde-kwin
deepin.dde-launcher
deepin.dde-session-ui
deepin.startdde
deja-dup
denemo
dfeet
dino
dropbox
dropbox-cli
easytag
elementary-xfce-icon-theme

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4k841z2aprxpq1mr2axmaybl56jbxs8p-adwaita-icon-theme-3.36.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4k841z2aprxpq1mr2axmaybl56jbxs8p-adwaita-icon-theme-3.36.1
ls -la /nix/store/4k841z2aprxpq1mr2axmaybl56jbxs8p-adwaita-icon-theme-3.36.1/bin

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

Maintainer pings
cc @hedning @jtojnar @worldofpeace for testing.

News looks okay


[Issue]: Current version not set

What happened?
Running nvm use 22.12.0 won't set the current version.

I had the previous version installed. After running the v1.2.0 installer, Node.js won't work anymore. I uninstalled NVM4W and removed any Node.js reference I could find. Re-installed v1.2.0 and run nvm install lts and nvm use 22.12.0 and it still didn't set the current version.
I tried different prompts/shells: PowerShell for Windows, PowerShell 7, Command Prompt, Bash.
I also tried to run the installer as Administrator.
I verified that Environment Variables are set.

Then I noticed that there was also no folder nvm4w under C:\.
After creating that folder and manually creating the symlink, I finally had a working current version.
C:\nvm4w>mklink /D "nodejs" "C:\Users\jeron\AppData\Roaming\nvm\v22.12.0"

(Probably creating the folder was enough, and adding the symlink was redundant)
Maybe the new C:\nvm4w\ directory is not created during installation and symlink fails silently.
What did you expect to happen?
NVM use should set the current version.
Version
1.2.0 or newer (Default)
Which version of Windows?
Windows 10
Which locale?
English (Default)
Which shell are you running NVM4W in?
Command Prompt, PowerShell
User Permissions?
Standard User, Elevated
Is Developer Mode enabled?
No (Default)
Relevant log/console output
No response
Debug Output
jeron is not using admin or elevated rights.

Windows Version:        10.0 (Build 19045)

Windows Developer Mode: UNKNOWN

NVM4W Version:          1.2.0
NVM4W Path:             C:\Users\jeron\AppData\Roaming\nvm\nvm.exe
NVM4W Settings:         C:\Users\jeron\AppData\Roaming\nvm\settings.txt
NVM_HOME:               C:\Users\jeron\AppData\Roaming\nvm
NVM_SYMLINK:            C:\nvm4w\nodejs
Node Installations:     C:\Users\jeron\AppData\Roaming\nvm
Default Architecture:   64-bit
Mirrors:                No mirrors configured
HTTP Proxy:             none

Total Node.js Versions: 2
Active Node.js Version: none
(run "nvm use <version>" to activate a version)
NVM_SYMLINK does not exist yet. This is auto-created when "nvm use" is run.

PROBLEMS DETECTED
-----------------
"pwsh in jeron" not recognized: the Command Prompt and Powershell are the only officially supported consoles. Some features may not work as expected.

Anything else?
Based on nvm debug results it appears that PowerShell (and also Command Prompt) is not correctly recognized within the Windows Terminal program.

This does indeed look like the installer is not creating the C:\nvm4w directory. The temporary fix is to create that directory manually, then run nvm use lts again.

@coreybutler On Windows 11, If I use the new 1.2.0 installer to use install into a new path E:\nvm and the symlink to E:\nvm\nodejs where are downloads actually getting installed to 😄 notice it downloaded lts into...where??
PS C:\WINDOWS\system32> nvm install lts
Downloading node.js version 22.12.0 (64-bit)...
Extracting node and npm...
Complete

Installation complete.
If you want to use this version, type:

nvm use 22.12.0
PS C:\WINDOWS\system32> nvm list

No installations recognized.
PS C:\WINDOWS\system32> nvm current
No current version. Run 'nvm use x.x.x' to set a version.
PS C:\WINDOWS\system32> nvm use 22.12.0
activation error: Version not installed. Run "nvm ls" to see available versions.
PS C:\WINDOWS\system32> nvm ls

No installations recognized.
PS C:\WINDOWS\system32>

@thadguidry run nvm debug to see where your installs are being saved. Here's my current one as a reference:

Powershell Non-Admin shows:
thadg is not using admin or elevated rights.

Windows Version:        10.0 (Build 26100)

Windows Developer Mode: UNKNOWN

NVM4W Version:          1.2.0
NVM4W Path:             E:\nvm\nvm.exe
NVM4W Settings:         E:\nvm\settings.txt
NVM_HOME:               E:\nvm
NVM_SYMLINK:            C:\Program Files\nodejs
Node Installations:     E:\nvm
Default Architecture:   64-bit
Mirrors:                No mirrors configured
HTTP Proxy:             none

Total Node.js Versions: 0
Active Node.js Version: none
(run "nvm use <version>" to activate a version)
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x28 pc=0x3f4670]

goroutine 1 [running]:
main.checkLocalEnvironment()
        D:/a/nvm-windows/nvm-windows/src/nvm.go:1572 +0x1950
main.main()
        D:/a/nvm-windows/nvm-windows/src/nvm.go:283 +0x645
PS C:\WINDOWS\system32>

Powershell in Admin mode shows:
PowerShell 7.4.6
PS C:\Users\thadg> nvm debug
Running NVM for Windows with administrator privileges.

Windows Version:        10.0 (Build 26100)

NVM4W Version:          1.2.0
NVM4W Path:             E:\nvm\nvm.exe
NVM4W Settings:         E:\nvm\settings.txt
NVM_HOME:               E:\nvm
NVM_SYMLINK:            E:\nvm\nodejs
Node Installations:     E:\nvm
Default Architecture:   64-bit
Mirrors:                No mirrors configured
HTTP Proxy:             none

Total Node.js Versions: 0
Active Node.js Version: none
(run "nvm use <version>" to activate a version)
NVM_SYMLINK does not exist yet. This is auto-created when "nvm use" is run.

IPv6 is enabled. This has been known to slow downloads significantly.

No problems detected.

⚠  We are currently building Author/Runtime to succeed NVM for Windows.

⚠  An upgrade is available: v1.2.1
   run "nvm upgrade" to update.

Find help at https://github.com/coreybutler/nvm-windows/wiki/Common-Issues
PS C:\Users\thadg>

Ah!!!, but a NEW PS Terminal now shows correctly in non-admin mode!
PowerShell 7.4.6
PS C:\Users\thadg> nvm debug
thadg is not using admin or elevated rights.

Windows Version:        10.0 (Build 26100)

Windows Developer Mode: UNKNOWN

NVM4W Version:          1.2.0
NVM4W Path:             E:\nvm\nvm.exe
NVM4W Settings:         E:\nvm\settings.txt
NVM_HOME:               E:\nvm
NVM_SYMLINK:            E:\nvm\nodejs
Node Installations:     E:\nvm
Default Architecture:   64-bit
Mirrors:                No mirrors configured
HTTP Proxy:             none

Total Node.js Versions: 0
Active Node.js Version: none
(run "nvm use <version>" to activate a version)
NVM_SYMLINK does not exist yet. This is auto-created when "nvm use" is run.

IPv6 is enabled. This has been known to slow downloads significantly.

No problems detected.

⚠  We are currently building Author/Runtime to succeed NVM for Windows.

⚠  An upgrade is available: v1.2.1
   run "nvm upgrade" to update.

Find help at https://github.com/coreybutler/nvm-windows/wiki/Common-Issues
PS C:\Users\thadg> nvm upgrade
Installation canceled by user
PS C:\Users\thadg>

So it seems like there needs to be a note on the last page of nvm-setup about
"IMPORTANT! Ensure that you start a NEW terminal after installing nvm-setup.exe, otherwise environment settings will NOT be read correctly."
Let's add that to the last page in the installer?

@thadguidry I'm more perplexed by the fact you installed the lts version and it didn't show up in E:\NVM.
Did you open the original powershell prompt using the installer (i.e. the checkbox on the last screen)?

Scratch the Let's add a note... seems there's more to the issue at hand:
even with a NEW PS window non-admin... debug shows that NVM_SYMLINK:  E:\nvm\nodejs  but yet, it doesn't seem to install under that symlink?  Even if I manually create the E:\nvm\nodejs subfolder... and try again... nvm install lts it still seems to download and install somewhere???
PS C:\Users\thadg> nvm install lts
Downloading node.js version 22.12.0 (64-bit)...
Extracting node and npm...
Complete

Installation complete.
If you want to use this version, type:

nvm use 22.12.0
PS C:\Users\thadg>

I am opening brand NEW terminals.. I've closed ALL terminals and just launched both a NEW PS as admin and without... same thing happens where a nvm install lts seems to download successfully but darn if I can figure out where/how?

I have even removed the nodejs subfolder symlink under C:\Program Files\ just to see what nvm debug then shows which at least now it shows there's an invalid symlink issue...so that's a "good thing".
PS C:\Users\thadg> nvm debug
thadg is not using admin or elevated rights.

Windows Version:        10.0 (Build 26100)

Windows Developer Mode: UNKNOWN

NVM4W Version:          1.2.0
NVM4W Path:             E:\nvm\nvm.exe
NVM4W Settings:         E:\nvm\settings.txt
NVM_HOME:               E:\nvm
NVM_SYMLINK:            E:\nvm\nodejs
Node Installations:     E:\nvm
Default Architecture:   64-bit
Mirrors:                No mirrors configured
HTTP Proxy:             none

Total Node.js Versions: 0
Active Node.js Version: none
(run "nvm use <version>" to activate a version)

IPv6 is enabled. This has been known to slow downloads significantly.

PROBLEMS DETECTED
-----------------
NVM_SYMLINK (E:\nvm\nodejs) is not a valid symlink.

⚠  We are currently building Author/Runtime to succeed NVM for Windows.

Find help at https://github.com/coreybutler/nvm-windows/wiki/Common-Issues
PS C:\Users\thadg>

Reinstalling nvm-setup.exe 1.2.0 doesn't seem to create the symlink back in place either?

PS C:\Users\thadg> nvm debug
thadg is not using admin or elevated rights.

Windows Version:        10.0 (Build 26100)

Windows Developer Mode: UNKNOWN

NVM4W Version:          1.2.0
NVM4W Path:             E:\nvm\nvm.exe
NVM4W Settings:         E:\nvm\settings.txt
NVM_HOME:               E:\nvm
NVM_SYMLINK:            E:\nvm\nodejs
Node Installations:     E:\nvm
Default Architecture:   64-bit
Mirrors:                No mirrors configured
HTTP Proxy:             none

Total Node.js Versions: 0
Active Node.js Version: none
(run "nvm use <version>" to activate a version)
NVM_SYMLINK does not exist yet. This is auto-created when "nvm use" is run.

IPv6 is enabled. This has been known to slow downloads significantly.

No problems detected.

⚠  We are currently building Author/Runtime to succeed NVM for Windows.

Find help at https://github.com/coreybutler/nvm-windows/wiki/Common-Issues
PS C:\Users\thadg> nvm use lts
activation error: Version not installed. Run "nvm ls" to see available versions.
PS C:\Users\thadg> nvm install lts
Downloading node.js version 22.12.0 (64-bit)...
Extracting node and npm...
Complete

Installation complete.
If you want to use this version, type:

nvm use 22.12.0
PS C:\Users\thadg> nvm use lts
activation error: Version not installed. Run "nvm ls" to see available versions.
PS C:\Users\thadg> nvm use 22.12.0
activation error: Version not installed. Run "nvm ls" to see available versions.
PS C:\Users\thadg>

Did you open the original powershell prompt using the installer (i.e. the checkbox on the last screen)?

YES, that's the very first output I showed you at the beginning.  Doesn't matter.. even with closing and reopening a new terminal, we can see that the ENV's are seemingly getting picked up by nvm debug and yet, they are not being used.

@thadguidry do you have an antivirus program running or do you have protected directories enabled? The download process creates a temporary directory and is supposed to move the files to the node install location. It looks like that is happening, but obviously it is not. An error would be thrown if there were a problem, so I'm wondering if AV/protected directories are removing the files right after they're copied to the destination directory (E:\nvm in your case).

Possible... only Windows Defender... let me check...

nothing in Windows Defender has been blocked or quarantined.

Shall I pull down source, compile in Go and debug or output logging to help ya'll troubleshoot?

If you have time to do that, it would be phenomenal, because I cannot recreate the error. I'm currently working on some other patches (the update system had some code that got copied into the release when it should not have). Look for the web.GetNodeJS method.
I'll try to put together a debugging version tomorrow. If you do find the source, or even additional info, I'd be very appreciative.

I forgot to mention, we have a tool for building/running the app called qgo (QuikGo). Once installed, you cd ./src and run qgo build or qgo build --no-cache.

I had a same problem, creating C:\nvm4w manually and running nvm use 22 again solved my problem 👍🏻

@coreybutler what moves or links the files under the root TEMP into the NYM_SYMLINK or perhaps NVM_HOME\nodejs\ ?
I just don't see that happening, but indeed the download and zip extraction happens just fine.
I've added a few DEBUG println statements into the code to help see a bit more and indeed all the Node files are there extracted nicely in the root TEMP folder:
PS F:\GitHubRepos\nvm-windows\bin> ./nvm.exe install lts
DEBUG arch.Validate:  C:\Users\thadg\AppData\Local\Temp\nvm-install-2291855044 22.12.0 64 false
DEBUG URL: https://nodejs.org/dist/v22.12.0/node-v22.12.0-win-x64.zip
Downloading node.js version 22.12.0 (64-bit)...
Extracting node and npm...
DEBUG Extracting node and npm:  <nil>
DEBUG zip:  C:\Users\thadg\AppData\Local\Temp\nvm-install-2291855044\v22.12.0\node-v22.12.0-win-x64
DEBUG root:  C:\Users\thadg\AppData\Local\Temp\nvm-install-2291855044\v22.12.0
DEBUG fs.Move: <nil>
Complete

Installation complete.
If you want to use this version, type:

nvm use 22.12.0

If I do nvm use 22.12.0 is there supposed to be some file movement?

And I've ensured that there's a correctly made (I made it manually just to be sure) symlink
PS F:\GitHubRepos\nvm-windows\bin> ./nvm.exe debug
thadg is not using admin or elevated rights.

Windows Version:        10.0 (Build 26100)

Windows Developer Mode: UNKNOWN

NVM4W Version:          1.2.1
NVM4W Author Bridge:    Not Detected
NVM4W Path:             E:\nvm\nvm.exe
NVM4W Settings:         E:\nvm\settings.txt
NVM_HOME:               E:\nvm
NVM_SYMLINK:            C:\Program Files\nodejs
Node Installations:     E:\nvm
Default Architecture:   64-bit
Mirrors:                No mirrors configured
HTTP Proxy:             none

Total Node.js Versions: 0
Active Node.js Version: none
(run "nvm use <version>" to activate a version)

IPv6 is enabled. This has been known to slow downloads significantly.

PROBLEMS DETECTED
-----------------
The author-nvm.exe file is missing or not executable.

⚠  We are currently building Author/Runtime to succeed NVM for Windows.

Find help at https://github.com/coreybutler/nvm-windows/wiki/Common-Issues

@thadguidry thank you for this.
I've been able to add a debugging mode for the install process, which I'll release in v1.2.1. I based some of the logs off of what you provided. That said, I think this conversation has veered off from the original post. I'll close this with the commit for v1.2.1. If you are still struggling with this after updating to the new version, open a new thread. Your issue is related to the download process whereas this issue is related to the linking process and the installer.
@jerone this is resolved and will be closed/available in v1.2.1 (later today, CST). Unfortunately, there was also an issue (#1199) with the upgrade capability, so it will require a reinstall.

I had the same issue today when nvm use didn't do anything, after spending a few hours, it was resolved by manually creating folder C:/nvm4w because nvm won't make it automatically.

@thadguidry I've released 1.2.1. It will require a reinstall, but now you can run nvm install lts --verbose and it will output a full detailed log of everything it's doing. If that still doesn't work, open a new issue.

@coreybutler Thanks. Confirmed.
The installation process, when NVM4W is already installed, could use a little friendly improvement:

I had to manually remove the folder that NVM4W (should) create itself.

@jerone - I saw that too and was annoyed by it ;-) I added a fix in the 1.2.1 uninstaller, but I can't change it in 1.2.0.


:arrow_up: eslint / redux-little-router / k-redux-factory

closes #137
closes #136

Pull Request Test Coverage Report for Build #287

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.25%

Totals

Change from base Build #281:
0.0%

Covered Lines:
265

Relevant Lines:
266

💛  - Coveralls


ffi bindings for credentials crate

From the crypto crate, foreign languages will need to be able to do the following at the very least:

[ ] Expose PresentationDefinition struct

Foreign languages will need to be able to inspect PresentationDefinition objects, as they are part of a tbDEX Offering. In order to do this, we must expose this type to foreign languages.

The initial scaffold for this is in place in this PR https://github.com/TBD54566975/web5-rs/pull/97
Will create more tickets for more detailed parts of this


feat(ConnectSession):  add method nullSafeGetSkipPermissionCheck,  add param maxConcurrentTaskCount in createPhysicalConnectionSession

What type of PR is this?
type-feature

What this PR does / why we need it:
support execute chart

add method nullSafeGetSkipPermissionCheck;
add param maxConcurrentTaskCount in createPhysicalConnectionSession

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

@LioRoger I've implemented your suggestion, please take a look when you have a chance.

@yhilmare I've implemented your suggestion, please take a look when you have a chance.


problem with returned Optional

Hello,
Is it expected that this code throws assertion, or I'm doing something wrong?
/+dub.sdl:
dependency "optional" version="~>0.7.3"
+/
import optional;

Optional!string f()
{
    Optional!string v;
    return v;
}

void main()
{
    Optional!string o;
    o = f();
    assert(o.empty, "must be empty");
}

Fix released in 0.7.4

Thanks!
пт, 16 нояб. 2018 г., 13:07 Ali Akhtarzada<EMAIL_ADDRESS>
Fix released in 0.7.4
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aliak00/optional/issues/26#issuecomment-439360752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNuSR7IDFp3Am1depJKy4FvCe9xZDC2ks5uvpxbgaJpZM4Yee4f
.


macro should use current sheet instead of Sheet1

current behavior is to use 'Sheet1'. when using the macro to import multiple tests to the same file the user must keep renaming not only the imported sheet but also the next one to 'sheet1' only to probably rename it later.

this would also fix the error thrown if Sheet1 is not found.

this would also fix the error thrown if Sheet1 is not found.


Add 2024-04-01 for StackHCI

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.

/azp run

@NikitaSingh-ms:  You still have not addressed the membership and permissions issues that I called out above.  This PR cannot move forward until you do so.   Please refer to the guide that I linked.

`I have updated the Codeowners file, please have a look now

/azp run

@ArthurMa1978 please take this as a beta release

/azp run net - codeowners-linter - ci


janitor: push all network requests to fetcher

Motivation

refactor the fetch package to achieve separation of concerns:

layer.go handle the per epoch/layer data
handler.go handle peer requests
fetch.go does all the actually peer requests

Changes

create a handler.go to decouple the circular dependency of Logic and Fetcher and allows mocking out network/peers
move the actual peer requests to Fetcher. Logic will handle the sync logic by epoch/layer

Test Plan

unit test, systest
DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

bors try

bors try

try
Build failed:

ci-stage2

unit test timed out

bors try

bors try

try
Build failed:

ci-stage2

https://github.com/spacemeshos/go-spacemesh/issues/3300

bors merge

Build failed:

ci-stage2

https://github.com/spacemeshos/go-spacemesh/runs/7140018303?check_suite_focus=true
all tests passed but ci-stage1 still exits with 1 😒

bors merge


How to update OTA firmware with Home Assistant (zigbee) (TZ-1029)

Question
Hi,
I need some help in OTA update of zigbee device through Home Assistant. How i can do this? I read many documents, i tried and failed to execute OTA update from Home Assistant.
Additional context.
No response

Any update?

i am using espressif zigbee light example with OTA client.

@mundevx Did you try https://github.com/espressif/esp-zigbee-sdk/tree/main/examples/esp_zigbee_ota example?

yes, i tried this example. This for OTA server and client and its working fine. But in my case, OTA server is Home assistant, who will update the firmware through GUI.  I combine espressif zigbee Light example with OTA client example. Now, how i can update firmware through Home Assistant.

@mundevx Could you try again use OTA  client example with Home assistant? BTW, could you share how to select firmware through GUI? I want to check it locally.

@lhespress, I also need answer for this.

@mundevx What coordinator do you have? Do you confirm whether support OTA cluster or not?

@lhespress ,  i am using Home assistant as coordinator? OTA cluster is supported in Home Assistant.

@mundevx I mean which Zigbee coordinator hardware work on Home assistant, eg: deCONZ, EmberZNet or other hardware.

@lhespress , i am working on EmberZNet hardware.

@mundevx

You need run ota_client firmware on H2 Device.
Your Home Assistant ZHA subsystem must be configured for over-the-air updates (OTA).
Copy the updated firmware file into the OTA directory using a file transfer program.
Run the ZHA Toolkitr and push the firmware to the device.

Closing as the questions is answered.


Denoised audio has 0db?

I do not know why. All the audio after denoising is almost silent.
`
{
"dataset": {
"extract_voice": true,
"in_memory_percentage": 1,
"noise_only_percent": 0.02,
"num_condition_classes": 29,
"path": "data/ShowerNoise/",
"regain": 0.06,
"sample_rate": 16000,
"type": "nsdtsea"
},
"model": {
"condition_encoding": "binary",
"dilations": 7,
"filters": {
"lengths": {
"res": 3,
"final": [3, 3],
"skip": 1
},
"depths": {
"res": 128,
"skip": 128,
"final": [2048, 256]
}
},
"num_stacks": 3,
"target_field_length": 1601,
"target_padding": 1
},
"optimizer": {
"decay": 0.0,
"epsilon": 1e-08,
"lr": 0.001,
"momentum": 0.9,
"type": "adam"
},
"training": {
"batch_size": 4,
"early_stopping_patience": 16,
"loss": {
"out_1": {
"l1": 1,
"l2": 0,
"weight": 1
},
"out_2": {
"l1": 1,
"l2": 0,
"weight": 1
}
},
"num_epochs": 15,
"num_test_samples": 50,
"num_train_samples": 450,
"path": "sessions/ShowerNoise",
"verbosity": 1
}
}
I have 500 audio files for training, and inside them, there are 100 files are clean audio, 7 files are noise-only with the silence output. I do not know why.
`

Its not possible for me to identify the issue with only the information you provided. But I see you're using a different dataset. Its likely structured differently than the NSDTSEA dataset we used to train, so you'll want to write a class that handles loading and batching your dataset appropriately.

Yeah, to test it, I also download the clean_trainset from the here. And then synthesize them together with my own noise using
pydub.AudioSegment.overlay()
For more details, the link is here

https://drive.google.com/file/d/1zqnS38KiKF1NNDL8dMqdSXUzfPgxhFV-/view?usp=sharing
This is the noise file. I only combine this audio with the clean_trainset files. For the config.json, I only change the dilation to 7 instead of 9 and the batch size to 4.


Mobile money

@dawbs #13188
### Tasks
- [ ] Add a draft title or issue reference here
```652174057
```rust

> ``

Unclear what this is.


Unable to use native scroller on windows phone (using   $.feat.nativeTouchScroll=true; )

We have hybrid app running on both android and window phone, on which we want to use native scroll er . Following piece of code works on android but the same is failing on windows phone.
$.feat.nativeTouchScroll=true;
Let us know if any thing specific to windows phone has to be done

No, per the documentation native scrolling is not supported on windows phone

Actually is posible, if you put this CSS:
*{
-ms-touch-action:auto;
-ms-overflow-style:auto;
}


Fix: [M3-6455] Add secondary as default buttonType to button component

Description 📝
Since the button refactor we witnessed a regression with buttons without a buttonType prop.
It only saw it with buttons containing only an icon. The fix put forward prevent the purple color to show for those and ensures a Button is never rendered without the proper styles.
Ideally we would not make this prop optional and replace all instances that are missing it, but essentially this fix achieves the same result. The change is not breaking, and all buttons meant to be rendered as primary already have the prop so it seems safe to default to secondary. I took a good look around the app and the change addresses several instances i had witnessed being wrong, and I did not catch any regression.
EDIT: I also edited the secondary palette to default to primary colors since we don't have a secondary set of colors, so in case another component suffers the same regression it won't give us a set of colors we don't support. thanks @bnussman-akamai for the suggestion
Preview 📷

Before
After

How to test 🧪
The two screenshots are respectively from database create and linode editable field but reviewers should test a few more areas to make sure no regression was introduced as part of this PR

@bnussman-akamai that's a very valid concern. I updated the secondary palette to default to primary colors since we don't have a secondary set of colors, so in case another component suffers the same regression it won't give us a set of colors we don't support.


Please make 10.2.13 available for download, for older Mac OS versions

Mac hardware from before 2012 cannot upgrade beyond Mac OS 10.11 El Capitan. Remote Desktop Client 10.2.13 is the last compatible version, and it works great. It was released 5/8/2019, but there does not seem to be anywhere to download it.

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

ID: 5982a31b-7825-9dde-8173-4bbb7cdb159b
Version Independent ID: a3d6fba1-4a23-6f0b-5bad-d7b7862126d9
Content: What's new in the macOS client
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/mac-whatsnew.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

I know it is an old post but I also need this version. Please help!

Same issue, would appreciate some help, Thanks!

Need this one, please help.

I'm found https://s.flipflip.ru/f/Microsoft_Remote_Desktop-10.2.13.dmg

Hey guys,
Anyone still have the latest RDC for El Capitan?
Tks

If some one still need it, and previous link not working.
https://bit.ly/3uOWaPl

https://shrinke.me/TgVRVK6
Download here for el capitan

https://sukerfotograper.blogspot.com/2023/02/microsoft-remot-desktop-untuk-macbook.html?m=1

I don't know what idiot's idea it was to put a million link shorteners and redirects for a download hosted on Google Drive for software they didn't make.
https://drive.google.com/file/d/16g8EWsnuxrz1rM2Dd_SZMrb0drMe5h8R/view?pli=1
Please send a direct link next time instead of shortened urls with so many ads it takes an hour to load the page.

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


⚠ ClamAV has degraded performance

In f03467b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 56 ms

Resolved: ClamAV performance has improved in 9890b5a.


No tests

Can someone write some tests? I'm not sure the code is correct for any given input.

Tests should include coverage analysis.


HmacSha512.cs MinKeySize value should be 32 not 64 bytes

Looking at src/Cryptography/HmacSha512.cs in NSec, I note the comment on lines 24-28 referring to minimum keySize for this function being 32 bytes.
//      Key Size - The key for HMAC-SHA-512 can be of any length. A length
//          less than L=64 bytes (the output length of SHA-512) is strongly
//          discouraged. (libsodium recommends a default size of
//          crypto_auth_hmacsha512_KEYBYTES=32 bytes.) Keys longer than L do
//          not significantly increase the function strength.

Checking the constants defined in libSodium, we can see this is indeed the case in crypto_auth_hmacsha512.h#L19 for the crypto_auth_hmacsha512_KEYBYTES constant:
#define crypto_auth_hmacsha512_KEYBYTES 32U

But looking back into src/Cryptography/HmacSha512.cs it seems to me there might be a mistake. Both minimum and maximum key sizes are set by the same libsodium constant.
public static readonly int MinKeySize = crypto_hash_sha512_BYTES;
public static readonly int MaxKeySize = crypto_hash_sha512_BYTES;

It appears that both MinKeySize and MaxKeySize are set to the length of crypto_hash_sha512_BYTES (64U) exported from libsodium, which is not equivalent to the crypto_auth_hmacsha512_KEYBYTES (32U) value referenced in the nsec comment, nor the value used in libsodium, and therefore I believe also not the correct constant to set here for MinKeySize.
It appears that the library correctly uses crypto_hash_sha512_BYTES to set MaxKeySize, but by using that same value to also set MinKeySize the API is incorrectly preventing the use of 32 byte keys with the HmacSha512 function.
I believe it should be:
public static readonly int MinKeySize = crypto_auth_hmacsha512_KEYBYTES;
public static readonly int MaxKeySize = crypto_hash_sha512_BYTES;

Happy to raise a PR if you concur. Also @ektrah, thank you for maintaining this library!

True, libsodium chooses a value of 32 bytes here. But RFC 2104, Section 3 clearly says that a key length of less than 64 bytes reduces the security strength of HMAC-SHA-512.
Of course, if you want a lower level of security, you can choose lower values. So I think there is no right or wrong with the constants unless you specify your desired security level.
Based on my reading of the RFC, it seems to me that the best strategy for a lower security level is to choose a key length of 64 bytes and then vary the length of the authentication tag. That's what’s currently implemented in HmacSha512.

This RFC was published in 1997, and written years before that. MD5 was the holy grail back then, and 5 years before SHA2 was announced.
The recommendation to have a key as large as the block size comes from the fact that the only proof that HMAC could act as a PRF was using that setting. For any reasonable output size (~ >= 192 bit) that all contemporary hash functions support, the output is indistinguishable from random, so theoretical PRF security doesn't do anything. Truncating the output also immediately achieves PRF security.
More importantly, it has been proven that HMAC offers PRF security with any sufficiently large key length (see When Messages are Keys: Is HMAC a dual-PRF?).
There's absolutely no point in using 512 bit keys. Or to use 512-bit authenticators for that matter. A 256 bit authenticator already guarantees that collisions will never occur.
For all these reasons, I'd recommend just using the regular high-level crypto_auth functions, which uses HMAC-SHA512 and truncates the output.

Thanks both for the insightful replies, much appreciated.

So I think there is no right or wrong with the constants unless you specify your desired security level.

I think that's sort of my point @ektrah, in NSec the MinKeySize is initialised to 64U, whereas in libsodium the minimum is 32U. Regardless of the discussion around key length efficacy, for me it is incorrect to force (without option to override, unless I'm mistaken?) a different set of constants on the user vs. those defined in libsodium.
It looks like there's a constructor signalling intent to allow adjustment of keySize, but from my reading both the MinKeySize and MaxKeySize constraints are set to 64, so I'm not seeing any practical use for the constructor?
Apologies in advance if I'm missing something obvious.

Thanks @ektrah!


Client instance unexpectedly restarts

Sometimes the client instance suddenly restarts and with it all the sandboxes on it.
We are collecting logs to investigate this in depth.

We will be deploying one latest fix, but the root cause should be addressed now.

The restarts were caused by incorrect closing of UFFD handlers.


Feature: Check for and download 'latest' image

I recently had a lab fail because it contained the 'latest' image for SR Linux, and a colleague had a different version of 'latest' deployed locally
When the image name contains the 'latest' tag, Containerlab should check if the latest image is indeed available, and pull an update if not

Hi Jeroen,
this behaviour is in line with docker-compose. Anyways, if you always want to latest image and to perform the check you can define the image-pull-policy as documented here:
https://containerlab.dev/manual/nodes/#image-pull-policy

Given that we have a knob aligned with default behavior of docker/k8s we are not considering changing the default


Potential race on keyspace initialization

I have observed a potential race issue with version 0.6 for Cassandra 3 when the database is clean:
Host replied with server error: java.lang.RuntimeException: java.util.concurrent.ExecutionException: org.apache.cassandra.exceptions.ConfigurationException: Column family ID mismatch (found 67ee9790-a293-11e5-b122-091830ac5256; expected 67e792b0-a293-11e5-b122-091830ac5256

This happens when an Akka Persistence based application is started for the first time. After restarting everything is fine.
To reproduce this an easy way, I suggest you use the Docker image for Cassandra.

I also had this issue several times, especially in the tests. I mentioned that in #121
I think the only way out, as said there and in #116, is to do a retry around schema and table creation. Planning to work on it this weekend. Any other ideas?

Continued here: https://github.com/akka/akka-persistence-cassandra/issues/4


cmd/go: Permission Denied when downloading a private module with go get

What version of Go are you using (go version)?

$ go version
go version go1.18.2 darwin/amd64

Does this issue reproduce with the latest release?
Yes. Assuming 1.18.2 is the latest release.
I tested with go1.16.5 to ensure that it wasn't a config issue on my machine, but I managed to successfully install the same module with go1.16 on the same machine.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/arieroos/Library/Caches/go-build"
GOENV="/Users/arieroos/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/arieroos/Workspace/go/pkg/mod"
GONOPROXY="github.com/Direct-Debit"
GONOSUMDB="github.com/Direct-Debit"
GOOS="darwin"
GOPATH="/Users/arieroos/Workspace/go"
GOPRIVATE="github.com/Direct-Debit"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/arieroos/Workspace/go/go1.18.2"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/arieroos/Workspace/go/go1.18.2/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/arieroos/Workspace/NewProject/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/vf/k2jjqxc950x3ss37bswvhnt40000gn/T/go-build145816414=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I tried downloading a private module by running:
export GOPRIVATE=github.com/Private-Org
go get github.com/Private-Org/go-commons

What did you expect to see?
I expected my private module to be downloaded.
What did you see instead?
I got this output:
go: module github.com/Private-Org/go-commons: git ls-remote -q origin in /Users/arieroos/Workspace/go/pkg/mod/cache/vcs/4371bddb1b5a61a8f85ece4f86eaa40bbac4cc02925be418880bcce25aafa433: exit status 128:
<EMAIL_ADDRESS>Permission denied (publickey).
        fatal: Could not read from remote repository.

        Please make sure you have the correct access rights
        and the repository exists.

Additional Notes

My public key is fine, I can successfully clone the repo in question and  ssh -T<EMAIL_ADDRESS>works as expected.
We tried reproducing the issue on my colleague's machine (Ubuntu 20.04), but it works correctly on his machine.

CC @bcmills maybe?

Show your git config and ssh setup?
also, go get -v -x ... output please.

You need either an insteadOf clause in your .gitconfig or a personal access token in your .netrc. See https://github.com/golang/go/issues/26134#issuecomment-403885803.

Duplicate of #26134

I don't think this is a duplicate of #26134, because I do have this in my gitconfig:
[url<EMAIL_ADDRESS>        insteadOf = https://github.com/

go get -v -x ... output:
mkdir -p /Users/arieroos/Workspace/go/pkg/mod/cache/vcs # git3 https://github.com/Direct-Debit/go-commons
# lock /Users/arieroos/Workspace/go/pkg/mod/cache/vcs/4371bddb1b5a61a8f85ece4f86eaa40bbac4cc02925be418880bcce25aafa433.lock# /Users/arieroos/Workspace/go/pkg/mod/cache/vcs/4371bddb1b5a61a8f85ece4f86eaa40bbac4cc02925be418880bcce25aafa433 for git3 https://github.com/Direct-Debit/go-commons
cd /Users/arieroos/Workspace/go/pkg/mod/cache/vcs/4371bddb1b5a61a8f85ece4f86eaa40bbac4cc02925be418880bcce25aafa433; git ls-remote -q origin
3.141s # cd /Users/arieroos/Workspace/go/pkg/mod/cache/vcs/4371bddb1b5a61a8f85ece4f86eaa40bbac4cc02925be418880bcce25aafa433; git ls-remote -q origin
# get https://github.com/Direct-Debit/go-commons
# get https://github.com/Direct-Debit/go-commons: 404 Not Found (1.131s)
# get https://proxy.golang.org/github.com/@v/list
# get https://proxy.golang.org/github.com/@v/list: 410 Gone (1.423s)
go: module github.com/Direct-Debit/go-commons: git ls-remote -q origin in /Users/arieroos/Workspace/go/pkg/mod/cache/vcs/4371bddb1b5a61a8f85ece4f86eaa40bbac4cc02925be418880bcce25aafa433: exit status 128:
<EMAIL_ADDRESS>Permission denied (publickey).
        fatal: Could not read from remote repository.

        Please make sure you have the correct access rights
        and the repository exists.

and what happens if you do:
cd /Users/arieroos/Workspace/go/pkg/mod/cache/vcs/4371bddb1b5a61a8f85ece4f86eaa40bbac4cc02925be418880bcce25aafa433; git ls-remote -q origin

@seankhliao That command runs successfully. It asks for my private key passphrase and the lists all the refs in the repo.
I also downgraded my project go version to 1.17, which gave the same issue. However, I don't get this issue with 1.16. So it seems that the go get changes between 1.16 and 1.17 might have something to do with this.

@arieroos , I've been fighting with a similar problem and this is what worked for me:
<EMAIL_ADDRESS>Permission denied (publickey) tells that go get is not using a password nor a ssh key.
Whilst commands like git fetch, git pull worked for me on the same repository because I was using an ssh key set up in ~/.ssh/config (as alias), it looks like go get does not take any configuration from ~/.ssh/config.
So next; I checked out why my ssh key was not used. Looking around, I found out that go get needed the key to be added to the ssh-agent. Hence,

I listed the keys added to the agent with ssh-add -l
I verified that the key I had to use for the authentication was not added to the agent
I added the key: `ssh-add ~/.ssh/<your_private.key>
And bingo! go get worked!

I hope this helps.

@al-villalba That worked! Thanks a million!

https://go.dev/doc/go1.17#password-prompts
go doesn't look at (or care about) ssh, it's up to git the user to ensure https://git/repo is translated into something that can be fetched automatically

It's not about what method to use for authentication, ssh or https. It's about how to use ssh+key, as this method is the preferred method for corporate private repositories and often is the only one.
Don't take me wrong, but if I can login to a remote machine with ssh, copy files remotely with scp, fetch content from private repositories with git fetch, using my keys set up in ~/.ssh/config, I will expect go get will be able to download using ~/.ssh/config. And this doesn't seem to be the case.


CNP ini config values with special characters fail silently if not quoted

The Vantiv setup script inserts values into the ini file without quoting. If values contain special characters like &, the php parse_ini_file function silently replaces those values with 0.
For example:
# cnp.ini
username = MyUser
password = my&password
...

yields the following php array:
$config = parse_ini_file('my-file.ini');
print_r($config);

/** Output:
array(
    "username" => "MyUser",
    "password" => "0"
)
*/

This happened to us when Vantiv gave us a production password with an ampersand (&) in it. Everything worked in the test environment, but when we switched to production, everything was failing with a "System Error - Call Vantiv" error message. This was because the config was writing our password as 0, rather than the password with the & sign in it that vantiv had given us.
While it was sufficient to simply quote the password value in the ini file, we had Vantiv change our password to something without an & in it.
Still, though, this should NOT fail silently. It caused an error in our production environment that lost us clients.

Thank you for pointing out the issue. It is fixed and will be available in the next release.


Undefined variable: "$base-font-weight"

Describe the bug
Unable to serve the site because of this error:
  Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/just-the-docs.scss':
                    Undefined variable: "$base-font-weight". on line 17

To Reproduce
Steps to reproduce the behavior:

Create a new GitLab Pages/Jekyll project, then clone it locally, and move to that folder
$ bundle install
$ bundle exec jekyll serve
See that it serves the site at http://<IP_ADDRESS>:4000/jekyll
Add gem "just-the-docs" as documented
Add theme: "just-the-docs" as documented
Repeat 2. and 3.
See that it cannot serve the site because of the error above.

Expected behavior
It should serve the site. Seeing a 404 page would be fine, given the missing docs folder.
Desktop (please complete the following information):

OS: macos Mojave
Ruby 2.6.3
Jekyll: 3.4.0 - same error with 3.8.5
just-the-docs: 0.1.6 - same error with 0.2.4

Additional context
After $ bundle update just-the-docs, and trying again, I got the same error on a different file:
  Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/dark-mode-preview.scss':
                    Undefined variable: "$base-font-weight". on line 17

Same error for me, also in Gitlab

Hi, I just installed a fresh copy of Jekyll and did the following successfully:

jekyll new jtd-gem-test
cd jtd-test
Edit Gemfile and replaced minima gem with just-the-docs
Edit _config.yml and replaced minima theme with just-the-docs
ran bundle install
ran bundle exec jekyll serve

The site served successfully locally. Try using the latest version 0.2.5 of the gem in your Gemfile and tell me if you're still getting the issue. I cannot reproduce it. I also do not use GitLab and have not tested on their pages service so I can't guarantee that any of this will work there.

I am not using Gitlab pages, but I got the same error.

gem install just-the-docs
Added theme: "just-the-docs" to config.yml
cd jekyllsite then jekyll serve


Reuse Greenlight connection

This will reuse the GL gRPC connection and reduce dramatically the number of tls handshakes.
From some tests I ran it shows 4x performance improvments.
This depends on greenlight to deploy their solution to constant URL per node.

@JssDWt @cdecker I am closing this in favor of https://github.com/breez/breez-sdk/pull/304 which seems to me a better approach.


Don't show own templates category in environment page

Don't show own templates category by default in environment page. Because it requires an E2E plugin to sign a message, show e2e pop-up only when the user needs to filter own templates.

more description:

To avoid having user enter pgp key password in Environments page, we need to remove showing Own templates group since templates owners are shown in All templates group when hovering mouse over them.
Also we need to make e2e plugin to ask pwd on Containers page only when a container is attempted to be exported to a template. Currently it asks it when opening the page.


Implemented BindWorkflowAssociationToContentType

Added method BindWorkflowAssociationToContentType to help support attaching workflow to content type.

PR will require manual porting to new PnP-Sites-Core repo + merging with current code

Hi Nanddeep,
Thanks for contributing this PR!
This PR has been open for a long time due to the fact that we've been restructuring the PnP repositories. Today the provisioning engine updates happen in the PnP-Sites-Core library and a lot of changes have been implemented over the last months.
Therefore it's very hard for us to merge this PR as is...we would like to ask you to resubmit this PR (if still relevant) as a new PR in the PnP-Sites-Repo. We apologize for the extra efforts this is resulting into.
This also means that we're going to close this PR in the PnP repo.
Best regards,
The PnP Core team

Hello Bert,
I can understand that, my PR is too old to merge now. I will revisit the recent implementation and raise new PR if required.
Thank you.


Visual mode mapping not working

I have the following lines in my init.vim:
	xnoremap <c-c> "+y

which makes "Ctrl-C" in visual mode copy the selected text.
This works in neovim. I added the following to vscode keybindings.json:
    { "key": "ctrl+c", "command": "vscode-neovim.send", "when": "neovim.mode == visual" },

hoping it would send Ctrl-C to neovim and do the copy. However, pressing "Ctrl-C" in visual mode inside vscode has no effect.

you need to unbind <C-c> from escape key first - escape keys are treated a kind specially:
                        {
				"command": "-vscode-neovim.escape",
				"key": "ctrl+c",
			},

I do have this already. In fact I also tried  and got the same outcome.

Ah, you forgot "args": "<C-c>" in your vscode-neovim.send keybinding

Thanks a lot! That addresses the issue.
Btw, I got my original settings by clicking "copy" here:

Copying this shortcut does not give me the "args" part so I didn't realize this.
Thanks again for your time


EE introspection content view

Provide an action that shows a menu of:

image os information
python packages
ansible version
???

This could use EE labels if available, EE metadata file if one is baked in, or mounts a python script and go get the information with that.

Fixed with #252


[Relay][TE] Add default param name if needed

#10516 used the Relay parameter name when lowering to TE. However, this creates an issue when the parameter name is empty. This is legal in Relay, but results in errors during code generation. For example, this is the generated CUDA kernel for bias add:
extern "C" __global__ void __launch_bounds__(1024) fused_raf_op_tvm_add_kernel0(
    float* __restrict__ T_add,
    float* __restrict__ , /* Name is missing and it results in compile errors. */
    float* __restrict__ _1) {
    T_add[((((int)blockIdx.x) * 1024) + ((int)threadIdx.x))] = ([((((int)blockIdx.x) * 1024) + ((int)threadIdx.x))] + _1[((((((int)blockIdx.x) * 16) + (((int)threadIdx.x) >> 6)) % 54) / 9)]);
}

This PR adds "placeholder" back as a default to make sure no empty string will be passed when lowering to TE.
cc @Lunderberg @tkonolige

Thank you for the catch!  Is there a test that should have caught this issue?

Agreed with @Lunderberg! Perhaps a regression test will be helpful (I don't know how hard it is to do so though)


[Dev] Move to Storybooks 3.2

Updates the storybook server to handle hierarchy, and removes the need to include your storybooks in the app - the list is pre-generated now.

I'll update the code extension after. I handle reaction.

Warnings

:warning:

New dependencies added: react-native-storybook-loader.

react-native-storybook-loader
Author: Chris Gestred
Description: A CLI for dynamically importing stories into [react-native-storybook](https://github.com/storybooks/react-native-storybook).
Homepage: https://github.com/elderfo/react-native-storybook-loader#readme

Created5 months agoLast Updated17 minutes agoLicenseMITMaintainers1Releases14Direct Dependenciescolors, dot, findup, glob and yargs

README
# react-native-storybook-loader
A CLI for dynamically importing stories into react-native-storybook.

Purpose
While using storybook for React Native, I repeatedly found myself manually creating a file with imports for all my stories. So I built an automated way to do it. react-native-storybook-loader can be run using configuration in your package.json or via the CLI interface and it generates a file with all the imports for you.
Installation
yarn add react-native-storybook-loader -D

Or
npm install react-native-storybook-loader --save-dev

Quick Start
Create a React Native project using create-react-native-app
create-react-native-app AwesomeProject

Add react-native-storybook to the project using getstorybook
cd AwesomeProject
getstorybook

Install react-native-storybook-loader
yarn add react-native-storybook-loader -D

Update index.js file in the ./storybook directory to point to storyLoader.js (this file will be generated before launching Storybook).
import { getStorybookUI, configure } from '@storybook/react-native';
import { loadStories } from './storyLoader';

configure(() => {
  loadStories()
}, module);

const StorybookUI = getStorybookUI({ port: 7007, host: 'localhost' });
export default StorybookUI;

Add the rnstl cli to the scripts tag of the package.json
{
  "scripts": {
    "prestorybook": "rnstl"
  }
}

Run Storybook
yarn storybook

Or
npm run storybook

Run react-native on the targeted platform
react-native run-android

Or
react-native run-ios

Note: If you have problems connecting from your device to Storybook using Android issue the following command: adb reverse tcp:7007 tcp:7007.
Configuration
Story loading is controlled by the react-native-storybook-loader section of the project's package.json.
Options

Setting
CLI Option
Type
Description
Default

searchDir
--searchDir
string or string[]
The directory or directories, relative to the project root, to search for files in.
Project root

outputFile
--outputFile
string
The output file that will be written. It is relative to the project directory.
./storybook/storyLoader.js

pattern
--pattern
string
The pattern of files to look at. It can be a specific file, or any valid glob. Note: if using the CLI, globs with **/*... must be escaped with quotes
./storybook/stories/index.js (The default React Native storybook file)

--silent

Silences output. This option is not supported in the package.json file.

Note: When using the CLI, any of option passed will override the values in the package.json

Examples:
Both examples below will search src and packages directories recursively for files that end with .stories.js and write the output to ./storybook/storyLoader.js
package.json
{
  "name": "AwesomeProject",
  "scripts": {
    "prestorybook": "rnstl"
  },
  "config": {
    "react-native-storybook-loader": {
      "searchDir": ["./src", "./packages"],
      "pattern": "**/*.stories.js",
      "outputFile": "./storybook/storyLoader.js"
    }
  }
}

CLI
Breaking Change
CLI can now be accessed from a terminal
./node_modules/.bin/rnstl <options>

or in package.json
{
  "scripts": {
    "prestorybook": "rnstl <options>"
  }
}

Note: When using a glob with **/* it is required to be wrapped in quotes
There is no longer a need to use node ./node_modules/.bin/rnstl <options>.
$ ./node_modules/.bin/rnstl --searchDir ./src ./packages --pattern "**/*.stories.js" --outputFile ./storybook/storyLoader.js

Support
Please log issues
Contributing
Coming Soon

yarn why react-native-storybook-loader output
Has been hoisted to "react-native-storybook-loader"This module exists because it's specified in "devDependencies".Disk size without dependencies: "980kB"Disk size with unique dependencies: "1.84MB"Disk size with transitive dependencies: "2.5MB"Amount of shared dependencies: 24

  Generated by :no_entry_sign: dangerJS

Dope 👍


PT-598: Publish event accessibility fix

Description :sparkles:
Issues :bug:
Closes :no_good_woman:
DEV-XXX:
Related :handshake:
Testing :alembic:
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #134 (f4fb0bb) into master (856646c) will decrease coverage by 0.06%.
The diff coverage is 63.63%.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
- Coverage   80.84%   80.78%   -0.07%
==========================================
  Files         172      172
  Lines        3039     3050      +11
  Branches      976      986      +10
==========================================
+ Hits         2457     2464       +7
- Misses        561      564       +3
- Partials       21       22       +1

Impacted Files
Coverage Δ

src/domain/event/EventSummaryPage.tsx
65.93% <63.63%> (-0.32%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #134 (f4fb0bb) into master (856646c) will decrease coverage by 0.06%.
The diff coverage is 63.63%.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
- Coverage   80.84%   80.78%   -0.07%
==========================================
  Files         172      172
  Lines        3039     3050      +11
  Branches      976      986      +10
==========================================
+ Hits         2457     2464       +7
- Misses        561      564       +3
- Partials       21       22       +1

Impacted Files
Coverage Δ

src/domain/event/EventSummaryPage.tsx
65.93% <63.63%> (-0.32%)
:arrow_down:

Continue to review full report at Codecov.

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


Merge redirection rule fixes

Merge redirection rule fixes

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Use the new gpt-3.5-turbo model

We want to explore how are the stories generated using the new chatgpt api model
Here's the documentation of the new model
https://platform.openai.com/docs/guides/chat
It should be as easy as changing the usage of the old endpoint for the new one and configure parameters properly.
But first we need to update the open ai gem.
We are using this gem:
https://github.com/alexrudall/ruby-openai
Otherwise you wont be able to use the new endpoint.
Please make sure you update every usage of the gem to the new endpoint.
They are mostly used in the jobs (app/jobs directory)
Take as an example this:
@client.completions(
      parameters: {
        model: 'text-davinci-003',
        prompt:, max_tokens: (DAVINCI_MAX_TOKENS - prompt.size)
      }
    )

That's using davinci 003
You need to update those calls to use gpt3.5-turbo
Please refer to the gem's documentation to do it accordingly.
You would also want to update the DAVINCI_MAX_TOKENS constant to 4096
The maximum number of tokens allowed for the generated answer. By default, the number of tokens the model can return will be (4096 - prompt tokens).

Hi @silva96, nice to meet you! I would love to take on this task. I'm still a beginner, but I'm excited to learn more about Turbo. It may take me a bit of time, is that okay?

I would also love to take a crack at this.

Hey there, thanks for giving us the opportunity to learn and grow. 😄 I am currently working on this issue.

Hey, @heyextra @JuzerShakir @Huiclaire here's the thing: I went to sleep and I got 3 offers from you three, but @JuzerShakir started right away.
As @Huiclaire requested the issue first, I'll give her the opportunity to take this issue and pay the bounty if the result is OK.
But also, as @JuzerShakir already opened a PR, I'll give him the chance to correct it and see how it works.
I'll keep the best result of both, but will pay both if both PR's are good.
@JuzerShakir for the next time, don't start a PR if I didn't assigned you to work on it. In this case, @Huiclaire asked first so I'd have assigned to her.
Thanks all!
@Huiclaire let me know if you are ok to start.

Sorry 🙏 @silva96 ,really sorry! I am new to this and I let my excitement carry me away. I have understood and I won't repeat that again!

Thanks everyone, the task is done, I just accepted @JuzerShakir PR because @Huiclaire didn't submit or respond anything.

Hi @silva96 , sorry for the late reply. I'm in Taiwan so the time difference may have caused the inconvenience. But thank you for assigning it to me,  and I hope we can have the opportunity to work together in the future!


create pulse penetration ratio feature extractor [2]

ask Zsofi for the exact explanation
Ratio of number of points within a selected layer to all point within the cylinder.
More possibilities to calculate:

[ ] ratio between ground, non ground
[ ] ratio within specific height layer to total number of points (depends on #34)
[ ] number of points above mean to total number of points

All feature extractors need to implement the feature extractor interface from #19.

Before I make a pull request, the ground points are labelled by a 2 in the raw classification right ?

yes 2 should be the ground points

thanks


Q: can I use this library to hook View methods?

Hello there,
Thanks for providing this awesome library.
I'm currently interested in hooking the standard android.view.View class public methods. Would it be possible to do this using this library? Or does it only support the code that is being built as part of my application?
Thank you!

@ibradwan Hey, at this time it can only hook your own code (not 3rd party libraries). We've had some other inquiries along the same lines in the past, and have tried a couple things to allow 3rd party library hooks but weren't successful in getting it to work.

I see, thanks a lot @eschlenz.


Update the OS values after the package upgrade of admin server

(cherry picked from commit 6e937d51ab8b610bac861cb8c02126fba483adda)
Backport of https://github.com/crowbar/crowbar/pull/2281

 Bug 1006770 - during upgrade, default_os and target_platform set incorrectly

travis: https://github.com/crowbar/crowbar/pull/2283

The change is in upgrade script which is not executed by gating job anyway; @MaximilianMeister could we merge?


SSH CONN ERROR: connect ECONNREFUSED <IP_ADDRESS>:22

Excuse me
chrome-page:
1、Input control can not input
2、Menu cannot be opened
3、SSH CONN ERROR: connect ECONNREFUSED <IP_ADDRESS>:22
so? i should to do.   I am a newbie.
clone code，npm install ，and npm start  ，open page
windows 10

Do you actually have an ssh server listening on <IP_ADDRESS> port 22 on windows 10?

oh，My local ssh server is broken.
so,after changing to ssh on the server, it succeeded.
thank you.


Update on support of multi-line and copy past on V2 version

Hi,
Multi-line support on tables is a blocker on our project, where we integrated quill.js v2. Could you share if there are technical solutions that you can think of but you didn't implement them because you don't need it, or if it is just not possible from your perspective?
Thank you !

Quill-better-table have already support Multi-line text in table cells, you can simply integrate and use.
About copying multi-line text and pasting into table cells, I have already solve it in my own project, it need to modify the quilljs original code. It was mentioned in the issue #33 .
Hope this helps:)


Fix defaults in EC2 creation (176 issue)

I've found and tested that if we replace vpc_id and vpc_subnet_id with empty value Ansible uses the appropriate defaults.

Related to #176


Blob PDF Support

Hi - I'm evaluating Vercel Blob for usage in a project, so I've been uploading and testing with different file types. Images uploaded to Blob render when using img or the NextJS Image component. Video files successfully play in a video tag. However when I upload a PDF to blob, and use iframe, embed or object HTML tags to render, the PDF is downloaded to my computer instead of being viewable inline on my HTML document. Is there a supported way to display PDFs from blob on a HTML page?

hey, the issue is on our roadmap, we are planning to allow embedding more mime types soon

glad to hear, thanks for sharing

@PeytonCleveland @jferrel7 the latest release of @vercel/blob creates pdf files with an inline content-disposition, allowing you to display them as you wish 👍

That's very nice @vvo 👏

@mfts AND we're working on a feature where you can add ?download=1 to any blob URL and it will force downloading the file. This will be the best of both worlds: you can show a pdf preview (page, or new tab directly) AND you can create a download link directly to them also (useful in many cases instead of asking customer to "right click, save as").
WDYT?

@vvo that will be a great feature!

you can add ?download=1 to any blob URL

Wouldn't that allow anyone else to add the ?download=1 param and download the file even if that's not intended?

Wouldn't that allow anyone else to add the ?download=1 param and download the file even if that's not intended?

@mfts If you can open or display a PDF file in your browser, you can download it already (save as). Adding ?download=1 is only a way to create download URLs for developers using Vercel Blob explicitly rather than you having to go through weird proxying to make it happen.
Is that ok for you?


feat: add isURL utility and parse urls without protocol

Happy to flip default or disable parseURL support for protocol relative URLs but I think better to accept in parseURL because of prevalence of them in the wild.

Codecov Report

Merging #16 (4445e56) into main (e0a1599) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
+ Coverage   86.91%   87.03%   +0.12%
==========================================
  Files           7        7
  Lines         214      216       +2
  Branches       60       62       +2
==========================================
+ Hits          186      188       +2
  Misses         28       28

Impacted Files
Coverage Δ

src/parse.ts
87.50% <100.00%> (ø)

src/utils.ts
98.43% <100.00%> (+0.05%)
:arrow_up:

Continue to review full report at Codecov.

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

@danielroe More I'm thinking isURL, should do real URL validation rather than checking protocol existence. Something like http://. is invalid and it can cause troubles if isURL being used in conditions to trsut a user input (see examples in this list: https://mathiasbynens.be/demo/url-regex)
What do you think if in this PR:

Keep/Add support for protocol relative parsing
Add flag to has protocol util to optionally allow relative syntax

And move isURL task to new PR with proper regex (or parser based on perf and complexity)?

@pi0 Sounds like a good plan. I'll make a stab.
(Separate comment) I wonder about differentiating utils that parse/validate input in accordance with spec compliance vs 'cheap' utils that assume pre validation.

@danielroe Agree for naming.  hasProtocol since even it is a cheap implementation, it's name is clear it is not validating entire URL.
BTW generally, there is not single standard for URL to follow and this PR is an example about conventions that are supported. (another example plus as space encoding)
My main goal for ufo is making stable utils for unversal/consistent behavior and sticking to standards as much as possible.


[Bug] Error when trying to delete by filter

Is this a new bug?

[X] I believe this is a new bug
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I've tried using the example docs on https://docs.pinecone.io/docs/metadata-filtering#deleting-vectors-by-metadata-filter to delete vectors by filter:
await index._delete({       filter: {           category: "favorites",       }, });
, and I receive error message:
PineconeError: PineconeClient: Error calling _delete: PineconeError: PineconeClient: Error calling _deleteRaw: RequiredError: Required parameter requestParameters.deleteRequest was null or undefined when calling _delete.
Expected Behavior
Expect the query to execute.
Steps To Reproduce

Add vectors to index with some meta data.
Ensure library is up to date: 0.0.14
Attempt to delete vectors using that meta data.

import { PineconeClient } from '@pinecone-database/pinecone';
const pinecone = new PineconeClient();
const index = pinecone.Index("example");
await index._delete({
  filter: {
      category: "favorites",
  },
 });
Relevant log output
error [PineconeError: PineconeClient: Error calling _delete: PineconeError: PineconeClient: Error calling _deleteRaw: RequiredError: Required parameter requestParameters.deleteRequest was null or undefined when calling _delete.]

Environment
- **OS**: Windows
- **Language version**: Node.js 18.14.0
- **Pinecone client version**: 0.0.14

Additional Context
No response

@CobaltBlueDW you'll need to put the object in a deleteRequest property. I looked inside the DeleteOperationRequest type, changed my argument and now it works for me.
await index._delete({
    deleteRequest: {
        filter: {
            category: "favorites",
        },
    }
});

Having a similar bug. Currently trying the above solution and getting a response of "{}". Hopefully someone from Pinecone will either fix the bug or the documentation, whichever is incorrect.

@jgard1 You've probably figured this out or moved on by now, but there's a missing top-level deleteRequest key needed as in the example above. Also (I'm not sure but) you may need to use the $eq and wrap your two conditions in an $and along the lines of
const deleteRequest = {
  filter: {
    $and: [
      genre: { $eq: "documentary" },
      year: { $eq: 2019 }
    ]
  }
}
await index._delete({ deleteRequest })

We shipped a big update in v1. Check out the release notes and v1 migration guide to see what's new.


[New game]: Johnny Smiter

🎮 Game Request
Welcome to Jar, a once peaceful land, now plauged by the evil fire creatures of the 13th realm.
As the 13th Knight of Jar it is your sacred duty to defend the lands of Jar from the invading hoards, banishing the evil creatures back to their fiery realm with your blessed spear.
But, be warned the presence of these creatures can burn the souls of even the most righteous of men.
Point down the features
Game points will be awarded as per smotes defended
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @S-ishita ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Fixes #170: Read description from .obsidian/obsidian.yaml file

[X] Have you created an issue and used it in the commit message?
[X] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[X] Have you built the project locally prior to submission with mvn clean install?

Don't we know the name of the Obsidian descriptor file from the catalog?

Only the obsidian-addon has access to the catalog. Catapult doesn't know about it

But I guess it should perhaps be provided as part of the Projectile instead and we need to figure out some way to pass that to CatapultResource

@gastaldi Right, TBH I'm hesitant about this because of the guesswork involved.  Let's try for something more predictable.  Passing the state into the Projectile makes more sense to me than having Catapult be aware of the obsidian descriptor or catalog.  Better separation of concerns that way.

Deal, rejecting this PR


[KeyVault] @azure/keyvault-secrets is missing deletedDate after deleting a secret

As reported by our customers.

I believe this is related to either specifying or not specifying the soft delete mode.

This issue is rather simple technically, but requires considerable feedback to solve properly. Here's what's going on:
At the common KeyVault core, all of the delete operations do have a deletedDate property available. For example, for secrets, the interface called DeletedSecretBundle has it here:
https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/keyvault/keyvault-secrets/src/core/models/index.ts#L296-L311
Which is indeed being used by the core's deleteSecret function:
https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/keyvault/keyvault-secrets/src/core/keyVaultClient.ts#L1365-L1416
The issue arises at the src definition of deleteSecret. There, we're retrieving the actual secret from the service using the bundle that we get from the delete operation. Therefore, we are currently not returning the object that contains deletedDate! See it here:
https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/keyvault/keyvault-secrets/src/index.ts#L245-L266
To solve this, we can easily just return the object right away from the core's deleteSecret. However: This would be a breaking change! (it will require other changes in the test files, but that's not really an issue). Should we apply that breaking change?

If I understand correctly, we could change the call to another function like getSecretFromSecretBundle like getDeletedSecretFromDeletedSecretBundle. Then, we could add the deletedDate to this return. The new object would just have some additional new properties? (if so, it wouldn't be a breaking change as it's unlikely to have any negative impact on anyone using the SDKs)

@jonathandturner this is what Alex suggested over email. This suggestion is fijne! The only issue with it is that this returned object will not be the same type as a returned secret, or as a secret bundle, it will be another type, which might not be compatible with the current APIs. But that doesn't mean I'm in disagreement, I'm just pointing it out. I'll send you those emails.

Ok I think I understood what needed to be done. PR soon.


building error which missing string_view

Describe your problem
building vineyard graph -> cannot find libgrape-lite -> building libgrape-lite (contains string_view)
vineyard graph also require string_view and it cannot be found

Hi @wuyueandrew, I have checked the installation of the latest vineyard, but don't encounter the error you reported. Could you please try the latest version again refer to the doc. If you encounter any errors after that, please feel free to let us know, and we'll handle them as soon as possible.

Hi @wuyueandrew, I have checked the installation of the latest vineyard, but don't encounter the error you reported. Could you please try the latest version again refer to the doc. If you encounter any errors after that, please feel free to let us know, and we'll handle them as soon as possible.

i built v6d  base on docker image ubuntu:latest, maybe ur env already install string_view.
shall i paste my dockerfile here ?

The patch #1289 has been merged. It was indeed a bug. Thanks for fixing.

thanks


Refactor inflight tracking and notify on completed instead of polling

This should get rid of any delays by using notifications.

Codecov Report

Merging #329 (3ec2eaf) into main (7aa0e41) will decrease coverage by 0.27%.
The diff coverage is 0.00%.

:exclamation: Current head 3ec2eaf differs from pull request most recent head 178c53e. Consider uploading reports for the commit 178c53e to get more accurate results

@@            Coverage Diff             @@
##             main     #329      +/-   ##
==========================================
- Coverage   66.64%   66.37%   -0.28%
==========================================
  Files          66       66
  Lines        3925     3943      +18
==========================================
+ Hits         2616     2617       +1
- Misses       1309     1326      +17

Impacted Files
Coverage Δ

common/mqtt_client/src/lib.rs
28.03% <0.00%> (-2.18%)
:arrow_down:

tedge/src/cli/mqtt/publish.rs
0.00% <0.00%> (ø)

mapper/collectd_mapper/src/batcher.rs
82.58% <0.00%> (+0.44%)
:arrow_up:

Continue to review full report at Codecov.

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


Upgrade integrations to the last version

Description
This PR update the versions of the integrations
Issues Resolved
Closes #445
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Replace with https://github.com/wazuh/wazuh-indexer/pull/447

Replace with https://github.com/wazuh/wazuh-indexer/pull/447


java.io.IOException: An existing connection was forcibly closed by the remote host  [JIRA: CLIENTS-891]

I run riak java client 2.0.4 in Windows and RHEL against riak server 2.0.6. I get the below error only in Windows and sometimes i see puts and get fail when this happens. Any reason as why this happens and any possible fix available?
ERROR [nioEventLoopGroup-3-3:]  com.basho.riak.client.core.RiakNode - Operation onException() channel: id:414833480 :8087 java.io.IOException: An existing connection was forcibly closed by the remote host

Hi @pradheeps,
We've received your GitHub Issue and ZenDesk ticket.  The issue you are seeing is that a network connection was forcibly closed by the Riak server, Load Balancer, or something in between your client and host. The client will take care of this case, so there is nothing for you, the user, to do.  If this happens often you may have a bad piece of hardware.
You should also see another log message that indicates what was happening with this connection at the time it was closed.  If the log message says "inAvailable channel closed", then the connection was not being used at the time. If it was being used you'll see another log message from the RiakNode class, and the client's retry logic should retry the operation on a different node/connection until it succeeds or runs out of retries.
I've also opened an issue(#628) to downgrade the severity of that log message, as it is an expected occasional state.
Thanks,
Alex

Hi @pradheeps,
We haven't heard back so I'm going to close this ticket.  If you were able to find more information about the order of events in your logs, please reopen it.
Thanks,
Alex

Need more information to decide if it's a bug or a user error.
[posted via JIRA by Alexander Moore]


node_modules folder not shown after installation

VSCode Version: 1.43.2
OS Version: macOS 10.14.6

Steps to Reproduce:

Open a React app or another project
Type yarn or npm i in terminal
After installation is finished, you need to relaunch vscode to see node_modules folder

Does this issue occur when all extensions are disabled?: Yes

So this does not happen when you disable all extensions? Which extension causes this?

Sorry, there was a mistake, It does occur when all extensions are disabled too.

Sorry, there was a mistake, It does occur when all extensions are disabled. I just edited issue.

I can not reproduce this.  Sounds like a missing file events
Can you please post the output if you run vscode with the verbose flag from the command line. So we see if the file events are coming. Thanks

Yeah, files created but there is no folder at the end till I relaunch.


feat(compare_map_segmentation): add dynamic map loading for voxel_based_compare_map_filter

Description
This PR to solve this issue 3085  for voxel_based_compare_map_filter nodelete.
The PR utilize the dynamic map loading interface to update current utilize pointcloud map cells.
Related links
TIERIV COMPANY INTERNAL LINK
https://github.com/autowarefoundation/autoware_launch/pull/257
Tests performed
I tested on sample rosbag from Autoware tutorial using split map here: sample-map-rosbag_split.zip

I have comfirmed that the performance is the same as current Autoware when use_dynamic_map_loading is set as false
I have confirmed that when use_dynamic_map_loading is set as true: the performance is almost the same with current Autoware, with very small degradation. The reason is:
。This PR storing voxel grid information of each current surrounding map cells + map cell position and size. However, the precise map cell region and position still cannot receive from map loader interface yet. It is still discussed here: https://github.com/orgs/autowarefoundation/discussions/3328. So map cell position and region are calculate by searching max and min pointcloud.
。So some input poincloud located between calculated map cell region will be passed through filter. However, this issue occurrs mostly around map cells with sparse pointcloud which far from lanelet and which should not have significant effect to the vehicle behavior.
This will be fixed after map loader interface is updated.

Current Autoware result:
rviz pc rendering, pink:filter output

Poincloud number:
red: filter input, blue: filter output

Updated Autoware:
rviz pc rendering, pink:filter output

Poincloud number:
red: filter input, blue: filter output

Notes for reviewers

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

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

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

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

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

[x] There are no open discussions or they are tracked via tickets.
[x] The PR is ready for merge.

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

@badai-nguyen
Hi, could you explain what does mutex_ptr_  protect?
I can see from your code that current_voxel_grid_dict_ is modified while mutex_ptr_ is locked, but reading of it is not protected.
I encountered a race condition bug at addMapCellAndFilter in the latest Autoware, but I cannot reproduce the race condition bug any more (that is what a race condition bugs are :(.
pthread_mutex_lock.c:94: _pthread_mutex_lock: Assertion `mutex->data.__owner == 0' failed.

I can see from your code that current_voxel_grid_dict_ is modified while mutex_ptr_ is locked, but reading of it is not protected.

@veqcc Thank you a lot for your pointing out my mistake.
Yes, some reads of current_voxel_grid_dict_ are missing the lock on mutex_ptr_.
Let me fix that.