Update the ownership policy of weights to prevent possible bugs

Currently, the weights are owned by the layer, and the manager keeps a reference. So, if a layer is deleted before the manager is, that leaves the manager with undefined references and there is no way to verify if a reference is valid in c++.
One of the solutions is to directly use shared_ptr for shared ownership. There might be other solutions as well.

:octocat: cibot: Thank you for posting issue #1030. The person in charge will reply soon.

Quick thought

we register layer instead of weights to the manager. when we delete layer, we notify manager
using weak_ptr

we register layer instead of weights to the manager. when we delete layer, we notify manager

IMO, if you want to register a layer, it must be to the graph. The graph can later provide necessary information to the manager. However, if the user allocates a layer object, the user can reset/delete it as well without the graph knowing it (unless there is a special interface to delete the layer from the graph and it's not a good design IMO) or the graph keeps polling it.
weak_ptr/shared_ptr for shared_ownership solves the above issue as well as the issue of reference_wrapper with a minor overhead.

This is done with layer_context and LayerV2 design. Closing this here.


chore: Add environment variable check script

Pull Request
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

#126  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JackPlowman and the rest of your teammates on  Graphite

Merge activity

Oct 5, 3:35 PM EDT: @JackPlowman merged this pull request with Graphite.


QFE: new middleware to force query statistics collection

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Adds a new middleware for both instant and range queries that might force stats collection.
Adds new query stats to logs
Update protobuf for stats package.

Verification

Can you share an example log line? Just want to see how it looks like

Can you share an example log line? Just want to see how it looks like

{
"caller": "handler.go:217",
"grafana_dashboard_uid": "jmwtDwa4k",
"grafana_panel_id": "120",
"host": "localhost:8084",
"level": "info",
"method": "POST",
"msg": "slow query detected",
"org_id": "anonymous",
"param_analyze": "true",
"param_dedup": "true",
"param_end": "1729183020",
"param_engine": "thanos",
"param_max_source_resolution": "0s",
"param_partial_response": "false",
"param_query": "sum(rate(nginx_ingress_controller_requests{}[5m])) by (ingress)",
"param_start": "1729172212",
"param_step": "360",
"path": "/api/v1/query_range",
"peak_samples": 409,
"query_range_hours": 3,
"query_range_human": "3h0m8s",
"remote_addr": "[::1]:55935",
"remote_user": "",
"time_taken": "3.518708209s",
"total_samples_loaded": 720187,
"trace_id": "a990d4d2c94a5530b48d94d0e5e55664",
"ts": "2024-10-24T20:11:58.16198Z"
}

@yeya24 do you mind reviewing the PR, pls?

The E2E tests are working locally:

I see so we are not calling AddFetchedSeries, AddFetchedChunkBytes, etc in Thanos querier.
This is fine. But I don't think collecting this data is conflicting of collecting the standard query stats. We can still do both.
Anyway, it is not blocking this PR so we can change if we feel useful

I can try working on this as a follow up.


[BUG] Junk in the log v1.3.0

Describe the bug
Some junk is being logged in console
Additional context

Looks like i left some debugging in there... will clear that up


Tooltip on Open button looks wrong

The tooltip for the open button says "open issueControl  in browser" - guess it should either be just "issue" (not "issueControl") or should replace with ID?  I think just "issue" would be fine.

Nice spotted - I guess a search/replace in the code at some time have been a bit too aggressive. ;-)


Fixes for release pages and package display names

@ronniegeraghty, you may want to get this https://github.com/Azure/azure-sdk/pull/6753/files#diff-dd42604ba12aca9015fe2cdaf131dae4551ad94571e5ea6daec3ec48765adfa9 merged before this PR, so that when the other one gets merged, it does not overwrite your changes.

A new PR to merge to avoid the entries being overwritten: https://github.com/Azure/azure-sdk/pull/6760/files#diff-dd42604ba12aca9015fe2cdaf131dae4551ad94571e5ea6daec3ec48765adfa9

If I understand correctly, dropping "Event Hubs - " here will result in it getting updated in the merged changelog (https://azure.github.io/azure-sdk/releases/2023-10/cpp.html) and release notes (https://github.com/Azure/azure-sdk-blog/pull/441/files#diff-ca9d0fef1e7ddccd45aad972bebd60b027b8241da48d0baa869af138407d6ed8R64).
Does "Azure Blob Storage Checkpoint Store" alone there, without the "Event Hubs - " prefix look good and clear?


Add colorspace option

One of the p5 shader examples uses hsb color space to convert colors to angles. I should try that and see if that changes the appearance of anything.

An option / setting to use hsb / rgb


issue #568: small embedding net as default

There you go @michaeldeistler

Thanks for the updates! There are merge conflicts now though, let me know if you need help with resolving those.

Closing in favor of #624


fix(console): avoid crash when accessing selected item

We should check the length before using the index to access it.
close https://github.com/tokio-rs/console/issues/565
Test locally:
https://github.com/tokio-rs/console/assets/29879298/5c4fd5da-e1c7-490b-bd67-1257972076d3
But it is difficult to view it, you can try it by following the steps from the issue.

@hds Need your approval to merge it. Could you please take another look? Thank you!

Thanks for your review!  💚 💙 💜 💛 ❤


labs:enable, labs:disable only filter on name

Currently the list of Features must be unique since the CLI just looks for the first thing by name:
https://github.com/heroku/heroku/blob/master/lib/heroku/command/labs.rb#L113
In order to support a future where we can have say a user feature named foo and an app feature named foo, the CLI will need to look for features by name and kind.

my thought was that we would keep it backwards compatible and rather than falling back on 404, we would query both, detect duplicates, and force a flag to pick between the two

Ugh why you gotta make my life hard @mikehale :)

Btw that is not the code, this is the code https://github.com/heroku/heroku-apps/tree/master/commands/labs

@dickeyxxx so actually I didn't realize that this feature is now handled by the node code in heroku-apps. @ransombriggs pointed me to plugins:commands, as I was confused by a simple grep of ~/.heroku.
Also this is not an urgent thing, but something that would be nice to support in the future.

there is also heroku which

@mikehale this somewhat involves this issue: https://github.com/heroku/cli/issues/259.
Essentially I plan to get rid of this command and replace it with apps:features/labs and user:features/labs


AzureTre master repository not cloned.

As a developer
When I clone the parent repository (as per guide https://microsoft.github.io/AzureTRE/0.8.0/) to my local machine, AzureTRE repository is not copied in as a subfolder (subrepository)
As a result, MakeFile cannot run

Clone AzureTre repository in Visual Studo Code (https://github.com/microsoft/AzureTRE-Deployment)
Re-open in a Docker container
run "Make auth"

@slawpast can you try building the container using:
 docker build -f ./.devcontainer/Dockerfile .
and provide the logs. The TRE repo is pulled in towards the end:
Step 52/54 : RUN oss_version_in_json=$(grep -oP '(?<="OSS_VERSION": ")[^"]*' /tmp/devcontainer.json)     && /tmp/install-azure-tre-oss.sh "${OSS_VERSION:-$oss_version_in_json}" "${AZURETRE_HOME}"     && sudo chown -R $USERNAME ${AZURETRE_HOME}


[fix-DMN15Test] Fix test

THis PR populate context for all rest invocations in the DMN15Test#commonUnnamedImport method

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

for pull request checks
Please add comment: Jenkins retest this

for a specific pull request check
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] tests

for quarkus branch checks
Run checks against Quarkus current used branch
Please add comment: Jenkins run quarkus-branch

for a quarkus branch specific check
Run checks against Quarkus current used branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] quarkus-branch

for quarkus main checks
Run checks against Quarkus main branch
Please add comment: Jenkins run quarkus-main

for a specific quarkus main check
Run checks against Quarkus main branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] quarkus-main

for quarkus lts checks
Run checks against Quarkus lts branch
Please add comment: Jenkins run quarkus-lts

for a specific quarkus lts check
Run checks against Quarkus lts branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] quarkus-lts

for native checks
Run native checks
Please add comment: Jenkins run native

for a specific native check
Run native checks
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] native

for native lts checks
Run native checks against quarkus lts branch
Please add comment: Jenkins run native-lts

for a specific native lts check
Run native checks against quarkus lts branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] native-lts

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

Quarkus-3 PR check is failing ... what to do ?

The Quarkus 3 check is applying patches from the `.ci/environments/quarkus-3/patches`.
The first patch, called 0001_before_sh.patch, is generated from Openrewrite .ci/environments/quarkus-3/quarkus3.yml recipe. The patch is created to speed up the check. But it may be that some changes in the PR broke this patch.
No panic, there is an easy way to regenerate it. You just need to comment on the PR:
jenkins rewrite quarkus-3

and it should, after some minutes (~20/30min) apply a commit on the PR with the patch regenerated.
Other patches were generated manually. If any of it fails, you will need to manually update it... and push your changes.

PR job #2 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-apps -u https://github.com/apache/incubator-kie-kogito-apps/pull/2034 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-apps-pr/job/PR-2034/2/display/redirect
Test results:

PASSED: 2783
FAILED: 1

Those are the test failures:

org.kie.kogito.index.ProcessDataIndexInfinispanIT.testProcessInstanceEvents
1 expectation failed.JSON path data.UserTaskInstances[0].potentialGroups[0] doesn't match.Expected: managers  Actual: null

PR job #1 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-apps -u https://github.com/apache/incubator-kie-kogito-apps/pull/2034 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-apps-pr/job/PR-2034/1/display/redirect
Test results:

PASSED: 1596
FAILED: 3

Those are the test failures:

