Install subversion17 fails on El Capitan

Hey there!
I realized that currently it is not possible to install subversion17 with El Capitan.
Right here https://github.com/Homebrew/legacy-homebrew/issues/49263 it was already discussed, but it seems not to give any indication on how to actually solve the problem.
I am working on a subversion respository which is rooted in svn 1.7. and it is currently also now an option to just update the repository to the newer version, because then all my other fellow team members would have to upgrade their svn version.
Is there currently a way to install subversion17 on my Mac with brew or any other tools?
All the best,
David

Is there currently a way to install subversion17 on my Mac with brew or any other tools?

It seems not currently. We'll accept a pull-request to fix it but we don't actively work on fixing old versions on newer macOS versions ourselves, sorry!

Basically:

Remove SSL support.
Vendor in OpenSSL 0.9.8, which is completely unsupported upstream and hasn't been receiving security fixes or even security analysis since the end of last year.

No other combination seems to work. Really I'd go so far as to say we're already letting users use insecure software with subversion17 that links to the outdated OpenSSL on Yosemite and below, but there we go.


Space around selector separator

I had a request for the VS Code beautify extension to support spaces around selector separators (>+~) in CSS (HookyQR/VSCodeBeautify#9). Although the reference was incorrect, this PR will enable css-beautify to support the requested option.
Scratched the previous branch and made the changes again.

Hello, I'm sorry for the delay.  I've had no time to review changes or to fix the issue with the CI tools. Please be patient.

Nope, this looks good.  Sanity check tests and everything.  Thanks!


fix(samples): do not pass location

This one is weird: the samples tests stopped working for no reason saying they don't like the parent field:
Error: 3 INVALID_ARGUMENT: Invalid request fields: parent.

I tried it and figured it works if we don't pass any location in the parent field. The comments in protos say this:
  // Format: `projects/{project-id}/locations/{location-id}`.
  //
  // If no location is specified, a region will be chosen automatically.

So... it all works without locations. Not sure if it's the right usage, but let's maybe merge this now to have the tests passing, and then figure out what's wrong with the API.

Thank you for catching this and making the change.
This is due to a late breaking change from in the service. The idiomatically correct fix is applied here:
https://github.com/googleapis/nodejs-document-ai/pull/35


as:command from:gh-r, download file is the directory.

Problem

I want to download fromgh-r.
But there are multi files to select, so I use use:"*.tar.gz"
I want to use a file as command which in the directory.
But it not work!

Example
youtube-dl
Here is my configuration.
zplug "rg3/youtube-dl", as:command, use:"*.tar.gz", from:gh-r

Ok, I have solved it, just use it as plugin.


Couldn't find tmdb id for media

Describe the bug
TMDb cannot fetch id for Die Hart. Die Hart is not the only movie this happens. Sometimes it works, sometime it doesn't. Is there maybe TMDb Rate Limiting at play?
On another Note which I think might be related. The Id couldn't be populated and got resolved to 'undefined'. I couldn't reproduce this tho:
2024-12-26T19:56:10.327687543Z [maintainerr] | 26/12/2024 20:56:10  [DEBUG] [TmdbApiService] GET request to https://api.themoviedb.org/3/movie/undefined?api_key=[REDACTED]&language=en&append_to_response=credits,external_ids,videos,release_dates,watch%2Fproviders failed: AxiosError: Request failed with status code 404

To Reproduce
Steps to reproduce the behavior:

Add Die Hart to Radarr
Go into Collection and Test Media on Die Hart.

Expected behavior
Request could be made an proceed with other rules.
Screenshots
2024-12-26T20:13:47.474176209Z [maintainerr] | 26/12/2024 21:13:47  [INFO] [RulesService] Successfully updated rulegroup '[Movies] Requested & Ready'.
2024-12-26T20:14:45.572031605Z [maintainerr] | 26/12/2024 21:14:45  [DEBUG] [OverseerrGetterService] Couldn't find tmdb id for media 'Die Hart' with id '79387'. As a result, no Overseerr query could be made.
2024-12-26T20:14:45.572067652Z [maintainerr] | 26/12/2024 21:14:45  [DEBUG] [OverseerrGetterService] Couldn't fetch Overseerr metadate for media 'Die Hart' with id '79387'. As a result, no Overseerr query could be made.

Additional context
Exported rules:
mediaType: MOVIES
rules:
  - "0":
      - firstValue: Tautulli.lastViewedAt
        action: BEFORE
        customValue:
          type: custom_days
          value: "555"
  - "1":
      - operator: OR
        firstValue: Tautulli.addDate
        action: BEFORE
        customValue:
          type: custom_days
          value: "180"
      - operator: AND
        firstValue: Tautulli.viewCount
        action: EQUALS
        customValue:
          type: number
          value: 0
  - "2":
      - operator: AND
        firstValue: Overseerr.isRequested
        action: EQUALS
        customValue:
          type: boolean
          value: "true"

https://docs.maintainerr.info/Common/#tvdbid-or-tmdbid-errors
Maintainerr pulls the TVDB/TMDb ID from the items Plex metadata (which isn't always great). Most likely Plex has added the wrong TMDB id somehow. You could verify this by looking in the items XML, on Plex, and looking for this section:
<Guid id="imdb://tt0095016"/>
<Guid id="tmdb://562"/>
<Guid id="tvdb://324"/>

1077280 is the correct TMDB id that should be showing for the movie Die Hart. TMDB Link
We use this ID to tell Radarr/Sonarr/Overseerr/etc. exactly which movie or show we are talking about. If it is incorrect, then the request to those services fails as it does not exist. Technically it does, but with a different TMDB id than what Maintainerr got from Plex.

2024-12-26T20:14:45.572031605Z [maintainerr] | 26/12/2024 21:14:45  [DEBUG] [OverseerrGetterService] Couldn't find tmdb id for media 'Die Hart' with id '79387'. As a result, no Overseerr query could be made.

TL;DR The best course of action is to try and use Fix Match on the item in Plex and see if it fixed the incorrect ID that was attached to the item.

Thanks to your comment I looked more into my metadata and as it turns out, I haven't refreshed my metadata for quite some time. Some movies were still in the old plex agent format, which doesn't work as expected with Maintainerr. After I refreshed my library everything seems to work, thanks!


Add optional support for EC implicitCurve parameters

Add X509_V_FLAG_INHERIT_PKPARAM and -inherit_pkparam verification apps option
implemented by new X509_PUBKEY_chain_inherit_parameter() etc.
Add -null_pkparam option to apps/x509 for producing certs with implicitCurve
Various corrections for misleading cert handling/verification errors
Add TODO for removing legacy X509_get_pubkey_parameters()
Add 10 test cases to 25-test_verify.t demonstrating these improvements

This fixes #12895.
Checklist

[x] documentation is added or updated
[x] tests are added or updated

I'm not sure we want to support this, even when you need to opt-in for the support.

It also improves the so far not very to-the point errors on trying to use such certs.
For instance,
openssl x509 -noout -text -in test/certs/p384-inter-implicitCurve.pem

used to print
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number: 2 (0x2)
        Signature Algorithm: ecdsa-with-SHA256
        Issuer: CN = ECDSA P-384 root
        Validity
            Not Before: Sep 26 17:43:38 2020 GMT
            Not After : Sep 27 17:43:38 2120 GMT
        Subject: CN = ECDSA P-384 inter
        Subject Public Key Info:
            Public Key Algorithm: id-ecPublicKey
            Unable to load Public Key
139970204808320:error:100DC08E:elliptic curve routines:eckey_type2param:decode error:../crypto/ec/ec_ameth.c:124:
139970204808320:error:100D7010:elliptic curve routines:eckey_pub_decode:EC lib:../crypto/ec/ec_ameth.c:151:
139970204808320:error:0B09407D:x509 certificate routines:x509_pubkey_decode:public key decode error:../crypto/x509/x_pubkey.c:125:
        X509v3 extensions:
            X509v3 Subject Key Identifier:
                B7:FC:08:8B:B1:7F:67:76:7C:52:3B:AD:B2:85:55:E0:90:26:00:CA
            X509v3 Authority Key Identifier:
                keyid:26:D0:8F:1D:7B:5F:DC:1E:7F:41:80:7D:63:CA:1C:75:81:D6:85:69

            X509v3 Basic Constraints: critical
                CA:TRUE
    Signature Algorithm: ecdsa-with-SHA256
         30:65:02:31:00:f7:4a:51:64:d1:fb:25:f4:0a:d8:05:55:2e:
         a9:c4:ce:d2:ef:b2:01:b5:c1:9f:67:59:23:46:a8:df:de:aa:
         b3:34:d8:fe:9a:15:8c:fc:8a:74:80:c2:d9:f7:66:37:a3:02:
         30:1f:7f:0c:bf:4b:f3:34:b5:fc:74:61:28:ac:87:1b:e3:ba:
         54:c6:91:06:24:c3:80:5d:21:c3:63:8a:0a:ec:f1:6d:d2:bc:
         f3:2e:76:c8:5d:79:87:4e:b2:8b:48:11:b4

and with this PR it prints
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number: 2 (0x2)
        Signature Algorithm: ecdsa-with-SHA256
        Issuer: CN = ECDSA P-384 root
        Validity
            Not Before: Sep 26 17:43:38 2020 GMT
            Not After : Sep 27 17:43:38 2120 GMT
        Subject: CN = ECDSA P-384 inter
        Subject Public Key Info:
            Public Key Algorithm: id-ecPublicKey
            Unable to load Public Key
C0578F63187F0000:error::elliptic curve routines:(unknown function):implicit curve:crypto/ec/ec_ameth.c:142:
        X509v3 extensions:
            X509v3 Subject Key Identifier:
                B7:FC:08:8B:B1:7F:67:76:7C:52:3B:AD:B2:85:55:E0:90:26:00:CA
            X509v3 Authority Key Identifier:
                26:D0:8F:1D:7B:5F:DC:1E:7F:41:80:7D:63:CA:1C:75:81:D6:85:69
            X509v3 Basic Constraints: critical
                CA:TRUE
    Signature Algorithm: ecdsa-with-SHA256
    Signature Value:
        30:65:02:31:00:f7:4a:51:64:d1:fb:25:f4:0a:d8:05:55:2e:
        a9:c4:ce:d2:ef:b2:01:b5:c1:9f:67:59:23:46:a8:df:de:aa:
        b3:34:d8:fe:9a:15:8c:fc:8a:74:80:c2:d9:f7:66:37:a3:02:
        30:1f:7f:0c:bf:4b:f3:34:b5:fc:74:61:28:ac:87:1b:e3:ba:
        54:c6:91:06:24:c3:80:5d:21:c3:63:8a:0a:ec:f1:6d:d2:bc:
        f3:2e:76:c8:5d:79:87:4e:b2:8b:48:11:b4

I've just removed in x509_vfy.c the two calls of the legacy function  X509_get_pubkey_parameters().
It would just add spurious error queue entries in some situation.
I suggest deprecating or entirely removing that function when we have more freedom to do API changes.

It also improves the so far not very to-the point errors on trying to use such certs.

Could we have just this instead of the "complete" support?

The currently reported Travis CI failure is unrelated and will be fixed by #13022.

Improving the error is reasonable enough, but I agree OpenSSL should not actually implement it. It's explicitly forbidden by RFC5480 and I'm not aware of any other implementation which allows this. It also risks introducing some weird problems coming from a certificate's public key being interpreted differently in context.
Cryptography is sadly mired with a history of specification mistakes and unnecessary complexity. This is a mistake that we have happily moved on from. Supporting this feature in OpenSSL would be a regression for both the project and the ecosystem as a whole.

Thanks @t8m and @davidben for your comments.
I am aware that since 2009, RFC 5480 forbids the use of implicitCurve (and specifiedCurve) parameters in PKIX
while by that time the key idea for implicitCurve was the following (quoting from RFC 3279):

If the elliptic curve parameters are specified as implicitlyCA in the
SubjectPublicKeyInfo AlgorithmIdentifier and the CA signed the
subject certificate using ECDSA, then the certificate issuer's ECDSA
parameters apply to the subject's ECDSA key.

I do not know what the specific actual concerns were when forbidding for EC certs everything else but namedCurve,
but there are certainly cases for which this is overzealous. And I do not really see what should go wrong
if in an - up to that point already verified - chain some security parameters from the issuing CA get inherited to the subject cert.
Actually right from its beginning in 1998, OpenSSL has been supporting implicitCurve,
with the help of the function  X509_get_pubkey_parameters(), which BTW has a rather unclear name.
This function apparently got neglected over the years and either should have been updated,
clarified, and documented or it should have been deprecated and removed. Neither happend.
Currently it simply does not work and in some situations even adds needless error queue entries.
So one of the commits in this PR removes its calls from verify_chain(), and a TODO for deprecation is added.
Following the current model of handling specifiedCurve, for which here is still quite some support in OpenSSL
and where just recently (in #/12683) a check has been added to throw an error if used in proper certificate chains
to be verified in verify_chain(), I updated/corrected and completed here the treatment of its sibling, implicitCurve.
Maybe some more discussion would be needed to find out whether there are still cases where it can be helpful
if OpenSSL can continue supporting such deprecated mechanisms where explicitly requested.
For the time being, as soon as I'll have some spare time for this non-urgent task,
I'm going to split this PR into

those changes that are indubitably helpful and even important (such as correcting misleading error output
and by default refusing to verify certificates that use the implicitCurve option with a proper error) and
those that - upon being explicitly enabled - support the use of the deprecated implicitCurve mechanism.

If we ever supported it, it's clearly been broken for a long time
without anybody complaining about it. Don't fix it, just remove
it. Don't bother with deprecating it first.

Closing as we are not going to add implictCurve support.


Fetch sequences from NCBI tool

The tool available here https://github.com/ARTbio/tools-artbio/tree/master/tools/fetch_fasta_from_ncbi is missing something:
I realized that the "fetch_fasta_from_ncbi" tool is not only retrieving sequences from NCBI nucleotide or protein databases. It also filter this sequences, modifying the header (putting underscores in place of space), eliminating first streches of N that trigger errors on the makeblastdb tools, etc..
Then, Metavisitor user still may wish you upload an use their own fasta sequences. In that case the "sanitization" performed by the Fetch sequences from NCBI tool has to be done in the same way.
Therefore another functionality should be added to the tool: the ability to take as an input a fasta file from the history.
It's tricky because the name of the tool will then be misleading :-/

The galaxy way would be to split the functionality in 2:
Fetch seqeunces and sanitize_fasta.

Yes, a mid-term would be to keep the tool as currently is, and to add a
sanitize_fasta for uploded fasta file...
Christophe Antoniewski
ARTbio http://artbio.fr/ - Tel +33 1 44 27 70 05
Drosophila Genetics & Epigenetics http://drosophile.org/ - Tel +33 1 44 2
7 34 39Mobile +33 6 68 60 51 50
https://twitter.com/ARTbio_IBPS
https://twitter.com/drosofff
2016-04-07 13:19 GMT+02:00 Marius van den Beek<EMAIL_ADDRESS>
The galaxy way would be to split the functionality in 2:
Fetch seqeunces and sanitize_fasta.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/ARTbio/tools-artbio/issues/76#issuecomment-206822257

Well I realize that it a lot of buzz for not much:
User have just to manipulate their fasta headers so that they do not
contain space characters. In some cases it can even work with space. We can
fix the problem with a bit of documentation.
What is as important is that de novo contigs in the Metavisitor workflows
do not contain spaces in their fasta headers.
The general problem is that makeblastdb cuts headers at first space
character, whereas the metavisitor blast parser tool does not.
Christophe Antoniewski
ARTbio http://artbio.fr/ - Tel +33 1 44 27 70 05
Drosophila Genetics & Epigenetics http://drosophile.org/ - Tel +33 1 44 2
7 34 39Mobile +33 6 68 60 51 50
https://twitter.com/ARTbio_IBPS
https://twitter.com/drosofff
2016-04-07 13:21 GMT+02:00 Christophe Antoniewski<EMAIL_ADDRESS>
Yes, a mid-term would be to keep the tool as currently is, and to add a
sanitize_fasta for uploded fasta file...
Christophe Antoniewski
ARTbio http://artbio.fr/ - Tel +33 1 44 27 70 05
Drosophila Genetics & Epigenetics http://drosophile.org/ - Tel +33 1 44
2
7 34 39Mobile +33 6 68 60 51 50
https://twitter.com/ARTbio_IBPS
https://twitter.com/drosofff
2016-04-07 13:19 GMT+02:00 Marius van den Beek<EMAIL_ADDRESS>
The galaxy way would be to split the functionality in 2:
Fetch seqeunces and sanitize_fasta.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/ARTbio/tools-artbio/issues/76#issuecomment-206822257

Closing this issue, follow up in #82


Help Return

Correct the help return to display data better and to make sure all the appropriate data is displayed

This is complete.
Also created a hidden group to remove events you want hidden from the -help command


convert add_limit to pipe step based limiting

Description
Up until now we were managing limits inside a somewhat conflated function that was wrapping generators. The problem was that limits where applied before incrementals where and that we had a certain amount of code duplication with respect to wrapping async iterators. This PRs solves this.

closes #2142

still missing two things:
1. actual test where limit is used with incremental to do the backfilling. my take is to add this to sql_database tests and (remember to add row_order)

2. having this example I'd add it to performance guide and say how to split large backfils. esp. mentioning that records should be ordered and not to retake data twice (ie. via WHERE clause)

3. on Py 3.11 some tests are not passing consistently. pls take a look

I have added a general test that combines incremental and add_limit. I will also add a nice example using the rfam database I think but will have to do this tomorrow.

@rudolfix I have delayed the iterator wrapping to the LimitItem binding now, I agree that this is probably a good idea. That said, the typing/importing is a bit messy now imho..


Adding code coverage

Pull Request Description
[Add a description of your pull request here]

Future Release Comment
[Add description of your change, to include in the next release]
[Delete any or all irrelevant sections, e.g. if your change does not warrant a release comment at all]
Breaking Changes:

None

Features:

None

Fixes:

None

Codecov Report

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

@@            Coverage Diff            @@
##             master      #65   +/-   ##
=========================================
  Coverage          ?   49.89%
=========================================
  Files             ?       28
  Lines             ?     4099
  Branches          ?        0
=========================================
  Hits              ?     2045
  Misses            ?     1911
  Partials          ?      143

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 1f78aa5...77d9492. Read the comment docs.


Map

created a new tap with a map. There are simple event listeners so that when you click on the map it will show up as a marker, and the markers are labeled alphabetically.

@meltsufin Yeah it was mostly just an exercise to see how Maps works. I originally was going to start looking at week5 materials but I figure it would be helpful to take a look at one of the frontend APIs.

Mike -- you are not seeing the possibilities here. This map on a touchscreen device can occupy a 2-4 year old for at least a half hour.

Haha, I'll give it a try!


Extension - Check bug Cannot read properties of undefined (reading 'length')

Describe the task
Check bug Cannot read properties of undefined (reading 'length')

Resolved in http://file.cdmteck.com/s/qCPp2c2XywgKM6S
=> Follow-up crash log after release


Some Skia BlendModes are rendered slow

Details
Example app:
https://gist.github.com/felixblaschke/4d6d1367b16bf0b30348ff4e4410b66c
Drawing with BlendMode.srcOver or BlendMode.plus results in 60fps in iOS Simulator. By swapping to BlendMode.darken or BlendMode.multiply the framerate drops to ~22fps. Maybe here is some room for optimization.

Logs

Logs

[✓] Flutter (Channel stable, 2.0.0, on macOS 11.2.1 20D74 darwin-arm, locale
    en-DE)
    • Flutter version 2.0.0 at /Users/*****/sdk/flutter
    • Framework revision 60bd88df91 (2 days ago), 2021-03-03 09:13:17 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

Different blend modes will have different costs, this is expected


refactor: remove rfsfun and refactor proofs

I realize this is a pretty big PR to review (sorry about that). It is the result of my attempts to make sense of rfsfun (which I unfortunately failed).
Instead I pursued the other direction and removed rfsfun altogether. Hence, it's debatable whether this PR needs to get merged but I believe the result is conceptually simpler code (which still can be streamlined more basing on this PR).
To ensure the lab, fpred and frf functions are defined over the same domain I merged those three functions into one function returning a triple.
I added some new lemmas and moved around a few existing lemmas and definition (e.g. lloc and is_write are in eventstructure.v now)  and also renamed some things.

This change is

@anton-trunov thanks for this PR.
I liked the idea to enforce the same-domain constraint through the tuple-valued function.
However, the use of (unnamed) projections worries me a bit.
How can we get around it?
Perhaps we can use a Record with meaningful names for fields (and also specify the types of the components, for clarity).

@anton-trunov thanks for this PR.
I liked the idea to enforce the same-domain constraint through the tuple-valued function.
However, the use of (unnamed) projections worries me a bit.
How can we get around it?
Perhaps we can use a Record with meaningful names for fields (and also specify the types of the components, for clarity).

I liked the idea to enforce the same-domain constraint through the tuple-valued function.
However, the use of (unnamed) projections worries me a bit.
How can we get around it?

Thanks for the review suggestion! Ok, let me make one more pass on the projections issue which admittedly I didn't want to invest much time into before having a general consensus on the rfsfun removal. I'll get back to this issue next week.

I liked the idea to enforce the same-domain constraint through the tuple-valued function.
However, the use of (unnamed) projections worries me a bit.
How can we get around it?

Thanks for the review suggestion! Ok, let me make one more pass on the projections issue which admittedly I didn't want to invest much time into before having a general consensus on the rfsfun removal. I'll get back to this issue next week.

@eupp Thanks for all the suggestions!


Reorganize integration tests as one crate with many modules. Issue #4867. Rebased.

Reorganize integration tests as one crate with many modules. Issue #4867. Rebased.
Tests into a testsuite directory, called as module from lib.rs. All of the extern crates moved to lib.rs.  Direct use of the crates functionalities referenced by: use crates name.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @matklad (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Hm, this still does not seem to merge cleanly?
Could you try rebasing it again? It's ok to do git push --force-with-lease to the PR branch, so as not to create a separate PR.
If you need any help with rebasing, just ask! If you don't feel like doing it, I can handle it myself :)


Added macOS sound re-enabler

Project URL
https://github.com/dragstor/mac-sound-fix
Category
Utilities
Description
Simple app made to fix the sound issues (it going mute and not responsive to controls) that can happen while using an external HDMI monitor on macOS or while attaching/detaching headphones.
Why it should be included to Awesome macOS open source applications  (optional)
Because it's open sourced and it's useful 😄
Checklist

[x] Edit applications.json instead of README.md.
[x] Only one project/change is in this pull request
[x] Screenshots(s) added if any
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

Umm.. TrelloApp is causing the build issues:
https://travis-ci.org/serhii-londar/open-source-mac-os-apps/builds/561236288#L1633

@dragstor Thank you! 👍


Add system function to interrogate threads.

This should allow to list and interrogate all threads, as well as those in thread pools.

Added in 0.22.0


The devfile registry repository needs to validate the devfiles inside it.

Currently, devfile stack creators are contributing stacks to the odo-devfiles/registry repository, where it performs some basic validation / snifftesting with odo. We need to make sure that we're doing similar levels validation on the devfile/registry repository as well.
Once https://github.com/openshift/odo/pull/4525 is merged, there will be a version of odo that supports OCI-based devfile registries, and we'll be able to run similar validation for those registries.
Effectively, this issue covers the following:

[ ] Porting over the test script in odo-devfiles/registry and run against Minikube in GitHub actions
[ ] Set up tests on the OpenShift CI for PR builds that stands up registry using the OpenShift templates for the public registry

We could use any one of the 3 methods to deploy a registry (Operator, Helm Chart, or OpenShift Template), but I think using the OpenShift template makes sense here since it's what is used to deploy registry.devfile.io.

PR for running the validation tests on Minikube is now up: https://github.com/devfile/registry/pull/20
I've also put up a PR that updates the build tools as required to build this repository on the OpenShift CI https://github.com/devfile/registry-support/pull/44

The PR for the validation tests has now been merged. I've now put up a PR that adds Dockerfile and OpenShift CI test scripts for running the validation tests on the OpenShift CI.


[Feature] Add required SQL objects

Required SQL Objects

[ ] Views
[ ] Functions
[ ] Triggers
[ ] Procedures

Table requirements

[ ] Checks
[ ] Unique
[ ] Default


feat(): add security context on the container level

It is recommended practice to give a resource (pod/container) the least privileges it needs. The Lenses Helm Chart already supports configuring the security context on the pod level. This PR adds the securityContext configurations on the container level (more info on container level securityContext here).
This change is fully backwards compatible, as the containerSecurityContext value configuration is optional and defaults to {}

@georgeyord Not sure what the merge process is but any review or feedback would be much appreciated.

Hello
I am also interested in this feature (internal security rules...).
When is it expected that this MR be merged ?
Thanks


404 with maven-metadata.xml

What happened?
Deployed a modularised maven project to a fresh reposilite instance
maven-metadata.xml file present locally but not accessible online unless an additional slash is appended
Compare:
https://maven.citizensnpcs.co/repo/net/citizensnpcs/citizens-parent/2.0.30-SNAPSHOT/maven-metadata.xml
https://maven.citizensnpcs.co/repo/net/citizensnpcs/citizens-parent/2.0.30-SNAPSHOT//maven-metadata.xml
Reposilite version
3.x
Relevant log output
No response

Both URLs works for me, are you sure it's not an issue with cache on your side?

Yes, it was a cache issue. Thanks.


Nikola Sample Site

To build, install Nikola and run "nikola build", output will be in output/
It could be made waaaaay simpler but I just took the default Nikola site from nikola init and did minimal modifications needed.
If there's anything wrong/missing in this sample site, or anything else I should do, let me know.

@ralsina  Wow. Fantastic. Thanks for adding Nikola. Will send out announcements later this week. Thanks again.

@ralsina  Maybe you can help / advise. Since the site gets hosted at  /site/nikola/...  - Do you know if there's a way I can add a prefix / site.path to absolute links? e.g. currently its
  <link rel="stylesheet" href="/css/style.css">

but the sample needs something like
 <link rel="stylesheet" href="[[prefix]]/css/style.css">  leading to
 <link rel="stylesheet" href="/site/nikola/css/style.css">

Not sure if that make sense e.g. for example can I use define a SITE_PATH=/site/nikola constant in config.py and use something like:
<link rel="stylesheet" href="%{SITE_PATH}/css/style.css">

Thanks again.  Still learning Nikola.

Just change the SITE_URL in conf.py to wherever the site will really be and it should Just Work (TM)

@ralsina  Great. Fantastic. FYI: The Nikola sample site is now live http://staystatic.github.io/sites/nikola. Thanks again.


Issue290 Make the rectNxM restart files valid again

Make the rectNxM restart files valid again
Fixes #(290)

Change Description
After changes to the grid and the swap of the array ordering, the init_rectNxM.nc are no longer valid.

Test Description
The fixed files can be verified by the model running the config_rectNxM.cfg configuration files successfully.

To be merged after #306

@timspainNERSC : Can you fix the conflicts on this one and request a review?

PR #380 reflects what this PR will look like after PR #373 is merged into develop.


OnDatabaseChange not available via version on maven

Hey,
am I right that OnDatabaseChange is not pushed to maven?
Cheers,
Jens

Pushed 0.4 with the change for #4. Let me know if you like the lib and how you use it. :sunglasses:


Relatives URLs are appended when multiple calls

What is the expected behavior?
define a base url when creating the Curl object that will never change, making possible to call multiple times get/post/push/... methods with the correct full url.
What is the actual behavior?
base url is modified by method setUrl(), causing to append a relative part to the previous relative part
What steps will reproduce the problem?
Instanciate a Curl object with a base_url, call any of get()/post()/put() method will append relative part to base_url, causing 404.
Code:
$curl = new Curl("https://my.domain.com/path/");
$response = $curl->get("relative/part");
// curl base url is now: https://my.domain.com/path/relative/part

$response = $curl->post('another/relative/path', ['data' => "content"]);
//curl base url is now: https://my.domain.com/path/relative/part/another/relative/path

Suspicious lines: https://github.com/php-curl-class/php-curl-class/blob/8b73ad5b3556585282ad58a22689d54aef614a9d/src/Curl/Curl.php#L1309-L1312

Thank you for your reply.
I understand what you mean, and in the meantime, an example is provided that tend to let me think we can call multiple times with relative and/or absolute path
https://github.com/php-curl-class/php-curl-class/blob/8b73ad5b3556585282ad58a22689d54aef614a9d/examples/get_relative.php#L6-L18
Also please note that when you set a base url with a path, and call get()/post()/... method, the path from base url will be removed, hence in your example, the url you set will be overwritten which is not evident to debug
$curl = new Curl('https://my.domain.com/path/');
$response = $curl->get('/custom/relative/part'); // https://my.domain.com/custom/relative/part
$response = $curl->post('/custom/another/relative/path', ['data' => 'content']); // https://my.domain.com/custom/another/relative/path

I used the absolute path to unlock my situation in the meantime, as you mentioned in your comment, but imo it would be a great improvment to handle this case :)
Thanks for your time !


docs: fix Suggest changes to this page link

The Suggest changes to this page link that can be found at the bottom of all documentation pages incorrectly points to the main branch, which leads to a 404 because the default branch is 0.x.

This PR:

[x] Updates the editLink.pattern option to rename the branch from main to 0.x.

I made it so that the branch name is determined programmatically. Not sure if that will work on Netlify though, let's see. Thanks for noticing the link issue!


Delete all AWS resources in an account based on a specific tag is failing.

I am trying to delete all AWS resources in an account based on a specific tag (e.g., tag:release-environment with value val-157373). The job is running in a GitLab pipeline, but it frequently encounters the following error. The process reaches this point, gets stuck and does not proceed further.
time="2024-12-17T10:38:09Z" level=error msg="listing failed:\n    RequestError: send request failed\n    caused by: Post "https://codestar.eu-west-1.amazonaws.com/": dial tcp 22.218.139:443: i/o timeout" error="RequestError: send request failed\ncaused by: Post "https://codestar.eu-west-1.amazonaws.com/\": dial tcp 22.218.139:443: i/o timeout" owner=eu-west-1 resource_type=CodeStarProject
nuke-config.yaml:
regions:
  - global
  - eu-west-1

blocklist:
  - ""

presets:
  common:
    filters:
      __global__:
        - property: tag:release-environment
          value: "val-157373"

accounts:
  <account_id>:
    presets:
      - common

gitlab-pipeline.yaml:
aws-nuke:
  stage: nuke
  image: <aws_account_id>.dkr.ecr.eu-west-1.amazonaws.com/ghcr/ekristen/aws-nuke:v3.36.0
  variables:
    AWS_ASSUME_ROLE_ARN: arn:aws:iam::<aws_account_id>:role/cleanup-iam-role-nuke
  script:
    - aws-nuke version
    - aws-nuke nuke -c aws-nuke-config.yaml --force --force-sleep 3
  rules:
    - when: always
  tags:
    - big

It's not stuck on that error, that error can be safely ignored. I recommend using this starter config and modify to your needs. https://aws-nuke.ekristen.dev/starter-config/.
Likely it's hanging up on either S3Object or CloudWatchLogGroups. You can turn on debug logging and find out for sure, the latest version tells you what it's currently iterating. You should 99% disable S3Object. (i.e. exclude)

It's not stuck on that error, that error can be safely ignored. I recommend using this starter config and modify to your needs. https://aws-nuke.ekristen.dev/starter-config/.
Likely it's hanging up on either S3Object or CloudWatchLogGroups. You can turn on debug logging and find out for sure, the latest version tells you what it's currently iterating. You should 99% disable S3Object. (i.e. exclude)

@ekristen That helps, but from my configuration it is not fetching the AWS resources in an account based on a specific tag.
project:
    filters:
      __global__:
        - property: tag:release-environment
          value: "val-157373"

accounts:
  "<account>":
    presets:
      - common
      - organization
      - defaults
      - project

I am trying to delete all AWS resources in an account based on a specific tag (e.g., tag:release-environment with value val-157373). Is it possible to achieve this using a global filter?

Currently the tool filters out what to keep, not what to nuke, but you can sorta achieve the opposite by using invert: true on the filter, everything that doesn't match will be kept instead.

Thanks @ekristen I have another following issue, do i need to open another issue?

It worked. Thanks


serverless --path parameter isn't working?

Hi, I've noticed that when I provide --path parameter in sls invoke local -f .. -p path/to/event.json the invocation finishes with error: The file you provided does not exist..
I guessed from the code here that I need to add include option to serverless.yml file so that event.json gets copied to .build directory: https://github.com/graphcool/serverless-plugin-typescript/blob/4bace79c1dd25cfb416cb7a4e4cc019c86fec309/src/index.ts#L149:L152
like so:
package:
  include:
    - src/**/event.json

However, any changes to event.json are not being updated when I do a new lambda invocation.
So I wonder if the --path paramter isn't working so far or am I doing it wrong?

I experienced the same behaviour... any update on this?


Add draft API specs for TRN allocation, QTLS and Alert webhooks

Should we tighten it up for early releases of the API and add nino? Currently around 30% coverage of active teachers (70%) with TPS data. May reduce support noise / unhappy path churn? (TPS would always have it - not sure about EWC)

Good shout - I've added a nationalInsuranceNumber property.


Add missing svg files in assembly page

To solve #689 and repair #641

Thanks @roman-dvorak for your contribution, this is a valuable feature and the documentation is very helpful too!


There is no way to close script opened windows.

Tell us about your environment:

Puppeteer version: v0.10.2
Platform / OS version: all platforms
URLs (if applicable): n/a

What steps will reproduce the problem?
Please include code that reproduces the issue.

I 'goto' a page(Page A) that will open a new window(advertisement).
Because the chrominium don't have adblock installed, the window just opened.
I 'goto' another page(Page B). the ad window stay opened.

There for, the window forever stays there, opened.
What is the expected result?

I want a 'window.open' event. Then I can listen to it and judge wether it will open.
I want a page handler to handle that window. Maybe I need to inspect content in that window,or I can close it using page.close() manually;

What happens instead?
I can't find a way preventing the window from opening, neither can I find a way closing the opened window. Please help me out.
P.S. How Can I get the default opened Page handler(the initial about:blank one)?
Thanks!!

We're working on an APIs for enumerating the open pages and catching events when pages open: https://github.com/GoogleChrome/puppeteer/pull/554
Will that work for you?

YES!!
That's exactly what I'm looking for! And the browser.pages() ! I'd love that!
Thanks so much!  I will close this issue and be looking forward to #554


Fix pagination slider when it's too closed to the end

Make the number of pages that are shown closed to the end, equal to
the number of pages that are shown closed to the beginning.

Please send with a before and after screenshot.

As you can see before the fix when current page is close to the end, the pagination shows 9 pages close the end, but when the current page is close to the beginning, it shows only 8 pages close to the beginning.
I thought it should be symmetric.


admissionregistration objects - mutating and validating webhook configurations

Kubernetes offers extensible admission controllers - https://kubernetes.io/docs/reference/access-authn-authz/extensible-admission-controllers/
These are available in kubectl, e.g. kubectl get mutatingwebhookconfigurations (or validatingwebhookconfigurations).
Why is this needed: these  are objects that may be deployed in a namespace. Given that they form part of a deployment / is a configured resource, it would be great if LENS could manage it, instead of needing to revert to other tools.
Specific use-case where this was a problem: trying to remove all resources and delete a namespace, this was the only resource from a broken deployment that i couldn't clean up with LENS.
Environment you are Lens application on:

Kubernetes distribution: EKS 1.19
Desktop OS: Windows 10 (latest)
LENS - latest update - 4.1.4

If you deploy ElasticSearch on K8s they use a validatingwebhook as well.  Plus they have a bad habit of having base configurations that need updating in order to deal with hosted K8s solutions (such as Digital Ocean).  Not that it's a problem to change it with kubectl, but, at this point, Lens is so good that I kind of smirk to myself when I have to use kubectl!
:)

What's the status of this? Are there impediments to adding those two resources?

any news ?

+1


encoding/xml: encoding a start token with a namespace set in Name and Attr results in invalid XML

What version of Go are you using (go version)?

$ go version
go version devel +6965b01ea248<PHONE_NUMBER>0706 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
https://play.golang.org/p/ZK8yhFoVwW9
	e := xml.NewEncoder(os.Stdout)
	e.EncodeToken(xml.StartElement{
		Name: xml.Name{Space: "space", Local: "local"},
		Attr: []xml.Attr{{Name: xml.Name{Local: "xmlns"}, Value: "spaceAttr"}},
	})
	e.Flush()
	// <local xmlns="space" xmlns="spaceAttr">

What did you expect to see?
One "xmlns" attribute.
What did you see instead?
Two "xmlns" attributes.
While the provided example is obviously very contrived this can happen in the real world (without a user doing something silly like manually adding a second xmlns attribute) in the situation where the user decodes some XML, makes modifications to the tokens, then encodes those tokens somewhere else (probably a fairly common use case). The decoder will add the "xmlns" attribute and fill out Name, then the encoder will duplicate it. The user would have to manually remove xmlns before encoding, or use RawToken (but this is not an option if the decoder is an arbitrary xml.TokenReader so this isn't always possible). Skipping xmlns attributes on marshal if start.Name is set is a simple patch that would fix this issue.

/cc @rsc


prepare for next development iteration

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Travis CI User seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @dyna-dot. What is the nature of your pull request? We have a snapshot version on develop but master shows the latest release. Therefore we merge pull requests into develop rather than onto master.


Fixed the wording on the explicit to be consistent with the code's implicit conversions.

Fixed the wording on the explicit conversions to be consistent with the code's implicit conversions.

LOL @ "Signifigant"


feat(react-instantsearch-hooks): Add useStats hook

Summary
Add a hook for useStats to react-instantsearch-hooks.
The goal is to be able to use use this hook to provide a widget, as well as replace the 'not implemented' documentation: https://www.algolia.com/doc/api-reference/widgets/stats/react-hooks/#about
Result
Review added test for basic functionality / expectation.
Next Steps

Add a widget to react-instantsearch-hooks
Update examples
Update documentation

This will be released next Tuesday normally. Will you prepare the PR to the documentation as well?


LwM2M: support Connectivity Monitoring object (object id 4)

Is your enhancement proposal related to a problem? Please describe.
Object 4 Connectivity Monitoring is Part of the LwM2M Tech Spec 1.0 and allows connected devices the ability to share statistics such as RSSI, bandwidth usage and other data.  Currently, Zephyr doesn't support this.
Describe the solution you'd like
Implement Connectivity Monitoring object in Zephyr.  Useful for modem-based, 802.15.4, bluetooth and wireless connections.

This is implemented in https://github.com/zephyrproject-rtos/zephyr/pull/17710

This is implemented in #17710

Can the issue be closed, then, @mike-scott ?


Re-align Dialog Actions

I believe the 'OK' action button should be to the right of the 'Cancel' button per the material-ui guidelines.
https://material.io/guidelines/components/pickers.html#pickers-date-pickers
BTW this library is awesome. Please let me know if I can help or if you'd prefer me to submit a PR.
Thanks!

Should be resolved with PR #11


Add ContentExcerpt feature for pages

Thanks for your PR :smiley:
Just one question, why ContentExcerpt and not Excerpt like with Jekyll?
Also can you add a test with the use of page/post.content_excerpt to display the excerpt?

Ok, I will do it.

Excellent, thanks!


Ollie: Watch skillsoniq.com

Ollie requests the watch of the watch_keyword skillsoniq\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

skillsoniq\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Backwards compatibility issue with cross config variable references

What happened?
I updated devspace from version 5 to 6, using the config version v1beta9 and ran into this error when trying to deploy:
create_deployments: error deploying Y: couldn't find runtime variable runtime.dependencies.X.images.x.image, make sure the dependency X was loaded
What did you expect to happen instead?
For the file to be backwards compatible.
How can we reproduce the bug? (as minimally and precisely as possible)
Try using cross file variable references, using config version v1beta9, with devspace v6.
My devspace.yaml:
version: ...
...

Local Environment:

DevSpace Version: 6.1.1
Operating System: mac
ARCH of the OS: ARM64
Kubernetes Cluster:
Cloud Provider: aws
Kubernetes Version: 1.21
Kubectl 1.22

Anything else we need to know?

/kind bug

@mahendrabagul

Do you have any specific need to keep the devspace yaml version at v1beta9?

This is a backward compatibility issue, In order to allow for a smoother upgrade, I was seeing if I could upgrade to devspace v6 (in order to prompt developers to upgrade and to upgrade the version on our tooling) before changing the files.

I assume you are using pipelines

I was not using pipelines as those features aren't available in v1beta9 as far as I know.


Draft: function calling

This goes with a few other PRs to enable OpenAI function calling in RTVI.

Cool. Approving! If you can squash all commits into one it would be great. Just to have a nice history. :pray:


Ignore $log

How can I disable the $logs?
SoundManager V2.97a.20140901 (HTML5-only mode)
SoundManager 2 HTML5 support: mp3 = true, mp4 = true, ogg = true, opus = true, wav = true
SoundManager 2: Ready. ✓

Never mind. I've jus did:
app.config(function() {
  soundManager.setup({
    debugMode: false
  });
});


rename BlockNumber = u64  alias

Describe the feature
this clashes with the rpc type of the same name and already caused a number of issues.
perhaps BlockNumberU64?
Additional context
No response

I think it's also the same for BlockId too.

closing since we renamed the BlockNumberOrTag


Double middle click on taskbar preview to close despite confirmation modal

Description of the new feature/enhancement

If I want to close a program/window, I'll often hover it in the taskbar, then middle-click the preview that appears. If there are multiple tabs open, though, a modal shows up asking me if I'm sure. That's fine. I like those kinds of warnings. But it completely blocks me from closing the window from the taskbar.
What several (sadly not all) programs do in those cases is interpret the second middle click as a confirmation and close right away. I wish Windows Terminal did this as well.
Proposed technical implementation details (optional)
When there's currently a confirmation modal and I send another "close" command through a taskbar middle click, I wish Windows Terminal would interpret this as "Yes, I'm sure" and close.

This seems like a reasonable request to me. Thanks!

That's definitely a good idea. Thanks!


ISSUE: Unlocking doesn't decrement Unlock events

With a lock code enabled, Enforce pin limit enabled and Unlock events set to 1, using the keypad code on a BE469ZP doesn't cause the Unlock events counter to decrement.  The lock code never becomes disabled in HA or on the lock.
Versions:
Home Assistant Core: core-2021.3.3
ZWave: zwavejs2mqtt, Zwavejs version 6.6.0, Zwavejs-server version 1.1.0, websockets enabled, MQTT disabled.
Keymaster version 0.0.27, installed from HACS
Here are the zwavejs and keymaster events posted for a manual and keypad lock and unlock operation:
Event 6 fired 9:50 AM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Manual unlock operation",
"parameters": {}
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:46.966844+00:00",
"context": {
"id": "578e9db597953b47460f8bcfb11e408d",
"parent_id": null,
"user_id": null
}
}
Event 5 fired 9:50 AM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Manual lock operation",
"parameters": {}
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:41.986789+00:00",
"context": {
"id": "a311088ba08837d193fe83d3ef97e2e2",
"parent_id": null,
"user_id": null
}
}
Event 4 fired 9:50 AM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Keypad unlock operation",
"parameters": {
"userId": 3
}
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:34.437041+00:00",
"context": {
"id": "c8215aadab40a57e391444be03d58627",
"parent_id": null,
"user_id": null
}
}
Event 3 fired 9:50 AM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Keypad lock operation",
"parameters": {}
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:29.114754+00:00",
"context": {
"id": "2e73be27a2d0c98026fa206c4c8dcfeb",
"parent_id": null,
"user_id": null
}
}
Event 3 fired 9:50 AM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "unlocked",
"action_text": "Manual unlock operation",
"code_slot": 0,
"code_slot_name": ""
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:46.967883+00:00",
"context": {
"id": "454242aaf2b8aeea3ce0e7338fe2ad91",
"parent_id": null,
"user_id": null
}
}
Event 2 fired 9:50 AM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "locked",
"action_text": "Manual lock operation",
"code_slot": 0,
"code_slot_name": ""
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:41.987761+00:00",
"context": {
"id": "2d04a443cf1fa5915d500af9dd0c5b9e",
"parent_id": null,
"user_id": null
}
}
Event 1 fired 9:50 AM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "unlocked",
"action_text": "Keypad unlock operation",
"code_slot": 3,
"code_slot_name": "Limited"
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:34.438140+00:00",
"context": {
"id": "d1d7c30f5a51f8e131b5d73cb9cd2035",
"parent_id": null,
"user_id": null
}
}
Event 0 fired 9:50 AM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "locked",
"action_text": "Keypad lock operation",
"code_slot": 0,
"code_slot_name": ""
},
"origin": "LOCAL",
"time_fired": "2021-03-13T14:50:29.116742+00:00",
"context": {
"id": "e8b31d22a12b53821789557aed29455d",
"parent_id": null,
"user_id": null
}
}
frontdoor.zip

