Update sample template takes very long when there are more than~250 samples

I have been waiting >20 mins for 1143 samples to update

Thanks @ackermag
For the time being, I've added this issue to the Alpha 0.1 milestone. My guess is that after the refactor that I'm doing in the metadata_template object this will go smoother. If after such refactor this is still an issue, we can re-assign to Alpha 0.2 and investigate the issue further.

@josenavas and I just tested with the latest version of master and a study with 1626 samples, the time went from 1 minute and 53 seconds (before relax-md-reqs) to 54 seconds. While this does reduce the amount of time it takes to update the sample template, it is still slow. @josenavas thinks that most of the time is spent opening/closing connections and this will likely be resolved with @antgonza's upcoming PR, I am bumping this to 0.2 for the time being.

@ElDeveloper could you test again to see if this is solved? Thanks.

Just tested this and it is still a problem.

Thanks. How long is it taking now?

About the same amount of time.

By the same amount of time you mean 54 seconds?

Ok, nix that it really took around 2-3 minutes (on microbio.me).

Ok, I think the only solution for making this happening faster is actually making sure that everything happens in a single transaction block. I don't see any other way of improving this w/o doing a considerable refactor. Let's test again once we make sure that everything in the code base is executed in transaction blocks.
If this is still an issue, then we will need to do some refactoring. which I more or less know how should be done, but is some kind of time consuming and we have other things that are more important than this.

Would it be possible to profile to determine what specifically is slow?

Moving this to alpha 0.3

Can this be closed?


Fix Integration Tests

Fix the following tests:
test_kafka_client_load_metadata
test_produce_fetch_messages
test_commit_offset
test_commit_offsets
test_consumer_poll
test_consumer_commit_messageset_no_consumes
This was broken due to the dependency and Rust upgrade
See this commit: https://github.com/kafka-rust/kafka-rust/commit/4e34584b6cf3fd0ddf950296c2211734bb94c108

@johnward running the tests locally worked fine for me. What issue were you seeing?

The SSL certificate is not set up localling so causes the secure mode to fail.
Running tests with KAFKA_VER=<IP_ADDRESS>, SCALA_VER=2.11, KAFKA_CLIENT_COMPRESSION=NONE, KAFKA_CLIENT_SECURE=secure
Need to check how the SSL certificates are set up for the Kafka docker image.
running 12 tests
test integration::client::test_commit_offset ... FAILED
test integration::client::test_commit_offsets ... FAILED
test integration::client::test_kafka_client_load_metadata ... FAILED
test integration::client::test_produce_fetch_messages ... FAILED
test integration::consumer_producer::consumer::test_consumer_commit_messageset ... FAILED
test integration::consumer_producer::consumer::test_consumer_commit_messageset_no_consumes ... FAILED
test integration::consumer_producer::consumer::test_consumer_non_existent_topic ... FAILED
test integration::consumer_producer::consumer::test_consumer_poll ... FAILED
test integration::consumer_producer::producer::test_producer_send ... FAILED
test integration::consumer_producer::producer::test_producer_send_all ... FAILED
test integration::consumer_producer::producer::test_producer_send_all_non_existent_topic ... FAILED
test integration::consumer_producer::producer::test_producer_send_non_existent_topic ... FAILED
failures:
---- integration::client::test_commit_offset stdout ----
thread 'main' panicked at 'called Result::unwrap() on an Err value: ErrorStack([Error { code: 336245135, library: "SSL routines", function: "SSL_CTX_use_certificate", reason: "ee key too small", file: "../ssl/ssl_rsa.c", line: 310 }])', tests/test_kafka.rs:116:44
stack backtrace:
0: rust_begin_unwind
at /rustc/cb75ad5db02783e8b0222fee363c5f63f7e2cf5b/library/std/src/panicking.rs:493:5
1: core::panicking::panic_fmt
at /rustc/cb75ad5db02783e8b0222fee363c5f63f7e2cf5b/library/core/src/panicking.rs:92:14
2: core::option::expect_none_failed
at /rustc/cb75ad5db02783e8b0222fee363c5f63f7e2cf5b/library/core/src/option.rs:1268:5
3: core::result::Result<T,E>::unwrap
at /home/john/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/result.rs:973:23
4: test_kafka::integration::get_security_config
at ./tests/test_kafka.rs:116:9
5: core::ops::function::FnOnce::call_once
at /home/john/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ops/function.rs:227:5
6: core::result::Result<T,E>::and_then
at /home/john/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/result.rs:708:22
7: test_kafka::integration::new_security_config
at ./tests/test_kafka.rs:105:14
8: test_kafka::integration::new_kafka_client
at ./tests/test_kafka.rs:79:57
9: test_kafka::integration::new_ready_kafka_client
at ./tests/test_kafka.rs:70:26
10: test_kafka::integration::client::test_commit_offset
at ./tests/integration/client/mod.rs:169:22
11: test_kafka::integration::client::test_commit_offset::{{closure}}
at ./tests/integration/client/mod.rs:167:1
12: core::ops::function::FnOnce::call_once
at /home/john/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ops/function.rs:227:5
13: core::ops::function::FnOnce::call_once
at /rustc/cb75ad5db02783e8b0222fee363c5f63f7e2cf5b/library/core/src/ops/function.rs:227:5
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.

Ah, it looks like openssl has made the default settings more strict, which is making the build host reject the key that the tests are making in memory. This should be a fixable problem though. I think when you see broken tests, the correct response is never to just delete the tests.

I agree. thanks


Negative KL divergence values

I have to congratulate you first on a great work done with this package. I think it will enable a lot of future work related to dataset shift.
My question is about my observation that this implementation sometimes produces negative values of the KL divergence. You did acknowledge in the paper that KL divergence is always non-negative. Is there an explanation for this?

Hi there,
Thank you for your kind words. Two possible explanations for this are that: (i) your probabilistic classifier is potentially misspecified, or (ii) your sample size is not big enough. Since we do not observe the true distributions or the density ratio (R-N derivative), the calculation can lead to negative estimates for the KL. This should not be a big problem if you also perform the hypothesis testing procedure (sometimes the p-value for Y|X can be misleading as well, it all depends on your model specification).
Does this help?

Ah that makes sense as the p-values of the hypothesis test tend to be relatively larger when the KL divergence is negative.
Thank you for the quick response.


Supports google cloud storage

What type of enhancement is this?
API improvement, Configuration
What does the enhancement do?
Supports google cloud storage service by opendal:

https://opendal.apache.org/docs/rust/opendal/services/struct.Gcs.html

Implementation challenges
No response

I'd like to work on this!!
#656 and #911 are helpful as references?

Thanks @NiwakaDev , This PR #1081 may also be helpful


Completion for ":cd" from 'cdpath'

The `cd` command should consider the `cdpath` setting for completion of
relative paths.

Currently the completion appears to use only the current working directory when
completing relative paths.

This means that `:cd foo<Tab>` should complete `foobar` if there exists
`/path/in/cdpath/foobar` (and `/path/in/cdpath` is in `cdpath`).

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 15 Jun 2015 at 6:55

Is this something that would be considedred to get merged into vim before I or someone else puts in the effort into implementing it?

Is this something that would be considedred to get merged into vim
before I or someone else puts in the effort into implementing it?

It's always nice when completion suggests all the matching values
correctly.
--
TALL KNIGHT: When you have found the shrubbery, then you must cut down the
mightiest tree in the forest ... with a herring.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I have the following custom commands with completion in viml, and I presume this should be implemented natively in C?
Any suggestions or pointers for where I can start?

I have the following custom commands with completion in viml, and I
presume this should be implemented natively in C?
Any suggestions or pointers for where I can start?

Yes, best is to implement this in C.
In vim.h you can find EXPAND_FILES_IN_PATH, which is like EXPAND_FILES
and using the 'path' option.  Similarly there would be
EXPAND_DIRS_IN_PATH instead of EXPAND_DIRECTORIES, and using 'cdpath'.
This code is in cmdexpand.c, e.g. set_context_by_cmdname().  But search
for all occurences of EXPAND_DIRECTORIES and check what needs to be done
for EXPAND_DIRS_IN_PATH.
I can't find a very good example, but parts that are needed are in the
findfile.c source file.  Probably you need to modify
ExpandFromContext(), which calls expand_files_and_dirs().  And probably
some lower level functions.
--
TIM: To the north there lies a cave,  the cave of Caerbannog, wherein, carved
in mystic runes, upon the very living rock, the last words of Olfin
Bedwere of Rheged make plain the last resting place of the most Holy
Grail.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Race Condition:  Attempting to load configurations before Zookeeper is connected

I'm using spring-cloud-zookeeper to configure my spring-boot app and roughly 1 out of 10 times the framework attempts to pre-instantiate beans before Zookeeper is actually connected, causing a failure. This appears to be a race condition.
To illustrate, whenever the following log message appears before application beans are instantiated by the application context, there's no problem:
[main-EventThread] o.a.c.f.state.ConnectionStateManager     : State change: CONNECTED

