How to calculate steps through epochs?

❓ Questions and Help
What is your question?
Hello, I want to compare the training time of transformer and my own model. By looking at the paper "Attention is all you need", I found an original sentence"We trained our models on one machine with 8 NVIDIA P100 GPUs. For our base models using the hyperparameters described throughout the paper, each training step took about 0.4 seconds. We trained the base models for a total of 100,000 steps or 12 hours. For our big models,(described on the bottom line of table 3), step time was 1.0 seconds. The big models were trained for 300,000 steps (3.5 days)."
How to calculate the steps here, the printed log of my model is as follows:
Code

Looking forward to your reply, thank you very much.
@myleott

The "ups" key corresponds to updates-per-second. So the step time here would be (1 / ups) = (1 / 1.11) = 0.9 seconds

The "ups" key corresponds to updates-per-second. So the step time here would be (1 / ups) = (1 / 1.11) = 0.9 seconds


[minor] sap.m.ViewSettingsDialog: Selecting all filter options causes a twitch

OpenUI5 version:
1.44.6
Browser/version:
Google Chrome/56.0.2924.87 on Windows 10
Any other tested browsers/devices(OK/FAIL):
Edge/38.14393.0.0: OK
IE 11: OK
URL:
https://openui5.hana.ondemand.com/explored.html#/sample/sap.m.sample.ViewSettingsDialog/preview
Steps to reproduce the problem:

Press on the first button (Open View Settings Dialog)
Go to the "Filters" page.
Open up Field1
Press the Select All checkbox

What is the expected result?
No twitch.
What happens instead?
The footer toolbar twitches for a split-second.
Any other information? (attach screenshot if possible)
https://giphy.com/gifs/openui5-view-settings-twitch-3o84U8blYyvSLiVkKk

Hi @serban-petrescu ,
Thank you for your findings. I have created an internal incident:<PHONE_NUMBER>
Regards,
Diana

The twitch should not happen anymore.
Regards,
Alex


Support OpenWRT 18.06 in luci device tracker

Description:
Updating luci rpc device tracker to use new APIs in OpenWRT 18.06.
As recommended in the old PR #15792 , I have moved RPC logic into a pip module called openwrt-luci-rpc
This is a retake of the PR #21233 which got messed up with the wrong CLA email in one commit. (apologies about that!)
Related issue (if applicable): fixes #8508
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Thanks @MartinHjelmare I appreciate the time spent on this review.


feat: Show API name in reporters

Issue
https://gravitee.atlassian.net/browse/APIM-3932
Description
Add api-name field in reporters data.
Related PRs:

https://github.com/gravitee-io/gravitee-reporter-elasticsearch/pull/83
https://github.com/gravitee-io/gravitee-reporter-common/pull/30
https://github.com/gravitee-io/gravitee-reporter-api/pull/85

Additional context

What about:

https://github.com/gravitee-io/gravitee-reporter-cloud
https://github.com/gravitee-io/gravitee-reporter-file
https://github.com/gravitee-io/gravitee-reporter-tcp
https://github.com/gravitee-io/gravitee-reporter-datadog

What about:

gravitee-io/gravitee-reporter-cloud
gravitee-io/gravitee-reporter-file
gravitee-io/gravitee-reporter-tcp
gravitee-io/gravitee-reporter-datadog

?

Seen together, we could update the dependencies, but it's not required as the modification is automatically provided to reporters


README should describe optional configuration variables

I learned the hard way (trial and error, reading source code) that capistrano-scm-copy supports a couple of very useful optional configuration variables (:tar_roles, :include_dir and :exclude_dir).  It would be nice if the README had a section that described these variables.  It would have saved me some time from digging into the source code.

+1

+1


Create extra option for placeholder option value

I believe theres value in adding an extra option which would allow you to choose the value (as opposed to the text that the current placeholderValue attribute is setting) of the placeholder. This I believe should be part of a bigger work that would set the value of the widget to the value of the placeholder when all options are removed (create "invisible" option with the set value). A scenario that I personally encountered issues with the current setup is when I put a multiple select in a form and I clear all values. Serialising or submitting the form you will see that theres no value for the select (not even an empty value) and this makes handling the clearing of the data on the server side considerably harder.

You can do it right now by overriding templates via onCreateTemplatesCallback

@tinovyatkin Thanks for the reply. I ended up doing it like so..
`
   const select = new Choices(widget, {
        maxItemCount: max,
        placeholder: true,
        placeholderValue: placeholder,
        .....

   select.passedElement.element.addEventListener('removeItem', function(event) {
        if(select.getValue().length==0){
            var el = select.setValue(['']);
            el.itemList.element.childNodes[0].classList.add('hidden');
        }
    });

    // Add event listener to select all options available when 'All' is clicked
    select.passedElement.element.addEventListener('addItem', function(event) {
        const value = event.detail.value;
        // If it was empty remove the empty value
        if(this.length>1 && this[0].value==''){
            select.removeActiveItemsByValue('');
        }
       .....

`


BeginnerTutorial.md: Include full render method

In the tutorial as is the original redux action creators onIncrement and onDecrement are not shown as props on the Counter component. This makes it so that the other buttons will no longer work if you follow the instructions to the letter.
Since the entire render method is very small I suggest including the whole render method like this:
function render() {
  ReactDOM.render(
    <Counter
      value={store.getState()}
      onIncrement={() => action('INCREMENT')}
      onDecrement={() => action('DECREMENT')}
      onIncrementAsync={() => action('INCREMENT_ASYNC')} />,
    document.getElementById('root')
  )
}

Making it much more clear how the main.js file should end up looking.
This is relating to the tutorial beginner tutorial:
https://github.com/justsayno/redux-saga/edit/master/docs/introduction/BeginnerTutorial.md and the instructions on adding the onIncrementAsync={() => action('INCREMENT_ASYNC')} to the render method in main.js in the tutorial project.

Thanks for contributing!


Unable to merge upstream changes

Related software versions

Description
As the history was abrutly reset when the code was made public, the tree is no longer shared with upstream httpx repository making it harder to get the updates.
I suppose we could script cherrypicks from upstream but it's easier to cherrypick once now to get the history properly done!
I have already done in the branch temporary_redo.
I pulled upstream tag v1.1.2 into an orphan branch and then I cherry picked our Initial commit and resolved all the conflicts (very few).
I'll now cherry pick the rest of the history up until now.
@gsilvapt @mariandppb if you guys are ok with it, we can just delete main and rename temporary_redo to main. From then on, merging upstream is possible again.
Use cases
Keep httpx up to date
References
No response

No idea how I managed to screw up by opening this issue here, but this is related to httpx as mentioned in the description...
https://github.com/surface-security/scanner-httpx/tree/temporary_redo


[Request] will you remove jQuery dependency

I am checking if you have any plan to remove jQuery dependency for the future release. It will reduce build size and improve performance (subjective topic), so it will be pure angular components (like angular material 2 library)
Thanks

Thanks for the update.


replica set with auth

When I try to run mongodb with --replSet and passed env variables for auth it gives error like
Error: Couldn't add user: not master
#179  I tried kalpakrg's solution too. However, it doesn't help either.

Same mistake

Ah, this is an interesting edge case of those new variables.  As a workaround, you'll probably need to specify just --auth instead of supplying the environment variables, and then set up the users yourself after setting up the replica set (since that needs to be set up first, IIRC).
I'm not sure if there's a good way for us to deal with --replSet + auth automatically. :disappointed:

It can only be corrected by overwriting the entrypoint
#!/usr/bin/env sh
# /custom-entrypoint.sh
if [ ! -f /data/db/.metadata/.replicaset ]; then
  mongod --fork --dbpath /data/db --port 27017 --logpath /var/log/mongod.log
  RET=1
  while [ $RET -ne 0 ]
  do
    echo "=> Waiting for confirmation of MongoDB service startup"
    sleep 5
    mongo admin --eval "help" >/dev/null 2>&1
    RET=$?
  done
  mongo admin --eval "db.createUser({user:'$MONGO_INITDB_ROOT_USERNAME',pwd:'$MONGO_INITDB_ROOT_PASSWORD',roles:[{role:'root',db:'admin'}]})"
  mongod --shutdown && mongod --fork --logpath /var/log/mongod.log --keyFile /run/secrets/MONGODB_KEYFILE --replSet $RS_NAME --shardsvr --dbpath /data/db --port 27017
  MYIP=<IP_ADDRESS>
  CONFIG="{_id:\"$RS_NAME\",version:1,members:[{_id:0,host:\"$MYIP:27017\"}]}"
  mongo -u $MONGO_INITDB_ROOT_USERNAME -p $MONGO_INITDB_ROOT_PASSWORD --authenticationDatabase admin --eval "printjson(rs.initiate($CONFIG))"
  # members="<IP_ADDRESS> <IP_ADDRESS>"
  # for member in $members; do
  #   if [ "$member" != "$MYIP" ]; then
  #     mongo -u $MONGO_INITDB_ROOT_USERNAME -p $MONGO_INITDB_ROOT_PASSWORD --authenticationDatabase admin --eval "printjson(rs.add('$member:27017'))"
  #     sleep 5
  #   fi
  # done
  mkdir -p /data/db/.metadata
  touch /data/db/.metadata/.replicaset
  mongod --shutdown && mongod --keyFile /run/secrets/MONGODB_KEYFILE --replSet $RS_NAME --shardsvr --dbpath /data/db --port 27017
else
  mongod --keyFile /run/secrets/MONGODB_KEYFILE --replSet $RS_NAME --shardsvr --dbpath /data/db --port 27017
fi

version: '2'
services:
  mongod:
    image: mongo:3.4.10
    environment:
      RS_NAME: $RS_NAME
      MONGO_INITDB_ROOT_USERNAME: $MONGO_INITDB_ROOT_USERNAME
      MONGO_INITDB_ROOT_PASSWORD: $MONGO_INITDB_ROOT_PASSWORD
    entrypoint: /custom-entrypoint.sh

Right, the initial user must be configured without --replSet, but then the daemon must be restarted with --replSet in order to run rs.initiate().
Here's the simple one-liner I've been using to test with:
$ docker run -it --rm -e MONGO_INITDB_ROOT_USERNAME=admin -e MONGO_INITDB_ROOT_PASSWORD=example mongo --replSet test

The following patch fixes the issue (and allows the above command to complete successfully), but at the cost of removing --replSet for that initial mongod instance, which may or may not be acceptable:
diff --git a/3.4/docker-entrypoint.sh b/3.4/docker-entrypoint.sh
index e1d5642..f4fd398 100755
--- a/3.4/docker-entrypoint.sh
+++ b/3.4/docker-entrypoint.sh
@@ -76,26 +76,45 @@ _mongod_hack_ensure_arg() {
 		mongodHackedArgs+=( "$ensureArg" )
 	fi
 }
-# _mongod_hack_ensure_arg_val '--some-arg' 'some-val' "$@"
+# _mongod_hack_ensure_no_arg '--some-arg' "$@"
 # set -- "${mongodHackedArgs[@]}"
-_mongod_hack_ensure_arg_val() {
-	local ensureArg="$1"; shift
-	local ensureVal="$1"; shift
+_mongod_hack_ensure_no_arg() {
+	local ensureNoArg="$1"; shift
+	mongodHackedArgs=()
+	while [ "$#" -gt 0 ]; do
+		local arg="$1"; shift
+		if [ "$arg" = "$ensureNoArg" ]; then
+			continue
+		fi
+		mongodHackedArgs+=( "$arg" )
+	done
+}
+# _mongod_hack_ensure_no_arg '--some-arg' "$@"
+# set -- "${mongodHackedArgs[@]}"
+_mongod_hack_ensure_no_arg_val() {
+	local ensureNoArg="$1"; shift
 	mongodHackedArgs=()
 	while [ "$#" -gt 0 ]; do
 		local arg="$1"; shift
 		case "$arg" in
-			"$ensureArg")
+			"$ensureNoArg")
 				shift # also skip the value
 				continue
 				;;
