Unchecking "Load on select" in Glitch harvester disables the "reload corruption" hotkey

It's also unclear how to load a stash in any way when load on select is disabled. Loading on double-click would be nice.
I might be doing something wrong, but load on select seems to make it a lot more tedious to merge stashes and stuff.

Issue is replicable (custom hotkey L for Reload Corruption no longer works when "Load on Select" is disabled) and there doesn't seem to be a way to load a corruption from the Stash into BizHawk if this option is disabled.

I've been able to modify / duplicate existing code to add features so double-clicking and the Reload hotkey skips checking if the "Load on Select" is enabled or not; testing seems to work successfully.

Any chance you could share the modified compiled files? Doesn't look like it would get merged any time soon, and I don't really want to try compiling it for multiple reasons.

the hotkeys are doing ui automation. we'd have a lot of refactors to do to
correctly implement hotkeys. if you're a coder and you want to help, you
can our discord. otherwise be patient, we do a bug blitz from time to time
but rtc dev has really slowed down to a crawl recently.
On Thu, Nov 30, 2023, 10:17 scrumpyman @.***> wrote:

Any chance you could share the modified compiled files? Doesn't look like
it would get merged any time soon, and I don't really want to try compiling
it for multiple reasons.
—
Reply to this email directly, view it on GitHub
https://github.com/redscientistlabs/RTCV/issues/488#issuecomment-1833979480,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5N3TZRBIGKQSFRYLO2R4DYHCPOVAVCNFSM6AAAAAA4V7A53SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZTHE3TSNBYGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Update readme.python.md

https://github.com/Azure/sdk-release-request/issues/4074

@msyyc , I leave this PR assignment to you as it's only readme changes.


chore: cleanup + various doc improvements

Please read contributing guidelines and remove this line.

Current dependencies on/for this PR:

master

PR #4282   👈

This stack of pull requests is managed by Graphite.


fix(explore): metrics caret not expanding in datasource panel

SUMMARY
This fixes an issue with the Antd Collapse in the Datasource panel that caused the accordion to fail.
Closes #12233
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
After:

TEST PLAN

Explore a chart
Expand the Metrics in the Datasource panel
Make sure the accordion works (only one section collapsed at a time)

ADDITIONAL INFORMATION

[x] Has associated issue: #12233
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

@junlincc @rusackas

@junlincc @rusackas

Codecov Report

Merging #12305 (62b46a9) into master (2b6a587) will increase coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12305      +/-   ##
==========================================
+ Coverage   63.00%   63.39%   +0.39%
==========================================
  Files        1002      485     -517
  Lines       49268    29865   -19403
  Branches     5003        0    -5003
==========================================
- Hits        31039    18932   -12107
+ Misses      18028    10933    -7095
+ Partials      201        0     -201

Flag
Coverage Δ

javascript
?

