Add logging of state output

Codecov Report

Merging #56 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   98.29%   98.28%   -0.02%
==========================================
  Files           6        6
  Lines         352      349       -3
  Branches       47       47
==========================================
- Hits          346      343       -3
  Misses          2        2
  Partials        4        4

Impacted Files
Coverage Δ

src/awsstepfuncs/abstract_state.py
98.83% <ø> (-0.04%)
:arrow_down:

src/awsstepfuncs/state_machine.py
94.36% <100.00%> (+0.24%)
:arrow_up:

Continue to review full report at Codecov.

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


[5.3] Old Session Isn't Flushed with Session::regenerate

Laravel Version: 5.3.28
PHP Version: 7.1
Database Driver & Version:

Description:
I am using file based Laravel session.
When running below code:
Session::forget('member');
Session::flush();
Session::regenerate();

The old session wasn't flush at all, all the data including the member info is still there. But the newly generated session start fresh.
How can I really flush the old session before moving to new session?

It also looks like the regenerate method accepts an optional boolean to destroy the session. So possibly, Session::regenerate(true); would help. If that is the case, then flush would probably be unnecessary.

@sisve Thanks for pointing out. Session::save() does the trick.
@devcircus Session::regenerate(true) does clear the old session file, but doesn't clear the previous session data if you didn't flush. Thus, below is my new code:
Session::flush();
Session::regenerate(true);

Thank you for your help. Have a nice day.


Fix missing dependencies for #1073

@mysticaltech @Silvest89 This basically fixes all the issues I experienced in #1073 .

[x] The problem mentioned in the last comment regarding k3s.service does not occur anymore
[x] I am able to destroy and deploy the same cluster config again and again

I 100% sure, that all other problems I had in #1073 were basically only happening because of pure randomness. Due these changes updating k3s configs only happens when first node is provisioned with base config and has running k3s.service. Also made sure that kustomization regarding all the other stuff CNI/CSI... is happening after all control nodes are deployed and have running k3s.

Thanks @aleksasiriski for testing this, really appreciate it!  🙏


Consolidate the GetDiagnosticInfo DM handling

do not ignore default values for the various metrics
cosmetic changes
alignment in naming for endpoint in the reply payload

//  Copyright (c) Microsoft Corporation.  All rights reserved.

I don't see any changes in this file

Refers to: common/src/Microsoft.Azure.IIoT.Agent.Framework/src/Models/AuthMode.cs:2 in 9d484e2. [](commit_id = 9d484e20c54ffd078abfc6342c0b4b860337a31b, deletion_comment = False)

//  Copyright (c) Microsoft Corporation.  All rights reserved.

just some trailing spaces after  and the explanation

In reply to:<PHONE_NUMBER>

Refers to: common/src/Microsoft.Azure.IIoT.Agent.Framework/src/Models/AuthMode.cs:2 in 9d484e2. [](commit_id = 9d484e20c54ffd078abfc6342c0b4b860337a31b, deletion_comment = False)


Add dataset cohort table to new ModelOverview experience

Description
Adding a table/heatmap for viewing statistics on the user-defined dataset cohorts. Note that this isn't exposed to users yet due to the feature flag added in #1306.
The coloring is custom per column/metric. Metrics are configurable. Cohort names are wrapped after 40 characters with a max of 2 lines to avoid overlapping.
Tests will be added separately since the feature isn't exposed yet and to keep the PRs small.
Expect several follow-up PRs to this one in the next days. The TODO for multiclass is intentional.
Areas changed

npm packages changed:

[ ] responsibleai/causality
[ ] responsibleai/core-ui
[ ] responsibleai/counterfactuals
[ ] responsibleai/dataset-explorer
[ ] responsibleai/fairness
[ ] responsibleai/interpret
[x] responsibleai/localization
[ ] responsibleai/mlchartlib
[x] responsibleai/model-assessment

Python packages changed:

[ ] erroranalysis
[ ] raiutils
[ ] raiwidgets
[ ] rai_core_flask
[ ] responsibleai

Tests

Tests to follow in a separate PR as mentioned above.

[ ] No new tests required.
[ ] New tests for the added feature are part of this PR.
[x] I validated the changes manually.

Screenshots (if appropriate):

Documentation:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Doc changes will happen once the feature flag is removed.

Codecov Report

Merging #1314 (7ef77b4) into main (0a002d7) will decrease coverage by 10.72%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1314       +/-   ##
===========================================
- Coverage   86.30%   75.58%   -10.73%
===========================================
  Files          99       10       -89
  Lines        4615      258     -4357
===========================================
- Hits         3983      195     -3788
+ Misses        632       63      -569

Flag
Coverage Δ

unittests
75.58% <ø> (-10.73%)
:arrow_down:

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

Impacted Files
Coverage Δ

raiwidgets/raiwidgets/cohort.py

...ponsibleai/responsibleai/modelanalysis/__init__.py

raiutils/raiutils/webservice/__init__.py

raiwidgets/raiwidgets/utils.py

.../responsibleai/_managers/counterfactual_manager.py

erroranalysis/erroranalysis/_internal/utils.py

erroranalysis/erroranalysis/report/__init__.py

...tools/error_analysis/dashboard_schemas/__init__.py

raiutils/raiutils/common/retries.py

...iwidgets/raiwidgets/model_performance_dashboard.py

... and 79 more

Continue to review full report at Codecov.

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


InChIKey property

