arm-linux build problem with any ports with wrong search path

when i want to cross compile it use wrong search paths so it build correctly but can not find compiled binary
how can i modify search path
the /opt/poky/3.1.3/sysroots/armv7vet2hf-neon-poky-linux-gnueabi/home/mohsen/vcpkg/packages/fmt_arm-linux/lib/pkgconfig/../../lib path is wrong  but /home/mohsen/vcpkg/packages/fmt_arm-linux/lib/pkgconfig/../../lib is okay
Computing installation plan...
The following packages will be built and installed:
  * fmt[core]:arm-linux
    spdlog[core]:arm-linux
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet arm-linux...
Starting package 1/2: fmt:arm-linux
Building package fmt[core]:arm-linux...
Could not locate cached archive: /home/mohsen/.cache/vcpkg/archives/b9/b9a3808a04e7b19ef83184c030938eab3908ee16.zip
-- Using cached /home/mohsen/vcpkg/downloads/fmtlib-fmt-4fe0b11195b7cd71f39253c44db2c9dddf6b82d4.tar.gz
-- Cleaning sources at /home/mohsen/vcpkg/buildtrees/fmt/src/dddf6b82d4-f2c5fef865.clean. Use --editable to skip cleaning for the packages you specify.
 If it is a system library use the
  SYSTEM_LIBRARIES parameter for the vcpkg_fixup_pkgconfig call! Otherwise,
  correct the *.pc file
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:298 (vcpkg_fixup_pkgconfig_check_files)
  ports/fmt/portfile.cmake:44 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:135 (include)

Error: Building package fmt:arm-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: fmt:arm-linux
  Vcpkg version: 2020.06.15-unknownhash

Additionally, attach any relevant sections from the log files above.

Thank you for reporting this issue, but I can't repro it locally. Do you have local changes? Or could you try with latest master branch?
xyb@ubuntu:~/src/vcpkg$ ./vcpkg install fmt:arm-linux
Computing installation plan...
The following packages will be built and installed:
    fmt[core]:arm-linux
Detecting compiler hash for triplet arm-linux...
Starting package 1/1: fmt:arm-linux
Building package fmt[core]:arm-linux...
Could not locate cached archive: /home/xyb/.cache/vcpkg/archives/2e/2e6f8e288e9fb5de59310b79c5274a4990e461e3.zip
-- Using community triplet arm-linux. This triplet configuration is not guaranteed to succeed.
-- [COMMUNITY] Loading triplet configuration from: /home/xyb/src/vcpkg/triplets/community/arm-linux.cmake
-- Using cached /home/xyb/src/vcpkg/downloads/fmtlib-fmt-4fe0b11195b7cd71f39253c44db2c9dddf6b82d4.tar.gz
-- Cleaning sources at /home/xyb/src/vcpkg/buildtrees/fmt/src/dddf6b82d4-f2c5fef865.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/xyb/src/vcpkg/downloads/fmtlib-fmt-4fe0b11195b7cd71f39253c44db2c9dddf6b82d4.tar.gz
-- Applying patch fix-warning4189.patch
-- Using source at /home/xyb/src/vcpkg/buildtrees/fmt/src/dddf6b82d4-f2c5fef865.clean
-- Configuring arm-linux-dbg
-- Configuring arm-linux-rel
-- Building arm-linux-dbg
-- Building arm-linux-rel
-- Installing: /home/xyb/src/vcpkg/packages/fmt_arm-linux/share/fmt/copyright
-- Fixing pkgconfig file: /home/xyb/src/vcpkg/packages/fmt_arm-linux/lib/pkgconfig/fmt.pc
-- Fixing pkgconfig file: /home/xyb/src/vcpkg/packages/fmt_arm-linux/debug/lib/pkgconfig/fmt.pc
-- Installing: /home/xyb/src/vcpkg/packages/fmt_arm-linux/share/fmt/usage
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: /home/xyb/.cache/vcpkg/archives/2e/2e6f8e288e9fb5de59310b79c5274a4990e461e3.zip
Building package fmt[core]:arm-linux... done
Installing package fmt[core]:arm-linux...
Installing package fmt[core]:arm-linux... done
Elapsed time for package fmt:arm-linux: 5.966 s

Total elapsed time: 6.696 s

The package fmt provides CMake targets:

    find_package(fmt CONFIG REQUIRED)
    target_link_libraries(main PRIVATE fmt::fmt)

    # Or use the header-only version
    target_link_libraries(main PRIVATE fmt::fmt-header-only)

xyb@ubuntu:~/src/vcpkg$ uname -a
Linux xybpc 4.19.104-microsoft-standard #1 SMP Wed Feb 19 06:37:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

i use master branch and my sdk produced by  latest stable yocto(dunfell)
set(VCPKG_TARGET_ARCHITECTURE arm)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE static)
set(VCPKG_BUILD_TYPE release)
set(VCPKG_CMAKE_SYSTEM_NAME Linux)

set(VCPKG_CHAINLOAD_TOOLCHAIN_FILE /opt/poky/3.1.3/sysroots/x86_64-pokysdk-linux/usr/share/cmake/OEToolchainConfig.cmake)

set(PORT_DEBUG on)

Still not repo. How do you generate your sdk? Or could you list your environment variables? I tried sdk from http://downloads.yoctoproject.org/releases/yocto/yocto-3.1.3/toolchain/x86_64/poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh, but can't repro.
xyb@xybpc:/tmp$ wget http://downloads.yoctoproject.org/releases/yocto/yocto-3.1.3/toolchain/x86_64/poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
--2020-11-02 06:51:40--  http://downloads.yoctoproject.org/releases/yocto/yocto-3.1.3/toolchain/x86_64/poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
Resolving downloads.yoctoproject.org (downloads.yoctoproject.org)... <IP_ADDRESS>
Connecting to downloads.yoctoproject.org (downloads.yoctoproject.org)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 474399995 (452M) [application/octet-stream]
Saving to: ‘poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh’

poky-glibc-x86_64-core-image-sato-armv7v 100%[================================================================================>] 452.42M  7.16MB/s    in 53s

2020-11-02 06:52:33 (8.54 MB/s) - ‘poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh’ saved [474399995/474399995]
xyb@xybpc:/tmp$ chmod +x poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
xyb@xybpc:/tmp$ sudo ./poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
Poky (Yocto Project Reference Distro) SDK installer version 3.1.3
=================================================================
Enter target directory for SDK (default: /opt/poky/3.1.3):
You are about to install the SDK to "/opt/poky/3.1.3". Proceed [Y/n]?
Extracting SDK.......................................................................................................................done
Setting it up...done
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
SDK has been successfully set up and is ready to be used.
Each time you wish to use the SDK in a new shell session, you need to source the environment setup script e.g.
 $ . /opt/poky/3.1.3/environment-setup-armv7vet2hf-neon-poky-linux-gnueabi
xyb@xybpc:~/src/vcpkg$ ./vcpkg install fmt:arm-linux
Computing installation plan...
The following packages will be built and installed:
    fmt[core]:arm-linux
Detecting compiler hash for triplet arm-linux...
Starting package 1/1: fmt:arm-linux
Building package fmt[core]:arm-linux...
Could not locate cached archive: /home/xyb/.cache/vcpkg/archives/c7/c720c12ed27e5a89faa309d345b87c668f9452d3.zip
-- Using community triplet arm-linux. This triplet configuration is not guaranteed to succeed.
-- [COMMUNITY] Loading triplet configuration from: /home/xyb/src/vcpkg/triplets/community/arm-linux.cmake
-- Using cached /home/xyb/src/vcpkg/downloads/fmtlib-fmt-4fe0b11195b7cd71f39253c44db2c9dddf6b82d4.tar.gz
-- Cleaning sources at /home/xyb/src/vcpkg/buildtrees/fmt/src/dddf6b82d4-f2c5fef865.clean. Use --editable to skip cleaning for the packages you specify.

Installing package fmt[core]:arm-linux... done
Elapsed time for package fmt:arm-linux: 3.456 s

Total elapsed time: 4.124 s

The package fmt provides CMake targets:

    find_package(fmt CONFIG REQUIRED)
    target_link_libraries(main PRIVATE fmt::fmt)

    # Or use the header-only version
    target_link_libraries(main PRIVATE fmt::fmt-header-only)

Still not repo. How do you generate your sdk? Or could you list your environment variables? I tried sdk from http://downloads.yoctoproject.org/releases/yocto/yocto-3.1.3/toolchain/x86_64/poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh, but can't repro.
xyb@xybpc:/tmp$ wget http://downloads.yoctoproject.org/releases/yocto/yocto-3.1.3/toolchain/x86_64/poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
--2020-11-02 06:51:40--  http://downloads.yoctoproject.org/releases/yocto/yocto-3.1.3/toolchain/x86_64/poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
Resolving downloads.yoctoproject.org (downloads.yoctoproject.org)... <IP_ADDRESS>
Connecting to downloads.yoctoproject.org (downloads.yoctoproject.org)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 474399995 (452M) [application/octet-stream]
Saving to: ‘poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh’

poky-glibc-x86_64-core-image-sato-armv7v 100%[================================================================================>] 452.42M  7.16MB/s    in 53s

2020-11-02 06:52:33 (8.54 MB/s) - ‘poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh’ saved [474399995/474399995]
xyb@xybpc:/tmp$ chmod +x poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
xyb@xybpc:/tmp$ sudo ./poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh
Poky (Yocto Project Reference Distro) SDK installer version 3.1.3
=================================================================
Enter target directory for SDK (default: /opt/poky/3.1.3):
You are about to install the SDK to "/opt/poky/3.1.3". Proceed [Y/n]?
Extracting SDK.......................................................................................................................done
Setting it up...done
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
SDK has been successfully set up and is ready to be used.
Each time you wish to use the SDK in a new shell session, you need to source the environment setup script e.g.
 $ . /opt/poky/3.1.3/environment-setup-armv7vet2hf-neon-poky-linux-gnueabi
xyb@xybpc:~/src/vcpkg$ ./vcpkg install fmt:arm-linux
Computing installation plan...
The following packages will be built and installed:
    fmt[core]:arm-linux
Detecting compiler hash for triplet arm-linux...
Starting package 1/1: fmt:arm-linux
Building package fmt[core]:arm-linux...
Could not locate cached archive: /home/xyb/.cache/vcpkg/archives/c7/c720c12ed27e5a89faa309d345b87c668f9452d3.zip
-- Using community triplet arm-linux. This triplet configuration is not guaranteed to succeed.
-- [COMMUNITY] Loading triplet configuration from: /home/xyb/src/vcpkg/triplets/community/arm-linux.cmake
-- Using cached /home/xyb/src/vcpkg/downloads/fmtlib-fmt-4fe0b11195b7cd71f39253c44db2c9dddf6b82d4.tar.gz
-- Cleaning sources at /home/xyb/src/vcpkg/buildtrees/fmt/src/dddf6b82d4-f2c5fef865.clean. Use --editable to skip cleaning for the packages you specify.

Installing package fmt[core]:arm-linux... done
Elapsed time for package fmt:arm-linux: 3.456 s