python
63.39% <ø> (-0.77%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/views.py
62.50% <0.00%> (-25.07%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/db_engine_specs/presto.py
74.67% <0.00%> (-8.66%)
:arrow_down:

superset/databases/commands/create.py
83.67% <0.00%> (-8.17%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/db_engine_specs/base.py
79.38% <0.00%> (-6.71%)
:arrow_down:

superset/sql_validators/base.py
93.33% <0.00%> (-6.67%)
:arrow_down:

superset/views/database/forms.py
83.33% <0.00%> (-5.56%)
:arrow_down:

... and 529 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #12305 (62b46a9) into master (2b6a587) will increase coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12305      +/-   ##
==========================================
+ Coverage   63.00%   63.39%   +0.39%
==========================================
  Files        1002      485     -517
  Lines       49268    29865   -19403
  Branches     5003        0    -5003
==========================================
- Hits        31039    18932   -12107
+ Misses      18028    10933    -7095
+ Partials      201        0     -201

Flag
Coverage Δ

javascript
?

python
63.39% <ø> (-0.77%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/views.py
62.50% <0.00%> (-25.07%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/db_engine_specs/presto.py
74.67% <0.00%> (-8.66%)
:arrow_down:

superset/databases/commands/create.py
83.67% <0.00%> (-8.17%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/db_engine_specs/base.py
79.38% <0.00%> (-6.71%)
:arrow_down:

superset/sql_validators/base.py
93.33% <0.00%> (-6.67%)
:arrow_down:

superset/views/database/forms.py
83.33% <0.00%> (-5.56%)
:arrow_down:

... and 529 more

Continue to review full report at Codecov.

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

thanks for the fix @geido!  can we keep the column section open while the metrics is expanded?

thanks for the fix @geido!  can we keep the column section open while the metrics is expanded?

I am not sure I understand @junlincc. The Columns section is closed as soon as the Metrics section is expanded.

I am not sure I understand @junlincc. The Columns section is closed as soon as the Metrics section is expanded.

@junlincc the "one at a time" option was there in the original implementation, just not working properly. This is what this PR fixes. Having both opened requires a different approach and a different implementation which I would not recommend.

@junlincc the "one at a time" option was there in the original implementation, just not working properly. This is what this PR fixes. Having both opened requires a different approach and a different implementation which I would not recommend.

+1 on keeping them both opened if users choose so. In the future we may even allow users to drag a column to the metric section to create a metric.

+1 on keeping them both opened if users choose so. In the future we may even allow users to drag a column to the metric section to create a metric.

@ktmud @junlincc I removed the accordion and set the two sections open by default. Have a look, please. Thank you

@ktmud @junlincc I removed the accordion and set the two sections open by default. Have a look, please. Thank you

Is it possible to update the screenshot/recording with change noted above?

Is it possible to update the screenshot/recording with change noted above?

Verified locally... looking good! You're off the hook, @junlincc

Verified locally... looking good! You're off the hook, @junlincc


Access Modifiers is missing in the output

I really like a lot this tool. So great what you have been doing.
How do you add the method access modifier (global, private, public) to the md file? Seems to be missing for me in the generated output.
apexdocs-generate -s force-app -t docs -r --scope global public  private -g docsify --includeMetadata

Thanks @davidbrowaeys
No way right now, but I agree those should be in there. Let me look into that!

I've newest a new version (https://www.npmjs.com/package/@cparra/apexdocs) which should include the access modifiers as part of the method and field descriptions. Let me know if you run into any issues!

Wow thank you so much @cesarParra didn't expect the quick so quick.


[FEATURE REQ] mgmt, support container size in convenience API on webapp/functionapp

https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/resourcemanager/azure-resourcemanager-appservice/src/main/java/com/azure/resourcemanager/appservice/fluent/models/SiteInner.java#L681-L702

Does customer asked for webapp as well?
Seems containerSize only applies on function app. Setting containerSize for webapp has no effect according to my test..

According to service, it's a memory limit indicator

1.1.1        Instance Size
The instance size property determines the amount of memory allocated to each instance of the function app, measured in mebibytes. CPU and network bandwidth are allocated proportionally. This is an app level property.
Proposed – reuse existing setting in our schema that is currently unused:
/site/{sitename}/containerSize

The effect it has on web app is unknown... We may support withContainerSize for both, but for web app, we'll not have test on it.

We can double check with Hanxiao on his usage.

Checked with Hanxiao, he'll use it for function app's preview flex consumption plan. He's fine with the approach.

2.29.0 released.


Issue adding a new DB spec: default_driver needs to have a non-empty value.

Using Superset 2.1.0, I'm trying to add a new db spec and using BasicParametersMixin so that a form is shown at database creation time with all the parameters in separate inputs.
class MyEngineSpec(BaseEngineSpec, BasicParametersMixin):

    engine = "myengine"

    engine_name = "My Engine"

    default_driver = ""

    sqlalchemy_uri_placeholder = (

        "my://user:password@host:port/dbname[?key=value&key=value...]"

    )

    encryption_parameters = {"sslmode": "require"}

    ...

My SQL Alchemy dialect does not need the specification of a driver, and therefore I've set default_driver to the empty string. I've tried that and also None, but Superset will not show me the form corresponding with the BasicParametersMixin unless I set a non-empty value in default_driver... which is wrong and makes SQL Alchemy fail because my dialect does not need any driver names.
Is there any way I can work around this issue?

I'm not sure I understand. What happens when you have default_driver set to an empty string? Can you share a screenshot?

What happens is that, for that data source, the BasicParametersMixin is ignored and no multi-input form --with separate server, port, user, password, etc. inputs-- is shown.
Instead, the standard SQL Alchemy URI form is shown.

Hi there, and sorry this has gone quiet for so long. We're no longer supporting Superset 2.x or prior, and it's been a while since this thread saw any activity, so I'm closing this as stale. If it is still an issue in Superset 3.x, we can re-open this, or feel free to open a new issue with updated context.


Allow unit converter to accept floating point numbers without leading zeroes

Summary of the Pull Request
What is this about: Allow unit converter to accept floating point numbers without leading zeroes
What is included in the PR: add a prefix "0" if the query starts with "."
How does someone test / validate: unit test / build and run the unit converter
Note that for the latter, the default activation command for Launcher-Program is "."; this may block the launcher from sending the query to plugins other than Program. Can add "%%" (default activation command for unit converter) at the beginning to do the test.
Quality Checklist

[x] Linked issue: #16287 ( for unit converter only)
[x] Communication: I've discussed this with core contributors in the issue.
[x] Tests: Added/updated and all pass
[ ] Installer: N/A
[ ] Localization: N/A
[ ] Docs: N/A
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries / WXS / YML

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Thanks for opening the PR!
It still seems to be failing some tests.

Thanks for opening the PR! It still seems to be failing some tests.

Sorry I didn't notice this earlier and thank you for letting me know!
I think I have fixed the problems now.

Please check decimal separator in current culture.

Done. Thank you for the suggestion.

@yifan-k looks like a few more tweaks are still being requested, correct @lncubusus?

Yes, as I mentioned it fails on machine with comma for decimal separator.

I missed the earlier message and I am so sorry that I did not reply.
I reviewed this problem and found that, as I just commented in Issue #16287:
Both Calculator and Unit Converter already support inputs without leading zeroes.
The problem in #16287 was caused by the fact that ".", the most common decimal separator, is used as the default activation command for Launcher - Program.
After changing the activation command to something else, both plugins work well with inputs starting with decimal separators.
So I decide to close this pull request.
Thanks to everyone above!


Parses NPM env variables

Fixes https://github.com/pnpm/pnpm/issues/8339.

Maybe we should extend the ENV variables cleanup to these settings too:

merge-git-branch-lockfiles-branch-pattern[]
ca[]

wouldn't the right fix be parsing the values from the env variables correctly?

Mainly for context and to understand better the right way: why in first place PNPM takes care of the exported NPM env variables (converting them to the original npmrc settings)?

Because that is how npm behaves. We use npm's configuration system.

Of course, but given the fact that PNPM forked the NPM config system, we are not required to follow strictly their logic.
In addition, the fact that PNPM loads its settings from npm_config_ env variables is not documented anywhere inside the PNPM documentation, so this behavior is mainly a side effect rather than a feature/public-api.
If the intent is to be able specify PNPM settings as env variables, this behavior should be at least documented somewhere inside the PNPM documentation.
That said, the fact that PNPM settings can be changed through env variables can be very handy, so I thing that it would be better, as you suggest, preserving the env variables instead of deleting them, adding this behavior to the documentation too.
I made some research inside the @npm/config source code, and the encoding seams to be quite simple and platform independent.
https://github.com/npm/cli/blob/4e81a6a4106e4e125b0eefda042b75cfae0a5f23/workspaces/config/lib/set-envs.js#L15
They are serializing array settings for supporting the ca[] setting which can contain escaped new line characters.

ca
Default: null
Type: null or String (can be set multiple times)
The Certificate Authority signing certificate that is trusted for SSL connections to the registry. Values should be in PEM format (Windows calls it "Base-64 encoded X.509 (.CER)") with newlines replaced by the string "\n". For example:
ca="-----BEGIN CERTIFICATE-----\nXXXX\nXXXX\n-----END CERTIFICATE-----"
Set to null to only allow "known" registrars, or to a specific CA cert to trust only that specific signing authority.
Multiple CAs can be trusted by specifying an array of certificates:
ca[]="..."
ca[]="..."

https://docs.npmjs.com/cli/v10/using-npm/config#ca
Digging inside the @npm/config source code we can see that array values are always joined using \n\n. For compatibility with NPM we should support this serialization format, but it would be very counter intuitive and awkward for the user to encode a simple list with new line characters, especially from shell, requiring to double escape the backslash \\n.
My idea is to implement a deserializer for the env variables that supports both the NPM serialization format and a simpler comma joined list. The comma joined list is aligned with the output of the pnpm config get public-hoist-pattern output so it seams the best choice to me.
What do you think?

I have updated the pull request source code with the proposed concept.
# Given an .npmrc with
# public-hoist-pattern[] = foo
# public-hoist-pattern[] = bar

npx pnpm config get public-hoist-pattern # returns
# foo,bar

npm_config_public_hoist_pattern=foo,bar npx pnpm config get public-hoist-pattern # returns
# foo,bar

@zkochan

Is public-hoist-pattern[]=['*foo*', '*bar*'] supported?


fix(delete-user-data): add RTDB locations function

fixes: #795
Checklist

add Realtime Database location choice to extensions.yaml
refactor SELECTED_DATABASE_INSTANCE to camelcase letters (selectedDatabaseInstance)
add selectedDatabaseLocation
add getDatabaseUrl function with tests
fix condition by checking if databaseUrl exist (was provided by user)
update README.md

Tests
Update Engine

Installed Extension

Extension Settings
Belgium(west-europe1) RTDB

United States(us-central1) RTDB

Added data
Belgium(west-europe1) RTDB

United States(us-central1) RTDB

Extension succesful logs
Belgium(west-europe1) RTDB

United States(us-central1) RTDB

Codecov Report

Merging #865 (4dcc168) into next (dc18013) will decrease coverage by 4.16%.
The diff coverage is 66.67%.

@@            Coverage Diff             @@
##             next     #865      +/-   ##
==========================================
- Coverage   50.19%   46.04%   -4.15%
==========================================
  Files          30       34       +4
  Lines        1329     1462     +133
  Branches      197      208      +11
==========================================
+ Hits          667      673       +6
- Misses        661      788     +127
  Partials        1        1

Impacted Files
Coverage Δ

delete-user-data/functions/src/config.ts
0.00% <ø> (ø)

delete-user-data/functions/src/index.ts
0.00% <0.00%> (ø)

delete-user-data/functions/src/helpers.ts
100.00% <100.00%> (ø)

delete-user-data/functions/src/logs.ts
0.00% <0.00%> (ø)


[Code Quality] Suggestions to improve performance

It seems like the application struggles at times, particularly with button/form actions.

Article I found on optimizing for PHP 7
Code analyzers: I ran a Exacat scan, and Phan might work for the PHP stuff. For JS, DeepScan flagged some minor issues.

I hope this is of some help.

Thanks for the suggestions! Improving the web interface is something I've been planning to do for a while.

#39 appears related to this, particularly with Javascript issues on older / TV browsers.


As user, I want scripts to not break, so I don't have to fix old projects all the time

[x] Store builds per version (codepen-snippets/1.2.3/math/Vector2.js)
[x] Expose builds via GitHub Pages
[x] Automate deploying with Travis CI

Apparently Travis CI needs a git token to commit and push changes.
http://stackoverflow.com/questions/23277391/how-to-publish-to-github-pages-from-travis-ci


Add page about Studio REST API

TODO:

[ ] replace demo-bank-customer-churn with example-get-started-private https://github.com/iterative/studio/pull/6338#issuecomment-1626045631

Looks good @aguschin! Do we have a story to add it to the model registry docs and anywhere else it's needed for higher-level stories? Where do we plan to explain why it's needed (esp. compared to dvc get) and how to use it (model deployment workflow)?

Where can we explain how to get models from a monorepo?

Should we rename from "download model" to something like "get model url"? I don't think either API downloads the actual files, right?

@dberenbaum, thanks for the feedback. I don't think these doc are ready to be released right now (even REST API part). My take was we need to complete all items in phase I before that. Phase I includes Python API - that's why it's here.

@aguschin I'm fine to cut the scope to the REST API (but we need directories there - my take). I think we can publish it then and make a FE change to reflect the curl command.

@shcheklein Could you delete or restore https://github.com/iterative/example-get-started-s3-private so I can push to it and use it for these docs?

@dberenbaum yep, I can do it. How do we expect though users to try it? Can we make it available anonymously?

I thought we were using that repo because we weren't focused on a publicly usable demo yet. If we want it to be public, I don't think it's worth working on https://github.com/iterative/example-get-started-s3-private unless I'm missing something. We could instead add a 2nd public s3 remote to https://github.com/iterative/example-get-started (or https://github.com/iterative/example-get-started-experiments).
We might need to address https://github.com/iterative/dvc/issues/5797. Can we generate the signed urls without that and will they work?
Also, what would be the risks of including credentials in the demo projects? How do we expect this to work for users who want to give read access to others to download models without exposing the underlying credentials?

Do we even want to publish this doc and expose the rest api publicly right now? Maybe it's simpler to leave it as a backend that gets used by the dvc cli/api.

I personally never considered this as an end goal, merely as a first step to build some basics. It might be useful in some scenarios, but I would also first go with a high level API, then would add this as a detail if needed.

I don't think we should spend time on worrying about the example or making it work publicly. We can add a note not to try the exact command because it will fail due to lack of credentials on studio

agreed.

Okay, let's close it for now and decide what we need after https://github.com/iterative/dvc/issues/9100

I don't think we should spend time on worrying about the example or making it work publicly. We can add a note not to try the exact command because it will fail due to lack of credentials on studio

How hard would it be to just return the http urls in this case to have a simple working example @amritghimire?

I don't think we should spend time on worrying about the example or making it work publicly. We can add a note not to try the exact command because it will fail due to lack of credentials on studio

How hard would it be to just return the http urls in this case to have a simple working example @amritghimire?

Sorry I didn't understand. Do we want to add support for http/https urls?

Do we want to add support for http/https urls?

Yes if it's easy enough. How much effort do you estimate it would take?

Do we want to add support for http/https urls?

Yes if it's easy enough. How much effort do you estimate it would take?

Should be probably 2 days effort.

Do we want to add support for http/https urls?

Yes if it's easy enough. How much effort do you estimate it would take?

Should be probably 2 days effort.

@dberenbaum Can you create an issue for this?

Waiting for https://github.com/iterative/studio/issues/7127 and https://github.com/iterative/studio/issues/7383 before merging

Per https://github.com/iterative/dvc.org/pull/4809#discussion_r1341264586, I added references and reworded the dvc artifacts CLI/API refs. @shcheklein PTAL 🙏.


Add a list of important properties.

Figured we might enumerate some of our more useful properties to make them more discoverable.

Nice :shipit:


PrefixAllGlobalsSniff: always flag blocked prefixes

Is your feature request related to a problem?

This is related https://github.com/WordPress/plugin-check/issues/523. In the Plugin Check plugin we need to find a way to flag the lack of prefixes for global functions.
Describe the solution you'd like

PrefixAllGlobalsSniff has a hardcoded blocklist of disallowed prefixes:
https://github.com/WordPress/WordPress-Coding-Standards/blob/7f766304b0654cee7c1dfa8e548f65fce197e05c/WordPress/Sniffs/NamingConventions/PrefixAllGlobalsSniff.php#L78-L91
However, this blocklist is only used to compare it against the user-provided prefixes list.
So if I provide myplugin,wp as valid prefixes, the sniff will warn about wp as disallowed and remove it from the list. After that, all functions are checked for the myplugin prefix.
Now, when I don't pass any custom prefixes, the sniff doesn't do anything, as it doesn't know what valid prefix to look for.
However, since we already know that wp is not allowed, the sniff could always check for that regardless.
So I am proposing that, if no list of valid prefixes is passed, the sniff should always at least flag the blocked ones.
In other words, if I don't pass any configuration wp_awesomefunc() would get reported as having a disallowed prefix.
Additional context (optional)

[x] I intend to create a pull request to implement this feature.

That's a good one @swissspidy !

I'm not keen on this suggestion for the following reasons:

This sniff is about prefixing all globals. You're suggesting changing the complete premise of the sniff to one of "banning certain prefixes completely". To me that is not an acceptable change.
It would also be very inhibitive for people using WPCS for in-company/private plugins/themes.
The ask is "flag the lack of prefixes for global functions", but if this would be handled via the PrefixAllGlobals sniff, it would not just affect global functions, but also namespace names, classes, variables, hook names, constants etc.

All in all, I don't think the proposed changed is acceptable.
It could be considered to have a separate sniff which only checks global function declaration names against a banned prefix list, though I'm not fully convinced of the usefulness of such a sniff as what we want is that devs use a consistent prefix, while selectively flagging unwanted prefixes would undermine that advise.
As an alternative, what about if the PrefixAllGlobals sniff would get a new error for "no prefixes passed" ? with the recommendation that plugins/themes should always use a prefix/prefixes and pass the prefix(es) selected to WPCS ?

Thanks, I appreciate the quick feedback!

It could be considered to have a separate sniff which only checks global function declaration names against a banned prefix list, though I'm not fully convinced of the usefulness of such a sniff as what we want is that devs use a consistent prefix, while selectively flagging unwanted prefixes would undermine that advise.

From a WPCS point of view that definitely makes sense. For the plugins team it's different though, at least that's my understanding as an outsider. So I think we'll have to create such a new sniff for Plugin Check to accommodate our needs.

As an alternative, what about if the PrefixAllGlobals sniff would get a new error for "no prefixes passed" ? with the recommendation that plugins/themes should always use a prefix/prefixes and pass the prefix(es) selected to WPCS ?

This would help people getting started with WPCS, as they might otherwise never realize that the sniff is silently not running. So that makes sense to me 👍

It could be considered to have a separate sniff which only checks global function declaration names against a banned prefix list, though I'm not fully convinced of the usefulness of such a sniff as what we want is that devs use a consistent prefix, while selectively flagging unwanted prefixes would undermine that advise.

From a WPCS point of view that definitely makes sense. For the plugins team it's different though, at least that's my understanding as an outsider.

I'm not so sure it is different. After all, unless you require a consistent - and preferably unique - prefix(es) in all plugins/themes, that "ban" list would just keep expanding until it is a dictionary.
That's something for the plugin team to discuss further though and outside the scope of WPCS.

As an alternative, what about if the PrefixAllGlobals sniff would get a new error for "no prefixes passed" ? with the recommendation that plugins/themes should always use a prefix/prefixes and pass the prefix(es) selected to WPCS ?

This would help people getting started with WPCS, as they might otherwise never realize that the sniff is silently not running. So that makes sense to me 👍

Let's give other people some time to pitch in to see if anyone has a good reason why this shouldn't be done.
If there are no objections, this is a change for which a PR would be welcome.
Notes for anyone who wants to work on this:

I think a warning would probably be the right error level.
This will need a private property in the sniff to remember whether the warning has already been thrown or no, as otherwise it would be thrown for every single function/namespace/class/variable/constant/etc definition and would become silly noisy without conveying any new information. Throwing the warning once should suffice.


return demo_sent_subjectivity instead of print

I simply changed the output format of the demo_sent_subjectivity to return instead of print to avoid the program to have to run longer because it prints out each. result to the console.

The reason the demo_* functions don't tend to return anything, is because they are not meant to be used much. In my opinion, these functions should not even be importable from NLTK. They are only meant to quickly and effectively show how functions/classes in that file can be used.
Because of that, I prefer printing over returning, as it helps with effectively displaying the output.

#2903 has fixed a broken test case which snuck into this PR. I've merged develop into this PR to resolve this broken test.
That said, this PR still has broken tests from the PR itself. (pre-commit fails because of incorrect formatting, running black or pre-commit run --all-files should solve it)

I'll be closing this.


Cannot Resize Regions added during .create

I'm looking to utilize Wavesurfer and the Regions plugin to create an initial region, and then allow the user to resize it.  I'm able to create the region, and it appears on the instance as expected:

I also explicitly set resize and drag to true on creation:
var wavesurfer = WaveSurfer.create({
      container: '#waveform',
      backend: 'MediaElement',
      waveColor: 'violet',
      hideScrollbar: true,
      autoCenter: false,
      progressColor: 'purple',
      plugins: [
        WaveSurfer.timeline.create({
          container: '#wave-timeline'
        }),
        WaveSurfer.regions.create({
          regions: [
            {
              id: 0,
              start: self.currentTime,
              end: self.currentTime + 15,
              color: 'hsla(400, 100%, 30%, 0.5)',
              resize: true,
              drag: true
            }
          ]
        })
      ]
    })

However, I'm unable to resize the created region.  If I enable dragSelection in the region options, I can create new regions and resize them, but not initially created regions.
Finally, if I log the created region:
console.log(wavesurfer.regions.list[0])
The region indeed reports drag and resize as being set to true:
**attributes: {}
color: "hsla(400, 100%, 30%, 0.5)"
data: {}
drag: true
element: region.wavesurfer-region
end: 15.603342
firedIn: false
firedOut: false
handlers: {remove: Array(3), out: Array(1)}
id: 0
loop: false
maxLength: undefined
minLength: undefined
resize: true
scroll: false
scrollSpeed: 1
scrollThreshold: 10
start: 0.603342
style: ƒ (e,t)
util: {…}
wavesurfer: t {handlers: {…}, defaultParams: {…}, backends: {…}, util: {…}, params: {…}, …}
wrapper: wave
_onRedraw: ƒ ()
__proto__: Object

What am I missing? Some of the examples do indeed allow for created regions to be resized.

What am I missing?

what version of wavesurfer.js? did you try master branch?

Ah, sorry for not including my version #.
I'm using 2.2.1 from npm.

Just tried master. Same behaviour.

Couldn't figure out how to get it to work during .oncreate, but it does work fine if you add the region after the instance is ready.
wavesurfer.on('ready', function () {
      wavesurfer.addRegion({
        id: 0,
        start: self.currentTime,
        end: self.currentTime + 15,
        color: 'hsla(1, 77%, 55%, 0.25)',
        resize: true,
        drag: true
      })
    })

Sadly that's not the case. The region doesn't appear when you create it using the ready event.

@iamsteadman does your PR #1673 fix this issue?

Still doesn't work. I'm using version 2.2.1. Any updates on this?


Restore last window size on startup, handled not correctly

Good Morning @terjeio
Today I checked the Box "Restore last window size on startup in the settings.
Theoretically it works, but the problem ist, that the upper left corner will still be the starting point.
For example I start the programm, the upper left corner ist maybe 300 pixel from the left desktop side and 300 pixel under the top area.
When I maximize the window and restart the programm, the upper left corner is still at 300x300 pixel, but the rest from the window is not shown...

Ok, I'll have to save the position as well then.

Ok, I'll have to save the position as well then.

Good Morning @terjeio
I downloaded your newest Version from today, it seems to save the position.
But one question.
Most time I use it in "full Screen" or Maximize Mode (don't know how to say :-) )
Is there the possibility that the software will restart in this mode?
But now it's better than before.

Good Morning @terjeio
I downloaded your newest Version from today, it seems to save the position.
But one question.
Most time I use it in "full Screen" or Maximize Mode (don't know how to say :-) )
Is there the possibility that the software will restart in this mode?
But now it's better than before.

Good morning @einencool
The latest edge version will now handle this correctly. It has also some changes to Tool length offset probing that I hope is ok, please be careful when using this and report any issues!

Good morning @einencool
The latest edge version will now handle this correctly. It has also some changes to Tool length offset probing that I hope is ok, please be careful when using this and report any issues!

Hello @terjeio
Today I made a little test, and it seems much better now.
Sadly the „maximized“ Position is not stored, but it‘s much better than before.
Since I didn‘t make a GCode with Tool Changes, I haven‘t tested the changed TLO part.
One question, somewhere you mentioned, that it would be able to see which Limit Pins raised an alarm.
Because I modified my „sensitive“ Endstops with stiffer springs, but I thing I got a noise Problem somewhere.
While the Spindle is running up, it happens two times, that Alarm:1 raises and the machine stops before any movements...

Hello @terjeio
Today I made a little test, and it seems much better now.
Sadly the „maximized“ Position is not stored, but it‘s much better than before.
Since I didn‘t make a GCode with Tool Changes, I haven‘t tested the changed TLO part.
One question, somewhere you mentioned, that it would be able to see which Limit Pins raised an alarm.
Because I modified my „sensitive“ Endstops with stiffer springs, but I thing I got a noise Problem somewhere.
While the Spindle is running up, it happens two times, that Alarm:1 raises and the machine stops before any movements...

Sadly the „maximized“ Position is not stored, but it‘s much better than before.

Do you have two monitors? Otherwhise I do not understand. Position is not saved, only size - and I set size to -1 when the window is maximized to indicate that it is to be maximized on startup.

One question, somewhere you mentioned, that it would be able to see which Limit Pins raised an alarm.

I cant remember that, only that it was mentioned that when two switches was used per axis it would be nice to see both in the "LED" status? If you have a problem with intermittent triggering of switches then this will not help as you might not even see a "LED" blink if the trigger pulse is shorter that the polling rate (200ms).
What could (or rather should?) be done is to log the reason for an alarm beeing triggered so that this could be reported separately via a new $-command.

Sadly the „maximized“ Position is not stored, but it‘s much better than before.

Do you have two monitors? Otherwhise I do not understand. Position is not saved, only size - and I set size to -1 when the window is maximized to indicate that it is to be maximized on startup.

One question, somewhere you mentioned, that it would be able to see which Limit Pins raised an alarm.

I cant remember that, only that it was mentioned that when two switches was used per axis it would be nice to see both in the "LED" status? If you have a problem with intermittent triggering of switches then this will not help as you might not even see a "LED" blink if the trigger pulse is shorter that the polling rate (200ms).
What could (or rather should?) be done is to log the reason for an alarm beeing triggered so that this could be reported separately via a new $-command.

Sadly the „maximized“ Position is not stored, but it‘s much better than before.

Do you have two monitors? Otherwhise I do not understand. Position is not saved, only size - and I set size to -1 when the window is maximized to indicate that it is to be maximized on startup.

No, I only use a Laptop with the built in screen.
I started the software several times, and it shows up in the last position, that‘s ok so far :-)
But the software is not „maximized“.
Then I clicked on „maximize“ and waited a few seconds.
After that I closed the Program and restarted it, but it has the same position and size as before and I have to click on maximize again.
That‘s all :-)
I can live with that, that‘s not the Problem :-)

One question, somewhere you mentioned, that it would be able to see which Limit Pins raised an alarm.

I cant remember that, only that it was mentioned that when two switches was used per axis it would be nice to see both in the "LED" status? If you have a problem with intermittent triggering of switches then this will not help as you might not even see a "LED" blink if the trigger pulse is shorter that the polling rate (200ms).
What could (or rather should?) be done is to log the reason for an alarm beeing triggered so that this could be reported separately via a new $-command.

Ah ok, I thought somewhere I read about it, that I can see in the Console which endstop was triggered to see where the alarm raises...
This option would be great, because I think it‘s a noise issue, because it shows up, while the spindle is starting. But the funny thing is, that it happens two times and I started the machine around 8 times....
So I have to check the wires if they are now lying to close to each other...

Sadly the „maximized“ Position is not stored, but it‘s much better than before.

Do you have two monitors? Otherwhise I do not understand. Position is not saved, only size - and I set size to -1 when the window is maximized to indicate that it is to be maximized on startup.

No, I only use a Laptop with the built in screen.
I started the software several times, and it shows up in the last position, that‘s ok so far :-)
But the software is not „maximized“.
Then I clicked on „maximize“ and waited a few seconds.
After that I closed the Program and restarted it, but it has the same position and size as before and I have to click on maximize again.
That‘s all :-)
I can live with that, that‘s not the Problem :-)

One question, somewhere you mentioned, that it would be able to see which Limit Pins raised an alarm.

I cant remember that, only that it was mentioned that when two switches was used per axis it would be nice to see both in the "LED" status? If you have a problem with intermittent triggering of switches then this will not help as you might not even see a "LED" blink if the trigger pulse is shorter that the polling rate (200ms).
What could (or rather should?) be done is to log the reason for an alarm beeing triggered so that this could be reported separately via a new $-command.

Ah ok, I thought somewhere I read about it, that I can see in the Console which endstop was triggered to see where the alarm raises...
This option would be great, because I think it‘s a noise issue, because it shows up, while the spindle is starting. But the funny thing is, that it happens two times and I started the machine around 8 times....
So I have to check the wires if they are now lying to close to each other...

What is saved in for window size in your App.config (in the sender folder) after you exit the sender with the window maximized?
It should be like this:
 <WindowWidth>-1</WindowWidth>
 <WindowHeight>-1</WindowHeight>

What is saved in for window size in your App.config (in the sender folder) after you exit the sender with the window maximized?
It should be like this:
 <WindowWidth>-1</WindowWidth>
 <WindowHeight>-1</WindowHeight>

Ahh Ok, that makes sense, didn't know what you mean before :-)
I don't know why, but the Config was not updated from the sender.
I always copy the "App.config" and the "ProbingProfiles.xml" to the newest Sender Folder, but the last saved date from the "App.Config" was the 16.01.2021 and not from yesterday...
So I enter the "-1" manually and test it in the next days, because I don't have the machine next to me and the Folder are synced to my PC at Home.
Thank you and for this I'll close this issue, because I think then it is solved :-)
Have a nice sunday evening
Greets
Chris

Ahh Ok, that makes sense, didn't know what you mean before :-)
I don't know why, but the Config was not updated from the sender.
I always copy the "App.config" and the "ProbingProfiles.xml" to the newest Sender Folder, but the last saved date from the "App.Config" was the 16.01.2021 and not from yesterday...
So I enter the "-1" manually and test it in the next days, because I don't have the machine next to me and the Folder are synced to my PC at Home.
Thank you and for this I'll close this issue, because I think then it is solved :-)
Have a nice sunday evening
Greets
Chris

What could (or rather should?) be done is to log the reason for an alarm beeing triggered so that this could be reported separately via a new $-command.

I am going to add a new command, $LEV (for Last EVents) that reports the inputs that was active when the core was last informed about a change via interrupt. This to aid finding which pin(s) caused problems due to interference. The output will contain the same letters for inputs as used for the |Pn: element in the real-time report.
I'll have to figure out how to best handle dual input limit switches (min & max) as this will require changes to both the core and all drivers. I am likely to report max limit switches by using the corresponding lowercase letters when this is ready.

What could (or rather should?) be done is to log the reason for an alarm beeing triggered so that this could be reported separately via a new $-command.

I am going to add a new command, $LEV (for Last EVents) that reports the inputs that was active when the core was last informed about a change via interrupt. This to aid finding which pin(s) caused problems due to interference. The output will contain the same letters for inputs as used for the |Pn: element in the real-time report.
I'll have to figure out how to best handle dual input limit switches (min & max) as this will require changes to both the core and all drivers. I am likely to report max limit switches by using the corresponding lowercase letters when this is ready.

That sounds interesting.
But one point.
Does it matter if it‘s the max or the min Endstop, that is triggered? Do some Boards have inputs for both sides?
On the Teensy there is min & max together on one input.
The only thing is that the Y1 OR Y2 Input could matter :-)

That sounds interesting.
But one point.
Does it matter if it‘s the max or the min Endstop, that is triggered? Do some Boards have inputs for both sides?
On the Teensy there is min & max together on one input.
The only thing is that the Y1 OR Y2 Input could matter :-)

