Loading jQuery 2.2.4 causes deadlock

Bug Report
Prerequisites

[x] Can you reproduce the problem in a MWE?
[x] Are you running the latest version of AngleSharp? AngleSharp v0.13.0
[x] Did you check the FAQs to see if that helps you?
[x] Are you reporting to the correct repository? (there are multiple AngleSharp libraries, e.g., AngleSharp.Css for CSS support)
[x] Did you perform a search in the issues?

Description
As the title says, loading jQuery 2.2.4 causes a deadlock. I'm using the following source:
https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.js
Steps to Reproduce
var config = Configuration.Default
	.WithRequester(new HttpClientRequester())
	.WithDefaultLoader(new LoaderOptions()
	{
		IsResourceLoadingEnabled = true
	})
	.WithJs()
	.WithEventLoop());

var document = await BrowsingContext.New(config).OpenAsync(r => r.Content(@"
<!DOCTYPE html>

        The body

"));

After investigation, it turns out that the issue comes from setting innerHTML while the document is loading. Here is a simple repro:
var config = Configuration.Default
	.WithRequester(new HttpClientRequester())
	.WithDefaultLoader(new LoaderOptions()
	{
		IsResourceLoadingEnabled = true
	})
	.WithJs()
	.WithEventLoop());

var document = await BrowsingContext.New(config).OpenAsync(r => r.Content(@"
<!DOCTYPE html>

        The body

"));

Expected behavior: OpenAsync should return the document with the script loaded
Actual behavior: OpenAsync is blocked and does not finish ever
Environment details: .Net Core 3.1
Possible Solution
So, after a bunch of debugging, I found the deadlock happens in AngleSharp.Html.Parser.Dom.HtmlDomBuilder.cs:L141.
After parsing the last html part of the fragment, the tokenizer returns an EndOfFile token and is passed through the following methods:

Consume
Home
InBody
End

The End method then sets _waiting = _document.FinishLoadingAsync(); which is causing the deadlock. So, I think the End method should not be called when a fragment is being parsed, but I'm not quite sure where exactly to put the condition. Also, I don't know if there are other cases appart of document fragments which could cause this issue.
Here's what I changed and now jQuery loads correctly.
HtmlDomBuilder.cs:L1511
case HtmlTokenType.EndOfFile:
{
+	if (IsFragmentCase)
+		return;

	CheckBodyOnClosing(token);

	if (_templateModes.Count != 0)
	{
		InTemplate(token);
	}
	else
	{
		End();
	}

	return;
}

Is my thinking correct? Also the changes are to be done in the AngleSharp repo, but I'm posting here because I tought the issue is more related to scripting. Should I open an issue in the AngleSharp repo as well or just a PR is enough?
Awesome project btw :+1:

Awesome investigation 🚀 !
I think indeed that something like that fix would be working, but I think we may rather need to look at the FinishLoadingAsync. In the fragment case there shouldn't be anything open so something is definitely fishy there.
I also fully agree that this is rather an AngleSharp issue, but I appreciate that you posted it here - since scripting is the only victim here.
I will try to spend some time tomorrow thinking about the potential root cause / a potential solution and I would post it here. I would then love to get your input or assertion / evaluation for this. Let's to fix this and make a PR for AngleSharp 0.14! :beers:

Thanks for your input!
You are absolutly right about the FinishLoadingAsync. At first, I tought it was using the same instance of HtmlDomBuilder for the fragment and the main document, so that the End call was closing the main document.
I looked further into FinishLoadingAsync. What happens is that it's queueing the DOMContentLoaded event and returning a task which would not be completed until the event is invoked on the event loop. The event then never fires because it's waiting for the current event to finish.
Here is where the event is queued
What causes FinishLoadingAsync to wait for the event to complete
I'm not sure how to fix this one. Maybe the event could be fired synchronously (direct call, without using the event loop)? I would be cursious to see how actual browsers implemented this.

I'm not sure how to fix this one. Maybe the event could be fired synchronously (direct call, without using the event loop)? I would be curious to see how actual browsers implemented this.

Same here. I thought about over it. I tend to actually use the early escape of the end of file case as shown in the OP.
Haven't reached a conclusion yet (not sure about any side effects).

(not sure about any side effects).

I think that FinishLoadingAsync still needs to be called to wait for resources (attached references) to load.
I had some spare time and looked at chromium source and they fire the event synchronously, see source. Ironically, there is a comment right above that says it should be asynchronous but the linked bug ticket has been closed with status WontFix.
I then looked at firefox source to compare and they fire the event asynchronously. Also, I'm not 100% sure (c++ is not really my thing), but I think it is not awaited.
So I can see 3 possible solutions:

Fire the event synchronously
Fire the event asynchronously without await
Don't fire the event in case of a fragment

IMO, the first would be the best.
What do you think @FlorianRappl ?

Yeah I think it is either

Fire the event synchronously, or
Fire the event asynchronously without await.

Let's go with the synchronous one. That way we delay / block with the resources, but we don't run into a (hidden) deadlock. I think it should be best.
Should I make the PR in AngleSharp or do you want to make it (I'll approve)? It would go into the 0.14 pre version, which we could then test and (if it works) I would make sure 0.14 is released ~start of next week.

:+1: I'll submit the PR in the next days.

I saw there's already some tests for jQuery, but they are not using the event loop. That causes the events to be fired synchronously, exactly as my changes does, so that's why it was passing.
I ran the tests with .WithEventLoop() and it passes when applying my changes.
Should I send another PR for the tests? Because the changes are in the other repo and tests would be in this repo, not sure if it would break things.

No its fine - send the PR for this repo, too. Since this is a AngleSharp.Js 0.14 pre-release you can reference the AngleSharp 0.14 pre-release, too.
Thanks - much appreciated! :beers:

Available in develop.


bugs when evaluate trivia_qa with parameter "all"

We have now fixed this bug, and you can pull the latest code.

If you have any other issues, please contact us.


Add Hooks support to podman

Also mv libkpod config into libpod.  Still needs more cleanup,
but this gets us to one library.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

Changes LGTM, but libostree is causing havoc with the tests.

@mheon Tests are passing,


Log the full TableMetadata

Description
Adds more logs. This is useful for determining the types of workloads happening on Polaris.
Type of change

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

How Has This Been Tested?

[X] Added log assertions

Checklist:

[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] If adding new functionality, I have discussed my implementation with the community using the linked GitHub issue

I removed the ReportMetricsRequest logging as it seems like we already log it as a tag with Invoking CatalogApi with params.

@RussellSpitzer @snazy Thank you for the feedback. I'm new to the Iceberg space and appreciate the context you have on these objects. My hope is to make it easier to see the "unstructured" data that gets self-reported by the query engines. Would you feel more comfortable if this was scoped down to printing the last 10 Snapshot ID & summaries?

@RussellSpitzer @snazy Thank you for the feedback. I'm new to the Iceberg space and appreciate the context you have on these objects. My hope is to make it easier to see the "unstructured" data that gets self-reported by the query engines. Would you feel more comfortable if this was scoped down to printing the last 5 Snapshot ID & summaries?

What's the use case for logging it at all, that cannot be done from an Iceberg client or curl?

@RussellSpitzer @snazy Thank you for the feedback. I'm new to the Iceberg space and appreciate the context you have on these objects. My hope is to make it easier to see the "unstructured" data that gets self-reported by the query engines. Would you feel more comfortable if this was scoped down to printing the last 5 Snapshot ID & summaries?

What's the use case for logging it at all, that cannot be done from an Iceberg client or curl?

I have no problem with the functionality but I think it probably should be part of an eventing api. We may want to keep the complete history of the table somewhere (possibly not in the Catalog itself)

What's the use case for logging it at all, that cannot be done from an Iceberg client or curl?

This is intended to be server-side so Polaris knows more about its callers.

I have no problem with the functionality but I think it probably should be part of an eventing api. We may want to keep the complete history of the table somewhere (possibly not in the Catalog itself)

Yes I think that would be a useful API and this change could help the project move in that direction. In some sense these 2 changes can make the value proposition clearer by first providing low-friction access to this unstructured, self-reported data.

Going to close this PR as there has been no activity on this PR, but most importantly because logging the whole table-metadata is very problematic for two reasons:

the shear size of that object (many kB up to MBs)
the fact that it contains potentially sensitive user data


Unexpected token = on first function in a class

This code works well, is just annoying to see this error all the time.
The problem is, when you create a class in ReactJs, and then create two functions inside this class, the first one will be coloured as an error saying:

Unexpected token = (null)

The second function has no error, is always the first. Note that this are arrow functions.
This code will do in Sublime Text 3 with babel-sublime, babel-linter and ESLint packages installed:
import React, { Component } from 'react';     class MyComponent extends Component {         firstFunction = () => {             return false;         }         secondFunction = () => {             return false;         }         thirdFunction = () => {             return false;         }         render() {             return(                 <div>                     Component                 </div>             );         }     }     export default MyComponent;
No matter what theme or color scheme are you using, this error will be present. I put a screenshot here so you can see the different color from the first function and the second and so on (no error present in the screenshot because I deactivated it on the linter rules).

I am putting my .eslintrc contents for you to see if something is making this error show up.
{   "parser": "babel-eslint",   "ecmaFeatures": {     "modules": true,     "arrowFunctions": true,     "classes": true   },   "env": {     "node": true,     "browser": true,     "es6": true,   },   "rules": {     "no-console": 0,     "no-mixed-spaces-and-tabs": 0,     "react/prop-types": 0,     "react/no-unescaped-entities": 0,     "react/no-string-refs": 0,     "no-useless-escape": 0,     "no-irregular-whitespace": 0,     "strict": 0   },   "extends": [     "eslint:recommended",     "plugin:react/recommended",   ],   "parserOptions": {       ecmaVersion: 6,       ecmaFeatures: {           jsx: true,       },       sourceType: "module"   }, }

Is this a regression to #312?

Same error here. I am using standard together with eslint and vscode plugins.
My eslintrc.yml:
`env:
browser: true
es6: true
node: true
extends:

'eslint:recommended'
'plugin:react/recommended'
globals:
Atomics: readonly
SharedArrayBuffer: readonly
parser: 'babel-eslint'
parserOptions:
ecmaFeatures:
jsx: true
ecmaVersion: 2018
sourceType: module
plugins:
react
rules:
indent:

error
2
SwitchCase: 1
linebreak-style:
error
unix
quotes:
error
single
semi:
error
never
`

Ok, after 2 days stuggling and a few minutes after posting my message here, I discovered that the problem was caused by standardjs plugin on vscode, nothing to do with eslint.

Thank you for the issue. Now that @babel/eslint-parser has been released, we are making this repository read-only. If this is a change you would still like to advocate for, please reopen this in the babel/babel monorepo.


It should be possible to set the ip address the server listens to by an environmental variable

In a previous version it was possible to set the ip address the node server is listening to by setting the environmental variable ADDRESS: https://github.com/DSpace/dspace-angular/blob/afc39022f86919d4005f72f3dae55adf1f8583c2/src/server.ts#L49
This was changed in #123. For docker it would be very useful to be able to set ip address and port by changing an environmental variable.

See PR #263.


libtexture_share_gl_client.so fails to load with undefined symbol: glad_glDeleteTextures

installed from the AUR, used through obs-plugin-texture-share-vk. glad was installed on system at time of build

I just updated the PKGBUILD, let me know if it works.

Yes it is working now, thank you


Nuget Erros

Nuget not compile correctly. the events namespace or the LatLng class is missing. could you publish a new version?

I'll second the request to publish an updated nuget.

The documentation on the front of this repo is for the current state of the master branch and unfortunately the API has changed significantly in the months since a version tag was last added and submitted as a new nuget version.
If you're using the public nuget rather than compiling master yourself,  you'll want to select the appropriate tag and view the documentation and samples as they were at that point.
For the current 0.4.0-alpha branch, this link should take you there: https://github.com/Mehigh17/BlazorLeaflet/tree/0.4.0-alpha


Support Ubuntu Jammy

From Jammy, we do not have python anymore. We have python3:
The following packages have unmet dependencies:
 builddeps:bazel : Depends: python but it is not installable

@wooksong PTAL

Ready!

Please take this patch for PPA build and please clean up PPA bazel, too. (use nnsuite for forking bazel and PPA)

@myungjoo Confirmed that it is possible to build with pdebuild on Jammy
The next steps would be

Merge it and move this repository to the nnsuite org.
Check the version that we need to build TF
If it is possible to use the provided one from https://bazel.build/install/ubuntu#install-on-ubuntu , remove all except for Jammy
Update documentation


Rename Module and Alias

Forking from here: https://github.com/ironmansoftware/psedit/issues/6#issuecomment-1329511879
Reasons for renaming:
psedit module exists in VS Code remote sessions and ISE
psedit alias also exists in VS Code remote sessions and ISE
Proposed Module Name: IronmanSoftware.PSEdit
Proposed Alias Behavior: Define psedit if it does not exist.
Definitely open to other suggestions!

psnote

Some arguments for NOT renaming the module: https://fosstodon.org/@jaykul/109423356465772117

My initial thought is IronmanSoftware.PSEditor. Although, the module only has a single command. Could there be related commands in the future related to editing files? I'm just brainstorming here, but maybe there would be a TUI hex viewer in the future. Maybe something with alternate streams? Planning for the future, I'd suggest a broader name like IronmanSoftware.PSEditorServices.  You might even consider a company prefix on the noun, like Show-IronEditor or Show-ISEditor. While I'm at it, shouldn't Open be the better verb?


Fix phpDoc type hint for phpCAS::client()

This pull request updates the phpDoc type hint for phpCAS::client() to match CAS_Client::__construct().
https://github.com/apereo/phpCAS/blob/89ce13044a629eb07c23e1bca0cb4926acf6ea00/source/CAS/Client.php#L888-L902

Actually, it looks like there are a few of these kinds of issues that are tripping up some static analyzers I'm running for one of my projects. Let me know if the maintainers would be open to me running the whole code base through something like PHPStan/Phan and submitting a PR for any major issues they find.

@kberzinch Sure we are happy to take any contributions. Pull requests are always greatly appreciated.


PHP Version constraint

Hi fabien,
well i guess this makes sense as symfony4 requires php7.1.3
cheers,

Hi,
See https://github.com/symfony/flex/pull/162
The approach of @nicolas-grekas is coherent.
Many companies use the default version, for Debian currently it is 7.0.
Regards

Closing as explained by @20uf.


DLG should utilize a new layout

Full screen:

dark top bar with the alert/prompt title
bright content area stretching the full height
dark bottom bar with branding on the left, and OK/Cancel buttons on the right

Windows:

new layout of a prompt

(closed by mistake)


[CRITICAL] core script makes lots of cache textures

iOS application debug log
2019-09-21 19:22:39.039393+0900 StarryNight_swift[12541:3674519] [ProcessSuspension]  0x2834ca780 - ProcessAssertion() Unable to acquire assertion for process with PID 0
2019-09-21 19:23:09.041738+0900 StarryNight_swift[12541:3674258] Could not signal service com.apple.WebKit.WebContent: 113: Could not find specified service

JS console debug log
[Error] Failed to load resource: The requested URL was not found on this server. (pixi.js.map, line 0)
[Error] Failed to load resource: The requested URL was not found on this server. (pixi-tilemap.js.map, line 0)
[Error] Failed to load resource: The requested URL was not found on this server. (pixi-picture.js.map, line 0)
[Warning] Total canvas memory use exceeds the maximum limit (288 MB). (rpg_core.js, line 1481, x2)
[Warning] Total canvas memory use exceeds the maximum limit (288 MB). (rpg_core.js, line 754)
[Warning] Total canvas memory use exceeds the maximum limit (288 MB). (rpg_core.js, line 754)

We are porting MV games to iOS and found the following flaws in the core script.
This problem is quite serious and is not very well suited to RAM optimization and modern GPU rendering methods.
Our game dev team have confirmed while porting to iOS, core script makes cache texture just one time uses for making every windows. (even when moving window cursor too...)
This approach is extremely inefficient.
Because of this problem, the web view we used to port to iOS, WkWebView, opens and closes the menu dozens of times, causing the web view will refresh page by cannot handle the huge memory heaps. It's very serious.
Here is reason:

Making cache texture (using canvas) are makes huge memory heaps, so when garbage collector(GC) will takes cleaning for long time. The real problem is that only GC knows when to clean. (GC will clean even playing game, that makes freezing the game)
Make every window using instant texture, then GPU will wait for next uploaded texture from CPU for drawing, every each frame! this is must be avoided with optimization.

There are two alternatives:

Create a blank canvas of 2048x2048 size, and draw in here and display it every time create a window. This is called dynamic texture. (only use one texture for displaying everything.)
Representing every window elements as Pixi's default objects. Because Pixi's assets are all managed objects.

Dev team should know why need to be more concerned about VRAM and RAM optimization issues, please check the link below.
What is a sprite sheet?
What is a sprite sheet? - Performance

Thank you for your report!
I don't know much about rendering, so it's better to let Pixi manage it.

I think it's going to require quite a lot of changes to core scripts in second option.
If need optimizing in a short time, I would recommend a dynamic texture solution.
I think this method can be expressed in the same way as it is now without creating many additional canvases.
If the situation is not good and there is no other way, I recommend the second option.

@Creta5164
Sorry for the late reply, but I'm still interested in your agenda.
Certainly the first method seems to be better. But since it is very technical, it may be wrong if I implement it.
Could you write a pull request?
I don't know if the publisher will officially release it, but we can publish your proposal as a new open source version on Github.

Sorry, but I'd not currently have enough margin of time to contribute significantly to this project.
So I created a demo to actually demonstrate the concept of dynamic textures.

jsfiddle demo
Pixi has a RenderTexture, that makes it possible to draw Pixi's DisplayObject.
This demo shows a demonstration of creating elements of a window into a Pixi Sprite, drawing a window on a RenderTexture, then tearing it off and recreating it into a Sprite.
This is a simple demo, and in order to actually use it, this will need function about lot of drawing and erasing textures to use dynamically in the RenderTexture. (Dynamic texture)
Check this post for how sprite packing algorithm works.

Thanks!
I will study more about rendering ;-)


MSL: Lookup table codegen causes spilling on NVidia drivers

This is sort of a cross-post of this https://github.com/KhronosGroup/glslang/issues/1430 - based on my renewed understanding of the SPIRV limitations it's starting to look like this problem can only be addressed on SPIRV-Cross side.
Given this shader:
float4 main(uint iid: SV_InstanceId): SV_Position
{
    static const float3 uvRoutingA[16] = { float3(0,0,1),  float3(1,0,0), float3(1,0,0), float3(0,0,1),  float3(1,0,0), float3(1,0,0),  float3(1,0,0),        float3(1,0,0),           float3(0,0,1),        float3(0,0,0), /* #10: */ float3(1,0,0), float3(0,1,0), float3(0,1,0), float3(1,0,0), float3(0,1,0),  float3(0,1,0) };

    return float4(uvRoutingA[iid], 1);
}

SPIRV-Cross MSL translation generates this code:
vertex main0_out main0(uint gl_InstanceIndex [[instance_id]])
{
    main0_out out = {};
    float3 _45[16] = {float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0), float3(1.0, 0.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0)};
    out.gl_Position = float4(_45[gl_InstanceIndex], 1.0);
    return out;
}

We currently use https://github.com/zeux/hlslparser; it has support for lifting static const arrays out to global scope, producing this code:
const static constant constexpr float3 uvRoutingA[16] = { float3(0, 0, 1), float3(1, 0, 0), float3(1, 0, 0), float3(0, 0, 1), float3(1, 0, 0), float3(1, 0, 0), float3(1, 0, 0), float3(1, 0, 0), float3(0, 0, 1), float3(0, 0, 0), float3(1, 0, 0), float3(0, 1, 0), float3(0, 1, 0), float3(1, 0, 0), float3(0, 1, 0), float3(0, 1, 0) };
...
vertex main0_out main0(uint gl_InstanceIndex [[instance_id]])
{
    main0_out out = {};
    out.gl_Position = float4(uvRoutingA[gl_InstanceIndex], 1.0);
    return out;
}

In one of our production shaders, the construct seen above happens two times (with two separate tables); on NVidia drivers in latest OSX, the shader generated from SPIRV-Cross output is very inefficient - on a full frame where only some of the draw calls use that, the GPU time jumps from 10 ms (as compiled by hlslparser) to 26 ms (as compiled by glslang+spirv-cross). I suspect that the driver compiler is spilling the array to perform dynamic indexing which takes forever.
It seems like SPIRV-Cross needs to learn to lift the table literals to the global scope similarly to hlslparser, and index them directly without copying them to local variables with spvArrayCopy or whatnot.

Just to have a more precise example from our production shader, not the simplified version above, here's an excerpt from SPIRV-Cross generated MSL (that takes 26 ms) - there's only 2 arrays, but there's 2 lookups into each so they are replicated twice:
    float3 _575[16] = {float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0), float3(1.0, 0.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0)};
    float2 _982 = _1008;
    _982.x = dot(_142.CB1[gl_InstanceIndex].uvScale.xyz, _575[_614]);
    float3 _576[16] = {float3(0.0, 1.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0, 1.0, 0.0), float3(0.0, 0.699999988079071044921875, 0.699999988079071044921875), float3(0.0, 0.699999988079071044921875, 0.699999988079071044921875), float3(0.699999988079071044921875, 0.699999988079071044921875, 0.0), float3(0.0), float3(0.0, 0.0, 1.0), float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0, 0.0, -1.0), float3(0.0, 0.0, 1.0)};
    float2 _984 = _982;
    _984.y = dot(_142.CB1[gl_InstanceIndex].uvScale.xyz, _576[_614]);
    float3 _577[16] = {float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0), float3(1.0, 0.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0), float3(1.0, 0.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0)};
    float2 _986 = _1008;
    _986.x = dot(_142.CB1[gl_InstanceIndex].scale.xyz, _577[_614]);
    float3 _578[16] = {float3(0.0, 1.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0, 1.0, 0.0), float3(0.0, 1.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0, 1.0, 0.0), float3(0.0, 0.699999988079071044921875, 0.699999988079071044921875), float3(0.0, 0.699999988079071044921875, 0.699999988079071044921875), float3(0.699999988079071044921875, 0.699999988079071044921875, 0.0), float3(0.0), float3(0.0, 0.0, 1.0), float3(0.0, 0.0, 1.0), float3(1.0, 0.0, 0.0), float3(0.0, 0.0, 1.0), float3(0.0, 0.0, -1.0), float3(0.0, 0.0, 1.0)};
    float2 _988 = _986;

