Force TLS negotiation to only support HTTP/1.1 - Issue #298?

Seems OKTA has a broken implementation of HTTP2?
There seem to be some quirks with they way golang handles HTTP2 requests (specifically when receiving GOAWAY frames) golang/go#20979, which result in the HTTP2 connection not terminating.

This does indeed appear to fix #298

This fixes #298 in our environment as well, however, I've reduced it down to a minimal patch that still works:
diff --git a/lib/okta.go b/lib/okta.go
index f17cf86..a5a420d 100644
--- a/lib/okta.go
+++ b/lib/okta.go
@@ -2,6 +2,7 @@ package lib

 import (
        "bytes"
+       "crypto/tls"
        "encoding/json"
        "errors"
        "fmt"
@@ -578,9 +579,13 @@ func (o *OktaClient) Get(method string, path string, data []byte, recv interface
                }
        }

+       tlsCfg := &tls.Config{
+       }
+
        transCfg := &http.Transport{
                Proxy:               http.ProxyFromEnvironment,
                TLSHandshakeTimeout: Timeout,
+               TLSClientConfig: tlsCfg,
        }
        client = http.Client{
                Transport: transCfg,

I don't yet understand why though.

I don't yet understand why though.

Default is HTTP1, says the Go docs.
https://golang.org/pkg/net/http/
    // TLSClientConfig specifies the TLS configuration to use with
    // tls.Client.
    // If nil, the default configuration is used.
    // If non-nil, HTTP/2 support may not be enabled by default.
    TLSClientConfig *tls.Config

Still, better to be explicit.

Ok, I'm going to modify this PR to match @sdann's change and merge. It'd be nice to understand this behaviour better though

Superseded by #300


Abstract the application interface to allow using it in deeper structures also

If later a portal system wants to use applications also in deeper paths, the application must be more abstract.

Abstract interface for VirtualHost or use a VirtualHostWrapper
Provide the path where the application is located
Provide aaa by default via VirtualHost
Provide the possibility of an framework HTML

Done. Expect the AAA features. This will be followed in another ticket.


[NEAT-108] Export transformations Part 2

I noticed that Pipeline creation is part of RulesToDMS step. I think it's better to separate into separate step, something like RulesToCDFPipeline. The reasoning is following :

RulesToCDFPipeline may contain a set of configuration parameters that have nothing to do with model creation in DMS
RulesToCDFPipeline may return errors or fail, and the root cause of the failure might also be outside of DMS schema creation.


Refactor reproduce_state for scene component

Description:

Add tests to reach full coverage for helpers/state.py.
Refactor reproduce_state function in helpers/state.py. Add two dicts,
service_attributes and service_to_state. Use these in combination
with the dict of services per domain from ServiceRegistry, to find
the correct service to use in a scene state change.
Add ATTR_CODE and ATTR_CODE_FORMAT in const. Import these in
alarm_control_panel and lock platforms instead of making duplicate
constants in multiple modules.
Use ATTR_MEDIA_CONTENT_TYPE and ATTR_MEDIA_CONTENT_ID in media_player
platform in SERVICE_PLAY_MEDIA and play_media methods, instead of
'media_type' and 'media_id'.
Fix PEP257 in modified files.

Related issue (if applicable):
Fixes https://github.com/balloob/home-assistant/issues/1166
Example entry for configuration.yaml (if applicable):
Nothing new, but we should update the docs with more examples.
scene:
  - name: Romantic
    entities:
      light.tv_back_light: on
      light.ceiling:
        state: on
        xy_color: [0.33, 0.66]
        brightness: 200

  - name: Movies
    entities:
      light.tv_back_light:
        state: on
        brightness: 100
      light.ceiling: off
  - name: play_batman
    entities:
      media_player.universal:
        media_content_type: 'movie'
        media_content_id: 'batman'

Checklist:

[x] Local tests with tox run successfully.
[ ] TravisCI does not fail. Your PR cannot be merged unless CI is green!
[x] Fork is up to date and was rebased on the dev branch before creating the PR.
[x] Commits have been squashed.
If the code does not interact with devices:

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

All services that I could find and that I thought made sense to activate as a result of a scene state update are now supported.
Services that were not added were, among others:
media_player services with no corresponding state or specific attribute, eg next track
mqtt publish and mqtt publish_template
device_tracker see service
toogle service
One thing that would be nice to add is a way of telling the rollershutter which position to go to. But I couldn't figure out an easy way of solving that. Something to think about...

@balloob How would you feel about adding two attributes to every service. One attribute would be the state which the service activates for an entity, the other a list of required attributes in data to call the service.
With such a change the two dicts I created for reproduce state would not need to be maintained.

Services already register which attributes they take in. See for example light/services.yaml. I think it's a great idea to add something like result_state to that file.

This is a very nice clean up ! Once you address both your own comments I think it's good to go :dolphin:
I would suggest we merge this before you migrate reproduce_state to use service description data.

Yeah, I saw services.yaml. My problem with those is that we have to maintain code at two places for every service, instead of configuring all attributes of the service when it's registered. This would be more natural and less error prone, in my opinion.
At the same time, I can see that it's potentially a lot of attributes per service, so it's nice to keep that code mass separate. Would it be possible to, somehow, make services.yaml required and maybe add even more functionality there, and have no configure possibilities when calling to register a service?
Correct me if I've misunderstood how services.yaml and registering a service is connected.

services.yaml is passed in when a service is registered. Because it's
metadata I decided to move it outside of the main component file.
On Sun, Mar 6, 2016 at 10:23 AM, Martin Hjelmare<EMAIL_ADDRESS>wrote:

Yeah, I saw services.yaml. My problem with those is that we have to
maintain code at two places for every service, instead of configuring all
attributes of the service when it's registered. This would be more natural
and less error prone, in my opinion.
At the same time, I can see that it's potentially a lot of attributes per
service, so it's nice to keep that code mass separate. Would it be possible
to, somehow, make services.yaml required and maybe add even more
functionality there, and have no configure possibilities when calling to
register a service?
Correct me if I've misunderstood how services.yaml and registering a
service is connected.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/pull/1485#issuecomment-192954472
.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.

I don't think it should not be required. If a service is not registering
anything, it just defaults to empty everything.
On Sun, Mar 6, 2016 at 10:27 AM, Paulus Schoutsen<EMAIL_ADDRESS>
wrote:

services.yaml is passed in when a service is registered. Because it's
metadata I decided to move it outside of the main component file.
On Sun, Mar 6, 2016 at 10:23 AM, Martin Hjelmare<EMAIL_ADDRESS>
wrote:

Yeah, I saw services.yaml. My problem with those is that we have to
maintain code at two places for every service, instead of configuring all
attributes of the service when it's registered. This would be more natural
and less error prone, in my opinion.
At the same time, I can see that it's potentially a lot of attributes per
service, so it's nice to keep that code mass separate. Would it be possible
to, somehow, make services.yaml required and maybe add even more
functionality there, and have no configure possibilities when calling to
register a service?
Correct me if I've misunderstood how services.yaml and registering a
service is connected.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/pull/1485#issuecomment-192954472
.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.

I guess my main issue is that today you can register a service that requires attributes for excecution, but it's optional to describe these attributes in services.yaml. This has lead to most services.yaml being empty, and the only way of finding the attributes that are required for a service is by manually reading through the code. Same problem with target state and services.

services.yaml was meant to be the documentation. It is also made available in the frontend when calling a service. They are indeed not all filled in but that's the problem we should try to solve.

Yes, agree.

Should be good now.
I'll have to look more closely at the service registry and the translations you were talking about. I'm not totally in on what you meant, but I think I'll figure it out, once I read the source.
"This data is registered with the service registry (hass.services). It is currently only exposed calling .services but that translates all services to a dict. I think that it should instead of translating things to a dict actually return the _services dictionary. Translating to a dict should happen in api.py (I think that's the only place it is used right now)."


Prevent some self-RSET/RPC in MultiplayerAPI

Allow calling yourself via RPC/RSET if the mode allows it. Better error messages for when you are not allowed to.
A potentially better solution to #27332 .
Closes #27332.
Closes #27436.

Allow calling yourself via RPC/RSET if the mode allows it. Better error messages for when you are not allowed to.

Shouldn't that be reflected in the PR title and commit message? Currently it sounds like it does the opposite :)

@mhilbrunner eheh, you are right :+1:


[bt_action_node-2] error while loading shared libraries: libplansys2_bt_actions.so

Desctiption
I get the following error when I try to run my own package. What could be the problem with libplansys2_bt_actions.so?
Environment

ROS Distro: Foxy
OS Version: e.g. Ubuntu 20.04
Debian packages install

Console output
[INFO] [launch]: All log files can be found below /home/splinter1984/.ros/log/2021-12-07-00-49-10-537554-notebook-31572
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [plansys2_node-1]: process started with pid [31574]
[INFO] [bt_action_node-2]: process started with pid [31576]
[bt_action_node-2] /opt/ros/foxy/lib/plansys2_bt_actions/bt_action_node: error while loading shared libraries: libplansys2_bt_actions.so: cannot open shared object file: No such file or directory
[ERROR] [bt_action_node-2]: process has died [pid 31576, exit code 127, cmd '/opt/ros/foxy/lib/plansys2_bt_actions/bt_action_node --ros-args -r __node:=move_1 -r __ns:=/ --params-file /home/splinter1984/robas_ws/install/rasms_manipulator/share/rasms_manipulator/config/params.yaml --params-file /tmp/launch_params_nlztctyb'].
[plansys2_node-1] RCUTILS_CONSOLE_STDOUT_LINE_BUFFERED is now ignored.  Please set RCUTILS_LOGGING_USE_STDOUT and RCUTILS_LOGGING_BUFFERED_STREAM to control the stream and the buffering of log messages.
[plansys2_node-1] [INFO] [1638812951.436647072] [domain_expert_lc_mngr]: Creating client for service [domain_expert/get_state]
[plansys2_node-1] [INFO] [1638812951.436832960] [domain_expert_lc_mngr]: Creating client for service [domain_expert/change_state]
[plansys2_node-1] [INFO] [1638812951.438587661] [executor_lc_mngr]: Creating client for service [executor/get_state]
[plansys2_node-1] [INFO] [1638812951.439080045] [executor_lc_mngr]: Creating client for service [executor/change_state]
[plansys2_node-1] [INFO] [1638812951.441712420] [planner_lc_mngr]: Creating client for service [planner/get_state]
[plansys2_node-1] [INFO] [1638812951.441801286] [planner_lc_mngr]: Creating client for service [planner/change_state]
[plansys2_node-1] [INFO] [1638812951.444177384] [problem_expert_lc_mngr]: Creating client for service [problem_expert/get_state]
[plansys2_node-1] [INFO] [1638812951.444238759] [problem_expert_lc_mngr]: Creating client for service [problem_expert/change_state]
[plansys2_node-1] [INFO] [1638812951.450281949] [domain_expert]: [domain_expert] Configuring...
[plansys2_node-1] [INFO] [1638812954.436581642] [domain_expert]: [domain_expert] Configured
[plansys2_node-1] [INFO] [1638812954.437494583] [domain_expert_lc_mngr]: Transition 1 successfully triggered.
[plansys2_node-1] [INFO] [1638812954.438646646] [domain_expert_lc_mngr]: Node domain_expert_lc_mngr has current state inactive.
[plansys2_node-1] [INFO] [1638812954.439391603] [problem_expert]: [problem_expert] Configuring...
[plansys2_node-1] [INFO] [1638812954.439768636] [problem_expert]: [problem_expert] Configured
[plansys2_node-1] [INFO] [1638812954.442343317] [problem_expert_lc_mngr]: Transition 1 successfully triggered.
[plansys2_node-1] [INFO] [1638812954.443679949] [problem_expert_lc_mngr]: Node problem_expert_lc_mngr has current state inactive.
[plansys2_node-1] [INFO] [1638812954.444580473] [planner]: [planner] Configuring...
[plansys2_node-1] [INFO] [1638812954.445519070] [planner]: Created solver : POPF of type plansys2/POPFPlanSolver
[plansys2_node-1] [INFO] [1638812954.445605362] [planner]: [planner] Configured
[plansys2_node-1] [INFO] [1638812954.448339989] [planner_lc_mngr]: Transition 1 successfully triggered.
[plansys2_node-1] [INFO] [1638812954.449397893] [planner_lc_mngr]: Node planner_lc_mngr has current state inactive.
[plansys2_node-1] [INFO] [1638812954.449695632] [executor]: [executor] Configuring...
[plansys2_node-1] [INFO] [1638812954.640050675] [executor]: [executor] Configured
[plansys2_node-1] [INFO] [1638812954.641131358] [executor_lc_mngr]: Transition 1 successfully triggered.

It seems like broken deb-package. You should try to install plansys2 foxy-devel branch from sources

Thanks for your reply. I compiled a package from the source of the foxy-devel branch and it worked.

I'm still getting this error, on both foxy-devel and master.  I believe it's due to a missing ${PROJECT_NAME} in the install section of plansys2_bt_actions CMakeLists.txt


Move genotyping unit over to Spark SQL

Resolves #233.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/186/Build result: FAILURE[...truncated 8 lines...] > git -c core.askpass=true fetch --tags --progress https://github.com/bigdatagenomics/avocado.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/239/merge^{commit} # timeout=10 > git branch -a --contains 9d939f5f0a57b4acc725d58d0d0bffd8693a9e42 # timeout=10 > git rev-parse remotes/origin/pr/239/merge^{commit} # timeout=10Checking out Revision 9d939f5f0a57b4acc725d58d0d0bffd8693a9e42 (origin/pr/239/merge) > git config core.sparsecheckout # timeout=10 > git checkout -f 9d939f5f0a57b4acc725d58d0d0bffd8693a9e42First time build. Skipping changelog.Triggering avocado-prb » 2.3.0,2.10,2.0.0,centosTriggering avocado-prb » 2.3.0,2.11,1.6.1,centosTriggering avocado-prb » 2.6.0,2.10,2.0.0,centosTriggering avocado-prb » 2.6.0,2.11,2.0.0,centosTriggering avocado-prb » 2.3.0,2.11,2.0.0,centosTriggering avocado-prb » 2.6.0,2.11,1.6.1,centosTriggering avocado-prb » 2.3.0,2.10,1.6.1,centosTriggering avocado-prb » 2.6.0,2.10,1.6.1,centosavocado-prb » 2.3.0,2.10,2.0.0,centos completed with result FAILUREavocado-prb » 2.3.0,2.11,1.6.1,centos completed with result FAILUREavocado-prb » 2.6.0,2.10,2.0.0,centos completed with result FAILUREavocado-prb » 2.6.0,2.11,2.0.0,centos completed with result FAILUREavocado-prb » 2.3.0,2.11,2.0.0,centos completed with result FAILUREavocado-prb » 2.6.0,2.11,1.6.1,centos completed with result FAILUREavocado-prb » 2.3.0,2.10,1.6.1,centos completed with result FAILUREavocado-prb » 2.6.0,2.10,1.6.1,centos completed with result FAILURE
Test FAILed.

Coverage increased (+1.2%) to 79.576% when pulling d46b4cafdb64e3920b187fb14a4f8b5b30b92e87 on fnothaft:issues/233-move-to-sql into ca662767f8c89f0f88bfb15e39c67bbcc832c4e2 on bigdatagenomics:master.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/187/Build result: FAILURE[...truncated 8 lines...] > git -c core.askpass=true fetch --tags --progress https://github.com/bigdatagenomics/avocado.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/239/merge^{commit} # timeout=10 > git branch -a --contains ebabd3818cda25f83d7837d27dfe2c7e1bfb972b # timeout=10 > git rev-parse remotes/origin/pr/239/merge^{commit} # timeout=10Checking out Revision ebabd3818cda25f83d7837d27dfe2c7e1bfb972b (origin/pr/239/merge) > git config core.sparsecheckout # timeout=10 > git checkout -f ebabd3818cda25f83d7837d27dfe2c7e1bfb972bFirst time build. Skipping changelog.Triggering avocado-prb » 2.3.0,2.10,2.0.0,centosTriggering avocado-prb » 2.3.0,2.11,1.6.1,centosTriggering avocado-prb » 2.6.0,2.10,2.0.0,centosTriggering avocado-prb » 2.6.0,2.11,2.0.0,centosTriggering avocado-prb » 2.3.0,2.11,2.0.0,centosTriggering avocado-prb » 2.6.0,2.11,1.6.1,centosTriggering avocado-prb » 2.3.0,2.10,1.6.1,centosTriggering avocado-prb » 2.6.0,2.10,1.6.1,centosavocado-prb » 2.3.0,2.10,2.0.0,centos completed with result SUCCESSavocado-prb » 2.3.0,2.11,1.6.1,centos completed with result FAILUREavocado-prb » 2.6.0,2.10,2.0.0,centos completed with result SUCCESSavocado-prb » 2.6.0,2.11,2.0.0,centos completed with result SUCCESSavocado-prb » 2.3.0,2.11,2.0.0,centos completed with result SUCCESSavocado-prb » 2.6.0,2.11,1.6.1,centos completed with result FAILUREavocado-prb » 2.3.0,2.10,1.6.1,centos completed with result FAILUREavocado-prb » 2.6.0,2.10,1.6.1,centos completed with result FAILURE
Test FAILed.

Jenkins, retest this please.

Coverage increased (+1.2%) to 79.576% when pulling d46b4cafdb64e3920b187fb14a4f8b5b30b92e87 on fnothaft:issues/233-move-to-sql into ca662767f8c89f0f88bfb15e39c67bbcc832c4e2 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/188/
Test PASSed.


⚠ Bofrost.be has degraded performance

In f780b32, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1136 ms

Resolved: Bofrost.be performance has improved in b0913b3 after 6 minutes.


FACE id changes constantly in version 4

Hi. I have recently been experimenting with the FACE stream on HERO10 cameras. With previous versions, the recorded ID seems to be consistent with the number of people in the frame, but with the HERO10 version (ver,confidence %,ID,x,y,w,h,smile %, blink %) the ID number seems to be increasing every few samples. Could I be interpreting something wrong?
Here's a snippet of what I see. The ID should be the third value in the array. I can provide the original file privately.
[
{
"STMP": 9443744,
"TSMP": 130,
"STNM": "Face Coordinates and details",
"TYPE": "BBSSSSSBB",
"SCAL": [1,1,1,65535,65535,65535,65535,1,1],
"FACE": [
[[4,100,3422,24114,27596,52259,19516,60,0]],
[[4,100,3707,31602,26333,56610,19513,57,0]],
[[4,99,3711,32754,26214,26149,19514,58,0]],
[[4,99,3712,32793,25975,30502,19515,59,0]],
[[4,100,3710,32434,25940,21542,29499,59,0]],
[[4,100,3710,32293,26060,52261,26170,58,0]],
[[4,100,3709,32165,26214,26148,58938,58,0]],
[[4,100,3713,33087,26418,12831,62265,57,0]],
[[4,100,3713,33074,26589,56607,35896,56,0]],
[[4,100,3713,33189,26777,39198,65335,55,0]],
[[4,100,3714,33407,26930,12830,26167,55,0]],
[[4,100,3715,33714,27084,52253,62263,55,0]],
[[4,100,3716,33983,27203,17181,55607,55,0]]]
},
{
"STMP": 10483744,
"TSMP": 143,
"STNM": "Face Coordinates and details",
"TYPE": "BBSSSSSBB",
"SCAL": [1,1,1,65535,65535,65535,65535,1,1],
"FACE": [
[[4,100,3717,34213,27340,52253,55606,54,0]],
[[4,100,3713,33266,25428,21538,9790,62,0]],
[[4,100,3714,33369,25514,43554,3135,63,0]],
[[4,100,3714,33458,25616,4129,65336,56,0]],
[[4,100,3714,33445,25753,39201,65336,56,0]],
[[4,100,3714,33394,25684,21538,16184,56,0]],
[[4,100,3714,33292,25736,34847,9784,56,0]],
[[4,100,3710,32319,25736,34849,55608,56,0]],
[[4,100,3709,32217,25650,12833,62264,56,0]],
[[4,100,3709,32153,25548,52257,65336,56,0]],
[[4,100,3709,32127,25514,43553,58936,56,0]],
[[4,100,3709,32140,25582,60961,52280,56,0]],
[[4,100,3709,32140,25684,21537,48951,55,0]]]
},
{
"STMP": 11523744,
"TSMP": 156,
"STNM": "Face Coordinates and details",
"TYPE": "BBSSSSSBB",
"SCAL": [1,1,1,65535,65535,65535,65535,1,1],
"FACE": [
[[4,100,3709,32127,25821,56609,48952,56,0]],
[[4,100,3709,32178,25906,12833,42552,56,0]],
[[4,100,3709,32153,25958,26145,39223,55,0]],
[[4,100,3709,32178,26179,17185,35895,55,0]],
[[4,100,3709,32140,26231,30497,32567,55,0]],
[[4,100,3709,32089,26282,43553,39223,55,0]],
[[4,100,3709,32025,26333,56609,32567,55,0]],
[[4,100,3708,31973,24575,65313,29502,62,0]],
[[4,100,3708,31884,24575,65313,39231,63,0]],
[[4,100,3708,31845,24609,8481,42559,63,0]],
[[4,100,3708,31820,24643,17181,35902,62,0]],
[[4,100,3708,31820,24609,8477,22846,62,0]],
[[4,100,3708,31820,24575,65309,32574,62,0]]]
}
]

Thank you

That is clearly not working correctly.  However it did not happen in the quick test I did here.  So factory reset the camera, and try again.  If repeats, maybe it is an issue with a particular video mode.

I switched to 1080P 25fps, disabled all the fancy stuff (hypersmooth, high bitrate) and got the same result, both on Labs firmware and fw versions 1.42 and the latest 1.46. The issue seems to be there. Could I be reading the data wrong? I remember the FACE data in the HERO7 was a bit tricky, as th array had a bunch of empty values "unuse[17]" before the smile.
Here's one of my test files (just the raw gpmf data, the video is a single face): https://www.dropbox.com/s/6jfrer4wniqk33o/GL014218.bin?dl=0

HERO7 Face data was a screw-up :)
Seems you are have parsing issues.  The ID will always start at zero, and only increase with each new face detected.  This is the output from one of my tools (first payload)
"STRM": {
  "STMP": 43443,
  "TSMP": 13,
  "STNM": "Face Coordinates and details",
  "TYPE": "BBSSSSSBB",
  "structure (SCAL)": "1-axis 9 samples of type 'S' (UNSIGNED_SHORT)",
  "SCAL": [1, 1, 1, 65535, 65535, 65535, 65535, 1, 1],
  "complex structure (FACE)": "14-byte samples",
  "FACE": [4,99,0,29559,18507,8191,18689,0,0],
  "FACE": [4,100,0,29559,18507,8191,18689,0,0],
  "FACE": [4,100,0,29559,18568,8191,18628,0,0],
  "FACE": [4,100,0,29559,18568,8191,18628,0,0],
  "FACE": [4,100,0,29559,18568,8191,18628,0,0],
  "FACE": [4,100,0,29559,18568,8191,18628,0,0],
  "FACE": [4,100,0,29559,18568,8191,18628,0,0],
  "FACE": [4,100,0,29559,18568,8191,18628,0,0],
  "FACE": [4,100,0,29559,18568,8191,18628,0,0],
  "FACE": [4,100,0,29559,18568,8191,20813,0,0],
  "FACE": [4,100,0,29559,20752,8191,18689,0,0],
  "FACE": [4,100,0,29559,20752,8191,18689,0,0],
  "FACE": [4,100,0,29559,20752,8191,18689,0,0],
  "grouped data (FACE)": "13 samples"
},