I believe I have fixed this with the linked PR. I will merge this in a sec but it may be a couple of days before the new release, I'd like to resolve some of the other issues reported before we release again

Quick test of one of my BE469ZP locks on v0.0.28 was still giving me this issue. No time to grab logs before work, but I'll do it tomorrow morning unless @rbauman70 beats me to it. 😛

Yes please do share logs, and also do me a favor and listen to the keymaster_lock_state_changed event, unlock or lock your lock with a PIN, and share the event data for that event as well. That will help me diagnose what could be happening

Yeah, I can confirm it's still not working in 0.0.28.  Here are the zwave and keymaster event logs:
Event 3 fired 5:42 PM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Manual unlock operation",
"parameters": {}
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:42:05.236187+00:00",
"context": {
"id": "16cc2f552b75c09906761f4b2c15676d",
"parent_id": null,
"user_id": null
}
}
Event 2 fired 5:41 PM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Manual lock operation",
"parameters": {}
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:41:58.550304+00:00",
"context": {
"id": "45cd02eee7e413f68698a413d630cbbe",
"parent_id": null,
"user_id": null
}
}
Event 1 fired 5:41 PM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Keypad unlock operation",
"parameters": {
"userId": 3
}
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:41:45.765445+00:00",
"context": {
"id": "581210761b6b1b3c9cc8fb7d6749581b",
"parent_id": null,
"user_id": null
}
}
Event 0 fired 5:41 PM:
{
"event_type": "zwave_js_event",
"data": {
"type": "notification",
"domain": "zwave_js",
"node_id": 3,
"home_id":<PHONE_NUMBER>,
"device_id": "4c13f86d958d890f6c30c6a99e0da63f",
"label": "Keypad lock operation",
"parameters": {}
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:41:39.799786+00:00",
"context": {
"id": "2c9d9bc7628b1c728d94e93600029127",
"parent_id": null,
"user_id": null
}
}
Event 3 fired 5:42 PM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"notification_source": "event",
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "unlocked",
"action_text": "Manual unlock operation",
"code_slot": 0,
"code_slot_name": ""
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:42:05.238068+00:00",
"context": {
"id": "f1cc37e724c2ecec3ea609dce2b4a4cd",
"parent_id": null,
"user_id": null
}
}
Event 2 fired 5:41 PM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"notification_source": "event",
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "locked",
"action_text": "Manual lock operation",
"code_slot": 0,
"code_slot_name": ""
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:41:58.552125+00:00",
"context": {
"id": "e96e31658fdb6d3a5002501f9d63bb96",
"parent_id": null,
"user_id": null
}
}
Event 1 fired 5:41 PM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"notification_source": "event",
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "unlocked",
"action_text": "Keypad unlock operation",
"code_slot": 3,
"code_slot_name": "Limited"
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:41:45.766521+00:00",
"context": {
"id": "57f3703c07eb2114f04386e80cabae3d",
"parent_id": null,
"user_id": null
}
}
Event 0 fired 5:41 PM:
{
"event_type": "keymaster_lock_state_changed",
"data": {
"notification_source": "event",
"lockname": "frontdoor",
"entity_id": "lock.touchscreen_deadbolt_z_wave_plus_current_lock_mode_frontdoor",
"state": "locked",
"action_text": "Keypad lock operation",
"code_slot": 0,
"code_slot_name": ""
},
"origin": "LOCAL",
"time_fired": "2021-03-17T21:41:39.800828+00:00",
"context": {
"id": "f6f1219cbf2d16bdb9a14d1417384c7c",
"parent_id": null,
"user_id": null
}
}