This PR adds InChIKey property for structure entries. InChIKey is a chemical structure descriptor alternative to SMILES (proposed in #368). InChIKey is said to avoid the the ambiguity the SMILES possesses, moreover, it does not have any internal structure (essentially being a "chemical checksum"), thus there should be no issues with its comparisons.
Pinging people who have expressed their interest for comments: @eimrek @utf @Austin243

Workshop: after the above workshop comment has been considered/handled this should be merged.
We have discussed if "complicated derived non-unique descriptors" really belongs inside the structures entry. However, before we have a decision/design on where they should go, we should accept fields that are useful and desired into the structures endpoint for now.
A similar regard holds for the desire to allow application area/subconsortia prefixes. Until we have a model for that, we merge fields without them.

@merkys @ml-evs What do you think - should we still add this to "core OPTIMADE" as decided in the web meet. Or, now that we are hopefully somewhat close to merging #473, should all of these 'chemically oriented fields" currently waiting in PRs by @merkys instead go in its own namespace?
These PRs are affected: #466 #465 #436 #398 - and the question is if they should be labeled 'status/blocked' (blocking on #473) or 'status/waiting-for-update' (since they all currently are sitting with comments to address before merging)

@merkys @ml-evs What do you think - should we still add this to "core OPTIMADE" as decided in the web meet. Or, now that we are hopefully somewhat close to merging #473, should all of these 'chemically oriented fields" currently waiting in PRs by @merkys instead go in its own namespace?
These PRs are affected: #466 #465 #436 #398 - and the question is if they should be labeled 'status/blocked' (blocking on #473) or 'status/waiting-for-update' (since they all currently are sitting with comments to address before merging)
Edit: possibly also #400, #396, or would those go in a "bio" prefix?

On purely technical/scientific terms I think this field would be perfect to seed a cheminformatics namespace (along with SMARTS/SMILES etc -- would have to figure out how to allow filter_smarts as a custom URL param too...). However on a purely development practice, I worry that we don't have the level of engagement or scale to spread ourselves so thinly across these various namespaces, in which case just loading up the core OPTIMADE namespace is maybe preferable. Happy to discuss!

@ml-evs maybe we can use this example to try out the infrastructure and see where we hit snags? I'm not sure we absolutely need engagement at this point, we already have 4-6 PR:s for properties to place in such a namespace-provider standard, which we can do under a "v0.1" to mark that things are highly experimental.
I've created a couple of new repos for this:

https://github.com/Materials-Consortia/definition-provider-template : GitHub template repo to create new definition-provider repos
https://github.com/Materials-Consortia/namespace-cheminformatics : live repo for the cheminformatics prefix where we can try to integrate @merkys cheminformatics definitions and see how far we get.

Great! Thanks for this @rartino -- I definitely stalled in my attempts to do the same thing. I will try to migrate https://github.com/Materials-Consortia/optimade-stability-namespace in the same direction.

Now about the remaining cheminformatics PRs. #436 introduces a new SMILES OPTIMADE data type and #398 introduces a new URI query parameter. I wonder whether property definition format and namespaces are ready to accept such extensions? If not, is this something that should be allowed to be extended in namespaces?

Once the current property defs etc. are merged, lets work on a similar design for user-defined datatypes. I'm thinking a similar declarative format as for units, properties for datatypes, where one with human language declare how every operator should work. However, I don't want to draft this until the property framework is merged.
The need for user-defined filer languages should perhaps inform the design of #398. Can we instead allow some syntax for the usual filter= to provide a list of filters with some kind of specifier what kind of filter it is? Then we can allow user-defined filter languages without new query parameters.

Once the current property defs etc. are merged, lets work on a similar design for user-defined datatypes. I'm thinking a similar declarative format as for units, properties for datatypes, where one with human language declare how every operator should work. However, I don't want to draft this until the property framework is merged.

Makes sense.

The need for user-defined filer languages should perhaps inform the design of #398. Can we instead allow some syntax for the usual filter= to provide a list of filters with some kind of specifier what kind of filter it is? Then we can allow user-defined filter languages without new query parameters.

For now queries in filters act on property values only. Query by SMARTS will not be bound to a specific property. A possible solution would be to allow queries on entries themselves by introducing property-less operators, viz. /structures?filter=SMARTS "[CX4]".

I suppose that by command line parameters you mean URL query parameters. Indeed, custom query parameters are already allowed. Thus #398 already can go to cheminformatics namespace.

It has been decided to move cheminformatics properties to a repository of its own, which has been done in https://github.com/Materials-Consortia/namespace-cheminformatics/pull/1 and https://github.com/Materials-Consortia/namespace-cheminformatics/pull/2. Closing this PR.


Lookup parent coin id

This adds a function to lookup the parent coin id of the xch spend. It gets the address data from an endpoint at api.aggsig.me/address, and if there are coins at that address it pulls the most recent one (by block_height_confirmed) which matches xch_amount_mojos.
I've tested it manually, on the xch -> btc workflow, but haven't done a txn with both sides.
I looked at scraping from explorers, but they all seem to make an ajax call to fetch the coins which makes the scraping approach more complex. It seems like a good use case for an api endpoint, so I created it on a digitalocean server. It should work fine for chiaswap, but is slow when pulling addresses with lots of coins.

It sounds like you're writing a web service that does this look-up?
The fact that explorers use an XMLHttpRequest kind of thing is actually good. It makes it easier to reverse-engineer the API.

I dug around a little bit and got these two useful-looking URLs:
https://api2.spacescan.io/1/xch/address/txns/xch18uh2ngsc7dcj5p35xervqrf4nmdqlwk6dtudk62e9aju4ncydhqqmcre0h?page=1&count=10
https://api2.spacescan.io/1/xch/coin/0xb80df1ff70cccfaf72eb449815a81834dc6b3e2516be0f511eb4df9d04c589a9
The first one can be used to find expected payments. The second can find the parent coin.

Hey, thanks for the feedback. I just added a commit to switch to using spacescan.io. It doesn't have an option to order the txns so we have to fetch all of them every time since the default sorting is oldest first. But like you said, since they're disposable addresses we wouldn't expect a lot of txns. Also wondering if I should add in some validation, eg handling badly formatted addresses?

Since the use case is to check known valid addresses, I'd say adding address validation in your method would be redundant. We would also reasonably expect the web services API to be able to deal with invalid addresses.

Also, note that the address api actually returns the coin parent id, so for the purposes of xchswap, we don't even need the coin api!


fix: avoid double-clicking and disable the save/create button during lock time

Issue
https://github.com/gravitee-io/issues/issues/XXXXX
Description
To keep the same logic to avoid double-clicking, but to allow the action to be restarted. Disable button during lock period
The choice of a 2s lock was made to avoid having to implement a lock mechanism on each use of the button. and to allow a new try if the 1st is in error for example. This is classic button save behavior.
Additional context

🧪  Gravitee.io Automatic Prerelease @gravitee/ui-particles-angular
npm install<EMAIL_ADDRESS>
yarn add<EMAIL_ADDRESS>

🧪  Gravitee.io Automatic Prerelease @gravitee/ui-policy-studio-angular
npm install<EMAIL_ADDRESS>
yarn add<EMAIL_ADDRESS>

📚  View the storybook of this branch here

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

npm package (@release-7.x dist-tag)
npm package (@release-7.x dist-tag)
npm package (@release-7.x dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The Power Fxs written over this document have typos ..

https://docs.microsoft.com/en-us/power-apps/teams/localizing-sample-apps
The Power Fxs written over here have typos, please fix the document accordingly.
Issue no.1
https://docs.microsoft.com/en-us/power-apps/teams/localizing-sample-apps#adding-a-new-language-to-your-app

“ru-RU” should be "ru-RU".
and might be here to remove. because there is no mention of 'staticLocalizationsNewMessages'.

Collect(colLocalization,Filter(staticLocalizationsNewMessages,LanguageTag = gblUserLanguage));

//user language
Set(gblUserLanguage,Switch(Left(Language(),2),"de","de-DE","en","en-US","es","es-ES","fr","fr-FR","it","it-IT","ja","ja-JP","nl","nl-NL",
"pt","pt-BR",“ru-RU”,
//default
"en-US"));
//build localization collection, with user's language
ClearCollect(colLocalization,Filter(staticLocalizations,LanguageTag = gblUserLanguage));
Collect(colLocalization,Filter(staticLocalizations,LanguageTag = gblUserLanguage));

Issue no.2
https://docs.microsoft.com/en-us/power-apps/teams/localizing-sample-apps#adding-and-localizing-a-new-label
All Semicolons should be modified as a comma.
With({varDefault: "Welcome to the app";varOOBTextId: "lblLoadingText_Welcome_locText"};With({varLocalizedText: LookUp(colLocalization;OOBTextID = varOOBTextId;LocalizedText)};Coalesce(varLocalizedText;varDefault)))

After modifying, that should be the following.
With(
    {
        varDefault: "Welcome to the app",
        varOOBTextId: "lblLoadingText_Welcome_locText"
    },
    With(
        {
            varLocalizedText: LookUp(
                colLocalization,
                OOBTextID = varOOBTextId,
                LocalizedText
            )
        },
        Coalesce(
            varLocalizedText,
            varDefault
        )
    )
)

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

ID: c40e1d02-821c-372a-6470-4850b5d04417
Version Independent ID: b7bd4543-5eba-c2b8-944e-be285f4455dc
Content: Localize sample apps (contains video) - Power Apps
Content Source: powerapps-docs/teams/localizing-sample-apps.md
Service: powerapps
Sub-service: teams
GitHub Login: @joel-lindstrom
Microsoft Alias: saperlmu

Thanks for your comment, @dora-taco. This issue has been assigned to the doc author for review.

Thank you for reaching out and your feedback. Documentation has been fixed. The changes will be reflected soon.


feat: add kyoshidajp/dep-doctor

kyoshidajp/dep-doctor: Diagnose whether your software dependency libraries are maintained
I run the below commands:
cmdx s kyoshidajp/dep-doctor
cmdx t kyoshidajp/dep-doctor
cmdx gr
cmdx new kyoshidajp/dep-doctor

Please review 🙏

Thank you for your contribution!

v4.157.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.157.0


InitialGoldenImage

Signed-off-by: matthew-frahry<EMAIL_ADDRESS>

I think this is good to go!

LGTM


Can u provide dependencies for java please?

[Enter feedback here]

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

ID: 35018f9d-bebe-4149-e26f-cb5086a1125d
Version Independent ID: 7dd66eac-f81d-12a5-dca0-b12b3280b772
Content: Azure App Configuration REST API - HMAC authentication
Content Source: articles/azure-app-configuration/rest-api-authentication-hmac.md
Service: azure-app-configuration
GitHub Login: @lisaguthrie
Microsoft Alias: lcozzens

@gitveysel - Thanks for posting the query. We are looking into it and will get back to you shortly.

#reassign: maud-lv

Hello and apologies for not updating this issue. I am closing it for the time being. If needed, please open a new issue to continue this conversation. #please-close


I am getting an error that prevents Desmos from Evaluating equations

DesModder Returns these Errors:
DesModder Loading Errors Manager
Looks like some Desmos change happened to break some of DesModder's patches.
The patches are for the following purposes. Some may be critical, some may just be styling:
Compile GLesmos code instead of running implicit plotter
Override the evaluation view
Distinguish 0 vs -0 and NaN vs -∞ vs +∞

This impacts Desmos critically. I can't evaluate anything.

This looks like the issue from 0.10.5

I am currently running on these specs: chromeOS Version 111.0.5563.118 (Official Build) (64-bit)
The version of the extension is v0.10.6.

Desmos started working again but I am still getting an error:
Compile GLesmos code instead of running implicit plotter

It's not impacting Desmos, however.


Swagger for Form Recognizer 2021-09-30-preview.

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[X] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
9/30/2021
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
10/5/2021
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[X] SDK of .NET (need service team to ensure code readiness)
[X] SDK of Python
[X] SDK of Java
[X] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[X] I commit to follow the Breaking Change Policy of "no breaking changes"
[X] I have reviewed the documentation for the workflow.
[X] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Will mark this PR as draft until the official announcement of the service.

@yangyuan The service has been released.  Can you please approve and merge?

@yangyu @anuchandy Can you please approve and merge this PR?  Thanks.


Sync interests when registering the device token if needed

What?

Sync any new interests persisted on the device when registering the device token with the Beams service.

Update the code in the example project to show that it's not necessary anymore to call the handler upon completion.

CC @pusher/mobile

Codecov Report

Merging #86 into master will decrease coverage by 0.38%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
- Coverage   80.92%   80.54%   -0.39%
==========================================
  Files          32       32
  Lines        1059     1064       +5
==========================================
  Hits          857      857
- Misses        202      207       +5

Impacted Files
Coverage Δ

Sources/PushNotifications.swift
16.75% <0%> (-0.47%)
:arrow_down:

Continue to review full report at Codecov.

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


Evaluator issue with preferred value

What version of CUE are you using (cue version)?

$ cue version
cue version 0.4.0 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What did you do?
package foo

import (
        "strings"
)

#something: {
        image:    string | * "some default"
        // This function call seems to apply BEFORE the default are computed
        toString: strings.Join([image], "")
}

onething: #something & {
        image: "now this will work"
}

What did you expect to see?

import "strings"

#something: {
    image:    "some default"
    toString:    "some default"
}
onething: {
    image:    "now this will work"
    toString: "now this will work"
}

What did you see instead?
import "strings"

#something: {
    image:    "some default"
    toString: strings.Join([image], "")
}
onething: {
    image:    "now this will work"
    toString: "now this will work"
}

Vaguely reminiscent of #507 ?

This feels similar, but not sure if it's the same issue...
package foo

#nothing: {
        image: string
        thisWorks: image
        // This function call applies TOO early, before image is actually set and this never succeeds
        if image != _|_ {
                thisDoesNotWork: image
        }
}