org.kie.kogito.index.mongodb.ProcessDataIndexMongoDBHttpIT.testProcessInstanceEvents
1 expectation failed.JSON path errors doesn't match.Expected: null  Actual: <[{message=Exception while fetching data (/UserTaskInstances[0]/schema) : Runtime service URL not defined, please review the kogito.service.url system property to point the public URL for this runtime., locations=[{line=1, column=85}], path=[UserTaskInstances, 0, schema], extensions={classification=DataFetchingException}}]>

org.kie.kogito.index.jdbc.mapper.JobEntityMapperIT.testMapToModel
java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors [error]: Build step io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor#startKeycloakContainer threw an exception: java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image quay.io/keycloak/keycloak:23.0.7 at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.startKeycloakContainer(KeycloakDevServicesProcessor.java:249) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.base/java.lang.reflect.Method.invoke(Method.java:568) at io.quarkus.deployment.ExtensionLoader$3.execute(ExtensionLoader.java:849) at io.quarkus.builder.BuildContext.run(BuildContext.java:256) at org.jboss.threads.ContextHandler$1.runWith(ContextHandler.java:18) at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2513) at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1538) at java.base/java.lang.Thread.run(Thread.java:833) at org.jboss.threads.JBossThread.run(JBossThread.java:501)Caused by: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image quay.io/keycloak/keycloak:23.0.7 at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:359) at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:330) at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.lambda$startContainer$4(KeycloakDevServicesProcessor.java:376) at java.base/java.util.Optional.orElseGet(Optional.java:364) at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.startContainer(KeycloakDevServicesProcessor.java:401) at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.startKeycloakContainer(KeycloakDevServicesProcessor.java:197) ... 11 moreCaused by: org.rnorth.ducttape.RetryCountExceededException: Retry limit hit with exception at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:88) at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:344) ... 16 moreCaused by: org.testcontainers.containers.ContainerLaunchException: Could not create/start container at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:563) at org.testcontainers.containers.GenericContainer.lambda$doStart$0(GenericContainer.java:354) at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:81) ... 17 moreCaused by: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for log output matching '.*Keycloak.*started.*' at org.testcontainers.containers.wait.strategy.LogMessageWaitStrategy.waitUntilReady(LogMessageWaitStrategy.java:47) at org.testcontainers.containers.wait.strategy.AbstractWaitStrategy.waitUntilReady(AbstractWaitStrategy.java:52) at org.testcontainers.containers.GenericContainer.waitUntilContainerStarted(GenericContainer.java:909) at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:500) ... 19 more

org.kie.kogito.index.postgresql.reporting.ProcessInstanceVariableMappingIT.testProcessInstanceVariableMapping
java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors [error]: Build step io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor#startKeycloakContainer threw an exception: java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image quay.io/keycloak/keycloak:23.0.7 at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.startKeycloakContainer(KeycloakDevServicesProcessor.java:249) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.base/java.lang.reflect.Method.invoke(Method.java:568) at io.quarkus.deployment.ExtensionLoader$3.execute(ExtensionLoader.java:849) at io.quarkus.builder.BuildContext.run(BuildContext.java:256) at org.jboss.threads.ContextHandler$1.runWith(ContextHandler.java:18) at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2513) at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1538) at java.base/java.lang.Thread.run(Thread.java:833) at org.jboss.threads.JBossThread.run(JBossThread.java:501)Caused by: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image quay.io/keycloak/keycloak:23.0.7 at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:359) at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:330) at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.lambda$startContainer$4(KeycloakDevServicesProcessor.java:376) at java.base/java.util.Optional.orElseGet(Optional.java:364) at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.startContainer(KeycloakDevServicesProcessor.java:401) at io.quarkus.oidc.deployment.devservices.keycloak.KeycloakDevServicesProcessor.startKeycloakContainer(KeycloakDevServicesProcessor.java:197) ... 11 moreCaused by: org.rnorth.ducttape.RetryCountExceededException: Retry limit hit with exception at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:88) at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:344) ... 16 moreCaused by: org.testcontainers.containers.ContainerLaunchException: Could not create/start container at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:563) at org.testcontainers.containers.GenericContainer.lambda$doStart$0(GenericContainer.java:354) at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:81) ... 17 moreCaused by: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for log output matching '.*Keycloak.*started.*' at org.testcontainers.containers.wait.strategy.LogMessageWaitStrategy.waitUntilReady(LogMessageWaitStrategy.java:47) at org.testcontainers.containers.wait.strategy.AbstractWaitStrategy.waitUntilReady(AbstractWaitStrategy.java:52) at org.testcontainers.containers.GenericContainer.waitUntilContainerStarted(GenericContainer.java:909) at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:500) ... 19 more

Failures on
continuous-integration/jenkins/pr-head unrelated


corgi: if job is aborted too fast, does not get the 'Job was aborted' message

When a job is aborted, in the Job detail dropdown I see ’Job was aborted.” message with black background. If a job is aborted too fast, I don’t get that message in the dropdown.
message from Thomas:
Yes please. If the job is aborted too quickly, concourse will never process it (and thus never write a log for it). Instead, the frontend should send a message to the backend as well. Maybe something like "Job was aborted before assignment"

Acceptance Criteria

An error message still occurs when a job is aborted before it is assigned to a concourse worker.

Testing on corgi prod (it was promoted to prod directly):
pdf, web preview, pdf-git, web preview git
college-algebra-bundle/precalculus
col11992
OK

Testing on corgi prod (it was promoted to prod directly):
pdf, web preview, pdf-git, web preview git
college-algebra-bundle/precalculus
col11992
OK


onedrive api in localhost

Hi,
How to test the onedrive api in localhost ?
Thanks,
SVNindia

"Testing OneDrive API in localhost" - what I understand is implementing and testing Authentication / Authorization process (OAuth 2.0) offered by OneDrive API.
Here is how you can accomplish that:
In the application you created at "Microsoft Account Developer Center", go to > Edit Settings > API Settings
Any URL like localhost or localhost:34342 is not allowed and gives an error on save like:

You must enter a valid domain that begins with http:// or https:// – query strings are not allowed. Length is limited to 248 characters.

To overcome this issue you can add a test domain entry in your development machine's host file (Path: C:\Windows\System32\drivers\etc\hosts).
Example:
<IP_ADDRESS>               localhost34342.com
<IP_ADDRESS>               www.localhost34342.com
Use following URL as Redirect URLs : http://localhost34342.com/redirectendpoint/ in Edit Settings > API Settings and save settings.
This will enable you to test the OneDrive API now on local host / development environment.
Hope it helps,
Jaspal

@svnindia Did you manage to get it working?  The OneDrive team has often used https://pages.github.com/ for quickly setting up a domain to auth against

Thanks @jsinh @peternied
Yes its working now for me. Implemented the Single Sign On with Drupal . To Simply Test
Since I am on linux I used below steps
vi /etc/hosts
added the below line
<IP_ADDRESS>	localhost.dev


fix(helm): ensure helm resources have correct namespace

PR Description
Helm chart resources don't have correct namespaces.
Which issue(s) this PR fixes

Issue can be reproduced by running helm template --namespace alloy alloy grafana/alloy --version 0.9.2.
Generated yaml files didn't have correct namespace.
Notes to the Reviewer
PR Checklist

[x] CHANGELOG.md updated
[ ] Documentation added
[ ] Tests updated
[ ] Config converters updated

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

I get nervous about the removal of the use of the alloy.namespace template function. That might break some users.
Perhaps instead of using .Release.namespace, we use alloy.namespace which is enabled by the namespaceOverride value

I get nervous about the removal of the use of the alloy.namespace template function. That might break some users. Perhaps instead of using .Release.namespace, we use alloy.namespace which is enabled by the namespaceOverride value

Didn't seem that was the case considering that it was undocumented value and some resources were already using the {{ .Release.Namespace }} like in the operations/helm/charts/alloy/templates/pdb.yaml file.
But updated the PR regardless.

I resolve the conflict, please review. For now I'm using my local fork as a workaround.

Thanks @clayton-cornell  , I've updated the PR as requested and rebased.

@petewall @wildum Is this PR ready to approve/merge?


Overwrite

Hey Piotr,
this adds the overwrite option to tftpd.

I've "seen the light" recently too so Gemfile is appreciated.


Create 'Attribut' modification control

The modification control from #10

:tada: This issue has been resolved in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add exception transitions

Currently, exceptions will break the control flow of an action, stopping the program early. Thus, if an exception is expected, the program will stop early. We will be adding the ability to conditionally transition based on exceptions, which will allow you to transition to an error-handling (or retry) action that does not need the full outputs of the prior action.
Here is what it would look liek in the current API:
@action(reads=["attempts"], writes=["output", "attempts"])
def some_flaky_action(state: State, max_retries: int=3) -> Tuple[dict, State]:
    result = {"output": None, "attempts": state["attempts"] + 1}
    try:
        result["output"] = call_some_api(...)
    excecpt APIException as e:
        if state["attempts"] >= max_retries:
           raise e
    return result, state.update(**result)

One could imagine adding it as a condition (a few possibilities)
@action(reads=[], writes=["output"])
def some_flaky_action(state: State) -> Tuple[dict, State]:
    result = {"output": call_some_api(...)}
    return result, state.update(**result)