And here's the hlslparser:
        float2 selector;
        selector.x = dot(uvScale.xyz, uvRoutingA[normalId]);
        selector.y = dot(uvScale.xyz, uvRoutingB[normalId]);
        uv = ((uv * selector) + matUvOffs.xy);
        selector.x = dot(scale.xyz, uvRoutingA[normalId]);
        selector.y = dot(scale.xyz, uvRoutingB[normalId]);
        uvStuds = (uvStuds * selector);

Just to keep everything nicely linked, here’s a DXC bug: https://github.com/Microsoft/DirectXShaderCompiler/issues/1398. DXC generates and extra branch that hopefully can be removed as part of this issue; my overarching goal is to make sure static constexpr array can be emitted from SPIRV produced by both compilers.
I think I more or less understand how to hack up a solution for this assuming there’s easy access to domination info... I’ll try to take a look at this later this week, although implementation suggestions are certainly welcome.

https://github.com/KhronosGroup/SPIRV-Cross/blob/master/spirv_cross.cpp#L3650 is the grand place where domination for all variables and temporaries live. I'm not sure this will be enough. It doesn't track all the info I think you'll need though.

I've started refactor some internals to make this analysis possible. This won't be easy though ...

Should "just werk" now when DXC fixes their static const init issue.


map, flatMap, and flatten work on one level

