[BTCP] Insufficient shielded balance for payment

Description
Running a BTCP pool for my miners. Found a block, and I get a payment error in the logs despite the payment really being done on the blockchain. There is also no entries logged in the payments table but the balance_changes table contains a "Balance reset after payment" entry.
Steps to Reproduce

Setup conf.json for BTCP with a z and non-z address as required.

Expected behavior
Payment going through OK and being logged in the DB.
Actual behavior
[2018-05-16 05:41:00.8658] [I] [PayoutManager] Processing payments for pool BTCP
[2018-05-16 05:41:00.8740] [I] [BTCP] [ZCash Payout Handler] Unlocked block 321435 worth 1.56261 BTCP
[2018-05-16 05:41:00.8740] [I] [PayoutManager] Processing payments for pool BTCP, block 321435
[2018-05-16 05:41:00.8767] [I] [BTCP] Adding 0.01547 BTCP to balance of b1CyV5r84i1YibEmNiezfUhpngsihV28DVy
[2018-05-16 05:41:00.9008] [I] [PPLNS Payment] Fetching page 0 of shares for pool BTCP, block 321435
[2018-05-16 05:41:01.2377] [I] [PPLNS Payment] Fetching page 1 of shares for pool BTCP, block 321435
[2018-05-16 05:41:01.5508] [I] [PPLNS Payment] Balance-calculation for pool BTCP, block 321435 completed with accumulated score 0.4067 (20.3%)
[2018-05-16 05:41:01.5523] [I] [PPLNS Payment] Adding 0.31459 BTCP to balance of b1CyV5r84i1YibEmNiezfUhpngsihV28DVy for 12.3 M (12295760) shares for block 321435
[2018-05-16 05:41:01.5642] [I] [PPLNS Payment] 12.3 M (12295760) shares contributed to a total payout of 0.31459 BTCP (20.33% of block reward) to 1 addresses
[2018-05-16 05:41:01.5914] [I] [BTCP] [ZCash Payout Handler] Shielding ZCash Coinbase funds
[2018-05-16 05:41:01.6039] [I] [BTCP] [ZCash Payout Handler] z_shieldcoinbase operation id: opid-e70c9340-a100-4c50-86c7-e8e8fda15c36
[2018-05-16 05:41:01.6087] [I] [BTCP] [ZCash Payout Handler] Waiting for shielding operation completion: opid-e70c9340-a100-4c50-86c7-e8e8fda15c36
[2018-05-16 05:41:02.8824] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:06.6087] [I] [BTCP] [Pool] Broadcasting job
[2018-05-16 05:41:07.1223] [I] [BTCP] [Pool] [0HLDORU6H6EIF] Share accepted: D=397
[2018-05-16 05:41:07.1223] [I] [BTCP] [Pool] [0HLDORU6H6EIF] VarDiff update to 398
[2018-05-16 05:41:11.6259] [I] [BTCP] [ZCash Payout Handler] Waiting for shielding operation completion: opid-e70c9340-a100-4c50-86c7-e8e8fda15c36
[2018-05-16 05:41:12.4382] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:14.9796] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:17.4005] [I] [BTCP] [Pool] Broadcasting job
[2018-05-16 05:41:21.6383] [I] [BTCP] [ZCash Payout Handler] Waiting for shielding operation completion: opid-e70c9340-a100-4c50-86c7-e8e8fda15c36
[2018-05-16 05:41:25.6914] [I] [BTCP] [Pool] [0HLDORU6H6EIF] Share accepted: D=398
[2018-05-16 05:41:26.6598] [I] [BTCP] [Pool] Broadcasting job
[2018-05-16 05:41:27.3614] [I] [StatsRecorder] Updating hashrates for pool BTCP
[2018-05-16 05:41:27.5282] [I] [BTCP] [Pool] [0HLDORU6H6EIF] Share accepted: D=398
[2018-05-16 05:41:28.1418] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:31.6401] [I] [BTCP] [ZCash Payout Handler] Waiting for shielding operation completion: opid-e70c9340-a100-4c50-86c7-e8e8fda15c36
[2018-05-16 05:41:34.7247] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:35.9271] [I] [BTCP] [Pool] [0HLDORU6H6EIF] Share accepted: D=398
[2018-05-16 05:41:37.0181] [I] [BTCP] [Pool] Broadcasting job
[2018-05-16 05:41:41.6424] [I] [BTCP] [ZCash Payout Handler] Waiting for shielding operation completion: opid-e70c9340-a100-4c50-86c7-e8e8fda15c36
[2018-05-16 05:41:46.6021] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:46.6550] [I] [BTCP] [Pool] Broadcasting job
[2018-05-16 05:41:50.8477] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:51.4839] [I] [BTCP] [Pool] [0HLDORU6H6EIE] Share accepted: D=96
[2018-05-16 05:41:51.6441] [I] [BTCP] [ZCash Payout Handler] Waiting for shielding operation completion: opid-e70c9340-a100-4c50-86c7-e8e8fda15c36
[2018-05-16 05:41:56.7066] [I] [BTCP] [Pool] Broadcasting job
[2018-05-16 05:42:01.6533] [I] [BTCP] [ZCash Payout Handler] z_shieldcoinbase successful
[2018-05-16 05:42:01.6561] [I] [BTCP] [ZCash Payout Handler] Insufficient shielded balance for payment of 0.33006 BTCP

Reproduces how often
Just found 1 block so far.
Version
Release 24
Additional Information
My wallet says I have the remaining funds, so not sure why it's complaining about insufficient shielded balance.
[btcp@btcp ~]$ btcp-cli z_listaddresses
[
  "zkJJzUnpXDkyfgR2aGGMU1Fjb6W6UmQ4SHGArT3Bwnd9xewcqv62sAN7zW5xsjKdvcHKZ75UqBP2iphkV2otAUPvt2iFADG"
]

[btcp@btcp ~]$ btcp-cli z_getbalance "zkJJzUnpXDkyfgR2aGGMU1Fjb6W6UmQ4SHGArT3Bwnd9xewcqv62sAN7zW5xsjKdvcHKZ75UqBP2iphkV2otAUPvt2iFADG"
1.23235460

I also don't have any accounts with names in my non-z addresses:
[btcp@btcp ~]$ btcp-cli listaccounts
{
  "": 0.00000000
}

