Rclone sync with --files-from tag does not remove deleted files/folders from destination

Hi,
I am using --files-from tag with the copy and sync command to migrate data from local to Google cloud storage. I am using flat-file generated from an external script to track/migrate files on my local storage.
Steps to reproduce:

I copied a source directory with the copy command and --files-from tag (File calculated through os walk). It worked fine.
Later I deleted some files and folders from inside the source directory. I calculated the flat-file again and the new file does not contain the deleted files. I used the new flat file with sync command.

Theoretically, Rclone should delete the files present on the destination which are not in the flat file. But the observed behavior is that the Rclone only tracks files present in the flat file. For a file which is not present in the flat file, but some kind of operation has been performed on that file (Update, Delete), Rclone will not migrate the changes to destination for that file.
Is this the intended behavior? I mean, I can calculate the difference in files using a script but this is just rewriting the behavior intended for the sync command.
Any thoughts on this issue?
Thanks
EDIT:
I am using Rclone: Version "v1.48.0-DEV" on a Windows 10 Machine.
Added Sample Flat file used:
With copy command (I uploaded a directory say C:\Users\Manish\Test to destination) with the following tag --files-from C:\Users\Manish\Temp\rclone_files.txt where the content of rclone_files.txt is:
/_5/_
/_6/_
/_7/_
/cvgr4wedfgbht4rwefgr3ef.rtf
/sdefr4wefgre.txt
/xcfedqwe.txt

Logs when copying:

PS C:\Users\manis> go run .\rclone.go copy C:\Users\manis\Desktop\0\ --files-from  C:\Users\manish\rclone_report.txt --config .\rclone.config -vvP
2019/07/08 22:41:04 DEBUG : rclone: Version "v1.48.0-DEV" starting with parameters ["C:\Users\manis\rclone.exe" "copy" "C:\Users\manis\Desktop\0\" "--files-from" "C:\Users\manis\rclone_report.txt" "--config" ".\rclone.config" "-vvP"]
2019/07/08 22:41:04 DEBUG : Using config file from "C:\Users\manis\rclone.config"
2019-07-08 22:41:06 INFO  : Storage bucket cloud-dev path data/5731798554247168/: Waiting for checks to finish
2019-07-08 22:41:06 INFO  : Storage bucket cloud-dev path data/5731798554247168/: Waiting for transfers to finish
2019-07-08 22:41:06 INFO  : sdefr4wefgre.txt: Copied (new) ETA 0s
2019-07-08 22:41:06 INFO  : xcfedqwe.txt: Copied (new)/s, ETA 0ss
2019-07-08 22:41:06 INFO  : 7/: Copied (new)98 Bytes/s, ETA 0s
2019-07-08 22:41:07 INFO  : cvgr4wedfgbht4rwefgr3ef.rtf: Copied (new)
2019-07-08 22:41:07 INFO  : 6/: Copied (new)84 Bytes/s, ETA 0s
2019-07-08 22:41:08 INFO  : 5/: Copied (new)65 Bytes/s, ETA 0s
Transferred:           282 / 282 Bytes, 100%, 58 Bytes/s, ETA 0s-
Errors:                 0 / 0, -7%          7/:  0% /0, 0/s, --
Checks:                 0 / 0, -3%          6/:  0% /0, 0/s, ---
Transferred:            6 / 6, 100%         6/:  0% /0, 0/s, - -
Elapsed time:        4.8s                   6/:  0% /0, 0/s, - -
2019/07/08 22:41:08 INFO  :                 5/:  0% /0, 0/s, - -
Transferred:           282 / 282 Bytes, 100%, 58 Bytes/s, ETA 0s -
Errors:                 0           xcfedqwe.txt: transferring
Checks:                 0 / 0, -ht4rwefgr3ef.rtf: transferring
Transferred:            6 / 6, 100%
Elapsed time:        4.8s
2019/07/08 22:41:08 DEBUG : 16 go routines active
2019/07/08 22:41:09 INFO  : The success report is available at C:\Users\manis\AppData\Local\rclone\success_07082019224105.csv
2019/07/08 22:41:10 DEBUG : success_07082019224105.csv: Couldn't find file - need to transfer
2019/07/08 22:41:10 INFO  : success_07082019224105.csv: Copied (new)
2019/07/08 22:41:11 INFO  : The failure report is available at C:\Users\manis\AppData\Local\rclone\failure_07082019224105.csv
2019/07/08 22:41:12 DEBUG : failure_07082019224105.csv: Couldn't find file - need to transfer
2019/07/08 22:41:12 INFO  : failure_07082019224105.csv: Copied (new)
2019/07/08 22:41:12 DEBUG : rclone: Version "v1.48.0-DEV" finishing with parameters ["C:\Users\manis\rclone.exe" "copy" "C:\Users\manis\Desktop\0\" "--files-from" "C:\Users\manis\rclone_report.txt" "--config" ".\rclone.config" "-vvP"]

Then I deleted folder /_5/_ and added a new file dwewwd.txt on local machine but the delete changes were not reflected on destination. I used the updated rclone_files.txt with the sync command this time and the content of the rclone_files.txt is:
/_6/_
/_7/_
/cvgr4wedfgbht4rwefgr3ef.rtf
/sdefr4wefgre.txt
/xcfedqwe.txt
dwewwd.txt

Logs when syncing

PS C:\Users\manis> go run .\rclone.go copy C:\Users\manis\Desktop\0\ --files-from  C:\Users\manis\rclone_report.txt --config .\rclone.config -vvP
2019/07/08 22:44:27 DEBUG : rclone: Version "v1.48.0-DEV" starting with parameters ["C:\Users\manis\rclone.exe" "copy" "C:\Users\manis\Desktop\0\" "--files-from" "C:\Users\manis\rclone_report.txt" "--config" ".\rclone.config" "-vvP"]
2019/07/08 22:44:27 DEBUG : Using config file from "C:\Users\manis\rclone.config"
2019-07-08 22:44:29 DEBUG : 5/: Excluded from sync (and deletion)
2019-07-08 22:44:29 INFO  : Storage bucket cloud-dev path data/5731798554247168/: Waiting for checks to finish
2019-07-08 22:44:29 DEBUG : cvgr4wedfgbht4rwefgr3ef.rtf: Size and modification time the same (differ by 0s, within tolerance 100ns)
2019-07-08 22:44:29 DEBUG : cvgr4wedfgbht4rwefgr3ef.rtf: Unchanged skipping
2019-07-08 22:44:29 DEBUG : sdefr4wefgre.txt: Size and modification time the same (differ by 0s, within tolerance 100ns)
2019-07-08 22:44:29 DEBUG : xcfedqwe.txt: Size and modification time the same (differ by 0s, within tolerance 100ns)
2019-07-08 22:44:29 DEBUG : 6/: Size and modification time the same (differ by 0s, within tolerance 100ns)
2019-07-08 22:44:29 DEBUG : 7/: Size and modification time the same (differ by 0s, within tolerance 100ns)
2019-07-08 22:44:29 DEBUG : sdefr4wefgre.txt: Unchanged skipping
2019-07-08 22:44:29 DEBUG : xcfedqwe.txt: Unchanged skipping0s
2019-07-08 22:44:29 DEBUG : 6/: Unchanged skipping/s, ETA 0s
2019-07-08 22:44:29 DEBUG : 7/: Unchanged skipping/s, ETA 0s
2019-07-08 22:44:29 INFO  : Storage bucket cloud-dev path data/5731798554247168/: Waiting for transfers to finish
2019-07-08 22:44:29 INFO  : dwewwd.txt: Copied (new)/s, ETA 0s
Transferred:            18 / 18 Bytes, 100%, 6 Bytes/s, ETA 0s
Errors:                 0 / 5, 100%         6/: checking 0/s, -
Checks:                 5 / 5, 100%         6/: checking 0/s, -
Transferred:            1 / 1, 100%         6/: checking 0/s, -
Elapsed time:        2.7s             dwewwd.txt:100% /18, 0/s, -
2019/07/08 22:44:29 INFO  :           dwewwd.txt:100% /18, 0/s, -
Transferred:            18 / 18 Bytes, 100%, 6 Bytes/s, ETA 0s, -
Errors:                 0
Checks:                 5 / 5, 100%
Transferred:            1 / 1, 100%
Elapsed time:        2.8s
2019/07/08 22:44:29 DEBUG : 10 go routines active
2019/07/08 22:44:30 INFO  : The success report is available at C:\Users\manis\AppData\Local\rclone\success_07082019224428.csv
2019/07/08 22:44:31 DEBUG : success_07082019224428.csv: Couldn't find file - need to transfer
2019/07/08 22:44:32 INFO  : success_07082019224428.csv: Copied (new)
2019/07/08 22:44:32 INFO  : The failure report is available at C:\Users\manis\AppData\Local\rclone\failure_07082019224428.csv
2019/07/08 22:44:33 DEBUG : failure_07082019224428.csv: Couldn't find file - need to transfer
2019/07/08 22:44:33 INFO  : failure_07082019224428.csv: Copied (new)
2019/07/08 22:44:33 DEBUG : rclone: Version "v1.48.0-DEV" finishing with parameters ["C:\Users\manis\rclone.exe" "copy" "C:\Users\manis\Desktop\0\" "--files-from" "C:\Users\manis\rclone_report.txt" "--config" ".\rclone.config" "-vvP"]

I am not the developer or designed so I cannot state what the intent was, but I strongly hope that the current behavior is intended!
And, for what it is worth, when using rsync with --delete (which has the same behavior as rclone sync) it does not seem to delete non-listed files

@ncw Can you please confirm?

I think this is working as designed.
If you want to delete things not in the --files-from list then you will need to use --delete-excluded.  Make sure you test with --dry-run first as it will delete everything that is not in the list!

Question looks to be answered and if there is anything else you need, please post a question to the forums.


Bug in Linear Explainer Correlation Correction

Considering a model made of two features that have correlation r, we were estimating Shap values for values of r between -1 and 1. In the current implementation the correction works the reverse way:
For correlation r=1 the features share their total shap values. However one can observe the opposite behaviour.

Was this closed because there was not a problem after all?


Update LTS version

New 6.x release on Sept. 5.

6.11.4 released in https://github.com/bucharest-gold/centos7-s2i-nodejs/commit/4f8c725ba809b9ffa46da9cce1832244ed42b8a0


David/coverage on release

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #5131 (8547263) into release-XXXX.XX (094c881) will increase coverage by 0%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           release-XXXX.XX   #5131   +/-   ##
===============================================
  Coverage               73%     73%
===============================================
  Files                  402     402
  Lines                26391   26391
  Branches              3798    3798
===============================================
+ Hits                 19470   19482   +12
+ Misses                5343    5335    -8
+ Partials              1578    1574    -4

Impacted Files
Coverage Δ

src/client/datascience/raw-kernel/rawSocket.ts
84% <0%> (-2%)
:arrow_down:

...lient/datascience/jupyter/kernels/cellExecution.ts
81% <0%> (ø)

src/client/datascience/baseJupyterSession.ts
76% <0%> (+<1%)
:arrow_up:

src/client/datascience/jupyter/jupyterNotebook.ts
76% <0%> (+<1%)
:arrow_up:

src/client/datascience/jupyter/kernelVariables.ts
76% <0%> (+<1%)
:arrow_up:

src/client/datascience/notebook/notebookEditor.ts
35% <0%> (+1%)
:arrow_up:

src/client/common/process/pythonDaemonPool.ts
83% <0%> (+4%)
:arrow_up:


Add map for saudi

Added a map for saudi. where i used the map to work on a torch tour map.

THanks a lot for your contribution @anasAbdulMajeed08  !


php 7.0 support

Is it possible to support also php7.0?

7.0 is dead. 7.1 is the lowest that is supported.


Fix user issue

Fixes #15094
Invalid filters weren't throwing the error that they should have been. This fixes the issue by putting the QueryOptions object in the ItemPaged instead of in the command

/azp run python - tables - tests

/azp run python - tables - tests

/azp run python - tables - tests


[Fix #118] Fix a false positive for DeletePrefix and DeletePrefix cops

Fixes #118.
Fix a false positive for Performance/DeletePrefix and Performance/DeletePrefix cops when using heredoc.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Hi @koic
I think the problem is not in any way related to heredocs.
Take any string containing newlines, and you cannot simply replace a call to .gsub(/^prefix/, '') with delete_prefix('prefix') since the latter has a different meaning, likewise for .gsub(/suffix$/, '')`.
The only thing that works in general is /\Aprefix/ and /suffix\z/.

@avdv Thanks for your feedback. I have rewritten this PR and added SafeMultiline option which is true by default. So it will detect only /\Aprefix/ and /suffix\z/ by default.

Thank you, LGTM!


[SDK][Typescript] Export Encryption and EncryptionUtils

Description
Export Encryption and EncryptionUtils and add OpenPGP as dependency in package.json

I don't know how you added OpenPGP package, but that affected the other modules. Please fix that to make Dependency Review check pass before merging this PR.


friendly_name_template not working

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:
0.64.3
friendly_name_template
i have this:
forecast_3:
friendly_name_template: >-
{%- set date = as_timestamp(now()) + (3 * 86400 ) -%}
{{ date | timestamp_custom("%A (%-d/%-m)") }}
value_template: >
{{states.sensor.dark_sky_daily_high_temperature_3.state|round(0)}}°/{{states.sensor.dark_sky_daily_low_temperature_3.state|round(0)}}°/{{states.sensor.dark_sky_precip_probability_3.state|round(0)}}
entity_picture_template: >-
{{ '/local/icons/dark_sky/' ~ states.sensor.dark_sky_icon_3.state ~ '.png'}}
But all it shows is forecast3:

Are you on Hassio?

Hassio here and it does not work for me either as reported on the thread. The old weather.yaml and python script work perfectly though.

Oh I think what's going on here: The templates for value_template, entity_picture_template, and friendly_name_template are execute after each other in that order. If any of those templates fail, then the other ones don't get executed. will be a relatively simple fix.


Published Date Box

Hi all,
We'd like to see a box in the metadata that lists a "Last Published On:" date where the user can enter a date. This will be good for articles that need frequent updating or reference specific data.
I'll reach out to Ana and see if I can get a mockup for this to include in this request.

@amol-anand do you know where you left off on this? I don't see "publication-date" in author-bio.js, but I see it is added to the index already.


Create a Terraform Template POC

Background 👀
Currently, we have a few terraform repositories that use a single large state file. This type of architecture for terraforming doesn't scale since deploy times increase linearly as the number of resources grows.
This results in terraform changes being made locally instead of through CI/CD, since deploying through pipelines takes significantly longer.
An example is dns-iac, which is deployed on local machines - taking ~1 hour for a terraform plan.
Task 🚀
The purpose of this ticket is to create a proof of concept (POC) terraform template project which will attempt to have the following characteristic for projects:

Scalable, i.e. as resources grow, deploy times stay consistent
Consistent/Automated, i.e. All deploys are handled through CI/CD
Resilient for team use, i.e. people raising multiple PRs are handled via CI/CD

Useful 🧠

ministryofjustice/terraform-template-poc

Acceptance Criteria 🎯

A terraform-template project has been created, which is automated, scalable and resilient for team use

The terraform-template-poc has now been created and contains several features that support the scalable creation of Terraform, suitable for team use. Some of these concepts have been tested at a small scale.
I will mark the ticket as closed and the POC can be tested when we next need to create a Terraform project.


[DOC] SLIXLineplotParameterGenerator example

#Is it possible to add an example for SLIXLineplotParameterGenerator (even a very brief one) based on one of the two datasets used for SLIXParameterGenerator and providing a text file with intensity values? This would make easier to try out that specific functionality.
https://github.com/openjournals/joss-reviews/issues/2675

I absolutely agree that we should include one or two examples for SLIXLineplotParameterGenerator in the README.md. I've added two example files in 10-SLIXLineplotParameterGenerator-example. The README.md is still in progress but will contain both instructions on how to use SLIXLineplotParameterGenerator and what results are being generated when running the program for the given examples (see update-readme).

I have updated the README.md with a more detailed explanation and an example of how to use SLIXLineplotParameterGenerator in the Section "Evaluation of Line Profiles". Please let us know if we should add anything else there.

@matteomancini: Please let us know if we can close this issue.

I think it looks great, it is clear and immediate to reproduce!


Make api-authentication-required expose the authenticationResult into the request/response

Originally from #148, but inability to get Travis to work with building that (probably because it's a PR from fork) I created this instead.
Closes #148, #138.

Thanks @typerandom !  Please make sure to amend the contributors doc when we merge in a contributed PR


fix: nyc command with shard option

Fixes: #256
Added a new environment variable to check if the shard cli option is used to run the nyc command conditionally.

Codecov Report
Base: 75.86% // Head: 75.86% // No change to project coverage :thumbsup:

Coverage data is based on head (db255fc) compared to base (725761b).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             next     #257   +/-   ##
=======================================
  Coverage   75.86%   75.86%
=======================================
  Files          11       11
  Lines         174      174
  Branches       36       36
=======================================
  Hits          132      132
  Misses         42       42

Impacted Files
Coverage Δ

src/util/getParsedCliOptions.ts
5.88% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Hey @bryanjtc thank you so much for this contribution (and the many others)!
Does this mean that for developers using shard option, they should figure out how to do the reporting themselves?

Hey @bryanjtc thank you so much for this contribution (and the many others)!
Does this mean that for developers using shard option, they should figure out how to do the reporting themselves?

Yes. Especifically the coverage json file for each ci job should be renamed using the shardIndex value.
Then a new ci job has to merge this coverage.json files using the nyc merge command to output the complete report.

Could you assist in writing some sort of recipe? Also there should be extra changes in reportCoverage I assume? To have the renaming bit right

@yannbf Done. The renaming part should be done in the yml ci file. I added a recipe for github, gitlab and circle ci.

You're the best @bryanjtc, thanks a lot for this contribution! There were some small issues regarding watch mode so I added a few fixes as well.


Postgres cluster cannot initialize on Azure Kubernetes version 1.26.6

I am using the latest helmchart (v0.18.2) with default values to deploy the operator.
This then installs version 1.20.2 of the operator.
I am running kubernetes version 1.26.6
when I then configure a simple postgres cluster like this
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: postgres
spec:
  imageName: ghcr.io/cloudnative-pg/postgresql:13.12-6
  instances: 1
  storage:
    size: 1Gi

The initdb pod (postgres-1-initdb-5pq6x) does not start with the following error message
Error: container has runAsNonRoot and image has non-numeric user (nonroot), cannot verify user is non-root (pod: "postgres-1-initdb-5pq6x_bynubian-dev-02(edf17686-61ad-4165-b280-2a19c9400eda)", container: bootstrap-controller)
I found 2 possible solutions here
It seems that "nonroot" user is configured in this DockerFile
Looking into the container, it seems that the uid for the nonroot user is 65532 and gid is 65532
Option 1 is to add a securitycontext to that initdb pod
securityContext:
  runAsUser: 65532

but I don't see a possibility to configure that with helm
Option 2 is to modify the DockerFile and replace
USER nonroot:nonroot

with
USER 65532:65532

I tried this with a custom docker image like this
FROM ghcr.io/cloudnative-pg/cloudnative-pg:1.20.2
USER 65532:65532

When applying this custom docker image, the original error is gone but I then get a permission error
Error: failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: eaccess /controller/manager: permission denied: unknown

Turned out that the problem is not caused by cloudnative operator but rather by an update of akv2k8s submitted an issue here


Runs always appear twice

@yveskerwyn Can you verify that this is still an issue?

We this issue again on Artilium Cockpit:

Software Version
Verified on 8.1.0 rc1


toggling output visibility breaks table

if you hide and then show an output with a table, it gets cropped.

reproduced on master for me as well


feat(@vtmn/svelte): add VtmnDropdown component

Refer to #1051
Based on the work of @jb-thery
Changes description

This P.R adds the Svelte dropdown component
Context

Related issue #364
Checklist

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Please, don't request directly from your main!
[x] Check commits & PR names matches our requested structure. It must follow the https://www.conventionalcommits.org pattern.
[x] Check your code additions will fail neither code linting checks.
[x] I have reviewed the submitted code.
[x] I have tested on related showcases.
[x] If it includes design changes, please ask for a review with a core team designer.

Does this introduce a breaking change?

No

Other information

Need to update code, refer A11y -> #1113

Need to update code, refer A11y -> #1113

What do you mean @Tlahey by this comment?

@Tlahey thanks! Is it ok for you? If yes we will take this PR ready for review again :)

@lauthieb yes it's good for me :)

FYI : add max-height 260px to fit 5 items without scroll on the dropdown

@jb-thery & @Tlahey , AWESOME!! Thanks :tada:


Indexer: Add Indexer and Hasura GraphQl to movement

Summary

RFCs: Link to RFC, Link to RFC, or $\emptyset$.
Categories: any of protocol-units, networks, scripts, util, cicd, or misc.

Add the indexer and Hasura GraphQl engine to process compose.
This PR depends on these one:

https://github.com/movementlabsxyz/aptos-core/pull/53
https://github.com/movementlabsxyz/aptos-indexer-processors/pull/2

Changelog

Create an Indexer crate under network/suzuka.
Update process compose to run Indexer and Hasura.
Remove the indexer started with Suzuka node.
Create 2 process-compose script: indexer and Hasura
The Postgres Db is now started by the indexer and not Suzuka-node.
Testing
If there's some issue for building (dieser error) you need to clean the repo with cargo clean
If you have the issue with fixed crate that need rustc version 1.79.0 execute this command: cargo update -p fixed --precise 1.27.0
To run everything:
CELESTIA_LOG_LEVEL=FATAL nix develop --extra-experimental-features nix-command --extra-experimental-features flakes --command bash  -c "just suzuka-full-node native build.setup.eth-local.celestia-local.indexer.hasura --keep-tui"

When the Hasura server is started, open a browser at this url: http://localhost:8085/console
To add the Suzuka Tx db to Hasura:

go in Data (top of the windows)
Connect database
Connect existing database
Put a name to Database name
Select database Url
set Database URL to postgresql://postgres:password@database:5432/postgres
Validate with connect dabase.
Under Datamanager select your database-> public
All the indexer table appears. Select track on the transactions table for example.
Go under API
Type this query

query MyQuery {
 transactions {
   hash
   success
   type
   version
 }
}

To see all Tx in the Db.

To run the indexer e2e test:
CELESTIA_LOG_LEVEL=FATAL nix develop --extra-experimental-features nix-command --extra-experimental-features flakes --command bash  -c "just suzuka-full-node native build.setup.eth-local.celestia-local.indexer.hasura.indexer-test --keep-tui"

Outstanding issues
I start only one indexer processor: dedfault_processor. I'm going to see how to start more processor to populate more data in the db.
Available processors are:
    AccountTransactionsProcessor,
    AnsProcessor(AnsProcessorConfig),
    CoinProcessor,
    DefaultProcessor,
    EventsProcessor,
    FungibleAssetProcessor,
    MonitoringProcessor,
    NftMetadataProcessor(NftMetadataProcessorConfig),
    ObjectsProcessor(ObjectsProcessorConfig),
    StakeProcessor(StakeProcessorConfig),
    TokenProcessor(TokenProcessorConfig),
    TokenV2Processor(TokenV2ProcessorConfig),
    TransactionMetadataProcessor,
    UserTransactionProcessor,
    ParquetDefaultProcessor(ParquetDefaultProcessorConfig),
    ParquetFungibleAssetProcessor(ParquetFungibleAssetProcessorConfig),

May need: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/libraries/libelf/default.nix

I've added an e2e test. For Indexer, I query the DB to see if there's some Tx. For Hasura, I just test if the console is available. I do some search to see how to configure Hasura and declare the indexer table to be able to graphQl query the transaction table, but I didn't see an obvious way, so I test the DB and the presence of Hasura.

@musitdev it looks like the indexer test has a legitimate error: https://github.com/movementlabsxyz/movement/actions/runs/10267756524/job/28452717854?pr=295

Ah yes, I didn't see. It's caused because Postgres db is not started, but I don't see why. The indexer depends on it in process-compose, but I don't see the logs of its start. I try to restart it.

The test pass after restart.


fix(protocol): add 1 to _REENTRY_SLOT in EssentialContract

Change _REENTRY_SLOT from 0xa5054f728453d3dbe953bdc43e4d0cb97e662ea32d7958190f3dc2da31d9721a to 0xa5054f728453d3dbe953bdc43e4d0cb97e662ea32d7958190f3dc2da31d9721a + 1
Based on https://github.com/code-423n4/2024-03-taiko-findings/issues/11

Emmm looks like we need to upgrade every contract we deployed both on L1 / L2 for this change...

We don't have to apply the change anytime soon. The previous code works fine.


[Feature Request] Better support for action masking for vectorized environments

Motivation
Stable-baselines3 (SB3) has introduced support for action masking (see here), which is a great feature. However, this API requires the user to provide an ActionMasker wrapper. The issue is that some environments (e.g., gym-microrts, pettingzoo) directly provides a vectorized interface, so there is no chance to apply this ActionMasker.
Feature
Extending MaskablePPO to work with vectorized environments natively. With this extension, SB3 + PPO + Mask can work in gym-microrts and pettingzoo.

@vwxyzjn

Hello,
this is a duplicate of https://github.com/Stable-Baselines-Team/stable-baselines3-contrib/issues/49#issuecomment-957629253
we would apppreciate a PR that solves this issue ;)


Please use the new syntax: '--references-files' has been replaced by '--references-file'.

Description
paket.targets uses od syntax: --references-files, which causes an additional log information:

Please use the new syntax: '--references-files' has been replaced by '--references-file'.

Expected behavior
paket.targets should use new syntax
Actual behavior
Old syntax is used, which causes a log message
I will be happy to submit a PR if you agree

Cc @agross
The problem is that this parameter takes multiple values comma separated. I
think plural would be correct.
Am 22.08.2017 9:52 vorm. schrieb "Łukasz Pyrzyk"<EMAIL_ADDRESS>
Description
paket.targets uses od syntax: --references-files, which causes an
additional log information:
Please use the new syntax: '--references-files' has been replaced by
'--references-file'.
Expected behavior
paket.targets should use new syntax
Actual behavior
Old syntax is used, which causes a log message
I will be happy to submit a PR if you agree
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2646, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNOMrXUa2qHqUKLyPC3_KnkOcV67Uks5saojJgaJpZM4O-Pd0
.

The new syntax reports the following: restore packages from a paket.references file; may be repeated
So instead of paket restore --references-files file1,file2,file3 which is a PITA w.r.t. completing **/*paket.references file names on the command line you would paket restore --references-file file1 --references-file file2 --references-file file3.

Ok I actually agree. Just someone needs to do the MSBuild magic here
Am 23.08.2017 6:19 nachm. schrieb "Alexander Groß"<EMAIL_ADDRESS>
:

The new syntax reports the following: restore packages from a
paket.references file; may be repeated
So instead of paket restore --references-files file1,file2,file3 which is
a PITA w.r.t. completing **/*paket.references file names on the command
line you would paket restore --references-file file1 --references-file
file2 --references-file file3.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2646#issuecomment-324387356,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNIgdzc_v42yCZkCZkvxE3AM7cifoks5sbFEBgaJpZM4O-Pd0
.

Yes please send PR :)


move away from onSettled fixes lifecycle issue

onSettled is deprecated and will be removed in the next major version of react query

Merging quickly to test the staging challenge portal site while GenUI is still available


Withdraw error kraken

Running this command on RPI4 results in an error:
docker run --rm -it --env-file=/home/xxx/docker/bitcoin-dca/config/bitcoin-dca.conf jorijn/bitcoin-dca:latest withdraw --all
In KrakenClient.php line 77:
EGeneral:Invalid arguments

Did you take the minimum withdrawal value into account? For Bitcoin, that's currently: 0.00050 XBT.

Just checked, withdrawing 0.00215 XBT pops up the same error:
In KrakenClient.php line 77:
EGeneral:Invalid arguments

Hi @ultrazeus
Too bad. Did you triple check the withdraw description configured in the .env file that is exactly matches the one added to the Kraken website?

Great, that was the glitch. I couldn't make up that this was actually required as it states "This option overrides any configured withdrawal address", assuming that the address I added in the .env file was enough.
The withdrawal was successful, but it did generate a php error:
PHP Fatal error:  Uncaught TypeError: Argument 2 passed to Jorijn\Bitcoin\Dca\EventListener\XPubAddressUsedListener::__construct() must be an instance of Jorijn\Bitcoin\Dca\Component\AddressFromMasterPublicKeyComponent, instance of Jorijn\Bitcoin\Dca\Component\ExternalAddressFromMasterPublicKeyComponent given, called in /app/var/cache/container.php on line 331 and defined in /app/src/EventListener/XPubAddressUsedListener.php:19 Stack trace: #0 /app/var/cache/container.php(331): Jorijn\Bitcoin\Dca\EventListener\XPubAddressUsedListener->__construct() #1 /app/var/cache/container.php(134): BitcoinDcaContainer->getEventListener_XpubAddressUsedService() #2 /app/vendor/symfony/event-dispatcher/EventDispatcher.php(267): BitcoinDcaContainer->{closure}() #3 /app/vendor/symfony/event-dispatcher/EventDispatcher.php(230): Symfony\Component\EventDispatcher\EventDispatcher::Symfony\Component\EventDispatcher\{closure}() #4 /app/vendor/symfony/event-dispatcher/EventDispatcher.php(59): Symfony\Component\EventDispatcher\EventDispatcher->callList in /app/src/EventListener/XPubAddressUsedListener.php on line 19

Thanks! Great that it worked out. I'll leave this ticket open to serve as a work item to clarify the documentation and figure out what caused the given error.


What is pyt_splade?

Where can we install that from?

https://github.com/cmacdonald/pyt_splade
[pyt_splade.png]
cmacdonald/pyt_spladehttps://github.com/cmacdonald/pyt_splade
github.comhttps://github.com/cmacdonald/pyt_splade
Craig
Sent from my iPhone
On 22 Jun 2023, at 20:00, Omar Khattab @.***> wrote:

Where can we install that from?
—
Reply to this email directly, view it on GitHubhttps://github.com/terrierteam/pyterrier_pisa/issues/19, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEXTCQLTEO5LVWSVW5I3HTXMSI45ANCNFSM6AAAAAAZQTZ43U.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks Craig! Hope you're well!

If you're keen on learned sparse stuff, there's also an integration with the pretty extensive lsr package here: https://github.com/thongnt99/learned-sparse-retrieval/blob/main/lsr/transformer.py


Propagate output_struct_name argument to ExtractSegmentationInterface

Motivation
The segmentation output from the Fee Lab has a custom name ("exOut") for output_struct_name, which is not possible to be modified from the interface level. Here I propose to propagate  output_struct_name to the interface level, so that it can be changed during conversion.
Alternatively, I can do this in a custom interface, where the __init__ would also be modified with adding timestamps for the segmentation data.
class ScherrerOphysSegmentationInterface(ExtractSegmentationInterface):
    """Data interface for ExtractSegmentationExtractor."""
    Extractor = ExtractSegmentationExtractor

    def __init__(
            self,
            file_path: FilePathType,
            timestamps_file_path: FilePathType,
            output_struct_name: str = "exOut",
            verbose: bool = True,
    ):

        super().__init__(
            file_path=file_path,
            sampling_frequency=30.0,
            output_struct_name=output_struct_name,
        )

        timestamps = get_timestamps_from_csv(file_path=timestamps_file_path)
        if not calculate_regular_series_rate(timestamps):
            # only use timestamps if they are not regular
            self.segmentation_extractor.set_times(times=timestamps)
        self.verbose = verbose

How to test the behavior?
Show here how to reproduce the new behavior (can be a bug fix or a new feature)

Checklist

[x] Have you thoroughly read our Developer Guide?
[x] Have you ensured the PR description clearly describes the problem and solutions?
[x] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
[x] If this PR fixes an issue, is the first line of the PR description fix #XXX where XXX is the issue number?
[x] Did you update the CHANGLOG.md file on your branch to describe your changes?

Yup, that's good. Always need to keep interfaces up to date with their corresponding extractor arguments.

@CodyCBakerPhD should I modify any of the current tests in test_gin_ophys.py?

@CodyCBakerPhD should I modify any of the current tests in test_gin_ophys.py?

I was wondering that but since the example data has the default name, specifying the keyword there wouldn't effectively test anything.
I suppose what you could do to ensure it's being passed is add a test where the passed group name is incorrect for the example files and use a self.assertRaisesWith to grab whatever error gets raised from ROIExtractors, just to ensure that the argument is getting passed

@weiglszonja Is this good to go?

@CodyCBakerPhD it seems like there are still some tests that are failing, looking into it.

Will approve this once https://github.com/catalystneuro/roiextractors/pull/210 is merged

Great, thank you. Should I go ahead an merge https://github.com/catalystneuro/roiextractors/pull/210?

Great, thank you. Should I go ahead and merge https://github.com/catalystneuro/roiextractors/pull/210?

Yep, any time you're done with a PR you're assigned to and it has approval, feel free to do the merge and branch deletion on your own 😃


Improve reddit video embed

Previously the videos could be played in the browser. Maybe we can do something similar as what reddit does, by adding a big play button on the image ⏯

Also change the icon wallstreetbets if possible. Same as what we do with Twitter users


EE-2260 - Added 'other' and 'unknown'

Branch based on EE-2261, so check commits beyond this ID: only pay attention to those that start with EE-2260. And please do ignore changes of JSON schema markdowns (check commit's name), since these are done automatically when the schemas are modified and encompass hundreds of files.

[ ] Added 'unknown' and 'other' to some AF fields
[ ] Added constraints for required optional fields when the main one is 'other'.

Related ticket: https://www.ebi.ac.uk/panda/jira/browse/EE-2260.

Awesome, thank you very much, @afoix and @malloryfreeberg for your reviews.
Since this branch was based on others, what I imagine I should do is wait for the other PRs to resolve and then merge them sequentially. I'll keep this open till then.


cmd: Modify structure of cmd directory

Split every command's flags into cmd file, and check if validate in pre-run stage.

How to build binary without a main package? Maybe we should move to cmd/qsctl package and add a main.go here?


How to use I'm not a developer

Help...did anyone know how to use it

Ok got it just using docker

Please tell me how. I still cant figure out


no benchmark/readme and the link in the root readme is broken

As title 🐙

Thanks, fixed.

Oops, it seems still broken...

I forgot to build the readme, updated now.


Enable Rabit test.

Closes https://github.com/dmlc/xgboost/pull/5031.

Closing to avoid spamming the list.
@chenqin Could you please help taking a look into #5031 when time allows?  I can't push to your branch as you are using master.


Reduce change of GL state when binding textures

reduce the change of GL state when binding textures by using a static variable that tracks the currently bound texture and its slot.
basic mechanism in shader in order to track the currently available slot when sending texture uniform to it.
use a flag for manual deletion when texture is created from a thread, automatic when created from main thread and nothing specified

Interesting, this isn't exactly what I imagined but it's better in some ways!
This ends up doing some extra work in Texture::bind(); It looks like even if a texture is already bound to the desired texture unit the method will make it the active texture unit and call bind again.


Editable - Wrap with ReadOnlyContext, ComposingContext, DecorateContext

The Editable component needs to be wrapped with ReadOnlyContext, ComposingContext, and DecorateContext providers.

This seems to be working.


Waveterm should respect user's login shell (or at least $PATH) (Mac specific)

Describe the bug
Waveterm always start /bin/bash as its base shell (see electron/emain.ts). If there is a different bashon the system, e.g. in /opt/homebrew/bin/bash, it is ignored. Also, the $PATH is ignored.
To Reproduce
Steps to reproduce the behavior:

Change user's login shell to /opt/homebrew/bin/bash
Start Waveterm
echo $BASH_VERSION
Version 3 bash is used.

Expected behavior
Waveterm should either

Use dscl . -read /Users/$USER UserShell to determine the user's login shell, and start that.
Find bash by looking at $PATH

Screenshots
Desktop

MacOS
Version v0.5.1

Additional context

you're 100% right.  obviously an oversight and will try to fix this ASAP.  using bash v4 or v5 is actually better for Wave as well.  thanks for submitting.

you're 100% right. obviously an oversight and will try to fix this ASAP. using bash v4 or v5 is actually better for Wave as well. thanks for submitting.

It would be interesting to understand what we give up by not using bash over zsh.

you're 100% right. obviously an oversight and will try to fix this ASAP. using bash v4 or v5 is actually better for Wave as well. thanks for submitting.

It would be interesting to understand what we give up by not using bash over zsh.

We plan on supporting zsh in the future.  Currently the issue around using zsh is our syntax parser (which power command completion and substitutions) is bash syntax specific.  Also the code that reads/restores the environment is bash specific.  We need to make modifications to support zsh's variable/function syntax.


docs(ee/vitals) add vitals 0.31 OAS file

This adds the Kong Vitals OpenAPI spec file for Kong EE 0.31.

Hey @travispamaral, not sure who reviews these, but I saw you as a reviewer on the last few PRs so figured I'd assign you to start!

 Vitals doc on getkong.org should link to publicly-accessible Vitals API Swagger doc


Add breakpoint range media query mixin

I think there should be a mixin to allow for a range - there should be an easier way than this: @media (min-width: breakpoint-min(sm)) and (max-width: breakpoint-max(md)) {}

Well, there is
@include media-breakpoint-up($name, $breakpoints) {
  @include media-breakpoint-down($name, $breakpoints) {
    @content;
  }
}

But your suggestion seems reasonable IMO.

Fixed by #17317.


ForbiddenError: User is not authorized to access this resource with an explicit deny

Versions

prismic-ts-codegen:
node:

Reproduction

Additional Details

Steps to reproduce
follow steps at loading-remote-models-with-the-custom-types-api
What is expected?
generate TypeScript types based on your Custom Type into output: "./types.generated.ts",
What is actually happening?
url: 'https://customtypes.prismic.io/slices',   response: {     message: 'User is not authorized to access this resource with an explicit deny'   }

I was able to fix it. You just need to remove the "Bearer" tag.
const config: Config = {   output: "./types.generated.ts",   repositoryName: repositoryName,   customTypesAPIToken: accessTypes,   models: {     files: ['./customtypes/**/index.json', './slices/**/model.json'],     fetchFromRepository: true,   }, };
The documentation in the Custom Type API tab was a bit confusing and I ended up adding the bearer tag.


Makefile: do not remove empty files from sys/*/gen

The existing "make clean" removes all directories with generated files,
even "empty.go" which are tracked by git.
This relaxes the clean rule to keep empty.go to stop these warnings:
|| package github.com/google/syzkaller/tools/syz-stress
||      imports github.com/google/syzkaller/pkg/csource
||      imports github.com/google/syzkaller/pkg/mgrconfig
||      imports github.com/google/syzkaller/sys
||      imports github.com/google/syzkaller/sys/windows/gen:
build constraints exclude all Go files in
/home/aik/p/go/src/github.com/google/syzkaller/sys/windows/gen
Fixes: 95777977ade5 ("sys: commit empty source files")
Signed-off-by: Alexey Kardashevskiy<EMAIL_ADDRESS>
Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Now I realized the last time I used make clean is... I don't even remember when :)

Now I realized the last time I used make clean is... I don't even remember when :)

Well, just recently syzkaller started crashing on my setup and I was bisecting, and while bisecting I went too far and started seeing these messages :-) Otherwise "make clean" is not horribly useful, true.

ah, makes sense
syzkaller bisection feature uses git clean -fdx, though one needs to be careful with it as it wipes all unrelated files.


add 'save_data' parameter to vak.cli.summary; add option to config

Really only relevant for 'learncurveso maybe add option inlearncurve` config section as in #21

Fixed by #43


Navigate() Question

Scott, Love and appreciate all the work you have done for so many years for the community. I feel I owe you more than a few pints. I am having my first go with the drag and drop and am super excited at the potential. I am stuck however on selecting an item to edit.
I have added the button in the item template as you described and it does update the text box with different GUIDS, however it does not seem to trigger the button action.
I have the button on the screen and it does fire a Navigate() correctly using the selected GUID, but only if you select it outside the dnd container.
I am still cutting my teeth on Power Apps and would appreciate and direction you may be able to provide.

Found a way to accomplish what I needed by adding the Navigate() code to the Master Zone onAction event. Not sure if that is the recommended or most elegant method, but it works nicely.

Hi @kipsmith1037 - thanks for the info and the update. The OnAction event will only be fired in the master zone - so I think what you are describing is correct!


chore: 5396 Upgrade to React 18

Reason for Change

#5396

Changes

React 18 offers various benefits such as better performance from batching updates and StrictMode to assist bug detection. Further reading see here

Update Datasets page Gene Count slider to use styled instead of makeStyles() since makeStyles is not React 18 compatible (CC: @MillenniumFalconMechanic @frano-m hope this is okay!)

Remove @mui/styles dep as a result of above point

Update bunch of HTML invalid attributes in our SVG in .tsx files. E.g., changing stroke-width to strokeWidth to comply with React's HTML attribute equivalent names

Enable React StrictMode to catch bugs and errors

Update frontend/src/views/Collection/components/Toast/index.tsx to NOT use Toaster.create() since it's not compatible with React 18. Instead, we explicitly add a new <div /> in _document.tsx and query the DOM to use React 18's createRoot() to attach Blueprint's Toaster component instance on it. I confirmed that adding invalid genes via comma separated list in GeneExpression results in showing error toaster as expected (CC: @seve in case I missed something!)

WARNING:

Pinning<EMAIL_ADDRESS>to avoid "placeholder" false error

https://github.com/creativetimofficial/material-tailwind/issues/528

Testing steps

I went to https://pr-6406-frontend.rdev.single-cell.czi.technology/ and tested the following pages:
Homepage, Collections, Datasets, GeneExpression, CellGuide, and Documentation - all seem to work compared to prod!

Checklist 🛎

[ ] Add product, design, and eng as reviewers for rdev review
[ ] For UI changes, add screenshots/videos, so the reviewers know what you expect them to see
[ ] For UI changes, add e2e tests to prevent regressions

Notes for Reviewer

Thanks so much everyone 😄 🙏 Will merge this now!


Is it possible to search through a list of objects?

One of the big reasons why I use Fuse is that I can pass a list of objects and it will search on a subset of keys. This requires additional indexing, I assume. Is it possible to do this with uFuzzy? It seems to only accept string[]

out of the box, no. but with bit of cleverness, yes.
the idea is to create a string[] by concat'ing the fields you're interested in using some rarely-used char like ¦. once you have this list you can use uFuzzy as normal.
it would take a bit more code to get the highlight ranges (info.ranges) properly partitioned based on offsets from prior ¦, but may not be necessary if you don't intend to do range highlighting in the UI.
this use case is on my demos TODO list, PRs welcome ;)

i should add that this strategy will only work in an AND fashion for multi-term needles.
if you want to do OR or  more advanced things, you'll probably want to make a different haystack array for each field and run separate uFuzzy matches for each haystack (maybe with different opts). then combine and "re-rank" matches that overlap, etc.
depending on what you need, at some point it may make sense to use a proper document search lib with an inverted index, etc. given its poor performance, Fuse wouldn't be my first choice tho.

I am actively looking for an alternative to Fuse, so I am certainly interested. To add a bit more context, It's not a complex search where something like Elastic would be applicable. My use case (btw, happy to be told I am doing it wrong) is something like, you fetch a set of records, maybe virtualized on the UI. Client side search to quickly filter the list down to what they need.
type RadioStation {
  callLetters: string
  title: string
  hostName: string
}

Say I want to search "Jim" on an array of the above structure. It should return because hostName has a "Jimmy Neutron".
Hopefully this isn't annoying, I'd love to use something besides Fuse.

something like this (incomplete):
const uFuzzy = require('../dist/uFuzzy.cjs');

let recs = [
    {
        foo: "Martha came to the conclusion that shake weights are a great gift for any occasion.",
        bar: "his seven-layer cake only had six layers.",
        baz: "The urgent care center was flooded with patients after the news of a new deadly virus was made public.",
    },
    {
        foo: "I never knew what hardship looked like until it started raining bowling balls.",
        bar: "It didn't make sense unless you had the power to eat colors.",
        baz: "For oil spots on the floor, nothing beats parking a motorbike in the lounge.",
    },
    {
        foo: "They were excited to see their first sloth.",
        bar: "The rusty nail stood erect, angled at a 45-degree angle, just waiting for the perfect barefoot to come along.",
        baz: "She wanted a pet platypus but ended up getting a duck and a ferret instead.",
    },
    {
        foo: "They say that dogs are man's best friend, but this cat was setting out to sabotage that theory.",
        bar: "He poured rocks in the dungeon of his mind.",
        baz: "He was willing to find the depths of the rabbit hole in order to be with her.",
    },
    {
        foo: "When I was little I had a car door slammed shut on my hand and I still remember it quite vividly.",
        bar: "She advised him to come back at once.",
        baz: "A purple pig and a green donkey flew a kite in the middle of the night and ended up sunburnt.",
    },
    {
        foo: "She looked at the masterpiece hanging in the museum but all she could think is that her five-year-old could do better.",
        bar: "We should play with legos at camp.",
        baz: "Trash covered the landscape like sprinkles do a birthday cake.",
    },
    {
        foo: "It was obvious she was hot, sweaty, and tired.",
        bar: "As he waited for the shower to warm, he noticed that he could hear water change temperature.",
        baz: "It took him a while to realize that everything he decided not to change, he was actually choosing.",
    },
    {
        foo: "She moved forward only because she trusted that the ending she now was going through must be followed by a new beginning.",
        bar: "Fluffy pink unicorns are a popular status symbol among macho men.",
        baz: "I am counting my calories, yet I really want dessert.",
    },
    {
        foo: "Happiness can be found in the depths of chocolate pudding.",
        bar: "Strawberries must be the one food that doesn't go well with this brand of paint.",
        baz: "Joe made the sugar cookies; Susan decorated them.",
    },
    {
        foo: "Poison ivy grew through the fence they said was impenetrable.",
        bar: "Nothing is as cautiously cuddly as a pet porcupine.",
        baz: "Dan ate the clouds like cotton candy.",
    },
];

let haystack = recs.map(r => `${r.foo}¦${r.bar}¦${r.baz}`);

let needle = 'was';

let uf = new uFuzzy();

let idxs = uf.filter(haystack, needle);

console.log(idxs.map(i => recs[i]));

Thanks for answering!

@leeoniya  i think the other part of doing objects is that you naturally want some fields to rank higher than other fields, so there's enough here that it should be a feature IMO

@sw-yx you should be able to use info.ranges to work out which portions of each match land within which segments, then use this to help with sorting.
of course each match can be cross-field when they're all concatenated. so you may want to run matches on each field as a separate haystack, then combine the results in the order of your most to least important haystack.
feel free to submit a demo if you get something interesting working. kinda related to https://github.com/leeoniya/uFuzzy/issues/19.
uFuzzy does not work with objects so any kind of object search would have to be a wrapper and not a core API.

thanks for the pointer - i have this in a todo issue right now.. not sure if i'll get to this but hopefully when i have some downtime

@leeoniya it would be great to add something like this https://github.com/leeoniya/uFuzzy/issues/7#issuecomment-1276777783 to documentation/readme!
My case is very similar to @christopher-caldwell case: want to migrate search from using lunrjs to something better... saw this project... and than... spend hours searching here and there for the answer to "how can I use uFuzzy with list of objects?"
I think many developers using lists of objects...

@leeoniya is it possible at least have some way to use objects as is and use an option to access the object's key we want to search in ?
like we have this:
const a = [
  {
     name: 'foo'
  },
  {
    name: 'bar'
  }
]

now having thousands of these, is it still efficient to map them ? wouldn't just passing the object to uFuzzy and then specifying name as key for search be more efficient ?

the overhead should be very small. the problem is that this approach becomes not useful when you have more than one property to search, or deep objects. the flat string array is universal.

I see
you are right


Japanese patch #4905

I added Japanese translation.

@freerdp-bot test

@freerdp-bot can you test that please ?

@freerdp-bot test

Ok, looks like you have a typo in the xml, please check that.

@freerdp-bot test

Thank you for this fix.


Python 3.9 import of rdkit.Chem fails with undefine symbol ImportError

Describe the bug
After a build from source and appropriate PYTHON_PATH and LD_LIBRARY_PATH setting
import rdkit works, but a subsequent import rdkit.Chem fails with
>>> import rdkit.Chem
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ludwig/git/rdkit/rdkit/Chem/__init__.py", line 20, in <module>
    from rdkit.Chem import rdchem
ImportError: /home/ludwig/git/rdkit/rdkit/Chem/rdchem.so: undefined symbol: _ZNK5RDKit8RingInfo17numRelevantCyclesEv

To Reproduce
Clone rdkit from github on debian bullseye. Satisfy requirements from the official debian packages and compile with cmake.
make install and import rdkit and rdkit.Chem
Expected behavior
Error free import
Configuration (please complete the following information):

RDKit version:
master git-hash e21e2c2af6b47f45d15ed3dbd3c70bd12b589077 (and some earlier versions)
OS:
debian bullseye
Python version (if relevant):
python 3.9.1+
Are you using conda?
No
If you are not using conda: how did you install the RDKit?
Compilation from source

Additional context
ldd of rdkit/rdkit/Chem/rdchem.so` reports no missing dynamic libraries.
objdump -T rdchem.so | grep _ZNK5RDKit8RingInfo17numRelevantCyclesEv
0000000000000000      DF *UND*<PHONE_NUMBER>000000              _ZNK5RDKit8RingInfo17numRelevantCyclesEv

The impression here is that rdchem.so is accessing a different version of the C++ library at runtime than the one it was built against. Try to check with ldd that the C++ libraries rdchem.so looks into are actually those you have just built.

Excellent.
Yes, that was indeed the problem.
It was binding against an old system wide installation.
Thanks a lot.


BlankZero and Editable modification are not detected as "not supported"

Please include the following with each issue:
1. Describe the bug
I was transforming code from C/AL to AL and neither the txt2al (it's okay) or the AL vsx did error (or at least warn) me about the following:

BlankZero modifications on a existing field inside a TableExtensions are not supported
Editable modifications on a existing part inside a PageExtensions are not supported

Only the publishing of the app has told me:
Reason: An error occurred while applying changes from the object of type 'PageExtension' with ID '50000'
in the 'xyz by xyz <IP_ADDRESS>' app to the application object of type 'Page' with the ID '5725'. Failing delta:
Type: ContentChangeDelta; TargetKind: FactBoxes; TargetId: Control1905767507; TargetType: ControlBaseDefinition
The error was: InvalidOperationException - Metadata delta application failed due to the following error(s):
Modifications to property Editable are not supported.

2. To Reproduce
Steps to reproduce the behavior:

Create TableExtensions and modify BlankZero of a base field.
Create PageExtension and modify Editable of a base part.

3. Expected behavior
VS Code should warn/error/tell me.
4. Actual behavior
Nothing.
5. Versions:

AL Language: v6.6.434101
Visual Studio Code: Version: 1.54.3
Business Central: 17

Maybe those changes should be OK, though? They can't affect the schema and are really just cosmetic (in fact I wonder if BlankZero even belongs on a table, other than for convenience!)

Is there any Progress on this feature?
I had the same problem with the Modifaction of the Editable Property.
Thankfully this error is recognized on publishing the extension in our buildpipeline resulting in the following error.

##[error]An error occurred while applying changes from the object of type 'PageExtension' with ID 'XXX' in the 'AppName publisher <IP_ADDRESS>60' app to the application object of type 'Page' with the ID '7374'. Failing delta: Type: 'ContentChangeDelta'; TargetKind: 'FactBoxes'; TargetId: 'Control2'; TargetType: 'ControlBaseDefinition'; PropertyNames: 'Editable'; The error was: InvalidOperationException - Metadata delta application failed due to the following error(s): Modifications to property Editable are not supported.

Same issue here with v20.1.ES SaaS.
CodeCops don't rise any error or warning...
I can build the .APP but surprisingly I can't deploy it.
Please, allow this type of changes or at least add it to CodeCops.

Closing as duplicate of #7670 which is considered for triage.

@qutreson is the "Editable" property for PageExtension parts also covered by the other issue? Or should I create a new issue for that?

@dannoe - We will cover both


Localization not working

OS: Ubuntu 16.04.1 LTS (kernel 4.10.0-32-generic)
Ver: DBeaver-CE 4.1.3 (Aug 21, 2017)
Problem:
When I use DBeaver, I noticed that the whole software are displayed in English, but in Preferences->Database->Result Sets->Data Formats->Locale it's setted zh - Chinese. I tried run 'dbeaver -nl cn' and 'dbeaver -nl zh' in shell but this won't work as well.
Does DBeaver-CE in Ubuntu contains localization? if it does, how to activate it?

just noticed that it's mentioned in https://github.com/serge-rider/dbeaver/issues/1867 that localization is wiped out in 4.x


Optimization for state

As discussed in the excel sheet with @TheForeverLost (upload it here as well). We need a way to log the previous values of modified data in the StateSet so that it can be used to undo the values in the Unity side of things to avoid the dramatic increase in StateLog object. (can be done by just added one prev_val field in the assign types state set or something more generic if possible).

Added details required here


🔥 Remove Newtonsoft.Json dependency

✨ What's this?
This PR removes the NuGet dependency on Newtonsoft.Json.
🔗 Relationships
Makes #57 obsolete.
🔍 Why do we want this?
It is no longer used.
🏗 How is it done?
Select. Delete.
💥 Breaking changes
Might break libraries and applications that depended on Newtonsoft.Json indirectly through Bearded.Graphics.
💡 Review hints
Confirmed that this works in TD.

depended on Newtonsoft.Json indirectly through Bearded.Graphics

Can you even do that?


why cache/xxxxxxxx/video_2_audio.mp3： No Such file or directory

如题

Please describe your problem in detail

Please describe your problem in detail

``

这个是ffcreator 启动后报错详细内容

Please describe your problem in detail

这个是creator层还是ffmpeg层出现问题？

且我只有一次FFVIDEO，为什么id会出现2？

Please describe your problem in detail

这个问题，发现是偶现，大概是什么原因呢？

I think you manually deleted the cache folder,
You can upload your project to gh. If you want to process a large number of videos, you can use ffcreatorlite


update electron to v 1.3.3

See changes here: https://github.com/electron/electron/releases/tag/v1.3.3

The current semvar allows the latest major and patch versions to be installed.


Missing Scenario: AD Group Based TeamsMeetingBroadcastPolicy

The article: https://docs.microsoft.com/en-us/microsoftteams/teams-live-events/set-teams-live-events-policies-using-powershell
Should be updated to include a new scenario under the first three scenarios, probably titled: “You want to disable live event scheduling for a large number of the users and allow a group of users to schedule them”
With a link to the Teams New-CsGroupPolicyAssignment Support document (https://docs.microsoft.com/en-us/powershell/module/teams/new-csgrouppolicyassignment?view=teams-ps) or an example like the below:
“
New-CSTeamsMeetingBroadcastPolicy -identity EnableBroadcastSchedulingPolicy
Set-CsTeamsMeetingBroadcastPolicy -identity EnableBroadcastSchedulingPolicy -AllowBroadcastScheduling $true
New-CsGroupPolicyAssignment -GroupId <GROUPID/UPN> -PolicyType TeamsMeetingBroadcastPolicy -PolicyName EnableBroadcastSchedulingPolicy -Rank 1 -Verbose
“

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

ID: 8cb395f1-fce4-2d64-c66e-36a6cd818ca6
Version Independent ID: 79bf1d7e-8cbc-5146-dfda-a6ae38d49be7
Content: Use PowerShell to set live events policies - Microsoft Teams
Content Source: Teams/teams-live-events/set-teams-live-events-policies-using-powershell.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@Jaredsfs1 Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@scanum this can be closed thank you.


No netlify edge function deployed with default project

Qwik Version
0.12.1
Operating System (or Browser)
chrome
Node Version (if applicable)
No response
Which component is affected?
Qwik City
Expected Behaviour
a netlify edge function is deployed
Actual Behaviour

Additional Information
There's no endpoint accessible/available after netlify deploy - adapter is installed

Hi @gkatsanos
Thanks for raising this issue 🙏
I just gave it a try on my side doing these steps:
yarn create qwik
cd <app dir>
yarn qwik add netlify-edge
git ....

selected the repo on netlify and published it successfully to the edge.
fn: https://warm-dasik-ac2b35.netlify.app/
repo: https://github.com/zanettin/qwik-netlify
would you mind to give it another try on your side?
cc: @adamdbradley @manucorporat

@zanettin try to create an endpoint and access it through your app..

oh sorry! sure will test that as well. thought you mean with endpoint any route 👍

hello again
can confirm the endpoint issue on netlify 👍
also tested cloudflare to see if it is a general issue with edge functions which is not the case: https://qwik-netlify.pages.dev/api
thanks again for reporting and sorry for the initial misunderstanding. hope @adamdbradley has an idea 🙏


Not an issue. Just a notice to a bounty.

A bounty has been out for MKDD and is at $3,000 in order to improve the LAN mode for MKDD. And thought it might interest you.
Regardless I plan to work towards it myself. However, my understanding of assembly is nonexistent. I am working towards understanding it. There are great guides out there. But I started the bounty with the hopes that someone more knowing of the process would beat me to a solution. Either way I refuse to claim the bounty as I am the one who started it. making it a clear conflict of interest, lol.
Whether or not is fine. As mentioned, I plan to keep working on it. and I may add to this once I become more comfortable with the process. I just brought it up in case the bounty might interest you. Feel free to ask questions if you have any.
Thanks.
https://app.bountysource.com/issues/91924449-a-fix-to-the-lan-code

I've been aware of the bounty however I'm not interested myself in going through the process of creating a patcher, rewrite the code (etc. etc.). I hope my decomp will be of any help when I'm going to start working on the LAN mode side.

ah ok. sorry to bring it up again then.

@SwareJonge hey there! Have you gotten to any LAN stuff yet? 😊

Fixing the LAN Code isn't a goal for this project and will never be, there are other projects that'll try to tackle this.


Improve Contributor Experience

Checking the patch commit message function as well.

@stathismor took that screenshot straight from the release party actually. With Alex M's comments, it going to go anyways. Commit coming right up.,

Thanks Versionbot, want more people's eyes on this before merging.
cc: @pdcastro when you have time.

@balena-ci I self-certify!

@balena-ci rebase

@balena-ci retest

@balena-ci rebase


Request: Add deploy to Azure feature

Purpose
In order to easily trial the product, it would be good to be able to easily set up a version to click around.
Desired solution
Restructure the project for images and add Azure deployment scripts
Alternatives considered
No response
Additional context
No response

Fixed the existing template to work on any tier.


Add sample_query_options to more functions

Re: issue #635

I'm starting to suspect changing these functions to kwargs-only was unnecessary, and it risks being incompatible with existing calls, although it does seem in keeping with the approach/style elsewhere in the package, and I suppose function calls can be updated... 🤔

I'm starting to suspect changing these functions to kwargs-only was unnecessary, and it risks being incompatible with existing calls, although it does seem in keeping with the approach/style elsewhere in the package, and I suppose function calls can be updated... 🤔

The approach I've taken so far is that private functions should be kwargs-only wherever possible, to help avoid bugs introduced by changing the order of parameters in a function call. But public functions can accept positional args.
If we changed public functions to kwargs-only we'd likely break some user code. I was tempted to do it a while back, but then thought better of it when I realised I do tend to use positional args when doing data analysis.

Thanks @alimanfoo . I'll amend this PR accordingly.


Expose publicly internal scripts/package.js for Monorepo maintainers

What problem does this feature solve?
As a monorepo maintainer, I want to normalize/synchronize fields in my packages’ package.json. AND Nuxt.js’ does it already. But it's hidden, and inaccessible for other monorepo maintainers.

This started based on a twitter thread
And also on RushJS.io's (a monorepo manager, like Lerna. But leveraging PNPM)  Gitter (link to channel)

What does the proposed changes look like?
See nuxt/nuxt.js#6373

This feature request is available on Nuxt community (#c9741)

It may be worth checking https://github.com/danielroe/siroc. I think we'll close this as we have moved to a different monorepo setup and open-sourced https://github.com/unjs/unbuild instead.


fix(job-form): fix parameter header

I accidentally took the header verbatim from the design, but Monitoring shouldn't be part of the header, so I simplified it to just Parameters, so that it applies everywhere.

FYI: approved because I already knew you were in the process of adding i18n.t() 😉

FYI: approved because I already knew you were in the process of adding i18n.t() 😉


Add onDidChangeConfiguration activation event

We already have workspace.onDidChangeConfiguration, however, it is impossible to activate the extension on changing configuration.
A workaround is to use onCommand:workbench.action.openSettings. However, this approach has some drawbacks described here. There is an edge case when the user has "Settings" page opened, the extension won't be activated. Besides it's not limited to the extension's config.

Can you please mention the use case to activate the extension on configuration change?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Flood protection triggers on the when sending to sp.php

It is probably because the browser sends first an HTTP OPTIONS to the server and then after it verified the CORS, it sends the actual HTTP POST. Maybe that causes a problem with the flood protect.
Here are the connections with the HTTP 429 error response:

One solution would be to only count HTTP POST or HTTP GET for the flood protection or double all values / have even values.
Edit: It still answers HTTP 429 after replying a second time 30 seconds after the first post.

Ive pushed a fix for this, and increased the default settings. These can be tweaked later if need be.

There is still something not right. It either triggers on normal use or doesn't reset.
Try testing the flood protection on its own.

Yup, you are correct, reopening. Ill have to look into this tomorrow.

Thanks.
I just used this one and verified that it saved a file to a /tmp path on an tmpfs and tested the triggering.

This should be good now. I'll leave this open for a while just in case it isn't.

Tested it a few times and it worked. Should be fixed then.

I was too early: After posting to a new thread it triggered it again.

Woops, I was writing the checks for submitting and quoting to the same file, so if one would try and grab a post multiple times, it would blast past the flood check for submitting too. This should be good now.


python310Packages.teslajsonpy: 3.7.4 -> 3.7.5

Changelog: https://github.com/zabuldon/teslajsonpy/releases/tag/v3.7.5
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 222602 run on x86_64-linux 1

2 packages built:

python310Packages.teslajsonpy
python311Packages.teslajsonpy


Add Karma configuration from Angular plugin configuration

Angular allows running tests using Karma via the @angular-devkit/build_angular:karma builder. However, configuration in Angular CLI workspace (aka angular.json) using that builder to run tests using Karma isn't taken into account as Karma configuration. But this is something that comes built-in when creating a fresh Angular project. This PR aims to take into account that builder configuration for knip.
To do so:

Karma plugin has been refactored: so that common functionality shared by both plugins lives in helpers.ts. In the same fashion that XO uses some ESLint helpers. The code to load a configuration file has been moved there too to keep clean the index.ts file.
Includes and excludes specified as builder options are used as included/excluded non-production entry points. If unspecified, Angular CLI defaults are used. Which actually come from the JSON Schema. Will try and see if can the script can be updated to obtain those values too.
Karma config file specified as karmaConfig in builder options is used to:

Load default Karma config if not specified
Load an alternative Karma config file if specified, but not one that Karma plugin would take by default.

Great, plugins use each other all over the place and it's chaotic and beautiful :)


Update package.json to fix ccache_enabled option, fixes #43633

Summary:
I found in 0.74.0-rc.x, ccache_enabled is introduced. However, it is not being delivered via npm.
fixes #43633
Changelog: [iOS] [Fixed] - Adding ccache_clang wrapper scripts to package.json for distribution
Test Plan
I ran the build with ccache_enabled to reproduce on 0.74-rc.4:
diff --git a/ios/Podfile b/ios/Podfile
index 9416525..faec5a7 100644
--- a/ios/Podfile
+++ b/ios/Podfile
@@ -34,7 +34,7 @@ target 'Newish' do
       installer,
       config[:reactNativePath],
       :mac_catalyst_enabled => false,
-      # :ccache_enabled => true
+      :ccache_enabled => true
     )
   end
 end

This fails at build in Xcode:

After applying the fix (npm publish --dry-run 2>&1 | grep "scripts/xcode") and confirming by replicating this in the above project, the builds was successful:

CLA signed... not sure why it is not updated

Thank you for this fix.

Added test plan notes I used to validate the fix (locally) to the summary.


[DevCenter] Weekly tests failure

Library name
Azure.Developer.DevCenter
Please describe the feature.
See here for more details:

DevCenter weekly test pipeline: https://dev.azure.com/azure-sdk/internal/_build?definitionId=5624&_a=summary

@pallavit @ShivangiReja looks like the issue is that tests-weekly isn't pulling in config from our variable group, Azure Dev Center Team Live Test Secrets.
@benbp do we need to take any additional steps to link that variable group to "net - devcenter - tests-weekly"? It's already connected to "net - devcenter - tests".

@sebrenna I just hooked up the weekly group, it should be good now.

The pipeline https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2019747&view=results is now green. Thank you @benbp!


Public folder clean up

... movapp-cover a movapp-bg jsem dal zatim taky do public/icons

Could have moved images that are not icons into a separate images folder, but we can sort that out later. Good job, merging :)

Good job @jejdacz 👍
Just for don't break public URL, I added two redirects here https://github.com/cesko-digital/movapp/pull/57 for https://www.movapp.cz/movapp-cover.jpg and https://www.movapp.cz/omalovanky.pdf
If anyone linked directly to our PDF file, the link will still work with redirect.
And the same for that cover image, that makes preview on social networks.
Rest of the files are ok without redirect (they are not suppose for public linking from outside world).

Hi @baryska just for you to know, that your PDF file for kids was moved.
Redirect make the old link work too.


Services: show information about consents given in user understandable form

Instead of API endpoints or abstractions, tell the user exactly what information the service uses about the user: name, email etc.
This obviously depends on the service.
Tunnistamo uses two different ways to do this at the moment. Only the other one (OpenID) makes it possible for us to list consents.

Tuomas Haapala is working on improvements that will provide more human-readable information

PR for this is in review in Tunnistamo.
https://github.com/City-of-Helsinki/tunnistamo/pull/74


[release-3.0] cherry pick: Generate flamegraph at runtime

Signed-off-by: Yang Keao<EMAIL_ADDRESS>What have you changed?
cherry-pick: #5697 #5933
What is the type of the changes?

New feature (a change which adds functionality)

How is the PR tested?
They have just been tested manually :(
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No
Refer to a related PR or issue link
#5666
Any examples?
Assume that tikv status_server is listening at localhost:8080
curl "http://localhost:8080/debug/pprof/profile?frequency=50&seconds=10" # this will start cpu sampler at 10/sec frequency and continues 10 secs.

curl "http://localhost:8080/debug/pprof/profile?frequency=50&seconds=10" --header  "Content-Type: application/protobuf" # this will return a pprof protobuf

Notice:
The implementation of rsperftools is not efficient (some performance improvement will land these days). So too high frequency will make it hold lock all the time and may lead to long response time and high cpu usage.
Panic:
Under these situations, rsperftools will crash or deadlock:

profiling tick happens when program is propagating thrown exception

flamegraph.svg.gz

/merge


:sparkles:Add space provider for KCP

Summary
Space provider for KCP runtime
Related issue(s)
Fixes #

/lgtm
/approve


xwayland: 22.1.5 -> 22.1.6

Description of changes
Changelog: https://lists.x.org/archives/xorg-announce/2022-December/003304.html
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Please fix the versions in the commit message.

Doh! Done.


why vits raised grad warn?

UserWarning: Grad strides do not match bucket view strides. This may indicate grad was not created according to the gradient layout contract, or that the param's strides changed since DDP was constructed.  This is not an error, but may impair performance. grad.sizes() = [1, 9, 98], strides() = [29498, 98, 1] bucket_view.sizes() = [1, 9, 98], strides() = [882, 98, 1] (Triggered internally at ../torch/csrc/distributed/c10d/reducer.cpp:325.)   Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
is this normal? how fix it?

This warning appears even if you use the original VITS repo to perform training. It will not cause any trouble or impair performance so simply ignore this.


Revert "Enable and fix all gocritic linters"

Reverts kubernetes-sigs/cri-tools#1506 and https://github.com/kubernetes-sigs/cri-tools/pull/1496
This seems to introduce a bunch of upstream test issues I'll investigate further.

/approve
/lgtm


Stopped users from being able to add duplicate tags

Closes #767
The problem is actually an API problem, but this should work as a stopgap. I'm having trouble recording the new testing fixtures, so this PR will be failing until Monday.

@TeslaDethray looking at CI, it times out when trying to run the commands. This has implied that the command being run in CI is entering an interactive mode. Maybe try running the command in the feature file locally and seeing if it works.

It works. The fixture PHP-VCR is running in CI DNE. If I comment out PHP-VCR var-including lines in the FeatureContext, it succeeds.

:+1:


Support Dynamic Connection String

Hi. thanks for this great library
I have a multi-tenant application and each tenant has its own connection string
how can I handle this?
if currently, the library supports dynamic connection strings, please tell me how

@castell0riz0n
Hi,
For multi-tenant purpose we can approach this way.

Make a directory name it as config
Create a separate local.settings file for each tenant in the config.
Once we have this folder ready Bases on the deployment environment we swap the existing  local.setting.json file with the one from config directory using pipeline line scripts usually yaml.

Thereby, settings can be configured based on pipeline we run.
Hope this helps.


Remove deprecated method [phab:D2227]

Created by schroederc at 2018-03-21 21:57:32:

fromberger accepted this revision at 2018-03-21 21:59:55

schroederc closed via commit 1db6a5a580399fc0eb1134a5c13744564000e0c6 at 2018-03-22 16:42:17


saving file does not save file

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description

Steps to Reproduce

save a .PHP file
make a minor change to the file
save the file

Expected behavior:

Blue "unsaved changes" dot disappears
Any changes to the file should be written to the file
Actual behavior:
Blue "unsaved changes" dot disappears
Changes to the file are not written to the file, as evidenced by examination of copy of the file subsequently sent to remote server via clonezilla.

Reproduces how often:
Intermittent.  Happens at least 10% of the time, sometimes several times in a row.

Versions

Microsoft Windows [Version 10.0.19042.746]
(c) 2020 Microsoft Corporation. All rights reserved.
C:\Users\meow>atom --version
Atom    : 1.54.0
Electron: 6.1.12
Chrome  : 76.0.3809.146
Node    : 12.4.0
C:\Users\meow>apm --version
apm  2.5.2
npm  6.14.8
node 12.4.0 x64
atom 1.54.0
python
git
visual studio
C:\Users\meow>systeminfo
Host Name:                 DESKTOP-S18HU3V
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.19042 N/A Build 19042
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free
Registered Owner:          meow
Registered Organization:
Product ID:                redacted
Original Install Date:     1/29/2021, 9:12:14 PM
System Boot Time:          1/29/2021, 9:35:36 PM
System Manufacturer:       Dell Inc.
System Model:              Latitude E5540
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
[01]: Intel64 Family 6 Model 69 Stepping 1 GenuineIntel ~2701 Mhz
BIOS Version:              Dell Inc. A24, 6/13/2019
Windows Directory:         C:\WINDOWS
System Directory:          C:\WINDOWS\system32
Boot Device:               \Device\HarddiskVolume1
System Locale:             en-us;English (United States)
Input Locale:              en-us;English (United States)
Time Zone:                 (UTC-06:00) Central Time (US & Canada)
Total Physical Memory:     12,193 MB
Available Physical Memory: 2,801 MB
Virtual Memory: Max Size:  39,841 MB
Virtual Memory: Available: 27,768 MB
Virtual Memory: In Use:    12,073 MB
Page File Location(s):     C:\pagefile.sys
Domain:                    WORKGROUP
Logon Server:              \DESKTOP-S18HU3V
Hotfix(s):                 5 Hotfix(s) Installed.
[01]: KB4586876
[02]: KB4562830
[03]: KB4580325
[04]: KB4598481
[05]: KB4598242
Network Card(s):           2 NIC(s) Installed.
[01]: Intel(R) Ethernet Connection I218-LM
Connection Name: Ethernet
DHCP Enabled:    Yes
DHCP Server:     <IP_ADDRESS>
IP address(es)
[01]: <IP_ADDRESS>
[02]: redacted
[03]: redacted
[04]: redacted
[05]: redacted
[02]: Intel(R) Dual Band Wireless-AC 7260
Connection Name: Wi-Fi
Status:          Media disconnected
Hyper-V Requirements:      VM Monitor Mode Extensions: Yes
Virtualization Enabled In Firmware: Yes
Second Level Address Translation: Yes
Data Execution Prevention Available: Yes
C:\Users\meow>
Additional Information

This computer is a laptop running on an SSD.  I'm guessing it's a caching problem.  If it is a caching problem, then maybe it's not an Atom issue, but an OS-level issue, but either way, I don't really care, if you're developing on windows, working around this kinda stuff is part of the cost of doing business.  it would be nice if Atom would do something to churn the cache.
I'll try to update this ticket as I gather more info on the problem.

I had the same issue.
Behaviour
The blue saving dot doesn't disappear for a while. I tried saving tens of times and the application crashes ones in a while. When I reopen the application, the file opens with an old version of data from one year ago(which copy I understand was from .atom/recovery folder. This would have been a serious loss of data if I had not used git to version this file a couple of weeks ago. Another thing that helped with recovery is that the recover folder has a version from last two days. So the data loss is limited to 2 days for me. Others may be unfortunate. See https://github.com/atom/atom/issues/18114 and a lot of similar reported issues.
The crash was reproduced a couple of times, each time the reopening of file takes me to the same old copy.
Observations
I tried the saving the file in safe mode and things seem to work fine. Atom team seems to close issues like this on the basis that it is a package issue. But this caused data loss to me and is a serious issue with Atom, if it allows packages to interfere with saving stuff.
I painfully disabled each package(not exactly each one but a bunch each time), while saving random junk to the same file every time, to check which package causes the issue. The main suspect is metrics package of Atom. The next candidate is tree view.
The reason I could not confirm it for sure is that after disabling metric package, the saving seems to work fine. However after enabling it again, the issue went away for good.
It could be the disabling metrics package which helped or it could be some other package's disabling which took effect by that time.
So for others who face the same issue:

Please try disabling metric package and
Report here if it fixes the issue

<tl;dr> Disabling 'metrics' package and restarting Atom restores expected file saving functionality.
I'm using Atom 1.56.0 on maccOS 10.15.7 Catalina, and experienced same issue when attempting to save files.
Sequence of events:

cmd-n new file
cmd-v paste json
cmd-s save file, dialog opens asking for file name.
enter file name then , or click 'save' button.
dialog closes
file remains unsaved, doesn't exist in the location or with name specified.
new file w/ contents still open in tab; tab top filename shows 'Untitled'

If I cmd-comma open settings --> packages, then search for 'metrics' and disable, then restart Atom, I can repeat the above sequence 1-6, but the file is saved in specified location and with specified name.
No idea why disabling metrics package restores file saving functionality, but it did so for me.

Update over my earlier comment.
The issue of not saving returned even with metrics package disabled. But the frequency is less.
So it looks like disabling any package effects a better behavior as per @jkjgabrielsen 's comment? Or it could be a trivial coincidence.

I am seeing this same issue on both Mac and Ubuntu, with .yaml files.


SyntaxError: leading zeros in decimal integer literals are not permitted

I'm trying to get the current date as a vector, so I'm using Vyxal's kð constant. However, when I try to run the Vyxal program, I get an error from the interpreter:
Traceback (most recent call last):
  File "/home/user/.local/bin/vyxal", line 8, in <module>
    sys.exit(cli())
  File "/home/user/.local/lib/python3.9/site-packages/vyxal/main.py", line 296, in cli
    execute_vyxal(file_name, flags, inputs)
  File "/home/user/.local/lib/python3.9/site-packages/vyxal/main.py", line 164, in execute_vyxal
    exec(code, locals() | globals())
  File "<string>", line 1, in <module>
  File "<string>", line 1
    [08,05,2022]
      ^
SyntaxError: leading zeros in decimal integer literals are not permitted; use an 0o prefix for octal integers

I can reproduce this error when running a simple expression in the Python interpreter:
i = 01

So it looks like Vyxal is creating a string that will then be executed as the program, but it's putting leading zeroes on the numbers. The same error occurs with kN as well

datetime.strftime unfortunately doesn't have a way to format without leading zeroes. Maybe wasn't caught because the element was written after the 10th of October, November, or December :P


Fix triggers for terminatingpod sanity check

In the current implementation, when pods are terminating, the terminating pod sanity check doesn't run because fix failed nodes check fails and the operator doesn't execute other checks. This PR will make sure to run the check.

Sorry for the delay @4n4nd. Apparently I didn't have notifications turned on for this repo. Will take a look this afternoon or maybe tomorrow. Thanks!

No worries! Could you please approve the tests on this one? @cin


Control surface setup

Pitch and roll inputs are getting switched. presumably the control surfaces are using the wrong axis somewhere. Possibly in part setup not code

cause is mirror axis using -ve scaling. Might need to switch materials in an interesting way for mirrored control surfaces

Hey.
I'm a bit new to updating mods through GitHub.  Do you mind giving me a quick tutorial or a link to a good one?
Thanks,
-defarm

Download the zip from the releases page
Install as normal (merge GameData from zip with KSP GameData. If you get "GameData \ GameData", you did it wrong. It should be "GameData/B9_Aerospace_ProceduralWings")

Hey, sweet. Thanks. I had done that earlier, thought I had done it wrong because the textures had flipped, realized I could have just changed them, forgot how to do it, and the rest is history. I'll dedicate a flag to you! Cheers!

fixed by not turning the mirrored wing inside out and handling material switching in code


Download events being omitted on the page level but chromedp listens for it on the Browser level

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.9.5
$ google-chrome --version
Google Chrome 123.0.6312.58
$ go version
go version go1.22.0 darwin/arm64

What did you do? Include clear steps.
I'm using chromedp's file download capability. The current version listens for download events on the browser level but the latest chrome omits download events on the page level

done := make(chan string, 1)
	chromedp.ListenTarget(ctx, func(v interface{}) {
		if ev, ok := v.(*browser.EventDownloadProgress); ok {
			completed := "(unknown)"
			if ev.TotalBytes != 0 {
				completed = fmt.Sprintf("%0.2f%%", ev.ReceivedBytes/ev.TotalBytes*100.0)
			}
			log.Printf("state: %s, completed: %s\n", ev.State.String(), completed)
			if ev.State == browser.DownloadProgressStateCompleted {
				done <- ev.GUID
				close(done)
			}
		}
	})

What did you expect to see?
The channel returning the guid and the program finishing with exit code 0
What did you see instead?
Channel never closes since the download events are omitted in different places.

Needed the WithEventsEnabled(true)


Asymetrical interfaces between the query() methods

There seems to be an asymetrical interface between the query() method on the global sql instance and the query() method on a Request instance.
This prevents the use of a string variable as parameter of the query() method on the global sql instance. Only a tagged template literal seems to work.
See below for an example of the four different ways to query, of which 3 work and one doesn't.
let sql = require('mssql');

(async () => {
    try {
        const pool = await sql.connect('mssql://sa:secret@my-server/my-database');
        var requestQuery = new sql.Request(pool);

        const q = `SELECT * FROM mytable`;

        // This doesn't work (Error: Could not find stored procedure 'S')
        const result1 = await sql.query(q);
        console.dir(result1);

        // This works
        const result2 = await sql.query`SELECT * FROM mytable`;
        console.dir(result2);

        // This works
        const result3 = await requestQuery.query(q);
        console.dir(result3);

        // This works
        const result4 = await requestQuery.query`SELECT * FROM mytable`
        console.dir(result4);

    } catch (err) {
        // ... error checks
        console.error('Error', err)
    }
})();

+1 I have this issue as well

So crazy. Even for a simple wrapper like this needs a workaround.
async function _sql(...args) {
    var result;

    if(args[0][0].length > 1) { // bad condition to check whatever it is template literal
        // is it with template literal? If so, use db.
        result = await db.query.apply(db, args);
    } else {
         // if not use request, otherwise it won't work!
        var req = db.request();
        var result = await req.query(args[0]);
    }

    return result.recordset;
}

Fixed and to be released in 4.2.


CMake error when using ASSERT_USE_EXTERNAL_CPPTRACE flag

When I tried installing libassert (release v1.2.1) using CMake with ASSERT_USE_EXTERNAL_CPPTRACE flag an error occurs after building the library which is raised here:
https://github.com/jeremy-rifkin/libassert/blob/56ddd631ccc133d4d25f7b8051371d931783a40b/CMakeLists.txt#L165-L170
The reason this error occurs is that the cpptrace target is not found due to the fact that the actual target name is cpptrace::cpptrace after it is added by find_package:
https://github.com/jeremy-rifkin/libassert/blob/56ddd631ccc133d4d25f7b8051371d931783a40b/CMakeLists.txt#L72-L74
After I edited the CMakeLists file locally by changing the target name to cpptrace::cpptrace inside the add_custom_command function I was able to install the library successfully. I think changing the fixed cpptrace value inside this function to some kind of local variable which would contain the valid target name based on ASSERT_USE_EXTERNAL_CPPTRACE flag value should do the trick.

Thanks for reporting this issue and the detailed triage, that sounds like it should work. I can fix it later or would be happy to accept a PR 🙂

Should be fixed now


NAT firewalled?

I will start several raspiblitz in China and need to know if they work correctly behind a NAT firewall. should work like any bitcoin/lightning node, or?

some background:
Here in China you get a router for your what-ever-configured internet connection. but you get only access to a user account of the modem. so your router receives ALWAYS an IP from the Modem via DHCP. In your router you can use that signal and bridge it to another network. But your traffic will always routed through the modem and it's IP and you can not create port forwarding  for example.
So you can for example NOT run your own server at home.
Due to the fact my wallets on PC and mac work properly I guess this should not be an issue, but I am afraid that incoming requests to provide an invoice could fail, because they never reach the destination.
Anybody experience with any kind of lightning full node in China at an internet connection providing dynamically IPs?

No experience with running nodes Nodes in China on my side. But NAT is a common problem for users. If your node is not public reachable othe nodes cannot open a channel with you. But you can open a channel with other public nodes. So then you can start to make payments. Also your channel gets published on the gossip network and other nodes have your node in their network map and should be able to send payments to you (if you have receiving capacity on your channel).
One trick to build up receiving capacity would be you have a public node outside the NAT that is your own. Then you start up your node behind the NAT and make a channel with your public node. If the channel is funded you sent all the funding to your public node (its stays in the family). Now you have the full receiving capacity of that node behind the NAT available and you can handout invoices.
All that said. Of course its the best if your node behind the NAT gets public available somehow. The RaspiBlitz has experimental TOR integration in the latest version. Even behind a NAT your node will then be publicly available over TOR.
Other ways to get this done could be to experiment with ngrok, cjdns or IPv6/VPN tunnels.

Thanks you very much. I am operating several lightning nodes on vps already which i can establish channels to.
Due to the fact i would like to become a forwarding relay within china the suggested solution will probably only support that indirectly. I needed to open lots of channels (especially to other in china located) nodes.
I am aiming to become an useful entrypoint within China. Harder than i thought, but tor will probably help.
We‘ll see.

Good luck. Share your experience :) Please keep in mind, that the RaspiBlitz is still a hackathon style project. Running big funds in production on mainnet, can lead to lost or stolen funds.


Pass spans by with 'ref' when possible

Nooo that's not what I meant 😢. This change will probably regress the performance since you are supposed to pass spans as is. The comment was that extra performance can be extracted by switching from spans to pointer arithmetics and unsafe, which is a very involved change hence the note "if you're feeling adventurous".

Ah, it seems that I have misunderstood then. I was planning on testing this tomorrow to check the perf but thanks for the clarification; I don't want to go into the pointers route, so I think I'll leave the code as is & close this PR tomorrow
Thanks a ton, again ❤


Macro for cause and description.

Greetings!
Am I missing something, but there exist #[source] for source(&self) method, #[backtrace] for backtrace(&self), but no macro for description and cause?

Correct, there is no reason to ever use those methods.

there is no reason to ever use those methods

Wait, why??