builder.with_actions(
   some_flaky_action=some_flaky_action
).with_transitions(
   (
      "some_flaky_action",
      "some_flaky_action",
      error(APIException) # infinite retries
      error(APIException, max=3) # 3 visits to this edge then it gets reset if this is not chosen
      # That's stored in state
)

Could also directly annotate it:
@action(reads=[], writes=["output"], error=(APIException, max=3))
def some_flaky_action(state: State) -> Tuple[dict, State]:
    result = {"output": call_some_api(...)}
    return result, state.update(**result)

Under the hood it would:

rerun the function with the same state inputs, except for incrementing an invocation counter.

and then it would display a loop back to the node based on error and then otherwise error the program out...

Or if it errors you want to set some state value.
@action(reads=[], writes=["output"], error={"write_to": "error_key"})
def some_flaky_action(state: State) -> Tuple[dict, State]:
    result = {"output": call_some_api(...)}
    return result, state.update(**result)

But I think what's missing here is

if some action errors
I want it to go to some specific "action" that I have defined on error
what's the best way to do that

Right now you have to manually define all the edges. But there could be a more systematic way.

Hello
I have the following use case:

I want to transition to a default action (let's call it RESET) when an exception occurs in any state action.
This RESET action would reset the state and transition back to the initial state.
Before transitioning, I'd like to catch the exception, modify the state to indicate that an exception occurred, and include a stack trace message for user communication.

The challenge I'm facing is that manually adding transitions from hundreds of states to the RESET state is impractical. Ideally, I'm looking for a catchall transition mechanism, something along the lines of (*, "RESET", when(exception_occurred=True))
A global exception handler might be a good approach:
def global_exception_handler(state: State, e: Exception) -> State:
   return state.update(
       exception_occurred=True,
       error_message=str(e),
       stack_trace=traceback.format_exc()
   )

builder.with_global_exception_handler(
   handler=global_exception_handler,
   target_state="RESET"
)

Some extra ideas from Slack:
@skrawcz says:
Yeah wildcard transition specifications seem like a good idea:
("*", "RESET", when(exception=True))

seems reasonable -- they would have to be at the end though.
Yeah interesting point <@599951704330338304> -- also I think we can expose state fields (E.G. exception) that suppresses and stores it. So:
@action(reads=..., writes=..., on_error=capture_as(field="exception")))
def ...
    ...

or on application build, for everything
app = (
    ApplicationBuilder()...
    .with_error_handling=capture_as(field="exception"))
    # then everything transitions to it
    .with_transitions("*", error_handler, expr("exception is not None"))
    .build


Redesign how couchapps are pushed to VM-based CouchDB nodes

Impact of the new feature
Central services and CouchDB
Is your feature request related to a problem? Please describe.
Yes, with the CMSWEB infrastructure migration, we are no longer deploying/enabling services in the VM nodes. This means, those nodes will not have couchapps updates (deployed with every WMCore service).
From the top of my head, couchapps are pushed in in the following way:

we deploy a service (e.g., reqmgr2) in the same CouchDB node that is supposed to run ReqMgr2-related couch apps (database, design document, map/reduce views, list functions, etc).
specific files are created - during deployment - at state/couchdb/stagingarea
when starting couchdb service, it scans the stagingarea and executes those files, which contain something like:

couchapp push /data/srv/current/apps/reqmgr2/data/couchapps/ReqMgrAux http://localhost:5984/reqmgr_auxiliary
couchapp push /data/srv/current/apps/reqmgr2/data/couchapps/ReqMgr http://localhost:5984/reqmgr_workload_cache
couchapp push /data/srv/current/apps/reqmgr2/data/couchapps/ConfigCache http://localhost:5984/reqmgr_config_cache

as one can see, it has a reference to the actual source code deployed under the current directory.

Describe the solution you'd like
We need to come up with a new procedure to push changes to the couchapps. Or redesign how we package CouchDB-related code such that we can keep deploying them in the nodes running the CouchDB services (3 or 4 nodes in production).
I still don't know how to properly resolve this problem, but a couple of ideas that come to my mind are:
a) try to push couchapps remotely (not sure our current couchdb setup would allow it)
b) build and package couchapps in a different way, such that it can be easily "deployable" on the VM-based nodes, and properly pushed in
Describe alternatives you've considered
A couple of suggestions made above, we still need to investigate what is doable or not, and what will be the best cost-benefit option.
Additional context
none

@goughes FYI

@goughes FYI


Albacore, Porechop and Indexing

Hi everybody,
I would like to ask about the processing of the reads and Nanopolish. Our workflow includes basecall FAST5 files using Albacore and then process Albacore's output with Porechop to trim barcodes and other sequences.
For example, if I have a run with 12 barcodes, after Albacore basecalling and Porechop filter, I get 12 FASTQ final files. Could I CAT those Porechop FASTQ files into 1 file (let's name it Porechop_CAT.fastq) and use it in the Index command?
nanopolish index -d /path/to/raw_fast5s/ -s sequencing_summary.txt Porechop_CAT.fastq
And then, if the previous is feasible, could I use the Porechop output for each barcode as the reads needed to align? An imaginary example, with BC01_Porechop.fa reads I assemble a genome (BC01_Porechop_genome.fa) with Canu.
bwa mem -x ont2d -t 8 BC01_Porechop_genome.fa BC01_Porechop.fa | samtools sort -o BC01_Porechop.sorted.bam -T BC01_Porechop.tmp -
samtools index BC01_Porechop.sorted.bam

Thank you so much for your help,
Quim

Hi @Jaoquien,
You can run nanopolish index on each barcoded file individually, passing in the complete directory of fast5s each time:
nanopolish index -d /path/to/raw_fast5s/ -s sequencing_summary.txt BC01_Porechop.fa

This should do the right thing, and let you polish the individual barcoded assemblies.
Jared

@jts Thanks!
But If I perform for example the CAT of all Albacore's FASTQs and then I make the Index, then I only need to do the index once and not with every Barcode, don't I? So my question was that if I index ALL the Albacore's FASTQs in one file, then I could work with each barcode separately without the need of indexing each time the FASTQ reads. Am I wrong?

nanopolish finds the index file based on the name of the input .fastq. So when you try to run on BC01_porechop.fa it won't be able to find the master index file you constructed. Its possible to hack around this by changing filenames but its probably not worth it - indexing the files individually is better.

@jts Perfect, thank you so much for your help!


I can't display subtitles

I'd like to display subtitles when I watch a video but I can't find how to do it.
Are captions displayed by default?
I use mpv as a video player and I run it on archlinux.
Here is my youtube-viewer.conf related to the player
  video_player_selected   => "mpv",
  video_players           => {
                               mplayer => {
                                 arg => "-prefer-ipv4 -really-quiet -title *TITLE*",
                                 audio => "-audiofile *AUDIO*",
                                 cmd => "mplayer",
                                 fs => "-fs",
                                 novideo => "-novideo",
                                 srt => "-sub *SUB*",
                               },
                               mpv => {
                                 arg => "--really-quiet --title *TITLE* --no-ytdl",
                                 audio => "--audio-file *AUDIO*",
                                 cmd => "/usr/sbin/mpv",
                                 fs => "--fullscreen",
                                 novideo => "--no-video",
                                 srt => "--sub-file *SUB*",
                               },
                               vlc => {
                                 arg => "--quiet --play-and-exit --no-video-title-show --input-title-format *TITLE*",
                                 audio => "--input-slave *AUDIO*",
                                 cmd => "vlc",
                                 fs => "--fullscreen",
                                 novideo => "--intf dummy --novideo",
                                 srt => "--sub-file *SUB*",
                               },
                             },
  videoCaption            => undef,
  videoDefinition         => undef,
  videoDimension          => undef,
  videoDuration           => undef,
  videoLicense            => undef,
  videoSyndicated         => undef,
  youtube_video_url       => "http://www.youtube.com/watch?v=%s",

Do I need to had an option in order to display captions?

Awesome ! It works :)
Thank you for your help.

How can I select other auto-generated captions than default (english)? Those languages that can be selected from "Auto-translate" submenu (Settings->Subtitles/CC->Auto-translate) in YouTube. For example, in this video: https://youtu.be/J0UhiO_pc90.


Try prompter for more consistent tooltips

Tippy seems to work only sporadically. We should try https://github.com/etiennebacher/prompter

Closed with b9df6e4


Tagging working at random, mostly not working.

I just started using this a few weeks ago but on almost all mkv's/mp4/m4v I keep getting the exact same error when it attempts to tag.
Trying to write tags
An unexpected error occurred, processing of this file has failed
cannot fit 'long' into an index-sized integer
There something that I have not set correctly?

As of commit 8375e799c53cfa9a0d87d62cccf747072875ee97 a workaround for this issue has been found and tagging should work on 32 bit systems regardless of file size


fix: move kubernetes autoscaling to v2

This version of autoscaling went GA back in December 2021 I think we can switch to it now. I don't think this is even a breaking change since you would need to be on a very old version of k8s to notice an issue.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip got a sec?


NTR: resistant starch

Preferred term label
resistant starch
Textual definition
Proceedings for the 2nd plenary meeting of EURESTA: European FLAIR Concerted Action
The total amount of starch, and the products of starch degradation that resists digestion in the small intestine of healthy people. The resistance of starch to digestion is inﬂuenced by the nature of the association between starch polymers, with higher amylose levels in the starch being associated with slower digestibility rates
Suggested parent term
soluble dietary fibre

From Wikipedia:
Resistant starch (RS) is starch, including its degradation products, that escapes from digestion in the small intestine of healthy individuals.

From Wikipedia:
Resistant starch (RS) is starch, including its degradation products, that escapes from digestion in the small intestine of healthy individuals.

I propose:
starch that escapes from digestion in the small intestine and are fermented in the colon of monogastric animals.
Raid, 1999
Tolulope et al., 2019

I propose:
starch that escapes from digestion in the small intestine and are fermented in the colon of monogastric animals.
Raid, 1999
Tolulope et al., 2019

In ruminants, starch that resists microbial enzymatic degradation in the upper gut sections (i.e., forestomachs) may be termed as bypass starch, undegradable starch, or ruminally resistant starch (RRS). More precisely, RRS is expected to resist degradation mechanisms in the rumen and be digested in the small intestine (Deckardt et al., 2013).
This is why we have to incorporate the term "monogastric animals".
As we should not use the term "starch" in the definition, we should use the definition from CHEBI:28017 for starch?
Hence the definition would be:
"Glucan that consisting of amylose and amylopectin that resists digestion in the small intestine of monogastric animals".