thanks, did you regenerate your package files before testing? this was a template change not a code change

That worked. Unlock events was decremented.  I had restarted HA a number of times after updating in HACS to the new keymaster version but I had not gone through Integrations..keymaster..options..submit again.  Thanks for getting it working.

Aha. My issue was the opposite. I didn't restart HA after package regeneration. After the restart, everything is working as intended. Thanks @raman325 :)


PD: Show slot of empty modules

overview
Currently empty modules have no way of signifying that they have a slot to add labware into, until you mouseover them. Empty modules are just empty gray rounded rectangles.
TODO: add slots on to modules
design
WIP

oops duplicate of #4310


Add StorageFeatures property to IDocumentStore

While attempting to add a sequence to the database through marten I discovered that IDocumentStore was missing the Storage property needed to refer to the sequence as per this documentation.
This change adds the missing property :)

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

So how do I reference the created sequence the way the documentation references them? Is there a read-only way of exposing the MatterId from the documentation example?

@elexisvenator So help me out, why would you even need to do that? What's your use case here?

I am doing exactly what is described in this article: https://martendb.io/scenarios/using-sequence-for-unique-id.html
I created an id class based on FeatureSchemaBase.
Added it to storage as part of the configuration options.
Then in one of my service calls I attempt to find the sequence again so I can get the next value. as shown in this example
But I cannot do that because theStore.Storage.FindFeature(typeof(MatterId)).Objects.OfType<Sequence>().Single(); refers to the property Storage which is not in the IDocumentStore interface.
Is there another way of getting the sequence again?