But I noticed the non-z address changed by itself. (I'm thinking maybe like bitcoin it gives a new address after every transaction) But not sure if relevent to this problem.
[btcp@btcp ~]$ btcp-cli getaccountaddress ""
b1EYBP7jNi3UyY2jAm1RYEibdUJrHUuWo6c

^^ not the non-z address in my json.conf

Fixed a long time ago. Closing.


where to find libDetXR?

I was trying to run this, but couldn't find the package for libDetXR.  Where do I get it?

Did you manage to solve this problem?

No, I haven't done anything since submitting this issue last year.  It looks like there hasn't been any activity on the project either.  It looks promising based on the screen shots.

Thanks.
I am now using
HdfView:
https://support.hdfgroup.org/products/java/release/download.html
Seems to work; downloadable binary.
cheers,
Stephen

On Sep 18, 2016, at 10:42 AM, John Readey<EMAIL_ADDRESS>wrote:
No, I haven't done anything since submitting this issue last year. It looks like there hasn't been any activity on the project either. It looks promising based on the screen shots.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@maheshpaliwal Do you, by any chance have the libDetXR file?


Make boot/vmrunner scripts use the python print function instead of the print statement

This addresses #1218.

Can one of the admins verify this patch with one of the following commands:

"jenkins test please" for a one time test run
"ok to test" to accept latest and future commits on this pull request for testing
"add to whitelist" to add the author to the whitelist


fix: increase cache for Inter font file [sc-12301]

Affected Components

[ ] Content & Marketing
[ ] Test
[ ] Docs
[ ] Learn
[x] Other

Pre-Requisites

[x] Code is linted (npm run lint)

Notes for the Reviewer

~Moves Inter.var.woff2 off of the self-hosting setup to requiring it from Google Fonts. This is due to bandwidth concerns, as that Inter font file was our #1 bandwidth consumer from Vercel (which charges an extreme amount for bandwidth).~

Changed to increase cache headers for the self-hosted Inter font file.

Last 14 Days:

See discussion with Daniel in Slack: https://checklyhq.slack.com/archives/C02CES5S4LS/p1666860610470859
New Dependency Submission

What's the lighthouse impact of this change? Did you test that? - Hannes

Hannes Lenke
Lindenpromenade 6
15344 Strausberg
*
*
EMail: @.***
Phone: (+49)<PHONE_NUMBER>
On Thu, Oct 27 2022 at 6:44 PM, Nahuel Alejandro Ramos < @.*** > wrote:

@.**** approved this pull request.
LGTM.
—
Reply to this email directly, view it on GitHub (
https://github.com/checkly/checklyhq.com/pull/562#pullrequestreview-1158659178
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAK5JTQRTAMMANKQZH3JS4TWFKWPPANCNFSM6AAAAAARQHIYHU
).
You are receiving this because you are subscribed to this thread. Message
ID: <checkly/checklyhq. com/pull/562/review/1158659178 @ github. com>

@ndom91 Also these caching settings for the font don't look alright to me. If we're not caching the font on the client it's not surprising, that it's driving so much bandwidth.
Generally speaking, self-hosting fonts is the way to go these days. 🙈

@ndom91 Also these caching settings for the font don't look alright to me. If we're not caching the font on the client it's not surprising, that it's driving so much bandwidth.
Generally speaking, self-hosting fonts is the way to go these days. see_no_evil

Yeah so it seems to be being built as if it were a code asset, i.e. like a css file, and appended a suffix for cache-busting, etc.
I'm not familiar enough with Hugo to say off the top of my head where this is coming from exactly, but I'm sure we can change it to tell it this is a static asset that never changes and cache client-side for as long as possible.
That seems to be a good middle-ground here.
Good eye!

@stefanjudis the Inter font file now gets loaded without a hash appended to the filename and with proper cache headers :partying_face:

I dropped the hashes because the hash would be different every deploy, so even if we set the cache header to cache it client-side for 1 month, for example, then as soon as we make a new deploy the filename would be different and it'll need to be freshly downloaded again. Right?

then as soon as we make a new deploy the filename would be different and it'll need to be freshly downloaded again

🤔  Aren't webpack file hashes file content dependent? So that the same file always has the same hash?
Anyways. Case closed. :D Thanks for handling this Nico!


How to send USDT from a address to other address using private key ?

How to send Tether USDT from a address to other address using private key ?
I using C# to dev tool send/receive  Tether USDT.
I just installed Omnicore to my server and synced all data. I use C# code connect to omnicore RPC API
Anybody can help me step by step how to create a transaction send Usdt from any address (with private key) to other address
Thanks a lot

the language you use isn't a matter, if you can connect to omnicore rpc then you can invoke the omni_send method.
At first you can test in testnet, please refer my issue to config your test environment https://github.com/OmniLayer/omnicore/issues/807


nominate windsonsea to the global reviewers

What type of PR is this?
/kind documentation

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

/cc @windsonsea @RainbowMango

/lgtm
@windsonsea has made a significant contribution, impressive!

/approve
/hold
for @windsonsea confirm. (Feel free to unhold )

wow, thanks, I would like to help you to review more about docs and text. 😄

/hold cancel


Get full description of what is part of a component given as argument

Be able to get full description of what is part of a component given as argument.
$ ocd describe <name>
Component ABC with source in https://git and scaled to 2 instances, exposed via http://blabla and connected to component DB1, and component DB2
Application context
Parameters were …
Env vars ...

The very basic information a component should provide is:

name
where's source code for the component
number of active instances
connections (routes, services consumed)
configuration

Subtask of #155
Dependency on #196

Describe is implemented.
It is showing name, component source, and linked storage. But some of the information is still missing.
There is no information about active instances, configuration, and connection to other components

Closing this, as the basic functionality is implemented.
We don't have support for scaling, or configuration right now, those informations will be added once we implement it.


Look for startxref further in from end if not found #828

PDFs provided in title issue have ~10k trailing characters before which comes a normal xref table.
(updated @arturadib): Link to issue #828

I'm fixing the ones in issue #828

Can we add one of the PDFs in #828 for testing?

Yes go ahead - I am not taking them down so feel free to test with them!

@zwadia, awesome. Can you please add a new entry to the https://github.com/mozilla/pdf.js/blob/master/test/test_manifest.json file?
Once that's done we can run the test bot and if someone else reviewed this, it can land :)

@jviereck thanks! added the two references and sent pull request.
I did not use the "link" attribute... not sure if that is needed.
You'll also need to add the two PDFs to the pdfs directory... here they are:
http://www.imageworkdownload.com/ccittg4compression1bpp.pdf
http://www.imageworkdownload.com/jpgcompression24bpp.pdf

@pdfjsbot test

Oh sorry, I thought the new PDF references are included in this pull request :/

I rrremoved them because of discussion in #872.

@fixplz, as you suggested, could you construct bad PDF similar to what we have in #828 and include them as a test?


[ERROR] The "path" configuration key was renamed. Please use "sast" instead.

I got two ERROR messages but the result shows success, do I need to worry about the ERROR?
[ERROR] The "path" configuration key was renamed. Please use "sast" instead.
[INFO] Official Documentation: https://docs.fluidattacks.com/machine/scanner/standalone/
[INFO] Namespace: OWASP
[ERROR] Computing commit hash:
[INFO] info HEAD is now at:<PHONE_NUMBER>000000000000000000000000000000
[INFO] Startup work dir is: /usr/scan
[INFO] Moving work dir to: /usr/scan
[INFO] An output file has been written: /usr/scan/Fluid-Attacks-Results.csv
[INFO] Summary: No vulnerabilities were found in your targets.
───────────────────────────────── 🏄 Success! ──────────────────────────────────

Here is the official documentation of our tool: https://docs.fluidattacks.com/machine/scanner/standalone
If you are trying to achieve Tier 2 CASA compliance, I recommend reading this section: https://docs.fluidattacks.com/machine/scanner/standalone/casa


Error while running teletraan server

Error: Could not find or load main class com.pinterest.teletraan.TeletraanService

Do you have more context how to reproduce this?

Compile the Teletraan server code:
$ cd $TELETRAAN_ROOT/deploy-service
$ mvn clean package -DskipTests

./teletraanservice/bin/run.sh start
Starting Teletraan server...
Teletraan server started.
:~/teletraan/deploy-service$ Error: Could not find or load main class com.pinterest.teletraan.TeletraanService

The server starts and then encounters this error

I could not reproduce with latest version. Are you sure the mvn build command finished successfully?

[INFO] teletraan .......................................... SUCCESS [  0.355 s]
[INFO] common ............................................. FAILURE [  2.799 s]
[INFO] teletraan-service .................................. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 3.327 s
[INFO] Finished at: 2016-11-01T13:09:13-04:00
[INFO] Final Memory: 10M/25M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project common: Fatal error compiling: invalid target release: 1.8 -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :common

You would need to install and use java8.

I have installed java 8:
$ java -version
openjdk version "1.8.0_91"

can you copy and paste the result of the following command:
mvn -version

Apache Maven 3.3.9

can you also run "echo $JAVA_HOME", and "$JAVA_HOME/bin/java -version"?

/usr/bin/java

version -- 1.8.0_91

You can close this issue, the problem was with setup of environment variables.


Remove ngBootsrap search-box component

Search-box component uses ngbTypeahead from Ng-bootstrap to open the dropdown when typing. This ticket should:

Remove the use of ngbTypeahead
Implement custom logic and styling mimicking ngbTypeahead
Not all the feature from the ng-bootstrap components are needed but the Spartacus behaviour should be the same

Don't forget to adjust the unit test and remove the BootstrapModule import from this component's module.

Based on discussions with UI team, we're changing searchbox configuration in the backend:
PR: https://stash.hybris.com/projects/C3PO/repos/spacceleratorsamplesaddon/pull-requests/49/overview

QA:

bootstrap should no longer be used in the html
the input should be selected when clicking anywhere in the searchbox
a 50% black overlay should appear over the page when the search box has any results (products, suggestions or messages)
clicking on the overlay (in case the result panel is shown) should not open any of the under laying links
the result panel should be hidden when the input is left
the result panel should appear again with the former results when the input is selected again
the input value should be cleared when the reset button is used
the result panel should be cleared and hidden when the reset button is clicked
the result panel should show a message when no results are found
the input value text part of the suggestions should be in plain font type, while the rest should be bold
when there are no suggestions returned, but there are products being found, the input value should appear as a suggestion (allowing to click on it)
clicking on a suggestion should open the PLP with the search results
clicking on a product should open the PDP
hitting enter when the input is focussed, should open the PLP with the search query
hitting escape when the input is focussed, should blur the input (and hide the result panel with that)
selecting a suggestion should not replace the input value
the cms searchbox configuration should be taken into account when using the search box .

show / hide products / suggestions
search after x-chars are typed
display x-number of products
display x-number of suggestions

(default Spartacus backend config) the searchbox behaviour should start searching after the first character is typed
(default Spartacus backend config) the searchbox behaviour should display max 5 suggestions
(default Spartacus backend config) the searchbox behaviour should display max 5 products (in desktop)

Mobile:

clicking on the search icon should open the input field
clicking on the search icon the 2nd time should hide the input field
hitting escape when the input is shown, should blur the input (and hide the result panel with that)
products should not be shown in the result, regardless of backend configuration

QA:

bootstrap should no longer be used in the html
:heavy_check_mark: verified
the input should be selected when clicking anywhere in the searchbox
a 50% black overlay should appear over the page when the search box has any results (products, suggestions or messages)
clicking on the overlay (in case the result panel is shown) should not open any of the under laying links
the result panel should be hidden when the input is left
the result panel should appear again with the former results when the input is selected again
the input value should be cleared when the reset button is used
the result panel should be cleared and hidden when the reset button is clicked
the result panel should show a message when no results are found
the input value text part of the suggestions should be in plain font type, while the rest should be bold
when there are no suggestions returned, but there are products being found, the input value should appear as a suggestion (allowing to click on it)
clicking on a suggestion should open the PLP with the search results
clicking on a product should open the PDP
hitting enter when the input is focussed, should open the PLP with the search query
hitting escape when the input is focussed, should blur the input (and hide the result panel with that)
selecting a suggestion should not replace the input value
the cms searchbox configuration should be taken into account when using the search box .

show / hide products / suggestions
search after x-chars are typed
display x-number of products
display x-number of suggestions

(default Spartacus backend config) the searchbox behaviour should start searching after the first character is typed
(default Spartacus backend config) the searchbox behaviour should display max 5 suggestions
(default Spartacus backend config) the searchbox behaviour should display max 5 products (in desktop)

Mobile:

clicking on the search icon should open the input field
clicking on the search icon the 2nd time should hide the input field
hitting escape when the input is shown, should blur the input (and hide the result panel with that)
products should not be shown in the result, regardless of backend configuration

qa steps are doc input, moving to done


Create a database initialization/server management tool

This tool should:
Given a database ip and admin login/pass:

Create segsdb user
create segs databases ( according to current sql files)
create database tables ( also using provided sql files)

Given a server address:

Set server IP's and ports correctly
If database info is available, put it into server configuration file.

Things to consider:

Database schema updates.

An update... dbtool covers all of this except, port and IP setting, and Schema Updates.

I'm going to close this so we can follow development progress in #285


[WIP] Eliminate annoying frontend race conditions involving i18next

Get translations.json from home.py
Add translations to i18next resources in translations.js

#9087

This can be closed!!


[WIP] Only log parsed config on rank 0

Context
As titled, for distributed recipes, config.parse logs the config on all ranks. Only do it on rank zero to not pollute logs.
Test plan

....

No they are not, I mixed up my branches and need to update

covered by #707


Create Method Class

Add functions to set and get Name and Type of Method.  Pending review and approval.


11/12 hide the care covered tooltip on mobile ui

lgtm, flex always -> hidden on small viewports, flex on med


Could Inline Highlighting use the same Theme which select in the setting?

Could Inline Highlighting use the same Theme which select in the setting?
The effect is very abrupt

They are using the same theme.

However if you select a custom theme in the plugin settings, the background color of that theme is applied to code blocks and not inline code blocks.

So the custom will not effect on the inline code block, it just mean "different theme"


Doesn't work on the MUI commonjs modules

I'm currently investigating an issue on @mui/material+styled-components+next (https://github.com/mui-org/material-ui/issues/29742). It turns that in the Next.js server bundle, the styled calls are not correctly being identified by babel-plugin-styled-components. The server bundle uses the commonjs version of @mui/material. This PR adds a failing test case for that format.
edit: 🤔  The new test doesn't seem to run in CI for some reason

@Janpot , any update on this?

Hello, is there a plan how to fix this?


Add WS2019 to list of supported OS for Accelerated Networking

@smflorentino : Thanks for your contribution! The author, @gsilva5, has been notified to review your proposed change.

@gsilva5 can we merge this?

@gsilva5 - can you take a look? thanks.

@gsilva5 can you merge?

The requested updates are currently in master; we'll close the pull request.


Add --snapshots-from  to pack command

Please add --snapshots-from in pack command so that one can select a subset of snapshots that are packed.

Sounds good to me. This is the code that would need updating. Someone may implement this over the coming months, if anyone is interested in taking a crack at it please shout and/or take assignment.
https://github.com/elfshaker/elfshaker/blob/1755d4fc69d6af519f562f7cdc9a822443c3859a/src/bin/elfshaker/pack.rs#L64-L66

Created https://github.com/elfshaker/elfshaker/pull/102 to address this.


Is it possible to show the state code on the map?

I've done something like that before and if @gabidavila thinks that would be a good addition to the project I might provide a PR with that.

@daviscabral Do you have any example where you did that? I am trying to do this right now

hi @mattspahr - the project is not online anymore - I'll check this repo code to see if I can remember how I did it and let you know.


Review comments: Release scripts

9th, June 2021.

[x] 1. Generate all the RCs - all the RCs

[x] 2. Add more verbose output.

$CMD | tee "$LOG_OUTPUT"

[x] 3. SVN staging dry-run

with DRY_RUN option, use a temporary directory.

[ ] 4. Generate sha512

[x] 5. Resolve /bin/sh vs /bin/bash

svn upload log

Possible modifications:

mvn -Dmaven.repo.local=${tmp_repo} -P'distribution' deploy -DskiptTests \
    -DaltDeploymentRepository=altDepRepo::default::file:${tmp_repo} \
    -Daether.checksums.algorithms=SHA-512

Error reading assemblies: Error locating assembly descriptor: src/assembly/source.xml
[ERROR]
[ERROR] [1] [INFO] Searching for file location: /home/ubuntu/Repo/systemds/src/assembly/source.xml
[ERROR]
[ERROR] [2] [INFO] File: /home/ubuntu/Repo/systemds/src/assembly/source.xml does not exist.
[ERROR]
[ERROR] [3] [INFO] File: /home/ubuntu/Repo/systemds/src/assembly/source.xml does not exist.

Further review comments: 11th June 2021

Generate sha-512
Fix the next version
Do not delete the artifacts (you can push them to /target)


changes needed for openmind/slurm

In addition the changes I made before Friday, I made some changes we discussed:

I've moved the data to /om/scratch (right now /om2/scratch/Sat/djarecka/bootstrap/inputs_data/HBN_BIDS)
removed the TMPDIR setting, just using default slurm tmpdirs

I also made some changes that were irrelevant right now in order to bring the code closer to the current version of the code from the Penn repo. However, I decided to leave the fmriprep arguments as it was in the Steve&Jeff script.
I've just started a longer run, logs can be found: /om2/user/djarecka/bootstrap/fmriprep_dec15f/analysis/logs/array_22480069_*
fyi. - I've tried to change the image to 21.0.0, but I'm getting some errors, will try to check more tomorrow (you can see /om2/user/djarecka/bootstrap/fmriprep_dec15e/analysis/logs/array_22477137_0.err)

please ping Steve, wasn't sure what is his github login

@smeisler

If we want to PR this back to their repo at some point or for feedback, may be good to make changes in the mit_slurm folder

Hi, what image was being used before switching to 21.0.0? I see that --output-layout legacy is not specified, meaning that outputs will be organized differently than versions 20.2.6 and below.

cmd="singularity run -e -B /om2 -B ${scratch}:/workdir -B ${bids_dir}:/input:ro -B ${outdir}:/out ${SING_IMG} /input /out participant --participant_label ${subject} -w /workdir --cifti-output 91k --mem_mb 40000 --output-spaces MNI152NLin6Asym:res-2 anat --fs-subjects-dir /out/freesurfer-${session}/ --bids-filter-file bids-filter-${session}.json --bids-database-dir ${myscratch}/mct/derivatives/bids_db/ --nprocs 16 --omp-nthreads 8"

i would suggest adding the mem_mb, nprocs and omp-nthreads flags with the appropriate number of slurm cores into the job script. it will speed things up significantly for an actual fmriprep run.

If we want to PR this back to their repo at some point or for feedback, may be good to make changes in the mit_slurm folder

I wasn't sure if that was the plan. Made this way for now, so it's easier to see differences. But we can change

cmd="singularity run -e -B /om2 -B ${scratch}:/workdir -B ${bids_dir}:/input:ro -B ${outdir}:/out ${SING_IMG} /input /out participant --participant_label ${subject} -w /workdir --cifti-output 91k --mem_mb 40000 --output-spaces MNI152NLin6Asym:res-2 anat --fs-subjects-dir /out/freesurfer-${session}/ --bids-filter-file bids-filter-${session}.json --bids-database-dir ${myscratch}/mct/derivatives/bids_db/ --nprocs 16 --omp-nthreads 8"

i would suggest adding the mem_mb, nprocs and omp-nthreads flags with the appropriate number of slurm cores into the job script. it will speed things up significantly for an actual fmriprep run.

Yes, I moved all setting to sbatch script, see here: https://github.com/sensein/TheWay/blob/301644b6a9c3940c00fe417693286e59bda0f932/scripts/cubic/bootstrap-fmriprep.sh#L268

@satra - I don't have good answer for some of your questions. Some changes are here only because I haven't actually started from the current master. I'm guessing that main reason is that @jsmentch and @smeisler had an earlier version. I could probably take just the current master and start editing from there to be more consistent

I could probably take just the current master and start editing from there to be more consistent

i think that would be good. in effect this PR is against the current master, so you can see what the differences directly are and adjust in your local branch.

I could probably take just the current master and start editing from there to be more consistent

i think that would be good. in effect this PR is against the current master, so you can see what the differences directly are and adjust in your local branch.

yes, I've already did several adjustment yesterday, but just didn't want to change anything that could make a change before testing thr current version first

@djarecka - checking on the status of this.

@satra - I've made some updates, but the code for some reason got stalked multiple times. Will try again...
(I still have some changes to make, but need to figure out this first)

I've  fixed the script and made some changes in order to configure slurm and fmriprep from files, for now these are simple scripts that I added to the repo to have an example.
Example of calling would be:
./bootstrap_fmriprep.sh -i <path to inputs_data>HBN_BIDS/ -t <path to tempdir> -v 20.2.3 -p <path to script that should be run to configure env, see env_setup.sh> -f <path to fmriprep options, see fmriprep_opt.txt>  -s <path to slurm options, see slurm_opt.txt> -o <path to output dir; optional>

@djarecka - perhaps work with @burdinskid13 to setup her options. those fmriprep options require some paths to files and it would be good to ensure that those things are also captured in the datalad repo. could be before execution, but necessary for truly replicating.

After my last changes the script could be called:
./bootstrap_fmriprep.sh -i <path to inputs_data>HBN_BIDS/ -t <path to tempdir> -v 20.2.3 -e <path to script that should be run to configure env, see env_setup.sh> -f <path to fmriprep options, see fmriprep_opt.txt>  -w <path to slurm options, see slurm_opt.txt> -p <path to project root dir (code and output will be there); optional> -l <path to license file> -s <subset of subjects (e.g. sub-NDARZZ*), optional>

new option ... -c <directory with files that should be copy to code/, e.g. scripts/cubic/config_examples/bootstrap-fmriprep/copy_code>

@jsmentch - for now, it's just a quick fix, you should be fine now without providing COPY_DIR. The message about the subject list was actually not treated as an error, so all the issues were related to COPY_DIR.
Will try to figure out why it complains about subject lists, but it seems to work fine.

I'm able to run bootstrap_fmriprep.sh with one or multiple subjects now, but now getting an error when I try to run sbatch_array.sh. participant_job.sh is not getting the job_id variable

/om/scratch/Thu/jsmentch/fmriprep_test/project_dir2/analysis/code/participant_job.sh: line 16: $4: unbound variable

ok, let me double check, I didn't test the slurm script after my changes, perhaps I broke something new...

added a basic readme with usage instructions: https://github.com/djarecka/TheWay/pull/1

Summarizing here some features that would be good to have before finalizing this:

ability to merge different runs into one dataset or run across multiple job arrays [necessary to run a dataset as large as HBN on openmind? or another solution to max job array size=2000 problem?]
ability to run a specific list of subjects [useful if just want to run everyone with ASD for example]
The outputs may need to be reorganized to look more like BIDS? Each subject’s fmriprep output is in a different subfolder eg
project_dir/merge_ds/sub-XXXX_fmriprep-20.2.3/sub-XXXX  Or we just reorganize this after running for upload to s3?

(Dorota and I discussed some of these a little before, but anyone please chime in if thoughts)

@jsmentch - I've added an option to read a list of subject from a file, so you can have -s /path/to/subject_list.txt
and  /path/to/subject_list.txt can have:
sub-NDARZZ740MLM
sub-NDARZZ810LVF

Would this solve the issue for you?

@djarecka Thanks, I will give this a try! yes, that will fix the second bullet above


RouteRules about nuxt.config in nuxt3

Describe the feature
In nuxt.config.js in nuxt2 I use this
Document address https://www.nuxtjs.cn/faq/http-proxy
proxy: {     "/*.txt": "https://example.com",   },
But using this in nuxt3 does not take effect
 nitro: {     routeRules: {       '/**.txt': {         proxy: 'https://example.com/**.txt'       },     }   },
Can this feature be made compatible in the future?
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

Let's track in unjs/nitro#1717. This is a nitro question.


[frontend][pytorch] Add a new test case for torch aten::fill_ operator implementation

Add a new test case for torch aten::fill_ operator implementation.
This new test is testing a case of usage of a sequence of operators aten::div() and aten::fill_() that was causing a failure during the deployment of Mask R-CNN model (torchvision 0.13.0 version of the model).
Issue: #12844

You need to include my change in frontend/pytorch.py from https://github.com/apache/tvm/issues/12844 as well.

Thanks for the comment. I have included yours changes as well to the PR.


File detection time

Sometimes, for some reason, it gets stuck.
I waited a long time and didn't even start loading the game.

Can you share your log file when the freeze happens?

I started the game a few times, and I think it's this.log file.
[23:10:12] [main/INFO]: Loading tweak class name net.minecraftforge.fml.common.launcher.FMLTweaker
[23:10:12] [main/INFO]: Using primary tweak class name net.minecraftforge.fml.common.launcher.FMLTweaker
[23:10:12] [main/INFO]: Calling tweak class net.minecraftforge.fml.common.launcher.FMLTweaker
[23:10:12] [main/INFO]: Forge Mod Loader version <IP_ADDRESS>60 for Minecraft 1.12.2 loading
[23:10:12] [main/INFO]: Java is Java HotSpot(TM) 64-Bit Server VM, version 1.8.0_401, running on Windows 10:amd64:10.0, installed at C:\Program Files\Java\jre-1.8
[23:10:12] [main/INFO]: Searching E:\Minecraft\G-BFS\1122 - ����.minecraft\mods for mods
[23:10:12] [main/INFO]: Loading tweaker net.jan.moddirector.launchwrapper.ModDirectorTweaker from !mod-director-launchwrapper-1.8.2-all.jar
[23:10:12] [main/INFO]: Loading tweak class name net.minecraftforge.fml.common.launcher.FMLInjectionAndSortingTweaker
[23:10:12] [main/INFO]: Loading tweak class name net.jan.moddirector.launchwrapper.ModDirectorTweaker
[23:10:12] [main/WARN]: Successfully replaced CA certificates with updated ones!
[23:10:12] [main/INFO]: Mod director loaded!
[23:10:12] [main/INFO]: ModDirector bootstrapped from Launchwrapper!
[23:10:12] [main/INFO]: Loading tweak class name net.minecraftforge.fml.common.launcher.FMLDeobfTweaker
[23:10:12] [main/INFO]: Calling tweak class net.minecraftforge.fml.common.launcher.FMLInjectionAndSortingTweaker
[23:10:12] [main/INFO]: Calling tweak class net.minecraftforge.fml.common.launcher.FMLInjectionAndSortingTweaker
[23:10:12] [main/INFO]: Calling tweak class net.jan.moddirector.launchwrapper.ModDirectorTweaker
[23:10:12] [main/INFO]: Loading config config\mod-director.bundle.json
[23:10:12] [main/INFO]: Loading config config\mod-director\optifine.url.json
[23:10:13] [main/ERROR]: Unable to launch
java.lang.NullPointerException: null
at net.jan.moddirector.core.manage.select.InstallSelector.accept(InstallSelector.java:58) ~[!mod-director-launchwrapper-1.8.2-all.jar:1.8.2-SNAPSHOT]
at net.jan.moddirector.core.ModDirector.activate(ModDirector.java:146) ~[!mod-director-launchwrapper-1.8.2-all.jar:1.8.2-SNAPSHOT]
at net.jan.moddirector.launchwrapper.ModDirectorTweaker.injectIntoClassLoader(ModDirectorTweaker.java:48) ~[!mod-director-launchwrapper-1.8.2-all.jar:1.8.2-SNAPSHOT]
at net.minecraft.launchwrapper.Launch.launch(Launch.java:115) [launchwrapper-1.12.jar:?]
at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.12.jar:?]
[23:10:13] [main/INFO]: [java.lang.ThreadGroup:uncaughtException:-1]: net.minecraftforge.fml.relauncher.FMLSecurityManager$ExitTrappedException
[23:10:13] [main/INFO]: [java.lang.ThreadGroup:uncaughtException:-1]: 	at net.minecraftforge.fml.relauncher.FMLSecurityManager.checkPermission(FMLSecurityManager.java:49)
[23:10:13] [main/INFO]: [java.lang.ThreadGroup:uncaughtException:-1]: 	at java.lang.SecurityManager.checkExit(Unknown Source)
[23:10:13] [main/INFO]: [java.lang.ThreadGroup:uncaughtException:-1]: 	at java.lang.Runtime.exit(Unknown Source)
[23:10:13] [main/INFO]: [java.lang.ThreadGroup:uncaughtException:-1]: 	at java.lang.System.exit(Unknown Source)
[23:10:13] [main/INFO]: [java.lang.ThreadGroup:uncaughtException:-1]: 	at net.minecraft.launchwrapper.Launch.launch(Launch.java:138)
[23:10:13] [main/INFO]: [java.lang.ThreadGroup:uncaughtException:-1]: 	at net.minecraft.launchwrapper.Launch.main(Launch.java:28)