In ruminants, starch that resists microbial enzymatic degradation in the upper gut sections (i.e., forestomachs) may be termed as bypass starch, undegradable starch, or ruminally resistant starch (RRS). More precisely, RRS is expected to resist degradation mechanisms in the rumen and be digested in the small intestine (Deckardt et al., 2013).
This is why we have to incorporate the term "monogastric animals".
As we should not use the term "starch" in the definition, we should use the definition from CHEBI:28017 for starch?
Hence the definition would be:
"Glucan that consisting of amylose and amylopectin that resists digestion in the small intestine of monogastric animals".

A portion of starch that cannot be digested by amylases in the small intestine
Englyst, 1985 as cited in Birt et al., 2013

A portion of starch that cannot be digested by amylases in the small intestine
Englyst, 1985 as cited in Birt et al., 2013


Update font-awesome.css.erb

add version value to the url so it matches upstream fontawsome changes to the delcation to work around ipad issues

so it matches upstream fontawsome changes

Could you please provide a link to the specific upstream changes that your referring to?
Adding  ?v=x.y.z cache busters should have no effect on Mobile Safari. The font_path helper already constructs a custom cache buster via the asset pipeline. Could you please elaborate on the underlying problem that you're experiencing?
Please review #174 and #175 for more about why we removed those ?v= cache busters.

Closing this due to lack of follow-up.


Add DarkSlayer102 to Contributors list

My first Contribution. Thank you!

Hey there,
Thanks for your PR & contribution :wave:!
Don't forget to join follow me, join our organization, & follow our organization to stay updated with the latest developments 💻
Thank you, and have a good day ahead 😊


setData function

What is the correct way to assign data from an external source?
How to use setData(data) function?
Two important improvements:

Drag & drop rows / reordering rows
JSON import-export

Thanks!

Please get the most recent updates from the repository.
You can find a example below:
http://bossanova.uk/jexcel/remote-updates
Basically, the example will load from a external JSON file, and you can download the data as a JSON string. In regards to sorting, as default you can double click on the Column Headers. Drag and drop is the road map, hope to get this done as soon as we can.

<link rel="stylesheet" href="http://cdn.bossanova.uk/css/jquery.jexcel.css" type="text/css" />

<textarea style='width:600px;height:100px;' id='test'></textarea>

<input type='button' onclick="$('#test').html(JSON.stringify($('#my').jexcel('getData')))" value="Get json data">

Just to let you know, I have just commit a new version with the dragRow feature.


Add charlie to the committer list

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


Update README.md

Replaced a left double quotation with a regular quotation as the left double quotation causes the powershell script to not be imported when run.

Thank you @Lotsamocha01, this kind of fix will save someone ages trying to figure out why the script does work. Thank you for sharing, this will be live shortly 🎉
Do you have a Twitter handle/name I can attribute this fix to? Also, have you registered for the PnP Recognition program, your submission will qualify for a badge :-)
Its opt in so please register here: https://pnp.github.io/recognitionprogram


Updated Readme with a Contributor ! Hope you like it and Merge

What type of PR is this? (check all applicable)

[ ] 🚀 Added Name
[ ] ✨ Feature
[ ] ✅ Joined Community
[ ] 🌟 ed the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
Add Link of GitHub Profile

Sorry bro ,i didnt get it . so how shall I put it
Can u help me a bit , i too wanna add mt github profile in your readme
repository
https://github.com/iamshubhamg
On Tue, Sep 8, 2020 at 7:30 AM Abhishek Naidu<EMAIL_ADDRESS>wrote:

Closed #253
https://github.com/abhisheknaiidu/awesome-github-profile-readme/pull/253
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/abhisheknaiidu/awesome-github-profile-readme/pull/253#event-3738771951,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMVP2BDDIV5F2MD5EMATUEDSEWF4NANCNFSM4P7XJASQ
.


CDI-642 Allow defining interceptor binding or qualifier with an AnnotatedTypeConfigurator

CDI-642 Allow defining interceptor binding or qualifier with an AnnotatedTypeConfigurator

Spec part not written yet: API should be discussed first.

Hm, shouldn't the methods accept Class<T> arguments? Similarly as javax.enterprise.inject.spi.BeforeBeanDiscovery.addAnnotatedType(Class<T>, String)?
I also don't like the names. Maybe we could name the methods configureQualifier(Class) and configureInterceptorBinding(Class) since we're basically configuring the original definitions?

Agree on both points. changing the PR in that direction

Looks good. We should merge it ASAP so that we can identify possible implementation problems in time ;-)


use omnibus 'gem' command

Hello,
I'm not sure if this is exactly correct but I noticed the recipe not doing what I expected. The end result was http and https in my sources list where I just wanted https. I think the commit itself will explain the issue.

:+1: good catch... can we get this merged?

This has been implemented by using the name :global with the new gemrc resource. Please let us know if you have any issues.


Error when generating a new CMakeLists.txt

Repro steps:

Create an empty folder
Create an empty main.cpp in that folder
Open that folder in VS Code
Run "CMake: Configure"
A prompt is shown providing the option to Create or Locate CMakeLists.txt.  Click "Create".
Enter a name and press enter
Select "Executable"

Result: A "Conguration is already in progress" error is displayed.
Expected: No error

Also, the initial configuration does not complete successfully.  The build directory is not created until I Configure again.


recommend auth code flow using signed requests

What type of PR is this?
Add one of the following kinds:

documentation

What this PR does / why we need it:
This PR recommends using signed OIDC authoritzation code flow requests.
Which issue(s) this PR fixes:
Fixes #205

My apologies, I created #233, before seeing this PR. Possible to use #233 instead?

I would like to clarify why the OIDC specification was chosen over RFC 9101. It is important because, although they are very similar, they have small but significant differences.
OIDC:

When the request parameter is used, the OpenID Connect request parameter values contained in the JWT supersede those passed using the OAuth 2.0 request syntax. However, parameters MAY also be passed using the OAuth 2.0 request syntax even when a Request Object is used; this would typically be done to enable a cached, pre-signed (and possibly pre-encrypted) Request Object value to be used containing the fixed request parameters, while parameters that can vary with each request, such as state and nonce, are passed as OAuth 2.0 parameters.

RFC 9101:

The client MAY send the parameters included in the Request Object duplicated in the query parameters as well for backward compatibility, etc.  However, the authorization server supporting this specification MUST only use the parameters included in the Request Object.

From my point of view, it is more interesting to follow RFC 9101, as it would ignore parameters that the client did not have to send and could potentially change how an authserver processes the request.

I would like to propose to append the following point regarding the sub claim for signed Request Object, as there are cases of implementation, the sub is wrongly included.
8< ------------------------
The sub (Subject) claim in the signed Request Object MUST NOT be present, to prevent cross-JWT confusion.

aud value. Telefónica believes that the aud field should be the issuer of the authorization server as per standards (both OIDC and RFC 9101)

Referring to the aud value statement in ODIC:
"The aud value SHOULD be or include the OP's Issuer Identifier URL."
Breaking it down:
a) The aud value SHOULD be the OP's Issuer Identifier URL, or
b) The aud value SHOULD include the OP's Issuer Identifier URL.
Point b) is read as "OP's Issuer Identifier URL" is used as a prefix.

sub claim requirement to avoid cross-JWT confusion.

Referring to OIDC, Client Authentication, private_key_jwt, the following statement is present:
"The JWT MAY contain other Claims. Any Claims used that are not understood MUST be ignored."
If this signed request object (for /authorize) aud value is the OP's Issuer Identifier URL and the 'sub' happens to be populated with the client_id of the OAuth Client, this could be used as the client_assertion for the /token or bc-authorize endpoint.


new attempt at clean PSB2 with five base problems

Things to check in review:

[x] Using IntRule as type for Integer in the grammars.jl (because Integer and Int are both reserved in Julia)
[ ] The command_while loop implementation in psb2_primitives.jl
[x] Are all the required links to the HerbAI framework in place?
[ ] Is the README clear on how to add new tasks in the benchmark

TODO: testing individual grammar functions and the command_while
Includes problems:

Basement
Coinsums
Fizz Buzz
Fuelcost
GCD

Codecov Report
Attention: Patch coverage is 22.09945% with 141 lines in your changes missing coverage. Please review.

Please upload report for BASE (dev@b9afc1d). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

src/data/PSB2_2021/grammar.jl
0.00%
95 Missing :warning:

src/data/PSB2_2021/retrieve_all_tasks.jl
0.00%
13 Missing :warning:

src/data/PSB2_2021/PSB2_2021.jl
7.69%
12 Missing :warning:

src/data/PSB2_2021/psb2_primitives.jl
63.63%
8 Missing :warning:

src/data/PSB2_2021/program_examples.jl
78.12%
7 Missing :warning:

src/utils/benchmarks_io.jl
0.00%
4 Missing :warning:

src/data/PSB2_2021/base_grammars.jl
0.00%
1 Missing :warning:

src/data/PSB2_2021/problem_grammars.jl
0.00%
1 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##             dev      #60   +/-   ##
======================================
  Coverage       ?   41.96%
======================================
  Files          ?       27
  Lines          ?      772
  Branches       ?        0
======================================
  Hits           ?      324
  Misses         ?      448
  Partials       ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Upsampling no work in "https://kiri.art/" or docker api?

error: return "Body excee..." , but I fix it by "export const config = {
api: {
bodyParser: {
sizeLimit: '4mb'
}
}
};"in pages/api/banana-upsample.ts.  Then next error: no this model of "RealESRGAN_x4plus"

