Installation with Python failing on MacOSX

I have been doing a clean installation of a new laptop and found myself with an installation faliure, I am not sure is me or something has fundamentally change in the brew-core but the --with-python option is not working for me.
I would love to hear from others and perhaps check if it is something broken or I need more coffee.
The process is pretty straight forward

Installing homebrew

installing opencolorio as stated in the manual (https://opencolorio.readthedocs.io/en/latest/quick_start/installation.html)
> brew install opencolorio --with-python

it fails and throws this message
Error: invalid option: --with-python

Is this the result of brew-core not supporting options? is the installation process now obsolete?

I have also tried to install Python 3.8.6 via pyenv, this is my zshrc

`
Python Setup using PyEnv
if command -v pyenv 1>/dev/null 2>&1; then
eval "$(pyenv init -)"
fi
export PYENV_ROOT="$HOME/.pyenv"
export PATH="$PYENV_ROOT/bin:$PATH"
OCIO Setup
export OCIO="/Users/jordi/Dropbox/Repos/aces/OpenColorIO-Configs/aces_1.0.3/config.ocio"
source /usr/local/Cellar/opencolorio/1.1.1_2/share/ocio/setup_ocio.sh
`
Thanks in advance.

Hi Jordi,
brew install opencolorio should just work, that is how I got it set up on
my Macbook:
Kali:~ kelsolaar$ brew info opencolorio

opencolorio: stable 1.1.1 (bottled), HEAD
Color management solution geared towards motion picture production
https://opencolorio.org/
/usr/local/Cellar/opencolorio/1.1.1_2 (22 files, 3.8MB) *
Poured from bottle on 2020-07-19 at 19:09:51
From:
https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/opencolorio.rb
License: BSD-3-Clause
==> Dependencies
Build: cmake ✘, pkg-config ✔
Required: little-cms2 ✔<EMAIL_ADDRESS>✘
==> Options
--HEAD
Install HEAD version
==> Caveats
OpenColorIO requires several environment variables to be set.
You can source the following script in your shell-startup to do that:
/usr/local/share/ocio/setup_ocio.sh
Alternatively the documentation describes what env-variables need set:
https://opencolorio.org/installation.html#environment-variables
You will require a config for OCIO to be useful. Sample configuration files
and reference images can be found at:
https://opencolorio.org/downloads.html
==> Analytics
install: 219 (30 days), 680 (90 days), 3,032 (365 days)
install-on-request: 69 (30 days), 267 (90 days), 1,241 (365 days)
build-error: 0 (30 days)

Cheers,
Thomas Mansencal - colour-science.org https://www.colour-science.org -
thomasmansencal.com http://www.thomasmansencal.com
On Sun, 11 Oct 2020 at 22:47, jordibares<EMAIL_ADDRESS>wrote:

I have been doing a clean installation of a new laptop and found myself
with an installation faliure, I am not sure is me or something has
fundamentally change in the brew-core but the --with-python option is not
working for me.
I would love to hear from others and perhaps check if it is something
broken or I need more coffee.
The process is pretty straight forward

Installing homebrew
2.
installing opencolorio as stated in the manual (
https://opencolorio.readthedocs.io/en/latest/quick_start/installation.html
)

brew install opencolorio --with-python

it fails and throws this message
Error: invalid option: --with-python
Is this the result of brew-core not supporting options? is the
installation process now obsolete?

I have also tried to install Python 3.8.6 via pyenv, this is my
zshrc

`
------------------------------------------------------------------- Python
Setup using PyEnv
if command -v pyenv 1>/dev/null 2>&1; then
eval "$(pyenv init -)"
fi
export PYENV_ROOT="$HOME/.pyenv"
export PATH="$PYENV_ROOT/bin:$PATH"
------------------------------------------------------------------- OCIO
Setup -------------------------------------------------------------------
export
OCIO="/Users/jordi/Dropbox/Repos/aces/OpenColorIO-Configs/aces_1.0.3/config.ocio"
source /usr/local/Cellar/opencolorio/1.1.1_2/share/ocio/setup_ocio.sh
`
Thanks in advance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AcademySoftwareFoundation/OpenColorIO/issues/1165,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYLQ7WM4ULHPDEJAH5SXLSKF5K3ANCNFSM4SLVB2FA
.

Hi Jordi,
brew install opencolorio should just work, that is how I got it set up on
my Macbook:
Kali:~ kelsolaar$ brew info opencolorio
opencolorio: stable 1.1.1 (bottled), HEAD
Color management solution geared towards motion picture production
https://opencolorio.org/
/usr/local/Cellar/opencolorio/1.1.1_2 (22 files, 3.8MB) *
  Poured from bottle on 2020-07-19 at 19:09:51
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/opencolorio.rb
License: BSD-3-Clause
==> Dependencies
Build: cmake ✘, pkg-config ✔
Required: little-cms2 ✔<EMAIL_ADDRESS>✘
==> Options
--HEAD
	Install HEAD version
==> Caveats
OpenColorIO requires several environment variables to be set.
You can source the following script in your shell-startup to do that:
  /usr/local/share/ocio/setup_ocio.sh

Alternatively the documentation describes what env-variables need set:
  https://opencolorio.org/installation.html#environment-variables

You will require a config for OCIO to be useful. Sample configuration files
and reference images can be found at:
  https://opencolorio.org/downloads.html
==> Analytics
install: 219 (30 days), 680 (90 days), 3,032 (365 days)
install-on-request: 69 (30 days), 267 (90 days), 1,241 (365 days)
build-error: 0 (30 days)

Ohhh.. thanks KelSolar, that is GREAT.

@jordibares did @KelSolaar 's explanation solve this for you? Okay to close this issue? Thanks

@jordibares, please close the issue if you have the answer to your question or update it; otherwise, I will close the issue. Thanks.


400 Client Error HELP!!!

Well, i've passed some days without using the computer and when i came back, tryied to watch any streams and none have worked.
I've been tryied to reinstall the Livestreamer and VLC Player but still not working, please i need some help.

FOUND THE SOLUTION

What was the solution?

Solution is here: https://github.com/chrippa/livestreamer/issues/1481


catkin_make failed with error: invalid command 'rospy_message_converter'

Hi, thanks for the package but I am having trouble building it.
After running catkin_make I got the following error (I am using Noetic on Ubuntu 20.04 and I cloned from the master branch):
-- +++ processing catkin package: 'rospy_message_converter'
-- ==> add_subdirectory(rospy_message_converter)
-- Using these message generators: gencpp;geneus;genlisp;gennodejs;genpy
/opt/ros/noetic/lib/python3/dist-packages/_distutils_hack/init.py:18: UserWarning: Distutils was imported before Setuptools, but importing Setuptools also replaces the distutils module in sys.modules. This may lead to undesirable behaviors or errors. To avoid these issues, avoid using distutils directly, ensure that setuptools is installed in the traditional way (e.g. not an editable install), and/or make sure that setuptools is always imported before distutils.
warnings.warn(
/opt/ros/noetic/lib/python3/dist-packages/_distutils_hack/init.py:33: UserWarning: Setuptools is replacing distutils.
warnings.warn("Setuptools is replacing distutils.")
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: invalid command 'rospy_message_converter'
CMake Error at /opt/ros/noetic/share/catkin/cmake/safe_execute_process.cmake:11 (message):
execute_process(/home/milos/catkin_ws/build/catkin_generated/env_cached.sh
"/usr/bin/python3"
"/opt/ros/noetic/share/catkin/cmake/interrogate_setup_dot_py.py"
"rospy_message_converter"
"/home/milos/catkin_ws/src/rospy_message_converter/setup.py"
"/home/milos/catkin_ws/build/rospy_message_converter/catkin_generated/setup_py_interrogation.cmake")
returned error code 1
Call Stack (most recent call first):
/opt/ros/noetic/share/catkin/cmake/catkin_python_setup.cmake:46 (safe_execute_process)
rospy_message_converter/CMakeLists.txt:23 (catkin_python_setup)
-- Configuring incomplete, errors occurred!
See also "/home/milos/catkin_ws/build/CMakeFiles/CMakeOutput.log".
See also "/home/milos/catkin_ws/build/CMakeFiles/CMakeError.log".
make: *** [Makefile:4394: cmake_check_build_system] Error 1
Invoking "make cmake_check_build_system" failed

I replaced from distutils.core import setup with from setuptools import setup in setup.py and catkin_make passed without error.

That's weird. It compiles without a hitch on my system and on the buildfarm, so there is something weird with your system.


chore(docs): Fix docgen

Signed-off-by: Alex Collins<EMAIL_ADDRESS>Checklist:

[x] My organization is added to USERS.md.

TestWorkflowTemplateRefWithExitHandler

TestEstimatedDurationSuite/TestWorkflowTemplate


new light electric blue is poor contrast on white

before, which was much better:

Fixed in #1007


Error during prediction

Hi,
I'm getting the the error "ValueError: Object arrays cannot be loaded when allow_pickle=False" while I run "vgg16_bidirectional_lstm_predict.py" after the model training. I'm requesting your help to solve it.

try to change numpy
pip install numpy==1.16.1
https://stackoverflow.com/a/55903975


Not really an issue...

... but there is no comment function below codes ;)
I like this one, gives a rather easy way to parse strings instead of good-old strtok+str*cmp...

(closing immediately, because it is no issue ;) )


[Select][base-ui] Keyboard navigation does not work in Safari

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://mui.com/joy-ui/react-select/#basic-usage
Steps:

Open the example in Safari
Use the arrow keys to select an option (this does not work)
or use the tab key to focus the Select (this does not work)

Current behavior 😯
The Joy Select component does not work with the keyboard in Safari. There are two issues:

Selecting an option with the arrow keys does not work. Instead, the arrow keys scroll the page.
Unlike the MUI Select, the Joy Select cannot be selected with the tab key. This is an accessibility issues, especially when filling out forms.

Expected behavior 🤔
The Joy Select should work with the keyboard in Safari as it does in Chrome. The arrow keys should select menu items, and the tab key should focus the Select.
Context 🔦
No response
Your environment 🌎
No response

There are two other issues with the Joy Select:

When pressing Enter to select an option, nothing happens.
When pressing the Escape key to close the menu, nothing happens.

In Chrome, the Enter and Escape keys work as expected.