-			"$ensureArg"=*)
+			"$ensureNoArg"=*)
 				# value is already included
 				continue
 				;;
 		esac
 		mongodHackedArgs+=( "$arg" )
 	done
+}
+# _mongod_hack_ensure_arg_val '--some-arg' 'some-val' "$@"
+# set -- "${mongodHackedArgs[@]}"
+_mongod_hack_ensure_arg_val() {
+	local ensureArg="$1"; shift
+	local ensureVal="$1"; shift
+	_mongod_hack_ensure_no_arg_val "$ensureArg" "$@"
 	mongodHackedArgs+=( "$ensureArg" "$ensureVal" )
 }
 # TODO what do to about "--config" ? :(
@@ -158,7 +177,11 @@ if [ "$originalArgOne" = 'mongod' ]; then
 		pidfile="$(mktemp)"
 		trap "rm -f '$pidfile'" EXIT

-		_mongod_hack_ensure_arg_val --bind_ip <IP_ADDRESS> "$@"
+		# remove "--auth" and "--replSet" for our initial startup (see https://docs.mongodb.com/manual/tutorial/enable-authentication/#start-mongodb-without-access-control)
+		_mongod_hack_ensure_no_arg --auth "$@"
+		_mongod_hack_ensure_no_arg_val --replSet "${mongodHackedArgs[@]}"
+
+		_mongod_hack_ensure_arg_val --bind_ip <IP_ADDRESS> "${mongodHackedArgs[@]}"
 		_mongod_hack_ensure_arg_val --port 27017 "${mongodHackedArgs[@]}"

 		sslMode="$(_mongod_hack_have_arg '--sslPEMKeyFile' "$@" && echo 'allowSSL' || echo 'disabled')" # "BadValue: need sslPEMKeyFile when SSL is enabled" vs "BadValue: need to enable SSL via the sslMode flag when using SSL configuration parameters"

One concrete side-effect is that if anyone was using rs.initiate() from within /docker-entrypoint-initdb.d, then this would break that (since we've stripped --replSet, and thus cannot initialize a replica set).  Perhaps the fix would be to only remove --replSet if we've got MONGO_INITDB_ROOT_USERNAME and MONGO_INITDB_ROOT_PASSWORD set (similar to other bits of this entrypoint).

For reference, here's that version:
diff --git a/3.4/docker-entrypoint.sh b/3.4/docker-entrypoint.sh
index e1d5642..871afa6 100755
--- a/3.4/docker-entrypoint.sh
+++ b/3.4/docker-entrypoint.sh
@@ -76,26 +76,45 @@ _mongod_hack_ensure_arg() {
 		mongodHackedArgs+=( "$ensureArg" )
 	fi
 }
-# _mongod_hack_ensure_arg_val '--some-arg' 'some-val' "$@"
+# _mongod_hack_ensure_no_arg '--some-arg' "$@"
 # set -- "${mongodHackedArgs[@]}"
-_mongod_hack_ensure_arg_val() {
-	local ensureArg="$1"; shift
-	local ensureVal="$1"; shift
+_mongod_hack_ensure_no_arg() {
+	local ensureNoArg="$1"; shift
+	mongodHackedArgs=()
+	while [ "$#" -gt 0 ]; do
+		local arg="$1"; shift
+		if [ "$arg" = "$ensureNoArg" ]; then
+			continue
+		fi
+		mongodHackedArgs+=( "$arg" )
+	done
+}
+# _mongod_hack_ensure_no_arg '--some-arg' "$@"
+# set -- "${mongodHackedArgs[@]}"
+_mongod_hack_ensure_no_arg_val() {
+	local ensureNoArg="$1"; shift
 	mongodHackedArgs=()
 	while [ "$#" -gt 0 ]; do
 		local arg="$1"; shift
 		case "$arg" in
-			"$ensureArg")
+			"$ensureNoArg")
 				shift # also skip the value
 				continue
 				;;
-			"$ensureArg"=*)
+			"$ensureNoArg"=*)
 				# value is already included
 				continue
 				;;
 		esac
 		mongodHackedArgs+=( "$arg" )
 	done
+}
+# _mongod_hack_ensure_arg_val '--some-arg' 'some-val' "$@"
+# set -- "${mongodHackedArgs[@]}"
+_mongod_hack_ensure_arg_val() {
+	local ensureArg="$1"; shift
+	local ensureVal="$1"; shift
+	_mongod_hack_ensure_no_arg_val "$ensureArg" "$@"
 	mongodHackedArgs+=( "$ensureArg" "$ensureVal" )
 }
 # TODO what do to about "--config" ? :(
@@ -158,7 +177,13 @@ if [ "$originalArgOne" = 'mongod' ]; then
 		pidfile="$(mktemp)"
 		trap "rm -f '$pidfile'" EXIT

-		_mongod_hack_ensure_arg_val --bind_ip <IP_ADDRESS> "$@"
+		# remove "--auth" and "--replSet" for our initial startup (see https://docs.mongodb.com/manual/tutorial/enable-authentication/#start-mongodb-without-access-control)
+		_mongod_hack_ensure_no_arg --auth "$@"
+		if [ "$MONGO_INITDB_ROOT_USERNAME" ] && [ "$MONGO_INITDB_ROOT_PASSWORD" ]; then
+			_mongod_hack_ensure_no_arg_val --replSet "${mongodHackedArgs[@]}"
+		fi
+
+		_mongod_hack_ensure_arg_val --bind_ip <IP_ADDRESS> "${mongodHackedArgs[@]}"
 		_mongod_hack_ensure_arg_val --port 27017 "${mongodHackedArgs[@]}"

 		sslMode="$(_mongod_hack_have_arg '--sslPEMKeyFile' "$@" && echo 'allowSSL' || echo 'disabled')" # "BadValue: need sslPEMKeyFile when SSL is enabled" vs "BadValue: need to enable SSL via the sslMode flag when using SSL configuration parameters"
@@ -214,12 +239,6 @@ if [ "$originalArgOne" = 'mongod' ]; then
 					roles: [ { role: 'root', db: $(jq --arg 'db' "$rootAuthDatabase" --null-input '$db') } ]
 				})
 			EOJS
-
-			mongo+=(
-				--username="$MONGO_INITDB_ROOT_USERNAME"
-				--password="$MONGO_INITDB_ROOT_PASSWORD"
-				--authenticationDatabase="$rootAuthDatabase"
-			)
 		fi

 		export MONGO_INITDB_DATABASE="${MONGO_INITDB_DATABASE:-test}"

(very interested in identifying more edge cases in this version, since it's something I'd be willing to accept as a PR if we can verify that most other currently-working use cases still work properly)

Hi, i think i have the same issues here, still very new to both docker and mongodb, i have a replica set all up and running, but i was going to enable auth mode on the replica set, so was just looking for the config file so i could add the line in to set the Key but there and switch it on, but could not find the conf file and i'm kinda stuck. not sure how to turn this on. any help / ideas ?

@tianon - So, just to confirm, was the merge that closed this issue supposed to support replica sets with auth, or is the solution proposed by @ft0907 the recommended approach?
I have been able to get this to work when brewing my own similarly to the comment immediately above, but have not been able to do it with just additions to /docker-entrypoint-initdb.d as per the documentation.  See https://github.com/PeterParker/replica-set-mongo-in-docker-with-auth as an example of this.

@PeterParker

the initial user must be configured without --replSet, but then the daemon must be restarted with --replSet in order to run rs.initiate().

The PR that closed this issue was not to let /docker-entrypoint-initdb.d do an rs.initiate with --auth (or the user/pass env vars).  You need to have the --replSet flag set and the PR removed it for the temporary mongod that runs during /docker-entrypoint-initdb.d if you are trying to create a user. The point of the PR was to remove options that don't work when both are turned on with the empty database.
Even if you do an rs.initiate with the initdb.d scripts, it will not be correct: https://github.com/docker-library/mongo/issues/339#issuecomment-485578503.  (though it seems you have found a clever hack around that with extra_hosts: ["database-no-auth:<IP_ADDRESS>"] in your compose file :heart_eyes:)

@yosifkit - Okay, thanks, that clarifies things.  It looks like a custom entrypoint is the way forward.
Also, by the way, many thanks for the work you and others have done on these containers -- they're awesome.

Just for clarification, how would I go about creating a replicaset that also has a user in it?
# docker-compose.yaml
...
 mongo_1:
    image: mongo:latest
    deploy:
      replicas: 1
      update_config:
        parallelism: 1
      restart_policy:
        condition: on-failure
    environment:
      MONGO_INITDB_ROOT_USERNAME: "{{ mongo_root_username }}"
      MONGO_INITDB_ROOT_PASSWORD: "{{ mongo_root_password }}"
    volumes:
      - {{ mongo_1_data_dir }}:/data/db
      - {{ token_dir }}:/tokens
    command: "mongod --replSet {{ mongo_replica_set_name }} --keyFile /tokens/{{ mongo_replica_set_name }}.key"
    ports:
      - "{{ mongo_1_port }}:27017"

This creates a replica set with no users. I'm slightly confused at how to go about getting this to work.  Can someone assist?

@coltenkrauter this might help

I've just spent 3h looking around to find out why this container cannot be setup correctly with authentication and a replicaSet. That's why...
You may want to put a caveat in your documentation 🙃

This is how I managed to create a replicaSet with an user on it:
The variables MONGO_INITDB_ROOT_USERNAME and MONGO_INITDB_ROOT_PASSWORD are set in the environment file
services:
  mongodb:
    image : mongo:5.0.5
    container_name: mongodb
    command : [ "/bin/bash","-c", "chmod +x /conf/mongo-init.sh && /conf/mongo-init.sh"]
    environment:
      - PUID=1000
      - PGID=1000
    restart: always
    env_file:
      - ${ENVFILE}
    networks:
      - default
    ports:
      - 27017:27017
    volumes:
      - database:/data
      - mongo-conf:/conf

--- mongo-init.sh
#!/bin/bash

logfile="mongo.out"
expected_message="Waiting for connections"

chmod 400 /conf/mongodb.key
mongod --quiet --logpath $logfile --replSet rs0 --bind_ip_all --port 27017 --dbpath /data/db/ --shardsvr --auth --keyFile /conf/mongodb.key &

while [ ! -f "$logfile" ]
do
    sleep 1
done

while :
do
    if grep -q "$expected_message" "$logfile"; then
        echo "Found the expected message: $expected_message"
        break
    fi
    sleep 1
done

mongo --eval '
  rs.initiate({
    _id: "rs0",
    version: 1,
    members: [
      { _id: 0, host: "localhost:27017" }
    ]
  })
'
sleep 5
mongo admin --eval "db.createUser({ user: '${MONGO_INITDB_ROOT_USERNAME}', pwd: '${MONGO_INITDB_ROOT_PASSWORD}', roles: [ { role: 'root', db: 'admin' } ] })"

tail -f $logfile


Update storage sample for Spring Boot 3