This is the same issue as mine, #21, is it a particular mod? What's you're mod list?

Can you share your log file when the freeze happens?

I don't know why the download crashed
https://gist.github.com/al-wei/b576f2dba8dac68377fa0079548ae9b2

Can you share your log file when the freeze happens?

I don't know why the download crashed
https://gist.github.com/al-wei/b576f2dba8dac68377fa0079548ae9b2

The download timeout occurred again.
https://gist.github.com/al-wei/d44ce6f0f13dd81377e8e00b1a0cffa0


WIP - Regenerate swagger code whenever files change

Description
It would be nice if ./bin/gen_server.sh was rerun every time a swagger yaml file changed.  This adds a new Makefile directive to do that.
Setup
Run make server_generate_and_run to start a process that will:

Reload the app when a .go file changes
Run make server_generate and restart the server whenever a YAML file in the swagger directory is modified.

This requires users to have entr installed in order to work. I haven't added it to the prereqs yet, but we can if this is generally considered useful.

@chrisgilmerproj Please take a look when you have a moment 😄

YES! This is exactly what I was hoping for.  And it works even better than I thought it would.  Can we merge this and share it with the team?


Combine webhook source code in beckn-sandbox source code

Description
Currently, the Beckn Sandbox and Webhook stand as distinct applications. However, since the Webhook predominantly comprises static functionalities we can consolidate the beckn-sandbox and webhook applications into a single, unified application. This consolidation aims to streamline development, maintenance, and hosting efforts while potentially improving efficiency and reducing resource utilization.