Does it matter if it‘s the max or the min Endstop, that is triggered? Do some Boards have inputs for both sides?

Not that I am aware of. But who knows what will become available at some point?
So I have decided to add core support for up to four limit switches per axis for limits and one or two for homing if separate from the limit switches. Now I need to retest auto-squaring again to be sure I have not broken anything...


Initial concept implementation

Just a concept implementation, no CI, no bintray publishing

I think that the concept is great, just like I imagined. Are you satisfied with this solution?

@davidbilik main concern for me was "HOT" Deferred in original adapterFactory by JW. Can't imagine a real situation where it could brake things down, but this eager execution seems sort of unnatural. I would expect execution to start whenever .await() is called in some coroutine scope, not when an instance of Deferred is obtained.
Chaining could be done much cooler by just using .fold, but i guess it is up to Interceptor whether to continue in chain, couldn't figure out a cool functional approach to it...


Added False number code (June Lunchtime)

Added code for False Numbers (June LunchTime)
Fixes #1491
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@ravikr126 Kindly review my PR


"import call expects exactly one argument after upgrade to 0.57.8 from 0.55

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
React Native Environment Info:
System:
OS: macOS 10.14
CPU: (8) x64 Intel(R) Core(TM) i7-4790S CPU @ 3.20GHz
Memory: 155.81 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 11.1.0 - /usr/local/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
IDEs:
Android Studio: 3.1 AI-173.4819257
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: ^16.7.0 => 16.7.0
react-native: ^0.57.8 => 0.57.8
Description
After doing yarn upgrade react-native@latest from 0.55.4 to 0.57.8, I'm experiencing the following issue:

When starting the bundler with yarn start.

I actually hit enter. Any way to remove the label?

facing the same issue with version 0.57.8
$ react-native info

React Native Environment Info:
    System:
      OS: macOS 10.14.2
      CPU: x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
      Memory: 8.03 GB / 32.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 10.12.0 - /usr/local/bin/node
      Yarn: 1.10.1 - /usr/local/bin/yarn
      npm: 6.4.1 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
    IDEs:
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.3 => 16.6.3
      react-native: 0.57.8 => 0.57.8
    npmGlobalPackages:
      react-native-cli: 2.0.1
      react-native-git-upgrade: 0.2.7

@defrian I "fixed" the issue by slowly commenting out code. First I reinstalled my dependencies, then I commented out the line where I registered my app, aka AppRegistry. Then I slowly reintroduced my App component back. Don't exactly know why but after lots of reloading the error went away.


create an SOP to maintain rare disease list from GARD and NORD

We need an SOP with GARD and NORD to keep the rare disease list current.
Best solution:
GARD and NORD deposit an updated list of their rare disease (downloadable, and kept up to date) that we can download.

@matentzn, I assigned both of us so we have this on our radar. I put the milestone to October for now.

I am making a bit of progress on this all the time, but I am not trying to push for it.

#6903
#7030
#7232


How to alias a column that is ambiguous in $joinRelation?

I have a $joinRelation property relating couple of tables and I receive sql error saying that ... "x" column is ambiguous in where clause. How and where can I alias the said column? I've only come across this question https://stackoverflow.com/questions/51411208/objectionjs-column-reference-is-ambiguous but I don't use where clauses myself so I would need to modify the alias somewhere else.
I've tried looking up Objection.js documentation about relationships and trying to see if there is a way to prefix the column with an alias but no luck so far.