Purpose
Update storage samples to provide a doc CONVERT_SAMPLE_TO_USE_SPRING_BOOT_3.md to convert the sample to run with Spring Boot 3.
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test
Refer to the doc CONVERT_SAMPLE_TO_USE_SPRING_BOOT_3.md to run with Spring Boot 3.
What to Check
Success to run the sample.
Other Information

These examples do not require a separate file to illustrate running with Spring Boot3.


Fix/Update for new IM versions

This makes the plugin work with newer ImageMagick version.
I needed to use this plugin as Sharp js library was not working on my old PC server CPU (AMD Phenom II), which did not support some instructions, and crashed during image transformations.

@fraunos ty published 3.0.0


update cache dropping doc

get rid of a now-unnecessary step
explain better how to set labels for kernel cache dropper

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ef4b663). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #650   +/-   ##
=========================================
  Coverage          ?   53.48%
=========================================
  Files             ?        8
  Lines             ?      316
  Branches          ?        0
=========================================
  Hits              ?      169
  Misses            ?      147
  Partials          ?        0

Flag
Coverage Δ

gha
53.48% <0.00%> (?)

python-3.9
53.48% <0.00%> (?)

unit
53.48% <0.00%> (?)

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

Continue to review full report at Codecov.

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

can someone please merge this?   I can't do it.


Slow connect if broker is bad

I am developing a sketch on ESP8266 board. I wanted to make a program for LED driver with a physical switch, which could operate with the broker.
The problem is such that if the broker is available, the module is connected instantly (13ms). But if the broker is not available, the module tries to connect approximately 6 seconds. Accordingly, at this time the module does not respond to user input (e.g. physical switching switch).
Changing the MQTT_SOCKET_TIMEUT does not solve anything. Tried even assign a value of 0 - no result.
Below I put part of the message from the module to the Monitor port. Connection check occurs every 20 seconds.

Time stamp reconnect server: 133440
WiFi connected
IP address:
<IP_ADDRESS>
Time stamp to try connect Mosquitto: 133440
Try Mosquitto to connect...
Time stamp the good connect Mosquitto: 133453
Mosquitto connected

Time stamp reconnect server: 113439
WiFi connected
IP address:
<IP_ADDRESS>
Time stamp to try connect Mosquitto: 113439
Try Mosquitto to connect...
Time stamp to failed connect Mosquitto: 119702
failed connect Mosquitto, rc=-2
What can I change in the library, in order to reduce the connection time if the broker Mosquitto is not available?

What network client are you using with the PubSubClient? All of that time will be spent within the network client trying to establish a connection.
There may be a configuration option in that network client to decrease whatever timeout it is using... but that is outside this library's control.

I use library <ESP8266WiFi.h>. Wi-Fi network. Router DIR-615. The network is fast. But the network is working. Disables only broker. Because I want to install it on a separate router and imitate him off.
So you want to say that the problem is in the client network library?
Hmm ..
Ie greatest time slot occurs at this stage:
https://github.com/knolleary/pubsubclient/blob/master/src/PubSubClient.cpp#L123

Yes, that line is where the PubSubClient is waiting for the network client to create a TCP connection.

Many thanks!


bug: Log is spammed by messages that contain the entire README file.

Cody Version
baed3871ad6d34b41e2a2c56258ccf4a52ae764e
IDE Information
IntelliJ IDEA 2022.1 (Community Edition)
Build #IC-221.5080.210, built on April 12, 2022
Describe the bug
I see 19 warnings containing the exact same text, which seems to be a HTML rendering of README.md. This adds up to several thousand lines of logs to scroll through.
Expected behavior
No warnings or at least shorter messages
Additional context
2024-05-16 05:46:15,064 [1284151]   WARN - #c.i.BundleBase - ignore extra mnemonic in
Sourcegraph Cody + Code Search

Use Sourcegraph’s AI assistant Cody and Sourcegraph Code Search directly from your JetBrains IDE.

Cody is an AI coding assistant that can write, understand and fix your code. Cody is powered by Sourcegraph’s code graph, and has knowledge of your entire codebase.

With Code Search, you can search code across all your repositories and code hosts—even the code you don’t have locally.

Cody for JetBrains IDEs is in beta. We’d love your feedback!

Cody Features

Autocomplete: Cody writes code for you

Cody autocompletes single lines, or whole functions, in any programming language, configuration file, or documentation. It’s powered by latest instant LLMs for accuracy and performance.

Chat: Ask Cody about anything in your codebase

Cody understands your entire codebase — not just your open files. Ask Cody any question about your code, and it will use Sourcegraph's code graph to answer using knowledge of your codebase.

For example, you can ask Cody:

"How is our app's secret storage implemented on Linux?"

"Where is the CI config for the web integration tests?"

"Write a new GraphQL resolver for the AuditLog"

"Why is the UserConnectionResolver giving an "unknown user" error, and how do I fix it?"

"Add helpful debug log statements"

"Make this work" (seriously, it often works—try it!)

Built-in commands

Cody has quick commands for common actions. Select the commands tab or right-click on a selection of code and choose one of the Ask Cody > ... commands, such as:

Explain code

Generate unit test

Smell code

We also welcome pull request contributions for new, useful commands!

Swappable LLMs

Cody supports multiple LLMs including Anthropic Claude, Claude 2, and OpenAI GPT-4/3.5 models, with more coming soon. Cody Pro users can swap models used for chat on-demand.

Usage

This plugin works for all Cody plans, including Cody Free, Cody Pro, and Cody Enterprise.

You can find detailed information about Cody's available plans on our website.

Programming language support

Cody works for any programming language because it uses LLMs trained on broad data. Cody works great with Python, Go, JavaScript, and TypeScript code.

Code graph

Cody is powered by Sourcegraph’s code graph and uses context of your codebase to extend its capabilities. By using context from entire repositories, Cody is able to give more accurate answers and generate idiomatic code.

For example:

Ask Cody to generate an API call. Cody can gather context on your API schema to inform the code it writes.

Ask Cody to find where in your codebase a specific component is defined. Cody can retrieve and describe the exact files where that component is written.

Ask Cody questions that require an understanding of multiple files. For example, ask Cody how frontend data is populated in a React app; Cody can find the React component definitions to understand what data is being passed and where it originates.

Cody Enterprise

Cody Enterprise is able to retrieve context from your entire remote codebase using Sourcegraph search. This gives Cody the ability to understand and answer questions about all of your code, even the repositories that don't live on your local machine.

Contact us to set up a trial of Cody Enterprise. If you’re an existing Sourcegraph Enterprise customer, contact your technical advisor.

Feedback

Issue tracker

Discord chat

Twitter (@sourcegraph)

License

Cody's code is open source (Apache License 2.0).

Code Search features

Search with Sourcegraph directly from inside the IDE

Instantly search in all open source repos and your private code

Peek into any remote repo in the IDE, without checking it out locally

URL sharing features

Create URLs to specific code blocks to share them with your teammates

Open your files on Sourcegraph

2022.1 is still ~6% of JetBrains users per https://plugins.jetbrains.com/docs/marketplace/product-versions-in-use-statistics.html so it isn't worth dropping support for it to silence this error.
On the other hand, it's not worth ameliorating, it will go away when we roll to 2022.2. So I'm going to close this, there's nothing actionable for us here.


Use case with Chrome extensions

I have a Chrome extension that I was hoping to use Chromy to load and test. However, since Chromy loads a fresh copy of Chrome on startup (no one signed in, etc.), this appears to be impossible.
Would it be possible to load an extension inside whichever instance Chromy runs?

Hi @shanemcgraw,
I've published new version to support loading extensions now!
Please try it out!
Here is an example.
const Chromy = require('chromy')

const chromy = new Chromy({
  visible: true,
  userDataDir: "/Users/YourName/Library/Application Support/Google/Chrome/",
  enableExtensions: true
})

chromy.chain().goto('https://example.com').end()

The new option enableExtensions enable loading extensions.
Generally this option should be used with userDataDir option because extensions are installed in userDataDir.

@dotneet   FANTASTIC!  I will add this to Backstop ASAP.

Thank you @dotneet!!

I'm trying to make it work, but unfortunately Flash extension in my Chrome is not running. Has something changed since this issue was resolved?
This config:
let chromy = new Chromy({
  visible:true,
  enableExtensions: true,
  launchBrowser: false,
  userDataDir: '/Users/seleckis/Library/Application Support/Google/Chrome/'
});

Opens Chrome, but flash extension is not running:
screen shot 2018-09-21 at 10 01 02


BufReader.read_line captures only 4096 bytes

original issue: https://github.com/rust-lang/futures-rs/issues/2175
might be related to #148.
original text
Crates used:
futures-util = 0.3.5
smol = 0.1.11
tracing = 0.1.15
tracing-subscriber = 0.2.5

I have some code like this:
    tracing_subscriber::fmt::init();
    smol::run(async {
        use futures_util::io::{self, AsyncBufReadExt, AsyncReadExt, AsyncWriteExt};
        let sess = /* Session Handle, implements AsyncRead + AsyncWrite */;
        let (srd, mut swr) = sess.split();

        // Create async stdin and stdout handles.
        let mut stdin = futures_util::io::BufReader::new(smol::reader(std::io::stdin()));
        let mut stdout = smol::writer(std::io::stdout());

        futures_util::future::try_join(
            async move {
                let mut line = String::new();
                while stdin.read_line(&mut line).await.is_ok() {
                    tracing::debug!("line length = {}", line.len());
                    swr.write_all(line.as_bytes()).await?;
                    line.clear();
                    swr.flush().await?;
                }
                Ok(())
            },
            io::copy(srd, &mut stdout),
        )
        .await
        .expect("I/O error");
    });

No matter how much input I provide on stdin on a single line, according to the line length =  debug statement it only captures at most 4096 bytes of it, and drops the rest. This is unexpected.

In the original issue, it was suspected that this might be caused by an alternation between Ok(n) with n > 0 and Ok(0), which shouldn't normally happen, as Ok(0) is interpreted as EOF.

ok, this bug is weird, and probably caused by the interaction of many layers of buffering and caching... It seems to depend on the terminal emulator I used to test this (it only returns up to 4096 bytes with each read)...

closing this, as it is a bug in xterm.


Implement HTL Sensor to support HTL language

Implement a new sensor that will support writing new rules for HTL language. Consider list of following rules when making decision on how the analysis should be performed:
< list of rules to be updated here >
(parser / regular expressions / jsoup / ?)

Hello @chutch ,
I would like to propose few rules that I recognize to valuable:

error: data-sly-attribute is used to render "javascript" attribute (e.g. onclick, onmousemove, etc), such attribute can not be processed via data-sly-attribute - nothing will be rendered
warning: data-sly-unwrap inside sly tag. In such case data-sly-unwrap is simply ignored as it is redundant
warning: Redundant HTL expression within data-sly-use or data-sly-include.
HTL expression allows to pass some additional parameters but in case if no parameters passed it only brings redundant complexity and overhead for the runtime

For the example for those you could refer to AEM Tools plugin
BTW, to implement all aforementioned no grammar parsing is required (jsoup or regular expressions would suffice)
Cheers,
Dmytro

Hi @deeprim, thanks for the list of potential rules, looks good.
Yeah, I was considering different options on how to approach new Sensor for HTL language and I'm torn apart, which way to choose. I'll keep that in mind. Have you considered contributing?


Issue with deployment of List View Command Set Extention

Category

[ ] Question
[ ] Typo
[X] Bug
[ ] Additional article idea