Total elapsed time: 4.124 s

The package fmt provides CMake targets:

    find_package(fmt CONFIG REQUIRED)
    target_link_libraries(main PRIVATE fmt::fmt)

    # Or use the header-only version
    target_link_libraries(main PRIVATE fmt::fmt-header-only)

could you share your triplet i still have same problem even with http://downloads.yoctoproject.org/releases/yocto/yocto-3.1.3/toolchain/x86_64/poky-glibc-x86_64-core-image-sato-armv7vet2hf-neon-qemuarm-toolchain-3.1.3.sh sdk

my triplet
set(VCPKG_TARGET_ARCHITECTURE arm)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE static)

set(VCPKG_CMAKE_SYSTEM_NAME Linux)

if(NOT CMAKE_HOST_SYSTEM_PROCESSOR)
    execute_process(COMMAND "uname" "-m" OUTPUT_VARIABLE CMAKE_HOST_SYSTEM_PROCESSOR OUTPUT_STRIP_TRAILING_WHITESPACE)
endif()

set(VCPKG_CHAINLOAD_TOOLCHAIN_FILE /opt/poky/3.1.3/sysroots/x86_64-pokysdk-linux/usr/share/cmake/OEToolchainConfig.cmake)

i commented vcpkg_fixup_pkgconfig() function from my port(fmt) and successfully compiled that


Option to normalise Windows EOL

It might within .getvalue() , but would be more conventional if the user had some way to reconfigure the streams.
Especially helpful for doctest where these EOLs break expectations.
https://github.com/bskinn/stdio-mgr/pull/79#discussion_r321917949

Definitely like this.
Should also allow getting rid of most of the convert_newlines usage all over the test suite.
Most would operate with the EOL normalization active, and then just a select few tests would sanity-check with it inactive.

Just noting the doctest problem is likely https://bugs.python.org/issue1812 ; there are many other bugs in doctest ive encountered over the years and they never seem to get fixed.  I've added a few issues at https://github.com/Erotemic/xdoctest/issues?q=is%3Aissue+is%3Aopen+author%3Ajayvdb+sort%3Aupdated-desc and getting traction, so that might be a way forward.

Do we note have to worry about  plain \r any more? (Was that old MacOS, before they switched to be atop Unixish?)
Yeah, doctest probably hasn't had a champion in the core dev team for a while. argparse is similar, and probably others.
I'm totally fine with adding new dependencies in the development and CI environments -- if we can get doctest quality/coverage parity (or better) with 3rd-party, I'm good with that. I guess that wouldn't really help people who need to use stdio-mgr with, e.g., doctest itself, though. :-/


Ravi/timer convert

Creating a draft PR for the timer conversion functionality. Do not merge this PR.

We can leave this as a branch so as not to clutter the PR page


[bug] Can't start the endstone, undefined symbol: PyProperty_Type

Describe the bug
Can't start under Ubuntu after building the latest version of the source code, Windows works fine.
I've tried using GitHub action and compiling locally, both with the same result.
Screenshots

Details (please complete the following information):

OS: Linux ubuntu 22.04
Endstone Version: endstone-0.5.6.dev41

Additional context
symbol lookup error: /usr/local/lib/python3.10/dist-packages/endstone/_internal/libendstone_runtime.so: undefined symbol: PyProperty_Type

What is your python version?

Python 3.9,3.10,3.12 tried it all.

try cleaning your env, reinstalling python and endstone

try cleaning your env, reinstalling python and endstone

Yes I tried that, originally I only had python 3.9 and 3.10, I newly installed python 3.12 and installed python 3.12 version endstone,still the same!

rm /usr/local/lib/python3.10/dist-packages/endstone/ -rf
then update pip and install endstone again?

rm /usr/local/lib/python3.10/dist-packages/endstone/ -rf
then update pip and install endstone again?

Yes I have done the install several times and uninstall will remove the directory and pip is up to date.

🥲

Thank you for reporting this! I can confirm that this is indeed a bug introduced by recent changes in the build pipeline. I will provide a fix soon.

Fixed in the latest commits.


[MAIN] add Python 3.12

This pull request introduces support for Python 3.12, which was  released as of October 2, 2023.
Python 3.12 release, see: https://www.python.org/downloads/release/python-3120/
Changes

add Python 3.12 to CI workflow.

coverage: 87.902% (-0.3%) from 88.235%
when pulling e5a60fa8cdf1ea392cb017639915786e199bc80a on INM-6:enh/add_python_3.12
into e2df1b96a4d82532f7d29b9e3eb8c1f271afa58b on NeuralEnsemble:master.

Wheels build successful on ubuntu:
10 wheels produced in 2 minutes:
  elephant-1.0.1b1-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl   1,282 kB
  elephant-1.0.1b1-cp310-cp310-musllinux_1_1_x86_64.whl                         1,820 kB
  elephant-1.0.1b1-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl   1,283 kB
  elephant-1.0.1b1-cp311-cp311-musllinux_1_1_x86_64.whl                         1,820 kB
  elephant-1.0.1b1-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl   1,283 kB
  elephant-1.0.1b1-cp312-cp312-musllinux_1_1_x86_64.whl                         1,820 kB
  elephant-1.0.1b1-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl     1,282 kB
  elephant-1.0.1b1-cp38-cp38-musllinux_1_1_x86_64.whl                           1,820 kB
  elephant-1.0.1b1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl     1,282 kB
  elephant-1.0.1b1-cp39-cp39-musllinux_1_1_x86_64.whl                           1,819 kB

Wheels build successful on windows:
5 wheels produced in 3 minutes:
  elephant-1.0.1b1-cp310-cp310-win_amd64.whl   571 kB
  elephant-1.0.1b1-cp311-cp311-win_amd64.whl   571 kB
  elephant-1.0.1b1-cp312-cp312-win_amd64.whl   571 kB
  elephant-1.0.1b1-cp38-cp38-win_amd64.whl     571 kB
  elephant-1.0.1b1-cp39-cp39-win_amd64.whl     571 kB


feat: tabs now support deep-linking

Adds support for deep-linking in tabs.  This will only work after #3618 goes it (note: it won't break anything if it does land before).
I updated all of these using this:
git grep -lE '<Tabs .*groupId' | xargs -J echo gsed -ri 's/<Tabs(.*)groupId="([^\"]*)"/<Tabs\1groupId="\2" queryString/'

https://user-images.githubusercontent.com/49578/224362123-8126614d-71e5-4a6f-a219-511d0ed73160.mp4

I don't think we generally update old versioned snapshots of the docs unless necessary. Reverting those changes can help with the size of this diff.
This doesn't fix existing inbound links that only include fragment references (without the new query params), right?

Answering out of order @motiz88:

This doesn't fix existing inbound links that only include fragment references (without the new query params), right?

