Do not auto-repair if the ci is unhealthy for very long

This avoids further db queries
Also corresponding test code change

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

Thanks for signing the CLA @jayeshpise !


Implement some sort of State handler with some sort of RO-API

 Issue by psych0d0g
Thursday Jan 14, 2016 at 14:41 GMT
Originally opened as https://github.com/plietar/librespot/issues/37

Is there a way to notice when a device connects / disconnects?
and could we supply some kind of state file for tracking?
what i have in mind is another argument to enable that feature and if enabled create a file where librespot writes the current state (like the proc file system on linux just a single line which gets updated on state changes)
example content of that file:
Device connects:
"connected"
Device disconnects:
"noendpoint"
Device disconnects and no music playing or end of playlist:
"noendpointnotplaying"

Closing this, as it comes under the broader issue of Metadata, which is being tracked in #7


Inconsistent Dependency Installation

Summary: Dependencies were not installed consistently across systems due to issues with requirements.txt not capturing all necessary packages.
Details: Some dependencies required for development (e.g., for testing or linting) were not specified in the requirements.txt, causing inconsistent environment setups between team members.
Affected Areas: requirements.txt.
Dependencies: All dependencies tracked in the requirements.txt.

Updated the requirements.txt to include missing dependencies (e.g., pylint, black, pytest) and re-tested installation across multiple environments.


Scorer should sum up scores into a double

Description
Addresses #12675 . Along with MultiSimilarity.MultiSimScorer found some others candidate scorer implementations for this fix.

Thanks @jpountz for the review! I have addressed the comments in the new revision.

Thanks for the approval @jpountz !


Question: Can we get all the intermidiate tensors when using gpu(opencl) delegate??

Hi guys, i wanna ask if there is any method that we can get the intermidiate tensors on gpu(opencl) delegate.   Which means, when delegate invoke is finished , all the intermidiate tensors' cl_mem are NAIVE stored and we can WRITE into a cpu ptr which is hold by TfliteTensor form the marched tensor id on CPU.
I see there is TensorTieFactory for subgraph global input & output's conversion. However I'm not really clear some details.
Could you help me ? Thanks a lot.!

@impjdi could you take a look at this?

Not easily achieved at the code level, as we implement https://arxiv.org/abs/2001.03288 buffers used by intermediate tensors are reused and overwritten.
However, one quick hack you can do is the following:

Assume the output of a CONV_2D is to be inspected.  Let that tensor be A.
Let the rest of the network intact, but attach a ADD 0 or MUL 1 to A, which is essentially a no-op.  Let its output be B.
Declare B to be the graph output when you use tflite_converter.

In case TFLite tries to do something smart and prunes out ADD 0 or MUL 1, change it to ADD 0.0000001 or MUL 0.99999999.

Not easily achieved at the code level, as we implement https://arxiv.org/abs/2001.03288 buffers used by intermediate tensors are reused and overwritten.
However, one quick hack you can do is the following:

Assume the output of a CONV_2D is to be inspected.  Let that tensor be A.
Let the rest of the network intact, but attach a ADD 0 or MUL 1 to A, which is essentially a no-op.  Let its output be B.
Declare B to be the graph output when you use tflite_converter.

In case TFLite tries to do something smart and prunes out ADD 0 or MUL 1, change it to ADD 0.0000001 or MUL 0.99999999.

I really appreciate your reply, i still got one more question. If we cannot get intermidiate buffers from internal op of graph. How can we debug the result of ops in graph if graph's result is wrong？(assume op single test is passed, but op may not be fully tested or there comes some platform-related bugs.)
BTW,  i figure we can use MemoryStrategy::NAIVE when create gpu buffers so that buffers won't be reused. Am I wrong?

@impjdi Please take a look.

How can we debug the result of ops in graph if graph's result is wrong？

While we can't rule out the possibility of a bug in the shader, it's often FP16 not being good enough (precision-wise), to do the math computation.  See whether FP32 works fine; if it does, you know it's a precision issue.
In our team, we bisect the graph and inspect the output.  Unfortunately, it's not easily achieved with an automatic tool, but you need to visually inspect the graph and find the right cut off point by yourself.

i figure we can use MemoryStrategy::NAIVE when create gpu buffers so that buffers won't be reused.