Add table to the column. For example:
{ query: { 'table.column': 1 }

Hi! I ran into this as well. The suggested fix will work, but it will break $eager: 'user' from returning anything.

Hi @mariusandra , if you have an issue, please open a new one with examples.

It's not an issue for me. I worked around it by making 2 queries and it was a throwaway project anyway.
I though I'll just leave a breadcrumb here for if anyone googling has a similar case. This person can then make a real issue :)


beacon scanner

hi @sandeepmistry  thanks for the awesome library. i was wondering that if you or any one in the community can provide an example to scan beacons in the area. if not then point me to functions which i can use to implement this functionality.
thanks

@cheeta1 this is a peripheral only library, so it does (and never will) support Central mode and scanning.

Hi @sandeepmistry, can i develop Central mode and scanning using this library? Maybe I need too add another some source files from Nordic SDK? Sorry for noobs questions,


Suggestion: Don't use @nuxtjs/router and override the default Nuxt routing functionality

@cretueusebiu thanks for sharing this repo, we thank you for it.
THis is just a suggestion, and I've seen the comment here:
https://github.com/cretueusebiu/laravel-nuxt/issues/33
What is the advantage of using this module: https://github.com/nuxt-community/router-module with a router.js file instead of auto-route generation from the pages/ directory?
Instead of overriding the default nuxt routing config functionality, why not add aliases or register custom routes instead of using @nuxtjs/router
This will allow this repo to be more compliant with other nuxt code, allowing us to copy paste from other repos into our application.

I always prefer to have my routes defined in a file, but I guess since this template is intended more as starting point I could remove it and leave it with the default router.
I will try to release a new version soon and will consider this change.

@cretueusebiu I am very new to nuxt (or vue for that matter) so I defer to the experts (aka you) as I don't know how much "extra power" the router.js file provides. I do believe that consistency with the other nuxt codebases would be helpful to me (and others will adopt it more easily), so thanks for the consideration.

It didn't make it into 2.0. It requires aliasing a bunch of routes...


[Fixes #121379429] Disable update on empty fields

What this PR does
Disables the update button in the user profile if the user has not entered any values, even if the input fields are touched.
Closes #40
Relevant Pivotal Tracker stories
Remove enabling update with empty fields

Coverage decreased (-0.4%) to 83.409% when pulling 3b90301d19e6a07091a78e49df4bc1c661244242 on bug/empty-field-userprofile-update into 7c27180b078ea7b2c6a10596737cd4ee6cd03709 on develop.


Doesn't recognize gulpfile.js

gulpfile.js appear to be a normal .js file, only "gulpfile" (no extension) use the gulp icon

Fixed.


adb command not found..

but i have already setting ANDROID_HOME environmental variable,adb command can found in terminal.

I have the same issue...

@dannysuen @Rowandjj I've released a new version to avoid the usage of this ANDROID_HOME var. Can you update your plugin installation. This is already fixed :)


Glyph image output order

Hey Marc –
Would it be possible to get the grid of diffed glyphs to output in a better order, i.e. GID or Unicode?
Thanks.

Another thing that would be beneficial on these images would be if they had different dimensions so that there are fewer glyphs per GIF. Like when it's too long, you dunno which is which when it flips back and forth between before and after. Or if the header (font name, before and after) repeated every X rows of glyphs.


Fix nodes names in edge's class

This is a fix for dagre-d3 rendering process.
There is no other solution to pass nodes names to the edge when rendering with dagre-d3.
I assume this solution is temporary, after merging #57  into master we'll pass nodes names in data- attributes:
https://github.com/applandinc/diagrams-js/pull/57/files#diff-a12813e59cc6d9ff69ab2da8f58567785e286a886ddd4bfd68d942e216bcda71R139-R141

This PR has been closed due to another PR#57 was merged with the same functionality.

This PR has been closed due to another PR#57 was merged with the same functionality.


cant't connect to docker hub in kitematic in window7

I can't log in the docker hub in the kitematic

are you using the proper credentials? Are you behind a proxy?

@FrenchBen yes,I am behind a proxy. I add proxy in the /var/lib/boot2docker/profile. And I can login the dockerhub in commandline. But still can't login the kitematic~!

Since you're on 2 different threads, I'm closing this one in favor of the previous mention.


Set default value for copysources

Update copysources.sh with the default value of the source path, so that it works even if no parameters are provided.
This will require changing the copysources.sh template and feeding it with the value of source folder relative to the output folder.

Update copysources.sh with the default value of the source path, so that it works even if no parameters are provided.
This will require changing the copysources.sh template and feeding it with the value of source folder relative to the output folder.

whatis the default value of source path ?

@seshapad Can you provide guidance to @PankhudiB on how to go about this activity?


Feature/Linux support

Update README
Handle Networking to support Linux

Have you tested it out on linux machine? Zvonimir can help you out on this.

Yes, @ZvonimirSabados tested out, and it works 👌

Awesome. Great work 🙌


Calico or Tigera operator should create CRDs automatically

Expected Behavior
Can the Calico or Tigera operator install Calico CustomResourceDefinitions (CRDs) directly? Similar to what Cilium does.
I noticed all the Calico and Tigera docs still talk about using kubectl to install either the Tigera operator or Calico directly.
When you look at the YAML, there are a ton of CRDs I'm expected to manage. The trouble is, we manage these declaratively. As a Kubernetes distro maintainer, it's becoming onerous to have to port Calico's CRDs every release. I'd rather just delegate this completely to the operator to install and manage. After all, it's Calico's own CRDs, it ought to be able to manage them.

Current Behavior
We're expected to apply CRDs either through kubectl or Helm (frowned upon) right now. Or otherwise pay attention to changes in the CRDs.

Possible Solution
cilium-operator creates or updates it's own CRDs when it starts. And it has RBAC to only allow managing those CRDs so its fine by any security team. As a result, nobody has to worry about the minor differences they make to their CRDs between releases, it's abstracted away.

Steps to Reproduce (for bugs)

Context

Your Environment

Operating System and version:
Link to your project (optional): Typhoon

This seems like a reasonable feature to add.
The operator does have the ability to update them if configured with a command line flag but does not have the ability to initially create them.
If someone wants to take this on, the expectation would be that in the main the operator would create them if they do not exist before starting up the controllers. If you have more questions please ask.


[Bug] Better visibility for overlapping items on gantt chart

What problem does this feature solve?
I have searched through all of the forums, documentation and tried messing with the logic of the gantt chart but I have yet to discover a good way of handling data that share the same time frame and row without entirely breaking the existing logic.
From what I've found, the default behaviour currently just hides the items behind each other, which can be quite limiting and there doesn't seem to be any options to change this.
Here is an example using the custom-gantt chart example.
Moving the item to the row below:

Item on the row below, blending together with the other items:

What does the proposed API look like?
This could be resolved with some sort of stacking behaviour to increase the height of the row when there are multiple items occupying the same space. I have quickly put together a rough of image of what I mean below:

I have tried to create this feature by modifying the the gantt chart's logic with no success.
I will continue to try and find work arounds for this scenario, but an official method would be greatly appreciated.
Have been loving the Echarts environment so far and look forward to all future improvements.

Still no success.
Any feedback regarding this issues would be greatly appreciated.

Just following up on this.
Does anyone know of a way of going about this?

Are there any plans to add this functionality?


Add signup testing with all error checks

Test the errors during signup

Can this be closed in favor of #112?
@mastrolinux


Remove the SetMaxRetries option

You can achieve the same via a custom Retrier.

@olivere, I can have a look at this if you want.

@eticzon Thanks. I think we should delay this (breaking) change until 6.0. If you want to tackle something hard, #415 is a good one. I currently don't have the time to do that. If you're looking for something much more straightforward, #297 is a long standing issue.

It doesn't make sense to remove this. If you need a complex/custom retrier, you can use SetRetrier, e.g.:
client, err := elastic.NewClient(
	elastic.SetHttpClient(httpClient),
	elastic.SetRetrier(elastic.NewBackoffRetrier(elastic.NewSimpleBackoff(100, 200, 300, 400, 500))), // 5 retries with fixed delay of 100ms, 200ms, 300ms, 400ms, and 500ms
	elastic.SetHealthcheck(false),
)

Otherwise, the SetMaxRetries(n) helper is a good companian if you want simple retries.


Toward typed equality in all things

Primarily looking for feedback though I wouldn't be averse to merging it as-is either.

I think this looks good, appreciate the tidying.

Unless anyone else has an opinion, feel free to merge (using the merge script in slamdata/devtools, if you haven't found it already).

[Current coverage][cc-pull] is 74.04%

Merging [#1272][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.01%

@@             master      #1272   diff @@
==========================================
  Files           192        192
  Lines         11005      11005
  Methods       10773      10781     +8
  Messages          0          0
  Branches        105         97     -8
==========================================
- Hits           8151       8149     -2
- Misses         2854       2856     +2
  Partials          0          0

Powered by Codecov. Last updated by [a7eb0a7...8a54256][cc-compare]
[cc-base-branch]: https://codecov.io/gh/quasar-analytics/quasar/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/quasar-analytics/quasar/compare/a7eb0a775913f8ee59bf63847876490e6169ec09...8a542563229701b92ae874f815296f5761478052?src=pr
[cc-pull]: https://codecov.io/gh/quasar-analytics/quasar/pull/1272?src=pr

I can do that, but I'm not used to merging my own pull requests, and I'm not completely clear on what the script accomplishes. Can you perhaps merge this one if that's long enough to allow others to comment, and I'll look to acquire a clearer idea of prevailing policy?

@paulp Sure I can merge, would you mind adding a file named revision.md in the CHANGELOG directory containing a line(s) describing the change(s) (see the other files in there for examples).

You’ll need to add a “CHANGELOG/revision.md” (literally called “revision” – the merge script renames it to the correct version number) containing a bullet point or more about what’s changed.
The script also ensures that you’ve rebased on master (but mostly it handles incrementing versions properly, since we publish a release on every merge). And we tend to squash commits – doesn’t have to be a single commit, but logical pieces.

As for squashing, I think these commits are fine as-is. Or at least I'd like for the existential one to survive for the nice commit message.

I'll push another one as described but my initial reaction is that it is largely busywork and better suited to tooling. I'll follow that up independently.

I rebased on master as well - it's unwise that we have so thoroughly normalized the practice of force pushing, but the way github has structured things there are no good alternatives.


Non-docker iptables rules are always removed

Scenario: docker daemon with --iptables=false --ip-forward=true --icc=false
After a container is created I added some iptables rules to/from another container.
When this container is killed and removed, those rules have disappeared.
I would expect (stale) iptables rules to stay there because docker did not create them in first place. Is this correct behaviour when --iptables is false?

I think this should be fixed now that https://github.com/docker/docker/pull/7003 was merged @gdm85 can you make sure?

@jfrazelle not sure when, but at some point it has been added this (reasonable) error when one tries to start docker daemon with arguments as OP:
You specified --iptables=false with --icc=false. ICC uses iptables to function. Please set --icc or --iptables to true.

Thus somehow it's no more possible to verify this problem ;)


New select list group

my SelectList PR keeps failing from an OOM issue for an unrelated test. using this PR to see if that PR is in a bad state for some reason

ok, guess the issue isn't specific to that PR 😕


Design a suite of Tests for DedupliFHIR

Design a suite of Tests for DedupliFHIR
Problem
There is currently only one test written for the DeupliFHIR backend.
Solution
Write a suite of tests to test specific inputs as well as test the efficacy of the underlying Splink algorithm to work on our generated data.
Result
There are now a bunch of new Pytest tests.
Test Plan
Run make test

Hmmmm there are 2 tests that are failing on my end:
______________________________________________ test_deduplicate_with_provided_data _______________________________________________

args = ()
kwargs = {'dedup_test_data':                             id  truth_value family_name given_name  ...            city state post...rance_Weber.xml"          NaN       weber   terrance  ...   South Quinton    NE       68843 NaN

[5 rows x 12 columns]}

    @wraps(func)
    def wrapper(*args,**kwargs):
>       fmt = kwargs['fmt']
E       KeyError: 'fmt'

deduplifhirLib/utils.py:140: KeyError
======================================================== warnings summary ========================================================
../.venv/lib/python3.11/site-packages/splink/blocking_rules_library.py:176: 6 warnings
cli/deduplifhirLib/tests.py: 20 warnings
  /Users/natalialuzuriaga/Desktop/ecqm-dedupe/.venv/lib/python3.11/site-packages/splink/blocking_rules_library.py:176: DeprecationWarning: `exact_match_rule` is deprecated; use `block_on`
    em_rules = [_exact_match(col) for col in col_names]

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
==================================================== short test summary info =====================================================
FAILED deduplifhirLib/tests.py::test_with_different_data_sizes[100-0.1] - splink.exceptions.EMTrainingException: Training rule `(l."street_address" = r."street_address") AND (l."postal_code" = r."pos...
FAILED deduplifhirLib/tests.py::test_deduplicate_with_provided_data - KeyError: 'fmt'
====================================== 2 failed, 6 passed, 26 warnings in 64.16s (0:01:04) =======================================


Help with corrupted bootloader

Hi Ardnew,
when I want to execute your script I got :"error: missing required file(s)". I would like to ask for your help and thanks for post about portenta h7 programming.
Kind regards
Filip

I see, looks like I left some local edits in the script last time it was committed. You probably only need to change the following two lines in flash-bootloader.sh#L17-L18:
#openocd_path="${tools_path}/openocd"
openocd_path="/usr/local/openocd"

to:
openocd_path="${tools_path}/openocd"
#openocd_path="/usr/local/openocd"

I'll go ahead and commit a fix, thanks for the feedback!

Thanks for your help, but still not working. Below logs from console:
Setting boot option bytes (boot Cortex-M7, gate Cortex-M4) ...
Open On-Chip Debugger 0.11.0+dev-00158-gd3a859cc4-dirty (2021-05-19-17:45)
Licensed under GNU GPL v2
For bug reports, read
http://openocd.org/doc/doxygen/bugs.html
Info : auto-selecting first available session transport "hla_swd". To override use 'transport select '.
Info : The selected transport took over low-level target control. The results might differ compared to plain JTAG/SWD
Info : clock speed 1800 kHz
Info : STLINK V2J37S7 (API v2) VID:PID 0483:3748
Info : Target voltage: 3.133807
Error: init mode failed (unable to connect to the target)


416 Range Not Satisfiable

https://github.com/weihanglo/sfz/blob/4dbd5fb51b9a084ac755baf46c100603113aa576/src/server/send.rs#L206

Reference: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/416
Specification: https://httpwg.org/specs/rfc7233.html#status.416

Thank you for the PR. Unfortunately I have no time working on it. See #108.


Problems building mxnet for AMD HIP

I post this awhile ago at ROCmSoftwarePlatform/mxnet and then realized that this this location was referenced for help.  Here is the repost...
Environment info
Operating System:
Ubuntu 16.04.3 LTS
Compiler:
hipcc / hcc (clang 6, see version output below)
hipcc --version
HIP version: 1.4.17494
HCC clang version 6.0.0 (ssh://gerritgit/compute/ec/hcc-tot/clang 42ceed861a212d9bd0aef883ee7981144f3ecc02) (ssh://gerritgit/compute/ec/hcc-tot/llvm 23e086be6f627e6e983c6789d2e77da6bf85ebb6) (based on HCC 1.1.17493-2f85d8a-42ceed8-23e086b )
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /opt/rocm/hcc/bin
Package used (Python/R/Scala/Julia):
MXNet version:
Or if installed from source:
MXNet commit hash (git rev-parse HEAD):
d053ae8
If you are using python package, please provide
Python version and distribution:
If you are using R package, please provide
R sessionInfo():
Error Message:
Please paste the full error message, including stack trace.
The initial issue was that with the latest rocm (1.7.60) install from the repositories there was problem with rocBLAS and hcRNG was missing so I built them from git. hcFFT was available as expected. At this point mxnet appear to compile but multiple errors reported I'm attached a build log from the second build attempt so it is less noisy.
I am also using cuda 9.1 but I did try cuda 8 which also failed. The environment vars in both cases were:
LD_LIBRARY_PATH=/usr/local/cuda/lib64 (this symlinked to 8 or 9.1 depending on what is installed)
HIP_PLATFORM=hcc
The current git version of mxnet also do not need the Makefile modification presented since it is always there.
build.log
Minimum reproducible example
if you are using your own code, please provide a short script that reproduces the error.
Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.
1.make -j $(nproc)
2.
3.
What have you tried to solve it?
The first stoppage in the log...
41 warnings and 2 errors generated.
Died at /opt/rocm/bin/hipcc line 500
...refers to a line in the hipcc script...
495 if ($runCmd) {
496 if ($HIP_PLATFORM eq "hcc" and exists($hipConfig{'HCC_VERSION'}) and $HCC_VERSION ne $hipConfig{'HCC_VERSION'}) {
497 print ("HIP ($HIP_PATH) was built using hcc $hipConfig{'HCC_VERSION'}, but you are using $HCC_HOME/hcc with version $HCC_VERSION from hipcc. Please rebuild HIP including cmake or update HCC_HOME variable.\n") ;
498 die unless $ENV{'HIP_IGNORE_HCC_VERSION'};
499 }
500 system ("$CMD") and die ();
501 }
However, my HIP configuration appears to be good...
hipconfig
HIP version : 1.4.17494
== hipconfig
HIP_PATH : /opt/rocm
HIP_PLATFORM : hcc
CPP_CONFIG : -D__HIP_PLATFORM_HCC__= -I/opt/rocm/include -I/opt/rocm/hcc/include
== hcc
HSA_PATH : /opt/rocm/hsa
HCC_HOME : /opt/rocm/hcc
HCC clang version 6.0.0 (ssh://gerritgit/compute/ec/hcc-tot/clang 42ceed861a212d9bd0aef883ee7981144f3ecc02) (ssh://gerritgit/compute/ec/hcc-tot/llvm 23e086be6f627e6e983c6789d2e77da6bf85ebb6) (based on HCC 1.1.17493-2f85d8a-42ceed8-23e086b )
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /opt/rocm/hcc/bin
LLVM (http://llvm.org/):
LLVM version 6.0.0svn
Optimized build.
Default target: x86_64-unknown-linux-gnu
Host CPU: znver1
Registered Targets:
amdgcn - AMD GCN GPUs
r600 - AMD GPUs HD2XXX-HD6XXX
x86 - 32-bit X86: Pentium-Pro and above
x86-64 - 64-bit X86: EM64T and AMD64
HCC-cxxflags : -hc -std=c++amp -I/opt/rocm/hcc-1.0/include -I/opt/rocm/includeHCC-ldflags : -hc -std=c++amp -L/opt/rocm/hcc-1.0/lib -Wl,--rpath=/opt/rocm/hcc-1.0/lib -ldl -lm -lpthread -lunwind -lhc_am -Wl,--whole-archive -lmcwamp -Wl,--no-whole-archive
=== Environment Variables
PATH=/opt/rocm/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
LD_LIBRARY_PATH=/usr/local/cuda/lib64
HIP_PLATFORM=hcc
== Linux Kernel
Hostname :
Linux 4.4.0-109-generic #132-Ubuntu SMP Tue Jan 9 19:52:39 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.3 LTS
Release:	16.04
Codename:	xenial
~ ~ ~
I'm not sure what to try next. My guess is that there are some function differences between mxnet code and the larger requirements but I don't know how to resolve that.

are you building from mxnet master? I don't think we have HIP support

I'm building following the instructions on ROCm's github.  Installation for the HIP port are there but say reference here for issues.
https://github.com/ROCmSoftwarePlatform/mxnet


Infinite loading on open

Expected Behavior
Main menu should open and display agile boards
Actual Behavior
Infinite loading
Steps to Reproduce the Problem

Create my account
Token
youtrack url

Specifications

T-REC Version: last one from release
Platform: windows

sorry for long response. I will try to verify this problem soon

Probably similar issue.
/rest/admin/agile responds 404
YouTrack version 2019.1 build 51932. Looks like you use old YT rest API(endpoints starts with /rest, new ones with /api) which is no longer actively maintained and will be deprecated soon https://www.jetbrains.com/help/youtrack/standalone/deprecated-rest-api-reference.html
Anyway i cant find /rest/admin/agile endpoint in old API docs.

https://github.com/hanzhinstas/timetracker/releases

This is related to #55, we are working on 1.4.0 release with fixes for temporary solution please use @arioki1  version


Existing locations to place Blackjack table

I will update this list as we pitch ideas:

Diamond City (upper stands bar)

Goodneighbor (Hotel Rexford)

Far Harbor

Nuka World

Combat Zone

Diamond City and Goodneighbor are good locations. I'm not sure about Far Harbor or Nuka World because I haven't played them much. Combat Zone sounds cool but I have never gone there without killing everyone.

I haven't played Nuka World much, but Far Harbor is a good place for it, the bar there at least.

Far Harbor sounds good if you know of a fitting bar there. The combat zone sounds good too if you can sneak the gambling room in. I'm sure we can find one for nuka world too if we poke around.

Ya, the combat zone would take a little more quest work than the others, so maybe that could wait until after the Alpha release

I'm cleaning up this thread and editing the list.

Fixed vault 81 dealer quest with commit below. Still chugging away on NPC's
https://github.com/Scrivener07/FO4_Games/commit/06e0cb6dbfa99912eeb5d9a9bc36587f948b4460

Prydwen dealer implemented with commit below. I have a concern there might be some missing files that I need to stage, so outside testing would be helpful.
https://github.com/Scrivener07/FO4_Games/commit/e03f6b5bff88480902d44471855755057425da65

Last commonwealth NPC is implemented! Also added missing actor files. With the commit below, commonwealth implementation is essentially done (minor tweaks needed like adding voice files). Leaving this issue open to track implementation of DLC area NPC's. DLC NPC's will be a new branch and a separate esp.
https://github.com/Scrivener07/FO4_Games/commit/7d6e37feb89aa961f901d4caac08b9badc595835

Starting on Far Harbor, did some of it via stream for Cascadia so I have to go back and do dialogue

Harry-Dugout Inn-Diamond City

Mr. Wallace-Colonial Taphouse-Diamond City

Douglas-Third Rail-Goodneighbor

Jackson-Diner-Vault81

Scribe Harrison-Mess Hall-Prydwen Airship

Toby-Railroad HQ-Old North Church

Far Harbor implementation is complete with this commit. It is ready to go out whenever we want to do that. Nuka World and Merged DLC plugin still to come.
https://github.com/Scrivener07/FO4_Games/commit/7a72d887c8b9471e979e740df72f909f13b9ac48

Terry 'Busch' Baldwater-Last Plank-Town of Far Harbor

Elliott-Acadia-Island of Far Harbor

Brother Julius-Nucleus-Island of Far Harbor

Max-Pack Den-Nuka World Hub

Zilch-Operator Hideout-Nuka World Hub

Rufio-Fizztop Mountain-Nuka World Hub

3 Gang specific NPC's complete. 2 NPC's left.
https://github.com/Scrivener07/FO4_Games/commit/16fc6b3f54b3e34d0502e50db9abb72288dc54f2

Removing the Hubologist NPC from this list, they have that big quest that is very specific to how many people they have so it wouldn't fit right, one NPC for NW left.

Sammy-Market-Nuka Town USA

Nuka World Implementation is complete with this commit, closing this issue!
https://github.com/Scrivener07/FO4_Games/commit/86e47baf5bb1ccc6a0d1655ab5c1e53593d0ab68


Docker: updated x86_64.stable images with CUDA-11.1 to support Nvidia 30x0 (Ampere) GPUs

Updated libtorch installer to fix the issue that Apollo-simulator in CPU mode didn't work properly.

Updated libtorch installer to fix the issue that Apollo-simulator in CPU mode didn't work properly.


srcPath should be relative to project the plugin is applied to

Hi,
I'm using this plugin in a multi-project setup. In my setup, this multi project repo hosts multiple projects with different versions each.
Currently, no matter which project I apply the plugin to, the version file is always generated relative to the root project instead of the project it's actually applied to, so I had to work around this:
srcDir = project.rootDir.relativePath(project.file('src/main/kotlin')).normalize()
However, I think it would make more sense that by default the srcDir would be relative to the project it's applied to.

I agree and done. let me know if 1.5.0 works for you.

Works great, thanks!


Test failure when following the README

I have followed the README file and when I try to run tests with pytest I get the following error:
(env) user@users-MBP cairo-contracts % pytest
========================================================= test session starts =========================================================
platform darwin -- Python 3.8.7, pytest-6.2.5, py-1.10.0, pluggy-1.0.0
rootdir: /Users/user/dev/cairo-contracts
plugins: web3-5.24.0, hypothesis-6.24.0, eth-brownie-1.17.1, xdist-1.34.0, forked-1.3.0
collected 0 items / 5 errors

=============================================================== ERRORS ================================================================
_______________________________________________ ERROR collecting tests/test_Account.py ________________________________________________
ImportError while importing test module '/Users/user/dev/cairo-contracts/tests/test_Account.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
../../.pyenv/versions/3.8.7/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/test_Account.py:3: in <module>
    from starkware.starknet.testing.starknet import Starknet
E   ModuleNotFoundError: No module named 'starkware'

shouldn't nile install take care of the dependencies?

Did you run nile init or nile install?
On Tue, 30 Nov 2021 at 12:48 poolpitako @.***> wrote:

I have followed the README file and when I try to run tests with pytest I
get the following error:
(env) @.*** cairo-contracts % pytest
========================================================= test session starts =========================================================
platform darwin -- Python 3.8.7, pytest-6.2.5, py-1.10.0, pluggy-1.0.0
rootdir: /Users/user/dev/cairo-contracts
plugins: web3-5.24.0, hypothesis-6.24.0, eth-brownie-1.17.1, xdist-1.34.0, forked-1.3.0
collected 0 items / 5 errors
=============================================================== ERRORS ================================================================
_______________________________________________ ERROR collecting tests/test_Account.py ________________________________________________
ImportError while importing test module '/Users/user/dev/cairo-contracts/tests/test_Account.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
../../.pyenv/versions/3.8.7/lib/python3.8/importlib/init.py:127: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
tests/test_Account.py:3: in
from starkware.starknet.testing.starknet import Starknet
E   ModuleNotFoundError: No module named 'starkware'
shouldn't nile install take care of the dependencies?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenZeppelin/cairo-contracts/issues/93, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM77F2DQI4IBA7LMMGPP7DUOTW3XANCNFSM5JCE2XDQ
.

Did you run nile init or nile install?
…
On Tue, 30 Nov 2021 at 12:48 poolpitako @.> wrote: I have followed the README file and when I try to run tests with pytest I get the following error: (env) @. cairo-contracts % pytest ========================================================= test session starts ========================================================= platform darwin -- Python 3.8.7, pytest-6.2.5, py-1.10.0, pluggy-1.0.0 rootdir: /Users/user/dev/cairo-contracts plugins: web3-5.24.0, hypothesis-6.24.0, eth-brownie-1.17.1, xdist-1.34.0, forked-1.3.0 collected 0 items / 5 errors =============================================================== ERRORS ================================================================ _______________________________________________ ERROR collecting tests/test_Account.py ________________________________________________ ImportError while importing test module '/Users/user/dev/cairo-contracts/tests/test_Account.py'. Hint: make sure your test modules/packages have valid Python names. Traceback: ../../.pyenv/versions/3.8.7/lib/python3.8/importlib/init.py:127: in import_module return _bootstrap._gcd_import(name[level:], package, level) tests/test_Account.py:3: in  from starkware.starknet.testing.starknet import Starknet E ModuleNotFoundError: No module named 'starkware' shouldn't nile install take care of the dependencies? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#93>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAM77F2DQI4IBA7LMMGPP7DUOTW3XANCNFSM5JCE2XDQ .

yep

OK, I found the problem.
% which pytest
/Users/user/.pyenv/shims/pytest

a surprise since my PATH starts with:
/Users/user/dev/cairo-contracts/env/bin
Using the full path fixes the problem, basically: /Users/user/dev/cairo-contracts/env/bin/pytest or a shorter version:
python -m pytest

how did you activate the virtual env?

how did you activate the virtual env?

great point. I just re-run source env/bin/activate and it worked.
I guess the issue is that after nile install the env should be activated again 🤷

No, it should not be activate again. nile install has to be run within it in the first place.

No, it should not be activated again. nile install has to be run within it in the first place.

that's how I run it. env was activated when nile install was run.
mac user with pyenv installed if you want to repro.

Mac user too, cannot reproduce :(
Are you able to reproduce it?

Mac user too, cannot reproduce :(
Are you able to reproduce it?

Env started working fine after the second source env/bin/activate and I haven't had the issue again.
It's fine, let's move on :)


WDCC not exported anymore

When exporting a repository with webdynpro component configurations, they are not listed in the repository. It seems as if the existence-check doesn't work properly - check generated where-clause

where-clause CONFIG_ID = 'ZMSG_RM_CREA_ZMSG_ROOM_ROOT 00' is wrong: it should be two different attributes addressed

If the key of the object's primary table is composed of multiple fields, /& addresses all of them - although the object key has all of them concatenated.


add markdown output for github actions display

example: https://github.com/UQ-PAC/BASIL/actions/runs/9884154629/attempts/1#summary-27300079028

Thanks this is great, it would be really good to have a statistical summary of the verification time as well, if you don't want to implement that I can do it

I'll leave that to you :) Let me know if there's any problems with the code.

patch to support more test suites: https://gist.github.com/katrinafyi/59a10338f58016bea0a84a34a81c7229


[iOS Verifier App] Verifiable Credentials with Matching KID But False Public Key Data Are Accepted

Describe the bug
We scanned verifiable clredentials with matching KID but manipulated public key data so that the signature was no more valid for the issuer with this KID.  The app scanned these without any error.
Expected behaviour
The app should recognize  that the credential has an invalid signature.

Technical details
Verifier App <IP_ADDRESS>(33) Hash & iOS Verifier 1.3.1 (31) Bloom

Copy of https://github.com/eu-digital-green-certificates/dgca-wallet-app-ios/issues/138

05.05.2022: Retest with iOS Verifier 1.3.1(34) was not successful. Actual result: "Barcode-Fehler"

tvt, 10.05.2022: Priority reduced from Medium to Low, because the described failed acception of the wrong Credential  is not occured anymore. Just only a not expected (consistent to Android) error messaged is now displayed.

tvt, 10.05.2022: retested successully by Ralica with IOS-version <IP_ADDRESS> (42) HL and 1.3.1 (38) BF.


Update zh_TW.json

Update zh_TW.json

Classic merge conflict

sec

You know what fix your branch first 👍


Could not load list of method overrides due to Method not found: int SQLitePCL.ISQLite3Provider.sqlite3_win32_set_directory(int,string)'

Since my last update to version 2.0.0 I didn't get the SQLitePCLRaw to run on Xamarin for Android.
At the point of SQLitePCL.Batteries_V2.Init() I got the exception as in the header meant.
Before I removed the SQLitePCLRaw.provider.e_sqlite3.android package because 2.0.0 isn't supported and I ran into the exception "Failed to resolve System.String SQLitePCL.raw::sqlite3_column_name(SQLitePCL.sqlite3_stmt,System.Int32)" if the sqlite-net-pcl calls the raw api.
"sqlite-net-pcl" 1.6.258-beta
"SQLitePCLRaw.bundle_green" 2.0.1
"SQLitePCLRaw.core" 2.0.1
"SQLitePCLRaw.lib.e_sqlite3" 2.0.1
"SQLitePCLRaw.lib.e_sqlite3.android" 2.0.1

sqlite-net has not yet been updated for compatibility with SQLitePCLRaw 2.0.  If you're using sqlite-net, you need to stay with SQLitePCLRaw 1.1.14 for now.

and what's about the SQLitePCLRaw.provider.e_sqlite3.android package? Should there be version 2.0 in future?

The SQLitePCLRaw.provider.e_sqlite3.android package does not exist in 2.0, but the same functionality is provided by other packages.
Since 2.0 reduces the number of packages, the easiest way to upgrade is to remove all the SQLitePCLRaw packages and then bring in one of the SQLitePCLRaw.bundle_ packages and allow it to transitively bring in its dependencies.  You will end up with different package ids, but the functionality is the same.
FWIW, this (and all nuget package operations) works best with PackageReference instead of packages.config.

The SQLitePCLRaw.provider.e_sqlite3.android package does not exist in 2.0, but the same functionality is provided by other packages.
Since 2.0 reduces the number of packages, the easiest way to upgrade is to remove all the SQLitePCLRaw packages and then bring in one of the SQLitePCLRaw.bundle_ packages and allow it to transitively bring in its dependencies. You will end up with different package ids, but the functionality is the same.
FWIW, this (and all nuget package operations) works best with PackageReference instead of packages.config.

Hi,
in my application I'm using microsoft.data.sqlite.core 5.0.5 that requires SQLitePCLRaw.core >= 2.0.4.
How can I make all things works?
If I update to microsoft.data.sqlite.core verision 5.0.5 the error
Could not load list of method overrides due to Method not found: int SQLitePCL.ISQLite3Provider.sqlite3_win32_set_directory(int,string)
is trigger.
I can't use the older version.
Thank you.

If the sqlite3_win32_set_directory() function is only present on Windows, why this wrapper tries to get its pointer on other platforms?
Check here
This makes it impossible to use the dynamic provider on Mac
Also, it should not fallback to another sqlite library if the one I am trying to load fails due to the above. In another words: the wrapper is loading another instance, not the one I am asking it to load

Closing old/stale issue.


added detailed guide in echo readme.md example folder

I have just updated echo readme,nd example files because most go users are confused including please accept my pr, you can also edit it a little bit if you want or let me know if you need anything else, thanks for the lib

why checks are failing i just added text in example docs..... i am new let me know if anyone sees my comment here

@nhooyr ...

Closing, feel free to open a new PR or issue if there's a specific concern you'd like addressed.


Checklist: Add New Check for AMP errors

Context
Based on #6184 to show when there are AMP violations in a story that has been published.
Summary
Adds a new check for if there are AMP errors present in a story. There must be either a story link present or the status is published/scheduled.

Relevant Technical Choices

Pascal did most of this in his proof of concept (#6184), I implemented in the new checklist structure.
Amp Error Validation is a new priority check. It updates on initial render and then whenever the isSaving prop from the story becomes false again. Right now, the actual check for errors will only run with a story has been published or scheduled. This is an intentional choice because 'saving' fixes most if not all errors with the AMP code.
We are filtering out errors in the validator that are checked internally. These are: missing story poster, missing publisher logo, missing video posters. The other priority check that is covered here is the story title, but AMP allows an empty title, so our check is better and that error never sees the light of day for the validator (as far as I can tell)
I have put this check behind its own feature flag because I'm not sure all of the issues we're filtering for are covered yet and it feels like something we might want to have the ability to exclude. Experiment is called enableAmpValidationCheck, set it to default true.

Fix three small bugs in checklist while making sense of the logic here:

the storyPageCount was digging one level too deep for pages and never rendering - pretty sure this was just a sneaky rebase
publish/update button was keeping the exclamation icon and tooltip even when all checkpoints were hit and issues resolved. Limiting based on if the dialog should appear when checkpoint ALL is hit (also updating the null to an empty string for tooltip to silence a console warning about the propType.
update Panel sections to only set isExpanded to true when there's an accompanying issue count and the isOpen boolean is true, this was causing a little hiccup where if a section was expanded when all issues were solved and now the checklist is empty the helper text associated with that panel wouldn't be hidden.

To-do

[x] Need to add some tests for the check
Look into Web Assembly implementation (new ticket here: https://github.com/google/web-stories-wp/issues/8477)
[x] Finalize copy
[ ] poster images for videos should trigger an error but can't find it!

User-facing changes
If you have AMP errors in your published/scheduled story you see this:

Testing Instructions
Have a bad story and see the error. This is surprisingly hard to trigger.
this is a list of AMP errors: https://amp.dev/documentation/guides-and-tutorials/learn/validation-workflow/validation_errors/?format=websites#amp-html-tag-and-attribute-errors  if you find a way to get this to fire please share! I was having luck before moving the check to be post publish. All errors I've found get resolved on save.
QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?
No
Does this PR change what data or activity we track or use?

I added a new trackClick (click_checklist_amp_test) for the link that sends users to https://search.google.com/test/amp to test their story and get more robust details about what went wrong in their story. I figure this is useful to see if people are following through on testing stories with issues or if we need a more involved solution for walking users through fixing these errors.
Also added a new trackEvent (amp_validation_errors_found) which fires once per session when amp validation errors are found.

Does this PR have a legal-related impact?
No
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #8442

@BrittanyIRL FYI there are now some merge conflicts


.NET Runtime Exception

Microsoft PowerToys version
0.41.3
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
I turned on my Surface Pro running Windows 10 Pro version 21H1, was received with an application exception dialog.
✔ Expected Behavior
There should not be any exception dialog
❌ Actual Behavior
Application: PowerLauncher.exe
CoreCLR Version: 4.700.21.26205
.NET Core Version: 3.1.16
Description: The process was terminated due to an unhandled exception.
Exception Info: System.TypeInitializationException: The type initializer for 'System.Windows.Threading.Dispatcher' threw an exception.
---> System.ComponentModel.Win32Exception (5): Access is denied.
at System.Diagnostics.ProcessManager.OpenProcess(Int32 processId, Int32 access, Boolean throwIfExited)
at System.Diagnostics.NtProcessManager.GetModules(Int32 processId, Boolean firstModuleOnly)
at System.Diagnostics.Process.get_Modules()
at System.Windows.WpfDllVerifier.VerifyWpfDllSet(String[] additionalDlls)
at System.Windows.Threading.Dispatcher..cctor()
--- End of inner exception stack trace ---
at System.Windows.Threading.Dispatcher.get_CurrentDispatcher()
at System.Windows.Threading.DispatcherObject..ctor()
at System.Windows.Application..ctor()
at PowerLauncher.App..ctor()
at PowerLauncher.App.Main()
Other Software
Windows 10 Pro 21H1 OS build 19043.1080

Looks like a duplicate of #12035

Yes we have a hot-fix coming soon to resolve this issue. Thank you for reporting!


build(android): update gradle

Signed-off-by: pingkai<EMAIL_ADDRESS>

Codecov Report

Merging #370 into develop will decrease coverage by 3.98%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #370      +/-   ##
===========================================
- Coverage    69.18%   65.20%   -3.99%
===========================================
  Files          214      214
  Lines        11706    11706
===========================================
- Hits          8099     7633     -466
- Misses        3607     4073     +466

Impacted Files
Coverage Δ

framework/tests/dataSource/ipList.h
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/demuxer/demuxerUnitTest.cpp
0.00% <0.00%> (-100.00%)
:arrow_down:

framework/tests/decoder/decoderUnitTest.cpp
0.00% <0.00%> (-93.62%)
:arrow_down:

framework/tests/dataSource/dataSourceUnitTest.cpp
0.00% <0.00%> (-93.39%)
:arrow_down:

framework/tests/demuxer/demuxerUtils.cpp
0.00% <0.00%> (-76.00%)
:arrow_down:

framework/data_source/ffmpeg_data_source.cpp
6.38% <0.00%> (-51.07%)
:arrow_down:

framework/demuxer/AVBSF.cpp
25.27% <0.00%> (-42.86%)
:arrow_down:

framework/data_source/curl/CURLShareInstance.cpp
69.44% <0.00%> (-27.78%)
:arrow_down:

framework/data_source/curl/CURLConnection.cpp
53.19% <0.00%> (-11.34%)
:arrow_down:

framework/data_source/ffmpeg_data_source.h
80.00% <0.00%> (-10.00%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

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


Fix dashed link lines in topology and group them in css

@miq-bot add_label bug, refactoring, ui, providers/containers

@epwinchell @martinpovolny please review

@miq-bot add_label topology


Create JSON schema

Create a JSON schema and add it to https://github.com/SchemaStore/schemastore so there will be support for validating / auto-completing configuration options within code editors.
For example, the YAML VS Code extension supports it.

I have made a PR for SchemaStore adding this: https://github.com/SchemaStore/schemastore/pull/3066

Thank you so much @hyperupcall! I appreciate your work! :tada:

That PR has been merged (accessible at https://json.schemastore.org/lefthook.json) - should this issue be closed now?

Thank you !


Update wm-powerbroadcast.md

Markdown fix

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


aws_profile segment added

Added AWS_DEFAULT_PROFILE segment for AWS CLI usage to support showing current profile.  Profile setup is defined in the AWS docs .

Just merged -- finally. Sorry it took so long. Will be released soon.

And thanks for the PR!


Add support for local PHP namespaces

Add support for local PHP namespaces. See https://github.com/fiatjaf/module-linker/issues/46#issuecomment-511207887 for details how it works.
Here are some PHP projects if you want to test this:

https://github.com/filips123/ConfigWriter/blob/v2/src/Config.php
https://github.com/Seldaek/monolog/blob/master/src/Monolog/Logger.php
https://github.com/sebastianbergmann/phpunit/blob/master/src/Util/Configuration.php
https://github.com/slimphp/Slim/blob/3.x/Slim/Container.php
https://github.com/laravel/framework/blob/5.8/src/Illuminate/Console/Application.php

This is great. You're awesome.


New package: Xencelabs.Driver version 1.3.0-39

Pull request has been created with komac v2.0.3 :rocket:

Resolve: #137776

Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！


I am having the same issue like @CarsonChen1129 . After i updated ionic-angular from 2.0.0-rc3, content.scrollTo stopped working.

Originally posted by @BViksa in https://github.com/ionic-team/ionic/issues/10521#issuecomment-284703645

Found solution
Replace this.content.scrollTo(...) with this.content._scrollContent.nativeElement.scrollTo(...)


Jeromegn dbversions count

clean up history and remove merge commits from https://github.com/vlcn-io/cr-sqlite/pull/310

This is a pretty big regression in terms of regular insert and update performance.
E.g.,
INSERT INTO base_table VALUES (...)
UPDATE base_table SET .... WHERE ...

Update Perf
Before

After

Insert Perf
Before

After

@jeromegn - what do you think of making this an optional bit of metadata to track?

Optional as a compile time option

Optional as a compile time option

That would work for us as a temporary workaround for the performance regression, but I'd say we'd eventually want to be able to profit from the full performance of cr-sqlite.
Any pointers on how to do that? I guess I can add a compile-time feature.

Looking at this some more: does the benchmark use a WAL journal? I didn't eyeball such a big regression in my manual testing, but maybe cr-sqlite got better before and I didn't get a feel for the new perf before my change.


Fix: disable power valid checks

This commit allows to ignore the battery prearm checks when CBRK_SUPPLY_CHK is disabled

Makes sense, did you check the other battery triggers as well?


DNS resolution

I have wsl-vpnkit working as root user. However, I am facing one issue. Name resolution works as root user, but not as a non privileged user. I am using the wsl-vpnkit default gateway in the resolve.conf file per the instructions.

Does it work for a normal user if you set a different DNS server like <IP_ADDRESS> in /etc/resolv.conf?

I tried several dns servers, <IP_ADDRESS> along with corporate dns servers.
None of these worked except for. nameserver <IP_ADDRESS>
As a normal user I can ping <IP_ADDRESS> and I can ping ips on corporate vpn. However, name lookups will not work as normal user.

Try running sudo nslookup example.com and nslookup example.com multiple times. Do they use the same servers?

silly mistake I had permissions 600 on /etc/resolv.conf . Thanks for the suggestions.