ID is 0 for most of the samples, the highest it got to was 8.
"TYPE": "BBSSSSSBB" which is a structure like this:
struct {
uint8_t  version;
uint8_t  confidence;
uint16_t  ID;
uint16  facebox[4];
uint8_t  smile;
uint8_t  blink;
}

Thank you. I will review my process

It was indeed a bug in my parsing process. All good now. Thanks.
Blinks seem to half-fire constantly in some shots. Smile and everything else works great! https://youtu.be/MeMIK2KVz3A


AzureStackHub FabricLocation new API

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[X] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
04/2022
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
04/2022
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[X] PowerShell
[ ] CLI
[ ] Terraform
[X] No refresh required for updates in this PR

Contribution checklist:

[X] I commit to follow the Breaking Change Policy of "no breaking changes"
[X] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[X] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[X] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[X] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@akning-ms PR checks fail in cross version breaking changes validation for adding Default Response. The change is expected. Do we still need to file a breaking changes scenario for this and attend monday meeting?

Yes pl follow the breaking change wiki to follow up

In reply to:<PHONE_NUMBER>

@rkmanda May I have an ETA on when this PR can be expected to go in?

@akning-ms Could you pls help merge this PR? ARM team has signed off.


Reduce the duplicate computation and remove the thread divergence in crossentropy

