Iterator doesn't seem to be matching type aliases

Trying to use an Iterator to reduce memory consumption of a large result pulled back from a database (the nodes go through a few transformations before being serialized as JSON) and getting this error:
instantiating 'Iterator::Map(Indexable::ItemIterator(Array(Neo4j::Type), Neo4j::Type), Neo4j::Type, DatabaseMetric)#next()'
in /usr/local/Cellar/crystal/0.27.2/src/iterator.cr:722: type must be Neo4j::Type, not (Array(Neo4j::Type) | Bool | Float64 | Hash(String, Neo4j::Type) | Int16 | Int32 | Int64 | Int8 | Neo4j::Failure | Neo4j::Ignored | Neo4j::LatLng | Neo4j::Node | Neo4j::Path | Neo4j::Point2D | Neo4j::Point3D | Neo4j::Relationship | Neo4j::Success | Neo4j::UnboundRelationship | String | Time | Nil)

In this case, Neo4j::Type is an alias for all of those types, including Nil.
I unfortunately don't have a reduced example for this yet. I'll see if I can find one soon.

My advice is to avoid using recursive aliases. They are slightly broken. You can use wrapper structs to achieve more or less the same result. That will probably make it work (I don't know if you control the Neo4j library).

@jgaskins can you try doing exp.as(Neo4j::Type) with the return value of the iterator?
Otherwise, without some reproducible code we can't help.

Actually, it's super easy to reproduce:
alias Rec = Int32 | Array(Rec)

a = [] of Rec
a << a

p a.each.map(&.itself).to_a

And simpler:
alias Rec = Int32 | Array(Rec)

a = uninitialized Rec

f = ->(x : Rec) {}
f.call(a.itself)

I think this can be fixed, it seems to be a problem with Proc type matching and recursive aliases.

@jgaskins Sorry for all the above noise, it was a really simple bug to fix. It'll probably be available in the next release.

Nice work! I was going to try to get a reduced example after work today but you beat me to it! :joy: Thanks!


Build and push docker image on git tag

I've added a new github workflow to generate and push a docker image to use the PowPeg-Node in.
The new workflow logs in on both DockerHub and GitHub Container Registry, builds the code, builds the docker image, then pushes it to both repos. For now, the action will be triggered when a tag is pushed.
The required secrets needed to be set up in this repo for this to work are:
DOCKERHUB_USERNAME : username to login on dockerhub
DOCKERHUB_TOKEN : token to use to login on dockerhub
Please contact the DevOps team to request these credentials
The images will be pushed as:
rsksmart/powpeg-node:<RELEASE_NAME>
ghcr.io/rsksmart/powpeg-node:<RELEASE_NAME>

Had to push a small change to the dockerhub repo name


Changed optimizer to optax

What does this PR do?
This PR changes the optimzer optax as requested.
I tested the code again on local gpu and same result as before.
Fixes # (issue)
Checklist

[x ] This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
checks if that's the case).
[] This change is discussed in a Github issue/
discussion (please add a
link).
[ ] The documentation and docstrings adhere to the
documentation guidelines.
[ ] This change includes necessary high-coverage tests.
(No quality testing = no merge!)

Part of #1053 and #568

please, have another look. The commits got a bit strange names by accident but the formatting is now included.

Thanks for the comment.  Yes. I am running it now again and I do the updates when finished in few hours.  Accuracy seems alright a bit higher with the same settings for the optimizer, maybe previous changes caused this too.

Codecov Report

Merging #1916 (7b2e401) into main (2a3efe9) will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1916      +/-   ##
==========================================
+ Coverage   70.54%   70.67%   +0.12%
==========================================
  Files          58       58
  Lines        4895     4920      +25
==========================================
+ Hits         3453     3477      +24
- Misses       1442     1443       +1

Impacted Files
Coverage Δ

flax/linen/module.py
93.56% <0.00%> (-0.23%)
:arrow_down:

flax/core/scope.py
89.15% <0.00%> (+0.05%)
:arrow_up:

flax/linen/linear.py
99.14% <0.00%> (+0.08%)
:arrow_up:

flax/errors.py
87.32% <0.00%> (+0.55%)
:arrow_up:

Continue to review full report at Codecov.

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

One more thing: in the end you'll probably need to squash some of the commits because we request to only have 5 commits per PR.
https://github.com/google/flax/blob/625dc1664c9fc98aaf1b5226ca8f57f322fb59a5/.github/workflows/build.yml#L35

Thanks for the comments. I tried to squash it. I hope that worked.
I tried to figure out what causes the accuracy change. I could not spot it. I suggest to leave it for now and later to improve it. The example is only educational and does not aim for accuracy.

alright. I did a rebase, I hope this resolves it.

I think, this should be fine now?


[NPM] add npm docker images apps

What does this PR do?
Add npm docker images apps
Which scenarios this will impact?
Motivation
Additional Notes
linked to this PR

/merge


Some ideas for future :)

Thanks for the hard work! Just wanted to throw some ideas out there (please feel free to ignore if out of scope).
The ability to lookup a key in PEM format for returning would be quite helpful for me, I'm right now using exec-with-secrets to populate the environment variables from aws-sm then I have another script to dump it to a file and generate the config for PKCS11, but skipping the dump step would be handy.
Just a side point, I think storing certificates in aws-secrets-manager is quite elegant because Lambda's can be written to automatically rotate the certificates if needed at any point and it's part of that system running in the background.
Here's some suggestions for future

Plugin system for storing/retrieving public certificates. So far we have file/directly in json config, but I could image more. Later plugins could be written for aws-secretstore, S3 or REST API. That means we can do PKI signing via the pkcs11 plugin without caring about where the certificates are stored. For now, this is a separate process to grab the cert and fill in the json config file.
Add option to return additional fields for the keys, these can be populated via the config file, e.g. kms, version (populated from pkcs11 code version)
Return key description of AWS key (description is stored in aws)
Add ability to lookup a key by KMS Key Alias instead of key_id
Add ability to use a pin code stored in either config file in plaintext, or a pre-encrypted string (can be decrypted with a kms key)

My recent PR adds the ability to get the CERTs from acm-pca which is a step in the direction you want. I wouldn't be hard to put an option to execute something here but when you start executing random things, there are security implications... oh well.
Key Alias already works, just use alias/foo as your key ID :-)
Otherwise I'm sure Ian welcomes pull requests :-)

+1


chore: package generator version to 0.7.0-alpha

The package generator fails on npm install due to version 0.6.0 of many packages not being available. This resolves it.

LGTM


release: release 0.6.0

This PR buttons up the 0.6.0 release. Ups POM versions, apply SonarLint refactoring suggestions:
* 358d474 - refactor: clean up the wording on metamodel generator messages
* 4199a64 - ci: update wordlist for spellchecker action
* 92c821c - docs: add Spring Data Redis dependent version to README
* 704834f - refactor: constructors of an "abstract" class should not be declared "public" (SonarLint java:S5993)
* 0a18374 - fix: second class type in Pair not properly declared in getFirstGetter
* 7ea1a46 - refactor: optional value should only be accessed after calling isPresent() (SonarLint java:S3655)
* 7556c47 - refactor: "@Override" should be used on overriding and implementing methods (SonarLint java:S1161)
* 3bdc48b - refactor: utility classes should not have public constructors (SonarLint java:S1118)
* b79cc9c - refactor: deprecated code should be removed (java:S1133)
* 31370ae - refactor: "equals(Object obj)" and "hashCode()" should be overridden in pairs (SonarLint java:S1206)
* e67a72c - refactor: boxing and unboxing should not be immediately reversed (SonarLint java:S2153)
* b5ba884 - refactor: "equals(Object obj)" should be overridden along with the "compareTo(T obj)" method (SonarLint java:S1210)
* ab26885 - refactor: boxed "Boolean" should be avoided in boolean expressions (SonarLint java:S5411)
* a2e1b19 - refactor: Strings should not be concatenated using '+' in a loop (SonarLint java:S1643)
* 11fa6df - refactor: consecutive AssertJ "assertThat" statements should be chained (SonarLint java:S5853)
* 03015af - refactor: chained AssertJ assertions should be simplified to the corresponding dedicated assertion (SonarLint java:S5838)
* e96d8be - refactor: lambdas should be replaced with method references (SonarLint java:S1612)
* 7baf520 - refactor: use the diamond operator ("<>") to reduce the verbosity of generics code (SonarLint java:S2293)
* d2d0ea1 - refactor: use Collections.emptyList() instead of Collections.EMPTY_LIST
* 1a62802 - refactor: re-order Java modifiers per conventions
* cabbe17 - docs: add contributors list to README
* 801c3af - release: 0.6.0

Some comments:

We're generally removing contributors from READMEs everywhere, as it's a maintenance sink. On the other hand, the release drafter, already includes the contributors on the release. Can you remove from the readme.
CI was broken and not reporting back - I've fixed that - but note that CI now fails.Please have a look@bsbodden

Question: Given the goal set of the JDK + how it works, but more importantly not having any OS specific code in here (other than the demos folder)... why run CI across the platforms?

On the merge side - once CI passes, and we're merged to master... master should properly publish snapshots to mavenhub. If that happens - we're good to use the drafter.

Some comments:

We're generally removing contributors from READMEs everywhere, as it's a maintenance sink. On the other hand, the release drafter, already includes the contributors on the release. Can you remove from the readme.

Ok

CI was broken and not reporting back - I've fixed that - but note that CI now fails.Please have a look@bsbodden

Can you point me to the config that is running the build, the command should be:
./mvnw clean verify
from the root folder of the project.

Question: Given the goal set of the JDK + how it works, but more importantly not having any OS specific code in here (other than the demos folder)... why run CI across the platforms?

There is no OS specific code as far as I can tell, but don't know that is necessary to build on anything other than Linux/Mac OS/X

@chayim @sazzad16 the issue is that Docker needs to be added to the workflow. I'll try to do that and let you know.


Readme Links Not Working

Thanks for opening an issue!
Please select the type of issue you're reporting. For questions.

[x ] Bug
[ ] Feature Request
[ ] Question

Problem Report

The readme links do not work, because they reference a jsx file, which was probably switched to js without updating the links.

System Info
N/A
Reproduction
Click on the example links in the readme

Fixed


Readme changes

Thanks for pointing this out. It's a typo for sure. Have raised a PR for rectification.


ARROW-5397: [FlightRPC] Add TLS certificates for testing Flight

This needs https://github.com/apache/arrow-testing/pull/2.

Codecov Report

Merging #4510 into master will increase coverage by 0.86%.
The diff coverage is 44.82%.

@@            Coverage Diff             @@
##           master    #4510      +/-   ##
==========================================
+ Coverage   88.27%   89.14%   +0.86%
==========================================
  Files         846      696     -150
  Lines      103662    91957   -11705
  Branches     1253        0    -1253
==========================================
- Hits        91512    81978    -9534
+ Misses      11903     9979    -1924
+ Partials      247        0     -247

