KafkaConnect configuration does not support clustered endpoint

Describe the bug
KafkaConnect is meant to run as a cluster with multiple hosts balancing the load using Kafka as communication mechanism (same as SchemaRegistry). Each node provides API endpoint that affects entire cluster. If one node goes down the cluster still functions.
This UI assumes only single endpoint in
Set up
Docker image 0.2.1
Steps to Reproduce
Steps to reproduce the behavior:

Have multiple KC nodes (in our case 5+)
Try to define KC configuration in envvars in a way that allows taking down KC nodes for maintenance without disrupting UI functionality (in this setup tried http://redacted1:8016,http://redacted2:8016,http://redacted3:8016)
Above is impossible because configuration takes only single host

Expected behavior
Can define comma-separated list of hosts (same as some other tools do)
Logs
07:56:06.544 [reactor-http-epoll-2] ERROR org.springframework.boot.autoconfigure.web.reactive.error.AbstractErrorWebExceptionHandler - [37d4100d]  500 Server Error for HTTP GET "/api/clusters/cluster_name/connectors?search="
org.springframework.web.reactive.function.client.WebClientResponseException$NotFound: 404 Not Found from GET http://redacted1:8016/,http:/redacted2:8016,http:/redacted3:8016/connectors
	at org.springframework.web.reactive.function.client.WebClientResponseException.create(WebClientResponseException.java:185) ~[spring-webflux-5.2.3.RELEASE.jar!/:5.2.3.RELEASE]
	Suppressed: reactor.core.publisher.FluxOnAssembly$OnAssemblyException:
Error has been observed at the following site(s):
	|_ checkpoint ⇢ 404 from GET http://redacted1:8016/,http:/redacted2:8016,http:/redacted3:8016/connectors [DefaultWebClient]
	|_ checkpoint ⇢ Handler com.provectus.kafka.ui.controller.KafkaConnectController#getAllConnectors(String, String, ServerWebExchange) [DispatcherHandler]
	|_ checkpoint ⇢ com.provectus.kafka.ui.config.CustomWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ com.provectus.kafka.ui.config.ReadOnlyModeFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.authorization.AuthorizationWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.authorization.ExceptionTranslationWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.authentication.logout.LogoutWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.savedrequest.ServerRequestCacheWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.context.SecurityContextServerWebExchangeWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.context.ReactorContextWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.header.HttpHeaderWriterWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.config.web.server.ServerHttpSecurity$ServerWebExchangeReactorContextWebFilter [DefaultWebFilterChain]
	|_ checkpoint ⇢ org.springframework.security.web.server.WebFilterChainProxy [DefaultWebFilterChain]
	|_ checkpoint ⇢ HTTP GET "/api/clusters/cluster_name/connectors?search=" [ExceptionHandlingWebHandler]
Stack trace:
		at org.springframework.web.reactive.function.client.WebClientResponseException.create(WebClientResponseException.java:185) ~[spring-webflux-5.2.3.RELEASE.jar!/:5.2.3.RELEASE]
		at org.springframework.web.reactive.function.client.DefaultClientResponse.lambda$createException$1(DefaultClientResponse.java:209) ~[spring-webflux-5.2.3.RELEASE.jar!/:5.2.3.RELEASE]
		at reactor.core.publisher.FluxMap$MapSubscriber.onNext(FluxMap.java:100) ~[reactor-core-3.3.2.RELEASE.jar!/:3.3.2.RELEASE]
		at reactor.core.publisher.FluxDefaultIfEmpty$DefaultIfEmptySubscriber.onNext(FluxDefaultIfEmpty.java:92) ~[reactor-core-3.3.2.RELEASE.jar!/:3.3.2.RELEASE]
		at reactor.core.publisher.FluxMapFuseable$MapFuseableSubscriber.onNext(FluxMapFuseable.java:121) ~[reactor-core-3.3.2.RELEASE.jar!/:3.3.2.RELEASE]
		at reactor.core.publisher.FluxContextStart$ContextStartSubscriber.onNext(FluxContextStart.java:103) ~[reactor-core-3.3.2.RELEASE.jar!/:3.3.2.RELEASE]
		at reactor.core.publisher.FluxContextStart$ContextStartSubscriber.onNext(FluxContextStart.java:103) ~[reactor-core-3.3.2.RELEASE.jar!/:3.3.2.RELEASE]

Additional context
It is same exact issue as #886

Thanks for reporting the issue, we'll aim to fix this in 0.4.

The implementation could be similar to #886, up for grabs


Questions about ‘Jacc score’ and ‘Jacobian determinant’

Hello, may I ask that ‘Jacobian determinant’ should stand for Jacobian determinant, which is used to evaluate the folding situation of the deformation field. So what does ’Jacc score‘ stand for? What does it mean physically in registration?
Looking forward to your reply!

Hello, I observed that in the article of ICCV2019, you used DICE as the evaluation metric, and in the article VTN, you used IoU as the evaluation metric. Since I do not understand the significance of IoU as a metric, I have read your VTN article again. You have this sentence in the article.
"The Segmentation IoU is computed for the volume and not the sections. It might well be the case that the overlap is not so satisfactory when viewed from those planes yet is better when viewed as a volume."
Q1:Why did you use different metrics in the two papers instead of both Dice and IoU？Is there any meaningful difference between the two metrics?
Q2:Could you give me more explanation about the difference between "Volume "and" section "  in the first sentence?
Q3:Is the Segmentation IoU computed for the sections? And IoU is computed for the volumes?
The problem I face is that I don't know which metric to use in my paper, and I still don't know the meaning of IoU.
Looking forward to your reply. Thank you.

Both Jacc score and Dice score are a measurement of segmentation IoU, and they are largely correlated. They are both computed for the volumes. We used Dice score in the ICCV article only because we found some other papers using Dice score so we thought that it may be more easily acceptable.

Thank you for sharing your Jupyter code. It is very helpful for me. I can successfully draw the flow_filed. Since I can draw flows, I assume that it is the same true for grids. But the grid graph has some problems.  I hope you can take a look at it. Thank you!
Fig.1 is my flow_filed, it is right.

Fig.2
results[key][0]               shape is :(128-128-128-3)
results[key][0][:, :, 64]    shape is :(128-128-3)

Fig.3 Call your jupter code and we do the "Debug" operation. As is shown in the Fig.4,  we find the value of the inv_flow between(-0.89,0.89). It's too small and the grid  is blank. So we multiply x times 64. Then we find the value of the inv_flow1 between(-57.0,57.5).
Fig.3

After the multiply operation, we can get a grid graph as Fig.4. But we find that the gird graph is not total. So we alter the code in line 47:plt.axis((-256, 256, 256, -256)).  At last, we obtain the gird graph in Fig.6.  However, it's so strange.
I think this figure is wrong, the figure is seriously distorted, but the flow field is correct. Could you please give me some suggestions? I watched it for a long time, and I didn't want to give up when I got to this point.Looking forward to your reply!
Fig.4

Fig.5

Fig.6

How did you compute the inverse field?

You are reminding me that I saw the inverse code you wrote, but I notice that you didn't call it. Could you tell me where should I call it?


Tappx Adaptater out of date

Hi, I'm Albert Grandes from Tappx.
Some of our publishers use the magnite adapter. However, we have had problems with these publishers because the magnite adapter is not up to date with the latest changes to the prebid adapter in go.
In the latest version of prebid Server we add new parameters (https://github.com/prebid/prebid-server/commit/c01c993fb5f0c86475f9650fbea6369a7cb3780e).
Also, in another commit we made important changes that should be integrated into the java adapter (https://github.com/prebid/prebid-server/pull/1777).
Thanks for everything.

Hi Albert, we're on track with changes in Tappx and will implement they next week.

https://github.com/prebid/prebid-server-java/pull/1410 merged. Closing.


DBT model fails to build when using a ClickHouse profile

Describe the bug
When building a dbt model in a code block targetting a ClickHouse profile, the block fails with the following error:
Exception: No configuration matching profile type clickhouse. Change your target in /home/src/default_repo/dbt/demo/profiles.yml or add dbt_profile_target to your global variables.
To Reproduce
Steps to reproduce the behavior:

Install dbt-clickhouse==1.4.2 through the Mage project requirements.txt file.
Create a profile in the profiles.yml file from the dbt project adding the needed configuration for ClickHouse, as specified here.
Create a new pipeline
Create a dummy DBT model using a DBT block

Click on Compile & preview
An error is thrown in the output.

Expected behavior
No errors in the output.
Desktop (please complete the following information):

OS: macOS Ventura 13.0.1
Browser: Firefox
Version: 114.0.2

Digging a little bit into the code I managed to provide a fix for this issue.


H743 and H743 DFSDM offsets wrong

The offsets appear to be correct for all other H743 variants. They all appear to be wrong on H743 and H743V. Working on a fix, but can't get changes to take ATM.
Attempting to apply a common patch of the form:
DFSDM:
  _modify:
    DFSDM_CHCFG0R1:
      addressOffset: 0x00
    DFSDM_CHCFG0R2:
      addressOffset: 0x04
    DFSDM_AWSCDR0R:
      addressOffset: 0x08
    DFSDM0_DAT0R:
      addressOffset: 0x0C
    DFSDM0_DATIN0R:
      addressOffset: 0x10

    DFSDM_CHCFG1R1:
      addressOffset: 0x20
    DFSDM_CHCFG1R2:
      addressOffset: 0x24
    DFSDM_AWSCDR1R:
      addressOffset: 0x28
    DFSDM0_DAT1R:
      addressOffset: 0x2C
    DFSDM0_DATIN1R:
      addressOffset: 0x30.
..

Additionally, the SVD names don't match the form of the RM. Eg FLT0CR2 vs  DFSDM0_CR2 and CH2CFGR1 vs DFSDM_CHCFG0R1
Also, indexed channels and filters would be nice, eg H7's DMA streams. I'm going to take a stab at the wrong addresses. Any ideas why the fixes I put in aren't taking?

Could you push a branch that shows them not working? It's hard to tell from just that snippet, which looks OK in isolation.

Done - https://github.com/stm32-rs/stm32-rs/pull/637

I think the "not working" is due to typos in the reg names I added.

If nothing else, we usually strip the peripheral name from field names using the _strip command for things like this (example).

Added strip, and the address offsets appear to have taken. Leaving as draft until further QC and ops testing. Appreciate it!

Nice! Annoyingly it seems like the names such as DFSDM3_CR2 get turned into plain 3_CR2 which is not valid (cannot start with a number). Looks like that's only an issue on 743/743v, so perhaps that patch file could also rename those registers to the FLT3CR2 name used elsewhere.

That would be idea; make it all match RM.
Of note, it seems like it's prefixing _ to make it valid, but I'm running into something very weird where it's expecting 2 of the regs (_1_FCR and _3_FCR to be methods (?) based on compile errors, and being ommitted from the top-of-the-module list of offsets. Not sure what's up. Could just make all the names match the RM.

svd2rust will prefix _, but the generated SVD file fails validation against the SVD specification because of the invalid names. Since you're already having to put the new address offset for the affected registers, you could also put name: in that same block to rename them.
If it's turning registers into methods, that's typically because they overlap or have the same address (like CCMR input/output mode registers on STM32 timers), so they are taken out of the register block and the methods are used instead.

Names fixed across H7

Fixed by #637


[WFLY-14022] Enable a way to ignore the MixedDomainDeployment700TestCase due to ARTEMIS-2800.

The first commit just allows the mixed-domain server to be overridden.
https://issues.redhat.com/browse/WFLY-14021
https://issues.redhat.com/browse/WFLY-14022

Custom PR run https://ci.wildfly.org/buildConfiguration/WFPR/228327

/retest

I updated CI instead and kicked off new jobs.


Issue with default scrollStrategy

As I understood from README, by default, when popover is visible and you scroll the background, it should stay attached to the anchor, repositioning itself. It worked just fine when I tested it in stackblitz.
In one page I used it, for some reason it didn't stay attached to the anchor.  It seems to be attatched to mouse position as I scrooll the background using the mouse wheel. I couldn't isolate the cause yet nor reproduce it in stackblitz.
If I have any progress with reproducing it, I'll let you know.

Is it consistent on that one page? Gif/video? And is it hover or click and on what kind of anchor?
Also yeah it's always reposition in beta.0 but you'll have the option to block in the next release!

In fact, I wanted the default behavior.
I don't have any gif/video because when I started oCam it interfered with the issue (it doesn't fix it, but it behaves different). I'll post a gif anyway in a few minutes (just trying some styling in popover).
It's on hover event.

Here it is. BTW, it's anchored in a regular div. Maybe the overall structure of the page is causing it. It's a flex-layout item and inside it there's a <mat-list>. I tried to take away <mat-list> and use a simple text instead of it, but the issue remained. Dispite the fact that oCam is messing a little with the behavior I'm seeing, basically it seems that when I scroll the page, the popover takes around 1 sec to close.

The popover is basically:

Legenda

<sat-popover #popover xPosition="after" [overlapAnchor]="false">

    <mat-list>

Legenda

      <mat-list-item>
        <mat-icon mat-list-icon>spellcheck</mat-icon>

 Indicador possui ação cadastrada

      </mat-list-item>
      <mat-list-item>
        <mat-icon mat-list-icon [style.color]="'#ffeeba'">format_color_fill</mat-icon>

 Desempenho abaixo da meta

      </mat-list-item>
    </mat-list>

</sat-popover>

Your video definitely looks like mouseleave is not firing until later. Is that because of oCam or is that the usual behavior? And can you verify it by wrapping the popover.close and including a console.log?
Another troubleshooting measure may be to switch to (click)="popover.toggle() to see if repositioning can still work like that.

Take a look at the results.
(click)="popover.toggle()"

wrapping the popover.close and including a console.log
In this case, clearly the (mouseleave) is beeing fired after I stop scrolling and move the mouse again.

Is the scrolling happening on the body or some other container (like a sidenav container)? I wonder if applying cdkScrollable to the scrolling container would fix it... I haven't seen any issues locally yet, but I've just been scrolling the body in all my tests.
Also I'll try to get the next release out early this week and you can see if the block/noop behaviors behave differently.

And if you really want to dig into it... 😄

Open<EMAIL_ADDRESS>

Find this section,
RepositionScrollStrategy.prototype.enable = function () {
    var _this = this;
    if (!this._scrollSubscription) {
        var /** @type {?} */ throttle = this._config ? this._config.scrollThrottle : 0;
        this._scrollSubscription = this._scrollDispatcher.scrolled(throttle, function () {
            _this._overlayRef.updatePosition();
        });
    }
};

Add a console.log before _this._overlayRef.updatePosition()

If it's not being called, then it's likely something that we can fix by applying cdkScrollable (or identifying a cdk bug?)

If it is being called, then maybe there's a bug in the popover position strategy that we can fix.

Is the scrolling happening on the body or some other container

Other container. I have a menu bar on the top that cannot be a part of the scrollable region
The first thing I'll try out is the cdkScrollable. Then I'll take a deeper look into the file you suggested.

cdkScrollable solved it 😃

Great! I'll make a note to add that to the readme

Yeah... it's a important note.

cdkScrollable solved it

I have the same problem and how 'cdkScrollable' should be implemented to solve it?


Subtitles and Thumbnails are downloaded multiple times when output template has format_id specified

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2021.12.01. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
yt-dlp -wv -o "%(uploader)s-%(upload_date)s-%(title)s-%(id)s.f%(format_id)s.%(ext)s" --restrict-filenames --write-thumbnail https://www.youtube.com/watch?v=0UyXSJhbFHA --write-sub en --convert-subs srt -f 18,22,140,251
Unless I'm mistaken(please correct me if I am), I don't know of any youtube/vimeo/dailymotion/twitch or any streaming video website's webpage, where subtitles and thumbnails change depending on the format selected.
The same thumbnails and subtitles are being re-downloaded for each -f format_id .
Subtitles and thumbnails should only be downloaded once regardless of how many -f I query.
Thank you.
Verbose log
$ yt-dlp -wv -o "%(uploader)s-%(upload_date)s-%(title)s-%(id)s.f%(format_id)s.%(ext)s" --restrict-filenames --write-thumbnail https://www.youtube.com/watch?v=0UyXSJhbFHA --write-sub en --convert-subs srt -f 18,22,140,251
[debug] yt-dlp version 2021.12.01 [91f071a] (py2exe)
[debug] Optional libraries: mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=0UyXSJhbFHA
[youtube] 0UyXSJhbFHA: Downloading webpage
[youtube] 0UyXSJhbFHA: Downloading android player API JSON
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, codec:vp9.2, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[debug] Downloading subtitles: en-US
[info] 0UyXSJhbFHA: Downloading 4 format(s): 18, 22, 140, 251
[info] Writing video subtitles to: vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.f18.en-US.vtt
[debug] Invoking downloader on "https://www.youtube.com/api/timedtext?v=0UyXSJhbFHA<REDACTEDREDACTEDREDACTEDREDACTEDREDACTEDREDACTEDREDACTED>&lang=en-US&fmt=vtt"
[download] Destination: vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.f18.en-US.vtt
[download] 100% of 5.74KiB in 00:01
[info] Downloading video thumbnail 41 ...
[info] Writing video thumbnail 41 to: vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.f18.webp
[SubtitlesConvertor] Converting subtitles
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.f18.en-US.vtt" -f srt "file:vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.f18.en-US.srt"
Deleting original file vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.f18.en-US.vtt (pass -k to keep)
[debug] Invoking downloader on "https://LONG_GOOGLEVIDEO_URL_THAT_I_HAVE_REDACTED"
[download] Destination: vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.f18.mp4
.
.
.
AND SO ON...BASICALLY RE-DOWNLOADS THUMBNAILS AND SUBTITLES ONCE FOR EACH -f I SPECIFY.

That is expected behavior. Otherwise what would the format_id in the subtitle's filename be? You can set a separate output template for the subs/thumbnails

Can't format_id and other non-subtitle specific,non-thumbnail-specifc output template fields be ignored while writing subtitles and thumbnails?
Since it's the same thumbnails and subtitles being re-downloaded over and over?
Basically while writing subtitles it should just be vlogbrothers-20211207-Reading_Hank_s_Decade-Old_Tweets-0UyXSJhbFHA.vtt

You can set a separate output template for the subs/thumbnails


Fluentbit configuration does not reload

Describe the bug

If you update the fluentbit config part, it's not reloaded in fluentbit
(I believe it can't even do this)
Steps to reproduce the issue:

Update the fluentbit config.
Expected behavior

Reload the config without rereading all log files
Screenshots

Additional context

Fluent-bit configuration is not (yet) intended to reload, it only works as a collector. The configurations are generated and only disruptive (re-create pods) updates possible (https://github.com/fluent/fluent-bit-docs/issues/210). Do you have specific use-case for this?

The use case would be testing and setting up the log collector.
I don't believe that during normal operation it should need to do this, but it's very confusing that all parts are updated by the operator, but the fluentbit part is not restarted.


SHA256 Intgerity Tag Issues with Firefox?

I'm having a strange issue in Firefox where my assets are failing the integrity check.
The console error from Firefox:

None of the "sha256" hashes in the integrity attribute match the content of the subresource.

My asset _config options:
# Asset Settings
assets:
  cache: false
  sources:
    - _assets/pdf
    - _assets/icons

This happens both on the production server and in a localhost environment. The site works fine in other browsers (Safari, Chrome). Is Firefox the only browser actually enforcing the integrity check? I'm not sure why it would fail to begin with though.
I link to my assets using the given tags:

{% css main %} and {% asset_path %} in HTML files
font_path() and image_path() in scss files

Any ideas on how I could trace down the issue here?

Chrome does indeed do integrity checks, so does Edge... Safari on the other hand has no support (AFAIW and last I remember -- that very well could have changed in the last few months.)  I just added an asset manually and edited the SHA and Chrome did reject the asset entirely.  Does Firefox give you a message in the console?  I know that only Firefox 43+ (12-15 Release) is the one that supports integrity (here we are on 46 Stable)
{% asset_path %} should not push out an integrity if it does please let me know, that is a bug.  Our helpers for CSS also do not push out integrities because CSS has a lack of support for them (right now, I don't know if that's planned or not.)   Actually I don't even think it's necessary for CSS since it's only images but perhaps.
1.) What happens when you switch to: {% css main crossorigin:anonymous %}?
2.) What error does Firefox give you?

The easiest way to debug is to shim the entire site in spots so that you can get the asset's cache path and run a manual integrity check on that path... I should probably log that. That might be too much work for you to bother with so the real easiest way would be to give us access to your source via Github so we can clone it and do that for you since I've tools designed to do exactly that in Jekyll.
However, I just double checked on my own site (which currently uses HEAD) -- it's only one feature ahead of 2.2.3 stable in that it has jekyll_assets_multi filter, so it is exactly the same for the feature you're having trouble with.

I tried `{% css main crossorigin:anonymous %}' with no success.
The exact error from the Firefox console (on localhost) is:

None of the "sha256" hashes in the integrity attribute match the content of the subresource. localhost:4000

The site is live with this issue, you can check it right now at:
http://wiki.shadowlinkit.com
And the github repo for the site:
https://github.com/ckreon/netp_wiki
I reproduced this on Firefox 45.0.2 running OS X 10.11.4 and Windows 7 x64.

This is definitely a bug in Firefox:
cat main-ca510268d1ad5ab5a3d92a7039be2963d00b27ee40942a5c4735276300bf5856.css \
  | openssl dgst -sha256 -binary | openssl enc -base64 -A

ylECaNGtWrWj2SpwOb4pY9ALJ+5AlCpcRzUnYwC/WFY=

However, I will concede that it could be because of the lack of COR's support on behalf of some servers (but I'll also be honest that, that should not bring down the asset, that should be fairly irrelevant.)  I think the best course of action is for me to just issue 2.2.4 to make integrity checks off by default on by choice that way this issue can be worked around easily.

Interesting - I'm surprised that it wasn't something on my end.
Is there steps I should take to submit this to Mozilla?
Also, is the full options list documented somewhere? When I first had this error, I checked to see if I could just turn off the integrity tags, but didn't find a relevant value (or so I thought).
Huge fan of your work on this project btw - thanks for everything!

If you can fork the source into a temporary repo that is reachable by web and hit up https://bugzilla.mozilla.org and file a bug they should be able to guide you from there.  @firefox on Twitter should be able to help you a bit with what they would need too. I would simply file a ticket and state that "sha integrity check fails" and give them the proof that the sha is valid and then link them to that fork and go from there.
We do provide a full list of our options at: https://github.com/jekyll/jekyll-assets#configuration at this time we do not allow the enabling and disabling of integrity, it's just there, I didn't assume browsers would have problems with it, so I just put it in there as a neat feature.  In the next few minutes I'll just that so it's off by default and a feature flag.

Ok, ya I use your whole README extensively, just wasn't sure if that was everything.
Thanks for the help!

Hi this is Hulk,
This ticket is flagged for a feature, I have reopened it for you, so that it can be closed with that feature.

I solved this issue by loading the page on chrome, which outputs the correct sri hash on console. Using the hash from the console fixed the loading in firefox too

I submitted this as a bug to the Mozilla team a while back. It is fixed and scheduled to be pushed with release 52, which should be available March 7th. You can verify the fix right now with either the nightly or developer build.

Hi @ckreon, I'm dealing with this issue right now. By chance chance would you be able to direct me to the bug report you filed with Mozilla? Thanks!

https://bugzilla.mozilla.org/show_bug.cgi?id=1269241

Remind Me to Work on this issue.

Even though the other ticket is newer it's already tracked so closing this in favor of the newer ticket #582.


Update contributing.md

Signed-off-by: Michael Aimino<EMAIL_ADDRESS>Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[ ] If the changes in this PR is part of the next future release, make this pull request against the docs-staging branch which will be published at the next release boundary. If the changes in this PR are part of the current release, use the default base branch, master. For more information about branches, see https://github.com/zowe/docs-site/tree/master#understanding-the-doc-branches.

[ ] If this PR relates to GitHub issues in docs-site or other repositories, please list in Description, prefixed with close, fix or resolve keywords.

Description (including links to related git issues)
Made an update to contributing.md. The first PR is created is mangled, so I deleted it and created this PR.
:heart:Thank you!

I'm actually planning to setup a Netlify preview link against the docs-staging branch.  So that with every PR - a Netlify link will be generated similar to what's generated against the master branch.
If the build fails. It will probably throw an error & will help to debug fast. :)

@covalentbond Thank you for reviewing and addressing the issues! For Netlify, I'll work with John to set it up for the docs-staging branch using the OMP Netlify account. : )


[DigitalOcean driver] Fails to remove created droplets after CTRL+C on create

I was creating some DO instances and the "Wait for SSH" bit was taking really long (possibly just straight-up freezing), so I hit CTRL+C.  I could see that they got created in the DO admin panel, so I'm not sure what the hangup was, but when I went to docker-machine rm the hosts they were still around and I just got a message saying:
$ ./docker-machine_darwin-amd64 rm prod1 prod3
INFO[0000] Digital Ocean SSH key doesn't exist, assuming it is already deleted
INFO[0000] Digital Ocean droplet doesn't exist, assuming it is already deleted
INFO[0000] Digital Ocean SSH key doesn't exist, assuming it is already deleted
INFO[0000] Digital Ocean droplet doesn't exist, assuming it is already deleted

The assumption is bad!  They were not deleted already.  Seems that the detection of their existence is off too.
Having the "broken" refs around in an error state is cool and expected but I think this seems a bug to me - rm should clean up instances that exist whether they're in a happy state or not.

Perhaps we need to SaveConfig earlier to get the creds, etc.

This seems to be an issue with other drivers as well. The instance gets created and if you ctrl-c while machine is waiting for ssh then the config ends up without the correct or any data.
So how about the Create() gets refactored to just do creation, then a SaveConfig() to end up in a defined state and use that to wait for ssh to get available?

@knarz We should be really close to that after this latest refactor.  We now have the drivers just creating the instance and then call "Provision" after that.  I will do some testing to see if this works with various drivers.

We need to revisit this in a different light given recent realizations


dxvk injection support

any future support?

if you are not using reshade then just try just renaming it to ReShade64.dll

@dnblank123 I got a solution, but you will have to be patient...
Firstly, english is not my primary language so sorry for any thing error in my grammar!
I made some changes to inject the dxvk, the dev (@34736384 ) can feel free to adapt it to their project.
First you need add this dlls (dxvk 64bits) in the same folders that you will add 'unlockfps.exe', but in 'UNLOCKFPS FOLDER/inject/64/'

Secondly you neesd add this lines later InjectReshade(pi.hProcess); and the delay is required because i dont know why this dont work when is instantly:
std::this_thread::sleep_for(std::chrono::milliseconds(1150));
Injectdxgi_64(pi.hProcess);
Injectd3d11_64(pi.hProcess);

and add this code to load the dlls:
(just copy and edit the function InjectReshade)

and done! If you wanna see if work it just use processhacker (you can find it in google, but open only later that alrealdy opened genshin, because genshin dont open when processhacker is alrealdy opened) and will see in module one 'd3d11.dll' and two 'dxgi.dll' (i dont know how unload system dxgi, but calm down, dxvk d3d11 only work if dxvk dxgi is working case else will show a resolution error)

I believe that this can be improved :)

@dnblank123 I got a solution, but you will have to be patient...
Firstly, english is not my primary language so sorry for any thing error in my grammar!
I made some changes to inject the dxvk, the dev (@34736384 ) can feel free to adapt it to their project.
First you need add this dlls (dxvk 64bits) in the same folders that you will add 'unlockfps.exe', but in 'UNLOCKFPS FOLDER/inject/64/'

Secondly you neesd add this lines later InjectReshade(pi.hProcess); and the delay is required because i dont know why this dont work when is instantly:
std::this_thread::sleep_for(std::chrono::milliseconds(1150));
Injectdxgi_64(pi.hProcess);
Injectd3d11_64(pi.hProcess);

and add this code to load the dlls:
(just copy and edit the function InjectReshade)

and done! If you wanna see if work it just use processhacker (you can find it in google, but open only later that alrealdy opened genshin, because genshin dont open when processhacker is alrealdy opened) and will see in module one 'd3d11.dll' and two 'dxgi.dll' (i dont know how unload system dxgi, but calm down, dxvk d3d11 only work if dxvk dxgi is working case else will show a resolution error)

I believe that this can be improved :)

@leonardo-spy Thank you very much. Managed to fix it by adding another sleep thread(?)

@dnblank123 i find the solution, just wait load this dll 'imagehlp.dll' and remove delay :)

code :
MODULEENTRY32 himagehlp{};
while (!GetModule(pi.dwProcessId, "imagehlp.dll", &himagehlp))
    std::this_thread::sleep_for(std::chrono::milliseconds(100));

Injectdxgi_64(pi.hProcess);
Injectd3d11_64(pi.hProcess);

update:
in some systems imagehlp dont is started, so replace with rsaenh.dll in your code

Check out the latest release https://github.com/34736384/genshin-fps-unlock/releases
Should solve this issue

My bad, dxgi.dll and d3d11.dll goes in the actual game folder, not the root folder. But it won't let me in

Not surprised, it was worth a shot. Hoping we could run this game in Vulkan somehow on Windows.

My bad, dxgi.dll and d3d11.dll goes in the actual game folder, not the root folder. But it won't let me in

Not surprised, it was worth a shot. Hoping we could run this game in Vulkan somehow on Windows.

@akippnn We have similar issue with your first comment, might be a delay or something idk, I'm closing the issue since I don't play the game anyway.

Hi, if i add dll's right here (in the image), will it work, or  i need to build with above changes?

i dont know is this still maintained / related, but now you can run genshin with vulkan without problem
dxvk gpl-async 2.2-4
is it bannable? i dont have any idea

are you use win10? how do you use dxvk?

i dont know is this still maintained / related, but now you can run genshin with vulkan without problem
dxvk gpl-async 2.2-4
is it bannable? i dont have any idea

are you use win10? how do you use dxvk?

im on windows 11. After the 4.0 update, there's a chance the game doesn't delete the dxgi.dll (i tested with dxvk and specialk local injection) and d3d11.dll crashed. they patched it somehow

i dont know is this still maintained / related, but now you can run genshin with vulkan without problem
dxvk gpl-async 2.2-4
is it bannable? i dont have any idea

are you use win10? how do you use dxvk?

im on windows 11. After the 4.0 update, there's a chance the game doesn't delete the dxgi.dll (i tested with dxvk and specialk local injection) and d3d11.dll crashed. they patched it somehow

so dxvk can't work after 4.0 updated? btw, what injector do you use?

DXVK is definitely working even after 4.0, just not when implemented this way.

DXVK is definitely working even after 4.0, just not when implemented this way.

all right, so can you use Vulkan to run genshin on windows after 4.0 update?

I have heard that dxvk can effectively increase fps, so I want to use dxvk.)
Can you help me find an injector with a close anti-cheat? I've searched the Internet and there's nothing available.

DXVK is definitely working even after 4.0, just not when implemented this way.

all right, so can you use Vulkan to run genshin on windows after 4.0 update?

Yes but you need to possibly remove the anti-cheat, so it's not worth it and is beyond the scope of this injector tool.

I have heard that dxvk can effectively increase fps, so I want to use dxvk.)
Can you help me find an injector with a close anti-cheat? I've searched the Internet and there's nothing available.


Where to start chapter

Maybe a check list of what needs to be done to get up and running.  Focus on deploying a container to K8S as a starting point.  Need to branch out for the different types of components such as monolith and serverless.

Deployments have been removed. The ORtelius CI/CD CLI now provides where to start directions.


Drag handle doesn't work in Firefox when it's a  element

In the following JsFiddle, the second list isn't draggable in Firefox (and maybe other browsers). Works on Chrome though.
http://jsfiddle.net/t6e66o10/1/

Moved example: http://jsbin.com/gebega/1/edit?js,output

Alas, this is FF bug.

http://jsbin.com/tujumopira/edit?html,output
https://bugzilla.mozilla.org/show_bug.cgi?id=568313
https://bugzilla.mozilla.org/show_bug.cgi?id=646823

Just for information, the same issue exists for Edge too.

But it breaks groups.


View manager for PostgreSql

Could be similar to what MongoDBViewManager is capable of by using the JSONB data type for the view.
Should provide a mechanism to easily create indexes into the JSON-serialized view instances, and it would be absolutely awesome if some basic LINQ spells were supported as well.

work is ongoing in feature/postgres-views

has been in master for a while now :)


Modifications to UI Elements

This PR  improves the overall UI:

Fixes #170 (Progress Bar Bug)
Fix Label getting cut off - labels will become '...' once you hit double digits
Improve /order select interface to:

Pull Request Test Coverage Report for Build 352

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 87.859%

Totals

Change from base Build 346:
0.004%

Covered Lines:
2873

Relevant Lines:
3270

💛  - Coveralls


Add Intune Section to Integration Doc

Description Of Changes
Add Intune section to integrations doc.
Motivation and Context
No prior reference to intune integration besides the setup and use docs. Also brought page up to current linting standards.
Testing

[X] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[X] Minor documentation fix (typos etc.).
[X] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.

Change Checklist

[ ] Requires a change to menu structure (top or left hand side)/
[ ] Menu structure has been updated
[X] No navigation changes needed

Related Issue
Fixes #436

@ryanrichter94 am I right in saying that you used the "big green" button here, and squashed the commits through the UI?

Yes


Replace @RequestMapping variable at runtime

Hi,
I want to replace a string in requestMapping URL at runtime. It's not a key-value pair. It's just an encrypted string.
@FeignClient(name = "base-service", url = "http://demo.india.com")
public interface BaseService {
	@RequestMapping(value = "/ClientOrderPayment.aspx?{q}", method = RequestMethod.POST)
	public Response getPaymentUrl(@QueryParam("q") String q, @RequestParam("clientCode") String clientCode);
}

I've tried @Param, @PathVariable and @QueryParam but it did not work. I'm getting the following exception:
Caused by: java.lang.NullPointerException: null
	at feign.RequestTemplate.replaceQueryValues(RequestTemplate.java:605) ~[feign-core-9.3.1.jar:na]

So your example is odd. q is a request parameter, but you've put it as a @PathVariable. We have tests that use these here.
Looks like this
		@RequestMapping(path = "/test2", method = RequestMethod.PUT, produces = MediaType.APPLICATION_JSON_VALUE)
		ResponseEntity<TestObject> getTest2(
				@RequestHeader(name = "Authorization") String auth,
				@RequestParam(name = "amount") Integer amount);

@spencergibb  Thanks for your reply.
But I want q to be appended to the URL in a post request. q does not have key value pairs, it's just a large encrypted string. What you are suggesting will send it as in body as x-www-form-urlencoded I guess.

I don't think that is supported. You could try a ReqestInterceptor

Closing due to age of the question. If you would like us to look at this issue, please comment and we will look at re-opening the issue.


Updates browser.md and browser-comparision.md with new callouts for e…

This PR updates browser.md and browser-comparision.md with new callouts for email insights service.
refers #4577
Covers:

[x] Browser Comparison Statistics
[x]  Browser Statistics

Hello @yashasingh,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @yashasingh,Thanks again for the PR!                                         You have earned 3 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


/proc/fs/lustre/health_check

Please add the new metric lustre_health for /proc/fs/lustre/health_check.
$ cat /proc/fs/lustre/health_check
healthy

(or not healthy)

We'll get it up there as soon as we can!

@roclark to the rescue, closing this issue (let us know if you have any concerns @knweiss)


#SearchFacade is missing when opening define a component menu

This class is missing :
MolWorld class>>openDefineComponentDialog
<script>
| searchClass list retValue |
list := SystemNavigation default allClasses select: [ :c |
	        c isTrait not and: [
		        (c allSuperclasses includes: Object) and: [ c isComponentClass ] ] ].
searchClass := **SearchFacade** classSearchIn: list.
searchClass title: 'Select the Molecule Component to define'.
retValue := searchClass openModal.
^ retValue answer

Config :

Windows 10
Pharo-11.0.0+build.459

The class was moved here : https://github.com/pharo-project/pharo/pull/12144

This evolution is only for Pharo 11. We needs to have a branch for P10 and less to preserve compatibility.


fixing settings for gh-pages deployment

Bob,
I think these are all of the changes needed to get it minimally working under github pages
After you merge this PR, pull to your local system and run the following from the rg-trees directory at the command line (assuming you have installed yarn):
yarn install
yarn deploy
This should deploy the built code to the gh-pages branch of the repository and then you should see it at https://rgorman.github.io/rg-trees
At that point, you would use the following settings in your html page for linking to Alpheios
data-alpheios_tb_url="https://rgorman.github.io/rg-trees/embed/DOC/SENTENCE"
and for a document e.g.
data-alpheios_tb_doc="aeschines-1-1-50-bu1"
@zfletch I think maybe we need to provide step by step instructions for setting up a repo. I am happy to help draft that if you want.

Bridget,
Thanks for this.
For some reason gh-pages is not a choice for me under GitHub pages. What’s our next move? ☺
<EMAIL_ADDRESS>From: Bridget Almas<EMAIL_ADDRESS>Sent: Tuesday, June 2, 2020 2:59 PM
To: rgorman/rg-trees<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [rgorman/rg-trees] fixing settings for gh-pages deployment (#1)
Bob,
I think these are all of the changes needed to get it minimally working under github pages
After you merge this PR, pull to your local system and run the following from the rg-trees directory at the command line (assuming you have installed yarn):
yarn install
yarn deploy
This should deploy the built code to the gh-pages branch of the repository and then you should see it at https://rgorman.github.io/rg-treeshttps://urldefense.proofpoint.com/v2/url?u=https-3A__rgorman.github.io_rg-2Dtrees&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=iXyIr_nljy2bP_XrNVOLBZgXexXg3mNR17f753VI1Zw&e=
At that point, you would use the following settings in your html page for linking to Alpheios
data-alpheios_tb_url="https://rgorman.github.io/rg-trees/embed/DOC/SENTENCEhttps://urldefense.proofpoint.com/v2/url?u=https-3A__rgorman.github.io_rg-2Dtrees_embed_DOC_SENTENCE&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=JiApt_EDl39djcXa7JaJ0rq-3KJs5yirUhqj7MuzAk0&e="
and for a document e.g.
data-alpheios_tb_doc="aeschines-1-1-50-bu1"
@zfletchhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_zfletch&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=dlXHy0w79vxeDyrD-uaqeWBqW7iFKV-Mfh65zPi0z70&e= I think maybe we need to provide step by step instructions for setting up a repo. I am happy to help draft that if you want.

You can view, comment on, or merge this pull request online at:
https://github.com/rgorman/rg-trees/pull/1https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman_rg-2Dtrees_pull_1&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=RAoC9tKg9rA4jcb4e2ENa36_JoyYc-I4HJJiJcs_rJI&e=
Commit Summary

fixing settings for gh-pages deployment

File Changes

M .envhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman_rg-2Dtrees_pull_1_files-23diff-2Df579cccc964135c7d644c7b2d3b0d3ec&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=shCS5FQKlSTrVBQGYTaJWtAuz07TkfyHhij4a_E9NHs&e= (2)
M package.jsonhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman_rg-2Dtrees_pull_1_files-23diff-2Db9cfc7f2cdf78a7f4b91a753d10865a2&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=bzIOzSs74xcjPvhm1sK1yb97V_EzF1W4kKpY7hCoszU&e= (4)

Patch Links:

https://github.com/rgorman/rg-trees/pull/1.patchhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman_rg-2Dtrees_pull_1.patch&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=idNwGAAhejL13knlcDsb9-pGCA3K3U7N04XHbjBAfxs&e=
https://github.com/rgorman/rg-trees/pull/1.diffhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman_rg-2Dtrees_pull_1.diff&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=NJjGIHyPF2eECSbCEv4kwjr8-OWAManIgicugOY2W-o&e=

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman_rg-2Dtrees_pull_1&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=RAoC9tKg9rA4jcb4e2ENa36_JoyYc-I4HJJiJcs_rJI&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABXXL2MJGZBLRW2AKEU7PG3RUVKZBANCNFSM4NRA773Q&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=YdnWgsGYX0vlkoZ1k3Z1TQ&m=Owu7CYHp6E4hQmp-ZJlDaeoWpZzXk8PUFeY8V201YU4&s=LEjHs9i7M1bQqY5JvWPXwOeJa9Ushl9JQnTvgRjChHo&e=.


Replace/Remove all uses of Common.h in the Engine

Closes #219
Closes #196

As I just noted in PR https://github.com/lairworks/nas2d-core/pull/221#issuecomment-498156045, we may want to simply remove "Common.cpp". It doesn't need to exist for backwards compatibility, and may simply cause diagnostic output by including the now deprecated header file.

As I just noted in PR #221 (comment), we may want to simply remove "Common.cpp". It doesn't need to exist for backwards compatibility, and may simply cause diagnostic output by including the now deprecated header file.

Removal of the files is a separate issue #220 in order to give downstream projects enough time to update. I'd prefer removing them all in one fell swoop because it requires modifying the project file (.vcxproj) itself and splitting that up between two PRs is a headache.

Yes, the Common.h file can affect downstream projects, if they've included it directly. The Common.cpp file won't affect downstream projects. When NAS2D is compiled into a library file, all object code is archived in a single .lib file, regardless of where the methods came from. The .lib file doesn't track the source object file. Moving the method implementation from one .cpp file to another .cpp is a distinction that will be lost during the compile/archive step. All the methods in Common.cpp have simply been moved to other .cpp files. It is safe to remove the Common.cpp file now, so long as you preserve the .h file for downstream projects.
That is of course assuming that Common.h was a public header. If it's a private header, and a downstream project has included it directly, that's their fault, and they should be broken. :wink:
I suppose this brings up the related question of whether or not the other header files are public, or only the main NAS2D.h file is considered public.
Anyway, doesn't matter which PR the removal is done in. I think it's safe to do it now, but you're welcome to do it in a future PR. Either way, this is good to merge now.

Reminder this still needs to be merged. This fixes a number of warnings which show up in downstream projects.

Additional tasks, which can be completed post merge:

Split Common.test.cpp
Remove Common.cpp

Ok, so Leeor isn't able to review right now. As this is a pretty innocuous change, and should remove warnings for downstream projects, I'm going to go ahead and merge this.


fix: Ignore ANSIBLE0014 in tox ansible-lint env

Running the tox ansible-lint environment results in the following error:
[ANSIBLE0014] Environment variables don't work as part of command
/home/jwcarman/projects/ibm/cluster-genesis/playbooks/tasks/cmd_bootstrap.yml:18
Task/Handler: Run Bootstrap Command
The error is generated due to using jinja2 variables inside the 'args'
value, not environment variables. This has been tested (lightly) but I
think it's safe to ignore this error class for now to allow tox to run
cleanly.

This doesn't seem to be failing anymore, which is good because I don't think it's actually violating ANSIBLE0014. I'm not sure what changed.

Oddly, 'ansible-lint' seems to only pass intermittently.


autoplay button is broken (2012 layout)

sorry to disturb and genuinely, customtube is awesome and I can't thank you enough for this cool piece of software that revives old youtube, but I have one small issue on videos on desktop, the autoplay button on any video I open glitches out and looks broken, even in full screen and if anyone knows how to fix this please write down in the comments

CustomTube is unofficially discontinued. I don't think this is going to be fixed anytime soon.


Allow pyelf tools to be installed on multiple platforms.

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

platformio depends on this package on more than just linux:
https://github.com/platformio/platformio-core/blob/develop/setup.py#L35

@conda-forge-admin, please rerender

@ivankravets since you use pyelf on multiple platforms, can you confirm that it indeed is useful to have this on windows and mac?

Bump for this change. It would help out platformio


Have issues with building from source

I try to build latest NTVS to try fixes, but having problem building by instructions.
I try to run regular build by running
msbuild NodejsTools.sln /p:Configuration=Release /p:VsTarget=14.0 /p:VisualStudioVersion=14.0

This setup is give me following error:
error CS1762 : Warning as error : A reference was created to embedded interop assembly 'c:\Program Files (x86)\Microsoft.NET\Primary Interop Assemblies\stdole.dll' because of an indirect reference to that as
sembly created by assembly 'c:\Program Files (x86)\Microsoft Visual Studio 15.0\Common7\IDE\PublicAssemblies\Microsoft.VisualStudio.CommandBars.dll'. Consider changing the 'Embed Interop Types' property on eit
her assembly. [<dir>\nodejstools\Nodejs\Product\InteractiveWindow\InteractiveWindow.csproj]

When building from VS that not an issue since project does not set WarningAsErrors setting.

For the the fix was change property in the InteractiveWindow.csproj:73 and  InteractiveWindow.csproj:76 from True to False. But that 's override settings introduced in the @8757515299cd6090502dfb4eb4706182eb716f64
After that change I was able to build the NTVS, but not sure that this is 100% correct fix.


Use assertNotEqual instead of assertNotEquals for Python 3.11 compatibility.

The deprecated aliases were removed in Python 3.11 in python/cpython#28268 .

Thanks. Merging.


[AutoPR network/resource-manager] Vtap swagger update and made the property as readonly

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3845

(message created by the CI based on PR content)


Command-line options for overriding configuration

This adds option parsing, such that configuration options can be overridden on the command-line.
For example, the command:
node server.js --port=80 --rethinkdb.host=rethinkdb

is equivalent to running server.js after editing config.js to include:
{
    port: 80,
    rethinkdb: {
        host: 'rethinkdb',
        //...
   },
    //...
}

This change also:

removes our dependency on the commander package, replacing it with minimist
makes DEBUG mode configurable through the DEBUG environment variable (instead of just the DEBUG config option)
makes the omega-logger log level configurable through the logLevel config option (instead of just the LOG_LEVEL environment variable)

@whitelynx: Interesting. I'm not sure how I feel about having command-line parsing in our config file... on the other hand, I don't disagree with the intention behind this.
After thinking about it, I'll give it a :+1: (with minor reservations).


Hyper fails to link on MacOs

Version
hyper = "0.14.11"
tonic = "0.5.0"
Platform
Darwin jhowarth52-mac 19.6.0 Darwin Kernel Version 19.6.0: Tue Jun 22 19:49:55 PDT 2021; root:xnu-6153.141.35~1/RELEASE_X86_64 x86_64
Description
I've tried to set up both warp and tonic as simple servers, but I get the following linker error when building. I get the exact same error when using both libraries, which makes sense because both of these libraries use hyper internally.
The error is massive so I uploaded the contents below instead of pasting.
Here's the top portion:
= note: Undefined symbols for architecture x86_64:
            "_$LT$hyper..error..Error$u20$as$u20$core..fmt..Debug$GT$::fmt::hf50eee56ebd28b77", referenced from:
                l___unnamed_7 in libtonic-b028be4b4c85066d.rlib(tonic-b028be4b4c85066d.tonic.1d94625b-cgu.2.rcgu.o)
            "_$LT$hyper..proto..h1..decode..IncompleteBody$u20$as$u20$core..fmt..Debug$GT$::fmt::hf4d525538f769eb5", referenced from:
                l___unnamed_2 in libtonic-b028be4b4c85066d.rlib(tonic-b028be4b4c85066d.tonic.1d94625b-cgu.2.rcgu.o)
            "_$LT$hyper..proto..h1..decode..IncompleteBody$u20$as$u20$core..fmt..Display$GT$::fmt::h2745005cda6487a4", referenced from:
                l___unnamed_91 in libtonic-b028be4b4c85066d.rlib(tonic-b028be4b4c85066d.tonic.1d94625b-cgu.2.rcgu.o)
                l___unnamed_2 in libtonic-b028be4b4c85066d.rlib(tonic-b028be4b4c85066d.tonic.1d94625b-cgu.2.rcgu.o)
            "hyper::proto::h1::io::WriteBuf$LT$B$GT$::buffer::CALLSITE::hc78ea883712a9e59", referenced from:

linker_error.txt
Has anyone seen this before or have any suggestions to get this working?

@JoeHowarth It may be possible that It is an XCode related issue, I have faced the same issue once and I uninstall XCode and install it again and it worked. You may try that also.

@JoeHowarth It may be possible that It is an XCode related issue, I have faced the same issue once and I uninstall XCode and install it again and it worked. You may try that also.


Add support for consuming and generating Open Threat Model (OTM)

Hello. I'd like to be able to both consume and generate OTM from Threat Dragon. On the consumption side, I'd like to be able to open an otm file directly. On the generation side, I'd like to be able to save models in otm format.
The Open Threat Model format is still early in development, but its goals are to standardize how data from threat models are represented, providing interoperability between different systems and tools.
Per the readme:

OTM allows both humans and computers to understand what are the components of a system, how are they distributed, the security risks that could be exposed to attackers and the mitigations that could be implemented to avoid those vulnerabilities.
OTM can be used to document your system and threat model, to keep you threat model aware of the changes that happens in the system and many other use cases.

Hello @stevespringett , the OPen Threat Model looks an excellent initiative.
The way we see it working is that Threat Dragon could read files in either format, and save in both formats. Typical Threat Dragon models are not large, it is a quick and accessible tool, so this could work well

The Open Threat Model is looking very promising and so we should try to get this into the next version of threat dragon, further to the discussion in [OTM under a standards body](OTM under a standards body)

Update. Matthew McDonald on my team at ServiceNow will be publishing a PR that adds support. He's currently testing round tripping between Threat Dragon and IriusRisk.

Thanks @stevespringett , very good news that this is progressing. I have assigned it to you and feel free to add Matthew McDonald.

Thanks for the pull request @mmcdonald4tw, and it will get reviewed this weekend


V1.0

Merge last weeks changes into v1.0 branch.

Sigh. Build didn't start because of name. Resubmitting using a different branch name.


Not able to run the docker

I am getting following error: When I launch my container.
sudo docker run -it -p 8888:8888 -v ~/myNotebooks:/notebooks/myNotebooks resin/rpi-raspbian
standard_init_linux.go:195: exec user process caused "exec format error"
Any idea how to fix this issue.

The command to run the container on a rapsberry pi model 3b is
docker run -it -p 8888:8888 -v ~/myNotebooks:/notebooks/myNotebooks
romilly/rpi-docker-tensorflow
which diiffers from what your message quotes.
Please try that and let me know if it's OK.
On 11 March 2018 at 03:22, raghavendrasri<EMAIL_ADDRESS>wrote:

I am getting following error: When I launch my container.
sudo docker run -it -p 8888:8888 -v ~/myNotebooks:/notebooks/myNotebooks
resin/rpi-raspbian
standard_init_linux.go:195: exec user process caused "exec format error"
Any idea how to fix this issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/romilly/rpi-docker-tensorflow/issues/6, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAJHPtVvNCYWTvOAdWAkfNNScoE7mIW9ks5tdJh4gaJpZM4SlirY
.

--
personal:@romillyc work:@rareblog skype:romilly.cocking web:
http://blog.rareschool.com/

No further feedback, so I am closing this issue.


Make Angular 2 dependency more relaxing

Currently the Angular 2 version is hardcoded and it permits repositories using it to upgrade to new versions as soon as possible and test it with new versions.
Dependency can be similar to ng2-translate's:
"peerDependencies": {
  "angular2": ">= 2.0.0-beta.0",
  "es6-promise": ">= 3.0.2",
  "es6-shim": ">= 0.33.3",
  "reflect-metadata": ">= 0.1.2",
  "rxjs": ">= 5.0.0-beta.0",
  "zone.js": ">= 0.5.10"
}

Is it acceptable? If so i would make a PR.

Can you guarantee that future versions will not broke bootstrap functionality?
I am not, until tested.
When ng2 will be released, dependencies will be relaxed a bit
Plus it is just peed devs, you can ignore warning
But at least bootstrap is honest, and giving a warning :)

No i can not guarantee, but at least it gives users of the library the ability.to upgrade and not make a hard dependency on it without error messages.
Or the other solution maybe to remove it from peerdeps, is it ok?

M yintention is to include it in my starter project https://github.com/blacksonic/angular2-es6-starter
And as a new version comes out i want to upgrade.

are you using npm3? than it is just a warning hint
I am completely with you about updating just in time
but for now even not all ng2 version a working
for example I had to skip 10 and 11

Using npm2 to be usable with node 4.x branch also

Stating incompatible versions in readme and be more relaxing?

Percent of users reading readme is low, really low. Update to npm 3 and
bother yourself with node 4. In April will be released node 6 LTS.
On Mon, Mar 28, 2016, 13:53 Soós Gábor<EMAIL_ADDRESS>wrote:

Stating incompatible versions in readme and be more relaxing?
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/valor-software/ng2-bootstrap/issues/334#issuecomment-202340089


Can't generate factories

Environment
@mizdra/graphql-codegen-typescript-fabbrica version: 0.5.0
typescript version (optional): 5.4.2
@graphql-codegen/cli version (optional): 5.0.2
@graphql-codegen/typescript version (optional):4.0.6
Summary
The file I'm generating is incomplete (basically empty). It looks like this:
import {
  type DefineTypeFactoryInterface,
  defineTypeFactory,
} from '@mizdra/graphql-codegen-typescript-fabbrica/helper';
import type {
  Maybe,
,
} from './graphql';

export * from '@mizdra/graphql-codegen-typescript-fabbrica/helper';

Step to reproduce
I'm honestly not sure how to provide steps to reproduce. I can provide some context however, if that helps.
I'm using this package in the context of an integration test suite using playwright. The configuration files are very simple:
codegen-config.ts
import type { CodegenConfig } from "@graphql-codegen/cli";

const config: CodegenConfig = {
  overwrite: true,
  schema: "http://<IP_ADDRESS>:3000/graphql",
  generates: {
    "codegen/graphql.ts": {
      plugins: ["typescript"],
    },
    'codegen/fabbrica.ts': {
      plugins: ['@mizdra/graphql-codegen-typescript-fabbrica'],
      config: {
        typesFile: './graphql', // required
      },
    },
  },
};

export default config;

tsconfig.json
{
  "compilerOptions": {
    "module": "Preserve",
    "moduleResolution": "Bundler",
    "paths": {
      "~*": ["./*"],
      "~test-helper": ["utils/testHelper"]
    }
  },
  "include": ["**/*.ts"]
}

I've tried different values for module and moduleResolution in tsconfig, as well as changing typesFile to ./graphql.js, but so far, no dice. Just to be clear: I am able to generate my types file (in my case graphql.ts) with the "typescript" plugin, so codegen is working in general.
What did you expect to happen?
I expected it to find all my types and create factories.
What actually happened?
It didn't create any factories ):
Participation

[ ] I am willing to submit a pull request for this issue.

Additional comments
No response

@alexthebake Thanks for submitting the issue!
Please create a project where I can reproduce the issue so that I can investigate the issue.
I recommend using StackBlitz. The following URL is a sample StackBlitz project using graphql-codegen-typescript-fabbrica. Please fork it and try to create a project that can reproduce your issue.

https://stackblitz.com/edit/playground-graphql-codegen-typescript-fabbrica?file=src%2Findex.test.ts&view=editor

@alexthebake
Please try https://github.com/mizdra/graphql-codegen-typescript-fabbrica/issues/82#issuecomment-2111447219. If it does not solve, reopen this issue.


Improve/add documentation for STM32F429I firmware management

Based on first time getting started with the STM32F429I this documentation is to help people getting started with that device, along with some additional generic getting started help.
Description

New:

STM32CubeProgrammer Utility to flash firmware guide
Trouble shooting guide for getting started (common getting started problems)
How version and checksum are managed between C# components and C++ assemblies (common getting started problem)
Images of Device Explorer, STM32CubeProgrammer, nanoff, etc to go with the guides

Updated:

Getting started guide
ST-Link guide
Index.md's for new content
order files for new content

Motivation and Context

Resolves nanoFramework/Home#754

Types of changes

[x] Improvement (correction, content improvement, typoe fix, formating)
[x] New Article (new document for docs website)
[ ] Config and build (change in the configuration and build system, has no impact on code or features)

Checklist:

[x] My doc follows the code style of this project.
[x] I have read the CONTRIBUTING document.

@edleno2 there are now conflicts in a couple of files. There was a PR waiting for review that had updated instructions with the new STM programmer CLI...

Considering that STLink has been marked as deprecated, it's better to just replace the STLink instructions with the new STM programmer. Please review & merge what is there now, so we can agree on how it will look like in the end.

@josesimoes  - Merged my changes with yours - copied your structure and removed the duplicate images I had made.  Added my new content to the TOC.  This should be ready to merge once the checks complete.


chore: Fix initialization startup error

lb-utils needs to be initialized when the user first starts

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Propagate -Swift.h files

Don't remember why we removed this but on further testing noticed that we need this being put under xchammer-includes/x/x so it gets to DD folder and building in Pure Xcode mode works

Confirmed that this is not necessary, it was bad state on my machine 🤷


Generate and download the element-templates for each workers/connector

2 downloads:

one per workers
one ZIP to download all templates in one

Done in 1.3.0


Badge colors, #714

Thanks, i think we can add this. But you can actually already specify badge color by specifying its bg- class like 1 :)

Thanks :-)


HARVESTER: Fix Harvester image issues

Summary
Fixes Harvester image issues

Occurred changes and/or fixed issues

https://github.com/harvester/harvester/issues/2562
https://github.com/harvester/harvester/issues/2563
https://github.com/harvester/harvester/issues/2569

Technical notes summary

Fix clone image lack of labels
Fix image can save the invalid name
Fix some settings that the input is focused and then press enter keyboard will refresh the page. This should be the implicit behavior： https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#implicit-submission

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

@mantis-toboggan-md Thanks for your review. Updated the PR per the comments.


about download features

After I read the data, features are [batchsize, 36, 2048]. Does they mean that there are 36 regions which are flatten to 2048.? If so what's the origin feature's size ? 2048=45.25..*45.25..。 How to flatten them?

You have downloaded the features , can you share it?


#259 Added support for overloaded methods to .any()

by explicitly specifying Kotlin/Java class as the arg.

Just in case: initially I mistyped .any() with .eq(). I have changed the description, but commit still has wrong message.

See #259

Thanks for pointing to existing solution, closing this PR now.


router deployment

I have not been able to test this yet, so do not merge. But I think it is interesting to have it visible for closer collaboration during ShiftWeek

/retest

It seems that https://github.com/gophercloud/gopherkube/commit/5f74be53df963e4038280384a236397bafb01236 broke the testing environment

It seems that 5f74be5 broke the testing environment

yes it did unfortunately. At a minimum, clouds.NewClient should be replaced with clouds.NewServiceClient (the Gophercloud generation was split in two).

Hey @jlarriba ! Do you mind squashing the commits? At this stage of the project, it's probably best to merge and iterate. I am going to merge a big refactoring and I don't want you to have to resolve 1000 conflicts

Hi, I  had the changes addressing the comments lingering in local, I thought I had pushed them :( Solved all the smaller things and squashed the commits.


fixed some minor things

painting texture fixed
Spotify link is now to my artist page (we can change with the album link if it actually gets processed by then)
safe key scaled up to be slightly larger and made more visible
changed painting dialogue to "There is something strange about these statues..."
@zoogies should change the yoyoengine book if you want it different, see "Assets > Dialogue > Bookcase > PastBookcase2" change the ink file
Note: the audio is in the candles but it is turned off for all the candles
Turned off roaring fire sound in outdoor level by shed

ggwp!


fix: add missing transform types

Love this repo! Amazing work :)
With the release of 0.8.0,
  transformPastedText: false,  // Allow to paste markdown text in the editor
  transformCopiedText: false,  // Copied text is transformed to markdown

was added, however was not added to the types definition, this pr fixes that :)

Thank you, it's released in v0.8.1


[Breaking change]: ListBox.DefaultItemHeight constant field is marked as obsolete

Description
ListBox.DefaultItemHeight is no longer used as the default item height for the ListBox control because now item height is proportional to the current application font. This change is necessitated by the introduction of the new default font for WinForms controls in .NET Core 3.1.
Version
.NET 8 Preview 2
Previous behavior
Applications referencing ListBox.DefaultItemHeight constant compiled with no warnings related to this value.
New behavior
Applications referencing ListBox.DefaultItemHeight constant will compile with the following warning:
WFDEV004: ListBox.DefaultItemHeight` constant is no longer used as the default item height. Default item height is now scaled according to the application default font.
Type of breaking change

[ ] Binary incompatible: Existing binaries may encounter a breaking change in behavior, such as failure to load or execute, and if so, require recompilation.
[X] Source incompatible: When recompiled using the new SDK or component or to target the new runtime, existing source code may require source changes to compile successfully.
[ ] Behavioral change: Existing binaries may behave differently at run time.

Reason for change
The new default ListBox item height value is calculated dynamically, based on the application font.
Recommended action
Use Control.DefaultFont.Height values instead.
Feature area
Windows Forms
Affected APIs
ListBox.DefaultItemHeight

Associated WorkItem - 63465

@Tanya-Solyanik It doesn't look like this made it in for Preview 2. Is that correct? https://github.com/dotnet/winforms/pull/8640/files

Now seems to depend on https://github.com/dotnet/winforms/pull/10142.


NETOBSERV-1151: move flp decode grpc pipe to netobserv agent to avoid flp code updates

move FLP grpc decode logic to the agent to just limit flp to pkg update w/o need to do any code changes

/ok-to-test

/ok-to-test

/ok-to-test

Ran the full-suite of automated regression tests for the above eBPF image. All except SCTP test-case passed(which is known issue, being investigated; issue unrelated to this PR). 204887

Also regression tested the PacketDrop feature and its working as expected!

/label qe-approved

/approve


The reason for the difference between table 1 and table 2 in the paper.

Hi! I‘m confused about the performance difference between Table 1 and Table 2 in the paper. For instance, in Table 1, the fid and mmdist of momask on HumanML3D are 0.045 and 2.958, respectively. However, in Table 2, the fid and mmdist of momask on HumanML3D are 0.051 and 2.957. Could you tell me the reason for the difference?

Results in Table 2 (in paper) are based on the setting of 18 inference steps. In our subsequent sweeping experiment on inference steps, we discovered that 10 steps of inference performed slightly better. Consequently, we report these final results in Table 1.


How to accept the server returns the response?

Yes, I am the output, or there is no "done" status

The problem is that event done is never triggered. Any fix on this ?

https://github.com/bleenco/ngx-uploader/blob/master/src/ngx-uploader/classes/ngx-uploader.class.ts#L187
use   number  4  replace  XMLHttpRequest.DONE
if (xhr.readyState === 4)

@sgyy Can you please specify your answer? What to do and where?
Also facing this issue right know. Is there a solution in the meantime?
Thanks.

Facing the same issue here.
Can someone please explain how to properly catch the server response after uploading?
@sgyy I don´t really want to change any code from inside the node_modules.

@Matze-Schmitt Unfortunately, the source of this bug is likely https://github.com/HubSpot/pace/issues/261 and manifests in ngx-uploader as describe in https://github.com/bleenco/ngx-uploader/issues/408.  Bottom line is pace needs to be fixed, not necessarily ngx-uploader (which is not the only component that's affected).  Pace is breaking a pretty basic component in their override of xmlhttprequest.

I don't use pace for my application, this worked for me:
In your onUploadOutput function: Add else if(output.type === 'done') { //output.file.response }
Added my entire onUploadOutput function incase of any confusion.
Good luck =)
onUploadOutput(output: UploadOutput): void {
    if (output.type === 'allAddedToQueue') {
      let token = this.authService.getToken();

      const event: UploadInput = {
        type: 'uploadAll',
        url: this.config.apiEndpoint + '/api/v1/attachments/upload',
        method: 'POST',
        headers: { 'Authorization': token }
      };
      this.uploadInput.emit(event);

    } else if (output.type === 'addedToQueue' && typeof output.file !== 'undefined') { // add file to array when added
      this.files.push(output.file);
    } else if (output.type === 'uploading' && typeof output.file !== 'undefined') {
      this.uploading = true;
      // update current data in files array for uploading file
      const index = this.files.findIndex(file => typeof output.file !== 'undefined' && file.id === output.file.id);
      this.files[index] = output.file;
    } else if (output.type === 'removed') {
      // remove file from array when removed
      this.files = this.files.filter((file: UploadFile) => file !== output.file);
    } else if (output.type === 'dragOver') {
      this.dragOver = true;
    } else if (output.type === 'dragOut') {
      this.dragOver = false;
    } else if (output.type === 'drop') {
      this.dragOver = false;
    } else if(output.type === 'done') {
      this.addFile(output.file.response);
    } else if(output.type === 'rejected') {
      console.log('Rejected', output);
    }
  }

Angular 6.1.7
ngx-uploader 6.0.2

I had the same issue, fact is that the output variable is bound to the
If you hide the input with *ngIf right after starting the upload, you also destroy the variable output, and will never catch the "done" event


auto-merge: instant merge after creation [skip ci]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>issue from an auto-merge PR: https://github.com/NVIDIA/spark-rapids/pull/1356#issuecomment-742175119

build

I think removing the 10 second wait (which I never understood the purpose) is a reasonable workaround to reduce the window of opportunity, but the window is still there.  I think a better solution is changing the workflow to recognize the merge failed because the SHA changed because new commits were pushed so it can automatically loopback and retry.
So I'm OK with this going in to make it less likely to occur, but I think it could be improved to eliminate this type of error entirely.

I think removing the 10 second wait (which I never understood the purpose) is a reasonable workaround to reduce the window of opportunity, but the window is still there. I think a better solution is changing the workflow to recognize the merge failed because the SHA changed because new commits were pushed so it can automatically loopback and retry.
So I'm OK with this going in to make it less likely to occur, but I think it could be improved to eliminate this type of error entirely.

yeah, totally agree. since the github API does not give the specific reason for us to identify detailed merge failures, so we may need to add some custom logic in our code. I just created https://github.com/NVIDIA/spark-rapids/issues/1363 for a future fix


Add lots of useful Rust Snippets

The old ones were lacking quite a lot of useful snippets, so I added them.

Hi @CryZe, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@CryZe, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

@CryZe First thanks a lot for the nice contribution. We were discussing in the team and realized that the snippets wouldn't better be contributed through an extension. That would give you more freedom of making changes, and gives others the opportunity to select if they want these snippets or not.
Maybe we should remove our default snippets, so they don't interfere with snippets from extension.
What do you think?

I'm not fully sure, but the developer of the RustyCode extension might have some opinions on that @saviorisdead (I hope mentioning works if he's not involved in this repo).

I think it would better move this to RustyCode, as @aeschli said, cause it would be easier to support/change it. But again it may force user to install RustyCode just for snippets.

I removed the snippets from the built-in extension to clear the way for snippet extensions and give users the choice if which snippets to install.
@CryZe It would be great if you could publish your snippets as extension. Let me know if I can help.


Remove "Fast"-suffix from moves

Some moves have "Fast" in their name. In pokemongo though, this suffix is not present. We should remove that too.

I'm also looking for a elegant solution for this.
The in-game move name comes from localization strings, not from game master file. I don't see this as a bug as propose of this project is to translate game master file to json format.
Similarly to game, maybe we should use separate files for localization in our projects (moves and pokemon)? Something like in PokeAlarm code: https://github.com/PokeAlarm/PokeAlarm/tree/master/locales

@vfcp
Good point.
But what I don't want to maintain is the localization strings by myself.
The Pokémon / Move strings could probably be fetched from PokéAPI or so. But I don't know if it is easily possible to fetch the PokemonGO-only localisation strings (e.g. Avatar Customization stuff).
And it is not the purpose of this repository to have localization files in it. This repository should only fetch its information from other reliable sources (pokemongo-game-master / PokéAPI) and not store them by itself.

I agree, the localization strings is not on scope of this repo. This tool propose is to have an accurate representation of game master file and localization is not part of it. My previous message was not clear, but when proposing localization files, I meant in our projects repos, if needed. This is why I don't see this issue as a bug.
The removal of "Fast" -suffix is not the only issue. For example "Mud Slap Fast" is "Mud-Slap" in game. There might be other slight differences and Niantics can change move names in any update. It would be a pain to maintain this manually and I need to research to see how to extract this from game resources.
Personally, I'll use localization as it fits my project long term plans. When I find a good source, I'll share it in this thread for those that might be looking for something similar.


Add Save & Activate Button to Batch Number Format Configuration Page

W-7954555
Critical Changes
Changes

Add Save & Activate button.

Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

Label looks great to me, @kenlewis-sfdo! Though, I think we'll actually end up creating the label in a separate PR that @ptewson-sfdo is working on for label freeze.


Remove https://github.com/compass-rs/sass-rs dependency

Hi,
https://github.com/compass-rs/sass-rs is a bit finnicky to build on windows, actually it will fail a lot of the time for particular environment, the owner recommended to switch to  https://github.com/kaj/rsass/ (https://github.com/compass-rs/sass-rs/issues/77#issuecomment-847085127).
I would like to know the feasability of such thing and in particular if that could be considered, or better if a feature flag could be introduced to not compile with sass support.
Regards,
Neil

Grass might be best considering it's going for 1-to-1 implementation with the new singular official Dart-based sass compiler and any deviations are a bug.  It's feature completion would be enough as it is now to at least offer as an option?

@OvermindDL1 when I originally built Trunk, I wanted to use grass. At the time it was not able to properly handle @imports & @use statements. I have not checked recently, but if it has made some progress on that front, then it is not a bad idea.
If there are any features missing which our current sass impl provides, then the update would be a regression, and we should not move forward with it. Thoughts?

What's preventing us from using the official dart sass compiler?

What's preventing us from using the official dart sass compiler?

Last I recall it doesn't work on some platforms that rust supports that users may want to use to compile on.

Done. Not yet released, but we are now using dart-sass on master.

@thedodd I've been using it in a production environment, it works rather well.


Add a security policy

Fixes OLINGO-1532

A security policy is already included here. This PR is redundant


When creating, deleting, updating an image tag, a spinner appears in the main application area

Steps to reproduce

Go to the Images page
Add/update/delete tag of any template or iso

Actual result

When creating, deleting, updating an image tag, a spinner appears in the main application area. See screenshot.
Expected result

The spinner should not be arrears during this operation
Environment

Browser:
- [ ] Chrome version XX
- [ ] Firefox version XX
- [ ] Safari version XX
- [ ] Opera version XX
- [ ] IE version XX
- [ ] Edge version XX

Others:

Connected feature ID

image_tag_create
image_tag_edit
image_tag_delete
Screenshots

Test on:
HeyRoach/1172-images-bug
Test to:
image_tag_create
image_tag_edit
image_tag_delete

An error dialog appears when editing a newly created tag
Steps:

Go to Images on any tabs (Templates or ISO)
Select any image from the list and go its sidebar to the "Tags" tab
Create new tag
Go to editing new tag
Change parameters and click to "Edit"

Actual result: An error dialog appears when editing a newly created tag (See to screenshot)
Expected result: Tag successfully updated, the error there isn't.

A deleted tag is displayed after refresh page (only ISO)
Steps:

Go to Images to "ISO" tab
Go to a sidebar of any your ISO to "Tags" tab
Create any new tag
Delete this tag
Refresh page

Actual result: Tag wasn't deleted and it is displayed in tag-list
Expected result: Deleted tag there isn't in tag-list

maybe this bug appears because BaseTemplateModel extends BaseModel
@tamazlykar, what do you think?

I don't think so. This your command failed. Try to find why when you delete it right after creation you have wrong request params.

While good one looks like:

@HeyRoach
Firefox
Doesn't repeat on the master
Steps:

Log in
Open section
VM/Storage/Images/Snapshots/Firewall/SSHkeys/Accounts
Open details sidebar
Refresh the page

Actual:
Bottom of the page becomes white, for example

Expected:
Bottom of the page not becomes white

@HeyRoach
Tested on 1174


[Feature]allow users to specify the kubeconfig file path while executing cli commands

Is your feature request related to a problem? Please describe.
I have a web server that uses kubevela to manage my clusters. Users don't install kubevela on their machine but send http request to my server so that it executes a vela command to manage clusters. Different users have different authorities(different kubeconfigs). Admin can send request like 'vela cluster join xxx' but common user can't. Currently, the vela CLI uses the default kubeconfig file located at $HOME/.kube/config to interact with the Kubernetes cluster, which means vela cli can't use users' own kubeconfig.
Describe the solution you'd like
I would like to propose the addition of a feature to the vela CLI that allows users to specify the kubeconfig file path while executing commands. For example:
vela --kubeconfig /path/to/kubeconfig.yaml status vela-app
Describe alternatives you've considered

Additional context

Workaround is KUBECONFIG=/path/to/kubeconfig.yaml vela status vela-app

I don't think vela should support this, since we can use kubectl context to control it.


Typo

[Enter feedback here]
Open the product, bundle, or family you want to add the products for.
Should be
Open the product, bundle, or family you want to add the properties for.

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

ID: 1f140aa3-375b-6eca-8916-cf8db4962633
Version Independent ID: 93366af9-a8c7-2134-3ad4-66c0ea4181e1
Content: Use properties to describe a product (Dynamics 365 Sales)
Content Source: ce/sales-enterprise/use-properties-describe-product.md
Service: dynamics-365-sales
GitHub Login: @shubhadaj
Microsoft Alias: shujoshi

@d3thomas: Thank you for the feedback. We've updated the topic and the changes will be live in few hours.


[RTE] Enable skipped tests after the migration

What

Why

How Tested

Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

Some screenshots sizes probably are changed because of the removed lineHeight style from richTextEditorWrapperStyle.
Rooster also added additional horizontal paddings for lists that weren't there before


Fixed bug with cast empty form data and filters

Issue:

some of our slices in staging were getting errors due to empty strings in integer fields
filters were broken in staging due to ast.literal_eval(v)

@mistercrunch

Would have had merge conflicts to I added your changes manually in my branch


Running openwhisk behind a corporate proxy

Greetings
I need some expert advise on how to get OpenWhisk installed and running behind a corporate proxy.
I find it surprisingly hard to figure out how to get Docker Containers working behind a http/https proxy server.
Here are some of the things I tried so far.

Setting options like in /etc/default/docker, /etc/environment etc.
Running squid as a local proxy to route traffic that comes via 'docker0' interface. I believe http can be achieved this way. How about https?
Passing 'Environment' variables in the Dockerfiles. I haven't tried this since docker operations are nicely abstracted by OpenWhisk and I don't want to break the abstraction yet.

So.. is there a good way to accomplish this? Does OpenWhisk offer a way to pass Environment variables into the containers (short of modifying Dockerfiles directly)
Appreciate your time.
Thanks
pj
Environment details:

ubuntu 14.04 with embedded couch DB

@pjdurai Did I understand your issue correctly: you're deploying openwhisk behind a firewall and want to route API traffic from outside to the openwhisk router on the inside? (Or do you have some hybrid deployment of the openwhisk container?)

Actually my concern is the other way round. I am trying to install/run openwhisk behind a corporate proxy server (All out-going traffic in my network goes through this one proxy server, usually specified in  https_proxy environment variable).
I am unable to complete openwhisk installation in an Ubuntu 14.04 VM because it halts when the containers try to download packages from internet. I haven't found a good way to specify this proxy to applications running inside the docker containers.
Thus I was wondering how others are doing it.
Thanks
pj

The cli does support http proxies but that won't help with your build. Maybe @domdom82 can help.

I am in the same problem. I went in and added ENV proxy setting to all dicker build files and got the build working, but when I run openwhisk with any triggers, all come back with econnreset error unable to connect socket. did you find an answer?

Likely your API host is not set correctly. Closing as stale.


matlab cv.findChessboardCorners

Hi,
the following matlab-function doesn't work:
[result, corners] = cv.findChessboardCorners(img, [6 10]);
The following error occurs:
Error using findChessboardCorners
cv::exception caught: /home/pewe/build/opencv3/opencv-3.0.0/modules/calib3d/src/calibinit.cpp:268: error: (-211)
Both width and height of the pattern should have bigger than 2 in function cvFindChessboardCorners

Adding some additional lines to the corresponding auto-generated source-file (build/modules/matlab/src/findChessboardCorners.cpp) & compiling this again:
#include
...
std::cout << "internal patternSize   = " << patternSize << std::endl;
Will produce the following output:
internal patternSize   = [0 x 0]
Error using findChessboardCorners
cv::exception caught: /home/pewe/build/opencv3/opencv-3.0.0/modules/calib3d/src/calibinit.cpp:268: error: (-211)
Both width and height of the pattern should have bigger than 2 in function cvFindChessboardCorners

This clearly shows that the BridgeVector::toSize() function doesn't work as supposed!
The function is defined here at line 321 (https://github.com/Itseez/opencv_contrib/blob/master/modules/matlab/include/opencv2/matlab/bridge.hpp):
cv::Size toSize() const { return cv::Size(); }
The function definition always returns an empty size (0x0).

Our current work around uses our own function and passes the pattern-size as two single ints.
Greetings
Peter

I'm having the same issue when trying to use the winSize parameter in calcOpticalFlowPyrLK. It always throws an exception that winSize.width and winSize.height need to be greater than zero. In matlab I've tried passing in a struct('width',5,'height',5) or an array [5,5] and it makes no difference. Seems the conversion to an openCV Size is not working right

I'm having the same issue when trying to supply the ksize parameter to cv.getStructuringElement in MATLAB. For example, executing
cv.getStructuringElement(cv.MORPH_RECT, [3, 3])
results in the error
Error using getStructuringElement cv::exception caught: c:\opencv\source\modules\imgproc\src\filterengine.hpp:363: error: (-215) anchor.inside(Rect(0, 0, ksize.width, ksize.height)) in function cv::normalizeAnchor
If cv.getStructuringElement is called without specifying an anchor, the anchor is set to the center of the size rectangle (i.e. anchor.x = ksize.width/2, anchor.y = ksize.height/2), so this error should only be occurring if ksize has zero width or height.
I've tried formatting the MATLAB parameter as a struct (as anuppari tried above), and I've tried formatting the argument matrix as all of the integer numeric types. I get the same result in all cases.
Here's the output of my cv.buildInformation():
Platform
OS:            Windows-6.1.7601
Architecture:  64-bit AMD64
Compiler:      MSVC 16.0.40219.1
Matlab
Version:       <IP_ADDRESS>0719 (R2016a)
Mex extension: mexw64
Architecture:  win64
Mex path:      C:/Program Files/MATLAB/R2016a/bin/mex.bat
Mex flags:     -largeArrayDims
CXX flags:     /D _CRT_SECURE_NO_DEPRECATE /D _CRT_NONSTDC_NO_DEPRECATE /D
_SCL_SECURE_NO_WARNINGS /Gy /bigobj /Oi
OpenCV
Version:       3.1.0-dev
Commit:        None
Configuration: Release
Modules:       dnn, core, imgproc, ml, imgcodecs, videoio, highgui,
objdetect, flann, features2d, photo, video, videostab,
calib3d, stitching, superres, xfeatures2d

I can confirm this bug with
Platform
OS:            Windows-6.1.7601
Architecture:  64-bit AMD64
Compiler:      MSVC 19.0.24213.1
Matlab
Version:       <IP_ADDRESS>1360 (R2016a)
Mex extension: mexw64
Architecture:  win64
Mex path:      C:/Program Files/MATLAB/R2016a/bin/mex.bat
Mex flags:     -largeArrayDims
CXX flags:     /D _CRT_SECURE_NO_DEPRECATE /D _CRT_NONSTDC_NO_DEPRECATE /D
_SCL_SECURE_NO_WARNINGS /Gy /bigobj /Oi
OpenCV
Version:       3.1.0
Commit:        None
Configuration: Release
Modules:       dnn, core, imgproc, ml, imgcodecs, videoio, highgui,
objdetect, flann, features2d, photo, video, videostab,
calib3d, stitching, superres, xfeatures2d

I also think the problem lies with Bridge::toSize() which is returning a new empty Size(). Can any of the developers confirm this? Are you accepting pull requests on this issue?


fix(STONEINTG-1074): set the option max file size to 2000M as in doc

Before you complete this pull request ...
Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

/retest


Issue with Vitis AI Profiler (No such file or directory: '/sys/kernel/debug/tracing/trace_clock')

Hi,
I am trying to use the Vitis AI Profiler following the tutorial https://github.com/Xilinx/Vitis-AI/tree/master/tools/Vitis-AI-Profiler.
I am using an Alveo U280, and Vitis AI Docker container for CPU.
The following command causes the issue "FileNotFoundError: [Errno 2] No such file or directory: '/sys/kernel/debug/tracing/trace_clock'".
sudo vaitrace ./resnet50 /usr/share/vitis_ai_library/models/resnet50/resnet50.xmodel

I am new to Linux and Vitis and I don't know how to solve this issue...
Thank you for your help.
Best regards,
Luc

Hi @Luc-Meunier
Could you please check the following?

Hello,
Thanks for helping me.
I added the line -v /sys/kernel/debug:/sys/kernel/debug --privileged=true \ in the docker_run.sh file.

However, I still get the error message ERROR:root:This tool need run as 'root'.
And I get an std::runtime error :

Facing same issue. It says Xilinx_xrt not set. Anyone any leads?

source /opt/xilinx/xrt/setup.sh
^ this script is responsible for setting that environment variable. Ideally it would have been set for you.
Sent from my iPhone

On Jun 17, 2021, at 3:19 AM, Karan Rawat @.***> wrote:

Facing same issue. It says Xilinx_xrt not set. Anyone any leads?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yes, solved it thanks.

Thank you, but unfortunately I still can't run vaitrace (still XILINX_XRT not set despite sourcing the script).
Could you tell me what am I missing ?

The latest VAI release has fixed this issue, please reopen this issue if there any other problems. thanks！

The latest VAI release has fixed this issue, please reopen this issue if there any other problems. thanks！

My VAI is 3.0. Aslo source /opt/xilinx/xrt/setup.sh
But still have the problem, how to solve it?

Hi @hocken-li , Can you share your steps and error message. Thanks!