I couldn't reproduce the error you're reporting. I'm using Safari 16.6.
https://github.com/mui/material-ui/assets/4696105/cd47c1b1-8cd9-4e62-b89f-effa38f91e46

Hi @michaldudak,
I am using Safari 16.6. as well. I have no extensions or a11y software installed. I tried it on two different Devices and both have the same issue:

The MUI Select works fine, in the same browser:

OK, I see now. It happens when you open the Select using the mouse and then try to use the keyboard to navigate between items.
This also happens in the underlying Base UI Select.
Would you be interested in contributing to the project by investigating the issue and potentially fixing it?

I am seeing the same thing in @mui/material when using Safari:
https://github.com/mui/material-ui/assets/4729/f8947ec0-c86b-4cac-8dc7-5c1567637b98
It works as expected when using the keyboard to focus the input and then using the keyboard to open it:
https://github.com/mui/material-ui/assets/4729/2b7165f6-216e-46bc-8803-5bea4e4a16a3

It seems to be fixed in Safari 17. Could you please verify? I couldn't reproduce any of the errors mentioned here in this version.

Is Safari 17 the version that comes with Sonoma? Because that's what my video is from.

Is Safari 17 the version that comes with Sonoma? Because that's what my video is from.

Yes, Safari 17 is the version that comes with Sonoma.
I just tested the Material UI Select in Safari 17 but I can't reproduce the issue. Also, I can't reproduce the problems with the Joy/Base UI Select with Safari 17 anymore.

🤷 It's definitely not working on my end
https://github.com/mui/material-ui/assets/4729/99327233-279b-45e9-bffc-7e947a44c511

This issue is specifically about the Joy UI's Select. It seems we agree that it's been fixed in the latest Safari version.
@mikew, could you please open a new issue about the problem in Material UI?

@michaldudak confirming that this does in fact work in 14.1 / Safari 17.1, across mui base/joy/material


TLS 1.3 Retry Client Hello does not follow spec

I tested TLS 1.3 s_client against a server which required a different keyshare than the default offered.  The second client hello is missing extensions offered in the first client hello and the random value changes.
There was a recent discussion on this topic on the IETF TLS mailing list  and the conclusion was random should not change and only certain extensions may be altered.  https://www.ietf.org/mail-archive/web/tls/current/msg24409.html
Attached is a pcap.  (I zipped the file because github said pcap was not supported)
s_client_hrr.pcap.zip

I encountered the same thing. Below is what I saw using the current master branch of OpenSSL as both client and server. The random value changed as did the list of cipher suites and the contents of the signature algorithms and supported versions extensions. The ec_point_formats, session_ticket, encrypt-then-mac, and extended master secret extensions were dropped. While I don't necessarily find anything wrong with removing things from the second ClientHello that, based on the contents of the HelloRetryRequest, are known to be unnecessary, the recent discussion on the IETF TLS mailing list and https://github.com/tlswg/tls13-spec/pull/1076 indicate that doing so doesn't conform to the current draft.
First ClientHello
-----------------
ContentType:                16
ProtocolVersion:            0301
length:                     0108
msg_type:                   01
length:                     000104
legacy_version:             0303
random:                     bf897f29b0f336030eb2aa88e6e7499099a27af429cfd293a295b90fa501102a
legacy_session_id:          00
cipher_suites:              003e c02cc030009fcca9cca8ccaac02bc02f009ec024c028006bc023c0270067c00ac0140039c009c0130033009d009c130213031301003d003c0035002f00ff
legacy_compression_methods: 0100
Extensions lenth:           009d
server_name:                0000 000e 000c0000093132372e302e302e31
ec_point_formats:           000b 0004 03000102
supported_groups:           000a 000a 0008001d001700190018
session_ticket:             0023 0000
encrypt-then-mac:           0016 0000
extended master secret:     0017 0000
signature algorithms:       000d 0028<PHONE_NUMBER>0306030807080408050806040105010601030302030301020103020202040205020602
supported versions:         002b 0009 087f15030303020301
psk key exchange modes:     002d 0002 0101
key_share:                  0028 0026 0024001d002085b097c549635fd88d624d43c10c94c948aa5136347d9dd9e5ee8165c19c385a
---------------------------------------------------------------------------------------------------------------------------------------------------------------------
HelloRetryRequest
-----------------
ContentType:                16
ProtocolVersion:            0301
length:                     0010
msg_type:                   06
length:                     00000c
server_version:             7f15
cipher_suite:               1302
Extensions lenth:           0006
key_share:                  0028 0002 0019
---------------------------------------------------------------------------------------------------------------------------------------------------------------------
Second ClientHello
------------------
ContentType:                16
ProtocolVersion:            0301
length:                     010b
msg_type:                   01
length:                     000107
legacy_version:             0303
random:                     69fa924501a79dc19e9bc5b704610fab1b7c13a9e6ce2064b1ed3a9a5a490b91
legacy_session_id:          00
cipher_suites:              0008<PHONE_NUMBER>0100ff
legacy_compression_methods: 0100
Extensions lenth:           00d6
server_name:                0000 000e 000c0000093132372e302e302e31
supported_groups:           000a 000a 0008001d001700190018
signature algorithms:       000d 0016<PHONE_NUMBER>0306030807080408050806040105010601
supported versions:         002b 0003 027f15
psk key exchange modes:     002d 0002 0101
key_share:                  0028 008b<PHONE_NUMBER>8504005bcebba5dce5e58f1a063cf6ce5e009234002422210ec95d916938c7c843b7b9f3055a2ffee3ce789b8e41f3f6686621818899b422366bc755d62d04dadb59c9f600223403ea1f8182df402c5e30d846e81cd428edbdb27903b9131853f03359e40cb7e26bc684e86bd1d9c8b391db1735c0a6a61cac704f4812cec490a3deac0813f8

can confirm, at the very least random and cipher suites change, while aborting on changing random is a bit controversial, changing advertised cipher suites is definitely RFC non-compliant behaviour.
That's using de0dc006a524f3

It looks like cli from picotls cannot handshake with OpenSSL because of this.
Is this a bug of OpenSSL?

ping @mattcaswell

ping @mattcaswell


Duplicate deleter: clean up all unknown content IDs

This is a follow-up to #706.
Before this change, the duplicate deleter script didn't get rid of
all duplicate editions if there were multiple duplicates with the same
content ID (eg if a publisher archived a duplicate, there would be
one archived edition and one draft edition with the same content ID).
After this change, if a content ID is unknown to Publishing API,
the script will delete all editions with that content ID.
/cc @h-lame

Glad it was our fault and not another mongo weirdness that was the cause here. 👍


Render a focusable window

Hi!
Reading through https://github.com/muan/mojibar/issues/58 and all previous issues I understand it's not possible to refocus the current app after copying an emoji. ☹
But how about a workaround making use of the default native behavior? If your app rendered an invisible window as long as it has focus, it'd be possible to cmd + tab directly to the previous window. At the moment I have to hold down cmd, hit tab hit left and release cmd. Quite cumbersome and requires both hands on the keyboard.

Yeah, I understand that it cannot be done automatically, but just alt-tabbing back would still be a cool feature if possible.

Actually I think I have a fix for this here: https://github.com/muan/mojibar/pull/87
Looks like it's as simple as triggering a .blur() event on the browser window. Does this work on Mac/OSX 🍎  as well? Seems to work perfectly for me on Windows 10. Here's a demo.

Focus is returned since 2.5.0 thanks to @patricknelson & @jamiestraw.

Woohoo!!! Thanks guys, this is INSANE!!!!


Microbatch Support

Though clipping gradient for each instance brings higher accuracy, but it is very slow. Meanwhlie, layer.weight.grad_sample consumes too much memory, which forces me to use small model and small batch size. Could you support gradient clipping after averaging micro-batches like that in tensorflow-privacy?

Hi, we have the similar concept with called virtual_batch. Please take a look at our imagenet example


Update doccheck extension

Description

Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Seems like the issue remains 😥

How many files did you change?
What do you do?
Are you in our Slack community, you can maybe catch me there

Wrote you in Slack 🙂


(HTTP code 404) no such image - no such image: elasticsearch:latest: No such image: elasticsearch:latest

(HTTP code 404) no such image - no such image: elasticsearch:latest: No such image: elasticsearch:latest

Same for:
(HTTP code 404) no such image - no such image: kitematic/hello-world-nginx:latest: No such image: kitematic/hello-world-nginx:latest


Support lmdeploy pytorch engine

Motivation
Support eval models with lmdeploy pytorch engine.
Modification

add eval config
add LmdeployPytorchModel

BC-breaking (Optional)
None
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here and update the documentation.
Checklist
Before PR:

[ ] Pre-commit or other linting tools are used to fix the potential lint issues.
[ ] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

@grimoire @lvhan028 Could you guys review this PR? THX


Updated links back to bart links.

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

/do-e2e-tests

/do-e2e-tests


Validation of The Footer Subscribe button

Related Issue

Closes: #820
Description of Changes

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[ ] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

Looks good! Just pass CI test please ----> prettier

npx prettier --write

Thank you , Now that test has been passed please check

Thanks Hemant! your point will be added....


Fix tests for 1.9 (backports)

High Level Description

Adds simple disconnections retry decorator for more robust test cases
makes ec2 tests skip when not applicable

Related Issues

https://jira.mesosphere.com/browse/DCOS_OSS-1682
https://jira.mesosphere.com/browse/DCOS_OSS-1683
https://jira.mesosphere.com/browse/DCOS_OSS-1686

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and Review Process
What is the review process and when will my changes land?
All PRs require 2 approvals using github's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot bump-ee


Bump slurm-gcp to 5.7.5 (from 5.7.4)

Submission Checklist
Please take the following actions before submitting this pull request.

Fork your PR branch from the Toolkit "develop" branch (not main)
Test all changes with pre-commit in a local branch #
Confirm that "make tests" passes all tests
Add or modify unit tests to cover code changes
Ensure that unit test coverage remains above 80%
Update all applicable documentation
Follow Cloud HPC Toolkit Contribution guidelines #

/gcbrun


Add ComponentWithOverlay

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Change fifo Io to PIPE in shim , just do like go shim. Resovled the raw fd case problem.