Impacted Files
Coverage Δ

python/pyarrow/flight.py
80% <ø> (ø)
:arrow_up:

cpp/src/arrow/flight/server.h
100% <ø> (ø)
:arrow_up:

cpp/src/arrow/flight/test-util.h
100% <ø> (ø)
:arrow_up:

cpp/src/arrow/flight/types.h
96.87% <0%> (-3.13%)
:arrow_down:

cpp/src/arrow/flight/types.cc
91.35% <100%> (+0.44%)
:arrow_up:

python/pyarrow/_flight.pyx
46.29% <35.71%> (-0.04%)
:arrow_down:

cpp/src/arrow/flight/test-util.cc
76.96% <38.63%> (-12.59%)
:arrow_down:

cpp/src/arrow/flight/flight-test.cc
93.08% <39.13%> (-6.92%)
:arrow_down:

python/pyarrow/tests/test_flight.py
77.35% <57.57%> (-2.73%)
:arrow_down:

cpp/src/arrow/flight/server.cc
91.63% <66.66%> (+1.03%)
:arrow_up:

... and 152 more

Continue to review full report at Codecov.

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

The submodule is now up-to-date, let's make sure the tests run in CI...

C++ with conda-forge appears to run the tests, but not Python with conda-forge (even though it sources travis_env_common.sh); I'll take a look at that when I can.

Looks like one of the C++ builds flaked when downloading deps, but otherwise tests run in CI now.

I restarted the failed build.

Thanks, looks like tests pass!

Gonna do a final review.

Did the Thrift download fail (I just opened https://issues.apache.org/jira/browse/ARROW-5576)?

@wesm Yeah, that was it.

OK, I looked into this and we're using the Apache dist system inappropriately, I commented at https://issues.apache.org/jira/browse/ARROW-5576?focusedCommentId=16863112&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16863112

Thanks @lihalite !


FB-729

FB-729 -- Refactor unit tests for fb-components-core schemas
This ensures that Runner has the latest Components Core as an explicit dependency

I forgot that dependabot is even a thing


Creating ELB on AWS not passing health check on EKS instances in ASG

Describe the bug
We have a (classic) ELB being created in AWS successfully, however, the health check is not passing for any of the instances, so the load balancer does not pass any traffic to EKS.
The instances are part of an Autoscaling Group, however, when I go to the ASG (and EC2 instances itself), they are listed as "Healthy".

The ASG has "EC2 & ELB" Health Checks enabled.

And you can see that they're all listed as "InService" and "Healthy" in the ASG Instance Management tab.

The configuration for the Health Check is as the Emissary CRDs configured it.

However you can see that none of the instances are listed as "InService"

I am using the Default Listeners, as well as 1 additional Listener, Host and Mapping.
These are the Defaults:
NAME                              PORT   PROTOCOL   STACK   STATSPREFIX   SECURITY   L7DEPTH
emissary-ingress-http-listener    8080   HTTP                             XFP
emissary-ingress-https-listener   8443   HTTPS                            XFP

The services they produced:
NAME                     TYPE           CLUSTER-IP      EXTERNAL-IP                                                                PORT(S)                      AGE
emissary-ingress         LoadBalancer   <IP_ADDRESS>   xxx-xxx.eu-north-1.elb.amazonaws.com   80:31042/TCP,443:32030/TCP   6h7m
emissary-ingress-admin   ClusterIP      <IP_ADDRESS>   <none>                                                                     8877/TCP,8005/TCP            6h7m
emissary-ingress-agent   ClusterIP      <IP_ADDRESS>   <none>                                                                     80/TCP                       6h7m

This is my own listener:
NAME                     PORT   PROTOCOL   STACK   STATSPREFIX        SECURITY   L7DEPTH
code-test-cakery-http    8080   HTTPS              code-test-cakery   XFP        0
code-test-cakery-https   8443   HTTPS              code-test-cakery   XFP        0

Created by the following config:
apiVersion: getambassador.io/v3alpha1
kind: Host
metadata:
  name: {{ .Chart.Name }}-host
  labels:
    labels.getambassador.io/hostname: {{ .Values.ingress.host }}
spec:
  hostname: {{ .Values.ingress.host }}
  tlsSecret:
    name: {{ .Values.ingress.host }}-tls
  requestPolicy:
    insecure:
      action: Redirect
---
apiVersion: getambassador.io/v3alpha1
kind: Listener
metadata:
  name: {{ .Chart.Name }}-http
  labels:
    labels.getambassador.io/listener: {{ .Values.ingress.host }}-80
spec:
  port: 8080
  protocol: HTTPS
  securityModel: XFP
  statsPrefix: {{ .Chart.Name }}
  l7Depth: 0
  hostBinding:
    namespace:
      from: SELF
    selector:
      matchLabels:
        labels.getambassador.io/hostname: {{ .Values.ingress.host }}
---
apiVersion: getambassador.io/v3alpha1
kind: Listener
metadata:
  name: {{ .Chart.Name }}-https
  labels:
    labels.getambassador.io/listener: {{ .Values.ingress.host }}-443
spec:
  port: 8443
  protocol: HTTPS
  securityModel: XFP
  statsPrefix: {{ .Chart.Name }}
  l7Depth: 0
  hostBinding:
    namespace:
      from: SELF
    selector:
      matchLabels:
        labels.getambassador.io/hostname: {{ .Values.ingress.host }}
---
apiVersion: getambassador.io/v3alpha1
kind:  Mapping
metadata:
  name:  {{ .Chart.Name }}-mapping
spec:
  service: {{ .Chart.Name }}.{{ .Chart.Name }}:80
  timeout_ms: 3000
  connect_timeout_ms: 3000
  retry_policy:
    retry_on: 5xx
    num_retries: 1
    per_try_timeout: 0.150s
  hostname: {{ .Values.ingress.host }}
  prefix: "/"

Versions (please complete the following information):

Emissary Ingress: 2.3.0 (Helm Chart 7.4.0)
Kubernetes environment: EKS
Kubernetes Version: 1.21

Doing an instance refresh on the instances appears to have resolved this.

Thanks, @alexbowers for the follow up and reporting on that outcome.


Merge MoonDAO style local cache into nance.app

https://github.com/nance-eth/MoonDAO-app/blob/main/ui/components/nance/ProposalLocalCache.tsx
has nice bar at top that says when the cache was saved. also allows for uuid based caching so you can edit multiple proposals and have local cache (only one new one though, could be improved upon)

implemented by https://github.com/nance-eth/nance-interface/commit/5818268d101bd1901d1c68c460f8f04cc987c313


parse_versa requires amara3 (reported by https://github.com/azaroth42)

Traceback (most recent call last):
  File "/Users/azaroth/install/bin/parse_versa", line 24, in <module>
    from versa.reader.md import from_markdown
  File "/Users/azaroth/install/lib/python3.4/site-packages/versa/reader/md.py", line 18, in <module>
    from amara3.uxml.parser import parse, event
ImportError: No module named 'amara3.uxml'

pip3 install amara gets amara-2.0.0 which doesn't install.
pip3 install amara3 doesn't find anything.
Giving up.

I am having the same issue and PyPI does not have the latest 0.4.1 version.

@jadeeshah very sorry for the delay in getting to this. I knew a new release was due anyway, and I finally squeezed out 0.5.0 and all its dependencies.
I have confirmed that a simple pip install pybibframe (with or without -U). You no longer need to mess with requirements.txt. If you continue to have trouble let me know. Thanks.


Refactor and speed up ci tests

Refactor/speed up CI tests:

[x] Have a job build and cache all the dependecies:

Use Swatinem/rust-cache@v2 to cache all the Rust files and dependencies
Use nextest --archive option to cache nextest files and dependencies
Use actions/cache/save@v3 to cache external dependencies builds (vit-ss,jormungandr etc)

Note that Rust files are only cached on merge to main as cache data created in a pull-request-triggered CI run can't be seen by other PRs. Instead nextest archive and external dependencies are rebuild at each run, cached to be passed to the matrix of runners and then cleaned up. For this untill this PR is merged not all the cache will hit. On my fork, when all cache hits the total running time is about 35min

[x] Use matrix with nextest --partition command to run the tests accross different runners.
[x] Make coverage and clippy job use the same cache as cargo build
[x] Add integration tests
[x] Divide building tests and dependecies in two different runners

Future work:

Pick the slowest tests and reduce their running time
Dont rebuild deps if is not necessary
Add remaining tests and features

nice!

@Mr-Leshiy this pr is tagged as do not merge and do not review :)

@kukkok3 do you think that .github/workflows/ci-tests-n-coverage.yml can be included in this PR?

@kukkok3 do you think that .github/workflows/ci-tests-n-coverage.yml can be included in this PR?

Yes it would make sense, should I update it?


Ikea Tradfri Wireless Dimmer

Version plugin 1.2.2
Version lib 0.28.1 / 0.28
Version firmware 3.0f
After upgrading the lib, the IKEA wireless dimmer does not work. When added is shows up as an generic device with only refresh. If I downgrade to 0.27.9 it works, and the dimmer is added correctly with all values and commands

will be fixed on lib 0.28.2

sounds great :-)

Hi again
Something is completely wrong with the IKEA support.
Tried to update to 0.28.2 and the dimmer is recognized as an IKEA dimmer, but the link quality is 0. I tried to refresh the dimmer and the "Commandes" page is filled with "weird" information :-)
I then tried to downgrade to 0.27.9 and the link quality is shown correctly.
One problem with all versions of the lib, is that I can't connect the dimmer to any bulb.
For testing purpose, I tried to install Abielle and everything work as it should. I can add the dimmer to the ZigBee network, connect the bulb to the dimmer, and control everything from Jeedom.
Is it correct that I can't see the dimmer state in Jeedom when using this plugin?? (eg. 10%) In Abielle I can see the dimmer state and the bulb state.

The remote_level_button is not getting change using the remote ?

No. Not at all.
This is how it looks on the dashboard

Looks like the device "type" is missing, maybe discovery hasn't work.
Could you try to call "Découverte" on that device ? Remember to awake the remote to get change.

About the Link Quality, it should works, if not maybe the device is not paired ?
When you said you can't connect the bulb and the dimmer, what are you doing with abeille ? and with zigate ?

I ordered a bulb and a dimmer, it will be more easy to test

I have tried to include/exclude it multiple times but nothing changed. I even tried to PDM reset. It’s like something is blocking the dimmer when included so I can’t connect the dimmer with the bulb. I tried “Découverte” and it was after that, that all the entireties when number showed up on the commandes page.
In Abeille I can connect the dimmer to ZiGate and it shows on the dashboard as a dimmer correctly. I can then afterwards connect the dimmer to the bulb as original with the Ikea Tradfri. It then afterwards shows up on the dashboard correctly as a bulb, so both a bulb and a dimmer is on the dashboard

ok I understand, there's some things that need to be clarified :