Feasibility Assessment: Conduct a thorough assessment to determine the technical feasibility and potential challenges associated with the proposed merger.
Impact Analysis: Evaluate the potential impact on existing functionalities, development workflows, and deployment processes.

Merge the webhook code into the beckn sandbox code.
Goals

[ ] Merge the webhook code into the beckn sandbox code.

Expected Outcome
When the webhook is merged into the sandbox and the deployment is done there should not be need of webhook and sandbox should work as expected.
Acceptance Criteria

[ ] Fully functional sandbox with webhook code merged.

Implementation Details
NA
Mockups / Wireframes
NA

[Please note that the below section of the ticket has to be in the format as mentioned as it is key to enabling proper listing of the project. Please only choose the options mentioned under the headings wherever applicable.]
Product Name
beckn-sandbox
Project Name
beckn-sandbox
Tech Skills Needed:
node js
Complexity
Medium
Category
Specifications
Sub Category

Migrated and updated the documentation in PR https://github.com/beckn/beckn-sandbox/pull/121


No longer builds against upstream Tencent/ncnn master

The ncnn::Pipeline::create() signature changed in Tencent/ncnn@1ea9de3, resulting in waifu2x-ncnn-vulkan failing to build. An easy solution to this is modifying the build instructions to specify a known working commit or tag.

Fixed in https://github.com/Nlzy/vapoursynth-waifu2x-ncnn-vulkan/commit/3b36a08f77a70d4d6243d86c1e29d5ab6fe68025
Thanks


Internet explorer support

Autocomplete dropdown doesn't appear in IE.
Body tag contains ng2-material-dropdown component, I think it's a style issue.
Can you help us to resolve that problem?
Tested in IE 10, 11.

@Gbuomprisco, could you provide workaround for us before issue will be resolved ?

@Gbuomprisco , Oh, I understand.
I'll try to solve this problem and create a pull request.
Thanks for quick answer.

@AndriiDidkivsky that would be awesome :)

Fixed in latest versions


Failed to setup

Version of the custom_component
Latest version
Describe the bug
After setting it up and entering the credentials I'm getting the following error:
Debug log
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 293, in async_setup
    result = await component.async_setup_entry(hass, self)  # type: ignore
  File "/config/custom_components/vienna_smartmeter/__init__.py", line 47, in async_setup_entry
    await client._refresh_token()
  File "/config/custom_components/vienna_smartmeter/api.py", line 76, in _refresh_token
    "code": await self._get_auth_code(),
  File "/config/custom_components/vienna_smartmeter/api.py", line 55, in _get_auth_code
    action = await self._get_login_action()
  File "/config/custom_components/vienna_smartmeter/api.py", line 50, in _get_login_action
    tree = html.fromstring(resp.content)
  File "/usr/local/lib/python3.9/site-packages/lxml/html/__init__.py", line 874, in fromstring
    is_full_html = _looks_like_full_html_unicode(html)
TypeError: expected string or bytes-like object

Hi!
Right now, the code in this repo is simply not working. I added an async module to the smartmeter repo some time ago and also updated the code for the hassio integration, but did not commit the changes so far, because I didn't have enough time to get the integration working properly.
I will create a PR with the changes I made and maybe someone can continue from there?

Hi, I wanted to start on the same project but stumbled on yours, I would have used a docker container to send the data over mqtt but HA, but a custom component is of course more elegant ;)

@platysma Please update the readme, as it is quite frustrating to follow the instructions only to get the same error message ;)


Emitted 'error' event on FSWatcher instance at FSWatcher._handleError

⚠ IMPORTANT ⚠ Please check the following list before proceeding. If you ignore this issue template, your issue will be directly closed.

[x] Read the docs.
[ ] Use Vite >=2.0. (1.x is no longer supported)
[ ] If the issue is related to 1.x -> 2.0 upgrade, read the Migration Guide first.

Describe the bug
A clear and concise description of what the bug is.
My react project is created by yarn create @vitejs/app vite-demo --template react-ts
When I start vite command and develop something and save files It always shows this error and stops vite runtime.
Reproduction
Please provide a link to a repo that can reproduce the problem you ran into.
A reproduction is required unless you are absolutely sure that the problem is obvious and the information you provided is enough for us to understand what the problem is. If a report has only vague description (e.g. just a generic error message) and has no reproduction, it will be closed immediately.
System Info

vite version: vite/2.0.0-beta.27
Operating System: Windows 10
Node version: both v15.5.0 and v14.5.0
Package manager (npm/yarn/pnpm) and version: both yarn 1.22.5 and npm 7.3.0

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.
Provide the error log here.
(node:15636) [DEP0148] DeprecationWarning: Use of deprecated folder mapping "./" in the "exports" field module resolution of the package at C:\Projects\vite-demo\node_modules\postcss\package.json.
Update this package.json to use a subpath pattern like "./*".
(Use node --trace-deprecation ... to show where the warning was created)
node:events:353
throw er; // Unhandled 'error' event
^

Error: EBUSY: resource busy or locked, lstat 'C:\DumpStack.log.tmp'
Emitted 'error' event on FSWatcher instance at:
at FSWatcher._handleError (C:\Projects\vite-demo\node_modules\vite\dist\node\chunks\dep-baca5bcc.js:11574:10)
at ReaddirpStream.NodeFsHandler._boundHandleError (C:\Projects\vite-demo\node_modules\vite\dist\node\chunks\dep-baca5bcc.js:10070:43)
at ReaddirpStream.emit (node:events:376:20)
at emitErrorNT (node:internal/streams/destroy:188:8)
at emitErrorCloseNT (node:internal/streams/destroy:153:3)
at processTicksAndRejections (node:internal/process/task_queues:80:21) {
errno: -4082,
code: 'EBUSY',
syscall: 'lstat',
path: 'C:\DumpStack.log.tmp'
}
error Command failed with exit code 1.

Follow the issue template requirements please.

Follow the issue template requirements please.

I've met exactly the same issue, so is it been resolved?

I've met exactly the same issue, so is it been resolved?

It's not because there was no reproduction.

It's not because there was no reproduction.


Add setupInitLayer() placeholder for Solaris

