Create .pre-commit-hooks.yaml

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature: pre-commit is a standard on how to integrate your repository with commands running before commit.

What is the current behaviour? (You can also link to an open issue here)

Closes: #493

What is the new behaviour (if this is a feature change)?

Easier use.

Other information:

If the gh-super-linter can use this, then pre-commit should be a no-brainer.

This change is

@kucherenko can you do a new release/tag as well? Or cherry-pick this file back onto the v3.5.3 tag? Else, pre-commit can't pick it up.

@killermoehre<EMAIL_ADDRESS>just released

Hmm, interesting. It breaks. @kucherenko
$ pre-commit run jscpd
[INFO] Initializing environment for https://github.com/kucherenko/jscpd.
[INFO] Installing environment for https://github.com/kucherenko/jscpd.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('~/.cache/pre-commit/reporrbwearr/node_env-system/bin/node', '/opt/homebrew/bin/npm', 'install', '--dev', '--prod', '--ignore-prepublish', '--no-progress', '--no-save')
return code: 1
stdout: (none)
stderr:
    npm WARN config dev Please use --include=dev instead.
    npm WARN config production Use `--omit=dev` instead.
    npm ERR! code ERESOLVE
    npm ERR! ERESOLVE unable to resolve dependency tree
    npm ERR!
    npm ERR! While resolving<EMAIL_ADDRESS>    npm ERR! Found<EMAIL_ADDRESS>    npm ERR! node_modules/typedoc
    npm ERR!   dev<EMAIL_ADDRESS>from the root project
    npm ERR!
    npm ERR! Could not resolve dependency:
    npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>    npm ERR! node_modules/typedoc-plugin-lerna-packages
    npm ERR!   dev<EMAIL_ADDRESS>from the root project
    npm ERR!
    npm ERR! Fix the upstream dependency conflict, or retry
    npm ERR! this command with --force or --legacy-peer-deps
    npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
    npm ERR!
    npm ERR!
    npm ERR! For a full report see:
    npm ERR! ~/.npm/_logs/2023-03-28T14_50_28_710Z-eresolve-report.txt

    npm ERR!  There is no magic here, as far as I understand node modules.
Also, cd packages/jscpd && npm install . still fails with ERESOLVE.

Ha, I got a solution. Will open another PR. Thank you for your patience.


Add an automated test

Use the Kiln framework to add at least one automated test to this repository

@BryceStevenWilley I am setting up some automated tracking projects. If this already has a test we can close the issue; the automation will be a bit overbroad although I am filtering it to repos that are published on courtformsonline.org.


Accept config as a JS object

This feature was previously discussed here: https://github.com/tailwindcss/discuss/issues/37
I needed this for a custom gulp build script where the user is able to override parts of the Tailwind config using a JSON file, so the final config is built dynamically rather than contained in a single file. I am not sure if I implemented it properly, as I didn't take the time to go deep in the codebase, but this worked for me and all the tests passed. I'm sorry if I missed anything; if so please let me know. Also I imagine it would be a good idea to add a test for this feature.

Hmm, also sorry for the three commits. I pushed each commit immediately after creating it, so I am not sure how I could squash them?

tailwind's configuration file exports the configuration, why can't you just merge the user config with the exported tailwind setup?

@phroggyy Yes, I'm doing that. But then there was no way to pass that merged config object to Tailwind, because it only accepted a path to a file. I could write it to a temporary file, but it seems like a better solution to make Tailwind accept either a file or an object.

Hey @benface any chance you can update this PR or resubmit without the package-lock.json changes?

Sure thing @adamwathan, done in #508


Abstractions to support other dependency injection providers?

I note that in the documentation you state that:

This library relies on the Microsoft.Extensions.DependencyInjection, so it only works with DI containers that integrate with those abstractions.

Have you considered structuring the library in such a way that use with alternative abstractions would be possible (if not supported currently)? This would allow extensions to be written to support additional DI containers.
The reason I ask is that I am also in the process of writing a mediator library using source generators, but am not nearly as far along in my endeavors. It is my intention to support compile-time DI containers such as StrongInject, Jab and Pure.DI.
Much of the logic would be shared between the implementations (e.g. analyzing the client code to collect information about all the registered handlers and pipeline behaviors), but would differ primarily in the source which is then generated from the collected information.

It's probably possible, as you say the analysis part is likely to be unchanged, so all the changes would basically be in the codegen templates. For example the options attribute there uses the ServiceLifetime enum, whereas StrongInject has it's own enum for this.
I'm not 100% sure what the best design would be here, creating common denominator abstractions around lifetime and service location/DI or just create a base template that is included and augmented in a Mediator.SourceGenerator.StrongInject library for example (then if you installed this package, the options attribute would use the Scope enum instead). I would need to investigate this. But basically the lifetime option, current AddMediator method that is generated and service location functions (GetServices, GetRequiredService etc) would need to be changed. And obviously it increases the unittesting scope as different DI containers can provide differences in functionality.
Overall I think it's a good idea, and the reason I haven't done anything about it was for simplicity sake since I'm not currently using anything other than the built in DI, so I'll probably look into this next time I'm working on this. Let me know if you want some input on your implementation and feel free to link it 😄

@martinothamar - Many thanks for your reply. I'll also take a look over the Mediator source code to see if there is a neat way to build in the extension point. I'm conscious that writing multiple implementations would significantly increase the project scope, but it would be great if it can be done in a way which permits additional community implementations without increasing your maintenance burden!

Hi and thanks for great package!
I would love to see support for other DI providers!
I believe that for Jab it should be pretty straitghforward. Additionally - if this information is helpful - Jab supports IServiceProvider interface.
Jab has module system so probably template should look like that:
  [Jab.ServiceProviderModule]
  {{~ for message in RequestMessages ~}}
  [Jab.{{ message.Handler.ServiceLifetimeShortName }}(typeof({{ message.HandlerWrapperTypeNameWithGenericTypeArguments }}))]
  [Jab.{{ message.Handler.ServiceLifetimeShortName }}(typeof({{ message.Handler.FullName }}))]
  [Jab.{{ message.Handler.ServiceLifetimeShortName }}(typeof({{ message.PipelineHandlerType }}))]
  {{~ end ~}}
  public interface IMediatorModule
  {
  }

Where message.Handler.ServiceLifetimeShortName should be just keyword "Singleton", "Transient" or "Scoped".
Then in our application it could be used like that:
[ServiceProvider]
[Import(typeof(IMediatorModule))]
public class ServiceProvider
{
}

Also another related question is - how to distribute Mediator for other DI providers?

I have pushed working proof of work for Jab DI container on my branch.
Not so much changes in comparison to default implementation, most of them were in template, message handlers and RequestMessage class.
Here you can take a look on changes:  https://github.com/martinothamar/Mediator/commit/c9249fa39cf9572308027d3036893e6bf8b32d4b


ci: lint pr title, move lint/format to ci and rm husky

removes the husky hooks (commit msg lint, formatting, eslint) in favor of ci checks
adds node matrix to test against v16,18,20 (and updates test to be 16 compatible)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


bot isnt working on replit

help command is working but the apis interaction have some problem...
the replit getting problem with the promises due to depreciated packages...
but working properly with my terminal
any solutions?

Can you send me the error code you get?


(java) SNAPSHOT release PR is empty

See: https://github.com/googleapis/java-bigtable-hbase/pull/3574

Another example in BQ: https://github.com/googleapis/java-bigquery/pull/1980/commits/dd4a739beb258dd9f23f659f2a464b9bd0c22985


Configure Renovate

Welcome to Renovate! This is an onboarding PR to help you understand and configure settings before regular Pull Requests begin.
🚦 To activate Renovate, merge this Pull Request. To disable Renovate, simply close this Pull Request unmerged.

Detected Package Files

docker-compose-example.yaml (docker-compose)
Dockerfile (dockerfile)
pyproject.toml (poetry)

What to Expect
With your current configuration, Renovate will create 3 Pull Requests:

Update dependency ipython to v8.13.2

Schedule: ["at any time"]
Branch name: renovate/ipython-8.x-lockfile
Merge into: main
Upgrade ipython to 8.13.2

Update dependency poetry-dynamic-versioning to ^0.22.0

Schedule: ["at any time"]
Branch name: renovate/poetry-dynamic-versioning-0.x
Merge into: main
Upgrade poetry-dynamic-versioning to ^0.22.0

Update fedora Docker tag to v39

Schedule: ["at any time"]
Branch name: renovate/fedora-39.x
Merge into: main
Upgrade fedora to 39

🚸 Branch creation will be limited to maximum 2 per hour, so it doesn't swamp any CI resources or overwhelm the project. See docs for prhourlylimit for details.

❓ Got questions? Check out Renovate's Docs, particularly the Getting Started section.
If you need any further assistance then you can also request help here.

This PR has been generated by Renovate Bot.

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
*Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Support only merged PRs

To re-run tox comment /tox in the PR.
To re-run build-container command /build-container in the PR.
To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
To re-run python-module-install command /python-module-install in the PR.
To add a label by comment use /<label name>, to remove, use /<label name> cancel

Supported labels

hold
verified
wip
lgtm
approve
target-version-


[sui framework] support limited reflection over types

Move allows type-safe implementations of key types like fungible tokens using the Coin<T> standard. However, distinguishing coins at the type level only is sometimes limiting--e.g., you might want to write a DEX that has at most one pool for a given pair of currencies (need to compare types + reify the result in a value) or write a lending protocol that supports an extensible pool of currencies (need to reify the currency types + map them to the reserve coin supply).
We have discussed supporting this via extending the types module in the Sui framework with an operator like get<T>(): Type, but the details need to be worked out. In particular, we would like to support the weakest form of reflection that supports use-cases like the ones above in order to keep static reasoning as simple/predictable as possible.

See https://github.com/move-language/move/pull/566 in upstream Move, which we will pull in

Now available!


Make SeqRegionSynonymAdaptor compatible with BaseAdaptor

I wanted to use fetch_all() on SeqRegionSynonymAdaptor but couldn't because it doesn't implement _tables(), _columns() and _objs_from_sth().

It's a fair criticism, and a reasonable fix

Coverage remained the same at 81.579% when pulling 2e48625a36979310319ed9da15be414b57abbd6f on willmclaren:feature/generic-seqregionsynonym-adaptor into 8bf4ff23aece533a38212a4af314a39e02aed089 on Ensembl:master.


FileNotFoundError: [Errno 2] No such file or directory: 'data/processed/videos/May/aud_esperanto.npy'