The dimmer can be paired to zigate : before 3.0f, zigate can only get some info like battery, with 3.0f zigate can catch more events, like on,off or level change. To do that both dimmer and zigate need to be in the same zigbee group (and in the future I'll try to do it automagically)

The bulb can be paired to zigate and can be control by zigate

When you pair the dimmer to the bulb using Ikea style, technically the dimmer create a zigbee group and put the bulb into the same group in order to control it because the dimmer send commands (on/off/level) to a zigbee group not to a device.
So if you put the zigate in the same group, you'll be able to catch events from dimmer. (group list can be retrieve using zigate command 0x0062)

Beside that, as convenient functions, for the dimmer Abeille add commands on the jeedom object to send on / off and level to the zigbee group but really it as nothing to do with the dimmer
Any time we could create a button and send a ON command to a group and if there's some bulbs in that group they will be turn on.

You can't control the dimmer using zigate, Abeille just mimic the dimmer.
If this is what you want, maybe we can add it in a next release

As long as I can see battery status, link quality and be able to add bulbs directly with the dimmer, I’m happy :-)
One thing though that could be nice, was to be able to see which devices in groups just like the ikea app, and to be able to move a device from group to group.

This will be added soon ;)

I planned to :

Automatically add zigate to group 0000 (to be able to catch event)
Automatically add ikea remote to group 0000 (to catch event)
Add UI to manage groups per device

You are a champ :-)
I really appreciate the great work you are doing.
Btw. Is the plugin working better in HA? Maybe I should try it again. Been over 6 months since I’ve tried it last time:-)

IMHO HA is better :smile:

Is the zigbee plugin for HA just as good as yours for Jeedom?

Since you seems interested by battery informations you have to know that there's no battery panel in HA (not natively) but you can still check battery level per device
some screen of my HA

https://github.com/doudz/homeassistant-zigate

That’s really nice:-)
Think I’ll take it for a spin later today:-)

is it only the ZiGate folder I have to clone into custom_components ?
What about the panels folder?

The zigate folder should be in custom_components and the panels folder at the same level of custom_components

And you can copy your zigate.jso n file from Jeedom to your ha config folder to keep your config

can I expose it to HomeKit? :-)

Yes https://www.home-assistant.io/components/homekit_controller/

Which one will you recommend. Hass.io or Hassbian? :-D

No Idea, I use a custom install.
https://community.home-assistant.io/t/what-is-the-difference-between-hassbian-and-hass-io/35108

Closing, since I made a test and it's working

@doudz
Maybe I'm just a complete fool, but now the Link Quality is not shown in Jeedom. I can see the LQI values in the json file though.
And now my Aqara wireless switch gets disabled after a day or so :-(

If you click on "Synchroniser" the LQI should appear.
You could also try to open "informations" of the jeedom object to see differents values like lqi or lastCommunication

It did not work, Now I'm installing on a fresh Jeedom to see if it helps.
Looks like Xiaomi problems are known, sadly :-( Hope it will be fixed soon. I'm planning to use the switch as a door bell, but that's not an option atm, when Xiaomi devices are so unstable

I'm not sure you use the last release of the plugin, on the "Santé" page  I see "Rssi" but it has been renamed LQI

I'm not sure you use the last release of the plugin, on the "Santé" page I see "Rssi" but it has been renamed LQI

I'm on the latest release, and it still says Rssi. Maybe it's changed in the pre-release?

Do you know how long it will take before the update is pushed to the Jeedom market? :-)

Jeedom Market is updated every night

Okay. Then I have to wait until tomorrow to test it, unless there’s a way to update it manually

working now :-)


Defining request stubs in an external config file.

I'm wondering if this is possible in the current version of webmock:
# spec/helpers/api_helper.rb
require 'webmock'

module APIHelper
  include WebMock::API

  stub_request(:get, api_path).to_return(status: 200)
end

And then including this config only for certain specs (for rspec)
# spec_helper.rb

config.include APIHelper, :type => :controller

Without having to explicitly call stub_request in  a before :each in the actual test suite?
For illustrative purposes, this is how I solve this problem currently (feels hacky):
# spec/helpers/api_helper.rb

require 'webmock'
require_relative '../spec_helper'

include WebMock::API

RSpec.configure do |config|
  config.before(:each) do
    stub_request(:get, api_path).to_return(status:200)
  end
end

As you can see, I haven't even wrapped the code in a module because then I got an error message saying that stub_request was not available.
Thanks :-)

Thanks for submitting, however I am going to close this because of the time elapsed. If it still something you'd like to see in webmock, please feel free to open a new issue, or pull request.


Rehydration bug

I'm using this plugin with the ssr option turned to true which is supposed to generate unique identifiers for React components to avoid issues during re-hydration in case the two render trees doesn't match perfectly.
However I'm observing inconsistent rendering with the wrong child being in the wrong parent component.
You can see a minimal reproduction of this issue here: https://github.com/mquandalle/repro-gatsby-issue-5993

Well it's going to render as the "else" of the ternary because isMobileLayout will always be false on the server. Then rehydration should kick in and replace it since client side knows about that thing.
Generally speaking, it's better to not have this sort of code or only do the switch on componentDidMount, because the recommended way of doing isomorphic React views is server and client should render the exact same thing prior to mount.

