EdgeX mqtt message bus failed to connect to mqtt-broker with TLS enable

Environment:

eKuiper version (e.g. 1.3.0):  lfedge/ekuiper:1.5-alpine,  lfedge/ekuiper:1.4-alpine
Hardware configuration (e.g. lscpu): x86_64
OS (e.g. cat /etc/os-release): ubuntu 20.04
Others:

What happened and what you expected to happen:
time="2022-05-18 14:42:32" level=error msg="The connection to edgex messagebus failed." file="edgex/edgex.go:126"
time="2022-05-18 14:42:32" level=error msg="can not create client for cfg : map[ClientId:edgexsource certificationpath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt format:json insecureskipverify:true messagetype:request optional_keepalive:50 port:8883 privatekeypath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key protocol:tcps rootcapath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/ca.crt server:mqtt-broker topic:edgex/events/# type:mqtt] have error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="clients/client_registy.go:103"
time="2022-05-18 14:42:32" level=error msg="found error when get edgex client, error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="source/edgex_source.go:80" rule=mqtt-tls-rule

How to reproduce it (as minimally and precisely as possible):

Used the following environment variables to deploy ekuiper service.

environment:
      EDGEX__DEFAULT__PORT: 8883
      EDGEX__DEFAULT__PROTOCOL: tcps
      EDGEX__DEFAULT__SERVER: mqtt-broker
      EDGEX__DEFAULT__CLIENTID: edgexsource
      EDGEX__DEFAULT__CERTIFICATIONPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt
      EDGEX__DEFAULT__PRIVATEKEYPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key
      EDGEX__DEFAULT__ROOTCAPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/ca.crt
      EDGEX__DEFAULT__INSECURESKIPVERIFY: 'true'
      EDGEX__DEFAULT__OPTIONAL_KEEPALIVE: 50

Create stream

{
    "sql": "create stream edgexsource() WITH (FORMAT=\"JSON\", TYPE=\"edgex\")"
}

Create rule

{
   "id": "mqtt-tls-rule",
   "sql": "SELECT * FROM edgexsource",
   "actions":[
      {
         "mqtt":{
            "certificationPath": "/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt",
            "clientId": "mqtt-tls-rule",
            "insecureSkipVerify": true,
            "privateKeyPath": "/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key",
            "rootCaPath": "/tmp/edgex/secrets/mqtt/client/edgex-kuiper/ca.crt",
            "server": "tcps://mqtt-broker:8883",
            "topic": "export"
         }
      }
   ]
}

Anything else we need to know?:
kuiper-connection-log.txt
mqtt-broker logs:
1652856152: New connection from <IP_ADDRESS>:48118 on port 8883.
1652856152: New client connected from <IP_ADDRESS>:48118 as mqtt-tls-rule (p1, c0, k30, u'edgex-kuiper').
1652856152: New connection from <IP_ADDRESS>:48122 on port 8883.
1652856152: OpenSSL Error[0]: error:1404A412:SSL routines:ST_ACCEPT:sslv3 alert bad certificate
1652856152: Client <unknown> disconnected: Protocol error.
1652856152: Client mqtt-tls-rule disconnected.

hi @cherrycl , I think the problem is you set the "insecureSkipVerify": true , please set it to true for this configuration.
{
   "id": "mqtt-tls-rule",
   "sql": "SELECT * FROM edgexsource",
   "actions":[
      {
         "mqtt":{
            "certificationPath": "/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt",
            "clientId": "mqtt-tls-rule",
            "insecureSkipVerify": false,
            "privateKeyPath": "/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key",
            "rootCaPath": "/tmp/edgex/secrets/mqtt/client/edgex-kuiper/ca.crt",
            "server": "tcps://mqtt-broker:8883",
            "topic": "export"
         }
      }
   ]
}

Hi @SuperRxan I set insecureSkipVerify: false in sink and stream connection, both got an error. But the error messages are different, please reference the following log.
insecureSkipVerify: false
time="2022-06-01 09:19:13" level=error msg="The connection to edgex messagebus failed." file="edgex/edgex.go:126"
time="2022-06-01 09:19:13" level=error msg="can not create client for connection selector : edgex.mqttmsgbus have error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : dial tcp <IP_ADDRESS>:1883: connect: connection refused" file="clients/client_registy.go:93"
time="2022-06-01 09:19:13" level=error msg="found error when get edgex client, error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : dial tcp <IP_ADDRESS>:1883: connect: connection refused" file="source/edgex_source.go:80" rule=mqtt-rule

insecureSkipVerify: true
time="2022-05-18 14:42:32" level=error msg="The connection to edgex messagebus failed." file="edgex/edgex.go:126"
time="2022-05-18 14:42:32" level=error msg="can not create client for cfg : map[ClientId:edgexsource certificationpath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt format:json insecureskipverify:true messagetype:request optional_keepalive:50 port:8883 privatekeypath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key protocol:tcps rootcapath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/ca.crt server:mqtt-broker topic:edgex/events/# type:mqtt] have error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="clients/client_registy.go:103"
time="2022-05-18 14:42:32" level=error msg="found error when get edgex client, error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="source/edgex_source.go:80" rule=mqtt-tls-rule

environment:
      EDGEX__DEFAULT__PORT: 8883
      EDGEX__DEFAULT__PROTOCOL: tcps
      EDGEX__DEFAULT__SERVER: mqtt-broker
      EDGEX__DEFAULT__CLIENTID: edgexsource
      EDGEX__DEFAULT__OPTIONAL__CERTFILE: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt
      EDGEX__DEFAULT__OPTIONAL__KEYFILE: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key
      EDGEX__DEFAULT__OPTIONAL__SKIPCERTVERIFY: 'true'
      EDGEX__DEFAULT__OPTIONAL__KEEPALIVE: 50

the way you set certffile and keyfile is incorrect

@SuperRxan Thanks for your suggestion.
I will test the stream connection first. I can't find any document to describe how to set configuration with TLS mqtt message bus on stream connection. Do you know where can I find it? As you said I cannot set cert and key on that way, then how to set it?
Thank you.

@SuperRxan When using the configuration you modified, still got error. The configurations are overrided, but edgex message bus still use port 1883 and protocol tcp.
time="2022-06-08 19:18:39" level=info msg="open source node with props map[ClientId:edgexsource connectionSelector:edgex.mqttMsgBus format:json messagetype:request optional:map[KeepAlive:50 certificationpath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt insecureskipverify:false privatekeypath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key] optional_keepalive:50 port:8883 protocol:tcps server:mqtt-broker topic:edgex/events/# type:mqtt], concurrency: 1, bufferLength: 102400" file="node/source_node.go:93" rule=log-rule
time="2022-06-08 19:18:39" level=info msg="Start source edgexsource instance 0 successfully" file="node/source_node.go:126" rule=log-rule
time="2022-06-08 19:18:39" level=info msg="Set config 'connection.edgex.mqttmsgbus.server' to 'mqtt-broker' by environment variable" file="conf/load.go:127"
time="2022-06-08 19:18:39" level=info msg="Use configuration for edgex messagebus {{mqtt-broker 1883 tcp} {mqtt-broker 1883 tcp} mqtt map[KeepAlive:50]}" file="edgex/edgex.go:54"
time="2022-06-08 19:18:39" level=error msg="The connection to edgex messagebus failed." file="edgex/edgex.go:126"
time="2022-06-08 19:18:39" level=error msg="can not create client for connection selector : edgex.mqttmsgbus have error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : dial tcp <IP_ADDRESS>:1883: connect: connection refused" file="clients/client_registy.go:93"

@SuperRxan it is not an edgex issue, but for ekuiper specifically
We would like to confirm whether the TLS connection with MQTT broker is supported by eKuiper
thanks.

@cherrycl  please paste all your configurations here
and check if your environment have this configuration, if have, please remove it
 environment:
      EDGEX__DEFAULT__CONNECTIONSELECTOR: edgex.mqttmsgbus

@SuperRxan Thank you. I removed EDGEX__DEFAULT__CONNECTIONSELECTOR from the environment variable, and did the test again by the following configurations
environment:
      EDGEX__DEFAULT__PORT: 8883
      EDGEX__DEFAULT__PROTOCOL: tcps
      EDGEX__DEFAULT__SERVER: mqtt-broker
      EDGEX__DEFAULT__TOPIC: edgex/events/#
      EDGEX__DEFAULT__TYPE: mqtt
      EDGEX__DEFAULT__MESSAGETYPE: request
      EDGEX__DEFAULT__CLIENTID: edgexsource
      EDGEX__DEFAULT__OPTIONAL__CERTIFICATIONPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt
      EDGEX__DEFAULT__OPTIONAL__PRIVATEKEYPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key
      EDGEX__DEFAULT__OPTIONAL__INSECURESKIPVERIFY: 'false'
      EDGEX__DEFAULT__OPTIONAL_KEEPALIVE: 50
      CONNECTION__EDGEX__MQTTMSGBUS__PROTOCOL: tcps
      CONNECTION__EDGEX__MQTTMSGBUS__SERVER: mqtt-broker
      CONNECTION__EDGEX__MQTTMSGBUS__PORT: 8883
      CONNECTION__EDGEX__MQTTMSGBUS__OPTIONAL__CERTIFICATIONPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt
      CONNECTION__EDGEX__MQTTMSGBUS__OPTIONAL__PRIVATEKEYPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key
      CONNECTION__EDGEX__MQTTMSGBUS__OPTIONAL__ROOTCAPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/ca.crt
      CONNECTION__EDGEX__MQTTMSGBUS__OPTIONAL__INSECURESKIPVERIFY: 'false'
      KUIPER__BASIC__CONSOLELOG: "true"
      KUIPER__BASIC__RESTPORT: 59720
      KUIPER__BASIC__DEBUG: "false"

