[SPARK-27254][SS] Cleanup complete but invalid output files in ManifestFileCommitProtocol if job is aborted

What changes were proposed in this pull request?
SPARK-27210 enables ManifestFileCommitProtocol to clean up incomplete output files in task level if task is aborted.
This patch extends the area of cleaning up, proposes ManifestFileCommitProtocol to clean up complete but invalid output files in job level if job aborts. Please note that this works as 'best-effort', not kind of guarantee, as we have in HadoopMapReduceCommitProtocol.
How was this patch tested?
Added UT.

cc-ing. same here as #24154
cc. @rxin since he originally authored the file.
Also cc to @tdas @zsxwing @jose-torres @arunmahadevan @gaborgsomogyi since it's related to SS.

Test build #103840 has started for PR 24186 at commit 002a184.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #103840 has finished for PR 24186 at commit 002a184.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103840/
Test FAILed.

retest this, please

Test build #103858 has started for PR 24186 at commit 002a184.

Test build #103858 has finished for PR 24186 at commit 002a184.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103858/
Test PASSed.

Test build #103932 has started for PR 24186 at commit 7519766.

Test build #103932 has finished for PR 24186 at commit 7519766.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103932/
Test FAILed.

Retest this, please

Test build #103948 has started for PR 24186 at commit 7519766.

Test build #103948 has finished for PR 24186 at commit 7519766.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103948/
Test FAILed.

retest this please

Test build #103964 has started for PR 24186 at commit 7519766.

Test build #103964 has finished for PR 24186 at commit 7519766.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103964/
Test PASSed.

@zsxwing Could you please review the latest change?

Ping again, as Spark+AI Summit 2019 in SF is end.

Test build #106855 has started for PR 24186 at commit 3785e35.

Test build #106855 has finished for PR 24186 at commit 3785e35.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106855/
Test PASSed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12369/
Test PASSed.

Test build #107200 has started for PR 24186 at commit 3785e35.

Test build #107200 has finished for PR 24186 at commit 3785e35.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107200/
Test PASSed.

Test build #107215 has started for PR 24186 at commit 933d00d.

Test build #107218 has started for PR 24186 at commit 57cd3bf.

Test build #107215 has finished for PR 24186 at commit 933d00d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107215/
Test PASSed.

Test build #107218 has finished for PR 24186 at commit 57cd3bf.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107218/
Test FAILed.

Test build #107224 has started for PR 24186 at commit d181d61.

Test build #107224 has finished for PR 24186 at commit d181d61.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107224/
Test PASSed.

@dongjoon-hyun Would you mind having another round of review? Thanks in advance.

@zsxwing @dongjoon-hyun Kindly reminder.

Ping.

Can one of the admins verify this patch?

ok to test

Test build #110686 has started for PR 24186 at commit d181d61.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15848/
Test PASSed.

Test build #110686 has finished for PR 24186 at commit d181d61.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110686/
Test FAILed.

retest this please

Test build #110858 has started for PR 24186 at commit d181d61.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15986/
Test PASSed.

Test build #110858 has finished for PR 24186 at commit d181d61.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110858/
Test FAILed.

retest this, please

Test build #110883 has started for PR 24186 at commit d181d61.

Test build #110883 has finished for PR 24186 at commit d181d61.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110883/
Test PASSed.

@zsxwing @dongjoon-hyun Kindly reminder.

Test build #111458 has started for PR 24186 at commit 177a4a4.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16492/
Test PASSed.

Test build #111458 has finished for PR 24186 at commit 177a4a4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111458/
Test PASSed.

Thanks! Merging to master.

Thanks for reviewing and merging!


Enable default HttpClient handler

This uses the feature implemented by @kjpou1 https://github.com/mono/mono/pull/10152. It means you can now do new HttpClient() directly and it works under Blazor on WebAssembly.
Note that we still register a HttpClient in the default DI config, because that instance has its BaseUri populated, meaning that it knows how to make requests with relative URLs. If you do new HttpClient() then you can only make requests to absolute URLs. This is standard for .NET's HttpClient.
Additionally, this involved updating to a newer version of mono (bfc35fc6e65). Benchmarks show that this newer version of Mono is consistently 10-20% faster on our checked scenarios (app startup, rendering, JSON handling), but it's also about 150KB bigger after linking.

cc @danroth27 because I know he's been waiting a while for this.

@rynowak I expect the only parts you'll want to review are the changes to the app startup process. Feel free to review the rest if you want, but it's mostly just the Mono update.

@SteveSandersonMS you can use this new mechanism to return a properly configured HttpClient.

@kumpera I thought that's what we already are doing in this PR. Or did you have something else in mind?

I'll merge this once the rebased branch build passes, but it's worth noting that once this goes in, the default app size will (temporarily) increase from 1.9MB to 2.5MB. This is due to it now including all the HttpClientHandler code paths (which transitively are pretty huge, and we don't even use them at runtime), which in turn is because we now call the default HttpClient constructor.
As a fix for the size bloat, I'm about to work on adding some infrastructure for pre-stripping parts of the BCL that we never want to use at runtime in WebAssembly builds. This will chop out HttpClientHandler completely (as it will never work, and is never required, on WebAssembly), giving us back that 500KB. It also makes it possible to strip out more - there might be some other easy wins to shrink typical apps further.

@kumpera How can we configure new HttpClient() to have a preset BaseUri?

If that's not true, we can change the callback to allow it to do so.

No problem - it's fine as-is!


Merge pull request #2623 from opscode/sersut/revert-1901

Preserve relative paths in Link resource
Conflicts:
spec/functional/resource/link_spec.rb
spec/unit/provider/link_spec.rb

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.


[New game]: playable piano game

🎮 Game Request
uses html,css and javascript to make the piano and music sound
Point down the features
music game
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @sanayamahajan-23 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign


Am I injecting right?

I think I injected right, but just to double check, where do you inject it? A more thorough walkthrough would be nice :)

Also, what do you inject, the .dll?

You inject using an DLL injector CSGhost is what I use


Update disks-types.md

@anasouma : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @anasouma - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? Thank you! We'll close this PR.


Realistic planets

closes #2

You could rewrite the history with git rebase -i and squash your commit "added planets glb to assets" with "use textures instead of gltf for planets" so that we don't need to have the big gltf part of the history (and therefore increase the size of the repo, even if you delete it afterwards).

these changes are also included in #118 so i'm closing this


Performance Improve load performance for variant type

remove phmap for padding rows
add SimpleFieldVisitorToScarlarType for short circuit type deducint
correct type coercion for conflict types bettween integers

In clickbench data, load performance:
12m36.799s -> 8m30.870s about 32.5%
In variant_p2/performance.groovy:
3min44s20 -> 2min21s07 about 36.7%
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run p0

run buildall

run buildall

run buildall


Fixed some markdown issues in README.md

minor changes to improve readability
link to LICENSE file

I signed it with a different email (could not change it).

Thanks @mojoaxel!


Correct typo

Changing "FirewassExecutionLog.txt" to "FirewallExecutionLog.txt".

Nice correction. The misspelled word could just as easily have said "FirewaterExecutionLog.txt" 😉

@illfated thank you!!


Wire up getLessonIndex method to front end

Codecov Report

Merging #163 into master will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #163      +/-   ##
============================================
- Coverage     50.01%   49.96%   -0.06%
  Complexity      573      573
============================================
  Files           125      125
  Lines          2611     2614       +3
  Branches        315      315
============================================
  Hits           1306     1306
- Misses         1196     1199       +3
  Partials        109      109

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/trackitnus/ui/upcoming/DayCard.java
0.00% <0.00%> (ø)
0.00 <0.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 621f983...9e38b74. Read the comment docs.


Update method get static content for Earning, General T&C

[x] Update method get static data for Earning Pool Detail modal
[x] Update method get static data for General T&C Modal

Version test: https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-311-1394-merge-2024-02-03T08-21-04.ipa
Device test: IP 11

[ ] Do not show Earning info

[ ] Do not update content after edit term_and_condition

Version test: https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-311-1394-merge-2024-02-03T08-21-04.ipa Device test: IP 11

[ ]  Do not show Earning info

[ ]  Do not update content after edit term_and_condition

Pls enable dev mode and check again

Version test: https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-318-1394-merge-2024-02-22T02-19-57.ipa
Device test: IP 11
Validation passed.

Install app(Enable dev mode)
Update app
Update content Intruction


range-v3: 0.5.0 -> 0.9.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/range-v3/versions.
meta.description for range-v3 is: '"Experimental range library for C++11/14/17"'.
meta.homepage for range-v3 is: '"https://github.com/ericniebler/range-v3"
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.9.1 with grep in /nix/store/56g65y0a57a2q51kd8js8jpbk0i4s1id-range-v3-0.9.1

directory tree listing: https://gist.github.com/11daff7f50a9c30c8518d17ad71d1963

du listing: https://gist.github.com/81a748825334c3e2d2fa349048134bbe

Rebuild report (if merged into master) (click to expand)

7 total rebuild path(s)
2 package rebuild(s)
2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
range-v3
tdesktop

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/56g65y0a57a2q51kd8js8jpbk0i4s1id-range-v3-0.9.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A range-v3 https://github.com/r-ryantm/nixpkgs/archive/e220644188aeae791eec85db5815697136846d55.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/56g65y0a57a2q51kd8js8jpbk0i4s1id-range-v3-0.9.1
ls -la /nix/store/56g65y0a57a2q51kd8js8jpbk0i4s1id-range-v3-0.9.1/bin

cc @xwvvvvwx for testing.

Missed this PR when I've prepared #70414 :o
This PR is fine but I'll close it in favour of the other one to add me as maintainer (it seems like range-v3 is currently unmaintained and since tdesktop is the only package in Nixpkgs that depends on it that seemed like a good idea).