all: {
        none: #nothing,
        other: #nothing
}

for k, v in all {
        all: "\(k)": image: k
}

Hi @dubo-dubon-duponey

#something: {
    image:    "some default"
    toString:    "some default"
}

Can you explain why this is your expectation? Is it consistency with the fact the image field has had its default selected? With eval we have this slightly weird beast which is a hybrid of actual CUE/not, somewhere between def and export. The difference with def in this case is that we are selecting the default, but then choosing not to evaluate the builtin. I don't know what the "right" answer is here. There is a plan to remove eval at some point (slightly ageing issue https://github.com/cuelang/cue/issues/337) and so as part of that we could consider what "right" means (for what will be the plain cue command).
Your second issue is more related to https://github.com/cuelang/cue/issues/990 AFAICT.

Hey Marcel ;)

Can you explain why this is your expectation?

The closest in my mental model would be:
image=${someVar:-default_value}
toString="$(dosomethingwithit "$someVar")"

I understand cue is not shell, but it's really hard to imagine why the default would only apply "after".
It also feels to me like it's defeating the purpose of a default here.
Pretty much this is saying toString is evaluating to anything BUT the final, actual value of the variable it is referencing.
For anyone who is new to this, it's really hard to explain why:
#something: {
        image:    string | * "some default"
        if image != _|_ {
                status: image // "works"
        }
        toString: strings.Join([image], "") // nope, this operates on a different value of image
        another: image // "works"
}

Your second issue is more related to #990 AFAICT.

Ok, this one is also hurting quite a lot.
Is this a bug in your opinion, or something that requires clarification / design?
Thanks Marcel for you time!

Hey Marcel ;)

If only I was Marcel ;-)
Whilst looking at this I uncovered a bug, raised as https://github.com/cuelang/cue/issues/1025. That won't be helping our intuition here. This bug also explains the strange "in between" behaviour I described in https://github.com/cuelang/cue/issues/1022#issuecomment-850771121.
Perhaps that clears things up? Because side-stepping that bug via:
exec cue eval

-- x.cue --
package x

import "strings"

#something: {
	image: string | *"some default"
	if image != _|_ {
		status: image
	}

	toString: strings.TrimSpace(image)
	another:  image
}

x: #something
y: #something & {
	image: "ok"
}

gives:
#something: {
    image:    "some default"
    toString: "some default"
    status:   "some default"
    another:  "some default"
}
x: {
    image:    "some default"
    toString: "some default"
    status:   "some default"
    another:  "some default"
}
y: {
    image:    "ok"
    toString: "ok"
    status:   "ok"
    another:  "ok"
}

Is this a bug in your opinion, or something that requires clarification / design?

I will defer to the real Marcel here: https://github.com/cuelang/cue/issues/990#issuecomment-846421752

Hey Marcel ;)

If only I was Marcel ;-)