Map and Flatten Enumerators are no longer recursive.
Added [Callable identity] to use in testing flatMap.
Removed testMapLiftsMappables - map shouldn’t lift mappables
All tests working

I have no idea why I made map lift mappables - thanks for sorting this out!


fix: Fixes a typo in the comment for content scope

@farhaanbukhsh can you rebase this? it looks like I don't have the ability to do so on my side otherwise I would do so and merge this fix.

@feanil  Done :)

@farhaanbukhsh 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


Apply fixes after discussion

Idea: we decided to keep things simple.

we put all data we want to copy into .neuro/
neuro-extras will be dumb: do not ask anything, do not overwrite. Just print: file/drectory .neuro/ exists, please delete it.

Merging to test with neuro-extras.


help(CDK/drag and drop): CDK drop and drop mixing up, down, left and right

What are you trying to do?
When I drag and drop columns from one list to another drop list, the drop area have up to 3 columns per row, and can add the whole rows. I hope the droped control can run around the implementation of an existing control.

What troubleshooting steps have you tried?
I checked the CDK document, the drag and drop samples only support up and down, or left and right with two list.
There are no examples of mixing up, down, left and right.
Reproduction
We can only help if we can reproduce the problem ourselves.
Use StackBlitz to demonstrate what you are trying to do:

Components: https://stackblitz.com/fork/components-issue
Harnesses: https://stackblitz.com/fork/harness-issue

Steps to reproduce:

Drag one of the left control list
The droped control can't run around the implementation of an existing control

Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

Duplicate of https://github.com/angular/components/issues/13372.

Duplicate of #13372.
Sorry, You misunderstood me.

I want to achieve the following GIF effect. Can angular CDK be implemented?

The droped control can be the left, right ,up or down sides of the existing control.

ok, I see. You should be able to implement that example by having multiple lists that the item can be dropped into (one for each row of fields).

Yes.
Any idear ?


Provide mono builds.

Please consider to provide mono builds alongside the normal builds, Mono C# is a official language too.

What would be different in a Mono build from the current one?

Sorry, I don't know. But we have AAR library for Android plugins (Mono / C#), so I thought that we should had to build the plugin with it for C#.
https://downloads.tuxfamily.org/godotengine/3.2.3/mono/godot-lib.3.2.3.stable.mono.release.aar


az image copy from one region to another does not work

The below is a command and full output. It cannot be simpler. I also tried it long time ago with "az feedback" but did not see any response. I simply want to copy my custom image from one region (eastus) to another (southcentralus).  I understand that the system is looking for "'MyHPCImage-7_os_disk_snapshot" and I do not have it. But why should I have it? I have an existing image "MyHPCImage-7" which I successfully use in my deployments. This should be enough. I want to add that the corresponding question by somebody else on stackoverflow was never answered:  https://stackoverflow.com/questions/45720966/how-to-copy-an-azure-image-from-one-region-to-another-region

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

I did that long time ago, never got a response
Related command
az image copy --source-resource-group "ResGroupForMetaBatchAccount" --source-object-name "MyHPCImage-7" --target-location  "southcentralus" --target-resource-group "ResGroupForMetaBatchAccount-southcentralus" --cleanup
Describe the bug
It returns ERROR, the image is not copied
To Reproduce
Run the above command
evgeniy@Azure:~$ az image copy --source-resource-group "ResGroupForMetaBatchAccount" --source-object-name "MyHPCImage-7" --target-location  "southcentralus" --target-resource-group "ResGroupForMetaBatchAccount-southcentralus" --cleanup
Getting OS disk ID of the source VM/image
Creating source snapshot
command failed: ['/usr/bin/python3.9', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'MyHPCImage-7_os_disk_snapshot', '--location', 'eastus', '--resource-group', 'ResGroupForMetaBatchAccount', '--source', '/subscriptions/53cb88b5-ffe7-4bd5-83ed-f3ac0d26469a/resourceGroups/ResGroupForMetaBatchAccount/providers/Microsoft.Compute/disks/vmimage_disk1_7cdff216f6ab46b8a41ccd5ada16c695', '--hyper-v-generation', 'V2', '--output', 'json', '--tags', 'created_by=image-copy-extension']
output: ERROR: (NotFound) Resource vmimage_disk1_7cdff216f6ab46b8a41ccd5ada16c695 is not found.
Code: NotFound
Message: Resource vmimage_disk1_7cdff216f6ab46b8a41ccd5ada16c695 is not found.

The command failed with an unexpected error. Here is the traceback:
Command '['/usr/bin/python3.9', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'MyHPCImage-7_os_disk_snapshot', '--location', 'eastus', '--resource-group', 'ResGroupForMetaBatchAccount', '--source', '/subscriptions/53cb88b5-ffe7-4bd5-83ed-f3ac0d26469a/resourceGroups/ResGroupForMetaBatchAccount/providers/Microsoft.Compute/disks/vmimage_disk1_7cdff216f6ab46b8a41ccd5ada16c695', '--hyper-v-generation', 'V2', '--output', 'json', '--tags', 'created_by=image-copy-extension']' returned non-zero exit status 3.
Traceback (most recent call last):
  File "/usr/lib64/az/lib/python3.9/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/evgeniy/.azure/cliextensions/image-copy-extension/azext_imagecopy/custom.py", line 108, in imagecopy
    run_cli_command(cli_cmd)
  File "/home/evgeniy/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 45, in run_cli_command
    raise ex
  File "/home/evgeniy/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 21, in run_cli_command
    cmd_output = check_output(cmd, stderr=STDOUT, universal_newlines=True)
  File "/usr/lib/python3.9/subprocess.py", line 424, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/usr/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/usr/bin/python3.9', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'MyHPCImage-7_os_disk_snapshot', '--location', 'eastus', '--resource-group', 'ResGroupForMetaBatchAccount', '--source', '/subscriptions/53cb88b5-ffe7-4bd5-83ed-f3ac0d26469a/resourceGroups/ResGroupForMetaBatchAccount/providers/Microsoft.Compute/disks/vmimage_disk1_7cdff216f6ab46b8a41ccd5ada16c695', '--hyper-v-generation', 'V2', '--output', 'json', '--tags', 'created_by=image-copy-extension']' returned non-zero exit status 3.
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
To open a new issue, please run `az feedback`