New CDN doesn't seem to work

I can't get Twemoji to work, it appears that MaxCDN doesn't host the files anymore?
For example this script:
<EMAIL_ADDRESS>leads to this URL:
https://twemoji.maxcdn.com/v/14.0.2/
However I didn't find anything there, under any version. All dead links.
:(
Any solution, or I'm doing something wrong?
Thanks!

Thanks for the reply, however I still can't seem to get it working.
For example when I'm trying to open/access this URL I get a 404:
https://cdnjs.cloudflare.com/ajax/libs/twemoji/14.0.2/twemoji.min.js
It seems older versions are working, for example 12.0.4, however that leaves me with another issue, for example this URL:
https://cdnjs.cloudflare.com/ajax/libs/twemoji/12.0.4/twemoji.min.js
The CDN referred is still maxcdn.com, for a newbie like me, could I ask how you made it work? I am looking hardly to get the flag emojis working on my websites, if I can't make it work, I'll just self-host it at the end, but you got my curiosity if there's any chance of figuring this out.
Thanks!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_6523ee18-c9af-44dc-b2d6-7990fbe89436/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_6523ee18-c9af-44dc-b2d6-7990fbe89436.git

Docs Build status updates of commit a7e42f9:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-BuildPermissionInsufficientError] Your repository https://github.com/VSC-Service-Account/docs-archive-test-source has enabled Docs Build service account. It seems that at least one of the account, e2ebd2, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master) and then retry the build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


What is the form of the output?

Hello! I noticed that this module returns an array of numbers.

What can I expect of the sizes of each number? The largest I got was 303 on a small example, and the smallest was 34. Is there a guaranteed lower bound? And upper bound?

What can I expect of the length of this array?

@papb - It looks like you might have confused the compression implemented in this library for something else. This library is useful for applying the popular Lempel–Ziv–Welch lossless compression algorithm on large JSON objects, so that it can be persisted into NoSQL databases or into the IndexedDB on the browser (for offline use).
As mentioned on the README, the output of the pack() call cannot be consumed directly - but if you persist this "packed" output instead of the original JSON into a NoSQL database, you should see the difference in the space utilization.
When you want to consume the "packed", large JSON object - you will then retrieve it from your persistence layer and the call unpack() from the library.
Does that help?

Hello, thanks for the fast reply! I already knew what you said, sorry for not being clear on the question. I don't have any knowledge on how the LZW algorithm works - what I know is that it is a compression algorithm. The fact that the output of this algorithm consists of an array of numbers was something I didn't know, and learned by running this module. Now I want to know what can I expect about the bounds in these numbers and the array length, in order to think better on the persistence layer that I will use. For example, if I have a way to persist an array of numbers from 0 to 1023, can I use it? Or is it possible that this algorithm generates a number above 1023 at some point?

I don't think we can definitively say what the upper bound is, for the numbers in the "compressed" array - and the reason it is like that is because the numbers will depend on the specific input passed into the core LZW compress function (which takes a random string and converts it into an array).
At the core (as can be seen on the function), the LZW algorithm starts with the ASCII character set loaded into a dictionary/map. And during the processing of a given input, it will start accumulating string segments that are repeated within.
Here is an example input of "awesome awesome", that is being processed by this core compress function along with the data structure within the function visualized as it is processed: Link to JavaScript Visualizer in PythonTutor.com
As you can see, for this input - the numbers go all the way up to 260 - hopefully this gives some insight into what you are dealing with.


Add on_hold RPC method

Preview is available here:http://developers-pr690.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr690.previews.kube001.services.stellar-ops.com

Something went wrong with PR preview build please check

Preview is available here:http://developers-pr690.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr690.previews.kube001.services.stellar-ops.com


Xfail ReactiveExtensions

ReactiveExtensions uses the legacy build system and it failed when we are using the
new driver. The new driver doesn't integrate with the legacy build system.
rdar://75437284

@swift-ci please test

@nkcsgexi Can you file a bug on bugs.swift.org?

@shahmishal to clarify, this is not necessarily a client-side issue. The legacy build system should be able to always pass down -disallow-use-new-driver and these tests will pass.

@shahmishal here you go: https://bugs.swift.org/browse/TF-1357

Thanks!

@swift-ci please test

We dont need this change because https://github.com/apple/swift-source-compat-suite/pull/497

Closing.


Log rotation of the airflow processes logs.

Is your feature request related to a problem? Please describe.
Currently, airflow processes logs are getting accumulated in one file which results into restarting of the container for cleaning up of space occupied by large files.
Describe the solution you'd like
Process logs should be rotated into multiple files on a daily so that that old logs deletion would be easy and quick.
Additional context
Log rotation along with archival of old logs would be great.

changes are done and merge in the main branch after testing. Added logrotation package & config file for airflow processes. Also, cron is scheduled for every hour.


NetworkPlugin cni failed on the status hook for pod 'nginx' - invalid CIDR address: Device "eth0" does not exist.

I have the following kubelet error on my minions:
Jul 02 16:20:42 sc-minion-1 kubelet[46142]: E0702 16:20:42.899902   46142 manager.go:309] NetworkPlugin cni failed on the status hook for pod 'nginx' - invalid CIDR address: Device "eth0" does not exist.
My 10-calico.conf on all nodes looks like this:
{
    "name": "calico-k8s-network",
    "type": "calico",
    "etcd_authority": "<IP_ADDRESS>:6666",
    "log_level": "info",
    "ipam": {
        "type": "calico-ipam"
    }
}

I also ran: calicoctl pool add <IP_ADDRESS>/16 --ipip --nat-outgoing on all nodes.

@jonathan-kosgei We've seen that error from a known k8s bug, where it queries the state of the pod before the CNI plugin has finished setting it up.  Were there any other symptoms you were seeing; did the pod come up and get networked?

@fasaxc Hi, the pod comes up, and I can curl ngin from the node it's been scheduled on. What led to to this however was that I couldn't curl the pod ip or the service ip from across hosts.

Were you curling from another host or from a pod on another host?  Was the other host running Calico? Do you have any security policy in place that could be blocking the traffic?  Please can you share the output from ip route on the host that you were curling from?

The two hosts, kubernetes minions have the exact same setup, automated install via ansible. I tried curling from both the host itself and a busybox pod via docker run --rm -it busybox sh; actually wget on the busybox pod. I have port 179 tcp/udp open and calicoctl status is established between the two hosts. Let me just get that for you.

On the host I was curling from:
root@sc-minion-0:~# ip route
default via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/24 dev lxcbr0  proto kernel  scope link  src <IP_ADDRESS>
<IP_ADDRESS> via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/24 dev eth0  proto kernel  scope link  src <IP_ADDRESS>
<IP_ADDRESS>/16 dev docker0  proto kernel  scope link  src <IP_ADDRESS>
<IP_ADDRESS> dev calife2d2bc6404  scope link
<IP_ADDRESS> via <IP_ADDRESS> dev tunl0  proto bird onlink
<IP_ADDRESS>/26 via <IP_ADDRESS> dev tunl0  proto bird onlink
blackhole <IP_ADDRESS>/26  proto bird

On the host with the nginx pod running:
root@sc-minion-1:~# ip route
default via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/24 dev lxcbr0  proto kernel  scope link  src <IP_ADDRESS>
<IP_ADDRESS> via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/24 dev eth0  proto kernel  scope link  src <IP_ADDRESS>
<IP_ADDRESS>/16 dev docker0  proto kernel  scope link  src <IP_ADDRESS>
<IP_ADDRESS> via <IP_ADDRESS> dev tunl0  proto bird onlink
blackhole <IP_ADDRESS>/26  proto bird
<IP_ADDRESS> dev cali765599c241a  scope link
<IP_ADDRESS>/26 via <IP_ADDRESS> dev tunl0  proto bird onlink

What's the IP of the pod you were accessing?

The pod nginx ip is <IP_ADDRESS>, the nginx service ip is <IP_ADDRESS>

The routing looks correct to me.  Can you try running traceroute to see where the packets are being dropped?  You could also run tcpdump -nli tunl0 on each host to see how far the packets get; is it the request or response that doesn't make it through?
Do you have any Calico network policy in place?

Not sure what exactly is the calico network policy, I don't have a deep knowledge of networking or running traceroute, I've uploaded the diags to https://transfer.sh/VcSyu/diags-040716-155209.tar.gz maybe that could help, or if you could walk me through.

@jonathan-kosgei I mean, have you configured any Calico security rules?  We provide networking and a distributed firewall; have you created any firewall rules?

@fasaxc No, to my knowledge no, only ports I have open on the nodes are 10250, 179 udp/tcp , 22, 80, 443

The diags confirm, looks like there's no policy in place.  Where are you running your cluster?  I wonder if the underlying network fabric is blocking IPIP packets.

Actually on azure, I had run calicoctl pool add <IP_ADDRESS>/16 --ipip --nat-outgoing on all the nodes

I think Azure blocks IPIP traffic, which prevents Calico networking from working.

I believe we're tracking the Azure issue here: https://github.com/projectcalico/calico-containers/issues/949

Thanks for the pointer, this sucks though, have to switch to weave or flannel.

Flannel and Calico are actually merging under the new Canal project so Calico security policy alreayd works with flannel.


Fixes for Python 3

Small changes that should allow people with Python 3 as their default system version to use installcab.py, without warnings. Should fix #2.

subprocess.check_output() returns bytes, so checking if it contains a str raises an error
getchildren() raises a DeprecationWarning on python 3

Thank you for your fixes and sorry for taking so long to apply them.


Number format