FileNotFoundError: [Errno 2] No such file or directory: 'data/processed/videos/May/aud_esperanto.npy'  ，Why hasn't it been generated

It means the esperanto extraction has failed. Please check the corresponding step. It is usually caused by hugging face downloading failed.

Hello,I meet this problem , how did you solve it,  could you tell me ,thank you very much

Has your issue been resolved? I encountered the same problem and couldn't find a solution.


can't set email on Create Issued Document Enity

POST /c/{company_id}/issued_documents
With the above endpoint, according with the docs
email: string
https://developers.fattureincloud.it/api-reference/#post-/c/-company_id-/issued_documents
we have to be able to set the email filed on the entity but if we do that this info is not predent in the issued document if we don't save the "entity" (client) on the backend, seems a bug.
Thaks
MM

Hi @Mattia-Marchiorato, we already discussed this topic several times in our github discussions (here for example).
So, creating an issued document the fields email, phone and fax are ignored, but they are present on the documentation because under the hood they use the same model and to generate consistent classes for our SDKs we need them like that.
Thanks,
Emanuele.


[Autopilot] approval for rule auto/volume-resize-pvc-5f851043-98ee-4ba1-a521-2174c1632d8d rule: volume-resize

This is a request to approve an autopilot action. The request was triggered based on an AutopilotRule volume-resize defined in your cluster.
What actions will be taken
Action: resize

Params: map[maxsize:100Gi scalepercentage:100]

ExpectedResult
PVC will resize from 60Gi to 100Gi
What objects will get affected

PersistentVolumeClaim auto/pgbench-data (pvc-5f851043-98ee-4ba1-a521-2174c1632d8d)

Object Owner(s):

ReplicaSet pgbench-6bc57495fd

How do I approve
Once you review the above,

To approve, simply approve and merge this PR
To declined, close the PR

Autopilot will be watching for the merged specs in the cluster and will proceed with the action if approved and declined the action if not.

[Autopilot] Closing PR as the approval auto/volume-resize-pvc-5f851043-98ee-4ba1-a521-2174c1632d8d was deleted in the Kubernetes cluster. The deletion could be due to one of below reasons:

The object: pvc-5f851043-98ee-4ba1-a521-2174c1632d8d for the approval no longer exists in the cluster.
The rule conditions for the object: pvc-5f851043-98ee-4ba1-a521-2174c1632d8d are no longer met in the cluster.
The approval was deleted by a user..


Bump tls-test-kit to v0.2.3 and inferno-core to v0.4.44

This PR adds the feature of using the inferno execute CLI and disabling TLS tests with INFERNO_DISABLE_TLS_TEST=true environment variable. It will unblock inferno-framework/inferno-reference-server#182 so the (g)(10) test kit can run in CI. This PR will also close #577.

I realized bumping both dependencies in one PR would be easier than two separate PRs. I ran this test kit locally to confirm it works. Ready for review and merge!


Can I change the options' background-color with the newest edition?

I want the options's background-color to inherit from the original options which i have set .Can I achieve that ?

Support for this was added in https://github.com/select2/select2/pull/3324, it will be available in 4.0.1.

Does this work now? I tried the way that Sagaio tried here https://github.com/select2/select2/issues/4483 but this is not working for me.
Do you have a tutorial, documentation about this?


Use Result for reverse geocode completion parameter

We have no immediate slated future release schedule for data-collection, please modify the base branch to master.


Netlify downtime workaround for moov.io