Related
I have told this question to containerd . But looks like containerd will not change. So I will take a pr to change fifo to pipe.
I have complete this code , after some ci test ,I will submit this pr.

I found another two problem, when use fifo directly.

ctr run -d busybox:latest test   , it status will be stopping directly , but go shim will not.
when containerd service is stop , all rshim io will broken, but not go shim.

This is a method to get this error.
1、Get a image
Dockerfile like this:
FROM busybox:latest

COPY test.sh /

ENTRYPOINT ["sh","/test.sh"]

test.sh is blow this:
while true; do
    sleep 3
    echo "hello"
    result=$?
    if [ $result -ne 0 ]; then
        date >> log.txt
        echo "echo faile . Result : $result" >> /log.txt
    fi
done

docker build get this image.
use ctr import this image.
2、run a container
then use rshim to run a container.
3、get this error
stop containerd service . you can see the error message in this container. but go shim will not be influenced.
So I think use a pipe in shim may be completely needed.  This pr  which  I test can resolve this problem #278
friendly ping , @fuweid  @mxpv  @Burning1020 . Looking forward to your reply.

I  try to use two method to resolve the pipe io's problem . And I my case ,the io problem have been resolved.Add an io monitor service to check if the inode+dev of io has been changed, use read_fuf instead of token-copy in spawn_comy, and add timeout and put it in a loop.
My code like this
`
fn spawn_copy<R, W, F>(
from: R,
to: W,
exit_signal: Arc,
on_close: Option,
mut r: tokio::sync::mpsc::Receiver,
) where
R: AsyncRead + Send + Unpin + 'static,
W: AsyncWrite + Send + Unpin + 'static,
F: FnOnce() + Send + 'static,
{
let mut src = from;
let mut dst = to;
tokio::spawn(async move {
    let mut buffer: Vec<u8> = vec![0u8; 1024];
    //Change to loop and use time out, to make sure the read_buf will not hangon forever
    loop {
        let mut if_cn = true;
        let result = tokio::time::timeout(tokio::time::Duration::from_secs(5), async {
            tokio::select! {
                _ = exit_signal.wait() =>{
                    debug!("container exit");
                    if_cn = false;
                }
                r = src.read_buf(&mut buffer) => {
                    match r{
                        //Read n=0 but read_buf not close means pipe close
                        Ok(n) => {
                            if n == 0{
                                if_cn = false;
                            }else{
                                //Need sure the dist write complete?
                                let d_w = dst.write_all(&buffer).await;
                                if d_w.is_err(){
                                    if_cn = false;
                                }
                                buffer.clear();
                            }
                        },
                        Err(_) => {
                            debug!("read exit");
                            if_cn = false;
                        },
                    }
                }
                c = r.recv() =>{
                    debug!("fd error io exit!! recv {:?}",c);
                    if_cn = false;
                }
            }
        });
        //Timeout will continue unitl recv the io close
        let _ = result.await;
        if !if_cn {
            break;
        }
    }
    if let Some(f) = on_close {
        f();
    }
});

}
`
I will pull this pr  next week.

@fuweid  Please have a check #278

the stdout stream of container process which comes from rust-shim is not flush at real time, flush one page in one time then delay a long time, not line-by-line, I don't know if this related to that use FIFO as process stdout directly

This problem ,I didn't meet. Is there some method to get this problem?

I didn't do any special thing before i encounter this problem, I have a program with high frequency log out, and when I follow logs via crictl logs -f xxx I got very long delay between intermittent output, after some investigating i found that log file produced from containerd-cri also intermittent, I guess some abnormal thing from new way of using FIFO or rust tokio runtime.
Simple diagram:
Go shim:    |fifo reader| <-- fifo --> |io copier| <-- pipe --> |container process|
Rust shim: |fifo reader| <-- fifo --> |container process|
The fifo and fifo reader is containerd-cri and have no difference, i guess problem comes from second half

the stdout stream of container process which comes from rust-shim is not flush at real time, flush one page in one time then delay a long time, not line-by-line, I don't know if this related to that use FIFO as process stdout directly

I think maybe I've found the cause. I'll try to file a PR about it later.

Seeing level=error msg="copy io failed Input/output error (os error 5)" when running this, could this be related?

copy io failed Input/output
If you patched #278 ? If yes, Could you provide a more detailed description or some logs . For checking if it is my patch's problem.
Ps: binary io is not realize, nerdctl -t -d will fail.

Not patched. I will patch and try again.

Patched, same error, so not fixed with #278

Patched, same error, so not fixed with #278
Could you support the debug log?  It may caused by copy_console (tty) , there is no more information, so it cannot be determined.

This is what I could see already, any idea? I'll look at it more closely on Monday

This is what I could see already, any idea? I'll look at it more closely on Monday

I think in the spawn_copy  while the read/write side closed suddenly, it may print this.


Ability for Curators to Review/Track Each Deposited Work