🤦🏾♂
Sorry! Never do emails on a Saturday evening!
Let me start again: "thank you for your time Paul, and apologies for calling you names :-)"
#1025 is exactly this problem I'm hitting here indeed (we can close this one if you prefer).
As for the other part (https://github.com/cuelang/cue/issues/1022#issuecomment-850647715), looking forward for the Real Slim Marcel to stand up :-))).

Another thing that ignores default values: disjunction of structs.
Maybe I just do not understand something fundamental about cue here.
So far (and as far as I understand this), disjunction of struct and builtins ignore default values, while everything else works as one would "intuitively" expect it (operators, interpolation, conditional fields).
(* this is unrelated to eval vs. export btw)

Another thing that ignores default values: disjunction of structs.

If you think there is an issue here, please do open a new issue because it's good to try and flush out these sorts of issues. Even if we conclude they're dupes/non-issues, the commentary is useful for others who might end up going down the same path (and conversely good material for explanations on why things are the way they are). But per your comment above I'll close this in favour of #1025.

This issue has been migrated to https://github.com/cue-lang/cue/issues/1022.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Unable to use xpath properly with Appium Helper

What are you trying to achieve?
Click on a app object when using xpath to select the second element with the same text
What do you get instead?
When I use I.click('//[@text="6º ano"]') it works
As I have several elements with the same text on the screen and I woul like to click on the second
I.click('(//[@text="6º ano"])[2]') is not able to find any element

Provide console output if related. Use --verbose mode for more details.

    Emitted | step.passed (I wait for visible<EMAIL_ADDRESS>    Emitted | step.finish (I wait for visible<EMAIL_ADDRESS>    Emitted | step.start (I click "(//*[@text="6º ano"])[2]")
    I click "(//*[@text="6º ano"])[2]"
    [1] Error | Error: Clickable element android=new UiSelector().text("(//*[@text="6º ano"])[2]") was n...
    Emitted | step.failed (I click "(//*[@text="6º ano"])[2]")
    Emitted | step.finish (I click "(//*[@text="6º ano"])[2]")
    [1] Error | Error: Clickable element android=new UiSelector().text("(//*[@text="6º ano"])[2]") was n...
 Clickable element android=new UiSelector().text("(//*[@text="6º ano"])[2]") was not found by text|CSS|XPath

Provide test source code if related

    I.click('(//*[@text="6º ano"])[2]')

Details

CodeceptJS version:v1.4.5
NodeJS Version:v8.11.4
Operating System:Windows 7
Appium: 1.9.1

When using the appiun session to inspect. it is able to locate the object
When playing the script even making sure the page is fully loaded script is not able to locate the second instance of the object.
For this instance
I.waitForVisible('(//*[@text="Ciências"]/../..//android.widget.TextView)[2]',10);
Script is able to detect the instance
When clicking, it cant be detected

Hello,
did you get any solution of this issue ?


Add malayalam courses

What does this PR do?
Add resource(s)
For resources
Added a list of Malayalam tutorials.
Why is this valuable (or not)?
These tutorials will help the people who know Malayalam.
How do we know it's really free?
These are youtube playlists.
For book lists, is it a book? For course lists, is it a course? etc.
These are courses.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

For youtube, we prefer "playlist" links and the name of the producer/creator. https://github.com/EbookFoundation/free-programming-books/blob/master/CONTRIBUTING.md#platforms-and-access-notes
The non-playlist links meet our criterion for "long enough to be a course" > 1 hour, so this is good. It would help the reviewers a lot if you could note this in the submission.
Thanks, and welcome to free-programming-books!

@eshellman Thanks

@iathul @eshellman is malayalam related with hindi hi, for which listings already exist to, or in the other hand it needs to be considered a language apart?
Moreabout, if this resources are for Malaysia, language locale is ms, See https://www.localeplanet.com/icu/ms/index.html

@davorpa Like Hindi Malayalam is a language used in Kerala (a state in India). It needs to be considered a separate language.

Thanks for that clarification

Like Hindi Malayalam is a language used in Kerala (a state in India). It needs to be considered a separate language.


Why loading is not showing but is showing on nested screens ?

What version of Remix are you using?
1.10
Steps to Reproduce
I want to show a loder when I fetch data. If I go on /expense it does not show, but in other nested files its show. Why it does not show when I visit the page /expense ? It waits until data fetched then it shows. I want to see the page wherever the data is loaded or not.
export default function ExpensesLayout() {
  const navigation = useNavigation();

  const expenses = useLoaderData();
  console.log('RENDERING EXPENSINVE LAYOUT');
  if(navigation.state === 'loading') {
    return <>
        <Outlet />

Loading

        </>
  }

  const hasExpenses = expenses && expenses.length > 0;
  return <>
    <Outlet />

        <Link to="add">
          <FaPlus />
          Add expenses
        </Link>

          <FaDownload />
          Load Raw Data

      { hasExpenses ?
        <ExpensesList expenses={expenses} />
        :

No expenses found

            Start <Link to="add">Adding some</Link>

      }

  </>
};

export async function loader() {
  return await getExpenses();
}

Expected Behavior
..
Actual Behavior
..

Hi @shop-fluencer, again, please use our discussion tab to ask support questions.


qemu

After using kas build how can I run qemu in my PC please

Hi, for more in depth explanations you can refer the docs: https://eclipse-leda.github.io/leda/docs/
As a tl;dr, run:
  kas shell /path/to/kas-file.yaml -c 'runqemu slirp nographic ovmf sdv-image-all'

E.g., if you've just built the qemux86 leda-distro image, use:
kas shell kas/leda-qemux86-64.yaml -c 'runqemu slirp nographic ovmf sdv-image-all'

Closing due to no reply


POC: implement nested error messages in modals

This introduces a bit of infrastructure to allow notifications to be nested inside modals and drawers, which could be used to improve error handling within these parts of the UI.
There are three pieces that work together:

A new useNotificationState hook, which attaches a new local notification list to a component, and provides notify and reportError functions for updating it, which are compatible with their global counterparts.
A change to withErrorReporting to optionally accept a function, to allow routing error messages to the local list instead of the global one.
A new LocalNotifications component which can be used to render the local notifications within any 'positioned' container, e.g. modals, drawers.

See the Environments.js change for how this would look in practice. I tested by trying to delete one of the 'stuck' runtimes in dev, which generates an error.
This seems to work pretty well, but there are some caveats we might consider before moving forward:

Currently we're rendering these similarly to the existing ones, as overlays in the upper right. Because modals are more compact, this might end up being overly disruptive. That said, the immediate use case is for errors, which are disruptive by definition.
The local notifications don't currently use animation. Due to the positioning, we probably couldn't use the same 'slide' animation anyway, so we'd need to come up with something new if we wanted that.

This was a PoC, so closing it out.


Blinking window in fullscreen mode

VSCode Version: Code 1.18.0 (dcee2202709a4f223185514b9275aa4229841aa7, 2017-11-08T12:44:48.751Z)
OS Version: Linux x64 4.4.0-98-generic
Extensions:

Extension
Author (truncated)
Version

aurelia
Aur
0.3.4

vscode-docker
Pet
0.0.21

sass-lint
gle
0.0.4

azure-account
ms-
0.1.3

sass-indented
rob
1.4.5

vscode-multiclip
sle
0.1.3

(22 theme extensions excluded)

Steps to Reproduce:

Go to Fullscreen mode (F11)
Hover on folder, file or (sic!) any block with title attribute in Chrome
Wait until title appears
Move mouse cursor out from the element

The result is the VSCode window blinks 1 time (per each 1-4 manipulation) - disappears for a few milliseconds and then appears again.

Having the same issue:

VSCode Version: Code 1.18.0 (dcee2202709a4f223185514b9275aa4229841aa7, 2017-11-08T12:32:39.870Z)
OS Version: Darwin x64 16.7.0
Extensions: the listing length exceeds browsers' URL characters limit

Steps to Reproduce:

Open VSCode
Click the maximize button in the top left corner of the window

Reproduces without extensions: Yes

Can you also reproduce this with Chrome?
Can you also reproduce this with a barebone electron application?
@bpasero you might have additional questions

@antony-chopin does it reproduce when you run "code --disable-gpu"?

Ok, there is more information.

@isidorn The bug is not reproducible in Chrome. Can't say anything about another electron applications 'cause I have no it.
The bug appears only after appearing modals (like confirmation of moving/deleting files)
@bpasero The bug is not reproducible with --disable-gpu

I suppose, the bug is not reproducible in Chrome in case of absent any modals :)

Code Version : 1.18.0, commit : dcee2202
Node : 7.9.0
Windows : Windows 10 Pro, 1709, 17025.1000

Yeah me too, but its not reproduced with --disable-gpu

The same issue on Ubuntu 18.04-beta, not reproduced with --disable-gpu

The same problem on Ubuntu 18.04, blinking in full screen mode when hover over a folder. It's been a while since the issue and nothing solved

Might be fixed on insiders with latest chrome, try it out in a couple of days when we jump on Chrome 71

Bug is not appearing anymore, BTW :)

Great, thanks for letting us know.


UI extension not showing on Thank You page

Please list the package(s) involved in the issue, and include the version you are using

ui-extensions v2023.10.x
ui-extensions-react v2023.10.x

Describe the bug

After deploying the checkout extensions and adding the block in the Thank you page via checkout editor, the extension shows just fine in the editor preview, but it does not render at all on the actual Thank you page of the store, when an order is placed.
If the extension is also exported to the order status page, it will show correctly on both checkout editor and on the real order status page.
Steps to reproduce the behavior:

Add extension on the Thank you page via checkout editor.
Verify that the extension is rendered in the editor preview.

Expected behavior

Extension should render on the real Thank you page when an order is created
Screenshots

Additional context

I've tried on different themes/stores/fresh installs of the app but the problem persists.

Hey @dnagoda thanks for getting back!

Are you using the purchase.thank-you.block.render target?

Yes, this is on my shopify.extension.toml file:
[[extensions.targeting]]
target = "purchase.thank-you.block.render"
module = "./src/Checkout.tsx"

And this is how i export it in Checkout.tsx:
function Confirmation() {
    return <Text>Lorem ipsum dolor sit amet</Text>;
}

export const thankYouBlock = reactExtension('purchase.thank-you.block.render', () => <Confirmation />);

Can you please provide the URL you're seeing for the thank-you page where the extension is not displayed?

This is one example: https://juliano-checkout-2.myshopify.com/checkouts/do/cea7b95d701d2e682c3c99bd160b1ac0/thank-you

you've enabled extensibility for the thank-you page in the shops admin Settings / Checkout area?

Yes, this is how it looks like:

Thanks.
This looks like a Draft Order based on the URL you provided. Unfortunately, extensibility is not fully supported on draft orders. We're working on it, but there's no public ETA yet.

Thanks for explaining @dnagoda


Suggestion!!! :D

Can add something like //paste -0
the -0 means paste the building but no paste air

//merge


prevent images and iframes reloading on patch?

Hi!
I have a view which renders two iframes (one on screen, and one translated off-screen). I slide the visible one off-screen, and the hidden one slides in, using CSS transitions.
There are three states to my transition:
1 when we are just showing the current slide (just one iframe),
2 when the next slide is loading off screen while the current slide is still on screen,
3 when the current slide is sliding out,  and the new one is sliding in
(and it goes in a loop, so after 3, we go back to 1 for Xms, then 2 for Yms and so on)
I have it all working correctly using reducers to set the states, and a "transition" effect that calls the reducers after setTimeouts. My view sets classes on the iframe-tags to make it use a CSS transition and slide
The only problem is, that the view we render at stage 1 causes the iframe to reload (making a blinking effect, which defeats the whole idea of this "load and slide in")
My only guess as to why this is, is that the patch-function is somehow determining that the whole iframe tag needs to be updated (thereby changing the src attribute as well), when I remove the "entering" class. But it doesnt seem to blink when I add the "entering" class.
I tried to read the patch-logic to find a way to add and remove classes without touching the src attribute but couldn't figure it out. Is there a way?
Or perhaps there is a way to use the lifecycle methods to capture and prevent the iframe from being updated, and it could let me "manually" modify the class property?

I tried to read the patch-logic to find a way to add and remove classes without touching the src attribute but couldn't figure it out. Is there a way?

What does the src attribute has to do with adding/removing classes?

Ok, to explain my issue more simply (probably should have started here): I have a view that renders an iframe basically according to this:
function (model, actions) {
    var cls;
    if (model.state === 'loading') cls = 'loading'
    if (model.state === 'transitioning') cls = 'entering'
    if (model.state === 'showing') cls = 'showing'
    h('iframe', {src: model.url, class: cls)})

I have noticed that when model.state goes from 'loading' to 'transitioning', there is no visible reload of the iframe. Indicating to me that hyperapp did not touch the actual dom-node, but only updated the class attribute of the domNode (as I intended)
From 'transitioning' to 'showing' there IS a visible reload of the entire iframe. Indicating either that the entire DOMNode was replaced (or at the very least, the src attribute was fiddled with.) -- which is what I would like to avoid.
When all i change is the class name, I don't want the entire dom node replaced, because that makes the browser reload the iframe, causing a visual "blink" which is exactly what I want to prevent with this preloading scheme.

Perhaps one reason hyperapp behaves differently in the cases of 'loading'->'transitioning' and 'transitioning'->'showing', is that in the former case, I still have two iframes in the before and after. But in the second case, I go from having two iframes (the old that slid out, and the new that slid in) to just one (currently showing)

@zaceno I'll read this more carefully when I am home. In the meantime, it could help us speed up any possible solution if you could come up with a fiddle or codepen that shows the problem.
I hope that's not a lot of work 🙏

Absolutely! Thanksk for taking the time to consider my question.
Here is a codepen example: http://codepen.io/anon/pen/BWNmjG
Note that there are three rectangles where an iframe can be, and it is the center one I call "on screen". In an actual implementation I would have this "zoomed in" so the left and right areas would be off screen and invisible.
Anyway: notice how as iframes start to slide from the right area to the middle, they don't reload. They start sliding because I changed the class (from 'loading' to 'leaving'). Hyperapp seems smart enough to understand that when I render the same iframe with a different class name, I don't need a new iframe-element, but just need the class updated on the existing iframe.
However, once the frame reaches the middle, there is a clear reload. This is what I am trying to prevent. I want to make this smooth. At this point I render with class: 'showing', instead of 'entering'. And also stop rendering the sibling iframe. This time, it appears hyperapp thinks the iframe needs to be completely swapped out for a new one.

Any ideas anyone?
I was thinking: if there's no way of avoiding the virtual-dom replacing the iframe, then perhaps this sort of problem shouldn't be managed by hyperapp at all. Perhaps the "right way" would be to capture the DOMElement in onRender, and then set up the timers and dom manipulations to do the transitions outside and parallel with hyperapp's state management.
... I'm not exactly sure how that would work, but it seem's like a hacky and messy way to go about things.
I realize I'm not explaining the issue well, but take a look at the code-pen example above. As it seems to me right now, hyperapp is unable to satisfy what I believe to be a relatively common and important use case. That should give this issue some priority, imo.
If no solution is found, I suggest making it clear in the docs, what it is (more generally) that is not supported, and providing a recommended workaround.
I would provide a PR, except I have no idea why the iframe is replaced in some calls to the patch-functions,  and not other times. Someone who understands how the patch-function works needs to look closer at this.

... of course as always, there's always the chance that the patch-function is fine, and it is me who made a dumb mistake in my code. But the example is there on the codepen for anyone to check.

@zaceno Sorry, I just haven't had time to look into this issue at all because of work. I'm using my free time to work towards the 0.7.0 release and the new plugin architecture, plus docs. If I manage to get all that done before the weekend, I may be able to take a look at this then.
Leave the CodePen around too please!

@zaceno Why are slides a local constant inside the view?

@jbucaran No worries, I get that we all have day-jobs too :) I didn't mean to try to rush you or anyone, just emphasize that I think this should be a priority (because I've rambled so much about it, I feel like that point got lost).
The Code pen is anonymous, so I don't know how long they will leave it around. I should probably sign up for an account to save it permanently. I'll look into that.
The const slides = [] is  just an implementation detail, I could just as well have written the view like this:
view: function (model, actions) {

    if (!model) return ''

    if (model.current && model.next) {
        return h('div', {class: 'slide-container'}, [
            h('iframe', {src: model.current, class:  'slide' + (model.state === 'transitioning' ? ' leaving' : ' showing')})
            h('iframe', {src: model.next, class: 'slide' + (model.state === 'transitioning' ? 'entering' : 'loading')})
        ])
    }

    if (model.current && !model.next) {
        return h('div', {class: 'slide-container'}, [
            h('iframe', {src: model.current, class:  'slide' + (model.state === 'transitioning' ? ' leaving' : ' showing')})
        ])
    }

    if (!model.current && model.next) {
        return h('div', {class: 'slide-container'}, [
            h('iframe', {src: model.next, class: 'slide' + (model.state === 'transitioning' ? 'entering' : 'loading')})
        ])
    }

  },

