[EPIC/NG] Fix Treeview

Epic grouping the 2 bugs and one critical performance improvement we need.

Recursive trees cannot be selectable.
Fully dynamic selection from the top of the tree doesn't work.
Every single node is eagerly created, which leads to terrible performance on large trees. (In addition to making it hard to do proper lazy-loading.)

Ok, so we have a plan.
 Depending on the state of the view, on how many nodes the user has expanded, we'd emit a different number of models every time. We cannot have a binding with this level of inconsistency.
So we're going to drop it, and that's a breaking change. This means that we would also drop <clr-tree> itself (unless we missed something), so we would only have a <clr-tree-node> component. The selection binding will now just be a simple [(clrSelected)] boolean binding on the tree nodes.
To obtain a "Select all" or "Select none" functionality, all the consumer will need to do is set selected to true or false on the root node. This will recursively select or unselect every other node.
Proper lazy-loading (#900)
We will extract the *clrIfExpanded from Datagrid: https://github.com/vmware/clarity/blob/master/src/clarity-angular/data/datagrid/datagrid-if-expanded.ts
This way, nodes that are not displayed simply do not exist, and their selection state doesn't matter. It will also come with all the advantages described in #903, including good performance for trees with tens of thousands of nodes.
There are two ways to use such a directive:
<clr-tree-node>
    A
    <clr-tree-node *ngIfExpanded>B</clr-tree-node>
    <clr-tree-node *ngIfExpanded>C</clr-tree-node>
</clr-tree-node>

<clr-tree-node>
    A
    <ng-container *ngIfExpanded>
        <clr-tree-node>B</clr-tree-node>
        <clr-tree-node>C</clr-tree-node>
    </ng-container>
</clr-tree-node>

In both cases, B and C will be instantiated and displayed when the node A becomes expanded.
When loading new nodes asynchronously, the spinner will be displayed using the new loading pattern, which is both less code for us and much easier to setup for the consumer.
Allow recursion (#487)
When the consumers don't know the depth of the tree, their only recourse is to create a recursive component: <my-recursive-node [node]="node"></my-recursive-node>.
The template of this component would typically look like this:
<clr-tree-node [clrLoading]="loading">
    {{node.label}}
    <ng-container *clrIfExpanded>
        <my-recursive-node *ngFor="let child of node.children" [node]="child"></my-recursive-node>
    </ng-container>
</clr-tree-node>

But doing so means the child nodes are hidden from @ContentChildren, which we use to propagate selection up and down. So we need to drop @ContentChildren for child detection and switch to a simple register/unregister pattern:
ngOnInit() {
    this.parent.register(this);
}
ngOnDestroy() {
    this.parent.unregister(this);
}

Dependency injection follows the DOM tree and crosses components, so this will allow us to support this recursive pattern.
The main issue with this is that we lose the ability to keep track of the order of children. But by dropping the top-level selection binding, we now don't have a single part of the treeview that relies on this order.
Why group them in a single task?
Each of these depends directly or indirectly on the others, and they require a big enough refactor of the tree (to make it simpler) that splitting them would just mean double the work for each step.
In addition, having a single, breaking change refactor of the tree will mean a single transition for the consumers. Sure we're ripping the band-aid, but having breaking changes on the tree every few weeks would be far worse.

Guys, could you please give some estimation when do you expect this to be finished.
Thanks in advance!

@bashibozuk
As a team, we avoid giving time-based estimations because our priorities change in real-time.
What I can tell you about this particular issue is:

It is a top priority and is currently in progress.
It will introduce a breaking change that will kick off the work we are planning to do for 0.10.0.
When this item is completed, we will most likely release a 0.10.0-alpha so that people will have access to the fix and not have to wait for the other work for 0.10.0 to be completed.

Right now, this issue is at the top of our list and is currently being worked on. When the fix is submitted, we will do our best to make available to you.
Thanks for checking back in on this.

Thanks for the fast response, good luck with the implementation!


[Bug] Consistent resource identification in API deisgn

Bug Description
When patching/updating some resources, some endpoints require id to be in the url while some require to be in the body.
e.g., update an existing a DNS record:
https://api.omg.lol/address/foobar/dns/2857074

update an existing status log:
https://api.omg.lol/address/adam/statuses/

with Id in the body.
It'd be much more consistent and predictable if we use the same way to identify a resource (independent of the operation). I personally prefer to put it in the url but other options are fine, as long as it's consistent across the system.
Steps to Reproduce
No response

Agreed. Will get this updated soon!

A bit more consistency with types would be appreciated also, for example GETing DNS records produces a list of records as:
"dns": [
    {
        "id": "2857074",
        "type": "A",
        "name": "foobar",
        "data": "<IP_ADDRESS>",
        "priority": null,
        "ttl": "3600",
        "created_at": "2022-11-26T04:30:13Z",
        "updated_at": "2022-11-26T04:31:33Z"
    }
]

Where the ID and TTL are returned as strings, but the response_received.data when POSTing returns ID and TTL as numbers:
"response_received": {
    "data": {
        "id": 41949560,
        "name": "testttle.elliott",
        "content": "example.com",
        "ttl": 60,
        "priority": null,
        "type": "TXT",
        "created_at": "2023-02-06T16:18:34Z",
        "updated_at": "2023-02-06T16:18:34Z"
    }
}

@ejstreet agree. Also later on when doing the overhaul, wondering if we can simplify the DNS model a little bit. The data_sent and response_received feels like the server is sending stuff to an upstream service and directly replicate the data to the client lol. Just a bit redundant.
And yeah the data and content naming thing would look better if unified. Cant recall if I mentioned this in another issue.


Request: Can we include unit testing skeletons with recipe generators?

I was thinking it might be useful to add the generation of unit test files for their associated cookbook files. If a person was to generate a recipe named 'foo', they would get the recipe file along with its associated spec file like so:
Compiling Cookbooks...
Recipe: code_generator::recipe
  * template[/opt/working/chef/cookbooks/foobar/recipes/foobar.rb] action create
    - create new file /opt/working/chef/cookbooks/foobar/recipes/foobar.rb
    - update content in file /opt/working/chef/cookbooks/foobar/recipes/foobar.rb from none to e3b0c4
    (diff output suppressed by config)
 * template[/opt/working/chef/cookbooks/foobar/spec/recipes/foobar_spec.rb] action create
    - create new file /opt/working/chef/cookbooks/foobar/spec/recipes/foobar.rb
    - update content in file /opt/working/chef/cookbooks/foobar/spec/recipes/foobar.rb from none to e3b0c4
    (diff output suppressed by config)

This would include modifying the chef generate cookbook command to add the spec/recipes folder and a default_spec.rb.

This was released in version 0.3.6 as part of #266


do not exit prematurely when using -manifest-dir

If using -manifest-dir, check images across all manifests for
emptiness before exiting with a NOP.
/assign @justinsb

/lgtm


Warden 0.5.1 doesn't start M2 services

Platform:

OS: macOS Catalina 10.15.4 (19E287)
Docker Desktop: 2.3.03

mutagen version: 0.11.4
warden version: 0.5.1

Steps to Reproduce:

Create & start the services from new M2 environment
warden env-init w2 magento2
warden sign-certificate w2.test
warden env up -d [This have problem with mutagen]

`Connecting traefik to w2_default network
Connecting tunnel to w2_default network
Starting w2_elasticsearch_1 ...
Starting w2_rabbitmq_1      ...
w2_redis_1 is up-to-date
Starting w2_rabbitmq_1      ... done
Starting w2_db_1            ... done
Starting w2_elasticsearch_1 ... done
w2_nginx_1 is up-to-date
w2_php-debug_1 is up-to-date
Starting w2_varnish_1       ... done
Installing agent...
Error: create failed: unable to connect to beta: unable to connect to endpoint: unable to dial agent endpoint: unable to install agent: unable to invoke agent installation: exit status 1
ERROR: Command mutagen sync create -c "${WARDEN_DIR}/environments/${WARDEN_ENV_TYPE}/${WARDEN_ENV_TYPE}.mutagen.yml" --label "warden-sync=${WARDEN_ENV_NAME}" --ignore "${WARDEN_SYNC_IGNORE:-}" "${WARDEN_ENV_PATH}${WARDEN_WEB_ROOT:-}" "docker://$(warden env ps -q php-fpm)/var/www/html" on line 43 failed with exit code 1`

I no longer have this issue. This is what I did and that seems resolved this for me.
Removed warden completely, installed again and followed recreating or re-initing environment file for existing M2 project and then now my local warden toolchain based dev environment works fine.

same problem, no solution at the moment

@fballiano The error in the OPs post indicated something is stuck with Mutagen. If that's the error you're getting, I'd try the solution I posted over here on an older issue: https://github.com/davidalger/warden/issues/26#issuecomment-546543118 If that doesn't work and all else fails, reset Docker Desktop, kill any mutagen processes and then spin things back up.

sadly it didn't work:
fab@MacBook-Pro-3 ~/P/prezzemolo (devel) [1]> warden sync start
Installing agent...
Error: create failed: unable to connect to beta: unable to connect to endpoint: unable to dial agent endpoint: unable to install agent: unable to invoke agent installation: exit status 1
ERROR: Command `mutagen sync create -c "${MUTAGEN_SYNC_FILE}" --label "warden-sync=${WARDEN_ENV_NAME}" --ignore "${WARDEN_SYNC_IGNORE:-}" "${WARDEN_ENV_PATH}${WARDEN_WEB_ROOT:-}" "docker://$(warden env ps -q php-fpm)/var/www/html"` at /usr/local/Cellar/warden/0.10.1/commands/sync.cmd:45 failed with exit code 1


package-lock.json won't update

Deliberate duplicate of https://github.com/TriPSs/conventional-changelog-action/issues/8 (closed automatically)
I'm about to rewrite my whole action without changelog because of this. Is there a reason why package-lock.json is not being taken into consideration when bumping version?
I'll gladly work on a patch and submit a PR if I'm given the green light.

Hi @andresmatasuarez, as far as i'm aware the conventional-changelog cli also does not do this. Could you clarify why this is needed?
Also you are welcome to create a PR that checks if there is a package-lock.json file, or maybe update the version-file to accept comma separated paths so you can add your package.json and your package-lock.json

Hi @TriPSs, not bumping version in both package.json and package-lock.json inevitably results in a discrepancy between the two which causes tools like npm ci not to work, which my team rely heavily on in our project, as do a lot of other teams as well, I'd say.
I like the comma-separated proposal so I'd go with it. I'll have it ready for review no later than tomorrow :)

This is released in v3.2.0 :tada:


Running the contributed command:'markdown.showPreviewToSide' failed.

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:26:50.689Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions: none

Running the contributed command:'markdown.showPreviewToSide' failed.
Steps to Reproduce:

Install visual studio code to a fresh new W10 pro PC
reboot
open vscode
open a .md file
ask for the preview with Ctrl+Shift+V
Then I've the message "error: Running the contributed command:'markdown.showPreviewToSide' failed." and it's not possible to show the preview

From my experience, network files (or symlinked files to network) do not work!

Effectively it was that. Moving the .md file to the drive of my PC works.
Thank you
regards

I see the error when trying to preview an .md file in my "Documents" folder - turns out that folder had been redirected to a UNC by GPO.  Moving to a local folder fixed.


OpenSeaDragon doesn't send cookie to Cloud CDN

OSD doesn't send cookie with ajax request even when ajaxWithCredentials is set to true and crossOriginPolicy set to "use-credentials". Any ideas on what to do?
I am using signed cookies to access Google Cloud storage bucket.
Thanks,
-Marcos

I don't think they work across domains. I am working with Google and I will
post the results.
Thanks,
-Marcos
On Mon, Jun 10, 2024 at 9:20 AM Ian Gilman @.***> wrote:

Is this for the tilesource or the tiles? If the latter, do you also have
loadTilesWithAjax turned on?
I don't know much about working with cookies. Aren't they just
automatically applied? Do they work across domain?
—
Reply to this email directly, view it on GitHub
https://github.com/openseadragon/openseadragon/issues/2540#issuecomment-2158806976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFLR7JJ3CAALSTH5TENJDZDZGXG6LAVCNFSM6AAAAABI7CVYPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYHAYDMOJXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Number of contributions does not update until the app is restarted

Summary:
After uploading an image, the number of contributions (the number in the bracket on the "Contributions" tab) does not increase. It increases only after the app is reopened. It doesn't update even after switching to and back from some other fragment.
Steps to reproduce:

Upload an image.
Notice that the number of contributions does not increase.

Device and Android version:
Android version 7.1.2 (API 25)
Would you like to work on the issue?
Yes

@ShridharGoel Are you currently working on this?

No

@albendz can I start working on it?

@madhurgupta10 Yes it is yours :-)