Commit d2bc5d62761c24866e4b0ff4099dd54264bd9325 (https://github.com/docker/docker/pull/26756) added
a setupInitLayer() for all platforms, but did not
add a placeholder for Solaris https://github.com/docker/docker/pull/26756#discussion_r79938294.
This adds the missing placeholder.
ping @amitkris @rhvgoyal ptal

@thaJeztah I am not sure if solaris requires an -init layer or not. Looks like that don't do anything in setup function. But it would be better if somebody who knows solaris better can confirm this.

@rhvgoyal I chatted with @amitkris on IRC, and this change looked good to him :smile:

@rhvgoyal a stub is sufficient for Solaris as of now. This can be implemented/addressed as a follow up to #23397 if required.

Ok, sounds good. If -init layer indeed is required, changing this is going to be really easy.
LGTM

LGTM


Easier debugging

This is related to #116. Right now we are using assert_is[OS] etc in tests. The output is meaningless as its the digest of the primitive.
Something like this would be much better and could be used with assert_eq:
trait Foo
  fun string(): String iso^
  fun eq(x: box->OS): Bool =>
    this is x
  fun ne(x: box->OS): Bool =>
    not eq(x)

primitive BSD is Foo
  fun string(): String iso^ =>
    "BSD".string()

primitive Linux is Foo
  fun string(): String iso ^ =>
    "Linux".string()

type OS is (BSD | Linux)

@Theodus I think we can close this, agree?

Yup


Add tests to github CI and remove circleci

Make CI change in https://github.com/o1-labs/proof-systems/pull/323 a dedicated PR.
@mrmr1993 plz take a look, thx!

Thanks!


ci : [TC-OO-1_1] Global attributes with server as DUT - write the default values to FeatureMap is disabled

SDK needs to add support for writing default values to FeatureMap
- label: "write the default values to optional global attribute: FeatureMap"
  disabled: true
  command: "writeAttribute"
  attribute: "Feature Map"
  arguments:
      value: 0
  response:
      error: 1

https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/Test_TC_OO_1_1.yaml

Closing it has been enabled a while ago.


Delete LICENSE

This can be closed. There was an account spamming github w/ PRs to delete licenses. Seems the account has been wiped.


[Skype for Business] Proactive messages not delivered if SFB window is closed

Bot Info
This issue applies to all SFB bots built on the Tangowork platform, but we'll provide the bot info for one of our test environments:

Bot handle: bobsheetz
App ID: 0046d486-6b93-49b0-9047-02c6dd535994
SDK Platform: Node.js
SDK Version: 3.11.0
Active Channels: Skype, Skype for Business, Slack
Deployment Environment: EC2 on AWS

Issue Description

We are implementing a Skype for Business chatbot at the head office of an 80,000-employee organization.
If a proactive message is sent while SFB is closed, or even if the current conversation window inside SFB is closed, it will never be delivered.
Our proactive message code works fine in Facebook Messenger, Skype, Slack, SMS, and Teams. It's only Skype for Business that has an issue.
Code Example
Our code looks like his:
// subscribeAddressHandler.js

const address = session.message.address;
addresses.push(address);

// sendProactiveHandler.js

const connector = new ChatConnector();
new UniversalBot(connector).beginDialog(address, '/default');

Addresses look like this:
{
  "id": "2",
  "channelId": "skypeforbusiness",
  "user": {
     "id"<EMAIL_ADDRESS>     "name": "Foo"
   },
   "conversation": {
      "isGroup": true,
      "id": "ZTFmMWYwM2Yjc2lwOmRlbW9AdGFuZ293b3JrLm9ubWljcm9zb2Z0LmNvbQ=="
   },
  "bot": {
     "id"<EMAIL_ADDRESS>     "name": "foobar"
   },
  "serviceUrl": "https://webpoolbl20r04.infra.lync.com/platformservice/tgt-3e269d45c99c53b8a53d91bd8610020f/botframework"
}

Reproduction Steps

Send a proactive message to Skype For Business while conversation window is closed, or while SFB is closed

Expected Behavior

We expect the message to to be visible for the user when they re-open Skype for Business, or when they reopen the conversation window with the bot.

Actual Results

The SFB user never receives the message.
On the bot side we see an error when the proactive message is sent while the conversation window is closed:
Error: Request to 'https://webpoolbl20r04.infra.lync.com/platformservice/tgt-3e269d45c99c53b8a53d91bd8610020f/botframework/v3/conversations/YjJmZTQ4ZTQjc2lwOmRlbW9AdGFuZ293b3JrLm9ubWljcm9zb2Z0LmNvbQ%3D%3D/activities' failed: [500] Internal Server Error
        at Request._callback (/botbuilder/node_modules/botbuilder/lib/bots/ChatConnector.js:512:46)
        at Request.self.callback (/botbuilder/node_modules/request/request.js:186:22)
        at emitTwo (events.js:106:13)
        at Request.emit (events.js:191:7)
        at Request.<anonymous> (/botbuilder/node_modules/request/request.js:1163:10)
        at emitOne (events.js:96:13)
        at Request.emit (events.js:188:7)
        at IncomingMessage.<anonymous> (/botbuilder/node_modules/request/request.js:1085:12)
        at IncomingMessage.g (events.js:292:16)
        at emitNone (events.js:91:20)
        at IncomingMessage.emit (events.js:185:7)
        at endReadableNT (_stream_readable.js:974:12)
        at _combinedTickCallback (internal/process/next_tick.js:74:11)
        at process._tickDomainCallback (internal/process/next_tick.js:122:9)

We've tried omitting the conversation part of the address, and this works fine in other messengers. However, in SFB, if we omit the conversation part of the address and send a proactive message to SFB:

on Mac OS, it opens a new conversation window (even there is an active one) and sends a cryptic "Meet Now" message.
on Windows, nothing happens

Because our proactive messages work fine on other channels, we believe this is a bug with botbuilder's integration with Skype for Business. But if it's not a bug, we'd be grateful for some advice on what we're doing wrong.

This seems like the expected behavior to me, but will ping the S4B team and see what they have to say.

@JasonSowers If you sent me an email, would you expect it to be delivered, even if I didn't have my email client open at the time? Of course. That's the situation we're talking about. Unless we are having an active conversation, SFB messages disappear as if they had never been sent.

Totally get what you are saying, just my guess that teams doesn't behave like that.

I'm seeing the same behavior, which is not expected, as other clients doesn't behave this way.

This is blocking us as well. The underlying service error seems to be the following:
{ Error:
   { Code: 'ServiceError',
     Message: 'A 480 (Temporarily Unavailable) response was received from
     the network and the operation failed. See the exception details for more information.'
  }
}

@intranetchris @wictorwilen @v46
All channels are independent of bot framework, even the internal channels like Skype, Skype for Business and MS Teams.  If one or more channels behave a specific way, this does not mean all channels behave like that.  All channels have their own independent features.  This is a question for Skype for Business as the behavior of channels themselves is well outside of the scope of the Bot Framework SDK.   In turn, this is well outside the scope of this forum.   If Skype for Business does not accept messages on the user's behalf while they are not logged in, there is nothing that can be done within the SDK to change that.
If a channel does not behave the way you would like, you need to get in contact with the channel and ask them if they can put your feature request on their roadmap.  The bot framework SDK just connects to the channels it does not enforce how they behave.


Added social cards.

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

Link to the original issue: #225

@timunie do i need to do anything else or is this done?

@AlexGipp Thank you! These cards are all awesome! So I will start her the vote for my 4 favorites:
Vote with 👍  for 1

with 😄 for 2

with 🎉 for 3

and with ❤ for 4

My favorites are 1 and 2

If you have any suggestions, feel free to let me know.

If you have any suggestions, feel free to let me know.

@AlexGipp I have a little one, the "Awesome icon packs for WPF and UWP in one library" could be maybe more generic, like "Awesome icon packs for your applications in one library", because one plan is to make a version for Avalonia too.

If you have any suggestions, feel free to let me know.

I don't know which tool you used for your cards, But if possible it would be nice if you also share the raw data with the community, e.g. as svg.
By the way: I am impressed by all of your cards in such a short time :-)

@timunie i used photoshop, i will make sure to add the psd files in the next push.

Hey guys, just added another card and the psd files. (also update the text @punker76 )

1 is missing a "." after MahApps, or was that intentional?

@batzen yeah i forgot that one, added it though

@AlexGipp your changes have been merged, thanks for your contribution 👍 Thank you for this nice work!

No problem, do you have any simple programming issues i could work on?


missing include

Environment
toml++ version and/or commit hash:
3.4.0
Compiler:
g++ 13.1.0
C++ standard mode:
C++17
Target arch:
x64
Library configuration overrides:
None
Relevant compilation flags:
None
Describe the bug
I installed the library with vcpkg and added the toolchain file with CMake, then tried to compile my program and I got an error from the compiler:
.../vcpkg_installed/x64-linux/include/toml++/impl/date_time.hpp:337:17: error: ‘optional’ does not name a type
I went to the file and added "#include " at the top. it worked and I could compile

This should not be the case if you are including the library 'normally'. 🤔 Can you show me the code that caased this, specifically, how were you #including toml++?

I used #include <toml++/toml.hpp> but I think maybe I'm suposed to just include  in the same file

No, you did the right thing. Looks like there's something amiss. I'll look into it, thanks 😃


update pre-comit

IMPORTANT: Please search among the Pull requests before creating one.
Description

How has this been tested?

Closes

Codecov Report

Merging #698 (8061bff) into main (540ad9a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #698   +/-   ##
=======================================
  Coverage   78.53%   78.53%
=======================================
  Files          33       33
  Lines        4710     4711    +1
  Branches      923      918    -5
=======================================
+ Hits         3699     3700    +1
  Misses        735      735
  Partials      276      276

Impacted Files
Coverage Δ

squidpy/_constants/_utils.py
95.00% <ø> (ø)

squidpy/pl/_color_utils.py
82.05% <ø> (ø)

squidpy/pl/_var_by_distance.py
88.05% <ø> (ø)

squidpy/read/_utils.py
75.00% <ø> (ø)

squidpy/_constants/_pkg_constants.py
93.84% <100.00%> (ø)

squidpy/datasets/_utils.py
79.31% <100.00%> (ø)

squidpy/gr/_build.py
87.34% <100.00%> (ø)

squidpy/gr/_ligrec.py
77.11% <100.00%> (ø)

squidpy/gr/_ppatterns.py
82.00% <100.00%> (ø)

squidpy/im/_feature_mixin.py
87.34% <100.00%> (ø)

... and 4 more


Allow SSID/password to be changed in web UI

At the moment they are hard coded which is a bit rubbish.

This is done.


Hyperlinks in job output undocumented

We support hyperlinks in job output using our underlying terminal-to-html renderer, however, this code is not documented and, presumably, left as an exercise for the reader to know we’re imitating iTerm 2 and co!
It’d be good to add this, perhaps alongside the existing images in job output doc.

Rad, thanks for making an issue about it @ticky! 🙌🏻I'll leave this open until it's added.

@harrietgrace I’d be happy to draft this up, do you think it should be its own page, or should the existing “Images in job output” be renamed, maybe to “Links and images in job output” (w/ redirect)?

I reckon the latter?

I have realised that our codes aren’t the same as iTerm’s - ours are in fact completely incompatible with iTerm’s! Opened this issue around that: buildkite/terminal-to-html#58


chore: move samples from python-docs-sample to here

Samples are being moved from an existing repo (python-docs-samples)

@busunkim96 Thank you for fixing the tests!


C# 3.0.3-stable signals not always triggering with exported binaries

Godot version:
3.0.3 Stable
OS/device including version:
Windows 10, Linux (Ubuntu 18) and OSX (macOS Sierra 10.12.6 - friend tested)
Issue description:
Problem is when you export a C# project, signals sometimes will not trigger. When I say sometimes I mean the debug version locally works flawlessly. On other pc's the debug has issue where my WeaponDischarged signal doesn't trigger. Release version on other pc's it'll work once or twice then fail to trigger the signals after trying to restart the game again.
I could've sworn this issue was fixed, but it seems like it's back
ERROR: emit_signal: Can't emit non-existing signal "WeaponDischarged".
    At: core/object.cpp:1139.

Steps to reproduce:

Create a game with signal usage.
Export game
Watch console spit out errors about signal

Minimal reproduction project:
SpaceShooter.zip
https://github.com/exts/godotsharp-shooter
Binaries: https://g4mr.itch.io/testing-godot
(Controls: WASD and SPACE, space to shoot)

CC @neikeq. Setting milestone 3.1 for now until we identify if it's reproducible in the master branch or a 3.0-only issue.

I could reproduce this using the last commit 4da621603eaf4b85dbaacb64a010ec2e62574ec5 from master using this example:
public class TestClass : Node
{
    [Signal]
    delegate void TestSignal(string message);

    public override void _Ready()
    {
        EmitSignal("TestSignal", "Hello World!");
    }
}

public class ReceiveSignal : Label
{
    public override void _Ready()
    {
        GetParent().Connect("TestSignal", this, "PrintMessage");
    }

    void PrintMessage(string message)
    {
        GD.Print(message);
        SetText(message);
    }
}

Although 7 out of 10 tries it will just "work", sometimes the signal will not connect giving me this error:

ERROR: In Object of type 'Node': Attempt to connect nonexistent signal 'TestSignal' to method 'Label.PrintMessage'
At: core\object.cpp:1404
ERROR: Can't emit non-existing signal "TestSignal".
At: core\object.cpp:1139

Hi everyone,
Can also confirm the bug using master from 5c5aafa
But from the OP's example project, I modified the GameScene script from
// connect the ship signal to a method inside our game scene, this would allow us to spawn bullets
// into the scene after we press spacebar. Allowing us to keep our code as separated as possible. Also
// allowing us to have bullets interact with other things like enemies in our case.
_ship = (Ship) GetNode("GameCanvas/Ship");
_ship.Connect("WeaponDischarged", this, "SpawnBullet");