Hey @xuechengjinxiu, welcome.
Firstly, congrats on getting so far :sweat_smile:  I know that currently the docs aren't so clear and the code is a bit of a mess :/  Most of my focus is currently on the docker api and kiri.art site, I hope in a few months to spend more time on making the stable-diffusion-react code base more clear and open for general cases.
Secondly, nice catch on the body size limit... I guess I've been sending smaller pics for up-sampling until now :sweat_smile:
Are you just playing locally or are you deploying somewhere too?
Unfortunately the upsampling code requires a different docker image than docker-diffusers-api (which only wraps HuggingFace's "diffusers" library for diffusion-based models*).  *The diffusion-based upsampling may already be possible, I just haven't spent much time with it yet).
There's a https://github.com/kiri-art/banana-upsample repo which has a docker image with RealESRGAN and FPGAN for upsampling and face restoration, respectively.  It deploys to banana too.  It's working on kiri.art but it's very slow, because banana can't currently optimize images with multiple models, so cold starts are very slow.  And I haven't had any time to try workaround that :(
Anyway, hope that all makes sense!  Let me know if anything is not clear or if you have any other questions.  And thanks for your post :pray:


Add ASCII grid option for standalone (2)

Hello,
I noticed the Output format 1 is very similar, but not exactly the same, as an ASCII Raster. Some programs (e.g., QGIS) can read ASCII Rasters directly, but do not provide the options to deal with the header in case it is not what they expect. I made a small change in the code to add the option to output the data as ASCII Rasters (Output format 2 ) and thought that could be helpful for other users.

I did not compile the exe file with Visual Studio, you might want to double check in case you merge the PR.

Cool! I hope it is helpful. I later noticed that Output format 1  loop does not write the data on the same order as ASCII rasters , I opened #2 that should fix it.


Content and Blog admin page: Metadata should be displayed on multiple lines on small screens

On Content and Blog admin page, the metadata should be displayed on multiple lines on small screens.
Ex :
Published: No Draft 4 weeks ago
1 comment
Last modified: 5 weeks ago By admin

I don't really understand how the flexbox css works and if it is possible.


Not clear about the context of Context Menu in empty area of notebook

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Create a notebook
Add 3 cells
Right click on the empty area at the bottom of the notebook
Click on Cut, Copy, Paste

What do each of these do?
Its not clear that they cut/copy/paste cells, or whether it applies to outputs.
The only way to find out is to try using them.
On the other hand the context menu items for Cells are Cut Cell, Copy Cell, Paste Cell
Hence the confusion.

Suggestion = Hide this context menu.


Configure default camera and speaker

What feature are you requesting?
I would like to set the default camera and speaker when initialising a call composite.
What would be the benefit of adding this feature?
The AzureCommunicationCallingUI currently selects the front camera of the phone, as well as using the ear speaker instead of the loud speaker. Our users now have to switch these for every call they make.
What solution would be ideal for you?
Extend LocalOptions to include fields for camera direction (front | back) and speaker, similar to cameraOn, microphoneOn.
What alternatives have you considered?
Swiftching from ACS to a different service entirely, including our javascript frontend and server infrastructure.
Is there any additional information for your request?
Attach screenshots or images add detail to your request:
iPhone is selected as default instead of Speaker

Front camera is used by default, and our users have to switch every time

Thanks a lot for your valuable feedback! We added a feature into our backlog (with no ETA at the moment); the feature will enable user to select from more options from the current list of camera and audio options.
If you have any immediate plans to ship utilizing the ACS Native UI SDK to production but need a feature specifically for your scenario, please feel free to reach out to me directly to provide more information about your desired workflow. From gathering more information about your business needs, we can better prioritize your asks in our backlog grooming sessions.
Thanks again for sharing your thoughts with us!
PS. We will keep this feature open for any other customer looking for this scenario.

+1


while creating order getting error with undefined status and error

 var key_id = '##########';
  var key_secret = '##############';

  var instance = new Razorpay({
    key_id: key_id,
    key_secret: key_secret
  });

  instance.orders.create({
    amount: 1000,
       currency: "INR",
       receipt: '9876',
       payment_capture: true
  }, (error, response) => {

    if ((error)) {
      // handle error
      console.log("error - ", error);
      res.send("err");
    } else {
      // handle success
      console.log("response - ", response);

      res.send("ok");
    }
  });

always getting   error - { statusCode: undefined, error: undefined }
can any one help me ?

same problem is occuring with me. Can someone please reply to this problem.

Hi. We're unable to reproduce this issue. Can you give us some more details? What are the version numbers of the SDK and Node.js that you are using? Can you try deleting node_modules and reinstalling all modules?

i was using firebase cloud functions to generate order id , one have to enable billing in firebase project to request to outbound api's.
for every api you use in firebase gives you undefined error code and status if you haven't enabled billing.
Thier is no issue in Razorpay SDk

We've usually seen this issue happen due to Network issues. Closing this.

The issue is indeed valid, I'm also getting the same problem.
request { amount: 30000, currency: 'INR', payment_capture: true, receipt: '#r1', notes: { key1: 'somthing' } }
response { statusCode: undefined, error: undefined }
note - I tried reinstalling node modules.

The issue is indeed valid, I'm also getting the same problem.
request { amount: 30000, currency: 'INR', payment_capture: true, receipt: '#r1', notes: { key1: 'somthing' } }
response { statusCode: undefined, error: undefined }
note - I tried reinstalling node modules it doesn't worked.
Plus I tried connecting through API call, I got ECONNRESET error.

You have to enable billing for the razorpay. This is just due to the fact your billing is not done.

hm. bit weird, I just restarted my system and things started working back. seems something was blocking connection with Razorpay API.

amount: 1000*100,

connect to internet

ran into it  because of wrong or expired keys, try  regenerating and using


Upgrade to id-tagging-schema v6

Just tracking that I updated the preset code to add partial support for the id-tagging-schema v6 that was recently released.
Some things we don't support at this time:

Röntgen icons (there are only a few of these anyway and they are hard to generate)
Preset "Aliases" (these are like terms but different)
New types of fields yet (like colour , date, or directionalCombo) but mappers can still add this information in the raw tag editor like they used to be able to.
Icons in combo field values

I'll link back to this issue if/when we add more support for these things.
Work was done in these commits

0bef7fbea - Resolve preset and field references new to id-tagging-schema v6
53699bbac - Fetch strings for id-tagging-schema v6 as well
df41f3d4d - Upgrade to id-tagging-schema v6

The Röntgen dependency is a PITA and really ridiculous. There's no way I'm adding all those prerequisites to my project, so for now I'm just copying them from iD from https://raw.githubusercontent.com/openstreetmap/iD/develop/svg/roentgen/*
Hopefully they fix this upstream.

FYI I created https://github.com/enzet/map-machine/issues/149 to track the issues of having the icons more easily consumed. I think there was no issue, yet, and it sounds like that really should be easier.

Yes, I know I could copy the icons from the iD repo, but I really don't want the increased maintenance burden of looking there for new icons.  For now these are the replacements I hardcoded:
https://github.com/facebook/RapiD/blob/0bef7fbea4a120201135335d39b7196564c8889f/modules/presets/index.js#L25-L37

I just added the Röntgen icons in 589d43d78 .
The work I just completed on #925 gives us much more flexibility to support new spritesheets.  These get converted to WebGL textures on the fly now, so we don't need to prepare a separate spritesheet for Pixi anymore.


Add margin

Your boxes need bottom margin cause float left will put boxes where ever they fit

This is fixed


Updated "Sales Forecast analysis," issue# 88.

Updated Sales Forecast analysis under "Financials" of the franchise.

please reopen the issue as I requested changes on it


Should julialauncher just be called julia?

IIUC, the vision here is that users will primarily interact with Julia through Juliaup's launcher, so I think it would make sense to call that the primary julia command. I am not sure whether there are any special considerations for Windows here.

Yeah, I thought about this. One benefit of the current setup though is simply that in Task Manager I can tell which process is which very easily, which can be quite helpful. And at least on Windows the store installer automatically creates a symlink from julia to julialauncher, so from a user point of view this is handled pretty transparently.
Not sure whether that task manager reasoning applies to other platforms as well?

That's a good point. On Linux htop usually shows the the full directory of the running command, so it would still be distinguishanle, but I could see that possibly being confusing. Perhaps we should just create a symlinks on Linux and MacOS in the default installer as well?

Ok, I have now changed the AUR recipe so that it creates julia as a symlink to julialauncher.

Should we close this issue, and say we generally leave the name and try to create symlinks?

Yes, I think we can


Add that need to clean the UWP project before building

X-Ref https://github.com/MicrosoftDocs/WindowsCommunityToolkitDocs/issues/289

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

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


Expose bin/reload as vendor binary for composer

added "bin" to composer.json
added some more entries to .gitignore
added "require-dev" with phpunit to composer.json
moved autoloading of tests into "autoload-dev" in composer.json
extended autoloading in bin/reload

Coverage decreased (-0.1%) to 46.21% when pulling 097a6f716032431dae2de8da78323d0d0a1ecd16 on hendrikmaus:configure-vendor-bin-with-composer into bc4aeeb4942c5cbfbbe54292179f7a910ca79260 on RickySu:master.

You're welcome. You might want to consider updating the lib on packagist; it still has the 0.1.0 release


shaurika.is-cool.dev

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description
I'll be using this domain for some self hosting, game servers and building websites for personal or to be used by a small group
Link to Website

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA


README updated

Mobile-config updated
Cleaned a bit the index file

It would be nice if you could bump the version and publish a new version on npm :)

Thanks again. New version is now available on NPM.


Issues with 'sprite overlaps tile' block

I feel there are a couple of problems with the 'on sprite of kind Player overlaps tile at location' block:

Using a pause block inside the overlap block causes the game to hang. Everything stops, and the effects of any other code within the overlap block are not seen until the time on the pause has resolved.
If you overlap two of the same blocks at the same time, the overlap code triggers twice. Obviously that's technically correct, but my students tend to find it counter-intuitive: for example, if they write code to take damage on overlapping a lava tile, they lose 2 life instead of 1 when walking into two lava tiles. I think it would be better if the overlap block did not trigger if it was already running, although I may be missing use cases that would become problematic because of that.