@nicolas-raoul I am unable to figure out why if (!ConfigUtils.isBetaFlavour()) {             setUploadCount();  }
is added in onCreateView in ContributionsFragment.java


Adding video

Please refer to the code of conduct : link

Please check if the PR fulfills these requirements :clipboard:

[x] The commit message follows our guidelines : link
[x] Docs have been added/updated (for bug fixes/features)
[x] A new branch is created for the change(s) and the changes are not committed on the main or master branch
[x] No credentials or secrets are committed

What kind of change does this PR introduce? (Bug fix, feature, docs update, refactor, chore, test, config) :information_desk_person: :scroll:
Docs update

Added Linked account flow video

Fixes :wrench: #103

Checklist for the current PR :page_with_curl:

[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

Build finished.

Build finished.


Allow users to specify the Node.js source path

It would be helpful to have an environment variable that users can set to specify the base path of their Node.js source code. This would be similar to the BP_GO_TARGETS environment variable that's supported in the Go Buildpack.
Having separate environment variables per language family would be useful for building polyglot apps such as a Go backend and Node.js frontend that might have a structure similar to the following:
src
  -> frontend
      -> package.json
      -> my frontend code
  -> go.mod
  -> main.go

Just to clarify, you want to be able to specify a root that is not the WorkingDir, you aren't looking to be able to pass a list of dirs to be built?

Just to clarify, you want to be able to specify a root that is not the WorkingDir

Right. Exactly this. So if I had the following filesystem structure:
workspace ->
   src
      -> frontend
          -> package.json
          -> my frontend code
      -> go.mod
      -> main.go

and my app is in the src directory, I'd want to be able to build my app like:
pack build polyglot-app --path ./workspace/ --env BP_INCLUDE_FILES="frontend/" --env BP_NODEJS_ROOT="frontend/" --buildpack gcr.io/paketo-buildpacks/node-engine --buildpack gcr.io/paketo-buildpacks/npm-install --buildpack gcr.io/paketo-buildpacks/go-dist --buildpack gcr.io/paketo-buildpacks/go-mod-vendor --buildpack gcr.io/paketo-buildpacks/go-build

@arjun024 sounds like this will get closed out by implementation of https://github.com/paketo-buildpacks/nodejs/issues/305, right?

@kvedurmu Doing a little clean up of issues. This problem should be solved by https://github.com/paketo-buildpacks/nodejs/issues/327


[terra-clinical-data-grid] Update Cell to have role undefined for non interactable cel

Summary

This Pr fixes the accessibility issue for selectable empty cells by making the role for cells as undefined.

Closes #703
Deployment Link

https://terra-clinical-deployed-pr-#.herokuapp.com/
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

Ok, so I messed up and merged this PR before it was ready, so this is re-opening it to get a11y Review
https://github.com/cerner/terra-clinical/pull/706

@scottwilmarth could you take a look at this?

I reviewed the fix and will solve the immediate need.


systemd deployment is broken

Good day.
After 1.5.3 release, I could not help but notice that converging on systemd OSes (> Centos 7) fails.
I suspect that this PR https://github.com/schubergphilis/nrpe/pull/31 should have changed something very important.
A piece from systemd log:
Jul 20 10:41:21 master0-el7 nrpe[14498]: Unable to open config file '/etc/nagios/nrpe.cfg' for reading
Jul 20 10:41:21 master0-el7 nrpe[14498]: Config file '/etc/nagios/nrpe.cfg' contained errors, aborting...
Jul 20 10:41:21 master0-el7 systemd[1]: nrpe.service: control process exited, code=exited status=2```

New release 1.5.4 is on its way and I hope it will fix also that issue.. Waiting for a
@anuriq Can you please re-try when 1.5.4 is available?

@anuriq v1.5.5 is released (due to my mistake, it is not 1.5.4 unfortunately). Can you please try when you have time?

will check now

looks OK. will confirm for sure after heavy converges. but let's close this for now.


Lefthook for Dart

Hello! I want to use lefthook with Dart. I can make a PR for that, but I have a couple of questions.
Do you publish all packages manaully? Who will publish new versions to pub (Dart package registry)?
P.S. Thanks for lib, it is really great! 😻

What do you think in general about this idea?

Hello @igorkamyshev, that's a great idea! You can do i simple wrapper and maintain it. For example like this wrapper for pip:
https://github.com/life4/lefthook
We can add your wrapper in our documentation. Is it possible for you?

Do you publish all packages manaully?
Yep, I publish it with goreleaser + npm publish + rake.

Nice, thanks. I will do it, and send notification for you.

Hello! I wrote Dart-wrapper for Lefthook. Can you add that link to official repo?
https://github.com/drt-pkgz/lefthook
Also, I have one question.
In repare-commit-msg hook I found next text:
#!/bin/sh

if [ "{$LEFTHOOK}" = "0" ]; then
	exit 0
fi

# lefthook_version: d41d8cd98f00b204e9800998ecf8427e

cmd="lefthook install"

if lefthook >/dev/null 2>&1
then
	exec $cmd
elif bundle exec lefthook >/dev/null 2>&1
then
	bundle exec $cmd
elif npx lefthook >/dev/null 2>&1
then
	npx $cmd
fi

cmd="lefthook run prepare-commit-msg $@"

if lefthook >/dev/null 2>&1
then
  exec $cmd
elif bundle exec lefthook >/dev/null 2>&1
then
  bundle exec $cmd
elif npx lefthook >/dev/null 2>&1
then
  npx $cmd
else
  echo "Can't find lefthook in PATH"
fi

I guess, it may be confusing, if folk uses this tool from Dart, but Lefthook try to use npx, ruby-stuff, etc. Can you provide API to set this commands implicitly by first install? I mean something like this:
lefthook install --install-command="pub global activate lefthook"

Yep, I can do that.
Right now I have semi-vacations days, so it took a while for this feature. Sorry about that.

It's okay, have a nice days-off.
Please, ping me, when you do this changes.

Yep, no problem.

Hi. What about this enhancement?

Sorry for so long, planned on following week.

Hello! Sorry for interruption. Do you plan make this enhancement? If you don't want to do this, it is okay, please just tell me about it 🤗 (because, this issue in my Things app and I can't just close any task without reason 😂)

I will! I just really have a very limited time :(
Or if you want to take this(it wiould be great) I can guide you through the code step by step.

@Arkweid yep. I am really bad in Go, but I want try 🤓 Please, provide instructions for me.

Ye @igorkamyshev ! Lets try this flow:

Add new config key, for example bin_path, like here:
https://github.com/Arkweid/lefthook/blob/master/cmd/root.go#L16-L23
Add new available flag for command install command lefthook install --bin-path=./node_modules/@arkweid/lefthook
https://github.com/Arkweid/lefthook/blob/master/cmd/root.go#L59-L60
You can find the reference in cobra repository for that part:  https://github.com/spf13/cobra
We using it for avoid CLI boilerplate.
Automatically add it into lefthook.yml config file when user provide bin_path flag. Template here:
https://github.com/Arkweid/lefthook/blob/master/cmd/install.go#L73-L108
Now we have all required things to add a hooks with desired bin_path. Template for that here:
https://github.com/Arkweid/lefthook/blob/master/cmd/add.go#L73-L84
Make the same changes for _windows.go files.
(optinal) If you feel a power - write the tests!
...
Profit!

@igorkamyshev hi! Is this issue still relevant for you by any chance?

I've moved the resulting feature request to a separate issue so that we can close this.


Add reviewers names in post meta ?

Each reviewers can add a commit in order to add himself in the post reviewers list metadata (eg: after authors). Reviewing a post is a contribution imo.
What do you think ?

it's a good idea :+1:

it would promote reviewing posts, so it's a good idea (it cannot be automated right ? because of squashes, didn't it?)

:+1: for me as well

Completely agree.

agree

so, we all agree, let's just say that when we post something, it does not have to be a unique commit, we will now squash cleverly to let people that have contributed to the writing of the post in the history

No I mean, we should add them. A reviewer can not push any commit.

i don't know how we can exactly handle this to mirror the reality but if we can display on the post all people that have commited or talked in the PR but that will not enhance the reviewer importance compared to the guy that just talks in the PR

i don't know how we can exactly handle this to mirror the reality

By hand, like the "authors" field.

The author(s) add themselves the reviewer names.

oh! by hand! ok! just let's put them in a reviewers field, the author has the be the star of the post :stuck_out_tongue:

Ok, so, i began to work on it, wanting to replace everything in each post with a reviewers field, but that is not really what we want. Today, the problem is that someone who talks on a PR helping the author will never be displayed as a contributor although the one that "just" fixes a mistake with the "edit" link on the post page will.
So i decided to keep the contributors like it is right now, and just merge with it the reviewers field.
Just look at #635 and you will understand.


Migrate Sunstone to React/Redux

Description
Sunstone needs an update to get rid of technology debt
Notes:

Existing FireEdge backend (with already a nodejs xmlrpc wrapper of the OCA) will be reused. Sunstone will be an app of FireEdge
Pool management most likely on the backend
Frontend will use pagination + FTS search (Sunstone won't feature search with sqlite backend)

Additional Context
All current functionality needs to be present. Probably new and old Sunstone will coexist for at least one release.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

[ ] bundling the font with the software (#4061)


err TypeError: Cannot read property 'text' of undefined

Hi I met this error when using addTable. Below is my code:

This is 'tab':

all the options object is 'options': { fill: '6f9fc9' }
and I got this error:

I couldn't figure it out. Please help me. thanks!

Hi @ninas880025 ,
Thanks for reporting this, and extra thanks for the stack trace! 👍
Those few undefined entries are likely the cause.
I've updated the code to avoid this error, so please verify if you can.

ok thanks! I use npm to install pptxgenjs so now I can renpm install right?

Hello,
I dowloaded new version (1.6.0) and then I found this error:
'Cannot read property 'opts' of undefined '
I didn't change my code, just download it and run.

can you help me with this? thanks!

Hi @ninas880025 ,
Thanks for the feedback!
I've fixed the issue above and found another place where null/undefined values would cause errors and fixed that as well.
Please let me know how the newest version works.

Hi, the newest version works! thanks so much!

Hi Sorry,
Could you help me to push this version to npm install? Cause my project need to use npm install to grab this version. thank you so much!!!


[Excel] (Custom Functions) Adjust custom functions overview article

This PR:

Adds a shared runtime note to the CF overview article, to address internal work item 3868818.
Moves the Known Issues section from the Overview to the Troubleshooting article, to match the Excel information architecture: Troubleshooting Excel Add-ins.

Docs Build status updates of commit b293f1e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 7e88908:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 f756e87:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 f756e87:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 1a22158:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 1a22158:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 a834a3c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 a834a3c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 1ef64f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 1ef64f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 36e7828:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
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 36e7828:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


fix(helm): Don't discard pre-releases upon index merge

Previously, when attempting to merge a new version into an existing
index.yaml which includes overlapping release and pre-release versions
(such as 0.1.0 and 0.1-dev), the merge would silently discard the
pre-release version from the newly merged index.
Fix this by performing an exact semantic comparison of the versions from
the provided index file.

I'm not sure if the current behaviour was specifically intended. In some sense, having a merge of {1.2.0} and {1.1-dev, 1.1.0, ...} should in theory end with the union of all input versions so that's the main argument point for the current approach. This also has the implication that over time the index will collect potentially a large number of pre-release versions.  Definitely open to discussion on this.
I can probably work around this issue on my side, but at the same time if upstream Helm worked a bit more predictably in this regard in future, then other projects could potentially benefit as well and avoid having their own variations of workarounds.

@joestringer We usually want an issue describing the problem and how you got it. Would you mind writing up an issue with all kinds of details.
I've not replicated it yet but this does look like a bug from the description.

@joestringer This is failing tests. Can you please correct the problem.

@joestringer This is failing tests. Can you please correct the problem.

CircleCI doesn't seem to provide a way to see your test results without signing up for their service, so I tried running the tests locally. I found unit test failure with the search and fixed that in the commit. I also hit a bunch of errors like this which I'm guessing I can ignore as they rely on some CI credentials:
--- FAIL: TestDependencyUpdateCmd (0.22s)
    dependency_update_test.go:41: Listening on directory /tmp/helm-repotest-463555852
    server.go:164: error logging into registry with good credentials

CircleCI doesn't seem to provide a way to see your test results without signing up for their service, so I tried running the tests locally. I found unit test failure with the search and fixed that in the commit. I also hit a bunch of errors like this which I'm guessing I can ignore as they rely on some CI credentials:
--- FAIL: TestDependencyUpdateCmd (0.22s)
    dependency_update_test.go:41: Listening on directory /tmp/helm-repotest-463555852
    server.go:164: error logging into registry with good credentials

Hmm, looks like circleci is still not happy. Is it possible to run the full testsuite locally? I have just been trying go test ./....

Hmm, looks like circleci is still not happy. Is it possible to run the full testsuite locally? I have just been trying go test ./....

make test will run the full test suite. In this case, make test-style is failing due to linting errors.

make test will run the full test suite. In this case, make test-style is failing due to linting errors.

Looks like it passed this time, thanks for the pointers.

Looks like it passed this time, thanks for the pointers.

@mattfarina I believe I've addressed your feedback, let me know if you want any further changes.

ping @mattfarina

Hi there. Checking in to see what's the status here and how you want to proceed. Should we close this, or is there something that we should still address here?

It sounds like there's no appetite upstream for making the merging behaviour consistent for versions that do not match semver v2 (ie merging v0.1-dev into an index with v0.1.0 results in the same output as merging v0.1.0 into an index with v0.1-dev), so we can just close this out.
If anyone discovers this PR in future, the simplest way to resolve this would be to ensure that your project uses Semantic Versioning 2.0.0 correctly.


Integrate features from @andreypopp

Based on https://gist.github.com/andreypopp/482c78a992acd3de70793bde62584392
Sessionless/unmanaged nvim instances per window (without running dntw)
Autoselecting tmux panes

@andreypopp Worked the ideas from your gist into a hybrid; I'm curious for your thoughts!


Added debug postfix d

to have separate lib names for debug and release build.
Otherwise the install option of cmakes overwrites the former installation and it is difficult to have debug and release version of the library at the same time.
As mentioned in issue #122

Can one of the admins verify this patch?

ok to test

Just to mention that it can be injected externally, and it's not very good practice to hardcode d postfix.


feat: add create command

test robot

@Lanture1064 test failed

test robot

robot failed...

Codecov Report

Merging #5 (4a2720b) into main (9abf768) will not change coverage.
The diff coverage is n/a.

: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            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           14        14
=========================================
  Hits            14        14

@Lanture1064 too many commits ...


parametrize ansilbe_lint

Issue Type

Feature request

Desired Behaviour
It would be good to  have option to disable some tests , add -x SKIP_LIST parameter to ansible_list.
I  have role where I need to use actioncommand to run service. This triggers [ANSIBLE0006] service used in place of service module and the automated tests failed.

Ansible lint offers this ability via skip_ansible_lint.  This won't be added to molecule, but molecule v2 has the ability to pass options CLI options to the linter.  Molecule v2 will not be out for another few months.

Thanks, this solved my problem. ;)


increase stop timeout to 30 seconds

Increase the timeout during container stop before shooting with SIGKILL.
Users have repeatedly reported that they are hitting the timeout, in
particular on small VMs with low resources.  Setting it to 30 seconds
will decrease the chance of hitting the timeout.
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>@mheon @giuseppe @baude @rhatdan ... this could be a controversial change as it also increases the time to stop misbehaving containers. These could still be silenced with podman-kill though.

I think misbehaving containers are going to be a problem, here... A lot of
programs do ignore SIGTERM. And there's no timeout flag to podman rm, so
you'll end up waiting the full 30 seconds on a sleep container to rm -f.
This seems like a good candidate for containers.conf so we don't need to
unconditionally up the timeout.
On Fri, Feb 14, 2020, 09:05 OpenShift CI Robot<EMAIL_ADDRESS>wrote:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: vrothberg
https://github.com/containers/libpod/pull/5211#
The full list of commands accepted by this bot can be found here
https://go.k8s.io/bot-commands?repo=containers%2Flibpod.
The pull request process is described here
https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process
Needs approval from an approver in each of these files:

OWNERS https://github.com/containers/libpod/blob/master/OWNERS
[vrothberg]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/pull/5211?email_source=notifications&email_token=AB3AOCBP4DAVWXKYBM5WWBDRC2QJHA5CNFSM4KVIVHE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELZED6A#issuecomment-586301944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3AOCCFTKWYIGOELRRNDMLRC2QJHANCNFSM4KVIVHEQ
.

This seems like a good candidate for containers.conf so we don't need to
unconditionally up the timeout.

That seems much better than increasing the timeout here. @rhatdan WDYT?

I'm way in on @mheon's thought of moving this to the conf file.

Fine with me.  Please open a PR with containers/common to add the field.  If we could ever figure out the one last blocker, we could get containers.conf merged into libpod.


Display graph per GPU in System Information

Feature request: Please display one graph per GPU in System Information window, similarly as Windows built-it Task Manager does it.
I have Lenovo ThinkPad E490 laptop with two graphics cards, one is "Intel(R) UHD Graphics 620" the other is "Radeon (TM) RX 550X". It would be much more convenient when developing a 3D application.
In attachment there are two screenshots, one from Process Hacker version 3.0.3521, other from Task Manager.

Duplicate of #65
The issue has been the difference between Windows 10 and earlier versions. Windows 7 and 8 don't have the same LUID identifiers needed for separating the devices across reboots.

Do you mean that LUID identifiers is not unique for GPU adapters on Windows 10?

@lovelifezhenzhen
They're too unique and change with the wind. Reboot and they change, sleep and they change, update the driver and they change, execute pnputil /restart-device and they'll change, install windows updates and they'll change, connect a new monitor  etc...
This is the complete opposite to LUIDs for other devices such as network adapters where the LUID only changes when you change the device socket/controller and they'll remain static across reboots and device changes. You can easily cache the NetLuidIndex and reuse later without it being invalidated unless its a different device and that's how PH is able to show adapter utilization for specific adapters because we cache NetLuidIndex and reuse it when calling functions like GetIfEntry/GetIfEntry2 with InterfaceLuid and easily querying the specific individual adapter utilization based on your preferences.
You can't do that with graphics devices and D3DKMTOpenAdapterFromLuid and are forced to constantly poll and enumerate devices which is a major performance problem especially since we still support Windows 7 and 8 which don't have the concept of graphics LUIDs... So we cache it anyway but you'll run into the problem where updating the nvidia driver invalidates the LUID and prevents PH from showing utilization without restarting the program.
The other more recent and very major problem is that Microsoft completely broke the D3DKMTQueryStatistics function on recent versions of Windows 10 forcing you to use performance counters which are massive string arrays (parsing string arrays are a major performance issue in themselves) and cannot be easily integrated with the existing code while also removing the ability to query individual processes/adapters.
It's a shitshow... I think this was intentional by Microsoft to stop third party software from correctly showing GPU usage like Process Explorer and Task Manager... I really can't see any other reason for so many stupid design changes or why they're broken these functions 5 times in 3 years - so much for Windows backwards compatibility 🤷♂

It seems to be less opening!
Do you know the detailed version of Windows 10 that broke the D3DKMTQueryStatistics function?

Do you know the detailed version of Windows 10 that broke the D3DKMTQueryStatistics function?

20H1 | May 2020 Update | 10.0.19041
It'll only return partial statistics and only for some processes. For example it'll report the correct usage until you start playing any graphics intensive games then both Process Explorer and Process Hacker report 1% utilization while Task Manager will report the correct utilization > 90%


Create L4S3RE

Measured manually and averaged the value over a 1 minute period because the power oscillated based on the music streamed

Thanks for the PR.
All other profiles for smart speakers are made by streaming pink noise. This is also used by the measure tool.
This audio file is being used: https://assets.ctfassets.net/4zjnzn055a4v/5d3omOpQeliAWkVm1QmZBj/308d35fbb226a4643aeb7b132949dbd3/g_pink.mp3
Pink noise gives a good representation across the frequency spectrum and also makes the measurements reproducable.
So would be preferable if you could redo the measurements. 15 seconds on each level is enough to get the measurements.

@malorezzanzo64 Did you see my comment?

@bramstroker Yes, I read. As soon as I have time I'll redo the calculation


Use of depreciated feature in ggplot2 by occAssess

Hello! Many thanks for creating this package, it's very helpful in my current research. While plotting a graph from an object created by assessSpatialBias, I get the following warning message:
The <scale> argument of guides() cannot be FALSE. Use "none" instead as of ggplot2 3.3.4.
i The deprecated feature was likely used in the occAssess package.
Please report the issue to the authors.
Thought I should let you know :)

Think I found the problem in the assessSpatialBias function in this bit:

Presumably should be updated to:

Thanks @ms-sherbert! Looks like you've found the fix. Great to hear you've found the package helpful


Problems with hidden automesh

Hi Martin,
I am trying to build up a mesh network out of a TP-Link Router with a hidden network and multiple D1 minis. Flashing the D1 minis works as expected and performance is perfect. But when I enable automesh (it does not matter if I do it via CLI or UI) the status LED just turns permanently on which according to the readme states that it is not connected to a AP...
Any help would be much appreciated.

I'm sorry, just messed up my config and didn't realize it^^

Sorry that I have to reopen this issue...
I configure my D1 minis like so (telnet):

set ssid [SSID_router]
set password [PASSWORD_router]
set speed 160
set am_sleep_time 0
set am_scan_time 0
set ssid_hidden 1
save
quit

until this point, everything works fine but of course there is no mesh and the D1mini makes the wifi visible.
When I do this config via telnet, the D1mini flashes its light every second and after a reboot, it just just goes to a static light.

set ssid [SSID_router]
set password [PASSWORD_router]
set speed 160
set am_sleep_time 0
set am_scan_time 0
set ssid_hidden 1
set automesh 1
save
quit

So...
Automesh is working when SSID is visible
Repeating a hidden SSID is working but makes it visible
Creating a hidden mesh around a hidden SSID is not possible...
Kind regards

Okay, think, I can explain: Automesh does a manual scan to find the AP and the other nodes. In this scan it doesn't see the hidden SSID and thus it fails. Guess, the only workaround would be an intermediate ESP node as fixed level 1 router with a non-hidden SSID. After this, the other nodes will mesh again.

Okay, sounds logical. So this is the part "AUTOMESH_LEARNING" , is it?
Is there any chance that I could hardcode a hidden SSID with WPA2 password into the firmware?

What exactly do you mean? The complete automesh hidden? That won't work. Need the scan for finding the possible/best route. The mechanism that allows for connecting to a hidden SSID is "Probe request/Probe response". These are management frames and the driver doesn't allow to use them in a user program.
You should be able to configure one (or more) ESP with the router firmware (without automesh enabled) to act as an intermediate between the hidded TP-Link AP and the automesh nodes. They may offer a different SSID than the TP-Link AP. So the TP-Link AP stays hidden and the automesh is visible.

Thanks for your response, yeah I wanted a complete automesh hidden. But your explanation makes perfect sense.
Maybe you could add this to the README in case others have the same thoughts as I did.


DBZ-941 Add connectionCreated template method in HistorizedRelational…

…SnapshotChangeEventSource
The method allows to define steps which have to be taken just after the
database connection is created (e.g. setting snapshot isolation level).
By default no operation is executed.

@jpechane I don't understand. The last build's logs shows:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 12:29 min
[INFO] Finished at: 2018-10-24T18:39:45+00:00
[INFO] Final Memory: 81M/1130M
[INFO] ------------------------------------------------------------------------

Besides this, other builds always fail due to (with different images):
Unable to pull 'debezium/postgres:9.6-devel' : received unexpected HTTP status: 503 Service Unavailable (Internal Server Error: 500)

@grzegorz8 It's passed now!

@grzegorz8 Applied, thanks.


Windows version clarification

With the "audio-cast-1.0.0-win32-ia32" name for the file, I came to the conclusion from the win32 portion that this would work on Windows XP. When I tried to run it I received a dialog box stating that the file was not a windows application. So, perhaps a listing of which windows operating systems this will work on would be helpful. From what I have read here in the issue section it appears that probably one would need at least Windows 7 to use this. Thanks again.

It should work on Windows XP, but I absolutely have not tested it..

I have a few more XP computers so I will try them and see what happens. I tried a couple of things on the one I was using but I was still getting a " not a valid windows application" dialog box.

Well I am still getting the same message. However, when I tried to set up the xp computer for the chromecast audio with google's installer I received and error message and have subsequently discovered that it needs a minimum of Win7 to run. I do not know if your app was meant to work without the chromecast setup from google or not. If so, then it is a definite no go for xp.

It was probably just compiled for 64bit instead of 32bit, to be honest. Are you running 32bit windows?

Yes, I am running XP 32bit. When I had these issues I tried it on my Win10 computer, which then ran save for the issues I noted in issue 11. Is this intended to work without the chromecast app?

Yeah, it works without any chromecast applications. It's 100% stand alone.

Is there a 32-bit version? I tried 0.2.5 but get "not a windows application". I am running WinXP!? (still)

This is never going to work with XP. As I looked into it some more I discovered that XP does not have the ability to capture the audio stream. So a 32bit version is moot. I don't remember the exact technical issue with it but I gave up on getting this running on XP when I came across the information. Probably should have posted what I found out here then. Sorry, my bad for not doing so.

I'm pretty sure that your XP system's ability to capture audio output is completely dependent on the sound card/driver. What's yours? Realtek?
See if you can find anything here: Start > (Settings) > Control Panel > Sounds, Speech and Audio Devices > Sounds and Audio devices > Audio tab
from: https://answers.microsoft.com/en-us/windows/forum/windows_xp-hardware/is-there-any-way-to-enable-stereo-mix-in-windows/e56077bf-e2ca-49e2-adc2-abaea9617aa3


Update Database

Hi,
is it possible to get an updated database at some point in the near future?
Or is there a way how to update/create a database by ourselves?
Thanks in advance :)

Thanks for reaching out. A database update is overdue. At minimum, I'm working on a writeup document for how to make your own eukdetect database - should be released within a month or so.

Hi,
Thank you for this really nice tool. We are currently working with a yeast that is not included in the current version of the database. Consequently, we are eager to learn how to create our own database :D

Thanks for reaching out. I'm currently collecting all the code and writing up how to create a database from scratch as well as how to amend an existing database. Just so I make sure to include your use case - is the yeast that you're working with something that's been sequenced and deposited in NCBI, or is it something you've sequenced yourself and hasn't been released yet? If it's not in NCBI, does the species have an NCBI taxonomy ID? Thanks.

Hi, we are working with Glaciozyma martinii which genome it's already available in the NCBI under taxid 119376. Thanks in advance :D


ocamlPackages.apron: 0.9.13 → 0.9.14 & enable PPLite support

Description of changes
https://github.com/antoinemine/apron/blob/v0.9.14/Changes
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build framac


Wrong class for skip button

Hi there
the skip button is not initialized with the enjoyhint_skip_btn CSS class as it should. It uses the enjoyhint_next_btn class (line 263 of enjoyhint.js).
Thanks!

Is there any difference between next and skip styles? I think that now EnjoyHint works well and some style rules just deprecated. Thanks for the issue, we will check it in short time, :smile:

According to the CSS, the skip style is supposed to pin the button in the bottom-right corner.
Having the right class for the skip button would also allow to easily retrieve it through a jQuery selector, to customize it for exemple (even if a proper way to do that would be to give access to some customization options of this button in the library).

I fully agree about style customization. But about button's location.... Locations of the buttons are hardcoded inside library, because some users likes to see buttons near each other. In your opinion, maybe better to pass location of that button to the constructor?

I guess the ideal would be to allow the user to customize the buttons (skip and next) the way he wants (style, location, text).
I'm not a javascript expert so I wouldn't really know what would be the best way to do that, but it could be by setting custom classes and text value through the constructor config hash for exemple.

Hi, at the moment the button control is implemented. It will be possible either hide or display the buttons. The user will be able to do it himself. We will add it to github today or tomorrow.

Bug is still present.
Issue is in EnjoyHint::clear() function and appears when that.nextUserClass is undefined due to wrong async execution order.

$(".enjoyhint_next_btn").removeClass(that.nextUserClass);

If that.nextUserClass === undefined, removeClass() cleans up entire element class and flushes next/skip buttons.
Following extra check makes it work perfectly:

$(".enjoyhint_next_btn").removeClass(that.nextUserClass || '');
$(".enjoyhint_skip_btn").removeClass(that.skipUserClass || '');

Feel free to make a commit.
Regards,
-- Alexander


[WIP] OCR monitoring

@krebernisak thanks a lot for the review. It offered much needed clarity. I refactored the package. Let me know what you think of the structure and separation of responsibilities. If you're OK with the approach I'll add tests.

Hi @krebernisak

Hi @krebernisak. Thanks again for the review
Re. https://github.com/smartcontractkit/chainlink-relay/pull/4#discussion_r753185268 Yes I agree. I looked into getting go-protobuf to generate private types but that’s not supported afaictl. So, I moved all the protobuf generated code in a separate generated package. Hopefully this makes it clearer. Alternatively I can have protobug generate in an internal package.
Re. https://github.com/smartcontractkit/chainlink-relay/pull/4#discussion_r753192160 and https://github.com/smartcontractkit/chainlink-relay/pull/4#discussion_r753197161
The service is the one that “maintains” the WSRPC connection. When you “Start” the service you get a Client which you can use in multiple instances of OCR. This does multiplex producers over the same WS connection, but the ordering of the messages is left to the golang goroutine scheduler. Client instance is thread-safe insomuch as the wsrpc.Client is threadsafe.
Re. the relay repo, I understand that for every instance of the ocr protocol we will instantiate a core#Services structure. This should receive the shared telemetry#Client instance instead of telemetry#Service - which wraps the connection. The question is then, where do I instantiate the Service? I can’t find an “entrypoint”, but maybe there doesn’t need to be one atm.


what does the '-ProtectPrivateKey' switch do?

It is a switch in...
PS C:> get-command Import-PfxCertificate | ft -a
CommandType Name                  Version Source

Cmdlet      Import-PfxCertificate <IP_ADDRESS> PKI

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

ID: e0fb050a-c1ce-ae8d-caea-418ba5891daf
Version Independent ID: f236cf2b-1cd1-f5d1-85be-9ea29bd23b16
Content: Import-PfxCertificate (pki)
Content Source: docset/winserver2019-ps/pki/Import-PfxCertificate.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Deprecate ctr cri load fork

Now that containerd merged: a62be32 We should change the impl of LoadImage to use the client.Import workflows rather than use the forked variant in pkg/containerd/importer

Ha, actually I found that I filed an issue for this before. https://github.com/containerd/cri/issues/909
How about we close this one, and track in that one?

No problem! Thanks!


add missing methods in cognito typescript definition

Issue #, if available:
n/a
Description of changes:
Added getIssuedAt() and decodePayload() to CognitoIdToken and CognitoAccessToken in amazon-cognito-identity-js/index.d.ts
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #393 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #393   +/-   ##
=======================================
  Coverage   89.33%   89.33%
=======================================
  Files          54       54
  Lines        2598     2598
  Branches      531      531
=======================================
  Hits         2321     2321
  Misses        266      266
  Partials       11       11

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 4f528a5...55a6489. Read the comment docs.


Keys are re-useable across different users

@Firesphere commented on Sat Sep 14 2019
As an admin
I would like to be able to prevent the same key to be used for multiple accounts
So that
I can be sure any secrets are not shared across users
Because
My CMS is not the size of Google/GitHub/etc. where it is common for users to have multiple accounts, compared to my website CMS.

@brynwhyman commented on Sun Sep 15 2019
Thanks for the suggestion! What 'key' are you referring to here?

@Firesphere commented on Sun Sep 15 2019
Specifically, physical keys (e.g. Yubikeys). A shared TOTP is practically impossible

@ScopeyNZ commented on Sun Sep 15 2019

I can be sure any secrets are not shared across users

Just FYI, most U2F keys generate private/key pairs for each registration, meaning that there's almost always a different public key per registration. Public keys should be different per registration (depends on device). Here's a good description here:
https://developers.yubico.com/U2F/Protocol_details/Key_generation.html
Also, I don't think that we should refer to a public key as a "secret". Disclosure of the registered key data does not give anybody elevated access as they still need the individual key. In fact, you can only use the original key to even check if registered key data applies to that key, so there's not really any information disclosure with registered key data.
I don't mind the idea of this as an enhancement - but I personally believe that it's really not any security improvement at all (right now) and should be treated as a very "impact/low" enhancement unless there's a good explanation for how this affects a users security at all.

I've moved this issue to the relevant authenticator repo (I should've used GitHub "transfer" rather than whatever I used)
In terms of implementation, it's impact/hard because the key storage architecture right now stores as a blob. This would have to change individual security keys to be a "has many" relation on the registered method.

@ScopeyNZ I'm aware that there are different public/private keys generated, I'm talking about the physical key.
As an admin I want

To prevent my staff from sharing their physical keys to log in to the CMS

Sure - that's why it's still open - I was just clearing it up for any future developer why it's tagged as impact/low 😃

Great idea but it's an old issue that we're not going to do anything with any time soon. Closing. If someone is keen for this and wants to put in the work to implement it, @ me and we can revisit.


Request: 64bit Windows builds

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Problem
64bit Windows shows this as primary remaining 32bit app.
Desired Solution
Run as 64bit application on 64bit Windows
Alternatives and Workarounds
No response
Additional Context
Clearly this is low priority, but I would expect x86/x64/ARM native operation on Windows.

Duplicate of https://github.com/microsoft/AzureStorageExplorer/issues/106


Update loop

This adds an optional argument to the top level loop method. Sometimes it makes sense to start iterating at a different number than 0.

Even though this is correct, maybe it's time to remove the argument to loop. It's not intuitive, it's Int32 by default, and it can overflow.
I'd vote to remove it.

If loop lacked the block argument, would you suggest people create their own counters?
Something like this?
my_counter = 0
loop do
  puts my_counter
end

Yes, that's what you had to do before. Well, almost, you'd also increment the counter ;-)

Oh yes, updating the counter inside the loop 😄

I love the counter in loop, make it Int128 and were all OK :)

@asterite I'd rather add a NOTE regarding possible overflow, since it's rather common flow to use a counter inside of a loop. Also, couldn't we change it to UInt32/64?

Imagine one day we raise on overflow. Then loop will always eventually raise. Doesn't sound good. Let's remove that block argument already :-)

Again, we can add a note. Someone doesn't read it and they have a loop in a server:
loop do
  # accept connection
end

if we ever introduce raise on overflow, that will eventually raise. Why not make it explicit and use a counter, instead of loop having a variable? I can't remember who had the idea to introduce a counter... maybe me? If so, I regret it and I'd like to remove it.

For iterating other indices Range can be used. I agree we can remove loop (and would love to see raise on overflow :-) )

Could we follow the example of each and each_with_index? Have a loop and a loop_with_index?

We can't iterate infinitely over a Range, though

Ref #1407

I think it would be fine to keep the iteration, especially since it gets optimized out when unused. If we ever introduce a raise on overflow, it would be easy to provide backwards compatibility by having the definition of loop rescue the exception. However, because Crystal isn't 1.0.0, I don't think backwards compatibility is all that necessary.

Can we do
loop do
 # no counter used
end

loop do |c|
  puts c
end

Yes, that's what happens currently. If you compile with the --release flag, the counter gets optimized out.

I can't really see any way to make this counter make sense after we add overflow detection, so I'm fine for removing it.

Without a counter, there is however little benefit for loop do over while true... It's just the block scope.

@straight-shoota the block scope is handy, if you want work being done inside the loop but don't want the work to leak outside

@straight-shoota I use loop do instead of while true because it's more descriptive. There don't need to be a concrete technical benefit, loop do is wirth it just for the naming.


Use official actions/jekyll-build-pages

GitHub Pages natively support Jekyll, it's what lets you publish index.md and have it show up a rendered index.html for example.
Instead of that custom action, this action can use the official https://github.com/actions/jekyll-build-pages

Sorry, what exactly are you suggesting here?

It's in the last sentence: use that action internally to build the Jekyll site.
From what I understand, "pr-preview-action" can be used to preview PRs for GitHub Pages websites. GitHub Pages is not just static HTML but it's actually Jekyll-based.
If my repo contains "readme.md" and I enable GitHub Pages publishing, the markdown file is built and displayed as index.html. If I add your action to said repo, will the preview also build it? I don't see any references to Jekyll at this point, so the only way to make that happen would be to use GitHub's action.

But… now that I think about it… the build part doesn't actually need to be part of this action probably. The action will still publish the files to GitHub Pages and then GitHub will take care of the Jekyll build.
Closing 😅

Thanks for clarifying. That's actually a good point and it's not one that I'd thought about. I don't see any reason that GH running Jekyll on the output preview directory wouldn't work out of the box, but it's definitely something to check.


[BT] list fix (IDFGH-8921)

Fix use after free in list_remove() and list_delete().
list->head / node are freed in list_free_node() / list_delete_node() and used after that to check if they are equal to list->tail.
Just a notice that hypothetical change in fixed_queue_new() form list_new(NULL) to list_new(osi_free) would lead to use after free of data returned by fixed_queue_dequeue() and fixed_queue_try_remove_from_queue(). In that case list_remove() should be replaced with list_delete() in fixed_queue_dequeue() and fixed_queue_try_remove_from_queue().

Thanks for your contribution.

Hi @tgotic, we reviewed your PR, and we think this PR is not very necessary. Reason is:
Although we free the node pointed to by list->head, we don't dereference list->head. We only access list->head itself. Therefore it cannot be called use after free.
If you still think this PR is necessary, feel free to correct me. Or, we will close this PR.
Finally, thanks for your contribution!

Hi @lennyJL. I agree that this PR is not very necessary.

Thanks for contribution again, feel free to reopen.


Figure out the icons and category

not sure who to assign this to

Categories sorted by @obucklin  in https://github.com/gramaziokohler/compas_timber/pull/209
Icons by @jonashaldemann ?


Insert Ethereum::CurrentReceipts in the AuxStore

Bound StorageProvider trait to FrontierBlockImport.
On FrontierBlockImport.import_block, read CurrentReceipts from pallet-ethereum and append it to the current block auxiliary storage.

@sorpaas didn't tried this yet but does it look reasonable?

@sorpaas Further on this, we could store the logs_bloom as part of the header digest and use it to scan for past events. Then on match access the auxstore to retrieve the actual logs.

Performing some tests.

Starting a node from scratch, generating 1200 Eth transaction over 10 blocks.
Waiting until it reaches block 20.
Query for logs from block 0 => 10

Current implementation: 1564ms
Auxiliary (tgmichel-consensus-logs): 69ms
StateBackend (tgmichel-consensus-logs-test2-statebackend): 80ms
We can see a huge improvement in using the Auxiliary implementation

I believe this was an attempt to fix a transaction receipt bug that was ultimately fixed by https://github.com/paritytech/frontier/pull/188
Is that right @tgmichel? If so, should this be closed?

@JoshOrndorff sorry the late response, I missed this comment.
No, this is related to finding a good way to access historical log bloom for past events. But the approach might be partially outdated.
I think we should take the same approach as Ethereum does @sorpaas :

Store the Ethereum header bloom in the substrate header digests (we already have that), and use this to deep matching events.
Use direct storage access (I prefer this once we have schema versioning) or auxstore, like in this PR, to access the receipt log data at the matching block height.

And probably combine all that with an optional subcommand to limit how far back can we go for scanning those events.

Whatever path we decide to take on this at some point, this is ancient.


Outdated jQuery

Foundation is distributing 1.5.1, while 1.6.4 is already available.

Good point, updated.


Added title function

Brings https://golang.org/pkg/strings/#Title function to life, useful when using variables in Camel Case notation

Thanks @thorstenhuhn


Move SegmentMetadata from pinot-common to pinot-core

This PR only moves the class into another package, no actual code change

Codecov Report

Merging #5156 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

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

Is this in preparation towards some other change?

Yes, and we have been wanting to do this for a long time, and now all the references to SegmentMetadata in pinot-common are finally cleared.

Will people's hadoop/spark job builds break because of this?

Should not because in order to build the pinot segment, we always need to import pinot-core.


named secrets add support for prefix

@ryanjbaxter ready to be reviewed, has been updated with config data tests also. Thank you

Can you link the issue for this here?
Also would you prefer to merge this before the test containers pr or does it not matter to you?

here is the issue. Just bare in mind please that we can't close it, even after this PR is merged, since I need to add same prefix support for Labeled Secrets.
That will happen after this one...
As to test containers, it really does not matter on the order.
Thank you

you're right, but this is only 1/2 of this feature. When I'll add support for prefix for Labeled Secrets also, documentation will be provided. wdyt? Support for that is coming in the nearest future


I want to  use 'rectangle'

How should I do?
could you  tell me how to cmd

I'm guessing you mean you want ctrl-V to work like it does in Vim, right?

other account this, thank you @wolftune
yes, this like picture
ctrV is..

I want to know how to this

@kenjimorita you can already do that because Atom just supports it directly. It's not identical but it works in your case. The Ctrl-V option might still be nice (so this could stay open as an issue) as it is specifically a rectangle selection. However, just do Alt-Shift-downarrow in Atom to have multiple cursors, and that will work as you want for the example you do above. In some ways the multi-cursor mode in Atom is greatly superior to Vim's rectangle selection.
Cheers

Would this be a duplicate of #468?

@wolftune are you using a package to get the the alt-shift-arrow functionality? (I'm using multi-cursor with custom key mappings.)

@jacekkopecky Yes, this is a duplicate of #468 and can be closed, although maybe my notes about the alt-shift-arrow function should be added there. No, I am not using any package, the alt-shift-arrow function is built into Atom.

Thanks @jacekkopecky and @wolftune!  @kenjimorita, please subscribe to #468 for updates.
@wolftune, feel free to copy your comments to the other issue :)


Bad IUPAC sequence in glycowork/glycan_data/v9_df_species.csv

I found the following bad IUPAC sequence in glycowork/glycan_data/v9_df_species.csv:
GlcNAc(b1-2)Man(a1-?)[Man(a1-?)]Man(b1-4)GlcNAc(b1-4)[Fuc(a1-6)GlcNAc
It is missing a ] just before the reducing end GlcNAc.

n

Thanks for picking that up! We'll fix it in the root dataset; though that will only roll out with 1.1 as changing the dataset usually causes some downstream changes etc.


Generic/LineLength: ignore line length for use statements

From PHP8 on use statements are considered a single token. For this reason they cannot be splitted up to shorter namespaces.
PHPCS doesn't account for that and will throw line length errors and warnings for the use statements. This change ignores the line length for the use statements.
Ofcourse this changes the current behavior so I want to use this PR to further discuss what might be needed to implement this change.
RFC link: https://wiki.php.net/rfc/namespaced_names_as_token
Issue link: https://github.com/squizlabs/PHP_CodeSniffer/issues/3606
Suggested changelog entry

Related issues/external references
Fixes https://github.com/squizlabs/PHP_CodeSniffer/issues/3606
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

[ ] This change is only breaking for integrators, not for external standards or end-users.

[ ] Documentation improvement

PR checklist

[x] I have checked there is no other PR open for the same change.
[x] I have read the Contribution Guidelines.
[x] I grant the project the right to include and distribute the code under the BSD-3-Clause license (and I have the right to grant these rights).
[x] I have added tests to cover my changes.
[x] I have verified that the code complies with the projects coding standards.
[ ] [Required for new sniffs] I have added XML documentation for the sniff.

@rjd22 Thanks for your interest in contributing to PHP_CodeSniffer.

I want to use this PR to further discuss what might be needed to implement this change.

As per the contributing guidelines, this should have been an issue to discuss the various options.
There are basically four options:

The property as implemented in this PR.
Having a separate error code for lines containing import use statements, which allows for selectively ignoring that specific error code.
Blindly ignoring import use statement lines altogether.
Toggling the behaviour of the sniff based on the value of the phpVersion config setting.

Also keep in mind that the tokenization of names in import use statements will change in PHPCS 4.0, which may inform how the solution should be coded.
Looking at the code and the tests, it also appears this PR:

Does not take into account that the use keyword is also used for trait use and closure use.
Does not allow for multi-use statements.
Does not allow for group use statements.
Does not allow for import use statements where the name is already on a second line.
Will significantly (and I mean that!) decrease performance of the sniff due to the way the findPrevious() call is made.

All in all, this is not an acceptable PR. I'd be happy to discuss/review a proposal to fix https://github.com/squizlabs/PHP_CodeSniffer/issues/3606, but this is not the solution.


Configure Nette application with NGINX in context

Hi, I have tried to make all possible configurations to deploy the Micka  in a NGINX context, but without good results. My configuration is as follows:
nginx.conf
location / {
    try_files $uri $uri/ =404;
}

location /visorgeo/catalogo {
    alias /usr/share/nginx/html/visorgeo/micka;
    location ~ \.php?$ {
        fastcgi_pass unix:/var/run/php-fpm/php-fpm.sock;fastcgi_param SCRIPT_FILENAME
        $document_root/index.php;
        include fastcgi_params;
        fastcgi_split_path_info ^((?U).+\.php)(/?.+)$;
        fastcgi_param PATH_INFO $fastcgi_path_info;
     }
}

The problem is that it does not redirect correctly the routers, always gives me a "404 Not Found", even though it displays correctly the initial page index.php.
I appreciate any help.

Hmm, we don't any experience with NGINX . Please check this documentation: https://jiripudil.cz/blog/configuring-nginx-php-fpm-nette

Here a solution in context url in NGINX:
location /micka/ {
    try_files $uri /micka/index.php$is_args$args;

    location ~ /micka/(.*\.php)(/.*)?$ {
        include snippets/fastcgi-php.conf;
        fastcgi_pass unix:/run/php/php5.6-fpm.sock;
        fastcgi_param  SCRIPT_FILENAME   $document_root$fastcgi_script_name;
     }
     location ~ /\.ht {
         deny all;
      }
}


Default the execution path to the execution directory

Summary or solution description
Removed the piece of code that used the shell path instead of the smart contract path.
How to Reproduce
neo3-boa compile path/to/smartcontract.py now has the same behaviour as neo3-boa compile path/to/smartcontract.py  --project-path path/to
Platform:

OS: Windows 10 x64
Python version: Python 3.9

(Optional) Additional context
Add any other context about the problem here.

Task linked: CU-864eek0cd Default the execution path to the execution directory

coverage: 91.14% (-0.01%) from 91.153% when pulling e713a0219a342401f25ce140be01fad8308daa60 on CU-864eek0cd into d37f8bdf791b0bf93c23cc190304790a519b4e5f on development.


Inferring region layer info and displaying read-only

Overview
Links to jira tickets

https://quartech.atlassian.net/browse/BHBC-1258
https://quartech.atlassian.net/browse/BHBC-1260

This PR contains the following changes

Implement UI and API changes to support project and survey boundaries

DB changes for project and surveys to link boundaries to parks, regions, WMU

Pending to fix UI and make cleaner

This PR contains the following types of changes

[x] New feature (change which adds functionality)
[x] Enhancement (improvements to existing functionality)

How Has This Been Tested?

Locally

Screenshots

In the edit location/project boundary, editing the region doesn't work.

In the edit location/project boundary, editing the region doesn't work.
Meaning: I cannot delete/add a region after the project is made

cool, please check now with my most recent commit :) thanks!


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.15.24",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=37f2d2cd8eadddbbd9c7bda327a9393399b2f89b"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "37f2d2cd8eadddbbd9c7bda327a9393399b2f89b",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "NFSD: Fix NFSv3 SETATTR/CREATE's handling of large file sizes\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.24 by commit 37f2d2cd8eadddbbd9c7bda327a9393399b2f89b. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000411


SSH ingress rule

Hope I'm not being thick here, but is this rule needed?
https://github.com/Guimove/terraform-aws-bastion/blob/master/main.tf#L85
Seems like you wouldn't want an ingress rule to your bastion from private subnets.  Just ingress from public (corp IP or something) and then ingress rules on your private sgs.  Or do I have this wrong?
Thanks for the work.

Hello,
I agree with you that linking sg to sg is really more secure, but many people asked this, but is personally I use the sg to sg old version.
I should find time to add the two solution with a switch to chose between them.


Libretro.RetroArch version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

While I have multiple PR related to our project to get in WinGet, this should be the final one. This contains stable where we fix with ninja release the directx9 popup and HASH values are valid.

- Architecture: x64
  InstallerType: nullsoft
  InstallerUrl: http://buildbot.libretro.com/stable/1.10.3/windows/x86_64/RetroArch-Win64-setup.exe
  InstallerSha256: 7FAE3676A37A2B174692C9FC3D17B0C4DB284966049E8630CF93A004AA588C60
  InstallerSwitches:
    Silent: /S
    SilentWithProgress: /S
- Architecture: x86
  InstallerType: nullsoft
  InstallerUrl: http://buildbot.libretro.com/stable/1.10.3/windows/x86/RetroArch-Win32-setup.exe
  InstallerSha256: 8F855E0AEA8702A6192812EF29FD6EFBBC10A5B86833C943A404B8D422F3A712
  InstallerSwitches:
    Silent: /S
    SilentWithProgress: /S

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@j82w @Trenly @ItzLevvie
I thought I could tag you since you have been provide helpful tips. So we pushed ninja release for directx9 thing and it is no longer a problem. I double check the hash(fresh build from today) and it was correct but Azure still fails for x64:
****************************************************************************************
Installer Validation
****************************************************************************************
Status: Completed
-----------------------------------------------------
Installer http://buildbot.libretro.com/stable/1.10.3/windows/x86/RetroArch-Win32-setup.exe
Result: Validation Succeeded
-----------------------------------------------------
Installer http://buildbot.libretro.com/stable/1.10.3/windows/x86_64/RetroArch-Win64-setup.exe
Result: Hash Validation Failed
-----------------------------------------------------
Result: Failed

I also made submission for both build where having incorrect detection for smartscreen and here is a part of MS response;
We can confirm that the application "retroarch-win64-setup.exe", f5fdb51e3ce4b7ab3a2af7859188aba3bef3a3ea4c3e76cb72481f76d551a074”) has since established reputation and attempting to
download or run the application should no longer show any warnings...

At this moment I'm about to give up. Scoop and Chocolatey working fine but I couldn't manage WinGet work least for this version so we can announce and see if it's usable for people or not.

@fpscan have you tried using https instead of http?

@fpscan have you tried using https instead of http?

Thank you for pointing that I was sure that I used https instead http but I saw it wasn't. So I'm on the go but I used browser desktop view and made the changes. So far it's is running and not failed.

x86 hash error:
2022-05-02 14:24:03.488 [CORE] WinGet, version [1.2.10271], activity [{D60A44EB-ECE1-404E-A857-FA39140484C4}]
2022-05-02 14:24:03.488 [CORE] OS: Windows.Desktop v10.0.19042.1052
2022-05-02 14:24:03.488 [CORE] Command line Args: C:\Users\Validator\AppData\Local\Microsoft\WindowsApps\winget.exe install --manifest D:\TRANSFER\24d28397-40ca-ec11-950a-000d3a9eda90\Manifest.yaml --accept-package-agreements
2022-05-02 14:24:03.488 [CORE] Package: Microsoft.DesktopAppInstaller v1.17.10271.0
2022-05-02 14:24:03.488 [CORE] IsCOMCall:0; Caller: winget-cli
2022-05-02 14:24:03.506 [CLI ] WinGet invoked with arguments: 'install' '--manifest' 'D:\TRANSFER\24d28397-40ca-ec11-950a-000d3a9eda90\Manifest.yaml' '--accept-package-agreements'
2022-05-02 14:24:03.509 [CLI ] Found subcommand: install
2022-05-02 14:24:03.509 [CLI ] Leaf command to execute: root:install
2022-05-02 14:24:03.521 [CLI ] Executing command: install
2022-05-02 14:24:03.526 [CLI ] Manifest fields: Name [RetroArch], Version [<IP_ADDRESS>]
2022-05-02 14:24:03.529 [CLI ] Starting installer selection.
2022-05-02 14:24:03.529 [CLI ] Installer [X64,nullsoft,Unknown,] not applicable: Machine is not compatible with X64
2022-05-02 14:24:03.529 [CLI ] Completed installer selection.
2022-05-02 14:24:03.572 [CLI ] Generated temp download path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Libretro.RetroArch.<IP_ADDRESS>\8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712
2022-05-02 14:24:03.576 [CORE] Downloading to path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Libretro.RetroArch.<IP_ADDRESS>\8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712
2022-05-02 14:24:03.576 [CORE] DeliveryOptimization downloading from url: https://buildbot.libretro.com/stable/1.10.3/windows/x86/RetroArch-Win32-setup.exe
2022-05-02 14:24:07.603 [CORE] Download completed.
2022-05-02 14:24:23.366 [CORE] Started applying motw to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Libretro.RetroArch.<IP_ADDRESS>\8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712 with zone: 3
2022-05-02 14:24:23.374 [CORE] Finished applying motw
2022-05-02 14:24:23.375 [CLI ] Package hash verification failed. SHA256 in manifest [8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712] does not match download [f847b92f8e6a13546abb88d0961b2f3f4f419de17a0a53ca895bf4094df03a37]
2022-05-02 14:24:23.375 [CLI ] Terminating context: 0x8a150011 at D:\a\_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\DownloadFlow.cpp:19d

x86 hash error:
2022-05-02 14:24:03.488 [CORE] WinGet, version [1.2.10271], activity [{D60A44EB-ECE1-404E-A857-FA39140484C4}]
2022-05-02 14:24:03.488 [CORE] OS: Windows.Desktop v10.0.19042.1052
2022-05-02 14:24:03.488 [CORE] Command line Args: C:\Users\Validator\AppData\Local\Microsoft\WindowsApps\winget.exe install --manifest D:\TRANSFER\24d28397-40ca-ec11-950a-000d3a9eda90\Manifest.yaml --accept-package-agreements
2022-05-02 14:24:03.488 [CORE] Package: Microsoft.DesktopAppInstaller v1.17.10271.0
2022-05-02 14:24:03.488 [CORE] IsCOMCall:0; Caller: winget-cli
2022-05-02 14:24:03.506 [CLI ] WinGet invoked with arguments: 'install' '--manifest' 'D:\TRANSFER\24d28397-40ca-ec11-950a-000d3a9eda90\Manifest.yaml' '--accept-package-agreements'
2022-05-02 14:24:03.509 [CLI ] Found subcommand: install
2022-05-02 14:24:03.509 [CLI ] Leaf command to execute: root:install
2022-05-02 14:24:03.521 [CLI ] Executing command: install
2022-05-02 14:24:03.526 [CLI ] Manifest fields: Name [RetroArch], Version [<IP_ADDRESS>]
2022-05-02 14:24:03.529 [CLI ] Starting installer selection.
2022-05-02 14:24:03.529 [CLI ] Installer [X64,nullsoft,Unknown,] not applicable: Machine is not compatible with X64
2022-05-02 14:24:03.529 [CLI ] Completed installer selection.
2022-05-02 14:24:03.572 [CLI ] Generated temp download path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Libretro.RetroArch.<IP_ADDRESS>\8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712
2022-05-02 14:24:03.576 [CORE] Downloading to path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Libretro.RetroArch.<IP_ADDRESS>\8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712
2022-05-02 14:24:03.576 [CORE] DeliveryOptimization downloading from url: https://buildbot.libretro.com/stable/1.10.3/windows/x86/RetroArch-Win32-setup.exe
2022-05-02 14:24:07.603 [CORE] Download completed.
2022-05-02 14:24:23.366 [CORE] Started applying motw to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Libretro.RetroArch.<IP_ADDRESS>\8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712 with zone: 3
2022-05-02 14:24:23.374 [CORE] Finished applying motw
2022-05-02 14:24:23.375 [CLI ] Package hash verification failed. SHA256 in manifest [8f855e0aea8702a6192812ef29fd6efbbc10a5b86833c943a404b8d422f3a712] does not match download [f847b92f8e6a13546abb88d0961b2f3f4f419de17a0a53ca895bf4094df03a37]
2022-05-02 14:24:23.375 [CLI ] Terminating context: 0x8a150011 at D:\a\_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\DownloadFlow.cpp:19d

Everytime I check I only get 8F855E0AEA8702A6192812EF29FD6EFBBC10A5B86833C943A404B8D422F3A712 for 32bit build. I may seem like I repeat myself but is there any chance manual merge can be apply at this time?

@denelon - Is there any chance someone could take an hour or two to look into why the hash would pass the Installer scan but not the installation validation? This isn't the only PR we've seen this on (I'll try and find a few others for reference, as well as open an issue if I can find them)

@Trenly, I've asked the team to take a look.

RetroArch requires DirectX 9.0C and it was not detected.

We're also getting a different hash.

RetroArch requires DirectX 9.0C and it was not detected.

We are working on this.

We're also getting a different hash.

I'm not sure what causes this but I get correct hash, stable build uses same hash until new version uploaded. We are using basic cloud flare.

@denelon Hi, I'm not sure why Hash is act like that even file didn't changed last time I uploaded. What we can do for hash at this moment? Winget is an official MS platform and directx is a fundamental part of many windows applications. Why MS think it is acceptable not to support the inclusion of directx when installing anything? I assume dependencies is not considering fo v1.3-Client so our installer won't be there. However I see zip installation for 1.3 in here https://github.com/microsoft/winget-cli/issues/182 and there https://github.com/microsoft/winget-cli/issues/140, so I guess there is nothing I can do beside waiting zip installation get merge for 1.3 and I make new PR for zip version instead installer(?)

@fpscan
Dependencies is still experimental and only supported in the simplest use case. We are looking to tackle dependencies after .zip. Unfortunately, portable apps was a bit more complex and took a bit longer than we anticipated. .zip is currently at risk for getting into 1.3, but we're going to keep working on it.

During our manual verification process, we are observing the hashes are changing for ex:

We think this is likely due to publisher is changing the installer on their end after publishing.  If the hash mismatch is still an issue after this is un-blocked for dependency, we will investigate further at that time.

During our manual verification process, we have been noticing the hashes for the installers are keep getting changed for ex. see 64 bit installer hash value in the below screenshot:

We think this is likely due to publisher is changing the installer on their end after publishing. If the hash mismatch is still an issue after this is un-blocked for dependency, we will investigate further at that time.

I don't know why HASH is changing but I have not published another build for 1.10.3 now or earlier. When I check the hash I get the same numbers.
I think we need to wait until WinGet gets either Dependencies or Portable app installer. The RA installer already includes a bundled directx installer - it seems that this is the problem, and packages on winget cannot include any external installers at all. Regardless of how you bundle directx, it has to call dxsetup.exe at some point - there is no way around this.
So what is needed is an RA installer that doesn't need directx at all.... I think the main problem there is that it's needed for the default windows audio driver - and there's not good alternative. If we don't have directx9 at all then we will have a lot of sound issue. I was expecting this could be painless experience for us but it is not right now, I'll check the milestone time to time. I would like to say that I am open for any suggestion.