@elexisvenator if you have IDocumentStore, you could type cast it to DocumentStore like (store as DocumentStore)?.StorageFeatures.FindFeature(typeof(MatterId)).Objects.OfType<Sequence>().Single();

@elexisvenator sample in docs uses the DocumentStore directly. It is safe for you to type cast IDocumentStore instance to DocumentStore for accessing Storage


Refactor ably wrapper

This pull request aims to improve Ably wrapper by

Providing the dispatcher for the callback calls
Reducing disconnect operation call to detaching only
Added optional ? to a nullable reason parameter as subscribing app would crash after stopping.

For the disconnect operation, I realised that detaching would implicitly end presence from documentation (I added comment into code) so it should be safe to reduce the whole operation to detach. I left channel and presence unsubscription code there.
After change this test fails on DefaultPublisherTest
should fail adding process when adding a trackable that is currently being added and it fails
I was confused by what this test was trying to do so I couldn't fix it. I'll leave it for now till Monday

I'm having issues with reformatting code to match ktlint style. It will complain about my indentation being 4 and not 5. But I will indent code to 5 and then it will complain otherwise 😩  Help please @KacperKluka

Superseded by https://github.com/ably/ably-asset-tracking-android/pull/513


Config level tags are ignored

The documentation promises that you can specify stack-level and config-level tags.  Running stack_master init us-west-2 test-stack even generates a stack-level tag for you.
stacks:
  us-west-2:
    test-stack:
      template: test-stack.json
      tags:
        environment: production