introduce const int bt = b * T + t to reduce the duplicated computation. [Running 5+ trials on 3070, no obvious improvement in the performance.]
remove the thread divergence if (i < B * T). [Running 5+ trials on 3070, no obvious improvement/decrease in the performance.

Isn't the if necessary for safety?

Isn't the if necessary for safety?

Correct, too focused on the thread divergence and forget about the basics : )

Include a kernel taking padded probs.

This code is not used anymore due to fused classifier pr


Update data files

I ran update-data.sh to produce this PR.  It has been over 7 months since the data files were last updated, and there have been lots of gTLD and PSL changes since then.

I just saw that .amazon is now a thing.  Is that a good enough reason for AWS Labs to update certlint's data files?
I just ran update-data.sh again: https://github.com/robstradling/certlint/commit/48186e4f242eda56930e1728e32da6361bef2ca9


Inventory should support Google Groups membership

Currently there is no API to expand membership

A lot more details are added in #126.  Will keep both linked for now.

Initial PRs #159, #165

Duplicate of #126
Closing.


local: move orchestration logic out of node-red (into separate runtime)

probably doesn't belong here, closing for now


add test for gRPC call responses without Trailers

Related issue: https://github.com/bufbuild/connect-go/issues/279
You might need to change the error message or use a predefined message accordingly.
And tests for streaming calls are missing.
assert.True(
	t,
	strings.HasSuffix(connectErr.Message(), "server closed the stream without sending trailers"),
)

Thanks for the PR, @lixin9311! Really appreciate the contribution 😻  Someone from Buf will take a look within the next few days.

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

Thanks for the test! I've merged it in to #288 and will commit it as part of that PR.


Fix a bug that automatically chosen a stage periodically

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #1104
Does this PR introduce a user-facing change?:

Fix a bug that the web periodically jumped to the log viewer of the running stage

/approve


RPM package and systemd service file fixes (for CentOS 8)

On CentOS 8, the generated RPM packages have some issues that prevent them from installing properly, this pull request fixes those.

For some reason the CentOS 8 RPM builds are not automatically determining the libcedar.so()(64bit) shared library provides for the "softether-common" package.  Without this the "softether-vpnserver" package will not install.  Not sure if this is the "correct" fix but manually adding the provision to CMakeLists.txt (using CPACK_RPM_COMMON_PACKAGE_PROVIDES) seems to resolve the issue
The "softether-vpnserver" RPM package is erroneously including the /lib/systemd/systemd directory hierarchy in its %files listing which leads to a conflict with the systemd package when you try to install the RPM.  Excluding these paths with CPACK_RPM_EXCLUDE_FROM_AUTO_FILELIST_ADDITION helps resolve this issue

I also tried to address #1111 by updating the softether-vpnserver.service file

@stevemuskiewicz , sorry for delay. I tested, all good. many thanks!


Support for other modules (rotary, xentropy, layer_norm)

The non-flash-attention modules in this repository seem to not be installable with AMD cards. I would be happy to help address this but need some guidance.
Progress:

I adapted the setup.py for the rotary embeddings module to get it building on an AMD machine.

Errors:

import rotary_emb returns ImportError: libc10.so: cannot open shared object file: No such file or directory.
If you import torch then import rotary_emb, the result is ImportError: /.../x86_miniconda3/envs/flash2/lib/python3.11/site-packages/rotary_emb.cpython-311-x86_64-linux-gnu.so: undefined symbol: _Z17apply_rotary_cudaN2at6TensorES0_S0_S0_S0_S0_b.

Configuration:

PyTorch 2.1.2
ROCM 5.6
MI250 GPU

/.../x86_miniconda3/envs/flash2/lib/python3.11/site-packages/rotary_emb.cpython-311-x86_64-linux-gnu.so: undefined symbol: _Z17apply_rotary_cudaN2at6TensorES0_S0_S0_S0_S0_b
It looks like there are some CUDA operators in the rotary lib which is not working on AMD GPUs. Therefore you cannot use it directly without kernel supports even you can build it.

@howiejayz hello I need rotary on amd, is this cuda operator still missing?  can you please specify which ones?  I can get the fix prioritized

Thanks, I will give it a try


TypeError in issuer endpoint for Uplink

There seems to be a bug in the new uplink implementation caused by the model validation in the publish_schema() method under schemas. Trying to publish a schema causes The following Type error:
File "/usr/local/lib/python3.8/abc.py", line 102, in __subclasscheck__
    return _abc_subclasscheck(cls, subclass)
TypeError: issubclass() arg 1 must be a class

Initially I discovered this when using the uplink controller in the cloudapi app and thought this might have been caused by some fastapi code also. However, I found this to be a problem with the uplink implementation after trying this with an AcaPyClient instance straight from python shell:
cl= aries_cloudcontroller.AcaPyClient(base_url="localhost:3021", api_key="adminApiKey")
>>> from aries_cloudcontroller import SchemaSendRequest
>>> schema_definition = SchemaSendRequest(
...         attributes=['hello', 'world'],
...         schema_name='planet',
...         schema_version="0.1"
...     )
>>> cl.schema.publish_schema(body=schema_definition)
<coroutine object SchemaApi.publish_schema at 0x7fafba05ed40>
>>> await cl.schema.publish_schema(body=schema_definition)
  File "<stdin>", line 1