Expected or Desired Behavior
When I bundle and deploy a List View Command Set Extension that checked out in test that it appears on a site it has been deployed on.
Observed Behavior
I built a custom solution for making a JSON Post to a MS Flow using the List View Command Set Extension. While it worked during gulp serve testing when I deployed to my tenant it would not appear.
I then back tracked and built out the basic extension based on the MSFT tutorial: https://docs.microsoft.com/en-us/sharepoint/dev/spfx/extensions/get-started/building-simple-cmdset-with-dialog-api
While it also worked fine during gulp-serve testing it did not appear when packaged and deployed to a site.
Steps to Reproduce
I have uploaded the project to Git Hub: https://github.com/john88benson/listViewCmdExtension
It is the "OOB" version of the ListViewCmdset stub created using the yo @microsoft/sharepoint (I updated my generator before hand: version 1.4.1).  After confirming it functioned during gulp serve I ran a gulp clean, gulp bundle --ship, gulp package-solution --ship then loaded the package into my App Catalog.  I added the App in the Site Contents for a site then checked for it in a Document Library but no dice.
Thanks for your contribution! Sharing is caring.

You can close this item, it is not a bug.  It's not obvious from the gulp serve testing but the RegistrationId in elements.xml specifies what type of list it appears on when deployed.  Once I updated the Reg Id to 101 it started appearing in the Doc Library as expected.
It may be worth making this a little more explicit in the CmdSet Tutorial since the early steps show the extension working on a Doc Library but it will display only on generic Lists unless the Reg Id is updated.
Thanks,
-JB


Is Box really gone in IDL 2.0?

Hi! I was under the impression @box doesn't get added in IDL 2.0. This comment on the BoxTrait class seems to confirm that:
https://github.com/smithy-lang/smithy/blob/e9cca364b7a4b0304510ebc9155e6bb44dbae5b5/smithy-model/src/main/java/software/amazon/smithy/model/traits/BoxTrait.java#L26-L27
but it's still being added to my simple shapes in IDL 2.0:

//> using scala "2.13.10"
//> using dep "software.amazon.smithy:smithy-model:1.45.0"
import software.amazon.smithy.model.Model
import software.amazon.smithy.model.selector.Selector
import software.amazon.smithy.model.shapes.ShapeId
import scala.jdk.CollectionConverters._
import software.amazon.smithy.model.transform.ModelTransformer
import software.amazon.smithy.model.shapes.ServiceShape
import software.amazon.smithy.model.shapes.OperationShape

val r = Model
  .assembler()
  .addUnparsedModel(
    "test.smithy",
    """$version: "2"
      |namespace test
      |
      |boolean MyBoolean
      |""".stripMargin,
  )
  .assemble()
// r: software.amazon.smithy.model.validation.ValidatedResult[Model] = software.amazon.smithy.model.validation.ValidatedResult@348fda3d

val m = r.unwrap()
// m: Model = software.amazon.smithy.model.Model@ca2e6244

m.expectShape(ShapeId.from("test#MyBoolean"))
  .getAllTraits()
  .asScala
  .toList
  .map(_._1)
  .foreach(println)
// smithy.api#box

Runnable snippet here
Is this expected? (i.e. the trait is still allowed in the model, but not in the IDL) - if so, should BoxTrait even be @Deprecated?

It's not allowed in IDL 2.0 models, but is a synthetic trait added to the semantic model and never serialized. It's added to the semantic model to provide a compatibility bridge between Smithy 1.0 and 2.0 for users of this library that expect a Smithy 1.0 semantic model. We marked the Java trait for box as @Deprecated to discourage people from continuing to check for it in code. Is the fact that it's marked as @Deprecated in Java causing an issue?

Nope, just wanted to clarify for the sake of knowing which traits can "reasonably" show up in a model :)
that explains it, thanks


improve semantics & accessibility

[x] add proper <nav>, <article> elements where needed
[x] use <div hidden> instead of display: none;
[x] use <time> for dates
[x] use roles
[x] improve disabled css: separate EN from HU version

completed, see commits addressing issue #7


foundationdb60: 6.0.15 -> 6.0.17

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build foundationdb60 python2Packages.foundationdb60 python3Packages.foundationdb60

@GrahamcOfBorg build foundationdb60 python2Packages.foundationdb60 python3Packages.foundationdb60


usage of _include and _revinclude search params

Any update on how to use Search params with _include and _revinclude params...??

Hi @anandanguru,
In the future, please ask questions on Stack Overflow with either the tag javascript-fhir or node-fhir-server-core.  It makes it easier for others with the same question to find the answer's and it makes issues better for tracking bugs and feature requests.
It's on our roadmap to add back some support for common parameters to all endpoints (not sure the current ETA since we have some other deadlines we need to meet). If you would like to submit a PR we would appreciate it.  In the meantime, you can modify any of the parameter files and add a section in for this. At that point, you just write your own query.
For example, add an entry to the src/server/standards/4_0_0/arguments/patient.arguments.js:
_include: {
  type: 'string',
  fhirtype: 'string',
  // The extra fields are not necessary, just there to help for building queries if needed
},

And it will now work, if you want it on all resources, add it to the src/server/standards/4_0_0/arguments/resource.arguments.js file.

Closing for now, if you are having trouble developing an implementation, feel free to post a question on StackOverflow and we would be glad to help.


Adding Contributing md file

A CONTRIBUTING md which will help in better contributions from new users

Hey @amitkumar-y thanks for opening this PR! But unfortunately Catppuccin does not use per repository CONTRIBUTING.md but rather uses organisation wide files.
Since this PR is invalid, I'll go ahead and close this.


NumPy 2.0 Incompatibility Issue

Issue
Robyn is currently not compatible with NumPy 2.0. I encountered this issue while trying to run Robyn, and I had to downgrade NumPy to an older version to get it working.

Same problem to me. This is the new code I replaced in install_nevergrad.R to make it work!
conda_create(envname = "r-reticulate", packages = "python=3.9")
use_condaenv("r-reticulate")
conda_install("r-reticulate", "numpy==1.26.4", pip = TRUE)
conda_install("r-reticulate", "nevergrad", pip = TRUE)

To make it more sustainable in env conflicts, probably we could keep the version of python libraries in requirements.txt, and using conda/pip on CLI to install it into the r-reticulate conda env

This approach would definitely help to mitigate conflicts in the future.
Additionally, I wanted to share that a fix has been deployed to the main branch of Reticulate. We can pull the latest changes from the main branch or wait until a new version is released. This should resolve the compatibility issue with NumPy 2.0.


How to reproduce result of Fig.1 in the paper?

How to reproduce the result of Fig.1 , which illustrates the loss(test error) as function of network weights in a two-dimensional subspace? Is it using PCA to get two main dimensions of weights?

Please see footnote 1 on page 2 of the camera-ready version of the paper:
http://auai.org/uai2018/proceedings/papers/313.pdf
There we tried to clarify the exact procedure for making the loss and test error surface visualizations.

I will close the issue for now, but I will be happy to answer if you will have further questions about those figures.

@izmailovpavel Hi and thank you for the great work, I've been implementing SWA in my research project and the results are great. I just have a few questions regarding the illustrations.

Are the weight vectors literally the weights (not biases) from a single linear layer of a network or are they the concatenation of the entire model?
Would you be comfortable providing the snippet of code you used to make the figures? (Does not need to be functional/polished or commented). Just so I can double check my own implementation.

Thank you for what you have done for the community.

Hi @roderickObrist, good to hear SWA is working well for you :) To make the figures we treat all the parameters of the network as one large (say 10 million-dimensional) vector. It includes both the biases and the weights. Say we have a total of D parameters. We treat the whole parameter space as just R^D. For each visualization we then pick three vectors v1, v2 and v3 in this R^D parameter space. These typically correspond to the weights of some networks, like SGD iterates from different iterations. Then, we construct the unique 2-d plane (affine subspace) that passes through these three vectors. We then plot the loss in the plane restricted to this 2-d subspace.
To answer your questions:

Weights include both weights and biases, and they are not from a single layer. This is the full vector of all the network's parameters.
We have a public implementation of a very similar visualization for our other paper here: https://github.com/timgaripov/dnn-mode-connectivity/blob/master/plane.py. I believe you would need to change this part here https://github.com/timgaripov/dnn-mode-connectivity/blob/master/plane.py#L96-L101, and load the weights of three networks v1, v2, v3 in the w list.

Thank you kindly, I will implement this in my own project over the next few days.


add nginx configurations for the odf-console

Changes required as per BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2139785
Resolves 4 issues:

BZ#2139785 itself: where on disabling ipv6 address on the cluster nodes, odf-console pod was resulting in error state (https://bugzilla.redhat.com/show_bug.cgi?id=2139785#c10)
There was CVE issue raised by IBM team where port 8080 was stated to be an insure port (https://bugzilla.redhat.com/show_bug.cgi?id=2166417). Port 8080 is present by default in the nginx configuration that we use for our pod (under /etc/nginx/nginx.conf: https://pastebin.com/ksX21dsr) and we did not had any control over that config earlier. Now all the nginx configurations (nginx's default and custom config for odf-console) are added via a ConfigMap created by odf-operator, so everything is under our control.
Also, now removed the port 8080 which was never needed and never used.
Earlier nginx configurations were added during build time, now they are added and can be easily updated during runtime.

There is one more CVE issue as per which we need to add readOnlyRootFilesystem: true spec for the odf-console pod, but on adding it is causing the UI's pod to go into an error state because nginx needs to create some files in pod's filesystem for its functioning but don't have access to do so (if we set this attribute), hence pod is never getting into "Running" status.
This PR adds the foundation for enabling us to be able to add the changes to fix this CVE (will send a follow-up PR for that).

Possible improvement ?
Currently for BZ#2139785 (if this PR gets merged), once odf-console pod errors due to ipv6 address issue, user will have to perform following operations (we will keep documentation team in loop):

Go to ConfigMap: odf-console-nginx-conf (introduced as part of this PR).
Comment out line listen       [::]:9001 ssl; --> # listen       [::]:9001 ssl;.
Restart the odf-console pod (just delete it) and Refresh.
Once done they will be able to see the UI once again as the pod will now be in running state.

We can automate this process so that user don;t have to take manual steps to resolve the issue.
Pro: no manual intervention needed.
Cons:

Not a priority (P0) use-case or something which is generally faced by users (rare).
odf-operator will need to introduce new a controller to keep a watch on resources which has nothing to do with ODF or storage, but are purely infra related.
This BZ is different from IPv4/IPv6 dual-stack networking where we can just watch over network.config.openshift.io cluster CR (https://docs.openshift.com/container-platform/4.12/networking/ovn_kubernetes_network_provider/converting-to-dual-stack.html)...
here we are disabling ipv6 addressing on the node's kernel level (https://access.redhat.com/solutions/5513111)... not sure if it is good idea to keep a watch for such changes.

/test odf-operator-e2e-aws

/test odf-operator-e2e-aws

I know the problem, I will get it fixed once #304 is merged

/test odf-operator-e2e-aws

/test odf-operator-e2e-aws

Need to update build Dockerfile once this is merged...

@bipuladh PTAL.

/lgtm

/assign @iamniting

/cherrypick release-4.13


Feature/swift 5 migration

Update to be in swift 5

Can this be committed as it resolve the Swift 5 errors


Need to get braket no in hostname

Need to get braket no in hostname
It shows up LL CSG#1 NAME VS NAME
Need to figure out if there is away to tweak that. Looks like get 5 determines that.

Yeah, this will be get5 - via get5-cli


format generated story modules

Pull Request Test Coverage Report for Build f1026e0a7bee0ec8a124f927199c2b535d06f753-PR-312
Details

4 of 6 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 89.764%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/mix/tasks/dog.gen.stories.ex
4
6
66.67%

Totals

Change from base Build 144b6eac92bb2fef6f28425e098a6e025d153e6f:
0.3%

Covered Lines:
684

Relevant Lines:
762

💛  - Coveralls


Spaces in default value

I would like to set a default value containing spaces like in - . It does work:
--name-sep=<sep>  name field separator [default:  - ]

But it is not so pleasing to the eye and prone to bad interpretation. I don't know what is the standard here, but I would imagine accepting a quoted default value, as in:
--name-sep=<sep>  name field separator [default: " - "]

Could be one way to go. Or am I missing something ?

This seems like a reasonable idea, but I would want to get it right, and there are a few rough cases:

how to do escaping. For instance I might want the string Dwayne "The Rock" Johnson. This should be easily explainable.
how to deal with escaping. eg if the string contains a ] then the current re.findall(r"\[default: (.*)\]") would fail. Of course, this also currently fails if you want a ] in your default, but currenlty it is a bit more obvious that this should fail. Once you add quoting, I think we would need this to work.