Wow you were super fast to answer, thank you so much! I updated the code 1 min after publishing this issue (it's the same but a bit more explicit about the issue).
The goal is indeed to have a SSR version that is a bit different than the first client render, which I understand isn't the best practice, but can happen on real-world occasions (for instance when we want to display different components depending on the screen size). A switch as in the reproduction works without styled-component: the page is first rendered using the SSR version and quickly after there is a small flash to replace it with the client-side version.
The problem with the styled-component version is that the resulting DOM is inconsistent with the React component (wrong component hierarchy), a problem I thought the componentId generated by the ssr: true option of this plugin was mean to avoid.

a problem I thought the componentId generated by the ssr: true option of this plugin was mean to avoid

Oh nope, separate things. The consistent componentId is specifically just making sure the same component definition has a consistent className hash between client and server. It doesn't have anything to do with changing out what component you are rendering.
One thing that occurred to me is if the underlying HTML element (e.g. <div>) is the same, I don't think React will do a full replacement on rehydration. It just warns that the classes are different.
To force React to behave, you could try using an alternate base tag in this edge case like <span> etc to force a re-render.

Yes I tried using different tags and it works in minimal examples, but in practice the issue is that as soon as there is a <div> in the page, the reconciler is going to re-use the SSR version and break the layout.

@mquandalle late followup but it looks like gatsby didn't have the ssr option turned on for certain rendering phases, which may have contributed to this issue: https://github.com/gatsbyjs/gatsby/pull/7912


Bump framework and basic auth version

Purpose
Note $subject
Related PRs

https://github.com/wso2/carbon-identity-framework/pull/5612
https://github.com/wso2-extensions/identity-local-auth-basicauth/pull/200

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/8532943854

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/8532943854
Status: success


Imporve the documentation

The documentation of the production deployment should be improved.
What are this variables ? ORG, OU, PIVPN_SERVER, PIVPN_CONF.
Are they boolean, string or int.
At least provide a sample or a documention about those variables.

@melakhvision
Fair point, I'll add some detail on those.
The first two are EasyRSA values intended to improve the cryptographic strength of generated keys and certificates.
ORG = Organization
You can use your company name or make something up if you don't have one.
OU = Organizational Unit
A typical value here would be IT (for Information Technology), or again you can make something up.
The second two are PiVPN values, which include the the unique ID PiVPN uses in creating your server certificate. If your look at the /etc/openvpn/server.conf file generated by PiVPN, you'll see it in the cert and key directives in the file. That long string, which starts with the hostname of your OpenVPN server and ends with a .crt or .key is what you want to use.
So on my test system:
PIVPN_SERVER = raspberrypi10_ca8554c5-6025-442f-9d06-25a5cab85a3c
PIVPN_CONF = server.conf (which is the default name created by PiVPN)

@melakhvision
I added some comments to the docker-compose, and I'll add some to the README as well:
version: '3'
services:
  gui:
    image: bnhf/openvpn-admin-plus:latest
    container_name: openvpn-gui-tap
    environment:
    - OPENVPN_ADMIN_USERNAME=admin # Leave this default as-is and update on first-run
    - OPENVPN_ADMIN_PASSWORD=b3secure # Leave this default as-is and update on first-run
    - COUNTRY=${COUNTRY} # Country, Province, City, Org and OU are used by EasyRSA
    - PROVINCE=${PROVINCE} # Province or State
    - CITY=${CITY}
    - ORG=${ORG} # Organization - Use your company name here or make one up
    - EMAIL=${EMAIL}
    - OU=${OU} # Organizational Unit - Use your company department name or make one up
    - PIVPN_SERVER=${PIVPN_SERVER} # The unique name used for your server's certificate and key. Found in the /etc/openvpn/server.conf file.
    - PIVPN_CONF=${PIVPN_CONF} # Filename for your OpenVPN server configuration - usually server.conf
    - TZ=${TZ} # Your OpenVPN server's timezone
    ports:
     - "8080:8080/tcp"
    restart: always
    volumes:
     - /etc/openvpn:/etc/openvpn
     - ./openvpn-data/db:/opt/openvpn-gui-tap/db

@melakhvision
I added the same commented environment section to the README, so I think we should be covered. I noticed PIVPN_SERVER and PIVPN_CONF were already discussed there, but of course it doesn't hurt to have some things described multiple times.
ORG and OU definitely needed clarification though. Those values used to be requested during a PiVPN script install of OpenVPN, but I don't think that's the case anymore. They're still used to improve cryptography when available, AFAIK.
I'm closing this one out, but feel free to re-open if needed. Cheers.


Step to ZIP a directory and then export the path as an environment variable

Inputs would be the target directory path (to compress) and the path where the zip file should be placed.
Output is an environment variable to that path, although this might not be required as you can provide the zip file path (input) as an env var too.

Additional inout: should the directory itself be part of the zip.

This issue was moved to https://discuss.bitrise.io/t/step-to-zip-a-directory-and-then-export-the-path-as-an-environment-variable/116


E2E test for init script

From @aerokite on October 2, 2017 9:41
Need to support e2e test
Use pg client to check particular data
Copied from original issue: kubedb/operator#119

Solved


Fix typo on the part of documenting inner_xml()

I find typo (maybe it is) on inner_xml() RDoc.
If it is actually, please merge it.

Thanks!


No error for unreachable code if other clauses in 'switch' don't return

function f(x) {
    switch (x) {
        default:
            return false;
        case 2:
            200;
    }
	return true;
}

I expected an error on reachability on case 2, its statement, or return true, but we don't correctly check this.

The actual semantics indicate that this issue is wrong :smile:


Trackbar jumps to wrong position when clicked.

https://i.gyazo.com/c185f613865f1ca38e36b02cbefc7044.gif
https://i.gyazo.com/bfde64ab95e0c0f4844d533fc246cba7.gif
The trackbar jumps when clicked on.  Appears to be made worse when the starting offset is non-zero.  Replicate with the setting:

REF: https://discord.com/channels/531175899588984842/599270434642460753/892811075966349353

I tried to mitigate this by subtracting 29 from _dragOffset in https://github.com/blish-hud/Blish-HUD/blob/dev/Blish HUD/Controls/TrackBar.cs#L96 like so:
var relMousePos = this.RelativeMousePosition - new Point(_dragOffset - 29, 0);
This seems to lessen the effect but the closer you get to the left or right of the bar there will be a tiny drift to either side.
I will investigate this further.

Found part of the issue:
If a MinValue other than 0 is specified, then this would be the wrong slider offset as seen in the examples above.
Working on a fix now.

Closed by #556!


Possible bug: Tools menu option disappears after switching to SD2 card

Hi ChristianHaitian,
I am working on setting up my devices, and I was having trouble getting to PortMaster. If I do a fresh install of ArkOS, it will show up under Options then Tools. But when I switch to SD2 for ROMS, that tool menu goes away. I have tried this on both a RG503 and RG351V. On both devices, that Tools menu disappears and won't come back, even if I switch it back to the main SD card.
I hope I am clear on my description of the issue. If you need any log files or screen shots, please let me know.
Thank you so much for your work on this fantastic OS!!

Hello,
The tools menu goes away because it doesn't find any tools in the roms2/tools folder.  Reinstall something like ThemeMaster or PortMaster there and it will reappear again.

Please forgive my ignorance, but I am not sure how to do reinstall ThemeMaster or PortMaster. If it isn't too much trouble, can you point me in the right direction?


[FIX] Promises Thenable Type, Media Find Type

Also includes eslint updates & a test file with my dummy configuration.
Contains the fix for Issues/18

Waiting for @jazibsawar's approval.
Will remove the test file and merge once approved.

I refactored to clean up the code. One scenario is not working though,
cosmic.media.findOne({})
  .then(res => console.log(res))
  .catch(err => console.log('THIS NEVER GETS CALLED EVEN ERROR');

I refactored to clean up the code. One scenario is not working though,
cosmic.media.findOne({})
  .then(res => console.log(res))
  .catch(err => console.log('THIS NEVER GETS CALLED EVEN ERROR');

Thanks for the cleanup. I've included a new condition in findOne parameter object to not be empty. Looks like the condition never goes into the catch block because the request pulls through successfully in the try block, which it shouldn't - making me wonder if we're considering this as a valid request on our backend.

I refactored to clean up the code. One scenario is not working though,
cosmic.media.findOne({})
  .then(res => console.log(res))
  .catch(err => console.log('THIS NEVER GETS CALLED EVEN ERROR');

Thanks for the cleanup. I've included a new condition in findOne method for the parameter object to not be empty. Looks like the condition never goes into the catch block because the request pulls through successfully in the try block, which it shouldn't - making me wonder if we're considering this as a valid request on our backend.

It wasn't about an error from the backend. It doesn't go to catch even if there is an error. I tested it by throwing an error

I refactored to clean up the code. One scenario is not working though,
cosmic.media.findOne({})
  .then(res => console.log(res))
  .catch(err => console.log('THIS NEVER GETS CALLED EVEN ERROR');

Thanks for the cleanup. I've included a new condition in findOne method for the parameter object to not be empty. Looks like the condition never goes into the catch block because the request pulls through successfully in the try block, which it shouldn't - making me wonder if we're considering this as a valid request on our backend.

It wasn't about an error from the backend. It doesn't go to catch even if there is an error. I tested it by throwing an error

I tried passing in an invalid string value (for which it should throw an error) and it did the error handling correctly for me.
Can you try with .findOne({ original_name: 'bad_request' }) in the test file to confirm?

I refactored to clean up the code. One scenario is not working though,
cosmic.media.findOne({})
  .then(res => console.log(res))
  .catch(err => console.log('THIS NEVER GETS CALLED EVEN ERROR');

Thanks for the cleanup. I've included a new condition in findOne method for the parameter object to not be empty. Looks like the condition never goes into the catch block because the request pulls through successfully in the try block, which it shouldn't - making me wonder if we're considering this as a valid request on our backend.

It wasn't about an error from the backend. It doesn't go to catch even if there is an error. I tested it by throwing an error

I tried passing in an invalid string value (for which it should throw an error) and it did the error handling correctly for me.
Can you try with .findOne({ original_name: 'bad_request' }) in the test file to confirm?
In your specific case with the empty object ({}) it did not go into the catch statement.

Are you using try-catch? Try catch do work. But then().catch() doesn't for me. I'll try again

I refactored to clean up the code. One scenario is not working though,
cosmic.media.findOne({})
  .then(res => console.log(res))
  .catch(err => console.log('THIS NEVER GETS CALLED EVEN ERROR');

Thanks for the cleanup. I've included a new condition in findOne method for the parameter object to not be empty. Looks like the condition never goes into the catch block because the request pulls through successfully in the try block, which it shouldn't - making me wonder if we're considering this as a valid request on our backend.

It wasn't about an error from the backend. It doesn't go to catch even if there is an error. I tested it by throwing an error

I tried passing in an invalid string value (for which it should throw an error) and it did the error handling correctly for me.
Can you try with .findOne({ original_name: 'bad_request' }) in the test file to confirm?
In your specific case with the empty object ({}) it did not go into the catch statement.

Are you using try-catch? Try catch do work. But then().catch() doesn't for me. I'll try again

Ah, gotcha. I was using Try-catch in the function. Just tried with the chaining methods and catch isn't working.


TabView fails to update tab contents

When the TabView widget's children change, we don't change the children
we give to the PageViewport because we fail to update _items. This patch
watches didChangeConfig and updates _items appropriately.
Fixes #1134
Fixes #1168

LGTM


Clientside rendering

Adds clientside rendering for pages.
Also adds page titles and active states for the top nav.
When you're offline it still does a full page reload from the cache.
fixes: https://github.com/lnug/lnug.github.io/issues/112
Well kinda, we still fetch related meetup and the contact page from the other repos at build time (bin/fetch-markdown.js)  and bring them into this repo as html files. That way they can easily be used for client side render from the pages folder.
We need to verify google analytics is behaving correctly with the clientside routing.

wow - nice PR


TypeScript fails on basic console.log

http://jsbin.com/tobaxejaco/edit?html,js,console,output
This is only a basic console.log("hi"); and it fails with the following error:
typescript processor compilation failed
jsbin-3.30.3.min.js:18 Object {}
jsbin-3.30.3.min.js:18 Uncaught (in promise) TypeError: Cannot read property 'html' of undefined
    at http://static.jsbin.com/js/prod/jsbin-3.30.3.min.js:18:11001(anonymous function) @ jsbin-3.30.3.min.js:18

TS is utterly out of date, so I'm making a plea for help, because I could never get the client side script to build: https://twitter.com/rem/status/617451826263269378
I do know that it's since moved to github rather than MS's version of repo control, which might make it easer...who knows...

Ok, I'll dig into it this week. I've used the client-side script with jspm, so I'll investigate how they're doing it.


Some attributes can't be used as arguments for filter()

Hi,
Thanks (again) for the nice API.
Not a bug per se, more of an feature request. Using pynetbox (2.1.0 / netbox 2.1.4 ), some attributes can't be used as query arguments for model.filter() - despite that Django "permits" it:
This works via nbshell/Django

Interface.objects.filter(device=1, name='gi0')[0].id
1499
lookup ip to interface
IPAddress.objects.filter(interface=1499)[0]
<IPAddress: <IP_ADDRESS>/25>

pynetbox will return the entire dataset for the same query

nb.dcim.interfaces.filter(id=1,name='gi0')[0].id
1499
nb.ipam.ip_addresses.filter(interface=1499)

Returns all IPaddresses
nb.ipam.ip_addresses.filter(description='FOO')
Returns all IPaddresses
This works as expected however:
nb.ipam.ip_addresses.filter(family=6)
nb.ipam.ip_addresses.filter(status=3)
nb.ipam.ip_addresses.filter('<IP_ADDRESS>/32')

Hey @oysteingy
I didn't have time to reproduce, so if let me know if I'm off with my explanation. The django shell can filter on any field in the database, but there's a more limited set of fields available to filter on in the API. E.g. if you're looking for what filters you can pass to the interfaces endpoint, they're defined here. Unfortunately, it doesn't look as though name is available right now.
Those variables that are django_filter objects are basically the named args you can pass to .filter() in pynetbox (device, device_id , mac_address, etc.). The reason you're getting the whole dataset is that's default NetBox behavior when you pass a filter that isn't defined.

Hi,
Sorry, this is just me being silly. After looking at the available query arguments for IPAM https://github.com/digitalocean/netbox/blob/develop/netbox/ipam/filters.py  the stars aligned for me .
'interface' is a valid query arguments in Django/nbshell, but not in the API.
'interface_id', however is available in the API for queries.
Sorry for the mess, and thank you very much for your help.


HITL unable to upload mission - missing battery params

It appears that this issue is still present in the current PX4 firmware build. I am trying to run HITL using AirSim on a Pixhawk 4 board and I am unable to run a mission due to the following errors:
When I boot up, it gets this issue with battery parameters:

Then, when trying to upload a mission, it gets these issues:

Also, I get a message saying that the vehicle requires setup prior to flight.

Steps to reproduce:

Flash Pixhawk with latest firmware
Set airframe to HIL Quadcopter X
Calibrate RC radio and set flight modes
Try to upload a waypoint mission with AirSim

Current setup:
QGC Version - v4.0.10
PX4 Firmware version - v1.11.1
Airframe - HIL Quadcopter X
What I have tried:

Re-flashing the board
Re-installing QGC
Resetting all parameters on the board
Setting the CBRK_SUPPLY_CHK to 894281

Nothing appears to work and I am unable to fly a mission. I am able to arm and fly it manually using an RC Radio controller, however. According to #14281 this has been fixed, so why isn't it working?

@bperseghetti Any insights?

After some looking around, I decided to try flashing an older version of PX4 on the Pixhawk to see if that works. Using v1.9.2 I have managed to successfully upload missions and fly them autonomously in AirSim. Is there a chance that something was fixed and then reverted?
I did notice that on the first board I tried it worked straight away with v1.9.2 and on another board I needed to swap out a perfectly functional SD card for another. Is it possible that the SD card needs to be wiped or re-formatted before uploading missions with HITL?

I had this exact issue a month or so ago, the issue was with cache in QGC and the params saved on the FCU. I had the issue though in Ubuntu using Gazebo. My solution for that was removing QGC and clearing the associated cache:
rm -rf ~/.cache QG*
Clear the Pixhawk by connecting to it using ./Tools/mavlink_shell.py /dev/ttyACM0.
Press enter a few times till you see the nuttx shell (nsh>) pop up and then run mtd erase.
It should look like:
nsh> mtd erase
Erasing /fs/mtd_params
Erased 16384 bytes
Erasing /fs/mtd_waypoints
Erased 16384 bytes
nsh>

@bperseghetti Thanks for the reply!
Unfortunately, this didn't seem to work for me. I couldn't find the QGroundControl cache on my windows machine (I don't know where to look or what it's called), so this is what I did:

Opened 'Add or Remove Programs' and uninstalled QGroundControl. Verified that the program was gone.
Ran the installer again, and installed QGC v4.0.10
Re-flashed my Pixhawk 4 with firmware v1.11.1 and went through the setup process again (radio, flight modes etc..). I also set the CBRK_SUPPLY_CHK to 894281.
Opened the NSH console using the mavlink console in QGC and ran 'mtd erase' and received exactly what you received in your reply.
Booted up AirSim, connected to the vehicle over serial, and it connected to QGC immediately, at which point I received a few of the 'missing battery status params' messages
Opened the mission planner screen, at which point I received a few 'maximum retries exceeded' messages
Attempted to upload a mission and received the 'GeoFence transfer failed. Retry transfer. Error: Mission write failed, vehicle failed to send final ack' and 'Rally Point transfer failed. Retry transfer. Error: Mission write failed, vehicle failed to send final ack' messages
Checked that manual mode still works, which it does
Checked the MAVLink Link Status Loss rate which is 0%

Somehow the vehicle is still refusing to upload these missions.

I think I managed to find the QGC cache located in AppData/roaming and AppData/local and it was called QGroundControl.org. I deleted this folder in both of those locations, reinstalled QGC v4.0.10 and ran the mtd erase command on the pixhawk with v1.11.1 and am still receiving all of the same error messages, although now I'm also receiving this error:

as well as the usual:

Want to move debugging this to slack?

@bperseghetti thanks, yeah I've just moved it over to the slack.

I managed to "fix" this issue by flashing ArduPilot on the Pixhawk, and then re-flashing PX4 v1.9.2. Somehow, this gave me a clean slate to work with and everything appeared to work fine. Mind you, this was on v1.9.2 and I haven't yet tested that this is okay on v1.11.1 (latest release), but I have a feeling that it will be fine on that too.

I managed to "fix" this issue by flashing ArduPilot on the Pixhawk, and then re-flashing PX4 v1.9.2. Somehow, this gave me a clean slate to work with and everything appeared to work fine. Mind you, this was on v1.9.2 and I haven't yet tested that this is okay on v1.11.1 (latest release), but I have a feeling that it will be fine on that too.

What that does is fully wipe the parameters.


Merge pull request #1 from origin

Sync the fork

Hi @Weipingh, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;


Enable sftp registry with fsspec

Update to PR

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.01% :tada:

Comparison is base (dca7d9b) 93.99% compared to head (36fa7d9) 94.00%.

:exclamation: Current head 36fa7d9 differs from pull request most recent head cb55c23. Consider uploading reports for the commit cb55c23 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
+ Coverage   93.99%   94.00%   +0.01%
==========================================
  Files          13       13
  Lines         699      701       +2
==========================================
+ Hits          657      659       +2
  Misses         42       42

Files Changed
Coverage Δ

tests/test_sshfs.py
99.02% <100.00%> (+<0.01%)
:arrow_up:

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

Hi @efiop, when the PR can be added to releases?
Thanks

@uunal Triggered a new release https://github.com/fsspec/sshfs/releases/tag/2023.10.0


Document 'un-pythonic' sdk behavior

per @zancas: in the sdk, when you use '()' in an expression, you're actually calling the BIG-IP. This behavior is 'un-pythonic' and therefore needs to be documented (user guide? developer guide?).
For example:
tm = mgmt.tm('<IP_ADDRESS>', 'admin', 'admin')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'Tm' object is not callable

^ error occurs because you're trying to call the tm module on the BIG-IP
instead, you'd use
tm = mgmt.tm

to instantiate the tm module.

Clarifications:

The behavior that deviates from default Python semantics is that objects are instantiated without the () "call" operator.

It is currently the case that use of the () "call" operator in SDK expressions is a clear indicator that a Resource object is communicating with a device.   This is one benefit of the slight deviation from default behavior.

It may be overstatement to call this behavior "un-Pythonic".
The above example code will not succeed because the root object must be constructed in the standard Pythonic way.  (This is the exception to the previously stated rule.)

Here's an example one-liner that will work:
tm = mgmt(IPADDRESS, USERNAME, PASSWORD).tm

I don't necessarily think the above one-liner is a good idea.
In fact I think the following version is likely to make for simpler code:
from f5.bigip import ManagementRoot
mgmt_rt = ManagementRoot(IPADDRESS, USERNAME, PASSWORD)
tm = mgmt_rt.tm

@jputrino What is the status of this issue?
Issue Review: I believe Zancas answered this, not sure if this has been implemented yet

@wojtek0806 -- the version in @zancas' latest comment is how the tm instantiation is shown in the docs at present (http://f5-sdk.readthedocs.io/en/v0.1.6/userguide/object_path.html#organizing-collection-section). What needs to be clarified in the docs is the behavior that deviates from default Python semantics, as noted above.


Remove italics on blockquotes

This removes italics and lightens the text in block quotes.
cc @sophshep - do you have any strong feels about this?
cc @wooorm @hzoo
Fixes #308
Before

…

After

…

@bkeepers Pretty! 👍

@sophshep - do you have any strong feels about this?

I like it! No more italic emojis _😆  😭 _


core: OCIO::SetLoggingLevel(...)

SetLoggingLevel wasnt being obeyed, if set before initialization. Now it works.

LGTM

Committed.


contract-sdk: Expose signature verification to wasm contracts

Per discussion: The batch verification stuff should just go away, because it's a giant pain.
While I'm commenting on this:
The Ed25519 signature verification in oasis-core/runtime/src/common/crypto exposes our flavor of Ed25519 (with our janky domain separation), and our idea how to handle certain edge cases.
Is it ok that we are forcing this on end user code?  I suppose that the circumstances that lead us to use something non-standard at the consensus side in the first place haven't really changed ("Ledger doesn't support Ed25519ctx/Ed25519ph"), but actual contract authors might want Ed25519pure (as opposed to Ed25519-prehashed-oasis-flavor-because-ledger).
If we do want this, I would recommend using one of ZIP-215/FIPS 186-5/Oasis flavored Ed25519pure, which would require additional code (or pulling in a crate), with a slight preference towards one of the latter two.

The Ed25519 signature verification in oasis-core/runtime/src/common/crypto exposes our flavor of Ed25519 (with our janky domain separation), and our idea how to handle certain edge cases.

Yeah we should definitely not impose our domain separation scheme and we should expose Ed25519pure here, possibly the ZIP-215 one?

Yeah we should definitely not impose our domain separation scheme and we should expose Ed25519pure here, possibly the ZIP-215 one?

I wish ZIP-215 was closer to the FIPS draft, though my preferred Ed25519 definition involves rejecting small-order-A.

Since we're going to expose Ed25519pure, the only primitive where a domain separation context makes sense is sr25519.  Should we nix the argument for now, or leave it?

The new API looks ok to me.


Question: Typescript -> (kotlin/Swift) with Integer

How do generate an integer when Typescript is the source, given that it only supports number which automatically becomes a Double in the target languages?

I have a similar problem.
I am converting typescript to swift.

i found use jsdoc
export interface Menu {
  /**
   * @type integer
   */
  seq?:         number;
  name:         string;
  isFavorite:   boolean;

  isAlcohol:    boolean;
  price:        number;
  position?:    MenuPosition;
}


Documentation fixes

A batch of documentation fixes before the Confluent Verification:

Adding support information to Confluent packaging
Minor fixes to README
Adding missing LICENSE file
Adding a QUICKSTART guide
Minor cosmetic fixes to configuration parameters

One missing thing: when we release the next version of scylla-cdc-java, pom file should be updated to use the newer version.

Hi, Can you add a QUICKSTART guide with a docker-compose version?

Hi, Can you add a QUICKSTART guide with a docker-compose version?

@avelanarius would that be a lot of work?


Add Module function to wrapper

And again: please add only related files and commits to a pull request


Everything seems to be working fine, but RTTI finds no results.

The picture below is an example of IDA Pro

As you can see in the pictures, everything is normal, but there is no result.

Is there run time type information present?

https://sourceforge.net/projects/classinformer/
I can extract the RTTI information with the plugin here.
2. When I go to the same address of one of those information in the picture, I see that it is not resolved.

https://sourceforge.net/projects/classinformer/
I can extract the RTTI information with the plugin here.
2. When I go to the same address of one of those information in the picture, I see that it is not resolved.

I see. Unfortunately for visual studio compiled programs my plugin piggybacks off of the rtti detected by the WindowsRttiAnalyzer that comes with Ghidra. If rtti is present but it isn't being found then it is most likely an issue with that analyzer and not the plugin.
I can take a quick look at the Ghidra log file to check for any indication of a problem for either analyzer if you'd like. Just be advised that the program name and information you are covering will be present in the logs. I have no problem with it if you redact that information.
The log file is located at something like C:\User\{user}\.ghidra\.ghidra_{version}\application.log for windows and `~/.ghidra/.ghidra_{version}/application.log for linux.

application.log
Thanks for the help, I hope we can fix the error.

application.log
Thanks for the help, I hope we can fix the error.

There are some really strange things going on here. For example multiple Ghidra installation files from 9.1.2 on the G drive are being used with Ghidra 10.1.3 which makes 0 sense. I doubt that is the causes of the problem here but figured I'd point it out anyway.
I do see indications that my analyzers have processed some classes but I'm not sure which session or program was the problematic one.

I was updating by copying file on top of previous ghidra version
I reset it now and it still doesn't work. But the interesting thing is that when I analyze another application it works, only this application does not work.

I think I got the point, your plugin lists the analyzed RTTI partition.
The reason for not being analyzed is that the "Windows x86 PE RTTI Analyzer" plug-in is not working properly.
This is because the app I'm trying to analyze was unpacked from a themida
It cannot analyze well because the PE section is not smooth.
I guess so. The analysis method of the ghidra and the analysis method of the class information application are different.

I noticed that it gives an error because the segment names are different.
But the main problem is that the RTTI part is in the .text segment, I'm looking for a way to split the segments.

After making the .text segment .data, the analysis was successful, but I wonder if this will do any harm to the analysis in ghidra software?

After making the .text segment .data, the analysis was successful, but I wonder if this will do any harm to the analysis in ghidra software?

You don't need to modify the PE itself. You can freely add/edit/delete memory blocks in Ghidra as you wish. Clicking this icon will bring up the menu where you can do that.

If I remember correctly the analyzer built in to Ghidra looks for .data and/or .rodata or maybe .rdata. Either way you can make the changes you patched into your PE in Ghidra itself without altering the underlying file.
Ghidra should be able to handle the hacked up PE just fine.

.text segmentini .data yaptıktan sonra analiz başarılı oldu ama acaba bunun ghidra yazılımındaki analize bir zararı olur mu?

PE'nin kendisini değiştirmeniz gerekmez. Ghidra'da istediğiniz gibi bellek bloklarını özgürce ekleyebilir/düzenleyebilir/silebilirsiniz. Bu simgeye tıklamak, bunu yapabileceğiniz menüyü getirecektir.

Doğru hatırlıyorsam, Ghidra'da yerleşik olan analizör arar .datave/ .rodataveya belki .rdata. Her iki şekilde de PE'nize yama yaptığınız değişiklikleri, temeldeki dosyayı değiştirmeden Ghidra'nın kendisinde yapabilirsiniz.
Ghidra , saldırıya uğramış PE'yi gayet iyi idare edebilmelidir.

I tried that, it didn't work, that's the thing that surprised me.
I guess the Ghidra software does not take into account the later edited segment names.

As you can see in the picture above, I fixed it later, but it still didn't work.
I have to fix the segment names before I send the file to the ghidra.


Linter prints errors twice in Sublime status bar.

As seen here:

I have the same error.
Sublime Test, Stable Channel, Build 3083
SublimeLinter v3.5.0
SublimeLinter-PyLint 1.3.1


Breaking change of setState()

component.setState() is not updated.
https://github.com/developit/preact/blob/4f671e7fb99425f84434c0c8b56d830ade4ea792/src/component.js#L55

Yup - I mentioned this to @marvinhagemeister earlier today. I think we may be working the same way as React's setState() as a result of this change though, so Unistore really needs the update here.

Here is a https://codesandbox.io/s/v81rznrm50 of the behaviour in react. They don't do an update when setState is called without any arguments.

I think it is better to documented to this change.
https://github.com/developit/preact/releases/tag/10.0.0-alpha.0

Fair point! I've just added it to the release notes :+1:


Replace bullet chart with PatternFly 4 component

This replaces the custom bullet chart with the newly created bullet chart component from PatternFly 4.
Fixes: https://github.com/project-koku/koku-ui/issues/930
Ocp on AWS details:

Ocp details:

Discovered a couple missing localizations.
src/pages/createCostModelWizard/generalInformation.tsx

src/pages/createCostModelWizard/index.tsx

Codecov Report

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

@@            Coverage Diff            @@
##             master     #950   +/-   ##
=========================================
  Coverage          ?   74.32%
=========================================
  Files             ?      187
  Lines             ?     3552
  Branches          ?      665
=========================================
  Hits              ?     2640
  Misses            ?      821
  Partials          ?       91

Continue to review full report at Codecov.

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


Documentation: Diagrams and READ.ME

[ ] Link to grant
[ ] Link to bread
[ ] Sequence diagram
[ ] Class Diagram
[ ] Textual Summary

Postponed due to possible changes in architecture incoming from #64


MTB: Add benchmark definition for "Block use of docker socket bind mount" (MTB-PL1-BC-CPI-3)

Add Benchmark definition markdown for MTB-PL1-BC-CPI-3

/assign @JimBugwadia

This benchmark is covered via MTB-PL1-BC-HI-1 Block use of bind mounts


Hypernova ignoring props sometimes

I am sorry if the description of my issue is a big vague, but I am not really sure where to start when fixing that error.
I am testing hypernova with hypernova-ruby in a Rails 4.2 environment. Server-side rendering works very well most of the time. But sometimes it seems like there are no props passed to the component for rendering (in hypernova).
I already checked the context (in getComponent function) and the props are passed to the server, anyhow the resulting string is build with the default props.
Any idea where to start on this?

What's your top-level component look like?

My top level component isn't special. I will try to implement a minimum-example which shows the problem. 😊

Okay, I figured out what my problem was: I am loading my javascript asynchronously in the head. So the javascript got executed before the dom was ready. This lead to the situation that the renderReactmethod got called when the placeholder-div was present but the script-tag containing the data was not. Also some components which appeared later in the dom were not initialized.
I am using the following code to make sure the dom is ready before initializing the components:
const renderReactComponents = () => {
  renderReact("Foo", Foo);
};

const domLoaded = () => {
  document.removeEventListener("DOMContentLoaded", domLoaded);
  window.removeEventListener("load", domLoaded);
  renderReactComponents();
};

if (typeof document !== "undefined" && document.readyState !== "loading") {
  renderReactComponents();
} else {
  document.addEventListener("DOMContentLoaded", domLoaded);
  window.addEventListener("load", domLoaded);
}


Fix build on GHC 7.6.3.

Tested with blaze-builder <0.4 and blaze-builder >= 0.4.

@hvr You're right, I'll correct that in a separate commit (unless @gregorycollins gets to it first).

Looking this over more closely: while the range is in error currently, the worst case scenario is pulling in an extra bytestring-builder dependency, which will never cause breakage (it conditionally exposes modules). Since this has already been tested by @gregorycollins, I'm going to release as-is. I'm happy to work on a more robust patch for hiding the bytestring-builder dependency once we get all our ducks in a row.

Github is not propagating my latest change to this branch, I'm going to make a new PR.


OCPBUGS-38601: The operand should not fail when the operator is uninstalled without prior deletion of the operator

When the operator is uninstalled without prior deletion of the operand, we should not let the operand fail. This is due to the removal of the service account and other RBAC objects. This commit will let the operator create the RBAC objects and additional service accounts detached from the OLM ones.
The first commit also includes an edge-case fix: when a deployment is deleted outside of the operand's lifecycle, the operator must be able to reconcile it, so we need to remove the finalizer.

Added tests to check reconciliation of RBAC objects
Some builders now return a pointer
deploymentsAreRunning|validateReconcile and deploymentsAreDeleted|validateDeletion are unified across integration and E2Es.
Minor fixes

Depends on #214

/jira refresh

/cc @Prashanth684 @AnnaZivkovic
(about https://github.com/openshift/multiarch-tuning-operator/pull/222/commits/cc8f805fe3d07b506ea9c6d236c9f76d88ca65be)
/cc @lwan-wanglin
(for testing and in particular about the changes in https://github.com/openshift/multiarch-tuning-operator/pull/222/commits/b93b63ee8d9665492f26e005a2474a71965c10ce)

/retest

/test e2e-gcp-multi-operator-olm e2e-gcp-multi-operator

/test e2e-gcp-multi-operator e2e-gcp-multi-operator-olm

/test e2e-gcp-multi-operator e2e-gcp-multi-operator-olm

/test e2e-gcp-multi-operator e2e-gcp-multi-operator-olm

https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_multiarch-tuning-operator/222/pull-ci-openshift-multiarch-tuning-operator-main-e2e-gcp-multi-operator-olm/1826084713811415040 passed

/test images e2e-gcp-multi-operator-olm e2e-gcp-multi-operator

/lgtm

/approve


Update Dockerfile

add new IMAGE

This branch has conflicts that must be resolved
I do it manually.  Sorry !


KeyError: 'location_file'

Hi,
When I run make demo-behavior, I see KeyError: 'location_file'.
In autonomy_node.py, looks like we are loading "location_file" ros parameter. However, I see that it's only being loaded in tb3_world.launch. I think that's probably reason why

Good catch! This caused an error in make demo-behavior-cpp as well for the exact same reason.
I added a wait to both nodes so you can actually start the behavior launch files before the world and the nodes will just wait for the world to start up.
Thanks!


Issue -378 Method details added

Issue -378 Method details added

PR Branch
#ADD LINK TO THE PR BRANCH
Motivation and Context
Checklist:

[x] Unit tests added?

[ ] Integration tests added?

[x] Test names are meaningful?

[x] Feature manually tested?

[ ] Branch build passed?

[ ] No 'package.*' in the imports

[ ] Relevant Wiki page updated with clear instruction for the end user

[ ] Http test added to http-testing module(if applicable) ?

[ ] Kafka test added to kafka-testing module(if applicable) ?

This PR resolves #378

Build successful @authorjapps Kindly review and close


Diacritic characters are ignored

Same as https://github.com/hrsh7th/nvim-compe/issues/541#issue-977420645

Try setting the keyword_pattern option to [[\k\+]].

See https://github.com/hrsh7th/cmp-buffer/issues/4
If you think it's unnoticavle, please create PR.

Try setting the keyword_pattern option to [[\k\+]].

It works thanks

I think this problem come back. I've installed cmp today I have the same problem. Do I  also need add something to cmp.setup.completion?

Please provide an example so that we can reproduce the bug

Sorry for delay. I had to clean my config cos I've just finished migration to lua config.
init.lua
vim.o.completeopt = 'menu,menuone,noselect'

local cmp = require 'cmp'
cmp.setup {
	snippet = {
		expand = function(args)
			vim.fn['UltiSnips#Anon'](args.body)
		end,
	},
	mapping = {
		['<CR>'] = cmp.mapping(cmp.mapping.confirm { select = true }, { 'i', 'c' }),
		['<C-Space>'] = cmp.mapping(cmp.mapping.confirm { select = true }, { 'i', 'c' }),
		['<C-j>'] = cmp.mapping(cmp.mapping.select_next_item { behavior = cmp.SelectBehavior.Insert }, { 'i', 'c' }),
		['<C-k>'] = cmp.mapping(cmp.mapping.select_prev_item { behavior = cmp.SelectBehavior.Insert }, { 'i', 'c' }),
		['<C-w>'] = cmp.mapping { i = cmp.mapping.abort(), c = cmp.mapping.close() },
		['<C-S-Space>'] = cmp.mapping(cmp.mapping.complete(), { 'i', 'c' }),
		['<C-y>'] = cmp.config.disable,
	},
	sources = {
		{ name = 'buffer', keyword_length = 2, keyword_pattern = [[\k\+]] },
		{ name = 'nvim_lua' },
		{ name = 'ultisnips' },
		{ name = 'path' },
		{ name = 'calc' },
	},
}

-- CMD mode - if you are in that mode and put / or : ' (if you enabled `native_menu`, this won't work anymore).
cmp.setup.cmdline('/', {
	sources = {
		{ name = 'buffer' },
	},
})
cmp.setup.cmdline(':', {
	sources = {
		{ name = 'cmdline_history', keyword_length = 3 },
		{ name = 'path' },
	},
})

plugins.lua
use {
	'hrsh7th/nvim-cmp',
	requires = {
		{ 'hrsh7th/nvim-cmp' },
		{ 'hrsh7th/cmp-buffer' },
		{ 'hrsh7th/cmp-path' },
		{ 'hrsh7th/cmp-calc' },
		{ 'dmitmel/cmp-cmdline-history' },
		{ 'quangnguyen30192/cmp-nvim-ultisnips' },
		{ 'hrsh7th/cmp-nvim-lua' }, -- vim.api
		{ 'hrsh7th/cmp-nvim-lsp' },
	},
}

I don't know how to proper debug nvim. If you like you can check out my dotfiles on github

@amiroslaw Well, the config looks fine. Can you also give a file with which the problem can be reproduced?

I've also checked out other languages and I have the same problem with different diacritics. The problem appears in any filetype.
Polish
księżyc głównie przyszłość trzydzieści rozdział szczególnie odpowiedź ostrożnie większość dwadzieścia dziesięć światło szczęście rzeczywiście przecież
Greek
monotonikó sýstīma grafīs
Portugues
Sinônimos (inglês) linguística diacrítico em inglês

That's really weird. For me it works fine (I think):

Yes I want to achieve this result. Unfortunately those patterns didn't work either.
I think that default keyword_pattern doesn't replace by user's one. When I replaced it in the source.lua the problem disappeared. I have the newest version.
local defaults = {
  keyword_length = 3,
	keyword_pattern = [[\k\+]],
  -- keyword_pattern = [[\%(-\?\d\+\%(\.\d\+\)\?\|\h\w*\%([\-]\w*\)*\)]],
  get_bufnrs = function()
    return { vim.api.nvim_get_current_buf() }
  end,
}

Oh I forgot, you have to specify the keyword_pattern like this:
cmp.setup({
  sources = {
    {
      name = 'buffer',
      option = { keyword_pattern = [[\k\+]] },
    },
  },
})

instead of
cmp.setup({
  sources = {
    {
      name = 'buffer',
      keyword_pattern = [[\k\+]],
    },
  },
})


Cyl dev: set workflow

Description
Please provide a brief introduction to the purpose and features of your Pull Request.
Type of change

[ ] Bug fix
[x] Feature enhancement/new feature
[ ] Documentation change
[ ] Style/UI change
[ ] Performance optimization
[ ] Refactoring/code optimization
[ ] Other changes

Related Issues
Please list the Issue numbers related to your Pull Request (if any).
Specific changes
configured jest as the test method and created a demo test file under tests folder
Testing
Please describe your testing methods and results to ensure that your changes do not introduce new issues.
Screenshots
If your Pull Request includes UI changes or new features, please provide relevant screenshots.
Other Information
If there are other important matters to note, please add them here.

人麻了，以后会注意在每个js开头加上import React from 'react';


Add runId as an output

This PR adds the triggered dbt Cloud job's runId as an output of the action. I've tested locally that this works, but let me know if I can try anything else!

And released in v4.1.
Thanks for contributing!


Write the dependency graph format for all package managers

This PR is part of #4069. It adds a generic converter for analyzer results that generates dependency graphs if necessary. It also adds a custom serializer that triggers this conversion automatically before writing the analyzer result.
Adding the custom serializer was a bit problematic because the conversion logic is part of the analyzer module; so the serializer cannot be placed in model where the standard object mappers are created. I therefore introduced a mechanism via Java's ServiceLoader that allows customization of these mappers from other modules. If you have better suggestions, please let me know.
Note that there are still some quirks until #4069 is completely solved. Currently, there are format conversions when loading and saving ORT results in single steps. (These conversions may change the exact dependency graph representation, if projects are processed in a different order.) This will be solved when the dependency graph becomes the "native" format.

Rebased again on master, as some conflicts were reported. @mnonnenmacher, @sschuberth, is there anything left here you want me to change?


test/peek_iter: add more unit tests for peek_iter

And fix remaining Clippy warnings.

This change is

r? @dirvine
(maidsafe_highfive has picked a reviewer for you, use r? to override)

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


python311Packages.pywaze: 0.4.0 -> 0.5.0

Diff: https://github.com/eifinger/pywaze/compare/refs/tags/v0.4.0...v0.5.0
Changelog: https://github.com/eifinger/pywaze/releases/tag/v0.5.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 255529 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.waze_travel_time
python310Packages.pywaze
python310Packages.pywaze.dist
python311Packages.pywaze
python311Packages.pywaze.dist


Nested query question

Hello. How can I perform query line this:
query := elastic.NewTermsQuery("Tags", tags...).MinimumShouldMatch(MINIMUM_TAG_MATCH)
query2 :=  elastic.NewTermQuery("Active", true)

I'm not sure I understand. You pass the query to the Search service via the Query func. If you want to combine queries, you can do so by e.g. using a BoolQuery.

Or do you want a Nested Query?

Yes! I mean - how to combine queries?

Like this example code:
query1 := elastic.NewTermsQuery("Tags", tags...).MinimumShouldMatch(MINIMUM_TAG_MATCH)
query2 :=  elastic.NewTermQuery("Active", true)
query := elactic.NewNestedQuery().Add(q1).Add(q)

The documentation is very good. Read the section about queries, e.g. BoolQuery. Then look up the tests, e.g. search_queries_bool_test.go if you're unsure about the exact syntax.

Thanks for link. https://github.com/olivere/elastic/blob/release-branch.v2/search_queries_bool_test.go

Tests are your friend. :-)


remove landing reference from history router

@cagataycivici latest commit broke build, minor fix.

It was fixed. Closing due to conflicts.
Thanks for your contribution!


1.20 invframework out.

in my earlier review you said you rely on a different plugin (https://www.spigotmc.org/resources/if-inventory-framework.57216/)
that version is out now so when will there be a new version for 1.20?

I see that IF has updated to 1.20.1. I started working on it last night, I will hopefully have it updated tonight.

This should be all good now, https://github.com/Code12x/WorldTP-For-Multiverse/releases/tag/v1.4.3 .
Thanks for letting me know it came out!

Its 1.20.1, that also supports 1.20 right?

It should, but I wasn't able to test 1.20 because when I tried to run java -jar BuildTools.jar --rev 1.20 it builds 1.20.1


Feature/holixon registry

Just a first MVP, not a single test so far... Let's exchange on that.

@jangalinski please get an impression and let's have a discussion if this goes in the right direction.... Then I'll start supplying unit tests and fixing issues...


DB Client: Fix get prototype in DbExecute interface

Environment Details

Helidon Version: 2.0-SNAPSHOT
Helidon SE

Problem Description
API inconsistency in io.helidon.dbclient.DbExecute interface
    default OptionalCompletionStage<DbRow> namedGet(String statementName, Object... parameters)

vs.
default CompletionStage<Optional<DbRow>> get(String statement, Object... parameters)

Both methods shall return OptionalCompletionStage<DbRow>

Looks like this issue is already done in this PR.
Should it be closed?


TypeError: Callback is not a function

Hi. Sorry for all the issues. So I started simplisafe and then I got this error
TypeError: callback is not a function
at SS3SimpliCam.identify (/usr/local/lib/node_modules/homebridge-simplisafe3/accessories/simplicam.js:98:5)
at PlatformAccessory. (/usr/local/lib/node_modules/homebridge-simplisafe3/accessories/simplicam.js:103:52)
at PlatformAccessory.emit (events.js:315:20)
at Accessory. (/usr/local/lib/node_modules/homebridge/src/platformAccessory.ts:85:12)
at Accessory.emit (events.js:315:20)
at Accessory.EventEmitter.emit (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/EventEmitter.ts:42:22)
at Accessory._this._identificationRequest (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Accessory.ts:256:12)
at Characteristic. (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Accessory.ts:1599:16)
at Characteristic.emit (events.js:315:20)
at Characteristic.EventEmitter.emit (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/EventEmitter.ts:42:22)
Do I need to turn off my firewall or rebuild node? Please help. Thanks!!

What version of homebridge are you using? If it was working before I would guess you changed something...

You did not use the issue template, please update this issue with the correct information.

The error makes me think you changed your version of homebridge or something, especially since it was working fine for you before.
Closing because you did not use the issue template, please update this issue with the correct information if needed.

@Ironboltx my sincerest apologies, you are correct there is a bug, it only happens when you are setting up the system (a big deal, obv) or if you hit "ID" in the Eve app, for example. I will release a fix soon. Thanks for your understanding.


[Question]: Are there docs available?

Hello,
I'm interested in potentially using this package to do SVG sanitizing in OctoberCMS, however I don't see any docs for how to go about doing that. I read https://blobfolio.com/2017/03/when-a-stranger-calls-sanitizing-svgs/ and it seems like this library would fit our needs perfectly but I need documentation in order to figure out how to use the library.

Hi Luke,
For usage details, you could take a look at the Wiki, starting here.
But if you're just looking to get some degree of SVG sanitizing into the OctoberCMS core, you probably don't want to add the entire blob-common library to your dependency chain — 99% of the code has nothing to do with SVGs, and some of the unrelated datasets are fairly large. Haha.
Instead, I'd recommend jumping into the source to get a sense of the tasks it is performing so that you can rewrite/port the few relevant methods directly to your project. Also, there are some nice example SVGs for unit testing, etc., here.
If you run into any issues or come up with any specific questions, please don't hesitate to reach out. I'm happy to help!


reflect rename of repo from python-bioimage-io to core-bioimage-io-python

as discussed in #101, the package was renamed to core-bioimage-io-python.
I did a quick replace to reflect this in the files of this repo.
The pip package name is unchanged (bioimage-core), I think that's fine.

The pip package name is unchanged (bioimage-core), I think that's fine.

Yes, that name should stay the same!

I will just go ahead and merge this :).


fix(props.text) pass proper callback to setState

Fix
in UNSAFE_componentWillReceiveProps, setState expects a callback as its second argument -- not a void | undefined. This small change fixes it.
Signed-off-by: Sam Apter<EMAIL_ADDRESS>

this._handleChangeText() is function...
What is this a fix for? What happened vs the expected behavior?
Unless I am fundamentally misunderstanding how all this works (I learned JS when promises became a thing).
I played around with the setState call back, here is a link to the stackblitz I was using.

@bell-steven
Yes, _handleChangeText is a function, but the way to pass that function to setState is just to give its name -- without invoking it (adding the () in the end). When you're passing an invoked function to setState you're actually passing the return value of that function rather than the function itself. So if _handleChangeText didn't take a parameter the fix would be just that: the second argument to setState should be changed from handleChangeText() to handleChangeText. But because it does take a parameter, that won't do it. So you need to change it instead to () => this._handleChangeText(nextProps.text). This way, you're still passing a function rather than its return value to setState and you're also specifying what argument to feed the function's parameter.
As to your stackpblitz, go ahead and try it again. The value you see logged by the console.log is off by one digit from the value in the counter on the screen. That's because the function is invoked right away rather than passing just the name of the function to be invoked by setState when it's done updating the state. Change this.checkAge() to this.checkAge or to () => this.checkAge() and look at the log and you'll see that now it matches what's in the counter on the page.

@sapter thanks for that.  Thats what I was thinking in the back of mind. I really appreciate the detailed explanation!
p.s. while I have you here, would you be able to shed some light on #558?

@bell-steven  I'll have a look at it in the morning. I already closed my computer for the night.

Thanks!


Make sure the webhook is not set when testing live getUpdates method!

Current coverage is 88.81% (diff: 100%)

No coverage report found for master at bbf8142.

Powered by Codecov. Last update bbf8142...2b381d1


tedge cert upload c8y: ambiguous file/folder not found error - missing path in error message

Proposed changes
While uploading the certificate, if the root certificate path does not exist, the error message must be more verbose, providing the info about which path or directory does not exist. The error must be as below.
Error: failed to upload root certificate

Caused by:
    Root certificate path /etc/ssl/certssaa does not exist

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue
https://github.com/thin-edge/thin-edge.io/issues/2419
Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

The system test should be moved as it is checking tedge cert upload and not tedge connect (as the test file is indicating).
Maybe moving the test case out into the following file:
tests/RobotFramework/tests/tedge/tedge_cert_upload.robot

Moved the test to tedge_cert_upload.robot as mentioned above.


For VB Contract.ThrowIfNull does not supress CA1062

Adding a Code.Contract Precondition as the error/warning instructions say does not suppress CA1062
 Public Function ConvertInputRequest(RequestToConvert As ConvertRequest) as Boolean
    Contract.ThrowIfNull(RequestToConvert,$"{NameOf(RequestToConvert)} cannot be Nothing")
    Dim languages As String() = {RequestToConvert.GetSourceExtension, RequestToConvert.GetTargetExtension}

The following does work to suppress the warning.
If RequestToConvert Is Nothing Then
    Throw New ArgumentException
End If

Also the message says Null Vs. Nothing where other messages gives separate instructions for VB

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

Content: CA1062
Product: visual-studio-windows
Technology: vs-ide-code-analysis
GitHub Login: @gewarren
Microsoft Alias: gewarren

@paul1956 Thank you for the feedback.
Can you verify if either of the following resolves the issue?

https://stackoverflow.com/questions/15188631/code-analysis-rule-ca1062-behaviour (also points to https://github.com/dotnet/roslyn-analyzers/issues/2369)

https://stackoverflow.com/questions/53448427/ca1062-not-evaluating-on-net-standard-2-0-class-library

@msebolt, not really. They are both C# issues,

C# and VB don't handle Null/Nothing Strings the same. In VB a Null/Nothing String = "" so it is valid to pass it to a function expecting a String.
I am using the recommended way to check for Null a Contract that shouldn't be necessary for a String.

There are 2 related issues, treatment of Null Strings and using Contract to address the issue.
 Public Function Y(OriginalText As String)
    Dim Text As String = OriginalText.Trim(" "c) ' There should be no warning here because if OriginalText is Nothing/Null or "" Text will be ""

 Public Function Y(OriginalText As String)
    Contract.ThrowIfNull(OriginalText ,"Error Message")
    Dim Text As String = OriginalText.Trim(" "c)

@paul1956 This sounds mostly like a problem with the analyzer itself rather than with the documentation, although we could add note to the "When to suppress warnings" section if the analyzer does not get changed. Can you please log an issue at https://github.com/dotnet/roslyn-analyzers/issues?

Both, I already filed #2815 against the analyzer as you asked, but you should always suppress this for VB and Strings (until fixed when it will not apear for strings). Your VB example using String and the test is always unnecessary. You do want an example but it should not use String.

Thanks, I agree the example for VB is bad. We'll also update the When to suppress section.

The roslyn-analyzers issue was closed as not reproducible, but the docs should still be updated.

Updated content link

Moved issue to .NET
Closing this in favor of https://github.com/dotnet/docs/issues/22041


Message: Cannot read property 'props' of undefined

I just upgraded from 2.0.0-alpha.0 to 2.0.0-beta.2 and that triggers the following error:
TypeError: Cannot read property 'props' of undefined
    at extractPropsFromVNodeData (vue.runtime.esm.js?2b0e:2270)
    at createComponent (vue.runtime.esm.js?2b0e:3227)
    at _createElement (vue.runtime.esm.js?2b0e:3434)
    at createElement (vue.runtime.esm.js?2b0e:3353)
    at vm.$createElement (vue.runtime.esm.js?2b0e:3494)
    at Proxy.render (LazyHydrate.esm.js?8416:144)
    at VueComponent.Vue._render (vue.runtime.esm.js?2b0e:3548)
    at VueComponent.updateComponent (vue.runtime.esm.js?2b0e:4055)
    at Watcher.get (vue.runtime.esm.js?2b0e:4479)
    at Watcher.run (vue.runtime.esm.js?2b0e:4554)

yeap. me to catch this error.

Hmm - for some reason 2.0.0-alpha.0 also triggers this error.

Look, people, I understand that it can be frustrating to use an Open Source project which has a bug that prevents you from doing the things you want to do. I've been there. But I do this stuff in my free time. The last two weeks I've worked probably 40+ hours on this in my free time. For almost 0 €.
Please read: https://github.com/maoberlehner/vue-lazy-hydration/blob/master/CONTRIBUTING.md

Found a problem? Want a new feature?

Provide a reduced test case or a live example.
Remember, a bug is a demonstrable problem caused by our code.

Could reproduce the problem myself.  Will be fixed with next release.


Change the minimum VS Code version that the Azure Databases extension supports

OS: Win10
Build Version: 20220830.1
Repro Steps:

Figure out the oldest vs code version that the extension supports, the version is 1.57.0 as below.

Install VS Code 1.57.1 from https://code.visualstudio.com/updates.
Install the Azure Databases extension from VSIX.
Expand the subscription node.
Check whether no error occurs.

Expect:
No error occurs.
Actual:
An error occurs.

I have bumped the VSCode engine version to 1.82.0.


retry connection in case of ECONNREFUSED

the client should tolerate instant unavailability with some backoff.

@chronark is this fixed?


issue #49 resolved session section changed

@championswimmer , I have to create this again , (deleted the branch mistake) , is it required ?


Update logo in header

Before you submit a pull request for review, please make sure...

[ ] Your code contains tests relevant for code you modified
[x] All new and existing tests are passing
[x] You have linted the project bundle exec standardrb --fix

If this PR is not ready for review, please make sure to submit it as a draft.

New logo WIP!

Hey @joemasilotti  which tool are you using to make the logo?

Sketch.app
https://www.sketch.com/

Thank you


IntelliJ + modular + Gradle does not load resources when running from IntelliJ

Hello,
The example for Intellij + modular + Gradle does not start when invoked directly from IntelliJ. It does not find the fxml file here:
    Parent root = FXMLLoader.load(getClass().getResource("scene.fxml"));

and throws a NPE.
Is there a way to fix this with the Gradle setup? Starting the application with gradlew run works, but then I don't have a debugger support and it is not straightforward.
I think the problem is that the resource files are build to a different output directory (build/classes and build/resources), but IntelliJ does not merge the directories.
I've imported the project into IntelliJ on Windows 7, set JDK 12 as the Gradle and the Project SDK, but did not do any other changes.

This is an issue with IntelliJ "default" configuration and how your project is configured as a "gradle" project.
Even if you don't set any VM arguments, IntelliJ will apply them for you. If you have a modular app, it puts everything in the module-path, and that's ok, but misses to add a required --patch-module: your resources and your Java classes are under the same package name but in different folders. That will leads to a "split-package" problem.
Therefore, the JavaFX gradle plugin adds this patch, and solves the issue, but this works only if you run a Gradle task (./gradlew run).
If you run from IntelliJ you will either have to:

run from the Gradle window, tasks -> application -> run.
or, merge source code and resources
or edit the run configuration, and add something like --patch-module hellofx=/path/to/resources

@jperedadnr Could you provide some more detailed instructions on how to get the same applications to work in IntelliJ? You say "edit the run configuration" - which part of the configuration exactly? Also, what should I replace "hellofx" in your example with? What does this represent exactly? Also, what should I place in "/path/to/resources" as this is obviously not a real path. Again, what does this represent exactly? Finally, could you please point to any kind of documentation either specific to JavaFX or to Java in general where this might already be explained in more detail.


Tools: get-help Add-Migration displays invalid related command

get-help Add-Migration
displays:
RELATED LINKS
    Remove-Migration
    Update-Database
    about_EntityFrameworkCore

about_EntityFrameworkCore doesn't seem to be there

That's the overview help topic that shows the unicorn.
Get-Help about_EntityFrameworkCore

Also, -Environment <String> is not a valid option anymore

@BenjiZombie Correct. You need to use the ASPNETCORE_ENVIRONMENT environment variable now.


brick, incorrect ranging

{clock,valid,ready,up:10} will disappear when close to bottom of the screen

Bad:

Good:

fixed by https://github.com/wavedrom/doppler/commit/707a3b6dab04368f15cc160ab79474ad89b26649


Made by Ra's al Ghul League

Token Address
e.g: https://scan.tomochain.com/address/0xd667b976e83de476a8d16116fb0c9e7eba8dd772
Token Symbol
e.g: TOMO, UFO
Token Name
e.g: TomoChain, UFO INU
Token Logo
http://ufoinu.com/coin_logo/256X256.png
Token Description
We will use a kind of artificial intelligence technology beyond the comprehensiveness of today humanity.
Website
http://ufoinu.com/
Social links
e.g Github, https://t.me/ufoinucoin, https://twitter.com/InuUfo

Made by Ra's al Ghul League

updated


update to latest dev branch and use package manager

Bump zig support to latest developement release
(0.12.0-dev.3596+fbb38a768). This includes switching to the included packagemanager.

This needs to be updated after https://github.com/mattnite/zig-mbedtls/pull/8 is merged.


The clear button of LineEdit can not delete the text on iOS devices.

Godot version:
3.2.x

OS/device including version:

Issue description:

after clear the text of LineEdit by click the clear button, the text disappeared, however, when continue to input characters, the original text still displayed and the new input characters is append to it.
Steps to reproduce:

add a LineEdit
enable clear button in the property of LineEdit
input some text
click clear button to clear the text
continue to input
the cleared text plus the new text displayed in the LineEdit.

The above behavior only happened on iOS devices.
And the following behavior happened on iOS and Android devices.
After clicking the clear button, the suggestion words on the soft keyboard  is not cleared, which will be cleared on the UITextField.
Minimal reproduction project:

Godot version:
3.2.x

Please be more specific, there are bug fixes in every single Godot version.

Godot version:
3.2.x

Please be more specific, there are bug fixes in every single Godot version.

all including 3.2.4 beta 6

all including 3.2.4 beta 6

Maybe someone from Android team can help? I don't entirely understand how android is handling keyboard input and LineEdit doesn't seem to be sending anything to platform code so we can actually catch it and clear the backing text view.

Maybe someone from Android team can help? I don't entirely understand how android is handling keyboard input and LineEdit doesn't seem to be sending anything to platform code so we can actually catch it and clear the backing text view.

@alexzheng can you try out this branch and check if it works for you? https://github.com/naithar/godot/commit/e3677a312bcb3bb47fdba6164e69c97bc8557754

@alexzheng can you try out this branch and check if it works for you? https://github.com/naithar/godot/commit/e3677a312bcb3bb47fdba6164e69c97bc8557754

@naithar  it works, thanks.

@naithar  it works, thanks.


Fix: Click on switch, click any menu, click in anywhere in the window, switch responds

Fixes issue where once a switch is mark active (by clicking on it), clicking anywhere else will trigger it on -mouseUp:.
Reproducible with the demo project included in repo.

Sure. Pushed a new commit.
PS: There's a mix of spaces and tabs else in the file though.