SyntaxError: 'await' outside function
>>> import asyncio
>>> loop = asyncio.get_event_loop()
>>> loop.run_until_complete(schema_definition = SchemaSendRequest(
...         attributes=['hello', 'world'],
...         schema_name='planet',
...         schema_version="0.1"
...
...
...
KeyboardInterrupt
>>> loop.run_until_complete(cl.schema.publish_schema(body=schema_definition))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/usr/local/lib/python3.8/site-packages/aries_cloudcontroller/api/schema.py", line 57, in publish_schema
    return await self.__publish_schema(
  File "/usr/local/lib/python3.8/site-packages/uplink/builder.py", line 95, in __call__
    self._request_definition.define_request(request_builder, args, kwargs)
  File "/usr/local/lib/python3.8/site-packages/uplink/commands.py", line 287, in define_request
    self._method_handler.handle_builder(request_builder)
  File "/usr/local/lib/python3.8/site-packages/uplink/decorators.py", line 62, in handle_builder
    annotation.modify_request(request_builder)
  File "/usr/local/lib/python3.8/site-packages/uplink/returns.py", line 64, in modify_request
    converter = request_builder.get_converter(
  File "/usr/local/lib/python3.8/site-packages/uplink/helpers.py", line 96, in get_converter
    return self._converter_registry[converter_key](*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/uplink/converters/__init__.py", line 54, in __call__
    converter = self._converter_factory(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/uplink/converters/__init__.py", line 114, in chain
    converter = func(factory)(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/uplink/converters/typing_.py", line 127, in create_response_body_converter
    return self._base_converter(type_)
  File "/usr/local/lib/python3.8/site-packages/uplink/converters/typing_.py", line 121, in _base_converter
    if issubclass(type_.__origin__, self.typing.Sequence):
  File "/usr/local/lib/python3.8/typing.py", line 774, in __subclasscheck__
    return issubclass(cls, self.__origin__)
  File "/usr/local/lib/python3.8/abc.py", line 102, in __subclasscheck__
    return _abc_subclasscheck(cls, subclass)
TypeError: issubclass() arg 1 must be a class
>>>

Interestingly, the stdout stream of the ledger shows that the schema does get published:
 14:35:14,523 indy.libindy.native.indy.services.ledger INFO   src/services/ledger/mod.rs:143 | build_schema_request() => Ok("{\"reqId\":1630593314522925994,\"identifier\":\"UXmTgD2HaqMkQKaPVUsJzD\",\"operation\":{\"type\":\"101\",\"data\":{\"name\":\"planet\",\"version\":\"0.1\",\"attr_names\":[\"world\",\"hello\"]}},\"protocolVersion\":2}")

This problem has been encountered - see pydantic here and fastapi here. So this appears to be a pydantic model/type checking issue that is possibly based on a Python standard library bug
@TimoGlastra You have pointed out that it likely is a problem here
I agree with that partly because the fact that the ledger does receive the definition implies that error is raised by the or one of the return types/models of the __publish_schema method.
Any thoughts and suggestions on how to fix that would be great.

Anyone able to reproduce this?
Do you have alternative ideas about the cause of this issue?
And how should one go about fixing this?

Most important to me is the latter question since the models are autogenerated so working around this manually in the model definitions would not be ideal. It would get overwritten by any consecutive release been created and used.

closing because fixed  by this pr
Uplink does not support Union types (see also this issue)
NOTE: This breaks transaction endorser logic for now


Minor improvements for basic lessons

Recently I've discovered a proselint tool and wanted to try it out.
Here is the first batch of changes suggested by the tool. While minor, I think that these changes make text better.
If this PR goes OK - I'll create a few more with fixes for other lessons and a CI check for it.

Thanks @brain-geek!


chore(package): update rollup to version 0.39.2

https://greenkeeper.io/

Current coverage is 45.77% (diff: 100%)

Merging #53 into master will not change coverage

@@             master        #53   diff @@
==========================================
  Files             1          1
  Lines           142        142
  Methods          43         43
  Messages          0          0
  Branches         12         12
==========================================
  Hits             65         65
  Misses           77         77
  Partials          0          0

Powered by Codecov. Last update 287856d...7e750f5


Tune the list of scoped available for Add to Scope operation

Problem
Currently, the sub-menu for Add to Scope operation contains all available scopes. But it could be more user-friendly if it would contain only relevant scopes.
Solution
Filter scopes that are assigned to all selected files out of the list of the sub-menu of Add to Scope operation.

declined: it is more user friendly to keep this list of scope the same


feat(consul,service_tags): Support escaped comma

#390
Example:
docker run -d \
      -p 4242:4242 \
      -e 'SERVICE_4242_TAGS=traefik.frontend.rule=Host: www.example.com\, Path: /api,traefik.enable=true' \
      company/service

Create only 2 tags into consul:

traefik.frontend.rule=Host: www.example.com, Path: /api
traefik.enable=true

I'm not sure about this code, mostly because there are no tests for this utility file and this seems like a good candidate for tests. @samber Mind adding tests? Once you do, I can totally merge this in for the next release :)

Done ;-)
Please note https://github.com/gliderlabs/registrator/issues/439 :trollface:

Is there an ETA on when this would actually be merged or released? I look forward to this for using Traefik.

@jakew no ETA at the moment. I'm not sure anyone knows how to make a release of registrator at the moment, which I'm trying to sort out (amongst other things).

Any update? I want to switch to Traefik and keep using this nice bit of kit  :D

This issue makes the registrator useless if you try to use it with traefik. Can somebody merge the PR? Its a quite small change and it is waiting for 6 months. Or should we consider this project dead and try to fix it ourselfs?

FYI for anyone affected by this: I just spent a while debugging issues with this and realized it's been merged but it's not part of the latest tag on Docker Hub.
https://hub.docker.com/r/gliderlabs/registrator/tags/
Make sure you're using gliderlabs/registrator:master.

could you please push this to "latest" tag do not want to use master image

@saurabhguptag We'll get to it when we get to it.


chown / chmod settings for containerized usage

Currently in containerized usage uses root perms

closed since just passing the right USER and GROUP into the docker run is Good Enough


[FEATURE REQUEST]

Maybe some file where you could add IPs + SSH Keys to allow other devices to connect to the same blog account to publish from other locations?
Hope this project isn't dead, cause it looks promising if it can be revived

thanks for the kind words, so far I haven't had a high demand for features, I'll see what can be done here. Of course I'm also looking forward to PRs =)


waitForResultWithin OMPromiseStateFailed crashes if no error code is present

        @throw [NSException exceptionWithName:@"WaitingForFufilledPromise"
                                       reason:@"The promise failed instead of getting fulfilled."
                                     userInfo:@{NSUnderlyingErrorKey: self.error}];

Fixed in version 0.4.1, which I just pushed to CocoaPods.


refactor: flag to use panda, closes #4255

Try out this version of Leather — download extension builds.

Refactored Flag component to use Panda.

@pete-watters the Radix card component has styling associated with it, where this doesn't. Also, in the card example, they're using flex to achieve the side-by-side layout, so doesn't implicitly make this layout.
<Flag /> here tries to create a component of a reusable a common layout pattern,


Update blog publish date for 1.28 feature blog for non graceful node shutdown to GA

Update the publication date for a 1.28 feature blog for non graceful node shutdown from August 15 (scheduled 1.28 release date) to August 16 per 1.28 enhancements tracking sheet
/assign @bradmccoydev
cc: @krol3 @mashby2022 @ramrodo @thschue

/milestone 1.28

/lgtm

@reylejano The date in the filename needed to be changed also right?

/approve

/approve


Can not login with password.

And I have a question about visual force page in classic when I disabled Third-Party Cookies.
I am using a visual force page as the call center adapter url. but in Salesforce classic mode it do not showing the visual force page at all. do you know is there any way to resolve it?

@aizazhakro could help take a look? thanks.


Add unit tests

Hooks should be tested, in the past we had issues caused by the root provider fetching blocks after unmount. We should re-introduce tests to the project.

This is now in main.


Mail we received to be reviewed

Reference: https://github.com/bithyve/internal-docs/issues/197

Closing this issue as discussed with dev.


Redeploy the demo

Hi, German&James, I've updated the navigation link, service icon and favicon. Please help redeploy the demo :-)

guys, we have a new release of STT + demo coming out today, please do not redeploy anything today (leave it for tomorrow), thanks a lot

@daniel-bolanos are saying that you will be updating the demo today?

yes, STT and TTS demos will be updated today

oh, shoot, sorry. The STT demo automatically deployed, and I just set up auto deployment for this one before I noticed this. Do we need to roll it back?

(Only master auto-deploys, and then only after the tests pass)


Tilemap vs sprite collisions arcade physique phaser 2.6.2

An error on the web site

I am trying to implement collisions between sprite and tile in arcade physique engine in phaser version 2.6.2
To learn how to use phaser for this task i am using this example:  http://phaser.io/examples/v2/tilemaps/map-collide
It doesn't seam to work because sprite character falls thru tile floor and doesn't collide with anything but world bound. Also it would be good to highlight importance off using body velocity so noobs like me wont think that they can use body position instead.

+1
I cant make Sprite > TileMap collisions work either.

im making something like this so it works:
in create:
game.physics.startSystem(Phaser.Physics.ARCADE);
game.physics.arcade.gravity.y = 1000;
var csv = '';
game.cache.addTilemap('cave', null, csv, Phaser.Tilemap.CSV);
var tilemap = this.game.add.tilemap('cave', this.tileSize, this.tileSize, layer);
var groundLayer = this.tilemap.createBlankLayer('ground', tileColumns, tileRows, tileSize, tileSize, layer);
	groundLayer.scale.set(1);
	groundLayer.renderSettings.enableScrollDelta = true;
	groundLayer.resizeWorld();
	groundLayer.debug = false;

var charactersLayer = game.add.spriteBatch();

var unit = game.add.sprite(...)
game.physics.enable(unit);
unit.body.collideWorldBounds = true;
charactersLayer.add(unit)

in update:
game.physics.arcade.collide(groundLayer, charactersLayer);

and ARCADE physics works only if you use velocity for moving objects (at least easiest way to make it work is to use velocity and acceleration)

My problem was that I passed tilemap.layers[0] to the collide function.
That is actually not the same as the result of tilemap.createBlankLayer().
A bit counter intuitive.

Man, I can't get this working by following their tutorial (or any other tutorial I've found) either. I've been trying to get collision working for like 2 hours now.

Andrew,
You should upload some code so others can see what's going on.
On Nov 19, 2016 11:05 PM, "Andrew Philpott"<EMAIL_ADDRESS>wrote:

Man, I can't get this working by following their tutorial (or any other
tutorial I've found) either. I've been trying to get collision working for
like 2 hours now.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/photonstorm/phaser/issues/2809#issuecomment-261757625,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABB3p-Wzaq98Icila-eISCA_6CSuRpyPks5q_8cZgaJpZM4Kb4Op
.

@hilts-vaughan I'm actually using the exact same code (aside from some variable names) as in this example right now just to make sure I'm not insane. This is the result I'm getting: http://d.pr/i/ZJB7.
The only thing that differs from that example is my JSON, so could that maybe be the issue?

yes scaling is totally useless i lost few weeks trying to fix it but end i got to conclusion that i would need rewriting most of engine so it would work and i resigned from scaling

Thank you for taking the time to open this issue. However, official support for Phaser 2 has now ended as we focus on preparing Phaser 3 for beta release. In a bid to keep the project tidy, and avoid confusion, we are closing all v2 related issues.
This does not mean your issue won't be looked-at. In November 2016 the Phaser CE (Community Edition) project began. The aim of Phaser CE was to allow the community to continue v2 development. Since then it has gained significant traction, with many updates and releases.
If you are still interested in having your issue investigated we strongly recommend testing the latest Phaser CE release. And only if the problem persists, opening an issue on the Phaser CE repo.


Add constantvallee.dev to list of 11ty websites

Adding my personal website to the list of 11ty websites.
Thank you for creating 11ty, it's pretty fun to work with.

Thank you!

Thank you!


Fails due to file exists on target branch

If any artifact exists in the target branch run that is not explicitly the coverage file, you'll get a conflict on download.
Download code coverage results from target branch
  ++ gh run list --status=success --branch=main --workflow=CI --event=push --json=databaseId --limit=1 -q '.[] | .databaseId'
  + LAST_SUCCESSFUL_RUN_ID=9388144932
  + '[' -z<PHONE_NUMBER> ']'
  + gh run download<PHONE_NUMBER> --name=unit-test-output --dir=.github/outputs
  error downloading unit-test-output: error extracting zip archive: error extracting "x-report.json": open .github/outputs/x-report.json: file exists

This likely occurs because to avoid the conflict, the previous step renames the coverage file to NEW_COVERAGE_PATH, but it only does so for the COVERAGE_FILE_NAME and not extraneous files in the artifact. I'd recommend renaming the output directory to --dir=.github/outputs/new_coverage and --dir=.github/outputs/old_coverage respectively.

Ah I see. So when you upload your code coverage artifact, you might actually chose to include other files as well such as different formats of the code coverage results.


Required optionals do not exist.

"Optional requirements" is an oxymoron. As such, I have fixed your README.md file. Please merge, or, preferably, rebase this into, or onto, respectively, your master branch! Thank you kindly.

Current coverage is 39.50% (diff: 100%)

Merging #1330 into master will decrease coverage by 0.14%

@@             master      #1330   diff @@
==========================================
  Files           109        109
  Lines         21174      21174
  Methods        2902       2902
  Messages          0          0
  Branches          0          0
==========================================
- Hits           8396       8365    -31
- Misses        12778      12809    +31
  Partials          0          0

Powered by Codecov. Last update 39646b6...61061b9

The most useful thing in this thread, so far, is a GIF from a children's movie. The only thing that could top the GIF is acceptance of the PR and a peaceful and rightful closure thereof.

I don't have much to say/decide, but my 5 cent for bike-shedding would be: "Dependencies" instead of "Requirements" (That works for both "Minimal Dependencies" and "Optional Dependencies"; plus I think it better captures the intend of the 'sentence').

Sorry, something that is optional is not a dependency. If you are dependent on something (such as oxygen) it's certainly not optional. This is in the general context of the application. In the sub context of the optional feature being used, then yes, they're a dependency, and a requirement (only of the optional feature, though, not the context of the readme file), but they're still neither, as they're optional.

Hm, ok.

I'm just going to leave this usage discussion here… and quote the most relevant paragraph:

I believe the confusion comes from the term "requirement". In the English language, a requirement is "a thing that is needed" or "a mandatory, compulsory, or necessary condition". However, in software engineering, the term requirement is simply a documented characteristic of a software system. The concept of optional and mandatory describe the priority of the documented characteristic of the software system.

The quality of that link is approximately equivalent to the quality of the prose in the offending document. As such reading it is entirely optional, and in no way required.

A comment from Iceland:

A comment from Iceland: Are they optional or are they requirements? They can't be both.
A comment from Canada: It's fairly clear from context that the error is the word "requirement".
A comment about Python developers: requirements.txt should probably hold product owner/user specifications about what the system should be able to do ;-)

I'm all for keeping this PR open for as long as possible to see how many more comments fredcooke will come up with.

You can still comment after it's closed ;-)

yeah, but you can't merge it


How to keep previous wizard data when we go back

Hi Michael,
First, thank you for creating this navigation library.
I am creating a wizard pattern using this library, and I am facing the issue to keep previous wizard data when we navigate back. I think it is a similar issue that was reported earlier. I have checked the suggested way but I think still I am missing something and the Back behavior is not working for me.
Could you please explain, where I need to make the changes to this sample to keep and bind the data to the previous wizard?

Hi @savan-bagtharia ,
Can you be a bit more specific? What data exactly are you trying to keep?

Hi Michael,
On the sample application, we can add an employee and for that employee, we can add/set value for the Firstname, LastName, Salary on different wizards, and goto next. Now if we go back to the previous wizard, then all values will not be populated there.
Let's say I have set some values like FirstName = ABC, LastName = XYZ, and Salary = 1000 and after moving to confirm screen, if we go back to other wizards (FirstName, LastName, Salary) then it should display all values.
Based on the documentation, I have changed the value for HistoricalNavigationMode but nothing worked.
Let me know if you need any further details.

Ok, I think I understand the problem.
Changing HistoricalNavigationMode  to UseSavedParameter won't help because that parameter doesn't do anything in populating the fields in the pages.
I think the best solution for this is a new history mode called "SaveViewModel". It will save a stack of all view models as you advance in history, and going back will use the same ViewModel instance from before.
Like with "SaveParamaterInstance" mode, it can lead to memory leaks because you're leaving instances in memory. But I think it's fine for many applications.
Does that sound like a good solution? If so, I can try and implement it.

@savan-bagtharia I went ahead and implemented "SaveViewModel" history mode. You can pull latest changes and see how it's used in "Add employee" flow. A new NuGet package (2.3.0) will be available within an hour. I'll update the documentation soon. Enjoy!

@michaelscodingspot Thank you for adding the new mode as SaveViewModel. That is working fine and keeping the data for the previous wizards.
But, I found one behaviour, when we go back to the previous wizard and if we go to the next then data will be lost for the next wizard.
e.g. Let's say, I have set values for FirstName = ABC, LastName = XYZ, and Salary = 1000. Now, when we go back to the previous wizard then data will be there (till FirstName). But after that when we go to the next wizard (LastName or Salary) then data is lost.
Can we get some flexibility to choose/decide, whether we want to keep data on Back/Next? I mean it could handle History Navigation completely in both way (start to end/end to start)

Sure thing.
As for what you're asking, it's a matter of usage. When clicking "Next" button, the current implementation navigates to a new page and not going forward in history. You can easily change it to check if "Forward" is possible and then navigate forward if yes or navigate to a new page otherwise.
In the sample application, in AddEmployeeViewModel.NextCommand change from
INavigationService navigationService = NavigationServiceProvider.GetNavigationServiceInstance();
AddWizardController controller = navigationService.GetController<AddWizard.AddWizardController>();
controller.Next();
RefreshButtons();

to
INavigationService navigationService = NavigationServiceProvider.GetNavigationServiceInstance();
AddWizardController controller = navigationService.GetController<AddWizard.AddWizardController>();
if (controller.CanGoForward)
{
    controller.GoForward();
}
else
{
    controller.Next();
}
RefreshButtons();

@savan-bagtharia Closing this issue as it seems all concern were resolved.


[Feature Request]: Add console log statements when enabling "debug mode"

What feature are you suggesting?
Add a setting which allows console log statements to be printed to assist with debugging. Default to off per Obsidian team's request.
What remote cloud services are you using, or suggesting adding the feature to?
No response
Ensure no sensitive information

[X] I ensure that no sensitive information is submitted in the issue.

Added in a75a12d. Let me know if any logs of interest are missing.


implement efficiency evaluation functions

Efficiency evaluation functions

construction cost function
Route efficiency function
Network efficiency function

Use a json file to store parameters used to adjust weights of different factors like transit time, transfers, stops, etc

#54


10th layer bug

🐛 Bug
If I add more than 10 layers, then I can still keep seeing the new ones on the menu, but I cannot select them and they will all show as layer 10.
To Reproduce
Steps to reproduce the behavior:

v = napari.view_image(image1)
v.add_image(image2)
Until layer 10, or a for loop with the same image would work too, I guess.

Expected behavior
Maybe if there is a limit at 10 one shouldn't be allowed to add new images anymore.
Environment
napari: 0.4.7
Platform: Windows-10-10.0.19043-SP0
Python: 3.8.12 | packaged by conda-forge | (default, Oct 12 2021, 21:19:05) [MSC v.1916 64 bit (AMD64)]
Qt: 5.15.2
PyQt5: 5.15.6
NumPy: 1.19.5
SciPy: 1.7.3
Dask: 2021.12.0
VisPy: 0.9.4

OpenGL:
- GL version: 4.6.0 NVIDIA 496.13
- MAX_TEXTURE_SIZE: 16384

Screens:
- screen 1: resolution 1920x1200, scale 1.0

Plugins:
- console: 0.0.4
- svg: 0.1.5

napari: 0.4.7
Your napari is quite old, current version is 0.4.12 with 0.4.13 nearing release.
I can't reproduce this problem on 0.4.12, so updating napari should solve the problem.

Thanks @psobolewskiPhD I confirm that updating napari solved the issue, sorry!


fix(aws): add missing base path to data routes

Related https://github.com/opennextjs/opennextjs-aws/issues/670
I was trying to figure out how to add (and run) a unit test in matcher.test.ts but then I saw this issue that mentions all unit tests are broken (?).
If e2e test(s) are required I'd appreciate any help as I'm not very familiar. And let me know if you need something else.
Thanks!

Unit tests should work again. I guess you could have one inside describe("fixDataPage".....

Added two tests. Test suite seems to run successfully.
Let me know if those are redundant, or you'd rather me add the base path to existing tests instead. I'm happy to update it as you see fit.

Looks like it failed on write permission reporting the test coverage in actions/test

So weird, the tests started failing locally for some reason now.

@alebelcor I think the reset are done on each describe, not on each test inside describe

@alebelcor sorry about that, looks like conico is right. you could add ts   beforeEach(() => {     NextConfig.basePath = "";   }); under line 471, or just do what you did before my earlier review.

beforeEach(()=> {
  NextConfig.basePath = undefined;
})```

if you add this under line 471 it seems to work here. might be that the global `beforeEach()` will only reset on each `describe`.

if you add this under line 471 it seems to work here. might be that the global beforeEach() will only reset on each describe:
beforeEach(()=> {
  NextConfig.basePath = undefined;
})```


Updated image path with CF199

Updated image path with CF199

 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.SathishJagajeevanram seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Added base map to dashboard

Co-authored-by: Margarita Salazar<EMAIL_ADDRESS>Co-authored-by: Jasmine Lane-Garcia<EMAIL_ADDRESS>Description

Loom summary

What work was done?

Describe in detail what's been modified/created
Added map to Dashboard

What feature / user story is it for?

As a user I can view a map on my dashboard
Any relevant links or images / screenshots

Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality) [X]

Change Status

Completed, ready to review and merge [X]

Checklist

My code follows the style guidelines of this project [X]
I have performed a self-review of my own code [X]
My code has been reviewed by at least one peer [X]
I have commented my code, particularly in hard-to-understand areas [X]
I have made corresponding changes to the documentation [X]
My changes generate no new warnings [X]
There are no merge conflicts [X]

Loom Summary


Very random weird error when using session sanic

Hi,
I'm using the sanic_session + asyncio-redis to handle the sessions in my web applications. But sometimes it generates below error(attached below) and happens randomly. It seems the operation on cookies are causing this issue, but I have no clue on the root cause, is there anyone who also met the same issue? Expecting suggestions and ideas. Thank you!
raceback (most recent call last):
8/27/2018 10:36:07 PM  File "/usr/local/lib/python3.6/site-packages/sanic/app.py", line 580, in handle_request
8/27/2018 10:36:07 PM    response)
8/27/2018 10:36:07 PM  File "/usr/local/lib/python3.6/site-packages/spf/framework.py", line 487, in _run_response_middleware
8/27/2018 10:36:07 PM    _response = await _response
8/27/2018 10:36:07 PM  File "/usr/local/lib/python3.6/site-packages/sanic_session/__init__.py", line 42, in save_session
8/27/2018 10:36:07 PM    await self.interface.save(request, response)
8/27/2018 10:36:07 PM  File "/usr/local/lib/python3.6/site-packages/sanic_session/base.py", line 129, in save
8/27/2018 10:36:07 PM    self._set_cookie_expiration(request, response)
8/27/2018 10:36:07 PM  File "/usr/local/lib/python3.6/site-packages/sanic_session/base.py", line 35, in _set_cookie_expiration
8/27/2018 10:36:07 PM    response.cookies[self.cookie_name] = request['session'].sid
8/27/2018 10:36:07 PM  File "/usr/local/lib/python3.6/site-packages/sanic/cookies.py", line 59, in __setitem__
8/27/2018 10:36:07 PM    self.headers[cookie_header] = cookie
8/27/2018 10:36:07 PM  File "/usr/local/lib/python3.6/site-packages/sanic/server.py", line 54, in __setitem__
8/27/2018 10:36:07 PM    return super().__setitem__(key.casefold(), value)
8/27/2018 10:36:07 PMAttributeError: 'MultiHeader' object has no attribute 'casefold'

@tonyzhangcn It would be very helpful if you could provide some more information about the key in this case. For some reason it's not a string as it should be.
To me, this looks like the issue should be raised in the Sanic repo instead of here. As somehow a MultiHeader ended up being used when trying setitem instead of a string.

Please reopen issue if you sill have such problems


Update 31_card_docs_update.sql doc about a style added

Hi. I think it was done here #355

I see! Can you make the change in the other pull request, then? Closing this one.

Hi, please take a look at #365. I hope it is ok now.


Better handling of stacked spatial dimension in matrix argument

There is a nasty misalignment of the matrix coordinates with the cutout coordinates in the land-use example. To ensure compatibility of matrices and cutout in the future, this PR proposes some changes to the convert_and_aggregate function.
Changes

Check coordinate alignment of a DataArray-matrix with cutout.grid in convert_and_aggregate.
convert_and_aggregate now supports a matrix of shape (N, X, Y). In this case the matrix must be a xarray.DataArray with dimensions x and y equal to those of cutout.data.
Due to ambiguity, conversion functions (.pv(...), .wind(...) etc.) now raise an ValueError if shapes and matrix are given.

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I tested my contribution locally and it seems to work fine.
[x] I locally ran pytest inside the repository and no unexpected problems came up.
[x] I have adjusted the docstrings in the code appropriately.
[x] I have documented the effects of my code changes in the documentation doc/.
[ ] ~I have added newly introduced dependencies to environment.yaml file.~
[x] I have added a note to release notes doc/release_notes.rst.
[x] I have used pre-commit run --all to lint/format/check my contribution

Codecov Report

Merging #203 (1821b47) into master (40dd1fc) will decrease coverage by 0.70%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   75.42%   74.71%   -0.71%
==========================================
  Files          18       18
  Lines        1481     1495      +14
  Branches      192      204      +12
==========================================
  Hits         1117     1117
- Misses        301      312      +11
- Partials       63       66       +3

Impacted Files
Coverage Δ

atlite/convert.py
76.52% <33.33%> (-4.04%)
:arrow_down:

atlite/gis.py
90.36% <0.00%> (-0.81%)
:arrow_down:

Continue to review full report at Codecov.

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

I would merge this if no one has an opposing opinion @euronion @fneum ?

No opposition from my side!

@FabianHofmann Sorry, I'm not happy with it. See comments.

@euronion thanks for your thoughts! Two things

You are right about the passing a non-stacked 3-dimensional DataArray as a matrix. This bloats the set of possible arguments.
However passing a DataArray with stacked coordinates was supported all the time and I strongly advocate to support it in the future. Two weeks ago I was "debugging" with an atlite user for 1 and a half hours, before recognizing that his matrix was stacked in the wrong way. The atlite conversion for a stacked coordinate axis is that both x and y are increasing, but x rolls over all values for each value in y, so y is monotonically increasing and x not (just like in cutout.grid). This is achieved by ds.stack(spatial=['y', 'x']). When matrix is a DataArray, the alignment of the stacked coordinates with this convention was not checked before. Now the alignment is double-checked before converting it to a sparse matrix. The example and the dynamic line rating code had a wrong stacking, this was also addressed.

1. You are right about the passing a non-stacked 3-dimensional DataArray as a matrix. This bloats the set of possible arguments.

I do like the idea of being able to pass an unstacked 3D DataArray. To me passing a 3D array feels more natural. But maybe not by overloading the same argument, but with a new argument, e.g. matrix_3d?

2. However passing a `DataArray` with stacked coordinates was supported all the time and I strongly advocate to support it in the future.

Of course, keep the backwards compatability! Sorry, I missed that when looking at the code.

This is achieved by ds.stack(spatial=['y', 'x']).

I get your point. It's just that I don't like hardcoding things like 'y','x' and 'spatial' into the code, because that's what causes misalignment if the hardcoding is in multiple locations.
You can access the order of stacked dimensions via
da.coords.indexes['spatial'].names        # FrozenList(['y', 'x'])

Alright, I think we can pull this in. If we want to support a 3d matrix as a separate argument, we can tackle that another time. Any objection @euronion?

LGTM


Support Backend acts as pure Zipkin collector

@peng-yongsheng @adriancole
This PR brings a new feature in SkyWalking. In old zipkin-skywalking adaptor, I can't make it 100% production ready(even 90% is hard). But in these months, I work around SkyWalking in service mesh, a pure Zipkin native format collector takes my attention.
In SkyWalking/Istio/Envoy solution, we could work in piece very well. And we miss tracing. SkyWalking's agents could work(that is for sure), and at the same time, Zipkin ecosystem could work too, especially, could be used in the production env and sampled based, because it would affect the metric, mesh observability provides 100% traffic/topology metrics.
So, I bring this PR here. Hope you will like this. Used to talk about this in #2044 (6.3.0 feature), I just speed this up.

Coverage decreased (-0.02%) to 13.88% when pulling 45a18f477978b4f5e134e66929ec25fd9c38ee21 on zipkin-trace into 16aca37e70baad8d967c919dab5acc2ae6aca834 on master.


Support deleting databases

Update protocol to support new methods:

client.deleteDatabase()
client.deleteDatabases()

Leverage new storage node endpoints from:

https://github.com/verida/storage-node/issues/34
https://github.com/verida/storage-node/pull/35

This is now included in Acacia release.
https://github.com/verida/verida-js/pull/238/commits/dee5a894c603d1e0e4d4e90087eaad1a2b56be71
https://github.com/verida/verida-js/pull/238/commits/4c36170249cd3b1a8b79588f83d9d1b0930483ab


run qemu for a foreign architecture

Now that we can run vmtest within a given chroot, we could host the chroot for a foreign architecture and run a vmtest in that chroot using a kernel matching that ar hitecture.
$ uname -a && cat /etc/lsb-release && RUST_LOG=debug cargo run -- -a aarch64 -k arm64/kbuild-output/arch/arm64/boot/Image.gz -r ../rootfs/ubuntu-lunar-arm64/ 'uname -a && cat /etc/lsb-release'
Linux surya 6.5.0-14-generic #14-Ubuntu SMP PREEMPT_DYNAMIC Tue Nov 14 14:59:49 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=23.10
DISTRIB_CODENAME=mantic
DISTRIB_DESCRIPTION="Ubuntu 23.10"
    Finished dev [unoptimized + debuginfo] target(s) in 0.05s
     Running `target/debug/vmtest -a aarch64 -k arm64/kbuild-output/arch/arm64/boot/Image.gz -r ../rootfs/ubuntu-lunar-arm64/ 'uname -a && cat /etc/lsb-release'`
=> Image.gz
===> Booting
===> Setting up VM
===> Running command
Linux (none) 6.6.0-rc5-ga4a0c99f10ca-dirty #40 SMP Thu Oct 26 18:28:11 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=23.04
DISTRIB_CODENAME=lunar
DISTRIB_DESCRIPTION="Ubuntu 23.04"

This is based on the discussion in #33. I had done the chroot part but dropped the ball on the architecture part.

putting this back into my queue... need to solve testing.

tests fixed.

lgtm! looks like we should land the Target::default() PR first though

yes. I will rebase on top and update this PR once #50 is merged.

rebased on top of #50

Oh, pls update docs too


[Tizen] Remove xwalk-launcher and xwalkctl.

Consequently remove the entire application/tools/linux directory.
Now that we have dropped support for using Crosswalk in shared process
mode on Tizen (commit e4f3799ac), nothing in that directory gets built
or used anywhere anymore.
BUG=XWALK-3203

lgtm


Dynamic bundles

Describe the solution you'd like
It would be good if we could set a bundle to monitor a folder to update and display its contents like it would now if we drag and drops folders in.

This kind of defies the use of Bundles. They are meant to provide a quick access to select files/folders. This can be already achieved by navigating to directory and having this kind of functionality in Bundles is redundant. Even more, single bundle wouldn't be able to fit large directories in the first place!

@d2dyno1 We could eventually figure out which items a user opens the most and prompt them to add the item(s) to a bundle

@d2dyno1 We could eventually figure out which items a user opens the most and prompt them to add the item(s) to a bundle

Good idea! However it's not related to the issue.


Swaylock support

I quite like betterlockscreen and right now it seems wayland lockscreens are all pretty basic. It would be great for betterlockscreen to have wayland/sway support

@ibrokemypie This is just a bash script that calls i3lock. i3lock would have to supprt wayland.

For sway at least, which is based upon i3, there is swaylock which is very similar to i3lock, however has some different options regarding indicator placement. It is otherwise a dropin replacement, the majority of the extra work would just be working out the different indicator placement configuration

I think we are open for a PR about that, as I personally have no experience with sway and currently not the time to get into it. It hink a possible change should abstract the option-names based on some config/tooling we can detect either i3 or sway.
Please Note: We are preparing a big update with mulit-monitor branch so if you are interested working on this please base it on multi-monitor :)

Just in case this gains traction, there seems to be a swaylock fork similar to i3lock-color https://github.com/mortie/swaylock-effects that already includes some features (a clock for example) that betterlockscreen uses.

Swaylock / swaylock-effects support images per output (https://man.archlinux.org/man/swaylock.1) and swaylock-effects also effects on it so I see no purpose for sway-support and complicating the Script even further (in case of swaylock-effects) so I'm going to close it for now.


PythonTools project build fails

I tried to build the latest (clone) and received this error that
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0234
The type or namespace name 'Web' does not exist in the namespace 'Microsoft.VisualStudio' (are you missing an assembly reference?)	PythonTools	 "Microsoft.VisualStudio.Web.WindowsAzure.Contracts"
I have checked that I have all the required components installed - like Azure SDK for .NET, Visual Studio SDK and VS Test Host release.
I also tried in multiple machines.

The requisite assembly should be in:
C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\Extensions\Microsoft\Web Tools\Azure\WindowsAzure
(adjusted for Visual Studio version that you're building for)
Can you please check whether it is physically there?

I checked in the above location and its not present there.

Which version of Azure SDK for .NET are you using?

Azure SDK for .NET 2.8.1

Any update on this?

Now that I look at the assembly name closely, I think this is a part of Web Tools rather than Azure. If you go to Tools -> Extensions and Updates, do you have the item titled "Microsoft ASP.NET and Web Tools" in the list under the "Installed" tab there?
If not, then it might have become an optional VS component that can be deselected during VS setup. Try doing repair/change on VS itself, and see if you can locate it in the component list.


Changelog for v3

👋 This issue is a new instance of https://github.com/puleos/object-hash/issues/87. v3.0.0 was released recently but README → Changelog only mentions 2.0.0:

Although it is possible to look at git log and find https://github.com/puleos/object-hash/pull/114, it is probably too much of a detour for such a popular library 🙂

Does v3 break compatibility with hashes?

Yes, v3.x generates different hashes but is otherwise compatible.
Feel free to open a PR.

@addaleax can you please explain what do you mean by "different hashes"?
// V2
hash({foo: 'bar'}) // => '67b69634f9880a282c14a0f0cb7ba20cf5d677e9'

// V3
hash({foo: 'bar'}) // => 'diferent hash' ????????????

So, sha1 in v2 is not same in v3?

@JaLe29 check https://github.com/puleos/object-hash/pull/114 — previous version wasn't distinguishing between typed arrays

It's better not to release a breaking change than to release it with zero (changelog) documentation.


It doesn't work vue 3

For me using the code in the docs
main.ts
const app = createApp(App);

import VueTelInput from 'vue-tel-input';
import 'vue-tel-input/dist/vue-tel-input.css';
app.use(VueTelInput)

It gives me [Vue warn]: Component is missing template or render function.

I have 6.0.0-beta.7 in my package.json

We're using 6.0.0-beta.7 as well and it's working for us with Vue3. Some of the options aren't working, but it loads and mostly works.
We're importing it locally within a single .vue component, not in the global config (we load components async)
import {VueTelInput} from 'vue-tel-input'
import 'vue-tel-input/dist/vue-tel-input.css';

export default defineComponent({
  components: {VueTelInput},
  ...
}

Im running into the same error with 6.0.1 :/

+1

https://www.npmjs.com/package/vue3-tel-input-js

Any update for this issue i have the same problem

same problem

import { VueTelInput } from 'vue3-tel-input-js'
  <vue-tel-input
          v-model="phone"
          :class="disable ? '!border-bg !bg-bg' : ''"
          :disabled="disable"
          enabled-country-code="true"
          mode="international"
          :name="name"
          style-classes="telinput"
          :type="type"
          :v-bind="options"
          :value="inputValue"
          @blur="handleBlur"
          @input="handleChange"
          @validate="validate"
        ></vue-tel-input>

try using import "vue-tel-input/dist/style.css";
It worked for me in vue3 with vue-tel-input version 8.1.0

the updated version import 'vue-tel-input/vue-tel-input.css';


Core dump after ^C when polling RTSP

On Ubuntu 22.04 (arm64) with py3.10.12, when I have a simple app that polls an RTSP stream with framegrab, when I hit ^C it shows KeyboardInterrupt, but doesn't exit right away.  If I ^C again, I get this error:
KeyboardInterrupt
^CException ignored in: <module 'threading' from '/usr/lib/python3.10/threading.py'>
Traceback (most recent call last):
  File "/usr/lib/python3.10/threading.py", line 1567, in _shutdown
    lock.acquire()
KeyboardInterrupt:
terminate called without an active exception
Aborted (core dumped)

Core dumps indicate something fundamentally wrong.

This was resolved in https://github.com/groundlight/framegrab/commit/0549bb80b33116fb9aaa04878edd5b402fd602c3


Feat/mm task admin app

Hello @Honeyloveet , the UI for the tasks general page and that showing task details has been completed. Kindly review it. Thank you

@tamsay There is no check on this PR, What did you do?

@peculiaruc I didn't get you. The PR is for the tasks general page, showing all tasks and details of each task.

Alright. Good job. Keep it up

Thank you.


Bind Ray ClusterRole to the new operator ServiceAccount

Fix a missing update from #125, to bind the Ray ClusterRole to the CodeFlare operator ServiceAccount.
It'll be simplified once project-codeflare/multi-cluster-app-dispatcher#635 is implemented.

/assign @anishasthana @sutaakar @jbusche

/lgtm
Though it doesn't have any effect yet as operator still use cluster edit role.

/lgtm
Though it doesn't have any effect yet as operator still use cluster edit role.

It doesn't seem like Kuberay resources are aggregated to the edit ClusterRole, kubectl get clusterrole edit -o yaml | grep ray doesn't return anything. If that is correct, then it should fixed the RBAC issue reported by @jbusche.


Pooled Contextual Embedding

Hello, how can I get PooledContextualEmbeddings as mentioned in http://alanakbik.github.io/papers/naacl2019_embeddings.pdf

You would need to create a data structure for keeping track of every unique word encountered in the text and its previous embeddings. This is done outside of the Keras/TF model in this repository. You could:

modify the code for the InputEncoder class in https://github.com/kensho-technologies/bubs/blob/master/bubs/helpers.py to add memory
implement a separate class outside of bubs.

If you go the former route, we are accepting pull requests:-)
Please refer to the beginning of Section 2 of the paper http://alanakbik.github.io/papers/naacl2019_embeddings.pdf. The embed() function they mention is equivalent to the example in https://github.com/kensho-technologies/bubs/blob/master/README.md - accepts text, outputs embeddings for each word. But the second part, the memory, is not implemented in bubs.

It requires an embed() function that produces a
contextualized embedding for a given word in a
sentence context (see Akbik et al. (2018)). It also
requires a memory that records for each unique
word all previous contextual embeddings, and a
pool() operation to pool embedding vectors.


Applying filters to dynamically generated target in a route

Hi,
We are currently exploring Spring gateways(Zuul, Spring Cloud Gateway, etc) for a specific use case in our environment.
Depending on the request header content coming in, we need to perform a reverse lookup(could be a mapping file or stored in DB) to determine which one of three downstream services(A, B, C) to proxy to.
After initial exploration and research, it appears one approach is to modify the  GATEWAY_REQUEST_URL_ATTR in a custom filter.
    @Bean
    public RouteLocator dynamicRoute(RouteLocatorBuilder builder) {
        return builder.routes()
                .route("dynamicRoute", r -> r.alwaysTrue()
                        .filters(f -> f
                                .filter(new DynamicGatewayFilter())
                                .requestRateLimiter(c -> c.setRateLimiter(demoRate()))
                        )
                        .uri("url://placeholder")
                )
                .build();
    }

However, by using this approach, as we understand it, we cannot apply filters(rate limiting, circuit breaker, etc) to each downstream service as it will apply globally to the RouteLocator route.
We want to limit the rate to each of the three downstream services(A, B, C) to 10 requests/sec per downstream service, not to the route as a whole, as 10 requests/sec to service A will prevent B or C from receiving requests.
Is there a way to do this cleanly?
Thanks!
Jon

Are you making sure your custom filter is running first?

Also the request rate limiter has an optional key redolver, so you should be able to put something as a request attribute, then look that up in the key resolver.

Hi @ryanjbaxter, yes we have confirmed that the filter is replacing the GATEWAY_REQUEST_URL_ATTR. We have confirmed it working once we changed the Order to a relatively large number such as 12345. We have verified that it does proxy to a test target, ie. httpbin.org/status/402 etc.
Hi @spencergibb, thank you, we will look into that. Seems somewhat related to the following issue: https://github.com/spring-cloud/spring-cloud-gateway/issues/924.

Hi,
I think I was able to get rate limiting working using the KeyResolver.
An example in case anybody was in the same boat.
.route(r -> r.path("/")
                        .filters(f -> f
                                .filter(dynamicGatewayFilter())
                                .filter(((exchange, chain) -> {
                                    exchange.getAttributes().put("RATEKEY", exchange.getRequest().getHeaders().getFirst("CUSTOM-HEADER"));
                                    return chain.filter(exchange);
                                }))
                                .requestRateLimiter(c -> c
                                        .setRateLimiter(demoRate())
                                        .setKeyResolver(exchange -> Mono.just(exchange.getAttributes().get("RATEKEY").toString()))))
                        .uri("url://placeholder")
                        .id("custom_route")
                )
                .build();

We want to be able to apply the same concept to hystrix filters as well. It appears that hystrix does not resolve off of KeyResolver.
Would configuring the hystrix config in a route work such as the following ?
.filters(f -> f.hystrix(config -> config.setName("DYNAMIC_HOST_TARGET_A_OR_B_OR_C")
If hystrix cannot be configured to do this, can one modify override the route in an exchange after entering an existing route by matching on route predicates?

Hi @spring-issuemaster , I just realized I cannot reopen this issue as it was not closed by me. Can you reopen?
Thanks,
Jon

@jlndev so you want to set the circuit breaker name based off some dynamic property of the request?  The circuit breaker wraps a route, not a specific request, so I dont see why you would want the name of the circuit breaker to be based on a property of the request.

Hi @ryanjbaxter ,
When we configure a placeholder route, that route is used to do a reverse lookup and proxy requests to services A or B or C depending on the content of the headers. If A goes down, the circuit breaker would trip, preventing subsequent traffic to be proxied to services B or C until the circuit opens for A. The gateway would block traffic to B and C for no reason.
In the rate limiting example in the beginning of this thread, we have the same use case, but since RateLimiter filter keys off of KeyResolver, we can rate limit and isolate services A, B, and C from each other(B cannot take bucket capacity from C, A from B, etc)
My questions are as follows:

What is the purpose of the hystrix circuit breaker name?

.filters(f -> f.hystrix(config -> config.setName("WHAT_DOES_THIS_DO")
2) If Hystrix can only be tied to a route, is there any way we can define a new route mid exchange, mutate the exchange and configure the new route?

Any other ideas on how this could be implemented?

Thanks!

The name is the key for configuring the circuit breaker in config.

@spencergibb, A, B, C are not different, they are just three identical monolith instances that we are trying to strangulate. The only reason we have to do a reverse lookup is to match the correct tenant/request to the correct legacy monolith(A|B|C). Our tenants/clients are tied to a specific monolith instance.
If A comes down, we still want tenants that were provisioned to B and C to continue to be able to call their respective monoliths. Essentially 3 circuit breakers for the placeholder route.
Can you give a little more detail on the custom filter for our further investigation?

Isn't the simplest answer to have 3 routes?

It is, but thats assuming we have all the predicates available to sort into the right routes to begin with.
We don't know where the request is going to be proxied until it enters a route and the custom filter dynamically determines which service to route to given the tenant id in the header. All tenants hits one endpoint(gateway) so there is no host header to each service A, B, C.
Twos ideas , not sure if supported or will work.
1)Define 3 routes to A, B, and C.  Intercept the request before it hits a SCG route, grab the header/tenant/client from HTTPServletRequest and add a ROUTEID header value. Then it will fall into the 3 pre defined routes matched on ROUTEID header value. (not sure if there is a clean way to do this in SCG)

Define the placeholder route, define the 3 routes to A, B , C.  When requests funnel through the placeholder route,  mutate the exchange route attribute and override route with route A, B, C (not sure if this is possible)

Thoughts?

How about a custom predicate?

Oh.
Will look more into this. Thanks!
(Sorry, new to Spring and SCG)


structurizr-cli takes 10 seconds to start

Hi,
I just wanted to check if this is a 'just my machine' thing, or to be expected.
The CLI (v1.4.5) and the latest release build (v1.5.0) both take 10 seconds to get to the point of printing Structurizr CLI v1.5.0, even if no command line arguments are given.
I know it's Java, and so I don't have an expectation that it will run immediately, but I just thought I should reach out and see if that's expected, or if you have a hint on how to debug why that might be the case.
Cheers,
Kirk

I just downloaded a fresh, separate copy of v1.5.0 and it's only 1-2 seconds for me (Java 13 on macOS Catalina).

Ok, cool - it's a "me" thing then... I'll close the issue. Cheers!

Just in case anyone is interested... it appears that it is something to do with how my wifi connection is configured. Using a different wireless access point makes a dramatic difference. I think something in the java classloading sequence doesn't like the way ipv6 is set up on my network:
"${JAVA_HOME}/bin/java" -Xlog:class+load:file=classload.txt -jar $SCRIPT_DIR/structurizr-cli-1.5.0.jar "$@"
Here's the 5 second delay in the classload log file:
[1.243s][info][class,load] java.net.Inet6Address source: shared objects file
[1.244s][info][class,load] java.net.Inet6Address$Inet6AddressHolder source: shared objects file
[6.248s][info][class,load] sun.net.InetAddressCachePolicy source: jrt:/java.base
[6.248s][info][class,load] sun.net.InetAddressCachePolicy$1 source: jrt:/java.base

Cheers,
Kirk

If you're on a Mac, there is/has been an outage that's causing a delay to starting apps -> https://twitter.com/lapcatsoftware/status/1326990296412991489 ... that might also be related.

Ugh... this was the cause. Sorry for assuming it was your app, and not my computer!
http://justthesam.com/2016/10/fixing-java-net-inet6addressimpl-lookupallhostaddr-slowdown/


Prevent infinite recursion by breaking on outer loop. All unit tests pass.

I added a unit test which fails (infinite loop) on current jael, but passes with this change.

Hi, and thanks for sending this PR! This will be present in the next 2.x release of Jael. I'll soon release a 3.x which uses reflectable instead of mirrors, probably.

Hi, @bpilot! Thanks again for this PR.
All Angel subprojects have been consolidated into a single monorepo: https://github.com/angel-dart/angel.
Please fork the monorepo, and open a new PR there instead.

Hi, @bpilot! Thanks again for this PR.
All Angel subprojects have been consolidated into a single monorepo: https://github.com/angel-dart/angel.
Please fork the monorepo, and open a new PR there instead.


display-dependency/plugin/property-updates: Make use of MessageUtils.getTerminalWidth

https://github.com/mojohaus/versions-maven-plugin/pull/564 introduced the feature to make the "displayTerminalWidth" configurable instead of having it fixed to anachronistic 80 chars.
as @rfscholte pointed out there is a method MessageUtils.getTerminalWidth which allows to detect the current terminal width if JANSI is available (since Maven 3.5.0).

issue is rejected, see #572


FileDropZone with touch backend

Hi there and thanks for your work... Very usefull !
I was wondering about this : https://react-dnd.github.io/react-dnd/docs-faq.html#112
How do I register a drop target for the native files?

If you are using the HTML5 backend, you can register a drop target for one of the NativeTypes it exports

But what if we use TouchBackend ?
Thanks

It's not possible. Only the native drag n' drop API allows you to drag things from outside your application.


Subdomains

This works fine for top level domain accounts but I'm having trouble getting it to work with sub domains.
For example, using the credentials Ntlm.setCredentials('TEST.COM', 'JACK', 'JACK') works fine.

However using the credentials Ntlm.setCredentials('SUB.TEST.COM', 'JACK', 'JACK')  tells me my credentials are wrong even though I know they are definitely right.

I appreciate any help in advance.

Hi. Is there still an issue here? I'd like to use this library in my project and I'm not sure if this works :)

Hey. That would be my guess. I haven't gotten around to fix this yet. In fact I haven't even looked into this enough to be able to determine if this is an actual bug or if this is something on the server side (e.g. https://support.microsoft.com/en-us/kb/896861). However, this will only affect you if you plan on using active directory subdomains.

Ok,
Thanks for clarification.


vulnfeeds: Write a .notes file when human intervention is needed.

Convert previous warnings into an of notes which are accumulated.
These are then written alongside the .yaml to indicate that human eyes
are required.
Also implement a false positives input file to allow users to ignore
such cases in the future.
Document the workflow as well for human triagers.

See the README.md changes in this PR for the proposed workflow for triagers.


Infrastructure: Helm charts for Mock Classifier

As a DevOps engineer
I want to have a helm chart for the Mock Classifier application
So that I can deploy the Mock Classifier to an already provisioned AKS cluster
Out of scope: provision of any cloud resources
Acceptance Criteria

[ ] Created a folder structure for helm charts for all applications in the Infrastructure repo
[ ] Created a helm chart for the Mock Classifier in the pre-agreed structure
[ ] Ensured that the helm chart deploys the application to the AKS cluster
[ ] Ensured that the values.yaml contains correct application configuration

@yahya130 @decodingahmed Where is the PR for this issue? It is not linked by the look of it. Thanks

Ignore. Added in error

All acceptance criteria met


The placeholder font-size of Input and Select should be same

Bug, feature request, or proposal:
Bug
What is the expected behavior?
The placeholder font-size of Input and Select should be same
What is the current behavior?
According to my usage, I found that the font-size of Input and Select are different:
Input: 20px
Select: 16px
So currently, I just reset it dynamically when the select change.
  setCssDynamical($event) {
    console.log($event);
    $event.source.trigger.nativeElement.childNodes[0].setAttribute("style", "font-size: 20px;font-weight: bold")
  }

      <md-select
        *ngIf="showTokenSelect"
        (change)="setCssDynamical($event)">
        <md-option *ngFor="let domain of domainsDemo" [value]="domain.value"
                   style="font-size: 20px">
          {{domain.viewValue}}
        </md-option>
      </md-select>

I think maybe these font-size should be same.
Which versions of Angular, Material, OS, browsers are affected?
"@angular/material": "^2.0.0-beta.2",
Is there anything else we should know?
According to my understand, I think just change the css of mat-select-trigger
.mat-select-trigger{display:flex;align-items:center;height:30px;min-width:112px;cursor:pointer;position:relative;box-sizing:border-box;font-size:16px}

font-size 16px to 20px

it is not just font size - there is general misalignment in select and input when used side by side

placeholders and underline are on different levels

I also has same issue as @janisr . any workaround while proper fix is released ?
I am not able to override .mat-select-trigger attributes from the component.css file.

This looks like a dupe of https://github.com/angular/material2/issues/2127. Input inherits its font size while select prescribes 16px. They should handle it the same way.


Update plugin-teamconcert-git.yml

Add Sahil Mansuri
Link to GitHub repository

When modifying release permission
If you are modifying the release permission of your plugin or component, fill out the following checklist:

### Release permission checklist (for submitters)
- [ ] The usernames of the users added to the "developers" section in the .yml file are the same the users use to log in to [accounts.jenkins.io](https://accounts.jenkins.io/).
- [x] All users added have logged in to [Artifactory](https://repo.jenkins-ci.org/) and [Jira](https://issues.jenkins.io/) once.
- [ ] I have mentioned an [existing team member](https://github.com/orgs/jenkinsci/teams) of the plugin or component team to approve this request.

When enabling automated releases (cd: true)
Follow the documentation to ensure, your pull request is set up properly. Don't merge it yet.
In case of changes requested by the hosting team, an open PR facilitates future reviews, without derailing work across multiple PRs.
Link to the PR enabling CD in your plugin

### CD checklist (for submitters)
- [ ] I have provided a link to the pull request in my plugin, which enables CD according to the documentation.

### Reviewer checklist
- [ ] Check that the `$pluginId Developers` team has `Admin` permissions while granting the access.
- [ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
- [ ] If security contacts are changed (this includes add/remove), ping the security officer (currently `@Wadeck`) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it.

I have unchecked

because that didn't happen yet.

@sbabno to approve for @sahilkmansuriibm

@NotMyFault , @timja , I am the member of the team who maintaining the Team Concert Git Plugin. I am releasing the new version of this plugin and I am currently stuck on last step where it will upload the new plugin version to Jenkins artifactory.
Hence, I would appreciate if you review and merge the PR so I can proceed with release process. Thanks.

I have unchecked  because that didn't happen yet.

sbabno is a maintainer


Fix "cannot import name 'force_text'" error to support Django > 4.0

The "ImportError: cannot import name 'force_text' from 'django.utils.encoding'" error occurs when a package or module tries to import the force_text function from Django's encoding module, but the function has been removed and replaced with force_str in Django version 4.

 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.


Allow bosh_cli to stop a job or all jobs in deployment

User can stop all instances of the job just skipping index in the stop command.
User can stop all jobs of the deployment skipping job name in the stop command.

--hard option means that corresponding vm(s) should be deleted
and disappear from 'bosh vms'.
Signed-off-by: Alexey Miroshkin<EMAIL_ADDRESS>

merged here: https://github.com/cloudfoundry/bosh/commit/43b2ab5848ef0cf614604fee1a49363205559e5b Changed :all to '*' to be consistent with other commands.


Implement Platform::parse_types_from_source

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 6 committers have signed the CLA.:white_check_mark: joshwatson:x: Brian Knudson:x: plafosse:x: KyleMiles:x: D0ntPanic:x: CouleeAppsBrian Knudson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

ok I think I have squashed and fixed issues 😃

What's there now seems like the original version

Ah, that would be because GitHub had a big red "Refresh" button at the top.

Okay, only the .as_ref() as *const [u8] as *const i8 -> .as_ref().as_ptr() as _ remains.


Modifying value before displaying in grid

I have a value I would like to modify before displaying in the grid... is there a way to make the field accept a function without having to modify the original data to contain a function which can be passed as a string? Does this make sense? I just would rather not modify the data source.
Thanks

A filter would be the normal way to do that - look at the cellFilter doco.

I found the cellFilter documentation http://ui-grid.info/docs/#/api/ui.grid.class:GridOptions.columnDef but the documentation doesn't really help me understand how to use it and what it does... would you know of an example of it's use? I did not find an example in the tutorials.

http://ui-grid.info/docs/#/tutorial/201_editable has a cellFilter in it.
It's the same as any other angular filter - it formats values.  In the example above it converts the codes (1, 2) to (male, female).

Now that clicks!!! thank you!!!!


feat: use npm-registry-client

Use npm-registry-client for communicating with the npm registry. This will guarantee that pnpm communicates to the registry the same way npm does.
Close #367

Landed in v0.36.0


Add enabled flag

Add a flag to enable or disable the integration at configuration time, this allows the service to be disabled for individual environments.

This is not needed, the start on boot flag is enough...closing.