Once we figure those out, and add tests and docs, then it sounds great!

Indeed I see the Pandora's box of string quoting.
The way I implemented in #47 cowardly avoid having to deal with it. It works as previously: re.findall(r"\[default: (.*)\]") but it just adds a check after the match is successful. If it is the case that the match starts and ends with the same quote character (both single or both double) than it strips them and that's it.
This example just works and result in Dwayne "The Rock" Johnson:
--main-actor=<name>    name of the movie's main actor  [default: Dwayne "The Rock" Johnson]

This might not be intuitive or elegant but it is easy to use and always has a solution, even for strings that do start and end with quotes:

enclose it with "the other" quote character [default: '"The Rock"']
enclose it with "the same" quote character [default: ""The Rock""]

However, confusion may arise when users of the library might not expect quoting to be implemented and, expecting "The Rock", writes this:
--main-actor-nickname=<name>   name of the movie's main actor  [default: "The Rock"]

One way around it would be to

specifically enable this feature when needed: arguments = docopt(__doc__, quoted_defaults=True)
document it properly and expect developers using the library to apply extra care when dealing with quotes anyway

As for string containing ], because .* is greedy, it doesn't pose any problem I reckon. The following example already works as expected (resulting in []) with or without quoting.
--group-enclosure=<pair>             open/close grouping characters [default: []]

Sorry for taking so long, but I think this has a path to life.

specifically enable this feature when needed: arguments = docopt(doc, quoted_defaults=True)

This is a reasonable idea, but I want docopt to be as opinionated and un-optioned as possible. Let's just go with the one least-bad behavior. I think it can be explained as
Quotes are not required (anything after `default: ` is taken as the value), but they *can* be used to make whitespace or other characters more clear.
If a default value is quoted (after stripping whitespace, begins and ends with `"` or begins and ends with  `'`), then those quotes are removed:
- `default:  leading space` -> ` leading space`
- `default: ' leading space'` -> ` leading space`
- `default:                 "Dawyne "The Rock" Johnson"           ` -> `Dawyne "The Rock" Johnson`

OK, I think that expected behavior you laid out seems reasonable. If you implement these test cases, I will accept #47:
[
    pytest.param(' leading space', ' leading space', id="leading_space_unquoted"),
    pytest.param('" leading space"', ' leading space', id="leading_space_quoted"),
    pytest.param(' "whitespace before"', "whitespace before", id="whitespace_before"),
    pytest.param('"whitespace after" ', "whitespace after", id="whitespace_after"),
    pytest.param('Dawyne "The Rock" Johnson', 'Dawyne "The Rock" Johnson', id="basic"),
    pytest.param('"Dawyne "The Rock" Johnson"', 'Dawyne "The Rock" Johnson', id="nested"),
    pytest.param('"Dawyne \'The Rock\' Johnson"', 'Dawyne "The Rock" Johnson', id="nested_mixed"),
    pytest.param('"The Rock"', "The Rock", id="unneeded_quotes"),
    pytest.param('"The Rock', '"The Rock', id="leading"),
    pytest.param('The Rock"', 'The Rock"', id="trailing"),
    pytest.param('"Dawyne "The Rock" Johnson', '"Dawyne "The Rock" Johnson', id="leading_with_inner"),
    pytest.param('Dawyne "The Rock" Johnson"', 'Dawyne "The Rock" Johnson"', id="trailing_with_inner"),
    pytest.param('"The Rock\'', '"The Rock\'', id="mixed"),
    pytest.param('[]', "[]", id="unquoted_brackets"),
    pytest.param('"[]"', "[]", id="quoted_brackets"),
    pytest.param('"]"', "]", id="quoted_trailing_bracket"),
    pytest.param('"]"', "]", id="quoted_trailing_bracket"),
]
...
doc = "default: " + inp

Please note the whitespace_before and whitespace_after cases, we haven't talked about that yet. I think if there is some sneaky extra wuitespace before/after a quote, it should get stripped, but I'm open to counterarguments.
If you can think of other ones, please add them!


fix(events): mark properties as object rather than string

Per docs it is an object that can contain value as string or number, and this will match in the same as in EventInput

Thank you @vhashimotoo for this pull request!
It looks good, it will part of the next release.


Incorrect off-topic detection for arithmetic

GitHub Copilot
Hi @AuracleTech, how can I help you?

AuracleTech
On my selection why does copilot recommends this : cost / (2.0 * ti.rows as f32)
instead of this : cost / ti.rows as f32

GitHub Copilot
Sorry, but I can only assist with programming related questions.

???

@AuracleTech  thanks for your patience. We've deployed an update that hopefully addresses this. We're still working on better understanding of context but please let us know if you're still hitting these issues!

Closing for now, as a new offtopic detection got deployed and your provided questions pass. Please file new issues for follow-up reports.


Progress messages are shown on multiple lines after upgrade to 4.39.0

Bug report

What is the current behavior?
After upgrading to from 4.38.0 to 4.39.0 the progress output is no longer shown on a single line but on multiple lines as visible here:

In the previous version the progress output was repeatedly (just a CR) shown on a single line that got overwritten on each step and now each step is shown on a separate (LF) line.
If the current behavior is a bug, please provide the steps to reproduce.

Just using the --progress option should be enough.
What is the expected behavior?
It's just a nuisance but having back the single line output of the progress messages not to clutter the screen would be nice.

Other relevant information:
webpack version:  4.39.0
Node.js version: 12.7.0
Operating System: Windows 7
Additional tools:

Which terminal?

ConEmu 190526

Could you try this PR #9513 if it fixes the problem?
add this to your package.json to test it: "webpack": "webpack/webpack#bugfix/terminal-length"

Wow, that's quick!
Yes, this PR fixes the problem!

I'm on that 4.39.1 patch release and still seeing these weird output things.

Could you file a new issue? Add information about which terminal is used.

With 4.39.1 I also see the extended progress output with iTerm and Terminal on macOS Mojave running zsh


ENH: hierarchy: add to_newick_tree function (#8274)

Reference issue

Closes #8274
What does this implement/fix?

From a linkage representing a hierarchical clustering, produce a string in Newick format. Complements to_tree and dendrogram in a sense - named it to_newick_tree.
Additional information

Context I found myself building a hierarchical tree and wanting a Newick output, for
manipulation/visualisation in ITOL.
Found #8274 and thought I'd put in a PR.
Implementation The proposed code is iterative, not recursive as in the issue.
With passing unit tests. I also built the documentation, and it looked alright to me ;)
Note: I also removed two verifications in existing function to_tree, as they are already covered by ìs_valid_linkage I believe (_check_hierarchy_uses_cluster_before_formed(Z))

This is useful!


cmd/go: script test hanging with blocked output pipes

greplogs -l -e 'goroutine \d+ \[chan receive, \d+ minutes\]:\n(?:.+\n\t.+\n)*cmd/go_test\.waitOrStop' --since=2022-01-01
2022-06-06T15:42:19-47e34ca/windows-amd64-longtest
2022-05-26T22:25:23-55c42c2/windows-amd64-longtest
2022-05-20T23:37:43-0a1a092/linux-amd64-longtest
2022-05-20T23:37:43-0a1a092/windows-amd64-longtest
2022-05-20T23:05:38-c3470ca/linux-amd64-longtest
2022-05-20T23:05:38-c3470ca/windows-amd64-longtest
2022-05-20T22:30:37-2b0e457/linux-amd64-longtest
2022-05-20T22:30:37-2b0e457/windows-amd64-longtest
2022-05-20T21:54:20-b580670/linux-amd64-longtest
2022-05-20T21:54:20-b580670/windows-amd64-longtest
2022-05-20T21:32:57-7ec6ef4/linux-amd64-longtest
2022-05-20T21:32:57-7ec6ef4/windows-amd64-longtest
2022-02-03T22:55:24-248ad85/netbsd-amd64-9_0

This is some script test hanging while waiting on the subprocess's output pipes to close, with the output pipes stuck open.
This is quite possibly a bug in the go command, but it will be much more straightforward to diagnose once #50436 is implemented. (A workaround in the interim is possible but more complex, because we would either have to run our own pipe-copying goroutines or direct all of the stderr and stdout output pipes for script tests to files.)
I plan to implement #50436 for Go 1.20, so milestoning to Go 1.20.

Duplicate of #50187.


Avoid redundant change event on load

Detecting input changes after page load can be useful for several reasons;
in these cases, the initial "change" event fired by salmonella widget is not desirable.

Thanks!


How to get id of inserted values

