Week 3 - Unit 7: get_authorizations method indentation code snippet

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

Hi Jaime,
thanks for the feedback.
We will have a look at it.
Kind regards,
Carine


DevilutionX from prebuild

https://github.com/zerojay/RetroPie-Extra/blob/a8e5eebcd992d0b5691f4729d220084e4edb5713/scriptmodules/ports/devilutionx.sh#L20
Woudn't it be faster to download the prebuild armhf release? With that the dependencies would also be limsted to just libsdl2-mixer and libsdl2-ttf.
With the latest release you can also specify the save dir so you don't need to copy the dir to "$md_conf_root/devilutionx", instead just specify --save-dir like you do with --data-dir already

Not really much faster at all, to be honest. It compiles fairly quickly though compilation speed is not generally a concern of mine to begin with unless it is prohibitively long. Generally speaking, I'd prefer not to use prebuilt binaries as they require more maintenance to deal with. If you want to use them, be my guest.

Copying configs/save files to $md_conf_root is a standard for RetroPie and it's what users expect, so that's not something I'm going to change here unless I'm misunderstanding something.

I'm saying you can have the game save directly in $md_conf_root and avoid having to copy between the two folders.

Thanks, I appreciate the heads up.


Add quest requirements to diary tasks

Will add quest requirements to diary tasks.

All pre-existing problems have now been resolved and this works without problem.


Examples on how to represent filtering terms in the GET query URL

Given that, for some time, we had only managed POST operations we haven't discussed in depth how to apply different filter types when used in the query string.
Should we provide examples of them?

Yes, we should provide examples.
The scout started off by considering GET requests and most of our filters discussions with partners used GET request examples, so we do have some already documented.
Where should we add in these examples?

Wonderful!
I guess that we can add these examples to 3 different places (in order my preference):

The Filters Scout document (the natural place, out of the repos but could be linked there)
The README here (also natural, but not where the users would look at it)
The README in the Models (where more users will go, but in the topic is broader than the Beacon v2 Model, hence is not the appropriate part. We must put a link there that points to the place where the examples fit).

Best!

@tb143 I think the point 1. by @jrambla above, obviously. But we already started the implementations repo which IMO is a much cleaner place for having a readable documentation with examples.
IMO we could have all this as part of a main RTD document (e.g. also adding examples in an "Appendix" section). There is a stub of a doc here (not in the main org yet...).
Pinging @mrueda and @laurenfromont.

@tb143 @jrambla etc.: Again, bump for using the implementations repo! The registry server cannot provide this kind of documentation - this is in parallel.
Alternative - moving this to a common documentation space, where then different core implementers add their examples? What is planned with the RTD documentation?


[AERIE-1785] Provide default priorities if not provided

Tickets addressed: AERIE-1785
Review: By commit
Merge strategy: Merge (no squash)

Description
Adds functionality to the SQL triggers from #123 to allow inserts to not provide a priority, in which case the trigger will select the next consecutive priority for the relevant specification or template.
Verification
I added a test to db-tests for this.
Documentation

Future work

Currently not sure if this will pass tests; for some reason I can't get the e2eTests to run locally like I did before. Using this PR as testing.

I started with a fresh docker but I was unable to run the test? I could be doing something wrong here.
Execution failed for task ':db-tests:test'.

No tests found for given includes: gov.nasa.jpl.aerie.database.SchedulerDatabaseTests

@goetzrrGit What command are you using? For some other reason I wasn't able to test this locally, so I just relied on the PR workflux. Also @camargo will probably know about this better.

I started with a fresh docker but I was unable to run the test? I could be doing something wrong here.
Execution failed for task ':db-tests:test'.

No tests found for given includes: gov.nasa.jpl.aerie.database.SchedulerDatabaseTests

Yeah, I believe we renamed that one to be :db-tests:e2eTest instead of test. The thought process there was that e2eTest expects a deployment of aerie to be available.

@goetzrrGit What command are you using? For some other reason I wasn't able to test this locally, so I just relied on the PR workflux. Also @camargo will probably know about this better, if this is an e2e-test thing.

I tried to run it in IntelliJ but if this a GitHub action thing then you can ignore my attempts to run the test locally.

@goetzrrGit What command are you using? For some other reason I wasn't able to test this locally, so I just relied on the PR workflux. Also @camargo will probably know about this better, if this is an e2e-test thing.

I tried to run it in IntelliJ but if this a GitHub action thing then you can ignore my attempts to run the test locally.

@goetzrrGit You should be able to run ./gradlew :db-tests:e2eTest. I hadn't considered the IntelliJ user case, might be worth looking into how to configure that in the future

@goetzrrGit What command are you using? For some other reason I wasn't able to test this locally, so I just relied on the PR workflux. Also @camargo will probably know about this better, if this is an e2e-test thing.

I tried to run it in IntelliJ but if this a GitHub action thing then you can ignore my attempts to run the test locally.

@goetzrrGit You should be able to run ./gradlew :db-tests:e2eTest. I hadn't considered the IntelliJ user case, might be worth looking into how to configure that in the future

ohhh, got it, thanks @mattdailis


[flake8-simplify] Include caveats of enabling if-else-block-instead-of-if-exp (SIM108)