My apologies if this has already been discussed, or if I'm missing something obvious in my understanding of this block!
Thank you.

Thanks for the feature request. We will look into this.


Accordion panel not working within Import users (security areas)

Link doesn't do anything

FYI: as of 44ccfa2 this works correctly, but it looks like a lot has changed since that commit :)

Confirmed fixed.
I've fixed a small unrelated issue in the same location at https://github.com/silverstripe/silverstripe-framework/pull/6240


Add a new option to fail on breaking changes diff

Let's add a CLI option to be able to make the process fail (return code != 0) in case the computed diff returns a breaking change.
Any ideas on the name of the option? --breaking or --ci, other?

I think we should use something explicit here, so I wouldn't argue in favor of --ci. Does --fail-on-breaking work for you?

Will call it --fail-on-breaking in the end (better be extra explicit and the limitation I mentioned was incorrect 😌). Thanks both for your comments


add H298A V9 sup

I need H298A V9 key or something i have this error when i try.. please add the support..
Signature: H298A V9
Failed! Trying again, with signature: H298AV9
Malformed decrypted payload, probably used the wrong key!

Waiting H298A V9 update!

+1

+1

Anyone has a fix yet for V9? I tried everything..

+1


Support for monotonically decreasing indices in interpolate_na

Currently interpolate_na requires all indices to be monotonically increasing. If thinking about geographical dataset, indices are generally always monotonic, however in the South hemisphere the latitude is monotonically decreasing.
The current workaround is to flip the image before and after the interpolation, however should not be a lot of effort to support all monotonic indices directly within interpolate_na.

A simple trick could be to not use the coordinate while interpolating. An example with the same data but the second dataset has a decreasing index.
import numpy as np
import xarray as xr

da = xr.DataArray([0, 2, np.nan, 3, 3.25], dims="x", coords={"x": [0, 1, 2, 3, 4]})
da1= xr.DataArray([3.25, 3,np.nan,2,0],dims='x',coords={"x":[4,3,2,1,0]})

da_inter=da.interpolate_na(dim='x', method='cubic')
da1_inter=da1.interpolate_na(dim='x',method='cubic',use_coordinate=False)

As far as I understand interpolation, the result is the same.

@TomNicholas where would I start from if I want to work on this?


Update faqs-app-service-linux.yml

Adding recommendations & clarifications around host disk space & the "no space left on device" exception
Reviewed w/ @btardif internally

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

@Kendubu1 if your content is ready for review and merge, type "#sign-off" in a new comment.

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

Updated hyperlink to remove /en-us/ segments

#sign-off

@btardif: I'm sorry - only the author of this article, @genlin, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@Kendubu1 @btardif thanks for your contribution. We will review your content and make the necessary updates.

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

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

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

@Kendubu1 I made minor changes. Let me know if there is any problem.

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

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

Looks good @AmandaAZ - I suppose 'data' can be both singular & plural 😆
I made a small edit for readability

@Kendubu1 The article is republished with the necessary changes. Thanks for your contribution.

This should help mitigate:
https://github.com/MicrosoftDocs/azure-docs/issues/83385


obfs4 混淆流量 有支持计计划吗

如题，混淆流量，Go版本不支持？

@penhauer-xiao I will try, if necessary

@arthurkiller I have tried google translate but it's not really working with this language(chines right?)
Can you translate the  issue?

@elico arthurkiller wants to use obfs4 from tor to disguise the ShadowSocks traffic as TLS traffic. This is not a great idea because obfs4 might get blocked too. The best solution is to use tools like Stunnel, to transport connection via REAL TLS. In my own case, I ditched ShadowSocks long ago. I think 3proxy + Stunnel is a better solution.

@testcaoy7 thanks.
I really can't read c\c++ that much compared to GoLang.
Is there something similar to 3proxy  in GoLang that you may know about?

@testcaoy7  You mean 3proxy + Stunnel compare ShadowSocks better.