to
// connect the ship signal to a method inside our game scene, this would allow us to spawn bullets
// into the scene after we press spacebar. Allowing us to keep our code as separated as possible. Also
// allowing us to have bullets interact with other things like enemies in our case.
_ship = (Ship) GetNode("GameCanvas/Ship");
_ship.AddUserSignal("WeaponDischarged");  // <<< add signal at setup
_ship.Connect("WeaponDischarged", this, "SpawnBullet");

and removed the signal declaration in Ship.cs
    public class Ship : Area2D
    {
        //[Signal]
        //public delegate void WeaponDischarged();

and it is working as it should, even exported in release.
I'm jaust having problems with the IsActionJustPressed in Ship.cs that seem to not be triggered, so I replaced it with
    Input.IsKeyPressed((int) KeyList.Space)

for the tests.

CC @paulloz

Thanks for the ping 😀, I don't have enough free time these days to check issues without being notified 😅.
I'll check this tomorrow.

Oh this is interesting @malbach. Updated my code, removed all signal annotations and just called AddUserSignal and my linux build works normally now O_o.
_ship = (Ship) GetNode("GameCanvas/Ship");
_ship.AddUserSignal("WeaponDischarged");
_ship.Connect("WeaponDischarged", this, "SpawnBullet");

var enemy = (Enemy) _enemyObject.Instance();
enemy.Position = new Vector2(_formations.XPosition, _formations.Positions[spawn]);
enemy.AddUserSignal("Destroyed");
enemy.Connect("Destroyed", this, "EnemyDestroyed");

Even emitting values to the signal works.
I uploaded a linux build here https://g4mr.itch.io/testing-godot called linux64-6202018.zip

FWIW, this is probably related to exports not being a tools build. There's probably some code not getting ran to handle delegate signals in those.

@paulloz I think I figured out the issue, but unsure how to fix it. void CSharpLanguage::reload_assemblies_if_needed method is tied to the tools ifdef which is the main code that updates the signals map which is then loaded from godot via void CSharpScript::get_script_signal_list right? If someone can figure out how to sort that then hypothetically it should work outside of the editor.
Maybe @neikeq could take a look I know you're very busy with personal life stuff. My C++ is pretty dated.

So I did test one last time just to confirm my findings.

I basically just updated the _get_signal to post a few ERR_PRINTS just to see me stepping into each if statement here  (no idea how to debug the editor/export templates) and when I built the editor off of 8bf5b502223d48ee1b7b88ab8613be7bacbb63d2 (since more recent commits were bugged) the editor showed me that's how the signals are being called.
Well in the debug export template only the instanced signal annotation was loaded via ScriptInstance *CSharpScript::instance_create(Object *p_this) which makes sense and the ship annotation not being loaded also make sense because it's only called in the reload assemblies if needed method so outside of the editor at runtime that code never gets executed. Hope that helps.
Here's a screenshot of the editor when I ran the test.

This can be closed @neikeq
I made some linux builds that work :) https://g4mr.itch.io/godot-export-test and my windows 10 build work as well with that PR change.

Can't be closed before the PR is merged :)


[Telegraf] Updated port for kubelet metrics collection

Port 10255, which is readonly and with no auth is disabled from
v1.11 and replaced by 10250, a secured one.

@huazhihao  -- I need you to sign our CLA before I can accept/merge this change.
https://www.influxdata.com/legal/cla/
Thanks for your interest and contribution!
Also, this will push the minimum version required to v1.11.  So, I may want to wait a little bit until we get some up-votes on this item.  Not sure how fast folks have moved to v1.11.

@timhallinflux  Sorry for the long silence as I almost forgot this commit. Thanks for your approval and information. The CLA has been signed. The commit has been rebased. So I think everything is good to go.


Test case for workaround for hparams loading issues

I added a test case to the workaround to https://github.com/Lightning-AI/lightning/issues/17889

I also incorporated the changes from the main branch, I wasn't aware of the nightly branch :)

Looks good, thank you!


Use cubic interpolation when upsampling NWPs to 5-minutely

Needs a buffer of 1 hour to start_hourly and end_hourly in get_nwp_example.  But also needs some fidly things fixing too:

When computing datetimes available for training, need to take this 1 hour buffer into consideration
In NWPDataLoader need to load data with an extra buffer.  Maybe this means dropping an hour from the start and end of all the contiguous segments, to leave that hour buffer for the NWPs.

perhaps with #135 this id no needed

Good catch!  I'll close this issue for now....


Installer docs

Windows was the one desperately needing an update, but the others also needed some care.

It looks pretty good to me! All the links work and the wording is direct and instructional enough to easily follow.


Blank Download Window

On linux the download window dialogue seems to be blank every time.

Resolve in Firefox 63


Issue with latest Vite

I used the latest vite and svelte project to use the medblocks ui library
The below is my package.json
  "name": "vite-project",
  "private": true,
  "version": "0.0.0",
  "type": "module",
  "scripts": {
    "dev": "vite",
    "build": "vite build",
    "preview": "vite preview",
    "check": "svelte-check --tsconfig ./tsconfig.json"
  },
  "devDependencies": {
    "@sveltejs/vite-plugin-svelte": "^2.0.0",
    "@tsconfig/svelte": "^3.0.0",
    "svelte": "^3.54.0",
    "svelte-check": "^2.10.0",
    "tslib": "^2.4.1",
    "typescript": "^4.9.3",
    "vite": "^4.0.0"
  },
  "dependencies": {
    "medblocks-ui": "^0.0.162"
  }
}

As per the docs, the below was added to the App.svelte.
<script lang="ts">
    import 'medblocks-ui'
    import 'medblocks-ui/dist/shoelace'
</script>

On importing a template related to blood pressure, it does not render correctly. The screenshot of the same is attached below. The issue seems to be with components like mb-quantity and mb-select.
<mb-form class="flex flex-col gap-3 p-5 shadow-lg rounded-lg border">
    <mb-context path="petetest/category" />
    <mb-context path="petetest/context/start_time" />
    <mb-context path="petetest/context/setting" />
    <mb-quantity
        default="mm[Hg]"
        path="petetest/blood_pressure/any_event:0/systolic"
        label="Systolic"
    >
        <mb-unit unit="mm[Hg]" label="mm[Hg]" />
    </mb-quantity>
    <mb-quantity
        default="mm[Hg]"
        path="petetest/blood_pressure/any_event:0/diastolic"
        label="Diastolic"
    >
        <mb-unit unit="mm[Hg]" label="mm[Hg]" />
    </mb-quantity>
    <mb-context path="petetest/blood_pressure/any_event:0/time" />

    <mb-context path="petetest/blood_pressure/subject" />
    <mb-context path="petetest/blood_pressure/language" />
    <mb-context path="petetest/blood_pressure/encoding" />
    <mb-context path="petetest/composer" />
    <mb-context path="petetest/language" />
    <mb-context path="petetest/territory" />

    <mb-submit>
        <sl-button type="neutral">Submit</sl-button>
    </mb-submit>
</mb-form>

@sidharthramesh I would appreciate your support here.

Are you using npm or another library like pnpm / yarn?

I am using yarn.
On debugging, I could find that in quantity.js file, the function ifDefined(this.label))is returning a function and this is what is displayed.

Thank you for the catch. Cc: @ATHULKNAIR
We noticed this same problem sometime ago, but it only happened when using yarn and not npm for some reason.
We will fix this soon. Would be glad to merge your PR if you’re interested @varghesethomase.

I can try @sidharthramesh . Is there someone who could assist me on this? In the mean time I also found issues with mb-select. Going through the code looks ok to me. But on rendering the html below,
mb-select
          path="covid.form.v1/clinical_background/covid_19_admission/hospital_admission_status"
          label="Hospital admission status"
        >
          <mb-option value="at0003" label="YES" />
          <mb-option value="at0004" label="NO" />
          <mb-option value="at0005" label="UNKNOWN" />
        </mb-select>
        <mb-date
          time
          path="covid.form.v1/clinical_background/covid_19_admission/date_first_admitted"
          label="Date first admitted"
        />

it displays  as it is in the screenshot

Thanks @sidharthramesh I can confirm that the package works as expected with npm instead of yarn. Since it might be still worth checking why the issue happens only with yarn, I'll keep this ticket open. I would like to see if you have the same feeling that it might be caused due to the difference in version of packages that yarn installs when compared to npm based on minor dependencies or so? Let me know your thoughts.

We usually use npm. And I have no idea why there are these issues when using yarn.
should I still keep this issue open? Maybe rename it to something like “issue with yarn”?


MQTT lags since 0.64.0

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.64.0
Python release (python3 --version):
3.6.3
Component/platform:
MQTT
Description of problem:
Since the update to 0.64 i noticed a lag from 0.3 to 1 seconds on my mqtt motion sensors/switches.
Expected:
Same response time like prior to 0.64
Problem-relevant configuration.yaml entries and steps to reproduce:
mqtt:
  broker: <IP_ADDRESS>
  port: 1883
  client_id: home-assistant-1

switch:
  - platform: mqtt
    name: "MCP23017 Test 01"
    state_topic: "home/mcp23017_1/cover_ez3_relay_man_auto"
    command_topic: "home/mcp23017_1/cover_ez3_relay_man_auto"
    qos: 0
    payload_on: "ON"
    payload_off: "OFF"
    retain: true

automation:
  - alias: mcp_test_01
    initial_state: True
    hide_entity: False
    trigger:
      - platform: mqtt
        topic: "home/mcp23017_1/cover_wz2_switch_down"
        payload: 'ON'
    action:
      service: switch.toggle
      entity_id: switch.mcp23017_test_01

Setup:
ubuntu 16.04, HA in python 3.6.3 venv, eclipse-mosquitto latest (1.4.12) in a docker container.
To do some measurement and get a timestamp in mosquitto_sub i installed moreutils which includes ts.

Reverting back to 0.63.3 fixes the issue.
Additional info:
0.64 Blog topic conversation wit @OttoWinter