No it doesn't fix that, just enables updating the query string when interacting (or the page if directly linked with the query string parameters set).  Interestingly there's some other work that I have to follow up with here to scroll after load & update the tabs (some of the ids aren't visible on load.  That'd also be a wider docusaurus contrib (but we can swizzle it for now).
FYI our CLI has a draft PR using this react-native-community/cli#1870

I don't think we generally update old versioned snapshots of the docs unless necessary. Reverting those changes can help with the size of this diff.

The change is automated and pretty low risk (IMHO).  The CLI change means we'd want to update all of our supported versions (0.71, 0.70, 0.69).


esLint integration

@jmeas  I integrated ESLint successfully on my test project. You can copy & paste. Just make sure you also copy the .eslint file in the root.
You find the code here: https://github.com/Kflash/ES2015/blob/master/gulpfile.js#L81-L98
esLint now works for source files and the test unit files.
There is a little issue. The jsNotify is not working out of the box with esLint it seems. I will investigate later. Other things to do now.
NOTE! You have to adjust and set your own rules, and remove the old lint files in your repo

@kflash, there's already an issue regarding ESLint over at #144. Gonna close this as a duplicate. I'll reference this when I go about adding ESLint, though, so thanks!

Sure. No problem. And I'm now out of your project. My time ended. I figure
waste of time to help out.
On May 12, 2015 1:14 AM, "James Smith"<EMAIL_ADDRESS>wrote:

@kflash https://github.com/kflash, there's already an issue regarding
ESLint over at #144
https://github.com/babel/babel-library-boilerplate/issues/144. Gonna
close this as a duplicate. I'll reference this when I go about adding
ESLint, though, so thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/babel/babel-library-boilerplate/issues/159#issuecomment-100981756
.


updated appdev cop members and repos

https://github.com/redhat-cop/org/issues/795

@jeremyrdavis can you approve, since your already one of the admins and this PR adds @pamenon and @deewhyweb


Updated WordPress docs

@ZubaeyrMSFT @BlackRider97  Why was the Terraform scripts deleted out? The comment says wordpress docs are being updated.

@ZubaeyrMSFT @BlackRider97 Why was the Terraform scripts deleted out? The comment says wordpress docs are being updated.

Added again in PR https://github.com/Azure/wordpress-linux-appservice/pull/183


fix: Change the token to secret

Because

We want the token is secret.

This commit

make the token secret

Found another bug: if the conversations array length is 0, it will return nil, but we should return an empty array. You can try make the start_date larger then now.

Please add "instillUIMultiline": true to message in TASK_WRITE_MESSAGE

Please add "instillUIMultiline": true to message in TASK_WRITE_MESSAGE
It is originally true in TASK_WRITE_MESSAGE

Please add "instillUIMultiline": true to message in TASK_WRITE_MESSAGE

@donch1989 It is originally true in TASK_WRITE_MESSAGE

Oh... Okay. My eyes are blurred.


Add namespace ID to NMT leaf hash

Fixes #145.

Converting to draft while we figure out the full extent of changes required.

render version of the the share section: https://github.com/lazyledger/lazyledger-specs/blob/adlerjohn-nmt_namespace_leaves/specs/data_structures.md#share


feat: makes a post to api/client/features possible by setting a cli arg

What
This exposes a boolean CLI arg called enable_post_features that will allow POST requests to api/client/features
Why
This is to support a very specific but ephemeral need for a user. This doesn't do anything useful outside of this need and it's absolutely not useful for our SDKs. To that end, I've placed the CLI arg in a new flattened CliArg called ExperimentalArgs and kept the code here isolated so that it's easy to tear out at a later stage

Was going to suggest making that argument global, so you don't have to remember whether to add it before or after the sub command

Was going to suggest making that argument global, so you don't have to remember whether to add it before or after the sub command

That would have been a good idea, I should have thought of that. We can move it in the next major version if we want but (and this is a terrible thing to say), I don't care much about this feature. When relevant user is done I'd either like to remove it or put it behind a compiler flag and leave that off by default


Add 'Move to view' and make 'Align to view' only align

Hi,
After discovering "Align with view" I played with it a bit and was dissatisfied with how it worked.
Maybe I'm picky, but for me "Align with view" should rotate but not move the selected node(s) 🙅♂
So I made it only rotate the selected nodes to be aligned with the view, and added a menu option "Move to view" which only does what "Align with view" did (do you follow ? 😛 ).
I also added a "focus_selection" at the end to avoid errors when the selected node is a camera (as reported in #23063).

Well, if we're being pedantic the semantics of "Align", then one would very much expect "Move" to only impact translation, not the rotation or scale.
To me align means the same as sync, as in "align/sync property X with property Y", so it's not limited to the rotation/"look at" angle only. So more than nitpicking about semantics, it would be interesting to know what are the use cases and if people need to be able to align rotation only, translation only, scale only, or full transform.

Moving to 3.2 milestone as we're about to enter the beta stage and release freeze for Godot 3.1. Only bug fixes against issues of the 3.1 milestone (or enhancements to core features of the 3.1 roadmap) will be considered until 3.1-stable is released.
Note: If it's merged after 3.1-stable is released, this change might be considered for cherry-picking into a later 3.1.x release.

Sorry for the late review.
We finally got to discuss this in a PR review meeting and bikeshed on the naming, see logs from 15:08.
We'd suggest to use the names "Align Transform with View" and "Align Rotation with View" to be explicit about what each operation does. Could you do this change, if you agree that it makes things clear? (And change the enum names accordingly.)

I pushed an update with the proposed label change.

Thanks!

hi,
Sorry I could not answer before, still in holidays :smile:
Thanks @akien-mga for doing it :+1:


[rule] Update rule health for append/commit fails

Similar to https://github.com/prometheus/prometheus/pull/8410 will
provide more context.

Are you willing to also call Rollback() then ? see the start of my work in #7592

Are you willing to also call Rollback() then?

I am willing to work on it, but can I do it in a follow up PR. Its trickier and I'd rather have the simple fix in first before adding more complexity to this PR.


listItemComments ability to filter mentionings

Observed Behavior
https://github.com/pnp/sp-dev-fx-controls-react/blob/master/src/controls/listItemComments/components/AddComment/AddComment.tsx
here  in new comment control on @ => opened list of users for using in the comment as mentions, this is nice, but when you have lots of users on the site will be nice to have filtering (either):

@ + letters afterwards, like if want to find user "MyFake, Name" => "@MyF" => suggestions  (same as it is in MS Teams, or here, in GitHub)
if leave on @ => suggestions - add search field there to be able to filter them by name

Category
[x ] Enhancement
[ ] Bug
[ ] Question
Version
Please specify what version of the library you are using: [ 3.5.0 ]

@fonbrauzen the control should allow you to filter in the way you describe in option 1 (@myname...) As you type, it will keep filtering, the same it does out of the box in a list item comments box.
Can you double-check that you have granted all the required graph permissions for the control to work?
Documentation mentions People.Read and User.ReadBasic.All but if you still don't get this experience, can you check if you get an error in the console related to insufficient permissions when trying to retrieve users?

sorry, User.ReadBasic.All was missing, now working


inquire on svs English phnum label

Dear tigermeat,
TIGER is wonderful project,I use it in openutau.
Could I ask how you train the English svs model? I am a Graduate student,have trained chinese svs model before, and I researched training English svs model, my questions are about making english trainset,especially the phnum labeled.Do you use code in https://github.com/openvpi/MakeDiffSinger or improved the code？
I will appreciate it if you can reply .Look forward for further communication,thank you very much

Hi! The best advice I can offer you is to join the DiffSinger discord server, if you can. Many users in the server will be able to provide you with guidance!
I will be closing this issue as it does not pertain to LabelMakr.


docs: add v8-memory-cage blog post

New blog post about the V8 memory cage and pointer compression.

Looks like this is approved and hasn't been touched for about a week.
Is there anything left to do on this before it can go live? I'd like to link to it next week in the E20 blog post. :slightly_smiling_face:


122 build status merge fix

https://github.com/Visual-Regression-Tracker/Visual-Regression-Tracker/issues/122

 Here is an overview of what got changed by this pull request:

Coverage increased per file
===========================
+ src/test-runs/dto/testRunResult.dto.ts  47

See the complete overview on Codacy


AZURE_LOGGING_DIR is also configurable in Linux Web Apps (Tomcat Stack)

The document says AZURE_LOGGING_DIR is Native Windows apps only, but actually, we can also use the setting in Linux Web Apps (Tomcat Stack).
I confirmed that in Java 11 / Tomcat 9.0.62.

AZURE_LOGGING_DIR
Native Windows apps only. Added to Java args as -Dsite.logdir. The default is %HOME%\LogFiles.

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

ID: f081389e-21ca-4d1a-80e1-d96c13ec1ffb
Version Independent ID: 9d1246c8-bfbf-78af-e87f-4d0dd287da90
Content: Environment variables and app settings reference - Azure App Service
Content Source: articles/app-service/reference-app-settings.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Default value in Linux is AZURE_LOGGING_DIR=/home/LogFiles.

@YusukeTobo
Thanks for your feedback! We will investigate and update as appropriate.

We have raised a PR for author's review, the published changes would reflect once the PR is merged. Thanks!


Code release

Hello!
Thank you much for this interesting work.
Any idea when the code will be released?
Thanks.

Hi @qsisi, Thanks for your interest in our project.
We are now preparing the code for release and I believe it will be no later than the end of September.
Thanks.

Thanks for your reply, I will close this issue after the code release.
Looking forward to your project!


Add branch name max length option to GitHub backend (closes #526)

- Summary
Adds branch name max length option to GitHub backend. Closes #526.

- Test plan
Tested manually.

- Description for the changelog

Add branch name max length option to GitHub backend (closes #526)

Deploy preview ready!
Built with commit c8d10cf6ef934877c4210ec88188c6e142b4fbe3
https://deploy-preview-862--netlify-cms-www.netlify.com

@tech4him1 addressed your comment, please re-review.

Deploy preview ready!
Built with commit 45b1598633ebb04fe8cfdf0f0e10cdee30b2523e
https://deploy-preview-862--cms-demo.netlify.com

@Benaiah Sorry for the delay. I'm seeing two things right now, one is a bug and one may not be:
If the branch_name_max_length is reduced after a post draft is created, the post will no longer show up in editorial workflow if it's slug had to be trimmed. I'm not sure if we can fix this, or just have a warning in the documentation.
If the branch_name_max_length is less than 5, we need to throw an error at config load point, since it always must be at least 5 chars to account for cms/.

I would say we aren't set up for branch names to change, so updating that config should only affect new branches and not existing branches.

@erquhart I agree, do you have any suggestions on how that could be implemented?

@erquhart I don't see how that works. If we save the metadata to a truncated name, how do we know what to get it back with without truncating? For example, if the slug is long-filename, and truncated it is long, how do we know what to get it as when attempting to read metadata for long-filename (assuming that the truncation length is different)?

I may be under thinking this, but I'd expect we'd set the same name for metadata and branch?
Sent with GitHawk

Well, that's not really what I'm trying to point out, but I could be wrong. If we call retrieveMetadata with a slug (from updateUnpublishedEntryStatus for example), isn't that going to be the full path, not the truncated one?

That's the thing -- this PR currently truncates both the metadata filename and the branch name.

Ah, understood! We shouldn't do that, agreed.

Also, if we don't do that, do you still think the best way to track the branch name is by saving in the metadata, or do you have other suggestions here?

Haven't dug into it very deep, but I can't think of a better way at the moment. You?


Magritte: execute actions in background

We also want an indicator that an execution is active (or finished with an error).

The following example shows a toolbar message that indicates an executed action. The second part of the video shows an error indication.
https://user-images.githubusercontent.com/465505/228077552-4dd16232-e45a-40ff-91bd-04e41eaab8d4.mov


build MediaPipe C++ with debug option on Windows

Hi,
I tried to build MediaPipe on Windows the C++ debug version, but it failed due to unresolved external for OpenCV functions. I found the issue. OpenCV has separate lib and dll for debug version with 'd' postfix.
A possible solution using config_setting:
cc_library(
    name = "opencv",
    srcs = select({
        ":opt_build": [
            "x64/vc15/lib/opencv_world" + OPENCV_VERSION + ".lib",
            "x64/vc15/bin/opencv_world" + OPENCV_VERSION + ".dll",
            ],
        ":dbg_build": [
            "x64/vc15/lib/opencv_world" + OPENCV_VERSION + "d.lib",
            "x64/vc15/bin/opencv_world" + OPENCV_VERSION + "d.dll",
        ],
    }),
    hdrs = glob(["include/opencv2/**/*.h*"]),
    includes = ["include/"],
    linkstatic = 1,
    visibility = ["//visibility:public"],
)

config_setting(
    name = "opt_build",
    values = {"compilation_mode": "opt"},
)

config_setting(
    name = "dbg_build",
    values = {"compilation_mode": "dbg"},
)

Peter

Hi @PeterPocsi, Could you please update your issue status.  Thanks!

Hi @sgowroji,
Yes, sure, I closed the issue because I have a PR for this.


Support for multi-class multi-label problems?

Currently, it seems that multi-class multi-label problems are not supported (e.g. a classification problem with 5 target variables (generally n_outputs) where each target variable can take three possible values (generally n_classes)).
Is this something that the authors think that should/should not be supported?
One issue is that currently the output of predict_proba methods has shape = (n_samples, n_outputs), whereas to support multi-class multi-label it should be a list with n_outputs elements, where each element has shape = (n_samples, n_classes) (see output of MultioutputClassifier in sklearn).
It seems to me that currently ClassifierChain supports multi-class multi-label problems; but LabelPowerset doesn't. I have tried to use the LabelPowerset transformation to a multi-class multi-label problem that I have, but judging from the output of unique_combinations_  it seems to me that the code currently assumes binary multi-label problems. However, the code runs without any exceptions/warnings, which results in wrong results, therefore I suggest that in case the authors do not want to support multi-class multi-label problems, there should at least be some checks/exceptions/warnings to refrain the user from using such data.

However, the code runs without any exceptions/warnings, which results in wrong results, therefore I suggest that in case the authors do not want to support multi-class multi-label problems, there should at least be some checks/exceptions/warnings to refrain the user from using such data.

In my opinion we shouldn't clutter the code with such things. It's the user's responsibility to check what the code does. We don't indicate that we support multi-target classification anywhere, I guess.
I'm not sure it would be beneficial to the library if we implemented support for this. I didn't look into it, but even now there are few people willing to participate.
@niedakh what's your take?

I think we shouldn't support multi-output tasks, there's no real foundation for that in scipy sparse too, I don't thinkg scipy matrices can handle sparse matrices with additional subvectors. We do rely a lot in the optimizations on this.

just wanted to ask : does this also apply to IterativeStratification? I tried with multi-class multi-label target array, (for instance, arrays like :
array([[2, 1, 0],
       [0, 0, 0],
       [1, 1, 0],
       [1, 1, 1],
       [0, 1, 0]])

(the first label has three possible classes)
After playing around with a bit, it seems that when using IterativeStratification with multi class + multi-label does not throw any errors, the resulting stratified folds do not have similar class proportions to the original distributions (i.e. not stratified)


it occors the error when i import 'package:bitmap/bitmap.dart';

* Error running Gradle:
ProcessException: Process "/Users/music/Documents/.../android/gradlew" exited abnormally:

> Configure project :bitmap
WARNING: The option setting 'android.enableR8=true' is experimental and unsupported.
The current default is 'false'
Consider disabling R8 by removing 'android.enableR8=true' from your gradle.properties before publishing your app.

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':bitmap'.
> NDK not configured.
  Download it with SDK manager.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s
  Command: /Users/music/Documents/.../android/gradlew app:properties

Finished with error: Please review your Gradle project setup in the android/ folder.

Appeared to me before I updated all android tools (sdk, AS, gradle...), have you tried to do that?

Appeared to me before I updated all android tools (sdk, AS, gradle...), have you tried to do that?

thanks, i'll try that.


Find header picture for "Call for papers" page

Find header picture for "Call for papers" page: https://2023.foss4g.org/call-for-papers/
Picture should be dark enough so the text is readable.

I've put a "random" picture for now but we need to find a picture related to the venue/topic.


taskwarrior3: 3.1.0 -> 3.2.0

Bumped taskwarrior version to 3.2.0.
Changelog: https://github.com/GothenburgBitFactory/taskwarrior/releases/tag/v3.2.0
Among other things it also fixes the annoying database locking issue (https://github.com/GothenburgBitFactory/taskwarrior/issues/3676).
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I have tried testing the build but for some reason I'm getting the following error:
$ nix build --file /nix/store/0nmjnwymxxvvq1lccjfdlar4v3qnhdph-nixpkgs-review-2.12.0/lib/python3.12/site-packages/nixpkgs_review/nix/review-shell.nix --nix-path 'nixpkgs=/Users/tim/.cache/nixpkgs-review/pr-356198-2/nixpkgs nixpkgs-overlays=/tmp/tmp0piq6p39' --extra-experimental-features 'nix-command no-url-literals' --no-link --keep-going --no-allow-import-from-derivation --argstr local-system aarch64-darwin --argstr nixpkgs-path /Users/tim/.cache/nixpkgs-review/pr-356198-2/nixpkgs --argstr nixpkgs-config-path /tmp/tmpu78p3jjn.nix --argstr attrs-path /Users/tim/.cache/nixpkgs-review/pr-356198-2/attrs.nix
error: builder for '/nix/store/0z9bs8wmia3wi81a7256dsfag1s8vxc5-taskwarrior-3.2.0.drv' failed with exit code 2;
       last 25 log lines:
       >    Compiling google-cloud-storage v0.15.0
       >    Compiling ureq v2.10.1
       >    Compiling strum_macros v0.25.3
       >    Compiling chrono v0.4.38
       > error: download of config.json failed
       >
       > Caused by:
       >   failed to download from `https://index.crates.io/config.json`
       >
       > Caused by:
       >   [6] Could not resolve hostname (Could not resolve host: index.crates.io)
       > make[2]: *** [src/taskchampion-cpp/CMakeFiles/cxxbridge_v1.0.128.dir/build.make:74: corrosion/cxxbridge_v1.0.128/bin/cxxbridge] Error 101
       > make[1]: *** [CMakeFiles/Makefile2:617: src/taskchampion-cpp/CMakeFiles/cxxbridge_v1.0.128.dir/all] Error 2
       > make[1]: *** Waiting for unfinished jobs....
       >    Compiling taskchampion-lib v0.1.0 (/tmp/nix-build-taskwarrior-3.2.0.drv-0/source/src/taskchampion-cpp)
       >    Compiling uuid v1.11.0
       >    Compiling cxxbridge-macro v1.0.128
       >    Compiling byteorder v1.5.0
       >    Compiling strum v0.25.0
       >    Compiling rusqlite v0.29.0
       >    Compiling taskchampion v0.9.0
       >     Finished `release` profile [optimized] target(s) in 46.06s
       > Copying byproducts `libtaskchampion_lib.a` to /tmp/nix-build-taskwarrior-3.2.0.drv-0/source/build/src/taskchampion-cpp
       > [ 13%] Built target _cargo-build_taskchampion_lib
       > make: *** [Makefile:166: all] Error 2
       For full logs, run 'nix log /nix/store/0z9bs8wmia3wi81a7256dsfag1s8vxc5-taskwarrior-3.2.0.drv'.
error: 1 dependencies of derivation '/nix/store/h0w9bw34dzkidsfl6f12a9byna5l6ib6-review-shell.drv' failed to build

Not sure what's going on there. Maybe a network issue on my side, I will dig it a bit later.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 356198

x86_64-linux

:x: 1 package failed to build:

taskwarrior3

:white_check_mark: 1 package built:

vimPlugins.taskwarrior3

Seems like an issue with upstream's rust dependencies...

In Nixpkgs we don't merge master into PR branches, but rather we rebase, in order to keep the PR's commits list as clean as possible.
As for the build error, I verified it is still an issue myself now, and opened:

https://github.com/GothenburgBitFactory/taskwarrior/issues/3705

In the meantime, this is marked as a draft, so no one will merge it by mistake.

Closing in favor of :

https://github.com/NixOS/nixpkgs/pull/359893

(Now not a draft).


When a photo is selected on create/edit, its preview doesn't show

On the create recipe form, click the camera tile to select a photo. It won't show, though it will be saved. Same goes for editing: if you pick a new photo, the old one will continue to be visible, even though the new one will be saved.

Done


Cleanup done

navbar, landing page and footer successfully cleaned up

Thank you for sending this in @Maloney18, I check the preview it seems you forgot to adjust the footer text to EventTrakka and making it a link that leads to the home page. Kindly adjust this.

@Maloney18  your PR  preview is breaking. Check Footer component, line 49 to be precise, there is <lin tag is supposed to be <li. Secondly, please remove the package-lock.json file. Since we are using yarn majorly. Sorry for the stress @Maloney18

Ok I will, I couldn't use my laptop at the time, so I edited from GitHub directly, I'm guessing I'll have to do it on my laptop then.


Allow fixnums as hash key

I got the error "Hash key is not a string" for a number in one of my apps during encoding of the data to be sent. I am not entirely sure where it came from since I wasn't able to reproduce that problem locally. My current assumption is that it was some key set during Raven.user_context.
While it could be said that the application is to blame for this, I think Sentry should be more forgiving. After all, Sentry is meant to collect errors, not produce any :) Sentry should do whatever it can to report errors in my application, no matter what is being thrown at it. In that light, I wonder whether "okjson" is the right way to do JSON encoding at all. From their repo:

Note for people who've used other ruby json
libraries: json does not represent symbols,
so okjson does not encode them. It won't
convert them to strings; that would silently
corrupt your data. The same holds for all of
ruby's other data types that don't exist in
json, including Range, Regexp, Proc, Time,
and user-defined classes.

Sentry should not be concerned with "corrupting data". Already Sentry is handling symbols in the vendored version since this is so common in Ruby. I think you're going to fight "upstream" okjson quite a bit going forward ...

Yeah, we've fought JSON before - unfortunately using something like multi_json or even just the built-in JSON causes us more problems for the reasons you describe - we need to be more forgiving. At least with OKJSON we have a recourse (modifying the implementation).

@michaelsauter Can you add a test for this? I'm 70% sure I'm OK with this but it will need a test.

@nateberkopec Thanks for commenting so quickly. I can look into that, but won't get to it until next week. Will update on Monday.

@nateberkopec Added some tests. Thought I'd restructure the original test a bit so it is easier to extend it with other types if there's a need to add more later.

Oh neat, I like the additions. Thank you!


Vision PDF/OCR

also remove reference to API keys and close ImageAnnotatorClient.
incomplete. does not work. returns errors when trying to write to GCS.

The tests should be fixed once we merge https://github.com/GoogleCloudPlatform/php-docs-samples/pull/623

The tests are failing with the following error:
1) Google\Cloud\Samples\Vision\visionTest::testPdfGcs
Error: Call to undefined method Google\Cloud\Vision\V1\AsyncBatchAnnotateFilesResponse::getMessage()
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/vision/src/detect_pdf_gcs.php:66
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/vision/vision.php:258
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/vision/vendor/symfony/console/Command/Command.php:249
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/vision/vendor/symfony/console/Tester/CommandTester.php:79
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/vision/test/visionTest.php:332
/home/[secure]/build/GoogleCloudPlatform/php-docs-samples/vision/test/visionTest.php:280
FAILURES!
Tests: 32, Assertions: 62, Errors: 1.

Travis errors:

Fatal error: Cannot use object of type Google\Cloud\Storage\ObjectIterator as array in /home/[secure]/build/GoogleCloudPlatform/php-docs-samples/vision/src/detect_pdf_gcs.php on line 91


RRFS smoke & dust

Additions to enable plotting of the new RRFS smoke and dust fields.  Dust is plotted with the same settings as smoke.

In the latest commit, the contour levels for dust were changed.  Here are new sample plots:


[9.x] Add support for table check constraints

This pull requests adds support for table check constraints (Documentation: MySQL | PostrgeSQL | SQLite | SQL Server).
Schema::create('users', function (Blueprint $table) {
    $table->id();
    $table->unsignedInteger('age');

    $table->check('age>=21');
});

Schema::create('events', function (Blueprint $table) {
    $table->id();
    $table->string('name');
    $table->datetime('starts_at');
    $table->datetime('ends_at')->nullable();
    $table->unsignedInteger('price');
    $table->unsignedInteger('discounted_price');

    $table->check('ends_at IS NULL OR ends_at>starts_at');
    $table->check('discounted_price <= price', 'check_valid_discounts');
});

In our use case, a Booking has many BookingItems, which morph to a contractor, who can either be a Hotel or a Flight. In case of a Hotel we need to known the room_id, for any other contractor_typethe room_id must be null.
While we will continue to validate this in the application, this PR allows us to add another layer of integrity for the database:
Schema::create('booking_items', function (Blueprint $table) {
    $table->id();
    $table->morphs('contractor');
    $table->foreignId('room_id')->constraint()->nullable();

    $table->check('contractor_type!="hotel" OR room_id IS NOT NULL');
});

All the default database drivers have support for check constraints (edit: For MySQL 5.7 The CHECK clause is parsed but ignored by all storage engines.). SQLite comes with a number of limitions:

SQLite cannot add check constraints to existing tables.
SQLite cannot drop check constraints.

Some similar limitations are currently covered in Blueprint::ensureCommandsAreValid() (link), while more recent PRs throw exceptions inside of the corresponding methods of SQLiteGrammar (e.g. compileSpatialIndex (link). I followed the later approach for now.
Note that the expression is considered "raw", meaning that it is not wrapped or parsed in any way. As long as it is valid SQL, the underlying database should handle it well.
I was torn between $table->check(...) and $table->checkConstraint(...) and I'm of course open for suggestions.
Please let me know if there's anything to be changed, improved or clarified.
If this gets merged, I'd be happy to look into inline column checks next as in $table->integer('age)->check('age>=21').

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Update iguanaTLS dependency

Today I tried building gyro with zig version 0.8.0-dev.2667+44de88498 and encountered this error:
./libs/iguanaTLS/src/asn1.zig:480:24: error: container 'std.builtin' has no member called 'endian'
        if (std.builtin.endian != .Big) {
                       ^
./libs/iguanaTLS/src/asn1.zig:462:41: note: called from here
        return try parse_length_internal(&bytes, der_reader);
                                        ^
./libs/iguanaTLS/src/asn1.zig:250:63: note: called from here
        const length = existing_length orelse try parse_length(der_reader);
                                                              ^
./libs/iguanaTLS/src/asn1.zig:198:54: note: called from here
        const res = try parse_schema_tag_len_internal(null, null, schema, captures, der_reader);
                                                     ^
./libs/iguanaTLS/src/main.zig:658:30: note: called from here
        asn1.der.parse_schema(schema, captures, reader) catch |err| switch (err) {
                             ^
./libs/iguanaTLS/src/main.zig:1234:75: note: called from here
            .default => certificate_public_key = try default_cert_verifier(
                                                                          ^
./libs/zfetch/src/connection.zig:92:58: note: called from here
                    self.context = try tls.client_connect(.{
                                                         ^
./libs/zfetch/src/connection.zig:85:84: note: called from here
    fn setupTlsContext(self: *Connection, trust: ?tls.x509.CertificateChain) !void {
                                                                                ^
./libs/iguanaTLS/src/asn1.zig:462:16: note: referenced here
        return try parse_length_internal(&bytes, der_reader);
               ^
./libs/iguanaTLS/src/asn1.zig:250:47: note: referenced here
        const length = existing_length orelse try parse_length(der_reader);
                                              ^
./libs/iguanaTLS/src/asn1.zig:198:21: note: referenced here
        const res = try parse_schema_tag_len_internal(null, null, schema, captures, der_reader);
                    ^
./libs/iguanaTLS/src/main.zig:658:57: note: referenced here
        asn1.der.parse_schema(schema, captures, reader) catch |err| switch (err) {
                                                        ^
./libs/iguanaTLS/src/main.zig:1234:50: note: referenced here
            .default => certificate_public_key = try default_cert_verifier(
                                                 ^
gyro...The following command exited with error code 1:
/home/dev/zig/zig build-exe /mnt/data/git/gyro/src/main.zig --pkg-begin build_options /mnt/data/git/gyro/zig-cache/gyro_build_options.zig --pkg-end --cache-dir /mnt/data/git/gyro/zig-cache --global-cache-dir /home/dev/.cache/zig --name gyro -target native-native-musl -mcpu=haswell+aes --pkg-begin clap /mnt/data/git/gyro/libs/zig-clap/clap.zig --pkg-end --pkg-begin version /mnt/data/git/gyro/libs/version/src/main.zig --pkg-begin mecha /mnt/data/git/gyro/libs/mecha/mecha.zig --pkg-end --pkg-end --pkg-begin tar /mnt/data/git/gyro/libs/tar/src/main.zig --pkg-end --pkg-begin zzz /mnt/data/git/gyro/libs/zzz/src/main.zig --pkg-end --pkg-begin glob /mnt/data/git/gyro/libs/glob/src/main.zig --pkg-end --pkg-begin zfetch /mnt/data/git/gyro/libs/zfetch/src/main.zig --pkg-begin hzzp /mnt/data/git/gyro/libs/hzzp/src/main.zig --pkg-end --pkg-begin uri /mnt/data/git/gyro/libs/zig-uri/uri.zig --pkg-end --pkg-begin network /mnt/data/git/gyro/libs/zig-network/network.zig --pkg-end --pkg-begin iguanaTLS /mnt/data/git/gyro/libs/iguanaTLS/src/main.zig --pkg-end --pkg-end --pkg-begin uri /mnt/data/git/gyro/libs/zig-uri/uri.zig --pkg-end --pkg-begin known-folders /mnt/data/git/gyro/libs/known-folders/known-folders.zig --pkg-end --enable-cache
error: the following build command failed with exit code 1:
/mnt/data/git/gyro/zig-cache/o/55d556ee8ca4eb00d082de177fdf4cb4/build /home/dev/zig/zig /mnt/data/git/gyro /mnt/data/git/gyro/zig-cache /home/dev/.cache/zig -Dbootstrap

I've seen that current zig's std no longer has std.builtin.endian and iguanaTLS was already updated to account for this change in https://github.com/alexnask/iguanaTLS/commit/da7b342cbc87afe0cb12c45623565d4b7eadf22c
So in this PR I'm updating the iguanaTLS dependency so that gyro builds again with current zig.

hey if you do a gyro update and commit the changes to the lockfile then these checks will pass

hey if you do a gyro update and commit the changes to the lockfile then these checks will pass

That's really cool!  I hadn't noticed that gyro manages its dependencies with gyro!  I've updated the commit.


Auth refactor

Motivation
Add auth page:

Info when user not logged in
Profile page
Add user info to side channel

Refactor auth to not be on critical path (sso-enabled).
Fix issues with other services if auth is not enabled.
Verification

Add required config files:
https://gist.github.com/wtrocki/7887e648d3d6a2cf848d529fd5dcb120
Go to profile page
Login

UI cases
When no auth enabled
Side panel

Profile screen

Enabled and authenticated
Side panel

Profile screen

Enabled and not authenticated

When no auth enabled
Side panel

Profile screen

Enabled and authenticated

Added photo

Note: check-sso always require user to perform login anyway. Authenticated is set to false.
Using require-login option for the moment.

👀

Auth disable
:heavy_check_mark: On profile shows Authentication not configured
Auth enable

Authenticated
:heavy_check_mark: On profile shows the profile info
:heavy_check_mark: On Side panel shows the user name and email
Not Authenticated
:heavy_minus_sign: Don't be forced to login when app starts if keycloak service is in mobile-services.js
:heavy_minus_sign: On profile shows not authenticated

@wtrocki I think I know the cause  of the problem you are having now with auth. The keycloak.js is different from AppAuth where the tokens are not kept in local storage, so it relies on check the browser every time to get the token. Probably something we need to improve in auth. I will create a ticket for it. For the moment, can we perform init on the profile page load instead, or when user click on the button?

jira created: https://issues.jboss.org/browse/AEROGEAR-8562.

@wei-lee My last commit disables check-sso option and uses require-login.
I haven't properly explained why I have done this so sorry for lots of text.
Over the last day I have poked with check-sso and found many issues and corner cases with that aproach. This aproach works perfectly in situation when login is optional and api allows unprotected access.
Currently our server by default do not allow this case - we apply keycloak middleware that will not allow users to gain access to server when not authenticated. We may log bug to support/document this case?
Additionally client needs to properly handle 401 and 403 cases. They are now pushed as application errors that users have control on.
Moving to require-login was just tradeoff considering amount of work and possible bugs that may appear. It helped me to avoid couple issues with handling different combination of states inside the app. Requiring login from start helped us with handling those additional states.  I have tried previous version of showcase app and it had the same issues.
I have tested require-login and I think that this is the option that we could propose for really short time as will come with simple handling and it will have much less corner cases. This will allow us to deliver showcase and do not land into the rabbit hole of bugs. However I truly recognize need for handling this use case and we should probably work on enabling it after delivering initial version.
Drawbacks here are obvious - users need to login first to the app when online and then carry away work on anything else, however next subsequent application launches will not require this (as long token is still valid).
This aproach also gives users ability to check if auth is enabled and token is applied - it will prevent from making graphql request when auth is still not enabled. I believe that our token provider should take that responsibility in the long run so there is work on SDK that will simplify entire application logic a lot.

After chat with @wei-lee we decided to make this PR about implementing various states for profile page.
Follow up ticket created will deal with the issue related to login on startup issue.


⚠ Bofrost.de has degraded performance

In ba8e52c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1488 ms

Resolved: Bofrost.de performance has improved in 36380ed after 20 minutes.


[style] add WhitespaceAround checkstyle rule.

Problem Summary: add WhitespaceAround checkstyle rule.
What is changed and how it works?
Proposal: xxx
Feel free to ping committers for the review!
Contribution Checklist

Make sure that the pull request corresponds to a GITHUB issue.

Name the pull request in the form "[Feature] [component] Title of the pull request", where Feature can be replaced by Hotfix, Bug, etc.

Minor fixes should be named following this pattern: [hotfix] [docs] Fix typo in README.md doc.

Purpose of this pull request

LGTM


update this repo's MIxS subsetting code to use GSC's MIxS v6.2.0

see

https://github.com/microbiomedata/nmdc-schema/issues/1224


make verify-iptables_blacklist raise AssertionError: The rule should no longer be in the map

currently, verifying the iptables_blacklist NF doesn't work.
from what I gathered, it is the only spec where a branch is actually "Multivalued!".
the assertion is actually sent by the spec, meaning in my understanding that the selected branch can't be taken.
but in tool/klint/verif/symbex.py:_symbex, it actually try for a "choice" exception but not for a "bad branching" exception. maybe the constraints are too loose and this path shouldn't happen?

this is probably because of that heuristic I removed in the commit fixing the verification for others. Oops! We can do the same "remove, write the commit tag in readme for those interested" approach.


Bump to 0.6.3 with SortedMap

Hi @dilyand! Can you remind me what we decided with it? Shall I close this PR?

I'm not sure we made a decision. We fixed the immediate issue with a custom mapping function in the bad row aggregator. However, I think the long-term plan was still to eventually fix it here as well, to ensure we don't need the custom mapping every time we want to work with bad rows in Beam. If we have another solution, then the PR can be closed.

I think long-term solution was to not use Iglu Client in Bad Rows at all, but as we speak I now think that SortedMap in Iglu Client would not hurt anybody and probably will help us to avoid further problems with Beam.
So scheduling for 1.1.0.

Cherry-picked into https://github.com/snowplow/iglu-scala-client/pull/148


Confusing "operation with the given volume ID" errors

Describe the bug
Ceph-CSI provisioner containers often output errors like below. To my understanding, this does not constitute a real error, as it only means that another Ceph-CSI process is handling the request. It often confuses Rook users as here https://github.com/rook/rook/issues/7202#issuecomment-780099804. Can these errors be reduced to Info messages?
E0216 20:15:38.416004       1 utils.go:136] ID: 1096 Req-ID: pvc-7f8ea9b2-dbe2-4cee-b6fa-0f5f71705444 GRPC error: rpc error: code = Aborted desc = an operation with the given Volume ID pvc-7f8ea9b2-dbe2-4cee-b6fa-0f5f71705444 already exists
E0216 20:16:40.427998       1 controllerserver.go:239] ID: 1098 Req-ID: pvc-1ac6f22b-2e19-4ead-8330-31bee8c5af16 an operation with the given Volume ID pvc-1ac6f22b-2e19-4ead-8330-31bee8c5af16 already exists
E0216 20:16:40.428033       1 utils.go:136] ID: 1098 Req-ID: pvc-1ac6f22b-2e19-4ead-8330-31bee8c5af16 GRPC error: rpc error: code = Aborted desc = an operation with the given Volume ID pvc-1ac6f22b-2e19-4ead-8330-31bee8c5af16 already exists
E0216 20:17:41.897098       1 controllerserver.go:239] ID: 1100 Req-ID: pvc-0d025ac3-1daa-47f0-a0da-aed85f501b2c an operation with the given Volume ID pvc-0d025ac3-1daa-47f0-a0da-aed85f501b2c already exists
E0216 20:17:41.897134       1 utils.go:136] ID: 1100 Req-ID: pvc-0d025ac3-1daa-47f0-a0da-aed85f501b2c GRPC error: rpc error: code = Aborted desc = an operation with the given Volume ID pvc-0d025ac3-1daa-47f0-a0da-aed85f501b2c already exists

@BlaineEXE I am diplomatic here. In one way its correct that, its informational message and can be logged as Info. In other angle, its a request reiteration from controller in above layer like sidecar and not really a retry from CSI driver layer. So I would like to get general opinion (@ShyamsundarR @nixpanic @Madhu-1 ) here before we make a move.

I agree that it is a confusing message. However, it usually is caused by an earlier failure that prevented the volume from getting created. We may be able to include more details about the original request, so that users have something they can lookup to troubleshoot a little more.
Unfortunately, I have seen occasions where this error is reported, because the initial CreateVolume request never returned. In these cases, the error message is actually correct. (Can happen if the RBD pool is configured correctly, like too few PGs.)
I wonder if we can detect actual hangs, and see if the go-routine is still active or not. If it is not, we should cleanup the journal so that a next try to create the volume can succeed.

I agree that it is a confusing message. However, it usually is caused by an earlier failure that prevented the volume from getting created. We may be able to include more details about the original request, so that users have something they can lookup to troubleshoot a little more.

I think this would help quite a bit if it's possible to include more information about where the root of the issue is. It's difficult when users merely report this error line, because it doesn't actually contain any information relevant to finding the source of the issue. Even just more text explaining that this could be a side-effect of another issue would be good and where (if possible) users should look for a different error.

We still get questions about this issue in Rook. Is it possible to get more information reported in these messages that can help narrow down the source of these issues when they occur?

@Yuggupta27 @Rakshith-R PTAL

Sure, We can definitely narrow the returned error which will help in reducing the noise and also the confusion. Will look into this :+1:

Hello, folks. We faced the same problem debugging the rbd plugin. Executing the rbd command from the provisioner pod with the given credentials had worked, but we were still seeing the "already exists" error in the CSI provisioner logs.
Frankly, we did not manage to reach the real cause of the issue, because suddenly the provisioner started working normally. I am writing to just bump the thread one more time. It is still hard to debug such errors.

Maybe a good start would be for the provisioner to "ping" the monitors in a way that is visible to the user?
Because at the moment a network misconfiguration is extremely easy to miss (first CreateVolume GRPC request timing out) and followed by a ton of noise (retries being aborted as per this issue)

It will be very difficult to get to the source of the problem as we don't keep track of what is the exact step going on in each RPC call.
Removing keepalive to get some attention if possible

Still think that even just a warning saying that « mons are unreachable [ip1, ip2, ip3] » in logs (and maybe even k8s resource events?) would go a long way into making this less confusing

Still think that even just a warning saying that « mons are unreachable [ip1, ip2, ip3] » in logs (and maybe even k8s resource events?) would go a long way into making this less confusing

@Tristan971 its not always the mon reachable problem, there could be many others reasons as well, if we don't receive any response to the ceph commands csi ran we will see this error message as well.


How to use this api?

Is it receiver api?
Can you please put some example app/code to use this api in an android app?

This is a sender API. I am not an android developer, so I cannot put an example app here. The code to call API methods should be same as provided in current examples.
I think it's better to use official android sdk for chromecast instead of this library. The goal of this library was to write senders for desktop/server applications.


Cell identifier and cellStyle override support

Current coverage is 95.41%

Merging #58 into master will not affect coverage as of 9f486b0

@@            master     #58   diff @@
======================================
  Files           16      16
  Stmts          764     763     -1
  Branches         0       0
  Methods          0
======================================
- Hit            729     728     -1
  Partial          0       0
  Missed          35      35

Review entire Coverage Diff as of 9f486b0

Powered by Codecov. Updated on successful CI builds.

Ahoy @Ewg777, these are both class methods, so self and [self class] are equivalent here, and the latter is less verbose.

This allows to subclass a table cell and set a custom identifier by using Storyboard.

You are already intended to subclass SSBaseTableCell or SSBaseCollectionCell if you need to provide your own identifier or cell style. I'm not sure then what this PR is changing?


Update mockito-scala-scalatest to 1.16.32

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.mockito", artifactId = "mockito-scala-scalatest" } ]

labels: test-library-update, semver-patch

Superseded by #2114.


split-diff as diff tool

Can I use https://github.com/mupchrch/split-diff as diff tool?

Any input on what would be the right path+command syntax here?
Or if the existing setting allows for commands (eg. split-diff:toggle), for installed packages?

List of plugin commands of split-diff - https://github.com/mupchrch/split-diff/blob/master/lib/split-diff.coffee#L28

@popstas i have tried already. just need someone to confirm to what script to point to exactly and the right syntax to put in the remote-sync diff tools path.

@mupchrch, maybe you can help with integrate your plugin split-diff with remote-sync?

It looks like remote-sync calls the diff tool by executing a child process. I don't think it is possible to invoke a package from a child process, so I think split-diff would have to be specifically integrated into the package, much like git-time-machine does it.
I am not sure when I'd have the time to try the integration myself, so if anyone wanted to try, then they should check out how this method does it. And feel free to ask me questions!

@mupchrch, thank you for answer.
Sorry, I have not experience in CoffeeScript and have no time for implementing this method.


Make torch out of glowy end plant

yeah

I doubt this mod will ever be active. Unless you're willing to continue this mod, then its dead.

Chiiiiiiiiiil gurl. If it's dead then let it die. Are you really this upset a mod that's seen no activity in 2 years isn't being continued or did you think that spamming comments on every issue would make me so excited I'd revive the mod?


MSI upgrade is deleting some files in lib/

Some admins are seeing that when upgrading the Windows MSI for GAM, they are left with an incomplete installation. Files are missing from lib/ and this causes GAM to crash with various missing files/libraries errors.
The issue seems to be that when we build the GAM MSI, heat.exe generates a new Guid ID for each file on each run. So the MSI installer sees them as unique files and for some reason fails to actually replace them.
We need a stable Guid for files across GAM MSI builds.

Workaround:

uninstall GAM7 MSI completely (gam.cfg and project files are NOT deleted).

install latest MSI.

That way we're not actually doing an upgrade and all files are copied.
More permanent fix incoming....

I've rewritten the tool script at:
https://github.com/GAM-team/GAM/blob/main/src/tools/gen-wix-xml-filelist.py
now after heat.exe runs the above script also runs and sets Guid to a uuid5 based on the source filename. That means that on every GAM MSI build a given filepath will get the same Guid.
This should fix MSI builds going forward but anyone with GAM 7.00.19 or older will need to do the uninstall, reinstall workaround at least once to get this solved.

In the legacy version, uninstalling GAM did not delete the GAM folder; now it does.
Users should be warned abount storing any user files in that directory.
Ross
Ross Scroggs
@.***

On Oct 16, 2024, at 2:00 PM, Jay Lee @.***> wrote:
Workaround:
uninstall GAM7 MSI completely (gam.cfg and project files are NOT deleted).
install latest MSI.
That way we're not actually doing an upgrade and all files are copied.
More permanent fix incoming....
—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1714#issuecomment-2417954166, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYLYEC2B5GB7WBWFIEFTZ33HXZAVCNFSM6AAAAABQCKFQ4SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJXHE2TIMJWGY.
You are receiving this because you are subscribed to this thread.

@taers232c do you know what change caused that? Ideally we would never delete files we didn't install. (and who really cares if c:\gam7 lives past MSI uninstall?)

I just checked, if there are any user files in c:\GAM, the folder is not deleted
Ross Scroggs
@.***

On Oct 16, 2024, at 2:09 PM, Jay Lee @.***> wrote:
@taers232c https://github.com/taers232c do you know what change caused that? Ideally we would never delete files we didn't install. (and who really cares if c:\gam7 lives past MSI uninstall?)
—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1714#issuecomment-2417968592, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL5VTPN2BWLNHSWUWSLZ33IX5AVCNFSM6AAAAABQCKFQ4SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJXHE3DQNJZGI.
You are receiving this because you were mentioned.

I'd call the new version 7.01.00 so it's easer to distinguish that this is the fixed version.
I have changes coming, I'll wait until you've got the issues resolved.
Ross
Ross Scroggs
@.***

On Oct 16, 2024, at 2:09 PM, Jay Lee @.***> wrote:
@taers232c https://github.com/taers232c do you know what change caused that? Ideally we would never delete files we didn't install. (and who really cares if c:\gam7 lives past MSI uninstall?)
—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1714#issuecomment-2417968592, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL5VTPN2BWLNHSWUWSLZ33IX5AVCNFSM6AAAAABQCKFQ4SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJXHE3DQNJZGI.
You are receiving this because you were mentioned.

This should be resolved now. Closing.


Add support for vision chat completion models

Describe the change
This PR let's you pass in image_url so we can pass in images to vision models like gpt-4-vision-preview
Describe your solution
The definition of ChatCompletionMessage is changed such that we can pass in a struct containing text and images for Content instead of a string.
Tests
This PR has been tested in example/chatvision
Issue: #539

Hey! is it possible to stream with gpt-4-vision-preview?

Hey! is it possible to stream with gpt-4-vision-preview?

I think I figured it out. However, the stream sometimes receives EOF during middle of the sentence.
'''
package main
import (
"context"
"errors"
"fmt"
"io"
"os"
openai "github.com/sashabaranov/go-openai"

)
func main() {
client := openai.NewClient(os.Getenv("OPENAI_API_KEY"))
ctx := context.Background()
req := openai.ChatCompletionRequest{
	Model: openai.GPT4VisionPreview,
	Messages: []openai.ChatCompletionMessage{
		{
			Role: openai.ChatMessageRoleUser,
			Content: []openai.ChatMessageContent{
				{
					Type: openai.ChatMessageContentTypeText,
					Text: "What's in this image",
				},
				{
					Type: openai.ChatMessageContentTypeImage,
					ImageURL: &openai.ChatMessageImageURL{
						URL: "https://upload.wikimedia.org/wikipedia/commons/thumb/d/dd/Gfp-wisconsin-madison-the-nature-boardwalk.jpg/2560px-Gfp-wisconsin-madison-the-nature-boardwalk.jpg",
					},
				},
			},
		},
	},
	Stream: true,
}

stream, err := client.CreateChatCompletionStream(ctx, req)
if err != nil {
	fmt.Printf("ChatCompletionStream error: %v\n", err)
	return
}
defer stream.Close()

fmt.Printf("Stream response: ")
for {
	response, err := stream.Recv()
	if errors.Is(err, io.EOF) {
		fmt.Println("\nStream finished")
		return
	}

	if err != nil {
		fmt.Printf("\nStream error: %v\n", err)
		return
	}

	fmt.Printf(response.Choices[0].Delta.Content)
}

}
'''

A few days ago, I encountered the same issue. Typically, I receive a sentence and a half, regardless of whether I send a URL with the https:// prefix or with an embedded image. I haven't tried without streaming enabled but will do so in a few hours. Have you noticed any differences in results when using streaming versus not using it?
@arif599

@arif599 I believe you need to set the MaxTokens to be higher value or highest(= 4096).
I guess it similar to this issue -> https://community.openai.com/t/gpt-4-vision-preview-finish-details/475911

@arif599 I believe you need to set the MaxTokens to be higher value or highest(= 4096). I guess it similar to this issue -> https://community.openai.com/t/gpt-4-vision-preview-finish-details/475911

Added MaxTokens: 4096 in ChatCompletionRequest and is working fine, thanks :)

A few days ago, I encountered the same issue. Typically, I receive a sentence and a half, regardless of whether I send a URL with the https:// prefix or with an embedded image. I haven't tried without streaming enabled but will do so in a few hours. Have you noticed any differences in results when using streaming versus not using it?
@arif599

Haven't noticed any difference with the quality of results but feel free to test it out yourself.
For streaming, just add MaxTokens: 4096 to ChatCompletionRequest in the example I provided.
So now it would be
`package main
import (
"context"
"errors"
"fmt"
"io"
"os"
openai "github.com/sashabaranov/go-openai"

)
func main() {
client := openai.NewClient(os.Getenv("OPENAI_API_KEY"))
ctx := context.Background()
req := openai.ChatCompletionRequest{
	Model: openai.GPT4VisionPreview,
	Messages: []openai.ChatCompletionMessage{
		{
			Role: openai.ChatMessageRoleUser,
			Content: []openai.ChatMessageContent{
				{
					Type: openai.ChatMessageContentTypeText,
					Text: "What's in this image",
				},
				{
					Type: openai.ChatMessageContentTypeImage,
					ImageURL: &openai.ChatMessageImageURL{
						URL: "https://upload.wikimedia.org/wikipedia/commons/thumb/d/dd/Gfp-wisconsin-madison-the-nature-boardwalk.jpg/2560px-Gfp-wisconsin-madison-the-nature-boardwalk.jpg",
					},
				},
			},
		},
	},
	Stream:    true,
	MaxTokens: 4096,
}

stream, err := client.CreateChatCompletionStream(ctx, req)
if err != nil {
	fmt.Printf("ChatCompletionStream error: %v\n", err)
	return
}
defer stream.Close()

fmt.Printf("Stream response: ")
for {
	response, err := stream.Recv()
	if errors.Is(err, io.EOF) {
		fmt.Println("\nStream finished")
		return
	}

	if err != nil {
		fmt.Printf("\nStream error: %v\n", err)
		return
	}

	fmt.Printf(response.Choices[0].Delta.Content)
}

}
`

@rkintzi can we send base64 encoded imgs?
Like https://platform.openai.com/docs/guides/vision/uploading-base-64-encoded-images
I wrote this encodeImage function in go
func encodeImage(imagePath string) string {
	imageData, err := os.ReadFile(imagePath)
	if err != nil {
		fmt.Println("Error:", err)
		return ""
	}
	base64String := base64.StdEncoding.EncodeToString(imageData)
	return base64String
}

but when I pass the encoded img into ImgURL
	req := gpt.ChatCompletionRequest{
		Model: gpt.GPT4VisionPreview,
		Messages: []gpt.ChatCompletionMessage{
			{
				Role: gpt.ChatMessageRoleUser,
				Content: []gpt.ChatMessageContent{
					{
						Type: gpt.ChatMessageContentTypeText,
						Text: prompt,
					},
					{
						Type: gpt.ChatMessageContentTypeImage,
						ImageURL: &gpt.ChatMessageImageURL{
							URL: encodeImage("example.jpg"),
						},
					},
				},
			},
		},
		Stream:    true,
		MaxTokens: 4096,
	}

I get error, status code: 400, message: Invalid image.

@sashabaranov @pufferffish closing this?


Fixed missing cpu information on nodejs

:)

@pprossi Ah, looks good! I'll try to review this as soon as possible. Thanks!

Coverage decreased (-0.1%) to 79.47% when pulling a71d134227512347773384cd561ddf0516c5d70d on pprossi:master into 98585f69582f56a730a7dccadb04496527343905 on jkphl:master.


[DOCS] README File is not properly organized.

Issue Description
The Prerequisites Section of the file is not organized.
All the information to run the project in our local is written in the same paragraph which decreased the readability of the project.
Screenshots or Examples (if applicable)

Proposed Solution (optional)
I will organize the Prerequisites Section properly by adding a new line after each instruction and properly adding the spaces.
Checklist

[X] I have checked the existing issues.
[X] I have read the Contributing Guidelines
[X] I have read the Code Of Conduct
[X] I joined onboarding group for knowledge flow and direct conversation and support from admins  Curious Developers Community - Curious Connect
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done
[X] If you need any help please join the default  (Google Meet Link)

I want to work on this issue. Please assign it to me under GSSOC.

@CDCHarshit  @CDCRishabh Kindly assign this issue to me under GSSOC.

please assign this to me.


[alertmanager-bot] new chart

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [radarr])

@onedr0p are these tests supported to run the real workload? In this case we would need working telegram credentials which I do not want to store in the public....

@runningman84 I am not sure, but you should be able to having the app run without specifying actual data...
Since this is a new chart, do you think it might be a good idea to use the common library @bjw-s built?
https://github.com/k8s-at-home/charts/tree/master/charts/common
Here's an example of it being used, but there are 8 or so charts using it too.
https://github.com/k8s-at-home/charts/tree/master/charts/qbittorrent
Moving forward we'd like to see new charts use this if it isn't much of a hassle, it reduces maintenance and LoC across the charts.

closing in favor of https://github.com/k8s-at-home/charts/pull/214


React 0.14

Currently redux-devtools is what is stopping me from upgrading to React 0.14.
Warning: React.findDOMNode is deprecated. Please use ReactDOM.findDOMNode from require('react-dom') instead.

in LogMonitor.

Is 0.14 released? It does not appear so.
We can't just go ahead and break compatibility with React 0.13 for the sake of a single warning until 0.14 is released as a stable version.

(To clarify: there are a few PRs fixing 0.14 warnings and I'll merge them when it's out, but not before that.)

And note that it's not really stopping you. It looks like an error but it's just a soft warning. Ignore it and you'll be fine.

If you don't like the error because of OCD (like me) or whatever... I've made a temporary fork here:
package.json
"devDependencies": {
  "redux-devtools": "git+ssh://git@github.com:tomatao/redux-devtools.git",,

Simply uses ReactDOM for findDOMNode instead of react.
I'll be deleting the fork when fixes merged.

Thanks @tomatau !

NP! Word of warning... heroku can't npm install from github... so have published this to npm as redux-devtools-14... which makes me feel VERY DIRTY.
Any chance a fix can be merged for this instead as it's such a tiny fix.
src/react/LogMonitor.js
import React, { PropTypes, findDOMNode, Component } from 'react';

becomes:
import React, { PropTypes, Component } from 'react';
import { findDOMNode } from 'react-dom';

I would make the PR here but tests wouldn't run locally for me and don't have time to debug why.

Got the tests running locally (bad npm version) and made a tiny PR
https://github.com/gaearon/redux-devtools/pull/135

It's a tiny fix but it breaks 0.13 compatibility.


prototype for front end

Develop front-end prototype with basic functions

The front-end prototype includes a registration page for new or returning users, alongside sections for voice recording and face capturing. It has two sections: one for recording voice and another for capturing a face using the camera. Users can switch between these sections with Next and Previous buttons, making it easy and clear to use.


remove encrypted tokens

Removing encrypted tokens from the .travis.yml file -> these are now stored as env variables in travis and have been rotated.
cc @springmeyer @artemp @flippmoke @millzpaugh @GretaCB @allieoop

Thanks @mapsam. 👍 looks good, will merge now.


Leverage Viper for reading Environment Variables

Provide a consistent experience:
https://github.com/spf13/viper#working-with-environment-variables
Risk: quality of life improvement, saves us time.

This is a quality of life improvement, low priority.

Will re-open if we plan on enhancing the go server


Create Voice

There should be the option to create a voice, since the API supports it. It would function almost identically to the "edit voice" function.

Thank you for the suggestion @platform-kit . This will feature be added in a future update

Here's how I did it https://gist.github.com/officialrobert/156a1cb6bdf42512f531df711ca36ef0

Thanks for the comment @officialrobert
Would you be open to contributing this solution to the package?


Improve LD HEV v_9mmar.mdl.

See if it's possible to reuse/share the same MP5 mesh with Op4/Blue Shift LD models.
PLAYER_ForeArm1.bmp is identical to textures/player/gordon/PLAYER_ForeArm1.bmp but they don't have the same dimension. Rework texture coordinates on the mesh to be able to reuse textures/player/gordon/PLAYER_ForeArm1.bmp.
Rework edge seams.
Fix left hand clipping through sleeve at frame 38 in reload animation.

The MP5 model looks a bit different in the expansion packs, i think merging the meshes might change the look too much.


Remove backslashed newlines on markdown

Fix https://gitea.com/gitea/gitea-docusaurus/issues/56

@lunny please fix the merge conflicts. :tea:


Inconsistent Errors in Different Environments

Hello,
We are running the limited release New Relic Kubernetes integration for Windows on our EKS clusters in AWS across several environments.
Description
In several of our environments it works cleanly. In another environment, on one cluster in a given account it seems to work on one of the two Windows nodes but not the other. On another cluster in the same new relic account and environment, it seems to not work on any of the 5 Windows nodes.
Steps to Reproduce
Simply deploy the daemonset
Expected Behavior
No error messages or failures.
Relevant Logs / Console output
When the integration fails we receive these errors consistently:
ime="2020-08-07T10:46:41-07:00" level=error msg="Integration command failed" error="exit status 1" instance=nri-kubernetes integration=com.newrelic.kubernetes prefix=integration/com.newr │
│ elic.kubernetes stderr="time="2020-08-07T10:46:31-07:00" level=warning msg="Cache file (c:\\var\\cache\\nr-kubernetes\\infra-sdk-cache.json) is older than 1m0s, skipping loading from disk."\ │
│ ntime="2020-08-07T10:46:41-07:00" level=panic msg="No data was populated"\ntime="2020-08-07T10:46:41-07:00" level=fatal msg="No data was populated"\n" working-dir="C:\Program Files\New Relic\ │
│ newrelic-infra\newrelic-integrations"
Your Environment
Running Kubernetes 1.16 on EKS.
Additional context
We would like to troubleshoot the issue and narrow down how we can address it.
Thanks!

Update: I've narrowed the problem a bit. On the nodes where the integration is failing, running the integration from the pod produces this result consistently:
C:\Program Files\New Relic\newrelic-infra\newrelic-integrations\bin>nri-kubernetes.exe -verbose
WARN[0000] Cache file (c:\var\cache\nr-kubernetes\infra-sdk-cache.json) is older than 1m0s, skipping loading from disk.
DEBU[0000] Integration "com.newrelic.kubernetes" with version 1.21.0 started
DEBU[0000] Found cached copy of "kubelet-client" stored at 2020-08-10 00:37:42 -0700 PDT
DEBU[0000] Kubelet node IP = <IP_ADDRESS>
DEBU[0000] Discovering KSM using DNS / k8s ApiServer (default)
DEBU[0000] Found cached copy of "ksm-client" stored at 2020-08-09 23:38:37 -0700 PDT
DEBU[0000] KSM Node = <IP_ADDRESS>
DEBU[0000] Running job: kubelet
DEBU[0000] Calling Kubelet endpoint: https://<IP_ADDRESS>:10250/pods
DEBU[0000] Calling Kubelet endpoint: https://<IP_ADDRESS>:10250/metrics/cadvisor
DEBU[0000] Calling Kubelet endpoint: https://<IP_ADDRESS>:10250/stats/summary
DEBU[0005] Calling Kubelet endpoint: https://<IP_ADDRESS>:10250/stats/summary
DEBU[0010] Job kubelet took 10.059s
DEBU[0010] populate errors:, error querying Kubelet. Get "https://<IP_ADDRESS>:10250/stats/summary": context deadline exceeded (Client.Timeout exceeded while awaiting headers)  datasource=k
ubelet phase=populate
PANI[0010] No data was populated
DEBU[0010] Integration "com.newrelic.kubernetes" exited
FATA[0010] No data was populated

So it seems like it can't query the kubelet for the node info at /stats/summary.
The helm chart is virtually the same in all environments, and in one environment one windows node is failing while the other is succeeding (which to me rules out RBAC issues and the like). This remained the case even after replacing both nodes.

Hey @andrewpurdin, thanks for filing this bug report, Reporter! We’ll be looking at it more in depth soon.

Hey there, good news, we upgraded to the latest Windows EKS optimized AMI and went to K8s 1.17 in our cluster and some combination of those changes seems to have resolved this issue. The problem may have been related to https://github.com/aws/containers-roadmap/issues/988, which I expect to close based on this AMI release.

Great news! Thanks for the update @andrewpurdin


Issuing credits for Immersive Shared Reality example

Issue 211250 credits to @lauerj.

 Follow up
On Wednesday, 06-Mar-24 17:10:29 UTC by Gov4Git dev
Issued 211250 credits to member @lauerj.


Copy input variables to GPU automatically

When variables with CPU arrays and GPU arrays are given to the same function, most users expect that the computation is done on the GPU, though currently it ends up with an error. We might change this behavior by silently copying the CPU arrays to the GPU device on which the other arguments exist.
Note that I made this issue since this functionality is indeed required in the implementation of #914. I will implement this change at the same PR with that for #914.

Since #914 is resolved without this feature, I close this issue for now.


[Feature] Support of perceptual loss in style transfer

The manipulation of images could be seen as a conditioned generation. Therefore, the commonly used perceptual loss in style transfer is essential in some generation tasks. However, the mmgeneration only supports perceptual loss in the LPIP metric.
Is it possible to support the perceptual loss for style transfer as in the mmediting?

Please check this issue. Have we support perceptual loss in MMGeneration?

Sure, it's an commonly used feature. We will add this to backlog. We will appreciate it a lot if you'd like to make a PR.

Thank you for your response. I am very willing to make a PR for adapting the perceptual loss from mmediting to mmgeneration. However, I have a few issues that need to discuss first.

Should the perceptual loss be implemented within mmgen/models/losses/gen_auxiliary_loss.py? Or I should create a new file like mmgen/models/losses/perceptual_loss.py
Should the class name PerceptualLoss be distinguished from the one in LPIP's PerceptualLoss? If it is true, what's your suggestions with the name?

Here are my suggestion.

You may implement the perceptual loss within mmgen/models/losses/gen_auxiliary_loss.py, then you can add it to your training config.
Since perceptual loss here is mainly used to calculate PPL metric. You may just use the same class name.

In 1.x, we just remove PeceptualLoss class and use lpips package instead. You may also use this package for your class.

Since now we are merging this repo into MMEditing. I also recommend you to try MMEdit-1.x and add this loss item there :).

Thank you for your suggestions.
I have a problem with the returned values. Typically, a loss module in mmgeneration returns one value. However, the PerceptualLoss often returns loss_percep and loss_style. Should the PerceptualLoss return a sum of these two losses or should I separate the loss_style into another loss module?


Fix 400 when "clyde" is in any webhook username

Discord just disallows this name.
The replacement character is Mathematical Sans-Serif Small E, which I felt was the best choice due to its similarity to a normal "e". However, I'm open to changing it to another character, putting it in parenthesis, or whatever else.

I believe the cyrillic е would be better here and seems to look the same as a normal e

Good find. The website I used didn't show Cyrillic characters as characters based on "e" (probably only considered latin alphabet). Cyrillic also has an upper case E so I will use that too. I wonder if there's a good way to do a case sensitive substitution. May just be easiest to chain two substitutions.

Do we also need to sub the names in reddit/python_news? The consistency is nice but having clyde in those is extremely unlikely

It's only one extra line of code, so better safe than sorry. I'd say this should be used for any name that's coming from an external source.


commands: allow to create send-to-self transactions with 'createspend'

Turns out it was straightforward. But i'm wary as these calculations are always pretty intricate.


I'm seeking some guidance on using exqlite under termux

G'day! I'm trying to use exqlite in a project running on termux.
While running mix deps.compile, the process fails to load the NIF module because the symbol trunc is not found. The following warning message is displayed when I run mix test at the root of the exqlite repository after cloning:
09:58:01.355 [warn]  The on_load function for module Elixir.Exqlite.Sqlite3NIF returned:
{:error,
 {:load_failed,
  'Failed to load NIF library: \'dlopen failed: cannot locate symbol "trunc" referenced by "/data/data/com.termux/files/home/develop/by-language/elixir/exqlite/_build/test/lib/exqlite/priv/sqlite3_nif.so"...\''}}

I'm supposing there is a missing library in my system but I fail to determine what library.
Could you guys offer-me some guidance?

@dominicletz maybe you can shed some light on this.
@leorodriguesrj I think you'll need to export CROSSCOMPILE=Android in order to get the correct symbols generated https://github.com/elixir-sqlite/exqlite/blob/main/Makefile#L38-L45

Interesting message on trunc. On Android generally, the CFLAG=-z global is important because of Androids' different treatment of dlopen scoping.
But in this case it could also just be a compiler version issue. The trunc method is defined in sqlite3 only when it's detected during compilation:
#if SQLITE_HAVE_C99_MATH_FUNCS
    MFUNCTION(trunc,             1, trunc,     ceilingFunc ),
#endif

I don't know which compiler and standard library termux is using, but maybe there is a problem?
I suggest first a quick check with CFLAGS=-z global and if the problem still occurs you might give LDFLAGS=-lm a try to see if it trunc is coming from the standard math library in termux. Otherwise you have to dig deeper into this termux specifically.
Cheers

@dominicletz The LDFLAGS solved the the problem. I am now facing a different issue since ExSqlean is compiled to EM_X86_64 instead of EM_AARCH64. At the moment I'm trying to compile ExSqlean with the correct architecture and if I manage to do that, how do I get the new version of ExSqlean to be used by exqlite? Is there a way to trigger it's compilation?

@leorodriguesrj ex_sqlean is really only for dev and test not necessary for downstream use.
{:ex_sqlean, "~> 0.8.5", only: [:dev, :test]},

@mindreframer might want to chime in here and see if they can support android compilation

@warmwaffles Great! I have my main project working correctly!

@leorodriguesrj good to hear!


Update README and Wiki

Is your feature request related to a problem? Please describe.
At the moment the readme and wiki are pretty bland and are needing a rework to help explain more what babblebot is and how to use it
Describe the solution you'd like
To give the viewer of the repo a better understand on babblebot and it's features
Describe alternatives you've considered
N/A
Additional context
N/A

Created branch update-readme to work on this issue, for PRs please merge to this branch

Created branch readme-update to work on this issue, for PRs please merge to this branch

Readme finished and pushed to staging

:tada: This issue has been resolved in version 2.1.6 :tada:
The release is available on:

v2.1.6
GitHub release

Your semantic-release bot :package::rocket:


Incorrect PSNR reporting from the c++ code

Sugimoto-san,
There seems to be an issue with the PSNR calculation in the C++ version. In the main.cpp::test_bilateral_filter function, cv::imread returns a 4-channel matrix even when a 3-channel (RGB) PNG is loaded (the 4th channel seems to be an empty alpha channel). As a result, the MSE is under-reported by a factor of ~4/3=1.333, making PSNR also seem higher.
Best,
Sean Young,

I just checked again, and cv::imread returns a matrix with the correct number of channels. However, if you check from within calc_psnr(), image1 and image2 are 4-channel matrices (this is true even for grayscale images).

Hello, seannz-san,
Thanks for your comments! I've checked the latest rep with a slight modification of setting to OpenCV4.2 and VS2017. The results seem to be good as far as I trace. I mean a matrix after imread() and two matrices in calc_psnr() are 3ch images when loading a 3ch PNG image (I check the bit-depth is 24 from file property dialog). May I ask you to send me an image file you are using?

Here's an image that I used for testing: http://r0k.us/graphics/kodak/kodak/kodim01.png
Thank you!

I've test the image and got the correct results (all the matrices mentioned above are 3-channel). I'm not sure why your code produces 4-channel images, but I'm happy if you correctly modify the code when seeing this strange behavior. Thanks!


Is it possible to validate 3 fields together ?

Versions:

VueJs: 2.5.13
Vee-Validate: 2.0.3

Description:
Hi,
this is not a bug, it's a question.
Is it possible to validate 3 fields together ?
I have 3 dropdown : day, month, year.
The date is not required.
But the buisness rule is :
If one on the field is set (month to april per exemple), then the two other fields (day and year ) become required.
I dont think it's possible with vee-validate, but if you have an idea it would be very appreciated.
Thanks,
Hugo

You could create a custom component that consists of those fields, then validate that component with v-validate.
Your component must be aware to emit the input event when the date value is complete otherwise it shouldn't. Otherwise it would be very annoying to tie in 3 fields in a validation rule.


Refactor miq report result

Have BinaryBlob work out the appropriate serializer for its data_type
Add some specs
Rearrange BinaryBlob specs so they don't all have the same shared
setup
Use FactoryGirl.build instead of .create
Remove a useless else clause
Remove a useless comment

I wanted to do some work to clean up a bit after https://github.com/ManageIQ/manageiq/pull/5420. Here's a first step by moving the logic done in MiqReportResult#report_results based on BinaryBlob's data into BinaryBlob
Tested this in the UI and everything looks good.
cc @gtanzillo
@miq-bot add-label refactoring, reporting, core

Had one question, but the rest looks great!

@Fryguy I addressed your comment above, so should be GTG when green :grinning:

:+1: Looks good. Nice improvement 🍻

@Fryguy just refactoring a little more based on @matthewd's suggestion

@Fryguy and....done. Can you rereview?

Very nice :+1:


Document permissions needed by Zot for S3 and DynamoDB

https://github.com/project-zot/zot/pull/1236

Done in https://github.com/project-zot/zot/pull/1236