Additional context
It is a linux image   . Please help to copy image to another region.

@zhoxing-ms for awareness

@evolkovgit Thank you for bringing this issue to our attention. I apologize for the inconvenience caused by the error in the az image copy process.
From my investigation, it appears that the error is caused by the missing "'MyHPCImage-7_os_disk_snapshot" file, which is created during the process of creating an image from a generalized VM. This file is necessary for the image service to store the image information and is also part of the billing process.
I understand that you only have the mirror file in your resource group "MyHPCImage-7" which only stores information such as the path and version of the mirror file. In the event that the snapshot file is deleted, the mirror will not be usable or movable.
I appreciate your feedback and we will consider reminding users of the consequences of the operation and requesting reconfirmation when they delete the snapshot file of the associated mirror.
If you have any follow-up questions, please do not hesitate to contact me.

Hi @wu5bocheng . Sorry, I am not satisfied with your answer. As I wrote in the initial BUG description, it was clear to me that the missing  "'MyHPCImage-7_os_disk_snapshot" was the reason.   I created the image "MyHPCImage-7" from running VM via Azure portal feature "Capture" (https://learn.microsoft.com/en-us/azure/virtual-machines/capture-image-portal) and the snapshot was NOT created automatically during the process.  "MyHPCImage-7" is still perfectly working  image. I successfully use it in deployments without having the "snapshot" . It works. I repeat, it is not a "mirror" as you wrote.  Also I never manually deleted any snapshot. There is nothing about the "snapshot" in the above link. Also there is no portal GUI to create a "snapshot" from an image. You wrote that the  "snapshot" is created during the process of creating an image from a generalized VM. It is not true.  It is never created for me. Which options I should enable so the "snapshot" IS CREATED while using the "capture" feature? So it is a bug.

Hi, @evolkovgit. I apologize for mislocating your issue earlier, I read it carefully and tried to reproduce your problem. I think you deleted the system disk of the VM where the image was created (or chose to delete the VM while creating the image). Your MyHPCImage-7  is based on the VM vmimage capture, but the system disk vmimage_disk1_7cdff216f6ab46b8a41ccd5ada16c695 that was deleted after the capture. Since the system disk could not be found, the image could not be automatically created by image-copy-extension automatically creates MyHPCImage-7_os_disk_snapshot and moves it, so it reports an error that MyHPCImage-7_os_disk_snapshot cannot be found.
If it is the case of your problem, here is a temporary solution that you can try to resolve the issue:

First, create a new virtual machine vm_temp by the image MyHPCImage-7.
Then, create a new image image_new for the virtual machine vm_temp.
Finally, use the new image image_new to perform a copy operation.

If I have mislocated your issue, could you please provide me with the output of
az resource list --resource-group ResGroupForMetaBatchAccount (if the output is too long to paste into the comments section, please store the output as a json file and mailto<EMAIL_ADDRESS>so that I can better help you solve the problem.
I hope this solution helps you. If you have any further questions or concerns, please do not hesitate to contact me.

Hi @wu5bocheng
Thank you for your response. You wrote:
>I think you... chose to delete the VM while creating the image
Correct, in the "capture" feature, I used the option (check-box) to delete the VM automatically after creating the image.  The reason I do it because the generalized VM is useless after the "generalization" process, since it is somehow removes  the Linux "azureuser" credentials and I cannot login there any longer.
I will use your temporary solution (without deleting the VM during the "capture" ) and it will probably help.
I still think that it is bug in Azure CLI. The "az image copy"  python code should simply create missing "'MyHPCImage-7_os_disk_snapshot" automatically if it needs it and delete it afterwards (by --clean) option .
Again, thank you. I will use your temporary solution.
Eugene

Hi @wu5bocheng
It still did not work (probably because my VM is Linux and you are experimenting with Windows VMs)

I created VM called "vmimage" using "MyHPCImage-7" image. The VM was OK, I am able to SSH there

I captured the image "MyHPCImage-8" from the VM "vmimage". I had the option "Automatically delete this virtual machine after creating the image" unchecked.

So now I have both VM "vmimage" and "MyHPCImage-8"  existing. I checked in portal "snapshots" if there is a snapshot for "MyHPCImage-8" : No, it was not there.

Anyway, I issued the command in Azure CLI:
az image copy --source-resource-group "ResGroupForMetaBatchAccount" --source-object-name "MyHPCImage-8" --target-location  "southcentralus" --target-resource-group "ResGroupForMetaBatchAccount-southcentralus" --cleanup
I got the result:
Don't specify an existing resource group in --temporary-resource-group-name when --cleanup is set

The above is wrong (bug) because I did not specify any --temporary-resource-group-name
5. Now (as you asked me) I am issuing :
az resource list --resource-group ResGroupForMetaBatchAccount
The result is long JSON text, I will send it to you via email (as you asked), but do not look for the snapshot in the list: it is not there.
Something to add:
a. The "generalized" "vmimage" Linux VM is useless now, because I cannot SSH-login there after it was "generalized" during the "capture" process.   So , since it still does not work, I will delete it. The captured image "MyHPCImage-8" is still OK, though there is still no corresponding snapshot. Since it is the same as the previous one "MyHPCImage-7" I will delete it as well after some attempts.
b. I tried to "move" the "MyHPCImage-8" image via Azure Web Portal GUI , it did not work either
regards,
Eugene

Sorry, I just tried again, but I couldn't reproduce your problem, so more assistance may be needed from you. Please run 'az image copy --source-resource-group "ResGroupForMetaBatchAccount" --source-object-name "MyHPCImage-8" -- target-location "southcentralus" --target-resource-group "ResGroupForMetaBatchAccount-southcentralus" --cleanup --debug' and send the output to me.

But I already deleted  "MyHPCImage-8" and VM "vmimage". I have to do it
again.
Why didn't you tell me to run it with the -- debug option from the very
beginning?
Why "move" image via Azure Portal GUI does not work either?

OK, I will do it tomorrow (I am in USA, California) one more time, but it
looks like it is easier for me to create a new image in "southcentralus"
than continue this.
FYI: I have experience in AWS (Amazon Cloud) and OCI (Oracle cloud) and
never experienced such issues there. We have to do Azure now, because our
customer also asked to add Azure to our software solution...
OK, please, wait for my email. but meanwhile please answer:

Why should users keep "generalized" VM after "capture", if it is useless
(cannot SSH login there after "generalizing")
I am not using "gallery" did you used gallery in your experiments when
"captured"

Eugene
On Mon, Feb 13, 2023 at 11:32 PM wu5bocheng @.***>
wrote:

Sorry, I just tried again, but I couldn't reproduce your problem, so more
assistance may be needed from you. Please run 'az image copy
--source-resource-group "ResGroupForMetaBatchAccount" --source-object-name
"MyHPCImage-8" -- target-location "southcentralus" --target-resource-group
"ResGroupForMetaBatchAccount-southcentralus" --cleanup --debug' and send
the output to me.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-cli/issues/25431#issuecomment-1429258770,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWPIOHY6DAJJWKFJUHLL5XLWXMYKXANCNFSM6AAAAAAUYM2YHQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

OK, I decided not to go to sleep :-) and do it now
Please let me know ASAP: I want to delete useless VM and image (not to pay for resources I do not need)

I created VM called "vmimage" using "MyHPCImage-7" image. The VM was OK, I am able to SSH there
I captured the image "MyHPCImage-8" from the VM "vmimage". I had the option "Automatically delete this virtual machine after creating the image" unchecked.
So now I have both VM "vmimage" and "MyHPCImage-8" existing.
I am issuing the az image copy command with the --debug option. Below is full output

Requesting a Cloud Shell.Succeeded.
Connecting terminal...

evgeniy@Azure:~$ az image copy --source-resource-group "ResGroupForMetaBatchAccount" --source-object-name "MyHPCImage-8" --target-location  "southcentralus" --target-resource-group "ResGroupForMetaBatchAccount-southcentralus" --cleanup --debug
cli.knack.cli: Command arguments: ['image', 'copy', '--source-resource-group', 'ResGroupForMetaBatchAccount', '--source-object-name', 'MyHPCImage-8', '--target-location', 'southcentralus', '--target-resource-group', 'ResGroupForMetaBatchAccount-southcentralus', '--cleanup', '--debug']
cli.knack.cli: __init__ debug log:
Enable color in terminal.
cli.knack.cli: Event: Cli.PreExecute []
cli.knack.cli: Event: CommandParser.OnGlobalArgumentsCreate [<function CLILogging.on_global_arguments at 0x7fe398dae160>, <function OutputProducer.on_global_arguments at 0x7fe398cccb80>, <function CLIQuery.on_global_arguments at 0x7fe398c5d160>]
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableCreate []
cli.azure.cli.core: Modules found from index for 'image': ['azure.cli.command_modules.vm', 'azext_imagecopy']
cli.azure.cli.core: Loading command modules:
cli.azure.cli.core: Name                  Load Time    Groups  Commands
cli.azure.cli.core: vm                        0.264        57       281
cli.azure.cli.core: Total (1)                 0.264        57       281
cli.azure.cli.core: These extensions are not installed and will be skipped: ['azext_next']
cli.azure.cli.core: Loading extensions:
cli.azure.cli.core: Name                  Load Time    Groups  Commands  Directory
cli.azure.cli.core: image-copy-extension      0.008         1         1  /home/evgeniy/.azure/cliextensions/image-copy-extension
cli.azure.cli.core: ai-examples               0.123         1         1  /usr/lib/python3.9/site-packages/azure-cli-extensions/ai-examples
cli.azure.cli.core: Total (2)                 0.130         2         2
cli.azure.cli.core: Loaded 58 groups, 283 commands.
cli.azure.cli.core: Found a match in the command table.
cli.azure.cli.core: Raw command  : image copy
cli.azure.cli.core: Command table: image copy
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableTruncate [<function AzCliLogging.init_command_file_logging at 0x7fe397f0bc10>]
cli.azure.cli.core.azlogging: metadata file logging enabled - writing logs to '/home/evgeniy/.azure/commands/2023-02-14.08-14-13.image_copy.134.log'.
az_command_data_logger: command args: image copy --source-resource-group {} --source-object-name {} --target-location {} --target-resource-group {} --cleanup --debug
cli.knack.cli: Event: CommandInvoker.OnPreArgumentLoad [<function register_global_subscription_argument.<locals>.add_subscription_parameter at 0x7fe397eb3820>]
cli.knack.cli: Event: CommandInvoker.OnPostArgumentLoad []
cli.knack.cli: Event: CommandInvoker.OnPostCommandTableCreate [<function register_ids_argument.<locals>.add_ids_arguments at 0x7fe397e7f0d0>, <function register_cache_arguments.<locals>.add_cache_arguments at 0x7fe397e7f280>]
cli.knack.cli: Event: CommandInvoker.OnCommandTableLoaded []
cli.knack.cli: Event: CommandInvoker.OnPreParseArgs []
cli.knack.cli: Event: CommandInvoker.OnPostParseArgs [<function OutputProducer.handle_output_argument at 0x7fe398cccc10>, <function CLIQuery.handle_query_parameter at 0x7fe398c5d1f0>, <function register_ids_argument.<locals>.parse_ids_arguments at 0x7fe397e7f1f0>]
az_command_data_logger: extension name: image-copy-extension
az_command_data_logger: extension version: 0.2.9
cli.azext_imagecopy.cli_utils: command: ['/usr/bin/python3.9', '-m', 'azure.cli', 'group', 'exists', '-n', 'image-copy-rg', '--output', 'tsv'] ended with output: true

cli.azure.cli.core.azclierror: Traceback (most recent call last):
  File "/usr/lib64/az/lib/python3.9/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/evgeniy/.azure/cliextensions/image-copy-extension/azext_imagecopy/custom.py", line 31, in imagecopy
    raise CLIError('Don\'t specify an existing resource group in --temporary-resource-group-name '
knack.util.CLIError: Don't specify an existing resource group in --temporary-resource-group-name when --cleanup is set

cli.azure.cli.core.azclierror: Don't specify an existing resource group in --temporary-resource-group-name when --cleanup is set
az_command_data_logger: Don't specify an existing resource group in --temporary-resource-group-name when --cleanup is set
cli.knack.cli: Event: Cli.PostExecute [<function AzCliLogging.deinit_cmd_metadata_logging at 0x7fe397f0be50>]
az_command_data_logger: exit code: 1
cli.__main__: Command ran in 1.531 seconds (init: 0.126, invoke: 1.405)
telemetry.main: Begin splitting cli events and extra events, total events: 1
telemetry.client: Accumulated 0 events. Flush the clients.
telemetry.main: Finish splitting cli events and extra events, cli events: 1
telemetry.save: Save telemetry record of length 3568 in cache
telemetry.check: Negative: The /home/evgeniy/.azure/telemetry.txt was modified at 2023-02-14 08:12:10.689799, which in less than 600.000000 s
evgeniy@Azure:~$

Thank you for bringing your issue to our attention. I apologize for any inconvenience caused. As the problem you are experiencing is specific and is challenging to reproduce. I appreciate your help in resolving this issue. I regret not suggesting "-- debug" option earlier. I will ensure that our team is more proactive in suggesting this option in the future.
As for your question about the "move" image via Azure Portal GUI, please note that the "az_image_copy" method is still under preview, and this feature is not yet supported by the portal. We appreciate your patience and understanding as we work to resolve this issue.
Thank you for your continued support. You can delete all the new formed items such as vmimage and MyHPCImage-8 except for raw image MyHPCImage-7, and your debug output should help us solve the problem.

Hello, @evolkovgit. Thank you for bringing to our attention the new issue you are experiencing with the --temporary-resource-group-name parameter in the image copy command. We understand that this error is caused by a conflict in the name of the temporary resource group, which results from the resource group image-copy-rg not being deleted during your previous image copy command execution.
To solve this problem, we recommend two possible workarounds. First, if the image-copy-rg resource group is not being used for any business operations, it is recommended that you delete it and run the image copy command again. Second, if the image-copy-rg resource group is being used and cannot be deleted, you can add the parameter --temporary-resource-group-name {unused_group_name} to the image copy command.
We would also like to address your previous feedback about the inability to move images through the portal. Please note that the image copy method is maintained by an extension and does not guarantee a stable development and operation rhythm, and therefore cannot be supported in the portal.
Also, make sure you fix the previous bug where the system disk was deleted before running copy.

First, create a new virtual machine vm_temp by the image MyHPCImage-7.
Then, create a new image image_new for the virtual machine vm_temp.
Finally, use the new image image_new to perform a copy operation.

We recommend that when you need to move an image in the future, please move the image first and then delete the original VM.
Thank you for your understanding, and please let us know if you have any further questions or concerns.

Hi @wu5bocheng ,
I was finally ably to copy my image to another region using the workaround you suggested.
I appreciate your help and I am impressed how fast you answer.  Such care makes all the difference. Thank you very much.
You can close the issue if you want, though I think you understand: there is much to be done to improve the "az_image_copy" command.
Thank you again,
Eugene

Dear Eugene,
Thank you for reaching out to us and providing feedback on the extension module. We are delighted to hear that the workaround suggestion was helpful and that you were able to copy your image to another region.
We appreciate your kind words about the prompt response you received from us. Our team is committed to enhancing the user experience and maintaining the stability of the extension module. We value your support and feedback.
As per your request, we will close the issue. If you encounter any further issues or have additional feedback, please do not hesitate to submit a new issue or reopen the closed one.
Thank you again for your feedback and support.
Best regards,
Bocheng Wu


Learn RF for blinds

Should I leave the config file as this to let the home app learn automatically RF?
"data":{
"open":"2600500000012...",
"close":"2600500000012...",
"stop":"2600500000012..."
Or should I set it like this?
      "data":{
        "open":"",
        "close":"",
        "stop":""

I assume you got this working?
There's no learning automatically - you use the learn accessories to get a hex code back.


[Feature] add synapse hyperparams

This PR adds 3 new hyperparameters as sudo-protected constants to the subtensor pallet.

ScalingLawPower: u8 = 50
SynergyScalingLawPower: u8 = 60
ValidatorExcludeQuantile: u8 = 10
Each value is stored as an unsigned 8-bit integer
Each value x here represents the value x * 10^-2
Meaning each can store floats between [0.0, 1.0] with a precision of 0.01

Implemented by #18


RandomDestination at first & round-robin

Hello.
Logstash-logback-encoder is woking by primary and secondary.
I need a balanced request to each destination.
@brenuart registered the issue #125

The first set destination is random.
If the size is exceeded, round-robin occurs.

I hope to review the code.
Thanks.

I think we need to introduce a DestinationSelectionStrategy concept.
I can see three strategies at the moment:

PreferPrimary - the current strategy of always preferring the primary destination if it is available  (secondaryConnectionTTL is only applicable for this strategy)
RoundRobin - a new strategy round-robin connects to a destination (starting at the first) every time a reconnect occurs
Random - randomly selects a destination every time a reconnect occurs

Would those satisfy your use cases?
From the changes in the current PR, it is not obvious (without studying the code pretty hard) which strategy is being used for any given set of configuration values.   For example, how does secondaryConnectionTTL come into play with your new strategies?  What happens when it is set?  So, I think a first-class DestinationSelectionStrategy concept is needed in order to make it very explicit what strategy is being used.

+1
That would be very nice indeed.

@philsttr
I will modify it using the concept of the strategy you have told me.

@philsttr
Hello. I have developed three strategies. Please check again.

Hey @withccm,
I've refactored your code a bit to extract a strategy interface, with three implementations.  This pulls the logic out the appender, and allows people to develop custom strategies.
I've committed it to the destination_connection_strategy branch.  I still have more testing and documentation to do, but this will provide a preview of how I'm thinking this should work.

@philsttr, Thank you for your positive review.
I have reviewed the code you have been working on.
I would be willing to help if I needed anything.

4.10 released with this change

Thanks @philsttr


Sorting data based on the field takes a long time,

Sorting data based on the field takes a long time,
And while waiting for this sort to finish checking other tables query will get stuck and never finish (you need to reopen queries).
Maybe the issue is that you can send just one query at a time.

I think this issue was solved. @SharperPS please inform me if you get any other issues with the performance.


To continue your session, complete a brief verification.

iam getting this error when am trying to start the uploader from rdp client
i have no idea what is the problem here.
full error message: To continue your session, complete a brief verification. This extra step helps us keep your account safe by making sure it’s really you.

hello @fawazahmed0 i just found this

on my rdp
maybe this is the problem

i managed to bypass it and iam running a expermenting cycle to get to know why i faced this issues before.

Yeah, please create a PR, I will be happy to merge it

@fawazahmed0 after a lot of investigations when somebody gets this error remove the yt_auth folder then re run the uploading cycle :) that will bypass it.

please consider adding it in the docs and close the questions related to it good luck brother :) @fawazahmed0

Thanks @Un1xMida

most welcome buddy keep up the good work.


Selected Items don't disappear in Table view when deletion denied in modal in Application page

Notes for Reviewers
This PR fixes #6754
Signed commits

[x] Yes, I signed my commits.

Hey @leecalcote Please review the PR.

Hey @leecalcote . Please suggest if I have to make some changes in the PR.

@hs05june  Let's discuss this on the Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

Yes @Chadha93 . Filters page also requires an update. Designs page has already been updated in my previous PR.

@Abhishek-kumar09 The delete function which is being passed as a prop in the custom toolbar component is similar for applications and filters page. But it is very different for designs page. I thought to do that but couldn't figure out how to tackle the delete function.

https://user-images.githubusercontent.com/103871275/209952769-797e7fb0-ae8e-47d2-ad25-24ebba6df5b5.mp4
@Abhishek-kumar09 Fixed for filters page using the same custom toolbar select. Since, the delete function for designs wasn't same as in application and filter. I didn't use the same toolbar as made in pr #6758.


Add a cargo feature to disable the getpwuid_r fallback.

Currently dirs-sys falls back to calling getpwuid_r when the HOME environment variable is not set, on most Unix-like platforms.
This is undesirable for at least some users, because such behavior is not specified in the XDG Base Directory Specification and differs from the behavior of the widely used libxdg-basedir. And, calling getpwuid_r means depending on a part of libc which, on some platforms, calls dlopen and runs third-party code that uses filesystem and networking in-process, which can create problems for some sandboxing scenarios.
This PR puts getpwuid_r under control of a feature flag, enabled by default to maintain compatibility with existing users of the crate, but which can be disabled by users who don't desire the getpwuid_r behavior.

Hi Dan,
the use of getpwuid_r follows the std lib implementation: https://github.com/rust-lang/rust/blob/master/library/std/src/sys/unix/os.rs#L633, it's not related to XDG.
My considerations:

Whether spending some extra effort of retrieving the home dir is a good idea, not sure.
Though I can see the appeal of getting rid of the libc dependency.
I wouldn't give too much weight to other implementations' choices, in libxdg-basedir's case correctness does not appear paramount, considering it's written in an unsafe language in the first place.

All in all, I'd be happy to integrate this change, though looking at the evolution of the analogous std lib code, I wonder if the cfg lines should be extended similarly here (in addition to the feature toggle)?
What do you think?
Thanks,
Simon

Actually, after looking at this more, I'm no longer convinced this is worth pushing for. And I see your point about not reading too much into libxdg-basedir.
There doesn't seem to be a conensus in the Unix world that getpwuid_r is wrong, just that checking HOME first is right. Some projects have even added getpwuid_r checks when they found checking HOME wasn't sufficient. Looking at that, I'm unable to convince myself that Rust programs in general shouldn't do the same thing.
I already do have ways to work around this for my projects, so I think the thing to do is to leave things as they are!

Rustix is only a system call wrapper library, so it doesn't support getpwuid_r (or any of the NSS functions) so it wouldn't help here.
Currently what I mostly do in projects where this is an issue is I stub out getpwuid_r with a function that aborts and just always make sure HOME is set. I also maintain Mustang, which is more of a libc implementation, and it now has implementations of the NSS functions, but in order to do that without pulling in all the code needed to do dlopen and support arbitrary C code that can get loaded in, it runs the getent command and parses the output.

Maybe it's time to retire getpwuid_r by default, and make people opt into the insanity that is libc-people-doing-libc-things.

I'm no fan of it, but I think dirs isn't such an effective place to push back against it. On a practical level, changing this behavior at this level would mean Rust programs would "just work" in slightly fewer existing places, which would be unfortunate.

At least it might give me some better understanding of the type of systems that lack $HOME ... :thinking:
Okay, maybe I'll merge this as-is and try to gather some additional feedback. WDYT?

For my part, I changed my mind and am no longer pushing for this PR.


Not Support with videojs-youtube

I try to use videojs-youtube with pre roll ads videojs-vast-plugin. But it is not work.
If i use mp4 video, the pre roll ads work perfectly.
Can you fix it?

A little off topic but I think youtube prohibits showing ads when embedding youtube videos.


Removed Interlock Dead Link

Proposed changes
This ToC entry was already removed here https://github.com/docker/docker.github.io/pull/8668/files. However as part of the ToC moves its managed to find its way back in.
Thanks @stevenfollis for reporting.
Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit 69aa43a1da9ac2fa8a900e7d915426b69b154a59
https://deploy-preview-10136--docsdocker.netlify.com


Parameterize CRIO_CNI_PLUGIN

Allow overriding CRIO_CNI_PLUGIN (default: /opt/cni/bin) and
make sure it gets written to our crio.conf file. This is
intended for running cri-o tests with containernetworking-cni
rpm which installs into /usr/libexec/cni
Signed-off-by: Ed Santiago<EMAIL_ADDRESS>

/ok-to-test

/test all

/test e2e_fedora
/test e2e_rhel

@stevekuznetsov @cevich Can't seem to get tests to run?

@rhatdan not sure what you mean -- looks like they ran, only Guestbook failed on the e2es

Yup that confused me, since they used to fail internally but report success. not they are failing and reporting failure.  But if only guestbook failed, we are good.

LGTM

CI passes. I also have a successful run on my end, Fedora 26 with containernetworking-cni-0.5.1-1.fc26 rpm installed, nothing in /opt/cni, and CRIO_CNI_PLUGIN=/usr/libexec/cni. What needs to happen to merge this?
Side note: after careful reflection, I think most instances of /opt/cni/bin in this source tree should be changed to /usr/libexec/cni to conform with the current packaging of containernetworking-cni. That's beyond the scope of this PR, though.

I will merge it.


update some debs mostly for minimal-versions

This bumps the minimal acceptable versions in Cargo.toml to versions that are compatible with -Z minimal-versions. This is part of the process of seeing how hard this is for crates to use in preparation for getting it stabilized for use in CI, specifically upstreaming the changes required to get criterion working with it. It is easy to use if all of your dependencies support it, but much harder if trying to impose it on them.
I am not sure I have this exactly correct. There may be more things that need to be updated for the benchmarking code to work with minimal-versions. I was not able to test with all the feachers. But I have verified that docopt can be made to work with this change to regex. So it is a step in the correct direction.

cc @BurntSushi follow up to our discussion at https://github.com/Amanieu/thread_local-rs/pull/13

Also to be clear this is enough for cargo check --tests to pass in the bench subfolder, the main folder, and a copy docopt pointing at this on my windows machine. Running test, and running on other machines may find more.

Thanks!

To be a little more blunt, is there an ETA for a release that has this merged? Is there follow up that could hasen it?

I don't really do scheduled releases, but I just put regex 1.0.3 on crates.io, which has this PR.


Schema @deprecated directive not supported for Input fields / arguments

Small thing before I forget - @deprecated directives don't stick (though they're not rejected) when used on Input fields in a grackle Schema. This is in the working draft of the GraphQL spec but not the latest released version. Shall I add support or leave it for now? Also looks it will also be allowed on arguments in that version

I think we'll probably want an overall ticket for supporting spec revisions.


Dataflow Worker: Add worker_id to windmill request headers

Adds 'worker_id' to header for 'commitWork' and 'getData' streaming requests from Dataflow worker to windmill.
+R: @boyuanzz

Hey @kennknowles, could you please take a look at this PR?

Run Java PreCommit

seems like the new added worker field broke 3 unit tests: https://builds.apache.org/job/beam_PreCommit_Java_Phrase/321/testReport/. Could you please adjust the failed unit tests in this PR? @rangadi

Thanks @boyuanzz. I have updated the affected tests. ./gradlew test -p runners/google-cloud-dataflow-java/worker succeeded on my machine.

@lukecwik could you please help merge this PR after all tests passed?

Run Java PreCommit

Run Java PreCommit

This PR is ready to merge if there is no more concern. Could any committer help merge this PR?


Update dependencies and workflows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@9b3c973). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff            @@
##             main      #293   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        98
  Lines           ?      3132
  Branches        ?       375