However, running stack_master diff against the template, the environment:production tag is nowhere to be seen.
arecker@localhost:testproject$ be stack_master diff
Executing diff on test-stack in us-west-2
Stack diff:
--- /var/folders/sc/b7xz6j0d57z5_d7j72mzyw800000gp/T/diffy20170411-39386-b6rlc6 2017-04-11 15:39:47.000000000 -0500
+++ /var/folders/sc/b7xz6j0d57z5_d7j72mzyw800000gp/T/diffy20170411-39386-e3a2o7 2017-04-11 15:39:47.000000000 -0500
@@ -0,0 +1,16 @@
+{
+  "AWSTemplateFormatVersion": "2010-09-09",
+  "Description": "Cloudformation stack for test-stack",
+  "Parameters": {
+    "InstanceType": {
+      "Description": "EC2 instance type",
+      "Type": "String"
+    }
+  },
+  "Mappings": {
+  },
+  "Resources": {
+  },
+  "Outputs": {
+  }
+}
\ No newline at end of file
Parameters diff:
+---
+InstanceType:
No stack found

How are these tags intended to work exactly?  I would have expected any taggable resource in the stack to receive this.  Maybe this feature isn't complete?

Wow, 1 year for me to reply to this issue. I'm really sorry about that.
The tags exist against the stack itself and hence are not present in the template. CloudFormation then propagates these tags to Resources within the stack where they support it.
Sorry again for taking so long to reply.