What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
Which database and its version are you using?
Last version of  gorm and postgresql
Please provide a complete runnable program to reproduce your issue.
type AttachmentsNew struct { 	gorm.Model 	id intgorm:"AUTO_INCREMENT"	ownerId int 	hashkey stringgorm:"size:64"	time int 	fileInfo string 	filepath string 	doctype stringgorm:"size:15"`
}
Database.Create(&AttachmentsNew{ownerId: ownerId, hashkey: "test", time:0, fileInfo:file.oldname, filepath:file.path, doctype:file.documentType})`
How i can get id of inserted value?

row := new(Article)
d := Gorm.Create(article).Scan(&row)
if d.Error != nil {
return 0, d.Error
}
return row.Id, nil

undefined Gorm

So, what is article?

My ID is autoincrement.
Your code return ID - 0, buy ID is 6;

Gorm automatically inserts the ID into the struct.
a := &AttachmentsNew{ownerId: ownerId, hashkey: "test", time:0, fileInfo:file.oldname, filepath:file.path, doctype:file.documentType}
Database.Create(a)
// a.id should be populated

Populated means extended?

@DuckerMan yes it means "filled".

So, why i should create variable that point to my structure?

so you can get use the ID after the creation. Otherwise I don't know any other method to achieve this.

i meet the same question, and i just can't get the id.

how can i solve it ? i tried the code like the pic.
please, is there anyone can help me?


[open62541] add amalgamation feature

required by qt6

Hi @Neumann-A
Thanks for your PR.
Have you tested this new feature amalgamation?

@NancyLi1013 the feature is used in #14333. One of the qt6 modules requires it.

LGTM now, thanks for your PR @Neumann-A.


"Delete after send" not respected in ODK-Collect

Originally reported on Google Code with ID 1132
What steps will reproduce the problem?
1. Fill out form
2. Finalize
3. wait

What is the expected output? What do you see instead?

If the "delete after send" option works as expected, one should not find the form in
the "delete saved forms" list.  The finalized form is listed with the sent date (and
appears in ODK-Aggregate) suggesting that the form was properly finalized and sent
and selecting "Send Finalized Forms" yields "Nothing available to display."

This can be dangerous in some situations and auto-deletion of finalized forms is important.
 If the option is broken in 1.45, a major effort should be made to inform users in
case some of their collectors are put a risk by having unexpected form data on their
mobiles.

What version of the product are you using? On what operating system?

1.45 on GalaxyS3 running Android 4.3.  ODK-Aggregate 1.45 on FreeBSD.

Please provide any additional information below.

If anyone else can verify this (set "delete after send" in general options, finalize
a form, then check in "delete saved form" to see if the should-have-been-deleted form
is still there, please post results.

Reported by dgessel on 2015-04-19 02:09:06

Same here. After sending in finalized form, it is not deleted - but needs to be manually deleted by "Delete Saved Form" > "Saved forms".
ODK Collect 1.4.7 (1053) running on phone with Android 5.0.1 Locale nb_NO


Add Chrome-specific instructions for symbolizing traces.

These help produce usable pprof flame graphs from Chrome release binaries.

Thanks Ken!


observability: add prometheus metrics

Add Prometheus metrics for Tornado server and Python Runtime. Other application specific metrics can be added in the future.
Description
Prometheus style metrics are a vendor neutral standard based on a pull model. Prometheus-enabled servers output time-series data on a route (usually /metrics), which can be scraped. Being an open standard, most monitoring tools know how to interface with Prometheus metrics (Open Telemetry support them too).
More info on the Prometheus standard:
https://prometheus.io/docs/concepts/data_model/
https://github.com/prometheus/docs/blob/main/content/docs/instrumenting/exposition_formats.md
This PR enables the basic built-in metrics, which cover the Tornado server (http metrics) and the Python runtime. More metrics, specific to Mage can be added in the future

How Has This Been Tested?
Manually run
./dev.sh default_repo
./dev.sh default_repo --enable_prometheus
Made sure /metrics is live only on the later case

[ ] Test A
[ ] Test B

Checklist

[ ] The PR is tagged with proper labels (bug, enhancement, feature, documentation)
[ ] I have performed a self-review of my own code
[ ] I have added unit tests that prove my fix is effective or that my feature works
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

[ ] If new documentation has been added, relative paths have been added to the appropriate section of docs/mint.json

cc:
@wangxiaoyou1993

cc: @dy46


Assert user-specified positions within image or image_view are in range

Description
These assertions validating user-specified x and y values for pixel positions at run-time should help GIL users to catch basic mistakes as early as possible, especially during complex processing algorithms.
(Easy to make a mistake calling view(y, x) instead of view(x, y)!)
The checks are deliberately implemented using assertions as debug-only tools disabled in NDEBUG builds, and not as exceptions to avoid potentially significant performance hit at run-time in optimised builds.
Add TODO comments where certain assumptions are not immediately obvious and may require further testing to clarify and documenting. For example, what are constraints on requesting locators with negative offsets using image_view::xy_at?
Tidy up image.hpp and image_view.hpp formatting vertically rather than horizontally with very long lines, align return statements to left for immediate display what is calculated and returned, instead of hiding it behind the right margin.
Tasklist

[x] Ensure all CI builds pass
[ ] Review and approve

(Easy to make a mistake calling view(y, x) instead of view(x, y)!)

Those assertions, if present, would help me and @miralshah365 avoid spending substantial amount of time on investigating what causes image heap corruption in one of processing algorithms we've been testing recently.

@stefanseefeld I'd like to have this merged, soonish, unless you have strong objections. Details, if anything necessary, can be tweaked in future, I think.

Thank you @stefanseefeld


Soft "elimination" for inactive members

Members that are inactive shall remain in VietSpeak Slack server, but shall not be allowed to post any messages on VietSpeak other than posting a valid audio submission for a task.
A member is considered inactive if they haven't posted any submissions for the last NUMBER_OF_RESTING_DAYS days, with NUMBER_OF_RESTING_DAYS = 14 (3 days to familiarize with the group + 11 days for the longest possible task)

Bot users should never be eliminated.


Facebook-config.txt and list.txt

../../config/facebook-config.txt -u ../../data/list.txt

I'm a beginner so I'm not that familiar with linux...
Is "facebook-config.txt" created automatically ?
And list.txt is a list of profile URLs that I need to manually enter ?
Because I can't seem to find anything with "list.txt" in .py script files.
I also can't find a package named 'SeleniumHelper', I installed Selenium, but the Helper is not included it appears.

I also cannot install "SeleniumHelper"


Implement JSON view model for GET on /register

Implement JSON view model for GET on /register according to the framework spec:

View Models
The login an registration views are dynamic.  The registration form has
configurable fields, and both login and registration need to dynamically render
buttons for any providers (Google, Facebook, SAML) that are mapped to the
specified Stormpath application.
For that reason, we support JSON GET requests of the /login and /register
views.  The response is a view model that the client can use to render the
form appropriately.
Please see login and registration for specific information on each
model.

Part of #298.

As this is almost identical to #306, I will wait for #310 to be merged and then do a small refactor to make it work for /register as well.


remove pause ability and replace it with restart ability

Should we also remove the reset button since it is no longer required?

Yes, it should be removed


Make more nations convert as 'major' nations

See these posts for details:
https://forum.paradoxplaza.com/forum/index.php?threads/v2-to-hoi3-converter-mod.582798/page-51#post-21050305
https://forum.paradoxplaza.com/forum/index.php?threads/v2-to-hoi3-converter-mod.582798/page-50#post-21027346

All great powers now convert as majors. Secondary powers could not be added, as I couldn't find them easily listed in the save.


[build] bump padder to v1.0.0 and fix breaking changes

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (6bab2ea) 31.92% compared to head (814b6b4) 31.92%.

:exclamation: Current head 814b6b4 differs from pull request most recent head 96f5b5c. Consider uploading reports for the commit 96f5b5c to get more accurate results

Files
Patch %
Lines

src/mock.rs
50.00%
4 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage   31.92%   31.92%
=======================================
  Files           5        5
  Lines         686      686
=======================================
  Hits          219      219
  Misses        467      467

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


API for plotting from a javascript function

A function to plot from a javascript function would be really handy.
Probably something in the form of
plotFunction(func: function, options: object)

https://github.com/sezanzeb/JsPlot3D/blob/master/USAGE.md#plotting-functions


build_ext fails if build_py was not called before

For example in aesara:
$ python setup.py build_ext -j12
running build_ext
UPDATING build/lib/aesara/_version.py
error: [Errno 2] No such file or directory: 'build/lib/aesara/_version.py'

During standard python setup.py install, I think build_py is called before build_ext, but it is not always the case that someone will want to call build_py implicitly in build_ext. For example, here is a StackOverflow post asking how to avoid running build_py before build_ext.
I note that we do avoid running the update if you use --inplace. Would updating your above command to python setup.py build_ext -j12 --inplace get you what you want?

Building in-place is not an option for us, as separate build dirs are required to handle different versions of Python.

For the record, the problem is related to the Python build infrastructure for Gentoo packages. I suppose the alternative "fix" would be to skip updating the version if build_py wasn't called but I'm not really sure how to check that. I suppose checking if build dir could be an option.

Might make sense to warn if the file doesn't exist with an error message that explains that it couldn't be updated and you might need to build_py first.
Setup.py does also allow you to pass multiple commands, each with their own options. For this use case, maybe:
python setup.py build_py build_ext -j12

The whole point is to avoid calling build_py because it makes everything much slower (and it's very slow anyway), and PEP517 build takes care of calling it anyway.

…and updated!


Wiki Documentation

Update wiki with usage details before the release.
The documentation must include all the scenarios that the extension covers.
Include documentation to achieve completions using razor tooling

documentation in readme.


hub open doesn't use the right browser

Hi :wave:
I really enjoy hub and I like to use hub open to open the current repository on github. I have quite a few browsers installed on my system and it looks like hub isn't using my default browser when opening the url.
Instead of using google-chrome-stable, it uses chromium.
I think the command should be using the value from my system settings:
Gnome Control Center -> Details -> Default Applications:

uname -a
Linux zbookarch.local 4.8.8-2-ARCH #1 SMP PREEMPT Thu Nov 17 14:51:03 CET 2016 x86_64 GNU/Linux

hub --version
git version 2.10.2
hub version 2.3.0-pre8-4-gd67ca0e

:cake:

Ah nevermind, I only had to set $BROWSER to the right executable, thanks to #1289 :+1: :tada:


Unable to run Body tracking on NVIDIA RTX A4000 in custom application

Suggested labels : Bug, Body Tracking, Triage Needed
Issue
When trying to use a custom Unity application based on the body tracking SDK, the body tracking fails to run, only when using an NVIDIA RTX A4000. The same application works on older GPUs.
Details
Different drivers were tested to solve this issue :
471.41 , 473.81 , 516.94
The application would be stuck in an infinite loop after opening the scene and had to be killed via the Task Manager.
516.25
The application would open properly, the body tracking was missing.
In all cases, the Azure Kinect Body Tracking Viewer would work properly.
In the Nvidia configuration pannel, the custom application uses the NVIDIA RTX A4000.
The Azure Kinect Body Tracking SDK was updated from 1.1.0 to 1.1.2.
I read issues #1735 and #1763 but could not find a solution.
Context
Body tracking worked perfectly on older configurations / GPUs. The NVIDIA RTX A4000 is the first problematic GPU for this custom application.
Configuration
Windows 10 Pro
Nvidia RTX A4000 - 471.41 , 473.81 , 516.25 , 516.94
Azure Kinect SDK 1.4.1
Azure Kinect Body Tracking SDK 1.1.2

Fixed with LightBuzz update and Azure Kinect Body Tracking SDK update (1.1.0 to 1.1.2)


Add simple guide to show how to use Py-EVM as a library

What was wrong?
So, far, we don't have a guide that shows how to use Py-EVM as an actual library to build things. Frankly, most of this isn't really Py-EVM specific and falls more into a "How to get started with Python" category but I feel it doesn't hurt to be as helpful as possible especially when it comes to onboarding new peoplel to the ecosystem.
How was it fixed?
Added simple guide that shows how to clone a boilerplate repo, add Py-EVM as a dependency, install it, and code a very very simple script that uses Py-EVM
The code is written based on the latest release. Might be worth cutting a new release soon and then rebasing this PR (as opposed to merging and breaking silently soon :sweat_smile:
Cute Animal Picture

Happy to hear this guide already helped @johnys222 to move forward (#1021).
Initially, I was tempted to keep this PR open until we cut a new Py-EVM release (because eth namespace changed) but I realize that this guide won't break as the py-evm version is locked down. So while it may become outdated, it won't break at least. Merging.


Feature/kpw15

Added a limited implementation of the SPS scheme presented by Kiltz et al. [1]
Note: Signs vectors instead of matrices
[1] Kiltz, E.,Pan, J., Wee, H.:
Structure-Preserving Signatures from Standard Assumptions, Revisited
https://eprint.iacr.org/2015/604.pdf
more closely follows:
[2] Sakai, Y., Attrapadung, N., Hanaoka, G.:
Attribute-Based Signatures for Circuits from Bilinear Map
https://eprint.iacr.org/2016/242.pdf

From my side the PR looks ready, besides the conflicts. 🎉

Good work.


To ensure clean builds, every buildkite user must manually override BUILDKITE_GIT_CLEAN_FLAGS env on every machine in their build fleet

https://github.com/buildkite/agent/issues/173 and subsequent tickets added support for BUILDKITE_GIT_CLEAN_FLAGS to allow users to customize the behavior of git clean.
However, because this is supported in an environment variable and not in a config file, this means that every build machine across the fleet must have its BUILDKITE_GIT_CLEAN_FLAGS set the same way manually.  If any machine neglects to set BUILDKITE_GIT_CLEAN_FLAGS, you can run into hard to repro situations such as https://github.com/buildkite/agent/issues/367
This is not a big deal on virtualized/containerized build fleets, but it can be hard to manage on machines that aren't easily virtualized (such as macs used for ios builds)
Either this should be configurable in build config files so that all machines will have the same config, or https://github.com/buildkite/agent/issues/367 should be fixed so that by default machines are properly cleaning their build directories.  Or both would also work.

@emmby this should already be configurable in your buildkite-agent.cfg check this commit: https://github.com/buildkite/agent/commit/5c50c5544fb88454f81b5dbd97b035011ff21acd
# Flags to pass to the `git clone` command
# git-clone-flags=-v

# Flags to pass to the `git clean` command
# git-clean-flags=-fdq

You should be able to configure both using that syntax, and by the looks of it they should be present (commented out) already in your buildkite-agent.cfg file - do you see them?

(Also, thanks again for submitting these issues! <3)

Ah, thank you.  Is the documentation on the website out of date?  It says here that BUILDKITE_GIT_CLEAN_FLAGS is only configurable via env: https://buildkite.com/docs/agent/configuration

@emmby great find - yeah those docs were 100% wrong. Sorry about that confusion!! @toolmantim has updated them now :)
I'll close this issue now - thanks again for reporting it! 👍


Page still mentioning Set-MsolUser

This document still refers to MSOnline cmdlets. Please change and explain how to do things using Microsoft Graph.

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

ID: 4cb129d2-9594-bf0c-5946-982f0ed0d888
Version Independent ID: 6522b038-6853-3552-55c8-f174cf56573d
Content: Enable per-user Multi-Factor Authentication - Microsoft Entra
Content Source: articles/active-directory/authentication/howto-mfa-userstates.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

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

@kloosterj
As per the latest update Graph docs are being updated with new license config option. It will be updated soon.


Error using MongoDB adapter example.

Question 💬
Using the exact code given in the MongoDB adapter guide, I get the following error.
https://next-auth.js.org/adapters/mongodb

How to reproduce ☕
https://github.com/callumok2004/nextjs-reproduction/
Contributing 🙌🏽
No, I am afraid I cannot help regarding this

the problem seems to happen before you actually use mongodb with next-auth. I would recommend looking around how to set up a Mongo client with Next.js.
I simply used the official example from here https://github.com/vercel/next.js/blob/canary/examples/with-mongodb/lib/mongodb.js

the problem seems to happen before you actually use mongodb with next-auth. I would recommend looking around how to set up a Mongo client with Next.js.
I simply used the official example from here https://github.com/vercel/next.js/blob/canary/examples/with-mongodb/lib/mongodb.js

That is the exact example I am using...?? Just without all the comments, added semi-colons etc

could you try using it outside of next-auth to confirm if it's actually a problem with next-auth or not? I have  a feeling this is an upstream issue.

@balazsorban44 the issue in the mentioned issue is not the same as this one...

my bad then. I'll transfer it here and reopen. in any case, if you have an adapter related issue, please open it in this repo next time. helps with maintenance


Request id is different than response id (-32000):

Rakudobrew on Ubuntu 15.04 & Linux Mint 17.2;
Running :~$ panda install Task::Star
Result:
~$ panda install Task::Star
==> Task::Star depends on JSON::RPC, Pod::To::HTML, p6doc
==> p6doc depends on File::Temp
==> File::Temp depends on File::Directory::Tree
==> Fetching JSON::RPC
==> Building JSON::RPC
==> Testing JSON::RPC
Request id is different than response id (-32000): {:request(${:id(Any:U), :jsonrpc("2.0"), :method("ping")}), :response(${:id(Any), :jsonrpc("2.0"), :result("pong")})}
# Looks like you planned 39 tests, but ran 29
t/client.t ..·
Dubious, test returned 255 (wstat 65280, 0xff00)
Failed 10/39 subtests·
t/server.t .. ok

Test Summary Report
-------------------
t/client.t (Wstat: 65280 Tests: 29 Failed: 0)
  Non-zero exit status: 255
  Parse errors: Bad plan.  You planned 39 tests but ran 29.
Files=2, Tests=58, 11 wallclock secs ( 0.04 usr  0.00 sys + 11.44 cusr  0.12 csys = 11.60 CPU)
Result: FAIL
test stage failed for JSON::RPC: Tests failed
  in method throw at /home/eric/.rakudobrew/moar-nom/install/share/perl6/runtime/CORE.setting.moarvm:1
  in method install at /home/eric/.rakudobrew/moar-nom/install/share/perl6/site/lib/Panda.pm:142
  in block  at /home/eric/.rakudobrew/moar-nom/install/share/perl6/site/lib/Panda.pm:214
  in method resolve at /home/eric/.rakudobrew/moar-nom/install/share/perl6/site/lib/Panda.pm:208
  in sub MAIN at /home/eric/.rakudobrew/bin/../moar-nom/install/share/perl6/site/bin/panda:18
  in block <unit> at /home/eric/.rakudobrew/bin/../moar-nom/install/share/perl6/site/bin/panda:145

Failure Summary
----------------
Task::Star(
    *test stage failed for JSON::RPC: Tests failed)

This is a duplicate of #17. Sorry!


[IT-3021] Add 'only-codes' parameter

Add an 'only-codes' query-string parameter to filter the chart of accounts based on the listed codes (meta codes are still injected).
Replace the 'enable_code_filter' parameter with debug logging output and a refactor to ensure that the code processing always occurs. This also renames 'filter_chart' to 'limit_chart' for clarity as it now only processes the 'limit' parameter.
This will allow us to choose which tags will be listed in the Service Catalog drop-down menu.

I'm going to re-work this as a priority-codes parameter defining codes to move to the top of the output list.


[BUG] Cloud Shell token API does not support https://cognitiveservices.azure.com

An issue filed in the azd repo shows that authentication for the https://cognitiveservices.azure.com audience is not supported in Cloud Shell. https://github.com/Azure-Samples/azure-search-openai-demo/issues/629
To Reproduce
curl http://localhost:50342/oauth2/token --data "resource=https://cognitiveservices.azure.com" -H Metadata:true -s

Also fails when running
azd auth token --output json --scope https://cognitiveservices.azure.com

This was discovered in
Observed Behavior
{"error":{"code":"AudienceNotSupported","message":"Audience https://cognitiveservices.azure.com is not a supported MSI token audience. Supported audiences: https://management.core.windows.net/,https://management.azure.com/,https://graph.windows.net/,https://vault.azure.net,https://datalake.azure.net/,https://outlook.office365.com/,https://graph.microsoft.com/,https://batch.core.windows.net/,https://analysis.windows.net/powerbi/api,https://storage.azure.com/,https://rest.media.azure.net,https://api.loganalytics.io,https://ossrdbms-aad.database.windows.net,https://www.yammer.com,https://digitaltwins.azure.net,0b07f429-9f4b-4714-9392-cc5e8e80c8b0,822c8694-ad95-4735-9c55-256f7db2f9b4,https://dev.azuresynapse.net,https://database.windows.net,https://quantum.microsoft.com,https://iothubs.azure.net,2ff814a6-3304-4ab8-85cb-cd0e6f879c1d,https://azuredatabricks.net/,ce34e7e5-485f-4d76-964f-b3d2b16d1e4f,https://azure-devices-provisioning.net,https://managedhsm.azure.net,499b84ac-1321-427f-aa17-267ca6975798,https://api.adu.microsoft.com/,https://purview.azure.net/,6dae42f8-4368-4678-94ff-3960e28e3630"}}

Expected behavior
Should be able to get an authentication token from Cloud Shell token API.
Is this specific to Cloud Shell?
Yes, this is specific to Cloud Shell
Interface information
How are you accessing Cloud Shell - https://shell.azure.com, https://portal.azure.com, via Windows Terminal, or some other method?
If a browser, which Operating System and browser are you using? (ex. Edge on Windows 10)
Accessing Cloud Shell by: https://portal.azure.com/
OS: Windows 11
Browser: Edge

Thank you @danieljurek for reporting this.  This should be a small fix on our side - I will follow up after the change has been made.

In context of the azure-search-openai-demo, it does apply to search.azure.com as well:
{"message":"fetching token: invalid CloudShell token API response code: 400, content: {"error":{"code":"AudienceNotSupported","message":"Audience https://search.azure.com is not a supported MSI token audience.

Is this fixed? I have hit the problem in https:://search.azure.com is not supported MSI token error, when I execute via cloudshell.
AzureDeveloperCliCredential.get_token failed: {"type":"consoleMessage","timestamp":"2024-04-18T18:06:11.892782392Z","data":{"message":"fetching token: invalid CloudShell token API response code: 400, content: {"error":{"code":"AudienceNotSupported","message":"Audience https://search.azure.com is not a supported MSI token audience. Supported audiences: https://management.core.windows.net/,https://management.azure.com/,https://graph.windows.net/,https://vault.azure.net,https://datalake.azure.net/,https://outlook.office365.com/,https://graph.microsoft.com/,https://batch.core.windows.net/,https://analysis.windows.net/powerbi/api,https://storage.azure.com/,https://rest.media.azure.net,https://api.loganalytics.io,https://ossrdbms-aad.database.windows.net,https://www.yammer.com,https://digitaltwins.azure.net,0b07f429-9f4b-4714-9392-cc5e8e80c8b0,822c8694-ad95-4735-9c55-256f7db2f9b4,https://dev.azuresynapse.net,https://database.windows.net,https://quantum.microsoft.com,https://iothubs.azure.net,2ff814a6-3304-4ab8-85cb-cd0e6f879c1d,https://azuredatabricks.net/,ce34e7e5-485f-4d76-964f-b3d2b16d1e4f,https://azure-devices-provisioning.net,https://managedhsm.azure.net,499b84ac-1321-427f-aa17-267ca6975798,https://api.adu.microsoft.com/,https://purview.azure.net/,6dae42f8-4368-4678-94ff-3960e28e3630,https://cognitiveservices.azure.com,48ac35b8-9aa8-4d74-927d-1f4a14a0b239,46da2f7e-b5ef-422a-88d4-2a7f9de6a0b2"}}\n"}}

I'm currently running into the same issue. This is really putting a crimp in my style for running a debug/troubleshooting app in an environment that I can't connect to from my dev system...


General Multi-Task Workers, and General Task Polling?

So, most of the documentation shows each worker having its own 'task type'. Is there any way to set up workers that can handle any, or a specified subset of task types? Also, is there a way then to simply query for the next task ready for processing in the Orchestrator's queues?
The main use case I have is this:
There are (very) many services that I want the orchestration's workers to call, each of which have a different amount of running time. Instead of trying to create a worker for each service (where each service is mapped to a task), I'd like to have general workers that can handle any general task, such that tasks are distributed between all workers.
Is there some other way this might work using the current Orchestrator's setup? Or, is there any issue preventing this from being added as a feature?

The project conductor-client has a class, WorkflowTaskCoordinator which polls for Worker input data in addition to other tasks like Task Worker thread pool management etc. Before polling for input data, the Worker instances have to be registered with the WorkflowTaskCoordinator object. Worker instance is  created by passing the unique Task Definition name to its constructor. Pls refer to the below code snippet:
TaskClient taskClient = new TaskClient();
taskClient.setRootURI("http://localhost:8080/api/");//Point this to the server API
int threadCount = 1; //number of threads used to execute workers. To avoid starvation, should be same or more than number of workers

Worker worker = new TaskWorker(“task_1");
Worker worker1 = new TaskWorker("task_2");
//Create WorkflowTaskCoordinator
WorkflowTaskCoordinator.Builder builder = new WorkflowTaskCoordinator.Builder();
WorkflowTaskCoordinator coordinator = builder.withWorkers(worker,worker1)
								  .withThreadCount(threadCount)
								  .withTaskClient(taskClient).build();
//Start for polling and execution of the tasks
coordinator.init();

In your case, you can create multiple instances of the same Worker class by passing different task definition names provided the single Worker class is implemented to handle the input data meant for all the tasks it has been initiated for.
WorkflowTaskCoordinator.Builder.withWorkers accepts Varargs.


Proof of concept: Collections support

What is this?
As it stands, this plugin only supports posts, i.e. documents in the default posts collection. Now that collections are mature and very useful, this plugin ought to support this Jekyll feature fully. This is my attempt at achieving this. Right now, it works, but it needs input and plenty of expert review.
How do I propose enabling collection archives?
To ensure backwards compatibility, documents in the default site.posts collection will continue to appear within enabled archives, but to add additional collections, you’ll need to add an archive: true value to each collection’s respective config, e.g.:
collections:
  albums:
    output: true
    archive: true

Is this the right approach? I also considered enabling archives via front matter defaults (and that way you could also exclude individual pages from being archived) but this felt like a separate feature.
Code review
Arguably, some of the code I’ve added here should be provided by Jekyll’s core. For example, Jekyll’s post_attr_hash utility only parses posts, but there doesn’t seem to be an equivalent function to parse documents across all collections; I’ve added a doc_attr_hash utility to perform that function here. Or have I missed something? It should be noted that the code within this function feels fragile even to my inexperienced eyes, so I welcome improvements, optimisations and refinement.
Next steps

[ ] Get consensus on approach to configuration
[ ] Confirm whether code should be forked to continue support for Jekyll 2.0
[ ] Review code and optimise
[ ] Add tests
[ ] Update documentation

I look forward to receiving feedback from the core plugin team, reviewers and others. /cc @alfredxing, @parkr, @pathawks

Going by the initial CI tests, looks like more code forking might be needed to provide compatibility with Jekyll 2.0. 🤔

Thanks for the PR! Just some of my thoughts here:

IMO all configuration for the plugin should be contained in the jekyll-archives hash; I don't think we should pollute the default Jekyll configuration spaces.
Documents don't have things like tags, categories, or date "by default". Tags and categories make sense, not sure if date does?
On attr_hash: if you think this would be useful to Jekyll in general, we can maybe think about including it in core? If so, you can open a PR there to add it. Ideally it can be a very general utility, taking in the attributes as well as a list of collections to look in. Then post_attr_hash can be refactored to just use that.
Jekyll 2 compatibility: this is kind of a pain. At the very least, you can probably refactor out the common if (Jekyll::VERSION >= '3.0.0') attr_hash(...) parts into a separate function

Thanks for the feedback @alfredxing!

IMO all configuration for the plugin should be contained in the jekyll-archives hash; I don't think we should pollute the default Jekyll configuration spaces.

Makes sense. How would you imagine configuring collections? Perhaps:
jekyll-archives:
  collections: [albums, books, films]

How might we deal with the posts collection in this case? Enabled by default, but if adding a collectionsarray, posts needs to be included?

Documents don't have things like tags, categories, or date "by default". Tags and categories make sense, not sure if date does?

Documents may not have these attributes by default, but they can – so this shouldn’t matter, no? (Most of my collection documents have dates, FWIW).

On attr_hash: if you think this would be useful to Jekyll in general, we can maybe think about including it in core?

I had a feeling you might say this! Will look at the core and see if anything has been proposed already. My general sense is that the core still treats posts as a special case, when really they should be treated as a default; i.e. site.tags only looks at posts, when it should probably look across all documents. Anyway, I’ll move this particular discussion to core.

you can probably refactor out the common if (Jekyll::VERSION >= '3.0.0') attr_hash(...) parts into a separate function.

You’re overestimate my programming abilities!! But yes, I think this might need abstracting out. I’m curious as to the general approach regarding compatibility with plugins, though?

@alfredxing Okay, I’ve had a stab at adding a global utility function to core (doc_attr_hash), and submitted a PR. I followed your idea of refactoring post_attr_hash to use this new function, too. I think it works well… 🤞

Has there been any development on this pr?

I know @spencertweedy already asked this question, but I wanted to reach out and ask if we might see this great initiative come to fruition. I'd be happy to make a donation if that helps!

Any news on this feature?

Hi all. I realized that the project has not been updated for a year, and the PR is from 2017, so I took the liberty to fork it and rebase the PR to 2.2.1 version. It seems to work for me.
To use it, I put this into Gemfile:
group :jekyll_plugins do
  gem "jekyll-archives", git: "https://github.com/positron96/jekyll-archives.git", ref: "ed7e67a"
end


 Unfortunately, both of these things are out of our control.
blob_id being converted to an integer is related to SQLite dumping bigint cols as integer columns, even for PK cols (see https://github.com/rails/rails/issues/31329, https://github.com/Shopify/maintenance_tasks/pull/424)
The order being dropped on the maintennace_tasks_runs table is also a SQLite adapter bug (that should be fixed by https://github.com/rails/rails/pull/41548 on Rails edge)
I don't think there's anything for us to do here, so I'm going to close this.

Thanks for the references, I learned a lot!


Multiscalar doesn't show the answers

See: https://app.seer.pm/markets/100/0x69cf7df28e14c16fededd377affc6d70b740732c?outcome=Clément
Questions are answered, those are shown in the search but not when browsing this particular market. The current answers need to be shown.

It's working now, but we didn't change anything. I'll try to replicate it in a new market (maybe it was a caching issue?)


Addition of Editorconfig

A .editorconfig file enforces some standards around whitespaces, tabs etc, which are adhered to in standard tools such as VS Code and Visual Studio.  The one being added in this PR was taken from the Umbraco CMS source.

Lovely! Thank you @LottePitcher!


Prompt to create flow.json if none exists

Instructions
When a user runs flow emulator and no flow.json exists in the directory:
Instead of halting, the cli should prompt the user and ask to create a flow.json
Optional:
The emulator can be started without a flow.json file present in the directory.

This is already possible to solve by using flow emulator --init.


单词错了，多了一个F

🚨
①、maven用最目前最新的稳定版本3.6.2编译时会报错，需要添加version才能使用。
②、有个单词写错了。

 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.

Codecov Report

Merging #2735 into master will decrease coverage by 0.01%.
The diff coverage is 55.55%.

@@             Coverage Diff              @@
##             master    #2735      +/-   ##
============================================
- Coverage     80.46%   80.44%   -0.02%
+ Complexity     7101     7098       -3
============================================
  Files           145      145
  Lines         26705    26705
  Branches       6373     6373
============================================
- Hits          21489    21484       -5
- Misses         3382     3384       +2
- Partials       1834     1837       +3

Impacted Files
Coverage Δ
Complexity Δ

...com/alibaba/fastjson/serializer/CalendarCodec.java
83.16% <0%> (ø)
19 <0> (ø)
:arrow_down:

...ava/com/alibaba/fastjson/serializer/JodaCodec.java
47.48% <0%> (ø)
35 <0> (ø)
:arrow_down:

...ba/fastjson/parser/deserializer/Jdk8DateCodec.java
74.83% <0%> (ø)
79 <0> (ø)
:arrow_down:

src/main/java/com/alibaba/fastjson/JSON.java
84.86% <100%> (ø)
147 <0> (ø)
:arrow_down:

...com/alibaba/fastjson/parser/DefaultJSONParser.java
88.09% <100%> (ø)
314 <3> (ø)
:arrow_down:

...ava/com/alibaba/fastjson/serializer/DateCodec.java
88.88% <100%> (-0.66%)
38 <0> (ø)

...main/java/com/alibaba/fastjson/util/TypeUtils.java
79.87% <66.66%> (ø)
622 <0> (-1)
:arrow_down:

...com/alibaba/fastjson/parser/JSONReaderScanner.java
72.61% <0%> (-1.28%)
44% <0%> (-1%)

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
87.88% <0%> (-0.19%)
198% <0%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

不能改，改了会导致不兼容


etcd-2.2.0 build fails for arm64

etcd-2.2.0 build fails for arm64 (go-1.5.1).
# github.com/coreos/etcd/Godeps/_workspace/src/github.com/boltdb/bolt
../gopath/src/github.com/coreos/etcd/Godeps/_workspace/src/github.com/boltdb/bolt/db.go:85: undefined: maxMapSize
../gopath/src/github.com/coreos/etcd/Godeps/_workspace/src/github.com/boltdb/bolt/db.go:85: invalid array bound maxMapSize

@mkaczanowski Could you confirm this is a problem?

As reported in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=803133
This problem also affects v2.2.1 on arm64, ppc64el and ppc64.

I verified the arm64 build fails with v2.2.1.

@glevand have you tried building with -tags arm64 ?

Closing this. I tried to build with amd64 with current 2.2 release branch https://github.com/coreos/etcd/tree/release-2.2 and it got built.
I used https://github.com/mitchellh/gox to build it.
Please ping us if it still fails.

Git bisect shows it was fixed in 4669b899cca9a326731a10e03c76605bf26470f6 (Godeps: update boltdb).

The boltdb in release branch may need to be updated.
s390x doesn't build with release-2.2.
Master builds fine.
I've figured out the root cause is boltdb. The bolddb version  in release-2.2 doesn't have bolt_s390x.go, but master has.

Commit https://github.com/coreos/etcd/commit/4669b899cca9a326731a10e03c76605bf26470f6 (Godeps: update boltdb) updated boltdb from v1.0-119-g90fef38 to v1.0-158-g81db894.
@gyuho can we get this into the next etcd release, either by merging https://github.com/coreos/etcd/commit/4669b899cca9a326731a10e03c76605bf26470f6, or branching from master after this commit?

I will ask @xiang90 if we could cherry-pick it in next release.

This is resolved in etcd 2.2.5 I believe.

Seems to be fixed.  Thanks.


Need help.

I want to make steam library folder on external usb drive. how to get proper permission on external usb drive when mounted?
Link: https://www.reddit.com/user/New_Asian_Dude16/comments/g2j5vq/the_three_things_sorry_im_not_being_a_bit_more/?utm_source=share&utm_medium=ios_app&utm_name=iossmf

@AsianLad16,
Your reddit post above seems to petain to Crostini, not crouton.
If your external drive is mounted noexec and/or without rw permissions then just remount it, for example:

sudo mount -i -o remount,exec,rw /media/removable/[NameOfDeviceMountpoint]

Hope this helps,
-DennisLfromGA


upgrade image-size version fix caculate webp image size bugs

When I try to use webp for app. But some webp images get an error message unsupported file type by<EMAIL_ADDRESS>And I in image-size repo I found a issue https://github.com/image-size/image-size/issues/65.
This is the same problem.
And try image-size^0.5.1 the error fixed.

@shergin The new pull request is here.
Thank you.

@shergin import failed but I don't know what makes test failed. Would you help me?

@Lxxyx I believe we also have to update DependencyGraph-test.js. Could you please do this?

It can not be landed because this diff breaks a packages consistency test.