Oh, I didn't know naive was one of the possible options.  If it's the same "naive" as in the paper (and I assume it is as it's the same dude who wrote the paper and implemented it), it may do the work.  I have never used it myself though.

@impjdi  thank you so much !
one last question about gpu layout:
seems like gpu(opencl) uses (C/4)HWB(C4) layout converted from BHWC(CPU layout)，why choose this layout？ maybe BHW(C4) or BH(C/4)W(C4) will be better?  These are my reasons:

data from different batches are independent;
channel data usually are accumulated together in conv ops which costs most of inference time in most CNN.

So Batch-size being outermost will reduce Scattered-access of threads in the same warp/wave as much as possible and Channel-size being innermost will reduce loop times when doing channel-accumulate. Plus, Channel-size being innermost will be more flexible in boundary procession.

@impjdi thank you so much !
one last question about gpu layout:
seems like gpu(opencl) uses (C/4)HWB(C4) layout converted from BHWC(CPU layout)，why choose this layout？ maybe BHW(C4) or BH(C/4)W(C4) will be better? These are my reasons:

data from different batches are independent;
channel data usually are accumulated together in conv ops which costs most of inference time in most CNN.

So Batch-size being outermost will reduce Scattered-access of threads in the same warp/wave as much as possible and Channel-size being innermost will reduce loop times when doing channel-accumulate. Plus, Channel-size being innermost will be more flexible in boundary process.

@impjdi please take a look.  thank you so much.

While I don't have any documentation to share, that's what the owner of OpenCL profiled and determined to be the fastest performing memory layout.  This usually goes with the parallelization of the workload and how GPU threads run.

@FdyCN Is this still an issue?
Please refer to the comment above.
Thanks!


GH-43400: [C++] Ensure using bundled GoogleTest when we use bundled GoogleTest

Rationale for this change
If we use bundled GoogleTest and system other dependencies such as Boost, our include path options may be:

-isystem /opt/homebrew/include (for Boost)
-isystem build_dir/_deps/googletest-src/googletest (for bundled GoogleTest)
-isystem build_dir/_deps/googletest-src/googlemock (for bundled GoogleTest)

With this order, GoogleTest headers in /opt/homebrew/include/ are used with bundled GoogleTest. It may cause link errors.
What changes are included in this PR?
This change introduces a new CMake target
arrow::GTest::gtest_headers that has include paths for bundled GoogleTest. And it's always used as the first link library of all test program. With this change, our include path options are:

-isystem build_dir/_deps/googletest-src/googletest (for bundled GoogleTest)
-isystem build_dir/_deps/googletest-src/googlemock (for bundled GoogleTest)
-isystem /opt/homebrew/include (for Boost)

With this order, we can always use our bundled GoogleTest.
arrow::GTest::gtest_headers is defined only when we use bundled GoogleTest. So this doesn't change the system GoogleTest case.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

GitHub Issue: #43400

@github-actions crossbow submit java-jars

This is an issue I have seen before with other dependencies when bundled deps are used (e.g. due to version mismatch) but they are also installed on the system, this solution works for gtest but I usually recommend to either upgrade or remove the system dependency...

this solution works for gtest but I usually recommend to either upgrade or remove the system dependency...

My recommendation is "use system dependencies as much as possible" or "remove the system dependency".
BTW, can we merge this?

Ah, are you suggesting the existing brew uninstall googletest approach?

@kou ah no, my question was why you need to create gtest_headers, could you not also just link against the normal gtest target first? Or would that mess with the link order for gtest deps?

I assume the special header target is necessary instead of just linking to gtest first?

Right.
If a test executable needs arrow_testing_shared, the test executable needs to link to arrow_testing_shared and GTest::gtest* but GoogleTest header files must exist before arrow_testing_shared and other system dependencies such as OpenSSL::*. If we have the special header target, we can use GTest::gtest_headers, OpenSSL::*, arrow_testing_shared and GTest::gtest order.

Ah, are you suggesting the existing brew uninstall googletest approach?

Sorry. I wrote this comment before I saw https://github.com/apache/arrow/pull/43465#pullrequestreview-2206426207 .


Render lilypond01a

This PR renders 01a-Pitches-Pitches.xml.
Divergences

The default time signature is 4/4 (vexml) instead of cut time (lilypond)
Measure labels are rendered for every measure and part by default (vexml).
Measure spacing has 100px buffer above the minimum amount of width needed (vexml).
Systems have a beginning barline (vexml).
The time signature is rendered for every system (vexml).

vexml

lilypond

I just noticed an issue where the measure numbers jump from 24 to 31 in the vexml rendering. I'm going to fix that before merging.

The issue was that 01a-Pitches-Pitches.xml had the wrong measure numbering and vexml was correctly reflecting that. I fixed this by updating 01a-Pitches-Pitches.xml with the correct numbering.


Stuck on action

Getting stuck here:
Run shrink/actions-document-publish@v1
6
/usr/bin/docker run --name c8ce414a4538b7784cc9b3f6051f7e48915b_d78eb0 --label 48c8ce --workdir /github/workspace --rm -e GITHUB_TOKEN -e INPUT_SOURCES -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_RETENTION_DAYS -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_SERVER_URL -e GITHUB_API_URL -e GITHUB_GRAPHQL_URL -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e GITHUB_ACTION_REPOSITORY -e GITHUB_ACTION_REF -e GITHUB_PATH -e GITHUB_ENV -e RUNNER_OS -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_c...
8
Started image server with image folder route '/github/workspace/.publish-docs/'.
9

10
Markdown files found: document.md
11
(node:14) UnhandledPromiseRejectionWarning: Error: ENOENT: no such file or directory, open '/github/workspace/.publish-docsdocument.md'
12
    at Object.openSync (fs.js:462:3)
13
    at Object.readFileSync (fs.js:364:35)
14
    at Function.getFileContent (/markdown-to-pdf.js:22:12)
15
    at GetFileBody (/github_interface.js:77:16)
16
    at /github_interface.js:139:33
17
    at FSReqCallback.oncomplete (fs.js:156:23)
18
(node:14) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
19
(node:14) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hi @AlexanderMohan, apologies for the inconvenience. Do you have a link to the repository that this is in, or can you provide a copy of the GitHub Workflow definition that you're using? The error is indicating that there's an issue with the path missing a directory separator, so it could be a platform issue (are you running on ubuntu-latest?) -- but it could also be that there's an environment variable having an impact. A copy of the Workflow (or link to the repository) will help me track down the issue for you.

Hi, have a look here: https://github.com/EdinburghUniversityTheatreCompany/Constitution
The action is in .github/workflows/markdown-pdf.yml

@AlexanderMohan Perfect, thank you for the link. At first glance I don't see an obvious issue, I'll investigate further now and get back to you today with an update.
I built this action over a year ago before I had perfected the art of creating GitHub Actions so it's very possible there's an obvious error that I missed due to the absence of tests (unlike the actions I've published more recently which are tested, I should probably fix up actions-document-publish at some point).

Great, thanks for such a fast response - this will eventually be perfect for us to use so I'd love if there is a solution for this, thanks @shrink

It seems like it would be an issue with input_sources but im not sure

@AlexanderMohan I've identified the issue. The upstream Markdown to PDF library has made some breaking changes, but due to the way I was depending on it -- not pinned to a version -- the breaking change broke this action. Entirely my fault!
I've just released a fix so if you re-run your Actions now they should work as expected. Sorry for the inconvenience! I'll add a rewrite of this action to my task list, not sure when I'll get to it though (but feel free to reach out again if you do encounter any new issues).


Add README for TrueNAS Scale Configuration for nfsn-ddns

I really appreciate your nfsn-ddns app and the effort to make it available in Docker with environment variables. Reading the source code and configuring it for my setup was a joy 😃
I have created a guide for setting up and testing the nfsn-ddns application on TrueNAS Scale. Please consider adding it as readme_truenasscale.md 🙏
Thank you, let me know if any modifications are needed.
Best regards,
Dustin
Proposed Guide:
# TRUENAS SERVER CONFIG

## Setup

**Login to NearlyFreeSpeech.net and get the API key**
- Member Interface > "Profile" tab > "Actions" > "Manage API Key"

**TrueNAS Scale > Datasets > Add Dataset**
- NFSN_DDNS_CONFIG

**TrueNAS Scale > Apps > Discover Apps > Custom App**
- Application Name: nfsn-ddns
- Image repo: ghcr.io/jdknight/nfsn-ddns
- Environment Variables:
  - NFSN_DDNS_API_LOGIN: example_username
  - NFSN_DDNS_API_TOKEN: example_token
  - NFSN_DDNS_DOMAINS: ddns.example.com
  - NFSN_DDNS_CACHE: 1
  - NFSN_DDNS_CACHE_FILE: /var/nfsn-ddns/cached-ip
- Storage: Add Host/Mount
  - Host Path: /mnt/Pool/NFSN_DDNS_CONFIG
  - Mount Path: /var/nfsn-ddns

## Testing

**Check the App Logs**
- Ensure that there are no errors and IP was sent to nfsn.

**Check the App Shell**
- `cat /var/nfsn-ddns/cached-ip`
- You should see your public IP cached in the file

**Log in to NearlyFreeSpeech.net and pull up the domain's DNS information**
- Ensure that ddns.example.com was added/updated

**If you have any ports open in your firewall (port forwarding), you can test they work with the new URL**

Thanks for the suggestion. I have written this up into a proposed document based on your instructions:
https://github.com/jdknight/nfsn-ddns/blob/prospect-truenasscale-readme/README_TrueNAS_SCALE.md
If you would like to review and propose any changes on this, comments are welcome.

Looks great!


Does not work on Linux

Does not work on linux. I have tried this with both package manager version of helix and compilesd from source version of helix. I have tried both master branch and snippet_placeholder branch for helix being compiled from source.

This is my helix branch erasin/helix/local-dev and config files erasin/helix-config that I has use these with archlinux.
The hx-lsp work with helix snippet_placeholder #9801.

Nevermind. It works, however themes do not work with the snippet_placeholder branch. Do not know why the functionality for tabstops, etc is working now whereas it was not working before.


Compiler warning in lib/timex.ex

Steps to reproduce

Use Elixir 1.4.4 with Erlang 19
Compile an Elixir project with Timex version 3.1.15

Description of issue
Compiler shows warning in `lib/timex.ex' on line 979:
==> timex
Compiling 57 files (.ex)
warning: this clause cannot match because a previous clause at line 979 always matches
  lib/timex.ex:979

P.S.: Thanks for this awesome library! <3

Fixed in master. Thanks!


UR5e + MoveIt trajectory execution timeout

I am opening this issue since I found no better way to ask for advice regarding code and the thoughts behind it:
I am running a UR5e robot with a customized robot model file and MoveIt. Due to safety reasons, the speed of the robot is set to half. This seems to cause wrong predictions for the execution duration of the robot, and the move execution gets cancelled about midway with a timeout error.
Now, I am trying to track down whether the issue is driver/Kinematics Plugin/MoveIt/config related.
The code line responsible for triggering the timeout is https://github.com/moveit/moveit/blob/9caa14eabc69c6ba2328eacfb5f6253a8acff085/moveit_ros/planning/trajectory_execution_manager/src/trajectory_execution_manager.cpp#L1270 and the expected_trajectory_execution is mainly influenced by  https://github.com/moveit/moveit/blob/9caa14eabc69c6ba2328eacfb5f6253a8acff085/moveit_ros/planning/trajectory_execution_manager/src/trajectory_execution_manager.cpp#L1203-L1208
What I do not understand is how the time_from_start attribute is used to deliver information about the expected execution duration. To my understaning, the attribute is set on every call to https://github.com/moveit/moveit/blob/9caa14eabc69c6ba2328eacfb5f6253a8acff085/moveit_ros/planning/trajectory_execution_manager/src/trajectory_execution_manager.cpp#L266-L267
At this point I got lost in the source code. When is this system invoked by which entities?

Due to safety reasons, the speed of the robot is set to half.

Do you set the scaling factor in the UR5 controller or in MoveIt?
In the former case, MoveIt cannot know about the slow down and will complain because execution lags behind (that's what you observed). You should change the max_velocity_scaling_factor:
https://answers.ros.org/question/251282/how-to-change-velocity-acceleration-in-motionplanrequest-topic/

Thx, that will most likely fix the timeout. Will test it asap.
Still curious after the code inspection on what parameters the move time estimation is calculated. Would appreciate some pointers.

Still curious after the code inspection at which point the move time estimation is calculated.

MoveIt simply compares the expected trajectory timing (as given in trajectory's member time_from_start) with the actual timing.

That part is clear, the unclear part is what instance provides the time_from_start value.

The time_from_start members of a trajectory are provided by the time parameterization plugin of your planning pipeline.

Thanks a lot for your help. The max_velocity_scaling_factor seems to work fine, at the cost of some hickups in the speed consistency. Maybe retrieving the UR controller scaling parameter and pass it somehow to the planning pipeline would be more elegant, but I suppose it will cost me way more time to implement than its worth. 👍


Translate 'library' to bahasa indonesia

translate the word 'library' to Indonesian language

Closing this because we're restarting work on the new website: #330


Cluster data class

❓ Questions & Help
Does the subgraphs returned by ClusterData has got between between cluster links or do we get 'x' number of disconnected subgraphs?

Indexing ClusterData only returns intra-cluster links. In case you are also interested in inter-cluster links between different partitions, you want to look at ClusterLoader.

Indexing ClusterData only returns intra-cluster links. In case you are also interested in inter-cluster links between different partitions, you want to look at ClusterLoader.

I want to make  each cluster as a minibatch without considering the inter-cluster links. So if I use DataLoader on the output returned by the ClusterData will that work?

I want to make  each cluster as a minibatch without considering the inter-cluster links. So if I use DataLoader on the output returned by the ClusterData will that work?

Yes, that will work :)

Yes, that will work :)

Thank you

Thank you

Will ClusterData return clusters comprising of only training nodes similar to   the inductive setting mentioned in the paper?

Will ClusterData return clusters comprising of only training nodes similar to   the inductive setting mentioned in the paper?

No, it will contain clusters of all the nodes in the graph.

No, it will contain clusters of all the nodes in the graph.

So in that case  in the ClusterGCN example reddit.py during the training message passing and update phase it sees all the  train,test and val nodes. So is that fully inductive?. I mean in the ClusterGCN github repo I've seen during them seperating out the train nodes and doing the training using the features of train nodes only. So just confused

So in that case  in the ClusterGCN example reddit.py during the training message passing and update phase it sees all the  train,test and val nodes. So is that fully inductive?. I mean in the ClusterGCN github repo I've seen during them seperating out the train nodes and doing the training using the features of train nodes only. So just confused

If you want to train it in an inductive fashion, you indeed need to separate graphs first, e.g., via torch_geometric.utils.subgraph.

If you want to train it in an inductive fashion, you indeed need to separate graphs first, e.g., via torch_geometric.utils.subgraph.

Hello,
For the data returned by ClusterData I noticed  a val_tensor inside the adj field. What does that val_tensor mean?

Hello,
For the data returned by ClusterData I noticed  a val_tensor inside the adj field. What does that val_tensor mean?

Can you give a brief example to what you are referring? I'm actually not sure what you mean. There shouldn't be any additional attributes besides the one original passed to Data.

Can you give a brief example to what you are referring? I'm actually not sure what you mean. There shouldn't be any additional attributes besides the one original passed to Data.


[EPIC] - Notion integration

Goal: allow writers to author & collaborate with Notion as a wiki CMS
Implementation: use the Notion API to sync Handbook pages into a Notion workspace.
Details (work in progress)

Possibility - Docu-notion on GitHub

Bi-directional sync? Perhaps start with a one way sync from The Handbook pages to Notion if that's easier to implement. Once the page updates are complete on the Notion side, a Notion page can be manually exported to Markdown for Handbook integration in the other direction

I'm slightly confused about the permissions given to writers and authors. Ideally, all authors should be able to work within the same space, which implies that anyone could delete or update content. This situation, however, could pose a threat to the safety of the entire content. What happens if an author accidentally alters someone else's article? I'm uncertain whether Notion allows writers to control their own content.
Another problem I'm grappling with is a persistent error message: "throw new Error(`Notion page not found "${uuidToId(pageId)}") ^ Error: Notion page not found "454a4e53ed0a46f6bd431b07328e8941". I'm at a loss as to why this is happening.


ERROR: Can't create the container

I setup Omnigress using docker
docker volume rm omnigres
docker volume create omnigres
docker run --name omnigres
-e POSTGRES_PASSWORD=omnigres
-e POSTGRES_USER=omnigres
-e POSTGRES_DB=omnigres
--mount source=omnigres,target=/var/lib/postgresql/data
-p <IP_ADDRESS>:5432:5432 -p <IP_ADDRESS>:8080:8080 --rm ghcr.io/omnigres/omnigres-slim:latest
when I try omni_containers  i get following errors:
omnigres=# create extension omni_containers cascade;
NOTICE:  installing required extension "omni_httpc"
CREATE EXTENSION
omnigres=# select * from omni_containers.docker_container_create('busybox', cmd => 'echo Hello world', pull => true);
ERROR:  Can't create the container
DETAIL:  Error code :
CONTEXT:  PL/pgSQL function omni_containers.docker_container_create(text,text,text,boolean,boolean,boolean,jsonb) line 103 at RAISE

@ranjanprj, thank you for the report; I will look into it.

@yrashk I think the issue is pretty straight-forward? Docker UNIX socket is not exposed to the container, thus container creation is failing? the error message could be better though

The error message can definitely be better!
I think your instinct is correct here – and it should be solvable.

I reran the same commands and now getting this error

Thank you, can you run make -j all before doing this (or more targeted, make -j omni_httpc)?


[BUG] Namespace bundle for topic not served by this instance. Please redo the lookup.

Describe the bug
Producer and Consumer throw errors when trying to publish/consume messages from topics created with bin/pulsar-admin and autoCreatedTopics. This error seems to be intermittent, some times when publishing gets an error sometimes seems to be able to publish messages and the same for consumers. Broker has configured brokerDeleteInactiveTopicsEnabled=false
To Reproduce
(Our setup is a 3 broker env)
Steps to reproduce the behavior:

Start up your Pulsar Broker with KOP
Create a partitioned topic
use bin/kafka-console-producer.sh to send messages
use bin/kafka-console-consumer.sh to receive messages

Expected behavior
Be able to use it every time the topic created
Screenshots
2022-05-25T23:54:24,792+0000 [pulsar-io-5-3] WARN org.apache.pulsar.broker.service.BrokerService - Namespace bundle for topic (persistent://public/default/angeltest-partition-0) not served by this instance. Please redo the lookup. Request is denied: namespace=public/default 2022-05-25T23:54:24,792+0000 [pulsar-io-5-3] WARN io.streamnative.pulsar.handlers.kop.KafkaTopicManager - Get partition-0 error [Namespace bundle for topic (persistent://public/default/angeltest-partition-0) not served by this instance. Please redo the lookup. Request is denied: namespace=public/default]. 2022-05-25T23:54:24,792+0000 [pulsar-io-5-3] INFO org.apache.pulsar.broker.PulsarService - No ledger offloader configured, using NULL instance 2022-05-25T23:54:24,792+0000 [pulsar-io-5-3] INFO org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl - Opening managed ledger public/default/persistent/angeltest 2022-05-25T23:54:24,794+0000 [bookkeeper-ml-scheduler-OrderedScheduler-7-0] INFO org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl - [public/default/persistent/angeltest] Closing managed ledger 2022-05-25T23:54:24,794+0000 [bookkeeper-ml-scheduler-OrderedScheduler-7-0] ERROR io.streamnative.pulsar.handlers.kop.KafkaTopicManager - [[id: 0x2254288d, L:/<IP_ADDRESS>:19092 - R:/<IP_ADDRESS>:58628]]Get empty non-partitioned topic for name persistent://public/default/angeltest 2022-05-25T23:54:24,794+0000 [bookkeeper-ml-scheduler-OrderedScheduler-7-0] ERROR io.streamnative.pulsar.handlers.kop.KafkaTopicManager - [[id: 0x2254288d, L:/<IP_ADDRESS>:19092 - R:/<IP_ADDRESS>:58628]] Failed to getTopicConsumerManager caused by getTopic 'persistent://public/default/angeltest-partition-0' returns empty
Additional context
Pulsar version: 2.9.1
KOP VERSION <IP_ADDRESS>

Could you try the streamnative/sn-pulsar:<IP_ADDRESS>  image? It looks more like a Pulsar side problem. There should be some fixes since 2.9.1 but I didn't remember the specific PRs.
In addition, could you give more details ?

The number of partitions.
The producer rate.
The complete log files of all brokers during the time that getTopic returns empty.

And if you can reproduce this issue, please run the following command to see what will happen:
./bin/pulsar-admin topics partitioned-lookup angeltest

or specify the specific partition:
./bin/pulsar-admin topics lookup angeltest-partition-0

I will try with the new version as mentioned, meanwhile sharing the details:
1.- Just one partition: bin/pulsar-admin topics create-partitioned-topic angeltest -p 1
2.- we just try one message.
3.- Broker logs attached.
Output of commands:
[root@broker-0 core]# bin/pulsar-admin topics partitioned-lookup angeltest
"persistent://public/default/angeltest-partition-0    pulsar+ssl://broker-0.brokers.pulsar-poc-v2.svc.cluster.local:6651"
[root@broker-0 core]# bin/pulsar-admin topics lookup [angeltest-partition-0]
"pulsar+ssl://broker-0.brokers.pulsar-poc-v2.svc.cluster.local:6651"
Logs:
broker-2.log
broker-1.log
broker-0.log

Hi @BewareMyPower did this information help to analyze the error shared?

Sorry I'm busy with other tasks recently, I will take a look at these logs soon.

It still looks like a Pulsar side bug and triggered by the bundle split.
You can try to reproduce it with bundle split disabled and see if it will still happen.
loadBalancerAutoBundleSplitEnabled=false

You can also try a newer Pulsar like I've said to see if this bug has been fixed.

Thanks for the info,
I have made the changes on the broker.conf file and set property mentioned as false. But we still see the issue.
This seems to happen when the Kafka client connects to KOP LB port but the request is resolved by the broker that is not the leader of the topic in the test.
Is there some other property that needs to be set or changed on the broker.conf? Unfortunately we are not able to test it on a Pulsar newer version.

No. It looks like an issue about K8s deployment. Could you describe how do you run KoP in K8s? Maybe @gaoran10 can provide some help.

KOP is running on a 3 k8s node cluster:
on each node I have:

zookeeper pod
bookie pod
broker/proxy pod with KOP enabled

I also have one NodePort svc to balance the load to the proxy running on every broker pod.
The same NodePort svc has a port that will target the port specified on kafkaListernes property.
The values are next:
kafkaListeners=kafka_internal://<IP_ADDRESS>:9092,kafka_external://<IP_ADDRESS>:19092
kafkaProtocolMap=kafka_internal:PLAINTEXT,kafka_external:PLAINTEXT
kafkaAdvertisedListeners=kafka_external://:19092,kafka_internal://:9092
The node-name used on the kafkaAdvertisedListeners is a resolvable address of external clients.

@BewareMyPower @gaoran10  I have enable a new pulsar cluster using image mentioned above: streamnative/sn-pulsar:<IP_ADDRESS>.
Error keeps ocurring when producer/consumer connects to the loadbalancer but the broker is not the leader of the topic in question:

2022-06-07T00:05:03,562+0000 [pulsar-io-4-2] WARN  org.apache.pulsar.broker.service.BrokerService - Namespace bundle for topic (persistent://public/default/angeltest-partition-0) not served by this instance. Please redo the lookup. Request is denied: namespace=public/default
2022-06-07T00:05:03,562+0000 [pulsar-io-4-2] WARN  io.streamnative.pulsar.handlers.kop.KafkaTopicManager - Get partition-0 error [Namespace bundle for topic (persistent://public/default/angeltest-partition-0) not served by this instance. Please redo the lookup. Request is denied: namespace=public/default].
2022-06-07T00:05:03,562+0000 [pulsar-io-4-2] WARN  org.apache.pulsar.broker.service.BrokerService - Namespace bundle for topic (persistent://public/default/angeltest) not served by this instance. Please redo the lookup. Request is denied: namespace=public/default
2022-06-07T00:05:03,562+0000 [pulsar-io-4-2] WARN  io.streamnative.pulsar.handlers.kop.KafkaTopicManager - [[id: 0x349cafb2, L:/<IP_ADDRESS>:9092 - R:/<IP_ADDRESS>:48024]] Failed to getTopic persistent://public/default/angeltest: Namespace bundle for topic (persistent://public/default/angeltest) not served by this instance. Please redo the lookup. Request is denied: namespace=public/default
2022-06-07T00:05:03,650+0000 [pulsar-web-37-5] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [07/Jun/2022:00:05:03 +0000] "GET /admin/v2/persistent/public/default/angeltest/partitions HTTP/1.1" 200 16 "-" "Pulsar-Java-v<IP_ADDRESS>" 1
2022-06-07T00:05:03,838+0000 [pulsar-io-4-2] WARN  org.apache.pulsar.broker.service.BrokerService - Namespace bundle for topic (persistent://public/default/angeltest-partition-0) not served by this instance. Please redo the lookup. Request is denied: namespace=public/default

Issue has been fix, we used to have one service for all the external connections, now we have one service for each broker for external connections.

Issue has been fix, we used to have one service for all the external connections, now we have one service for each broker for external connections.

Assuming you installed this with helm, how did you manage that?


com.fasterxml.json.core.JsonParseException constantly and randomly thrown.

I have a trading bot using the REST API that when asking for market data, sometimes gets this exception:
com.fasterxml.jackson.core.JsonParseException: Unexpected character ('<' (code 60)): expected a valid value (number, String, array, object, 'true', 'false' or 'null')
 at [Source: okhttp3.ResponseBody$BomAwareReader@7699ccb9; line: 1, column: 2]

These are randomly thrown but constantly through out the day. Could be 5 times in 5 minutes, 10 times in 5 minutes, and then a few hours without any exceptions thrown, this is what is making it hard for me to figure this out. Any help appreciated.

Something is making me think this has something to do with issue #10 , I will confirm later.

For some reason this is related to API requests as I suspected, even though I am sure I am not surpassing the 1200 Requests (weighted) per minute, since I did that and the error was:
Exception caught!! Too many requests; current limit is 1200 requests per minute. Please use the websocket for live updates to avoid polling the API.
Instead of the one I am getting. However, turning the API requests per minute down to around 800 or 700, makes this error disapear.
Any help?

Seems this error is produced by CloudFront.
Binance Support refuses to tell me how to avoid this, while my app only polls about 800 times per minute (weighted) it still gets those cloudfront pages every few minutes or hourse. Just had to add a few try-catches to avoid this stupid error messing with my program, but it makes no sense to advertise 1200 weighted requests on the API while the firewall intercepts packets at waaaaaay less.
Lets hope this gets fixed at some point on Binance's side, and that the API is updated to manage it correctly.

Closing this ticket, as at the moment, there is nothing that can be done from the API side to fix it.


Tracking PSBT refactoring & PSBTv2 Epic

This is a tracking issue for ongoing PSBT work, including but not limited to implementing support for PSBT v2 BIP-370.
For reference we do support PSBT v0 BIP-174.
We can look to the achow101 PR https://github.com/bitcoin/bitcoin/pull/21283 for guidance since he's the BIP-370 author.
1⃣ Refactor
Refactor PSBT Consensus Encoding Traits
Now we implement consensus::Encodable and Decodable for PSBT types. We should not implement those traits since PSBT data moves at the application layer and not at the p2p consensus layer. We may consider removing serde support for PSBT or reimplement it using BIP-174 serialization spec base64 or binary. We should remove the Consensus Encoding traits as we roll out PSBTv2, or else we're building deeper into a regressive pattern.

[x] Serialize without Consensus Traits

[x] https://github.com/rust-bitcoin/rust-bitcoin/issues/934
[x] https://github.com/rust-bitcoin/rust-bitcoin/pull/988

Improve Psbt Error handling

[x] https://github.com/rust-bitcoin/rust-bitcoin/issues/837 circular dependencies, Parse(String), dependence on consensus Error types
[x] https://github.com/rust-bitcoin/rust-bitcoin/pull/1310
[x] https://github.com/rust-bitcoin/rust-bitcoin/pull/1532
[x] #1589
[x] https://github.com/rust-bitcoin/rust-bitcoin/issues/1590 https://github.com/rust-bitcoin/rust-bitcoin/pull/1609

2⃣ Add PSBTv2
PSBTv2 allows inputs and outputs to be added to (and removed from) the PSBT after creation. To do this the struct eliminates the global unsigned transaction. Input and output maps replace it. Each element of those maps has complete information about each input or output. Shipping PSBTv2 is the primary goal  of this epic
Discussion

[ ] https://github.com/rust-bitcoin/rust-bitcoin/pull/1924
[ ] https://github.com/rust-bitcoin/rust-bitcoin/pull/1218
[ ] Add new input map
[ ] Add new output map
[ ] Add test vectors from BIP-370

Prior attempts:

https://github.com/rust-bitcoin/rust-bitcoin/pull/726
https://github.com/rust-bitcoin/rust-bitcoin/pull/769

Document and Exemplify

[x] https://github.com/rust-bitcoin/rust-bitcoin/pull/935
[x] https://github.com/rust-bitcoin/rust-bitcoin/issues/893 https://github.com/rust-bitcoin/rust-bitcoin/pull/999 good first issue
[ ] Decide to include a finalizer or include rust-miniscript as a dev-dependency for examples https://github.com/rust-bitcoin/rust-bitcoin/issues/630#issuecomment-1189611845

working standalone psbt crate including v2
The BP-WG/descriptor-wallet version supports the internal v2 struct but only v0 de/serialization. That makes working with the psbt a lot easier in rust code with one conversion. It's a neat approach.
Improve capabilities
Enable Signing

[x] https://github.com/rust-bitcoin/rust-bitcoin/pull/957

Shall we close this @DanGould and focus on https://github.com/tcharding/rust-psbt?

yes


Bugged trees

I made my own server and when i try to chop down trees i cant, nobody in my server can, they seems like they have a force field, bigger than the tree, i've tried to reinstall everything, but nothing, somebody can help me?
The server is made in Ubuntu 14.04

Obviously not a rocket issue


Add labels to radar chart

Add labels to radar charts which display the score

You should ask John-Morten about this, but I believe the score he want to see the ones used internally by the algorithm. That would be the ones  produced by the "x_score()" functions used in optimizer.py:81 calculate_performance().

You should ask John-Morten about this, but I believe the score he want to see the ones used internally by the algorithm. That would be the ones  produced by the "x_score()" functions used in optimizer.py:81 calculate_performance().

Gjerne sjekk om det ble brukt riktig standard deviation

Gjerne sjekk om det ble brukt riktig standard deviation


docs: remove non-existent shared_context overload

Fixes #2374.

I've investigated and this documentation was added by @myronmarston in March 2014, in 5b342a31b8630d11f173afba1d9991a13d1c0eb4, technically the first argument was/is used for the name but this isn't clear from this documentation so I'm in favour of removing it.

Thanks :)


Update supported linux platforms

Updates

RHEL 9.x: amd64, arm64, ppc64le, s390x

https://hub.docker.com/r/redhat/ubi9/tags

SUSE Linux Enterprise Server 15: amd64, ppc64le, s390x

https://registry.suse.com/static/suse/sle15/index.html

Ubuntu 18.04: amd64, arm64/v8, arm/v7, ppc64le, s390x

https://hub.docker.com/_/ubuntu?tab=tags

CentOS 7: amd64, arm64/v8, arm/v7, ppc64le and docker image available

https://hub.docker.com/_/centos?tab=tags

Remove

Rocky Linux 8.x from ppc64le, because not supported

https://rockylinux.org/download

Checklist#### Checklist

[x] npm test passes
[x] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[x] contribution guidelines followed here

@gdams I removed the extra distributions.

sorry for a late comment:
rhel9 was recently added into "installer" jenkinsfile, but till we have our next CPU in about 2weeks, we do not actually support for it yet (https://adoptium.jfrog.io/ui/repos/tree/General/rpm/rhel/7)
as for rocky, we do have rpm released at https://adoptium.jfrog.io/artifactory/rpm/rocky/8/


[FEATURE] Add support for Mistral via the mistral Python SDK

📥 Pull Request
📘 Description
Adds support for the Mistral models tracking for mistral >= 1.0.1 Python SDK with examples
🎯 Goal
To extend the functionality of Agentops
🧪 Testing
Tested the functions with the provided tests file

@bboynton97 @areibman this is ready for review!

Cleaned and fixed up some items while creating the #382 PR.
This can also be reviewed.

@bboynton97 @areibman can you guys review this?


Remove outdated +build directive

The code is still using the old +build directives from the sources
https://go.googlesource.com/proposal/+/master/design/draft-gobuild.md
We could remove all +build directives, as we have to go:build ones
This is similar to what has been done in testify https://github.com/stretchr/testify/pull/1671
Originally posted by @ccoVeille in https://github.com/gofr-dev/gofr/pull/1158#discussion_r1821995280

Fixed by #1185


calendar  is not loaded  in view

I have opened the modal in the another page  after that i open the calender in mypage the calendar month view in not loaded in the html

@vikas-kalyan Sorry, I don't quite understand how to reproduce your issue.
Are you saying you have a page containing the calendar, but it's not displayed by default.
You go to another page to open a modal.
Then go back to the previous page to somehow display the calendar, but it doesn't appear?
Is there any error printed out in the console?

Yes your r right i also want to say that thing i mail you the screenshot
that appear it show one error but if i remive that it still nit shws the
calender

Yes your r right i also want to say that thing i mail you the screenshot
that appear it show one error but if i remove that it still not show the
calender

Calender is open properly untill i m not open the modal in the another page but once i open modal and come back again open calender page  then i can't see the calender u can see in the image that i attach

@vikas-kalyan I can't reproduce this issue.
I have my home page having the calendar displayed.
In another page, it contains a "open modal" button.
When I click the "open modal" button, the modal dialog pops up. After I dismiss it and switch back to the home page, the calendar is still there.

Please make  cache of states true

@vikas-kalyan Sorry, I have no idea about cache state? Is it related to this calendar component?


Access rights for /tmp/nuget

If one first run monodevelop from root and restore some packages,
then directory /tmp/nuget will be owned by root
If one then run modevelop from user, she/it will receive an error like:
Adding SharpHSQL...
Installing 'SharpHSQL 1.0.3'.
Could not find a part of the path "/tmp/nuget/akb0dm2d.wn4/lib/net40/SharpHsql.dll".
removing /tmp/nuget and start monodevelop form user helps, ofcourse

NuGet does not currently do anything special for paths on Linux or Mac. It will just use the directories returned by .NET which on Windows are OK but on Linux/Mac they may not be the best or traditional paths.
So for the temp directory NuGet will use Path.GetTempPath(). On Windows this path will be a directory inside the user's profile and not a global directory like the default Linux path.
I guess NuGet could do something similar to MonoDevelop, and not rely on the path returned from Mono, and use a different path such as:
~/.cache/nuget/temp

This directory would be owned by the current user instead of being global.

You can specify the Folder GetTempPath() returns by setting the environment variable TMP, as it will be checked before TEMP and others.
TMP=~/.cache/ nuget restore


Direct Initialization vs Brace Initialization

The code presented in the section Handling Exceptions uses direct initialization instead of brace initialization. See screenshot below.

vector<int> indices(3) will result in [0, 0, 0] making indices[1] valid.
vector<int> indices{3} will result in [3] making indices[1] invalid.
Thanks.

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

ID: 6232c6e0-d7c5-70d4-9b9d-623d25ced117
Version Independent ID: bfc01a2a-12a3-5d14-a37d-d42c95deb27c
Content: Examples of Lambda Expressions
Content Source: docs/cpp/examples-of-lambda-expressions.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @corob-msft
Microsoft Alias: corob

I think the real typo is that it should have been indices[-1]=1; to throw the exception. I've fixed the topic which will be live Monday afternoon.


⚠ Thumbnails API Endpoint has degraded performance

In accba29, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1944 ms

Resolved: Thumbnails API Endpoint performance has improved in ed49384.


refactor: refine links in readme.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

refactor: refine links in README due to an update to the KusionStack web page content.

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
23789

Relevant Lines:
39374

💛  - Coveralls


Yandex Metrica Support

This is almost certainly the definition of scope creep, but I wanted to share an idea from our Discord.
They ask if it's possible to adjust this tool to support Yandex Metrica.
Does this seem feasible? And, if it is, should it live as part of this tool or in a separate one?

I'm interested in taking.


Fixing language on comment

@pranavkm

Hi @ZestyBread, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Shouldn't you keep the TODO just change what it says

Should have been a cleaned up TODO. Other TODOs address it already.

:shipit:


Incorrect checksum hash in Github Release

Title: Incorrect checksum hash on GitHub releases
Description:
Checksum Hash listed in GitHub release asset is SHA-512 while the one being used is SHA-256.
Example: https://github.com/envoyproxy/envoy/releases/tag/v1.26.1
Repro steps:
Download the checksum file.

the files are hashed with sha256, and the message is hashed and signed with sha512 (the gpg default)
i think this is a reasonable scheme

i think this is an issue of documentation which i will follow up on in #27179
im gonna close this issue, but if you think there is something we can improve with our signing process @hbagdi please do reopen


Update registry.json - Shareslake bridge metadata key

Signed-off-by: Miguel Angel Cabrera Minagorri<EMAIL_ADDRESS>Update registry.json to add a metadata key to route Shareslake's bridge transactions.
We will introduce an object as the value of the key containing the destination/origin network and the destination address / original network Tx.

Looks like another reviewer is required, maybe @KtorZ ?

Thank you guys!


Sieve of eratosthenes algorithm in Python

🛠 Issue (#173)

Issue no #173
👨💻 Changes proposed

Sieve of eratosthenes algorithm in python added
Readme updated for the same

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

@Kumar-laxmi please review it

@Kumar-laxmi please review it

Gentle request to add a few comment lines explaining your approach to the problem

@Kumar-laxmi please review it

Gentle request to add a few comment lines explaining your approach to the problem

Okay I will add.

@zugzwang03 I have added comment please review it


Works with current release (2.4.0), but not earlier versions of ChartJS

Most up to date version of Chart.BarFunnel.js works with ChartJS v2.4.0 and 2.1.0. Make adjustments to code to support other versions.

One option here is to bump the version number to something like 0.2.0 and also bump the Chart.js version number.


Implemented named arguments

Hey Alexis,
Here's an implementation of named parameters. For example:
.mixin(@a: 1, @b: 2, @c: 3, @d: 4, @e: 5) {
  ...
}

.class {
  .mixin(1, 2, 3, 4, 10);
  .mixin(@e: 10);
}

If you wanted to override the 5th argument you would need to know the default value for the first 4 before supplying the 5th and if the defaults changed you now have to find everywhere where you've done this.
Not good.
Now you just specify the name of the argument you want to override.
It might be nice to throw an error if you place any named arguments before any "positional" arguments. as it stands it works regardless but you may get unexpected results.
Cheers
James

Any progress on this ticket?

Would love to see named arguments added.

Currently working on an updated version of this here; will be submitting another pull request after I've tested it a bit more thoroughly.


the Basic Usage portion in the documentation could be improved in the following way

Issue:
the current command is shown as:
instaloader profile [profile ...]
which could be a little  confusing because if I'd like to get user 'JohnDoe', I initially tried instaloader profile [JohnDoe], instaloader profile [JohnDoe_1, JohnDoe_2], etc...
Propose:
a better way is to separate the above command into 2, one is to scrape a single user, and the other is to scrape a list of users:
instaloader profile
and
instaloader profile_1 profile_2

Enclosing arguments with [ and ] is a well-known notation to denote them as optional. The syntax instaloader profile [profile ...] follows exactly the example from Python's ArgumentParser documentation:
https://docs.python.org/3/library/argparse.html#usage
Especially the reuse of the same argument name is perfectly in accordance.
We do not want to deviate from the current description, because it follows a well established convention.


Data get's cutting with getParsedBody()

Hello, i have a problem with the getParsedBody() function.
I send data to the slim framework (v. 3) with the unirest framework.
Client Code:
$deliveryJSON = $_POST['deliveryJSON'];
$delivery = json_decode($deliveryJSON);

$headers = array('Accept' => 'application/json', 'Authorization' => 'Bearer ' . $_SESSION['login_token']);
$dataForm = Unirest\Request\Body::Form($delivery);
$response = Unirest\Request::post(REST_URL . '/MMMScannerApp/Deliveries', $headers, $dataForm);

When i get the the parsed body with the slim framework, the array has only <= 125 objects.
So all object from the position 126  are not in the array.
Server Code:
$parsedBody = $request->getParsedBody()
It must be the getParsedBody() function, because in the $request->getBody() there are all data available.
For now i send the json file directly and i call json_decode and work with Std Class objects. But i want to use the parsedBody() function.

Uh... Can you paste some code that is slim related?
I have no idea what Unirest\Request is or what it does...

Unirest is a lightweight HTTP request library.
Here is some code (Old code, which i want):
$app->post('/Deliveries', function (Request $request, Response $response) {
        // send the whole body, because of the complexity from the object
        $parsedBody = $request->getParsedBody();
        // $articleList is an array. the max size is 125 objects.
        // all object over 125 are not in the array
        $articleList = $parsedBody['articleList'];
        foreach ($articleList as $article) {
               // do what ever
        }
}

New code (works, but not really nice)
$app->post('/Deliveries', function (Request $request, Response $response) {
// read the json string
$body = $request->getBody();
$parsedBody = json_decode($body);
$articleList = $parsedBody -> articleList;
// now $articleList has all Objects
foreach ($articleList as $article) {
// do what ever
}
}

Hello, i tried another framework (Httpful), there it works without problems ;). So i close this issues now.

glad you were able to get it resolved :)


Increase accessibility of Trigger agent options

Increase accessibility of Trigger agent options by using a Hash instead of an Array. Includes a patch to facilitate migration to the new rules format. The deprecated array format is still works. When the agent is next updated(i.e. saving from edit page) rules in the array format will be converted into the hash format. Tests/fixtures were updated that specified rules using array specific logic.
Inspired by this conversation https://github.com/cantino/huginn/pull/1139#discussion_r44586348
Basically, it was really inconvenient to update options inside an object(rule) that was also inside an array(rules). So I replaced that array(rules) with a hash(rules) and gave each object(rule) a name(hash['recipe_name']).

To clarify, the reason you'd like to change this is so that a CommanderAgent can identify a rule by name instead of having to manage an Array, right?
@dsander / @knu, any opinions?

Not sure if I fully understand the use case, wouldn't it be possible to have multiple trigger agents instead of one? With multiple CommanerAgents each just needs to update its own trigger agent and can safely override the options.

That is correct @cantino.
@dsander using multiple commanders/triggers should also work.

Maybe we should allow both hashes and arrays, but not deprecate the old format or convert between them?

I personally do not use the TriggerAgent but it sounds a bit odd to change the options format to "fix" the commander agent. Would it be an option to make the commander agent smarter when configuring agents that expect the options to be an array? Like adding some kind of placeholder to the array which means the commander will keep that index instead of replacing the whole array?

@cantino I like that solution.
@dsander that makes sense, but, it would likely mean more loc and negatively add to the "options configuration learning curve" for new users.

@brianpetro, could it be something like:
{
  'configure_options': {
    'path.to.array[2]': 'hello!'
  }
}

Where the JSONPath stored in the key name tells it what element to update?

@brianpetro, maybe it could be something like this?
{
  'configure_options': {
    'path.to.array[2]': 'hello!'
  }
}

So, is there a way to change the value field using a trigger agent here?

{
  "expected_receive_period_in_days": "2",
  "keep_event": "false",
  "rules": [
    {
      "type": "regex",
      "value": "foo\\d+bar",
      "path": "topkey.subkey.subkey.goal"
    }
  ],
  "message": "Looks like your pattern matched in '{{value}}'!"
}

If so, what would that look like?


Add unit tests for compare256 variants.

@Dead2 we need a new label for "testing framework". Maybe it can be used for fuzzers/benchmarks/gtest..


Upload to PyPI on GitHub release publish event

If you tag your release by publishing a release on GH and trigger it with the release: published event then the pypi upload step gets skipped.

So, I think that this not working is actually https://github.com/actions/runner/issues/2788 and it should work as documented here: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#release
We might want to consider building in a workaround though.


[requires.io] dependency update on master branch

Travis tests have failed
Hey @silasary,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
1st Build

     python run.py tests -m "not tappedout"

CONFIG: web_cache=.web_cache
CONFIG: whoosh_index_dir=whoosh_index
Fetching https://mtgjson.com/json/version.json (cache ok)
CONFIG: slow_fetch=10.0
CONFIG: magic_database=cards
CONFIG: mysql_host=localhost
CONFIG: mysql_port=3306
CONFIG: mysql_user=root
CONFIG: mysql_passwd=
CONFIG: slow_query=5.0
Creating database cards
Database update required
CONFIG: scratch_dir=.
Storing https://mtgjson.com/json/AllCards-x.json.zip in ./zip/zip.zip
Storing https://mtgjson.com/json/AllSets.json.zip in ./zip/zip.zip
Fetching https://pennydreadfulmtg.github.io/modo-bugs/bugs.json (cache ok)
UNKNOWN BUGGED CARD: Teferi’s Realm
Fetching http://whatsinstandard.com/api/v5/sets.json (cache ok)
Traceback (most recent call last):
  File "run.py", line 86, in <module>
    run()
  File "run.py", line 35, in run
    multiverse.init()
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/multiverse.py", line 26, in init
    update_database(current_version)
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/multiverse.py", line 131, in update_database
    update_pd_legality()
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/multiverse.py", line 159, in update_pd_legality
    if s == rotation.current_season_code():
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/rotation.py", line 37, in current_season_code
    return last_rotation_ex()['code']
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/rotation.py", line 56, in last_rotation_ex
    return max([s for s in sets() if s['enter_date_dt'] < dtutil.now()], key=lambda s: s['enter_date_dt'])
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/rotation.py", line 98, in sets
    __SETS.extend(init())
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/rotation.py", line 34, in init
    return [postprocess(release) for release in set_info]
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/rotation.py", line 34, in <listcomp>
    return [postprocess(release) for release in set_info]
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/magic/rotation.py", line 68, in postprocess
    setinfo['enter_date_dt'] = dtutil.parse(cast(str, setinfo['enter_date']), '%Y-%m-%dT%H:%M:%S.%fZ', dtutil.WOTC_TZ)
  File "/home/travis/build/PennyDreadfulMTG/Penny-Dreadful-Tools/shared/dtutil.py", line 36, in parse
    dt = datetime.datetime.strptime(s, date_format)
  File "/opt/python/3.6.3/lib/python3.6/_strptime.py", line 565, in _strptime_datetime
    tt, fraction = _strptime(data_string, format)
  File "/opt/python/3.6.3/lib/python3.6/_strptime.py", line 362, in _strptime
    (data_string, format))
ValueError: time data '2015-10-02T00:00:00.000' does not match format '%Y-%m-%dT%H:%M:%S.%fZ'

Um?

I think one of the feeds changed. Prices are not updating because of this same thing.

actually 2015-10-02T00:00:00.000 is a pure bs date so one of the feeds changed AND it's buggy.

Oh.  It's almost certainly https://github.com/glacials/whatsinstandard/commit/606e2fbbac7275be984aabb273e0556f643dc60e

ah, good find. "nonbreaking change" ho ho ho. these people have got too used to php's strtotime :D

bors r+


[5.7] PHPDoc coding style fixes

This PR modifies only PHPDoc blocks and has no functional changes.
According to the Contribution Guide:

@param attribute is followed by two spaces, the argument type, two more spaces, and finally the variable name

Which was NOT true in many places.
This PR fixes it for all Laravel codebase.

This honestly just creates more headache than its worth with merge conflicts and such.

PRs like this are best done RIGHT BEFORE a big release.

I see.
By saying "big release", do you mean 5.8 or something like 6.0?
Anyways, do you interested in such PR? I can do it again if we can pick the right moment.

@taylorotwell So, do you interested in that PR later?


Update Project.toml

Increment Catlab version max to 0.16.8 and AlgebraicRewriting to 0.3.2

Unfortunately we'll also need to increment the stockflow version to propagate these changes

@Saityi Do you know how to do this automatically?  Whenever Catlab updates, update the Project.toml to accept the new one and increment the version?

@neonWhiteout  I think all you need to do is to modify https://github.com/AlgebraicJulia/StockFlow.jl/blob/main/.github/workflows/julia_ci.yml#L43 along the lines of https://juliaregistries.github.io/CompatHelper.jl/stable/#GitHub -- with bump_version=true https://juliaregistries.github.io/CompatHelper.jl/stable/#CompatHelper.main
See https://github.com/AlgebraicJulia/.github/blob/f8ec80a3d3418c495ede00bb9aa8cafa805f5d2b/.github/workflows/julia_compat.yml#L18 for the one we're using via the uses reference in julia_ci.yml; you may be able to just copy-and-paste it into our julia_ci.yml with a modified main on L48.
I'm not sure if there's a more DRY way of handling it (uses the AlgebraicJulia/.github version, but modifies the main) ...


Add tag list to interface

Could you add the tag documentation somewhere.

You can add title, description, author and tags clicking Save button.
If you don't want to create an account just click "Dismiss" on the form and download the ZWO file.
Let me know if I answered the question.

No im talking about the actual documentation:
<IntervalsT Repeat="3" OnDuration="60" OffDuration="120" OnPower="1" OffPower="0.748" pace="0">
  <textevent timeoffset="0" message="Hammer down, 1 minute!"/>
  <textevent timeoffset="60" message="... and cool down for a few!"/>
</IntervalsT

Ryan Ramsey | Sr. Systems Engineer (Enterprise) | Nutanix Inc.
ryan.ramsey<EMAIL_ADDRESS>|  +1.805.551.2854 Mobile
[Image result for linkedin signature logo]<linkedin.com/in/ryan-ramsey-5b2b321>
<EMAIL_ADDRESS>From: Carlo Schiesaro<EMAIL_ADDRESS>Sent: Wednesday, December 9, 2020 11:00 AM
To: breiko83/zwo-editor<EMAIL_ADDRESS>Cc: Ryan Ramsey<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [breiko83/zwo-editor] Add tag list to interface (#11)
You can add title, description, author and tags clicking Save button.
If you don't want to create an account just click "Dismiss" on the form and download the ZWO file.
Let me know if I answered the question.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_breiko83_zwo-2Deditor_issues_11-23issuecomment-2D741981142&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=uucYufoLmOMSgduH2MOd7xt32h8e4YZ3cAqJChonjIE&m=ABxDvVDIF_3TchbOo_9zStd-0Cbx3gF7aTq-8Wt0eoY&s=y6Ne0rVqb8fpw9RITL6F0UIERDrWDe7TXQtZ7MNV5t4&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AJD4TUIRDIFE6A4BX5L4Y2DST7CMDANCNFSM4UT5ERRQ&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=uucYufoLmOMSgduH2MOd7xt32h8e4YZ3cAqJChonjIE&m=ABxDvVDIF_3TchbOo_9zStd-0Cbx3gF7aTq-8Wt0eoY&s=8k-JUtLxbrwM6QNT2sqNpSicrGkDnlyFHp4tp8ErLWI&e=.

I'm still not sure I understand your suggestion. Would do you mean with tags and documentation?
Can you make an example so I can understand?
Thanks!

I meant documentation for each of the tags… I saw it once on the internet but I lost the link.
For example
 -           Creator of the workout
 -             Name that will be displayed in Zwift.
…
       - Display a text message on the screen.
      The time of which an event triggers (in seconds)
         The text to display on the screen
       - Create an Interval block
            How many times to repeat the interval cycle
   # seconds to sustain the interval
  # of seconds to sustain the rest period
Etc…
Ryan Ramsey | Sr. Systems Engineer (Enterprise) | Nutanix Inc.
ryan.ramsey<EMAIL_ADDRESS>|  +1.805.551.2854 Mobile
[Image result for linkedin signature logo]<linkedin.com/in/ryan-ramsey-5b2b321>
<EMAIL_ADDRESS>From: Carlo Schiesaro<EMAIL_ADDRESS>Sent: Thursday, December 10, 2020 1:41 AM
To: breiko83/zwo-editor<EMAIL_ADDRESS>Cc: Ryan Ramsey<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [breiko83/zwo-editor] Add tag list to interface (#11)
I'm still not sure I understand your suggestion. Would do you mean with tags and documentation?
Can you make an example so I can understand?
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_breiko83_zwo-2Deditor_issues_11-23issuecomment-2D742404348&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=uucYufoLmOMSgduH2MOd7xt32h8e4YZ3cAqJChonjIE&m=wXBuVzJLa1L1_wqQrMp8AH1CoV8z8APoX86o4pttv-Y&s=p0EtYQ9sg16JoZ3GdP5P_dNBsCYD8sS-7qGxcb0TL1Q&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AJD4TULR6C7HWRCRKNSUF2TSUCJRBANCNFSM4UT5ERRQ&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=uucYufoLmOMSgduH2MOd7xt32h8e4YZ3cAqJChonjIE&m=wXBuVzJLa1L1_wqQrMp8AH1CoV8z8APoX86o4pttv-Y&s=yTODCK8rKCwZSivBYdvhFsf_5oaMMeJ4ONWVDKHoL60&e=.

You are referring to the final ZWO file. Why would you need instructions to manually edit this?
The point of having the web interface is actually not to edit text files.
Let me know if you need more help or I'll close this.
Thanks!

Great question.
I needed to do it for the overlap in the text fields.. you can’t grab them with the current interface. There are also a LOT of tags that are not in the builder right now that someone might want to add later on.
Just a suggestion… I’d like to see the community grow and that comes from knowledge and creativity.
Ryan Ramsey | Sr. Systems Engineer (Enterprise) | Nutanix Inc.
ryan.ramsey<EMAIL_ADDRESS>|  +1.805.551.2854 Mobile
[Image result for linkedin signature logo]<linkedin.com/in/ryan-ramsey-5b2b321>
<EMAIL_ADDRESS>From: Carlo Schiesaro<EMAIL_ADDRESS>Sent: Thursday, December 10, 2020 1:19 PM
To: breiko83/zwo-editor<EMAIL_ADDRESS>Cc: Ryan Ramsey<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [breiko83/zwo-editor] Add tag list to interface (#11)
You are referring to the final ZWO file. Why would you need instructions to manually edit this?
The point of having the web interface is actually not to edit text files.
Let me know if you need more help or I'll close this.
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_breiko83_zwo-2Deditor_issues_11-23issuecomment-2D742805578&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=uucYufoLmOMSgduH2MOd7xt32h8e4YZ3cAqJChonjIE&m=6p0Qi1SKQW788S1LOzBA3IAVvjRq2ojukfSlDuNdFM0&s=Ue_9Q3J3p1DM_AOeB-g4lm5br61ltEX6gvBpetrmP50&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AJD4TUIIDGIVIY65XXEGAHTSUE3LNANCNFSM4UT5ERRQ&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=uucYufoLmOMSgduH2MOd7xt32h8e4YZ3cAqJChonjIE&m=6p0Qi1SKQW788S1LOzBA3IAVvjRq2ojukfSlDuNdFM0&s=C1KN9Ur-M9wQI6bVcXXknwV3ZeUKCY9V3OZMfq4ADBA&e=.

Sorry, I'm closing this as it's not relevant to the workout editor.
Thanks for your contribution.


Add shapevalidation test

Test that the expected amount of memory, number of CPUs, and number of NUMA nodes appear

Exclude this test from existing CIT pipelines that trigger frequently

Add a new release pipeline that will run all tests on an image, triggered manually

Add API for tests to force a specific zone and VM shape when it is required for a test, and log when they do so

Test new API functions

Unrelated to merging this PR, do we want the shape validation tests to run on a 12 hour interval, or would it potentially use more quota than we want?
If so, we might want to exclude it from the cit-periodics dashboard here in a different PR:
https://github.com/GoogleCloudPlatform/oss-test-infra/blob/2f7280c9d7d4a3cf96c156fa60a52011188d3454/prow/prowjobs/GoogleCloudPlatform/gcp-guest/gcp-guest-config.yaml#L412

/lgtm

Definitely not, that would be too much quota and isn't likely to change once it's passing. I have a change ready to exclude that.


Add Health Deidentification SDK client library for java

Description
This PR adds the Azure Deidentification REST client library for java
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run prepare-pipelines

/azp run java - healthdataaiservices - ci

/azp run java - healthdataaiservices - ci

/azp run prepare-pipelines

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-health-deidentification

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-health-deidentification

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-health-deidentification

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-health-deidentification

/azp run prepare-pipelines

/azp run prepare-pipelines


fix(es/minifier): mangle.safari10 is causing identifier reuse

When the mangle.safari10 option is set to true identifiers are re-used in the try/catch block which results in a parse error.
Description:

Using the mangle.safari10 option causes identifiers to be re-used in a try/catch block.
BREAKING CHANGE:

Related issue (if exists):
https://github.com/swc-project/swc/issues/8837

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

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

You can do UPDATE=1 cargo test from ./crates/swc_ecma_minifier to update test snapshot files

@kdy1 I think we'll have divergence with the terser output for the broken_safari_catch_scope test if we remove this option. Is that OK?

Yeap, it's fine

Can you run UPDATE=1 cargo test --test projects --test tsc from ./crates/swc?


Haute42 g16 rgb led not working after reset

Hi all my g16 haute42 rgb lighting is not working i tried installing the firmware again but still not working i also tried following the same config in the docs page no improvement either please help

Hi @DarkSim0 ,
Please join up in their Discord and submit a support ticket there.
https://discord.com/invite/hcCyQRffbU


Rename from method in model builder.

I will force merge this PR desipte failing test. It's a false positive.


pull request for e2e test at 02/04/2017 18:45:08

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


Add Data.Maybe

This is a mathematical library, not sure how Maybe is going to be used.
We may have another programming library

Maybe may be useful, but it will require some additional lemmas about its relationship with non-decidable subtypes. Anyway, we're adding constructions as they're needed.
Also, in our code style, constructor names begin with lower case letters.

Turns out this is very useful in writing some solvers. Can we merge this?


validateSearch called when hovering unrelated routes on Links

Describe the bug
I was investigating performance issues in my own project where i have a table with a 300~ Links that had performance issues and when I added a console.log in the validateSearch of the current route it printed for every Link I created. I then managed to somewhat reproduce this in the Kitchen Sink example.
I'm not sure if the performance issue is related, but it seems unperformant to call validateSearch for Links that do not match the path.
Your Example Website or App
https://stackblitz.com/edit/tanstack-router-zzmuer?file=src%2Froutes%2Flogin.tsx
Steps to Reproduce the Bug or Issue
Go to:
https://stackblitz.com/edit/tanstack-router-zzmuer?file=src%2Froutes%2Flogin.tsx
Here I've just changed validateSeach to
`validateSearch: (search) => {
console.log('validating login', Math.random());
return z
  .object({
    redirect: z.string().optional(),
  })
  .parse(search);

}`
Hovering any Link pointing to users/invoices/other routes will call the login validateSearch, multiple times.
Expected behavior
Only call validateSearch on Links with a "to" matching "/login", or similar.
Screenshots or Videos
https://github.com/TanStack/router/assets/87984222/c041b1e7-5070-4d43-9f58-7bb08b49fb88
Platform
Linux, ubuntu 22
Firefox/Chrome, latest
Additional context
No response

The validateSearch option gets called here in router.matchRoutes for all the matches.
Looking into this for this scenario, the validateSearch call gets called in these instances (as far as I can tell):

During the instantiation of the <Link> component.
When hovering a link (aka preloading).
When using a matchRoute call.

This is because at each of these places we either end up calling router.buildLocation (which internally calls router.matchRoutes) or directly call router.matchRoutes. These calls end up calling validateSearch option on each of the matches internally.
This issue is also related to #1481 where validateSearch gets called on other routes.
I'll investigate what's the effect of running the validateSearch only onLoad.

This would definitely require consideration since making changes to this behaviour affects how route preloading works.
For the time being, I don't think we can alter this behaviour since the router depends on the search params being validated in the router.matchRoutes method (as mentioned here) for it to create the actual route matches used downstream.
Going to have to leave this for the time being.

I'm closing this since the router relies on the validateSearch function being called to validate the URLs being generated for use in the <Link> component. Also, since this issue, being made performance improvements to memorize all the routes internally as well as further refinements into how the <Link> component's props are handled.


ROX-14100: enrich vuln report with package-level 'fixedBy'

Description
This creates a new "fixedby" enricher which attempts to determine the lowest version to which a package must be upgraded to fix all fixable vulnerabilities.
Note the following is possible:
Let's say a package is on version A which is affected by some vulnerability fixed in version C, such that A < C. It is possible that another vulnerability only affects the package starting on version B, such that A < B < C, and is fixed in version D, such that C < D. This enricher does not account for this vulnerability and will just say the "fixedby" version is C instead of D.
Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Here I tell how I validated my change
Unit tests
Deployed Central Services and scanned the images listed below

The tests were created by checking against real images. These images include:

docker.io/library/alpine:3.18 - Alpine
docker.io/library/amazonlinux:2023 - Amazon Linux
docker.io/library/debian:11 - Debian
quay.io/stackrox-io/scanner:4.3.0 - Go/RHEL
quay.io/rhacs-eng/qa:elasticsearch6-rhel8 - Java/RHEL
quay.io/rhacs-eng/qa:bitnami-express - Node.js
docker.io/library/oraclelinux:8 - Oracle
docker.io/library/photon:3.0-20230923 - Photon
docker.io/library/python:3.12.0-bookworm - Python
quay.io/rhacs-eng/main:4.3.0 - RHEL/RHCC
quay.io/rhacs-eng/qa:ruby-simple - Ruby
docker.io/opensuse/leap:15 - SUSE
docker.io/ubuntu:20.04 - Ubuntu

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/test ocp-stable-scanner-v4-qa-e2e-tests

/test ocp-stable-scanner-v4-qa-e2e-tests

/retest

going to merge to get this tested asap, but I highly encourage @dcaravel and @jvdm to review this once back in office even post merge


Correct scan order of Recurrent Neural Network with multiple timeseries?

I am reading the code of SimpleRNN and LSTM, but I am still not sure how keras trains a network to deal with multiple samples (multiple time series). Someone also called the problem "Multi-Dimensional Recurrent Neural Networks": http://people.idsia.ch/~juergen/icann_2007.pdf
It seems that Kera can handle it since it gives us a 3D tensor where one dimension can be the specific sample. However, when I read the code, it seems that it will reshuffle the tensor, and scan along the time dimension first by Theano. Here is the scan order that I learn from the code:
[timestamp 1, sample 1, features11]
[timestamp 1, sample 2, features12]
[timestamp 2, sample 1, feature21]
[timestamp 2, sample 2, feature22]
...
Here comes the question: how do you guarantee the feedback loop (the hidden layer output) is for the same sample? Will the output of sample1 be used as an input of sample2?
Thanks,

Someone also called the problem "Multi-Dimensional Recurrent Neural Networks": http://people.idsia.ch/~juergen/icann_2007.pdf

This designates RNNs with multidimensional outputs (also supported by Keras) but has nothing to do with processing multiple samples at once.

Will the output of sample1 be used as an input of sample2?

No, in Keras RNN layers each sample is processed independently.

Thanks for the reply. If each sample is processed independently, does it mean that it is the same as having #number_of_sample of RNNs independently? or is it one single RNN for arbitrary number of samples?


Admission from Rockfield IT OpenZiti Adopters:

Admission from Rockfield IT OpenZiti Adopters:

Hi @RockfieldIT - would you mind setting up signed commits? The project can't merge a commit unless they're signed. Once you setup signed commits you'll be able to amend your previous commit and force push your branch and it should then show up as 'signed'.
thanks for the pr!

Sorry for the delay !


Use the FrameTracer class in the executed notebook

Updates the notebook cell templates for activating tracing to use FrameTracer instead of calling create_collector and tracing_on/tracing_off directly.

Codecov Report

Merging #160 (d05e163) into main (8d24740) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #160   +/-   ##
=======================================
  Coverage   95.77%   95.77%
=======================================
  Files          33       33
  Lines        1917     1917
=======================================
  Hits         1836     1836
  Misses         81       81

Impacted Files
Coverage Δ

pybryt/execution/__init__.py
100.00% <100.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 8d24740...d05e163. Read the comment docs.


Disable build dependencies.

This should fix #6973. Was accidentally enabled when migrating the argument names.

Also need to comment line #28 in zip.yml and rebase master to get the storage/spm build fix

Reverted GHA checks - they all passed in 326f335


the installation methods and steps not defined in markdown readme

the readme.md was not completed, is there a other way to install this? there is no documentation to see or read.

We are in active development.
The completed version will be released soon.
You can try to read the docs first https://qlib-server.readthedocs.io/ .
If you encounter any problem, please contact us.
Before try the Online version of Qlib,  you can try the offline version first

thank you

@Fliw   We've finished the documents now.
If you have any problems when reading the docs, feel free to contact us.
Otherwise, we'll close this issue later.

okay, thank you ✌🏻 i will close this issue now


Curly/smart quotes result in an invalid XForm

If you have an XLSForm with curly/smart quotes, when you convert it to an XForm, the form will contain invalid (as far as XML and ODK Validate) characters.
dumb/straight quotes, form is valid

curly/smart quotes, form is invalid

I propose pyxform either throw a warning or convert the smart quotes to dumb quotes.

Duplicate of #85

Oh, hah!


Update seller - change LENS token

As a seller, if my wallet owns several LENS tokens, I should be able to switch from a token to another one
This will call updateSeller() protocol method to change the authTokenId of the seller account

Do we know how many Sellers are using Lens now? Does this impact anything else like royalties or anything?


Hott-1284 Add Slack notification.

Jira link
HOTT-<1284
What?
Add Slack notification in case of FailedUpdatesError

A couple of the lint violations are to do  with the files we've added :smile:


Polyglot Notebook: [DevExE2E] The execution result of "Output" doesn't change after setting value in JavaScript when using split view

Describe the bug
Using extension Polyglot Notebook in VS Code insider.
The execution result of "Output" doesn't change after setting value in JavaScript.
Detailed Repro Build info: VSCodeSetup-x64-1.79.0 + dotnet-interactive-vscode-1.0.4309011.vsix
VSCode Link: https://code.visualstudio.com/download
Version: 1.79.0 (system setup)
Commit: b380da4ef1ee00e224a15c1d4d9793e27c2b6302
Date: 2023-06-07T14:26:35.552Z
Electron: 22.5.5
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.20348
dotnet-interactive-vscode-1.0.4309011.vsix
https://dev.azure.com/dnceng/internal/_build/results?buildId=2197700&view=artifacts&pathAsName=false&type=publishedArtifacts
sdk info: dotnet-sdk-7.0.302-win-x64
Note:
1.It's not a regression issue. It also repro on VSCodeUserSetup-x64-1.78.1 + dotnet-interactive-vscode-1.0.4259021.vsix.
Steps:
Start with an empty text file open in VSCode
2.Ctrl+Shift+P => "Polyglot Notebook: Create new blank notebook"
3.Select "Create as .dib" ->Select "C#" -> Specify the code in cell-> execution
var x=123;
4. Add a new code cell below as html. -> Specify the code in cell-> execution
The value is not yet set.
5. Add a new code cell below as javascript. -> Specify the code in cell-> execution
#!share --from csharp x
document.getElementById("output").innerText = The value is ${x}.;
console.log(The value is ${x}.);
Expected Results:
After executing Step5, the Output should be shown as "The value is 123" in step4.

Actual Results:
The execution result of "Output" doesn't change after setting value in JavaScript.

Please complete the following:
Which version of .NET Interactive are you using? (In a notebook, run the #!about magic command. ):

OS

[ √] Windows 11
[ ] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[√ ] Visual Studio Code
[ ] Visual Studio Code Insiders
[ ] Visual Studio
[ ] Other (please specify)

does it repro even when you don't use split view but only a single view for the notebook?

this is vscode insiders 1.80.0

@colombod This issue only repro in split view. It doesn't repro when using a single view.

This issue repro on VS Code 1.93.1 + dotnet-interactive-vscode-1.0.5468011.vsix + dotnet-sdk-8.0.401-win-x64 as split view.

This issue still repro on VS Code 1.93.1 + dotnet-interactive-vscode-1.0.5468011.vsix + dotnet-sdk-8.0.401-win-x64 as split view.


Fixed an input bug, where due (next|last) week is not captured by a regex

Hi @gammons
It has been a while.
Here is my fix for #126, it seems the regex that detects due, does not detect  and thus not setting a todo's due date.
AS always happy to add to this as you need.
Regards
Stuart

hi @stuartskelton this looks great, thank you!


[TreeView] Add checkbox support

Thanks for the Component, it works really well. Unfortunately I have come across on problem in the implementation which I can't seem to find a workaround for.

[X] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡
I am looking to use the TreeView / TreeItem components with a check box on each row. Ideally I would like a prop to suppress the event that expands and collapses the tree if the CheckBox is clicked. Or maybe a prop to enable a checkbox with a prop to set the value and a call back when clicked that also avoids the list toggle.
At the moment when a check box is toggled it also expands and collapses the tree when the node has children.
Thank You.
Examples 🌈

This is the current code...
const checkBoxClicked = (event, checked, id) => {
   setOrgStructureElement(checked, id, selected, orgStructure);
};

const createOrgStructureLevel = orgStructureElement => {
   const elements = [];

    orgStructureElement.forEach((v, k) => {
      const { id, children } = v;
      if (k.length !== 0) {
        const label = (
          <div style={{ display: 'flex', alignItems: 'center' }}>
            <Checkbox
              id={`checkbox-${k}`}
              className={classes.globalFilterCheckbox}
              checked={selected.has(id)}
              onChange={(event, checked) => checkBoxClicked(event, checked, id)}
              color="primary"
            />
            <Typography variant="caption">{k}</Typography>
          </div>
        );
        elements.push(
          children && children.length > 0 ? (
            <TreeItem key={id} nodeId={id} label={label}>
              {createOrgStructureLevel(children)}
            </TreeItem>
          ) : (
            <TreeItem key={id} nodeId={id} label={label} />
          ),
        );
      } else if (children) {
        elements.push(createOrgStructureLevel(children));
      }
    });
    return elements;
  };

  const handleExpanded = (nodeId, nodeExpanded) => {
    // cache expanded nodes
    if (nodeExpanded) {
      addOpenOrgStructurePanel(nodeId);
    } else {
      removeOpenOrgStructurePanel(nodeId);
    }
  };

  return (
    <TreeView
      defaultExpanded={openOrgStructurePanels}
      onNodeToggle={(nodeId, nodeExpanded) => handleExpanded(nodeId, nodeExpanded)}
      defaultCollapseIcon={<ExpandMoreIcon />}
      defaultExpandIcon={<ChevronRightIcon />}>
      {orgStructure.length !== 0 ? createOrgStructureLevel(orgStructure) : null}
    </TreeView>
  )
};

Motivation 🔦
see above

@mbrookes I can confirm no, Event.stopPropagation() doesn't help. Using the following handler:
  handleCheck(event) {
    // Prevent the checkbox-clicked event from firing the collapse/expand action on the tree
    // This makes no difference to behaviour
    event.stopPropagation()
    console.log('CHECKBOX CHANGED')
  }

@cdwyer-cognito did you fix this in the end?

Ok. fixed.
After delving into the mui code, I realised the problem here. The Checkbox is handling the onChange event from its base input, not an onClick.
So capturing the onChange and stopping it from propagating won't do any good - the original onClick event bubbles up the DOM to the TreeItem where it's handled here. Sigh.
Anyhoo, simple fix is to give the checkbox an onClick handler as well as an onChange handler. This isn't documented in the checkbox api, but it ends up being given to the IconButton here
So here's a Checkbox which will work for this purpose:
<Checkbox
    id={`checkbox-${yourId}`}
    checked={isChecked}
    onChange={(e, checked) => (console.log('you checked it!', checked)}
    onClick={e => (e.stopPropagation())}
    {...props}
/>

So sorry, I've not looked at this thread for a while. That looks a good solution, I'll give it a go and let you know. Thank you

Great, that works perfectly. I had tried to stop the propergation on the onChange, I understand why that didn't work now. Thank you again.

can you shair current code of checkbox treeview

@RjSingh1529 my solution
`import React from 'react';
import TreeView from '@material-ui/lab/TreeView';
import ExpandMoreIcon from '@material-ui/icons/ExpandMore';
import ChevronRightIcon from '@material-ui/icons/ChevronRight';
import TreeItem from '@material-ui/lab/TreeItem';
import { Checkbox, FormControlLabel } from '@material-ui/core';
interface RenderTree {
id: string;
name: string;
children?: RenderTree[];
}
const data: RenderTree = {
id: '0',
name: 'Parent',
children: [
{
id: '1',
name: 'Child - 1',
},
{
id: '3',
name: 'Child - 3',
children: [
{
id: '4',
name: 'Child - 4',
children: [
{
id: '7',
name: 'Child - 7',
},
{
id: '8',
name: 'Child - 8',
},
],
},
],
},
{
id: '5',
name: 'Child - 5',
children: [
{
id: '6',
name: 'Child - 6',
},
],
},
],
};
export default function RecursiveTreeView() {
const [selected, setSelected] = React.useState<string[]>([]);
function getChildById(node: RenderTree, id: string) {
let array: string[] = [];
function getAllChild(nodes: RenderTree | null) {
  if (nodes === null) return [];
  array.push(nodes.id);
  if (Array.isArray(nodes.children)) {
    nodes.children.forEach((node) => {
      array = [...array, ...getAllChild(node)];
      array = array.filter((v, i) => array.indexOf(v) === i);
    });
  }
  return array;
}

function getNodeById(nodes: RenderTree, id: string) {
  if (nodes.id === id) {
    return nodes;
  } else if (Array.isArray(nodes.children)) {
    let result = null;
    nodes.children.forEach((node) => {
      if (!!getNodeById(node, id)) {
        result = getNodeById(node, id);
      }
    });
    return result;
  }

  return null;
}

return getAllChild(getNodeById(node, id));

}
function getOnChange(checked: boolean, nodes: RenderTree) {
const allNode = getChildById(data, nodes.id);
let array = checked
? [...selected, ...allNode]
: selected.filter((value) => !allNode.includes(value));
array = array.filter((v, i) => array.indexOf(v) === i);

setSelected(array);

}
const renderTree = (nodes: RenderTree) => (
<TreeItem
key={nodes.id}
nodeId={nodes.id}
label={
<FormControlLabel
control={
<Checkbox
checked={selected.some((item) => item === nodes.id)}
onChange={(event) =>
getOnChange(event.currentTarget.checked, nodes)
}
onClick={(e) => e.stopPropagation()}
/>
}
label={<>{nodes.name}</>}
key={nodes.id}
/>
}
>
{Array.isArray(nodes.children)
? nodes.children.map((node) => renderTree(node))
: null}

);
return (
<TreeView
defaultCollapseIcon={}
defaultExpanded={['0', '3', '4']}
defaultExpandIcon={}
>
{renderTree(data)}

);
}
`

@RjSingh1529  my solution in typescript
https://codesandbox.io/s/typescript-data-driven-treeview-dhkx2?file=/src/App.tsx

We probably need to have a dedicated TreeItemCheckbox anyway to get the a11y story right. Then we also have better control over event propagation quirks.

plea

Ok. fixed.
After delving into the mui code, I realised the problem here. The Checkbox is handling the onChange event from its base input, not an onClick.
So capturing the onChange and stopping it from propagating won't do any good - the original onClick event bubbles up the DOM to the TreeItem where it's handled here. Sigh.
Anyhoo, simple fix is to give the checkbox an onClick handler as well as an onChange handler. This isn't documented in the checkbox api, but it ends up being given to the IconButton here
So here's a Checkbox which will work for this purpose:
<Checkbox
    id={`checkbox-${yourId}`}
    checked={isChecked}
    onChange={(e, checked) => (console.log('you checked it!', checked)}
    onClick={e => (e.stopPropagation())}
    {...props}
/>

please share your complete code with us . if it is possible.

@RjSingh1529 my solution in typescript
https://codesandbox.io/s/typescript-data-driven-treeview-dhkx2?file=/src/App.tsx

I don't understand the getChildByID code. Can you re-explain how you checked the child when the parent is checked.

Added auto selection for parent when children are selected. And the same for deselection.
Improved a bit search for selected node.
@dnquang1996vn Thank you for your example!
https://codesandbox.io/s/edeyu

Added auto selection for parent when children are selected. And the same for deselection. Improved a bit search for selected node. @dnquang1996vn Thank you for your example! https://codesandbox.io/s/edeyu

@hamsishe  great work! Any tips on how I could extend this to where the children could be a group of radio buttons (in addition to have checkboxes in the same parent)

Great sample.


Fix other files in dist not being published

Pull Request Test Coverage Report for Build 434389267

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

Totals

Change from base Build 430997499:
0.0%

Covered Lines:
20

Relevant Lines:
20

💛  - Coveralls


[bug]: generate.py as module (API) requires explicit argument for log_tokenization

Is there an existing issue for this?

[X] I have searched the existing issues

OS
macOS
GPU
mps
VRAM
64GB shared
What happened?
Documentation describes "simplified API" at https://invoke-ai.github.io/InvokeAI/features/OTHER/?h=api .
Steps to reproduce this bug:
Enter console to activate venv and start Python, then enter the commands from the documentation, or even the commands from the tutorial embedded in generate.py:
./invoke.sh
[5. open the developer console.]
>>> from ldm.generate import Generate
>>> g=Generate()
>>> outputs=g.txt2img("any text")
or
>>> outputs=g.prompt2png(prompt="any text", outdir="./outputs/img-samples")

Error:
.venv/lib/python3.10/site-packages/ldm/invoke/conditioning.py:96 in _get_conditioning_for_prompt
|   93 │   Process prompt structure and tokens, and return (conditioning, unconditioning, extra   │
│    94 │   """                                                                                │
│    95 │                                                                                │
│ ❱  96 │   if log_tokens or Globals.log_tokenization:                                             │
│    97 │   │   print(f"\n>> [TOKENLOG] Parsed Prompt: {parsed_prompt}")                           │
│    98 │   │   print(f"\n>> [TOKENLOG] Parsed Negative Prompt: {parsed_negative_prompt}")         │
│    99
AttributeError: 'Namespace' object has no attribute 'log_tokenization'

Steps to test:
repeat all steps above, except with the extra argument to txt2img or prompt2png,
log_tokenization=True
and the image is generated without error.
Fix?
Ensure default for log_tokenization is set. I don't know where.
While we're at it, the documentation could be update to match at least one of the the tips in generate.py, which says that txt2img is deprecated.
Screenshots
No response
Additional context
No response
Contact Details
No response

Oops, I had written in this report that I entered "./install.sh" when I had actually entered "./invoke.sh" at the first example code. Hopefully nobody dismissed this report because of that mixup.


No way to set goal?

I cannot find an option to set a headpat goal. There's the button in config to show notification when the goal is reached, but no command to do so.

Only purpose of option is to hide splash screen image - unfortunate naming on my part, never intended that to be editable.
Just for extra flavor and reward after reaching 5, 15, 25 and every +25 after that.

understandable have a good day


I want to enable the hub to close (and settle?) my channel

User Story
As Chris I want to enable the hub to close (and settle?) my channel so that I don't have to do it
@andrevmatos Did I understand that correctly? Reference is https://github.com/raiden-network/raiden-contracts/issues/1123
Acceptance criteria
Tasks

That's not exactly for the hub. HUB can always close and settle whenever they want. The issue is about having generic 3rd-parties willing to pay for the gas, to do so, through an off-chain signature from us. The main use-case is #172 , where the main account (seen as a 3rd-party from contract's perspective) pays for the gas instead of the eth-less subkey. Other possibilites may arise as well, like companies willing to pay for the first couple of on-chain txs for users for easy onboarding, but strictly speaking, if the user wants to close, they're the one willing to pay for the gas, not the hub.

Thanks for the explanation :-) Closing it