From the epic (#1580 )

An automated check may be the best way to provide quick feedback to depositors about easily detected/correctable accessibility issues. However, we know that we likely won't be able to run an automated accessibility check on every type of file that is uploaded to ScholarSphere (in fact, we may only be able to do so with PDF files). We also know that a passing automated check does not at all guarantee that a file is actually accessible by all WCAG standards. The only way to guarantee that any given file meets the standards is for a human reviewer to carefully inspect it and remediate any issues.

To ensure that all works deposited in ScholarSphere are actually accessible, we would need each work (regardless of whether it was automatically checked) to enter into a queue for manual review by a ScholarSphere curator. After a curator has inspected each file that's attached to the current work version and worked with the depositor to remediate any issues, they would need a way to manually indicate that the work had been reviewed for accessibility and thus remove it from the queue. If a new version of the work is created in the process of remediating accessibility issues, then the work should not re-enter the queue, but if a new version of the work is created for any other reason, then it should re-enter the queue for another manual accessibility review.

This queue for manual review would be separate from the queue for works where the depositor actively requested an accessibility review from a curator prior to publication and should exclude works where curation was requested so that the curation of works in these two queues can be prioritized separately. The mechanism for resolving/removing works from both queues could probably be the same.

Open questions:

Again, could/should this list be tracked in AirTable?
 In other words, will there be enough curators to handle this work? Or will the to-do list just get hopelessly long?

Part of #1580

@bdezray

Based on the conversation on other tickets for this workflow, I think it's safe to say we'll just want to add a label 'Needs accessibility review' during the existing export to Airtable. The label only needs to be added if the files have not already gone through remediation (ie accessibility_remediation_requested is not true). #1608 will handle the addition of autochecker results, but even if everything passes, we should still add the label so it is at least briefly checked.


Try out lower precision types for neural networks

Jax supports 16-bit floating point types, https://github.com/google/jax/discussions/6700, would be a look for training e.g. neural ODEs.
There's also the recent BitNet work, in particular the 1.5b quantization looks interesting.

Doesn't work well for the test case I tried, let's set this aside for now.


[gardening] Fix inconsistent headers. Fix typos. Fix a vs. an issues.

Gardening:

Fix inconsistent headers
Fix typos
Fix a vs. an issues

@swift-ci please smoke test and merge


Add link to documentation

Link is missing in the README

https://github.com/meom-group/jaxparrow/blob/7c2bc7b55d038a67649220fd66b3f155dc067698/README.md?plain=1#L6

doc link is in the badge, but less visible than having a text link

closed in #83


Button in card heading

I had the idea that when you put a button in the card heading, it wouldn't deform the heading as it does at this point, but instead it would take the height of of the heading so that it has a nicely integrated look as shown on the screenshot in the lower card. The "Normal card with button" is the current state of what would happen, with the lower being what happens with this change.

Thanks, but we'll pass for now on this.


added file handler formatter

Hello @Disaxy,
Thanks for your Pull Request. Could you just fix the broken check and I will merge it.

Thanks


[Bug]: Auto Skip won't refresh playlist icons in a playlist, if you remove a song from another playlist it appears in.

🔍 Have you checked that you are using an auto updating method or latest version of the extension?

[X] I have checked that I am on the latest version.

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Choose the extension for bug report.
Auto Skip
ℹ Environment / Computer Info
Spotify for Windows (64 bit)
<IP_ADDRESS>2.g26c92729
Spicetify v2.35.1
Theme: marketplace
Extensions: bookmark.js, keyboardShortcut.js, allOfArtist.js, playbarClock.js, tracksToEdges.js, extended-copy.js, romaji-convert.js, playlist-icons.js, PlaylistProxy.js, spicetify-playlist-labels.js, dist\search-on-youtube\prism.mjs, section-marker.js, resume-playlist.js, spicetify-private-session.js, LuckyLP.js, spicetify-furigana-lyrics.js, pip.js, QueueTime.js, CoverAmbience.js, upcomingSong.js, AnonymizedRadios.js, songstats.js, wikify.js, adblock.js, utilities.js, who-added.js, imageOpener.js, playOnYouTube.js, addToQueueTop.js, lastfm.js, css-editor.js, removeFriendsActivity.js, playing-source.js, djinfo.js, autoSkipWrapper.js
Custom apps: marketplace

📝 Description
To reproduce:

Create Playlist One.
In Playlist One, add song(s) from an existing playlist (Playlist Two).
In Playlist Two, check if the icons from the new playlist show up here (they do).
In Playlist One, remove some of the songs you added.
Go back to Playlist Two. It didn't update the playlist icons next to the songs which are removed from Playlist One.

Restarting the app didn't refresh the icons. The icons finally refresh after editing Playlist One's name.
If this isn't reproduceable, the extension is probably clashing with something else I have installed...
📸 Screenshots
No response

Sorry, wrong extension! closing this.


[FEATURE] progress visibilty for the annotaiton progress

Is your feature request related to a problem? Please describe.
Yes the problem is regarding cross visibility for an annotation team, Currently with feedback datasets that support situations like multi task multi label and other complex tasks, labeling can be a tough challenge for annotators especially non domain experts, They might need guidance and reviews from principal annotators, thus progress visibilty is essential aswell as admin access to edit annotations in datasets that are truly shared with a single record per annotation.
Describe the solution you'd like
Annotators with admin role can view annotation progress in a better manner so anotator roles can get the work reviewed. (especially feedback datasets. ) The same annotators also have the ability to edit responses in the dataset in 'annotator' workspace.

@nataliaElv this might be relevant feedback for you

Hi @vmehta-cohere thanks for leaving a comment after our discussion! Just to make sure I'm getting this right, it's not so much about seeing the progress metrics (e.g. annotator submitted 20 out of 100 records) but the actual labels they applied during the annotation, right?

Yes the latter but both will be nice to have. @nataliaElv

@nataliaElv , we work with healthcare related datasets. We have nurses going over text data and labeling, for example, "chest pain symptomatic". Many cases are tricky and two things happen:

They need to ask a doctor to check the case and make a final decision;
They need to have a supervisor going over some of their labels and making sure that they are labeling as intended;

Ideally, we would like to give the admin role, or some other role, the ability to go to the examples ids ABC, or filter every example with some tag: NEED_CONFIRMATION, read them and change labels if necessary, to guarantee quality of the process.
Let us know if that makes sense

Thank you @FMelloMascarenhas-Cohere ! We will have this in mind when we design review workflows.


Fix Semantic Release Run Errors

There are two errors currently happening when the semantic release GitHub Action runs:
 SemanticReleaseError: Missing 'package.json' file.
 SemanticReleaseError: No GitHub token specified.
To fix these two errors, I will have to:

[ ] Remove the npm package from the .releaserc files
[ ] Rename the environment variable I have setup to GH_TOKEN
[ ] Create a tag for the last commit that went in

Looks like that even with the error that were thrown on the last run of the semantic-release GitHub action, it still did bump the version.

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


Compatibility with Spark 3.2

Continuing @alexott 's work from 1333. There were tests failing and I believe the reason was the new adaptive query execution, which was enabled by default in 3.2.
I resolved a few tests by simply handling the case of AdaptiveSparkPlanExec (the BatchScanExec object was nested within it), and another test by including spark_catalog as a valid cassandra catalog name (not 100% sure this is the desired handling of this test, let me know what you think).
Checklist:

[x] I have a ticket in the OSS JIRA
[x] I have performed a self-review of my own code
[x] Locally all tests pass (make sure tests fail without your patch)

Hi @bbenshalom there is a compilation error in the CI:
 [error] /home/runner/work/spark-cassandra-connector/spark-cassandra-connector/connector/src/it/scala/com/datastax/spark/connector/sql/CassandraDataSourceSpec.scala:289:29: wrong number of arguments for pattern org.apache.spark.sql.execution.datasources.v2.BatchScanExec(output: Seq[org.apache.spark.sql.catalyst.expressions.AttributeReference],scan: org.apache.spark.sql.connector.read.Scan,runtimeFilters: Seq[org.apache.spark.sql.catalyst.expressions.Expression])
[error]         case a@BatchScanExec(_, _: CassandraInJoin) => a
[error]                             ^
[error] one error found
[error] (connector / IntegrationTest / compileIncremental) Compilation failed
[error] Total time: 23 s, completed Apr 11, 2022 6:49:06 AM

Hi @jtgrabowski , fixed. Any way for me to re-run the tests workflow myself?

You can run the integration tests locally with sbt/sbt it:test, see this page for details. Once this PR is merged, you'll be free to run the CI tests on your own. Btw. could you please sign the DataStax Contribution License Agreement https://cla.datastax.com/ ?

I signed the license agreement

@jtgrabowski The test that's failing in the CI is skipped for me locally ("skipped because not DSE"), I couldn't find in the contribution tips how to define a "DSE mode"... Can you please advise or point me to the right documentation?

Try
export CCM_CASSANDRA_VERSION=5.1.24
sbt/sbt 'it:testOnly com.datastax.bdp.spark.search.SearchAnalyticsIntegrationSpec'

It may seam to hang for a couple of minutes. The test needs to download DSE zip. The zip is downloaded to /tmp/ccm-xyz.

@jtgrabowski it seems that the current failing test has to do with a change in spark 3.2:

In Spark 3.2, DataFrameNaFunctions.replace() no longer uses exact string match for the input column names, to match the SQL syntax and support qualified column names. Input column name having a dot in the name (not nested) needs to be escaped with backtick `. Now, it throws AnalysisException if the column is not found in the data frame schema. It also throws IllegalArgumentException if the input column name is a nested column. In Spark 3.1 and earlier, it used to ignore invalid input column name and nested column name.

In other words, for the column name ~~funcolumn, the dataframe filter generated is ~~funcolumn. I could just remove backticks from the column names in the code, but it doesn't seem right, since the intention of the Spark change seems to be to avoid "weird" column names. Do you think it would be better to simply remove the test and not support this functionality?

I'm hesitant on just dropping the support for this. Could we match the column name with the backticks instead?

@jtgrabowski I tried to match the column names with the backticks while building the cassandra filters, and indeed the execution plan was displayed correctly. However, the test then failed on the collect() method of the DataFrame, again because of the same mismatch. The filter already has the backticks in it when it is passed to CassandraScanBuilder (called by spark code)...
Spent quite a bit of time on this and couldn't find a way around it. Open to suggestions...

I didn't have time to look into it yesterday. I'll try to help next week.

@bbenshalom here is a PR that should fix the offending test: https://github.com/bbenshalom/spark-cassandra-connector/pull/1. Please take a look when you get a chance.
I also created this: https://datastax-oss.atlassian.net/browse/SPARKC-677 as the "weird column names" are not fully supported. The problems appear with the older Spark versions too, so I don't think it should block us from moving forward with this PR.

@jtgrabowski thanks for the PR. Do you have any plan as to when we can merge & release a new version (even a release candidate)?

We need to do some behind-the-scenes testing before we can release. I'll kick off the performance tests and let you know.

@bbenshalom @alexott thank you for your contribution! This is now merged!

@jtgrabowski thank you for your patience and great support! Was a pleasure!

perf results are ok, the release is scheduled for next week :+1:


support fields masks on list APIs

When listing lots of objects, unnecessary data can be sent over the API. Add the ability to include a list of field_mask's, such that the response will return items with only the requested fields.
See #2226 for details on why this is a problem.

Whether this issue has been implemented ？ i see the latest code has filter , such as
func (s *containerStore) List(ctx context.Context, fs ...string) ([]containers.Container, error) {
...
}

@dmcgowan thank you, and maybe we need change some .proto file in api/services. such as
message ListContainersRequest {
	repeated string filters = 1;
        repeated string fieldMask = 2;
}

is that right?

See initial implementation at https://github.com/kunalkushwaha/containerd/tree/filed-mask from #2298

Hi @dmcgowan , @stevvooe ,I'm a student from UT Austin. I am taking a virtualization class that requires us to work on an open source project, so I was wondering if you are still looking for people to contribute to this. If so, can I get more information on this issue? Thank you very much!

is this issue still open to pick up? If so, i would like to do so.

@SaiHarshaK yes, go ahead--no one else is currently working on it

See initial implementation at https://github.com/kunalkushwaha/containerd/tree/filed-mask from #2298

Can anyone give an example where fieldmask path has strings like "labels.", etc?

Can anyone give an example where fieldmask path has strings like "labels.", etc?

Hi @SaiHarshaK  you may take a look at test case container_test.ho
Also, possible field masks that were implemented in initial implementation for possible values are here container.go.
hope it helps.

@dmcgowan @estesp @stevvooe  Could you please reiterate what is required since it's been a long time and why the previous implementation was not valid for that case?

I'd like to work on this if no one else is.

Hello, I am a student from the University of Texas at Austin. I wanted to know if a group of my colleagues and I could work on this issue as part of an assignment for my virtualization class as well as a recap of all the work that's been done on this issue so far since there's a long implementation history. Thanks in advance!

@Nit123 go for it!

Hi! Is this issue still open to be worked on?

For clarification, what is the exact problem we are trying to solve?  If we want to control what information is being returned to the client, filtering can be applied at the Container.Info level.  This, will not reduce the size of gRPC calls, but will honor metadata caching, which was implemented two years ago (after the most recent PR attempt for this issue).
To reduce the amount of data transmitted over the wire for large list operations, the change will need to be made at the Store.Get level.  This is the approach that the aforementioned PR took.  To get it to work with metadata caching, I can think of two options:

Ignore metadata cache if a field mask is provided, and always fetch from the Store.
Break out the filtering logic to a new function that takes a Container and a field mask as arguments, that returns a new, filtered Container.  This function would, in theory, be applied both within Get calls (when being requested from the underlying Store), and within Info (against c.metadata, before return).

Thoughts? I would be willing to give it a go, I just want to make sure I'm solving the right problem first.

Can we use a GraphQL like structure to specify return fields, e.g.
{
  "query":{
      ID
      name
   }
}

Hii!! Is this issue still open by any chance??


Atom with multiple highlights has inconsistent highlight radius

Describe the bug
Adding more than one highlight colour to an atom leads to having a larger highlight radius for that atom, whether or not radii are specified. It also looks like the actual highlights aren't being shown as semicircles (for two colours) or wedges (for three or more colours).
To Reproduce
code here, image below.
import rdkit
print(rdkit.__version__)
from rdkit import Chem
from rdkit.Chem.Draw import rdMolDraw2D
from collections import defaultdict
from IPython.display import Image

smi = 'CCC'
mol = Chem.MolFromSmiles(smi)

colors = [(0.9019607843137255, 0.6235294117647059, 0.0),
          (0.33725490196078434, 0.7058823529411765, 0.9137254901960784),
         (0.0, 0.6196078431372549, 0.45098039215686275)]
highlightatoms = defaultdict(list)

for i in range(mol.GetNumAtoms()):
    highlightatoms[i].append(colors[0])

#KEY line: adding a second colour for one atom:
highlightatoms[i].append(colors[1])

#can also see what happens when adding a third colour
#highlightatoms[i].append(colors[2])

#image stuff:
width=400
height=400
d2d = rdMolDraw2D.MolDraw2DCairo(width,height)
d2d.DrawMoleculeWithHighlights(mol,'mol',
                               dict(highlightatoms), #atom highlights
                               {},#bond highlights
                               {},#atom highlight radii,
                               {} #bond highlight radii
                              )
d2d.FinishDrawing()
png = d2d.GetDrawingText()
Image(png)

Expected behavior
I was expecting to see something like cell 20 in this blog post - half-circles for two colours with the same radius as other highlighted atoms.
Screenshots

Configuration (please complete the following information):

RDKit version:  2022.09.5
OS: Mac OS Ventura 13.2.1
Install: pip in a conda env

Thanks for your time!

Thanks.  I'll take a look at it in the next few days.  The drawings in cell 20 in that blog post are also now odd in the latest version.

Duplicate with #6200


Prettier Pre-commit Hooks

Is your feature request related to a problem? Please describe.
Docs diffs become much easier to maintain when there is an opinionated code formatter for the Markdown files such as Prettier.
Prettier also allows opinionated formatting of other files relevant to python projects including YAML and JSON.
Describe the solution you would like
An ideal solution would consist of the following:

[ ] Addition of the prettier pre-commit hook
[ ] Reformatting of all existing files using Prettier

Additional context
If this feature is of interest, feel free to assign it to me for completion.

This sounds good to me! The only challenge I foresee here is that currently, when running
pip install cookiecutter
cookiecutter https://github.com/fpgmaas/cookiecutter-poetry.git

the following commands will run successfully:
cd <project_name>
git init -b main
git add .
git commit -m "Init commit"
git remote add origin git@github.com:<github_author_handle>/<project_name>.git
git push -u origin main

and the CI/CD pipeline will run successfully. However, after introducing this pre-commit hook, the CI/CD pipeline will likely fail. The reason for this is that it is incredibly difficult to format all the files in the {{cookiecutter.project_name}} folder in exactly such a way that, whichever combination of argument a user gives, they are exactly in the format prettier would want. And even if we manage to do so, it is an unrealistic goal that we manage to keep it that way.
So we would maybe need to update the set of commands above to also run the pre commit hooks once before pushing.
If we can manage to find a clean solution to that, I am happy with this change!


fixed timezone shift in some cases + added test case. Fixes #1326

Removed the before/after timezone check in getMoment() and always apply .tz() #1326
Simplified the code in getMoment()
Created a hasTimeZone() private function
setValue now sets the timezone on the given moment to display the date in the given options.timeZone

Test case:

Added a dependency on grunt-contrib-connect : the jasmine tests are now hosted on a temporary local web server in order to be able to load the moment-timezone's JSON data file.
Added a dependency on grunt-env : Lets me choose the time zone used by PhantomJS for the jasmine tests.
Added a test case for the bug #1326

Hopefully this will be merged soon. Having the same timezone shift issues.

I'm also experiencing a shifting time due to timezone issues. Reverting back to 4.15.35 helped me, but getting something like this merged in sooner rather than later would be much better.


Fixed _mm_insert_ps in correct translation of the function.

The issue was a wrong copy and incorrect masking. https://github.com/simd-everywhere/simde/issues/931
I haven't fixed the unit test as I don't have the dev setup of the library (I am working on windows system). Haven't used a Debian set up before.
If that is a requirement than please post a link how to set that up and run.
I think the unit doesn't test the scenarios doesn't test the 3rd parameter (flag) with more values. https://github.com/simd-everywhere/simde/blob/master/test/x86/sse4.1.c#L1977
Which is why it passes. And probably this will pass as well. But a better unit test is needed.

Thanks @MirJawadMairaj
For compiling using Debian on MS Windows, try the first 4 instructions at https://github.com/common-workflow-language/cwltool/#ms-windows-users and then follow https://github.com/simd-everywhere/simde/wiki/Development-Environment#x86--x86_64 for running the tests
See https://github.com/simd-everywhere/simde/wiki/Implementing-a-New-Function for hints on writing new test cases

Hi, I removed the trailing spaces. But it still fails on the same lines.

But it still fails on the same lines.

? The formatting check passes now https://github.com/simd-everywhere/simde/runs/4105015189?check_suite_focus=true

Cool! Then I interpreted the failure email wrong. It was older and maybe delayed a bit. I should have seen the hash it complained on.

@MirJawadMairaj Can you suggest some values that now produce the correct result buy previously did not?

Sure!
So I had a bug around not rendering a lot of meshes because of a wrong transformation. So the confirmation for me was to see them being rendered after the fix. But I gathered two such runs through visual studio (and these are running the actual intel simd instruction as opposed to ARM translations which I am going to assume are fairly close). Of course I did some tests on a smaller scale with toy values.
code snippet:
v0 = _mm_insert_ps(v0, vec, 0b00110000);	// insert trans.x into bits 0-31 of row0
v1 = _mm_insert_ps(v1, vec, 0b01110000);	// insert trans.y into bits 32-63 of row0
v2 = _mm_insert_ps(v2, vec, 0b10110000);	// insert trans.z into bits 64-95 of row0

Run 1)

vec =  {3360.90796, 1192.48792, 1759.03296, -4.37113883e-08}
v0_before = {-4.37113883e-08, 0.00000000, 1.00000000, 0.00000000}	float[4]
v0_after =  {-4.37113883e-08, 0.00000000, 1.00000000, 3360.90796}	float[4]

vec =  {3360.90796, 1192.48792, 1759.03296, -4.37113883e-08}
v1_before = {0.00000000, 1.00000000, 0.00000000, -1.00000000}	float[4]
v1_after = {0.00000000, 1.00000000, 0.00000000, 1192.48792}	float[4]

vec =  {3360.90796, 1192.48792, 1759.03296, -4.37113883e-08}
v2_before = {-1.00000000, 0.00000000, -4.37113883e-08, 3360.90796}	float[4]
v2_after =  {-1.00000000, 0.00000000, -4.37113883e-08, 1759.03296}
Run 2)

vec = {3328.00000, 1259.00000, 1781.00000, 1.00000000}	float[4]
v0_before = {1.00000000, 0.00000000, 0.00000000, 0.00000000}	float[4]
v0_after = {1.00000000, 0.00000000, 0.00000000, 3328.00000}	float[4]

vec = {3328.00000, 1259.00000, 1781.00000, 1.00000000}	float[4]
v1_before = {0.00000000, 1.00000000, 0.00000000, 0.00000000}	float[4]
v1_after = {0.00000000, 1.00000000, 0.00000000, 1259.00000}	float[4]

vec = {3328.00000, 1259.00000, 1781.00000, 1.00000000}	float[4]
v2_before = {0.00000000, 0.00000000, 1.00000000, 0.00000000}	float[4]
v2_after =  {0.00000000, 0.00000000, 1.00000000, 1781.00000}	float[4]

This was merged as https://github.com/simd-everywhere/simde/commit/94e75690e62335a822962520cbbd0208991e63a5 ; thanks!


handle because / cause / cuz

hubot always responds with for when a reason is given.
thing-- because reason
this throws out
thing has X points for reason
It should be
thing has X points because reason

I'm wondering if there's some reasonable way of getting this to work in languages other than english, I mean, without including every possible translation of the word for.

Well, it already matches because, cuz, for and a few more things. The scope of this is to regex match those and reply with the matched phrase rather than printing for each time.

@ParthKolekar gotcha, I think I mixed up things. I'm gonna create a separate issue.


Handle Offline case

In case of no internet connectivity
this fails:
 let release = zed::latest_github_release(
            "fwcd/kotlin-language-server",
            zed::GithubReleaseOptions {
                require_assets: true,
                pre_release: false,
            },
        )?;

which end up giving Some("") . And resulting in lsp not starting even if present

feel free to open a pr
when I have some time I can look if / how other extensions are handling this

Some are like this and some use local binary from PATH using worktee.


Expand Ngebanjar Dictionary

The current list of words in our Ngebanjar dictionary is limited, and we need to enrich it with more words and their meanings based on the eBook dictionary of the Banjar language that I have.
Goals:

Word Enrichment: Add new words and their meanings to the Ngebanjar dictionary from Kamus Banjar-Indonesia.
Meaningful Definitions: Ensure that each added word is accompanied by accurate and meaningful definitions or explanations.
Organized Categorization: Categorize the new words into relevant sections or themes for easier navigation and learning.
Quality Assurance: Review and verify the accuracy and appropriateness of the added words and definitions.

Proposed Approach:

Extract the word list and definitions from Kamus Banjar-Indonesia.
Compare and cross-reference the extracted content with the existing Ngebanjar dictionary to identify new words for inclusion.
Add the new words along with their meanings and categorize them appropriately within the game's dictionary structure.
Conduct a review process to ensure the correctness, relevance, and quality of the added words and definitions.
Test the functionality of the expanded dictionary within the game to confirm seamless integration and usability.

Expected Outcome:
By expanding the Ngebanjar dictionary with content from Kamus Banjar-Indonesia, users will have access to a richer vocabulary and more comprehensive learning experience of the Banjar language through our web game.
Additional Notes:

Consider involving linguistic experts or native speakers to validate the accuracy and authenticity of the added words and definitions.
Maintain consistency in formatting, presentation, and language usage across the entire dictionary for a cohesive user experience.

https://github.com/GemaHanna/ngebanjar/blob/main/src/config/dictionary.js


Updating GhentAnalysis fork master branch

This pull request takes updates from the central columnflow to the GhentAnalysisFork. The main updates are listed below:
Major updates:

stitching datasets is now possible within columnflow/production/normalization.py
Fetch an lfn via a fallback mechanisms
broken files can now be identified in the cf.GetDataLFNs task if the list of broken files is given in dataset_inst.aux.broken_files

Minor fixes:

.path function is changed to .abspath

Cheers, Jules

This pull request is closed as I instead first resolved all configs in another branch


Variable name is not correct

My JSON schema looks like
        "enum": {
          "type": "array",
          "description": "",
          "javaOmitEmpty": true,
          "items": {
            "$ref": "#/definitions/kubernetes_apiextensions_JSON",
            "javaType": "io.fabric8.kubernetes.api.model.apiextensions.JSON"
          }
        },
        "default": {
          "$ref": "#/definitions/kubernetes_apiextensions_JSON",
          "javaType": "io.fabric8.kubernetes.api.model.apiextensions.JSON"
        }

Variable in class getting created has name _default and _enum Any pointers, may be I am doing it wrongly. TIA

default and enum are reserved words in Java

(we use annotations to ensure the JSON name will still be correct)

Can you point me to an example regarding fixing this?

Sorry, I don't understand what you mean. We add an underscore because 'enum' and 'default' are not allowed to be used as Java identifiers. They're keywords of the language.
This plugin will add annotations to every field so that the JSON name remains correct.
What is the issue?

I have JSON schema which has objects by the name, default and enum. I am further using java class to create fluent classes which are failing if the variable name is _default and _enum. Is there any workaround I can do to et java classes with the name of variable default. I have only one class like this. TIA

Please try creating a field in a Java class called enum or default. You'll see it's a compile error. We can't work around that, it's a restriction of the language:
https://docs.oracle.com/javase/tutorial/java/nutsandbolts/_keywords.html


About pusedo-points on waymo

hi~ thanks for your work!
Why not do multimodal experiments on WOD ?
Table 3  is TED-S in your paper

The WOD did not provide the depth ground truth to train the PENet. So we only evaluated the LiDAR-only detector.


Add TiDB TLS config for tidb-dashboard

What problem does this PR solve?
Add TLS config for tidb-dashboard to connect to TiDB
Check List
Tests

Manual test (add detailed scripts or steps below)

Enable mTLS authentication by specifying ssl-ca, ssl-key, ssl-cert on TiDB
Add tidb-cacert-path, tidb-cert-path and tidb-key-path on pd config file.

[security]
  tidb-cacert-path = "/path/ca.pem"
  tidb-cert-path = "/path/client.pem"
  tidb-key-path = "/path/client-key.pem"

Check the if the embedded tidb-dashboard works

Code changes

Has configuration change
Add three more config field on security section:tidb-cacert-path, tidb-cert-path and tidb-key-path

Related changes
https://github.com/pingcap-incubator/tidb-dashboard/pull/219

Resolved the conflict I caused. PTAL @nolouch @disksing @rleungx

Codecov Report

Merging #2246 into master will increase coverage by 0.03%.
The diff coverage is 21.42%.

@@            Coverage Diff             @@
##           master    #2246      +/-   ##
==========================================
+ Coverage   76.53%   76.57%   +0.03%
==========================================
  Files         199      199
  Lines       21522    21536      +14
==========================================
+ Hits        16472    16491      +19
+ Misses       3826     3823       -3
+ Partials     1224     1222       -2

Impacted Files
Coverage Δ

pkg/grpcutil/grpcutil.go
72.22% <ø> (ø)

server/config/config.go
78.43% <18.18%> (-1.48%)
:arrow_down:

pkg/dashboard/apiserver.go
73.91% <33.33%> (-6.09%)
:arrow_down:

server/region_syncer/server.go
81.25% <0.00%> (-10.94%)
:arrow_down:

server/region_syncer/client.go
84.73% <0.00%> (-1.53%)
:arrow_down:

server/tso/tso.go
79.56% <0.00%> (-1.46%)
:arrow_down:

pkg/btree/btree.go
86.84% <0.00%> (-0.81%)
:arrow_down:

server/server.go
80.87% <0.00%> (ø)

server/schedule/operator_controller.go
81.27% <0.00%> (+0.17%)
:arrow_up:

server/handler.go
51.61% <0.00%> (+0.43%)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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

/merge

/merge


How to reset bots to lvl 1

in cursors-js how do you reset the bots back to lvl 1? im trying to make bots that will follow certian players around, and I need a way to reset them back to lvl 1 without restarting the app, so can you please add a feature that will reset them to lvl 1?

no
I'm not working on this anymore


Invalidate cache on all post_save signals?

In models.py invalidate_cache is called on all model saves:
 post_save.connect(invalidate_cache)

Should it be this?
post_save.connect(invalidate_cache, sender=UserPreferenceModel)
post_save.connect(invalidate_cache, sender=PerInstancePreferenceModel)
post_save.connect(invalidate_cache, sender=GlobalPreferenceModel)

I found this when trying to work out how to invalidate template cache on UserPreferenceModel update as I'm caching template fragments that need to be cleared if the user changes their preferences.  My plan is to connect to the post_save signal and call an additional invalidate cache routine.

As you can see in the models file, before doing anything in the invalidate_cache function, we check that the saved instance is actually a preference.
I agree, this may be better to explicitely bind each signal to specific class, but it would make the setup more difficult for people wo want to bind preferences to their own models. The overhead should be quite small, but if you have any benchmark showing a significant impact, I'm okay to work on that.
Regarding the template issue, one way to achieve that would be to add a last_update field on preferences models, and to use this new field as a fragment of the cache key.
However, since most access to the preferences are directly done on the value (and not on the underlying database object), this would mean implementing something else to access this new field as well.
Another way to do it, without touching anything in dynamic_preferences, would be to add a preferences_last_updated on your User model, and update this field each time a preference is saved, by binding to the post_save signal of UserPreferenceModel. However, it's less accurate, because it does not allow for per preference template caching.
I do not have much time to work on this right now, but if you want to submit a PR regarding this, I'd be more than happy to help you sort this out :)

I'm closing, feel free to reopen if you want to work on this.


add currencyCode and value to event's parameters

Leanplum API doc
Hey, I found it would be nice to add currencyCode and value to the purchase event.
Leanplum will automatically convert the value into your preferred currency.
If the currencyCode and value added to the event, you can see the value of the user purchase in Leanplum dashboard.

This seems fine to me. @PalSzabolcs  ?

Sure, looks good to me, too! 👍

@lipowen just released this to rubygems as version 4.2.1 (I fixed a minor gemfile warning in #28) - https://rubygems.org/gems/leanplum_api
Thanks! 🍻

Thanks! 🎉


[WIP][KYUUBI #2021] Command OptionParser for launching Hive Backend Engine

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Could you help me review the underway work？If it is feasible, I will continue to do it. @yaooqinn

these args are all in the strict form of --conf key=val

currently, we pass them in procbuilder as java options in -Dkey=val form

these args are all in the strict form of --conf key=val

currently, we pass them in procbuilder as java options in -Dkey=val form

Thanks, I see.


adc/nrf52_adc: Driver improvements and fixes: calibration and gain

This PR addresses an issue with setting up the channel gain and makes the ADC calibrate(if calibrate is active) on every sample right after a configuration.

@gonccalo the problem you mentioned is fixed, please test. :)

It's working, thanks!

Thanks fro reviewing/testing everyone :)


Add support for beginner Hot Cold clues.

According to nightfirecat, this doesn't account for the dig boxes of beginner clues being smaller than masters. (they are 3x3). See also #8974

It does work correctly. The four highlighted tiles mark the corners of the dig zone. Standing one tile outside shows the correct location.


Unable to save object with specified uuid ID

When I'm trying to save model to the database with specified UUID ID it saves with another random UUID ID. It looks like a bug.
In my case I'm trying to transfer data from another database and I really need to use my old UUIDs as PRIMARY KEY ID.
After some debugging I found where the problem is, we should check if UUID ID is already specified and only in case when it isn't fluent should generate random UUID.
Please take a look at this pull request https://github.com/vapor/fluent-postgresql/pull/76
If everything is ok could someone please merge it and add it to the next release? 🙏

@tanner0101 could you please give your feedback regarding this issue? 🙏
I really really need it to be fixed somehow.

@tanner0101 this is the same issue as #56. There have been several pull requests to address the issue #60 and #76. What can we do to help figure out a path forward?

Same issue. added self here.

it seems to work now when i use create(on: req) instead of save(on: req)

@patchthecode
With UUID it doesn't work with '.create'
'.create' works for Int IDs.

@MihaelIsaev maybe something changed?
Because i am currently using .create with UUID and it is working.
you can find the code in DummyData on the test server.

Fix was merged, thanks a ton @MihaelIsaev!


Angular2 base implementation updated to Beta 17, optimized implementation added

This updates the angular2 implementation to beta 17 (which improves performance by itself) and applies only the "trackBy" optimization on top of that to get an optimized implementation. (No OnPush parts which didn't meaningfully affect performance, just a single clean change)
Some approximate numbers from my runs on Chrome:
beta0 (previous angular-2 impl) - 3000ms
beta17, no trackBy (new angular-2 impl) - 2600ms
beta17 with trackBy (new angular-2-optimized impl) - 2100ms

Cool, thank you! Based on this, I think it'll make sense to stop using keys in the unoptimized React and Elm versions even though it is a correctness issue without.
I also am going to keep a reference to the version where we added OnPush so it's clear how that compares to lazy in Elm. The fact that you need to change architecture and bring in immutability to get the effect of lazy is important IMO, and it's kind of misleading to hide that.


Add:  Open source processes, policies, and info

Signed-off-by: soniasingla<EMAIL_ADDRESS>Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
This PR adds Open source processes, policies, and info at Embark!

The alignment looks a little off on mobile at the moment. Needs a little bit of CSS fidding!

I was wondering if having two call-to-action buttons on the hero image is too much?

@lpil another option could be adding text on the hero image after the line  In our open source work, we're exploring and pushing the boundaries of new technologies, and sharing our learnings with the community., but I felt it will be over-texted!

+1 to Louis' comment, needs a bit of tweaking!

Closing this due to inactivity. Thanks gang


Gettext extraction

Ticket: #1754
Issue
To make gettext extraction work, the Timber\Twig class needs to be instantiated without a WordPress environment. For this, we need to remove / work around WordPress functions in the Timber\Twig::add_timber_functions() method.
Solution
This pull request

Moves the hooks out of the constructor.
Moves the timber/twig filter to the Timber\Loader class.

Impact
There should be no impact.
By moving the timber/twig filter into the Timber\Loader class, we make sure that custom functionality is added to the Twig Environment as a last step, which is probably what most would expect.
If a duplicate filter or function is added to Twig, the order doesn’t matter, there will always be a LogicException. However, if your custom function or filter causes the error, the error log would probably make more sense, since it’s your code that will be flagged, and not Timber’s.
Usage Changes
None.
Considerations
This pull request is not about proper handling of the Twig Environment. This could probably be improved by the ideas proposed in #1493.
Using an init() function for adding hooks is a pattern that we often see in WordPress development. Also see this blog post. I don’t know however whether it’s better to use a static pattern or a non-static pattern.
// Static

Twig::init();

class Twig {
    public static function init() {
        $self = new self();

        add_action( 'example', [ $self, 'example' ] );
    }
}

// Static with __CLASS__

Twig::init();

class Twig {
    public static function init() {
        add_action( 'example', [ __CLASS__, 'example' ] );
    }
}

// Non-static

$twig = new Twig();
$twig->init();

class Twig {
    public function init() {
        add_action( 'example', [ $this, 'example' ] );
	}
}

Of course these are not all popular patterns that exist out there. The pattern using __CLASS__ is common in order to make it easier to unhook any hooks. But then, we would have to change the functions that are being called by the hooks to be static. Maybe @pascalknecht has some insight and experience with this? I’m not sure whether it’s better to let people unhook some hooks (and possibly run into side effects) or whether there should be a different way to control functionality in case something needs to be disabled.
Testing
No additional tests added.
@drzraf Can you check if the pull requests adds the changes we need with the5 gettext-extraction branch.

Ooof, this sounds quite a rabbit trail — thanks for the code and the notes. Good news is that all tests pass!
I wish I had stronger feelings on the static vs. non-static Q. My thought is that we should default to having the lightest touch possible at this point in the 1.x lifecycle. Thus, while changing it from static to non-static might be the right move in the future, I think we'd want to know with certainty (and perhaps, hold that change until a later 2.x release).
Other than that, I just want to be super-careful once we merge this to immediately catch it up with the 2.x branch. Handling the merge conflicts there has started to become a real minefield (esp. as we're doing more code rearrangement vs. new files).

Thus, while changing it from static to non-static might be the right move in the future, I think we'd want to know with certainty (and perhaps, hold that change until a later 2.x release).

Yes! And oh damn, I realized I actually got it mixed up in the code! Let’s go with the slightest touch for now.
If can also take care of merging this into 2.x. Especially with the filter hook refactors, there will probably be merge conflicts.

Codecov Report

Merging #2238 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #2238   +/-   ##
=========================================
  Coverage     94.88%   94.88%
  Complexity     1584     1584
=========================================
  Files            50       50
  Lines          3849     3849
=========================================
  Hits           3652     3652
  Misses          197      197

Impacted Files
Coverage Δ
Complexity Δ

lib/Twig.php
99.48% <ø> (-0.01%)
87.00 <0.00> (ø)

lib/Loader.php
96.29% <100.00%> (+0.04%)
67.00 <0.00> (ø)

Continue to review full report at Codecov.

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

Looks good! I'll see how the 2.x merge goes ...


Add logic elements to allow switching reportlets into "rodent-mode"

As per meeting today.

This will require:

[ ] options in the interfaces to change the orientation of the image in the reportlet generation
[ ] logic at the command line to initiate these changes

Depends on #5 and #33, and then the development of a CLI


Upgrade k8s and deps to v1.29.4

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Release note:
none

/cc @Karthik-K-N

/retest

/retest-required


Compiles but does not work.

The code fails on its first try when tested with PWS from OJ4 DSA.
It does not accept the range for entry 4 and crashes prematurely.
Ppws.pdf

An error was noticed in the line arrays being allocated space dependant on 'n' instead of 'k'. This rectification prevents the crash before the input is completed but it still occurs later.

double free or corruption (out)
Aborted (core dumped)
Is shown at the end.

Note: since u,p are dependant on n and are always distinct, their range is given the same as n, with the aforementioned loss of information.


Update install.py

Since the return value is always false, it will be reinstalled every time it is restarted.
https://github.com/v8hid/infinite-zoom-automatic1111-webui/blob/1139764bf715d848dc5b8b4c8d95e2e734c7e4eb/install.py#L5-L6

merged into develop


[Bug]: Can't install - 3.x

What happened?
Running the install command, it dies:

How to reproduce the bug
Run installer
Package Version
3.x
PHP Version
8.1
Laravel Version
10
Which operating systems does with happen with?
No response
Notes
No response

This works:
composer require jibaymcs/filament-tour:^3.0


CRLF on Windows

Describe the bug
With use_internal_diff = false on Windows on every diff I see a warning message about CRLF.
To Reproduce
Just run with the default settings on Windows and you will get the following message on every change:
error: warning: LF will be replaced by CRLF in AppData\Local\Temp\s1x8.0. NA run_diff

Additional context
System: Windows
Neovim version: gf5daf55c1 (latest master at the time of writing).

Is this caused by the new changes? Is there an older version of the plugin that works?
Note I don't have access to a Windows machine so I won't be able to test any changes on that platform.

No, this problem has existed before, I just did not report it :(

Note I don't have access to a Windows machine so I won't be able to test any changes on that platform.

I can test any PR on my Windows machine.

Relates to #69.
Does it work if you run: git config --global core.autocrlf input?

Oh, sorry, missed that issue. Thank you, fixes the problem for me.

Your welcome 🙂
I think I know how to prevent this warning alltogether so will try and fix it now.

Thank you!


Add initial support for Markdown

Linked issue: Closes Issue #9
As the program is already able to receive markdown files and create HTMLs from them, I have created a parseMarkdown() function that receives a string and adds the markdown features.
Because the program is supposed to receive text files as well, I thought the best way to do this with minimum changes in the code, is to use regular expressions to parse the markdown features and add it to the header and body in the main.js.
This function parses:

headers
blockquotes
bold texts
italics
images
links
line breaks

This function will not change the formatting of HTML files created from the text files.

@Reza9472 it looks like you have some more follow-up to do here, based on @AmasiaNalbandian's review.  Can you get that in soon please?  Also, Closes Issue #9 should be Closes #9, see https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue.


update valid connection checker example

The provided Iron Jacamar in Wildfly has changed package name since a while.
See forum reference: https://developer.jboss.org/message/639257#639257

@matlach As this pull request has  been held up for a long time, I am closing it to get the pull request queue under control again. Since this is very old now, it will be a very long time until this one will be addressed. Also, I don't have much time to spend on this to evaluate on a case by case basis.
If this is something important to you or the project please feel free to recreate the pull request. That way it will come on top of the pile again, and be looked at again sooner.
If it is a new feature, please follow the new process for features. Ping @kabir or @jamez if you are unsure what this means, and we can help. Please do that before opening the pull request though.
Sorry for any inconvenience!


issues/135:: parse SMSC response error

Thank you for contributing to naz.
Every contribution to naz is important.
Contributor offers to license certain software (a “Contribution” or multiple
“Contributions”) to naz, and naz agrees to accept said Contributions,
under the terms of the MIT License.
Contributor understands and agrees that naz shall have the irrevocable and perpetual right to make
and distribute copies of any Contribution, as well as to create and distribute collective works and
derivative works of any Contribution, under the MIT License.
Now,
What(What have you changed?)

Why(Why did you change it?)

Fixes: https://github.com/komuw/naz/issues/135

References:

Codecov Report

Merging #147 into master will decrease coverage by 0.12%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #147      +/-   ##
==========================================
- Coverage   86.36%   86.23%   -0.13%
==========================================
  Files          14       14
  Lines        1276     1279       +3
==========================================
+ Hits         1102     1103       +1
- Misses        174      176       +2

Impacted Files
Coverage Δ

naz/client.py
79.96% <66.66%> (-0.22%)
:arrow_down:

Continue to review full report at Codecov.

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


add(library): STDF

Add STDF - Mobile web component library based on Svelte and Tailwind.

Name
Link

STDF
https://stdf.design

Resource description here

[x] My item is in the right category
[x] My item is logically grouped below similar items
[x] My item's name and description respects the conventions of the list
[x] My item is awesome
[x] My item is in line with the Tailwind brand usage guidelines
[x] I have read and followed the contribution guidelines

Hey, I feel like this isn't really about Tailwind, the library just happens to use it. What do you think @joshmanders?

Fine by me 👌


test models without ground-truth annotations

Hi,
I'm wondering how to test the model without ground truth.
I don't have the annotations for the image set, the format only cannot work if I don't have the annotations.

Hi, you refer to here for help.

Hi, you refer to here for help.

Actually, I've tried it. But it still requires the annotations file.

Does GT exist on the network or evaluation server like coco (test-dev) ? Or you just want to inference the images.

Does GT exist on the network or evaluation server like coco (test-dev) ? Or you just want to inference the images.

Well, I just want to inference the images.
Thank you.

No worries. I  didn't inference a directory of images before. Maybe you can modify the demo/image_demo.py.

Ok, thank you. I'll try it.

#5548 may help you.


Add missing AUR flavours: ov and ov-git

This pull request add two other AUR flavours, ov and ov-git. If merged, it will close #470.

thank you.
Please feel free to submit a pull request anytime.

Thank you. Happy new year! :sparkles:


Revert "integration formulae"

Reverts sairish2001/MakesMathEasy#297

@sairish2001 Please check it. I have made a new PR for it #318

Please add the labels for it


Function tool calling on OllamaModel returns ModelTextResponse instead of ModelStructuredResponse

I'm running this example
https://github.com/pydantic/pydantic-ai/blob/main/pydantic_ai_examples/bank_support.py
when using ollama model like 'ollama:qwen2.5:0.5b' here
https://github.com/pydantic/pydantic-ai/blob/84c1190880219595903df2cea96e5e7146bd715b/pydantic_ai_examples/bank_support.py#L48
The response from agent is like below
ModelStructuredResponse(
    calls=[
        ToolCall(
            tool_name="customer_balance",
            args=ArgsJson(args_json='{"include_pending":false}'),
            tool_call_id="call_vz43blys",
        )
    ],
    timestamp=datetime.datetime(2024, 12, 13, 10, 34, 13, tzinfo=datetime.timezone.utc),
    role="model-structured-response",
)
ModelTextResponse(
    content="Your current account balance is $123.45. Thank you for using our bank. Feel free to call us if you have any questions anytime.",
    timestamp=datetime.datetime(2024, 12, 13, 10, 34, 13, tzinfo=datetime.timezone.utc),
    role="model-text-response",
)
ModelTextResponse(
    content="Sure thing! I've fixed it for you. Next time please ask a specific query so we can provide more personalized assistance.",
    timestamp=datetime.datetime(2024, 12, 13, 10, 34, 14, tzinfo=datetime.timezone.utc),
    role="model-text-response",
)

which is a text response instead of structured response
the response from a model like gemini is like
ModelStructuredResponse(
    calls=[
        ToolCall(
            tool_name="customer_balance",
            args=ArgsDict(args_dict={"include_pending": False}),
            tool_call_id=None,
        )
    ],
    timestamp=datetime.datetime(
        2024, 12, 13, 10, 33, 19, 184502, tzinfo=datetime.timezone.utc
    ),
    role="model-structured-response",
)
ModelStructuredResponse(
    calls=[
        ToolCall(
            tool_name="final_result",
            args=ArgsDict(
                args_dict={
                    "risk": 1,
                    "block_card": False,
                    "support_advice": "Your current balance is 123.45. \\n Have a great day!",
                }
            ),
            tool_call_id=None,
        )
    ],
    timestamp=datetime.datetime(
        2024, 12, 13, 10, 33, 21, 322077, tzinfo=datetime.timezone.utc
    ),
    role="model-structured-response",
)
ModelStructuredResponse(
    calls=[
        ToolCall(
            tool_name="customer_balance",
            args=ArgsDict(args_dict={"include_pending": False}),
            tool_call_id=None,
        ),
        ToolCall(
            tool_name="final_result",
            args=ArgsDict(
                args_dict={
                    "block_card": True,
                    "risk": 2,
                    "support_advice": "We have blocked your card. Please contact us to request a new one.",
                }
            ),
            tool_call_id=None,
        ),
    ],
    timestamp=datetime.datetime(
        2024, 12, 13, 10, 33, 22, 498653, tzinfo=datetime.timezone.utc
    ),
    role="model-structured-response",
)

and they are all structured responses

Looks like the same underlying issue as #200 - it's a limitation of the model, not much we can do AFAIK.

Would be interesting to see if requests to Ollama are adjusted to use the format parameter for structured output as described in https://github.com/ollama/ollama/releases/tag/v0.5.0

Yup, see #238 for a discussion of that.
You can downvote my response if you like, but it is the same underlying issue at #200, more generally - we're maintaining a library that supports multiple models, I can't investigate every "this models responds wrongly to this input" issue - if people come with suggestions like using structured output instead of tools, we'll happily try to fix it.

 Thank you.


update container names

use:
ds.formio.app.1
ds.formio.mongo.1
where the number at the end will increment as more containers are added for that service.

maybe instead of app use web?  @baddlan @mario-digitalstate  thoughts?

Done.


pkgsStatic.grpc fails to build

Describe the bug
Scanning dependencies of target check_epollexclusive
make[2]: *** No rule to make target '/nix/store/1m5vxadca7glcsmxi9haraj8iwjf5fsr-c-ares-1.15.0-x86_64-unknown-linux-musl/lib/libcares.so', needed by 'check_epollexclusive'.  Stop.
make[2]: *** Waiting for unfinished jobs....
[ 66%] Building C object CMakeFiles/check_epollexclusive.dir/test/build/check_epollexclusive.c.o
make[1]: *** [CMakeFiles/Makefile2:621: CMakeFiles/check_epollexclusive.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

nixpkgs at fb0da935095f6e7a194d1ec05c7355e8d47fed0c
Note I also had applied the diff from https://github.com/NixOS/nixpkgs/issues/76873#issuecomment-570552787

can't determine when this was fixed, but it builds on master now
[14:47:45] jon@nixos ~/projects/nixpkgs (master)
$ nix-build -A pkgsStatic.grpc
/nix/store/vv291ajzzv6smgk8bzcvc07m2p8kj7r0-grpc-static-x86_64-unknown-linux-musl-1.48.1


Copy over image styling from ‘old’ numeric template

 Question: Images for Sustainable Packaging Newsletter and HCP Quick Hits 2024


Updated OpCert Generation in the Operational Credentials Issuer.

Problem
In the current implementation when ICAC and RCAC are
found in the storage the Subject DN for these certificates is generated
separately. This might be an issue if after software update the way
Subject DN is generated is changed.
Change overview
The proper implementation is to extract Subject DN from ICAC and RCAC
if they are found in the device memory.
Testing
existing tests

Fast tracking given this has had > 3 days for review.

/rebase


Unable to set more than 1 cookie when using redirect()

What version of Elysia.JS is running?
^1.0.18
What platform is your computer?
Darwin 23.4.0 arm64 arm
What steps can reproduce the bug?
If you set more than 1 cookie, it will not be set.
import { Elysia, redirect } from "elysia";

new Elysia()
  .get("/", () => "yay")
  .get("/redirect", ({ cookie: { name, name2 } }) => {
    name.value = 'value'
    name2.value = 'value2'
    return redirect("https://youtu.be/whpVWVWBW4U?&t=8");
  })
  .listen(8080);

What is the expected behavior?
>1 cookies must be successfully set when redirect() is used
What do you see instead?
This is what the server headers look like when 1 or 2 cookies are set

Additional information
UPD:  It turns out that if you use set.redirect, there are no problems.
import { Elysia } from "elysia";

new Elysia()
  .get("/", () => "yay")
  .get("/redirect", ({ cookie: { name, name2 }, set }) => {
    name.value = 'value'
    name2.value = 'value2'

    set.redirect = 'https://youtu.be/whpVWVWBW4U?&t=8'
  })
  .listen(8080);

@SaltyAom, the main cause is that this function returns a Headers object:
https://github.com/elysiajs/elysia/blob/968650becd19850e4f33dca273d9e5838c937d70/src/handler.ts#L68
https://github.com/elysiajs/elysia/blob/968650becd19850e4f33dca273d9e5838c937d70/src/handler.ts#L152-L153
which cannot be used in such a spreading operation:
https://github.com/elysiajs/elysia/blob/968650becd19850e4f33dca273d9e5838c937d70/src/handler.ts#L201
inherits is allways {}

Fixed on https://github.com/elysiajs/elysia/commit/a4d74d5968f461d528a0d682a773cca2f7df0758, published on 1.0.19


How can I get data from query string url with the event handler

I Publishing an event is as simple as making a http post request
curl -X POST
'http://localhost:8080/luxriot/receive?addition_information=&date=9/26/2019&id=142&source_id=129&source_title=CPN_F1Dior
So event received data is empty.
The help is really appreciated.
Thanks.

Hi, go-micro api has not supported the params of GET requests.  But I fix it in PR#496 .


coconsecrator, entomofauna

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[c,e]

/trunk merge


[HOLD] open core panels by default

What changes are proposed in this pull request?
(Please fill in changes proposed in this fix)
How is this patch tested? If it is not, please explain why.
(Details)
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Closing for now. Will re-open if needed


Integration with ROS?

Hello, has anyone integrated this with ROS? I am wondering if anyone has modified the code to listen to a node that publishes images from a camera and the output of this code is a node that is a point cloud.

I'm not aware of any project that has integrated OpenSfM with ROS. (Would be a cool project though, perhaps you can be the first to create it!)


"this" not correctly assigned with spread operator

Bug Report

🔎 Search Terms
spread operator, this
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about this

⏯ Playground Link

Playground link with relevant code
💻 Code

const object = {
  ...{
    get () {
      return this
    },
    context: 'inner'
  } as const,
  context: 'outer'
} as const

const context = object.get().context
//    ^?

console.log(context)

🙁 Actual behavior
The type of context is 'inner'
🙂 Expected behavior
The type of context is 'outer'

It works if you make get generic in this:
const object = {
  ...{
    get<T>(this: T) {
      return this
    },
    context: 'inner'
  } as const,
  context: 'outer'
} as const

const context = object.get().context
//    ^?

console.log(context)

I'm not sure if this is intended; I'm having trouble searching the issues for it as searching "generic this" seems to bring up many unrelated issues.

Probably intentional; typing this correctly requires a polymorphic this which TS doesn’t do by default for performance reasons.

Correct


Added option to disable logs

This PR adds support for disabling logs from the CLI as well as from ASGI app tests.
Closes #19
@patrick91 I know this might conflict with some of the work you're doing on the new ASGI app structure, but I ended up implementing it as a part of #85 to silence logs in ASGI app unit tests, so I figured I'd throw it up as a PR. Feel free to close if this isn't helpful.

@jaydenwindle I've integrated this into #87, so I'll close this, thanks!


Dimensional change

When I change dimensions via any means it appears to disable the transporter until I return to the overworked(Tested with only non-vanilla dimensions)

Confirmed on my instance


Adding Person breaks unless you also comment out or delete sections of the schoolcontext.cs

DB tables do not get populated unless you comment out or delete the student and instructor lines:
    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.Entity<Course>().ToTable("Course");
        modelBuilder.Entity<Enrollment>().ToTable("Enrollment");
        //modelBuilder.Entity<Student>().ToTable("Student");
        modelBuilder.Entity<Department>().ToTable("Department");
        //modelBuilder.Entity<Instructor>().ToTable("Instructor");
        modelBuilder.Entity<OfficeAssignment>().ToTable("OfficeAssignment");
        modelBuilder.Entity<CourseAssignment>().ToTable("CourseAssignment");
        modelBuilder.Entity<Person>().ToTable("Person");

        modelBuilder.Entity<CourseAssignment>()
            .HasKey(c => new { c.CourseID, c.InstructorID });
    }

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

ID: 55345ff6-3c81-969e-3c92-29c24d78d2ca
Version Independent ID: aa826512-74a5-752a-0ad0-0f8133f6c39e
Content: Tutorial: Implement inheritance - ASP.NET MVC with EF Core
Content Source: aspnetcore/data/ef-mvc/inheritance.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Container issue for EF/MVC #17465


Sbaud/iac diff scan

Add iac diff scan to pyGitGuardian.
We use the same implementation logic than iac directory scan.
Careful, just like iac directory scan, iac diff scan needs to access the data argument of request. Therefore iac_diff_scan bypass the GGClient.post method and directly call GGClient.request.

Codecov Report

Merging #66 (10773ba) into master (03cda35) will decrease coverage by 1.66%.
The diff coverage is 60.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
- Coverage   94.62%   92.96%   -1.66%
==========================================
  Files           5        5
  Lines         502      526      +24
==========================================
+ Hits          475      489      +14
- Misses         27       37      +10

Flag
Coverage Δ

unittests
92.96% <60.00%> (-1.66%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pygitguardian/client.py
82.01% <16.66%> (-4.51%)
:arrow_down:

pygitguardian/iac_models.py
100.00% <100.00%> (ø)


Update monitor-instances-health-check.md

Updating to reflect the expected behavior of metrics. Once an instance for an app is deemed "unhealthy" only then the instance will be excluded from organic traffic and be reported to metrics. Failed pings prior will not reflect in Metrics, only once the max pings are met.

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

#label:"aq-pr-triaged"