Wow, 1 year for me to reply to this issue. I'm really sorry about
that.

Lol - no worries, mate.  We have since been using stack level tags, and
I should have circled back and closed the issue myself.  Thanks!

Feel free to join our slack workspace if you have further questions, we should be more responsive there.


Replace HTTP URLs with HTTPS for sites with HTTPS versions

This PR replaces HTTP URLs with HTTPS where applicable (only for sites with HTTPS versions, and also example/dummy links). I also fixed a few instances of outdated links, such as http://continuousphysics.com/Bullet -> https://bulletphysics.org.
While making this PR I noticed that there are many URLs that are dead links. I left all of those alone, but at some point another PR should replace those with working links or with web.archive.org links.

I wouldn't edit any third-party license notice, since we don't control them.

I undid the changes to the Apache licenses and FreeType. I also sent a message to the Apache maintainers about this. For other licenses, the GPL and MPL both use HTTPS links in the official version of the licenses and auto-redirect to HTTPS, so I kept them.

Thanks!


build(flow): ignore config-chain/test/broken.json

Description:
Ignore invalid json files in flow config to prevent expected failures from producing errors.
See https://github.com/facebook/flow/issues/869
Motivation and Context:
We are getting this error when running flow:
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ node_modules/npm/node_modules/config-chain/test/broken.json:11:3

Unexpected string

      8│     "url": "https://github.com/dominictarr/config-chain.git"
      9│   }
     10│   //missing , and then this comment. this json is intensionally invalid
     11│   "dependencies": {
     12│     "proto-list": "1",
     13│     "ini": "~1.0.2"
     14│   },

See https://github.com/facebook/flow/issues/2364
How Has This Been Tested?
run yarn flow and verify error is no longer produced,
Screenshots (if appropriate):
Types of changes:
Bug fix
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage remained the same at 16.201% when pulling 1de5ae65a66f3574466ee0b5c159319c2871e9e3 on mrfelton:fix/flow-ignore-broken-config-chain into 800ece4cc779f8b229dfd503f61f319c1ef803fe on LN-Zap:master.


Docs increment

Moving more examples over, with minor changes. My Sybil setup can't handle the \" escapes necessary to use a double-quote character in a string literal. Making that work reliably looked difficult.
I can either avoid string literals in the docstrings (usually by using symbols instead) or use the <<# comment strings instead (for the inner or outer, doesn't matter which). Too bad we don't have proper heredocs, or even triple-quoted strings like Python. Common Lisp has character-stream reader macos and can implement them, but Clojure and Emacs lisp also have this problem. To maintain general Lisp editor compatibility, that's the best Lissp can do.

There's still, what, ten sections left, or so? We'll get there. Then lots of rereading and editing.


LZ4, BZ2, snappy compression support

Added in 22.10: https://github.com/ClickHouse/ClickHouse/pull/42071

LZ4 is blocked:

https://github.com/pierrec/node-lz4 does not support modern Node.js versions
https://github.com/antoniomuso/lz4-napi lacks streaming support

WIP branch (unmerged) https://github.com/ClickHouse/clickhouse-js/compare/main...lz4


fix: Refactoring to replace all usages of isEqual method from lodash with equal method from fast-deep-equal/es6

Description
The change is a refactoring work. All usages of isEqual method from lodash has been replaced with equal method from fast-deep-qual/es6. This shall optimise the speed of execution as already analysed by the task creator.
Fixes #16421
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
This has not been tested other than reviewing the code changes.
Checklist:

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

/ok-to-test=9170474

/ok-to-test sha=9170474

Thanks a lot for the contribution on this @smrutiparida, we will review this and take it forward from here. Please do keep a lookout for any change requests, If would like to get this merged ASAP if all looks good 😄 Let us know if you have any question meanwhile!
cc @SatishGandham

@SatishGandham do we need some manual QA around this? Can you shed light on that?


Split requirements.pip into developers and users requirements.

Currently all the requirements both for using vumi and developing vumi are stored in one requirements.pip file and all of these are incorporated into the setup.py file. I think we should split the file into requirements.pip which contains just the dependencies need to run Vumi and requirements-dev.pip which includes the dependencies need to develop Vumi and run its test suite.

Fix merged in https://github.com/praekelt/vumi/commit/ca0b386bde1642f129f34f9713822ad06baf6aac


No kinematics solver instantiated for group 'arm'

Description
Hi. I build moveit_config_package using moveit_setup_assistant from my own urdf. I install pick_ik plugin and modified kinematics.yaml as below.
arm:
  kinematics_solver: pick_ik/PickIkPlugin
  kinematics_solver_timeout: 0.05
  kinematics_solver_attempts: 3
  mode: global
  position_scale: 1.0
  rotation_scale: 0.5
  position_threshold: 0.001
  orientation_threshold: 0.01
  cost_threshold: 0.001
  minimal_displacement_weight: 0.0
  gd_step_size: 0.0001

I write a simple node base on the instruction in here. When I run this simple node, I got the following error log. And real arm didn't move.

[INFO] [1721283620.149320877] [moveit_rdf_loader.rdf_loader]: Loaded robot model in 1.02484 seconds
[INFO] [1721283620.149397762] [moveit_robot_model.robot_model]: Loading robot model 'rml_63_gripper_description'...
[WARN] [1721283620.195971420] [moveit_ros.robot_model_loader]: No kinematics plugins defined. Fill and load kinematics.yaml!
[INFO] [1721283620.205834940] [move_group_interface]: Ready to take commands for planning group arm.
[INFO] [1721283620.206336457] [moveit_ros.current_state_monitor]: Listening to joint states on topic 'joint_states'
[INFO] [1721283621.206451246] [moveit_ros.current_state_monitor]: Didn't receive robot state (joint angles) with recent timestamp within 1.000000 seconds. Requested time<PHONE_NUMBER>.206350, but latest received state has time 0.000000.
Check clock synchronization if your are running ROS across multiple machines!
[ERROR] [1721283621.206508374] [move_group_interface]: Failed to fetch current robot state

I wrote another complicated node, which control real arm to execute trajectory. I used code as below:
geometry_msgs::msg::Pose goal;
move_group_->setJointValueTarget(goal, "Link00");
move_group_->move();

After I run this code, I also get the same warning:

[WARN] [1721283620.195971420] [moveit_ros.robot_model_loader]: No kinematics plugins defined. Fill and load kinematics.yaml!

And I got a new error log:

[ERROR] [1721283094.638409609] [moveit_robot_state.robot_state]: No kinematics solver instantiated for group 'arm'
[INFO] [1721283094.639067653] [move_group_interface]: Plan and Execute request accepted
[INFO] [1721283094.920604840] [move_group_interface]: Plan and Execute request complete!

But I try to get ros2 param and get result as below, it seems that I loaded kinematics solver successfully. So why these things happened?

Your environment

ROS Distro: Humble
OS Version: Ubuntu 22.04
Binary install

Expected behaviour
No Error log, and the real robot arm work as what I exepected.
Actual behaviour
I got Error and real robot arm doesn't move at all. unless I use these kind of function:
std::vector<double> goal_joints = {0.0067008, -0.422063, 1.29788, -0.032265, 1.97607, 0.0046417};
move_group_->setJointValueTarget(goal_joints);
auto res = move_group_->move();

If I give moveit the exact joints, the real arm will move.

I have the same thing running ROS2 Iron, both with gazebo and the model of a real arm. Very weirdly, however, this sample works (as the poster writes)! But just running  move_group_interface.getCurrentJointValues(); fails
https://robotics.stackexchange.com/questions/111430/failed-to-fetch-current-robot-state-when-using-mgi-in-wall-timer-callback-functi


Support for launching matlab via CMake in setup-matlab@v2