But that means unnecessary repetition of code. I found it more concise to just set up an array of children for the slide container, and populate that array with one or both iframes depending on state, before rendering the container.

Ok here's the exact same pen, but no longer anonymous. I presume it doesn't risk being deleted:
http://codepen.io/zaceno/pen/gmPbzV

Here is another pen exemplifying the problem. This time using images instead of iframes.
http://codepen.io/zaceno/pen/bqwdgG
(also I locked it to using the 0.6.0 tag. The previous pens were not locked to a specific version, and might break if hyperapp changes backward compatibility in 0.7.0 for example)
Notice, however, that the reload-blink doesn't happen for all images. Only two out of three. What makes those images special, I don't know. Maybe it's just more apparent for two out of three because they are larger, server is redirecting... who knows.

@zaceno Thanks! Now that 0.7.0 is out, I have more time to look at this. We'll get to the bottom of it 💪.

Sweet! Congrats on getting 0.7.0 out. Didn't realize it had just been released. Will test this on 0.7.0 then, just to see if it magically resolved itself like bugs sometimes do :)

@zaceno Don't think so, but miracles do happen in movies.
I still haven't published any release notes mentioning the breaking change.
Rename: reducers/effects → actions. It's all a single object now.

@jbucaran Thanks! I tested it and it is unfortunately still a bug.
But also, I noticed another breaking change (or separate bug?) in 0.7.0: my actions (at least those that previously were 'effects') were not recieving the actions object any longer. I had to pass it along from the subscription, as a data-object (fortunately this pen doesn't need to pass data to actions, so it works for me, but not sure how that would work otherwise) Should I open a separate issue for this?
http://codepen.io/zaceno/pen/JWRdVm?editors=1111

Ok I "fixed" it. I had some time to look at this, and figured out the only way to make this work is with a minor change to the patch function, which allows you to specify that a node must be inserted as a new node, and not replace or update an existing node. I made a pull request (#135) for that. It may not be the best way to implement it, but at least it works.
I have updated this codepen, to demonstrate how I would use the new "force insert" functionality.
http://codepen.io/zaceno/pen/JWRdVm
But I haven't figured out how to get my modified hyperapp into the codepen. So it doesn't actually work on code pen, yet. I'll look into that.

https://github.com/hyperapp/hyperapp/pull/135

I have updated the pen to use my modification in PR 135, and it now demonstrates the desired behavior
http://codepen.io/zaceno/pen/JWRdVm

@zaceno The result looks great! We just need to figure out a way to do this without _insert.

Yes, I agree the _insert flag is not pretty. But I don't see any other way at the moment.
By the way, the _insert flag allows a potentially is a potentially pretty good performance boost for apps working with long lists of things. Without it, if you add something to the top of a long list, the patch function has to perform DOM operations for every item in the list. But if you add something to the top of a list with the insert flag, the only DOM operation will be to insert the new item. Nothing else, since the rest of the lists won't diff.

... so it's kind of like an optimization flag. You don't ever have to use _insert:true (except for use cases like mine), but you can, and if used judiciously will improve performance of your app.

... if used un-judiciously, it should not harm your performance in any way that I can tell (but your app might not work at all as intended ;) )

Following some discussions elsewhere, I feel it's best to discard the _insert: truesolution of PR 135, and instead pin my hopes to https://github.com/hyperapp/hyperapp/pull/141 which solves this problem through the use of "keys" to dom nodes, instructing the patch function to reuse dom-nodes when it sees identical keys being used.

@zaceno Close? 😉


parent is undefined during SSR

In the dropdown component, parent is referenced (https://github.com/carbon-design-system/carbon-components-svelte/blob/afa692c9e0db235d0e54975281fa89f831109077/src/Dropdown/Dropdown.svelte#L158-L168) in the onMount and onDestroy functions, however this is not defined during SSR in SvelteKit.
A possible fix that would work in Svelte as well as SvelteKit would be to check for globalThis.window:

during SSR it is globalThis.window is undefined
in the browser globalThis.window is defined

So this would work:
  let parent = globalThis.window ?? undefined;

  onMount(() => {
    if (parent) {
      parent.addEventListener("click", pageClickHandler);
    }
  });

  onDestroy(() => {
    if (parent) {
      parent.removeEventListener("click", pageClickHandler);
    }
  });

Alternatively checking if parent is defined would work in case globalThis is not an option (yet Svelte uses globalThis since version 3, so it should be save). This one will always work I guess:
if (typeof parent !== "undefined" && parent) {

Minimal example: https://github.com/ptrxyz/carbon-bug-testing/

I reproduced this as well; the solution is to return a function in onMount since onDestroy does in fact run during SSR. (@metonym, you were right all along).
As discussed on Discord, @ptrxyz will provide a PR.

This is now released in v0.72.1
Resolved by #1639

Thank you both for the quick fix and release.


Revert "Fix up rough edges in Polish translation"

Reverts Bluewire-OSS/templates#8
yh i accidentally added code
sorry

im a dumbass sorry


updates for CATS report

@v-kents,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

globalization/docfx.json
:white_check_mark:Succeeded

globalization/index.md
:white_check_mark:Succeeded

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


Meta or Facebook Open Source?

Similar to this issue: https://github.com/facebook/react-native-website/issues/2893, because Facebook open source is now meta open source, is this logo still valid? Or should it be changed to match the new logos on the meta open-source website? This applies to https://reactjs.org if you scroll to the bottom. If this is invalid, feel free to close this issue, I just wanted to bring it up.

Oh, and I just realized that the copyright was updated. So, my only wondering is about the image.

can you assign it to me?

There is no need to do anything. The new site doesn't have the logo anyway.


flatfield correction

We have master_flats, but when normalising them in the 2D plane, the resulting master_flats are not useful for a flat-field correction. Should the flat-field normalisation to the Master Flat better be applied after order extraction?

I do not understand this issue anymore. The Flat is not being normalised for each CCD to the nan-max value. That seems to work when doing the flat-field correction. Note that the flatfield correction is only applied after the tramlines extraction!


Add proper SNOMED codes for segmentation output (RMS, necrosis, etc.)

Added metadata_config method from Chris Bridge (created for IDC publishing of original model results) to read correct codes.


Update README.md

Added link to the bot and a TaskList

Thanks for the PR. Looks great, merging 👍


Failed to import -<EMAIL_ADDRESS>

Failing module<EMAIL_ADDRESS>
GitHub: https://github.com/nodeca/pako/tree/1.0.11
npm: https://www.npmjs.com/package/pako/v/1.0.11

import { Inflate, Z_SYNC_FLUSH } from "https://esm.sh/v120/pako@1.0.11/es2022/pako.mjs?cjs-exports=Inflate,Z_SYNC_FLUSH";
console.log(Inflate, Z_SYNC_FLUSH);

Error message
After onload I got this error:
Uncaught SyntaxError: The requested module 'https://esm.sh/v120/pako@1.0.11/es2022/pako.mjs?cjs-exports=Inflate,Z_SYNC_FLUSH' does not provide an export named 'Inflate'

Additional info
The following code works. Unfortunately I do not have access to change this import, as it is in a sub-dependency of a library. Thank you for your help!
import * as Pako from<EMAIL_ADDRESS>console.log(Pako.default.Inflate, Pako.default.Z_SYNC_FLUSH);

esm.sh version: v120
Browser version: Edge 113.0.1774.35/Chrome 113.0.5672.92

We can see an example of this issue in the following import:
<script src="https://esm.sh/v120/<EMAIL_ADDRESS>type="module"></script>

/* esm.sh - esbuild bundle<EMAIL_ADDRESS>es2022 production */
import { Buffer as __Buffer$ } from<EMAIL_ADDRESS>import { Z_SYNC_FLUSH as k, Inflate as y } from<EMAIL_ADDRESS>
Error
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'Z_SYNC_FLUSH' (at bgzFilehandle.ts:130:3)

please use import { Inflate, Z_SYNC_FLUSH } from "https://esm.sh/v120/pako@1.0.11?cjs-exports=Inflate,Z_SYNC_FLUSH" instead

We can see an example of this issue in the following import:
<script src="https://esm.sh/v120/<EMAIL_ADDRESS>type="module"></script>

/* esm.sh - esbuild bundle<EMAIL_ADDRESS>es2022 production */
import { Buffer as __Buffer$ } from<EMAIL_ADDRESS>import { Z_SYNC_FLUSH as k, Inflate as y } from<EMAIL_ADDRESS>
Error
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'Z_SYNC_FLUSH' (at bgzFilehandle.ts:130:3)

i will fix this

Thank you, I really appreciate it

For now, I am using an import map to work around this issue:
<script type="importmap">
  {
    "imports": {
<EMAIL_ADDRESS>"https://esm.sh/pako@1.0.11?cjs-exports=Inflate,Z_SYNC_FLUSH&exports=Inflate,Z_SYNC_FLUSH"
    }
  }
</script>

@park-brian v121 should work fine

free to open this if not work for you thanks!

Great, thank you so much!


missing input file 'external/androidsdk/build-tools/28.0.3/aapt2

So when i Build AAR file in WSL then it's give me error missing input file (in picture)  but when i check the directory /root/.cache/bazel/_bazel_root/0505f157c249bce4e2dbda8c7d819117/external/androidsdk/build-tools/28.0.3/aapt2 in linux it's already exist
Here :

Please help me with this issue i have spend so much time for this.
Thanks a lot !!!

I am getting the same issue

Hi @NguyenLong-02, Could you please check with the latest release and let us know if you still see the same error .

Any solution?


Not showing all browsers

open-link-with is not showing all my installed browsers.  My default browser is Vivaldi, a Chrome clone.  I also have Firefox installed but as a backup browser.  I am using Kubuntu 22.04 which is set  to default to opening Vivaldi.  Only Obsidian refuses to open Vivaldi so I installed your plugin.  However the Vivaldi browser does not show up in the browser list so the plugin doesn't work.  I am using open-link-with v0.1.10 in Obsidian v1.3.7.

I found more info.  It has to do with Obsidian installed as a snap.  The snap version of Obsidian only sees the snap version of Firefox.  The non-snap Vivaldi browser which I have checked and is definitely set as my default browser and also works as such for my non-snap apps.  This is a known issue which I found at https://forum.snapcraft.io/t/default-browser-not-used-from-within-snap/4347/35

@Exomancer Did you find any solution?

In case anyone else has this issue in the future, I was able to find a solution using the custom profiles detailed on the readme. I'm on windows 11, so mileage may vary for linux.
I use firefox as my default browser, and chrome for professional work. My plugin wasn't detecting chrome in the browser drop down. Using the template in the readme, I made this:
{
    "chrome": [
        "C:/Program Files/Google/Chrome/Application/chrome.exe"
    ],
    "chrome-private": [
        "C:/Program Files/Google/Chrome/Application/chrome.exe",
        "-incognito"
    ]
}

pasting this into the Customization dialog box in plugin settings produced the expected behavior.

In case anyone else has this issue in the future, I was able to find a solution using the custom profiles detailed on the readme. I'm on windows 11, so mileage may vary for linux.
I use firefox as my default browser, and chrome for professional work. My plugin wasn't detecting chrome in the browser drop down. Using the template in the readme, I made this:
{
    "chrome": [
        "C:/Program Files/Google/Chrome/Application/chrome.exe"
    ],
    "chrome-private": [
        "C:/Program Files/Google/Chrome/Application/chrome.exe",
        "-incognito"
    ]
}

pasting this into the Customization dialog box in plugin settings produced the expected behavior.

This trick works fine for me. Copy the path to the executable file, use chrome or edge in the combo (same as the json file)


chore(package): update lint-staged to version 7.1.2

Closes #175

Codecov Report

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

@@            Coverage Diff             @@
##           master     #178      +/-   ##
==========================================
+ Coverage   84.43%   85.99%   +1.55%
==========================================
  Files          11       11
  Lines         257      257
==========================================
+ Hits          217      221       +4
+ Misses         40       36       -4

Impacted Files
Coverage Δ

src/baseRequest.js
79.31% <0%> (+6.89%)
:arrow_up:

src/request.js
92.85% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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


Loading signature pad with existing data URL

Maybe this already exists and I just didn't see it, but it would be awesome to be able to load the signature pad with a signature that was previously captured.

Thanks for the feature request! It doesn't exist yet but I'm happy to merge in a PR that enables it :) Let me know if you want to discuss potential implementations


How to change font size?

I am not sure how to change the font size of the printer.

Hello, I know it's a little late but I wrote a simple extension class that achieve the font size setting (@mtmsuhail could be interesting to see this implemented)
public static class PrinterExtensions
{
	[Flags]
	public enum PrintStyle
	{
		HeightNormal = 0x00,
		Height2X = 0x01,
		Height3X = 0x02,
		Height4X = 0x03,
		Height5X = 0x04,
		Height6X = 0x05,
		Height7X = 0x06,
		Height8X = 0x07,
		WidthNormal = 0x00,
		Width2X = 0x10,
		Width3X = 0x20,
		Width4X = 0x30,
		Width5X = 0x40,
		Width6X = 0x50,
		Width7X = 0x60,
		Width8X = 0x70,
	};

	public static void SetStyle(this Printer printer, PrintStyle style)
	{
		printer.Append(new byte[] { 0x1D, 0x21, (byte)style });
	}
}

you can then use it like this:
Printer printer = new Printer("FooBar");
printer.SetStyle(PrintStyle.Height2X | PrintStyle.Width2X);

Source


Can we have a config option to disable the log output?

I love this plugin and am very happy with the development! I do, however, wish to disable the continuous logging because it's pretty hard to troubleshoot other plugins when the log files are completely filled with lines like these:
[9/7/2021, 4:25:22 PM] [P1] Electricity: set Total Consumption from 36128.027 kWh to 36128.028 kWh
[9/7/2021, 4:25:22 PM] [P1] Electricity: set Total Consumption Normal from 16578.191 kWh to 16578.192 kWh
[9/7/2021, 4:25:22 PM] [P1] Electricity: set Current Consumption from 340 W to 330 W

Set Log Level to 0 in the electricity accessory.


Location Activity not going on-screen messages

My last two messages is JSQLocationMediaItem they always showing activity indicator on it, when CollectionView scroll up and again scroll down then indicator gone and show location placeholder on placeholder.

Hey @UmerAfzal1 :smile:
At some point, you'll need to reload those collection view cells.


chore(): update stencil

Pull request checklist
Please check if your PR fulfills the following requirements:

[] Tests for the changes have been added (for bug fixes / features)
[] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (npm run build) was run locally and any changes were pushed
[x] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe):

Updates core dep of stencil to 1.7.4

Can you update from master? Why is the changelog off?

Hmm, I did update from master with rebase

Did you do a force push?


backend/headless: use config-less context

This is preliminary work for https://github.com/swaywm/wlroots/pull/2505.
With wlr_swapchain, we stop using EGLSurface entirely, so we can skip selecting an EGLConfig.

However, it seems that libweston will fall back to using some EGL config attribs in case that the EGL extensions are not present. Is there someplace we can find how widely supported these are?

I thought of doing that too, but I figured we could try without the workaround first and see if anyone complains about the missing extension.
Mesa unconditionally enables the extension for all drivers: https://gitlab.freedesktop.org/mesa/mesa/-/blob/661b29283425c568d1fa403be703775e02c50d08/src/egl/drivers/dri2/egl_dri2.c#L928


update csi-addons dependency from 0.5.0 to 0.8.0

since we are going to be installed alongside w/ odf-operator inorder to conform to u/s CI changing the dependency of csi-addons from 0.5.0 to 0.8.0

ref https://github.com/red-hat-storage/odf-operator/blob/096506a5855fff8491b5a70f35d29f84d833a821/hack/make-bundle-vars.mk#L93-L94

by the time I realized the miss https://github.com/red-hat-storage/ocs-client-operator/pull/131#issuecomment-2036912588, earlier PR was merged.
However, having current one in a new PR explicitly doesn't hurt.


How to simulate a substance being broken in two, cut, or smashed?

Does MuJoCo have such bodies?
I see the skins and the fabrics. But I'm not sure what this means, from the docs... " MuJoCo skins are only used for visualization and do not affect the physics in any way. In particular, collisions involve the geoms of the bodies to which the skin is attached, and not the skin itself."
In the hammock XML example it seems like the humanoid is right up against the hammock fabric. From the quote above it seems like the documentation is saying the skin materials are some kind of animation.

You could create the pieces, weld them together, and disable the welds at run time. This will simulate the effect of breaking.
But more generally no, MuJoCo was not designed for this type of high-dof simulation and is focused rather on articulated systems.

@yuvaltassa
Hi, I saw your name on the Maximum a Posteriori Policy Optimisation paper and I wonder if you wouldn't mind me asking what you think is the best algorithm right now for a multi-objective humanoid robot with continuous controls. Sorry, I know this is probably the wrong place to ask.
I was looking at the MO-MPO algorithm.


Resolve syncing conflicts from from-prod-live to live

Pull request opened by Docs to resolve syncing conflicts from from-prod-live to live.

Docs Build status updates of commit b72b338:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/OPS-E2E-PPE/azure-docs-powershell, working directory: W:\u1a0-s, target branch: live, target branch commit: 4e4265f48465520d0e27fd785fdee5bb7976ea3a, source commit: b72b338525ee302597b3a54a11453f4881d22689.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit b9b2dea:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/OPS-E2E-PPE/azure-docs-powershell, working directory: W:\sc30-s, target branch: live, target branch commit: 4e4265f48465520d0e27fd785fdee5bb7976ea3a, source commit: b9b2dea3441d1532a5564ddca3dced45424fe2d6.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit bd7edc4:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/OPS-E2E-PPE/azure-docs-powershell, working directory: W:\qgs0-s, target branch: live, target branch commit: 4e4265f48465520d0e27fd785fdee5bb7976ea3a, source commit: bd7edc4d48b6a8a8bec864edc876e16af0a49505.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0f2a1ca:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RunPsScriptFriendlyFailed] Run script failed with no detailed information. Please retry building your content again. If the issue still happens, open a ticket in https://SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[1.13] gateway-api: update types to read v1beta1 if present (#41238) …

