support dumping metrics by instance

What problem does this PR solve?
For a large cluster, getSeriesNum may always fail due to quota limit. As a result, significant metrics like tidb_tikvclient_request_seconds and tikv_grpc_msg_duration_seconds are often missing, which blocks troubleshooting.
What is changed and how it works?
With this PR, if getSeriesNum failed, diag will fallback to by-instance dump mode, which get instance list first and call collectMetric by instance.
Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Code changes

[ ] Has exported function/method change
[ ] Has exported variable/fields change
[ ] Has interface methods change
[ ] Has persistent data change

Side effects

[x] Possible performance regression
[ ] Increased code complexity
[ ] Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation

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

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4c0b393) 30.21% compared to head (913f819) 30.21%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #470   +/-   ##
=======================================
  Coverage   30.21%   30.21%
=======================================
  Files          52       52
  Lines        4416     4416
=======================================
  Hits         1334     1334
  Misses       2911     2911
  Partials      171      171

Flag
Coverage Δ

unittests
30.21% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Compatibility with TFS versions

Is there an overview with TFS versions and the latest sdk version supported for that TFS version?
VSTS = latest version
TFS 2018.1 = 5.127.1 ???
TFS 2018 =
TFS 2017 =

A range would be nice as well, as in the range of versions that will work on a TFS version, for those of us that want to support older and newer versions.

We try not to break compat in the SDK, only some rest APIs might not be available.

I've had compatibility broken a few times. As an example, recently one of my extensions stopped working. At some point witClient.getFields() changed from return type WorkItemComment[] to WorkItemComments in VSTS, with no announcement (that I'm aware of). Is the WorkItemComments type in VSTS only? or VSTS and 2018.1? What about 2018 or 2017.x? There's no easy way to know what will work in different versions.

getFields should return WorkItemField[], is that not what you're seeing?

Oops, I mean witClient.getComments().

@damirvarga Is this what you're looking for?


need annotation for nginx proxy_buffer_size

If an upstream app has a header that's bigger than the default proxy buffer size, nginx returns a 502 and the client receives nothing.
Example logs...
2016/11/29 18:10:10 [error] 64#0: *59606 upstream sent too big header while reading response header from upstream, client: <IP_ADDRESS>, server: ~^subdomain\.(?<domain>.+)$, request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:80/", host: "subdomain.example.com"

[2016-11-29T18:10:10+00:00] - <IP_ADDRESS> - - - 502 - "GET / HTTP/1.1" - 732 - "-" - "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2936.0 Safari/537.36" - "~^subdomain\x5C.(?<domain>.+)$" - <IP_ADDRESS>:80 - subdomain.example.com - 0.005 - 0.005

The explanation/fix...
https://ma.ttias.be/nginx-proxy-upstream-sent-big-header-reading-response-header-upstream/
So, essentially just need to be able to add those proxy buffer lines to the config...
server {
  listen        80;
  server_name   host.tld;

  location / {
    proxy_pass       http://upstream;
    ...

    proxy_buffer_size          128k;
    proxy_buffers              4 256k;
    proxy_busy_buffers_size    256k;
  }
}

Reference:
http://nginx.org/en/docs/http/ngx_http_proxy_module.html#proxy_buffer_size

Do you have a expected relase date with this changes??

Thanks again @krancour!

No problem, @jshimko.

@gemoya these changes are now included in the latest Router release, v2.11.0.  Testing will commence shortly for the next Workflow release to include this version (Workflow v2.12) and we expect it will be available by tomorrow.


About the loss_distill

Hi, thank you for the excellent work and the release of the code!
I am a little confused about the approach to calculating loss_distill in line 1429 of xbert.py as shown in
                  loss_distill = -torch.sum(F.log_softmax(prediction_scores, dim=1)*soft_labels,dim=-1)

I think the size of both prediction_scores and soft_labels would be (batch_size, seq_len, vocab_size). And F.softmax is used in the last dimension for soft_labels in line 237 of model_pretrain.py, as shown in
                  mlm_output = self.text_encoder(input_ids,
                                                 attention_mask = text.attention_mask,
                                                 encoder_hidden_states = image_embeds,
                                                 encoder_attention_mask = image_atts,
                                                 return_dict = True,
                                                 labels = labels,
                                                 soft_labels = F.softmax(logits_m,dim=-1),
                                                 alpha = alpha
                                                )

Why is F.log_softmax used in the second dimension (dimension of seq_len) for prediction_scores?

Hi, thanks for your interest in our work.
Have you printed out the size of prediction_scores and soft_labels?

Hi, thanks for your response.
I printed out the size of them by setting batch_size in Pretrain.yaml to 2 and find that they have the same size, e.g. (2, 11, 30522).

Thanks for pointing this bug. I fixed this issue and merged to the mainline. More details can be found in this commit: https://github.com/uta-smile/TCL/commit/74a3e4f963a77ba43f2a2e2abe02bbeea22eba09


Neo4jGraphSail test is not functioning properly

When the test is run in IntelliJ, it works. When via mvn clean install, it fails. I believe it has to do with directory issues when getting a working directory. For now, I have commented out the test so Josh can have a look and fix it.
https://raw.github.com/tinkerpop/blueprints/master/blueprints-graph-sail/src/test/java/com/tinkerpop/blueprints/pgm/oupls/sail/Neo4jGraphSailTest.java

@peterneubauer, do you have any idea under what circumstances you would get a "lock obtained timeout" in a Neo4j Lucene index?  I can't make heads or tails of this issue so far, looking for any kind of clue.
testGetStatementsSP_OG(com.tinkerpop.blueprints.pgm.oupls.sail.Neo4jGraphSailTest)  Time elapsed: 1.149 sec  <<< ERROR!
java.lang.RuntimeException: org.apache.lucene.store.LockObtainFailedException: Lock obtain timed out<EMAIL_ADDRESS>        at org.neo4j.index.impl.lucene.LuceneDataSource.getIndexWriter(LuceneDataSource.java:604)
        at org.neo4j.index.impl.lucene.LuceneDataSource.getIndexSearcher(LuceneDataSource.java:487)
        at org.neo4j.index.impl.lucene.LuceneIndex.query(LuceneIndex.java:248)
        at org.neo4j.index.impl.lucene.LuceneIndex.get(LuceneIndex.java:187)
        at com.tinkerpop.blueprints.pgm.impls.neo4j.Neo4jIndex.get(Neo4jIndex.java:70)
        at com.tinkerpop.blueprints.pgm.oupls.sail.GraphSail$DataStore.getReferenceVertex(GraphSail.java:276)
        at com.tinkerpop.blueprints.pgm.oupls.sail.GraphSail.<init>(GraphSail.java:135)
        at com.tinkerpop.blueprints.pgm.oupls.sail.GraphSail.<init>(GraphSail.java:103)
        at com.tinkerpop.blueprints.pgm.oupls.sail.GraphSailTest.createSail(GraphSailTest.java:39)
        at com.tinkerpop.blueprints.pgm.oupls.sail.SailTest.setUp(SailTest.java:61)
        at junit.framework.TestCase.runBare(TestCase.java:132)
        at junit.framework.TestResult$1.protect(TestResult.java:110)
        at junit.framework.TestResult.runProtected(TestResult.java:128)
        at junit.framework.TestResult.run(TestResult.java:113)
        at junit.framework.TestCase.run(TestCase.java:124)
        at junit.framework.TestSuite.runTest(TestSuite.java:232)
        at junit.framework.TestSuite.run(TestSuite.java:227)
        at org.junit.internal.runners.JUnit38ClassRunner.run(JUnit38ClassRunner.java:79)
        at org.apache.maven.surefire.junit4.JUnit4TestSet.execute(JUnit4TestSet.java:62)
        at org.apache.maven.surefire.suite.AbstractDirectoryTestSuite.executeTestSet(AbstractDirectoryTestSuite.java:140)
        at org.apache.maven.surefire.suite.AbstractDirectoryTestSuite.execute(AbstractDirectoryTestSuite.java:127)
        at org.apache.maven.surefire.Surefire.run(Surefire.java:177)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:597)
        at org.apache.maven.surefire.booter.SurefireBooter.runSuitesInProcess(SurefireBooter.java:338)
        at org.apache.maven.surefire.booter.SurefireBooter.main(SurefireBooter.java:997)
Caused by: org.apache.lucene.store.LockObtainFailedException: Lock obtain timed out<EMAIL_ADDRESS>        at org.apache.lucene.store.Lock.obtain(Lock.java:84)
        at org.apache.lucene.index.IndexWriter.<init>(IndexWriter.java:1097)
        at org.neo4j.index.impl.lucene.LuceneDataSource.getIndexWriter(LuceneDataSource.java:591)
        ... 27 more

We should push to get closure on this. It works from IntelliJ, but from mvn clean install command line, exceptions are thrown. NOTE that this is not happening in Neo4jGraph. I suspect that this .clear() model that @joshsh is using in his test case is not solid enough.

1

Will also try to hunt down the batch graph thing, later this week.
/peter
Sent from my phone.
On Oct 19, 2011 12:09 AM, "Marko A. Rodriguez" <
<EMAIL_ADDRESS>wrote:

We should push to get closure on this. It works from IntelliJ, but from mvn
clean install command line, exceptions are thrown. NOTE that this is not
happening in Neo4jGraph. I suspect that this .clear() model that @joshsh is
using in his test case is not solid enough.
--
Reply to this email directly or view it on GitHub:
https://github.com/tinkerpop/blueprints/issues/174#issuecomment-2448540

If you find a Neo4jGraphBatch issue, please add a test case to:
https://github.com/tinkerpop/blueprints/blob/master/blueprints-neo4jbatch-graph/src/test/java/com/tinkerpop/blueprints/pgm/impls/neo4jbatch/Neo4jBatchGraphTest.java
If you can't solve it, add the test case and then comment it out and tell me in another issue, please. Thanks Peter.


Agave's slack communication channel not working

Agave's slack communication channel (for Q and A) is not working anymore. Here is link (not working):
https://slackin.agaveapi.co/

Sorry for the inconvenience. The project domain moved to agaveplatform.org http://agaveplatform.org/. You can find the registration app at: https://slackin.agaveplatform.org/ https://slackin.agaveplatform.org/.

On Jun 15, 2020, at 9:26 AM, Jiarong Guo<EMAIL_ADDRESS>wrote:
https://slackin.agaveapi.co/ https://slackin.agaveapi.co/

Hi Rion,
It seems that I can not join with this link
https://slackin.agaveplatform.org/
Is there any requirement? I am a researcher at Ohio State and developing
apps for cyverse.
best,
jiarong
On Tue, Jun 16, 2020 at 6:08 PM Rion Dooley<EMAIL_ADDRESS>wrote:

Sorry for the inconvenience. The project domain moved to agaveplatform.org
http://agaveplatform.org/. You can find the registration app at:
https://slackin.agaveplatform.org/ https://slackin.agaveplatform.org/.

On Jun 15, 2020, at 9:26 AM, Jiarong Guo<EMAIL_ADDRESS>wrote:
https://slackin.agaveapi.co/ https://slackin.agaveapi.co/

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cyverse/cyverse-sdk/issues/29#issuecomment-645036819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADN5TNKKGL4GOEQAVYX5YLRW7UOTANCNFSM4N6HNHAA
.

Ah, the Joys of Slack API changes. A redirect should propagate out in the next day sometime. Until the, you can access the redirect URL directly. https://join.slack.com/t/agaveplatform/shared_invite/zt-f7trw4eq-~xOlq~gR7LGSebOFWjhSWg https://join.slack.com/t/agaveplatform/shared_invite/zt-f7trw4eq-~xOlq~gR7LGSebOFWjhSWg.

On Jun 17, 2020, at 1:30 AM, Jiarong Guo<EMAIL_ADDRESS>wrote:
Hi Rion,
It seems that I can not join with this link
https://slackin.agaveplatform.org/
Is there any requirement? I am a researcher at Ohio State and developing
apps for cyverse.
best,
jiarong
On Tue, Jun 16, 2020 at 6:08 PM Rion Dooley<EMAIL_ADDRESS>wrote:

Sorry for the inconvenience. The project domain moved to agaveplatform.org
http://agaveplatform.org/. You can find the registration app at:
https://slackin.agaveplatform.org/ https://slackin.agaveplatform.org/.

On Jun 15, 2020, at 9:26 AM, Jiarong Guo<EMAIL_ADDRESS>wrote:
https://slackin.agaveapi.co/ https://slackin.agaveapi.co/

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cyverse/cyverse-sdk/issues/29#issuecomment-645036819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADN5TNKKGL4GOEQAVYX5YLRW7UOTANCNFSM4N6HNHAA
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/cyverse/cyverse-sdk/issues/29#issuecomment-645179447, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAF7WMSLVJSJLK76MLT5EKDRXBPHFANCNFSM4N6HNHAA.


#1326 - Support for property paths in the SparqlBuilder

This PR addresses GitHub issue: #1326.
Briefly describe the changes proposed in this PR:

Very basic property path support "proposal" which already works for my use cases

This should get some refinement and testing before it gets merged. Consider it a proposal to potentially work on together.
This is a PR is a "fork" of the Path class I'm developing for https://github.com/inception-project/inception because we need property path support there now.

I was thinking if it might be a sensible idea to have a SparqlBuilder.path(...) method instead of Path.of(...) - in particular given that Path is also the name of a Java standard class.

I was thinking if it might be a sensible idea to have a SparqlBuilder.path(...) method instead of Path.of(...) - in particular given that Path is also the name of a Java standard class.

Good point, hadn't thought of that. A suggestion: rename the Path class you introduced here to PropertyPath, and in addition introduce SparqlBuilder.path as a convenience shortform.

add to whitelist

@rdf4j-bot run tests

@reckart I have left this open because you mentioned you wanted to do further improvements to it. I'm happy to accept what you have you as-is, but if you want to improve it further soon I'll leave this open.  Keep in mind that we are planning a 2.5.1 patch release for this friday or early next week.

@jeenbroekstra I have applied the suggestions you made. Additionally, I have changed the signatures to accept only RdfPredicate instead of QueryElement. At least for the cases I am using this, it seems to be sufficient. Some thoughts:

there are no unit tests
the PropertyPath class contains only static methods. Maybe PropertyPaths would be a name for the class more in-line with the general naming conventions, e.g. GraphPatterns?
I have added the SparqlBuilder.path method as you suggested, but I am not sure it makes too much sense since the user in many cases will find other methods like zeroOrMore in PropertyPath. So I think it might be more sensible to rename PropertyPath.of to PropertyPaths.path(...) method and leave this out of SparqlBuilder.

Actually... I have applied the thoughts from the last two bullet points now.

So if you are fine with untested code, I think this looks ok now.

Yeah, the unit tests for sparqlbuilder in general are missing - this is something we missed when we accepted the original contribution. Improving test coverage for it is on our radar. See #1385

@rdf4j-bot run tests


Support for content without background

Currently, when content inside SwipeableActionsBox doesn't have any background or has translucent background color, backgroundUntilSwipeThreshold or swiped actions background is visible.
With backgroundUntilSwipeThreshold=Color.Transparent and content without background, it looks like this when swipe threshold is reached:

Can you apply a solid background to your content?

No because I have screens with gradient background

Whoops! I should have only drawn the background behind the active swipe action instead of drawing it behind the entire content. Wanna help me out by sending a PR? Making this change would also remove the need to have backgroundUntilSwipeThreshold.
Great app btw!

Great app btw!

Thanks!
I tried to implement this in #5. Decided to keep backgroundUntilSwipeThreshold because onSwipe doesn't get triggered until swipeThreshold so backgroundUntilSwipeThreshold can be used as UI indication of that.


Webauthn support types

📦 Pull Request
Types for WebAuthn support
🗜 Versioning
(Check one!)

[ ] Patch: Bug Fix?
[x] Minor: New Feature?
[ ] Major: Breaking Change?

✅ Fixed Issues
N/A
🚨 Test instructions
No tests for types
⚠ Update CHANGELOG.md

[x] I have updated the Upcoming Changes section of CHANGELOG.md with context related to this Pull Request.

:rocket: PR was released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>:rocket:


Formatted the creation date for the post creation into something more…

For this commit, I simply formatted the date into a more readable format before adding it to the ticket, in addition to setting the "assigned user" field to the user-inputted value for "assigned user" in the form. Both changes were in the "CreateTicket.jsx" file, so hopefully merging this should be smoother.

I added another field to the ticket, for the name of the user who created the ticket post. This field makes use of the react-client-session library to get the name of the user and put it in the ticket JSON when they create a new post.


[UI] Navigation highlight not working properly for subsections under Management section

Description
Subsections under Management section are not getting highlighted when navigated to those sections. But rather, Management is getting highlighted
Expected Behavior
The navigated subsection should be highlighted.
Screenshots/Logs
https://user-images.githubusercontent.com/75248557/123373245-b00d1e00-d5a2-11eb-8cc3-c6ce42317257.mov
Environment:

Meshery Version: stable-v0.5.26
Kubernetes Version: v1.19.7
Host OS: Big Sur
Browser: Brave

Contributor Resources:

(Contributing Docs](https://docs.meshery.io/project/contributing)

Hello, I am a newcomer to open source contributions and would like to work on this.
Currently reviewing the documentation/guidelines and setting up my environment.
I have joined the Layer5 Slack workspace.
Thanks!

Welcome @nishit-prasad 🥳 , there are a lot of people in the community willing to help people. Just ping on slack if you face any difficulties ✌🏽

Thank you! I really appreciate it!

@sudo-NithishKarthik
I think I found the root cause.
Part of the highlight logic is tied up with link value of the subsections (children).
https://github.com/layer5io/meshery/blob/master/ui/components/Navigator.js#L334
Setting them true seems to fix it. Wanted to make sure whether they're any issues with this change before I commit in my scratch branch?
I have posted this concern in #meshery-ui slack channel.

Also:
(If you refer screen recording)
I also noticed that the subsection SMI Results under Conformance have same href URLs.
So, if you select SMI Results subsection, you will see both Conformance and SMI Results highlighted.
That's because both have same href links (and another part of highlight logic is tied with it).
So the question is, if we have a distinct URL in place - I can add the fix in this or I am okay with anybody raising as a new issue.
Please advise.


Non-deterministic crashes of Legion/Realm/Regent examples in the CI

We have noticed some examples crash non-deterministically in the CI, so I plan to use this issue to track them, such that people won't be surprised if they see the same error in their development branch.
Such non-deterministic crashes are difficulty to reproduce. We may need to run multiple containers concurrently to increase the contention on the machine.

Realm::MutexChecker::lock_fail
This one is often seen in GASNetEx CIs, need to keep track if it also happens with GASNet.
Here is a job log: https://gitlab.com/StanfordLegion/legion/-/jobs/2775315984

Realm::ChunkedRecycler<T, CHUNK_SIZE>::~ChunkedRecycler()
At least one GASNetEXEvent is allocated but not freed. This one is often see in GASNetEx CIs.
Here is a job log: https://gitlab.com/StanfordLegion/legion/-/jobs/2791336163
Here is another issue to track it https://github.com/StanfordLegion/legion/issues/1304

FIXED: realm proc_group
Here is a job log: https://gitlab.com/StanfordLegion/legion/-/jobs/2810475096
(Fixed by https://gitlab.com/StanfordLegion/legion/-/merge_requests/784)
Crash again in Mac OS: https://gitlab.com/StanfordLegion/legion/-/jobs/4518715817

FIXED: crash in examples/separate_compilation.rg https://github.com/StanfordLegion/legion/issues/1478
unclear whether this is a crash in the code itself or an issue in the launcher (MPI/PMI)
Job log: https://gitlab.com/StanfordLegion/legion/-/jobs/3095304675

realm compqueuewith ucx
Here is the job log: https://gitlab.com/StanfordLegion/legion/-/jobs/4241109046

legion spy
https://gitlab.com/StanfordLegion/legion/-/jobs/4318846145

FIXED attach_file_mini on Mac OS with c++17
https://gitlab.com/StanfordLegion/legion/-/jobs/4433345645

Regent non-deterministic segfaults #1490

realm test_profiling
https://gitlab.com/StanfordLegion/legion/-/jobs/4505374541

realm deferred_allocs
https://gitlab.com/StanfordLegion/legion/-/jobs/4505374448

realm event_subsribe
https://gitlab.com/StanfordLegion/legion/-/jobs/4513555518

realm ctxswitch
https://gitlab.com/StanfordLegion/legion/-/jobs/4521720755

jupyter notebook timeout
https://gitlab.com/StanfordLegion/legion/-/jobs/4521720751

The completion queue test: https://gitlab.com/StanfordLegion/legion/-/jobs/3701536692

Do we have a bullet for {realm}: network still not quiescent after 10 attempts? Failing here:
https://gitlab.com/StanfordLegion/legion/-/jobs/4481778384

Here's a new failure for MutexChecker: https://gitlab.com/StanfordLegion/legion/-/jobs/4482517781

I fixed the attach_file_mini test.

Just another failure with realm_reductions.cc:

FWIW, that failure mode of the network not being quiescent is not specific to that test. I've seen it on lots of different tests.

Here's a fun variation of the mutex checker overflow:
https://gitlab.com/StanfordLegion/legion/-/jobs/4802099679

Another one for gcc9_cxx11_release_gasnetex_ucx_regent:
https://gitlab.com/StanfordLegion/legion/-/jobs/4803853742
Not sure if that's a duplicate

Is this a new failure mode?
https://gitlab.com/StanfordLegion/legion/-/jobs/4810462264

Another one for gcc9_cxx11_release_gasnetex_ucx_regent:

That is a known issue in the Legion master branch that is fixed in the control replication branch and is too hard to backport.

Is this a new failure mode?

No, it's already fixed and it was not intermittent.

Have you seen this one before (on latest control_replication)?
https://gitlab.com/StanfordLegion/legion/-/jobs/4811257857

In the latest control_replication, I'm still seeing:
https://gitlab.com/StanfordLegion/legion/-/jobs/4811257849

Have you seen this one before (on latest control_replication)?

Yes, and master and every other branch I've worked on. PMI does not like something in our docker setup.

In the latest control_replication, I'm still seeing:

Try again.

Some clues as to the problems with the PMI setup:
https://stackoverflow.com/questions/23237026/simple-mpi-program-fail-with-large-number-of-processes
https://stackoverflow.com/questions/29315216/mpich-example-cpi-generates-error-when-it-runs-on-multiple-fresh-installed-vps
Better to describe machines as IP addresses than host names.

We're setting mpirun -n 2 from https://gitlab.com/StanfordLegion/legion/-/blob/master/.gitlab-ci.yml?ref_type=heads#L312, which I believe will implicitly refer to localhost. Do you really think that will be an issue with the host name?

I have a suspicion that those failures are actually related to this commit that I made today:
https://gitlab.com/StanfordLegion/legion/-/commit/831103cb94264df3f6c7326cd84d3df0b1425b1f
It handles the race where you launch Legion on multiple nodes, the whole program runs on one node, and then that node exits even before the other nodes have even finished starting up., so the job scheduler starts trying to tear down the other processes before they're even done.

I am adding two more failures to make sure they get addressed:

https://gitlab.com/StanfordLegion/legion/-/jobs/6565586841
https://gitlab.com/StanfordLegion/legion/-/jobs/6578006791

We need to audit all reported failures here to see if any still remain

At least 1, 2, and 15 are still occurring since they also show up in the fuzzer tests (see #1745)

Here's a recent CI failure for 2: https://gitlab.com/StanfordLegion/legion/-/jobs/7919090909
Here's a CI failure for 15: https://gitlab.com/StanfordLegion/legion/-/jobs/7919090855

This might be related to 15, but not sure:
https://gitlab.com/StanfordLegion/legion/-/jobs/7920864325
runtime/realm/gasnetex/gasnetex_internal.cc:3596: bool Realm::GASNetEXInternal::check_for_quiescence(size_t): Assertion `0' failed.

There's also this, which might be a hang: https://gitlab.com/StanfordLegion/legion/-/jobs/7920864333
HELP!  Alarm triggered - likely hang!

@elliottslaughter This is a probably discussed already somewhere else but I recall you have done a "fuzz testing" done a relatively short time ago that exposed a number of bugs. Would you be able to describe what sort of fuzz tester is it? Or perhaps point to a place that has some context on it. I would be open to discuss integrating the fuzz testing for Realm. Either as a standalone tool that we run/maintain ourselves or something derived from what you have already done.

#1745 is the fuzzer-specific issue, maybe I'll answer over there since this thread is already quite long?


Does not work

I can not authorize the account.
Google complaining to
Authorisation Error
Error 400: redirect_uri_mismatch
I have no idea what is wrong with this.
The content in this section has been provided by the app developer. This content has not been reviewed or verified by Google.
If you’re the app developer, make sure that these request details comply with Google policies.
redirect_uri: https://www.google.com

Hi @JanosTurk, this is an error you receive in the Homebridge logs? Or when you're following the getting started guide?

The first part of the setup guide is fine. When I try to authorise my sandbox to get the refresh token and subscription id the google page comes up.
So I opened this URL:
https://nestservices.google.com/partnerconnections/project-id/auth?redirect_uri=https://www.google.com&access_type=offline&prompt=consent&client_id=oauth2-client-id&response_type=code&scope=https://www.googleapis.com/auth/sdm.service+https://www.googleapis.com/auth/pubsub
of course with my details. Assigned my gmail account as requested. Allowed permission to have access to my door entry. Then came up this error message above…

I wasted 12 hours on this one. I have done every single step again and again and I'm not able to authorize my Google Account. I gave up on this... It does not work or the documentation is wrong.
Access blocked: Door’s request is invalid
You can’t sign in because Door sent an invalid request. You can try again later, or contact the developer about this issue. Learn more about this error
If you are a developer of Door, see error details.
Error 400: redirect_uri_mismatch

I'm sorry you're having so much trouble @JanosTurk, however I really am not able to offer support for the Google, only for issues with the plugin itself.  As the FAQ says:
"The Nest plugin for Home Assistant uses much the same process (don't forget the "ONE IMPORTANT DIFFERENCE" section above). It has an illustrated guide that you may find helpful. You can also try reaching out to others on Discord or Reddit, some people there may be able to help."


Feature Request Operator | Custom CA bundle configuration in Console UI

Description
Today: Customers have to add CA bundles to the operator configuration at deploy time or if they know how they can add it to the user spec in the console.
Expectation: Customers can configure CA bundles via Console UI.

Worth testing this one using the existing option in Advanced settings tab.


[Discussion] information leakage while intaking the conversition?

Discussion
Hi, I have one concern about the model taking in the conversation. One conversation might have multiple questions and answers and it seems (from my understanding) the model is tokenizing and consuming the QAs in one conversation simultaneously. I wonder if my understanding is True, and if yes, could it influence the model? If some answers are based on the previous question, could that lead to some potential information leakage?

Yeah! This is a good question! A perfect conversational data should aviod such lekage!


Remove email campaigns factory

depends on !147
COM-1053
Description
The factory is not needed. I'd like to simplify creating the EmailCampaignsPage.

This needs a rebase @raphaelblum 🙏🏻


Change format for editable date and date ranges

Description
The goal would be to answer/extend this need even for editable date. The goal is not for display purposes but to change the order from MM/DD/YYYY to DD/MM/YYYY or more.
Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%.
[ ] Create related issue in taipy-doc for documentation and Release Notes.
[ ] Check if a new demo could be provided based on this, or if legacy demos could be benefit from it.
[ ] Ensure any change is well documented.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

works for me (tm)

I mean I tried it and it works ie you can edit a date with the specified format

I mean I tried it and it works ie you can edit a date with the specified format

Could you provide the code and taipy version used?

Here is the code that I tried on Taipy 3.1.1 Taipy GUI 3.1.4
from taipy.gui import builder as tgb
from taipy.gui import Gui
import datetime as dt
date_range = [dt.datetime(2024,1,5), dt.datetime(2024,1,10)]
with tgb.Page() as page:
    tgb.date_range(dates="{date_range}", format="dd-MM-yyyy")

Gui(page,).run(debug=True, use_reloader=True, port=5111)

Tried it on the develop branch (ie 4.0.0)

Indeed, sorry I didn't checked the dev version.

Implemented in 4.0.


Getting Data from a Database without refreshing using AJAX and PHP

Getting Data from a Database without refreshing using AJAX and PHP
Most Websites that operate without AJAX will have to reload the whole web page as soon as content changes from the database.
While searching from a search engine and having to click the search button to get information and reloading the whole web page can be frustrating Most times.
As web developers, we want to build a search engine that can throw out information from the database immediately after a search has been initiated in the search engine. AJAX (Asynchronous JavaScript and XML) is a technique used for creating fast and dynamic web pages, AJAX allows web pages to be updated asynchronously by communicating with the database in the background and carrying out small data exchange. This implies that it is possible to update parts of a web page, without reloading the whole page.
Key takeaways
At the end of the tutorials the reader should be able to:

Understanding AJAX and PHP
Create a database with xampp
Connecting Database to PHP
Create a Search engine with AJAX

Article quality
In this article, I will be teaching you the fundamentals of AJAX and how to build a search engine operating with AJAX and PHP
Reference:
https://www.tutorialspoint.com/php/php_and_ajax.htm

Thank you for submitting your topic. @Adespscientist
After some careful consideration, it struck us that this topic may be a bit over-saturated throughout other blog sites and official documentation.
We believe this is the best way for students to build a great portfolio (for potential employers) is by building what does not exist and what can provide the most value.
Please feel free to suggest an alternate topic to explore. 🚀

I write about cybersecurity but most of my topics are rejected why
That what i do
On Thu, Dec 23, 2021, 11:10 AM Ahmad Mardeni @.***>
wrote:

Closed #5705
https://github.com/section-engineering-education/engineering-education/issues/5705
.
—
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/issues/5705#event-5809080211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4IMNSAFLIA7CSBB36QWEDUSLYQBANCNFSM5KSEYF5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
<section-engineering-education/engineering-education/issue/5705/issue_event/5809080211
@github.com>

https://github.com/section-engineering-education/engineering-education/issues/5699
This my topic was rejected
Why?
On Thu, Dec 23, 2021, 11:13 AM Peter Adeshina @.***>
wrote:

I write about cybersecurity but most of my topics are rejected why
That what i do
On Thu, Dec 23, 2021, 11:10 AM Ahmad Mardeni @.***>
wrote:

Closed #5705
https://github.com/section-engineering-education/engineering-education/issues/5705
.
—
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/issues/5705#event-5809080211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4IMNSAFLIA7CSBB36QWEDUSLYQBANCNFSM5KSEYF5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
<section-engineering-education/engineering-education/issue/5705/issue_event/5809080211
@github.com>


172 - Integration add FileCriteria to the getFilesCountInfo use case

What this PR does / why we need it:
This PR adds the FileCriteria to the  getFilesCountInfo use case, so the files count takes into account the filters and search applied to get the files.
Which issue(s) this PR closes:

Closes #172

Special notes for your reviewer:
The e2e tests for the getFilesTotalDownloadSizeByDatasetPersistentId won't work because these PR need to be merged https://github.com/IQSS/dataverse/pull/9960
Suggestions on how to test this:
Step 1: Run the development environment

Run npm I
cd packages/design-system && npm run build
cd ../../
Check that you have a  .env file such as the .env.example, with the VITE_DATAVERSE_BACKEND_URL=http://localhost:8000 variable
cd dev-env
./run-env.sh 9907-file-counts-with-criteria
To check the environment go to http://localhost:8000 and check your local dataverse installation

Step 2: Test Dataset View mode with the implemented changes for the getFilesCountInfo

Go to http://localhost:8000
Login as admin using username: dataverseAdmin and password: admin1
Create a new Dataset
Upload some files with different types, or tags, or access.
From the dataset view mode copy the search parameters from the url. Ex.: ?persistentId=doi:10.5072/FK2/LHGRHP&version=DRAFT
Go to http://localhost:8000/spa/datasets and paste the search parameters. Ex.: http://localhost:8000/spa/datasets?persistentId=doi:10.5072/FK2/LHGRHP&version=DRAFT
You are seeing now the same Dataset in the SPA, check that when you apply some filter to the Files Table, the filters dropdown options are updated with the new available filters after narrowing the search.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
No
Additional documentation:

Same building error here! Can you check other PRs? @MellyGray

@GPortas I fixed the unit tests and there is an opened issue to fix the e2e

We have a build error in develop when deploying to beta testing:
https://github.com/IQSS/dataverse-frontend/actions/runs/6731104689/job/18295128987
It seems related to storybook. If it's not too big, can we make the fix in this PR to stabilize develop? @MellyGray

@GPortas I fixed the Storybook error, I think that should fix beta


Django IDOM

This is the most minimalistic Django configuration possible that can support Websockets.
I've used folder structure and file naming schemes suggested by Django documentation.
Also included the Daphne webserver in the requirements for testing on more than just the development webserver.
In order to get this configuration running

cd <repo_root_dir>
pip install requirements.txt
python manage.py migrate to create the initial database
python manage.py runserver to run the django development test server. Alternatively, daphne dj_idom.asgi:application to run the  production-grade webserver.

Here's the files in the repo:
dj_idom/static/scripts.js: Client side websocket
dj_idom/templates/base.html: HTML base template.
dj_idom/consumers.py: Server side websocket
dj_idom/asgi.py: Websocket URL routing
dj_idom/urls.py: HTTP URL routing
dj_idom/settings.py: Django boot time config
manage.py: Django project management utility

Thanks, I'll look this over shortly.

@Archmonger IDOM 0.29.0 was just released and contains a fix so that we won't need to do pip install idom[stable] here.

@rmorshea Resolved all startup errors under this repo's default configuration, I'll look to implement some tests soon.

@rmorshea Getting a test_suite fail on GH actions, not sure what to do to resolve it.

Yup, I think a test that goes something like this should be totally sufficient:
import idom

def test_simple_counter():
    run_the_app_somehow()
    web_driver = make_the_web_driver_somehow()

    @idom.component
    def Counter():
        count, set_count = idom.hooks.use_state(0)
        return idom.html.div(
            idom.html.button(
                {"id": "incr", "onClick": lambda event: set_count(count + 1)},
                "Click me!",
            ),
            idom.html.p(
                {"id": "count", "data-count": count}, f"Current count is: {count}"
            )
        )

    counter_button = web_driver.find_element_by_id("incr")
    counter_view = web_dirver.find_element_by_id("count")

    for i in range(5):
        assert counter_view.get_attribute("data-count") == i
        counter_button.click()

Anything beyond that is nice but probably not necessary right now.

I think I'll need to tag up with you regarding the frontend-backend connection via django. Not sure how to get those components to actually appear in a django fashion.

I think there's some docs on on how to set up a test using Selenium: https://docs.djangoproject.com/en/3.2/topics/testing/tools/#liveservertestcase
If there's not enough info there, grab a weekly office hour slot and we'll figure it out: https://github.com/idom-team/idom/discussions/312

There's some critical bugs in IDOM that need attention. Probably won't have time to work on this for another week or so.

Does the commit from 6 days ago address the critical bugs you brought up?

It does. I plan to release it soon. I'll then have some time to return to this after the two remaining open PRs are merged..

Ok, I'm not free to start working on this again.

Ok, I'm not free to start working on this again.

😂
I've been focused on Pyarr development but I'll have time to return to this soonish

Edit: I'm now free to start working on this again.

Can't wait to test this out!! This is the last piece of the puzzle for developing the Conreq community app store!

I'm gonna go ahead and merge this, but I really do think we should provide an app that encapsulates what was done in test_app since there's a lot of little things you have to understand about IDOM to make it work. Unfortunately, I don't think I understand enough about Django to make that happen.
Ideally one would be able to install django_idom as an app and then just embed a template into you HTML where you want an IDOM view to show up. I think I'll have to do some pair programming with someone who knows how to make re-usable apps though.

I'm gonna go ahead and merge this, but I really do think we should provide an app that encapsulates what was done in test_app since there's a lot of little things you have to understand about IDOM to make it work. Unfortunately, I don't think I understand enough about Django to make that happen.
Ideally one would be able to install django_idom as an app and then just embed a template into you HTML where you want an IDOM view to show up. I think I'll have to do some pair programming with someone who knows how to make re-usable apps though.

I'll can show you how to make reusable templates. That one is fairly simple, but it might be awkward given the current repo structure of using test_app

@Archmonger it'd be great to organize another working session to get through that.

@Archmonger it'd be great to organize another working session to get through that.

We can plan something out around your schedule, let me know your availability.

@rmorshea Something came up so I'll need to cancel our 11. I'll be able to do later in the evening if you're free.

Should be fine. Just let me know when you're able.

@Archmonger still able to meet?

@rmorshea just freed up, lemme know when you wanna meet

Moved the event.


Remove Preview Notice for embeded canvas apps in model driven apps

This feature is no longer in preview as per the April'19 release notes https://docs.microsoft.com/en-us/business-applications-release-notes/April19/microsoft-powerapps/embedding-canvas-apps-model-driven-forms-generally-enhanced

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

@amoedo - This feature won't GA until later this month. See the line item here: https://docs.microsoft.com/en-us/business-applications-release-notes/April19/microsoft-powerapps/planned-features, which states June. Although this PR was merged with live, it won't be merged into the publishing repo per my previous comment.


cache_range_requests plugin optional support for special IMS header

To support slice plugin self healing a regex_revalidate type feature can be enabled for cache_range_requests (param -c or --consider-ims):
X-CRR-IMS: (date) (ie: Tue, 19 Nov 2019 13:26:45 GMT)
Use case: During slice stitching if 2 adjacent range requests disagree for etag/last modifed, the "older" of the 2 block ranges will be re-requested with this X-CRR-IMS: (newer block range date) which can propagate up through layers of a CDN to an origin if necessary (assuming they all use the cache_range_request plugin with the --consider-ims param).
When enabled and using with the slice plugin, for security purposes it is advisable to add a header_rewrite rule to strip the X-CRR-IMS header at the edge if the incoming port (php) is NOT 0 (the internal ATS loopback port).
Converted the plugin configuration to use getopt_long() and deprecated ps_mode:cache_key_url in favor of -p or --ps-cachekey Modified base autest for both new and deprecated option check.
Also included: TSMBuffer/TSMLoc variable name changes, switching a txndata::char* to std::string (no extra malloc/freees), switching static const char* to constexpr std::string_view, moving functions into a private namespace.
Extra testing:  valgrind with small canned set of requests, automated range requester which dual issues through ATS and origin.

Oof the PR pulled way to much code in.

[approve ci]

Added a cache_range_requests admin guide.

@traeak Thanks for adding the cache_range_requests to the admin-guide however, a link to the doc is needed in doc/admin-guide/plugins/index.en.rst so that the doc can be found.  Also 'make html' gives this error:
/trafficserver/doc/admin-guide/plugins/cache_range_requests.en.rst: WARNING: document isn't included in any toctree

Cherry-picked to v9.0.x branch.


explain install/uninstall of database target

Not clear when install/uninstall runs
https://github.com/NLog/NLog/wiki/Database-target

http://nlog-project.org/2010/09/25/deploying-nlog-configuration.html

fixed with https://github.com/NLog/NLog/wiki/Installing-targets


port decentralized epoch manager

This is a major rewrite of the epoch manager. Previously, Peloton implements a centralized epoch manager, which is unscalable, especially for logging. This PR introduced a decentralized epoch manager we designed to resolve potential centralized contention points. The next step is to rewrite the logging component, which is a missing piece in Peloton.
I also fixed some random bugs in the networking layer, but please note that the jdbc bug still exists.
@MattPerron @jessesleeping as discussed before, could you please help review this PR?

Coverage decreased (-0.2%) to 66.165% when pulling dd976ea20e262606bf6126ceb71b979648685c1e on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Coverage decreased (-0.2%) to 66.144% when pulling<PHONE_NUMBER>b6d68e0fbc93ab206a1c20a1e623ca on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

@jessesleeping please review for the second time.

Coverage decreased (-66.3%) to 0.0% when pulling 6e3871e7bb43614a1c20550c6b0c40aac629e51d on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Please note that I have fully removed the older version of the epoch manager.

Coverage decreased (-0.9%) to 65.387% when pulling 41590de92633fb4d573d5ecbf9ba7060f57a6065 on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Coverage decreased (-0.3%) to 65.982% when pulling 41590de92633fb4d573d5ecbf9ba7060f57a6065 on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Coverage decreased (-66.3%) to 0.0% when pulling b716db38139d5143e71441a2f47d9f4861b1331e on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Coverage decreased (-0.2%) to 66.106% when pulling a712574210e2c42754507942a4655d695e204d01 on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Coverage decreased (-0.7%) to 65.653% when pulling 514adf3030164ea640171cff5cd219de4037fa75 on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Coverage decreased (-0.7%) to 65.664% when pulling fd81bc64ed9a8cc17547b9f217c4d25cf4b3af8f on yingjunwu:transaction into ab37f74fb1278a4e644f46bf89ef86c830629407 on cmu-db:master.

Coverage decreased (-0.7%) to 65.647% when pulling ea721dee71d17c5099ef7a8a46c6f4ba89adc1c3 on yingjunwu:transaction into b9c7d00b67b46539d04026c8c51809ded7bbee63 on cmu-db:master.

@apavlo @jessesleeping @MattPerron This PR can be merged now. I will port the logging component in the next PR.

Coverage decreased (-0.8%) to 65.568% when pulling c67514cfa5a073910d9a9cdd69700b0c2282efac on yingjunwu:transaction into b9c7d00b67b46539d04026c8c51809ded7bbee63 on cmu-db:master.

65% coverage :-1:

I know this is already merged, but does this actually work when the # connections > # networking threads?
Do you have a test for this?


JObject ?

Hello.
I would like to know if we can use JObject with your package ?
Visual Studio can't find the type

and there is nothing under Newtonsoft.Json.Linq

I want to achieve something like that :

If found a solution to my issue without using JObject.
But I am still looking forward to know if JObject is in this UnityPackage.


Reduce the frequency of Metadata retrival

Currently the refreshing federation and eduGAIN metadata is too frequent. Update the following parameters to requce the frequency.
maxRefreshDelay to PT4H for AAF amd 'PT24H' for eduGAIN. Upper bound on the next refresh from the time calculated based on the metadata's expiration.
refreshDelayFactor to 0.75. A factor applied to the initially determined refresh time in order to determine the next refresh time (typically to ensure refresh takes place prior to the metadata's expiration). Attempts to refresh metadata will generally begin around the product of this number and the maximum refresh delay.
Current settings of 'PT2H' and '0.125' have IdP reloading around every 20 minutes.

Values have been updated.
maxRefreshDelay to PT4H
refreshDelayFactor to 0.75


APT source whitelist request for ppa:terry.guo/gcc-arm-embedded

https://launchpad.net/~terry.guo/+archive/ubuntu/gcc-arm-embedded
deb http://ppa.launchpad.net/terry.guo/gcc-arm-embedded/ubuntu YOUR_UBUNTU_VERSION_HERE main
deb-src http://ppa.launchpad.net/terry.guo/gcc-arm-embedded/ubuntu YOUR_UBUNTU_VERSION_HERE main

I also need this.  Why is it taking so long to add this one?

+1, previous issue is https://github.com/travis-ci/apt-source-whitelist/issues/34

I found some instructions here that I found useful:
http://blog.travis-ci.com/2015-03-19-uc-berkeley-calsol-team-runs-on-travis-ci/

Thanks @dale-stewart but I would prefer to have benefits of both containers and cached packages.
Your reference is in fact downloading the whole toolchain source code and compiling it, which isn't a bad idea by itself.
If we were 100 doing the same thing at the same moment, it would be a total waste of energy.

I agree, it is just a stop-gap measure for me until the package becomes available.  However, there is no compilation involved, only downloading the binaries and setting up the path appropriately.  I think the caching could be used to avoid doing the download more than once, but I haven't gotten that far yet.

@dale-stewart, my apologizes: you were totally right about the avoided compilation: your link shows how to use the precompiled linux32 ARM GCC toolchain in the container context. Ihave read it too fast.
Cheers,
T.

No apologies needed, hope it helps!
Sent using CloudMagic Email
[https://cloudmagic.com/k/d/mailapp?ct=pa&cv=7.4.10&pv=5.1&source=email_footer_2] On Sun, Oct 18, 2015 at 4:31 AM, Thibaut VIARD <<EMAIL_ADDRESS><EMAIL_ADDRESS>> wrote:
@dale-stewart [https://github.com/dale-stewart] , my apologizes: you were totally right about the avoided compilation: your
link shows how to use the precompiled linux32 ARM GCC toolchain in the container
context. Ihave read it too fast.
Cheers,
T.
—
Reply to this email directly or view it on GitHub
[https://github.com/travis-ci/apt-source-whitelist/issues/132#issuecomment-148995081] .[https://github.com/notifications/beacon/AAd5U8C5g1XRL203zBpM1s0onxBZTsYtks5o815TgaJpZM4F4cKg.gif]

I need this package too to move to the container infrastructure. Seems like no-one at travis-ci is actually listening to these issues, though. Pathetic.

One more vote. But shouldnt this instead point to :
https://launchpad.net/~team-gcc-arm-embedded/+archive/ubuntu/ppa

And i just noticed that as per #390 new apt-sources are not currently added at all :(


docs/FAQ: fix HOMEBREW_NO_INSTALL_FROM_API mention

There is no HOMEBREW_NO_INSTALL_WITH_API; it should be HOMEBREW_NO_INSTALL_FROM_API.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Review period will end on 2023-02-20 at 08:51:37 UTC.

Review period skipped due to critical label.


cant change vocab vector

When I try to change the word embeddings nothing happens
`
import spacy_universal_sentence_encoder
nlp_use = spacy_universal_sentence_encoder.load_model('en_use_lg')
vector = nlp_use('her').vector
nlp_use.vocab.set_vector("Ertha", vector)
print(vector)
print(nlp_use("Ertha"))
`
The two print statements should be equal.

Hi @DanielBeck93,
At the moment this wrapper library did not allow to set the vectors, it provided only a read-only copy of the values from TensorFlow Hub.
Given your example, I understood that the feature is quite easy to be provided.
With the new v0.4.2, you can achieve setting any vectors to tokens, spans and even entire docs.
The comparison is done on the string that you pass to the vocab.set_vector function. Only exact matches will count.
import spacy_universal_sentence_encoder
nlp_use = spacy_universal_sentence_encoder.load_model('en_use_lg')

vector = nlp_use('her').vector
nlp_use.vocab.set_vector("Ertha", vector)

# compare the vectors, now they are equal
print((vector == nlp_use("Ertha").vector).all())

Please note that changing the vector to a single word does not change the vector of the spans/sentences/documents containing it.
If you try to compare the sentences "good for her" and "good for Ertha", they will be different, even if you have set the value of "Ertha" to be equal to "her".
If you want to substitute the words in the document, my suggestion would be to substitute the string "Ertha" with "her" in the text before creating the document with nlp_use(text).
Let me know if this works for you.
Best,
Martino

The version 0.4.2 has been yanked due to a problem found in #14


Implement Argon2id

I currently find the current KDF (scrypt) implementation adequate, however I think it would be nice to move to a KDF that will most likely be supported much longer and more widely used. Argon2id is also a more safe and versatile than scrypt.
I believe switching the primary KDF will be safe as long as we check the hash string for the correct KDF and use the appropriate KDF. The current KDF method would be renamed and only used for validation (if needed.)
I would be more than glad to implement Argon2id and deprecate scrypt myself and share the test builds if the project would be willing use Argon2id.
Edit:
Sorry, I forgot to link to GoDoc. Argon2 is supported.
https://godoc.org/golang.org/x/crypto/argon2

Please describe why changing the KDF would make restic better?
Because from what I can tell, changing the KDF now will not improve the security (because scrypt is not broken) or affect the speed of backup (KDF is only used once before the backup starts).

Thanks for the suggestion and your offer to work on restic!
Adding the actual implementation is not much work, modifying the repository format in a backwards-compatible way is much more work.
I think scrypt is adequate for the foreseeable future. Adding a second KDF will introduce additional complexity and may introduce new bugs. In my opinion, increasing the complexity just because it would be nice does not convince me.
I'm closing this issue for now, please feel free to add further comments. Thanks!


Nested Nunjucks variables aren't rendered when config variable points to a file

Describe the bug
Nested Nunjucks variables, such as {{ foo.bar }} aren't rendered when rendering the content of a variable in prompt that points to a file.
To Reproduce
See the shared promptfoo: https://app.promptfoo.dev/eval/f:701acdb5-d5f6-48ec-bad8-647780a25d79/
Config:
prompts: Output the following "{{ prompttest }}"
providers:
  - openai:gpt-4o-mini
tests:
  - description: 'Test translation to French'
    vars:
      test1: test1-content
      test2:
        sub: hello-content
      prompttest: file://test.txt
    assert:
      - type: contains
        value: hello-content

Test.txt contains the following line:
"Hey, this is {{ test1 }}, this is {{ test2 }} and this is {{ test2.sub }}"

Expected behavior
{{ test2.sub }} is rendered as hello-content
Actual behaviour
Screenshots
If applicable, add screenshots to help explain your problem.
System information:
If possible, please output the results of promptfoo debug and paste the output here.
{
  "version": "0.99.1",
  "platform": {
    "os": "darwin",
    "release": "23.3.0",
    "arch": "arm64",
    "nodeVersion": "v20.17.0"
  },
  "env": {},
  "configInfo": {
    "configExists": false,
    "configContent": null
  }
}

Currently, nested objects in vars are not officially supported, which is why you may see a schema warning in your editor when defining test2 (if your editor is configured with a schema validator).

To work around this, I recommend flattening your vars structure. For example, { key1: { key2: ... } } -> { 'key1.key2': ... }.
If you’re interested in adding support for nested vars, we’d love your help! The main piece of code to explore is here.
Let us know if you have any further questions or need guidance!

Thanks @mldangelo! The use case I'm trying to solve is: the AI agent I'm working with receives a context Nunjucks variable with all the required data as a nested object. I can't control the AI agent's behaviour re:rendering, and I want to use promptfoo for testing the prompt. Is there a workaround I'm not seeing? I tried using transformVars, but it doesn't support objects as well, it seems.

Ah! Makes sense. In that case I recommend using a prompt function https://www.promptfoo.dev/docs/configuration/parameters/#prompt-functions in python or javascript. You can do the vars substitution right in the function without nunjucks.


Animation for the view elements in custom view is not working

The showcase view is awesome and easy to use.Unable to ad a simple animation for the view element like Imageview or Textview in custom layout. its not working do you have any suggestions its helpful.

@Rahulrsannidhi , Glad to hear that you found this useful and easy. :)
Adding animations to individual views is currently not supported by the library. That said, this feature is in the pipeline and under development. However, cannot commit any ETA as of now. Will update this issue as and when its done. Hope this helps!


Fix prometheus-operator’s git sha to use v0.30.0

Fixes #105
Let me know if there should also be a commit to update the kube-prometheus version in the jsonnetfile.lock.json—something like:
@@ -8,7 +8,7 @@
                     "subdir": "jsonnet/kube-prometheus"
                 }
             },
-            "version": "81b9c9f9f6886ba1fbd61b05cdf0cc4f4d95eba8"
+            "version": "24c21574823556526d8eb486ff02be1e7df8af72"
         },
         {
             "name": "ksonnet",

Also, if you do approve of this fix, how would you feel about cutting a v0.1.1 release?

cc @metalmatze

It would be great, if we could actually reference release-0.30 in jsonnet/kube-prometheus/jsonnetfile.json and then run jb update again. Then we'll already forward to v0.30.1 :blush:
Otherwise this LGTM :+1:

Thank you! lgtm :+1:

@metalmatze @brancz sounds great! I'll make those changes and update my branch shortly.

@metalmatze @brancz The branch should be good to go.

lgtm thanks!


Change the systemd default back to non-socket-activation

This reintroduces the contrib/init/systemd/socket-activation subdirectory for the "socket activation" variation of the Docker systemd service file.
In the context of "restart policies" and the Docker daemon lifecycle, socket activation, while a neat feature with solid use cases, is not a sane default for the majority of users anymore.
cc @philips @lsm5 @jfrazelle

I'm pretty sure there are docs changes needed to go with this.

LGTM

I'm just curious, but why not just just enable the docker.service unit so that it starts by default (in addition to the docker.socket unit)?
One of the reasons systemd introduced the whole socket unit thing was so that service B, which depends on service A, can start up without waiting for service A to fully come up. The socket will be there, and any connection requests will just block until A finishes coming up.
So by doing this you make the startup process take longer if any other units depend on it. Not that docker takes a while to start, but I don't feel like this is the right solution.

LGTM, we weren't really using it anyways
@phemmer we could do that too

I like @phemmer 's suggestion. It keeps us from a needless fork of the config files.

I think the discussion on https://github.com/docker/docker/pull/10144 explains pretty well why I think it was a mistake to take the socket file generation out of Docker's control -- while it's nice to be able to have other service files depend on docker.socket to know that they have some kind of socket they can connect to, we actually lose functionality by doing so, including the ability to even tell if Docker itself is up and ready for commands without blocking.

@phemmer @philips any comments on that

The only reason I see from #10144 on why systemd should not control the socket is because when doing so, docker can't control the ownership. And I don't see this as an issue. It's neither good nor bad behavior in my opinion. Each method just has a different way of setting permissions.

the ability to even tell if Docker itself is up and ready for commands without blocking.

Why is this necessary? If systemd is starting docker, and everything is properly depending on docker.socket, and is properly waiting for docker to finish starting, why do we need to know that it's still starting without blocking?
Lets say you want to start service Foo, which needs docker. With systemd docker.socket, the app connects and it blocks waiting. Without systemd docker.socket, something has to periodically poll docker to know that it's ready before starting service Foo. Polling will always be slower as docker can start responding in between polling intervals. I can't think of any scenario where having to poll would be a good thing.

I agree with @phemmer. What part of #10144 implies things would be better if docker owned the creation of the socket file?
The root problem in this issue is restart policies don't work if Docker doesn't start. This is fixed by making Docker start in the default.target. This can be done w/o removing socket activation which has desirable traits like removing polling as @phemmer points out.

That said if @tianon wants to split things back out I am fine doing that; I just don't see a sound argument in relation to fixing this particular bug.

Ok, fair enough! :+1:
Is that something you can point me to some simple hrefs for how to do in
the context of our current packages? :innocent:

Is that something you can point me to some simple hrefs for how to do

You mean how to change the docker.service unit to start by default? The unit already does this here. But for the [Install] section to be evaluated, something needs to enable the service. This would be up to the package maintainer or user. Basically whoever is currently calling systemctl enable docker.socket should instead call systemctl enable docker.service.
(You could modify the docker.socket unit to start docker.service when it is started, but I would consider this to be bad behavior)


Problem with multiple button presses and analogue.

Hi. I am currently adapting a GVG100 panel, I have made all of the modifications in your Wiki, my panel only has a single 7805 regulator though but other than that, all component values and positions appear to be the same.
I have found that if I press more than 1 button at a time, I get very strange results with it appearing to select many other buttons in a random way. Have you seen this before?
Also I have found that the analogue seems to be very erratic. I have tried cleaning the potentiometers but not sure how sealed they are, I see that you are sampling twice so I assume you had similar issues?
Thanks
Phil

Hi.
No, i have not see that. In theory every button will be read individually.
The analog on mine worked pretty fine. I just do the sampling multiple times just to stabilize the values a bit more just because it's analog.
Also i have not played with it since this project was released so i'm not up to date on the technical details.

Hi, I have managed to diagnose the analogue a bit better. It looks like sometimes I am getting 512 more than the actual reading so it appears that bit 10 is "intermittent". I suspect that this may be a timing issue but I can do a bit of a 'hack' to fix this by just ignoring the status of the 10th bit if it is different to the last reading I guess? Not an ideal fix and if it is a fault on the DAC1001, this could be difficult(expensive) to replace as it is an obsolete part.

Interesting. This is too far back to remember anything. But i think i kept an eye on the timings from the datasheet.
I just know that the code as it is works fine for my panel and someone else who currently is using it without any problems.
But the arduino code is mostly just a template as how to interface with it, you can play with the timings, add handling for that stuff if you want.

I'll close for now, feel free to continue the discussion here.


closes #13034 Fixes the ScrollViewMock methods

Motivation (required)
Solves https://github.com/facebook/react-native/issues/13034
Now the ScrollView mock has all the methods available.
Test Plan (required)
React Native tests pass.
To test this specific part of the code,
$ react-native init Test
$ cd Test/
$ yarn add react-navigation

Then, add a simple project that uses react-navigation:
import React from 'react';
import { Text } from 'react-native';
import { StackNavigator } from 'react-navigation';

class HomeScreen extends React.Component {
  static navigationOptions = {
    title: 'Welcome',
  };
  render() {
    return <Text>Hello, Navigation!</Text>;
  }
}

const SimpleApp = StackNavigator({
  Home: { screen: HomeScreen },
});

export default SimpleApp

Run the default render tests:
$ npm run test

@facebook-github-bot shipit

Hey @satya164!
Can I help you with something else with this PR?

@alvaromb Seems this PR breaks tests. Could you please take a look?

Hi @shergin!
Sorry, I forgot about this PR. Please ping me if you need a hand with this. Thanks!

@alvaromb I would love to see passed tests. 😄

Hi @shergin!
This PR got merged? If not, what can I help you with to merge it?

@alvaromb I got bitten by this issue an burned some time trying to track it down.  a local change to how jest mocks out the ScrollView works but it would have been nice to have that time back :)
looks like your PR fails in CI and it now has a merge conflict with master.
CI
i can't see the details for the failure in CircleCI ... but the build was from a few months ago so it may have been flushed.  The details for the travis-ci build failure are still available: https://travis-ci.org/facebook/react-native/jobs/255491208#L3150
The failure is:
node_modules/react-native/Libraries/Components/ScrollView/__mocks__/ScrollViewMock.js:23
23: const ScrollViewComponent = jest.genMockFromModule('ScrollView');
                                ^^^^ identifier `jest`. Could not resolve name

This occurs during the flow check portion of the iOS build.  Based on the date of the build i think this is the line number in the build script where the failure occurs: https://github.com/facebook/react-native/blob/29d9c35e122861d86c70b1e27a6e34eda4d82369/scripts/run-ci-e2e-tests.js#L203
So the rub is, your source file to create the mock specifies the use of flow, but flow doesn't recognize the jest identifier.
Two options (and I am not sure which is the react-native preference because there are examples of both in the source):

strip off the @flow annotation from the file: example of that
explicitly declare the jest identifier: example of that

Merge Conflict
Looks like master was updated to support a flow v0.53.  I don't think that this impacts your PR, you just need to rebase.
Let me know if you want to see this through or if you want to close out this PR and have me open up a replacement.

Just merged against master, tests fails due to other issues though.

Honestly, I don't know how I should proceed from here, should I merge against another branch? @shergin @carlost

¯_(ツ)_/¯
Previous landing was unsuccessful because fb internal tool detected that this diff got new version on GH.

Should I open a new PR @shergin @satya164 ?


Create secure cookies when running with HTTPS

As an administrator, I would like to be able to configure the application to create secure cookies when running under HTTPS.
Acceptance

[ ] If the setting ROCKET_HTTPS is set to true, cookies are marked as secure.
[ ] The new setting is documented in the README.

Notes
The only place we currently create a cookie is in the cookie session creation endpoint.

We no longer use cookies for authentication.


SherlockActionbar conflicts with latest FacebookSdk facebook-android-sdk:[4,5)

Hey Guys I am having conflict issues with SherlockActionbar when I upgraded my Facebook sdk to latest version because of this warning from Facebook Reminder: Graph API v2.4 will be deprecated on Oct 09, 2017
My project is about 3 years old . I can't change SherlockActionbar now
Please see my post on Stack https://stackoverflow.com/questions/46299215/actionbarsherlock-conflicts-facebooks-sdk-facebook-android-sdk4-5

You can't use this library and AppCompat together. You'll need to upgrade to AppCompat only.


Avoid setting duplicated testID

When testing a component that uses react-native-modal using the testID prop, the testID is set in the Modal component and also in the container view leading to a duplicated testID.
This makes things tricky when using any of the recommended test libraries since querying by that testID finds two native elements instead of only one.
A possible solution would be to assign the testID only to the Modal component since someone using this package will have control over the content of the Modal and can set a different testID if they like.
This change should not affect the current behavior in any way.

A possible solution would be to assign the testID only to the Modal component since someone using this package will have control over the content of the Modal and can set a different testID if they like.

I like this approach, thanks for the PR!
I guess you already had a chance to test it, is it working well for you?

@ssaornil merged :) Marked as major just to be safe because it's a potential breaking change for test suites.
Thanks again! 🙏

:tada: This PR is included in version 11.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bug when Unpack double from Any

When we unpack a double from any, the output result is a junk value. @ccifra

@ccifra  closing this as it is fixed in the latest release


File Watcher

When the option - "Phpstan: File Watcher = true" is set, your extension starts analysis of all project files - this creates a load and takes a lot of time.
Is it possible to run the analysis of either only the file being changed or use the cache to speed up the analysis?

Hi,

When the option - "Phpstan: File Watcher = true" is set, your extension starts analysis of all project files - this creates a load and takes a lot of time.

The first analysis can be slower because the cache is empty and it requires scan all files.

Is it possible to run the analysis of either only the file being changed or use the cache to speed up the analysis?

The extension only runs the analysis when it changes a file that it matches with the phpstan config (parameters.paths, parameters.fileExtensions).
PHPStan already has a file cache system for speed up the analysis.
How many files are you analysing? The parameters.excludes_analyse config is not yet supported.

My phpstan.neon
`includes:
- ./vendor/nunomaduro/larastan/extension.neon
parameters:
paths:
- app
- config
- database
- nova-components
- resources
- routes
- tests
# The level 8 is the highest level
level: 1

ignoreErrors:
- '#Call to an undefined static method Illuminate\\Support\\Facades\\DB::rollback\(\).#'
-
    message: '#Call to an undefined method Illuminate\\Support\\Fluent::on\(\).#'
    path: ./database/migrations/*
-
    # Undefined property resources
    message: '#Access to an undefined property App\\Nova\\Resources[\\a-zA-Z]+Resource::\$[a-z\_]+.#'
    path: ./app/Nova/Resources/*
-
    # Undefined property models
    message: '#Access to an undefined property App\\Models\\[a-zA-Z]+::\$[a-z\_]+.#'
    path: ./app/Models/*

excludes_analyse:
    - vendor

checkMissingIterableValueType: true`

when changing one file, analysis of all files is always started. My log:
`# Command: analyse
Note: Using configuration file phpstan.neon.
0/215 [>---------------------------]   0%
  40/215 [=====>----------------------]  18%
  60/215 [=======>--------------------]  27%
  80/215 [==========>-----------------]  37%
 100/215 [=============>--------------]  46%
 120/215 [===============>------------]  55%
 140/215 [==================>---------]  65%
 160/215 [====================>-------]  74%
 180/215 [=======================>----]  83%
 195/215 [=========================>--]  90%
 215/215 [============================] 100%
{"totals":{"errors":1,"file_errors":0},"files":[],"errors":["Ignored error pattern #Call to an undefined method Illuminate\\Support\\Fluent::on\(\).# in path database\migrations\* was not matched in reported errors."]}`


improve: add /xsrf endpoint to get XSRF token only

Webssh might be used a backend service for an other large system. And so
this system must do a first HTTP request to get XSRF token. To make
getting XSRF token more convenient, let's make a special endpoint that
returns a TSRF token is the JSON response (it is more convenient than
parsing HTML form from index page).

To bypass the xsrf protection, you can simply issue a http request with the same _xsrf key value pair in the POST body and the Cookie header.
So I don't think it is necessary to implement this xsrf service.

To bypass the xsrf protection, you can simply issue a http request with the same _xsrf key value pair in the POST body and the Cookie header.

Sorry, I don't get the idea.
Tornado matches _xsrf tokens from parameters and cookies, but before that it checks its format, and that format is an own tornado's format, so I cannot (do not want to generate on a client). It's more convenient to get token from the backend.
Also, there is a question, why then front-page cannot set xsrf token to the Cookie header and get it from a form's hidden parameter?

Also, let me explain my intentions.
I (not exactly me... my collegues) develop an Angular component that must use the webssh as a backend. So we use an other front-end for the webssh. And on the post a request with a parameters we can the "XSRF cookie does not match POST argument" error. How can we avoid this with a proposed single POST request?

O, I've got it working! Angular components must be set up appropriately


PHP 8.1: Add ReturnTypeWillChange attribute to Response

Fixes #24

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Converting to Draft, as I'm finding some other deprecation notices as I go through my manual testing.

Thanks @chrispenny


Try to import the python 3 urllib.parse, and fall back to urlparse

With this patch, mock_services can be used with python 3.

@toopy what's the problem with the ci build ? should we re-run it ?

I'd like to merge this, could you rebase on top of the current master which has a fix for ci ea06695 so we can have a test run that doesn't fail in setup ? Thanks !

Seems like there are more errors on Python3: https://travis-ci.org/jpic/mock-services/jobs/134221259

Looks good .. OPM!


SQL Profiler is not capturing parameterized queries from .Net SqlClient Data Provider

Azure Data Studio Version: 1.2.4
SQL Query Profiler : 0.3.0

Steps to Reproduce:

Connect to Azure SQL Instance
Execute via Web App, code that further executes parameterized SQL queries using the ADO.Net /  .Net SqlClient Data Provider.  Further details - In my case, very often, this is done in the application code via PetaPoco.

Result: I can see the rpc_completed calls, and the reads / writes information, but not the actual parameterized SQL call itself in the TextData column.
This may be related to issue #2125.
I would test version 0.4.0 of SQL Query Profiler if someone could point me to a installable OSX build.

@spursrox please try this Azure Data Studio insider build: https://github.com/Microsoft/azuredatastudio/releases/tag/1.3.1 and install v 0.4.0 profiler extension.

@alanrenmsft Tried latest builds, no luck. "rpc_completed" rows, for the TextData column, are still empty. Where is the definition of the trace file associated to the Session Template? Since I'm trying to profile Azure SQL Server, I assume I'm supposed to be using session template "Standard_Azure".
Maybe its just the configuration of the hard-coded Session Templates?
[https://dba.stackexchange.com/questions/10269/sql-profiler-not-showing-rpc-calls]
In the "Events Selection" tab of the trace definition, make sure that the TextData column is checked for RPC:Completed events.
(This is unchecked in the default trace definition)

@spursrox Please try this build: https://github.com/Microsoft/azuredatastudio/releases/tag/1.3.4 and make sure the profiler extension version is 0.5.1.

@alanrenmsft Works great now, thank you.


[STORY] As a passport holder, I want to send tokens to the Smart Contract so that I can stake them on someone else’s identity (Community Staking)

Acceptance
Background:
GIVEN I am logged in to The Staking App with my Passport
AND I’ve approved the contract to accept my tokens
Scenario: Enter the amount to stake
WHEN I click the stake button
THEN I see a modal that lets me enter the amount of GTC that I want to stake
Success Path: enough GTC balance
WHEN I put in multiple amounts of GTC
AND I have enough GTC in my balance to match the sum of all the amounts entered
AND I click the “stake” button
THEN I see a second modal confirming the amount of GTC I want to stake
AND a list of addresses/Passports that I am staking on
AND I see important details about how long my staking will be locked
AND I see information about how the smart contract has been audited
Fail Path: insufficient GTC balance
WHEN I give it an amount of GTC
AND I do not have enough GTC in my balance to match the amount I entered
THEN the “stake” button turns to disabled state and reads “insufficient balance”
OR
WHEN I give it an amount of GTC
AND I do not have enough GTC in my balance to match the amount I entered
AND I click the “stake” button
THEN I see a second modal that informs me that wallet balance is insufficient
Scenario: Confirm the staking
Success Path: confirms the staking
WHEN I’ve read the details (amount, locking info, audit info)
AND I click the confirm button (”stake my GTC”?)
THEN I receive a message from my wallet to sign a transaction
AND WHEN I accept the transaction
THEN my GTC is staked and locked
Return Path: return to the staking amount
WHEN I’ve read the details (amount, locking info, audit info)
AND I click “back to the previous step”
THEN I return back to the modal where I enter how much to stake
Scenario: Staking is complete
WHEN my GTC is transaction is confirmed
THEN I see a toast telling me it is complete
AND WHEN I look at the details on the Staking App dashboard
THEN the amount of GTC that I have staked is reflected on the Stake on others option
Notes

The failpath options are for @farque65 to decide based on whichever is faster

story points 2

For designs check out: https://www.figma.com/file/cpTUBYmCxRIjh5A4cV2RJl/🥩Staking-App?node-id=24%3A3185
What to look for:


Enable to annotate :string type to endpoint

I want annotate primitive types like string to endpoints.
When use symbols, annotate those symbol types as follows.
class BooksController
  extend OpenApiAnnotator::ControllerAnnotatable

  endpoint [:string]
  def index
    # some code
  end
end

 "books":
    get:
      responses:
        '200':
          description: Returns an array of string
          content:
            application/json:
              schema:
                type: array
                items:
                  type: string

@ngtk Please review

@kobayang
Oh, now we can not handle primitive types and these are treated as a custom component.
Sorry about that! 😨
But, can we use a class instead of a symbol in order to specify the OpenAPI type? 💡
Because of interface consistency, which means annotation should be ruby level, not OpenAPI spec.
Also, we can handle this because the granularity of the OpenAPI type is bigger than Ruby's one. but, if we want to handle the OpenAPI format, we should use a symbol.
But, for now, I think that it should be used Ruby class for the interface and annotator should convert a Ruby class to an OpenAPI type.
FYI: Definition of type and format https://swagger.io/docs/specification/data-models/data-types/
Anyway, thank you for pull requesting! 😄

@ngtk
Does this mean endpoint [String] is better than endpoint [:string] ?
But we can not know the String class is customized or not.
Or can we use OpenApi::DataType class like endpoint [OpenApi::DateType(:string, :string)] ?
cf. https://github.com/ngtk/open_api/blob/master/lib/open_api/data_types.rb

Does this mean endpoint [String] is better than endpoint [:string] ?
But we can not know the String class is customized or not.

What do you mean "customized"? Do you mean "user-defined"? or "Singular class"?
Anyway, I think we can convert String to OpenApi::DateType(:string, :string) in this library.

f279d71 Enable to annotate some primitive ruby classes (String, Integer, and Float).
cf.  https://github.com/ngtk/open_api/blob/master/lib/open_api/data_types.rb

Thank you for contributing! 🎉
This change has been released in v0.4.0! https://rubygems.org/gems/open_api_annotator


VCS editable regresses to using deprecated egg= syntax

Issue description
This is basically the same issue noted here #5626, it seems to be have been reintroduced by #5928.
Pipfile:
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
pyjwt = {extras = ["crypto"], editable = true, ref = "2.6.0", git =<EMAIL_ADDRESS>
[dev-packages]

[requires]
python_version = "3.9"

The easiest way to see this is, after lock, the older pipenv returns the expected new syntax:
> pipenv --version
pipenv, version 2023.9.1
 > pipenv requirements
-i https://pypi.org/simple
cffi==1.16.0; python_version >= '3.8'
cryptography==41.0.4
pycparser==2.21
pyjwt[crypto]@ git+ssh://git@github.com/jpadilla/pyjwt.git@7665aa625506a11bae50b56d3e04413a3dc6fdf8

The newest version does not:
> pipenv --version
pipenv, version 2023.10.3
> pipenv requirements
-i https://pypi.org/simple
cffi==1.16.0; python_version >= '3.8'
cryptography==41.0.4
pycparser==2.21
-e git+ssh://git@github.com/jpadilla/pyjwt.git@7665aa625506a11bae50b56d3e04413a3dc6fdf8#egg=pyjwt[crypto]

The newer version has the deprecated warnings, as well, that have now returned, e.g.:
> pipenv sync --dev
PipDeprecationWarning: DEPRECATION: git+ssh://****@github.com/jpadilla/pyjwt.git@7665aa625506a11bae50b56d3e04413a3dc6fdf8#egg=pyjwt[crypto] contains an egg fragment with a non-PEP 508 name pip 25.0 will enforce this behaviour change. A possible replacement is to use the req @ url syntax, and remove the egg fragment. Discussion can be found at https://github.com/pypa/pip/issues/11617
  deprecated(

EDIT:
As a follow-up, I should note, also, that without editable, it also generates a deprecation warning on lock:
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
pyjwt = {extras = ["crypto"], ref = "2.6.0", git =<EMAIL_ADDRESS>
[dev-packages]

[requires]
python_version = "3.9"
~

And then lock:
pipenv lock
Locking [packages] dependencies...
Building requirements...
Resolving dependencies...
✔ Success!
⠴ Locking...Warning: INFO:pip.subprocessor:Running command git clone --filter=blob:none --quiet<EMAIL_ADDRESS>/private/var/folders/jh/0_l94d8n51n2gjzt_5l8mv8r0000gq/T/pip-temp-tbxaqx59/pyjwt_2de1d1d332894184a987c8ebdc9489d2
INFO:pip.subprocessor:Running command git checkout -q 7665aa625506a11bae50b56d3e04413a3dc6fdf8
INFO:pip.subprocessor:Running command git clone --filter=blob:none<EMAIL_ADDRESS>/var/folders/jh/0_l94d8n51n2gjzt_5l8mv8r0000gq/T/tmplbwvrzyq
INFO:pip.subprocessor:Cloning into '/var/folders/jh/0_l94d8n51n2gjzt_5l8mv8r0000gq/T/tmplbwvrzyq'...

INFO:pip.subprocessor:Running command git checkout -q 7665aa625506a11bae50b56d3e04413a3dc6fdf8
/.../.pyenv/versions/3.9.17/lib/python3.9/site-packages/pipenv/patched/pip/_internal/models/link.py:445: PipDeprecationWarning: DEPRECATION: git+ssh://****@github.com/jpadilla/pyjwt.git@7665aa625506a11bae50b56d3e04413a3dc6fdf8[crypto]#egg=pyjwt[crypto] contains an egg fragment with a non-PEP 508 name pip 25.0 will enforce this behaviour change. A possible replacement is to use the req @ url syntax, and remove the egg fragment. Discussion can be found at https://github.com/pypa/pip/issues/11617
  deprecated(
Locking [dev-packages] dependencies...
Updated Pipfile.lock (a9d82110488b564f4c7da14f298280fd55109dab9f3889067b3104fc9227efeb)!

I don't see where the pipenv  utils would be using  the old format for non-edtiable; if it was editable: when I took this deprecation warning  up originally, it appears that the new format doesn't work for editable.

I actually am not sure if the editable requirement is even needed for most VCS installs

I wasn't sure of that either when I did the refactor away from requirementslib, but it turns out it affects which directory in the virtualenv it gets installed into, and some users have expectations around  that.
Sounds like the lock file retains a prior editable setting  even if it was removed in subsequent lock?  If so that sounds like an  edge case we should clean up


create release 3.2.1

Create release 3.2.1 and stage to maven central.

Seems we need to update to our new maven-parent:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-javadoc-plugin:2.9.1:jar (attach-javadocs) on project sonar-devon4j-plugin: MavenReportException: Error while creating archive:
[ERROR] Exit code: 1 - javadoc: error - The code being documented uses modules but the packages defined in http://docs.oracle.com/javase/7/docs/api/ are in the unnamed module.
[ERROR]
[ERROR] Command line was: /Users/hohwille/projects/software/default/java/11.0.5_10/Contents/Home/bin/javadoc @options @packages
[ERROR]
[ERROR] Refer to the generated Javadoc files in '/Users/hohwille/projects/devon/workspaces/stable/sonar-devon4j-plugin/target/apidocs' dir.

Indeed urgent to rework:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.8.2:deploy (default-deploy) on project sonar-devon4j-plugin: Failed to deploy artifacts: Could not transfer artifact com.devonfw.tools:sonar-devon4j-plugin:jar:3.2.1 from/to ossrh (https://oss.sonatype.org/service/local/staging/deploy/maven2/): Failed to transfer file https://oss.sonatype.org/service/local/staging/deploy/maven2/com/devonfw/tools/sonar-devon4j-plugin/3.2.1/sonar-devon4j-plugin-3.2.1.jar with status code 401 -> [Help 1]

OMG:

typeId | signature-staging
-- | --
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://pool.sks-keyservers.net:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://keyserver.ubuntu.com:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://keys.gnupg.net:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://pool.sks-keyservers.net:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://keyserver.ubuntu.com:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://keys.gnupg.net:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://pool.sks-keyservers.net:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://keyserver.ubuntu.com:11371/.  Upload your public key and try the operation again.
failureMessage | No  public key: Key with id: (345d2874aa2a78c3) was not able to be located  on http://keys.gnupg.net:11371/.  Upload your public key and try the operation again.

The last one was because of this legacy config:
https://github.com/devonfw/sonar-devon4j-plugin/blob/3ba66bc80a5525f6667839142b04e57281935b04/pom.xml#L343
The 401 because of this:
https://github.com/devonfw/sonar-devon4j-plugin/blob/3ba66bc80a5525f6667839142b04e57281935b04/pom.xml#L363
instead of this:
https://github.com/devonfw/maven-parent/blob/329cc0c7a39673565af8dec967b31c1e3632dc6b/pom.xml#L551
The JavaDoc issue because of this:
https://github.com/devonfw/sonar-devon4j-plugin/blob/3ba66bc80a5525f6667839142b04e57281935b04/pom.xml#L237
I will simply update to our maven-parent to get rid of all these problems.

Available in maven central:
https://repo1.maven.org/maven2/com/devonfw/tools/sonar-devon4j-plugin/3.2.1/
Done.


Discrete Markers and Radar Graphs

Im having an issue with discrete markers and radar graphs.
I took your working codepen from #286 and changed the chart type to radar graph and the discrete markers dont take effect.
Can discrete markers be used with radar Graphs
https://codepen.io/craicerjack/pen/ewMBbj

You're correct.
Discrete markers don't work in Radar Charts. I will release a fix soon.


setup CI for this repo

Summary
For now I just plan on using flake8 to add linting to our CI.
Implementation
I will add a fairly basic flake8 setup to the repo, we can add exclusions as we encounter rules that we do not agree with.

Perfect! Go for it.

@fisher60 this can be closed, right?


"Latest" release is from 2018

Can you fix this?

click on +118 releases see is updated
to download binaries
https://ungoogled-software.github.io/ungoogled-chromium-binaries/

@jacob755 I'm aware of that, but it might give the wrong idea to people that are just taking a look at the repo.

@jacob755 I'm aware of that, but it might give the wrong idea to people that are just taking a look at the repo.

Yes! When I first saw this GitHub repo and found the lastest release to be 1 yo I thought the developement was halted

I tried looking a bit into this, there are a lot of bloated node.js-based solutions. Ideally I'd need something that:

Is very barebones, just creates empty releases whenever a tag is pushed. No extra features
Does not involve a complex dependency system, e.g. npm
Can be self-hosted, i.e. something we can fork into the ungoogled-software organization and maintain ourselves to guarantee safety and security.

...but none of the existing solutions I found fit all three criterion. I suppose we might have to create our own.


Inconsistent inclusion of a footer template

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.23.3
Expected results
Adding a file superset/templates/appbuilder/footer.html as per the appbuilder documentation should include the same fragment on all pages.
Actual results
The templates defined in superset do not reference the footer template and are thus not included. Others such as those directly from appbuilder still support this giving inconsistent results.
Footer included in, for example, login/, databaseview/list/ but not included in superset/dashboard/..., superset/welcome.
Steps to reproduce
Include a file superset/templates/appbuilder/footer.html with any content that would be visible when rendered such as
Hello, World
 and open the views indicated above.
Possible solutions
Some method of adding end-of-view content without patching a large set of templates would be great. I am thinking of internal/external support links, legal notices here. Other mechanisms would be fine too though I am reluctant to use JS to add it dynamically to pages, which may or may not work for all users.
As a possible alternative, a documented way of including a notice on the login page only (but for all authentication mechanisms) may work but I cannot be sure this covers all such use cases (e.g. anonymous access).

Sounds like you want to add footer.html to the basic.html template (or whatever what the base template for React apps is). Keep in mind this is not a public api, we're not explicitly supporting people forking Superset to change the templates or that kind of things.

Thanks. For the benefit of anyone else looking for a similar solution: I replaced my existing patches with one adding the footer inclusion code to the basic.html file as suggested. It appears to be sufficient but I have not had a chance to test all sections.
I am happy for the issue to be closed if you do not intend to implement footer support in project-specific templates.

Happy to review if you want to upstream your change.

@b881659b I have a similar situation. Could you please share your footer inclusion code?

@b881659b I have a similar situation. Could you please share your footer inclusion code?

@guptaraul89: I do not have access to that code anymore. From what I recall replicating the footer inclusion
 section to basic.html was sufficient to get this to work but the original templates did more than just include the footer template. They added a little formatting, which you could find in the appbuilder templates. This would ensure consistent margins across pages.
An alternative I explored at the time, which may be easier but not applicable in all cases, was to include javascript in the header. The header appeared to be handled more consistently. This could dynamically add content to the end of the page or in the margins such as a support button. We solved our problems in different ways so I not know how much this part of superset has changed since and if the above still works.


bump go.mod authzed/spicedb to v1.25.0

Fixes https://github.com/authzed/examples/issues/12
I removed go.buf.build/* from go.mod and then updated spicedb dependency by
go get<EMAIL_ADDRESS>Had to drop server.WithDashboardAPI(util.HTTPServerConfig{HTTPEnabled: false}),

I have read the CLA Document and I hereby sign the CLA


Inspection dialogue should delete the temp image file safely

Sometimes MUSCIMarker crashes when inspecting objects because the image file no longer exists. The deletion in CropObjectView.inspect() popup dismiss binding should be made safe.

Maybe it doesn't need to delete the image at all, since tmp gets cleared out anyway when the application exits.


fix: rename f to fn

Codecov Report

Merging #7 (0cab984) into main (de15a0d) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   67.67%   67.67%
=======================================
  Files          14       14
  Lines         631      631
=======================================
  Hits          427      427
  Misses        146      146
  Partials       58       58

Flag
Coverage Δ

unittests
67.67% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

recovery/recovery.go
83.33% <100.00%> (ø)

retry/retry.go
92.85% <100.00%> (ø)

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 de15a0d...0cab984. Read the comment docs.


Bot.loop is None in new versions

The Bot.loop in None after https://github.com/timoniq/vkbottle/commit/730c8143dcce2a25a28322a6034778d058588dcc#diff-eef9235318e8d85a06766f3daf94071f commit.
We are creating a background task by using Bot.loop.create_task. It worked fine on version 2.7.8. I think that commit https://github.com/timoniq/vkbottle/commit/730c8143dcce2a25a28322a6034778d058588dcc#diff-eef9235318e8d85a06766f3daf94071f broke loop creation in vkbottle/framework/user/user.py line 74.

true. it will be fixed in 3.0 release, now please use stable version


Flag changes unexpectedly

I selected a flag and started typing.
But after typing about 3 digits, the flag changed to "CURRENT" flag.
So I can't select other flags anymore.
If some experiences this issue, please reply.
Thanks.

Same issue