Hello everyone, thanks a lot for work on MATLAB-related GitHub Actions.
I looked in updating the CI of my projects to setup-matlab@v2 (see https://github.com/robotology/idyntree/pull/1181). As my project is a C++ library that also provides MATLAB bindings, all the tests are handled via CMake test infrastructure (see for example CMake's FindMatlab matlab_add_unit_test command). On setup-matlab@v1 everything worked fine, while on setup-matlab@v2 the test fail as matlab is not able to find the license.
The problem is similar to https://github.com/matlab-actions/setup-matlab/issues/89, but in my case I do not directly set the command used to launch the tests, as that is handled by CMake. I wonder if there is any workaround for launching matlab command using setup-matlab@v2 ? Thanks a lot in advance!

Hi @traversaro, like #89, we do not have an official solution for supporting this type of workflow yet.
In the meantime, you may be able to create an adapter script that translates the matlab command CMake is attempting to call into a run-matlab-command call.
For example, in your job log, I see CMake is attempting to call matlab -nodisplay -nodesktop -nojvm -batch SOME_COMMAND_HERE. You could potentially write a bash script named matlab that does something like this:
# Example:
# matlab -nodisplay -nojvm -batch "disp hello"

# Treat the last arg as the command
# In the example: "disp hello"
cmd=${@: -1}

# Treat everything else but the last two args as args for run-matlab-command
# In the example: -nodisplay -nojvm
args=${@:1:$#-2}

# Call run-matlab-command with the cmd and args
# In the example: ./run-matlab-command "disp hello" -nodisplay -nojvm
run-matlab-command "$cmd" "$args"

Then you have to get CMake to call your version of matlab instead of the matlab in the MATLAB bin folder. I am not familiar enough with the FindMatlab module to know if it supports doing that.
Like I mentioned in #89 though: please note the "run-matlab-command" binary is undocumented and subject to change in the future.

Good idea, thanks! I was thinking on modifying the CMake build system to use run-matlab-command, but avoiding to modify the CMake script would be welcome.

Like I mentioned in https://github.com/matlab-actions/setup-matlab/issues/89 though: please note the "run-matlab-command" binary is undocumented and subject to change in the future.

Sure, thanks!


webview: Fix: multiple messages are posted on Android.

@borisyankov Can you please try this PR on your physical device, as a fix for #1600

I don't understand why this would fix it.
It looks the identical functionality.

(https://stackoverflow.com/a/26146974/5612089)
BTW it seems to work when I encountered the same problem (https://chat.zulip.org/#narrow/stream/mobile/subject/meeting/near/400559)

In the example, they are actually calling addEventListener twice. I don't see any relation to our issue.

Can you confirm if this has any effect?

Looks like this is working now. Closing.


Subscribers to peer messages shouldn't have to wait for a timeout when the peer disconnects

What is wrong?
When a peer disconnects, right now, subscribers to messages wait for a timeout before they realize that the peer is gone. This slows down syncing and other things (and in pathological cases with no timeout, can pause forever).
How can it be fixed
~Maybe cause all subscribers to the peer to receive a TimeoutError immediately on peer disconnect.~
Implement deregister_peer in ResponseCandidateStream and immediately raise TimeoutError on any pending requests.

I think we should raise PeerConnectionLost instead of TimeoutError, like Peer.read() does


Draft: boards: Implement Pin Control Logic for all T-Mobile Dev Boards

Add Kconfig, pinctrl, and wake-up-triggers to efm32pg_stk3402a. Restore testing flag and add pwm, uart nd spi to yaml file. Put pinctrl file in dts folder so alternate boards can be used. Use SiLabs settings for gpios and pinctrl on uarts. The leuart is not implemented yet so only usart0 @ 115200. Add CONFIG settings to match SiLabs and enable testing. Add compile conditional around USART_NUM. Will need to add this? Add pinctrl device compatible to "silabs,gecko-pinctrl"

Replaced


[WIP] Jet Stirred Reactors

[ ] Ready for review

Code is functional and successfully runs JSRs given a ChemKED JSR file (https://github.com/pr-omethe-us/PyKED/pull/126)

Todo:

[ ] Multiprocessing
[ ] Efficently save .h5 files
[ ] Refactoring

[ ] eval_model.py
[x] simulation.py

[ ] Error functions

[x] Difference between area under the curves using numpy.trapz

[ ] Multiple datasets

I would suggest integrate the difference between the curves, rather than find the difference of the integrals.

I would suggest integrate the difference between the curves, rather than find the difference of the integrals.

I'm trying to visualize the difference here... in practice the answer should be the same, no? Though it's easier/more efficient to just integrate once.

I was imagining these two curves

They have the same area under them, so the difference in area under curve is zero, suggesting they are identical and there is no error in the model.
If you find the difference between the curves, then the area between that and the y=0 axis (ok, so perhaps I should have said integrate the absolute value of the difference between the curves), the error is significant.

@rwest got it, that makes sense!

It's by no means a perfect measure of fit. For example, these two pairs of curves would have the same "error".

But it's a start, to get the code working.

Agreed.
FYI, when I was implementing the error evaluation for perfectly stirred reactor simulations in my older MARS, which marched along the upper branch, I used three points: near extinction turning point, near equilibrium (e.g., a residence time of around 0.1 s, and the logarithmic midpoint between those two points.
The overall error was the the max of the error in residence time at the extinction turning point and the error in temperature at the other two points.

For our JSR simulations it's maybe worth reading 2E08.pdf and keeping an eye on https://github.com/Cantera/enhancements/issues/31 and https://github.com/Cantera/cantera/pull/1021

I'm resurrecting this PR because I've got a flame speed addition in the works that depends on this.
Here's what I've done:

Finished adding multiprocessing
Added handling for multiple datasets
Made a bunch of PEP8/style changes because I can't help myself
Refactored the simulation files back into a single simulation.py file

Here's the remaining work:
[ ] Get existing tests back up and running
[ ] Add JSR tests
[ ] Refactor jsr_eval_model into eval_model
Here's what I don't plan to do for this PR:

Improve the error calculations. I made a curve_matching branch that can (in theory) handle this, but it deserves its own PR
Plotting

Okay, @rwest I think this one is ready for review.


Where to go for support with Gateway issues?

This page tries to explain how to "Restore, migrate, or take over a gateway in a non-default region" and shows a screenshot that provides no indication how you get there. (maybe because I'm using a corporate account / tenant).
This page also does not mention how to change the region
https://learn.microsoft.com/en-us/data-integration/gateway/service-gateway-app

Thanks @NHasgreenb for pointing this out. We have made the change now. #please-close


GREGWT example

The GREGWT example on pg. 112 isn't running.
data_in <- prepareData(cbind(age, sex), ind, census_area_id = F, breaks = c(3))
Error in [.data.frame(Tx, names(Tx)[i:b]) : undefined columns selected

Any ideas @emunozh ? Just reproduced this on a new system starting from scratch. Thank for reporting @tyokota and sorry for the slow response.

As a reference I am including the example here from 06-alternative-approaches.Rmd
# Install GREGWT (uncomment/alter as appropriate)
# devtools::install_github("emunozh/GREGWT")
# load the library (0.7.0)
library('GREGWT')

# Load the data from csv files stored under data
age = read.csv("data/SimpleWorld/age.csv")
sex = read.csv("data/SimpleWorld/sex.csv")
ind = read.csv("data/SimpleWorld/ind-full.csv")
# Make categories for age
ind$age <- cut(ind$age, breaks=c(0, 49, Inf),
               labels = c("a0.49", "a.50."))
# Add initial weights to survey
ind$w <- vector(mode = "numeric", length=dim(ind)[1]) + 1

# prepare simulation data using GREGWT::prepareData
data_in <- prepareData(cbind(age, sex),
                  ind, census_area_id = F, breaks = c(3))

There have been some modifications in the GREGWT code. Plese note that the example listed above referes the reader to the library version 0.7.0. Plese make sure that you are uning the right version. I just run the code with 0.7.0 and it runs. Let me know if you need version 0.7.0. Here is the output of my session:
> sessionInfo()
R version 3.3.1 (2016-06-21)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Arch Linux

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] GREGWT_0.7.0   nvimcom_0.9-21

loaded via a namespace (and not attached):
[1] tools_3.3.1

Now, if you want to run it with the lattest version (0.7.3 just updated the repo) you have to make a small modification to the code:
# prepare simulation data using GREGWT::prepareData
data_in <- prepareData(cbind(age, sex),
                  ind, census_area_id = F, breaks = c(2))

It was confusing why the brak was on 3 (it assumed a column of area ID), now the break is on column 2, as it should be.
Plese let me know if you have other difficulties with the library. I haven`t been able to take the library out of beta...


CI is broken

@jamwaffles Do you have any idea why CI isn't working at the moment? It did work correctly in the PR, but fails on the master branch. At first I suspected this might be a temporary problem, but it seems to be persistent.

Very strange... the deploy key CircleCI uses vanished from Github's side. I've reauthed through Circle's UI and it looks like it's working now. Thanks for letting me know.

Thanks for letting me know.

Thanks for fixing it so quickly.

Any time :)


Clearing storybook

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@helen-v Is this nearing ready to review? These updates will be nice to have to incorporate into upcoming components!


Add DefaultFrameworks field to InstallationData struct

User description
This will allow the operator to receive the default frameworks as a configuration

PR Type
Enhancement

Description

Added new DefaultFrameworks field to InstallationData struct to allow configuring default frameworks via operator
Field is defined as string array with proper JSON/BSON serialization tags
Includes omitempty option to maintain backward compatibility

Changes walkthrough 📝
Relevant filesEnhancement

portaltypes.goAdd DefaultFrameworks configuration field to InstallationData

armotypes/portaltypes.go
Added DefaultFrameworks field to InstallationData struct as a string array  Field includes JSON and BSON tags with omitempty option

+1/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

@slashben can you pls review


Start using Cap'n Proto instead of Google Protobuf

Start using Cap'n Proto instead of Google Protobuf for client-server communications. This will speed up visualization and reduce memory consumption.

I could not find supported stable cap'n proto library for browser javascript:
Here are all options I could find:

capnproto/node-capnp only for node.js
jscheid/capnproto-js outdated
jdiaz5513/capnp-ts unstable + it's typescript version, so it will add extra complexity to the plugin

Next option to check is FlatBuffers

FlatBuffers is much faster.
It takes ~800ms to create and serialize a tree with 524288 nodes with flatbuffers, and ~3s with protobuf.
Although resulting file is bigger 50Mb vs 20Mb. But disk space should not be a problem and serialization/deserialization time is much more important.
Also FlatBuffers should reduce runtime memory consumption.


Check/imporove: Installation: Does installation proceed as outlined in the documentation?

from https://joss.readthedocs.io/en/latest/review_checklist.html

https://heudiconv.readthedocs.io/en/latest/installation.html

[ ] install via pip
[ ] install via neurodebian?
[ ] use with docker
[ ] build singularity container from docker
[ ] TODO add developer install

TODO add link to README

pip install works, though it currently does not install dcm2niix or DataLad.

[Ubuntu 22.04] Using Neurodebian wget -O- http://neuro.debian.net/lists/jammy.us-tn.full
sudo apt-get install heudiconv succeeds
$ heudiconv --version
0.11.6

Docker pull and run nipy/heudiconv:latest appears to work, but install instructions do not include a sanity check.

Singularity pull and run appears to work , but output returns usage help

closed by https://github.com/nipy/heudiconv/pull/640


jax2tf: support mixed-precision inputs to dot_general

This is the issue that led to the rollback in https://github.com/google/jax/pull/17150, because jnp.einsum started to lower to mixed-precision dot_general and broke downstream code.
Minimal repro:
import jax
import numpy as np
from jax.experimental import jax2tf

x = np.ones(4, dtype=bool)
y = np.ones(4, dtype='int32')

print(jax.lax.dot(x, y))
# 4

print(jax2tf.convert(jax.lax.dot)(x, y))
# InvalidArgumentError: Value for attr 'LhsT' of bool is not in the list of allowed values...

Depending on the serialization mode, this may also affect other mixed types; e.g.
x = np.ones(4, dtype='int8')
y = np.ones(4, dtype='float32')

print(jax.lax.dot(x, y))
# 4.0

print(jax2tf.convert(jax.lax.dot, enable_xla=False, native_serialization=False)(x, y))
# InvalidArgumentError: Value for attr 'T' of int8 is not in the list of allowed values:...

There seem indeed to be a problem with the non-native jax2tf, but starting with #16749 most people use native serialization, which does not have this problem. (@jakevdp please check your local setup for running the above example if the jax2tf_default_native_serialization config flag is set to True).
I might not fix the non-native serialization behavior because most users should switch aware from it (will investigate the internal test failures).

Thanks for the fix @necula!
It looks like there's one more related issue where jax2tf errors on code that executes correctly in XLA. The issue arises in non-native serialization when preferred_element_type is narrower than the inputs:
import jax
from jax.experimental import jax2tf

x = jax.numpy.ones(4, dtype='int32')
y = jax.numpy.ones(4, dtype='float16')

func = lambda x, y: jax.lax.dot(x, y, preferred_element_type='float16')

print(func(x, y))
# 4.0

print(jax2tf.convert(func, enable_xla=False, native_serialization=False)(x, y))
# tensorflow.python.framework.errors_impl.InvalidArgumentError: cannot compute MatMul as
# input #1(zero-based) was expected to be a int32 tensor but is a half tensor

Sorry, I just realized this new repro does not actually hit the new problem... I'll try to come up with a better one


add support to D3 i18n

Another change I needed in my project; this uses D3's tickFormat to translate x axis labels.

Can you rebase from master ?

I'll publish a new version tomorrow. This project also need a serious refactor to conform with reusable charts that d3 proposes. If you are up to it, feel free :)

Branch updated. About the refactor, I'm sorry but I don't think I'm having time for that anytime soon :/
Thanks!

Last thingy: Can you update the readme with this parameter described? Thanks

No prob. I'll update it tomorrow.

Updated.


FR: grpc_cli TLS flags: providing explicit CAs, certs, remote host names

If you --enable_ssl in grpc_cli you get default grpc::SslCredentialsOptions, which causes grpc_cli to use the system CAs and no client certificates. Furthermore, if you don't have DNS for your target (e.g., an VM-based service with an ephemeral IP address), you may need to specify grpc::ChannelArguments::SetSslTargetNameOverride in order to correctly validate the certificate.
With appropriate flags, this can allow a variety of command-line use cases involving self-signed certificates or private authorities used in testing services.
This correlates to a portion of https://issuetracker.google.com/issues/111599303.

It looks like GRPC_DEFAULT_SSL_ROOTS_FILE_PATH env variable controls alternative CAs, so that's solved already at least.

@yang-g - Not sure if you know about this. Please feel free to reassign (or unassign yourself) if you are unfamiliar with this.


Rounding to pixel grid is not dpi aware

#734 rounds sizes to integer number of px, but this is not the correct behavior when the dpi scaling factor is not 1.0.
Fixing this is not trivial. To know the dpi requires context, and this is a value that can change dynamically.
#1037 brings up much deeper architectural concerns. I chose px rather than device pixels as the units for layout because it's meaningful from a design perspective, and because it minimizes the chance of user layouts changing unintentionally with dpi, but it might be worth reconsidering this choice. There are a number of computations, of which this is one, and layout of windows in a virtual desktop (on Windows; mac is probably more geared to "points" than device pixels) is another, for which device pixels are probably a more natural unit of computation. Maybe we should consider using device pixels for layout, and do conversions as appropriate.
Or maybe we should keep px, but have mechanisms to round to device pixels using context.

See also #1091


Python 3.11

Could you please support Python 3.11 on PyPI?

I still have issues using ScaNN with Python 3.11

Hello, I have the same issue as well. Could you please provide support for python 3.11?


Having trouble running lats on hotpot in 'standard' prompt sample mode

Hi andyz245,
Thank you very much for your excellent work!
I have some trouble running lats on hotpot. When I run your code in the CoT prompt mode, it works fluently. However, if I switch the mode to 'standard', I find that the code will be caught in a dead loop. After some debugging, I noticed that the problem occurs at this line:
https://github.com/andyz245/LanguageAgentTreeSearch/blob/e08d36fee744c6b8edbe75c448766fb556b9f3ed/hotpot/lats.py#L305C15-L305C15
where the sampled_action fails to parse a valid action (probably because the response of GPT does not contain words such as Search[something]?) and therefore MCTS cannot successfully generate new states. Do you have any suggestions on this issue?

Hi,
Thanks for reaching out. The current code is only designed to run it in "CoT" mode, which actually corresponds to ReAct (sorry for the confusion). The standard setting corresponds to CoT, which is not supported by thee current code. If you need to run the code with CoT (and not ReAct), I can try to reimplement it.

closing this for now


Add a flag to dump all https requests please

I frequently get bad request errors from the Google API Servers.  Because all requests are HTTPS, intercepting them is a major hassle.
Please add a flag so that all HTTP requests and responses are dumped to the output window.

Request/response logging is now available, since release v1.23.0.


WEBBPSF_EXT_PATH handling when not ending in '/'

If the WEBBPSF_EXT_PATH environmental variable isn't set such that it ends in a '/', line 73 in webbpsf_ext/init.py doesn't add a '/' to the end, but changes the data path to be simply the root directory, '/'.

So, this is curious. Someone else brought this to my attention a couple weeks ago and I added / modified the code block up above line 73 that should catch invalid directories. Specifically, this line [62, 63]:
if (data_path is None) or (data_path == ''):
    raise IOError(f"WEBBPSF_EXT_PATH ({data_path}) is not a valid path! Have you set WEBBPSF_EXT_PATH environment variable?")

should catch that the path is invalid and throw an error asking you to set you env variable. So, it's a mystery to me how the code is even getting to line 73 (unless it somehow things your computer is a readthedocs servers, but in that case it would at least provide a temporary data path).
Can you provide the full error output for me?
In any case, make sure you have both WEBBPSF_PATH and WEBBPSF_EXT_PATH environment variables set.

Also, what is your operating system? I suppose it's possible that os.getenv() could be behaving differently and returning an output I'm not expecting that is OS dependent.

Oh sorry, that wasn't the line I was referring to. I can get you a more extensive error message if needed (it was on my student's computer), but the crux of the error message was this:
Unable to write file '/psf_coeffs/NIRCam' (Unknown (FileSystemError): Error: EROFS: read-only file system, mkdir '/psf_coeffs')

The WEBBPSF_EXT_PATH is being set and found by the code, so the lines that you mentioned above checking if that path is None or an empty string aren't triggering. I actually ran the os.getenv('WEBBPSF_EXT_PATH') command in the environment where we were getting the issue and it successfully retrieved the path. It's just that, at some point, it's changing it to the root directory path, '/'.
We traced it to the section below:
    if '/' not in data_path[-1]:
        # Make sure there is a '/' at the end of the path name
        data_path = os.path.join(data_path, '/')

The WEBBPSF_EXT_PATH that the system was successfully retrieving didn't end in a '/', so it was trying to add one. But os.path.join here reads the '/' that it's trying to add on the end as the root directory path, not just a slash. So just returns '/' instead of the original data_path with a '/' on the end.
>>> # for example
>>> os.path.join('my/path','/')
'/'

We were able to work around it by making sure that the WEBBPSF_EXT_PATH does end with a '/' when set, but since this section of the code looked like its purpose was making the code robust against whether WEBBPSF_EXT_PATH ended in a '/' or not, I thought I'd point it out. I think fixing it would be as simple as handing os.path.join an empty string instead of '/':
>>> os.path.join('my/path','')
'my/path/'

Fixed in v1.2.3.


The Graph sync status

Fixes #464 .
Changes proposed in this PR:

show to users the current sync status of the displayed financial data, and if it is out of sync, display a warning to users

Good, so not my bad. Was strange for me too, tried it in many ways thinking that I missed something

Pushed a fix, needs some refactoring and some cleaning but it works

How can I determine how much the Graph is out of sync, so I know when to show a warning? How much the difference between the 2 numbers should be?

In theory it should be the same block number

Now the sync is checked on component mount, network change and wallet connect/disconnect. Should I make it to check on a certain interval?
I also displayed the warning message inside the announcement banner but not sure how that should actually work. I guess we want to keep the current announcement so that adds up to 2 messages, if so, we have to differentiate them better.  And then should the announcement component support multiple messages(more than 2)? If It should than it need some changes.

We need some messages on tooltip or something. Maybe show the block difference
I have no ideea what this means.

After i wrote the comment i remembered about the top message. If i didn't remember i wouldn't have seen it. We need something to warn people.

I think that's the Announcement component's job to display the warnings. But I see your point, the warning symbol down there can be confusing. Maybe we can drop the symbol change down and only show the warning in the announcement banner.

refactored a bit, so we get a clear separation of concerns between banner, network stuff, graph sync stuff:

new atoms/AnnouncementBanner : banner component reduced to presentation only, where its content is always passed as props
revised molecules/NetworkBanner: the former AnnouncementBanner  now holds all the specific network detection logic, in the end also returns the atoms/AnnouncementBanner
new hook hooks/useGraphSyncStatus: move all the graph fetching logic in there so we can use its status in multiple places in the app without all this props passing. This also decouples the SyncStatus component in footer from its logic
in App.tsx, add the graph sync warning banner in another atoms/AnnouncementBanner, getting its values from the hook

Resulting in:

For the info in footer ended up with this:

almost everything working nicely now, refactored some of the data flow and made everything more universal. When wallet is connected, head block is always coming from web3.eth.getBlockNumber(), set early in our Web3Provider. This block number is then used by useGraphSyncStatus to figure out if additional calls are needed to get head block, so only for users without a wallet:

when on mainnet, get head block from blocklytics/ethereum-blocks graphql query
for all other networks, create new web3 instance with config.nodeUri, and use web3.eth.getBlockNumber() again

But that only works for ETH networks, cause there is no infura for polygon, so slapping the infura id at the end of the config.nodeUri does not work here.
So we might end up having to create unique solutions to fetch the head block for non-ETH networks.

phew, made sure the banner is never briefly appearing upon user network switching, trick was to work with a new web3Loading value from our Web3Provider.
Tested all combinations again, remaining issue stays Polygon without a wallet which gives us this:

It would not prevent app from working, nor would it falsely show the banner. But if the head block and subgraph are out of sync in Polygon there would be no warning banner.

The bug still appeared for me so I added a loading state for the subgraph fetching too