…(#41259)

[1.14] gateway-api: update types to read v1beta1 if present (#41238)

gateway-api: update types to read v1beta1 if present

The upstream API is dropping support for the v1alpha2 types for HTTPRoute, Gateway, and GatewayClass in the next release. So, unlike Istio where we always read the alpha versions in Istiod, we must switch to reading the beta types or users on v0.6.0 (next release of API) will be broken.
This means that the minimum version of the API is v0.5.0; most users are upgrading quickly since v0.4.0 was still an alpha API.
(cherry picked from commit fc33f94ecd1367de8e060aa78d615e106c77b952)

fixes

(cherry picked from commit dbf424bf27552290f563d0c1ee8238bb5671d63d)

fixes

fixes

fix conformance

lint

remove entirely

(cherry picked from commit 84a45e740be465b47712cbfcc7d8a094488f2a94)
Please provide a description of this PR:
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ ] Does not have any user-facing changes.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio_release-1.13
f297c316f432cd0e13d262bf20a5e2fdbbef6ed9
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio_release-1.13
f297c316f432cd0e13d262bf20a5e2fdbbef6ed9
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio_release-1.13
f297c316f432cd0e13d262bf20a5e2fdbbef6ed9
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Decouple static Linux binaries from libc

When Go builds static Linux binaries, can these have the option of being decoupled from the specific libc implementation? I'm no kernel hacker, so I am curious if Go on Linux could be replatformed onto POSIX-standard calls moreso than any particular Linux libc interface.
Traditionally, Linux binaries have been GNU/Linux. But with the rise of alternative Linux libc implemenations (uClibC, musl, Android / Bionic, etc.), it's easy to mistakenly package a Go Linux binary in a way that other Linux users are not able to run on their particular Linux flavor. For example, Go binaries built with CentOS may run on Debian or vice-versa, but may not run on Alpine, Void Linux, OpenWrt, or busybox, as they link against different C libraries.

Standard Go binaries already do not use libc - they issue system calls directly.
We only use libc if you use cgo, or if you import certain packages (net is the common one I believe. There may be others.)
Are there specific incompatibilities you have run into? Or is this speculative?

We already do this as much as possible on GNU/Linux.  I'm going to close this issue, but please comment if you have specific suggestions for things that we should change.


*: improve latency when streaming series from Prometheus

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

Changes
I've found that when requesting many series (in the order of ten
thousands), the Thanos sidecar spends half of its time computing the
number of received series. To calculate the number of series, it needs
to build a label-based identifier for each chunked series and compare it
with the previous identifier. Eventually this number is only used for
logging and tracing so it doesn't feel like it's worth the penalty.
This change adds an histogram metric,
thanos_sidecar_prometheus_store_received_frames, which tracks the number
of frames per request received from the Prometheus remote read API
(buckets: 10, 100, 1000, 10000, 100000). It can be used to evaluate
whether expensive Series requests are performed.
Verification
CPU profile before this PR (half of the time is spent in prompb.(*Label).String)