However, when that message doesn't appear in the console, errors such as the following occur:
2015-09-26 21:20:45.460  WARN 89108 --- [main] s.c.a.AnnotationConfigApplicationContext : Exception encountered during context initialization - cancelling refresh attempt
java.lang.IllegalArgumentException: Could not resolve placeholder 'readSql' in string value "${readSql}"
    at org.springframework.util.PropertyPlaceholderHelper.parseStringValue(PropertyPlaceholderHelper.java:174) ~[spring-core-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.util.PropertyPlaceholderHelper.replacePlaceholders(PropertyPlaceholderHelper.java:126) ~[spring-core-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.core.env.AbstractPropertyResolver.doResolvePlaceholders(AbstractPropertyResolver.java:204) ~[spring-core-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.core.env.AbstractPropertyResolver.resolveRequiredPlaceholders(AbstractPropertyResolver.java:178) ~[spring-core-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.context.support.PropertySourcesPlaceholderConfigurer$2.resolveStringValue(PropertySourcesPlaceholderConfigurer.java:175) ~[spring-context-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory.resolveEmbeddedValue(AbstractBeanFactory.java:807) ~[spring-beans-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.doResolveDependency(DefaultListableBeanFactory.java:980) ~[spring-beans-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.resolveDependency(DefaultListableBeanFactory.java:967) ~[spring-beans-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor$AutowiredFieldElement.inject(AutowiredAnnotationBeanPostProcessor.java:543) ~[spring-beans-4.2.1.RELEASE.jar!/:4.2.1.RELEASE]
    ... 23 common frames omitted

A workaround is to autowire a CuratorFramework member variable and block until connection is acquired.  E.g.:
    @Autowired
    public void setCuratorFramework(CuratorFramework cf) {
        this.cf = cf;
        log.debug("Checking ZK connection status...");
        if (!cf.getZookeeperClient().isConnected())
            try {
                log.warn("wasn't connected so blocking for up to 3 minutes...");
                boolean connected = cf.blockUntilConnected(3, TimeUnit.MINUTES);

                if (!connected) {
                    log.warn("Couldn't connect to ZK after 3 minutes!!");
                } else {
                    log.debug("ZK Connected...sleeping for 5 seconds to allow properties to load");
                    Thread.sleep(5000);
                }
            } catch (InterruptedException ie) {
                ie.printStackTrace();
            }

    }

Of course, this isn't ideal since one has to be sure that the above code executes before any application beans are loaded.
It seems that the CuratorFramework.blockUntilConnected() method should be called internally by the framework, perhaps in ZookeeperPropertySource.start().  The race condition mentioned in the comments of that method regarding TreeCache.process() may not be resolved by blocking until connectivity is achieved but it might be made somewhat less likely to occur(?).

Setting --spring.cloud.zookeeper.config.cacheEnabled=false (from #38) turns the cache off and hence this issue doesn't happen. Still need to fix it for caching.

Actually I can still make it happen with the cache off.  It happens intermittently when Zk takes longer to connect than usual.  I still think that CuratorFramework.blockUntilConnected()  needs to be in there somewhere.

Has there been any activity on this issue by any chance?  It's a very serious issue for me at this point. If necessary I can try to put together a test that will reproduce the issue.

That would be helpful.

OK I'll work on that

BTW, I was building with M1 when I said that I could reproduce the issue with with
--spring.cloud.zookeeper.config.cacheEnabled=false so the flag wasn't doing anything.  I'm sorry about that, forgot I had switched it.
However, with latest BUILD-SNAPSHOT and --spring.cloud.zookeeper.config.cacheEnabled=true, my app is hanging forever at the following line, right after ConnectionStateManager reports CONNECTED:
s.c.a.AnnotationConfigApplicationContext : Refreshing org.springframework.context.annotation.AnnotationConfigApplicationContext@24fb6a80: startup date [Thu Oct 01 23:57:16 PDT 2015]; parent: org.springframework.context.annotation.AnnotationConfigApplicationContext@4f18837a
This doesn't happen with the cache turned off (but I do encounter my race condition I was mentioning earlier).

@zacwolfe if it's hanging that would cause the race condition. What needs to happen, is that the property source need to not be cached. As a separate concern, we setup the cache so that @ConfigurationProperties beans can get auto-refreshed when zookeeper pushes an update.

I'm sorry Spencer, it was late and I said the opposite of what actually is happening.  With the cache turned off, the app is hanging right after zookeeper is reporting CONNECTED.  It does this every time and does so indefinitely--my colleagues who have been experimenting with it have reported the same thing.  With the cache turned ON we experience the race condition.
So if we could get it working with cache turned off I think that would allows us to move forward with our POC.

I'm unable to reproduce the hanging with the cache turned off. Do you have a project that can reproduce?

I can put one together but I have a feeling it's related to the time it takes to connect to ZK.  It only seems to happen when running locally and connecting to a remote ZK instance over VPN.
When I get a chance I'll do some more experiments...maybe I can simulate a slow ZK connect.

Ah, which is why I wouldn't see it.

Maybe something like https://github.com/h2non/toxy could help.

Hi Spencer, I have a test app that should reproduce the problem.  I put everything in a 1.3M zip file.  How should I get it to you?

I think you can attach arbitrary files to the issues now.

But not zip files https://help.github.com/articles/file-attachments-on-issues-and-pull-requests/
Email (spencer at gibb dot us) or dropbox or something.

I tried to attach my zip but it's complaining that "Unfortunately, we don’t support that file type.  Choose Files Try again with a PNG, GIF, JPG, DOCX, PPTX, XLSX, TXT, or PDF."

OK I had to give it a PNG extension. Instructions below:
I have an example that should reproduce the problem. I have a modified version of zookeeper that just sleeps for 5 seconds during connect and adds a 100ms slowdown during request handling.
The test app is based on this example http://therore.net/distributed-configuration-with-zookeeper-curator-and-spring-cloud-config/
There's a little setup involved.
Please download zookeeper 3.4.6 and untar http://apache.osuosl.org/zookeeper/zookeeper-3.4.6/zookeeper-3.4.6.tar.gz
Configure ZK with default settings using default port 2181 and start it.
Enter some data:
zkCli.sh
create /config ""
create /config/myapplication ""
create /config/myapplication/name Smegman
quit
Run the test app:
gradlew bootRun
It should boot up successfully and eventually start printing "Hello Smegman" every 10 seconds.
Kill the app, in zookeeper dir rename zookeeper-3.4.6.jar to zookeeper-3.4.6.jar.good and drop in the zookeeper-3.4.6_slow.jar I put in the zip.
Rerunning the test app should result in a failure.  Currently I have cacheEnabled: false in the bootstrap.yml so the app should hang.  Changing cacheEnabled: true should invoke race condition, resulting in the following error:
java.lang.IllegalArgumentException: Could not resolve placeholder 'name' in string value "${name}"
BTW, I'm running in windows 7...hopefully this behaves the same on your box. Please let me know if there's anything else I can do to assist.

Any luck reproducing the issue?  Please let me know if there's anything I can do.
thanks,
Zac

I plan on looking at it today.

So with no cache it's not "hanging" per se, it's doing a ton of work. Everytime a property is asked for, it asks zookeeper for all of it's properties. That's too much work. Still looking at this.

The cached version with blockUntilConnected(10, TimeUnit.SECONDS); works as the vanilla zk version.

Tested in several different environments and it's working great now.  Thank you!

@zacwolfe thanks for the follow up.

Worked for me:
spring.cloud.zookeeper.config.cacheEnabled=false


Migrate to foreign_type! macro

See https://docs.rs/foreign-types/0.4.0/foreign_types/
What's not clear to me is how this works with a parent type semantics: current macro in metal-rs supports it and uses it, but I don't see it in the foreign_types crate. cc @sfackler

What do you mean by parent type semantics? Upcasting? You'd probably end up making a separate trait and macro that wraps foreign_types!.

@sfackler I mean this - https://github.com/gfx-rs/metal-rs/blob/1b0c776d889a222bf986cb5adfdb5af3d0bfd9ea/src/lib.rs#L66
On a second look, it's just one impl Deref away from what you have. So we can just add it ourselves.


Patching reports success when it did not do anything

I'm running the command below and it reports success but the finalizers remain unchanged ("Finalizers: [kubernetes.io/pvc-protection]"):
kubectl patch pvc pvc1 -p '{"metadata":{"finalizers": []}}' --type=merge

This happens with some service account. If I run the patch command on my laptop, it works fine.
The patch command should report an error when it fails to apply the patch.

Just a quick thought, I haven’t had a chance to try it, but what if you use null or [null] instead of [].
I wonder if it reported success because it didn’t think there was anything to do... not sure how Json merge patch interprets [], maybe the same as {} which would be to do nothing.

@aleb Sorry, I missed the part where you said it works find from your laptop.
I tested and it works fine for me too.
Can you provide your OS (e.g. cat /etc/os-release) and the output of kubectl version on the machine that doesn't work and also for your laptop?

The OS is "Debian GNU/Linux 9 (stretch)".
The kubectl version:
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:17:28Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

If I use the service account credentials on my laptop with kubectl v1.18.0 I get a proper failure:
$ kubectl patch pvc pvc-1 -n namespace-1 -p '{"metadata":{"finalizers": []}}' --type=merge
Error from server (Forbidden): persistentvolumeclaims "pvc-1" is forbidden: User<EMAIL_ADDRESS>cannot get resource "persistentvolumeclaims" in API group "" in the namespace "namespace-1": requires one of ["container.persistentVolumeClaims.get"] permission(s).


Remove google-spreadsheet

Deploy preview for recalc ready!
✅ Preview
https://recalc-poyurvgct-tervay.vercel.app
Built with commit 810b83e0a5b04e359b7db1623093bbb2681fd17a.
This pull request is being automatically deployed with vercel-action


Search editor opens file from wrong directory when workspace has multiple directories opened

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
Version: 1.73.0 (user setup)
Commit: 8fa188b2b301d36553cbc9ce1b0a146ccb93351f
Date: 2022-11-01T15:34:06.111Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000
Sandboxed: No
OS Version:  21h2 build 22000.675

Steps to Reproduce:

Create a workspace with two directories of the same name but different path, e.g. D:\a\b\x, D:\c\d\x
Use search editor to search for something where the result is a file like D:\c\d\x\result.txt
Double clicking on result vscode tries to open file D:\a\b\x\result.txt which doesn't exist.

Duplicate of https://github.com/microsoft/vscode/issues/198159
Closing this
/duplicate


Setup CI/CD flows for API/UI

/gcbrun

/gcbrun

Temporary Url for review: https://business-ar-dev--pr-27-ss987aa9.web.app


Update api.py

Convert SparseDataFrame to dense before writing #221

Current coverage is 80.04% (diff: 50.00%)

Merging #224 into master will decrease coverage by 0.06%

@@             master       #224   diff @@
==========================================
  Files            12         12
  Lines           885        887     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            709        710     +1
- Misses          176        177     +1
  Partials          0          0

Powered by Codecov. Last update f621ec0...210f42d

moved to #223


feat(mpt): TrieNode retrieval

Adds a function to the trie node that allows for opening the trie at a
given path. As the trie is opened, intermediate nodes are persisted
within the trie node passed.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#174
#173  👈
#172
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @clabby and the rest of your teammates on  Graphite


refactor: remove background queue config options from SdkConfig

Part of: https://linear.app/customerio/issue/MBL-120/public-api-remove-unused-properties-from-sdkconfig
The BQ feature that deletes expired tasks will no longer work after this change. Because most of the BQ implementation remains inside of the codebase and has not yet been removed, I commented out the code that is no longer working so the code compiles. The commented out code could be deleted or fixed when the rest of the BQ code is taken care of in the codebase.

Stack:

#547
#546
#540
#539
#534 ⬅
#533
#532

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #540


TODO:

Documentation
Add test to verify that the nodeVersion points to the latest released Airnode version
Test it

I ran multiple examples, one for each cloud provider and local with HardHat. It works great, I found only a few little issues:

When trying the weather-multi-value example I can see that an object was (almost) printed into secrets.env after yarn create-airnode-secrets run.

The example curl command in weather-multi-value example for testing the API key doesn't work because the timestamp there is older than 5 days. I would suggest (if possible) using an endpoint that doesn't require a timestamp.
Also, I would suggest adding a warning the it takes a while for API key to become active (still weather-multi-value example)
When I tried to remove an Airnode instance from GCP I got an error saying that that's supported only for AWS

Also the Docker command for remove is missing GCP credentials.


ctrl+p file search returns "recent files" from files not within my currently opened project. Can we disable this?

This has happened too many times to me now where I am working on multiple projects within the same day, and after opening a new site directory within vscode and then pressing ctrl+p I click one of the top files expecting it to be within the directory I am sitting in, but then after making loads of changes and testing I realise nothing is changing... and that's because vscode has presented the recently opened file from the last project I was working on that had the same filename. This happens to me all the time and I wonder if others have this same issue?
In short, is there a way to disable recent files entirely so that I don't get this issue, or even better.. can we add an option to disable files from other workspaces or outside directories from displaying within recent files for the ctrl+p menu? If not is this an option that can be added?
Thanks, I would be very interested if anyone else would appreciate this feature :)

100% agreed, this is very disruptive..

Yes, I found this since I posted, and it does what I need. Thanks


 Now we're using a new version making it easy to pin back to 1.20. Later I'll look at why 1.21+ are failing (#313).
While I was at it I added tests on 1.21 (1.22+ is failing, again will look later, #312), and upgraded the setup-go action itself.
I have:

[x] Written a clear PR title and description (above)
[x] Signed the Khan Academy CLA
[x] Added tests covering my changes, if applicable (n/a)
[x] Included a link to the issue fixed, if applicable (n/a)
[x] Included documentation, for new features (n/a)
[x] Added an entry to the changelog (n/a)

merging before review since it's blocking some PRs and it's pretty safe, but lmk if there are any concerns and I'll follow up later!


fix: prevent attachment filename from overflowing the table cell

The filenames generally don't contain breakpoints and therefore
if a filename is longer than (1/3) of the page width it will overflow
and be displayed on top of the description.
This PR fixes the issue by inserting zero-width spaces between every
character in the filename.
Before

After

I guess it works 😅 but I wonder if Typst rendering (s|w)ould have a better way to break words

https://github.com/typst/typst/issues/674 is open, so should :D

I asked the folks on the Typst Discord server and this is the solution I was given. Couldn't figure out a better way myself either.


New part in ST.{lib,dcm}: STM32F030F4P6

I just received and assembled the boards that use this part, which I added to your library quite some time ago. The boards work fine, hence I submit my work. I'm happy to iterate over this if you feel something should be done in a different manner. Also happy to discuss on IRC.

Thank you!
Mostly looks good. Could you please add a datasheet URL in the "Datasheet" field though?


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-monocdk" workflow

See https://github.com/cdklabs/cdk-monitoring-constructs/pull/140#issuecomment-1127954067


test: validate missing encoding in to_string

Validates https://github.com/cloudflare/workerd/issues/1777 bug.

We might have a bug with character sets.

The application/x-www-form-urlencoded percent-encode set is the component percent-encode set and U+0021 (!), U+0027 (') to U+0029 RIGHT PARENTHESIS, inclusive, and U+007E (~).
The component percent-encode set is the userinfo percent-encode set and U+0024 ($) to U+0026 (&), inclusive, U+002B (+), and U+002C (,).
The userinfo percent-encode set is the path percent-encode set and U+002F (/), U+003A (:), U+003B (;), U+003D (=), U+0040 (@), U+005B ([) to U+005E (^), inclusive, and U+007C (|).

The user info percent-encode set contains /, but it doesn't get encoded.
cc @lemire

@anonrig Nice fix. :-)
I like that you added a test.


主题配置能否添加一个自定义选择颜色的配置呀

没必要，想要什么颜色自己改配置。

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days


[WIP] Fix auto-tagging in release_testing_setup.sh for prerelease.yml

https://github.com/firebase/firebase-ios-sdk/pull/13217 did not work as expected and the prerelease nightly has been broken. I think the takeaway is that the GITHUB_TOKEN cannot be used in new clones, only in the workflow's original checkout context.
The release workflow also uses this script, and I want to preserve its function until I get the prerelease workflow working, so I have some temporary logic added to do so.
#no-changelog

Seemingly working in manual invocation: https://github.com/firebase/firebase-ios-sdk/actions/runs/10187798681/job/28183076729


How to remove Cluster Caption in header?

Hi, how to remove Cluster Caption in balloon header?

if my balloon content body and cluster caption are same, in right side we have duplicated content

Hi @AliveDD! Can you specify which behaviour from Yandex.Maps JS API you are trying to recreate?

I went through Yandex maps docs quickly and couldn't find an easy solution with their API. Will keep this issue open if somebody can come up with one 👍

Closed due to inactivity

Did somebody find the solution?

clusterBalloonItemContentLayout: ymaps.templateLayoutFactory.createClass('{{ properties.balloonContentBody|raw }}'),


NaiveRateLimit might be too naive?

From the comment over on https://github.com/vladimir-bukhtoyarov/bucket4j/discussions/243:

Small off-top regarding to source code of RateLimitUtil.kt - storing limiters inside concurrent hashmap without expiration can be a cause of OOM because hashmap will continuously increase each time when new key found, and will not decrease when old key are not used for long time. In the my example I will use Bucket4j integration with Caffeine in order to bring expiration and maximum size of collection with buckets.

@vladimir-bukhtoyarov, the whole map is cleared at the specified interval: https://github.com/tipsy/javalin/blob/95ab1d2a75ed6af3b6863c9dd8af6ab29acb59ac/javalin/src/main/java/io/javalin/http/util/RateLimitUtil.kt#L28
Is this not sufficient?


Coerse ClosedStruct to Hash

This makes ClosedStruct able to behave like an Hash.
imagine the following method:
def create_connection(ip:, port:, **)
  puts "connecting to #{ip}:#{port}"
  # doing crazy stuffs [...]
end

The ip and port are both available in my config struct:
class Configuration < ClosedStruct
  def initialize(port: 80, ip: '<IP_ADDRESS>')
    super(port: port, ip: ip)
  end
end

I want to directly pass the struct into the method using the double splat operator like this:
config = Configuration.new(port: 8080)
create_connection(**config)
# "connecting to <IP_ADDRESS>:8080"

Since the to_h method is already defined this pr cannot be changed conform the request.
Thanks for your time :)

Ah, of course it is :D


Fix SSL update bug

Thank you for writing serverpilot-shell! You've saved our company so much time by allowing us to easily automate ServerPilot app creations and updates.
Enabling AutoSSL for an app using:
serverpilot apps ssl update $appid auto true
Returns this error:
Error: Invalid argument: auto
Removing the quotes for these boolean values fixes it.

@n8kowald I'm glad you've found this tool to be helpful, thank you for the pull request!


Things to keep in mind

Maybe is it good to add a point that this applies to the OS and Server roles only, and that additional Software (like SQLServer) is not subject to Automatic exclusions - to date it was not clear to me.... (I just assumed this list was not listing all Automatic exclusions and that other lists could exist for e.g. SQLServer)

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

ID: 1664d524-cf0b-a4d4-695a-114f5df1d33e
Version Independent ID: 31bff983-8c0c-915c-6f88-f5a0a719987c
Content: Configure Microsoft Defender Antivirus exclusions on Windows Server
Content Source: microsoft-365/security/defender-endpoint/configure-server-exclusions-microsoft-defender-antivirus.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@msbemba Thank you very much for the contribution and updating the documentation with PR. @andrePKI Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


fix broken josm-link

only "zoom" dont works

I finally got back to this and decided to offer both load and select, and to only load the object itself.
Thanks to both of you for your pull requests and comments!


problem with travis?

I'm getting this weird message about not being able to clone the repository to test it
https://travis-ci.org/niallrobinson/iris/builds/45550198

Looks like a travis problem. have you tried re-starting the build?

I've restarted it about 5 times now, over that last three days, but it failed again just now.

sorry - wrong button

Looks like you've created your branch from an old version ... this was fixed in #1113.

wow - that is old...thanks


feat: updated 404 not found page

Pull Request
Description
#1298 solved
Closes #
Type of change

[ ] Adding a new Resource
[ ] Adding a new Category
[ ] Documentation enhancement or fixes
[x] Website enhancement or fixes

Checklist:

[x] I read carefully CONTRIBUTING.md
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Screenshots (if appropriate):

responsive

Additional

hi @jfmartinz review the PR


Update DB semantic conventions

https://github.com/open-telemetry/opentelemetry-specification/pull/575

This may fall out of scope but would it make sense to have a shared opentelemetry-semantic-conventions package that exports the enum of all attribute names and such? If so should this be package be hosted in the core repo?

There is already such a package. https://github.com/open-telemetry/opentelemetry-js/tree/master/packages/opentelemetry-semantic-conventions

perfect!

@dyladan re: https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/semantic_conventions/database.md#mysql
Please take note of span name and db statement.
Shall we proceed like as such? Currently span name is  e.g.: mysql:query:SELECT, the value of db statement is that of span name and we have a custom attribute MYSQL_VALUES with the  values. To adhere to spec, MYSQL_VALUES would be gone, they would be templated into the sql for db statement, and the non templated sql statement would be the span name.
If we proceed as per spec, should we worry about sanitization?

We should follow the spec

FYI I don't plan on tackling pg or pg-pool so those are up for grabs.


Remove support for BRIG from openmp plugin

I think this is obsolete. Any objections?

This is fixed on dev (by removing it)


WIP composable primer styles

This introduces a new module, Primer::Styleable which adds methods to a
component that generate primer based class names.

Personally I think that View Components should treat primer classes as second class citizens (in the future these perhaps shouldn't be allowed?). Hence I'm actually more in favour of using them (for now) explicitly (i.e.: classes: "d-inline-block etc...") or maybe even primer_classes: "...".

Personally I think that View Components should treat Primer classes as second class citizens (in the future these perhaps shouldn't be allowed?). Hence I'm actually more in favour of using them (for now) explicitly (i.e.: classes: "d-inline-block etc...") or maybe even primer_classes: "...". Finding new ways to express them could make it hard to get rid off them.

I think that's an interesting thought, and brings up a really good point, how much do we want to couple these components to Primer through methods? Additionally, how coupled do we want to allow people to opt-in to components being coupled to primer?
The original goal of this approach is to help remove the need to use **kwargs in so many components. The thought behind this approach was to build a module that you can include that gives you explicit methods that generate primer classes. I think one advantage here is that the DSL could be replaced/updated to support any CSS framework's classes. So if we update primer classes upstream we could easily update these methods to support the new version's CSS classes.
I'd also be 👍 on removing **kwargs in favor of something like:
def initialize(tag_name:, primer_classes:)

Assuming that would be a 1-to-1 replacement.

So if we update primer classes upstream we could easily update these methods to support the new version's CSS classes.

Is this really a realistic scenario?

Is this really a realistic scenario?

I believe so, it looks like one of the last times we had a breaking CSS name change was in December 2019 here, https://github.com/primer/css/pull/966 so there is some precedent.

Closing for now, since we want to stick with a more React-like API


some README fix

WHAT
Some cosmetic changes
WHY
English correction, doc consistency, etc.

Codecov Report

Merging #51 (45ed4d3) into main (d22984e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #51   +/-   ##
=======================================
  Coverage   84.67%   84.67%
=======================================
  Files          17       17
  Lines         535      535
=======================================
  Hits          453      453
  Misses         74       74
  Partials        8        8

Continue to review full report at Codecov.

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


can you offer native code?

can you offer native code?

There is no plan to open the source code at present.

@jpxiong Can you tell me about the steps of the RTMP publishing?

Depending on librtmp.


Add methods to HttpHeaders.Common for IfMatch and IfNoneMatch header creation

IfMatch and IfNoneMatch should have methods that accept an ETag parameter for creation of these header types.

Considering it's not a customer-facing API we can handle a bit of complexity in client implementations.


add a local (example) credential revocation plugin

@italobb please add details

The plugin will implement the ICredentialStatusManager interface, allowing the revocation of credentials using a simple centralized revocation method, which we'll call SimpleStatus. The credentialStatus field of a credential using the SimpleStatus will have the following format:
    "credentialStatus": {
        "id": "https://example.com/credential/status/74636220-fd6c-49ba-bcbf-fe858bcc2729",
        "type": "SimpleStatus"
    }

How will the credentialStatus field of a VC be generated using this plugin?
The plugin MUST receive an URL prefix (in our example https://example.com/credential/status/) as parameter during its instantiation. The plugin will generate a random UUID for the credential and simply append it to the URL prefix. This new URL will be the credentialStatus.id.

How the credential status will be served?
This plugin must be instantiated by an agent in a remote-server which will pass it's server URL as a parameter for the plugin instantiation. The plugin will offer a method which can be used by the server to serve the credential status response.

Where will the revocation information be stored?
The plugin will receive a storage provider as parameter and a simple storage provider will be implemented in the remote-server module.

How secure is this credential status method?
It's really a demonstration of a credential status plugin which shouldn't be used in production. Some important issues in this example:

It won't check if the one generating the credentialStatus is the same who will issue the credential.
There's no strong linkage between the credential itself and the credentialStatus.id. Anyone can use that same reference in another VC without any check.
The URL which respond by the status won't return a signed payload or VC. It`ll simply return if that VC was revoked or not without confirming the authorship of the VC.


Terminal window needs to be closed twice after code.cmd invoked

Launch Code from the prompt via %installdir%\bin\code.cmd
Exit the prompt (via X-out, exit command, etc)

Expected Terminal exits
Actual Terminal does not exit, it becomes unresponsive.  Need to take exit action a second time to close the window.
Version 0.10.5
Commit dfc08dc9edf6607483b193bf934d3badc3fa2e11
Date 2015-12-17T16:08:04.766Z
Shell 0.34.5
Renderer 45.0.2454.85
Node 4.1.1

Seems related to https://github.com/Microsoft/vscode/issues/1109. There are different quirks specific to launching with code.exe and code.cmd

Likely a problem with the Command Prompt, won't do nothing here.


MusicAssistant' object has no attribute 'aiobrowser'

What version of Music Assistant has the issue?
2.0.0b103
What version of the Home Assistant Integration have you got installed?
2014.1.0
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
2024-02-22 10:28:24.521 ERROR (MainThread) [music_assistant.webserver] [546471939024] Error handling message: CommandMessage(message_id=130, command='config/providers/save', args={'provider_domain': 'airplay', 'values': {'enabled': False}, 'instance_id': 'airplay'})
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/controllers/webserver.py", line 374, in _run_handler
result = await result
^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/controllers/config.py", line 259, in save_provider_config
config = await self._update_provider_config(instance_id, values)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/controllers/config.py", line 711, in _update_provider_config
await self.mass.unload_provider(config.instance_id)
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/server.py", line 455, in unload_provider
self.aiobrowser.types.discard(mdns_type)
^^^^^^^^^^^^^^^
AttributeError: 'MusicAssistant' object has no attribute 'aiobrowser'
How to reproduce
Disable airplar or change airplay name
Music Providers
Spoitfy
Player Providers
Airplay
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.1.6
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

Thanks. This is being looked at.

his is fixed in the next release, only happens when you reload a provider.


Rendering a new View.Grid displaces content inside

When switching from one View.Grid to another, it seems to displace the content inside as it reuses the attributes such as the coldefs and rowdefs from the previous grid.
I believe this could be a similar issue to #622 which was fixed in PR #627. However, I am experiencing similar behaviour to this issue now.
I am using Fabulous 0.54.0 and Xamarin.Forms <IP_ADDRESS>6
Reproducing this issue with:
let view (model: Model) dispatch =
    match model.Page with
    | One ->
        View.ContentPage(
            content = View.Grid(
                coldefs = [Star; Stars 25.; Star],
                rowdefs = [Auto; Auto],
                children = [
                    yield View.Label(text = "Page One").Row(0).Column(1)
                    yield View.Button(text = "Change", command = (fun () -> dispatch RenderTwo)).Row(1).Column(1)
                ]
            )
        )
    | Two ->
        View.ContentPage(
            content = View.Grid(
                rowdefs = [Auto; Auto],
                children = [
                    yield View.Label(text = "Page Two").Row(0)
                    yield View.Button(text = "Change", command = (fun () -> dispatch RenderOne)).Row(1)
                ]
            )
        )

When clicking Change from Page One to Page Two, the content of the View.Grid in the second view shows as

I understand that the reason for this is when updating the views, the views are being reused from the canReuseView function
I worked around this issue by adding a unique automationId to both grids, in order to get a false return in canReuseView for the canReuseAutomationId prevChild newChild condition:
| One ->
    View.ContentPage(
      content = View.Grid(
          automationId="PageOne",
          coldefs = [Star; Stars 25.; Star],
          rowdefs = [Auto; Auto],
          children = [
              yield View.Label(text = "Page One").Row(0).Column(1)
              yield View.Button(text = "Change", command = (fun () -> dispatch RenderTwo)).Row(1).Column(1)
          ]
      )
    )

I'm not quite satisfied with this workaround as the automationId should be used for automation testing. Is this expected behaviour from the view reuse logic? or could there be some missing checks/edge cases inside this canReuseView function causing views to be reused when they shouldn't be?
TIA

@rillstone Thanks for the report!
v0.54 seems ok. I don't reproduce with the sample you provided.
But I do reproduce in 0.55-preview. Will try to fix that.
In the meantime, please use v0.54 instead.

Should be fixed in 0.55.0-preview3


aplayer怎么安装hexo的？看中文的文档没看懂

中文用户请注意：请尽量用英文描述你的 issue，这样能够让尽可能多的人帮到你。
If you want to report a bug, please provide the following information:

The steps to reproduce.
A minimal demo of the problem via https://jsfiddle.net or http://codepen.io/pen if possible.
Which versions of APlayer, and which browser / OS are affected by this issue?

是在hexo里面安装aplayer吧，可以使用npm安装或者直接引入js
https://github.com/YuVincentxuan/YuVincentxuan.github.io/tree/master/dist


bosh create-release should warn you for ruby cli based releases

When creating a release with the go cli on a repo which has releases previously created with the ruby cli, the cli doesn't complain but just creates a release which doesn't contain any blobs. You have to execute bosh reset-release to prepare the bosh release for the new go  cli. It will be more helpful if the cli warns you or fails in this case.

Closing - duplicate of #145


update codebase to use proper logging

In each of our Python files, implement proper logging (in lieu of print statements` via the following:
LOGGER = logging.getLogger(__name__)
...
LOGGER.info('test')
LOGGER.warning('careful')

I pushed a commit that fixes this in latest registry.

https://github.com/cga-harvard/HHypermap/commit/3ac723dcc57c1a17e46b1ad6a30789370af7f258


Add non_maximum_suppression that supports both CPU and GPU arrays.

Merge after #145.
This PR moves transforms/bbox/non_maximum_suppression.py to links/utils/bbox/non_maximum_suppression.py.
Here are few changes.

It supports both CuPy arrays and NumPy arrays. Supporting CuPy is very critical for training Faster RCNN. Usually, NumPy implementation (not the one in this PR, but Cythonized version) is 10x slower than CuPy implementation. Not only that, the magnitude of speed is quite big taking 200ms per iteration for NumPy  version.
Change interface of non_maximum_suppression to not return transformed bounding boxes.
Add score option.
Add Cython dependencies. The Cythonized part is important for performance. Plain Python is order of magnitude slower.

@Hakuyume
Can you check this PR?

Updated directory structure and rebased dependencies.

I'm checking this PR. Please wait.

Thanks.
I found a bug in CPU mode when size of bbox is big and score is given.
I changed a test to catch this case b60ad48 and fixed it 4f8caca.
Other than that, I reflected your comments.

Jaccard overlap seems to be not a valid word since I could not find a usage of that word on any of the Google search results.
How about using Intersection over Union (IoU)?
Wikipedia says it is one of the correct word.

The Jaccard index, also known as Intersection over Union and the Jaccard similarity coefficient (From Wikipedia)

@yuyu2172 Sorry, Jaccard overlap is not a valid word. Jaccard index is valid. In the area of object detection, IoU (Intersection over Union) is more familiar. I think IoU is OK. Of course, we have to keep consistency.

I reflected our discussions to the code.

Fixed.

LGTM


Read stream of a pdf instead pdf path

Hi, is it possible in some way to put a stream of a pdf file instead the correct path of it? Unfortunatelly i've got only the stream and not the path. So i can't do something like:
var iFramePdfFile = "../vendor/ViewerJS/#" + myPdfPath;
because i've got only the stream. Thank you

:,(

Any update on this one?


⚠ Bofrost.lu has degraded performance

In 3e82412, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1008 ms

Resolved: Bofrost.lu performance has improved in 45eeb9c after 10 minutes.


Added Visual Studio prerequisites to getting-started

I could not get the sdk to work (debug) without update 2

Hi @peteraritchie, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, AZPRBOT;

@seanmck - Sean, could you review this update proposed by @peteraritchie in the public repo?

@seanmck - Second notice, please review

@peteraritchie: Thanks for the contribution. We actually hadn't realized that this Update 2 dependency had snuck in.
@tysonn: Since we now support multiple versions of VS, I will handle this in a slightly different way. You can go ahead and close without merging.


pre-commit complains about keys/SSL cert from this repo?

Hi, I hope this is relevant here and wish it could help/be resolvable. If not, I'll try opening an issue at pre-commit.
What version of this package are you using?
"standard": "^17.0.0",
What operating system, Node.js, and npm version?
macOS 12.6.2
node v18.13.0
npm 9.2.0
pre-commit 2.21.0
What happened?
When I run pre-commit autoupdate or pre-commit run --all-files , pre-commit complains about unmatching SSL certificates for/from this very repo.
What did you expect to happen?
It used to work and just run fine and apply its rules. It's been warning me for a while, and now it has stopped working completely.
Are you willing to submit a pull request to fix this bug?
I would if I could. I can't do Js.
Here is my config .pre-commit-config.yaml :
repos:
  - repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v4.4.0
    hooks:
      - id: detect-private-key
      - id: check-added-large-files
      - id: check-case-conflict
      - id: check-merge-conflict
        args: [--assume-in-merge]
      - id: check-symlinks
      - id: check-yaml
      - id: check-toml
      - id: check-json
      - id: pretty-format-json
        args: [--autofix, --no-sort-keys]
      - id: trailing-whitespace
        args: [--markdown-linebreak-ext=md]

  - repo: https://github.com/standard/standard
    rev: v17.0.0
    files: "./**/*.(js|jsx|ts|tsx|mjs|cjs)"
    hooks:
      - id: standard
        args: [--fix]

Here's an output:
$ pre-commit autoupdate
[WARNING] Unexpected key(s) present on https://github.com/standard/standard: files
Updating https://github.com/pre-commit/pre-commit-hooks ... already up to date.
Updating https://github.com/standard/standard ... already up to date.

$ pre-commit run --all-files
[WARNING] Unexpected key(s) present on https://github.com/standard/standard: files
[INFO] Installing environment for https://github.com/standard/standard.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command<EMAIL_ADDRESS>'-mnodeenv', '--prebuilt', '--clean-src', '/Users/gacallea/.cache/pre-commit/repoc1kllni1/node_env-default')
return code: 1
stdout: (none)
stderr:
    Traceback (most recent call last):
      File<EMAIL_ADDRESS>line 1348, in do_open
        h.request(req.get_method(), req.selector, req.data, headers,
      File<EMAIL_ADDRESS>line 1282, in request
        self._send_request(method, url, body, headers, encode_chunked)
      File<EMAIL_ADDRESS>line 1328, in _send_request
        self.endheaders(body, encode_chunked=encode_chunked)
      File<EMAIL_ADDRESS>line 1277, in endheaders
        self._send_output(message_body, encode_chunked=encode_chunked)
      File<EMAIL_ADDRESS>line 1037, in _send_output
        self.send(msg)
      File<EMAIL_ADDRESS>line 975, in send
        self.connect()
      File<EMAIL_ADDRESS>line 1454, in connect
        self.sock = self._context.wrap_socket(self.sock,
      File<EMAIL_ADDRESS>line 513, in wrap_socket
        return self.sslsocket_class._create(
      File<EMAIL_ADDRESS>line 1071, in _create
        self.do_handshake()
      File<EMAIL_ADDRESS>line 1342, in do_handshake
        self._sslobj.do_handshake()
    ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File<EMAIL_ADDRESS>line 196, in _run_module_as_main
        return _run_code(code, main_globals, None,
      File<EMAIL_ADDRESS>line 86, in _run_code
        exec(code, run_globals)
      File "/usr/local/lib/python3.10/site-packages/nodeenv.py", line 1519, in <module>
        main()
      File "/usr/local/lib/python3.10/site-packages/nodeenv.py", line 1093, in main
        args.node = get_last_stable_node_version()
      File "/usr/local/lib/python3.10/site-packages/nodeenv.py", line 1026, in get_last_stable_node_version
        return _get_versions_json()[0]['version'].lstrip('v')
      File "/usr/local/lib/python3.10/site-packages/nodeenv.py", line 1002, in _get_versions_json
        response = urlopen('%s/index.json' % src_base_url)
      File "/usr/local/lib/python3.10/site-packages/nodeenv.py", line 624, in urlopen
        return urllib2.urlopen(req)
      File<EMAIL_ADDRESS>line 216, in urlopen
        return opener.open(url, data, timeout)
      File<EMAIL_ADDRESS>line 519, in open
        response = self._open(req, data)
      File<EMAIL_ADDRESS>line 536, in _open
        result = self._call_chain(self.handle_open, protocol, protocol +
      File<EMAIL_ADDRESS>line 496, in _call_chain
        result = func(*args)
      File<EMAIL_ADDRESS>line 1391, in https_open
        return self.do_open(http.client.HTTPSConnection, req,
      File<EMAIL_ADDRESS>line 1351, in do_open
        raise URLError(err)
    urllib.error.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)>

Check the log at /Users/gacallea/.cache/pre-commit/pre-commit.log

I think that this is two unrelated problems, and the warning about Standard is that you have added files to a "repository mapping", which doesn't have a key config.

thank you @LinusU , removing files worked for the warning.
Regarding the python issue: I thought it was related sorry. Just found out that is a python issue, most probably with the brew version that has recently updated. Will open an issue there, as it is the first time this happens. In the past brew python has never complained. They may have forgotten to install certs with it.


move env var handling into rust and expose summarize env as a command

Description
We don't currently really handle env vars in a uniform way. This sets up clap to handle this by adding the clap env feature and apply the appropriate args, as well as pull the detection logic out of go.
This should also provide definitive semantics around overrides; Env vars are only considered for flags that have not been provided.

Should we add a note in the command line reference page that you can configure some arguments with env variables?

yeah, let's tackle that separately though, we can do it once for all flags


[web] add tex_image_* fns for VideoFrame

Hey 👋
texImage* bindings for VideoFrame were recently merged into web-sys (https://github.com/rustwasm/wasm-bindgen/pull/3077)
this PR allows using them on web through glow
gated them behind web_sys_unstable_apis, since VideoFrame is unstable in web-sys

bors r+


Update vulnerable packages for v5.2

5.2 has the following vulnerable packages:
System.Formats.Asn1
System.Private.Uri
Microsoft.Extensions.Caching.Memory
This PR updates already existing packages or takes direct dependencies on packages that can't be updated otherwise (at least I wasn't able to)

hmm... the 5.2 build seems to be broken and it seems it's not based entirely on YAML, so I'm not sure I can fix that.
e.g. the ADO-UB20-SQL19 needs to change to ADO-UB20-SQL22 it seems:
https://sqlclientdrivers.visualstudio.com/public/_build/results?buildId=101979&view=logs&j=3ad9154c-b66f-5e48-f703-44743de98f45
##[error]Failed to request agent. Exception Image ADO-UB20-SQL19 doesn't exist in pool ADO-CI-1ES-Pool
,##[error]The remote provider was unable to process the request.

@cheenamalhotra

We're aware of the issue, and have been working with engg teams offline to fix the ADO-UB20-SQL19 image so it can be used and we can continue testing with SQL 19.


Improve card design on the home page

Hi @shubhagarwal1 , I have assigned this task to you. Kindly complete within 2 days. Good luck!

Hey @shubhagarwal1 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


AZ - ELR feed not transported on 2/6

Our ops dashboard (https://prime.cdc.gov/metabase/question/23-needs-investigation-reports-not-transported-last-5-days)
shows that AZ's primary data feed on 2/6 failed to transport to the STLT.
Report ID a36593ac-d238-4bc0-ba7f-4a90e2094126
Action ID 6919991

Outcomes: Make sure AZ gets the file, and root cause analysis why the report failed

@jorg3lopez is looking into this.

Findings:
It appears that the transport params were missing at the time of sending the file during the batch process. Below is how it should look with the params:

Below it show this step is missing:

This needs further investigation. (03/02/2022)

I am blocked.  I still can't access the Azure storage using my CDC credential.

@anshulkumar-usds Test results have been resent to AZ. (03/17/2022)
File name:
az-covid-19-42ee57bb-6a6c-46fa-87f6-25c022125f02-20220317204900

3/18: AZ notified.


Feature Request: More Pleasing Design for Text Heavy Blogs

The Liebling design assumes blog has content with visuals (i.e. cover images). Many Ghost users write text-only articles, and we can see this from popularity of Medium-like minimal themes.
Liebling does not look so great if a post has no cover art. Can there be a more pleasing design for text-only articles? Either trigger a more minimal design for posts with no art, or user might choose to toggle entire Liebling theme to be "text-mode" for the article cards. Maybe like Casper but without the images.

There are different types of themes that focus on different needs, you can find themes for magazine-link content, minimal, photography, etc., I can't think of a theme that covers all those needs. This theme is not focused on text-only publications, if you want something like that then I would suggest you to look for a minimal (text-focused) theme.


"docker-machine create" on OSX / VirtualBox fails to start docker w/ devicemapper as storage driver

(docker-machine version: 0.7.0;   VirtualBox version: 5.0.16 r105871;  OSX version: 10.11.4 (15E65))
I've been merrily creating / wiping-out images for a while now, but sometime in the last two weeks, something broke.
I can no longer get a functioning VM w/ the devicemapper storage engine. (Note : aufs works just fine!)
docker-machine -D create -d virtualbox  --engine-storage-driver=devicemapper dev errors out with
Error creating machine: Error checking the host: Error checking and/or regenerating the certs: There was an error validating certificates for host "<IP_ADDRESS>:2376": dial tcp <IP_ADDRESS>:2376: getsockopt: connection refused
You can attempt to regenerate them using 'docker-machine regenerate-certs [name]'.
Be advised that this will trigger a Docker daemon restart which will stop running containers.

On connecting to the newly created vm, i see that docker has not started
root@dev:/mnt/sda1/var/lib/boot2docker# /etc/init.d/docker status
Docker daemon is not running

Running /etc/init.d/docker start fails, with the following in /mnt/sda1/var/lib/boot2docker/docker.log
time="2016-04-15T20:55:40.547380117Z" level=debug msg="[graphdriver] trying provided driver \"devicemapper\""
time="2016-04-15T20:55:40.547475500Z" level=debug msg="containerd connection state change: READY"
time="2016-04-15T20:55:40.547888243Z" level=debug msg="devicemapper: driver version is 4.31.0"
time="2016-04-15T20:55:40.549580492Z" level=debug msg="received past containerd event: &types.Event{Type:\"live\", Id:\"\", Status:0x0, P
id:\"\", Timestamp:0x5711554c}"
time="2016-04-15T20:55:40.547905237Z" level=error msg="devmapper: Udev sync is not supported. This will lead to data loss and unexpected
behavior. Install a dynamic binary to use devicemapper or select a different storage driver. For more information, see https://docs.docke
r.com/engine/reference/commandline/daemon/#daemon-storage-driver-option"
time="2016-04-15T20:55:40.549636885Z" level=debug msg="Cleaning up old mountid : start."
time="2016-04-15T20:55:40.549667567Z" level=fatal msg="Error starting daemon: error initializing graphdriver: driver not supported"

These are happing with a clean install of VirtualBox and Docker-Toolbox.
Perhaps something to do with static vs dynamic linking of devicemapper? Something else that changed recently?

I think that it's just not really supported in upstream Docker anymore.
Why use devicemapper?

Thats what I thought, just wanted to make sure that it wasn't something I was doing :-)

You're OK !  Give overlay a whirl if you're curious.  It's pretty good.

might as well - intertia had kept me on devicemapper lo all this while...

It's worth mentioning, there's certain things you can't do with overlay that you can do with devicemapper- notably install a JDK on an Ubuntu/Debian system.

notably install a JDK on an Ubuntu/Debian system.

Can you link the relevant issue? Why would using overlay prevent this?

Hello @dieswaytoofast ,
The default base os docker-machine uses for virtualbox ( boot2docker ) doesn't support the devicemapper file system.
I wrote a article how to fix this issue here: https://medium.com/@menzo/using-the-device-mapper-storage-driver-on-mac-osx-6327dbee3250


Issues with purged private data reconciliation

Scenario1
Create private data on peer1 and peer2, then stop peer2, purge private data on peer1 (block6), then restart peer2.
Upon restart of peer2, it should receive block 6 and gracefully handle the purge transaction.
Issue1 - peer2 gets block from orderer but can’t pull private data from peer1 (presumably since peer1 has already purged it). After 1 minute of trying, peer2 commits block with missing private data:
2022-12-14 23:06:29.665 EST 04a6 WARN [gossip.privdata] RetrievePvtdata -> Could not fetch all 1 eligible collection private write sets for block [6]…Will commit block with missing private write sets

Expected behavior - When peer2 requests private data from peer1, peer1 should respond that the requested private data has been purged so that peer2 doesn’t keep trying.
Then, upon commit with missing private data, peer2 purges the private data as expected:
2022-12-14 23:06:29.720 EST 04d9 INFO [pvtdatastorage] deleteDataMarkedForPurge -> Processed private data purges from private data storage channel=mychannel numKeysPurged=2 numPrivateDataStoreRecordsPurged=1

Issue2 - peer2 attempts reconciliation of the missing private data and succeeds:
2022-12-14 23:06:39.261 EST 051c DEBU [kvledger] CommitPvtDataOfOldBlocks -> [mychannel:] Committing pvtData of [1] old blocks to the pvtdatastore
2022-12-14 23:06:39.283 EST 051f INFO [gossip.privdata] reconcile -> Reconciliation cycle finished successfully. reconciled 1 private data keys from blocks range [6 - 6] channel=mychannel

Expected behavior: peer2 should not attempt to reconcile the missing private data in the first place, because the private data has already been purged. Shouldn’t the data be removed from the missing data queue when it got purged? Potentially this is happening because the purge logic runs prior to the commit logic, so the missing data entry gets added to the missing data queue but doesn’t get purged from the missing data queue (similar to issue #3828, so perhaps the same fix would apply).
Additionally, was the private data really reconciled per the log messages? Peer1 shouldn’t have the purged data anymore and therefore peer2 should have no way to get it. So why do we get messages on peer2 “Committing pvtData of [1] old blocks to the pvtdatastore” and “reconciled 1 private data keys from blocks range [6 - 6]”?
Peer1 log seems to show that it actually found the private data and returned it to peer2:
2022-12-14 23:51:06.272 EST 12f4 DEBU [gossip.privdata] filterNotEligible -> Found 1 for TxID  , collection assetCollection for <IP_ADDRESS>:7055 channel=mychannel
2022-12-14 23:51:06.272 EST 12fc DEBU [gossip.privdata] func1 -> Returning <IP_ADDRESS>:7055 1 elements channel=mychannel

But perhaps these messages are misleading... perhaps peer1 did respond to peer2 correctly that the private data is missing, and peer2 honored it by removing the entry from the missing data queue.
Scenario2
With peer2 down, create (block 10) and then purge (block 11) private data on peer1.
Upon restart of peer2, it should receive block 10 and block 11, and gracefully handle the create and purge transaction.
This issue starts similar to the prior issues… peer2 attempts to retrieve the private data of the ‘create’ block 10 for one minute and then gives up:
2022-12-15 00:00:05.138 EST 0ba1 WARN [gossip.privdata] RetrievePvtdata -> Could not fetch all 1 eligible collection private write sets for block [10]…Will commit block with missing private write sets

Issue3 - when peer2 attempts to reconcile the missing private data, this time it enters a reconciliation infinite loop with peer2 constantly requesting the private data from peer1:
2022-12-15 00:00:17.325 EST ab19 WARN [kvledger] logSkippedCollection -> Failed to reconcile pvtdata for chaincode=private collection=assetCollection blocknum=10 txnum=0 reason="Only subset of expected keys present"
2022-12-15 00:00:17.328 EST ab48 WARN [kvledger] logSkippedCollection -> Failed to reconcile pvtdata for chaincode=private collection=assetCollection blocknum=10 txnum=0 reason="Only subset of expected keys present"
2022-12-15 00:00:17.331 EST ab77 WARN [kvledger] logSkippedCollection -> Failed to reconcile pvtdata for chaincode=private collection=assetCollection blocknum=10 txnum=0 reason="Only subset of expected keys present"

We may want to address Issue3 first, since we may not be able to reproduce it after Issue1 and Issue2 are resolved.

Issue1 will be addressed in this issue.
Issue2 is not an issue.
Issue3 was resolved in PR #3854
Here's logs to help with Issue1.
peer3 was down while data was purged on peer1 in block 9. The logs demonstrate peer3 starting and attempting to pull the private data from peer1 for 60 seconds before giving up.
requesting_peer3.txt
replying_peer1.txt
Extracting the main INFO statements here for context:
2022-12-15 23:31:45.904 EST 0115 INFO [gossip.privdata] StoreBlock -> Received block [9] from buffer channel=mychannel
2022-12-15 23:31:45.907 EST 0164 INFO [committer.txvalidator] Validate -> [mychannel] Validated block [9] in 3ms

<60 seconds of retrieve attempts>

2022-12-15 23:32:46.306 EST 0b95 WARN [gossip.privdata] RetrievePvtdata -> Could not fetch all 1 eligible collection private write sets for block [9] (0 from local cache, 0 from transient store, 0 from other peers). Will commit block with missing private write sets:[txID: 8088ec18478c82e1b3144aeeb24678de3779f021a1baa486b32b010c03fb44ad, seq: 0, namespace: private, collection: assetCollection, hash: 4650cb9dc89a557bcd4efd0164bef7eef7147fdbe95fdf2230b0504f9468f08a
] channel=mychannel
2022-12-15 23:32:46.340 EST 0bc8 INFO [pvtdatastorage] deleteDataMarkedForPurge -> Processed private data purges from private data storage channel=mychannel numKeysPurged=2 numPrivateDataStoreRecordsPurged=1
2022-12-15 23:32:46.448 EST 0be4 INFO [kvledger] commit -> [mychannel] Committed block [9] with 1 transaction(s) in 141ms (state_validation=6ms block_and_pvtdata_commit=91ms state_commit=22ms) commitHash=[2173a46924c1b4326f42f767aa8e19ce1b9cf070c43eea26fd3b6b5de468fbd3]

Issue 1 resolved in https://github.com/hyperledger/fabric/pull/3907


s3 bucket access denied

Not clear which role I need to attach S3 actions to. Can you point me in the right direction?

@bollig This is the API stack, we're adding the permissions in a future version but to change it now via the api stack role

@bollig Highly recommend you use PERSISTENT_2 for the cost / bucket linking functionality. I put together a gist on how to use it w/ pcluster.
https://gist.github.com/sean-smith/be2954de5516c8b2b33abacc87ac9b61

3.1.0 will have a better error message for this.

@bollig do you still have problems related to this?


PULL Resquest to Monolog Official Repo

Hey why won't you make a pull request to the monolog official repo ?

i gonna do this

They consider my code as a third party package.
https://github.com/Seldaek/monolog/issues/1811#issuecomment-1598881011


[TTT] Show created games on games screen

This implements showing created games/challenges on the games screen.
Also changes Firebase implementation to Realtime Datastore.
fixes: #1121

Can you add a screenshot?

@snario A video is worth a thousand screenshots https://www.loom.com/share/04e72ba506d648d2a29e4aba919bd9e8


You are receiving this because you are subscribed to this thread.Message
ID: @.***>

ok : https://www.youtube.com/watch?v=W1iF_93XJAE
El mié, 4 may 2022 a las 8:29, PrismLight @.***>)
escribió:

@.**** requested changes on this pull request.
commit die
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/pull/8472#pullrequestreview-961849588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAFXN33WVWDFBCCMOHPOKDVIJ3SPANCNFSM5VAMLRJA
.
You are receiving this because you commented.Message ID:
@.***>

what are you

i dont get it, kinda unfunny i think


Prevent merge mined currencies using the same chain id from blowing up

This should either be tested for on startup (if practical), or caught/handled in such a way that doesn't cause jobs creation to fail

it should be able to. Call getauxblock and check if chainid is in a list of chainid's were using. Just like whats done with address versions on the frontend right?

@ahmedbodi Definitely, just needs to be coded up.

Okay. will get this coded up and a pr submitted in a few

@ahmedbodi did you ever create this?


Add plasma comparsion table

Adds Plasma Design Comparison table to docs. Closes #92

Thanks for the great work! @mul53


Redundant quantification when rendering GADT constructors from external packages

(Yet another issue noticed in well-typed/generics-sop#92.)
Run Haddock on this code:
module Bug ((:~:)(..)) where

import Data.Type.Equality

And you'll get this:

Notice that the type signature for Refl quantifies b, which is completely redundant! The culprit is these lines of code:
https://github.com/haskell/haddock/blob/fd56ac4d031963ecc137a5027d800bd8ed588264/haddock-api/src/Haddock/Convert.hs#L337-L342
This is constructing a type signature for a GADT constructor by displaying all of the universally quantified type variables following by the existentially quantified type variables. But this is not always what the user has written, as shown in the example above. We really ought to be using dataConUserTyVars instead, which takes this information into account.
Patch incoming.

Fixed in #1022.


GitPodify project

Add a .gitpod.yml file with some basic configuration. https://www.gitpod.io/docs/references/gitpod-yml

I would like to do, can you tell me what things we have to include

This should do-
# List the start up tasks. Learn more https://www.gitpod.io/docs/config-start-tasks/
tasks:
  - name: yarn
    init: yarn install
    command: yarn run dev

# List the ports to expose. Learn more https://www.gitpod.io/docs/config-ports/
ports:
  - port: 3000
    onOpen: open-browser

github:
  prebuilds:
    master: true
    branches: true
    pullRequests: true
    pullRequestsFromForks: true
    addCheck: true
    addComment: false
    addBadge: true

Can you assign me

I though you use npm but its yarn ok no problem, so i should simply copy all this at the root of the project in a yml file

sounds good, and just saw that there are both yarn and package locks. @FrancescoXX should we use yarn or npm?

for the time being you can choose any one @RishiKumarRay

@avneesh0612 please review


Usage problem with wix/react-native-navigation on Android

Hello, I've sucessfully used both libs together on<EMAIL_ADDRESS>but after upgrading to<EMAIL_ADDRESS>I encountered a new problem that I didn't have before:
To perform the upgrade I created a new react-native project in a separared directory, moved in the source files I wrote and reapplied all the manual integrations one by one (namely: rn-navigation, rn-share-extension and rn-swiss-knife), so we can assume there are no stale settings somewhere that didn't get properly updated. Also I used<EMAIL_ADDRESS>with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>with<EMAIL_ADDRESS>Now the problem is as follows:
For reasons that I don't fully understand, react-native-navigation changes the way the app is loaded by skipping the AppRegistry.registerComponent step and while before it didn't cause any problem, now "forcing" the execution of the main application makes the share extension crash.
myapp/MainActivity.java
import com.reactnativenavigation.controllers.SplashActivity;

public class MainActivity extends SplashActivity {

}

index.android.js
import { AppRegistry } from 'react-native'
import App from './src/app'
import Share from './src/share'

AppRegistry.registerComponent('MyShare', () => Share)

let app = new App()

The App instance, after some setup, calls Navigation.startSingleScreenApp and the main application starts.  Without that line, the share extension works perfectly but the main application doesn't load (I get a white screen), but, with it, the share extension crashes.
On iOS this doesn't cause any problem even with the new version of react-native, it's something on android that has changed.
EDIT: Apparently it keeps working on iOS when building in debug mode, when using the release scheme I get the same error and also in this case the "fix" is to delete the new App line.
Is there any way of checking "who" whats to call "what" and only instantiate App when the user wants the main application? Or anything else that can be done to obtain the previous convenient behavior?
For reference:
myapp/MainApplication.java
import android.app.Application;

import com.facebook.react.ReactApplication;
import com.facebook.react.ReactNativeHost;
import com.facebook.react.ReactPackage;
import com.facebook.react.shell.MainReactPackage;
import com.facebook.soloader.SoLoader;

import java.util.Arrays;
import java.util.List;
import com.reactnativenavigation.NavigationApplication;
import com.alinz.parkerdan.shareextension.SharePackage;
import com.github.alinz.rnsk.RNSKPackage;

public class MainApplication extends NavigationApplication {

  private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
    @Override
    public boolean getUseDeveloperSupport() {
      return BuildConfig.DEBUG;
    }

    @Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
          new MainReactPackage(),
          new SharePackage(),
          new RNSKPackage()
      );
    }
  };

  @Override
  public boolean isDebug() {
     // Make sure you are using BuildConfig from your own application
    return BuildConfig.DEBUG;
  }

  @Override
  public List<ReactPackage> createAdditionalReactPackages() {
     // Add the packages you require here.
     // No need to add RnnPackage and MainReactPackage
    return Arrays.<ReactPackage>asList(
          new RNSKPackage()
      );
  }

  @Override
  public ReactNativeHost getReactNativeHost() {
    return mReactNativeHost;
  }

  @Override
  public void onCreate() {
    super.onCreate();
    SoLoader.init(this, /* native exopackage */ false);
  }
}

Is it working with wix/reac-native-navigation???


CI[pyFV3]: retry even when connection is refused

According to @phschaad, the pyFV3 workflow occasionally fails with connection refused errors when attempting to download the test data set for regression testing. This PR configures wget to retry even in case of fatal connection errors (like 404 (not found) or 503 (connection refused)).
Let's see if this helps in the short term. In the long run, we should think about leveraging caches for that data because it rarely changes and it's kind of a wast to download it every time from scratch.
Follow-up PR from https://github.com/spcl/dace/pull/1731#issuecomment-2458989370.
/cc @FlorianDeconinck FYI

I'll see if we can mirror those data somewhere else as well


Error calling New-TfsTeam with non-existent backlog iteration

VERBOSE: [16:36:10.0054] [New-TfsTeam] Getting Iterations under path '\CM2021004 - Alocação Consultor DevOps'
new-tfsteam : Error retrieving Iterations from path '\CM2021004 - Alocação Consultor DevOps' (VS402485: The
Area/Iteration name is not recognized: CM2021004 - Alocação Consultor DevOps. )
At line:1 char:1
+ new-tfsteam $p -DefaultAreaPath $p -BacklogIteration $p  -Verbose
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [New-TfsTeam], Exception
    + FullyQualifiedErrorId : VssServiceException,TfsCmdlets.Cmdlets.Team.NewTeam

Fixed by #144 (release 2.2.0)


bug in the Polish language license deed for CC 3.0 Attribution license (possibly also for others, they only use CC BY)

Describe the bug
There is a link attached to the word “licencja” which leads to the following URL that is non-existent:
https://creativecommons.org/licenses/by/3.0/pl/
It seems that the proper link should be:
https://creativecommons.org/licenses/by/3.0/pl/legalcode
To Reproduce
Visit link above
Expected behavior
Link goes to correct resource
Screenshots

Desktop (please complete the following information)

OS: (ex. iOS)
Browser (ex. chrome, safari)
Version (ex. 22)

Smartphone (please complete the following information)

Device: (ex. iPhone6)
OS: (ex. iOS8.1)
Browser (ex. stock browser, safari)
Version (ex. 22)

Additional context
Submitted by A.Tarkowski. @kgodey

Hi, what is the typical time to fix this sort of bug? I'm asking because I signalled this issue to Diane (thank you @peterspdx for submitting the ticket) - upon request from the Polish Ministry of Digital Affairs which uses CC licenses across the Polish gov.pl public administration service and sees the faulty link as a critical issue. I hope that this can be fixed relatively quickly, thank you!


Updating dependencies and fixing error in run command

Adding some updates in order to keep the project functional and up-to-date.

Hey @guilhermeDTNA, Appreciate your contribution.  Will merge the PR, I'll also update to latest tailwind and update gulp accordingly this weekend.


Fix Travis breakage caused due to unimported idx module

Summary
I broke the CI build by using idx in the DraftTreeAdapter file, which is a FB-internal module not available on this project
Test Plan
yarn run test now passes

Closed by 3306ddf - lost the PR <-> Phabricator link by changing the summary in Phabricator.


Fixes for issues FreeRDP#4571 and FreeRDP#5620

Just took solutions from these issues to pull.

Sorry, my first pull, not familiar wtih github logic yet.
Will close this one and create as it should be.

@dragonuaix well, thank you for the try then ;)


Add cluster name prefix before secrets

What this PR does / why we need it:
Fixes https://issues.redhat.com/browse/MULTIARCH-2794
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/cc @mkumatag

/test e2e-aws

/approve

/retest

/retest-required
/hold cancel

/retest

/retest-required

/retest-required

/retest-required


⚠ Bofrost.lu has degraded performance

In 6464c1d, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1611 ms

Resolved: Bofrost.lu performance has improved in 2805a0f after 12 minutes.


DPO trainer example

Describe the bug
In trying DPO trainer example getting a bug with batch size and sharding , may be shard axis are not properly set or could be jax error as well , system used is V3 -32 , 4 hosts
To Reproduce
Steps to reproduce the behavior
Just run the dpo trainer examples
the error is this
outputs = self.layers(

File "/home/spars/.local/lib/python3.10/site-packages/easydel/modules/qwen2/modeling_qwen_flax.py", line 862, in call
output = pecs entry. Consider checking that in_specs are correct, and if so consider changing the mesh axis sizes or else padding the input and adapting 'functools.partial(<function ring_attention at 0x7f2e71b196c0>, axis_name='sp', float32_logits=True, platform=None, backend=None, autocheck=True, blocksize_c=None, blocksize_k=128, blocksize_q=128, dtype=<class 'jax.numpy.float32'>, softmax_scale=0.08838834764831843, deterministic=True, dropout_rng=None)' appropriately.
layer(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/modules/qwen2/modeling_qwen_flax.py", line 502, in call
attn_outputs = self.self_attn(
File "/home/spars/.local/lib/python3.10/site-packages/flax/linen/partitioning.py", line 568, in inner
return rematted(variable_groups, rng_groups, *dyn_args)
File "/home/spars/.local/lib/python3.10/site-packages/flax/linen/partitioning.py", line 565, in rematted
y = fn(scope, *args)
File "/home/spars/.local/lib/python3.10/site-packages/easydel/modules/qwen2/modeling_qwen_flax.py", line 382, in call
attentions = self.attention_performer(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/modules/attention_module.py", line 529, in call
return self.ring_attention(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/modules/attention_module.py", line 649, in ring_attention
attn_output = shard_map(
ValueError: shard_map applied to the function 'functools.partial(<function ring_attention at 0x7f9c76b396c0>, axis_name='sp', float32_logits=True, platform=None, backend=None, autocheck=True, blocksize_c=None, blocksize_k=128, blocksize_q=128, dtype=<class 'jax.numpy.float32'>, softmax_scale=0.08838834764831843, deterministic=True, dropout_rng=None)' was given argument arrays with axis sizes that are not evenly divisible by the corresponding mesh axis sizes:
The mesh given has shape (1, 32, 1, 1) with corresponding axis names ('dp', 'fsdp', 'tp', 'sp').

args[0] of shape float32[1,1,28,128], where args[0] is bound to functools.partial(<function ring_attention at 0x7f9c76b396c0>, axis_name='sp', float32_logits=True, platform=None, backend=None, autocheck=True, blocksize_c=None, blocksize_k=128, blocksize_q=128, dtype=<class 'jax.numpy.float32'>, softmax_scale=0.08838834764831843, deterministic=True, dropout_rng=None)'s parameter 'query', corresponds to in_specs[0] of value PartitionSpec(('dp', 'fsdp'), 'sp', 'tp', None), which maps array axis 0 (of size 1) to mesh axes ('dp', 'fsdp') (of total size 32), but 32 does not evenly divide 1

args[1] of shape float32[1,1,28,128], where args[1] is bound to functools.partial(<function ring_attention at 0x7f9c76b396c0>, axis_name='sp', float32_logits=True, platform=None, backend=None, autocheck=True, blocksize_c=None, blocksize_k=128, blocksize_q=128, dtype=<class 'jax.numpy.float32'>, softmax_scale=0.08838834764831843, deterministic=True, dropout_rng=None)'s parameter 'key', corresponds to in_specs[1] of value PartitionSpec(('dp', 'fsdp'), 'sp', 'tp', None), which maps array axis 0 (of size 1) to mesh axes ('dp', 'fsdp') (of total size 32), but 32 does not evenly divide 1

args[2] of shape float32[1,1,28,128], where args[2] is bound to functools.partial(<function ring_attention at 0x7f9c76b396c0>, axis_name='sp', float32_logits=True, platform=None, backend=None, autocheck=True, blocksize_c=None, blocksize_k=128, blocksize_q=128, dtype=<class 'jax.numpy.float32'>, softmax_scale=0.08838834764831843, deterministic=True, dropout_rng=None)'s parameter 'value', corresponds to in_specs[2] of value PartitionSpec(('dp', 'fsdp'), 'sp', 'tp', None), which maps array axis 0 (of size 1) to mesh axes ('dp', 'fsdp') (of total size 32), but 32 does not evenly divide 1

args[3] of shape float32[1,1,1,1], where args[3] is bound to functools.partial(<function ring_attention at 0x7f9c76b396c0>, axis_name='sp', float32_logits=True, platform=None, backend=None, autocheck=True, blocksize_c=None, blocksize_k=128, blocksize_q=128, dtype=<class 'jax.numpy.float32'>, softmax_scale=0.08838834764831843, deterministic=True, dropout_rng=None)'s parameter 'bias', corresponds to in_specs[3] of value PartitionSpec(('dp', 'fsdp'), 'tp', 'sp', None), which maps array axis 0 (of size 1) to mesh axes ('dp', 'fsdp') (of total size 32), but 32 does not evenly divide 1

Another issue , with ORPO trainer ,
TypeError: Can't instantiate abstract class ORPOTrainer with abstract methods _eval_epoch, _execute_eval_step, _execute_train_step, _finalize_training, _run_evaluation, _run_training_loop, _train_epoch

So tried, the DPO Trainer test , python_test/trainers/dpo_test.py , but it is also getting stuck at
params={"params": model.shard_params(model.params)},
File "/home/spars/.local/lib/python3.10/site-packages/transformers/modeling_flax_utils.py", line 272, in params
raise ValueError(
ValueError: params cannot be accessed from model when the model is created with _do_init=False. You must call init_weights manually and store the params outside of the model and pass it explicitly where needed.
Traceback (most recent call last):
How can I initialize from initial , I already have a SFT model , is there a way to transfer the parameters from trained model to initialized model , and the logic for this I couldn't understand, why it was necessary ?

Hi
Yea i can fix that for u in next 2,3 hours ill give you a Dpo example with an example for transferring model

@sparsh35, I have added a tutorial for the DPO trainer, and I am sorry if I was slow on that; I was checking some other things.
https://github.com/erfanzar/EasyDeL/blob/main/notebooks/dpo-trainer.ipynb

@erfanzar No problem at all , I know you are the sole maintainer of this project, things do get hectic.

Still getting issue , with dtypes,
Traceback (most recent call last):
File "/home/spars/train/dpotraining.py", line 161, in
main()
File "/home/spars/train/dpotraining.py", line 135, in main
output=dpo_trainer.train()
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 1160, in train
output, run_exception = self._run_training_loop(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 920, in _run_training_loop
current_step, run_exception = self._train_epoch(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 993, in _train_epoch
loss, metrics, run_exception = self._execute_train_step(batch)
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 1102, in _execute_train_step
self.model_state, dpo_out = self.sharded_train_step_function(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/func_utils/creators.py", line 271, in dpo_step
(__loss, (__chosen_rewards, __rejected_rewards)), grads = grad_fn(state.params)
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/func_utils/creators.py", line 254, in calculate_loss
losses = _loss_func(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/func_utils/loss_funcs.py", line 77, in _kto_pair_dpo_loss
chosen_kl = jax.lax.clamp(
TypeError: lax.clamp requires arguments to have the same dtypes, got int32, bfloat16, float32.
Using same code as given in example.

hello and thanks for re-opening issue, i have that fixed rn

@erfanzar Thanks , i also modified code for hinge loss and kto also to check, I think there is another bug related to jax array sharding , this code maybe isn't compatible with multi-host, I can't debug this error , can you help
[1 1 1 ... 0 0 0]]
jax.errors.SimplifiedTraceback: For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/spars/train/dpotraining.py", line 153, in
main()
File "/home/spars/train/dpotraining.py", line 127, in main
output=dpo_trainer.train()
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 1160, in train
output, run_exception = self._run_training_loop(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 920, in _run_training_loop
current_step, run_exception = self._train_epoch(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 993, in _train_epoch
loss, metrics, run_exception = self._execute_train_step(batch)
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 1102, in _execute_train_step
self.model_state, dpo_out = self.sharded_train_step_function(
ValueError: Passing non-trivial shardings for numpy inputs is not allowed. To fix this error, either specify a replicated sharding explicitly or use jax.experimental.multihost_utils.host_local_array_to_global_array(...) to convert your host local numpy inputs to a jax.Array which you can pass to pjit. If the numpy input is the same on each process, then you can use jax.make_array_from_callback(...) to create a jax.Array` which you can pass to pjit. Please see the jax.Array migration guide for more information https://jax.readthedocs.io/en/latest/jax_array_migration.html#handling-of-host-local-inputs-to-pjit-like-batch-etc. Got arg shape: (32, 1536), arg value: [[1 1 1 ... 0 0 0]

@sparsh35 thanks ill fix that one asap

can you try again and tell me if u still facing this issue

No this is the error ,
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/spars/train/sa.py", line 124, in
output=dpo_trainer.train()
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 1198, in train
output, run_exception = self._run_training_loop(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 951, in _run_training_loop
current_step, run_exception = self._train_epoch(
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 1024, in _train_epoch
loss, metrics, run_exception = self._execute_train_step(batch)
File "/home/spars/.local/lib/python3.10/site-packages/easydel/trainers/direct_preference_optimization_trainer/dpo_trainer.py", line 1140, in _execute_train_step
self.model_state, dpo_out = self.sharded_train_step_function(
File "/home/spars/.local/lib/python3.10/site-packages/jax/_src/array.py", line 1047, in _array_mlir_constant_handler
raise RuntimeError(
RuntimeError: Closing over jax.Array that spans non-addressable (non process local) devices is not allowed. Please pass such arrays as arguments to the function. Got jax.Array: bfloat16[3584,152064]

I haven't fix one yet... gime one week

There is error in ORPO trainer as well, getting Nan losses, maybe I might have incorrect format of data , looking at the code of test and trainer, I think one problem is not all tokenizers have BOS token id , it is not handled here, even after debugging this I am gettting Nan losses, can't find the issue though , if you can please check it.

@erfanzar .

@sparsh35 sure ill check and update orpo trainer


How to access

Salam,
I want to know how to login to test it
thanks

Hello, it's been very long since last time I logged in this system, but if you can run it successfully, maybe try the admin account, account name is admin and the initial password might be 10086, then you can try to create different types of accounts to test.

Thanks,
I tried it, but it doesn't work.


Dataflow semantic of assignments

Hi, I just noticed that data does not propagate over multiple assignments, e.g., a = b = c.  The dataflow semantic for assignments should include the return value as output.

Actually, it's my understanding that the return value should be considered as output for all expressions, e.g., a = val++ does not work with the current semantic as well. Something like sink(val++) does work, though.
void sink(int);

void foo() {
	int val = 42, a, b;

	a = b = val;
	sink(a); // broken
	sink(b); // ok
}

void bar() {
	int val = 42, a;

	a = val++; // broken
	sink(a);
}

void baz() {
	int val = 1336;

	sink(val++); // ok
}


Feature Request : make groupedLines of exports based on empty lines

Following this closed feature, since TS 4.7 organizeImports is groupping imports base on empty line separator.
By default, organizeImports also organize exports.
I need the same behaviour on exports, which are sometimes dependant of previous export (and which should not be reorder globally) :
Example - Before
export d from "D";
export a from "A";
export b from "B";

export c from "C"; //depends on D

{...}

Current behaviour
export a from "A";
export b from "B";
export c from "C";  //depends on D which is not currently exported => causes a build error
export d from "D";

{...}

Desired behaviour
export a from "A";
export b from "B";
export d from "D";

export c from "C";  //depends on D => no error

{...}

Thanks

Has anyone found a workaround/way to disable export sorting? My team relies on the other features of organize imports (like removing unused imports), but has a few index files that need to be manually sorted like the examples above. This feature clobbers our ability to do that.


Various type related fixes as found by Sonar

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9f74de0). Click here to learn what that means.
The diff coverage is 73%.

@@           Coverage Diff           @@
##             main   #14856   +/-   ##
=======================================
  Coverage        ?      65%
=======================================
  Files           ?      550
  Lines           ?    25948
  Branches        ?     3677
=======================================
  Hits            ?    16895
  Misses          ?     8361
  Partials        ?      692

Impacted Files
Coverage Δ

src/client/activation/node/manager.ts
27% <0%> (ø)

src/client/activation/progress.ts
30% <0%> (ø)

src/client/common/installer/productInstaller.ts
44% <0%> (ø)

src/client/common/utils/sysTypes.ts
56% <0%> (ø)

src/client/pythonEnvironments/base/locators.ts
55% <ø> (ø)

src/client/activation/languageServer/manager.ts
79% <100%> (ø)

src/client/common/process/pythonToolService.ts
100% <100%> (ø)

src/client/debugger/extension/adapter/logging.ts
97% <100%> (ø)

...ion/interpreterSelector/commands/setInterpreter.ts
94% <100%> (ø)

src/client/providers/symbolProvider.ts
88% <100%> (ø)

... and 2 more


request for the “T+S+H+K” checkpoint

Thanks for your excellent work.
Can you release the checkpoint that trained on “T+S+H+K”  dataset ?
Also, can you tell me which of the 4 weight files provided, trained on the most data, has the best performance?
Thanks a lot.

Thank you for your attention. The checkpoints trained on "T+S+H+K" are streamflow-sintel.pth and streamflow-kitti.pth. The difference between them is that the latter has undergone more fine-tuning with more KITTI data. However, if you're referring to a checkpoint trained on a broader range of datasets, it would be streamflow-spring, which is fine-tuned on the Spring dataset based on streamflow-sintel.pth. Based on some prior observations, streamflow-things.pth and streamflow-spring.pth generally exhibit better generalization capabilities, as these two datasets contain much more samples.