@penhauer-xiao I find that TLS is a better solution to bypass "China" firewall beacause it is a widely used protocol thus cannot be banned easily. It also allow you to bypass QoS system. You can replace stunnel to any tunnel you like, such as kcptun. It also works well with 3proxy.
Obfs4 alreadly get blocked in Kazakhstan (which means it can be blocked and this is why I don't want to use it).

@testcaoy7   Ok， thanks

how about this?

looks cool, and wanna support
Best regards
✉✉✉✉✉✉✉✉✉✉✉✉✉✉
Arthur lee
Sent from my iPhone

On 8 Mar 2018, at 12:05 PM, 陈飞<EMAIL_ADDRESS>wrote:
how about this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Disable email account confirmation on Heroku

The Heroku build fails because ENABLE_ACCOUNT_CONFIRMATION_BY_EMAIL is true by default which in turn requires configuring a storefront domain. We cannot assume where the storefront is deployed so this has to be configured manually. This PR disables email account confirmation which I think is fine for test Heroku builds.

Here is the report for 46ebabba08057b88e96afc110f6d57bf41ac55ba (maarcingebala/saleor @ heroku-change-env)
Base comparison is b0810a541ad2f0a5ed3daa63bfdf575c97ecc331.
No differences were found. (click me)

# api.benchmark checkout
  test name                                  	left count 	right count	duplicate count
  -------------------------------------------	-----------	-----------	---------------
  add billing address to checkout            	         34	         34	             20
  add shipping to checkout                   	          7	          7	              0
  checkout payment charge                    	         10	         10	              0
  complete checkout                          	          8	          8	              0
  create checkout                            	          5	          5	              1

# api.benchmark homepage
  test name                                  	left count 	right count	duplicate count
  -------------------------------------------	-----------	-----------	---------------
  retrieve main menu                         	          5	          5	              0
  retrieve product list                      	          4	          4	              0
  retrieve secondary menu                    	          5	          5	              0
  retrieve shop                              	          2	          2	              0

# api.benchmark product
  test name                                  	left count 	right count	duplicate count
  -------------------------------------------	-----------	-----------	---------------
  product details                            	         18	         18	              4
  retrieve product attributes                	          9	          9	              0

# api.benchmark variant
  test name                                  	left count 	right count	duplicate count
  -------------------------------------------	-----------	-----------	---------------
  product variant bulk create                	         51	         51	              3
  retrieve variant list                      	         23	         23	              9

# api product sorting attributes
  test name                                  	left count 	right count	duplicate count
  -------------------------------------------	-----------	-----------	---------------
  sort product not having attribute data     	         21	         21	              0


Correct ada-lovelace conversion in name of unit test

[x] Update incorrect conversion of 2^32-1 lovelace to ada from ≈4.295 ada to ≈4295 ada in the name of a unit test

Comments

2^32/1e6 = 4 294,967296
I had been thinking about it wrong the whole time 😱
This means fee coin values on mainnet would basically always be encoded as 5 bytes (distributeSurplus should still be useful, even on mainnet, because of change-output values though)

Issue Number
None.

bors r+

bors r+


Redirect protocol.ai to ipn.io

https://github.com/protocol/main/issues/164

Currently protocol.ai returns a 404. Until we get the new website up, it should at least redirect to ipn.io so that people using our email addresses to find our website will have something to look at.

closing in favour of #55


As a server admin, I want to be able add, modify, and remove authors.

Done. Django's admin handles this.


RFC: Stable API Change: Rename hci_rpmsg to hci_ipc

Problem Description:
The HCI driver in drivers/bluetooth/hci/rpmsg and the HCI sample in
samples/bluetooth/hci_rpmsg are no longer appropriately named. Both now use
the IPC subsystem, which can in turn use (but does not have to) RPMsg under
the hood.
Proposed Change (detailed):
The proposal, as shown in Pull Request #64391, is to rename anything related
to HCI and RPMsg to use the term IPC instead.
Detailed RFC:

Function call changes
None that are public
Device Tree changes (source and bindings)
The following chose node has been changed:
zephyr,bt-hci-rpmsg-ipc has been renamed to zephyr,bt-hci-ipc
Kconfig option changes
CONFIG_BT_RPMSG has been renamed to CONFIG_BT_HCI_IPC
Sample names changes
samples/bluetooth/hci_rpmsg has been renamed to samples/bluetooth/hci_ipc

Dependencies:
Users will need to use the new Kconfig option and Devicetree chosen node in
their out of tree boards and applications.
Users will need to compile the sample with the new name and path.

Will these changes break messaging between cores? E.g. if we flash core A after this change has been merged, and we have an older firmware on core B?

Will these changes break messaging between cores? E.g. if we flash core A after this change has been merged, and we have an older firmware on core B?

No, because there are no runtime changes at all. Everything is compile-time.

Architecture WG:

No objections


Add star rail battle chronicle

Seems only CN has this for now.
See hewang1an/StarRail-plugin

Hey there, just a heads up that Global has this now
It seems quite similar to Genshin's battle chronicles
https://www.hoyolab.com/article/19152801/?lang=en-us&utm_source=sns&utm_medium=link
CC @EffortlessFury too, since you were the one that got the daily check-ins working


[ESP32]:  Update WiFi Scan Network Implementation for New API

Problem:
Starting from ESP-IDF v5.1.3, a new API was introduced that allows fetching individual AP records one at a time, eliminating the need for preallocating a list of AP records.
Change Overview:

ESP32 Platform Updates: Updated NetworkCommissioningDriver.cpp and NetworkCommissioningDriver.h to support the new API introduced in ESP-IDF v5.1.3 while maintaining backward compatibility with earlier versions.
Refactored ESPScanResponseIterator and OnScanWiFiNetworkDone() method to handle WiFi scan results differently based on the ESP-IDF versions.

Testing:
The changes were tested on an ESP32-S3 chip using ESP-IDF versions v5.1 and v5.2.2
Both versions successfully read WiFi scan result lists using the CHIP tool with the following command:
./chip-tool networkcommissioning scan-networks 1 0

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.esp seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


forwardRef helper

Reach-UI has incredible typings. We could steal + adapt the following typings for us to make creating components easier regarding typings:
https://github.com/reach/reach-ui/blob/4cb497f530b0f83f80c6f6f2da46ab55b1160cb6/packages/utils/src/index.tsx#L185-L209
/**
 * This is a hack for sure. The thing is, getting a component to intelligently
 * infer props based on a component or JSX string passed into an `as` prop is
 * kind of a huge pain. Getting it to work and satisfy the constraints of
 * `forwardRef` seems dang near impossible. To avoid needing to do this awkward
 * type song-and-dance every time we want to forward a ref into a component
 * that accepts an `as` prop, we abstract all of that mess to this function for
 * the time time being.
 *
 * TODO: Eventually we should probably just try to get the type defs above
 * working across the board, but ain't nobody got time for that mess!
 *
 * @param Comp
 */
export function forwardRefWithAs<Props, ComponentType extends As>(
  comp: (
    props: PropsFromAs<ComponentType, Props>,
    ref: React.RefObject<any>
  ) => React.ReactElement | null
) {
  return (React.forwardRef(comp as any) as unknown) as ComponentWithAs<
    ComponentType,
    Props
  >;
}

Closed via #106


[Feature Request] A prediction pipeline for combining upstream and downstream model

I want to use s3prl models to create a api for my services, if we can create a pipeline to automatic loading a ckpt for inference will be more convenient.
As an example, I have trained a ctc model './dev-clean-best.ckpt' with dict char.dict`
model = s3prlModel('./dev-clean-best.ckpt', './char.dict')
print(model('./sample1.flac'))

It will return the prediction result:
"GOING ALONG SLUSHY COUNTRI ROADS AND SPEAKING TO DAMP ORDIANC IS IN DROFTY'S CHOOL ROMS DAY OF TER DAY FOR FOULT NIGHT HE'LL HAVE TO PUT IN AN APPEARANCE OF SUMONG PLACE OF WUSH IT POND SOME DAY MONING AND HE CAN COME TO USSI MEDIATELY OFTOWAODS"

my code for this case:
import torch
import torchaudio

class s3prlModel:
    def __init__(self, cpkt, dict_path='dict.txt'):
        from s3prl.downstream.runner import Runner
        model_dict = torch.load(cpkt, map_location='cpu')
        self.args = model_dict['Args']
        self.config = model_dict['Config']
        # config patch
        self.args.init_ckpt = cpkt
        self.args.device = "cuda" if torch.cuda.is_available() else "cpu"
        self.config['downstream_expert']['datarc']["dict_path"] = dict_path
        runner = Runner(self.args, self.config)
        self.upstream = runner._get_upstream()
        self.featurizer = runner._get_featurizer()
        self.downstream = runner._get_downstream()

    def __call__(self, wav_path):
        wav, sr = torchaudio.load(wav_path)
        wav = wav.mean(0).unsqueeze(0)
        with torch.no_grad():
            features = self.upstream.model(wav)
            features = self.featurizer.model(wav, features)
            result = self.downstream.model.inference(features, filenames=[])[0]
            return result

Also It would be better if we can save char.dict into ckpt checkpoint

Hi!
Sounds good. This is naturally supported in the version > 0.4.
I am not very sure for the old version (downstream folder), what change do you propose to make?

i think i missed that in the document, I have no problem then ~


Run keccak benchmark

Demonstration of Keccak benchmark for the following Rust code: https://gist.github.com/akashin/a02b4b72883674072e09b226904f2b5e

Closing as I don't intend to submit this for now. We would eventually need to revive this to complete https://github.com/near/wasmtime/issues/173


🚨 URGENT ISSUE: Login and Logout Not Working, Both Buttons Shown Simultaneously 🚨

Description
The login and logout functionality on our website is currently experiencing a critical issue. Users are unable to log in or log out successfully, and both the "Login" and "Logout" buttons are being displayed simultaneously, causing confusion and hindering user interactions.
Expected Behavior:

When a user is not logged in, only the "Login" button should be displayed, prompting users to log in.
After successful login, the "Logout" button should be shown, allowing users to log out.
Current Behavior:

Both "Login" and "Logout" buttons are being shown together, regardless of the login status of the user.
Clicking on either button does not perform the expected login or logout action.

Steps to Reproduce:

Visit the website's home page or any restricted page.
Observe the presence of both "Login" and "Logout" buttons simultaneously, regardless of login status.
Attempt to log in or log out using the respective buttons.
Notice that the expected login or logout action is not performed.

Possible Solution:
The root cause of this issue needs to be identified and addressed. The login and logout mechanisms should be reviewed to ensure they function correctly and display the appropriate buttons based on the user's authentication status.
Screenshots

Additional information
This is an emergency issue
What browser are you seeing the problem on?
Microsoft Edge
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)
[X] I am a GSSoC'23 contributor

@AnirudhPanda ,please assign this issue to me ,its an high emergency issue

@AnirudhPanda ,Please review and merge my pull request


Remove SNIP-20 specific code for preventing query/message execution

When executing a contract, there is some SNIP-20 specific code that handles and warns the user if they are not providing either an address or key for a specific query as follows:
https://github.com/stakeordie/griptape.js/blob/a0d0f94052d8e2dae3b3e896f0265bff5030a929/src/contracts/index.ts#L104-L114
We should remove this and maybe provide and API to handle specific scenarios in which we want to prevent the execution of a query.

@luis-stakeordie .


WHL: Update to PROJ 9.3

[x] Closes #1327
[x] Fully documented, including history.rst for all changes and api/*.rst for new API

TODO: vcpkg & PROJ release

I get a test failure using PROJ 9.3.0 with pyproj 3.6.0:
=================================== FAILURES ===================================
____________________________ test_repr__conditional ____________________________

    @pytest.mark.grid
    def test_repr__conditional():
        trans_repr = repr(Transformer.from_crs("EPSG:4326+3855", "EPSG:4979"))
        if grids_available("us_nga_egm08_25.tif"):
            assert trans_repr == (
                "<Unknown Transformer: unknown>\n"
                "Description: unavailable until proj_trans is called\n"
                "Area of Use:\n- undefined"
            )
        elif PROJ_GTE_92:
>           assert trans_repr == (
                "<Unknown Transformer: noop>\n"
                "Description: Transformation from EGM2008 height to WGS 84 "
                "(ballpark vertical transformation, without ellipsoid height "
                "to vertical height correction)\n"
                "Area of Use:\n- undefined"
            )
E           AssertionError: assert '<Other Coord... 180.0, 90.0)' == '<Unknown Tra...\n- undefined'
E             - <Unknown Transformer: noop>
E             + <Other Coordinate Operation Transformer: noop>
E               Description: Transformation from EGM2008 height to WGS 84 (ballpark vertical transformation, without ellipsoid height to vertical height correction)
E               Area of Use:
E             - - undefined
E             + - name: World.
E             + - bounds: (-180.0, -90.0, 180.0, 90.0)

test/test_transformer.py:544: AssertionError

See #1315


because of race condition of displaying transactions when user is log…

…ged in before market data is pulled, add a conditional when getting description
fixes bthaile/augur#504
This issue also occurs on local chain.

Coverage decreased (-0.01%) to 55.49% when pulling 16f65d9c6daa50e76fc1feba02b20ec75918c540 on fix-display-message-race into 05ffbe76fdbaba9e695370dd54ef5ef1271b90fa on master.


fixed dotenv

added dotenv module
added dotenv plugin in webpack config
added TEST env variable
console.log TEST successfully in src/index.js

The .env file used for testing:
TEST=helloworld


Bump keyseq to 0.4.0 and add action::trigger* convenience systems, CHANGE NOTATION, and prep for bevy-input-sequence 0.6.0 release.

Hey Elm,
I added some convenience methods to trigger events instead of sending them. But that's not my whopper.
My whopper is that I changed keyseq and published 0.4.0. After a lot of deliberation, I decided to switch to capitalized modifier keys, so Ctrl-A instead of ctrl-A. I also added a feature flag to permit the even more standard Ctrl+A. This change breaks any keyseq users, which right now is only this crate, so I say break 'em while they're small.
My hope is to get bevy-input-sequence out the door and then push my first release of bevy_minibuffer for bevy 0.14 before bevy 0.15 gets published, which may be as soon as this weekend.
If you don't viscerally object to the notation change, let me know how I can help. If you're ok with this PR and permit it, I can publish to crates.io since we're both owners. That's purely an offer to help and expedite if you lack time or access.
Be well!
-Shane

I definitely don't like the parentheses (). Cargo's formatter takes weird liberties with what's inside, which is why I had switched to curly braces {}. Apparently one of the few differences between those macro notations are that curly braces permit statements. While () and [] only permit expressions. key*! are all expressions, so that's another reason to prefer [].
Do you have a link to any docs about the macro [] auto completion? I can put the comment for the code in as you have it, but it doesn't make sense to me. Would the names have to line up? What editor are you using for macro completion?
/// ```
/// bevy_pkeyseq![Ctrl-W Ctrl-D Ctrl-S Ctrl-A]
/// ```
pub fn bevy_pkeyseq(input: proc_macro::TokenStream) -> proc_macro::TokenStream {
}

Bummer. Cargo format does the same to [] as it does to (). Look at this:
#[rustfmt::skip]
#[test]
fn before_cargo_format() {
    assert_eq!(
        [key![Ctrl-A],
         key! [Ctrl-A],
         key! [ Ctrl-A ],
         key!{Ctrl-A},
         key! {Ctrl-A},
         key! { Ctrl-A },
         key!(Ctrl-A),
         key! (Ctrl-A),
         key! ( Ctrl-A ),
        ],
        [
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
        ]
    );
}

#[test]
fn after_cargo_format() {
    assert_eq!(
        [
            key![Ctrl - A],
            key![Ctrl - A],
            key![Ctrl - A],
            key! {Ctrl-A},
            key! {Ctrl-A},
            key! { Ctrl-A },
            key!(Ctrl - A),
            key!(Ctrl - A),
            key!(Ctrl - A),
        ],
        [
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
            (Modifiers::empty(), KeyCode::KeyA),
        ]
    );
}

I really don't like how it adds the space between the dash. I want it to look like how it would look in the manual. How do you feel about curly braces?

@shanecelis
I use Rust Rover and Vscode and both work.
I found the following page that mentions this behavior.
https://users.rust-lang.org/t/default-delimiters-for-macro-invocation-vs-code-rust-analyzer/96814/3

I checked it on my end, and indeed, the space is being added.
I feel this problem is surprisingly easy to face,  so there might be a workaround, but I think curly braces would be fine.


chore(shared): html tags

add html tag: portal
remove deprecated tag : hgroup、rtc
detail see : Obsolete and deprecated elements

AFAIK <portal> isn't formally part of the spec yet.

AFAIK <portal> isn't formally part of the spec yet.
fix!


Uncaught Error: Cannot call function 'getPosition' on missing remote object 220Error: Cannot cal...

Note: I am not fully aware what triggered this issue. But this is not the first time I am having this.
[Enter steps to reproduce:]

...
...

Atom: 1.40.1 x64
Electron: 3.1.10
OS: Kali GNU/Linux
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot call function 'getPosition' on missing remote object 220
Error: Cannot call function 'getPosition' on missing remote object 220
at throwRPCError (/usr/share/atom/resources/electron.asar/browser/rpc-server.js:145:17)
at EventEmitter. (/usr/share/atom/resources/electron.asar/browser/rpc-server.js:367:7)
at EventEmitter.emit (events.js:182:13)
at WebContents. (/usr/share/atom/resources/electron.asar/browser/api/web-contents.js:294:13)
at WebContents.emit (events.js:182:13)
At /usr/share/atom/resources/electron.asar/renderer/api/remote.js:219

Error: Cannot call function 'getPosition' on missing remote object 220
Error: Cannot call function 'getPosition' on missing remote object 220
    at throwRPCError (/usr/share/atom/resources/electron.asar/browser/rpc-server.js:145:17)
    at EventEmitter.<anonymous> (/usr/share/atom/resources/electron.asar/browser/rpc-server.js:367:7)
    at EventEmitter.emit (events.js:182:13)
    at WebContents.<anonymous> (/usr/share/atom/resources/electron.asar/browser/api/web-contents.js:294:13)
    at WebContents.emit (events.js:182:13)

    at metaToException (/usr/share/atom/resources/electron.asar/renderer/api/remote.js:271:17)
    at Object.exception (/usr/share/atom/resources/electron.asar/renderer/api/remote.js:219:30)
    at metaToValue (/usr/share/atom/resources/electron.asar/renderer/api/remote.js:223:28)
    at Object.remoteMemberFunction (/usr/share/atom/resources/electron.asar/renderer/api/remote.js:123:16)
    at AtomEnvironment.getWindowDimensions (/usr/share/atom/resources/app/static/<embedded>:1:705395)
    at AtomEnvironment.storeWindowDimensions (/usr/share/atom/resources/app/static/<embedded>:1:705810)
    at WindowEventHandler.handleWindowBlur (/usr/share/atom/resources/app/static/<embedded>:11:285354)

Commands
  3x -3:45 core:move-right (input.hidden-input)
     -3:43.9.0 editor:newline (input.hidden-input)
  5x -3:41.9.0 core:backspace (input.hidden-input)
     -3:36.6.0 core:save (input.hidden-input)
  5x -2:52.4.0 core:backspace (input.hidden-input)
     -2:48.8.0 core:move-down (input.hidden-input)
  2x -1:15.9.0 core:undo (input.hidden-input)
  2x -1:13.5.0 core:redo (input.hidden-input)
     -1:12 core:save (input.hidden-input)
     -1:02.2.0 core:undo (input.hidden-input)
     -0:55.3.0 core:backspace (input.hidden-input)
  5x -0:54.8.0 core:move-right (input.hidden-input)
     -0:53.2.0 core:backspace (input.hidden-input)
     -0:52.8.0 core:save (input.hidden-input)
     -0:48.8.0 core:backspace (input.hidden-input)
     -0:48.3.0 core:save (input.hidden-input)

Non-Core Packages

Thanks for contributing!
All new issues now require steps to reproduce (also called "repro steps") the problem or the behavior you wish to see in Atom. We do this so we can be certain that we are understanding what it is you're describing and don't prematurely close the issue or implement the wrong feature. If you need help writing detailed repro steps, please see the How Can I Contribute? section of the Atom CONTRTIBUTING guide.

Thank you @Kpraful for opening this issue. This was issue present in electron versions 4 and below. The latest atom version v1.51.0 uses electron version 5.0.3. The issue was fixed on this commit https://github.com/electron/electron/commit/a8f2646ba682d65f94ea628ac11b1a5521bcedcd. The fix has been available from electron version 5.0.0. I would advice you to upgrade to the latest atom version. I am closing this for now but happy to reopen if you still experience the issue.


Simplify and improve performance of F1 calculation

This is a cherry-pick rebase of https://github.com/Chia-Network/chiapos/pull/70
It also changes ProofOfSpace.exe to use create instead of generate to match python

My k=32 test plot completed successfully.
F1 complete, time: 233.849 seconds. CPU (76.54%)
Approximate working space used (without final file): 286.580 GiB
Final File size: 101.326 GiB
Total time = 82625.894 seconds. CPU (50.220%)
Older version for comparison:
F1 complete, time: 469.209 seconds. CPU (99.99%)

@wjblanke I think my optimization should be in a separate pull request from your implementation, which adds multithreading, since those are distinct features and they can work independently. If it's okay with you, I'll create a separate PR with only my optimization on top of master, with fixes based on Arvid's feedback.

Closing in favor of already merged https://github.com/Chia-Network/chiapos/pull/96


Default Class options are changing

I create a rectangle
While creating, I call initHook and creating a new instance of a custom class (L.Class.extend)
I change the options of the class on the rectangle
The options of the prototype of the class are changed

Expected behavior
The options of the extended L.Class should never changed from outside
Environment

Leaflet version: 1.7.1

Test
L.PM = L.Class.extend({
   options: {
  	test: 1,
        something: true
  }
})

function initRectangle() {
  this.pm = new L.PM(this);
}
L.Rectangle.addInitHook(initRectangle);

console.log(L.PM.prototype.options)

var rect = L.rectangle(map.getBounds()).addTo(map);
rect.pm.options.x = 'option changed'

console.log(L.PM.prototype.options)

https://jsfiddle.net/falkedesign/gwj6dphf/
When I add following the connection / reference between the options breaks:
var rect = L.rectangle(map.getBounds()).addTo(map);
L.setOptions(rect.pm,{a: 'xxx'})
rect.pm.options.x = 'option changed'

Perhaps it is fixed by #6766

@johnd0e your PR doesn't fix it. I create one by my self

That's good.
I can confirm the issue, but your description is unclear, so here is my try: https://github.com/Leaflet/Leaflet/pull/7459#pullrequestreview-586639979


integration with sendgrid?

first of all thanks for putting up this amazing set of tools together..
i was trying to integrate sendgrid rather than mailgun but couldn't make it work.
Is it possible to use email client for sending emails via sendgrid smtp or even web api?

You may try one of these methods:

Send via SendGrid HTTP API: you may have a look in my custom Email Client library (/application/libraries/Email_client.php) and add the "sendgrid" platform as you wish; you will need to add the SendGrid PHP library by composer as well.

Send via SendGrid SMTP: you may use traditional email setup (as seen from CodeIgniter official doc here) and update the values from /application/config/email.php.


Allow to specify a VIP to access the kube-apiserver

Add new variable $lb_api_server_ip to specify a loadbalancer IP to access the kube-apiserver

I don't really share whatever use case is driving this? A really compelling case is needed to add new configuration options, but please re-open if I'm missing something.

This is use to access HA k8s cluster apiserver from a LoadBalancer IP.
Need to have the IP of LoadBalancer declare into the apiserver certificate.

If you've setup a load balancing approach and know its IP, resolve that IP via a DNS record (e.g. cluster-foo.some-domain.com), even if you never plan to change that IP. You'll avoid creating TLS certificates that are hardcoded to a specific IP address (good practice) and if you ever do need to change the address, you won't have to replace cluster certificates.


Threads not shown

I've joined some continuous threads, but these threads are not shown in discordo.
Discord Official FAQ about Threads:
https://support.discord.com/hc/ja/articles/4403205878423-Threads-FAQ

According to the Discord Developer Document, Threads have been designed to be very similar to channel objects, but do NOT seem to be the same as channels.
Therefore, retrieving and chatting in the thread may need additional development.
https://discord.com/developers/docs/topics/threads

I have an opposite problem
There is a text channel with 2 threads I am in. I can read/write to threads, but I cannot view content of text channel itself.

i like the idea of dropping one more electron app from my workflow, but we heavily rely in threads - so i cannot use discordo yet :/
what's missing / how would i get started if i wanted to hack threads into discordo?

Guild thread events are not sent to the client by default. When a guild is selected in the guilds tree, you must send the guild subscription event (opcode: 14) with the fields guild_id as the selected guild ID and threads as true to receive guild thread events (you can see the structure of the guild subscribe event here). Currently, the official client sends the guild subscribe event every time a guild is selected:

Join our Discord server for further assistance.