CPU profile after this PR

CPU graph showing the 50% improvement (the patched version of the Thanos sidecar has been deployed around 15:50)
https://snapshot.raintank.io/dashboard/snapshot/mufpH3k2yRJUYLM284SXESwVLG0Ns14i?orgId=2

https://github.com/prometheus-operator/prometheus-operator/pull/3457 is the prometheus-operator PR including this change.

@bwplotka friendly ping :)

Thanks, amazingly spotted :heart:


int parser fails on trailing "e"

int parser fails to parse 42efg while parsing 42abc works.
ellie: https://ellie-app.com/4jqm8pFGXyLa1
Related issues: https://github.com/elm/parser/issues/14 and https://github.com/elm/parser/issues/25

I've run into this issue as well, while parsing Bencoded content.
In this example, parsing "2" from "i2e" is the goal.

Input
Result
Expected

i2
Success
Success

i2h
Success
Success

i2e
Problem
Success

i2.5
Problem
Success

Code:
https://ellie-app.com/5cLQ4CgfWvGa1
The "i2.5h" and "i2e" examples fail in the same way, making it seem like Parser.int is interested in floats as well as integers. I also tried using Parser.number and only allowed int, hoping the specificity would improve the situation, but alas, the same results.

e also causes problems with parsing floats.
run float "e" --> "Expecting float at column 2"
run float "e0" --> "Expecting float at column 1"

I'm facing a similar issue with character e messing with my parser :-(
Did anyone found a temporary fix ?

@matsumonkie See here for example:
https://github.com/elm/parser/issues/14#issuecomment-450547742
You may need to adapt if you want to handle negative numbers.

Thank you for your answer @rlefevre
I think my issue is not related to your comment though.
When I use the [https://package.elm-lang.org/packages/elm/parser/latest/Parser#variable](variable parser) almost exactly like this:
typeVar : Parser String
typeVar =
  variable
    { start = Char.isLower
    , inner = \c -> Char.isAlphaNum c || c == '_'
    , reserved = Set.fromList [ "let", "in", "case", "of" ]
    }

I noticed that it worked fine as long as the variable name doesn't start with the letter e.
So "foo" works fine but "efoo" doesn't.

@matsumonkie I've used this workaround for now:
{-| The built-in float parser has a bug with leading 'e'.
See <https://github.com/elm/parser/issues/28>
-}
parseFloat : Parser Float
parseFloat =
    {- By making it backtrackable, even if the input starts with an 'e',
       we will be able to try out other alternatives instead of getting
       stuck on it as an invalid number.
    -}
    Parser.backtrackable <| Parser.float ExpectingFloat InvalidNumber

@matsumonkie If the problem comes from Parser.variable, you should open a new issue about it, this issue is about Parser.int.
But I don't think that there is a problem with variable, for example:
https://ellie-app.com/8Mq3xfyKVn5a1
Instead you most likely have a Parser.int or Parser.float before the typeVar parser, leading to the issue:
https://ellie-app.com/8Mq4tJVgKJga1
This is the Parser.int that chomps the e from the string and make the whole oneOf fails (because of the issue we are commenting in).
So you can either use the parser I linked to above from the other issue:
https://ellie-app.com/8Mq5nMzBw79a1
It checks that only digits are used to parse a number.
Or you can just put the typeVar parser before the int one:
https://ellie-app.com/8Mq6fCJP59ta1
If this is possible, this is likely the most efficient work-around.
backtrackable is an option, but it is likely significantly less efficient.

Instead you most likely have a Parser.int or Parser.float in a oneOf before the typeVar parser, leading to the issue:
https://ellie-app.com/8MqcZtr6mmka1
This is the Parser.int that chomps the e from the string and makes the whole oneOf fail (because of the issue we are commenting in).

Well... that was exactly it! Sorry for the inconvenience :-S
And many thanks for your answer, it helped a lot!


Sign in to confirm age

Please describe your problem in as much detail as possible:
I have to sign in to confirm my age even when NSFW: true in the options.

Further information:

DisTube version: 3.3.2
discord.js version: 13.6.0
Node.js version: 16.13.2
Operating system: Linux Ubuntu 20.04

Use cookie to bypass it. Make sure your account is over 18 (FAQ)


Allow the user to scroll the content of the details view

Aallow the user to switch the focus between the form buttons and details pane (when visibile). In this way, the user could scroll the details view content by using the up/down arrow keys.

/lgtm


SnpEff cloud patch

The SnpEff config and database files are not currently being staged by Nextflow because they are being supplied as an argument via the modules.config file, rather than as a process input. This is only an issue when running on cloud because the files must be staged before they can be used. Fixing this issue revealed several other issues with the current format. Specifically, only the name of the database is supplied via the "--species" flag in MycoSNP, which means the database files are never staged. My workaround for this was to supply the directory containing the config file (i.e., ${projectDir}/assets/snpeffdb/) and database files instead of just the config file (i.e., ${projectDir}/assets/snpeffdb/snpEff.config) as input for the "--snpeffconfig" flag. This also solved issues with how SnpEff assumes that database files are within a subdirectory of the config file directory. This approach assumes that your config file is named "snpEff.config" and that your database is in a subdirectory called "data". There are probably better ways to approach this, but I aimed for making the smallest number of changes possible.

Reopening to merge into a clean v1.6.0_dev branch. Easier to identify contributions this way.


Issues #82 Add workflow to re-open tasks once they have been marked c…

…ompleted

@phil-monroe can you review when you get a chance?

LGTM, thanks @kristysnet!


Decentralized Trust example scenario

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/apps/decentralized-trust.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/decentralized-trust/DecentralizedTrust.png
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/apps/decentralized-trust.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/decentralized-trust/DecentralizedTrust.png
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/apps/decentralized-trust.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/decentralized-trust/DecentralizedTrust.png
:white_check_mark:Succeeded
View

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


Something Removed? And Idea

Hey, 1st i would like to say thanks for making the plugin its nice, been using and sharing the packs i made around with other people 1k shots+ its nice
So, i re got it after windows reset, but i noticed that when i try to use auto fill while using the plugin it still is set to unlimited boost? I change around setting in rocket league to figure out why but nothing has changed.
Using auto refill or the standard boost amount make the shot more realistic.
So how do i get a older version of freeplaycheck point or will it be added again into a newer update?
Ideas
Hey so not sure if it possible but maybe you can take nameplataes or car hitbox from the replays and put them into freeplay depending on each shot and use them as a obstacle or something to make it a bit more realistic instead of having to do image trainning based on where the other team can be from.
Not possible?
Well there is a bot in Starbase arc in freeplay for rumble, can you edit that? I also seen other plugins spawn in bots from freeplay like the kickoff bot where it takes your recording of a kickoff and you can practice kickoffs or get someone elses attempt and also try there kickoffs and beat it and stuff.
Even if it slows down how long it takes for getting the shot it self it would help to have something to avoid and stuff.
I have more ideas if needed
Thanks for reading

I don't think the plugin should impact your boost settings.  If you want to try using older versions, though, you can download them from the releases page: https://github.com/NitrOP7674/FreeplayCheckpoint/releases (I'd recommend sticking to the release versions -- not "-preX" suffixed, as they likely have bugs that I found and fixed later).  Each checkpoint stores your current boost amount, so if you took the checkpoint with 100 boost you will have 100 again when you restore it.