I got the error x509: certificate signed by unknown authority. Capture the following error logs for your reference.
time="2022-06-09 16:32:59" level=info msg="open source node with props map[ClientId:edgexsource format:json messagetype:request optional:map[KeepAlive:50 certificationpath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt insecureskipverify:false privatekeypath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key] optional_keepalive:50 port:8883 protocol:tcps server:mqtt-broker topic:edgex/events/# type:mqtt], concurrency: 1, bufferLength: 102400" file="node/source_node.go:93" rule=log-rule
time="2022-06-09 16:32:59" level=info msg="Start source edgexsource instance 0 successfully" file="node/source_node.go:126" rule=log-rule
time="2022-06-09 16:32:59" level=info msg="Use configuration for edgex messagebus {{mqtt-broker 8883 tcps} {mqtt-broker 8883 tcps} mqtt map[KeepAlive:50 certificationpath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt insecureskipverify:false privatekeypath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key]}" file="edgex/edgex.go:54"
time="2022-06-09 16:32:59" level=error msg="The connection to edgex messagebus failed." file="edgex/edgex.go:126"
time="2022-06-09 16:32:59" level=error msg="can not create client for cfg : map[ClientId:edgexsource format:json messagetype:request optional_keepalive:50 port:8883 protocol:tcps server:mqtt-broker topic:edgex/events/# type:mqtt] have error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="clients/client_registy.go:103"
time="2022-06-09 16:32:59" level=error msg="found error when get edgex client, error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="source/edgex_source.go:80" rule=log-rule
time="2022-06-09 16:32:59" level=info msg="EdgeX Source instance 0 Done." file="source/edgex_source.go:296" rule=log-rule
time="2022-06-09 16:32:59" level=info msg="source edgexsource done" file="node/source_node.go:114" rule=log-rule
time="2022-06-09 16:32:59" level=error msg="sending error: Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="infra/saferun.go:52" rule=log-rule
time="2022-06-09 16:32:59" level=error msg="closing rule log-rule for error: Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="server/rule_manager.go:119"
time="2022-06-09 16:32:59" level=error msg="sending error: <nil>" file="infra/saferun.go:52" rule=log-rule

Thank you.

@SuperRxan I updated the configuration below, and also got the error x509: certificate signed by unknown authority.
environment:
      EDGEX__DEFAULT__PORT: 8883
      EDGEX__DEFAULT__PROTOCOL: tcps
      EDGEX__DEFAULT__SERVER: mqtt-broker
      DGEX__DEFAULT__TOPIC: edgex/events/#
      EDGEX__DEFAULT__TYPE: mqtt
      EDGEX__DEFAULT__MESSAGETYPE: request
      EDGEX__DEFAULT__CLIENTID: edgexsource
      EDGEX__DEFAULT__OPTIONAL__CERTIFICATIONPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt
      EDGEX__DEFAULT__OPTIONAL__PRIVATEKEYPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key
      EDGEX__DEFAULT__OPTIONAL__INSECURESKIPVERIFY: "true"
      EDGEX__DEFAULT__OPTIONAL_KEEPALIVE: 50
      CONNECTION__EDGEX__MQTTMSGBUS__PROTOCOL: tcps
      CONNECTION__EDGEX__MQTTMSGBUS__SERVER: mqtt-broker
      CONNECTION__EDGEX__MQTTMSGBUS__PORT: 8883
      CONNECTION__EDGEX__MQTTMSGBUS__OPTIONAL__CERTIFICATIONPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt
      CONNECTION__EDGEX__MQTTMSGBUS__OPTIONAL__PRIVATEKEYPATH: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key
      CONNECTION__EDGEX__MQTTMSGBUS__OPTIONAL__INSECURESKIPVERIFY: "true"
      KUIPER__BASIC__CONSOLELOG: "true"
      KUIPER__BASIC__RESTPORT: 59720
      KUIPER__BASIC__DEBUG: "false"

Logs:
time="2022-06-10 11:15:34" level=info msg="open source node with props map[ClientId:edgexsource format:json messagetype:request optional:map[KeepAlive:50 certificationpath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt insecureskipverify:true privatekeypath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key] optional_keepalive:50 port:8883 protocol:tcps server:mqtt-broker topic:edgex/events/# type:mqtt], concurrency: 1, bufferLength: 102400" file="node/source_node.go:93" rule=log-rule
time="2022-06-10 11:15:34" level=info msg="Start source edgexsource instance 0 successfully" file="node/source_node.go:126" rule=log-rule
time="2022-06-10 11:15:34" level=info msg="Use configuration for edgex messagebus {{mqtt-broker 8883 tcps} {mqtt-broker 8883 tcps} mqtt map[KeepAlive:50 certificationpath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt insecureskipverify:true privatekeypath:/tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key]}" file="edgex/edgex.go:54"
time="2022-06-10 11:15:34" level=info msg="open sink node 1 instances" file="node/sink_node.go:152" rule=log-rule
time="2022-06-10 11:15:34" level=info msg="Opening func collector" file="collector/func.go:50" rule=log-rule
time="2022-06-10 11:15:34" level=error msg="The connection to edgex messagebus failed." file="edgex/edgex.go:126"
time="2022-06-10 11:15:34" level=error msg="can not create client for cfg : map[ClientId:edgexsource format:json messagetype:request optional_keepalive:50 port:8883 protocol:tcps server:mqtt-broker topic:edgex/events/# type:mqtt] have error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="clients/client_registy.go:103"
time="2022-06-10 11:15:34" level=error msg="found error when get edgex client, error Failed to connect to edgex message bus: An error occured while performing a 'Connect' operation: network Error : x509: certificate signed by unknown authority" file="source/edgex_source.go:80" rule=log-rule

@cherrycl  please modify your configuration to this
environment:
      EDGEX__DEFAULT__PORT: 8883
      EDGEX__DEFAULT__PROTOCOL: tcps
      EDGEX__DEFAULT__SERVER: mqtt-broker
      DGEX__DEFAULT__TOPIC: edgex/events/#
      EDGEX__DEFAULT__TYPE: mqtt
      EDGEX__DEFAULT__MESSAGETYPE: request
      EDGEX__DEFAULT__CLIENTID: edgexsource
      EDGEX__DEFAULT__OPTIONAL__CERTFILE: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.crt
      EDGEX__DEFAULT__OPTIONAL__KEYFILE: /tmp/edgex/secrets/mqtt/client/edgex-kuiper/mqtt.key
      EDGEX__DEFAULT__OPTIONAL__SKIPCERTVERIFY: 'true'
      EDGEX__DEFAULT__OPTIONAL__KEEPALIVE: 50
      KUIPER__BASIC__CONSOLELOG: "true"
      KUIPER__BASIC__RESTPORT: 59720
      KUIPER__BASIC__DEBUG: "false"

@SuperRxan  It works. Thank you very much.


Adding Adobe XD UI KIT

I'd love to know if I can create a UI kit for material UI in adobe XD and get featured in the website

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

Summary 💡
I started creating an Adobe XD UI kit based on the material ui components because I've always wanted to use something like that in my day to day life as a UX/UI designer and never found one. So, my request is to add a section like for premium themes but UI Kits (created with Adobe XD or other prototyping tools).
Can I sell this UI Kit on the website and provide others for free?
Motivation 🔦
Tons of Developers use Material ui, and so the designers working with these developers should have something similar for rapid prototyping and much better designer/developer relationship. This would give material ui more popularity not only for developers but for UI designers as well.
I hope you get the idea and come back to me with an open minded response.
Thank you for making a Ruckus! 💖

Thanks for the interest. I have contacted you by email.

I'd love to know about the development of UI kit for Adobe XD, if still in debate/planning or if it's gonna happen.

@selhari To be clear, we are talking about deploying the same as https://material-ui.com/store/items/sketch-react/. @belhassen07 Would be the best one to answer it.

@selhari I've already started the development of the UI Kit. We will publish it as soon as it's ready for use.

@belhassen07 I am a UX designer making extensive use of MUI for a project right now. I am making my own Adobe XD components as needed, (the full set is extensive). Is the Adobe XD UI kit still on track for release?

@belhassen07 is it possible to help out with this effort?

@andrewkfiedler Belhassen has stopped the effort in this direction mid-year.
Starting this year, we have been collecting a list of designers that are interested in such a product. If you don't mind, I have added your email to the list (using your public emails used on commits).
At this point, it seems that the user demand is high enough to justify exploring it. The results of this design survey for 2020 will provide further validation on the need.
We have started to POC with it, Adobe XD has proven to be harder to work with than Sketch and Figma.

@selhari @andrewkfiedler We have a first version available: https://material-ui.com/store/items/adobe-xd-react/. You access the live preview too: https://material-ui.com/store/previews/adobe-xd-react/.


feat: fix testcases of DVM

Description

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Added Features

[ ]

Modified Features

[ ]

Test Cases

[ ] Test A
[ ] Test B

Change Log

Documentation

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

go test ./x/dvm/...
ok      github.com/sge-network/sge/x/dvm        0.064s
ok      github.com/sge-network/sge/x/dvm/client/cli     76.485s
ok      github.com/sge-network/sge/x/dvm/keeper 0.141s
?       github.com/sge-network/sge/x/dvm/simulation     [no test files]
ok      github.com/sge-network/sge/x/dvm/types  0.043s


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1824 ms

Resolved: Bofrost.de performance has improved in 485abab after 8 minutes.


s/ToggleNotifications/SetNotifications/g

@cjb asked for this change.
@akalin-keybase can you take a look and update KBFS accordingly?

lgtm!

Cc: @keybase/react-hackers and also @cjb who asked!

Awesome, I was confused by this too! LGTM.


Integration fails to start with HA 2023.2.1