Interesting... Could you please check something for me? I would like to know whether this delay is caused by the publishing or subscribing or even the automation aspect of Home Assistant.
So do you notice this when you manually toggle entities through the front-end? Does the trigger work when the publishing action: part is replaced with a non-MQTT action? Do the entity updates in the front-end happen quickly or is there this 1s delay? Is there something in your mosquitto logs (maybe about too many subscriptions? Does this disappear when you only have a single MQTT device in your config (thus not creating many subscriptions)? Thanks!

Hi @OttoWinter, thanks for looking on this.

do you notice this when you manually toggle entities through the front-end?

No, when i click the switch there's no lag.

Does the trigger work when the publishing action: part is replaced with a non-MQTT action?

There's also a lag when i call a shell_command in the action.

Do the entity updates in the front-end happen quickly or is there this 1s delay?

They happen with the same lag to the trigger.

Is there something in your mosquitto logs (maybe about too many subscriptions?

Absolutely nothing.

Does this disappear when you only have a single MQTT device in your config (thus not creating many subscriptions)?

For this i have to wait to do some tests. :-)
BTW, seems like i'm not the only one.
https://community.home-assistant.io/t/delay-between-ha-and-mqtt/44958/4?u=vdrainer

So I guess with this comment it's quite clear that it's due to subscriptions, and more precisely our new way of re-sending every single subscription to the broker (which was necessary because of #12003). Maybe the reason I'm not noticing this is that I have quite the beastly machine running MQTT here (though I still have ~100 MQTT topics that are subscribed to, some even multiple times).
I'm not sure how to go about fixing this though. One option would be to revert to saving retained messages ourselves as proposed in #12004, though that would definitely not be too nice. Another option would be to be a bit smarter about creating subscriptions. For example, while we know a new subscription has not been established yet we might not need to create a new one, but I think that would only really be a "hack" fix and wouldn't necessarily be scalable. Any thoughts @balloob?

MQTT specs:

If a Server receives a SUBSCRIBE Packet containing a Topic Filter that is identical to an existing Subscription’s Topic Filter then it MUST completely replace that existing Subscription with a new Subscription. The Topic Filter in the new Subscription will be identical to that in the previous Subscription, although its maximum QoS value could be different. Any existing retained messages matching the Topic Filter MUST be re-sent, but the flow of publications MUST NOT be interrupted [MQTT-3.8.4-3].

So I guess mosquitto doesn't follow standard here :/ (as multiple subscriptions to the same topic shouldn't cause any slowdowns...) At least if my theory is correct...

So before we come up with solutions, we should really understand the problem here. What behavior is causing the delay and why is this happening?

is this publish or subscribe?
Is the delay in the Python code or in the MQTT server?
Is is Mosquitto specific or do we see the same behavior with other MQTT servers?
What is the difference between 1 and 100 subscriptions / publishing?

Looking at 0.63 and the new code, the biggest difference is that with the old code we would schedule all callbacks at the same time on the event loop. With the new one we do it in sequence.
@VDRainer, can you change async_run_job on this line to async_add_job and run your benchmark again?

@balloob , thanks for your response.

No hardware, wifi, physical switches or something involved.
Only one mqtt switch and an automation, nothing else with mqtt.
switch:
  - platform: mqtt
    name: "MQTT Test"
    state_topic: "mqtt_test/switch01"
    command_topic: "mqtt_test/switch01"
    qos: 0
    payload_on: "ON"
    payload_off: "OFF"
    retain: true

automation:
  - alias: mqtt_test
    initial_state: True
    hide_entity: False
    trigger:
      - platform: mqtt
        topic: "mqtt_test/trigger"
        payload: 'ON'
    action:
      service: switch.toggle
      entity_id: switch.mqtt_test

Triggered with mosquitto_pub -t "mqtt_test/trigger" -m "ON"
The results are same as in my normal home environment.

Add at top of file:
import time

Add this to print timestamp:
print('<function name ie async_publish>', time.time())

And then add such code to (before the highlighted line):

https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/mqtt/__init__.py#L208
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/mqtt/__init__.py#L392
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/mqtt/__init__.py#L407
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/mqtt/__init__.py#L482
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/mqtt/__init__.py#L591
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/mqtt/__init__.py#L596
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/switch/mqtt.py#L139
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/switch/mqtt.py#L153

Puuh, took some time for this.
On my productive system no chance, because the log runs to fast.
Fiddled it into hass.io test env from my last post.
Was not sure about the function names, so included the line numbers from your response.
591 async_add_job<PHONE_NUMBER>.0046504
596 _mqtt_handle_message<PHONE_NUMBER>.882419
2018-02-27 12:30:39 INFO (MainThread) [homeassistant.components.automation] Executing mqtt_test
611 async_run_job<PHONE_NUMBER>.8863292
switch-mqtt 153 async_turn_off<PHONE_NUMBER>.887972
208 _build_publish_data<PHONE_NUMBER>.8921425
392 async_publish_service<PHONE_NUMBER>.8993862
407 async_publish<PHONE_NUMBER>.8995473
482 _paho_lock<PHONE_NUMBER>.8996415
591 async_add_job<PHONE_NUMBER>.9069247
596 _mqtt_handle_message<PHONE_NUMBER>.8830576
611 async_run_job<PHONE_NUMBER>.8839433

591 async_add_job<PHONE_NUMBER>.302853
596 _mqtt_handle_message<PHONE_NUMBER>.883064
2018-02-27 12:31:05 INFO (MainThread) [homeassistant.components.automation] Executing mqtt_test
611 async_run_job<PHONE_NUMBER>.8871262
switch-mqtt 139 async_turn_on<PHONE_NUMBER>.9035816
208 _build_publish_data<PHONE_NUMBER>.9111507
392 async_publish_service<PHONE_NUMBER>.911728
407 async_publish<PHONE_NUMBER>.9149196
482 _paho_lock<PHONE_NUMBER>.9149723
591 async_add_job<PHONE_NUMBER>.922455
596 _mqtt_handle_message<PHONE_NUMBER>.078753
611 async_run_job<PHONE_NUMBER>.0793777

Hope that helps!

That's certainly a very interesting result, so here's my interpretation of what's happening here:
Something is causing a very significant slowdown between when our callback _mqtt_on_message is called and when we handle the message in _mqtt_handle_message. We see that because in your logs there's a ~0.6 second delay between those two calls. If we look at the code path from before and after it's more or less the same concerning receiving messages:
before:

_mqtt_on_message -> dispatcher_send(signal)
-> hass.loop.call_soon_threadsafe(async_dispatcher_send)
-> async_dispatcher_send -> hass.async_add_job(FINAL_CALLBACK)

after:

_mqtt_on_message -> hass.async_add_job(_mqtt_handle_message) (SHOULD BE add_job 🤕)
_mqtt_handle_message -> hass.async_run_job(FINAL_CALLBACK)

So apart from the error with using async_add_job instead of add_job the code should execute more or less the same. Even the async_run_job(FINAL_CALLBACK) should more or less be the same speed since callbacks shouldn't do any expensive tasks. What this also probably means is that my previous theory was wrong and this is not do to the broker, I simply didn't expect such a significant slowdown to be caused by anything else (but I guess tasks+asyncio can do that now).
Could you try replacing the async_add_job on line 591 with a add_job?

BINGO!
591 async_add_job now add_job<PHONE_NUMBER>.4511173
596 _mqtt_handle_message<PHONE_NUMBER>.4547071
611 async_run_job<PHONE_NUMBER>.455014
2018-02-27 13:25:59 INFO (MainThread) [homeassistant.components.automation] Executing mqtt_test
switch-mqtt 139 async_turn_on<PHONE_NUMBER>.4609294
208 _build_publish_data<PHONE_NUMBER>.4636903
392 async_publish_service<PHONE_NUMBER>.4643748
407 async_publish<PHONE_NUMBER>.464434
482 _paho_lock<PHONE_NUMBER>.464483
591 async_add_job now add_job<PHONE_NUMBER>.4667833
596 _mqtt_handle_message<PHONE_NUMBER>.4682815
611 async_run_job<PHONE_NUMBER>.4684873

591 async_add_job now add_job<PHONE_NUMBER>.3577397
596 _mqtt_handle_message<PHONE_NUMBER>.3623252
611 async_run_job<PHONE_NUMBER>.3625004
2018-02-27 13:26:09 INFO (MainThread) [homeassistant.components.automation] Executing mqtt_test
switch-mqtt 153 async_turn_off<PHONE_NUMBER>.3698487
208 _build_publish_data<PHONE_NUMBER>.3710778
392 async_publish_service<PHONE_NUMBER>.3716028
407 async_publish<PHONE_NUMBER>.3716664
482 _paho_lock<PHONE_NUMBER>.371715
591 async_add_job now add_job<PHONE_NUMBER>.3746936
596 _mqtt_handle_message<PHONE_NUMBER>.3753254
611 async_run_job<PHONE_NUMBER>.3754404

Great to see you found a fix! Thanks for helping us debug @VDRainer ❤

Yes I want to say thank you too @VDRainer. Without your great debugging support this wouldn't have been possible to fix this quickly. 👍

My pleasure! :relieved:
Thanks!


ARROW-5978: [FlightRPC] [Java] Properly release buffers in Flight integration client

Fixes a bug where dictionaries weren't properly released in cleaning up a flight stream.
Travis build: https://travis-ci.com/lihalite/arrow/builds/119807464
Recreated from https://github.com/apache/arrow/pull/4905

Codecov Report

Merging #4913 into master will increase coverage by 2.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4913      +/-   ##
==========================================
+ Coverage   87.44%   89.58%   +2.14%
==========================================
  Files         995      661     -334
  Lines      140479    96636   -43843
  Branches     1418        0    -1418
==========================================
- Hits       122841    86572   -36269
+ Misses      17276    10064    -7212
+ Partials      362        0     -362

Impacted Files
Coverage Δ

cpp/src/arrow/json/converter.cc
90.53% <0%> (-1.78%)
:arrow_down:

cpp/src/arrow/json/chunked-builder.cc
79.91% <0%> (-1.68%)
:arrow_down:

r/src/recordbatch.cpp

r/R/Table.R

js/src/util/fn.ts

go/arrow/array/bufferbuilder.go

r/src/symbols.cpp

rust/datafusion/src/execution/projection.rs

rust/datafusion/src/execution/filter.rs

rust/arrow/src/csv/writer.rs

... and 328 more

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 e1559f2...0805c3c. Read the comment docs.

@lihalite is this ready for review?  I just tried restarting failed travis jobs.

@emkornfield thanks for taking a look. Looks like Travis flaked on some downloads, so I've rebased and pushed to try again. Otherwise it should be ready.

Ok, tests finally pass!

@lihalite could you add a test that reproduces the bug?

@BryanCutler good point, I've added a test to make sure that both issues are fixed (dictionaries not being freed/refcount going negative when dictionaries are freed).

This looks reasonable to me.  @BryanCutler any concerns with merging?

Let me push a fix for the Travis failures - I got blocked yesterday by the failure on master.

Personal fork: https://travis-ci.com/lihalite/arrow/builds/123360416

Looks like the C++/Python build flaked out.

On flakiness, I think if you rebase this has been fixed on master.   I'll merge tomorrow if @BryanCutler doesn't have any more concerns.

I'll try to look at this soon, but don't block on me and feel free to merge

+1, thank you @lihalite


FIX Issues with AdaLora initialization

Resolves #1647

AdaLoraConfig now converts target_modules to set, same as LoRA
AdaLoraConfig now raises when used with DoRA
AdaLoraConfig now raises when used with LoftQ
AdaLoraModel now raises when trying to call add_weighted_adapter
Add tests for those in test_initialization.py
Small clean ups in test_initialization.py

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Unable to generate an independent node instance

Godot version: 3.1.1-stable (FreeBSD ports build)

OS/device including version: FreeBSD 11.3-STABLE r349638

Issue description:

Consider the following two scripts, Simple.gd and Driver.gd
extends Node
var a = 17
func _ready():
	pass
func get_a():
	return self.a
func set_a(value : int) :
	self.a = value

extends Node
var oldd
onready var d = load("res://Simple.gd").new()
func _ready():
 	pass
func _on_Button_pressed():
	print("#### PRESSED")
	if (oldd != null) :
		print("Old A is " + str(oldd.get_a()))
	var newd = d.duplicate(DUPLICATE_USE_INSTANCING|DUPLICATE_SCRIPTS)
	print("New 'instance' attempt, a is " + str(newd.get_a()))
	newd.set_a(22)
	print("Set a to 22: " + str(newd.get_a()))
	if (oldd != null) :
		print("Old a still should be 17: " + str(oldd.get_a()))
	print(oldd)
	print(newd)
	oldd = newd

The output from the 2nd and subsequent button presses indicates that the old node (oldd) is getting changed along with the new node:
#### PRESSED
Old A is 22
New 'instance' attempt, a is 17
Set a to 22: 22
Old a still should be 17: 22
[Node:1780]
[Node:1991]

I would expect that newd would be a fresh instance and set_a to only change the fresh instance's copy of a. Instead, it changes all of them (tested this in a much more complex project). I do not understand. Is this my naivete, a bug in Godot, or a combination of both?
Note that I have tried new(), duplicate(), and instance() to get a fresh instance. All three, while returning slightly different semantics when printed, produce the same result.

Old a still should be 17

On second button pressed, this statement can't be true because you set a to 22 and assign the node to oldd during the previous button press. So on the next button press, oldd is still 22.

Yep, this test case is whacked. :) Closing.