Not possible?

Very possible.  I had experimented with a way to capture part of a replay, including all the cars, and pull it into freeplay a couple years ago.  It was semi-successful, but needed a ton more work than time I had available, especially as hacking on this plugin requires a lot of experimentation with no real support in the form of documentation or stability guarantees.  It also felt different enough from this plugin that I eventually abandoned it.
Unfortunately, with Psyonix/Epic continually taking features away from the game, I fear bakkesmod could be next, so I don't have much motivation to continue working on this plugin except for critical bug fixes.
Of course all this code is open source, so if you or anyone else is interested in using it in another plugin, you are welcome to do so in accordance with the license (mostly there as a CYA, though it stipulates you need to post the MIT license text).  And if any changes are sent here as PRs, I will do my best to review & merge & push them to bakkesplugins.com.


Add Lin tests for Base.Buffer

Add Lin tests for Base.Buffer. This is split into two commits to keep track of the functions that can raise exceptions when used in parallel (while they shouldn’t in a sequential use).
Segfaults happened on 5.0.0 in CI, for instance there:
random seed: 255066454
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s Lin DSL Base Buffer test with Domain
/home/runner/work/_temp/cfbbcd46-3b2b-4f6b-b063-35a9068fc155.sh: line 2: 3690717 Segmentation fault      (core dumped) opam exec -- dune exec "$ONLY_TEST" -- -v
[ ]    0    0    0    0 / 1000     0.0s Lin DSL Base Buffer test with Domain (generating)

This is hopefully another symptom of ocaml/ocaml#12103 which was fixed on trunk by ocaml/ocaml#12104.

For some reason, enabled_if seemed to work locally but is not enough to tell dune not to compile the test in CI, so I updated the PR with a workaround using a select ... from. It didn’t seem to work in a test stanza, though, so I unfolded it into an executable and a rule.

(Runs on trunk will break, it was updated to 5.2, ppxlib must follow suite)

I've given these tests a spin on 5.1 and trunk with this branch and cannot reproduce with a repeats count of 50: https://github.com/ocaml-multicore/multicoretests/tree/shym-base-buffer
This indicates that the issue is indeed fixed by ocaml/ocaml#12104.

Closing this for now, as we abandoned the idea of testing Base.
Thanks for the PR and for confirming that this could be useful too! :smiley:


han-transformer not work

i have tried p1_HierarchicalAttention_model_transformer.py with your test function,but the loss dose not converge。

I think you are run on toy task, and loss is decreasing. if you want coverage, you can use real task,

I run p1_HierarchicalAttention_model.py ,its work fine .and i just want to try transformer,but i dosen't work,so i try you test function,but it dose not work too?

thanks

I think transformer work better now. as someone find a bug which was fixed:
https://github.com/brightmart/text_classification/issues/93

hey,that's me . and i just try your transofrmer code.it's work.    but i campare to textcnn, just as the same conclusion in your github(https://github.com/brightmart/text_classification). transformer isn't better than  textcnn ,may be my dataset is not large enough.  But, i think transformer may be more potential. I will  try pre-train like bert and find whether it's help.

good to hear from you. please keep me update.


reactor:multiple delete executor

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

 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.liiiiiibpm 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.

Codecov Report

Merging #482 (a20c3e3) into master (cf5661e) will increase coverage by 0.12%.
The diff coverage is 45.71%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #482      +/-   ##
==========================================
+ Coverage   39.02%   39.14%   +0.12%
==========================================
  Files         145      146       +1
  Lines        9638     9778     +140
==========================================
+ Hits         3761     3828      +67
- Misses       5565     5637      +72
- Partials      312      313       +1

Impacted Files
Coverage Δ

...kg/datasource/sql/exec/at/multi_delete_executor.go
45.71% <45.71%> (ø)

pkg/datasource/sql/exec/at/base_executor.go
39.13% <0.00%> (+1.18%)
:arrow_up:

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

参考这个PR，把at_executor.go文件完善下
https://github.com/seata/seata-go/pull/481/files#diff-84f163a33322f2f1282a877450c2dbe24769b8caf415962d4724f8fbfd90554a

这里加上这些代码：

multi_executor文件改动：

for _, beforeImage := range beforeImages {
    m.execContext.TxCtx.RoundImages.AppendBeofreImage(beforeImage)
}
for _, afterImage := range beforeImages {
    m.execContext.TxCtx.RoundImages.AppendAfterImage(afterImage)
}


Batch Size

I have enjoyed reading your paper, while I was implementing your work on my dataset, I have increased the default batch size from 1 to 8 in base options. And it showed this error:
Traceback (most recent call last):
File "train.py", line 44, in
model.data_dependent_initialize(data,data2)
File "sb_model.py", line 106, in data_dependent_initialize
self.forward()                     # compute fake images: G(A)
File "sb_model.py", line 184, in forward
for t in range(self.time_idx.int().item()+1):
ValueError: only one element tensors can be converted to Python scalars"

Hello, I also tried running this code with batch_size 8, but it didn't work.
The authors said "we trained the model for 400 epochs, with batch size of 1." on their paper.
And i added some for loops, and multi-batch-training works.
However, learning did not proceed normally. Maybe i am wrong.
Or i think this code maybe didn't support multi-batch-size training.

Hi, in this version, the training process is available with only single GPU and batch size of 1.
As the training process is optimized on batch size=1 setting, there would be some problems with larger batch size.
We will upload more generalized version later.
Sorry for inconvenience.


feat(stacks): support automated sync for stacks [EE-248]

Close https://github.com/portainer/portainer/issues/5461


Consider expanding max supply allowed for user-issued assets

Currently at 10^9 precision, the max supply of a user-issued asset is 9999999.99999999. This means UIAs can't map 1-1 to many cryptocurrencies (including Bitcoin, for example). If there's no technical reason for this limitation, we should raise the limit I think.

The reason @bytemaster stated for this limitation is to stay within the 53-bit mantissa of doubles, which eases integration of other languages.  Also, doubles are used at some places in our code.  While my staunch opinion is that a financially oriented product like us should never use doubles anywhere, unfortunately the existing code does sometimes use doubles.
We would have to do a fair amount of auditing to change BTS_MAX_SHARE_SUPPLY, not least because I don't believe anyone anticipated ever changing it.  You could always set precision lower and use bits [1], or resign yourself to never being able to have more than 10 million UIA-BTC.
[1] Many bitcoin related websites use the word "bit" to refer to a micro-BTC, one millionth of a bitcoin.

Note that no state transitions depend on doubles/floats, and any usage of them in the code is purely for convenience and can be changed without a fork. In some cases, they cause user-facing precision issues (e.g. when entering decimals in the API), and very much need to be changed.

Too low priority; shelving for now unless we get some demand for it.


fix: give stats initial values

Otherwise the compiler cannot derive the type and thinks stats.snapshot returns {}

The resulting types:
    /**
     * Returns a clone of the current stats.
     */
    get snapshot(): {
        dataReceived: Big;
        dataSent: Big;
    };
    /**
     * Returns a clone of the internal movingAverages
     */
    get movingAverages(): {
        dataReceived: MovingAverage[];
        dataSent: MovingAverage[];
    };


Sync main branch with upstream opendatahub main

Sync main branch with upstream opendatahub main
Fixes: https://github.com/red-hat-data-services/notebooks/pull/68

merging to keep main up-to date.


Document the "labels' loading" functionality

 Issue by carlos.garcia
Wednesday Nov 02, 2016 at 16:59 GMT
Originally opened as https://git.tk.informatik.tu-darmstadt.de/SPIN/ID2T-toolkit/issues/14

When a pcap file is loaded and a corresponding labels file is found, ID2T states that the labels are loaded.
The purpose of this is not documented.

 Comment by patrick.jattke
Friday Nov 04, 2016 at 00:02 GMT

This functionality is now described in the README.md, see commit feeae3db73 on the development branch.

 Comment by patrick.jattke
Friday Nov 04, 2016 at 00:02 GMT


Update error checking to use type(of:) rather than typecasting

Verify error status using type(of:) is NSError.Type rather than using as? NSError
This was raised due to iOS 16 String extending Error, meaning any Promise will always return as rejected even if the actual promise was successful
fix #185

Note: This code passed all tests as well as the additional tests located below, which stemmed from the same issue and failed prior to my changes when building against iOS 16 with SafariServices.framework connected to the package
https://github.com/google/promises/issues/88#issuecomment-572834083


Enable consumer_group function interpolation in Kafka Input

Address #618 by enabling function interpolation for consumer_group and thus allowing unique consumer_groups per session.

Thanks @DeeJayBro, this implementation looks good but I've been thinking about the implications of allowing interpolations here and there's a few things I want to understand before we proceed, going to follow up in https://github.com/Jeffail/benthos/issues/618

Thanks @DeeJayBro, this implementation looks good but I've been thinking about the implications of allowing interpolations here and there's a few things I want to understand before we proceed, going to follow up in https://github.com/Jeffail/benthos/issues/618

Went with enabling empty consumer groups: https://github.com/Jeffail/benthos/commit/76264dfe8c6afba31b5383093ee97041578fc92b