I had to correct the example JSON files by hand in order to adjust the representation of numbers in various places. In particular timestamps, such as
2019-10-21T11:00:30.000+01:00
may be written as 2019-10-21T11:00:30+01:00 which is the same number but displayed at a different precision leading to a different hash. The same happened for sensor data, e.g.
<sensorReport type="gs1:Speed" value="161.0" uom="KMH"/> was translated to have a value of 161 in JSON (no .0).
Should we address this in our algorithm? E.g. remove trailing "0" after the "."?
Originally I considered this out of scope, but in practice this will cause lots of trouble if we do not specify a way to canonicalize numbers.

Thanks for making me aware of that, @Echsecutor!
For the canonicalisation, how do you like us to specify that the millisecond (SSS) part of an ISO timestamp MUST be present when building the pre-hash string? That means that even though the abbreviated version was technically correct, it MUST be padded with three zeros.

 This was my original intention, but I am not sure if people will be happy with it... ;)

As discussed: out algorithm needs to pad the timestamp to ms precision (see https://github.com/RalphTro/epcis-event-hash-generator/commit/60042f3b12f609a4fd5e900257362bafc901dcb6 ). All other figures should not have trailing 0s


Add dir() to ObjectProtocol.

@m-ou-se
Could you please rebase on the master?
#888 would resolve the travis failure.
In addition, any test for this API is appriciated.

A general comment about these kinds of functions - dir, repr etc.
At the moment they're trait methods, but I personally would prefer them to be free functions like this:
fn dir<T>(obj: T) -> &PyList
where T: AsRef<PyAny>
{ ... }

// Use later as
pyo3::dir(obj);

It's not a strong feeling, but I think it is more consistent with how these work in Python, where they're also used as dir(obj), repr(obj) etc.
Wonder how you feel about this?
(If we wanted compatibility we could leave them on ObjectProtocol but mark as deprecated?)

A general comment about these kinds of functions - dir, repr etc.

A bit off-topic and should be discussed in another thread?

A bit off-topic and should be discussed in another thread?

I thought it was relevant given we're adding dir now, but sure I'll move this to another issue.

Many thanks @m-ou-se for updating this PR. As I guess you have a need for dir(), I'd be interested to hear what you think of the idea proposed in #892 . (We could have pyo3::dir() and o.__dir__() in the way suggested by that PR.)

@davidhewitt Ah, didn't see that one yet. Looks good.

@davidhewitt Ah, didn't see that one yet. Looks good.

I think I pushed that PR about 20 seconds after you updated this branch¬ 😄  I'll wait for some more feedback to see what others think of that PR, but I will definitely make sure to add dir() .

FWIW please do merge this PR without waiting for #892 - we might wait a little while to decide the right design there.

Thank you!


Checking mail notifcation stays forever if server is not reachabel

It happens from time to time that k9 mail is not able to reach the server and keeps the checking mail notification for ever. I observed this i.e. in wifi networks that have no uplink. Sometimes when i force k9 mail to pull mails in such a network (of course by mistake) that the "cheking mail" notification pops up and stays until i reboot the phone. Killing k9mail via the app settings dialog of android is not helping. It also stays there after k9 successfully pulled mails trough a different wifi network.
I'm running k9 on an nexus 5 with android 5.1.1 the mail server on the other side is a dovecot imap on debian old stable. Latest k9 version tested is 5.006 from fdroid.

I have no idea what could be causing this. But most of the notification code was rewritten in PR #795 and it's likely this problem was also fixed by the rewrite.
I will close this issue once the pull request is merged into master. If you're still seeing the issue with the next stable release (5.2xx, not yet released, no ETA) please open another issue.


Large Result collection observation issue (time lag)

Goals
I'm trying to observe large Result object and obtain notifications quickly. There's time lag while observing Results contains of 40k entities , each with extra subentity, like Dog->Owner, one to one. Data is provided using pagination algorithm.
Is there is a way I can obtain Results change notification in faster way?
Expected Results
Observing small list (65 entities):
15:46:25006 new pax value = 1 (property change)
15:46:25039
filteredInsertions = []
filteredDeletions = []
filteredModifications = [0]
delta is 33ms
Works pretty fast, tableview modifies it's cell according to new value almost instantly (places checkmark on cell)
Actual Results
40k entities (like described above)
15:44:15178 new pax value = 4 (property change)
15:44:15832
filteredInsertions = []
filteredDeletions = []
filteredModifications = [0]
delta is 654ms
Works pretty slow, it takes time (even after swipe left/right cell animation finished) to renew cell state
Steps to Reproduce
Code Sample
Just regular Result collection observation.
notificationToken = rsvps.observe { [weak self] (changes) in

                guard let self = self else {
                    Logger.log("Unable to resolve self")
                    return
                }

                self.updateRsvpCounters()
                switch changes {
                case .initial:
                    self.tableView.reloadData()
                case .update(_, let deletions, let insertions, let modifications):

                    Logger.log("""
                        \nfilteredInsertions = \(insertions)
                        filteredDeletions = \(deletions)
                        filteredModifications = \(modifications)
                        """)

                    self.tableView.beginUpdates()
                    self.tableView.insertRows(at: insertions.map { IndexPath(row: $0, section: 0) },
                                              with: .fade)
                    self.tableView.deleteRows(at: deletions.map { IndexPath(row: $0, section: 0) },
                                              with: .left)
                    self.tableView.reloadRows(at: modifications.map { IndexPath(row: $0, section: 0) },
                                              with: .none)
                    self.tableView.endUpdates()
                case .error(let error):
                    Logger.log("Error: \(error)")
                }
            }

Version of Realm and Tooling
Realm framework version: 3.13.0
Xcode version: Version 10.1 (10B61)
iOS/OSX version: Mojave 10.14.2
Dependency manager + version: CocoaPods 1.6.0.beta.2
PS: Issues which can be related to my, but I found no answer there: #5689

@dmitryzawadsky

Is there is a way I can obtain Results change notification in faster way?

The first thing to consider are the other properties of rsvps. With collection notifications every nested object property could trigger a notification.
You could explore inverse relationships for properties you do not want to trigger. And Object notifications are another method to more granularly observe objects.
And in the event that you're measuring the delta time via the UI, you may also want to use dispatch groups to update the UI. I've seen cases where a developer's notifications are firing promptly, but the UI update is de-prioritized, causing a visual lag.


iso4217.json relative path

It seems that iso4217.json path is relative to implementing src project, should be relative to module src.

never mind pypi is not including data files :facepalm:

Fixed at https://github.com/juanpsenn/python-todopago/commit/baa09c30194a2122b2dd1fc8c89065ef431b1359


Invisible proxy icon (AKA window title icon)

Steps to reproduce
Start without MacVim running.

open -a MacVim file.md. mvim file.md also works the same (I have a symlink to mvim in my PATH).
Observe the lack of the proxy icon.

Actually, the proxy icon still works if you know where to start dragging, but it is invisible.
Workaround (further adventure)

Go to appearance settings

Check "Hidden title bar" and uncheck it again
The proxy icon is visible now.

Expected behaviour
The proxy icon should be visible regardless.
Version of Vim and architecture
MacVim r180, Vim 9.1.727, ARM, on
Environment
Sonoma 14.7
I'm using the dark theme for it. Somewhere in Apple's Settings, I also selected the option to make it show the proxy icons without having to click on the title first. Update: I mean the "Apple Settings -> Display -> Show window title icons" option.
How MacVim was installed
From GitHub
Logs and stack traces
No response
Vim configuration where issue is reproducable
No response
Issue has been tested with given configuration

[ ] by running MacVim.app from GUI macOS interface
[X] by running vim/gvim/etc installed by MacVim
[ ] by running other versions of vim (e.g. /usr/bin/vim)

Issue has been tested with no configuration

[X] by running mvim --clean (or gvim, supplied by MacVim distribution)
[ ] by running vim --clean (in terminal, supplied by MacVim distribution)
[ ] by running vim --clean (in terminal, other suppliers, e.g. /usr/bin/vim)

Other conditions

[ ] The both Homebrew packages "vim" and "macvim" are installed

This was actually done intentionally when I implemented #1038. It isn't really a bug as I just thought this looks cleaner, but in retrospect providing an option to show file name or not might have been the easier / cleaner solution.
Was the motivation for this issue post entirely for the ripdrag linked issue? I was curious what you use the icon for.

I would currently mainly use it for that, yes. I may or may not open the file only to drag it.
It feels like a bug that toggling the "hide window bar" option changes the look.
By default, Mac OS Sonoma hides the icon until you click on the window title. I'm not sure how easy that is, but ideally you'd follow that behavior and respect the "show window icons" system option for people who want to avoid that click.

Yeah I think having a separate option properly makes more sense. Sometimes trying to be too smart is detrimental to having a consistent setting. I'll put this to backlog.

By default, Mac OS Sonoma hides the icon until you click on the window title. I'm not sure how easy that is, but ideally you'd follow that behavior and respect the "show window icons" system option for people who want to avoid that click.

I think this only applies to the combined toolbar mode. For example, Finder and Preview does that, but TextEdit doesn't.
In fact, MacVim also does that if you use the toolbar as well (:set go+=T). Either way it wouldn't affect our decision here.


Apply authentication to agent apiserver endpoint

Antrea agent uses apiserver to interact with antctl. However, unlike the
controller, the API endpoint is exposed only locally which limits antctl
to local execution.
Additionally, we would like to reuse the listener for Prometheus metrics,
which would require external access to the API endpoint's metrics path.
Having authentication in a similar way to the controller's implementation
could help here as it will allow external exposure of the API endpoint.
Fixes: #613

/test-all

/test-all

/test-all

/test-all

/test-all

/test-e2e
/test-conformance

/test-e2e

@jianjuns do you have other comments? If not, I think we could merge this now.

@jianjuns do you have other comments? If not, I think we could merge this now.

I am good.

/test-all

/test-all


\ERROR in src/app/app.component.ts(27,54): error TS2339: Property 'getDownloadURL ' does not exist on type 'AngularFireUploadTask'.

Hi Please help me to show this error getDownloadURL

This is not an issue with the package, and this is not the place to ask for help troubleshooting your code. In the future you should ask for help on StackOverflow.
There's a post from a week and a half ago for this exact error message:
https://stackoverflow.com/questions/50541836/property-downloadurl-does-not-exist-on-type-angularfireuploadtask/50663965#50663965


[Bug] subprocess.CalledProcessError: Command '['D:\Program\Anaconda3\envs\mlc-chat\python.exe', '-m', 'mlc_chat.cli.check_device', 'vulkan:0']' returned non-zero exit status<PHONE_NUMBER>.

🐛 Bug

I used mlc llm to compile Llama-2-7b-chat-hf, but when using mlc-chat, I reported an error:
subprocess.CalledProcessError:  Command '['D:\Program\Anaconda3\envs\mlc-chat\python.exe', '-m', 'mlc_chat.cli.check_device',  'vulkan:0']' returned non-zero exit status<PHONE_NUMBER>.
To Reproduce
Steps to reproduce the behavior:
python sample_mlc_chat.py

Complete error reporting:

Traceback (most recent call last):
File "E:\code\mlc-llm-win\mlc-llm\sample_mlc_chat.py", line 8, in
cm = ChatModule(model="Llama-2-7b-chat-hf-q4f16_1")
File "D:\Program\Anaconda3\envs\mlc-chat\lib\site-packages\mlc_chat\chat_module.py", line 660, in init
self.device = detect_device(device)
File "D:\Program\Anaconda3\envs\mlc-chat\lib\site-packages\mlc_chat\support\auto_device.py", line 27, in detect_device
if _device_exists(cur_device):
File "D:\Program\Anaconda3\envs\mlc-chat\lib\site-packages\mlc_chat\support\auto_device.py", line 58, in _device_exists
result = subprocess.check_output(cmd, stderr=subprocess.STDOUT).decode("utf-8")
File "D:\Program\Anaconda3\envs\mlc-chat\lib\subprocess.py", line 420, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "D:\Program\Anaconda3\envs\mlc-chat\lib\subprocess.py", line 524, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['D:\Program\Anaconda3\envs\mlc-chat\python.exe', '-m', 'mlc_chat.cli.check_device', 'vulkan:0']' returned non-zero exit status<PHONE_NUMBER>.

Environment

Platform (e.g. WebGPU/Vulkan/IOS/Android/CUDA): Vulkan
Operating system (e.g. Ubuntu/Windows/MacOS/...): windows
Device (e.g. iPhone 12 Pro, PC+RTX 3090, ...)

My device is 3060TI 8G and I also encountered this problem.

Is it possible that this is because I am a GTX 1650 4G graphics card, because MLC currently does not support the CPU, and then the PC's graphics card is not enough memory?

It’s not about GPUs, but more like an OS issue. @Tao-begd @MaTwickenham Are you both on windows?

Could you run the following command and share the error message:
python -m mlc_chat.cli.check_device vulkan:0

python -m mlc_chat.cli.check_device vulkan:0,the result is 1

It seems to actually work…Did it crash or any error message popped up?

It’s not about GPUs, but more like an OS issue. @Tao-begd @MaTwickenham Are you both on windows?

Could you run the following command and share the error message:
python -m mlc_chat.cli.check_device vulkan:0

python -m mlc_chat.cli.check_device vulkan:0,the result is 1

It seems to actually work…Did it crash or any error message popped up?

Yes, I encountered that error on Windows. And my error message is just like @Tao-begd posted.

Seems that the subprocess checking is a bit unreliable in windows. Maybe we can return to vulkan().exist check in the same process ?

Thanks to Professor Tianqi, the problem is solved

Thank you for reporting!

I'm able to reproduce this issue:

using:
from mlc_chat.support.auto_device import detect_device
print(detect_device("auto"))

Removing "vulkan" in AUTO_DETECT_DEVICES avoids the sub-process from crashing, which means it's a TVM-specific bug with its Vulkan runtime @tqchen

In fact, directly runs the command below actually does fail, but cmd.exe just hides this problem silently:

https://github.com/mlc-ai/mlc-llm/pull/1350


Update last load/today and DST indicators to pass accessibility

Current Carbon Last Load (yellow) and DST (gray)

Update to orange square and line and blue/gray diamond and line:

NEW blue/gray: #405978
NEW orange: #C97318
Instead of using the triangle for DST, we've determined to use the diamond. You can use the existing regular diamond icon from Carbon with the new color.

:tada: This issue has been resolved in version 2.14.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


single-pool-js: redo packaging

please work please work please work

alright i think... im finally free. cc @steveluscher @joncinque you dont need to code review im gonna merge as-is but you might be interested in the details of what i can only pray to god is the resolution to all this
the tldr is use tsc to produce separate es6 and cjs versions of the library, do not use type in the main package.json but monkeypatch it in dist/. the only weird fucking thing about all this is i cant for the life of me get import './internal' to work, so i added .js extensions (in ts files... lol...) as far as i can tell node deliberately broke filename resolution in 19, which would explain why stake pool works fine on 18... i dont know

This removed solana-bankrun and still somehow passed, so I looked at the CI, and it seems like it's still broken unfortunately https://github.com/solana-labs/solana-program-library/actions/runs/6800097574/job/18488277665#step:6:317

well the good news is that's unrelated, i removed bankrun from the one of the two packages that doesnt have tests in it, so it shouldnt affect anything
the bad news... is:

i have no idea why the script failing still makes ci think it passed...?
this is because node removed another flag i depend on to make node actually work. and they did it on a minor version of the lts release. this should pass on 20.5 but starts failing with 20.6

i will... take care of it

Ah ok, that makes sense! Sorry for not looking further into it. But yeah, something strange is going on...

oh i know why, its because i modified the test script to be edit package.json && run test ; edit package.json because the reversion needs to happen regardless of if the tests pass or fail but this means now the test command always returns 0. annoying

thank you for catching this!


fix(.travis.yml): support [0-1] leading v's

Updated that, @mboersma. Thanks for the regex beautification suggestion.


Zeto domain - foundation with 2 Zeto token implementations

Based off of the initial implementation from @awrichar , I think here's a bullet list of things needed for the next phase (still not an exhaustive list for getting to the end of job of the Zeto domain):

make the "Paladin domain interface" for the Zeto domain more generic, so that it doesn't have all the specific knowledge of how Zeto internals work like the list of constructor parameters (addresses of the various ZKP verifier contracts, etc.)
make the domain support multiple Zeto token implementations via standard "Zeto interface(s)" to be defined in Solidity. this was the specific ABI and bytecode can be dynamically loaded from a folder of the user's choice, rather than being embedded in the binary. this allows clients to provide additional or custom Zeto implementations for Paladin to work with
add the other Zeto token implementations to be supported via the above mechanism
allow the flexibility for users to pre-deploy Zeto tokens and tell the domain about it via configuration

already implemented


Can we get a new release?

There hasn't been a release of google.golang.org/protobuf in over six months. There 18 commits, some of which look pretty useful for users. Could we get a v1.29.0 release?
Also, there's been a small fix in this older repo(in jsonpb package of github.com/golang/protobuf), sitting on master unreleased, for a year and a half. Maybe we need a v1.5.3 ("final release") of that, which could be cut first and referenced from the newer module's go.mod?

@lfolger, thank you!!


Distributed Objects and better global state API

Goals

To distribute state across processes; different processes would be responsible for part of the global state; processes would then make RPC calls to an object "owner" to get access to it;
First implementation is going to be naive (based on lock groups); lots of opportunities for optimization here, by finding optimal object placement (depending on how often processes access global state, detection of communication channels, etc.)
This would include a better, location-independent API that allows processes to access global state. Instead of having globalState.GetStringCollection(), we could have queue.Get(), where queue is a struct defined in the pgonet package that handles Get according to the variable type. All types implement the same interface. Also, Get could be translated to a local data-structure access or an RPC call to the processes that "owns" the object.

Closing this since the existing runtime API in the state-server strategy was chosen.


Test fails because the string is not in the correct case

At least on my computer, this test failed because the key in my registry is in uppercase - SOFTWARE and not Software
I think it will be fine to make the check case insensitive, cause either way windows doesn't care the case of the registry key - it could also be SoFTwarE.

Thanks. Will update this soon

Duplicate of #36


Add zdb unit tests

Which package/s are you suggesting this feature for?
grid_client
Describe the solution you'd like
Add unit tests for zdb.ts

Already done here

testcase


Implement filesystem watch

Current listeners only listen for saves occurring in-editor, but it would be nice if it listened for general filesystem changes.

Covered in #51


Placeholder in DSP runtime configuration prepends unnecessary data

Whilst configuring DSP runtime configuration I got stung by this, the placeholder prepends a :port/pipeline to the route url (it also says service as opposed to route --just a nit pick).
It doesn't work with it, but works without it.

Only for notebook-devs:
Goal: Figure out service work in the endpoint URL , if not change the placeholder to route-url.
Acceptance criteria

[ ] Test if this issue is actually happening
[ ] Setup a cluster with RHODS/ODH, create any of the notebook with elyra.
[ ] Setup the runtime-configuration with service:port URL, Test the pipeline.
[ ] Change the placeholder, if the service:port URL doesn't work.

Based on the discussion with community contributors.
The users would like to have the option to use service-url if it possible.
https://odh-io.slack.com/archives/C035J0KA6UB/p1689970211014419?thread_ts=1689965765.550419&cid=C035J0KA6UB
https://odh-io.slack.com/archives/C035J0KA6UB/p1690179701867089?thread_ts=1689965765.550419&cid=C035J0KA6UB
as per this comment, it would be worth investigating

Enhancement to the current flow to enable users to use service-URL.

As this require some more investigation,
Moving this to next sprint 1.32.

Based on the work,
Pipeline can be used from service directly , by using the port
Example: https://ds-pipeline-pipelines-definition.test.svc.cluster.local:8888

@harshad16
Can you please confirm the documentation requirement for this. Would you like to add this to release notes as a known issue?
Would you recommend updating the existing user-facing documentation to further clarify this fix?

@chtyler, It is not an issue.
More documentation updates.
we need to update the documentation, stating that, the user can use the service-based URL.
instead of the route-based one.
Let's find a better way of updating this in the documentation.

@harshad16
I have created the following Jira to cover this work: https://issues.redhat.com/browse/RHODS-12954

This work is now completed with effort.
Service can be used with port info, as now the network policies are set in place.
Closing this issue. Thanks.


image.naturalwidth/height vs image.width/height

When referring to an image's size in an image element, the width will be the DOM width, whereas naturalWidth will be the image's actual width. This causes hashing differences if images are not drawn at 1:1 scale in the browser.
Workaround would be to copy the element and set the size of that element to the image's natural size

Oh good catch. I changed it so that it uses naturalWidth/height now in 2f2190794e03be75f169f220e11fda0df4d76e1e


Update guestagent suite

Test that pre and post snapshot scripts are executed when a snapshot request with guestflush is sent The windows equivalent of this is on OS feature doesn't have any guest environment components
Fix a bug with the telemetry test where it was triggering incorrectly and restart the agent after changing telemetry settings until we reevaluate agent telemetry behavior
/cc @dorileo @koln67 @drewhli

/lgtm


support for Xcode 10.2 swift5

I have try with "rm -rf /Library/Caches/Sourcery" and "--disableCache", but it`s does not work.The Xcode show error With "module compiled with Swift 4.2 cannot be imported by the Swift 5.0 compiler: /var/folders/xc/36050s51s4qwthtcyhgh0000gn/T/SourceryRuntime.framework/Modules/SourceryRuntime.swiftmodule/x86_64.swiftmodule
import SourceryRuntime"

I suppose that you're using version 0.15.0 of Sourcery which is not compatible with the Swift 5.0 compiler. This is a known issue that's been around for every Swift release.
However, from version 0.16.0, this will not be an issue anymore.
So if you need to use the Swift 5.0 compiler, please use version 0.16.0.


Use STM

The state management in Hackagebot is starting to get a bit hairy.
It works now, but long term we should use STM to enforce the relationships between all these moving parts.

Won't do


BUG: sparse.linalg: add locks to ensure ARPACK threadsafety

The ARPACK fortran code uses SAVE variables, and is not threadsafe or
re-entrant. Add locks and reentrancy checks to the Python side to deal
with this issue --- previously the code used to segfault.
As we use an explicit lock, it is not necessary for the Fortran code to
hold the GIL, so adjust the .pyf file accordingly.
Fixes gh-5846

@@            master   #5848   diff @@
======================================
  Files          235     235
  Stmts        43388   43402    +14
  Branches      8167    8168     +1
  Methods          0       0
======================================
+ Hit          33784   33798    +14
  Partial       2592    2592
  Missed        7012    7012

Review entire Coverage Diff as of bdaeaa8

Powered by Codecov. Updated on successful CI builds.

LGTM as is.
However, this will mean that we have two slightly different versions of reentrancy check code in sparse.linalg alone.  The version that this adds and the existing code in isolve/iterative.py. This seems like a sign that maybe this should be factored out somewhere.  _lib maybe?

refactored

removed unnecessary imports

Thanks, merged.


Please Help

I need help with Eagle-Eye:
=> Opening Webdriver
Traceback (most recent call last):
File "/home/kali/Descargas/EagleEye/EagleEye/eagle-eye.py", line 229, in
main(skipFB=args.skipfb, FBUrls=[], jsonRep=jsonRepFile, dockerMode=aDocker, dockerName=aName)
File "/home/kali/Descargas/EagleEye/EagleEye/eagle-eye.py", line 99, in main
f.grabData()
File "/home/kali/Descargas/EagleEye/EagleEye/grabber/facebook.py", line 16, in grabData
driver = cfg.getWebDriver()
File "/home/kali/Descargas/EagleEye/EagleEye/utils/config.py", line 45, in getWebDriver
return webdriver.Firefox(profile)
File "/usr/lib/python3/dist-packages/selenium/webdriver/firefox/webdriver.py", line 170, in init
RemoteWebDriver.init(
File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/webdriver.py", line 152, in init
self.start_session(capabilities, browser_profile)
File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/webdriver.py", line 249, in start_session
response = self.execute(Command.NEW_SESSION, parameters)
File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/webdriver.py", line 318, in execute
self.error_handler.check_response(response)
File "/usr/lib/python3/dist-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: Process unexpectedly closed with status 1

Check your geckodriver version.
I tested the current version with the latest firefox and geckodriver on MacM1 and it worked perfectly with for example Megan Fox.


[Bug] dydx api ignores 'fromISO' time-date spec

Background / motivation
In safe_fetch_ohlcv_dydx() when we call dydx and specify fromISO parameter, it ignores it, and instead returns values from now.
To see this, uncomment the last 3 lines in test_safe_fetch_ohlcv_dydx__real_response()

I've done a bunch of tests.

v4 API:

fromISO is ignored
toISO is ignored

v3 API

fromISO is ignored
toISO is ignored

v3 pylib

fromISO is ignored
toISO is seen. Good!
alas, dydx v3 is on the path to being deprecated. It's different than v4. We don't want to use it :(

How I got the v3 pylib results:
Clone the library. In console:
git clone https://github.com/dydxprotocol/dydx-v3-python.git

Install virtualenv. In console:
python -m venv venv
source /venv/bin/activate

pip install -r requirements.txt

Fix getargspec issue. In console:
pip install git+https://github.com/ethereum/web3.py.git

# Fixes this error
# ImportError: cannot import name 'getargspec' from 'inspect'<EMAIL_ADDRESS>
Open Python console:
python

In Python console:
from web3 import Web3

from dydx3 import Client
from dydx3.constants import MARKET_BTC_USD

client = Client(host='https://api.dydx.exchange')

candles = client.public.get_candles(
    market=MARKET_BTC_USD,
    resolution='5MINS',
    limit=1,
    to_iso="2024-03-16T03:35:00.000Z",
)

print(candles.data)

How I got the v3 pylib results
(Referenced in previous comment)
Clone the library. In console:
git clone https://github.com/dydxprotocol/dydx-v3-python.git

Install virtualenv. In console:
python -m venv venv
source /venv/bin/activate

pip install -r requirements.txt

Fix getargspec issue. In console:
pip install git+https://github.com/ethereum/web3.py.git

# Fixes this error
# ImportError: cannot import name 'getargspec' from 'inspect'<EMAIL_ADDRESS>
Open Python console:
python

In Python console:
from web3 import Web3

from dydx3 import Client
from dydx3.constants import MARKET_BTC_USD

client = Client(host='https://api.dydx.exchange')

candles = client.public.get_candles(
    market=MARKET_BTC_USD,
    resolution='5MINS',
    limit=1,
    to_iso="2024-03-16T03:35:00.000Z",
)

print(candles.data)

It should output:
{'candles': [{'startedAt': '2024-03-16T03:30:00.000Z', 'updatedAt': '2024-03-16T03:34:19.844Z', 'market': 'BTC-USD', 'resolution': '5MINS', 'low': '69123', 'high': '69218', 'open': '69191', 'close': '69192', 'baseTokenVolume': '1.5023', 'trades': '31', 'usdVolume': '103905.7281', 'startingOpenInterest': '1781.6770'}]}

@trizin - thanks for taking this one. Note the two newest comments above this one.
Ideas to pursue, from easiest / least work to hardest:

Q: is it possible that fromISO needs to be at a very specific value, ie exactly every 5 minutes? (If yes, that's awesome, a very simple solution)
Ask for help in dydx support channels (discord? Ask Veronica)
Directly call the V4 smart contracts. (The v3 py lib has a nice interface. Maybe there's already a v4 python wip?)

Seems like fromISO only works when used together with toISO.

That's great news! A straightforward path!


Wrong code formatting using Handlebars

Is there any reason for disabling the formatting of 'a', 'span', 'img', 'strong',... ???
https://github.com/beautify-web/js-beautify/blob/master/js/lib/beautify-html.js#L115

Thanks for the info.

@bitwiseman is there any special reason for leaving the following tags unformatted =>
https://github.com/beautify-web/js-beautify/blob/v1.5.4/js/lib/beautify-html.js#L115
unformatted = options.unformatted || ['a', 'span', 'img', 'bdo', 'em', 'strong', 'dfn', 'code', 'samp', 'kbd', 'var', 'cite', 'abbr', 'acronym', 'q', 'sub', 'sup', 'tt', 'i', 'b', 'big', 'small', 'u', 's', 'strike', 'font', 'ins', 'del', 'pre', 'address', 'dt', 'h1', 'h2', 'h3', 'h4', 'h5', 'h6'];

In this case "unformatted" means, not put on a new line and not reindented.  This makes sense as these tags can occur like
Example where the spacing around the tags
 matters.  If we reformatted these, we'd end up adding unintended whitespace to the output.

I would agree if we were talking about pure HTML but in Handlebars you have sometimes if-blocks inside span for example:

    SMALL TEXT

        {{#if isOwner}}

        {{else}}

        {{/if}}

    {{userName}}: {{text}}

I formatted the previous code manually, but if I let beautify do it form me the code will look like:

    SMALL TEXT

        {{#if isOwner}}

    {{else}}

        {{/if}}

        {{userName}}: {{text}}

I marked this as a bug already.  The behavior is obviously wrong, and should be fixed.
Are you trying to say that the issue you're seeing is caused by the "unformatted" setting?

I'm not sure, just trying to understand the real issue & maybe find a temporary workaround :)

This appears to have been fixed:
{
"indent_size": "4",
"indent_char": " ",
"max_preserve_newlines": "5",
"preserve_newlines": true,
"keep_array_indentation": false,
"break_chained_methods": false,
"indent_scripts": "normal",
"brace_style": "collapse",
"space_before_conditional": true,
"unescape_strings": false,
"jslint_happy": false,
"end_with_newline": false,
"wrap_line_length": "0",
"indent_inner_html": false,
"comma_first": false,
"e4x": false,
"indent_handlebars": true
}


ImageCard-TodaysLunch: New feature that allows to display multiple dishes on the same day

Q
A

New feature?
yes

What's in this Pull Request?
It's possible now to plan multiple dishes on the same day, these will be displayed in the adaptive card slider

@giuleon Thanks for the update. I have approved it.


"Reverse for 'interface bulk_disconnect' not found." Problem adding Interfaces to Devices

Issue type: bug report

Python version: 2.7.5
NetBox version: 2.0.5

Created a Device and Added Interfaces with "LOM [1-2]" form factor: 1000BASE-T (1GE) and then I get the "Reverse for 'interface_bulk_disconnect' not found." error message.

The bulk disconnect feature was just added in v2.0.7. It looks like you've upgraded your code base but still need to restart the WSGI service.

Thank-you!  I restarted gunicorn and it's working fine.  Thanks again.


features not working

Is your feature request related to a problem? Please describe.
As i tried the features of this project from side nav bar , then these features are not working like I choose circle there is no response of the web app
Describe the solution you'd like
it should response when i try the features from left nav bar

@jatinjain001 were you using any local server like localhost or go live server of vs code ? because app will respond if open directly

yeah i checked it
On Thu, Jun 17, 2021 at 11:05 PM 111arpit1 @.***> wrote:

@jatinjain001 https://github.com/jatinjain001 were you using any local
server like localhost or go live server of vs code ? because app will
respond if open directly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/makesmatheasy/makesmatheasy/issues/5286#issuecomment-863431035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCUOJASJMIZECF376ETHV3TTIW7TANCNFSM4634X5UQ
.

https://user-images.githubusercontent.com/55352601/122456362-16a2a280-cfcb-11eb-8000-526a7c8c8595.mp4
@jatinjain001 I couldn't find any such issue with the app can you you please share a video or mention some specific  section  so that i could check them

Ok, I cross checked you can close my issue its working fine now
On Fri, Jun 18, 2021 at 12:23 AM 111arpit1 @.***> wrote:

https://user-images.githubusercontent.com/55352601/122456362-16a2a280-cfcb-11eb-8000-526a7c8c8595.mp4
@jatinjain001 https://github.com/jatinjain001 I couldn't find any such
issue with the app can you you please share a video or mention some
specific section so that i could check them
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/makesmatheasy/makesmatheasy/issues/5286#issuecomment-863485012,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCUOJBMH67FRU6ZUQ24USLTTJAA5ANCNFSM4634X5UQ
.

Ok, I cross checked you can close my issue its working fine now
…
On Fri, Jun 18, 2021 at 12:23 AM 111arpit1 @.***> wrote: https://user-images.githubusercontent.com/55352601/122456362-16a2a280-cfcb-11eb-8000-526a7c8c8595.mp4 @jatinjain001 https://github.com/jatinjain001 I couldn't find any such issue with the app can you you please share a video or mention some specific section so that i could check them — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#5286 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALCUOJBMH67FRU6ZUQ24USLTTJAA5ANCNFSM4634X5UQ .

No worries @jatinjain001 take your time see its a very big app so its obvious to have some bugs .
Take your time explore the whole app and if you find any such kind of bug kindly create a new issue for that. And try to give some more details about the issue .
happy coding :)


create model error

When I call m = timm.create_model("efficientnet_b0", pretrained=True)
There is an error: RuntimeError: unexpected EOF, expected 72233 more bytes. The file might be corrupted.
I don't know how to fix it.
Please help me.

Initially I run the code and download pretrained file but there is no progress so I type CTRL+C stop it. Then came out this error.

Sorry, I deleted cache file and redownload, it worked. My bad


[Bug] Card componet causing unresponsiveness on mobile screen

Description
The card component in the website has extra width, which is overflowing content outside the main div, hence causing unresponsiveness on a mobile screen.
The size needs to be fixed in small screen media queries .
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Assigning but make sure to do responsive changes for the small screens @pb0126


Clicking refresh freezes rqt

Opening rqt and then opening parameter reconfigure and hitting Refresh freezes whole rqt and it needs to be force shutdown.
Info: Ros2 Foxy, built by source. rqt_reconfigure dashing branch. Ubuntu 20.04.

This is happening to me too:
$ ros2 run rqt_reconfigure rqt_reconfigure
^C[INFO] [1605727363.846724404] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
PluginHandler.save_settings() plugin "rqt_reconfigure/Param#0" raised an exception:
Traceback (most recent call last):
  File "/opt/ros/foxy/lib/python3.8/site-packages/qt_gui/plugin_handler.py", line 191, in save_settings
    self._save_settings(plugin_settings, instance_settings)
  File "/opt/ros/foxy/lib/python3.8/site-packages/qt_gui/plugin_handler_direct.py", line 114, in _save_settings
    self.emit_save_settings_completed()
  File "/opt/ros/foxy/lib/python3.8/site-packages/qt_gui/plugin_handler.py", line 207, in emit_save_settings_completed
    callback(self._instance_id)
  File "/opt/ros/foxy/lib/python3.8/site-packages/qt_gui/plugin_manager.py", line 459, in _close_application_save_callback
    self._close_application_shutdown_plugins()
  File "/opt/ros/foxy/lib/python3.8/site-packages/qt_gui/plugin_manager.py", line 466, in _close_application_shutdown_plugins
    self._shutdown_plugin(
  File "/opt/ros/foxy/lib/python3.8/site-packages/qt_gui/plugin_manager.py", line 353, in _shutdown_plugin
    handler.close_signal.disconnect(self.unload_plugin)
TypeError: disconnect() failed between 'close_signal' and 'unload_plugin'

^C[INFO] [1605727364.574836832] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
^C[INFO] [1605727365.030892332] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
^C[INFO] [1605727365.359810927] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
^C[INFO] [1605727368.262765530] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
^C[INFO] [1605727368.406749937] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
^C[INFO] [1605727368.542732863] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
^C[INFO] [1605727368.694778147] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)

I'm also seeing a different stack trace, along with the node actually closing. This started happening after I saw the freeze and need to force-close rqt. In both cases, the behavior was instigated by clicking "Refresh".
$ ros2 run rqt_reconfigure rqt_reconfigure
^C[INFO] [1605727622.828869366] - rclcpp:/tmp/binarydeb/ros-foxy-rclcpp-2.2.0/src/rclcpp/signal_handler.cpp:signal_handler:202: signal_handler(signal_value=2)
[ERROR] [1605727622.927742918] - rqt_reconfigure:/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py:_prune_nodetree_pernode:438: Reconfigure GUI cannot connect to master.
Traceback (most recent call last):
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 452, in _refresh_nodes
    self._prune_nodetree_pernode()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 439, in _prune_nodetree_pernode
    raise e  # TODO Make sure 'raise' here returns or finalizes func.
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 436, in _prune_nodetree_pernode
    nodes = find_nodes_with_params(self._context.node)
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 129, in find_nodes_with_params
    return list(
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 131, in <lambda>
    lambda node_name: _has_params(node, node_name),
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 122, in _has_params
    client.wait_for_service()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/client.py", line 164, in wait_for_service
    return self.service_is_ready()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/client.py", line 143, in service_is_ready
    with self.handle as capsule:
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/handle.py", line 124, in __enter__
    return self._get_capsule()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/handle.py", line 106, in _get_capsule
    raise InvalidHandle('Tried to use a handle that has been destroyed.')
rclpy.handle.InvalidHandle: Tried to use a handle that has been destroyed.
Error in sys.excepthook:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/apport_python_hook.py", line 153, in apport_excepthook
    with os.fdopen(os.open(pr_filename,
FileNotFoundError: [Errno 2] No such file or directory: '/var/crash/_opt_ros_foxy_lib_rqt_reconfigure_rqt_reconfigure.1000.crash'

Original exception was:
Traceback (most recent call last):
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 452, in _refresh_nodes
    self._prune_nodetree_pernode()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 439, in _prune_nodetree_pernode
    raise e  # TODO Make sure 'raise' here returns or finalizes func.
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 436, in _prune_nodetree_pernode
    nodes = find_nodes_with_params(self._context.node)
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 129, in find_nodes_with_params
    return list(
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 131, in <lambda>
    lambda node_name: _has_params(node, node_name),
  File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 122, in _has_params
    client.wait_for_service()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/client.py", line 164, in wait_for_service
    return self.service_is_ready()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/client.py", line 143, in service_is_ready
    with self.handle as capsule:
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/handle.py", line 124, in __enter__
    return self._get_capsule()
  File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/handle.py", line 106, in _get_capsule
    raise InvalidHandle('Tried to use a handle that has been destroyed.')
rclpy.handle.InvalidHandle: Tried to use a handle that has been destroyed.

Same, though for me the trouble only appears when rviz2 is running in parallel. With a custom node using the parameters so far it does not crash and refreshes the list to include my node.
Once rviz2 is opened it freezes.
Am on Ubunutu 20.04 with a ros2 install by binaries.
and its consistent if I start it with rqt or with ros2 run rqt_reconfigure rqt_reconfigure
`

$ rqt
^C[INFO] [1607535108.410624788] [rclcpp]: signal_handler(signal_value=2)
[ERROR] [1607535108.546704721] [rqt_reconfigure]: Reconfigure GUI cannot connect to master.
Traceback (most recent call last):
File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 452, in _refresh_nodes
self._prune_nodetree_pernode()
File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 439, in _prune_nodetree_pernode
raise e  # TODO Make sure 'raise' here returns or finalizes func.
File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/node_selector_widget.py", line 436, in _prune_nodetree_pernode
nodes = find_nodes_with_params(self._context.node)
File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 129, in find_nodes_with_params
return list(
File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 131, in
lambda node_name: _has_params(node, node_name),
File "/opt/ros/foxy/lib/python3.8/site-packages/rqt_reconfigure/param_api.py", line 122, in _has_params
client.wait_for_service()
File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/client.py", line 164, in wait_for_service
return self.service_is_ready()
File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/client.py", line 143, in service_is_ready
with self.handle as capsule:
File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/handle.py", line 124, in enter
return self._get_capsule()
File "/opt/ros/foxy/lib/python3.8/site-packages/rclpy/handle.py", line 106, in _get_capsule
raise InvalidHandle('Tried to use a handle that has been destroyed.')
rclpy.handle.InvalidHandle: Tried to use a handle that has been destroyed.

`
Which strikes me as weird is the prompt that no master can be contacted, which sounds a lot like ROS1.

The root of the problem is that there are a couple incorrect assumptions in the code (specifically, the _has_params function in rqt_reconfigure's param_api.py module).
def _has_params(node, node_name):
    client = node.create_client(
        ListParameters,
        '{node_name}/list_parameters'.format_map(locals()))
    if not client.service_is_ready():
        client.wait_for_service()
    ret = len(client.call(ListParameters.Request()).result.names) > 0
    node.destroy_client(client)
    return ret

First, some background context: A node doesn't have to support the parameter services. It can set start_parameter_services in NodeOptions to false. Also, some nodes in the system may not be spinning and therefore don't processing incoming messages. These nodes might have the parameter services, but they won't respond to the incoming service request since they aren't processing messages.
However, _has_params is creating a service client for ListParameters, waiting for the service (and assuming a successful result), and is then doing a synchronous call, which blocks forever  since the node isn't spinning to process the request, resulting in a hang.
This code needs to be updated to both ensure that the service is actually present and ready and then to an async call with a timeout in case the node doesn't respond to the request.
I've entered an issue on this here: #96

I am having the same issue. Any progress on this?

I am having the same issue. Any progress on this?

Carved out some time to fix it: #98. Please give it a try and let me know how it goes. It will now timeout and print an appropriate message if a node doesn't respond to a parameters-related service request.

Carved out some time to fix it: #98. Please give it a try and let me know how it goes. It will now timeout and print an appropriate message if a node doesn't respond to a parameters-related service request.


Headless Chrome/Firefox instead of PhantomJS for example integration

The lead developer for PhantomJS is stepping down and both Chrome and Firefox are introducing a headless mode. We should look into updating the example code for how to integrate a headless browser with simplecrawler!

Agree, would like to see how to combine simplecrawler with a headless browser.

I'm using a headless browser. Basically, after headers are fetched, in the download condition I tell simplecrawler not to download the page and instead tell my headless module to do so.
Not very reliable when downloading 10000s of pages though.


Add checker pattern to pump probe for I24 serial

Closes #65
Adds new setting to to the pump probe options, which sends a command to the pmac to only have the pump_repeat applied every other city block.

Yes, it's small, which means that it's a really easy, small ophyd device. It seems like we might as well add this in ophyd now. Otherwise we'll end up adding more functionality in the old way of doing it whilst also trying to move to the new way of doing it, seems like wasted effort?


Cannot ping/dig domains even though i can ping ip addrs

In raising this issue, I confirm the following:
{please fill the checkboxes, e.g: [X]}

[X] I have read the documentation
[] Is it a feature request? please consider opening a [Discussion] (https://github.com/pivpn/pivpn/discussions/new)
[X] I have read and understood the contributors guide.
[] The issue I am reporting can be replicated.
[] The issue I am reporting is directly related to the pivpn installer script.
[X] The issue I am reporting isn't a duplicate (see FAQs, closed issues, and open issues).

Has your install failed?
no

Describe the issue

Expected behavior
I can ping/dig domains
i setup pivpn wireguard on a debian vps on OVH. I can connect to the vps via my pc via archlinux with the provided wireguard config file but i cannot  access any domain names. it is probably not a resolv.conf issue since dig @someotherdnsresolver gnu.org does not work either.
pinging an ip
[ak@aspire7:~] 13μs $ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=55 time=282 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=55 time=316 ms
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 2 received, 33.3333% packet loss, time 2002ms
rtt min/avg/max/mdev = 281.932/299.101/316.271/17.169 ms

pinging a domain
[ak@aspire7:~] 13μs $ ping gnu.org
ping: gnu.org: Temporary failure in name resolution

/etc/resolv.conf
[2] [ak@aspire7:~] 13μs $ cat /etc/resolv.conf
# Generated by resolvconf
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

dig gnu.org with <IP_ADDRESS>
[ak@aspire7:~] 13μs $ dig @<IP_ADDRESS> gnu.org

; <<>> DiG 9.18.1 <<>> @<IP_ADDRESS> gnu.org
; (1 server found)
;; global options: +cmd
;; connection timed out; no servers could be reached

I can ssh into the vps via both its public and private (wg) ip.
setupVars.conf
debian@vernvps:~$ cat /etc/pivpn/wireguard/setupVars.conf
PLAT=Debian
OSCN=bullseye
USING_UFW=0
pivpnforceipv6route=1
IPv4dev=ens3
install_user=debian
install_home=/home/debian
VPN=wireguard
pivpnPORT=51820
pivpnDNS1=<IP_ADDRESS>
pivpnDNS2=<IP_ADDRESS>
pivpnHOST=<IP_ADDRESS>
INPUT_CHAIN_EDITED=0
FORWARD_CHAIN_EDITED=0
INPUT_CHAIN_EDITEDv6=
FORWARD_CHAIN_EDITEDv6=
pivpnPROTO=udp
pivpnMTU=1420
pivpnDEV=wg0
pivpnNET=<IP_ADDRESS>
subnetClass=24
pivpnenableipv6=0
ALLOWED_IPS="<IP_ADDRESS>/0, ::0/0"
UNATTUPG=1
INSTALLED_PACKAGES=(git dnsutils grepcidr bsdmainutils iptables-persistent wireguard-tools qrencode)

Please provide your system information
What type of hardware are you running PiVPN at?

OVH Cloud VPS running Debian GNU/Linux 11 (Bullseye)

Output of uname -a
Linux vernvps 5.10.0-11-cloud-amd64 #1 SMP Debian 5.10.92-1 (2022-01-18) x86_64 GNU/Linux

Output of cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

Console output of      pivpn debug

debian@vernvps:~$ pivpn debug
::: Generating Debug Output
sudo: unable to resolve host vernvps: Name or service not known
::::		PiVPN debug		::::
=============================================
::::		Latest commit		::::
Branch: master
Commit: 0b5aab87221e83ed0e0d66a0279d04f1c59f8cb4
Author: 4s3ti
Date: Tue May 24 16:46:48 2022 +0200
Summary: pre-merge changelog update
=============================================
::::	   Installation settings    	::::
PLAT=Debian
OSCN=bullseye
USING_UFW=0
pivpnforceipv6route=1
IPv4dev=ens3
install_user=debian
install_home=/home/debian
VPN=wireguard
pivpnPORT=51820
pivpnDNS1=<IP_ADDRESS>
pivpnDNS2=<IP_ADDRESS>
pivpnHOST=REDACTED
INPUT_CHAIN_EDITED=0
FORWARD_CHAIN_EDITED=0
INPUT_CHAIN_EDITEDv6=
FORWARD_CHAIN_EDITEDv6=
pivpnPROTO=udp
pivpnMTU=1420
pivpnDEV=wg0
pivpnNET=<IP_ADDRESS>
subnetClass=24
pivpnenableipv6=0
ALLOWED_IPS="<IP_ADDRESS>/0, ::0/0"
UNATTUPG=1
INSTALLED_PACKAGES=(git dnsutils grepcidr bsdmainutils iptables-persistent wireguard-tools qrencode)
=============================================
::::  Server configuration shown below   ::::
[Interface]
PrivateKey = server_priv
Address = <IP_ADDRESS>/24
MTU = 1420
ListenPort = 51820
### begin aspire7-testing ###
[Peer]
PublicKey = aspire7-testing_pub
PresharedKey = aspire7-testing_psk
AllowedIPs = <IP_ADDRESS>/32
### end aspire7-testing ###
### begin tildeserver ###
[Peer]
PublicKey = tildeserver_pub
PresharedKey = tildeserver_psk
AllowedIPs = <IP_ADDRESS>/32
### end tildeserver ###
=============================================
::::  Client configuration shown below   ::::
[Interface]
PrivateKey = aspire7-testing_priv
Address = <IP_ADDRESS>/24
DNS = <IP_ADDRESS>, <IP_ADDRESS>

[Peer]
PublicKey = server_pub
PresharedKey = aspire7-testing_psk
Endpoint = REDACTED:51820
AllowedIPs = <IP_ADDRESS>/0, ::0/0
=============================================
:::: 	Recursive list of files in	::::
::::	/etc/wireguard shown below	::::
/etc/wireguard:
configs
keys
wg0.conf

/etc/wireguard/configs:
aspire7-testing.conf
clients.txt
tildeserver.conf

/etc/wireguard/keys:
aspire7-testing_priv
aspire7-testing_psk
aspire7-testing_pub
server_priv
server_pub
tildeserver_priv
tildeserver_psk
tildeserver_pub
=============================================
::::		Self check		::::
:: [OK] IP forwarding is enabled
:: [OK] Iptables MASQUERADE rule set
:: [OK] WireGuard is running
:: [OK] WireGuard is enabled (it will automatically start on reboot)
:: [OK] WireGuard is listening on port 51820/udp
=============================================
:::: Having trouble connecting? Take a look at the FAQ:
:::: https://docs.pivpn.io/faq
=============================================
:::: WARNING: This script should have automatically masked sensitive       ::::
:::: information, however, still make sure that PrivateKey, PublicKey      ::::
:::: and PresharedKey are masked before reporting an issue. An example key ::::
:::: that you should NOT see in this log looks like this:                  ::::
:::: YIAoJVsdIeyvXfGGDDadHh6AxsMRymZTnnzZoAb9cxRe                          ::::
=============================================
::::		Debug complete		::::
:::
::: Debug output completed above.
::: Copy saved to /tmp/debug.log
:::
debian@vernvps:~$

Similar issue happening here as well. It failed during installation as DNS resolution started failing midway through installation.
Running Ubuntu 20.04
dig @<IP_ADDRESS> google.com

Looking in /var/log/kern.log I see the response is being dropped
Aug 30 07:14:59 fouad kernel: [403452.156544] [UFW BLOCK] IN=enp197s0 OUT= MAC=f0:2f:74:97:9e:b2:80:7f:f8:79:1d:e1:08:00 SRC=<IP_ADDRESS> DST=<PUBLIC_IP> LEN=83 TOS=0x00 PREC=0x00 TTL=60 ID=46336 DF PROTO=UDP SPT=53 DPT=40649 LEN=63

Is there a way to revert all changes made by the installer?

Is there a way to revert all changes made by the installer?

https://docs.pivpn.io/install/#uninstall


Bottom bar with error message is changing its height when tooltip appears

Environment
Atom: 1.0.19
atom-typescript: 7.0.0
OS: Mac OS 10.10.5
Bottom bar is changing its height when you have any errors in the file/project and you hover the source code. When tooltip appears the bottom bar is "jumping" (see image below).

I've inspected the bottom bar's view and it seems that when you hover the code, atom shows the loading icon (with github logo) for less than a second. The problem is that logo is higher than the line with error message, so bottom bar is "jumping".
The bottom bar is not jumping when you have no errors in the file/project, since there's no line with error message.
I've temporarily fixed it on my local machine by adding
line-height: 24px;

to the line with error message in mainPanelView.ts (line 81)
Thanks,
Sergey

Good fix. Thanks! :rose:  7.0.1


Code style migration from JSCS to ESLint

Sure, I guess we can just remove the section too since eslint is mentioned above already


[Fit and finish] Runtimes widget

Display runtime type (e.g. Kubeflow Pipelines or Apache Airflow). While this might not seem that useful until another runtime is supported we should incorporate the information into the design.
Change "Runtime history" label to "Pipeline run history" since there isn't really a runtime history, only a pipeline run history.

@ptitzler would addressing both of these issue in line fix be okay?
Eg. Runtime History -> Kubeflow Pipelines History
That would make what the link is clearer while also clarifying the instance type.

Similar to how we use Cloud Object Storage, can't we just say something like Kubeflow Pipelines UI with some sort of tooltip like See submitted pipelines and associated executions?

@ajbozarth yes, for the time being (only KFP supported) that should be sufficient. However once we support multiple back-ends a user would have to expand each node to find out what it driving pipeline execution. I don't think we can rely on users choosing a meaningful name to make that distinction.
@lresende tool tips are bad from an accessibility point of view. They also won't help should there ever be a need to filter runtimes by type. However, maybe  I'm too optimistic how many runtimes a typical installation would include, lol.


loss_heatmap reduction abnormal

hillo，I trained the pandaset dataset with your code。The training went well with transfusion_pillar_L and training log see
link：https://pan.baidu.com/s/1PFjYIedIrYY1qKw5SwUe1A?pwd=4cho
Extraction code：4cho
but bad with transfusion_pillar_LC and training log see
link：https://pan.baidu.com/s/10en2ppjXWTYlhE_i4wyXqw?pwd=zxbb
Extraction code：zxbb
The performance of the model transfusion_pillar_LC trained on transfusion_pillar_L basis is degraded。What I'm confused about is that loss_heatmap fall first and then rise and the last matched_ious should be the highest。the MAP of the final test on data is lower than transfusion_pillar_L。
there is no problem in data validation，what do you think may be the wrong place and Can you give me some advice？

Hi @study1994 Here are some suggestions:

Try removing GlobalRotScaleTrans and RandomFlip3D when you train the LC model. These augmentations will bring some mismatch for the image-guided query initialization (although does not hurt the final performance on nuScenes) and I have already commented it out for TransFusionLC in the last commit. You can find some detailed discussion here
Is your 2D backbone pre-trained on pandaset? If not you may first pre-train the 2D backbone on pandaset for detection or segmentation task so that the img feature can learn semantic on pandaset.


added giovannixxiiicosentino

@pietrodev07 lib/domains/it/edu/giovannixxiiicosentino: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


[WIP] Reflection based Nd4j/SameDiff api parity tests

Reflection based, similar to #6287, may expand to cover it (although I probably can't use reflection there, the method names don't always match the op).
Very rough atm, but it works.

Report

Self-contained in https://github.com/SkymindIO/deeplearning4j/blob/rn_api_parity_tests/nd4j/nd4j-backends/nd4j-tests/src/test/java/org/nd4j/api/TestINDArraySDVariableParity.java


net.minecraft.client.util.SmoothUtil name is bad

Besides not following the usual convention of these kinds of classes ending in *Helper rather than Util, the name also implies that it's a class of static utility methods, which isn't at all true. It's actually a class Mojang uses as a generic "smoother".
They instantiate it twice in the Mouse class and call its smooth method incrementally, with the current x or z and the change versus time to smooth out the values.
Relevant usage (found in Mouse.updateMouse, variable names added for clarity):
        double time = GlfwUtil.getTime();
        double timeDifference = time - this.lastMouseUpdateTime;
        this.lastMouseUpdateTime = time;
        if (!this.isCursorLocked() || !this.client.isWindowFocused()) {
            this.cursorDeltaX = 0.0;
            this.cursorDeltaY = 0.0;
            return;
        }
        double sentitivity = this.client.options.getMouseSensitivity().getValue() * (double)0.6f + (double)0.2f;
        double speed = Math.pow(sentitivity, 3) * 8.0;
        if (this.client.options.smoothCameraEnabled) {
            double i = this.cursorXSmoother.smooth(this.cursorDeltaX * speed, timeDifference * speed);
            double j = this.cursorYSmoother.smooth(this.cursorDeltaY * speed, timeDifference * speed);

Suggested name changes:
SmoothUtil -> TrailingInterpolator
SmoothUtil.smooth(double original, double smoother) -> TrailingInterpolator.step(double value, double timeDifference)
Suggested package change:
net.minecraft.util -> net.minecraft.util.math.noise

Definitely shouldn't be in util.math.noise if it's used for mouse input (which is not worldgen noise), I think a rename to Smoother would be fine?

Definitely shouldn't be in util.math.noise if it's used for mouse input (which is not worldgen noise), I think a rename to Smoother would be fine?

Smoother sounds good!
My reasoning for putting it in noise is it seems to be related to those kinds of noise-y operations. I.e. like smoothing the output of a noise function.
Nothing I could see to indicate the package is for world generation.

The package currently contains noise generation code, the smoother class isn't really related to those.


Generate a longer header guard if modules are used

          Thoughts on making the header guard longer? There may be multiple `Time` messages in different modules.

https://google.github.io/styleguide/cppguide.html#The__define_Guard
Before:
#ifndef _Time_H_
#define _Time_H_

#ifdef __cplusplus
extern "C"
{
#endif

#include <stdint.h>
#include <stdbool.h>

typedef struct builtin_interfaces_msg_Time

After:
#ifndef _BuiltinInterfacesMsgTime_H_
#define _BuiltinInterfacesMsgTime_H_

#ifdef __cplusplus
extern "C"
{
#endif

#include <stdint.h>
#include <stdbool.h>

typedef struct builtin_interfaces_msg_Time

Originally posted by @Ryanf55 in https://github.com/eProsima/Micro-XRCE-DDS-Gen/issues/48#issuecomment-1452187908
Please assign this to me.

generated C source file doesn't include module path.
Here are my steps.

clone rcl_interfaces (containing builtin_interfaces)# contain builtin_msgs/msg/Time.msg
git clone https://github.com/ros2/rcl_interfaces
# contain std_msgs/msg/Header.msg
git clone https://github.com/ros2/common_interfaces
# Header.msg rely on builtin_msgs/msg/Time.msg so merge them into same folder
mkdir ros2_interfaces
cp -r rcl_interfaces/* ros2_interfaces/
cp -r common_interfaces/* ros2_interfaces/

translate Time.msg and Header.msg into .idl file at where their .msg file stay with ros2_idl tools.
use Gen tool to translate into C code.
Have tried./script/microxrceddsgen -I rcl_interfaces -example rcl_interfaces/builtin_interfaces/msgs/Header.idl

and./script/microxrceddsgen -I rcl_interfaces -example -d rcl_interfaces/std_msgs/msg rcl_interfaces/builtin_interfaces/msgs/Header.idl

Both of the commands generate header file with #include "builtin_msgs/msg/Time.h" as expected but Header.c is #include "Header.h" rather than #include "std_msgs/msg/Header.h".