During startup following error occurs:
Logger: homeassistant.components.sensor
Source: custom_components/discovergy/sensor.py:256
Integration: Sensor (documentation, issues)
First occurred: 8:13:43 AM (1 occurrences)
Last logged: 8:13:43 AM
Error while setting up discovergy platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 293, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/discovergy/sensor.py", line 228, in async_setup_entry
DiscovergySensor(
File "/config/custom_components/discovergy/sensor.py", line 256, in init
self.precision = precision
AttributeError: can't set attribute 'precision'
HA:  2023.2.1
Integration version: 0.4.1

Updating to latest beta, solves the problem though.

Thanks for reporting, but this is a duplicate of #16
With HA 2023.2 please use either <= v0.3.0 or >= v0.4.3.


post setup-node throws an error - missing pnpm store dir

Description:
post setup-node throws an error about a missing store directory for pnpm
Action version:
3
Platform:

[x] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[x] Hosted
[ ] Self-hosted

Tools version:

node16, pnpm 7.5.1
Repro steps:
workflow:
https://github.com/sveltejs/vite-plugin-svelte/blob/main/.github/workflows/ci.yml
run with error:
https://github.com/sveltejs/vite-plugin-svelte/runs/7306898304?check_suite_focus=true
Expected behavior:
pnpm store is detected correctly and cached in post-action - no error thrown.
Even if the run completed correctly but for whatever reason the pnpm store could not be cached, post should not throw an error, causing a CI fail.
Actual behavior:
An error is thrown in post setup-node, failing the run

pnpm store is not found initially (ok i guess, it will be populated later)
pnpm install fills store
post action throws an error that store dir does not exist

Hello @dominikg. Thank you for your report. I think it can't save the cache because pnpm store path returns such output:
Using hooks from: /home/runner/work/vite-plugin-svelte/vite-plugin-svelte/.pnpmfile.cjs
readPackage hook is declared. Manifests of dependencies might get overridden
/home/runner/.local/share/pnpm/store/v3

It gets extra logging that is why the action can't get correct path. As temporary solution could you please create .npmrc file with such content:
loglevel=warn # or error

hmm, that's very unfortunate and could break a lot of CIs, In fact i wonder why it hasn't caused more trouble until now.
Can confirm that the additional logging appears when updating from pnpm 7.5.0 to 7.5.1
cc @zkochan

can confirm that loglevel=warn in .npmrc avoids the problem
https://github.com/sveltejs/vite-plugin-svelte/runs/7309633235?check_suite_focus=true#step:16:6

pnpm store path should only print the path to the store, so it is a bug.

Hello everyone. I've merged fix in the main branch. Could you please confirm that it works as expected  for actions/setup-node@main ?

Hello everyone. We released a new version and updated the major tag with fix. Could you please confirm that everything works as expected ?


Problem with the notification

I have just tried running this module with some version of YouTube (from 11.01.53 to 11.06.54) and I get always the same problem: instead of the Background Player notification I get one of Youtube Red saying "With YouTube Red you can play videos in background: change you settings in the app" or something similar. Videos are still played in background but I can't have the control notification back. I'm using the 1.8.5 version of the module on Android 6.0.1

This is interesting one...
AFAIR when i played YouTube video along with the YTBP module for the first time, it asked me if I want to enable background playback in a popup window. I selected "yes".
Maybe you have selected "no"?
This option may not be easy to change after that, as the setting is not visible in the menu at all.
I think that cleaning YouTube data should be helpful.
Then, after running background playback for the first time it might ask you again, so choose to enable background playback then.
Please report back if possible.
Thank you.

I tried with your method but It didn't work.
Then I tried to uninstall both YouTube and module and then reinstall them in few different way (the app before the module,  vice versa, from play store and from apkmirror, etc) but it didn't work too.
I also tried to install 1.7.6 version of the module with YouTube 10.49.59 but it's not working.
I hope to find a fix and in that way I'll try to be useful for you

Ok.
Before I try to look into the code I have thought of two options:

try to disable all the notifications in YouTube settings
maybe it's some kind of ad? Did you try to combine it with YouTube AdAway module by Wanam?

I tried both methods but they did't work out.
Now I've installed the latest version of Youtube from APKMirror (11.07.59 Arm 320dpi) and I'm keep facing the same issue.

And after clicking "ok" nothing changes, hm?
I may not see it because I live in a country with no YouTube Red support.
I'll check deeper and be back.

Yes. If I click "ok" or I just ignore the message, it never reappears again.
P.S. I noticed only today that in the app's settings there is no more the "YouTube Red" section: the one in which I should have been able to set up the background playback offered by YouTube Red


Why require PHP 7.4?

This commit broke compatibility between phposinfo and platforms running PHP 7.3: https://github.com/loophp/phposinfo/commit/0526c49dbf963233b76020884dabec4a53dec8f7
PHP 7.3 is still officially supported until December 2021, and still widely used. Is there a particular reason you removed support? Can we get it back? Our downstream project (https://github.com/acquia/blt) still needs to support PHP 7.3 users.

Hi Dane,
I will restore it, sorry about that.

Fixed! https://github.com/loophp/phposinfo/releases/tag/1.7.1

Thanks for the quick response!


Make Labels readable

We need to make the labels readable. One solution, is to rotate the labels 90 degrees and/or have smaller text.
Another problem with labels currently, is that some labels are being rendered outside the lyphs.
For example:
Turn on the labels for the Lyphs, the labels for those lyphs inside Regions are being rendered in a position that doesn't match that of the lyph.
https://user-images.githubusercontent.com/4562825/148138990-2885a442-eecb-4cff-a08e-5d944df55d15.mp4
Labels should be more readable, as in screenshot below.

Just for my own reference, this is called "multiple label placement problem" in computer science.


Discord Invites are closed

Is there an ETA for reopening?
Otherwise I'd take a private invite too. DiscordID is falcatos

Reopened recently.


ruby_linux_x64_centos6 distribtest is failing on master

https://source.cloud.google.com/results/invocations/1f7d7a7f-ede9-4e8d-9766-786aac2dafeb/targets/github%2Fgrpc/tests
Bundle complete! 3 Gemfile dependencies, 7 gems now installed.
Use `bundle info [gemname]` to see where a bundled gem is installed.
+ bundle exec ./distribtest.rb
bundler: failed to load command: ./distribtest.rb (./distribtest.rb)
LoadError: /usr/lib64/libstdc++.so.6: version `GLIBCXX_3.4.14' not found (required by /usr/local/rvm/gems/ruby-2.3.8/gems/grpc-1.25.0.dev-x86_64-linux/src/ruby/lib/grpc/2.3/grpc_c.so) - /usr/local/rvm/gems/ruby-2.3.8/gems/grpc-1.25.0.dev-x86_64-linux/src/ruby/lib/grpc/2.3/grpc_c.so
  /usr/local/rvm/gems/ruby-2.3.8/gems/grpc-1.25.0.dev-x86_64-linux/src/ruby/lib/grpc/grpc.rb:20:in `require'
  /usr/local/rvm/gems/ruby-2.3.8/gems/grpc-1.25.0.dev-x86_64-linux/src/ruby/lib/grpc/grpc.rb:20:in `<top (required)>'
  /usr/local/rvm/gems/ruby-2.3.8/gems/grpc-1.25.0.dev-x86_64-linux/src/ruby/lib/grpc.rb:19:in `require_relative'
  /usr/local/rvm/gems/ruby-2.3.8/gems/grpc-1.25.0.dev-x86_64-linux/src/ruby/lib/grpc.rb:19:in `<top (required)>'
  /var/local/git/grpc/test/distrib/ruby/distribtest.rb:17:in `require'
  /var/local/git/grpc/test/distrib/ruby/distribtest.rb:17:in `<top (required)>'

I'm not 100%, but this was likely introduced by https://github.com/grpc/grpc/pull/20441
or some other PR in this diff https://github.com/grpc/grpc/compare/6c2eb034f7159d3c74509a35e47bd0e1605de29e...c8c755ac9de28f810af44b1939265a3414a3404c

thanks for report. PR (https://github.com/grpc/grpc/pull/20496) to fix the problem is in progress.

Fixed


Support Kubeflow Pipelines

What would you like to be added:
I would like to support the ML pipeline toolkit, Kubeflow Pipelines.
Why is this needed:
The Kubeflow Pipelines is the production-grade MLOps pipeline that is widely known and used in many productions.
Completion requirements:
This enhancement requires the following artifacts:

[ ] Design doc
[ ] API change
[ ] Docs update

The artifacts should be linked in subsequent comments.
The Kueflow Pipelines internally uses the Argo workflow's Emissary Executor. So, although we may be able to support the kubeflow pipelines once we resolve #74, we need to investigate whether or not we need to implement some features/controllers for this integration.

This would be cool.

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale


Add seek and tell to Python CuFile object

To make it easier to drop CuFile objects in where other Python file objects are accepted, it would be nice to have seek and tell methods. This would allow a user to go to a file offset and also check what file offset they are at.

I think it could be a good idea, but it will require some smart chunking under the hood. This is because all of our reads and writes (incl. all of the cuFile API) takes ranges.

Could we simply store an off_t offset inside the CuFile object that we then simply do some integer math with to determine where to read and write from?
Or is there some existing behavior under-the-hood that would complicate this approach?

Sorry, yes that would work. I was more thinking of using it together with https://github.com/rapidsai/kvikio/issues/577 where stream read and write would not use GDS or parallel IO if we don't do something clever :)


Unknow database type ds3622xs+_host_v7.db

I have the following error "Unknown database type ds3622xs+_host_v7.db"
After the command syno_hdd_db.sh -fr
Do you have any ideas?

Can you save a copy of your ds3622xs+_host_v7.db file, zip it and then reply with the zip file attached (just drag and drop the zip file onto the reply before clicking on the Comment button).
To copy the ds3622xs+_host_v7.db file to your home directory:
sudo -i cp /var/lib/disk-compatibility/ds3622xs+_host_v7.db $HOME

@7ussana not responding.

Same error here ; ERROR Unknown database type ds213j_host_v7.db.new!
ds213j_host_v7.zip
ds213j_host_v7.db.zip

@mrmuiz
The script did correctly add your WD10EZRX-00A8LB0 and HD204UI to ds213j_host_v7.db
Your ds213j_host_v7.db.new file is 0 bytes (empty). So I'm not surprized the script was unable to determine the version.
Try deleting the ds213j_host_v7.db.new file. I'll update the script to deal with empty db and db.new files so other people don't get the same error.

@mrmuiz
I've released a new version that ignores empty .db.new files so it won't show an error.
https://github.com/007revad/Synology_HDD_db/releases/tag/v3.5.97


Add scss to css compiler

Description

This adds an gulp tasks, to build existing css files out of scss files and enables advantages for working with style rules.
Also this will reduce the complexibility of heavy css files in all future css changes.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[x] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?

(This changes modify and unify the structure of css code only.)
All we have to check yet, is if .css file-changes include only: indentation, spaces and quotes.
It's very useful to reduce the diff to non whitespace changes here:

@webermaximilian92 thanks for the PR!
Can you provide some more details about the changes this PR is providing? I mean, how this will play out in ACS. When will the CSS compilation take place? At runtime when the page is being loaded? What are the benefits that we get with these changes?

Hey Rafael,

sass and scss are preprocessors of css. Similar to ES6 or TypeScript for JavaScript.
SCSS provides us many logical functions and advantages to organise CSS rules more easily
Using css is not the common standard for such complex pages like cloudstack anymore.
We can make the css code more modular and maintainable with sass/scss, by splitting files.
We could also split css files, but this is not best prectice, since many css files will result in many server requests, and with sass it's possible to compile many files into one css file in just under a second after scss change.
Later we could minify the final css file automatically, this will enhance loading times of pages.

Thanks @webermaximilian92.
Can you check the licensing issues?
https://builds.apache.org/job/cloudstack-pr-analysis/8442/artifact/target/rat.txt

Very sorry I had to create new PR after running in problems with my forked cloudstack after switching my username.
https://github.com/apache/cloudstack/pull/3307


Saltern janitor closet needs a space cleaner dispenser

Description

Reproduction

Screenshots

Additional context

it HAS one

Yeah it already has one

man I'm blind as hell


[HOTFIX] Some basic fix for 1.3.0 release

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[X] Any interfaces changed?
NA

[X] Any backward compatibility impacted?
OK

[X] Document update required?
OK

[X] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3496/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2257/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2266/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3504/

LGTM


EXP-3467 change fling method to getChannel in the model, override get…

…Channel name for Data model

:+1:


Fix: Drop mime-db dependency for http-compression rule

Pull request checklist
Make sure you:

[x] Signed the Contributor License Agreement
[x] Followed the commit message guidelines

For non-trivial changes, please make sure you also:

[ ] Added/Updated related documentation.
[ ] Added/Updated related tests.

Short description of the change(s)
mime-db was used as a source to check if something should be compressed, but the project is planning to drop the compressible data in the near future in favor of the compressible project.
All of the cases currently covered by compressible are already covered by our code, so this change just adds the cases that were covered by mime-db and not by our code, hence, maintaining backwards compatibility.

Fix: #1108

@alrra shouldn't this be a chore because nothing changes to the end user?

shouldn't this be a chore because nothing changes to the end user?

It should, but I wanted to trigger a new release.

Also, this is in some cases a performance improvement, since we no longer need to query mime-db.


Update aws_config

Caught this in another PR

https://docs.rs/aws-config/latest/aws_config/struct.BehaviorVersion.html#method.latest maybe we should just use latest?
From the description in the docs, all versions are compatible, it just changes some performance characteristics. And we certainly arent "tuning to rust SDK behaviors" as they say.


Remove translation processing using cheerio.

Current translation process is below:

Look up localized template file from challenge-content- directory.
Transform the content read from template file into some other form using cheerio(jQuery).

I'm now translating contents into Japanese and content building does not work, because the transformed Japanese contents are escaped badly in the second process.
We manages contents separately for each language and these are already translated properly. Thus we have no need for the second transformation process, I think.

translateLocale function I removed had a role for HTML escaping(like ' to '), so my modification may be a bit problematic. It changed the result of English contents.

I made a mistake. The builded contents merely are numeric character references. So It's properly displayed when running the program.


Secret Scanning for Forge Apps

Don't merge this in yet, I still need to work on finishing all of the cases for the definition analysis


ShapeNet server down?

Hello and thank you for your work!
I am trying to log in to the ShapeNet website to download the dataset (I already have my account approved), but I am getting the following error message:
Exception: MongoError: topology was destroyed
I have seen somebody mentioning this message in https://github.com/daerduoCarey/partnet_dataset/issues/18. Could you, please, try to fix it this again?
Best regards!

I tried again today and now it seems to be working, thanks anyway!


Add dynamic circuits as a backends filter

Summary
Make it slightly easier for people to find backends that support QASM3.
cc @taalexander in case you want to update docs.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-2 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-2.7%) to 35.509%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_provider/ibm_backend_service.py
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_provider/ibm_backend_service.py
1
23.11%

Totals

Change from base Build<PHONE_NUMBER>:
-2.7%

Covered Lines:
2388

Relevant Lines:
6725

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-2.7%) to 35.509%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_provider/ibm_backend_service.py
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_provider/ibm_backend_service.py
1
23.11%

Totals

Change from base Build<PHONE_NUMBER>:
-2.7%

Covered Lines:
2388

Relevant Lines:
6725

💛  - Coveralls


Fill svg after drawing is complete

Hi there,
How can the fill be applied after the drawing is complete. Or is this not possible?
Would be awesome to have this.
Cheers,
Stephan

Ok, I found a "hacky" way to get the result. Seems to be working as I expected.

Hi @udarts , can you share what you've done to implement the svg fill after animate completion? Thanks.

Should this be SVG path (d) dependent?


build images for all supported pg versions

Publish docker images for all the versions that we run tests for / are supported.

I thinks with docker like this:
ARG PG_VERSION
ARG PGRX_VERSION

FROM quay.io/coredb/pgrx-builder:pg${PG_VERSION}-pgrx${PGRX_VERSION}

WORKDIR /app

COPY --chown=postgres:postgres . .

ARG EXTENSION_NAME
ARG EXTENSION_VERSION

RUN cd extension && cargo pgrx package

# Add commands to build and push Docker image
FROM postgres:${PG_VERSION}

COPY --from=0 /app/target/release/${EXTENSION_NAME}-${EXTENSION_VERSION}.so /usr/lib/postgresql/${PG_VERSION}/lib/

ENTRYPOINT ["postgres"]

and action like this:
name: Build and Push Docker Images

on:
  push:
    tags:
      - 'v*.*.*'

jobs:
  build:
    runs-on: ubuntu-latest
    strategy:
      matrix:
        pg_version: [13, 14, 15]  # Add all supported PG versions
        pgrx_version: [0.9.8]     # Add relevant PGRX versions

    steps:
    - name: Checkout repository
      uses: actions/checkout@v2

    - name: Set up Docker Buildx
      uses: docker/setup-buildx-action@v1

    - name: Log in to Docker Hub
      uses: docker/login-action@v1
      with:
        username: ${{ secrets.DOCKER_USERNAME }}
        password: ${{ secrets.DOCKER_PASSWORD }}

    - name: Build and push
      uses: docker/build-push-action@v2
      with:
        context: .
        push: true
        tags: |
          yourdockerhubusername/pgmq:${{ matrix.pg_version }}-${{ matrix.pgrx_version }}
          ${{ steps.set-latest.outputs.latest_tag }}
      build-args: |
        PG_VERSION=${{ matrix.pg_version }}
        PGRX_VERSION=${{ matrix.pgrx_version }}

    - name: Set latest tag
      id: set-latest
      run: |
        PG_VERSIONS=(${{ matrix.pg_version }})
        HIGHEST_PG_VERSION=$(echo ${PG_VERSIONS[*]} | tr ' ' '\n' | sort -nr | head -n1)
        if [ "${{ matrix.pg_version }}" == "$HIGHEST_PG_VERSION" ]; then
          echo "::set-output name=latest_tag::yourdockerhubusername/pgmq:latest"
        else
          echo "::set-output name=latest_tag::"
        fi

We can achieve this, but I'm not sure where to make those changes. Should we create a new action or edit the one we already have?

The current workflow  is similar to what you've suggested. However, the extension is only built for a single pgrx version (and soon wont be built by pgrx).
We can add the matrix strategy similar to how you suggested and can add a new repo for each major version of Postgres, e.g. pgmq-pg15, pgmq-pg16, pgmq-pg17` that we want to support images for (I think pg14+ is sufficient). Continue to use the short sha when publishing off every commit and latest + semver tag when publishing off a release.
In the end we'd end up with images published something like this:
quay.io/tembo/pg14-pgmq:626f74e
quay.io/tembo/pg14-pgmq:v1.2.1
quay.io/tembo/pg14-pgmq:latest
...
quay.io/tembo/pg16-pgmq:626f74e
quay.io/tembo/pg16-pgmq:v1.2.1
quay.io/tembo/pg16-pgmq:latest

Thoughts?

interesting!

resolved in https://github.com/tembo-io/pgmq/pull/237


[Task]: Page Switcher

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Visual Requirements

Animate slide down the panel to reveal the list of pages
Show the list of pages

https://github.com/appsmithorg/appsmith/assets/12022471/3283a2ea-f53f-409e-ab52-810afaf9374f
Functional Requirements

Have a state controlled by the Top Bar trigger.
Adding new pages and navigating to it
Re order the pages via drag
Switch pages on click
On switch, restore the context of the  destination page

@albinAppsmith what's the estimate?


Owner Name sorting is not ignoring lower/uppercase

Descriptions
In the settings of a safe > owners you can sort names asc/desc
The sorting is putting lower case characters as higher than the uppercase characters.
My guess is that is sorting by ASCII value instead of alphabetically
URL: https://rinkeby.gnosis-safe.io/safes/0xd7CFF0711F41c3c9702a4f5df82334679Be7CfF0/settings
Snapshot:

Steps:
1 - Enter into a safe settings
2 - Edit some owners names using lower/uppercase characters
3 - Sort the column

Still an issue. "sa" should go before "Second".

This should be done only once @fernandomg merges his PR.


extract into IO

Is it possible to extract a specified file into a target IO rather than a directory? I want to access particular files from a tgz archive and work with them in-memory without extracting them to disk.

Would be fixed by https://github.com/JuliaIO/Tar.jl/pull/95, which I've been meaning to return to.


I have probleme when i install fsevents !! do you have some idea to resolve this issues thank's

$ npm install fsevents
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "fsevents"
npm ERR! node v6.10.0
npm ERR! npm  v3.10.10
npm ERR! code EBADPLATFORM
npm ERR! notsup Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm ERR! notsup Valid OS:    darwin
npm ERR! notsup Valid Arch:  any
npm ERR! notsup Actual OS:   win32
npm ERR! notsup Actual Arch: x64
npm ERR! Please include the following file with any support request:

It's an OS X-only module, you can't install it on Windows.


[Select]: New Select component (Beta)

name: 'Pull request'
about: Select component
New Pull Request
Checklist

[x] I am not disclosing a vulnerability.
[x] My code is conform the code style
[x] I have made corresponding changes to the documentation
[x] I have updated Typescript definitions when needed

Issue Description

Related issue: #749 , #728
Solution Description

Folder structure changed
New select component added in beta version since more changes need to be done in further iterations.
change tryBeta prop to true to try this component
Search feature added
Multi-selection added
Accessibility (Keyboard) added

wow this looks great @AbhinavMV i think i only have two TINY bit of feedback

on change it would be nice to see the action in storybook, i dont even see the console log actually
each name could use like 4px padding left on the span
2 countries flags fail, prob not your bad

wow this looks great @AbhinavMV i think i only have 3 TINY bits of feedback:

on change it would be nice to see the action in storybook, i dont even see the console log actually
each name could use like 4px padding left on the span
2 countries flags fail, prob not your bad

Nice catch.
Got it will fix all these soon. 🙌

on change it would be nice to see the action in storybook, i dont even see the console log actually

Added action for this 🙌🏻

each name could use like 4px padding left on the span

Fixed 🙌🏻

2 countries flags fail, prob not your bad

Yeah no image present for that, can't do anything 😅
@BillyG83 I guess all the mentioned issues are fixed.
Also new things added:

default index can be set now
Custom error and description messages

Validation is pending - probably add it in another PR? this is getting huge would make it hard to review 😅

great job @AbhinavMV lets merge this and test in Admin to see if QA will pass it, and get feedback from them. Also if you could look into a fallback image for any flags that are missing or fail for any reason, the broken image thing is a little ugly and @MateusbrMedeiros will not pass that I am sure 😁


Add separate PropTypes package

relates #16

Thanks for adding this! Sorry for the merge delay, things have been busy.


Update git dependency

It seems that this project requires the 0.1 version of pascalmh/git.php. Unfortunately this does not exist. For now there is only a v0.1.1-rc.
This happens when I try to install this plugin:
$ composer require blankogmbh/kirby-git-commit-and-push-content
Using version ^1.4 for blankogmbh/kirby-git-commit-and-push-content
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for blankogmbh/kirby-git-commit-and-push-content ^1.4 -> satisfiable by blankogmbh/kirby-git-commit-and-push-content[1.4.0].
    - blankogmbh/kirby-git-commit-and-push-content 1.4.0 requires pascalmh/git.php ^0.1 -> satisfiable by pascalmh/git.php[v0.1.1-rc] but these conflict with your requirements or minimum-stability.

Installation failed, reverting ./composer.json to its original content.

@Pascalmh is this related to #21?

Works great now. Thank you.


How to convert DReyeVR_logging.py output to CSV ?

Hi!
Is there any way to use the DReyeVR_logging.py with the .rec file ? Currently, I have to run it with my simulation to log manually.
Thanks.

ok, I figured out a way. Basically, if we run DReyeVR_logging.py with start_replaying.py, we can get the log file.
But my question now is, what's the output format of that log file?

Generally, we don't use the DReyeVR_logging.py file to get recorded information, rather we recommend using the DReyeVR parser as described here. What information are you trying to get?

I want to perform some data analysis. For example, I want to see how the average pupil diameter varies for n participants. So I can have n columns containing pupil data for each participant and a last column that contains the average. So yeah, stuff like that.
Also yeah, the parser works great! Its perfect to analyse data individually. The graphs it gives have the "carla_time" on the x-axis. I am interested in data vs "real world time". Can you lemme know which variable is associated with "real world time" ?

Ok, figured it out. So 'timestamp_device' is the one associated with real-time. Need to divide by 1000 to get the value in seconds.

Update:
An easier way is to run the text file that you get from ./show_recorder_file_info.py -a -f /PATH/TO/RECORDER-FILE > recorder.txt on this code to get your desired variables. You can modify as per your needs.
import re
import csv

# Open the input text file
with open('4a.txt', 'r') as file:
    content = file.read()

# Use regular expressions to find all relevant values
timestamp_carla_values = re.findall(r'TimestampCarla:(\d+)', content)
pupil_diameter_values = re.findall(r'PupilDiameter:(-?[\d.]+)', content)
eye_openness_values = re.findall(r'EyeOpenness:(-?[\d.]+)', content)
hit_values = re.findall(r'FocusInfo:Hit:(\d+)', content)
actor_name_values = re.findall(r'ActorName:([\w]+)', content)

# Open a CSV file for writing
with open('z.csv', 'w', newline='') as csvfile:
    writer = csv.writer(csvfile)

    # Write the header row to the CSV file
    writer.writerow(['TimestampCarla', 'PupilDiameter A', 'PupilDiameter B', 'EyeOpenness A', 'EyeOpenness B', 'Hit', 'ActorName'])

    # Determine the minimum length of the lists to avoid index errors
    min_length = min(len(timestamp_carla_values), len(pupil_diameter_values), len(eye_openness_values), len(hit_values), len(actor_name_values))

    # Write the values to the CSV file, alternating EyeOpenness values
    for i in range(min_length):
        row = [
            timestamp_carla_values[i],
            pupil_diameter_values[i * 2] if i * 2 < len(pupil_diameter_values) else '',
            pupil_diameter_values[i * 2 + 1] if i * 2 + 1 < len(pupil_diameter_values) else '',
            eye_openness_values[i * 2] if i * 2 < len(eye_openness_values) else '',
            eye_openness_values[i * 2 + 1] if i * 2 + 1 < len(eye_openness_values) else '',
            hit_values[i] if i < len(hit_values) else '',
            actor_name_values[i] if i < len(actor_name_values) else ''
        ]
        writer.writerow(row)

print("CSV file 'z.csv' has been created with the specified values.")


Action overwrites GITHUB_PATH, making other installed tools unaccessible

Trying to use this action in my pipeline, and currently, the action overwrites GITHUB_PATH, thus making other installed tools unusable, such as protoc in my case.
 thread 'main' panicked at 'Could not find `protoc` installation and this build crate cannot proceed without
      this knowledge. If `protoc` is installed and this crate had trouble finding
      it, you can set the `PROTOC` environment variable with the specific path to your
      installed `protoc` binary.If you're on debian, try `apt-get install protobuf-compiler` or download it from https://github.com/protocolbuffers/protobuf/releases

https://github.com/houseabsolute/actions-rust-cross/blob/9a602122c2f4df61140216897231df93c0922a22/action.yml#LL29C12-L29C60

Ah, I just saw this issue. This should be a simple fix.

Actually, I'm a bit confused as to how this is happening. From reading the GH Actions docs it seems like appending to $GITHUB_PATH should just add a directory, not wipe out any that already exist. Can you point me to workflow you're using that's having this issue?

Hey @autarch ! Thanks for replying. Of course, see below:

Latest run: https://github.com/omnilium/pandorica/actions/runs/4397996712/jobs/7701515589
Workflow file: https://github.com/omnilium/pandorica/blob/main/.github/workflows/build.yml

I wrote some tests to check for this. AFAICT it's not overwriting the $PATH. See https://github.com/houseabsolute/actions-rust-cross/actions/runs/4399492494/jobs/7704067598#step:7:10

Oh wait, I see what's happening. The problem is that builds for non-native architectures are run inside a Docker container. That container doesn't have protoc in it.
There's a number of ways to solve this but I think the simplest is to add pre-build hooks for cross that install protoc -https://github.com/cross-rs/cross#pre-build-hook
You might find my ubi project handy for this.

I'm going to close this since it's not a bug with the action. But on the plus side, it got me to write some tests for it, which is great!

@autarch Thank you so much for looking into this and for providing this very valuable insight. It makes perfect sense why it doesn't work and it totally slipped my mind checking for the fact that it runs inside a docker container.
I will go about adding pre-build hooks as linked above.


[pigeon] Java Result<T> doesn't allow communicating nullability

As far as I can tell there's no way to pass nullability annotations through a generic in Java. To allow for expressing nullability of return values, we should make a breaking change to add a new NullableResult<T>, which is the same as Result<T> except for the annotations we put on the argument to success. (The breaking part being that some methods that currently have a Result parameter will change to a NullableResult parameter.)
For now I'm suppressing the UnknownNullability warning on Result<T>'s success method.
@tarrinneal I think we should do this at the same time we replace Reply with Result for https://github.com/flutter/flutter/issues/118243 since we'll be making a related breaking change for Java at that point (although this one will affect more code).

@stuartmorgan is error suppression worse than added boilerplate and complexity? Is it required for kotlin compat?

It's not just about suppressing the error, it's about not being able to accurately express the nullability semantics.

worse than added boilerplate and complexity

The boilerplate should just be a small amount of Pigeon-generated Java; since generating boilerplate to increase type safety is literally the whole point of Pigeon, that tradeoff seems reasonable to me 🙂  The added complexity for the client seems smallish; not ideal, but making it impossible to express the correct semantics seems worse to me. (This is something I really dislike about the Obj-C generator when I use it, actually; all returns are nullable even when they aren't because of the out-error patterns that is standard for Obj-C. We can't fix it there without doing something very non-idiomatic, but we can here.)

Is it required for kotlin compat?

To have the correct nullability in the types for any Kotlin code interacting with this, yes. (That's probably not a major consideration though, since I imagine people who are consuming this code from Kotlin are using the Kotlin generator.)


Changes in redirection of footer links

Updated the links for the footer social media icons.
Added redirection links for product, blog, pricing, resources(in the footer).

Signed-off-by: anupriya0703<EMAIL_ADDRESS>

have a alternate PR for similar changes https://github.com/mayadata-io/kubera-docs/pull/24


Issue a error if version 2.0.0 defines global OS specific tasks.

This was never supported in 2.0.0 but a user might be hit when simply changing the version number. We should make this clear in the schema and issue an error message.

To verify: tasks.json
{
	// See https://go.microsoft.com/fwlink/?LinkId=733558
	// for the documentation about the tasks.json format
	"version": "2.0.0",
	"windows": {
		"tasks": [
			{
				"taskName": "mmm",
				"command": "echo"
			}
		]
	},
	"tasks": [
		{
			"type": "gulp",
			"task": "build",
			"problemMatcher": [
				"$tsc"
			],
			"group": { "kind": "build", "isPrimary": true },
			"presentation": {
				"echo": true,
				"reveal": "always",
				"focus": false,
				"panel": "dedicated"
			}
		}
	]
}

Should show an error under windows>tasks and an message in the output channel. If tested under different OS change windows to osx or linux


Region list - include sort by alpha

Correct me if I'm wrong, but the list of regions is only sorted by proximity. Would be nice to have the option to sort alphabetically. Just like Android.

If you haven't started this, it seems fine to move it to 4.0.3. You did a lot for 4.0.2 already - feels like a solid update.

It's cool, I'm almost done with this... just one annoying UI issue I'm fighting with.

https://github.com/scottwainstock/pbm-ios/commit/3ec03fefa430b22c92434f24569940eb0d75480f

Sick! Resolved on iphone and ipad.


Delay on triggering onSnapToItem

Thank you for providing this great package. I'm moving from react-native-snap-carousel to react-native-reanimated-carousel.
Describe the bug
I have a Carousel (Multiple version) with two controller buttons to go to the next and previous slide.
when I click on buttons, onSnapToItem changes the current index and based on that I change the opacity of the buttons.
But there is a noticeable delay in the opacity changes and I think it's related to onSnapToItem and the current index update.
To Reproduce
Steps to reproduce the behavior:

build a simple carousel
add buttons that by pressing triggers onSnapToItem
observe the behavior
See the Delay

Expected behavior
onSnapToItem should act immediately after the slide transition
Versions (please complete the following information):

react: v18.2.0
react-native: v0.71.4
react-native-reanimated: v^3.0.2
react-native-reanimated-carousel: v^3.3.0
react-native-gesture-handler: v^2.9.0

Smartphone (please complete the following information):

Device: iPhone12 and Emulator
OS: iOS 16.3.1

Additional context
The issue might be related to how the transition animation ends.

As I guessed the issue is related to the way animation ends.
The solution for having a snappy transition between slides with no delays:
Changing Easing: react-native-reanimated-carousel/src/coontants/index.js
export const Easing = {
  easeOutQuart: _Easing.bezier(
    0.1,
    0.2,
    0.1,
    0.2,
  ) as unknown as Animated.EasingFunction,
};

It could be a good idea to define different props for animations to achieve the best result based on your case. For now, the change above makes the transition more snappy and gives the end user a better experience with no delay.

I tested on a less powerful phone (iPhone 7) and the delay is still there. Changing the elements of the transitioning animation just minimizes the issue.

Now I tested the carousels on the old device in testflight mode and there is no delay. I think the problem was related to the react native 0.71.4 with the new debugging system that the states are being updated with delay.
I think this issue can be closed.


boolean variable escaping in a raw instruction

Hello and thank you for knex !
Knex version: 0.20.13
Database + version: postgres 12.2
OS: MacOs 10.13.6
Bug

Explain what kind of behaviour you are getting and how you think it should do

knex.raw('? as ??', [true, 'editable']): the first variable should be a boolean, but it is a string.

Error message

No error message. The type of the variable is not correct

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to MySQL or PostgreSQL, then single file example which initializes
needed data and demonstrates the problem.

const connection = {
  database: 'knex-bug',
  host: process.env.PGHOST,
  password: process.env.PGPASSWORD,
  port: Number(process.env.PGPORT),
  user: process.env.PGUSER
}

const knex = require('knex')({
  client: 'pg',
  connection
})

knex.schema
    .createTable('users', function(table) {
      table.increments('id')
      table.string('user_name')
    })
    .then(function() {
      return knex('users').insert({ user_name: 'Tim' })
    })
  .then(function() {
    return knex('users').select(knex.raw('? as ??', [true, 'editable']))
  })
  .then(function(rows) {
    rows.map(function(row) {
      console.log(row, typeof row.editable)
      // { editable: 'true' } string -> BUG: type should be boolean
    })
  })
  .catch(function(e) {
    console.error(e)
  })

This happens, because of the way how node-pg driver creates prepared statements
When
knex('users').select(knex.raw('? as ??', [true, 'editable']))

is sent to database, driver first creates prepared statement, with one parameter and after that it calls the prepared statement with arguments.
Prepared statement (created by node-pg driver) doesn't declare typing information for its parameters so the query just doesn't know that the selected literal in boolean SQL type (I cannot remember details how node-pg driver actually sends the data so, I'm sorry for really vague explanation how this happens).
Anyways knex cannot help you automatically with this, since knex actually passes the boolean as a boolean value to the node-pg driver.
There are 2 ways how you should be a able to work around this
Cast data in query to boolean:
knex('users').select(knex.raw('?::boolean as ??', [true, 'editable']))

or hardcode boolean literal to SQL string
knex('users').select(knex.raw('true as ??', ['editable']))

ps. This could be helped if we add some SQL type wrapper classes for allowing to pass information about SQL types of the parameters in query... though that could require also changes to database drivers and a lot more investigation if it really is viable feature to implement.

Thank you for the explanation


Update removing-sensitive-data-from-a-repository.md

Add gitleaks as another way to prevent secrets leaks.

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@diberry Thanks so much for opening a PR! I'll get this triaged for review ✨


changes to device trust screen

Motivation

JIRA: https://issues.jboss.org/browse/AEROGEAR-2965
Progress

[x] Screen Layout same as Xamarin
[x] Percentage trust score show no decimal places
[x] Update Icons
[x] Update wording
[x] Update dialog box

Screen Shots

Warning Dialog
Device Trust Page

Tried on 2 different emulators and they all work fine.

@ciaranRoche Can you rebase?


Fetching Github data often leads to being throttled by Github

Worked around this temporarily with:
const fetchGithubData = async data => {
  await new Promise(resolve => setTimeout(resolve, Math.random() * 30000));

😅

This worked better: https://github.com/react-native-community/native-directory/commit/0c8c38b6414d7c44efbc821538bab54920e53685


AttributeError: module 'imgaug.augmenters' has no attribute 'MultiplyHueAndSaturation'

Hi,guys,I meet this error when using imgaug. Meet this error when running train.py, I can use imgaug normally on other .py which is at same project. Is there a solution for this error?
I would appreciate it if you have suggestions.
OS:ubantu16.04
imgaug:0.3.0

images = iaa.MultiplyHueAndSaturation(mul_hue=(0.8, 1.3), mul_saturation=(0.5, 1.5),per_channel=True)(image=images)
Original Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/torch/utils/data/_utils/worker.py", line 178, in _worker_loop
    data = fetcher.fetch(index)
  File "/usr/local/lib/python3.5/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/usr/local/lib/python3.5/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "utils/datasets.py", line 107, in __getitem__
    img_ia = color_enhance(img_ia)
  File "utils/augmentations.py", line 151, in color_enhance
    images = iaa.MultiplyHueAndSaturation(mul_hue=(0.8, 1.3), mul_saturation=(0.5, 1.5),per_channel=True)(image=images)
AttributeError: module 'imgaug.augmenters' has no attribute 'MultiplyHueAndSaturation''

Do you mean you can import MultiplyHueAndSaturation in other files, but not in your train.py? That would be weird. At least the augmenter should exist in 0.3.0. Do you maybe have a folder or file called imgaug?
You could run print(dir(iaa)) to see what augmenters and functions are actually available. Also run import imgaug; print(imgaug.__version__) to make sure that you actually use 0.3.0 and not accidentally an older version.

Thank you for your help.
You got it right.
I tried to print that information as you mentioned. When i print at other .py,there are 'MultiplyHueAndSaturation' and 'MultiplySaturation' and so on, the version is 0.3.0. But it is not when i print at train.py. the version is 0.2.9 even though I've never installed it before.
It is strange. Could i solve this problem via uninstalling the 0.2.9 version?
Thanks for your time.

Do you run all of these files with the exactly same command from the same directory? My best guess would be that they are executed in different python environments and that one of them has 0.2.9 installed and the other has 0.3.0 installed. Uninstalling 0.2.9 would then lead to an import error in one of the environments.
You can also try import imgaug; print(imgaug.__file__) to see from which path each of these files imports imgaug.

Hi, aleju.
Thank you very much. you are right.
They are from different python environments.
I have setup the same environment.
Thank you


Shouldn't Middleware functions take &mut self?

Is there any reason why middleware functions cannot take &mut self? Middleware could enable storing some state when a request comes in and then use that state during the response phase. I'm facing the same issue right now when implementing #8. I figured that at the minimum when someone firsts starts using Tide, they should see a simple log in the terminal of the format:
<timestamp> <http method> <path> <statuscode>

But in order to do so, I need to store method and path information which is only present on the Request object and access them when the response function is called by the call method in Server.
I imagine that there are many such usecases where middleware would be storing information when the request method is called. An example that comes to mind is a caching middleware.
Therefore, I think that the request and response methods of the Middleware trait should take &mut self. Or is there a better way to go about solving this?

Sorry accidentally closed this! My bad!
Meant to leave this comment:
It might be worth storing a optional mutable context that is passed along with each request as opposed to just the middleware itself? That'll allow devs to separate out the functionality of the middleware vs what needs to be stored on each request? I dunno - just a thought!

Middleware may be shared among multiple threads, so we can't take a mutable reference (it leads to data races). Instead, you can use some sync types for middleware state.

@tirr-c I did not realize that. Correct me if I'm wrong, but I'm guessing the reason Middleware may be shared amongst threads is because the executors themselves are running in a threadpool and the futures could be spawned on any of those threads right?
If that's the case, then I have to put the shared state behind a Mutex, right?

@bIgBV correct.
I think it'd actually be a good idea to open an issue for talking about the threading model in general -- I haven't put a ton of thought into it and there are lot of potential options.

@aturon what do you mean by the threading model? Isn't Tide using hyper internally, meaning it's running on tokio? How can we have a different threading model than the one on top of which we are building? Or am I not understanding this right?

Just opened #50 and my original question has been answered, so closing this. issue.


[Packaging][Release] Missing amazon-linux/2/x86_64/repodata/repomd.xml repo metadata file

Describe the enhancement requested
We re heavily relying on this file being available on JFrog's server.
[dejan@server ~]$ curl https://apache.jfrog.io/artifactory/arrow/amazon-linux/2/x86_64/repodata/repomd.xml
{
  "errors" : [ {
    "status" : 404,
    "message" : "File not found."
  } ]
}

Is there an alternative place we can try?
Component(s)
Packaging

Hi @dejlek I am not familiar enough with this area, do you know if this file was previously there?
I just released Apache Arrow 13.0.0 and from what I can see we dropped support for Amazon Linux 2 on that relase as seen here: https://github.com/apache/arrow/issues/36241
The file is available for Amazon Linux 2023:
https://apache.jfrog.io/ui/native/arrow/amazon-linux/2023/x86_64/repodata/
I wouldn't expect the file to be removed if it was previously there by our release scripts, that's why I am asking.
CC @kou

Yes, files were definitely there for months...

Oh, sorry. I've restored the repodata for Amazon Linux 2. We should have not removed the existing repodata for Amazon Linux 2 even if we dropped support for Amazon Linux 2.

I'll update our release script to prevent this in future releases.

@kou just for my personal interest, how did you regenerated the repodata for Amazon Linux 2? Did you downloaded it from the previous release and uploaded it manually?

Artifactory has the "Trash Can" feature: https://apache.jfrog.io/ui/repos/tree/General/auto-trashcan/arrow
We can restore removed files from here. :-)

Artifactory has the "Trash Can" feature: https://apache.jfrog.io/ui/repos/tree/General/auto-trashcan/arrow
We can restore removed files from here. :-)

Nice, thanks @kou this is good to know.


Adding new constant for tracking polling retries.

Adding new constant to track polling count for task list build.
Issue: LMCROSSITXSADEPLOY-681

Unnecessary


JobStatusLite crash with code can't encode char

Error message is
'ascii' codec can't encode character u'\u2018' in position 1424: ordinal not in range(128)

even though the source code is defining the utf-8 codec. In addition to that, it has the ignore value, which I'd expect to skip anything that can't be decoded...
Full traceback as follows:
2019-04-15 04:45:52,072:139744026318592:ERROR:SimpleCondorPlugin:No job report for job with id 5058715 and gridid 95267.128
2019-04-15 04:45:52,152:139744026318592:ERROR:SimpleCondorPlugin:No job report for job with id 5182715 and gridid 95964.25
2019-04-15 04:45:52,156:139744026318592:ERROR:BossAirAPI:Exception while completing jobs!
'ascii' codec can't encode character u'\u2018' in position 1424: ordinal not in range(128)
2019-04-15 04:45:58,932:139744026318592:ERROR:BaseWorkerThread:Error in worker algorithm (1):
Backtrace:
  <WMCore.BossAir.StatusPoller.StatusPoller object at 0x7f18b1a0a490> <@========== WMException Start ==========@>
Exception Class: BossAirException
Message: Exception while completing jobs!
'ascii' codec can't encode character u'\u2018' in position 1424: ordinal not in range(128)
        ModuleName : WMCore.BossAir.BossAirAPI
        MethodName : _complete
        ClassInstance : None
        FileName : /data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/BossAir/BossAirAPI.py
        ClassName : None
        LineNumber : 534
        ErrorNr : 0

Traceback:
  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/BossAir/BossAirAPI.py", line 526, in _complete
    self.plugins[plugin].complete(jobsToComplete[plugin])

  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/BossAir/Plugins/SimpleCondorPlugin.py", line 324, in complete
    condorReport.addError(exitType, exitCode, exitType, logOutput)

  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/FwkJobReport/Report.py", line 568, in addError
    errDetails.details = errorDetails.decode('utf-8', 'ignore')

  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/external/python/2.7.13-comp/lib/python2.7/encodings/utf_8.py", line 16, in decode
    return codecs.utf_8_decode(input, errors, True)

<@---------- WMException End ----------@>  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/WorkerThreads/BaseWorkerThread.py", line 182, in __call__
    tSpent, results, _ = algorithmWithDBExceptionHandler(parameters)
  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/Database/DBExceptionHandler.py", line 35, in wrapper
    return f(*args, **kwargs)
  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/Utils/Timers.py", line 24, in wrapper
    res = func(*arg, **kw)
  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/BossAir/StatusPoller.py", line 68, in algorithm
    self.checkStatus()
  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/BossAir/StatusPoller.py", line 92, in checkStatus
    runningJobs = self.bossAir.track()
  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/BossAir/BossAirAPI.py", line 494, in track
    self._complete(jobs=jobsToComplete)
  File "/data/srv/wmagent/v1.1.20.patch3/sw/slc7_amd64_gcc630/cms/wmagent/1.1.20.patch3/lib/python2.7/site-packages/WMCore/BossAir/BossAirAPI.py", line 534, in _complete
    raise BossAirException(msg)

Maybe not that urgent, I restarted the component to get further logs and the error hasn't shown up again...

Same issue just hit vocms0283 (1.2.4.patch2)
2019-07-25 08:31:09,574:140484889880320:ERROR:SimpleCondorPlugin:No job report for job with id 1227112 and gridid 3197.99
2019-07-25 08:31:09,703:140484889880320:ERROR:BossAirAPI:Exception while completing jobs!
'ascii' codec can't encode character u'\u2018' in position 1403: ordinal not in range(128)
2019-07-25 08:31:11,058:140484889880320:ERROR:BaseWorkerThread:Error in worker algorithm (1):
Backtrace:
  <WMCore.BossAir.StatusPoller.StatusPoller object at 0x7fc5376bb310> <@========== WMException Start ==========@>
Exception Class: BossAirException
Message: Exception while completing jobs!
'ascii' codec can't encode character u'\u2018' in position 1403: ordinal not in range(128)
        ModuleName : WMCore.BossAir.BossAirAPI
        MethodName : _complete
        ClassInstance : None
        FileName : /data/srv/wmagent/v1.2.4.patch2/sw/slc7_amd64_gcc630/cms/wmagent/1.2.4.patch2/lib/python2.7/site-packages/WMCore/BossAir/BossAirAPI.py
        ClassName : None
        LineNumber : 534
        ErrorNr : 0

Traceback:
  File "/data/srv/wmagent/v1.2.4.patch2/sw/slc7_amd64_gcc630/cms/wmagent/1.2.4.patch2/lib/python2.7/site-packages/WMCore/BossAir/BossAirAPI.py", line 526, in _complete
    self.plugins[plugin].complete(jobsToComplete[plugin])

  File "/data/srv/wmagent/v1.2.4.patch2/sw/slc7_amd64_gcc630/cms/wmagent/1.2.4.patch2/lib/python2.7/site-packages/WMCore/BossAir/Plugins/SimpleCondorPlugin.py", line 345, in complete
    condorReport.addError(exitType, exitCode, exitType, logOutput)

  File "/data/srv/wmagent/v1.2.4.patch2/sw/slc7_amd64_gcc630/cms/wmagent/1.2.4.patch2/lib/python2.7/site-packages/WMCore/FwkJobReport/Report.py", line 604, in addError
    errDetails.details = errorDetails.decode('utf-8', 'ignore')

  File "/data/srv/wmagent/v1.2.4.patch2/sw/slc7_amd64_gcc630/external/python/2.7.13-comp/lib/python2.7/encodings/utf_8.py", line 16, in decode
    return codecs.utf_8_decode(input, errors, True)

I'm going to trap that exception and be done with this issue. It happens once every 6 months anyways.


[Question] TF2 strange farmer.

How do I make it so that I can get more value than 300000 at a time?
If I write "repeat": at least more than 300001, it does not work, it shows that everything is successful, logged off, but nothing happens, I met one person who has 77 million on 20+ guns, and wherever there are additional counters the same number, he can't be so long all 300k times to run the script for what?
For example: https://steamcommunity.com/id/goodgalRiRi/inventory/#440 (just look , that's amazing, LOL)

The script isn't perfect. There are lots of things to optimize. It could simple be the TF2 backend detecting spam and blocking it or something. I can look into it some time later and see if I can hopefully make it better.

Kay, im waiting, bro.

I have done a few of changes to the code. It should work a lot better now for large amounts. Download again and see if this solves your problem.

IDK What's im doing wrong, but

I noticed when testing not all increases are applied properly but I never had to be off by that much. I will look again and try with 15M to see what happens, hopefully I can optimize it

I did some changes and 15 million went through without problems. Try again!

I did some changes and 15 million went through without problems. Try again!

Lemme try

If you copy the link printed on the third line from bottom does it pre-select the correct item in your inventory?

Yes! On my mask, now i trynna to my sniper rifle....
It's different from event type 64(point scored) to 0(strange)

Strange, but my cosmetic still have 300k points, since i create a this thread. Now im awaiting when the script will finished for farm my sniper rifle.

No new updates?!


vep update

updated genome configs for vep v88
updated effects.py for vep v88
added vcfanno config for dbNSFP and dbscSNV

related to chapmanb/cloudbiolinux#250

Hi Brad,
quick question. Since the PR for cloudbiolinux was merged, new installations will only get the latest versions of the reference data files. For clinvar there's no issue, but won't this cause conflicts for 'fresh' installs of the genome resources here aren't updated too? This is the case for the vep cache and the dbsnp file, since the name changed too.
Cheers
M

Matthas;
Agreed, thanks for the push. Apologies, I'd done some testing but hadn't had a chance to finish. Thank you again for all this work getting it in place. We'll aim for a soonish release after folks have a chance to test so we can get data and code all synchronized in a stable release. Thank you again.

Hi Brad,
My pleasure. If you're interested, I'm willing to commit to keeping the datasets we use up to date. Is it possible for me to get some kind of notification if you're about to release a new version? That way I can check if everything's still up to date and prepare the PR's needed to include the latest data in the new release.
Cheers
M

Matthias -- thanks so much. I'll try to provide a heads up when we're reaching time for a new release. We're aiming to keep bcbio stable in development and just break off new releases over time. In general we try to get a release out in a month/two-month time frame so practically anytime around then is reasonable to start putting this together. I'd suggest only doing updates when you think it provides some value for interpretation, since there is a cost to needing to re-download and validate everything for users. Thank you again.

No problem,
judging by past releases, I'll start looking into new datasets every two months, at the beginning of the month, and put together a PR.
Cheers
M


fix: Improve "default" export conditions.

Noticed some issues with our exports, specifically with Jest v28, is that jest-environment-jsdom does not resolve our browser condition and throws a "Cannot find module" error.
By default, jest only considers the browser condition (which contains import/module) (https://jestjs.io/docs/configuration#testenvironmentoptions-object) and does not match against the outer node or default.
"exports": {
  "./package.json": "./package.json",
  ".": {
    "types": "./dts/index.d.ts",
    "browser": {
      "module": "./esm/index.js",
      "import": "./esm/index.js",
+     "default": "./lib/index.js"
    },
    "node": {
      "import": "./cjs/index-wrapper.mjs",
      "require": "./cjs/index.cjs",
+     "default": "./lib/index.js"
    },
    "default": "./lib/index.js"
  }
}

Because of this, our exports are basically ESM only, but jest does not support ESM that well yet. I'm trying to figure out the best way to handle this... I think the only solution is to add default to each condition again (above).

Hey @azu, since you've been using Packemon a bit lately, curious if you've run into this problem yet.

I have not met this issue yet.
I did a quick search, I could not found thie default issue
https://github.com/facebook/jest/issues/9771
https://github.com/facebook/jest/blob/05469328f49bd1d2435c47a70c47983355bc4f54/packages/jest-resolve/src/defaultResolver.ts#L11-L14
https://github.com/lukeed/resolve.exports#readme
@dai-shi written an atritcle about conditional exports. He may know somthing about this issue.
How Jotai Specifies Package Entry Points · Daishi Kato's blog

tbh, we are still having issues with jest. ex. https://github.com/dai-shi/proxy-memoize/issues/42#issuecomment-1110495594

I think the only solution is to add default to each condition again (above).

If adding default solves it, I think it's a good solution.

@dai-shi Yeah, I think to unblock for now, I'll add the default. I think in v3 I'll revisit the exports a bit and improve them if applicable.


Remove has-part from whitelist

Would we ever want has_part in annotation extensions?

@cmungall
The canonical Wnt signaling model uses has_part to denote the members of the beta-catenin destruction complex (as represented by the GO CC term):
http://noctua.berkeleybop.org/editor/graph/gomodel:56d1143000003025
Is that the kind of thing you're thinking of?

It was in response to this:
http://noctua.berkeleybop.org/workbench/annpreview/?model_id=gomodel:5900dc7400001088
while in this case it could be improved by filtering out to root, there will be other cases where has-parts shows up a lot in inferred extensions, this will likely irk people and I couldn't think of where curators currently use it in extensions

I think you are correct that curators have not used has_part in AEs. Karen will/has open/ed another ticket for filtering out any references to root nodes unless they are used in the conventional way to indicate that we know nothing about a gene product.

Okay, got it.  I'm not aware of cases where has_part has been used in manual annotation extensions.  I actually don't mind seeing them in the inferred GPAD extensions, but if we think they're going to be annoying to people, and they haven't been used in manual annotation, then it might be prudent to take them out.

So it sounds like we have consensus to take this out.


Add plots for results to smirky

I have added the data plots to that @camizanette and @davidlmobley have been using for smarty to smirky. I also updated the load_trajectory function to use pandas so it is more general. That means that the parsing works even when there are commas in the SMIRKS patterns. Not a lot of changes, but I thought I would make it available. I'm working on a few other updates to odds in smirky, but I'm not quite done yet.

Cool! Any chance you could post an example plot?

LGTM, I'll merge. Do post a plot for John when you have a chance, @bannanc . John, they're basic at this point -- primarily score versus time if I remember correctly.

@jchodera I left a longer simulation going when I left the office tonight, I'll post the resulting plot in the morning.


Surface area and volume of cylinder

fuction to find total surface area and volume of cylinder
Can I add above mention functions ?

@DhrutikPatel i think these functions have already been added. Please check the readme file.

@Swayam221 You're right. Issue closed.


GraphQL support

No beautifier yet. Would be terribly nifty to have. I use graphql-tag/language to put the queries into separate .graphql files. There is syntax coloring available as graphql-language, but no beautifier in atom-beautify.

see https://www.npmjs.com/package/graphql-formatter


⚠ Turnos 24x7 has degraded performance

In 267b1c6, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 3323 ms

Resolved: Turnos 24x7 performance has improved in 6b0b895.


Fix 404 paper url in README

The current link 404s, the new link is from the SIGGRAPH 2018 youtube video description.

Thanks!


Allow currency filter to specify places

This adds another parameter to the currency filter which allows specifying of the places to display. This is useful if you only want to display a currency as £100.

Where are my modals?! I jumped right into the code review 😳
Thanks for the PR. The jsdoc is indeed wrong

@jbrooksuk What is the use case for displaying 0, 1 or more than 2 decimals on a currency?

@jbrooksuk What is the use case for displaying 0, 1 or more than 2 decimals on a currency?

@posva mainly it's just when you have a more precise number but need to display it differently.

@jbrooksuk I see the concern on arbitrary numbers but what about currencies?

@jbrooksuk I see the concern on arbitrary numbers but what about currencies?

When you don't want the trailing ".00" on the end,

@posva @jbrooksuk What about instead of an option to specify decimal places, having an option to display whole numbers without decimals? So for example in USD, 22 would become "$22", but 22.2 would become $22.20. That way, you can guarantee you're not hiding rounding errors or other miscalculations.

Perhaps a boolean such as preferIntegers?

@jbrooksuk @posva  I believe your concern is valid for removing trailing '.00'.
For inspiration: Angular provides a similar parameter called fractionSize on their currency filter which you can reference here: https://docs.angularjs.org/api/ng/filter/currency
@chrisvfritz In that case an optional flag to the currency filter such as no_trailing_zeros could be an option.

@posva @jbrooksuk What about instead of an option to specify decimal places, having an option to display whole numbers without decimals? So for example in USD, 22 would become "$22", but 22.2 would become "$22.20". That way, you can guarantee you're not hiding rounding errors or other miscalculations.

@chrisvfritz that doesn't work in some scenarios. For example, the app I'm building needs to display the whole price, not the pence, but elsewhere it needs the pence.

Remember that this is an optional parameter and the default behaviour continues to be the same. I feel this adds more flexibility.

@jbrooksuk So in some cases there is a non-zero pence in the price, but you still prefer to hide it?

@jbrooksuk So in some cases there is a non-zero pence in the price, but you still prefer to hide it?

Yeah, and without re-implementing the filter, it's not "impossible". Like I said, this is an optional extension of existing functionality. It's just not strict anymore.

@FadyMak I like the noTrailingZeroes name! :smiley:
@jbrooksuk Hmm, I worry that might be an edge case. I feel like suppressing non-zero pence/cents/etc could very easily (and unintentionally) hide errors in other applications, if people used it hoping to only suppress trailing zeroes.

@jbrooksuk Hmm, I worry that might be an edge case. I feel like suppressing non-zero pence/cents/etc could very easily (and unintentionally) hide errors in other applications, as people might use it hoping to only suppress trailing zeroes.

If they do, then that's a matter of not understanding what the filter is doing.

@jbrooksuk I agree, but I think in designing APIs, it's important to be wary of releasing features that are especially prone to misunderstanding. I also have a preference for explicitness though. Since that's a feature that could easily hide rounding/calculation errors, I would want to make sure I (and other team members) remained very aware of that. So I would personally never use such an option even if it existed. I would instead opt for explicitness - with something like a computed value called priceWithoutPence that I used the currency filter on instead.
What are your thoughts @posva @FadyMak?

@jbrooksuk I agree with the point of being explicit. I believe the core API should provide basic functionality such as specifying the decimalPlaces as an optional parameter to the currency filter. This would result in the currency filter rounding to the specified number of decimal places.
Since the number of decimal places varies by locale (i.e.: the US Dollar ($) is usually represented with 2 decimal places whereas the Kuwaiti Dinar (KWD) is represented with 3 decimal places), the decimalPlaces parameter would default to the maximum number of decimal places for that specific locale.
Please reference the ISO 4217 World Currency Format to see the default number of decimal places a specific currency displays.
If a developer wishes to perform more complex formatting, they can achieve that by defining a custom filter of their own and chaining it to the currency filter.

Defaulting based on locale could be dangerous?

@FadyMak Good to know about some currencies using 3 decimal places instead of 2 or 0 - thanks for linking to that resource!

I would actually create a precision filter away from this to allow for more use cases,  and chain the filters.  This will help add more to the API at the same time as resolving this issue.

@jbrooksuk thanks for the PR! So to sum it up:

We want to avoid making built-in filters more complex than they are right now, because Vue is not a utility library and domain-specific logic is best left in userland. The default filters were provided for convenience but not for completeness.

As @blake-newman suggested, chaining an extra filter seems like a cleaner and smarter solution.

Although we are not merging this, we really appreciate your contribution!


Workflow Executions: Cancel

What does this PR do and why?
Describe in detail what your merge request does and why.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other pull requests.
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

[ ] I have evaluated the PR acceptance checklist for this PR.

@joshsadam @ericenns I've put this PR on hold for now as I was advised by Phil that WES isn't cancelling jobs at the moment, so we can't test it to ensure it works


Increase plot size or font size whenever needed

As mentioned by Gaël, some plots could have better readability after increasing either their size or font size.
One example is the Solution for Exercise M6.02, where the labels overlap the plot.
This is a nice-to-have but not a priority for v2.

Regarding the font size, I think that we are fine we this file: https://github.com/INRIA/scikit-learn-mooc/blob/master/python_scripts/matplotlibrc
So in the notebook and jupyter-book, it will be OK. It is true that some of the figures would benefit either from a slight increase or sometimes move the legend outside of the figure. I think that this second point should always be considered with the amount of matplotlib code that such change will involve.

TODO: put a list of problematic figures

Putting the legends outside the figures improved things a lot so the problem can be considered solved generally.
Once could still review all figures to generally increase the figure sizes overall. But at least they are readable enough now.


buidoperon

I am completely new to this, I followed your document step by step and when I did the 'make -j pyoperon' step at the end, I got this error, can you please tell me what is the reason? THANKS a lot!!!

Build command

mkdir build && cd build
cmake .. -G "Visual Studio 15 2017" -A x64 -DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake -DCMAKE_BUILD_TYPE=Release  -DBUILD_PYBIND=ON -DUSE_OPENLIBM=ON -DUSE_SINGLE_PRECISION=ON -DCERES_TINY_SOLVER=ON
make -j pyoperon

Hi, it seems like the pyoperon target is not generated by CMake. Could you post the whole output of the cmake command?

Hi, it seems like the pyoperon target is not generated by CMake. Could you post the whole output of the cmake command?

Thanks for your reply, here is my whole output，what I should do?

Hi,
I think you are missing the Python libraries which are necessary to build the python module. Normally, CMake should find them and display something like this:
-- Found Python3: /nix/store/k3agmx8hndsfd68cal0m9qc9dnbvfq1i-python3-3.9.6-env/bin/python3.9 (found version "3.9.6") found components: Development Interpreter Development.Module Development.Embed

Possibly you need to install Python3 via vcpkg install Python3:x64-windows or similar. Apologies, this should have been mentioned in the documentation.

Thanks a lot. I tried your suggestion, but vcpkg tells me Python3:x64-windows already exists,

I tried adding my python path to cmake, however the problem still exists

To rule out this cause could you make the following change to the CMakeLists.txt file and then re-run cmake?
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 27e5746..f2eceb6 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -40,7 +40,7 @@ set_package_properties(doctest PROPERTIES TYPE OPTIONAL) # unit tests
 set_package_properties(cxxopts PROPERTIES TYPE OPTIONAL) # command-line parser

 # python binding
-set_package_properties(Python3 PROPERTIES TYPE OPTIONAL)
+set_package_properties(Python3 PROPERTIES TYPE REQUIRED)
 set_package_properties(pybind11 PROPERTIES TYPE OPTIONAL)

 # get revision information from git

This way, if there is a bug in the Python3 port and for some reason CMake can't find it, then it will throw an error.

According to your suggestion, I modified the CMakeLists.txt file, not sure if the changes are correct

the output is as follows(┯_┯)

I made a few more attempts, and this time CMake should find python3
But the problem still exists(┯_┯)


Unable to use with go-jwt/v4

I'm trying to use the library with go-jwt/v4 but due to different signature of KMSSigningMethod in v2 it's not working.
Is there a way to go mod tidy import version 1.1.0 of library?
i think there is an error on version of go.mod in v1.
https://go.dev/doc/modules/version-numbers
require (
	github.com/aws/aws-sdk-go-v2/config v1.18.27
	github.com/aws/aws-sdk-go-v2/service/kms v1.22.2
	github.com/golang-jwt/jwt/v4 v4.5.0
	github.com/matelang/jwt-go-aws-kms/v2 v1.1.0
)

❯ go mod tidy
go: errors parsing go.mod:
/Users/fams/src/jws-signer/go.mod:9:2: require github.com/matelang/jwt-go-aws-kms/v2: version "v1.1.0" invalid: should be v2, not v1

Hi @fams ,
I'll have to check, sorry for the delay, I missed the notification from this issue.

Yes, the RSA-PSS is the alg used by me
On Fri, Jul 28, 2023 at 9:57 AM Máté Lang @.***> wrote:

Did you try version 1.0.0?
The only difference is support for RSA PSS? Do you need that?
—
Reply to this email directly, view it on GitHub
https://github.com/matelang/jwt-go-aws-kms/issues/18#issuecomment-1655643329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADVPG2QLZ64STI7BVKOJI3XSOZKXANCNFSM6AAAAAAZRYBCUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I will close this issue since I did not have time to backport that functionality to v4.
I will continue to maintain the compatibility with go-jwt v5 and AWS SDK version 2.