Summary
This aims to acknowledge the existing issues (#14017, #5528) and help users decide whether to enable this rule.
Test Plan

Awesome, thank you


Changed code-of-conduct link in navbar

Adressing #414
Appliedd the various modifications as suggested here

changed code of conduct link in navbar with the link to code-of-conduct in policies repository
added a small sentence message in CODE_OF_CONDUCT.md as suggested here

The checks have failed. Could you check and fix the errors?

I verified in the netlify logs! This is what is causing the issue

Any idea of How to go about it ?

The checks have failed. Could you check and fix the errors?

I verified in the netlify logs! This is what is causing the issue

Any idea of How to go about it ?

Or maybe I should another pull request and close this one?

The checks have failed. Could you check and fix the errors?

I have fixed the other checks but this one is persisting!

The reason is.... The funders.md file contains an image which has a HTTP link. Having a HTTP S link will solve the problem
What should I do @bebatut

You still need to remove the code_of_conduct.md file but could you change the link in the _posts/2020-10-01-annual-report-part-1.md
The other error about the mixed content, I can fix it in another PR

You still need to remove the code_of_conduct.md file but could you change the link in the _posts/2020-10-01-annual-report-part-1.md The other error about the mixed content, I can fix it in another PR

Which link should I change there ?
This is becoming really consufing!
I can close the two PR and restart from scratch in one Pull Request....
What do you think about this ?

When I try to combine the two PRs, into one.... It is prompting me to CREATE A PULL REQUEST
Seems like it is not possible to combine pull requests @bebatut

Yep start again from scratch it will be much cleaner.
The idea,

go to the main branch
update it
create a new branch
make all changes related to the code of conduct in the branch
push to GitHub
open 1 PR

For the link in the _post file, you need to change it to link to the code of conduct in the Policy GitHub repository, same link as in the navbar

Yep start again from scratch it will be much cleaner.
The idea,

go to the main branch
update it
create a new branch
make all changes related to the code of conduct in the branch
push to GitHub
open 1 PR

I am working on it NOW 🦸
Thanks very much for your guidance and support @bebatut
I'm grateful...☄


feature/sync-title-field

Removing the HTML input field for title in edit entry. For each document-type document on field property has an attribute isTitle. That means that will copy every change to the blog.data.title to blog.title, because every document must have a title. So every document type must declare 1 data field that will function as a title

I'm only thinking of what to do if it's not only the input-text field that is configured as a title (for example what if a date or something makes more sense for some types). But let's worry about that if we encounter it. This looks good for now.


Xamarin.Auth + TLS 1.2 (CloudFlare SSL)

I'm having issues authenticating to TLS 1.2 HTTPS sites using Android. I get the following error:
ERROR Error: SecureChannelFailure (The authentication or decryption has failed.)
I use Xamarin.Auth 1.3 .
On iOS I was able to bypass the problem by selecting:

HttpClient implementation: NSUrlSession
SSL/TLS implementation: Apple TLS
in the project options.

Are there Android specific options also to support TLS 1.2?

This issue appears resolved with Xamarin.Auth 1.3.1 alpha-2 and using the XA_HTTP_CLIENT_HANDLER_TYPE environment variable:
https://developer.xamarin.com/guides/cross-platform/transport-layer-security/

Can you check 1.3.1 alpha-03 please and confirm if it is solved, so this issue can be closed.
thanks

Xamarin.Auth is not really at fault here. To get it to work with TLS 1.2 and Android you need to set:

XA_HTTP_CLIENT_HANDLER_TYPE environment variable: https://developer.xamarin.com/guides/cross-platform/transport-layer-security/
Select "AndroidClientHandler" in HttpClient implementation in the XS Android Build options.
If using the Linker, you need to add "using Xamarin.Android.Net;" somewhere in your Android project so that the linker does not remove this library.


Update dependency mkdocs-mdpo-plugin to v0.0.16

This PR contains the following updates:

Package
Type
Update
Change

mkdocs-mdpo-plugin
extra
patch
==0.0.14 -> ==0.0.16

Release Notes

mondeja/mkdocs-mdpo-plugin
v0.0.16
Compare Source

Added ignore_msgids setting to add a list of messages that shouldn't be dumped inside PO files.

v0.0.15
Compare Source

Refactored plugin.
Don't write empty compendiums.
Add more test cases.

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 95.084%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
851

Relevant Lines:
895

💛  - Coveralls


hiredis crashes when unix socket connection is used

Hi.
I'm using latest hiredis library on my C application.
The app crashes under the following steps:

unix socket is used to connect to redis server
The redis server is then killed
Now, when a command is sent to the redis pipeline (with redisAppendCommand), the app crashes.

The stacktrace shows __redisAppendCommand at the top of the stack (hiredis.c: 910).
It seems that the call to sdscatlen is failing.
My program is explicitly ignoring SIGPIPE signal, that will be sent to it when the remote end on the unix socket is killed.
By the way, same crash happens if I use a TCP socket connection with the redis server. If the redis server is down, the app crashes when calling redisAppendCommand at hiredis.c: 910 line.
Any idea?
Thanks.
-Bob

Please add additional information

Code to reproduce
Backtrace of failing programm

The backtrace of the failing program, using valgrind, is as follows.
==4251== Invalid read of size 8
==4251==    at 0x54690A4: __redisAppendCommand (hiredis.c:910)
==4251==    by 0x546912B: redisvAppendCommand (hiredis.c:942)
==4251==    by 0x5469226: redisAppendCommand (hiredis.c:956)
==4251==    by 0x402F5E: processPkt (main.c:599)
==4251==    by 0x403284: cb (main.c:715)
==4251==    by 0x5056EBA: ??? (in /usr/lib/x86_64-linux-gnu/libnetfilter_queue.so.1.3.0)
==4251==    by 0x525F376: nfnl_handle_packet (in /usr/lib/x86_64-linux-gnu/libnfnetlink.so.0.2.0)
==4251==    by 0x404624: main (radius.c:1162)
==4251==  Address 0x90 is not stack'd, malloc'd or (recently) free'd
I have tried to reproduce the problem with a minimal program, but I am not able to reproduce it, to be honest. It seems that the command is sent to the output buffer without any problem, even if the redis server is down.
The minimal program that I have used is:
#include <stdio.h>
#include <stdlib.h>
#include <signal.h>
#include <unistd.h>
#include <hiredis/hiredis.h>
int main(int argc, char **argv)
{
signal(SIGPIPE, SIG_IGN);
signal(SIGHUP, SIG_IGN);
struct timeval timeout = {0, 500000}; // 0.5 seconds
redisContext *redisctxt = redisConnectUnixWithTimeout("/tmp/redis.sock", timeout);
if (redisctxt == NULL || redisctxt->err) {
if (redisctxt) {
printf("Redis connection error: %s\n", redisctxt->errstr);
redisFree(redisctxt);
} else {
printf("Redis connection error: can't allocate redis context\n");
}
exit(1);
} else {
printf("Redis connection started\n");
}
printf("Waiting 20 secs to give you time to kill the redis server ...\n");
sleep(20);
/* Send command to the redis pipeline */
redisAppendCommand(redisctxt, "SET key1 value1");
printf("Sent command to the redis server ...\n");
redisReply *reply;
if (redisGetReply(redisctxt, (void **)&reply) == REDIS_OK) {
printf("Successful Redis Reply: %s\n", reply->str);
freeReplyObject(reply);
} else {
printf("Unsuccessful Redis Reply\n");
}
exit(0);
}
It seems to be a problem on my program itself.
But it only shows up when connection is lost with the redis server. And valgrind always says that redisAppendCommand is on top of the stack.
I will try to investigate it further.
Thanks.
-Bob

The problem seems to be in redisAppendCommand, that is before the socket is touched at all (data is only written on redisGetReply).
Looks like the context (and its buffer) is destroyed for some reason.
Are you using multiple threads by any chance? Is the context object otherwise concurrently accessed?
Without a way to reproduce there's not much I do here.

Well, I have a main thread, running the core of the application, and an auxiliary thread, which is in charge of trying to reconnect to redis server from time to time.
But the thing is that, once I kill the redis server, the only way my app has to know about it, is by issuing some command and getting an error (on the main thread). When this error is returned to my app, the redis context is freed, and the auxiliary thread is started in order to try to get another one once the redis server comes back to life.
From what you say, I have to doublecheck that the redis context is in good shape when the redisAppendCommand is issued. At first sight, I would say that it should be ok, since my app creates it at the very beginning successfully, and when I kill the redis server, my app has not been "notified" about it yet, and so, the redis context is not freed and it is still considered the one to be used.
I will investigate and let you know.
Thanks a lot.
-Bob

OK, there are two things I have observed:

I have removed all the places in my code where the redis context is freed.
And the problem is still there, the app crashes at the same point.
So, we can discard the assumption that the redisctxt has somehow been already freed when the redisAppendCommand is called.

It seems that the 'obuf' field inside the redis context is used to contain the formatted command to be appended. I can see its content, and it is the original value I sent, but with some extra characters, which I suppose is the redis format itself.
But the strange thing is that this obuf pointer changes whenever I call redisAppendCommand. I'm talking about the pointer value.
Is that normal?
I presumed that only the contents would change, but not the address of the first character of the string.

thanks.
-Bob

I presumed that only the contents would change, but not the address of the first character of the string.

The buffer might be reallocated, and thus might change.

Closing this issue .....
I have found where the problem was.
@badboy, you were right, the redis context had been already freed when redisAppendCommand was being called.
I do not understand why the problem persisted when all redisFree calls were commented out, but that is a different thing.
Thanks a lot for your help.
-Bob


Add command to display tags

This closes #299.

Coverage increased (+0.02%) to 94.492% when pulling 2dc68d99c71d0a459c2fa4f69363277f8151188c on drdavella:schema-command into e3b0eae3d48830848a60db988aec8e40a67ea096 on spacetelescope:master.


logical quotas are vulnerable to changes by the collection owner

I had to set the quota using an owner of the collection. How can I set a quota that a regular user can't just alter themselves? I imagined having an admin user be the owner of the top-level collection and grant more restrictive ACLs for end-users, but I'm wondering if a user has to be able to update at least the current size values for this to work at all. If that's the case I'm concerned that the end user could just alter the current sizes themselves to bypass quota enforcement.

Closing - I believe this is now handled cleanly by the implementation at https://github.com/irods/irods_rule_engine_plugin_logical_quotas
If not, please open an issue over there...


Refactoring search pipeline for HNSW and inverted dense

Refactored to create HnswDenseSearcher and InvertedDenseSearcher - these will provide bindings from Python
SearchHnswDenseVectors and SearchInvertedDenseVectors wrapper the two searchers above to provide main
Improved test coverage

@tteofili please review when you have a chance - I'll run tests in parallel.


Hue

Is it possible to add Hue to this cluster?

Yes, I'm glad to accept pull requests! Should be a breeze, considering HiveServer2 Thrift server port is already exposed.


Authority Key Identifier reported as missing even though it's present

When calling the verify command on this root certificate and its intermediate certificate with the strict option, the authority key identifier of the intermediate is reported as missing even though it is present in the certificate:
$ openssl3 verify -CAfile ./R6.pem -x509_strict ./I6.pem
C = DE, ST = Germany, L = Berlin, O = mendelson-e-commerce GmbH, OU = (c) 2016 mendelson-e-commerce GmbH - for authorized use only, CN = mendelson Public Primary Certification Authority - I6
error 85 at 0 depth lookup: Missing Authority Key Identifier
C = DE, ST = Germany, L = Berlin, O = mendelson-e-commerce GmbH, OU = (c) 2016 mendelson-e-commerce GmbH - for authorized use only, CN = mendelson Public Primary Certification Authority - R6, emailAddress =<EMAIL_ADDRESS>error 86 at 1 depth lookup: Missing Subject Key Identifier
error ./I6.pem: verification failed

Output when inspecting the intermediate certificate:
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number: 42 (0x2a)
    Signature Algorithm: sha1WithRSAEncryption
        Issuer: C=DE, ST=Germany, L=Berlin, O=mendelson-e-commerce GmbH, OU=(c) 2016 mendelson-e-commerce GmbH - for authorized use only, CN=mendelson Public Primary Certification Authority -<EMAIL_ADDRESS>        Validity
            Not Before: Jun 29 11:05:09 2016 GMT
            Not After : Jan  1 01:01:01 2026 GMT
        Subject: C=DE, ST=Germany, L=Berlin, O=mendelson-e-commerce GmbH, OU=(c) 2016 mendelson-e-commerce GmbH - for authorized use only, CN=mendelson Public Primary Certification Authority - I6
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
                RSA Public-Key: (2048 bit)
                Modulus:
                    00:df:fa:5b:c4:d6:f1:86:07:80:ac:47:c5:78:3a:
                    0b:2d:69:65:18:57:1e:be:96:06:a6:94:c1:38:25:
                    34:17:fb:f3:cc:b7:5c:97:5a:1b:f7:b5:93:06:b0:
                    32:e6:00:74:e6:16:b7:cb:26:db:16:26:a6:62:68:
                    ad:c6:3b:9b:88:13:75:cb:8c:1f:c7:c2:6a:b0:ec:
                    45:57:53:d8:13:17:c5:9f:49:4c:81:b8:13:d6:1f:
                    97:33:90:32:f3:a7:fa:d6:7a:53:23:e4:81:b7:d7:
                    db:37:ca:c9:d4:87:e4:52:62:63:7d:50:35:b0:5b:
                    dc:c2:35:cc:01:54:60:2d:e6:aa:63:04:82:b1:f0:
                    9b:9a:93:16:b6:1b:a7:7b:fc:64:19:c1:89:64:51:
                    f4:7e:f6:21:a5:75:66:31:8f:f7:41:f2:87:ac:3f:
                    a5:08:e8:f9:32:67:2c:89:25:2d:a4:ef:e8:27:7b:
                    d1:25:86:86:78:49:c1:d3:cf:ff:d6:2f:cd:f4:97:
                    b7:db:16:f3:08:fa:3e:45:d3:d9:ca:96:d3:ca:38:
                    86:49:9c:73:de:4c:a0:87:6c:1f:aa:de:f8:6b:f4:
                    77:7a:ad:d5:ad:f2:32:71:1d:1c:71:7c:83:eb:74:
                    3f:65:5e:cc:41:7e:93:43:1d:89:bf:37:1c:2f:10:
                    01:a3
                Exponent: 65537 (0x10001)
        X509v3 extensions:
            X509v3 Basic Constraints: critical
                CA:TRUE
            X509v3 Authority Key Identifier:
                DirName:/C=DE/ST=Germany/L=Berlin/O=mendelson-e-commerce GmbH/OU=(c) 2016 mendelson-e-commerce GmbH - for authorized use only/CN=mendelson Public Primary Certification Authority -<EMAIL_ADDRESS>                serial:C8:2C:93:05:F0:7F:53:0B

            X509v3 Subject Key Identifier:
                6C:F1:70:37:E5:13:46:FC:5D:99:E8:5A:4A:0A:E8:B6:B9:AB:32:FE
            X509v3 Key Usage: critical
                Digital Signature, Certificate Sign, CRL Sign
            X509v3 CRL Distribution Points:

                Full Name:
                  URI:http://ca.mendelson-e-c.com/mendelson6.crl

OpenSSL version used:
$ openssl3 version -a
OpenSSL 3.0.7 1 Nov 2022 (Library: OpenSSL 3.0.7 1 Nov 2022)
built on: Tue Nov  1 14:14:36 2022 UTC
platform: darwin64-x86_64-cc
options:  bn(64,64)
compiler: clang -fPIC -arch x86_64 -O3 -Wall -DL_ENDIAN -DOPENSSL_PIC -D_REENTRANT -DOPENSSL_BUILDING_OPENSSL -DNDEBUG
OPENSSLDIR: "/usr/local/etc/openssl@3"
ENGINESDIR<EMAIL_ADDRESS>MODULESDIR<EMAIL_ADDRESS>Seeding source: os-specific
CPUINFO: OPENSSL_ia32cap=0x7ffaf3bfffebffff:0x40000000029c67af

To me it looks like it should complain about the root not having a subject key identifier (which it does), but the intermediate should be fine. The RFC states that it is fine to construct the authority key identifier from subject and serial number which is the case here.

The error is reported also in case when x->akid->keyid == NULL which is incorrect - we should check also whether there is the issuer/serial pair in akid. Only when it would be completely empty we should fail this.
IMO fairly minor but nevertheless a bug.


Generalize production of Ergo runtime

The Ergo runtime is a patched-up version of the Q*cert one. It is composed of vanilla JavaScript functions without module layout for node consumption. We should improve upon this in several ways:

[x] Automate the production of the runtime from Q*cert itself so changes/fixes there are propagated to Ergo automatically
[x] Allow the runtime generation to be parameterized by foreign type libraries (e.g., various support for dateTime)
[x] Generalize the runtime generation so it works for both browser and node

Should be best on new JavaScript backend, see #191

Most of this is done in release-1.0 branch. One more item to add before we can close this issue:

[ ] Do not duplicate the Qcert runtime in ./runtime/javascript, but import it from the now published Qcert library


Is it possible to get custom installed apps?

I am looking to install a custom app but  I cant seem to use the install command in commands (terminal), this would greatly help me as I wont want to have to write the code for all the apps I use and I need to install specific dependencies for my projects I am trying to launch in prozilla OS for testing.

Hey @ImAGoofyGoober-YourAGoofyGoober, there is now a guide on how to install custom apps in ProzillaOS: https://os.prozilla.dev/docs/guides/custom-app
It involves some coding but it allows you to turn any React.js app (or any website, using an iframe) into a ProzillaOS app.


Realtime Not Working

The default realtime deployment isn't working. All other services are working fine, except realtime where I'm getting errors like the following:

Do I have to do any extra configuration for realtime? Would love to have a fix here asap.

It seems it's trying but failing to connect to itself? The unable to connect to is its own IP.

Also having issues with this

My database doesn't have a _realtime schema, only a realtime schema, and it has this. Nowhere in my database is tables for broadcasts / channels / presences / subscription, like how supabase.com has.

I am dealing with this too. In the tenants table I found that my tenant was called realtime-dev-tenant, and the code was looking for realtime-dev
I updated this code in supabase-stack

And now I am getting a signature error in the logs, so maybe a step in the right direction:

Still debugging but thought I'd share in case it helps someone sort this out. Thanks

I am dealing with this too. In the tenants table I found that my tenant was called realtime-dev-tenant, and the code was looking for realtime-dev
I updated this code in supabase-stack
And now I am getting a signature error in the logs, so maybe a step in the right direction:
Still debugging but thought I'd share in case it helps someone sort this out. Thanks

If you clear out the requirements schema, then update the version of realtime in the stack to the latest image: ecs.ContainerImage.fromRegistry('public.ecr.aws/supabase/realtime:v2.28.34'), and redeploy, it'll correctly insert as realtime-dev instead of realtime-dev-tenant

Hey @mats16 would love some help here 🙏❤

Update: The self host docker seems to have the same problem by default. Weirdly, the cli docker works fine. Looking into this.

Now, the websocket for realtime fails with a 426 response. For example, using websocat
websocat "ws://MYPUBLICIP/realtime/v1/websocket?apikey=XYZ&vsn=1.0.0"
websocat: WebSocketError: WebSocketError: Received unexpected status code (426 Upgrade Required)
websocat: error running

And kong logs at the same time
"GET /realtime/v1/websocket?apikey=XYZ&vsn=1.0.0 HTTP/1.0" 426 0 "-" "-" kong_request_id: "XYZ"

Using websocat on a project on supabase.com, it connects correctly.

Am also now getting these logs in my realtime
@dubscode How goes over there? Did healthcheck improve your issue?

Now, the websocket for realtime fails with a 426 response. For example, using websocat
websocat "ws://MYPUBLICIP/realtime/v1/websocket?apikey=XYZ&vsn=1.0.0"
websocat: WebSocketError: WebSocketError: Received unexpected status code (426 Upgrade Required)
websocat: error running

And kong logs at the same time
"GET /realtime/v1/websocket?apikey=XYZ&vsn=1.0.0 HTTP/1.0" 426 0 "-" "-" kong_request_id: "XYZ"

Using websocat on a project on supabase.com, it connects correctly.

 Wish realtime would've had better logs for this, but after I fixed Nginx this is up and running. Now it's just a matter of merging #95

@ConProgramming Hope you doing well mate, was there any official resolution to this? I tried to follow the JSON for the health check on another issue you posed in but it seems like the issue still persists? been battling it for a full day now.


setup test should throw if nose isn't installed

Currently, running 'setup test' just makes a Command that has no arguments and does nothing if nose isn't installed. It should raise an exception in the arguments-parsing stuff to let the user know. Right now, it just complains because the command line arguments are missing, which is difficult to diagnose.

@ericvaandering @ticoann do you guys know if that's still the case?

I dunno, but this is in my wheelhouse now.

It still is the case

--
Sent from Gmail Mobile

@ericvaandering, Is this still the case?

Nothing changed since 2015 on this.


[CVP-1644] - get replicaset logs for operators that fail olm-deployment

Adding a task that saves the output of the command:
oc get replicasets -o go-template='{{range .items}}{{.status.conditions}}{{"\n"}}{{end}}' -n $NAMESPACE
(adjusted for usage in Ansible) to the deploy-olm-operator role to create more explicit way of debugging issues that occur when olm-deployment test fails and pod doesn't get created.

/retest

Seems like there is currently error with a server
https://jenkins-cvp-stage-ci-operatortestplaybooks.cloud.paas.psi.redhat.com/job/cvp-redhat-operator-bundle-image-validation-test/11/console
Getting 500 - server error.

/retest

LGTM


Add iamhsa/pkenv

https://github.com/iamhsa/pkenv

Thank you.
I'm working on it.
Maybe you're aware, but you can manage packer with aqua too.
aqua g -i hashicorp/packer

I see.
I created a pull request to support pkenv.
I noticed the issue that every time you update pkenv you need to install Packer.
Please see the pull request description. https://github.com/aquaproj/aqua-registry/pull/20287#issue-2151925094

v4.141.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.141.0

Just curious, when I try using aqua with tfenv or pkenv, and following this guide:
https://aquaproj.github.io/docs/guides/build-container-image#remove-aqua-from-image
The tfenv and pkenv commands are in lib and libexec. Is there a way to handle this?

https://aquaproj.github.io/docs/guides/build-container-image/#notes-of-aqua-cp
aqua cp doesn't work for tfenv and pkenv well.
So if you want to install them in Docker images and don't want to keep aqua in images,
we don't recommend aqua to install them.

I might work on an option for cp for this kind of thing.

Thank you.
If you come up with an idea about it, please open a discussion.
https://github.com/orgs/aquaproj/discussions


Mark flutter_gallery__preview_dart_2_bulid unflaky

This was fixed in #13150.
That commit rolled the engine to:
flutter/engine@93b21795971357edbfd646f61b8d7f62388b0dae,
which rolled Dart to:
dart-lang/sdk@70e5deacb54aea295665215837adaedd3d6a5bfa
which includes:
dart-lang/sdk@d38c08d97358ebe4ee30e557bfee339dddda9b7a
Which fixes the breakage introduced in #13134.
This reverts commit f5d9c7775dabc9f3e78b87b4ea8679edcae73b36.


tests(api) run kong with a single worker

ensure kong runs with a single worker; resolves
worker consistency issues, decreasing flakiness

:tada: This PR is included in version 4.18.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Conv2DTranspose in Hexagon delegate crash

System information

Host OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
MacOS, Ubuntu...
TensorFlow installed from (source or binary):
source
Tensorflow version (commit SHA if source):
Tag v2.5.0
Target platform (e.g. Arm Mbed OS, Arduino Nano 33 etc.):
Android

Describe the problem
When I run my tflite quantized model on my mobile device's DSP, I get a crash:
ERROR: Bias/channel scales number mismatch for bias tensor: model/conv2d_3/BiasAdd;model/conv2d/BiasAdd/ReadVariableOp/resource;model/conv2d_3/Conv2D
Segmentation fault
The crash occurs in Hexagon delegate, in the recently added code related to Transposed Convolution with bias.
The crash does NOT occur if I set use_bias=False to Conv2DTranspose
The error log comes from the ProcessPerChannelQuantizedBias() method in conv_2d_helpers.cc file.
See attached a small sample code that creates a model reproducing the crash.
model.zip
inputs = keras.layers.Input(shape=(128, 128, 1))
x = keras.layers.Conv2D(96, kernel_size=2, strides=2, padding='valid')(inputs)
x = keras.layers.Conv2D(96, kernel_size=2, strides=2, padding='valid')(x)
x = keras.layers.Conv2D(96, kernel_size=2, strides=2, padding='valid')(x)
x = keras.layers.Conv2D(96, kernel_size=2, strides=2, padding='valid')(x)
outputs = keras.layers.Conv2DTranspose(16, kernel_size=2, strides=2, use_bias=True)(x)
model = keras.Model(inputs=inputs, outputs=outputs)
The crash can be easily reproduced with the benchmark tool:
> benchmark_model --graph=model.tflite --input_layer=normalized_input_image_tensor --input_layer_shape=1,128,128,1 --warmup_runs=100 --num_runs=1000 --run_delay=0 --num_threads=4 --use_hexagon=true --enable_op_profiling=false
STARTING!
Log parameter values verbosely: [0]
Min num runs: [1000]
Inter-run delay (seconds): [0]
Num threads: [4]
Min warmup runs: [100]
Graph: [model.tflite]
Input layers: [normalized_input_image_tensor]
Input shapes: [1,128,128,1]
Enable op profiling: [0]
#threads used for CPU inference: [4]
Use Hexagon: [1]
Loaded model model.tflite
INFO: Initialized TensorFlow Lite runtime.
INFO: TfLiteHexagonDelegate delegate: 5 nodes delegated out of 5 nodes with 1 partitions.
ERROR: Bias/channel scales number mismatch for bias tensor:  model/conv2d_3/BiasAdd;model/conv2d/BiasAdd/ReadVariableOp/resource;model/conv2d_3/Conv2D
Segmentation fault

@srjoglekar246 could you take a look at this?

Hi, any workaround to propose in the meanwhile ? Thanks.

@twkx  Can you share how you quantized the model?

(Maybe not the actual data you used to quantize, but just the code snippet)

@twkx Can you share how you quantized the model?

The full code is in the zip file attached in issue description and runs in standalone.

(Maybe not the actual data you used to quantize, but just the code snippet)

See the model.py file in the model.zip file attached previously.
Thanks for the help

This is an issue on our side, I will ping back here once I have a fix.


Revert change from 0f1d011

Change in 0f1d011 broke th:replace, [[#{some.message(arg)}]] and structureHandler.replaceWith in Thymeleaf

Coverage remained the same at 60.292% when pulling 1412493e856433523b4b44d4b51683120902573c on sbearcsiro:fix-template-handler-breaking-replace into 3ff03f65a76692125669747f0bade7176dfd5795 on apereo:master.


Implement cfscrape for Crunchyroll extractor and check version for cfscrape

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[ ] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[ ] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
I not find which bug is now related to crunchyroll 503 because the previous one was closed.
I just reimplement back what I did before (https://github.com/rg3/youtube-dl/pull/11730) and added the cfscrape version check.
Cheers

https://github.com/rg3/youtube-dl/pull/17120#issuecomment-409281371


Add Anchored component

Usage:
import A from "@components/Anchored";
foo

Note that Markdown cannot be used inside. The bold style can be turned off with bold={false}.
See #157

https://deploy-preview-280--reverent-edison-2864ea.netlify.app/authoring/guidelines/kata#.novelty

Yay great! IT looks soooo nice, but some questions from me:

Does it have to be styled with a red squiggle? For a moment I thought I am looking at my browser's spell checker being triggered by something i the new docs. If it has to be a squiggle, I'd make it not as similar to existing things (i.e. browser spellcheck mark), or maybe highlighting similar to the one from Chrome (https://www.theverge.com/2020/6/4/21280115/google-search-engine-yellow-highlight-featured-snippet-anchor-text)
There's a mark to click on to grab the link, cool!
Would the anchor work when empty, just ID without text? I get nothing would get highlighted then, but would it still be possible to be linked to it, and grab its link?
I think what when not triggered, initial styling should be neutral. Styling could still be driven by Markdown  outside of the anchor, right?

Does it have to be styled with a red squiggle?

No, it's just using the primary brand color at the moment. The wavy line was chosen because it's rarely used on web and stands out while not being too much. I don't have a strong opinion, we can try other styles.

Would the anchor work when empty, just ID without text?

Yeah

I think what when not triggered, initial styling should be neutral. Styling could still be driven by Markdown outside of the anchor, right?

Oh, I didn't realize that's valid. I'll change it.

Yellow/Amber on dark mode makes it difficult to read, so:


PP-3764 Cypress testing - session handling

##WHAT
Includes Cypress.io plugin and scaffolding to 'inject' a logged in user session via the use of some injected cookies that bypass a login flow and take the user to authenticated pages, based on stubs provided by contract/pact.
New pacts/interations have been added on connector to return a gateway account with ID = 666, as well as transaction summary values which are requested by the the dashboard on load. To have these stubbed, they need to be created as pact interactions to be picked up by pact-stub.
Note, in order to run these new browser tests locally, the following steps need to take place.

'npm test' is first run, which runs unit tests and generates up to date contract/pact files
'npm run pact-stub' is run in a new terminal window, which mounts pact files into a container which then stubs all requests made in expected formats, with expected responses.
'npm run server:cypress' is run in a new terminal window. This will now run the self service application, with environment variables pointing the API endpoints at the pact-stub server.
'npm run test:cypress' is run in a third terminal window. This will execute Cypress.io which then uses information in the /cypress directory to initialise and run the browser tests

I'm getting one error in the cypress tests when I run them
 1) Login Page Form validation Valid submissions should progress to 2FA page if provided valid username and password:
     CypressError: Timed out retrying: expected 'Sign in to GOV.UK Pay' to equal 'Enter security code - GOV.UK Pay'

I guess this was already like this, but we should probably start running these tests as part of the pipeline soon to keep us honest


Modified python version check to accommodate for legacy version styles

What does this PR do?
Fixes  #13347
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes? Standalone tests only
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@otaj updated accordingly.


⚠ Processors has degraded performance

In 0845680, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 42 ms

Resolved: Processors performance has improved in a0d5b6e.


.RSSLink is deprecated, but still in the example

You can see it here: https://github.com/gohugoio/hugo/blob/master/examples/blog/layouts/partials/header.html

Still open.


How to seed a lot of files?

I have a lot of gallary images, so how would you seed a lot of files the smartest, like 100.000+ gallary images?
I am right now just seeding them with client.seed but it does not really scale.
Preferably I would only start seeding something when a peer connects.
I did search the issues and could not find anyone asking, so I thought this could be useful for others also to know.

The way that torrents work requires a high number of messages just to remain "findable" by peers. There are messages to the DHT (which take ~3 seconds per torrent) and to the tracker (much faster). These need to be resent every 15 minutes.
So, 300 torrents * 3 seconds = 15 minutes. So, adding around 300 torrents will mean that there's a constant low-level amount of UDP traffic to the DHT. Going too many factors above this number will probably overwhelm your router.
(This is just for the traditional BitTorrent DHT. Talking to WebTorrent trackers should be a lot more efficient.)
I recommend taking a look at "web seeding", which lets you host the files via http/https with something like Apache or nginx (which is simpler and more efficient) and include the http link in your .torrent file. Most torrent clients will only use the web seed when there are few or no peers available.
npm install create-torrent -g

create-torrent --urlList http://example.com/files/my-torrent1 /path/to/my-torrent1

You can read a bit more about web seeding here: http://getright.com/seedtorrent.html

Thanks for that really detailed answer, I can see the problem.
The reason however that I am using torrent are because the internet connection really often drop here in South America, so downloading by http is often a really bad experience. You would be amazed how bad it is! 😅
I think the solution is, whenever the client hits the page, the server callbacks and starts the torrent seeding. This way I am only seeding the pictures which I know are in demand.

@kevinsimper I think you'll find that torrent clients like WebTorrent do a better job of downloading over http than a traditional web browser because they're fetching smaller pieces each time, so if one piece request fails the whole download won't fail.
But I think your approach of on-demand seeding sounds workable too! Kudos to you for making things better for yourself and others in South America -- good stuff!
(Did you know that rsync was invented by a New Zealander who wanted something that made fewer roundtrips than scp because New Zealand is so far away from everything?)


opae-sdk: remove ase documentation

-remove ase documentation from doc
Signed-off-by: anandaravuri<EMAIL_ADDRESS>

Coverage increased (+0.03%) to 65.516% when pulling e7de1e8ad31baaf6765adaebd428cbea8371383d on aravuri/ase_doc_remove into 2b8fcbfcf068518ad89442e65703fd5d915ac672 on master.

Will the removed files be added to opae-sim?

[Ananda]:  moved ase doc from opae-sdk to opae-sim repo
https://github.com/OPAE/opae-sim/pull/23


Feature/model level datalibs

Closes #173
Changes
Implements interfaces for model-level ERA5 and HRES data.
Features

Add ModelLevelERA5 and ModelLevelHRES interfaces for accessing ERA5 and HRES data on model levels.
Update ECMWF tutorial notebook with instructions for using model-level datalibs.

Internals

Create static copy of dataframe for determining pressure at ECMWF model levels.
Extract model-level utilities in ARCOERA5 to their own module for reuse in ModelLevelERA5 and ModelLevelHRES.
Update Makefile so make doctest and make nb-test use data cached by make ensure-era5-cached when run locally.

Tests

[x] QC test passes locally (make test)
[x] CI tests pass

Reviewer

@mlshapiro
@trdean1

Yes, testing those parts of the new datalibs would be good!
I've added tests to test/unit/test_ecmwf that cover portions of ModelLevelERA5 and ModelLevelHRES that don't use metview and don't access upstream APIs. I've parameterized tests by datalib where it makes sense, so these changes should also expand test coverage for other ECMWF datalibs.

Amazing work - I don't have much to comment here, mostly naming conventions and trying to not open up too many hairy edges with inputs.
Just want to confirm that this will cache the pressure level data (not the original Grib or model level data)

This will always cache the pressure-level data, and will optionally cache the original grib data if cache_grib=True is passed as a keyword argument. (I think you'd only want to do that if you're changing something in the grib processing and don't want to wait for the grib file to be re-downloaded each time you process it, so I think of this option as an aid for developers rather than something useful for users.)


Delay win-by-default in gesture arena

Wait until the end of the microtask to tell gesture recognizers that
they've won in the gesture arena. This lets recognizers dispose reject
themselves at arbitrary times without triggering gestures in awkward
call stacks.
Fixes #3183

Not really ready for review.  Definitely needs test.

@Hixie Ready for review now.

LGTM


PR: Remove file path from the Editor's plugin title.

Fixes #3972

Simple enough, thanks @dalthviz!


Bump hyper to 0.9

This makes multipart work with latest hyper.

The problem with bumping Hyper alone is that it breaks other integrations that haven't updated. However, I was thinking about switching to using a version range for compatibility with other crates using an older version of Hyper. Probably >=0.8,<0.10, if you want to try that out on this PR.

I will make this version change myself, thanks.


User contributed scripts

Please describe the change / addition you'd like to see made
One of the great things about "ad" is how easy it is to write little scripts to adapt how the editor works. Some of these will be very bespoke to the user, others will be scripts that many users could be interested in and could avoid "reinventing the wheel"
for example a script to comment out lines of code (a simple adaption to the indent script)
#!/usr/bin/env bash
source "$HOME/.ad/lib/ad.sh"

requireAd
id="$(currentBufferId)"
addr="$(bufRead "$id" addr | sed -E 's/(.+):.+,(.+):.+/\1,\2/')"
# append comment for sh, python or R
adEdit '. x/.+/ s/^/#/'
echo "$addr" | bufWrite "$id" addr

Could there be a discoverable way for users to contribute scripts like this to ad? Say in a "contrib" directory?
Is this a feature you have seen in other text editors?
This isn't an example for a text editor but plan9 has a "contrib" directory. Listed here https://plan9.io/wiki/plan9/Contrib_index/index.html
Other editors Emacs, Vim, VSCode have package repositories such as MELPA for emacs https://melpa.org/
A package repository is very likely overkill for ad but a place to contribute and find useful scripts such as in plan9 might be useful?

I'd be very happy to have users contribute scripts in this way. I think the best thing would be to place them in a new data/contrib directory I think, so if you don't want to raise a PR with scripts like the one you are showing here then please do 🙂


Annotation function from completing-read should be invoked lazily

@oantolin and I are attempting to move annotation functionality from embark and consult to another common package "marginalia", see https://github.com/oantolin/embark/issues/41 and https://github.com/minad/consult/issues/28
From selectrum I am missing ~two~ three things:

(FIXED) The annotation function provided by the completing-read candidates callback must be invoked lazily otherwise the slowdown is unacceptable for example if calling describe-variable with annotations enabled. This is important if we want to use the more standardized annotation function or potentially later on the affixation function (#240). This has been discussed before https://github.com/raxod502/selectrum/issues/241#issuecomment-735791743. Right now consult circumvents this issue since it uses the selectrum-highlight-function and not the standard mechanism for consult-annotate-mode. But I think this is not the approach we are favoring for marginalia.
The face properties of the string provided by the annotation function is overwritten. The full annotation string is shown italic and gray in my case. This is somehow related to #236 I think.
~3. There is another issue it seems. When I am enhancing describe-face with consult-annotate-mode or marginalia-mode, the metadata annotation function does not seem to be used. If I use the selectrum-highlight-function however it works. The difference is that describe-face is doing a multiple completing read it seems?~

Right now consult circumvents this issue since it uses the selectrum-highlight-function and not the standard mechanism for consult-annotate-mode. But I think this is not the approach we are favoring for marginalia.

I agree, this needs to be fixed, this isn't the purpose of selectrum-highlight-function, too.  I have worked on it in #248 can you confirm that works for you?

The face properties of the string provided by the annotation function is overwritten. The full annotation string is shown italic and gray in my case. This is somehow related to #236 I think.

Yes, let's dealt with that in this issue.

Currently hacking on an initial prototype of marginalia see https://github.com/minad/marginalia

@clemera There is another issue I think, see 3. above.

Do you have a test case for me? It works with this one:
(completing-read-multiple
 "Use annotations to note which is longest: "
 (lambda (input predicate action)
   (if (eq action 'metadata)
       `(metadata
         (annotation-function
          . (lambda (str)
              (when (string= str "longest")
                " <- This is the longest."))))
     (complete-with-action action
                           '("longest" "short" "longer")
                           input
                           predicate))))

@clemera I will do - maybe something else was wrong and I attributed it wrongly to selectrum what I saw. Up to now I only experimented shortly with overwriting the default annotation function in combination with selectrum due to issues 1 and 2 which are kind of a road block.

@clemera I made a test again. Issue 1 is fixed and works nicely, issue 3 was invalid and things actually work. Eagerly waiting for issue 2 now! Then I can remove the rest of the selectrum specifics from marginalia :)

Should be fixed?

Fixed, thank you!


Slack channel engagement 12/6

[ ] post to Slack channel

We hope you all had a nice holiday last week channel! Are you thinking of learning a new tool in the new year? Here's a throwback to some resources shared by @ DawnWright during last year's Pathways series if you're interested in mapping and data visualization in a new way.
The Essential ArcGIS skills lesson series is a set of tutorials to help new users learn skills to create maps and utilize ArcGIS online. Lessons range from 15 to 45 minutes and will help you feel confident using these tools on your own.
Another pro tip? Through tutorials, you are given licensing during the course and can practice your skills right in the web browser. Check it out star!


Make it easier to develop pre_processor and post_processor without spark

from #448: We could consider turning generate_udf into a class so that we can take care of the pickling and the pandas_udf constructor call in the base class? This way the hack is more isolated

duplicated with #533


Update wrapper to 8.5 RC3

WARN: The milestone of this PR is 8.5, but the base branch version is 8.6

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


Migration tests: parametrize fresh_create.

This will allow us to write migration tests that look at specific scenarios; right now we call initial_db to create the objects, which means the database we have to migrate from is fixed for all tests.
In particular, I want to shove some NetworkingActions in the DB as part of testing #683, without a whole lot else to complicate things.

can you explain a bit how your 'make_objects' works? It seems like it would be adding something in addition to what is in initial_db but I'm not positive.
I'm going to go review Alembic again, but want a starting place for what to look for.

It would be instead of initial db; each migration test would supply its own set of objects to test with., so we can write tests for different scenarios. This would obviously need to be coordinated with the database dump.

each migration test would supply its own set of objects to test with

Shouldn't there still be a standard (minimum) set of objects as well though? If the objects are different for each migration test it doesn't seem like the same things would be tested across multiple migrations.

You're still going to be running each migration test against all scripts from when the database dump was made through the current. What's different from test to test is the set of objects that exist in the database before & after, not what migration scripts are tested.

@kylehogan, ping.

thanks for the reminder. Could you write an addition to the docs that describes how to use make_objects? it seems that this is currently set up to use initial_db, is this correct? (if so, how would you use something other than initial_db?)

@kylehogan, I added a proper docstring to that function. Note that this is a parametrized test, similar to those in tests/unit/api/auth.py. The logic for the test is parametrized over the three arguments to that function. The second argument to parametrize is a list of sets of arguments to pass to the function; each one is a test. Right now there's only one, but looking at tests/unit/api/auth.py gives a better picture of what this looks like when you have more than one test.
To use something other than initial_db, you'd define another function to use instead, take a database dump, and then add another element to that list that's being passed to parametrize, which would include the path to the dump, the function you defined, and the configuration you used when creating the dump.
It would probably be easier to see what was going on if I'd actually added another such test in this patch. I'm planning on doing this as part of port_revert, which should make it more clear, but I also wanted to keep that pr as small as I can, so doing this separately seemed like a good idea.

thanks for adding the longer docstring. makes sense!
+1

@henn, @shwsun, @knikolla, could one of you give this a second review?

@zenhack I have the sense that this PR means instead of using init_db to populate database for each and every test case, now we can populate database with different set of objects. Does this mean that we now will have a stack of init_db, switch_op_db stuff later on?
And I would guess the value would be that only facing a PR with switches changes will require switch_op_db to populate the database, right?

@shwsun, I assume you mean initial_db?
This is just for the migration tests, but the point is to give us more flexibility in terms of what objects are used for what tests.
It might be a good idea to factor out the db object population later on, and we can discuss how that should look. The concrete use case I have in mind is for the work on port_revert, which adds a column type to NetworkingAction. In this instance:

The type column needs to be added
Any existing NetoworkingActions need to have their type field set to 'modify_port'

I'd like to write a test that excercises the migration script in the case where there are NetworkingActions in the db. None of our existing fixtures does this, and while we could keep piling stuff on, it would be nice if this could be self-contained -- the test will be easier to understand with just a few objects in the DB. Furthermore, having NetworkingActions in the db is likely to complicate how other tests need to deal with things.

@zenhack I now understand that it is for migration testing. +1


This is super fun & good

original image:

I am happy you like it!! Any question just ask me! If you have any business cooperation， please contact<EMAIL_ADDRESS>


Change systemd deb package architecture to amd64

Currently systemd deb package released as all architecture, but it includes amd64 binary.
I'd like to change package architecture to amd64 so that package architecture will be the same as including binary.

LGTM

Thanks!


Add eslint "required" config

Checklist

[x] npm start and/or npm test it worked
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[x] commit message follows commit guidelines

Description of change
From #1

The tests aren't passing but it seems to be a problem with cjpatoilo/eslint-config-styled or its usage.

I don't know the real problem here, but looking at cjpatoilo/marshmallow, a .eslintrc file is being used together with the -c styled in the command line.
If you have a better approach to solve this problem, please let me know.

@alisonmonteiro Good call 🚀


Basic LSP implementation

The syntax/semantic analysis APIs should now be sufficient for a basic LSP implementation. We don't need to be super ambitious here - just performing semantic highlighting would be a great first step.
Depends on:

[x] #30

e2dbe09 implemented syntax classification.
Now we just need to hook up an LSP handler to our workspace and classification APIs.


Background Colour of AnimatedIntro.tsx component is only behind the ball.

Hey there! Sick tutorial on this.
Was following you for the AnimtatedIntro.tsx, and it seems like the background colour is changing just for the area behind the ball, and not the entire screen like your demo. Code is copied paste from your repo, with no changes.

Nevermind, turns out I didnt set the flex property in the main index.tsx.
<View style={{ flex: 1 }} >
  <AnimatedIntro />
  <BottomLoginSheet />
</View>


Wasm: Language feature for the JavaScript Promise Integration proposal (JSPI)

Motivation
The JavaScript Promise Integration proposal (JSPI) for WebAssembly adds a superpower to WebAssembly, compared to JavaScript. It allows Wasm code to perform synchronous calls to Promise-returning JS functions, and suspend when the Promise is not resolved yet. This is a power basically equivalent to Virtual Threads (aka green threads, coroutines, etc.), something that is not possible to implement with JavaScript alone. Basically a non-blocking await(promise) function.
I successfully leveraged JSPI with Scala.js-on-Wasm in a hobby project to dramatically simplify the internal API :
https://github.com/sjrd/funlabyrinthe-scala/commit/18bc1f2b93e6e235087cbbd17bd344d055b6f32f
However, to achieve it, I had to textually post-process the main.js file generated by Scala.js, hacking and slashing its abstractions to build my own. I used my internal knowledge of how the linker generates code to do this. This is not something we're supposed to do, obviously, and it will likely break when I next uprade Scala.js.
It would be nice to have dedicated language support to leverage JSPI from Scala.js code.
Proposal
There are two sides to JSPI : importing a WebAssembly.Suspending function, and wrapping an export into a WebAssembly.promising call.
Promising Exports
We need a way to express that an exported def must be wrapped in a WebAssembly.promising call. Note that the Wasm function fitself must be given to promising(f). Currently Scala.js wraps every exported function in a JavaScript-defined function to protect its abstractions. This is why we cannot currently call promising in user-space.
I propose a distinct @JSExportPromising annotation, to be used like @JSExportTopLevel on defs:
object Container {
  @JSExportPromising("name", "module.js")
  def myPromisingMethod(x: Int, y: String): String = ???
}

As seen from JavaScript, it would return a js.Promise[String], following the API transformation applied by WebAssembly.promising.
Suspending Imports
On the other side, we need a way to express that we import a JS function as wrapped into a new WebAssembly.Suspending(f) constructor. Here as well, the resulting Suspending instance must be directly imported into Wasm. Since Scala.js generates custom arrow functions for all imports to protect its abstractions, once again this cannot be achieved in user-space.
I see two possibilities here.
The first one is simpler from the Scala.js language specification point of view. The second one is closer to the JSPI API design.
First option: offer a single primitive await:
def await[A](p: js.Promise[A]): A = throw new Error("stub")

This can be implemented as a unique JS helper of the form
"await": new WebAssembly.Suspending((x) => x),

When calling any JS function that returns a js.Promise in its API, we can follow up with a call to this primitive to exploit JSPI and give us the result.
Second option: offer a generic @JSImportSuspending annotation for @js.native defs:
@js.native
@JSImportSuspending("readFile", "node:fs")
def readFile(f: String, charset: String): String = js.native

This is closer to the spirit of the JSPI API design, but it means we also need an @JSGlobalSuspending, a variant with a globalFallback, etc.
Implementation concerns
On the call path between a promising export and a suspending import call, there cannot be any JavaScript frame. This cannot reasonably be checked statically nor dynamically. However, it will be checked by the engine at run-time, which will trap if the condition is not upheld. There is currently no way to test ahead of time, and since a trap is not catchable from Wasm, we cannot recover from that. That is quite annoying, since otherwise in fastLinkJS our linker guarantees that the code it generates never traps. For example, even a testing framework would have no way of catching this condition; it will crash instead.
Design concerns
This feature is unique to the combination of JavaScript and Wasm. JS alone cannot do it, nor can Wasm alone. Therefore, this feature would be unique to Scala.js-on-Wasm! Given the current experimental status of our Wasm backend, it's not something we can really do today. However, it gives such a new superpower that I believe we should already start considering how we could offer that superpower to our users.

Interestingly, #5082 uses JSPI to implement async/await in the Wasm backend. If we did have #5082, then maybe the only thing we need for full, unrestricted support of JSPI is to let js.await be used anywhere instead of being restricted inside a js.async block. When linking on JS, unrestricted JSAwaits would be flagged as linking errors; but on Wasm, they would be fine.

Just to check my understanding of JSPI:

I would be surprised if they are optimized in any specific way, because by spec each await must happen in a separate microtask of the job queue.

(from https://github.com/scala-js/scala-js/issues/5079#issuecomment-2488224718)
Not being forced to have a microtask tick is essentially the superpower we are granted here, correct? (or does the import interface spec force the tick?).
On the surface, it also looks like we can suspend in any function (not just ones as marked async), but IIUC, if we want to have a static guarantee that we do not trap (or otherwise fail) on suspend, we essentially need to introduce a similar "flag" on functions. (arguably, not being strictly forced to have that flag and use other means to not trap is also a superpower).

Not being forced to have a microtask tick is essentially the superpower we are granted here, correct? (or does the import interface spec force the tick?).

No, that's at best something that could be more efficient. Anyway, even JSPI's Suspending functions suspend and resume at best on the next tick as well. Under the hood it uses the same Await(p) "spec function" as the one used by JS' await keyword.

(arguably, not being strictly forced to have that flag and use other means to not trap is also a superpower).

That is the superpower. Not so much that we don't need to flag functions, but that we can suspend an arbitrary number of stack frames! The functions in the middle of the stack (between the entry through promising and the exit through Suspending) don't need to know that they can be suspended. In fact, they can either be called from code that does not suspend, in which case they won't suspend.
This mechanism solves the "what color is your function?" problem. Every function can act blue or red, depending on who calls them. That's the semantic superpower. In addition, it does that efficiently (through suspension instead of CPS-transforming every function).

On the surface, it also looks like we can suspend in any function (not just ones as marked async), but IIUC, if we want to have a static guarantee that we do not trap (or otherwise fail) on suspend, we essentially need to introduce a similar "flag" on functions.

If we want that guarantee, then we're back to coloring functions, indeed. For the traps, it looks like I managed to convince the proposal champions to throw catchable exceptions instead. So that's good, because we can still promise to generate code that never traps. We don't need to guarantee that no exceptions will be thrown. (It is possible to build abstractions that guarantee it with Scala 3's research "capture checking", notably as done in the research Gears library; but that's out of the scope of what we need to think about.)

It’s great to see that https://github.com/scala-js/scala-js/pull/5082 only provides the js.async and js.await APIs, without exposing "low-level" APIs like suspendable to end users 👍

the only thing we need for full, unrestricted support of JSPI is to let js.await be used anywhere instead of being restricted inside a js.async block. When linking on JS, unrestricted JSAwaits would be flagged as linking errors; but on Wasm, they would be fine.

I’m curious about how "letting " let js.await be used anywhere instead of being restricted inside a js.async block" looks like.
AFAIK, js.async is translated into a Wasm function that is exported and wrapped by WebAssembly.promising, while js.await(...) is a JS function wrapped by WebAssembly.suspending and imported into Wasm (at least in the current state of https://github.com/scala-js/scala-js/pull/5082).
If we allow js.await outside a js.async block when linking to Wasm, the generated code could result in a RuntimeError (Uncaught RuntimeError: attempting to suspend without a WebAssembly.promising export on Chrome). Do you have a plan to additional change to the PR?

No, I don't think we should attempt to check at the Scala.js level. With the changes in https://github.com/WebAssembly/js-promise-integration/pull/55 the RuntimeError will be a SuspendError error instead (a normal JS exception that we can catch, for example in a testing framework). Beyond that, we would leave it to the responsibility of the developer to write correct code (like we do for all sorts of APIs that can throw exceptions).
As I mentioned, capture checking would provide a way to build safer abstractions on top of those building blocks, but that's beyond our scope.

Beyond that, we would leave it to the responsibility of the developer to write correct code (like we do for all sorts of APIs that can throw exceptions).

I'm somewhat with @tanishiking on this one. Or at least, I feel we should:

Try to build a deeper understanding of how this case relates to other cases where we "resort" to runtime exceptions, or
Have a solid plan of what to do if we realize we need to restrict this capability further.

It doesn't seem bad at all to me to have a predictable exception. We have plenty of APIs, both Scala.js-specific and not, that work that way. Examples include division by zero, string concatenation of a js.Symbol, access out of bounds of a collection. Arguably all of these can be detected with a straightforward test ahead of attempting the operation. For erroneous conditions that cannot be straightforwardly tested ahead of time, we have even worse APIs with unspecified behavior: notably, all sorts of modifications to collections while an iteration is in progress. Here we will get (with SuspendError) a predictable exception instead of unspecified behavior.
The problem with restricting this is that currently known solutions either

restrict too much, preventing one from actually exercising the superpower offered by JSPI (functions need to be colored again)
or too little, resulting in the exception being thrown anyway in some cases, so we still have no guarantee.

Finding a solution that hits the sweet spot in the middle is precisely what capture checking set out to achieve, but that is a multi-year research project. js.async would introduce a capability to await, and js.await would require that capability. Capture checking ensures that the capability does not leak out of the js.async block (even through captures, which is the origin of the project name).

but that is a multi-year research project

I think this is the key insight here. So I agree, we must find a way to expose this at least as a low-level API (and if we actually get exceptions, that's indeed more than enough IMO).
I wonder if we should offer a "coloring" based sugar at the language level for users that just want async (IIUC #5082 essentially does that in its current state).


KOGITO-7119: Establish test runner for serverless tooling in vscode

Adds first testing project greeting-flow
Adds first smoke test with example
Add helpers to work with serverless files in VSCode
Add helpers for asserts and SWF editor and text editor.
@tomasdavidorg  ready for inital checks,  I will add some polish later.

@tomasdavidorg @tiagobento modified the smoke test to be more lenient and useful, and added it to build commands, let me know if still ok.

Disabled the tests until #1052 is merged. @tiagobento Please merge if ok.


TST, MAINT: test_matrix_input failing

On latest main this fails: python dev.py test -t scipy/interpolate/tests/test_rgi.py::TestInterpN::test_matrix_input -b all
This one has an explicit opt-in marker but also uses matrix so I'll defer to @ev-br

https://github.com/scipy/scipy/pull/21536 suggests a fix, WDYT?


[Android] leftWindow does not respond to click events

What am I doing wrong here? Seem's like this should be pretty straight forward but I keep having issues. iOS works fine with Windows, android will not respond to any click events in the slidView(leftWindow). Any help would be appreciated.
var NappDrawerModule = require('dk.napp.drawer');

var data = [{title:'test1'}, {title:'test 2'}, {title:'test 3'}];

var slideView = Ti.UI.createView();
var tableView = Ti.UI.createTableView({data:data});
slideView.add(tableView);

var centerView = Ti.UI.createView();
var label = Ti.UI.createLabel({text:'center view'});
centerView.add(label);

tableView.addEventListener('click', function(e){
	Ti.API.info('clicked index '+e.index);
});

slideView.addEventListener('click', function(){
	Ti.API.info('clicked slideView');
});

var drawer = NappDrawerModule.createDrawer({
	leftWindow: slideView,
	centerWindow: centerView,
	closeDrawerGestureMode: NappDrawerModule.CLOSE_MODE_ALL,
	openDrawerGestureMode: NappDrawerModule.OPEN_MODE_ALL,
	fullscreen:false,
	showShadow: true,
	leftDrawerWidth: 250,
	fading: 0.2,
        parallaxAmount: 0.2,
	statusBarStyle: NappDrawerModule.STATUSBAR_WHITE,
	orientationModes: [Ti.UI.PORTRAIT, Ti.UI.UPSIDE_PORTRAIT]
});

drawer.open();

I am having the same issue. Click events do not seem to work under android, works fine under IOS. How did you finally resolve this? Thanks

Ah just found it myself.
https://github.com/FokkeZB/nl.fokkezb.drawer/issues/9
Under android do NOT use CLOSE_MODE_ALL.
If seems clicks do not work if used !?
Thanks anyway.


Checkpointed WAL tsm1 cache

This work shows the tsm1 cache taking shape. The idea behind this cache is that it is the queryable version of the data that exists in the WAL. In fact it may be a super-set of data in the WAL, and even data in the WAL has been compacted into tsm files data in the cache may be queried instead of hitting tsm files. Systems with large amounts of RAM may use this to increase query performance.
Of course all caches must have an eviction policy, and this is where checkpoints come in. When the WAL compacts data into tsm files, it informs the cache via the checkpoint mechanism. This allows the cache to evict data received up to and including the checkpoint, if necessary.
If data cannot be added to the cache, because its maximum memory footprint has been reached, it will reject additions.
Performance is critical in this code, and this code will be carefully tested and benchmarked. Some distinct types have been introduced to help with construction, but these types must not impact performance.
Please consult the changes to DESIGN.md for more details.

@jwilder @pauldix

Few comments, but otherwise :+1:

OK, I'm going to merge this now actually, as I got +1 from @jwilder -- more work coming in next PR.

it's vital to get some benchmark testing on this when the cache is large. I can pitch in and just write some of those tests myself if it'll be helpful.

Thanks @pauldix. I am going to change the implementation somewhat so not
quite ready for more tests yet.
On Tuesday, November 17, 2015, Paul Dix<EMAIL_ADDRESS>wrote:

it's vital to get some benchmark testing on this when the cache is large.
I can pitch in and just write some of those tests myself if it'll be
helpful.
—
Reply to this email directly or view it on GitHub
https://github.com/influxdb/influxdb/pull/4812#issuecomment-157565073.


Polars MVP documentation

Include examples of using Polars for sums, means, groupby, and quantiles

(depending on time, this may move to the next, smaller release)

In the API docs, rust/src/measurements/code/make_private_lazyframe.rst already has sums and groupby, adding mean wouldn't be hard, and with a data set this trivial, median might be sufficient to demonstrate quantiles?

closing in favor of multiple notebook issues


Can not navigate slack history by hotkey

Expected Behavior
In any slack service, try navigate history using hotkey (CMD + [, CMD + ])
Channel can be navigated back and forth

Current Behavior
In any slack service, try navigate history using hotkey (CMD + [, CMD + ])
Channel can not be navigated back and forth

Steps to Reproduce (for bugs)

Open a slack channel (service)
select back and forth some channels
Use hotkey to navigate channel history
Nothing changed

Your Environment

Franz Version used: Version 5.0.0-beta.16 (5.0.0-beta.16.762)
Operating System and version: Macos Sirrea 10.12.6 (16G1212)

Many keyboard shortcuts are not working with the Slack integration. Common ones that are frustratingly not registering are CTRL/CMD+T to switch between conversations and CTRL/CMD+F that is intercepted by Franz to go into fullscreen mode. Why not just using f11 for that?

Hi guys, what about the history (prev/next channel or DM) navigation? Why is this issue close @AndyHoang?

@suxstellino since its the limitation, so I think there're nothing we can do. So I closed this


Implement caffe export

Please check #3572.

@okuta Is it still WIP?

I changed title.
Cloud you start review this?

I fixed.

A flake8 error is occurring. Please check the test results.

And please import from chainer import utils and call utils.experimental('chainer.exporters.caffe.export') at the top of export() function.

I fixed.

Thanks! I will add.

@okuta Thanks for adding the README.md. Could you check the Jenkins test result? The doc test is failing due to this WARNING:
WARNING: [autosummary] failed to import 'chainer.exporters.caffe.export': no module named chainer.exporters.caffe.export

@mitmul  The test was passed.


Feature request: flag unused context

Issue
I just spent a large chunk of my Claude balance when I accidentally left a 5,000 line file in the context after I was done editing it.  I eventually noticed my mistake when I wondered why aider/Claude was taking longer to respond than expected.  (This is not how I prefer to organize things but it was someone else's project.)  What do you think about adding a line to the output like this?

Applied edit to foo.py
Unedited files in context: bar.py, baz.py

Version and model info
Aider v0.56.0

If you are unsure what is in your context, you can use /tokens to check.

Thanks for trying aider and filing this issue.
Aider already shows all the files in the context above each prompt:
─────────────────────────
aider/scrape.py
>


Recipe for testing ES6 modules

It's kind of covered by the code coverage recipe, but it might be good to have a simpler recipe that didn't include that.
Some stuff from this comment would apply here as well.

See https://github.com/jamestalmage/__ava-0.8-with-babel-6 for a working example
Repository name is a misnomer now - it uses<EMAIL_ADDRESS>

It dould be great to have a recipe of how to use ava with rollup.

@boutros What do you mean? Rollup is a module bundler—surely you aren't bundling AVA for production right?

Rollup is a module bundler—surely you aren't bundling AVA for production right?

No, not for production of course. Anyway I solved it with rollup, so my remark can be ignored. If anyone is interested here is the relevant commit: https://github.com/boutros/koblersken/commit/c00e3a098c035c7a585ab72d1dde91fd2ac47e91
It's easy to get lost in the javascript build jungle :)

I think we're kinda already covering this. Of course if you use new language features that are supported by your targeted Node.js version you don't need to do anything special. If you need to transpile your source code we've covered that in the Transpiling imported modules section.
I'm closing this issue, but if you have ideas on how to improve the documentation please open a new issue or PR!


Extra behat tests for participant/wishlist page

+ fix to not include ads in test and test_travis env to avoid blocking the behat tests

@tvlooy now all critical parts are tested I think.

creating a party
all actions in a unstarted party + start it
all actions in a started party
participant and wishlist page

Possible next steps are also testing the email content and command line jobs we have


Improve the DEV network effect

You really want everyone with a project on stakes.social to spread the word about DEV in some way. It could be as simple as a badge in the project readme, a tweet, or something else. It would of course be totally optional, but encouraged, since it benefits the project owner too.
One idea I had is that right after adding a GitHub repo onto Stakes.social, there could be a modal dialog with a wizard of some recommended steps. For example, for the wizard:

Add badge to repo.
Tweet about it (with a tweet button that opens Twitter with a predefined tweet).
Some examples of how other people are using and spreading the word about DEV.

Alternatively, each asset page could show some kind of checklist with some steps to take. This is how GitHub does it:

Please keep this open.


Update build.gradle

In flutter version 3.24.3, This plugin not working so I just increased sdk compile version.
It work for me now.

Here is the quick fix :
We got the issue , because the plugin is using Kotlin Gradle version lower than 1.5.10, now its time to upgrade the Kotlin version of this plugin. You might get this issue with another plugin as well.
Download and unzip the plugin.
Then  open /android/build.gradle
Replace ext.kotlin_version = '1.5.10' with ext.kotlin_version = '1.7.10' // OR any latest version
Now, save the build. Gradle file of the plugin folder.
Now go to the pubspec. yaml file on your project, and add this package like the below:
#before
flutter_html_to_pdf: ^0.7.0
#after
flutter_html_to_pdf:
path: lib/flutter_html_to_pdf
I put the plugin in the lib folder in the root directory, you can put it wherever you want.
Now it works Properly.


Improve docker-compose local unit-test, minor cleanup

Unfortunately the "unit-test" isn't much of a unit test, clearly connecting to the database containers...
... but after these changes, it at least starts up on my machine, and doesn't collide with other instances / port use / etc.
It also finally connects to postgres, though that should still be fixed eventually.
To summarize the changes:

ports -> expose to keep networking internal.  ports are no longer publicly mapped, so you can run other things without colliding.
command array -> a string.  because wow was that verbose.
adding postgres connection info to unit-test, as it was using the wrong host + username previously.

Unfortunately tests still don't pass locally, I have not yet traced down the reason.
They stop here for me, on a few errors like this:
--- FAIL: TestSQLHistoryV2PersistenceSuite (3.70s)
    --- PASS: TestSQLHistoryV2PersistenceSuite/TestConcurrentlyCreateAndAppendBranches (1.23s)
    --- FAIL: TestSQLHistoryV2PersistenceSuite/TestConcurrentlyForkAndAppendBranches (0.61s)
        historyV2PersistenceTest.go:752:
            	Error Trace:	historyV2PersistenceTest.go:752
				historyV2PersistenceTest.go:570
				asm_amd64.s:1357
            	Error:      	Expected nil, but got: &types.InternalDataInconsistencyError{Message:"corrupted history event batch, eventID is not continouous"}
            	Test:       	TestSQLHistoryV2PersistenceSuite/TestConcurrentlyForkAndAppendBranches

Still, it's progress.

Looks like you have some integration tests failing with the new changes.

should be totally unrelated - buildkite doesn't use the -local files. (restarting tests)

Coverage decreased (-0.05%) to 61.379% when pulling fe0807cc06e46d465d60d6f3ffb7b418f8ce2e7f on compose into e895640226c5f07ae4be52624f9b78b0cf9c72a2 on master.


Fix PaymentMethodType case

Make sure PaymentMethodType is always uppercase. Make the code similar to PaymentMethodService.

@Betree E2E tests are still failing, looking at it.


Create Docker Image CI

Require secret DELETE with delete:packages permission
Closes #14

What it does:

Runs on every commit in the main branch and on every new tag
Checkout code
Connects to ghcr
Gets the name of the git (secure-77/Perlite) and sets it to lower case
Builds and pushes the images, e.g.:

perlite-latest, perlite-dev-latest and web-latest` tags for the main branch
perlite-2 and web-2 for a new tag 2

Finally, delete the old images that have been untagged

I'm doing this for the first time btw so I hope I'm not messing up 😁

Thanks for your effort, but at the moment I only want to push final releases to the docker hub and not every commit! Maybe I will take a look at that in the future.


[Tabs] scrollIntoView blocked by scrollable tabs

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I am trying to use Tabs component to navigate user on page - on tab change I call scrollIntoView to some section on page. It works well in most cases, except when I am on small device tabs turns into scrollable tabs - it that case when I click on tab which is not entirely visible nice animation to switch tabs appear but scrollIntoView is not working (page is not scrolled to section).
Update: window.scrollTo works fine, also without behaviour: smooth it works without problem
Expected behavior 🤔
scrollIntoView is working in all cases OR there is a way how to know that tabs scroll animation ended so I can delay my scroll functionality
Steps to reproduce 🕹
Steps:

Open minimal example https://codesandbox.io/s/react-mui-forked-dfm2q5?file=/index.js

Resize preview so some tab is only partialy visible e.g.

click on partially hidden tab (3rd tab) - tabs are switched but view is not scrolled to 3rd tab

click on some visible tab or click again on 3rd tab - view is scrolled using scrollIntoView

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
System:
OS: Windows 10 10.0.22000
CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
Memory: 3.82 GB / 15.89 GB
Binaries:
Node: 16.14.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.5 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 8.5.5 - C:\Projects\my-business-v3\node_modules.bin\npm.CMD
Managers:
pip3: 20.2.3 - C:\Python39\Scripts\pip3.EXE
Utilities:
Git: 2.21.0.
Virtualization:
Docker: 20.10.13 - C:\Program Files\Docker\Docker\resources\bin\docker.EXE
IDEs:
Android Studio: Version  <IP_ADDRESS> AI-183.6<IP_ADDRESS>22156
VSCode: 1.67.2 - C:\Users\OndrejBrynda\AppData\Local\Programs\Microsoft VS Code\bin\code.CMD
Visual Studio: 16.11.32407.337 (Visual Studio Professional 2019)
Languages:
Python: 3.9.2
Browsers:
Chrome: 101.0.4951.67
Edge: Spartan (44.22000.120.0), Chromium (101.0.1210.53)
Internet Explorer: 11.0.22000.120

Tested in Chrome, Firefox

This seems like an interesting issue to work on, but isn't it working as intended? User need a way to read other tabs that are blocked. And when someone click on a blocked tabs, then most probably they wouldn't want to scrollIntoView, because they don't fully know what they are clicking yet

Well.. I don't think this is intended behaviour..

Why should something going on internally in mui component block anything going on in outside code? thats bold assuming user dont want scroll or something else
There is contradiction: onChange is triggered so I assume that tabs has been changed, but action in my handler is blocked so app ends up in inconsistent state
User can see where are they scrolling - active tab is in the end visible, moreover they can be in fixed div so they are always visible, also thats why they are clicking the tab - if they want to read it they would probably scrool tabs first

oh, you're right. I misunderstood something. Very well then, I'll see what I can do

My conclusion about this problem is that the animation for showing the partially hidden tab cancels your animation. It works when scroll-behavior is not "smooth" because... there is no animation so it can go to the anchor.
More details here: https://stackoverflow.com/a/73055175
A solution could be to wait the end of the animation of the tab, then calling scrollIntoView. I have to do it but I don't know when I will take the time so I hope someone will find it before me !


The fast rtps will NOT work when Domain id value is from 0xc16c to 0xc16f ?

     Hi, we discovered by accident that the subscriber will not receive the data if domain id setting as 0xc16c - 0xc16f. So is there any restrictive condition to set domain id, or any bug when handle domain id? Thanks!

RTPS uses UDP to send and receive data. The port number used is related with the domain_id and the participant_id. Section 9.6.1 of the RTPS standard specifies the default way in which the ports are calculated to allow interoperability. As UDP has a port number limit of 65535, the maximum domain_id value to create a valid port number is about 230.

Closing this, as the question has been answered long ago.


gcsfuse 0.20.1

Thanks for your contribution to Homebrew!  You rock!


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.6 to before v5.15.8",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ec97ecf1ebe485a17cd8395a5f35e6b80b57665a",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=d86216dfda7c98375f809e26a30bfdaaba21d46e"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ec97ecf1ebe485a17cd8395a5f35e6b80b57665a",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "d86216dfda7c98375f809e26a30bfdaaba21d46e",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net/sched: fq_pie: prevent dismantle issue\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.8 by commit d86216dfda7c98375f809e26a30bfdaaba21d46e, it was introduced in version v5.6 by commit ec97ecf1ebe485a17cd8395a5f35e6b80b57665a. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002594


nixos: fix formatting of option examples

A number of option examples in the Configuration Options section of the manual are badly formatted.
For instance, the services.fail2ban.jails option example looks like this (the image cuts off the full example):

I found eight other examples in the manual which are also badly formatted:

networking.vpnc.services
environment.freetds
security.pki.certificates
services.nsd.zones
services.phpfpm.poolConfigs
services.zope2.instances
services.xserver.inputClassSections
system.activationScripts

These commits use the literalExample function to properly format the examples like so:

The manual has been built on master and checked.
I thought it would be cleaner to make a separate commit for each module, but I can squash the commits into one if you would prefer.
Cheers

Looks good to me.

:+1:

I do not think this is really problem with how examples are formated in nixos options, more problem that documentation generator (does not have yet) support for correctly displaying attribute sets, and that's something that should be fixed.

@offlinehacker Agreed.
For example, the "default" value of fonts.fontconfig.ultimate.rendering is also badly formatted in the manual, but this cannot be fixed using the literalExample function because then the module, by default, would set the option to be text rather than an attribute set.
Getting the document generator to support correctly displaying attribute sets (and lists) would solve this.

@lihop could you rebase? I'll merge then.

@domenkozar Rebased.
Zope2 formatting was fixed by https://github.com/NixOS/nixpkgs/commit/a04a7272aa38ada45e694281071b720d4abbd0df#diff-3a432d6d0d049e13f4335c4d77fa7283 so I removed the commit for that one.


Feature/login email or display name

Description
-Small fixes in error handing with login/sign up
-Ability to login using email or username with same form
Type of change
Please delete options that are not relevant.

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

Change Status

[ ] Complete, but not tested (may need new tests)

How Has This Been Tested?

[ ] Test A

The new login function is a super long and redundant and probably not the best way to do it, but it works. Can make it better later.


An AsyncHttpResponseHandler Error

Hi all !
I developed an Android APP by Android Studio, which is almost finished.
My app works normally when I RUN the app in Android Studio, but when I build an APK and installed it to my cell phone , the app crashed when I log in.
The error info is as follow:
29413-29413/? E/AsyncHttpResponseHandler﹕ User-space exception detected!
    java.lang.NullPointerException: Attempt to invoke interface method 'java.lang.Object[] java.util.Collection.toArray()' on a null object reference
........
07-29 17:42:04.458  29413-29413/? D/AndroidRuntime﹕ Shutting down VM
07-29 17:42:04.468  29413-29413/? E/a﹕ java.lang.RuntimeException: java.lang.NullPointerException: Attempt to invoke interface method 'java.lang.Object[] java.util.Collection.toArray()' on a null object reference
........

     Caused by: java.lang.NullPointerException: Attempt to invoke interface method 'java.lang.Object[] java.util.Collection.toArray()' on a null object reference

Thanks.

Hi, from first line of LogCat read User-space exception detected! This means, the problem is in your implementation of callback method, and you have to find out which one. Search for toArray() in your code, this is not problem of library in my opinion.


Need documentation (release 4.2) describing veg_hist inputs

Need to add documentation (to release 4.2) for veg_hist inputs, similar to meteorological input files but optional.  Also need to make it clear that if these are used, their values will be taken instead of those from the veg_lib or veg_param files.

Agreed.  This will need to go directly onto master for the docs and a PR is welcomed.

I believe this issue can be closed, as it was addressed in PR #330.

@tbohn I'll close the issue but in the future, you can use this syntax to have these automatically closed when your PR is merged.

Ah, that's pretty slick.  I'll use it.
On Wed, Jan 20, 2016 at 2:21 PM, Joe Hamman<EMAIL_ADDRESS>wrote:

@tbohn https://github.com/tbohn I'll close the issue but in the future,
you can use this
https://help.github.com/articles/closing-issues-via-commit-messages/
syntax to have these automatically closed when your PR is merged.
—
Reply to this email directly or view it on GitHub
https://github.com/UW-Hydro/VIC/issues/328#issuecomment-173380746.


AvroSchemaRegistryClientMessageConverter should attempt to create caches, or document them

Spun off from #1566:
The AvroSchemaRegistryClientMessageConverter uses these caches:
	public static final String CACHE_PREFIX = "org.springframework.cloud.stream.schema";
	public static final String REFLECTION_CACHE_NAME = CACHE_PREFIX + ".reflectionCache";
	public static final String SCHEMA_CACHE_NAME = CACHE_PREFIX + ".schemaCache";
	public static final String REFERENCE_CACHE_NAME = CACHE_PREFIX + ".referenceCache";

I suggest either or both of the following:

Document that these caches are required
Enhance the converter to attempt to auto-create these caches if missing.

What version of spring-cloud-stream you are referring to?  I can't correlate line 355 to anything of significance going back to 2.1.x and 2.0.x didn't even have that many lines.

I think I figured out versioning issue. . . anyway
Also, based on the comment I left in #1566 "...First if you look at implementation of the cache manager (RedisCacheManager for your case), they do create caches if they are missing, so I can't reproduce NPE you're referring to..."
Basically what I am trying to say is that it is the responsibility of the CacheManager to pre-create caches. Some do and some perhaps don't. I do not believe we have to be concerned with that at the stream level.
I am going to put an assertion to avoid NPE and log a more meaningful message. That's about the best we can do. I'll back port it as well to 2.1.x

Thanks @olegz , that assert message is great


[BUG] iOS build failure with JWPlayerKit 4.20.0

What does this Pull Request do?

Remove soon to be deprecated callback
Only use a specific version of the iOS SDK.

Why is this Pull Request needed?

Prepare for next iOS release with breaking deprecations
Ensure the version in the podspec is tested and viable with this wrapper

Are there any points in the code the reviewer needs to double check?

no

Are there any Pull Requests open in other repos which need to be merged with this?

no

Addresses Issue(s):
GitHub Issue

For anyone waiting for this, you will need to ensure your podfile.lock is updated to only use the specific version as stated in the podspec here.

Thanks!


module "QtMultimedia" is not installed

Embedded linux board with iMX6ULL, I added following items to my image:
   qtbase \
   qtbase-plugins \
   qtserialport \
   qt5-env \
   libxkbcommon \
   qtquickcontrols \
   qtquickcontrols2 \
   qtmultimedia \
   qtmultimedia-plugins \
   qtmultimedia-qmlplugins \
   gstreamer1.0 \
   gstreamer1.0-libav \
   gstreamer1.0-plugins-good \
   gstreamer1.0-plugins-bad \
   gstreamer1.0-plugins-imx \

Building qtbase with:
PACKAGECONFIG_append = " no-opengl accessibility fontconfig linuxfb libinput xkbcommon"
PACKAGECONFIG_remove = " examples tests gbm eglfs gl gles2 kms"

and qtmultimedia with:
PACKAGECONFIG_append = "gstreamer"

camera example with Qt5 widgets works properly.
declarative camera demo or any other QML example with multimedia module doesn't work because of following error:
QQmlApplicationEngine failed to load component
qrc:/main.qml:3 module "QtMultimedia" is not installed

In /usr/lib/qml there are no QtMultimedia related files:
> ls /usr/lib/qml/
Qt  QtQml  QtQuick  QtQuick.2  QtTest  builtins.qmltypes

also
>  opkg files qtmultimedia-qmlplugins
Package qtmultimedia-qmlplugins (5.12.3+git0+64981aee3b-r0) is installed on root and has the following files:

returns no files installed.
How to build QML multimedia plugins with Yocto?

Do you have any luck with the issue?

@G-S-E unfortunately no. According to my investigations qtmultimedia qml plugins needs opengl support to be compiled.
In the end I found way to connect to QML web camera image preview that I needed using gstreamer videosink from this project:
https://github.com/muddog/QRScanner
and rendering it as QML element using QPainter like in this example code
https://doc.qt.io/qt-5/qtqml-tutorials-extending-qml-example.html

add in your conf/local.conf with latest version 5.15
IMAGE_INSTALL_append = " qtmultimedia"
depends on DISTRO_FEATURES_append = " opengl"


Remove unnecessary clock disparity check (nitpick)

There is no need to mention clock disparity when comparing two static slot values (assuming the clock disparity is less than a slot, even then I don't think that's the intention).

Yup, you're right. I'm going to make a new PR pointing this to v011x for immediate release

Closing in favor of #1683 for immediate release


docs: fix --healthcheck-command option

Make the documentation agree with the code.
Related #3507

/ok-to-test
/approve

Commit needs a signoff - please run git commit --amend -s and re-push

/lgtm
Thanks @stefanb2


iComfort S30 supported?

Please confirm if this plugin supports the Lennox iComfort S30 thermostat or just the iComfort Wifi model.   My understanding is the API's are different.
Thanks!

I only have the iComfort WiFi, so I'm unable to test with other versions fo the thermostat. I just saw you posted to Dan's project as well. He understands the APIs a lot better. I'll try to watch for a reply from him and see if there is any updates on his side and if something should be updated on this project (and I can find time to do it).
Thanks

Thanks very much for the reply.   We'll both be watching for Dan's reply  :)