filter_parser: Add suppress_parse_error parameter

Related to https://github.com/fluent/fluentd/issues/1480
This is acceptable for me because configuration becomes error prone without this parameter for ignoring unneeded log lines case.
<label @ERROR>
  <match {tag1,tag2,...}>
    @type null
  </match>
</label>

@ERROR label is for capturing unexpected error event so blackhoke usecase is misuse.

@tagomoris Could you check this?

@tagomoris ping?

Use raise when no emit case.
With reserve_data true, filter_parser passes incoming data to next filter so can't use raise.

LGTM.


[cppzmq] draft api feature

Is your feature request related to a problem? Please describe.
cppzmq can be built with ENABLE_DRAFTS option. I want to be able to build using this flag to get extended functionality of the library.
Proposed solution
Add feature draft that sets ENABLE_DRAFTS option.
Additional context
See CMake file that contains the ENABLE_DRAFTS option: https://github.com/zeromq/cppzmq/blob/03243ad64db652c43d150e9fb5a80a13c1a1a4a9/CMakeLists.txt#L34

This patch can be used:
From 835cd51df568bfefd625c996b8aa3870f871bf5c Mon Sep 17 00:00:00 2001
From: Daniel Lundborg<EMAIL_ADDRESS>Date: Tue, 17 Nov 2020 18:41:06 +0100
Subject: [PATCH] [cppzmq] draft api feature

Feature to build and install draft APIs.

Signed-off-by: Daniel Lundborg<EMAIL_ADDRESS>---
 ports/cppzmq/CONTROL        |  3 +++
 ports/cppzmq/portfile.cmake | 10 +++++++++-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/ports/cppzmq/CONTROL b/ports/cppzmq/CONTROL
index bec4b67c6..577399bd5 100644
--- a/ports/cppzmq/CONTROL
+++ b/ports/cppzmq/CONTROL
@@ -3,3 +3,6 @@ Version: 4.7.1
 Build-Depends: zeromq
 Homepage: https://github.com/zeromq/cppzmq
 Description: lightweight messaging kernel, C++ bindings
+
+Feature: draft
+Description: Build and install draft APIs
diff --git a/ports/cppzmq/portfile.cmake b/ports/cppzmq/portfile.cmake
index f8c2ed28d..d1a902843 100644
--- a/ports/cppzmq/portfile.cmake
+++ b/ports/cppzmq/portfile.cmake
@@ -6,10 +6,18 @@ vcpkg_from_github(
     HEAD_REF master
 )

+vcpkg_check_features(
+    OUT_FEATURE_OPTIONS FEATURE_OPTIONS
+    FEATURES
+        draft ENABLE_DRAFTS
+)
+
 vcpkg_configure_cmake(
     SOURCE_PATH ${SOURCE_PATH}
     PREFER_NINJA
-    OPTIONS -DCPPZMQ_BUILD_TESTS=OFF
+    OPTIONS
+        -DCPPZMQ_BUILD_TESTS=OFF
+        ${FEATURE_OPTIONS}
 )

 vcpkg_install_cmake()
--
2.25.0


VW Passat GTE: Not allowed to start climatisation without plugged in power

The system refuses to start climatisation without plugged in power unless that setting is sent in the request:
...body {'callback': 'https://mobileapi.apps.emea.vwapps.io/vehicles/callbacks/.../climatisation/start', 'targetTemperature_K': 297.15}
...
...info The process could not be started. Please make sure that the charging cable is plugged and try again.
But with the setting added,
...body {'targetTemperature_K': 297.15, 'callback': 'https://mobileapi.apps.emea.vwapps.io/vehicles/callbacks/.../climatisation/start', 'climatisationWithoutExternalPower': True}
...status successful
...body {'callback': 'https://mobileapi.apps.emea.vwapps.io/vehicles/callbacks/.../climatisation/start', 'climatisationWithoutExternalPower': True, 'targetTemperature_K': 297.15}
it works.
I added this value to settingsDict in controls.py. This should probably reflect the value from the settings instead:
                elif 'targetTemperature_F' in settingsDict:
                    settingsDict['targetTemperature_K'] = farenheitToKelvin(settingsDict['targetTemperature_F'])
                else:
                    settingsDict['targetTemperature_K'] = celsiusToKelvin(20.5)
+            settingsDict['climatisationWithoutExternalPower'] = True

        data = json.dumps(settingsDict)
        controlResponse = self.vehicle.weConnect.session.post(url, data=data, allow_redirects=True)
        if controlResponse.status_code != requests.codes['ok']:
            errorDict = controlResponse.json()

Hey, thank you for reporting and the excellent analysis. I will release the fix as soon as possible.

Should be fixed in 0.47.1 release

Great, thanks!


rlly amazin game i made

check out this sick game

@nathan962 I'm happy to merge this in and move it to the right directory for you.


Implement OAM DMA transfer

https://gbdev.io/pandocs/OAM_DMA_Transfer.html

Initial implementation here: https://github.com/pineman/fpt/blob/main/fpt/src/memory/mod.rs#L161.
Maybe the DMA OAM transfer timings should be respected?


Added constructors taking PyObject

Added constructors taking PyObject
Description
Added overloaded constructor with PyObject as parameter. PeriodCountConsolidatorBase constructor convert PyObject to delegate .
Related Issue
#3780
Motivation and Context
Though there is workaround available, suggestion is not to rely on System package. So overloaded C# class constructors
Requires Documentation Change
NA
How Has This Been Tested?
Using snippet in #3780
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Removed code where, I tried to convert C# delegate to PyObject. it's failing on CI unit test run, though it's alright on my win/linux dev run. Just kept step where function from Python module is converted to PyObject.
There is a separate Python file, which can be tested in isolation.

Superceeded/extended by #3816


Please stop polute global scope

For modern ES lib it's absolutely wrong behavior https://github.com/Blankeos/tsdot/blob/main/src/index.ts#L233
it's should be enabled optional during bundling process.

Thanks for the comment. Wasn't aware that it was widely considered wrong.
I was just trying to keep the old behavior when importing it via a  tag in Vanilla HTML.
I'll make a way to make it optional. Thanks!

I was just trying to keep the old behavior when importing it via a  tag in Vanilla HTML.

Yes, I think it's good point, but I believe it should be enabled only during bundle browser stand alone script. For ES6 or TS module it shouldn't be enabled.


Refactor VideoPlayer component to improve rewind and forward function

Issue: Tapping on the forward or rewind button sets the currentTime to 0 until buffering So quickly tapping on the buttons more than once starts the video from start.
I throttled forward and rewind function to use accumulated time.
if this is not the correct way please fix this issue.

Not sure what's the issue here. Can you give an example or a reproduction repository? That would be helpful

Not sure what's the issue here. Can you give an example or a reproduction repository? That would be helpful

quickly press the fast-forward or the rewind button 3 or 4 times and the video will start playing from the beginning.

@himanshu8443 if it reaches the end, it's meant to start from beginning. What's your wanted behaviour?

check this video I just pressed the forward button and the video started playing from the beginning.
https://github.com/LunatiqueCoder/react-native-media-console/assets/99420590/345261b7-3455-43c9-891a-8c8678b5036e

@himanshu8443 Ok, I will have to look into it, but I can't promise it will happen very soon :(

I am noticing this same thing on Android only, when i click the back button twice fast it starts the video from the start.

@LunatiqueCoder hey I found that its because in VideoPlayer.tsx there is _onSeek function
const _onSeek = (obj: OnSeekData) => {
    if (!seeking) {
      setControlTimeout();
    }
    setCurrentTime(obj.seekTime);  // <--here
    console.log('seek time',obj.seekTime);

    if (typeof onSeek === 'function') {
      onSeek(obj);
    }
  };

logging obj.seekTime prints 0 means it sets currentTime to 0 which causes video playing from start and the actual seek time is returned in obj.currentTime
after setting setting  setCurrentTime(obj.currentTime); it fix the issue
 const _onSeek = (obj: OnSeekData) => {
    if (!seeking) {
      setControlTimeout();
    }
    setCurrentTime(obj.currentTime); // <--here

    if (typeof onSeek === 'function') {
      onSeek(obj);
    }
  };

@himanshu8443 are there any plans to resolve these CI failures to get this merged? 😅

@himanshu8443 are there any plans to resolve these CI failures to get this merged? 😅

well, it's not necessary now, as I said above setCurrentTime(obj.seekTime); was the issue I think it's fixed in the new version of react-native-video. I should have closed this pull request.


Update dependencies to match documentation.

They're also now more flexible.
This is as an alternative to #597 which makes the dependencies match the docs.
(We moved from using Bower for Backgrid to using NPM because of the ~1.1.0 on Backbone preventing us from using 1.2.x -- hence, I'm a believer in having more flexible version restrictions - but understand if others believe otherwise :-) )

Taken #607 instead


No location behaves as there is

When there is no location fix at the start, there is no blue marker, but when you drag the map, the crosshairs button appears.

Fixed in 3.0.


Fix Typo in stdcopy.go

Signed-off-by: Chengxia Yu<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:feixueliantian1/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.


Arabic tokenization is broken

The T5 tokenizer doesn't seem to know what to do with all of the Arabic text in the dataset (along with some other non-Latin scripts). Here's an example from Natural Instructions followed by its tokenization:
Example:

Teacher:You are given a sentence in Arabic. Your job is to translate the Arabic sentence into English.
Teacher: Now, understand the problem? Solve this instance: الممرضة في عيادة ذات ضغط نرى فيها من 50 إلى 100 مريض يومياً ، يترك لها فقط بضع دقائق لتقضيها مع كل مريض — دقائق لكل مريض.
Student:

Tokenization:

[17476    10  3774    33   787     3     9  7142    16 19248     5   696
   613    19    12 13959     8 19248  7142   139  1566     5 17476    10
   852     6   734     8   682    58  5175   162    48  3421    10     3
     2     3     2     3     2     3     2     3     2     3     2     3
     2     3     2   943     3     2   910     3     2     3     2     3
     2     3     2     3     2     3     2     3     2     3     2     3
     2     3     2     3     2     3     2     3   318     3     2     3
     2     3     2     5  6341    10     1]

Is this known? Were scripts not in the T5 tokenizer excluded from FLAN runs?

Hello gahdritz, any luck in finding a solution for this problem?

@gahdritz Sorry for the delay on answering this -- I missed it. The T5 tokenizer is just for English so cannot reliably handle other languages/scripts. The Flan Collection does include multiple languages (especially within the NIv2 submixture), but we did not exclude this training data in our runs (either for Flan-T5 or Flan-PaLM, where the latter is multilingual).
To accommodate multilingual datasets you can either change the tokenizer used in the Flan Collection, or just save the pretokenized inputs and outputs yourself and tokenize them later.