========================================
  Hits            ?      3132
  Misses          ?         0
  Partials        ?         0

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


added revamped web app code

Great job @rajatsonibkn2002 👏
I will clone and try out the flows locally. Post this, I'll merge your PR.


title() and author() methods overwrite each other in the q param

Hi. I'm evaluating this library for use in a project and I've noticed that the title() and author() methods, rather than appending the value of the q parameter – as per the comments – seem to replace the q parameter entirely with the new key/value pair.
example:
public function searchByKeywords($phrase) {
  $client = new GoogleBooksApi(array('key' => API_KEY));
  $client->author($phrase)->title($phrase)->order('relevance')->take(MAX_RESULTS);
  return $client->get()
}

And the results with "gertrude stein" as the $phrase argument:
...
{
  "paramaters": {
    "country": "US",
    "q": {
      "intitle": "gertrude stein"
    },
    "download": "",
    "filter": "",
    "startIndex": 0,
    "maxResults": 40,
    "printType": "",
    "projection": "",
    "orderBy": "relevance"
  }
}

The results only show books about Gertrude Stein, not those by Gertrude Stein. I would expect the parameters property to look like {"intitle": "gertrude stein", "inauthor: "gertrude stein"} and to get results for both title and author.
Is this the intended behavior or a bug? Or am I missing something?
I can see a work-around by using string concatenation and the search() method, but the interface as described in the comments looks much cleaner.

Hello!
Sounds like a bug by all accounts. It's been awhile since I've looked at or used this package.
Are you able to make a pull request to fix this or did you need me to have a look?

I'll make a PR when I find a minute to fix it : )


Vagrantfiles: bump version to 144

This new VM caches the following image docker.io/cilium/istio_proxy_debug:1.0.0
Caching this image will reduce the amount of time the Istio CI tests take.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please

Coverage decreased (-0.03%) to 42.017% when pulling cf10fed879fdc05f22abf975fec1857d2cd381c2 on pr/ianvernon/bump-vm-image-istio-proxy into b60d512501f089854e25370fbc6768f20c1656d6 on master.

CI panicked? https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/11520/execution/node/83/log/

test-me-please


Smallfile dockerfile

This is the first time I've run smallfile inside an AWS cluster successfully.   I had to start trying out run_snafu.py inside a container built with smallfile_wrapper/Dockerfile.   A corresponding commit in ripsaw will be coming shortly, but you can test this on your local laptop by just building the container and running run_snafu.py within it.

better now?


GlWindow creation fails on Win10

I'm trying to run the window example on the master but it crashes at https://github.com/tomaka/glutin/blob/master/examples/window.rs#L11
OS: Windows 10
rustc 1.18.0 (03fc9d622 2017-06-06)
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', src\libcore\option.rs:329
note: Run with `RUST_BACKTRACE=1` for a backtrace.
^C
E:\Desktop\glutin-master>set RUST_BACKTRACE=1

E:\Desktop\glutin-master>cargo run --example window
    Finished dev [unoptimized + debuginfo] target(s) in 0.1 secs
     Running `target\debug\examples\window.exe`
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', src\libcore\option.rs:329
stack backtrace:
   0: std::sys_common::backtrace::_print
             at C:\projects\rust\src\libstd\sys_common\backtrace.rs:94
   1: std::panicking::default_hook::{{closure}}
             at C:\projects\rust\src\libstd\panicking.rs:354
   2: std::panicking::default_hook
             at C:\projects\rust\src\libstd\panicking.rs:371
   3: std::panicking::rust_panic_with_hook
             at C:\projects\rust\src\libstd\panicking.rs:549
   4: std::panicking::begin_panic<collections::string::String>
             at C:\projects\rust\src\libstd\panicking.rs:511
   5: std::panicking::begin_panic_fmt
             at C:\projects\rust\src\libstd\panicking.rs:495
   6: std::panicking::rust_begin_panic
             at C:\projects\rust\src\libstd\panicking.rs:471
   7: core::panicking::panic_fmt
             at C:\projects\rust\src\libcore\panicking.rs:69
   8: core::panicking::panic
             at C:\projects\rust\src\libcore\panicking.rs:49
   9: core::option::Option<&winit::platform::platform::events_loop::ThreadLocalData>::unwrap<&winit::platform::platform::events_loop::ThreadLocalData>
             at C:\projects\rust\src\libcore\macros.rs:21
  10: winit::platform::platform::events_loop::send_event::{{closure}}
             at E:\Rust\registry\src\github.com-1ecc6299db9ec823\winit-0.7.4\src\platform\windows\events_loop.rs:268
  11: std::thread::local::LocalKey<core::cell::RefCell<core::option::Option<winit::platform::platform::events_loop::ThreadLocalData>>>::with<core::cell::RefCell<core::option::Option<winit::platform::platform::events_loop::ThreadLocalData>>,closure,()>
             at C:\projects\rust\src\libstd\thread\local.rs:253
  12: winit::platform::platform::events_loop::send_event
             at E:\Rust\registry\src\github.com-1ecc6299db9ec823\winit-0.7.4\src\platform\windows\events_loop.rs:266
  13: winit::platform::platform::events_loop::callback
             at E:\Rust\registry\src\github.com-1ecc6299db9ec823\winit-0.7.4\src\platform\windows\events_loop.rs:305
  14: CallWindowProcW
  15: CallWindowProcW
  16: GetTopWindow
  17: KiUserCallbackDispatcher
  18: NtUserCreateWindowEx
  19: CreateWindowExW
  20: CreateWindowExW
  21: CreateWindowExW
  22: glutin::api::wgl::load_extra_functions
             at .\src\api\wgl\mod.rs:703
  23: glutin::api::wgl::Context::new
             at .\src\api\wgl\mod.rs:91
  24: glutin::platform::platform::context::Context::new
             at .\src\platform\windows\context.rs:71
  25: glutin::platform::platform::Context::new
             at .\src\platform\windows\mod.rs:69
  26: glutin::GlWindow::new
             at .\src\lib.rs:322
  27: window::main
             at .\examples\window.rs:11
  28: panic_unwind::__rust_maybe_catch_panic
             at C:\projects\rust\src\libpanic_unwind\lib.rs:98
  29: std::rt::lang_start
             at C:\projects\rust\src\libstd\rt.rs:51
  30: main
  31: __scrt_common_main_seh
             at f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl:253
  32: BaseThreadInitThunk

After a bit of debugging the issue seems to be caused by the extra dummy window created for context creation:

A new temporary window is created using the same callback function as the main window (https://github.com/tomaka/glutin/blob/master/src/api/wgl/mod.rs#L681-L711)
The callback function recieves commands from both windows (the original and the dummy window), but only our original window has an associated event loop, which initialized the thread-local data.
send_event and other events get executed during the lifetime of the dummy window, trying to access the thread-local storage on the main thread, not the thread owned by the main event loop causing a panic.

What would be the best way to fix this? Just early-out on the mentioned places if there is no thread local data?

Err thanks for the investigation. I really didn't think about that.
I think the proper fix would be to use a different window class, which is annoying.

Ok, thanks! I'll try to implement your solution as it sounds reasonable and better than the one I suggested.


typo: poartal -> portal

@nschonni : Thanks for your contribution! The author, @mattbriggs, has been notified to review your proposed change.


Appended "update" @visibility to BaseResourceProperties::marketplace field

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[x] Other, please clarify:

Extending Organization_Update operation for "edit recurring billing" feature. As we need to change @visibility of BaseResourceProperties::marketplace field, we can't control changes for existing API versions.

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

/pr RequestMerge

/pr RequestMerge

As requested offline, this PR is ready for merge. Adding the label.


Add credential presentation, rather than just a copy of the credential

we should be able to present the credential and the user can accept/reject, without copying it first.

will be in roots-react-native


fix(triggers): support disable mps by unbind

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SPARK-21843:testNameNote should be "(minNumPostShufflePartitions: 5)"

Signed-off-by: iamhumanbeing<EMAIL_ADDRESS>What changes were proposed in this pull request?
testNameNote = "(minNumPostShufflePartitions: 3) is not correct.
it should be "(minNumPostShufflePartitions: " + numPartitions + ")" in ExchangeCoordinatorSuite
How was this patch tested?
unit tests
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Is this test failing for you?

@srowen : this test is not failing, but the test name is not correct. it should be "minNumPostShufflePartitions: 5", cause the unit test code set minNumPostShufflePartitions to 5, not 3.
determining the number of reducers: aggregate operator(minNumPostShufflePartitions: 3)
determining the number of reducers: join operator(minNumPostShufflePartitions: 3)
determining the number of reducers: complex query 1(minNumPostShufflePartitions: 3)
determining the number of reducers: complex query 2(minNumPostShufflePartitions: 3)

Test build #3906 has started for PR 19058 at commit 4e201e5.

Test build #3906 has finished for PR 19058 at commit 4e201e5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to master


Add relate parameter to Otsuthreshold

This is related to OtsuThreshold is missing the relate parameter #403.
Adds a relate parameter to be passed to image_filters.OtsuThreshold class.

Description
Added relate arguments in init, and call functional otsu_threshold with.
Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[x] Enhancement/optimization
[x] Documentation

Issues Fixed or Closed by This PR

Fixes: #403.

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
I just changed the function documentation.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@ernestoarbitrio the CI is not passing because openslide-python has been updated, I opened PR #416 to support the new version. When that is merged we can rebase this PR on master and then merge it


Let Lyche admin see previous reservations

I think this should be a pretty straight-forward issue.

Don't really know why this was opened, there is already archive functionality in the Lyche admin panel.


request: change main accessory or separate tiles

I found that this purifier is being shown as an air quality tile, with options underneath. This makes no sense to me.
Can this be made an air purifier instead, which can actually have a state? (on/off).
Or, alternatively, separate the tiles in order make them 2 accessories instead.

Yes. I thinks it's strange

#17


Add support for fabric-permission-api

Are there any plans on adding support for fabric-permission-api?
Would be very useful.

closed by #5


chore(deps): update dependency opentelemetry.extensions.hosting to 1.10.0

This PR contains the following updates:

Package
Type
Update
Change

OpenTelemetry.Extensions.Hosting (source)
nuget
minor
1.9.0 -> 1.10.0

Release Notes

open-telemetry/opentelemetry-dotnet (OpenTelemetry.Extensions.Hosting)
v1.10.0

Bumped the package versions of System.Diagnostic.DiagnosticSource and other
Microsoft.Extensions.* packages to 9.0.0.

Added support for new APIs introduced in System.Diagnostics.DiagnosticSource
9.0.0:

InstrumentAdvice<T>
For details see: Explicit bucket histogram
aggregation.

Gauge<T>

ActivitySource.Tags
(supported in OtlpExporter & ConsoleExporter)

Experimental features promoted to stable:

CardinalityLimit can now be managed for individual metrics via the View
API. For details see: Changing cardinality limit for a
Metric.

The overflow
attribute
(otel.metric.overflow) behavior is now enabled by default. The
OTEL_DOTNET_EXPERIMENTAL_METRICS_EMIT_OVERFLOW_ATTRIBUTE environment
variable is no longer required. For details see: Cardinality
Limits.

The MetricPoint reclaim behavior is now enabled by default when Delta
aggregation temporality is used. The
OTEL_DOTNET_EXPERIMENTAL_METRICS_RECLAIM_UNUSED_METRIC_POINTS environment
variable is no longer required. For details see: Cardinality
Limits.

Added OpenTelemetrySdk.Create API for configuring OpenTelemetry .NET signals
(logging, tracing, and metrics) via a single builder. This new API simplifies
bootstrap and teardown, and supports cross-cutting extensions targeting
IOpenTelemetryBuilder.

Removed out of support net6.0 target and added net9.0 target.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Branch automerge failure
This PR was configured for branch automerge. However, this is not possible, so it has been raised as a PR instead.

Branch has one or more failed status checks


Transport to zip. Select multiples transport requests

Implements #578

I would like to show a screen with the transports selected after TRINT_SELECT_REQUESTS, this can be a future enhancement.

Yeah,  I'll look into it in a few hours. 😺


Add Java vagrant files

Added Java vagrant files. Please do not merge, requires further testing.
@alayek can you check on your system? I did not copy the bashrc as it was acting weirdly spewing several errors.

@koustuvsinha works mostly, except for not being able to recognize mvn.
This is the output of env command
M2=/bin
TERM=xterm
SHELL=/bin/bash
HISTSIZE=32768
CLICOLOR=1
DERBY_HOME=/usr/lib/jvm/java-8-oracle/db
SSH_CLIENT=<IP_ADDRESS> 47694 22
SSH_TTY=/dev/pts/1
LC_ALL=en_US.UTF-8
HISTFILESIZE=32768
USER=vagrant
LS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:su=37;41:sg=30;43:ca=30;41:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arj=01;31:*.taz=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.dz=01;31:*.gz=01;31:*.lz=01;31:*.xz=01;31:*.bz2=01;31:*.bz=01;31:*.tbz=01;31:*.tbz2=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.war=01;31:*.ear=01;31:*.sar=01;31:*.rar=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.webm=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.axv=01;35:*.anx=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=00;36:*.au=00;36:*.flac=00;36:*.mid=00;36:*.midi=00;36:*.mka=00;36:*.mp3=00;36:*.mpc=00;36:*.ogg=00;36:*.ra=00;36:*.wav=00;36:*.axa=00;36:*.oga=00;36:*.spx=00;36:*.xspf=00;36:
MAIL=/var/mail/vagrant
PATH=:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
PWD=/home/vagrant
JAVA_HOME=/usr/lib/jvm/java-8-oracle
EDITOR=/usr/bin/vim
LANG=en_US
PS1=[\[\e[0;32m\]\t \u@\h:\[\e[0;33m\]\w\[\e[0m\]]\[\e[0;36m\]$(parse_git_branch)\[\e[0m\]$(env_set) $ \[\e[m\]
HISTCONTROL=ignoredups
M2_HOME=/usr/share/maven3
SHLVL=1
HOME=/home/vagrant
LOGNAME=vagrant
J2SDKDIR=/usr/lib/jvm/java-8-oracle
SSH_CONNECTION=<IP_ADDRESS> 47694 <IP_ADDRESS> 22
LC_CTYPE=en_IN
LESSOPEN=| /usr/bin/lesspipe %s
J2REDIR=/usr/lib/jvm/java-8-oracle/jre
LESSCLOSE=/usr/bin/lesspipe %s %s
HISTTIMEFORMAT=%F %T
_=/usr/bin/env

The gradle is added to the path, along with Java and JVM.

@alayek I am running this to add the Maven path to bashrc, but nothing is being appended there :confused:
su - vagrant -c "echo 'export M2_HOME=/usr/share/maven3' >> ~/.bashrc"

btw @alayek please check whether tomcat is running on port 8080 (mapped with host) :white_check_mark:

Can you please try with the full path, /home/vagrant/.bashrc, instead of ~/.bashrc?
Also, please share the command I should use to start the Tomcat server.

@alayek bingo! although su -vagrant was not recognizing $M2 so manually replaced it!
Tomcat Server is already started (set to auto-start) when the box has been loaded.

I think the lesson here is that when you do su - <username> -c "some command", it evaluates the full path first before creating a subshell in user mode. Would be interesting to see if the same thing also happens in a here document.

If possible, please add Some bats based tests so that one can run them once the machine boots up. If you need help, refer to the dev-rails branch on this repo. @zcassini has added some nice and nifty tests!

Here are my typical vagrant up stats.
➜  java git:(java) date && vagrant up > logfile.txt && date
Fri Jun  3 00:07:51 IST 2016
Fri Jun  3 00:25:05 IST 2016

Here's my logfile.
I checked for Java, Java compiler, Maven, Gradle versions.
Please add the bats tests, with Tomcat, and I would merge.

@alayek can you check the bats tests? i am not much of a bash-foo so made only very primitive tests

New vagrant up stats
➜  java git:(java) date && vagrant up > logfile.txt && date
Fri Jun  3 13:51:00 IST 2016
Fri Jun  3 14:01:29 IST 2016

Here are the outputs from bats tests:
[01:34:22 vagrant@trusty64:/vagrant/tests] $ bats gradle.bats
 ✓ gradle version is 2 or higher

1 test, 0 failures
[01:34:28 vagrant@trusty64:/vagrant/tests] $ bats maven.bats
 ✓ maven version is 3 or higher

1 test, 0 failures
[01:34:38 vagrant@trusty64:/vagrant/tests] $ bats postgres.bats
 ✓ postgres is running
 ✓ can connect with psql
 ✓ postgres version is 9.3 or higher

3 tests, 0 failures
[01:34:44 vagrant@trusty64:/vagrant/tests] $ bats java.bats
 ✓ tomcat is running
 ✓ java version is 1.8 or higher

2 tests, 0 failures

@zcassini might wanna take a look at the bash tests.

It doesn't suppress red-text reliably, but since we are on a clock; let's merge for now.
LGTM :+1:
Let's :shipit: :package:


Allow some retries for setup_molecule run

Allow some retries for setup_molecule run
We see some unrelated failures while running the pod, during setup
stage: apparently, the DNS may fail to answer in time, and this crashes
the env setup.
With this patch, we should be in better position, since it would retry
to install the libs in case of failure, with a small delay.
Hopefully, this change will stabilize enough the job so that we may
consider it "voting" in a near future.
Ensure we don't run deploy jobs on unrelated modifications
A couple of files nested in ci/playbooks aren't used at all in any
deployment job.
Let's avoid useless resources usage whenever we modify them!
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

Note: the pre-commit job in zuul WILL fail - it needs #1538 to pass correctly.

/lgtm

/lgtm


Clean Junk

hey..plz this code give me in objective c

@Urvashi-iPhone You're getting down voted for just asking for a ton of work to be done, without attempting any of it yourself.
Have you tried to write the following repo in Objective C? If so, where did you have troubles?

Please close.


Can this support multiple spouses?

Hello,
Please add this feature. Thanks for this great work.

Hi!
This is already supported. The data attribute for marriages takes an array of spouses.
/Erik


util: Convert find_missing_*_translations to typescript

Working on the new test_timeline.ts file, I wanted to move the building of argparse and handling of args for each util out of index.ts and to their own file so that the index.ts file doesn't bloat as more tools are added.
While trying to move said code responsibility out of index.ts, I ran into issues because these files weren't typescript, so I converted them.
Also there were some bugs that I fixed along the way.

Looks good for fr. I didn't check any of the other languages, since they should all function the same?
$ npm run find-translations -- --locale=fr -f cactbot

><EMAIL_ADDRESS>find-translations
> npm run util -- findTranslations "--locale=fr" "-f" "cactbot"

><EMAIL_ADDRESS>util
> ts-node util/index.ts "findTranslations" "--locale=fr" "-f" "cactbot"

(node:1472) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
/mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/radar/radar.ts:176 "const pullText: LocaleText = {"
/mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/radar/radar_config.ts:72 "name: {"
/mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/radar/radar_config.ts:83 "name: {"
/mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/pharos_sirius.ts:42 "text: {"
/mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/the_lost_city_of_amdapor.ts:36 "text: {"
/mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/03-hw/trial/sephirot-ex.ts:45 "text: {"
/mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/03-hw/trial/sephirot-ex.ts:58 "text: {"

If you have the time to run all of them, it might be worth doing:
npm run find-translations -- --locale=de -f .
npm run find-translations -- --locale=fr -f .
npm run find-translations -- --locale=ja -f .
npm run find-translations -- --locale=cn -f .
npm run find-translations -- --locale=ko -f .

and sending it to a file and comparing the output of that before and after this patch?

It takes a surprisingly long time to run, that's why I only checked one lang key, but I guess I can script it out...

Also there were some bugs that I fixed along the way.

I see the Set length vs size there.  Were there others that you were thinking about here?

There was a call to basically new Set(['a']).delete(new Set(['a'])) which did nothing:
https://github.com/quisquous/cactbot/pull/4466/files#diff-bf3d028e14b6b0f30ea4df0fd83c6f9d11feaaaccd2facf0f95fe144f143c39bL110-R118

And some fun stuff with strings and sets:
https://github.com/quisquous/cactbot/pull/4466/files#diff-bf3d028e14b6b0f30ea4df0fd83c6f9d11feaaaccd2facf0f95fe144f143c39bL114-R123
I think there were one or two other things along those lines as well.

The old and new versions produce the same output, with a difference in file order which I think isn't caused by this change?
$ for i in de fr ja cn ko; do echo $i;diff ../test_tl-*-fix-$i.txt;echo;echo; done
de
9c9
< (node:1891) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
---
> (node:2692) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time

fr
9c9
< (node:1973) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
---
> (node:3340) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time

ja
9c9
< (node:2013) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
---
> (node:3958) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
20a21
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/oopsyraidsy/oopsy_live_list.ts:19 "const errorMessageEnableACTWS = {"
162,165d162
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:35 "text: {"
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:70 "text: {"
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:104 "text: {"
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:130 "text: {"
169a167,170
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:35 "text: {"
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:70 "text: {"
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:104 "text: {"
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/04-sb/trial/rathalos.ts:130 "text: {"
363a365
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/06-ew/hunts/labyrinthos.ts:34 "followCharge: {"
369d370
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/06-ew/hunts/labyrinthos.ts:34 "followCharge: {"

cn
9c9
< (node:2053) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
---
> (node:4592) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
12,13d11
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/snowcloak.ts:22 "text: {"
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/the_lost_city_of_amdapor.ts:36 "text: {"
15a14,15
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/snowcloak.ts:22 "text: {"
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/the_lost_city_of_amdapor.ts:36 "text: {"

ko
9c9
< (node:2093) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
---
> (node:5231) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
14d13
< /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/the_lost_city_of_amdapor.ts:36 "text: {"
15a15
> /mnt/c/Users/valarnin/Source/valarnin-cactbot/ui/raidboss/data/02-arr/dungeon/the_lost_city_of_amdapor.ts:36 "text: {"

Yeah, that ordering difference is very weird and would maybe be nice to fix up later, but not an issue for this PR.  Thanks for taking the time for the full check; it just seemed like no reason to not make sure all the output was identical.