Terraform will perform the following actions:

  # google_dns_record_set.moov-A will be updated in-place
  ~ resource "google_dns_record_set" "moov-A" {
        id           = "moov-io/moov.io./A"
        name         = "moov.io."
      ~ rrdatas      = [
          - "<IP_ADDRESS>",
          + "<IP_ADDRESS>",
        ]
        # (4 unchanged attributes hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Warning: Version constraints inside provider configuration blocks are deprecated

  on gcp.tf line 6, in provider "google":
   6:   version     = "~> 2.20"

Terraform 0.13 and earlier allowed provider version constraints inside the
provider configuration block, but that is now deprecated and will be removed
in a future version of Terraform. To silence this warning, move the provider
version constraint into the required_providers block.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

google_dns_record_set.moov-A: Modifying... [id=moov-io/moov.io./A]
google_dns_record_set.moov-A: Modifications complete after 3s [id=moov-io/moov.io./A]

Apply complete! Resources: 0 added, 1 changed, 0 destroyed.


Sleep mode minimum seems too bright in our RV

Hi, in our RV we are just appreciating the great benefits of using adaptive lighting.
However, the sleep mode seems too "aggressive" when one wakes during the night. It may be because of the unusually small space lighted by the bulb. I'm using a high quality RGBW bulb (Zigbee) by Ledvance (Osram GmbH): https://www.amazon.it/gp/product/B07VHRP4PS (4x 60€)
Default config values are OK, but it would be nice to be able to lower the temperature <1000°K and/or the brightness <1% (float), without tampering locally with VALIDATION_TUPLES in config.py, thus loosing the click-to-update laziness.
Thank you,
Piero

Has nothing to do with this integration, but with your lightbulbs. Your Osram lights are fairly cheap and I know that most cheap lights don't have 1% brightness. They have the setting, but the light itself doesn't go any lower than like 5-10%. I used a few Osrams before and noticed the same thing with the cheaper IKEA ones. You can't lower the brightness visually than what the light itself supports.
I also use Hue RGBW lightbulbs, which are more expensive but perform a lot better. 1% brightness is extremely dim.

Thank you for your... illuminating reply.
I will continue using your Adaptive Lighting, but as I don't like to spend 40€ for one E27 bulb, I will look for other hardware soutions, eventually DIY, like ESPhome wifi bulbs instead of Zigbee, or other DIY solutions.
Thank you again for your addon and your support.
Piero


Make E741 not apply to stub files

I think it would make sense to have E741 (ambiguous-variable-name) not apply to stub files.
Unlike with runtime .py files, stub authors generally have no control over the names of variables that appear in a stub. A well-written stub should aim to faithfully represent the interface of the equivalent .py file as it exists at runtime, and that includes any ambiguously named variables in the runtime module. Even if the source code for the stubs is located in the same repository as the runtime module being stubbed, you likely won't be able to change an ambiguously named variable in the stub without also changing the corresponding runtime module, and tools like stubtest will generally help stub authors ensure consistency between the runtime module and the stub.
Typeshed has always ignored this rule for .pyi files in our ruff config (and before that, in our .flake8 config), and I can't think of a situation in which it would be useful when applied to stubs.
Would we need some kind of deprecation period for this change? Or could we just make it?
(F403 and F405 (regarding * imports) similarly flag things that are out of the control of stub authors. But these rules highlight places where ruff's inference capabilities will be severely limited due to the specific idioms being used, so I think we should probably keep these enabled by default, even for .pyi files.)

I think this makes sense! We could ship it as part of v0.5.0 if you want?

We've now made this change in preview mode; we'll be stabilising it in Ruff 0.7. Thanks @calumy!


Dead Letter Queue implementations can leak personal data to log

Basic information

Axon Framework version: 4.6.0
JDK version:  17
Complete executable reproducer if available (e.g. GitHub Repo): N/A

Steps to reproduce

Add  message with personal data to DLQ
Content of message is logged

Expected behaviour
Only message-identification is logged, not the contents, since the contents might be sensitive.
Actual behaviour
The message content is logged

Closing this issue, as it's resolved in pull request #2398.


Changes to enable the overview topic for reference

Plus there were other minor changes.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

SP-Framework/breadcrumb/toc.yml
:white_check_mark:Succeeded

SP-Framework/overview/sharepoint.md
:white_check_mark:Succeeded

SP-Framework/sp-odata-types/iodatausercustomaction.yml
:white_check_mark:Succeeded

SP-Framework/spfx-toc/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

.openpublishing.redirection.json
:white_check_mark:Succeeded

SP-Framework/breadcrumb/toc.yml
:white_check_mark:Succeeded

SP-Framework/overview/sharepoint.md
:white_check_mark:Succeeded

SP-Framework/sp-odata-types/iodatausercustomaction.yml
:white_check_mark:Succeeded

SP-Framework/spfx-toc/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add test script for link node

Before you hit that Submit button....
Please read our contribution guidelines
before submitting a pull-request.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

If you want to raise a pull-request with a new feature, or a refactoring
of existing code, it may well get rejected if it hasn't been discussed on
the mailing list or
slack team first.
Proposed changes
Describe the nature of this change. What problem does it address?

There is no test script for link node.
Add test script & test cases for link node.
Also fix red/red.js to implement RED.events for link node test support.

Checklist
Put an x in the boxes that apply

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.2%) to 87.775% when pulling 3ac41cecea0d971f45d6cbf3b5a30879a9d301bf on HiroyasuNishiyama:feature/test-for-link-node into f57a0d4d6b3cea2fa782cf8fe2d12a887869c79d on node-red:master.


Time between lines

How can the time it takes blobs\people between the green\red in\out latch lines be calculated? effectively trying to figure out speed the person is walkng\running.
cool project

Hi @davidw212
There are two options.
You can use ofxCv's getVelocity function
or
If you make two variables in the Blobs Class.
int starttime;
int endtime;
Set them to both to 0 in the setup loop.
Then record the unix time using [ofGetUnixTime()] into starttime when it hits the first line (http://openframeworks.cc/documentation/utils/ofUtils/#!show_ofGetUnixTime).
just before the blob hits the second line, record the unix time again into the endtime. minus the starttime from endtime.
i.e
int difftime = endtime - starttime;
That should give you the desired time.
Best


register_permissions() plugin hook, WIP

Refs #1939
From this comment: https://github.com/simonw/datasette/issues/1939#issuecomment-1343872168

[x] Unit test for the registration plugin hook itself
[x] Use them in check_permission_actions_are_documented test in conftest.py
[x] Add description field to Permissions (and update tests and docs)
[x] Documentation for datasette.permissions dictionary
[ ] If no default= provided in call to permission_allowed() then use default from datasette.permissions list
[ ] Throw an error if two permissions are registered with the same name or abbreviation (but other attributes differ)
[ ] Log a warning any time you try to call permission_allowed() on a permission that was not registered (but only once per permission to avoid spamming the logs)
[ ] Update authentication and permissions documentation to explain that permissions are now registered and have a registered default

:books: Documentation preview :books:: https://datasette--1940.org.readthedocs.build/en/1940/

Here's my first test failure:
tests/test_permissions.py .......F
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> traceback >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>

allow = {}, expected_anon = 403, expected_auth = 403, path = '/fixtures/compound_three_primary_keys'
padlock_client = <datasette.utils.testing.TestClient object at 0x107c09d20>

    @pytest.mark.parametrize(
        "allow,expected_anon,expected_auth",
        [
            (None, 200, 200),
            ({}, 403, 403),
            ({"id": "root"}, 403, 200),
        ],
    )
    @pytest.mark.parametrize(
        "path",
        (
            "/",
            "/fixtures",
            "/fixtures/compound_three_primary_keys",
            "/fixtures/compound_three_primary_keys/a,a,a",
            "/fixtures/two",  # Query
        ),
    )
    def test_view_padlock(allow, expected_anon, expected_auth, path, padlock_client):
        padlock_client.ds._metadata_local["allow"] = allow
        fragment = "🔒</h1>"
        anon_response = padlock_client.get(path)
>       assert expected_anon == anon_response.status
E       assert 403 == 200
E        +  where 200 = <datasette.utils.testing.TestResponse object at 0x107aea680>.status

/Users/simon/Dropbox/Development/datasette/tests/test_permissions.py:61: AssertionError
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> entering PDB >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>

>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> PDB post_mortem (IO-capturing turned off) >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
> /Users/simon/Dropbox/Development/datasette/tests/test_permissions.py(61)test_view_padlock()
-> assert expected_anon == anon_response.status
(Pdb) anon_response
<datasette.utils.testing.TestResponse object at 0x107aea680>
(Pdb) anon_response.status
200
(Pdb) path
'/fixtures/compound_three_primary_keys'
(Pdb) padlock_client.ds._metadata_
*** AttributeError: 'Datasette' object has no attribute '_metadata_'
(Pdb) padlock_client.ds._metadata_local
{'databases': {'fixtures': {'queries': {'two': {'sql': 'select 1 + 1', 'name': 'two'}, 'from_async_hook': {'sql': 'select 2', 'name': 'from_async_hook'}, 'from_hook': {'sql': "select 1, 'null' as actor_id", 'name': 'from_hook'}}, 'source': None, 'source_url': None, 'license': None, 'license_url': None, 'about': None, 'about_url': None}}, 'allow': {}}
(Pdb) allow
{}

It looks like I've broken the allow logic that notices that if there's an "allow": {} on the root then anonymous users should not be allowed to view any pages.

I'm going to revert that last commit, see if I can get the tests running again and then apply the changes a line at a time to figure out which ones broke things.

It's this change which triggers the failures:
diff --git a/datasette/app.py b/datasette/app.py
index 760063d5..defa9688 100644
--- a/datasette/app.py
+++ b/datasette/app.py
@@ -707,9 +707,12 @@ class Datasette:
                 )
         return crumbs

-    async def permission_allowed(self, actor, action, resource=None, default=False):
+    async def permission_allowed(self, actor, action, resource=None, default=None):
         """Check permissions using the permissions_allowed plugin hook"""
         result = None
+        # Use default from registered permission, if available
+        if default is None and action in self.permissions:
+            default = self.permissions[action].default
         for check in pm.hook.permission_allowed(
             datasette=self,
             actor=actor,

Actually one last thing: I said that the error would only occur if the permissions differed in some way.

This PR ended up bundling part of the implementation of:

#1636

I'm going to be bad an NOT untangle that from this before I merge it.


[FE] Enable readonly / editable mode for the postcoordination

We need to implement two states for the PostCoordination view , ReadOnly and editable
the state "readOnly" should be on the entire view (scale values + table)
Both table and scale value cards should have an internal state "readOnly" and have a mutator that can change the state.
when in ReadOnly mode for the table , the checks are disabled and can't be edited.
when in ReadOnly mode for the scale values cards , the buttons are not shown.
when In ReadOnly mode the modal is not shown in the cards.
If "Cancel" is hit, we fetch the data from the server and refresh the view.

i'll work on this for the scale value cards also.


I get the following error on build only... TypeError: b is not a function

🤓 Question
Hi There!
I'm trying to use a couple different animations and it works great when I locally run this on my laptop. When I do a build and deploy it to a server I get the following error when I open the page in the browser "TypeError: b is not a function"
Is there a setting I need on build to make sure everything works as expected? I cleared out my node_modules and package-lock.json and re-installed to no avail. I have also tried to modify settings in my tsconfig.app.json which also didn't help.
Here is a simplified version of one of my animations...
const traverseTransitions    = useTransition(toggle, p => p, {
from:  { opacity: 0, width:"100%" },
enter: { opacity: 1, width:"100%", paddingRight:"10px" },
leave: { opacity: 0, width:"100%" },
});
{traverseTransitions.map(({ item, key, props }) =>
<animated.div style={props} key={key}>{menuTraverse()}</animated.div>
)}
A twist to my configuration is I am using Angular 7, I have been converting my app to React for some time and everything has worked fine with those pieces so far. Once I add react-spring for the transitions I am getting the above error.
Note: I have React 16.13 installed with react-spring 8.0.27.
Any suggestions you could provide would be extremely helpful.
Thanks in advance!

Please try with v9:
npm install react-spring@next

Just installed v9. It says that "g is not a function" now

If you can't reproduce in CodeSandbox, please provide a Github repository I can debug. 👍

looking at this code in the bundle, shouldn't it be transpiled at this point and not using "class" and such?

Oh, you'll be interested in #601. The solution for v9 is here.

funny I was just looking at that one. With v9 - I'm now trying to use use the 'react-spring/web.cjs'
import { Transition, animated } from 'react-spring/web.cjs'
I don't see any ES6 arrow functions any longer in the bundle which is good, but I'm getting a completely different error at runtime.
I changed away from using the hooks version to see if that would help. I tried following this very simple example. Again this works fine except when I do a build
https://github.com/react-spring/react-spring-examples/blob/renderprops/demos/renderprops/pagetransitions/index.js

Not sure this helps but this is my current tsconfig, I've tried several variations of this>

"compilerOptions": {
"sourceMap": true,
"declaration": false,
"moduleResolution": "node",
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"jsx": "react",
"target": "es5",
"lib": [
"es2016",
"dom"
],
"outDir": "../out-tsc/app",
"module": "es2015",
"baseUrl": "",
"types": ["node"],

"paths": {
"ag-grid/": ["../node_modules/ag-grid/"]
}
}

Please see https://github.com/react-spring/react-spring/issues/947#issuecomment-600892282

In the process of getting a simplified repo of the problem to share I was able to figure out what was going on...
The build optimizer for angular was somehow causing this issue. Turning this off everything works as expected. I am confused why it solely causes a problem in this library and no others to date as we have a large app with lots of dependencies...
Still digging...


Adds metrics for GCS events

Adds metrics for the following events:

bucket listing (for a certain cloud project) - will ping on each listing
blob browsing - will ping on each browse operation (e.g. each folder traversal)
usage of each right-click menu action
StorageExceptions from bucket listing, and blob browsing

PTAL.
Cleaned it up a bit. Decided to get rid of that pingMetrics method altogether since it was 1 line of code it was replacing.


Please add support for Swift Package Manager (Dependencies)

Many people will be moving away from CocoaPods and Carthage and using Swift Package Manager (Dependencies). Can you please add support? Thank you!

I've just added basic SwiftPM support in #26 but it has one limitation: since SwiftPM doesn't support resources or bundles so far and APNumberPad comes with default backspace icon.
Nevertheless, it can be used with custom styleClass proper set (on APNumberPad instance) which would refer backspace icon from user's resources.
I guess it can be worked around by drawing that icon as a shape via code, I'll consider the next steps in the new version of APNumberPad which is coming soon.

Xcode 12 and Swift 5.3 (both available as dev betas) now include resource and closed-source binary support for SPM. Hopefully SPM can now be done for this?
https://www.hackingwithswift.com/articles/218/whats-new-in-swift-5-3

Sure, thanks for the link. I need some time to catch up with the new features.

Xcode 12 is now out as of today, just for an update on this.

As an update: I made Swift PM work fine but there was an issue with CocoaPods. Hopefully, I'll solve it within a few days.

Swift PM support branch has been merged, if you have any issue please let me know.

Awesome thank you! So far SPM support seems to be working great.

I'll close the issue then. Feel free to post any issues you have.


BBC iPlayer subtitle format

How to get SRT subtitles from BBC iPlayer?
youtube-dl --all-subs --skip-download  --convert-subs srt http://www.bbc.co.uk/iplayer/episode/p03nnns9/murdered-by-my-father
I can only get ttml format......

Remove --skip-download. Duplicate of #9073.

@dstftw  If I remove --skip-download, I will download the video，but I just want only the srt subtitle

It's not possible see linked issue.

Sorry, I don't get it. Could you give me a command to just download the SRT subtitle  of this program? Thank you very much!!! http://www.bbc.co.uk/iplayer/episode/p03nnns9/murdered-by-my-father

It's not possible to download srt because there is no srt provided by BBC. You can only download ttml and convert to srt but currently only with downloading the video itself.

Thank you very much!


Shortcut Guide shortcut not working

Microsoft PowerToys version
0.37.0
Running as admin

[X] Yes

Area(s) with issue?
Shortcut Guide
Steps to reproduce

Hold Windows key for 1 second
Shortcut Guide doesn't appear

✔ Expected Behavior
Shortcut Guide appears after holding Windows key
❌ Actual Behavior
Shortcut Guide never appears
Other Software
The only other thing I can possibly think of, is I have Razer Synapse installed. But with that, there are no options using the windows key. I also have Gaming Mode off (which prevents the usage of the Windows key). This used to work with the same keyboard and Razer Synapse installed.
I also only have MS Edge (Chromium) open when attempting the shortcut for the guide to appear.

Please let me know if I should attach the Zip file generated from the bug report via the tool.

@megaroeny
thanks for reporting the issue.
The plan for 0.39 is to drop support for long press Win key and have a default shortcut Win + ? that will be user configurable.
Win + ? is already available as an alternative to long press Win key.

@megaroeny
thanks for reporting the issue.
The plan for 0.39 is to drop support for long press Win key and have a default shortcut Win + ? that will be user configurable.
Win + ? is already available as an alternative to long press Win key.

Got it! Yes I prefer that much more. Thank you!


httpie: 2.2.0 -> 2.4.0

Motivation for this change
Replaces #102466
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 112381 at 56b97d57 run on x86_64-linux 1

2 packages built:

http-prompt
httpie

1 suggestion:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/tools/networking/httpie/default.nix:15:5:
   |
15 |     ./strip-venv.patch
   |     ^

Result of nixpkgs-review pr 112381 at 56b97d57 run on aarch64-linux 1

2 packages built:

http-prompt
httpie

1 suggestion:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/tools/networking/httpie/default.nix:15:5:
   |
15 |     ./strip-venv.patch
   |     ^

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

2 packages built:

http-prompt
httpie

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112381 run on x86_64-linux 1

2 packages built:

http-prompt
httpie

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112381 run on x86_64-linux 1

2 packages built:

http-prompt
httpie


x-tagGroups for redoc

Hello, how would I be able to make use of the x-tagGroups in combination with redoc ?
x-tagGroups

@michelbieleveld Unfortunately not at the moment, there is no extension field in OpenApi struct. This could be added in future though.


Port CI to Github Actions

Ported from https://github.com/neuropoly/spinalcordtoolbox/blob/master/.travis.yml
Based on https://github.com/actions/starter-workflows/blob/main/ci/python-package.yml
I had to add in some fixes for bugs Github exposed. I also went off-topic and did some nit-picky cleanups I found in the process; sorry about that, I can pull those out if it would make it easier to review.
Main part:

tests.yml
Associated bug fixes:

one missing case of sct_test_path()
create intermediate folders if missing while testing on them instead of assuming install_sct did it successfully
force install in-place (to be undone in a follow up: #3140)

Off-topic parts:

set PIP_PROGRESS_BAR=off globally
Source SCT from the user's shell for more accurate testing, and make that shell file ~/.bash_profile not ~/.bashrc

some distros intentionally make ~/.bashrc incompatible with noninteractive sessions, because it's not meant for them
this is how things like brew get installed too

Use that to spot-check the install result a bit more thoroughly
drop pylint (part 1 of #3142)
Unify Travis' triggers with the new Github triggers

they weren't far off, but it's just nice to keep everything consistent

You can see that every platform passes, even the nightlies, if you fork this branch and add your fork to the on: list
https://github.com/neuropoly/spinalcordtoolbox/blob/029bbc835171bd0bd04022fb01251e8db57ef3fd/.github/workflows/tests.yml#L3-L7
Speed: where Travis takes an hour to do a run, Actions takes 15 minutes, because it actually runs everything in parallel, and maybe their machines are a bit faster too; it's much faster to get assigned a machine, especially a macOS machine: Travis has to wait for 20 minutes sometimes for the tests to even start.
Parity: I've made sure not to break Travis while doing this. Though I fixed some things, I did it in a way that works on both platforms (and therefore hopefully the wider world). Once this goes in both Travis and Actions will be active in tandem for a time as checks against each other. See https://github.com/neuropoly/spinalcordtoolbox/issues/3071#issuecomment-751832655 for my plan about that.
Open issues:

We hit and reported a bug (https://github.com/actions/virtual-environments/discussions/2352) with the macOS VMs. It's one of those things that should work but also kind of shouldn't because we were doing things the messy way, and in order to work around it I had to switch everything to the non-interactive mode it expects (i.e. passing -y flags, mostly) instead of trying to force it into a mold. So, this now includes #3102, so make sure to review and merge that first, because that's where I first put the fix. - now merged, thanks everyone
I really wish I could have used a build matrix for the jobs instead of copy-pasting them but I couldn't get it to work https://github.com/neuropoly/spinalcordtoolbox/issues/3071#issuecomment-7506736281
https://github.com/neuropoly/spinalcordtoolbox/issues/2957#issuecomment-752886419 is an intermittent failure, but we might have to deal with it before merging this because for some reason it's much more common on Actions macOS than anywhere else. Quieted with pytest -n 1, but I would really really like feedback or an alternate fix. This is a real concurrency bug! We should squash it.
Changing to ~/.bash_profile probably has unintended consequences; at a minimum, it's going to make
https://github.com/neuropoly/spinalcordtoolbox/blob/d64af23b69dbc396322cdc963a90a899b73736e0/install_sct#L677-L682
do the wrong thing. dealing with this later, it got worked around in a different way: https://github.com/neuropoly/spinalcordtoolbox/pull/3125#discussion_r551070363; but I'm still going to do it in https://github.com/neuropoly/spinalcordtoolbox/issues/3149
This doesn't test "package" mode: https://github.com/neuropoly/spinalcordtoolbox/issues/3140
Do we have a lot of users on Windows? Booting up WSL is still slower than anything else. Maybe we could leave the WSL build to cron?

Checklist
GitHub

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[x] I've applied the relevant labels to this PR
[x] I've assigned a reviewer

PR contents

[ ] I've consulted SCT's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[ ] I've added relevant tests for my contribution
[ ] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description

Linked issues

Closes #3071

I've got all the platforms passing now, e.g. https://github.com/neuropoly/spinalcordtoolbox/actions/runs/452633468
I want to clean up the workflow file some. It's not the worst but I think I can make it more straightforward by:

using ${{env.NIGHTLY}} in my if:s
skipping the build matrix (actually, matricies, because I couldn't get all the combinations we use to fit in one) and just using explicitly jobs; this'll be more verbose but it seems like build matrices are not quite fully featured yet

This is currently green except for https://travis-ci.com/github/neuropoly/spinalcordtoolbox/jobs/466205848, which is #3099, and should be fixed soon by upstream.

This is ready for a first round of review. Any takers? :trumpet:

I'm trying to -n 1 workaround in 8f3dc5c42c271d1a03f366e350cc5d834f2ec4c1. But I expect it to greatly slow down the tests.

Multithreaded: https://github.com/neuropoly/spinalcordtoolbox/actions/runs/454468649
Singlethreaded: https://github.com/neuropoly/spinalcordtoolbox/actions/runs/454488205

Hello- I can't actually comment on that PR, but why not have an object that
holds what needs to be checked for, with a repository for getting and
setting said items
This would mean that you could get any new items that someone else has
added.
On Mon, Jan 4, 2021 at 4:52 PM Julien Cohen-Adad<EMAIL_ADDRESS>wrote:

@jcohenadad commented on this pull request.
In .ci.sh
https://github.com/neuropoly/spinalcordtoolbox/pull/3125#discussion_r551562149
:

echo *** CHECK PATH ***
-ls -lA bin  # Make sure all binaries and aliases are there
-source python/etc/profile.d/conda.sh  # to be able to call conda
+# Make sure all binaries and aliases are there
+for tool in \
I agree that maintaining the list is tricky. I'm open to suggestions?
Maybe @Drulex https://github.com/Drulex or @cfhammill
https://github.com/cfhammill or @Drulex https://github.com/Drulex
have some? Can I tag the newer people too for their ideas too?
@joshuacwnewton https://github.com/joshuacwnewton @ahill187
https://github.com/ahill187 @gisellemartel
https://github.com/gisellemartel @charjl https://github.com/charjl
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neuropoly/spinalcordtoolbox/pull/3125#discussion_r551562149,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASV47GKZBL6HQEAHKMM273SYITCDANCNFSM4VHRWPHQ
.

--
E. Charlotte V. Jantz-Lee
613.818.6068

Hello- I can't actually comment on that PR, but why not have an object that
holds what needs to be checked for, with a repository for getting and
setting said items
This would mean that you could get any new items that someone else has
added.
On Mon, Jan 4, 2021 at 4:52 PM Julien Cohen-Adad<EMAIL_ADDRESS>wrote:

@jcohenadad commented on this pull request.
In .ci.sh
https://github.com/neuropoly/spinalcordtoolbox/pull/3125#discussion_r551562149
:

echo *** CHECK PATH ***
-ls -lA bin  # Make sure all binaries and aliases are there
-source python/etc/profile.d/conda.sh  # to be able to call conda
+# Make sure all binaries and aliases are there
+for tool in \
I agree that maintaining the list is tricky. I'm open to suggestions?
Maybe @Drulex https://github.com/Drulex or @cfhammill
https://github.com/cfhammill or @Drulex https://github.com/Drulex
have some? Can I tag the newer people too for their ideas too?
@joshuacwnewton https://github.com/joshuacwnewton @ahill187
https://github.com/ahill187 @gisellemartel
https://github.com/gisellemartel @charjl https://github.com/charjl
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neuropoly/spinalcordtoolbox/pull/3125#discussion_r551562149,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASV47GKZBL6HQEAHKMM273SYITCDANCNFSM4VHRWPHQ
.

--
E. Charlotte V. Jantz-Lee
613.818.6068

I can't actually comment on that PR

thank you for your input @charjl ! i've added you with write permission in the team (unless there is another way to allow ppl to actually comment on PR?? that would prevent having me manually adding ppl)

I can't actually comment on that PR

thank you for your input @charjl ! i've added you with write permission in the team (unless there is another way to allow ppl to actually comment on PR?? that would prevent having me manually adding ppl)

I still can't comment on this project, but I am fine cloning it.

hum, that's annoying though-- can someone pls have a look? @alexfoias @joshuacwnewton ? thanks

I still can't comment on this project, but I am fine cloning it.

hum, that's annoying though-- can someone pls have a look? @alexfoias @joshuacwnewton ? thanks

@jcohenadad @charjl I'll check right now.

@jcohenadad @charjl I'll check right now.

@charjl could you please retry ?

@charjl could you please retry ?

Do we have a lot of users on Windows? Booting up WSL is still slower than anything else.

Hum, I really don't know. My guesstimate would be 50-200 Windows users.

Maybe we could leave the WSL build to cron?

👍

Do we have a lot of users on Windows? Booting up WSL is still slower than anything else.

Hum, I really don't know. My guesstimate would be 50-200 Windows users.

Maybe we could leave the WSL build to cron?

👍

Do we have a lot of users on Windows? Booting up WSL is still slower than anything else.

Hum, I really don't know. My guesstimate would be 50-200 Windows users.

Maybe we could leave the WSL build to cron?

93287fc2859850b86cc990338d3c2672febc8177

Do we have a lot of users on Windows? Booting up WSL is still slower than anything else.

Hum, I really don't know. My guesstimate would be 50-200 Windows users.

Maybe we could leave the WSL build to cron?

93287fc2859850b86cc990338d3c2672febc8177

Ola, this is great feedback!

1. By grouping everything into a single `.ci.sh` script, we end up with a single "CI" step in the workflow, which makes it a little harder to navigate because of the wall of text.

I think Travis has a good fix for this: just break up your scripts. We can give .ci.sh subcommands, or write separate test scripts (that maybe..all source a set of common subroutines?). They suggest it in https://docs.travis-ci.com/user/speeding-up-the-build/#parallelizing-your-builds-across-virtual-machines as a way to get better performance, but it improves readability too.

I got around this in #3198 by using steps:, which makes the CI interface a little easier to read, I think. The trouble is, steps: would involve a lot of copying and pasting. And... we're not the only ones to run into this limitation in GitHub Actions. actions/runner#646 (comment) does a really good job at summing up what I'm trying to get at.

Honestly I think CI is still in a dark age. It's at the point where it's still reinventing scripting. This sort of thing is par for the course with Jenkins and Travis, and working around it is up to the devs doing clever things.

Apparently in Q2 2021, workflow templates are coming? actions/runner#646 (comment)

Oh I hope so. But I will believe it when I see it. Travis never had composable CI. Maybe Github will do it better, but their first versions are going to be glitchy and need rethinking. If we can get composability just based on shell in the meantime I bet that'll serve us better.

EDIT: Aha! GitHub provides a way of grouping lines within shell scripts. This whole guide seems full of useful, underdocumented tricks.

NICE.
:+1: to it being underdocumented. Actions is super new! It already had a CVE!

1. `pytest` displays monochrome output in GH Actions by default. _(Context: [pytest-dev/pytest#7443](https://github.com/pytest-dev/pytest/issues/7443))_

Color needs to be added back via an environment variable:
https://github.com/neuropoly/spinalcordtoolbox/blob/796d3011b1e7b50104b833a979272b1fae4ab2d4/.github/workflows/tests.yml#L56-L58
(The rationale for needing an environment variable to do this is a bit of a rabbit hole, but if you're curious about it, the pytest team talks about it here: pytest-dev/pytest#7462)

Legit, thanks for diving into that rabbit hole.
aaaaaaah Actions not providing a tty is probably why yes | doesn't work right in it either. https://github.com/actions/runner/issues/241 is a good thing to pin somewhere.

tbh Actions seems really half-baked. If they didn't have about a trillion dollars behind them to pay for infra I would say it's not worth it.

I've spoken with @kousu, and we've agreed to let me take over this PR.
The work is mostly done. But, there is a fix in this PR for #2957 that may not be necessary, now that that issue has been resolved by #3212. So, I'll clean this up and seek a re-review.

As a note, to limit the scope of changes, I've reverted commits related to test data.
This includes d80e7ab, which was a good fix, but also only one instance of a hardcoded test data path (there are others). And, because there is a bigger effort in the works to turn the sct_test_data function into a fixture, hardcoded instances of "sct_testing_data/" can be handled then.

Closing and re-opening this PR to trigger a new build, as to refresh the "Expected — Waiting for status to be reported" status of Travis.

It all passed! Hooray!
If you want to test the nightlies too, you can temporarily add ng/ci-gh-actions to
https://github.com/neuropoly/spinalcordtoolbox/blob/af01471ee52fbe4ed401d092116dc3d624cefff6/.github/workflows/tests.yml#L5-L7


android: feat(terminal): add bindings for reader software update status, battery level, and simulator configuration

Adds new events to report reader software update status, an event to report reader battery status, and setSimulatorConfiguration method.

Any update on this?  The only thing preventing me from using this plugin is not being able to listen for firmware updates so I can convey that to the user.

Sorry for the delay. I looked it over and it is excellent! Thanks for the great pull request following the code so far.
Merge and test further.

Note: It will take some time to release the product as we will be working on the iOS implementation. Also, there is a possibility of specification changes to make it consistent with iOS behavior.

Note: iOS can't get errorCode, so change type.
https://github.com/capacitor-community/stripe/pull/375/commits/6db8444730cab2b1d7f95d9ec0fcfaf5b3ef6161

@DoctorMcKay Released! Thanks awesome pull request! https://github.com/capacitor-community/stripe/releases/tag/v6.1.0

Great, thanks for the release!
I am using this in production, but it's a private app that's not publicly available.


KeyError: 'returns'

Expected Behavior
Continue work...
Actual Behavior
[2016-07-26 19:12:50] [x] Captured Doduo! [CP 189] [IV 0.51]
[2016-07-26 19:12:50] [#] Doduo has been exchanged for candy!
Traceback (most recent call last):
File "pokecli.py", line 220, in
main()
File "pokecli.py", line 211, in main
bot.take_step()
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/init.py", line 36, in take_step
self.stepper.take_step()
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/stepper.py", line 62, in take_step
self._work_at_position(position[0], position[1], position[2], True)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/stepper.py", line 139, in _work_at_position
self.bot.work_on_cell(cell, position, pokemon_only)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/init.py", line 93, in work_on_cell
worker.work()
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/cell_workers/move_to_fort_worker.py", line 30, in work
self.stepper._walk_to(self.config.walk, *position)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/stepper.py", line 88, in _walk_to
alt, False)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/stepper.py", line 139, in _work_at_position
self.bot.work_on_cell(cell, position, pokemon_only)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/init.py", line 76, in work_on_cell
if self.catch_pokemon(pokemon) == PokemonCatchWorker.NO_POKEBALLS:
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/init.py", line 190, in catch_pokemon
return_value = worker.work()
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 30, in work
response_dict = self.api.call()
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/src/pgoapi/pgoapi/pgoapi.py", line 80, in call
response = request.request(api_endpoint, self._req_method_list, player_position)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/src/pgoapi/pgoapi/rpc_api.py", line 93, in request
response_dict = self._parse_main_response(response, subrequests)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/src/pgoapi/pgoapi/rpc_api.py", line 201, in _parse_main_response
response_proto_dict = self._parse_sub_responses(response_proto, subrequests, response_proto_dict)
File "/Users/David/Dropbox/pokbot/PokemonGo-Bot/src/pgoapi/pgoapi/rpc_api.py", line 208, in _parse_sub_responses
del response_proto_dict['returns']
KeyError: 'returns'
Steps to Reproduce
Run about 1 hour, it just crashed.
Other Information
OS:  Mac OS X 10.11.5
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)

Please run this command: git log -n 1 --pretty=format:"%H" and tell me your result.

@TheGoldenXY =format:3aca6d798f8a3e173dce0beb7b13c9505cc5b251

It happens at startup for me...

See this thread [https://github.com/tejado/pgoapi/issues/77] and the fix seems to have been pushed on 1f25e90

Thank for the fix : )  @kriswong

No problem!  If you want an easy way to fix this (and avoid the 1 hour log out, still to be tested) go into the ~/PokemonGo-Bot/src/pgoapi/pgoapi/ folder and run the following:
git reset --hard HEAD
git pull
Shoudl resolve at least it did for me.

This issue has been solved.
/closed

We have an issue.
This solution conflicts with Issue # 1042. If you do one you will get the other issue.

I fixed issue #1042 and after 2 hours of running the bot, got the same 'KeyError: 'returns'' error


🧑💻: Contributors' page

:red_circle: Title : Enhancing  Contributors' page
:red_circle: Enhancement Aim : To align with the website theme and colors
:red_circle: Changes : The Get Details button's color should not be blue, a bit styling will be added to each card to give a minimalist look and on clicking the button, their profile should open
Screenshots 📷

Kindly assign this issue to me under hactoberfest, gssoc-ext and relevant tags.

:white_check_mark: To be Mentioned while taking the issue :

Full name : Bhumika Sharma
What is your participant role? GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

On click show the data of their working in this repository, not redirect to their profile

Noted


[GEP-18] Adapt ssh-keypair to new secrets manager

How to categorize this PR?

/area security
/kind enhancement
/merge squash
What this PR does / why we need it:
This PR adapts the ssh-keypair secrets to new secrets manager. Given that there is not a single responsible component for it, the Botanist generates it.
The ShootStatus is extended with information about the SSH keypair rotation (similar to how it looks like for CA rotation). This is a requirement for the secrets manager since it needs to remember the last started time of the rotation process, and it's also an improvement for users and operators to see when the rotation was last started and/or completed.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/3292#issuecomment-1036058890
Special notes for your reviewer:
:x: Depends on #5524 which needs to be merged first.
/invite @timebertt @BeckerMax
Release note:

There is a new section in the `ShootStatus` under `.status.credentials.rotation.sshKeypair` describing when the SSH keypair rotation was last initiated and last completed.

/cc @ary1992

Thank you for the well organised PR.

/lgtm

/rebase


Large text does't paste very well

I use this with my gemini account. I can paste fairly large text into the browser while using it manually.
But when I use talking head module, sometimes the lib gets lost with even 15 lines. It just gets stuck and never clicks the send button.
I have no experience with selenium. It might be something with my setup alone.

Hello, apologies for the delayed response.
In order to investigate this issue, it would be great if you could kindly provide the logs next time you encounter this problem. It’s possible that this might be a problem regarding  Selenium, but I’d like to examine the logs to confirm.


AB#4780 Keycloak GraphQL Integration

Implements the keycloak-connect-graphql library with OCTI. It uses the current Keycloak configuration in the default.json so that will need to be applied for it to work. The environment variable KEYCLOAK_DISABLE can be used to disable the Keycloak feature. The variable is a flag and therefor accepts the values 0 or 1.  This variable will ultimately be ignored if built in webpack production mode.

NOTE: this is a server side configuration and env var

The other change not related to Keycloak is the buildContext method for graphql. I made it a bit more flexible for adding to the context by working on an object.

@r-joyce adding you as reviewer in the case you have time.


Use set literal syntax for set prettyprinting

pls add a test in tests/test_format.py in test_pprint

pls add a whatsnew note in 0.17.1 as well

can you add a whatsnew note (api changes section)

done

can you add a test as well
tests/test_format.py in test_pprint

done

awesome. ping when green.

ping!

merged via https://github.com/pydata/pandas/commit/76d4c49a10b2606458d2622a05bb14aea578e9a5
thanks!


[EPIC] Back-Ends logs

We need to work on logs on all Back-End project.
What should be done
Message type
Add into Logger config metadata [:initial_call, :file]
emergency: when the system cannot be used (ex : openfass_not_reachable after 5 attempts)
alert: something got wrong and need our attention, (ex: openfass_not_reachable less than 5 times, mongo_connection, ecto connection error, ...)
critical: when the error can make system crash (ex: view genserver crash/ query genserver crash), basicly we can found this error in try/catch
error: normal error
warning: something goes wrong but it not impact workflow
notice: hightlight a message
info: info message (ex: socket started, channel_started)
debug: debug message
Genserver
We need to make our genserver more stable, for each pattern match add a default function, try/catch for a sensitive call.
Log and return errors (error/critical) and in some cases send a message to the channel.
List of task

[ ] Add try / rescue on every:

[ ] Function that can raise
[ ] GenserverCall (avoid no process)

[ ] Add default function for PatternMatch function, following the case Log
[ ] Debug Log for logical function

[ ] Add debug Log on start on function with params
[ ] Add debug log on end of function with result

[ ] Info Log

[ ] Log socket start/stop
[ ] Log channel join
[ ] Log environment start/stop
[ ] Log session start/stop

[ ] notice log

[ ] Log Environment/Session genserver start/end (QueryServer, ViewServer/...)

[ ] Warning log
[ ] Error Log
[ ] Critical Log

[ ] Add try/rescue on genserver Call for more complete log that error 500 (add metadata)

[ ] Alert/Emergency

[ ] Create an error agent in lenra-common that logs an error alert and if the error occurs more than 5 times in 10 minutes, logs an emergency and waits 15 minutes before logging this alert again.

@pichoemr Is it finished ?


[co] Add new port

Does your PR follow the maintainer guide?

Yes

https://github.com/idealvin/co

Documents

English: github gitee
中文: github gitee

@NancyLi1013  Hi, I have updated the portfile. It works now.

Have you tested these two features libcurl, openssl? Do these features build successfully?

Have you tested these two features libcurl, openssl? Do these features build successfully?

It shows Building package co[core,libcurl,openssl]:x64-windows... done.

Have you tested these two features libcurl, openssl? Do these features build successfully?

@NancyLi1013
Yes, It works well.
$ vcpkg install co[libcurl,openssl]:x64-windows
Computing installation plan...
The following packages will be built and installed:
    co[core,libcurl,openssl]:x64-windows -> 2.0.1
  * curl[core,non-http,openssl,schannel,ssl,sspi]:x64-windows -> 7.78.0
  * openssl[core]:x64-windows -> 1.1.1k#8
  * zlib[core]:x64-windows -> 1.2.11#11

LGTM now, thanks for adding this port. @idealvin

LGTM now, thanks for adding this port. @idealvin

ok. Thanks.

The name co is extremely short and there are a very limited number of names like that (676!). Could this be renamed to idealvin-co to avoid confusion in the future and preserve this very precious resource?

The name co is extremely short and there are a very limited number of names like that (676!). Could this port be renamed to idealvin-co to avoid confusion in the future and preserve this very precious resource?

@ras0219  I'm sorry. I think CO is more convenient to use. idealvin-co is not so friendly to users and will not be used.

The name co is extremely short and there are a very limited number of names like that (676!). Could this port be renamed to idealvin-co to avoid confusion in the future and preserve this very precious resource?

I found cr, dx, qt, etc. in vcpkg/ports, and I think co is not a problem.

The name co is extremely short and there are a very limited number of names like that (676!). Could this port be renamed to idealvin-co to avoid confusion in the future and preserve this very precious resource?

@ras0219

Hi there, @idealvin :)
The main reason we haven't merged this yet is that we are concerned about taking the short/desirable name "co" because we're having difficulty finding strong/broad association with that name. That doesn't mean we're necessarily "blocking" over it, but would feel a lot better if the name was idealvin-co or something like that instead. How does that sound to you?

We still haven't made the go/no go decision on name here. I note that it isn't used on Repology. This discussion did trigger adding a note about preferring longer names to avoid confusion: https://github.com/microsoft/vcpkg/pull/19956
I "vote" weakly in favor of merging this but would really prefer the prefix.

We still haven't made the go/no go decision on name here. I note that it isn't used on Repology. This discussion did trigger adding a note about preferring longer names to avoid confusion: #19956
I "vote" weakly in favor of merging this but would really prefer the prefix.

How about alphaco?

Thanks. I'll rename my project and send a PR a few days later.

After some discussion with maintainers I got something like:
SF F N SA WA
 0 2 1  0  0

so I would merge this if you reopen @idealvin

(The maintainers who voted are @BillyONeal @ras0219 and @vicroms )

After some discussion with maintainers I got something like:
SF F N SA WA
 0 2 1  0  0

so I would merge this if you reopen @idealvin

ok, Thanks!

Thanks for your contribution :)


Tesseract inserting additional alternative characters

Environment

Tesseract Version: <3.x stable and 4.0 alpha/beta> for English language text (using Fast and Best trained data) Command line

Platform: <Windows, version 64-bit and linux (Ubuntu/centos)-->

Current Behavior:
All versions of tesseract mentioned above tend to insert additional alternative characters (probably) whenever its not very confident. For example - if theres a "#" in the image file it often spits out "#H" or "A#" or even "AH"... Thats 2 characters for 1. Another example: If theres a "$" in the image then it gives "S$" or "$s" etc.. happens very often for other characters like 0,O,!,%,^ etc etc...
My application is very sensitive to length of the string hence an extra character throws many things off.
I am currently a command-line user and may later use it in Java whenever a wrapper for 4.0 becomes available.
Expected Behavior:
Expect tesseract to give out only one character for each character in the image.  I should be able to control this behaviour using command line parameters (assuming there isn't one yet..). I have looked into the parameters but there are hundreds and mostly non-self-explanatory. Hence raising this as an issue. Also is it possible to get a "Character-level" HOCR output - current one is at word level granularity.
Suggested Fix:

For English, please also try the file from tessdata repo with --oem 0.
That will use the legacy tesseract engine. It is possible that it will be a
better fit for your use case.
On Tue 10 Apr, 2018, 6:28 PM jghare<EMAIL_ADDRESS>wrote:

Environment

Tesseract Version: <3.x stable and 4.0 alpha/beta> for English
language text (using Fast and Best trained data)
Platform: <Windows, version 64-bit and linux (Ubuntu/centos)-->

Current Behavior:
All versions of tesseract mentioned above tend to insert additional
alternative characters (probably) whenever its not very confident. For
example - if there a "#" in the image file it often spits out "#H" or "A#"
or even "AH"... Thats 2 characters for 1. Another example: If theres a "$"
in the image then all it gives "S$" of "$s" etc.. happens very often for
other characters like 0,O,!,%,^ etc etc...
My application is very sensitive to length of the string hence an extra
character throws many things off.
I am currently a command-line user and may later use it in Java whenever a
wrapper for 4.0 becomes available.
Expected Behavior:
Expect tesseract to give out only one character for each character in the
image. I should be able to control this behaviour using command line
parameters (assuming there isn't one yet..). I have looked into the
parameters but there are hundreds and mostly non-self-explanatory. Hence
raising this as an issue. Also is it possible to get a "Character-level"
HOCR output - current one is at word level granularity.
Suggested Fix:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1465, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AE2_ozUU7WV9vOy5bqQWuoe5mAeKl_7Qks5tnKx3gaJpZM4TONQm
.

Hi Shreeshrii
When I say English I mean the English alphabet and special characters. The words themselves are not dictionary words and are cryptic and long sequences of mixed characters... 20-30 characters long.
The 4.0 alpha and beta give me far superior results on the OCR than legacy on my images. Is there no way to tell tesseract 4.0 to not insert extra alternatives?
Also would be good to give it a white-list of characters. I see that issue is also open...

Please fix this.. It's a big problem.

If it is a big problem that provide user case. Description by words is difficult to test and developers are forced to spent useless time on find what is your problem instead of solving problems.

#1011 might be related

There are a number of issues regarding this, for different languages etc. Listing them below.
Incorrect recognotion of specific words - additional letters inserted #1011
tesseract add similar characters in Japanese text (ambiguity management?) #1063
German - Characters added to result multiple times (aä / AÄ) #1060
Tesseract LSTM 4.0: letters repeat in recognized text #884

Possibly related:
recognizes more characters than present #1362

This is still present in the latest master branch. It seems to happen after retraining (finetuning) the original tessdata files - in my case eng - and appears to be a result of ambiguous output from the LSTM, where it is providing more than one character for a bounding box (or at least that's how it appears without actually checking) - i.e. it is giving its possible or "unconfident" words as well. More training does seem to balance this out slightly, but it's very hit or miss.

When I say English I mean the English alphabet and special characters. The words themselves are not dictionary words and are cryptic and long sequences of mixed characters... 20-30 characters long.

In that case try to disable the dictionary.
Also try to fine tune the model.

Also is it possible to get a "Character-level" HOCR output - current one is at word level granularity.

Yes. It is. Use  -c hocr_char_boxes=1  hocr in your command line. Output is of the format:

             B
             A
             S
             O
             B
             C

Hi,
I tried to use it, but it is not working for me. Any idea

C:\Program Files (x86)\Tesseract-OCR>tesseract testImage.PNG out -l check -c hocr_char_boxes=1 hocr
Could not set option: hocr_char_boxes=1
Tesseract Open Source OCR Engine v4.00.00alpha with Leptonica
OSD: Weak margin (0.63), horiz textlines, not CJK: Don't rotate.
Detected 3 diacritics
It looks like this config is not longer there. I want the output that on a char level but it does not seen possible.

@jghare, can you provide some simple images which show this issue? That would help testing new code which tries to fix it.

@stweil I encounter this issue on nearly daily basis with about 5% of the cases (around 2 per day on average). I will try to save the problematic files and the settings if that will be helpful to you.
In the mean time, here are some examples (called from python):

--
file: stelnum -> text: 1C4BUOOOOKPJ60479 -> extracted text: 1Cc4dBUOOOOKPJ60479 -> config: --oem 1 --psm 1 -c tessedit_char_whitelist=ABCDEFGHJKLMNOPSRTUVWXYZ1234567890 lang=dan
--

file: regnumber -> text: AJ38906 -> extracted text: AIJ38906 -> config: --oem 1 --psm 1 -c tessedit_char_whitelist=0123456789ABCDEFGHIJKLMNOPQSRTUVWXY lang=dan
--
You can find attached the images in an archive, hopefully they will prove useful test material. Please tell if you would like me to send more
images.zip
--

For random sequence of characters you'll need to:

Disable the dictionary.
Fine tune the eng model with similar images.

If you have questions about fine tuning, use the forum.

@Shreeshrii

There are a number of issues regarding this, for different languages etc. Listing them below.
Incorrect recognotion of specific words - additional letters inserted #1011
tesseract add similar characters in Japanese text (ambiguity management?) #1063
German - Characters added to result multiple times (aä / AÄ) #1060
Tesseract LSTM 4.0: letters repeat in recognized text #884

recognizes more characters than present #1362

allow me to add #1465 and #2738.
My diagnosis for this bug was that it is specific to the Tesseract CTC implementation (with its NodeContinuation trick conflating paths to avoid the combinatorial explosion but creating an additional ambiguity of two adjacent nulls). I called these fake CTC duplicates diplopia. Someone definitely needs to work on this.

hello,
I have this problem too, any idea how I can help fix it?
I have this simple example :

OCR result : 1921.14K
Analyzing other tests the problem is probably in the 9 because sometimes I get wrong results with 2s instead of 9s, In this case I got both.
I'm using this .NET wrapper (https://github.com/charlesw/tesseract/tree/feature/321-Tesseract-4) but I build and updated the tesseract and leptonica DLLs to the latest ones (leptonica-1.83.0 & tesseract50), using the "best" eng.traineddata and char whitelist "0123456789.,KMB".

I have just created pull request https://github.com/tesseract-ocr/tesseract/pull/4211 which I consider to be an improved solution for diplopia.
I encourage everyone on this trail to try this out and test it with as broad a range of cases as possible.
Note by the way, there are some new configuration values that can only be set in code as things stand. These configuration values are:
bool kRemoveDiplopia - if true, enables diplopia removal functionality. If false, my changes have no effect
int kMaxDiplopiaGap - maximum number of timesteps apart to be considered diplopia, default 2
Obviously if my diplopia change is of value, then these configuration items should be made into settings.


Provide a setup.py file for installation and packaging

Thanks a lot for the patch. I tried it on Ubuntu 11.10 and I got:

ondrej@eagle:~/repos/py2js(thenewguy-dev)$ make
make -C examples generate
make[1]: Entering directory `/home/ondrej/repos/py2js/examples'
Compiling [gol.py]
Traceback (most recent call last):
  File "gol.py", line 1, in <module>
    from py2js.decorator import JavaScript
  File "/home/ondrej/repos/py2js/py2js/__init__.py", line 8, in <module>
    __version__ = get_version()
  File "/home/ondrej/repos/py2js/_version.py", line 46, in get_version
    raise ValueError('Could not determine version.')
ValueError: Could not determine version.
make[1]: *** [gol.html] Error 1
make[1]: Leaving directory `/home/ondrej/repos/py2js/examples'
make: *** [all] Error 2

Does make work for you on Windows?

Also, try to run tests:
./run_tests.py

I run them in your branch by using the generated files from the master and some of them failed for me. Do all tests pass for you?

Make and tests work for me on Ubuntu.  I have provided a list of commands to produce the environment in an AWS EC2 instance.  Is it acceptable to require an install before make will work?
I have also tested on Windows 7 and all tests pass.
Note that it took a recent copy of v8 compiled from svn for the tests to pass. Otherwise roughly 23 failed.
aws instance: ami-a7762ae2 (Ubuntu 11.10)

sudo apt-get install -y git python-setuptools make subversion scons lynx gcc g++

cd ~
svn checkout http://v8.googlecode.com/svn/trunk/ v8
cd v8

sudo scons d8 -j4

sudo cp ./d8 /usr/bin/js

cd ~
git clone https://github.com/thenewguy/py2js.git
cd py2js

sudo python setup.py install
sudo make
sudo python run_tests.py

Currently py2js works without having to use "sudo" nor to install it. (Of course, you should be able to install it as well.)
I would like to keep it that way, so that one can develop/test it quickly without having to install it.
It must be possible to improve your patch to make this work. I am currently busy to figure it out, but if you get stuck, I'll try to find some time for it.
Here is the tests log from the latest master with Ubuntu's v8:
https://gist.github.com/1383861
as you can see, all tests pass. Can you post here what tests fail for you with Ubuntu's v8? Does it fail only with your branch or also with master?
Thanks again for all your work.

I think so too, it should be simple. If you do it, then I can test that things still work on my machine and I can merge this patch in.


CPLY Loader/Writer

I think some of the built-in vertex shaders are exactly the same, you can cache the same asset under multiple paths.

done


bug in the code to calculate

@daizuozhuo  There is something wrong in the function eval_func_gpu, the default max_rank = 50. When the numbers of images of the same id is larger than 50, it will return a wrong mAP.

you can create a merge request for this bug.


Feature: Option to export punched tape to png

Describe your feature or enhancement suggestion
Add option download punched tape as png
Describe a concrete use case

I see how this could be useful but I can't offer frontend-side rendering of PNGs for this use case, right now.


update travis config to match master

This was fixed a few days ago but only on the master branch. Found while trying to troubleshoot Netlify failures to post deploy URLs to 1.11 PRs. Might as well fix builds while we're at it ...

/assign @mistyhacks

OK, looked just quickly at Travis log and think I know what's going on. But I'll investigate and fix up tomorrow. My guess is that this build will fail until we merge master back into 1.11. But any and all corrections/fixes by others most welcome.

@Bradamant3 The 1.11 gate job is completely broken. It is due to upstream change to the kubernetes/kubernetes code.

@Bradamant3 As I have commented in another PR, the gate is broken due to the following change upstream:
https://github.com/kubernetes/kubernetes/pull/63206
You cannot simply turn the go test command on without fixing the code. I have pinged the author of the above PR, but so far ... no response.

@Bradamant3 Ideally, the release-1.11 branch of website should be constantly tested against the master branch of kubernetes. When 1.11 is out, our release-1.11 will become the master, we change the gate job once and make the whole site always tested against the release-1.11 branch.
The correct fix, imo, should come from the upstream. We need some big fish to swim there NOW and yell out "you break us!!!"

Regardless, thanks for the nudge to rebase 1.11 on master, I'll do that this afternoon.


Contribution guidelines should cover how to create a branch for a pull request

Closing, since this isn't really needed with the current project velocity.


Bootstrap native equivalent of twbs #22049 and #21079

Hi and thanks for this wonderful piece of code.
I am opening an issue because Bootstrap Native prevents the click event from being propagated when an input triggers a collapse. In other words, when a checkbox is used for toggling a collapse using the data api, the collapse will toggle but the checkbox state remains the same: it can't be checked or unchecked.
As said, bootstrap had this exact same behavior before the fix #22272, as the issues #22049 and #21079 mentions.
It seems a similar fix could quickly be applied in the native side of the force.
Best regards.

I think you could do some CSS/JS to make it work your way, we don't really rush to bloat our codebase.

I think you could fork and do some CSS/JS to make it work your way, we don't really rush to bloat our codebase.

Thanks for your fast response.
I will fork your project to pull theses changes then. I just discovered an other undesired behavior that blocks an element from triggering multiple collapses at once, i think i will take the time to correct this too.


Artifacts while in Beyond Compare

I got heavy artifacts when another windows was in foreground of beyond compare while comparing to files.
Version:
Beyond Compare: 3.3.3 (Build 14128)
Windows: windows server 2008 r2
freerdp: 1.0.2
os: Thinstation 5 SMP kernel amd64
wm: Xfce4

Closing due to lack of activity.
Reopen/create a new issue if still relevant.


Remove flake8 excludes

Assures that all in-repo code is run under flake8, only excludes
allowed should be those already listed in .gitignore file.
Signed-off-by: Sorin Sbarnea<EMAIL_ADDRESS>Please include details of what it is, how to use it, how it's been tested
PR Type

Bugfix Pull Request

CI failures unrelated (flaky blah blah https://travis-ci.com/ansible/molecule/jobs/191901246).
Merging, thanks!


How can I use texture.frame?

Description
export default function ( props ){
const { x, y, texture, name, data, } = props;
console.log( data, name );
const Rectangle = PixiComponent( 'Rectangle', {
	create: () => new Graphics(),
	applyProps: ( g, _, props ) => {
		console.log( props, 'props' );
		const { x, y, width, height, } = props;

		g.clear();
		g.beginFill( 0xff700b, 1 );
		g.drawRect( x, y, width, height );
		g.endFill();

		texture.frame = g;
	}
} );

const frameData = data.frames[ name ].frame;

return (
	<Container position={[ x, y, ]}>
		<Rectangle x={frameData.x} y={frameData.y} width={frameData.w} height={frameData.h}/>
		<Sprite texture={texture}/>
	</Container>

);

};
It did not work when I change "x" and "y" in "applyProps", components show the first image all the way
Additional info

@inlet/react-pixi version: 1.1.11
React version:16.8.6
ReactDOM version:16.8.6
PIXI version:5.2.0

Can you please create a codepen/codesandbox example?

I'll close this issue for now, please let me know if you think we should reopen this issue.
Here's an example of a custom component icm graphics:
https://codepen.io/inlet/pen/RMLJKm
As you can see the props are passed through correctly.
Btw, you can use react-pixi's Graphics.draw as well to achieve something similar.


Finish the render

Basically this is the main issues I'm currently tackling. I'm right now adding an opengl wrapper or glw which is in the has a bunch of opengl functionality that is wrapped to make it more focused on the modern pipe. Internally I've decided to use OpenGL 4.x as my main render api. I might decided later on to support Direct3D 11 if I need it (aka driver issues).  Here a list of stuff I need for glw

[x] Shader
[x] Uniform
[x] UniformBlock
[x] VertexArray
[x] Program
[x] VertexBuffer
[x] IndexBuffer
[x] Texture2D
[x] Texture2DArray
[x] UniformBuffer
[x] FrameBuffer

That should sum up what I need for GLW its not a big list because the game engine isn't a 3D game engine so I don't need to add in Cubemaps or RenderBuffer because I'm don't need to Depth buffer for any thing. I might add that in to allow for Z ordering down the line. Next is for the actual render interface. The goal for this render interface is to only allow for the functionality that I'll need for the engine and nothing more. For instance sprite rendering... I can use just the main program to handle most of it but the drawing routine will require me passing a custom VertexBuffer for texCoords. Another example will be for TileMaps and ObjectMaps this will require both a custom VertexBuffers for both the vertices and texCoords. The other thing about the render is that internally it will have a fixed resolution at 640x480 (4:3) or 640x360 (16:9) so the actual render be independent to the actual resolution of the window. This means I'll have to make a FrameBuffer combo and a Texture2D that can resize if the use changes the resolution in the eventual setting menu.

I forgot one thing... I also need to add in font render here too. I need to display text output to the player which will be useful for alot of thing such as huds, custom ui elements, messagebox, etc.

Ok here is a simple example of what the render can do at this time...

I doesn't do much just yet but its a start.

Here is the introduction video of the issue as well as a bit of progress

Got done adding UniformBlock and UniformBuffer to the render. I also added a render interal interface for stuff that the rest of the application doesn't need to access to. Alright time to add in the other stuff.

Alright I've added in Texture2DArrays which is cool. Here is what it actually does.

What does a Texture2DArray does.
Here is the code example to initialize a Texture2DArray from file. This example is from main.cpp currently.
"""
render::glw::Texture2DArray::createTexture2DArrayFromFiles(&icon_array, {
"data/icon/icon_32.png",
"data/icon/icon_32_green.png",
"data/icon/icon_32_blue.png",
"data/icon/icon_32_magenta.png"
});
"""
What I'm upload to are these images.
icon_32.png

icon_32_green.png

icon_32_blue.png

icon_32_magenta.png

In order to get the rough fx we'll have to use custom texturecoords...
"""
icon_array_texCoords.init();
icon_array_texCoords.add3(0.0f, 0.0f, 0.0f);
icon_array_texCoords.add3(1.0f, 0.0f, 1.0f);
icon_array_texCoords.add3(0.0f, 1.0f, 2.0f);
icon_array_texCoords.add3(1.0f, 1.0f, 3.0f);
icon_array_texCoords.update();
"""
In order  to render it we'll need to use the TEXTURE2D_ARRAY shader and the rest will work the same way as the normal textures.
"""
render::startShader(render::ShaderType::ST_TEXTURE2D_ARRAY);
//render::setView(glm::mat4(1.0f));
render::setModel(
glm::translate(glm::mat4(1.0f), glm::vec3(this->postion, 0.0f)) *
glm::scale(glm::mat4(1.0f), glm::vec3(32.0f, 32.0f, 0.0f))
);
icon_array.bind(GL_TEXTURE0);
render::draw(icon_array_texCoords);
icon_array.unbind(GL_TEXTURE0);
render::endShader();
"""
This is complete and I can't wait to start using them for tilemapping.

Time for the framebuffer.

Alright got done with the frame buffer. Which is cool. I'm going to create a quick video to document how it works because its a little hard to describe its functionality of without showing it off.

Here is the video.

Here is a video to the post processing demo to demonstrate what the current render could do.

Alright I'm back to the main branch. The last thing I want to add into the render section of the engine is font or text rendering. I've already started using my faviorit font pack which has a bunch of nice fonts for games. They are located in bin/data/font directory. I'll be trying to load them using freetype and then rendering them using OpenGL textures.

[ ] Load Freetype
[ ] Grab data from freetype and construct a texture.
[ ] Render Texture to screen
[ ] Design it to allow the use of multiple fonts with different sizes.

Alright I'm back. Had to do some family stuff which made up most of my time. Went to a dinner to honor my uncle who tragically died saving his men during Vietnam war from a tragic explosion that happened on a plane they were riding on. He was a great man according to my mom. Well I'm back

                                                                       ##

############             #########         ###########       ################
#####          ####   #####       ####     #####     #####
###         ###       ####     ###          ##     ##
###       ###         ####    ##            ##    ##
###      ###          ###    #     #####   ##    ##
##      ##   #####    ##    #   #### ##   ##    ##
##     ###   ## ###   ##    #   ###   ##  ##    ##
##    ##    #    #   ##    #   ##    ######    ##
#####    ##    ##    #    ######    #   ##              ##
#  ###   ##    ##    ###            #   ##              ##
#   ##    #     ##   ######         #   ##              ##
#   ##    #     ##     ######       #   ##              ##
#    #    #      ##      #####      #   ## #########    ##
#    #    #      ####       ###     #   ## ##      #    ##
#    #    #        #####     ##     #   ## ##      #    ##
#    #   ##            ###    ##    #   ## ##      #    ##
#   ##    #              ##   ##    #   ## #####   #    ##
#  ##    ##              ##   ##    #   ##    ##   #    ##
#####    ##    #######   ##   ##    #    ##   #    #    ##
#     ###  ### ###   ##    #    ######   ##    ##
##     ##    #####    ##    ##    #####   ##    ##
##     ##            ###    ##            ##    ##
##       ##          ###     ##            ##    ##
###       ###        ####      ##          ##     ##
##  ##         ######  #####       #### #   #####     ##
#############          #########            ###########      ################

                                                                       **

**

Here is an example of freetype rendering text to a cli

Alright added in font rendering :)

I'm going to add in a separate shader to render fonts because it should be rendered on top of the game render unless it being used with in the render. Plus I want to add in coloring.

Alright added in a font shader to render out color font :)

Here is the postprocessing image...

Here is the final video for a bit. I'm going to move onto the sound issue and add in the sound api finally.

Alright I'm going to be adding in 4 new additions to the render system.

Texture2D Manager
Texture2DArray Manager
VertexBuffer Manager
IndexBuffer Manager
Reload functions for all of these system + render global as well.

Texture2D manager will manage the 2D textures from a centralizer location. Now not all Texture2D will require a file load because they are being use to render to texture or some other purpose.

Texture2DArray manager will be very similar to the Texture2D manager but will manage Texture2DArrays

Reload function are functions that will be called when the game reloads. I'm still debating on how this will work because I might need two functions for this one were I'm releasing resources and another reinitializing them. I might split it up into 2 different functions

reloadInit()
reloadRelease()

This actually might be better.


Define user story around accessing served models via Seldon

When a user serves a model using Seldon in their Kubeflow namespace, there are different use cases for accessing a model that they might want to satisfy:

accessing the model from a Kubeflow Notebook
accessing the model from a Kubeflow Pipeline
accessing the model from another user's Kubeflow Notebook (eg userA serves the model and userB' Notebook consumes it)
accessing the model from another user's Kubeflow Pipeline (eg userA serves the model and userB's Pipeline consumes it)
accessing the model from outside Kubeflow with authentication required
accessing the model from outside Kubeflow without authentication required (eg: true public access to the model)

For each of these use cases, we should have:

an automated test demonstrating it works
a user guide showing a user how to use this feature

canonical/bundle-kubeflow#557 cites issues with this access story.  In general, we need to define some clear manual or automated tests and see exactly what does and does not work, then provide guides for users.


Charts don't always have all 3 sections; section labels are not evenly padded

summary
https://staging.covidactnow.org/us/ca
https://www.dropbox.com/s/ig4hhreaqo77fb6/Screenshot 2020-04-27 11.49.02.png?dl=0
https://www.dropbox.com/s/8ql38wue4fjai84/Screenshot 2020-04-27 11.49.15.png?dl=0
expected result
Charts always have 3 sections (and are always the same static height ideally).
Padding around labels is even
actual result
Charts don't always have all 3 sections;
section labels are not evenly padded
environment
(device, browser, mobile / web)
link
screenshot
https://www.dropbox.com/s/ig4hhreaqo77fb6/Screenshot 2020-04-27 11.49.02.png?dl=0
https://www.dropbox.com/s/8ql38wue4fjai84/Screenshot 2020-04-27 11.49.15.png?dl=0
(you can copy/paste images directly in)

1. Charts don't always have all 3 sections
Currently, the maximum value shown in the chart is computed from the data, so if the max value for Rt is in the "medium" zone, the red region is not going to show.
https://github.com/covid-projections/covid-projections/blob/ca6589d9305144d19081062a02f93e61f6c53227/src/components/Charts/zoneUtils.js#L41-L51
We can fix this by forcing the red area to be the maximum between the data and the point where the orange and red regions would have the same height
// Something like this should work
const [minYAxis, maxYValues] = const [minYAxis, maxYAxis] = roundAxisLimits(0, getMaxY(data));
const  mediumRegionHeight = CASE_GROWTH_RATE.MEDIUM.upperLimit - CASE_GROWTH_RATE.LOW.upperLimit;
const maxYAxis = Math.max(maxYValues, CASE_GROWTH_RATE.MEDIUM.upperLimit + mediumRegionHeight)

2. Section labels are not evenly padded
The section labels are vertically centered in their respective regions. The boundaries between the regions are defined in https://github.com/covid-projections/covid-projections/blob/develop/src/enums/zones.ts - I just put values to mimic the mocks, I don't think that the values are correct. The values might not be evenly spaced either.
@maxwhenderson do we know how we will determine what is low, medium, and high for each one of the charts (Rt, positive test rate, hospital usage rate)?
I can fix 1) this evening or help someone else fix it during the day, for the second one, I think we need to see how it looks once we land definitive values for low, medium, and high for the charts to see if we need to adjust the positions.

@pnavarrc made a bunch of fixes / tweaks.  If there are outstanding issues, please open a new bug.


fixes #558 fixes #1038 construct an array of lambda functions

"f(x)=x; g(x)=x; [f, g]"

Codecov Report

Merging #1097 (0131498) into develop (84be552) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #1097      +/-   ##
===========================================
- Coverage    59.00%   58.93%   -0.07%
===========================================
  Files           82       82
  Lines         4398     4398
===========================================
- Hits          2595     2592       -3
- Misses        1803     1806       +3

Impacted Files
Coverage Δ

...ucer/Reducer_Dispatch/Reducer_Dispatch_BuiltIn.res
59.34% <100.00%> (ø)

.../Reducer/Reducer_Expression/Reducer_Expression.res
81.25% <0.00%> (-9.38%)
:arrow_down:

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

It looks like mapping works as expected, neat.

The interface works for me, will merge.

Impressive how tiny the fix turned out to be!
There's this note in Reducer_Expression.res:

TODO: remove on solving issue#558

Should it be updated?


bump version to 0.11.3

Add two improvements to the ratechecker data-loading routine:

Fix an IndexError that led to server errors.
Re-sequence data loading to catch partial-load conditions.

I don't understand how this change could have affected frontend test coverage. Very strange, codecov.
More seriously, I noticed what seems to be minor a docs bug in the newest version, that breaks one of the doc footnotes.

@chosak thanks for catching the docs bug. It is fixed.

Minor request before merging: would you mind re-titling this PR so that the cfgov-refresh release log has a more descriptive message that mentions owning-a-home-api?


refine dir in PaddlePaddle/Mobile and add rounding

refine dir in PaddlePaddle/Mobile and add rounding

Too much update in this repo, need to Separate this into several parts


Impossible to use same fonts hashable with markdown

Due too maping bug in version 0.4.1 currently there is not possible to have same fonts with different styling in deck.

Fixed in 75ee035f7f5f5caf2067cf33ed0c24d684629978


Provide approved content for HOME

the last approved mockup can be found here: https://invis.io/7WXZSH0E8U6#/425309410_Homepage_-_D56
Content will be directly shared by Zia via telegram an AI file. Maximilian will coordinate.
Final text changes will be made when on staging.

AI file provided via Telegram.