📖 Add rebase documentation

Summary
Add documentation on the rebase process.

This is great and will be an amazing help in the future :-)

Maybe a dumb question: Why does kcp-dev/apimachinery requires k8s.io/api? Sounds like a 'reversed' dependency? Thanks!

@waltforme I don't know where the kcp-dev/apimachinery name came from in the beginning, to be honest. It's not related to k8s.io/apimachinery and untangling it might be a good use of time. The contents in the repo are related like this:

kcp-dev/apimachinery/pkg/cache is similar to k8s.io/client-go/tools/cache here
kcp-dev/apimachinery/pkg/client is the machinery behind the cross-cluster clients generated by kcp-dev/code-generator and committed to kcp-dev/client-go

Perhaps deleting the repo and subsuming the content into the more appropriate places would be good.

I see. Now the dependency makes sense. Also it explains why the repository is so small.
Maybe I should take a closer look before asking this.
Thanks for the detailed explanations! @stevekuznetsov


doesn't make discord channels

When I open the .exe file, nothing happens. It just loads but it doesn't create Discord channels in my server to control the pc.

i dont even know what to do after i run the bat file can u help me get to where u are

Als ik het .exe-bestand open, gebeurt er niets. Het laadt gewoon, maar het maakt geen Discord-kanalen op mijn server om de pc te besturen.

has the bot admin on the server

i dont even know what to do after i run the bat file can u help me get to where u are

The keylogger from the same person is not working either.

I will take a closer look into this issue tomorrow

you need to have those enabled

@RubenZW does the bot go online once you execute the malware? If it does make sure to check @alvgame's comment and give us feedback :)

i dont even know what to do after i run the bat file can u help me get to where u are

The keylogger from the same person is not working either.

Also @RawSpam you added a few comments on our issues which weren't true or fully true all the time. If you know a solution to a specific issue then I don't mind if you comment on it. Also, You can't just say this other software from this creator didn't work for me, so it's no wonder that this doesn't work either. Both work to my knowledge. And you also admitted that his keylogger is working as well as giving someone advice on how to install it here. I guess that was before my PR which explained the process a bit better and also changed the file names of some instructions in his README to make it easier for people to understand which command uses what file.

@RubenZW does the bot go online once you execute the malware? If it does make sure to check @alvgame's comment and give us feedback :)

i had the same problem too, and i turned those intel on and it still doesnt work

then you have a different issue, please open your own issue because I am closing this one due to inactivity of the inactivity of Ruben on this problem.


Axios v0.25.0 update

Axios has been updated to v0.25.0. Among other things, it includes a fix for a vulnerable dependency (follow-redirects).
Can we get an update to @nestjs/axios with the latest version of Axios? Thanks!

https://github.com/nestjs/axios/pull/300


Allow month-day date-times in selections

It would be very nice if the selection file format supported "Year-Month-Day" dates in addition to "Year-DayOfYear" form.  The challenge is made more complex by a) the date-times are allowed to be short, specifying only a date, only an hour, etc. and b) the fields in the file can be in different columns depending on file format.
Allowing both date formats might require more strictness in the forms.  Review is required to determine if this can be detected unambiguously.

...along these lines I was caught up by a short endtime not including the entirety e.g.
endtime year 2018
I would expect to include 2018:365:24:60:60
or 2019:00...
Perhaps I'm alone on this, and I can just add 1 to the year

As of v3.0.6 any time formats recognized by ms_timestr2nstime() may now be used in selection files.
@dsentinel, yeah that's a tricky problem.  Adding context ("start" versus "end") when converting short times is non-trivial and can make usage really confusing.  But it's a common problem across web service interfaces, library APIs, etc.
I've been thinking along the lines of supporting time designations that are non-inclusive.  For example, a user requests and end time of <2019 (or (2019 for traditional interval syntax) which would mean anything before the year 2019 but not including 2019-1-1T0:0:0.


global reader app permission policies visibility

Mention about the missing option and visibility of app permission policies, if you're not a global admin nor teams admin.

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

File
Status
Preview URL
Details

Teams/teams-app-permission-policies.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

missed my mention


Add a new Timeline visual

src/Clients/PowerBIVisualsTests/helpers/performanceTestsHelpers.ts, line 490 [r1] (raw file):
I'd just leave this out of the perf tests for now.

Comments from the review on Reviewable.io

Please integrate Cong's latest changes, then I will take another look.

Comments from the review on Reviewable.io

src/Clients/Visuals/visuals/timeline.ts, line 887 [r2] (raw file):
We can't go outside the visual like this. There is no way to enforce a minimum width, please remove this.

Comments from the review on Reviewable.io

Review status: 0 of 9 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

src/Clients/Visuals/visuals/timeline.ts, line 887 [r2] (raw file):
I removed it - Done

Comments from the review on Reviewable.io

src/Clients/Visuals/visuals/timeline.ts, line 738 [r3] (raw file):
Should be using DataViewObject.getValue() for these.

Comments from the review on Reviewable.io

Reviewed 7 of 9 files at r1, 1 of 1 files at r3.
Review status: 8 of 9 files reviewed at latest revision, 8 unresolved discussions, some commit checks failed.

src/Clients/Visuals/visuals/timeline.ts, line 161 [r3] (raw file):
Prefer "== null"

src/Clients/Visuals/visuals/timeline.ts, line 232 [r3] (raw file):
This is not the newest filter code.

src/Clients/Visuals/visuals/timeline.ts, line 445 [r3] (raw file):
Uh oh, do we need this? Can you elaborate on the need for this if so?

src/Clients/Visuals/visuals/timeline.ts, line 510 [r3] (raw file):
Can't do this. You should just be able to do this with .sprite() call in the less file.

src/Clients/Visuals/visuals/timeline.ts, line 1132 [r3] (raw file):
Please use ObjectEnumerationBuilder.

Comments from the review on Reviewable.io

Reviewed 1 of 9 files at r1.
Review status: all files reviewed at latest revision, 8 unresolved discussions, some commit checks failed.

Comments from the review on Reviewable.io

src/Clients/Visuals/visuals/timeline.ts, line 510 [r3] (raw file):
Actually, let's change this to a base64 encoded image (https://css-tricks.com/data-uris/)

Comments from the review on Reviewable.io

This PR seems to be broken and contains unrelated changes, closing.... @IgnatVilesov feel free to send another one  if required


test/params_test.c: diverse fixes

79c0b33 test/params_test.c: make construct_api_params() static
9eaf623 test/params_test.c: make more global variables static
f2003c8 test/params_test.c: use TEST_double_eq to check doubles

I'll just wait for the CIs to confirm as well...

Cis happy.  Merged.
d3620841cc39aaa4a3d75ea32e94ccd110b5bef4 test/params_test.c: use TEST_double_eq to check doubles
932c3d0f6706a08932d89c8610b06cb57fba13ce test/params_test.c: make more global variables static
847275075f971afeb5fa928c078edf3bc17cddb3 test/params_test.c: make construct_api_params() static


bump spec version for test runtime

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #199

PR #198

PR #202   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 24, 4:56 PM: @rphmeier started a stack merge that includes this pull request via Graphite.


It does not work !

hello, I did everything correctly, it puts me in the terminal "Successfully logged in as ????#4659" but nothing happens

Hello @Ayasticall
Thank you for getting in touch. You've followed all the steps correctly, and your question is appreciated. To activate the related
features, a member must first boost or unboost your server, at which point the bot will execute the necessary actions.
All steps have been thoroughly tested, but if you encounter any bugs, please let me know.

Sorry, I didn't know you had to boost ! Thanks


Old MFA recommendations referenced

This public doc still lists the old deprecated recommendations: https://learn.microsoft.com/en-us/azure/defender-for-cloud/multi-factor-authentication-enforcement
As per this it's a v1 and v2 should be in all new MFA docs.
https://learn.microsoft.com/en-us/azure/defender-for-cloud/release-notes#deprecation-of-identity-recommendations-v1

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

ID: 4a76960f-b0c2-d820-5a49-29f19e937e29
Version Independent ID: f66f0b4f-89fd-5b91-66b7-93ee7a0e1d1d
Content: Microsoft Defender for Cloud's security recommendations for MFA
Content Source: articles/defender-for-cloud/multi-factor-authentication-enforcement.md
Service: defender-for-cloud
GitHub Login: @ElazarK
Microsoft Alias: elkrieger

@kev654321
Thanks for your feedback! We have assigned the issue to author and will provide further updates.

This is outdated doc not sure why my comment was edited https://learn.microsoft.com/en-us/azure/defender-for-cloud/multi-factor-authentication-enforcement

@kev654321
Hi, Initially you mentioned two links so I thought that updation is required  to this document "https://learn.microsoft.com/en-us/azure/defender-for-cloud/release-notes#deprecation-of-identity-recommendations-v1"
But I have changed the comment to this document now "https://learn.microsoft.com/en-us/azure/defender-for-cloud/multi-factor-authentication-enforcement"
Thanks for your update. I have assigned the case to author who will update accordingly.

Thank you for your comment. We'll investigate and get back to you.

#label:"backlog-item-created"

@kev654321 you are indeed correct and the document will be changed to reflect the correct information

#please-close


refactor(hooks): add indexes to selectState; enable suggestions

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Example workflows re-enabled

Description
The example workflows were marked to be expected failure. This oversight provided us false confidence that the test cases were passing when in reality they were still failing.

Excellent!  Thanks James!


Improve test framework

There is a few main issues here:

I have to set the bundleThreshold to > 1 in order to get bundles to be generated.

Even with it > 1, the bundle doesn't seem to be correctly referenced in the files.

The sourcemaps have absolute urls in them. Probably related to: https://github.com/systemjs/builder/issues/157 (this is so that @crisptrutski will have an easy test)
I'm not sure we need the absolute path to angular in the output? Instead of System.register("github:angular/bower-angular@1.3.15"), can't we just have it say System.register("angular")?

At some point when this is pretty stable and working, I'll write some passing tests. I guess mostly I just want to grep some of the output JS code and look for specific things.

bundleThreshold should have a value between 0 and 1.0.

Sure, but run this PR and see that the left hand side == 1, so it doesn't catch it. I just set it to 2 to prove the point.
Jon

On Apr 29, 2015, at 12:45 PM, Marjan Georgiev<EMAIL_ADDRESS>wrote:
bundleThreshold should have a value between 0 and 1.0.
—
Reply to this email directly or view it on GitHub.

Fixed mocha timeout to be longer cause travis is slow. PR passing now again. =)

@marjan-georgiev ping! =) I'd like to move forward with more tests and get the points above resolved. Thoughts?

@amcdnl Ping?

Merged it. Sorry for the delay.


Issue with Youtube auto subtitle

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.02.04. (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
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Description
The auto subtitles from youtube no longer work properly.  This used to not happen, and does not seem to be caused by a change in yt-dlp since the behaviour changed without a change in yt-dlp. I have also tested and it does not seem to be a one off issue with this video, it seems to happen on all videos (only on auto generated subtitles not traditional subtitles).
 Thanks

He committed a fix,

Would anyone be able to describe what the fix does? Specifically does it make it work correctly for auto-subs? or does it avoid some kinds of auto-subs to prevent the new junk - but also prevent the auto-subs completely in such cases?
It's not obvious to me by looking at the relevant commit https://github.com/yt-dlp/yt-dlp/commit/d49669acad71f640ffd8b78f0ea7911ae1f67720

Auto-sub URLs normally look like https://www.youtube.com/...lang=so&...tlang=en&.... But now, when lang and tlang are same, youtube doesn't want us to pass tlang in the URL. Doing so causes the damaged subs to be returned

But now, when lang and tlang are same, youtube doesn't want us to pass tlang

Thanks.
Is there a way to identify auto-subs (transcripted) by examining at the URL?
I think before this change the presence of tlang was such indication, but now that it can be removed, can it still be identified somehow?

Not entirely sure, but I believe you can look for kind=asr

Thanks. Appreciate the info/tips.

@pukkandan This issue is still occurring in the latest release 2<IP_ADDRESS>.
But it only occurs when sub langs are , on  for example it is ok.

Provide verbose log

yt-dlp 'https://www.youtube.com/watch?v=AT3fCgxEAJE' --skip-download --write-auto-subs --write-subs   --sub-langs en-en-nP7-2PuUl7o  --verbose -o 'tmp/%(id)s/s'
[debug] Command-line config: ['https://www.youtube.com/watch?v=AT3fCgxEAJE', '--skip-download', '--write-auto-subs', '--write-subs', '--sub-langs', ',en-en-nP7-2PuUl7o', '--verbose', '-o', 'tmp/%(id)s/s']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, err UTF-8, pref UTF-8
[debug] yt-dlp version 2<IP_ADDRESS> [c0c2c57d3]
[debug] Python version 3.6.13 (CPython 64bit) - Darwin-21.2.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Optional libraries: brotli, Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=AT3fCgxEAJE
[youtube] AT3fCgxEAJE: Downloading webpage
[youtube] AT3fCgxEAJE: 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, en-en-nP7-2PuUl7o
[debug] Default format spec: best/bestvideo+bestaudio
[info] AT3fCgxEAJE: Downloading 1 format(s): 22
Deleting existing file tmp/AT3fCgxEAJE/s.en.vtt
[info] Writing video subtitles to: tmp/AT3fCgxEAJE/s.en.vtt
[debug] Invoking downloader on "https://www.youtube.com/api/timedtext?v=AT3fCgxEAJE&asr_langs=de%2Cen%2Ces%2Cfr%2Cid%2Cit%2Cja%2Cko%2Cnl%2Cpt%2Cru%2Ctr%2Cvi&caps=asr&exp=xctw&xoaf=5&hl=en&ip=<IP_ADDRESS>&ipbits=0&expire=1646850707&sparams=ip%2Cipbits%2Cexpire%2Cv%2Casr_langs%2Ccaps%2Cexp%2Cxoaf&signature=DF42CD76201958731864CC1B91B55D1D6F6338EB.3FEC425F6797374F623EA9F0BADD1A6E911C23BC&key=yt8&kind=asr&lang=en&fmt=vtt"
[download] Destination: tmp/AT3fCgxEAJE/s.en.vtt
[download] 100% of 29.01KiB in 00:00
Deleting existing file tmp/AT3fCgxEAJE/s.en-en-nP7-2PuUl7o.vtt
[info] Writing video subtitles to: tmp/AT3fCgxEAJE/s.en-en-nP7-2PuUl7o.vtt
[debug] Invoking downloader on "https://www.youtube.com/api/timedtext?v=AT3fCgxEAJE&asr_langs=de%2Cen%2Ces%2Cfr%2Cid%2Cit%2Cja%2Cko%2Cnl%2Cpt%2Cru%2Ctr%2Cvi&caps=asr&exp=xctw&xoaf=5&hl=en&ip=<IP_ADDRESS>&ipbits=0&expire=1646850707&sparams=ip%2Cipbits%2Cexpire%2Cv%2Casr_langs%2Ccaps%2Cexp%2Cxoaf&signature=DF42CD76201958731864CC1B91B55D1D6F6338EB.3FEC425F6797374F623EA9F0BADD1A6E911C23BC&key=yt8&lang=en&name=en&tlang=en-en-nP7-2PuUl7o&fmt=vtt"
[download] Destination: tmp/AT3fCgxEAJE/s.en-en-nP7-2PuUl7o.vtt
[download] 100% of 5.85KiB in 00:00
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=AT3fCgxEAJE
[youtube] AT3fCgxEAJE: Downloading webpage
[youtube] AT3fCgxEAJE: Downloading android player API JSON

idk why the language code looks like en-nP7-2PuUl7o. Will need to look into it more


add image container and graph examples/tutorials

adding & fixing examples / tutorials for images

z-stacks example
updated image container example -> extended to tutorial
MIBI-TOF tutorial (only works with https://github.com/theislab/squidpy/pull/354)
fixed processing examples
fixed segmentation examples
fixed feature extraction examples

I'd also propose to add a new subsection in examples for the ImageContainer examples, to differentiate between sq.im API and ImageContainer methods
Additionally, how about adding a separation between dataset analysis tutorials and Squidpy tutorials (loading spatial data, dealing with z-stacks, etc)?

locally, building docs now works, CI will fail, because of dependency on "dev" branch from squidpy. Shall I update the requirements for this PR, or should we just merge @giovp ?

The only thing that is not working locally is the preview images and hover info of all auto-generated examples + tutorials. Any idea how to fix this / why its not working?

The only thing that is not working locally is the preview images and hover info of all auto-generated examples + tutorials.

not sure sorry, what does it mean hover info? also, is it for all previous images or only some?

Shall I update the requirements for this PR, or should we just merge @giovp ?

I think it's fine to merge?

The only thing that is not working locally is the preview images and hover info of all auto-generated examples + tutorials.

not sure sorry, what does it mean hover info? also, is it for all previous images or only some?

with hover info I mean the text that is shown if you hover the mouse over the gallery images. Usually a text is show then. This now only works for the pre-generated external tutorials, not for the autogenerated ones.

The only thing that is not working locally is the preview images and hover info of all auto-generated examples + tutorials. Any idea how to fix this / why its not working?

Hopefully it can still be fixed by doing the thing mentioned in https://github.com/theislab/squidpy_notebooks/issues/27#issuecomment-774774145.

The only thing that is not working locally is the preview images and hover info of all auto-generated examples + tutorials. Any idea how to fix this / why its not working?

Hopefully it can still be fixed by doing the thing mentioned in #27 (comment).

Unfortunately that is not it. I checked, and all rst files have the correct format with the extra line. I looked at the built html files, and html/auto_examples/index.html contains the correct images, but html/examples.html does not.
I suspect that there is something out of date on my system that that this is not an actual issue.

@michalk8 I saw that there is a dev branch - should we merge this PR to dev instead of master?

@michalk8 we'd be almost ready to merge this and sync it with master, but there is this thumbnail issues that we are not sure.
In the squidpy_notebooks/docs/build/html/examples.html file it looks like:

but in the squidpy_notebooks/docs/build/html/auto_examples/index.html it's correct

it's also correct the hover text.
I honestly don't remember if this was the case also before, and that CI will evenutally render thumbnails correctly. I don't think either of ones changed anything on the docs side so we don't know what could be the issue.
unrelated to that, I will have to update couple of tutorials (today or tomorrow) and then we can merge to master both this and squidpy main repo I think. Release then will need to wait another 2 weeks (when we submit revision).

@michalk8 I saw that there is a dev branch - should we merge this PR to dev instead of master?

Currently, it doesn't make a difference (haven't enabled dev main docs yet [some issue when building it], will fix it with new release) but afterwards, yes.

@michalk8 we'd be almost ready to merge this and sync it with master, but there is this thumbnail issues that we are not sure.

If you run tox -e shallow-docs, thumbnails for examples are not created because the examples are not run.
Will run tox -e docs after https://github.com/theislab/squidpy/pull/360 is merged.
As for missing hover info, not sure how to fix it - it was always like this. However, hover info when looking from API (e.g. here) works.

unrelated to that, I will have to update couple of tutorials (today or tomorrow) and then we can merge to master both this and squidpy main repo I think. Release then will need to wait another 2 weeks (when we submit revision).

Sounds good.

If you run tox -e shallow-docs, thumbnails for examples are not created because the examples are not run.
Will run tox -e docs after theislab/squidpy#360 is merged.

I stand corrected, tox -e docs also doesn't show the images, I suspect this is the issue (haven't tried fixing it yet).
However, when seeing using these files in final docs, it's fine, see below

The CellProfiler logo is broken because in the main repo, I do not download yet all images (I have a TODO here).

1 small tip: In references.bib, please remove url = ... and eprint = ... (DOI should be sufficient). The problem is that for url, it gets prefixed with https://arxiv.org and I haven't found a way to change this (don't like this tbh).
I've also changed the requirements.txt to use dev (so that we know tests pass), therefore this should be merged to dev as well (still need to find a nice way to regen dev docs) and the above comment about dev https://github.com/theislab/squidpy_notebooks/pull/64#issuecomment-856093128 should be ignored.
Btw, the tutorials looks really cool!

If you run tox -e shallow-docs, thumbnails for examples are not created because the examples are not run.
Will run tox -e docs after theislab/squidpy#360 is merged.

I stand corrected, tox -e docs also doesn't show the images, I suspect this is the issue (haven't tried fixing it yet).
However, when seeing using these files in final docs, it's fine, see below

The CellProfiler logo is broken because in the main repo, I do not download yet all images (I have a TODO here).

Ok, this looks great! I'll make a note in the contributing guide that thumbnail images only work in the final docs. Could you remind me again how you generated the final docs? Is it easily possible to do that locally? I think there was something about a token needed -- is that still the case now that everything is public? Am I right in thinking that I could add dev here https://github.com/theislab/squidpy/blob/c0220c907577196db5f0dd18ff572648a1450612/docs/source/conf.py#L36 and then it would use the dev branch of squidpy_notebooks? I'm happy to have a quick chat about this as well, as this process of linking the repositories in the docs still seems like magic to me. I wanted to document this a bit for future reference.

ok I think we are done from my side re notebooks. Shall we merge ?

will fix linting and also the napari tutorial is not updated yet, will do it by this evening

will fix linting and also the napari tutorial is not updated yet, will do it by this evening

Please also check that everything renders as expected, esp. lists (linting can't catch this since it's still valid RST).

ok, the updated tutorials don't have lists etc cause they are external notebooks

Also, how are the examples.rst and tutorials.rst in the main repo updated? I changed them in Squidpy_notebooks, but of course they are still old in the main repo. Is this a CI thing, or do we do that manually?

Already done in dev branch (though the function needs to be slightly refactored to use git clone).

I wanted to document this a bit for future reference.

Good idea, I can do this once it's also working with dev.

Also, how are the examples.rst and tutorials.rst in the main repo updated? I changed them in Squidpy_notebooks, but of course they are still old in the main repo. Is this a CI thing, or do we do that manually?

Yes, they need to also be updated (it doesn't need to be this case, if the file order we get from globbing would be the one we want - alt would be to rename the files to e.g. 001_<old_name>_py or similar).

Yes, they need to also be updated (it doesn't need to be this case, if the file order we get from globbing would be the one we want - alt would be to rename the files to e.g. 001_<old_name>_py or similar).

Ok, its fine to have this as a manual step as long as its documented somewhere. Now we are mixing exeternal tutorials with rst tutorials in one sections (Napari should be sorted in the "internal" tutorials), so your renaming scheme would not work in this case.

Going to merge this because of https://github.com/theislab/squidpy/pull/365 (want to get stable/latest dev running).
We can then better see if there are any problems. Also now, tox -e docs in squidpy's repo will try to get your local notebooks/examples, see the contributing.


[PyTorch] Fix pad_common for float pad_value

This PR intends to fix the impl of pad_common for float pad_value found in models like pytorch_unet and timm_efficientdet, as well as add a couple new tests for the torch.nn.functional.pad.

@vinx13 @masahi This PR should be ready for another look. Thanks!

@tvm-bot rerun


[SPARK-45752][SQL] Unreferenced CTE should all be checked by CheckAnalysis0

What changes were proposed in this pull request?

This PR fixes an issue that if a CTE is referenced by a non-referenced CTE,  then this CTE should also have ref count as 0 and goes through CheckAnalysis0. This will guarantee analyzer throw proper error message for problematic CTE which is not referenced.
Why are the changes needed?

To improve error message for non-referenced CTE case.
Does this PR introduce any user-facing change?

NO
How was this patch tested?

UT
Was this patch authored or co-authored using generative AI tooling?

NO

@cloud-fan

The code has been updated. Please take a look.

thanks, merging to master!


Allow filtering on multiple semantic types

Current Behavior
Currently, q2-taxa only allows you to operate on a Frequency feature table semantic type, and you can only perform taxonomic filtering on Sequences.
Proposed Behavior
I'd like to be able to collapse relative frequency and compositional tables. (I think it would be harder to manage with presence/absence), filter relative frequency and presence/absent tables, and filter aligned sequences.

@hagenjp Just did something like this for q2-taxa so she is going to take a stab at this issue!

[ ] introduce ability to collapse relative frequency and compositional tables
[ ] Add ability to filter relative frequency
[ ] Add ability to filter aligned seqs
[ ] allow FeatureTable[PresenceAbsence] as an input to filter, and add a TypeMatch such that the output FeatureTable subtype matches the input FeatureTable subtype


Added laravel 3rd party plugin

Added laravel plugin to the 3rd party list

I think the awesome-list addition is sufficient, no need to put it here. @ozh?

Yeh, the readme is much less valuable than the awesome-yourls repo


Azure Container Instance group docs; does "create" with same properties restart the containers?

Details
Service: Azure Container Instances
Relevant documentation: https://learn.microsoft.com/en-us/azure/container-instances/container-instances-update
Feedback:
The docs do not explicitly state whether the containers are updated, when the properties of the az container create command stay the same.
It does say:

Modify or add at least one property of the group that supports update when you redeploy.

But this does not state explicitly what happens when no properties are modified.
It is very useful to know this, because you might use it during the following deployment process:

Create (or update) container instances
Restart container (optional or not?)

Preferred situation:
State explicitly what happens when NO properties are being updated. Does the container get restarted? Or not? Do we manually need to restart the container?

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

ID: df5263d2-e8da-ad3e-efcc-03c0a1088849
Version Independent ID: bbb97f05-ca9a-1558-a95e-bca1fe610752
Content: Update container group - Azure Container Instances
Content Source: articles/container-instances/container-instances-update.md
Service: container-instances
GitHub Login: @tomvcassidy
Microsoft Alias: tomcassidy

@timotk
Thanks for your feedback! We will investigate and update as appropriate.

@timotk
Thank you for bringing this to our attention.
I've delegated this to content author @tomvcassidy, who will review it and offer their insightful opinions.

@tomvcassidy
Could you please review add comments on this, update as appropriate.

Hi @timotk ! Sorry for the delay. I checked with the product team and running a create command a second time with all the same properties and values you included the first time and with no properties added or values modified will result in a container restart. You could also use az container restart to achieve this.
I've added a note to the document to clarify this point for future readers. It is merged as of https://github.com/MicrosoftDocs/azure-docs-pr/pull/273393 and will be published later today.
Thanks, and if you find anything else that needs clarification in the documentation, please let us know.
#please-close

Hi @tomvcassidy , thanks for updating! However, the PR link doesn't work, so I can't see the exact changes.


put an error check for initialStartModels to avoid user input mistake

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
#355
What are the significant changes in functionality due to this change request?
Just an error check.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code. Done
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts). Manual updated, no conversion script needed
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details. yes
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass. Yes
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True. No significant functionality
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync. No change
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.  Close review done
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added? No change

Checklist passes.
Checklist done by @joshua-cogliati-inl on 2017-Sept-14

@joshua-cogliati-inl @alfoa Here is the fix for PR#348

Okay, I'm review it.

Thanks.


Raspberry Pi as mesh node

I want to write a python port for this library that basically allows a Raspberry/Orange/Banana/Panda board to create a wifi hotspot and have at least one ESP8266 node connected to it.
After some easyMesh source code skimming, I'm thinking of doing something like this :

create a wifi hotspot with a name like _meshPrefix+"00001" and _meshPassword that is started before all other nodes
a python script (flask maybe ?) that listens on _meshPort and parses the packet/message
the python script is also able to broadcast a message to the mesh

What do I want from this port, ordered from most important to least :

the raspberry is able to receive/listen to every broadcast witch will contain some sensors readings that I will put in a database (influxDB)
have a updated list of all the nodes and their connections to display a visual representation of the graph
send a message to individual/every node, the esp will parse that message into a command (IR emitter, LED state, OLED display message )

I wrote a easyMesheD that can receive messages and send messages to the mesh. It does need to actively connect to the mesh, so if you want to also connect to the wider network at the same time then you need two network cards on your raspberry pi.
I am currently writing an improved version as well that basically act as a full node as part of painlessMesh, which is a fork of easyMesh (easyMesh seems to have been abandoned).

This seems like a good enough solution, thank you!
I'll try to put it on a raspberry and check it out.

Hi,
Had you succeeded in using raspberry pi as a mesh node ?

No, I had problems with the whole mesh and postponed it.
There is this working library, but again, some mesh problems and the fact that it's written in D  made me not bother with finding a solution.
https://gitlab.com/BlackEdder/painlessMeshListener

Thanks. But the painlessMeshListener is not working. Infact there is no any procedure regarding its usage.

I put some notes on its usage on the painlessMesh wiki and in the readme (although indeed minimal). Maybe if you raise some issues I can help you solve your problem. You can also try to contact me on gitter. https://gitter.im/painlessMesh/Lobby


Add rootAttributes rendering option

Hi,
I want to set the root attributes of a menu from inside a Twig template. I am working on a project where different themes need to have different CSS classes for the menu, and it seems quite ugly to override the MenuBuilder just to change the childAttributes of the root item.
My suggestion is to add a rootAttributes option to the default renderers.
Example usage (Twig renderer):
{# render Bootstrap nav #}
{{ knp_menu_render('main', {
    'currentClass': 'active',
    'rootAttributes': { class: 'nav navbar-nav' }
}) }}

If you agree, I could submit a PR with all necessary changes to the default renderers, docs and twig template.

I'm not entirely certain why it was done this way, but the main issues are here:

https://github.com/KnpLabs/KnpMenu/blob/master/src/Knp/Menu/Renderer/ListRenderer.php#L46
and
https://github.com/KnpLabs/KnpMenu/blob/master/src/Knp/Menu/Resources/views/knp_menu.html.twig#L18

This means that even if you define a Builder as such:
$menu = $factory->createItem('root', $options);

 $menu->setAttribute('class', 'root-class');
 $menu->setAttribute('id', 'root_id');
 $menu->setChildrenAttribute('class', 'child-class');

... the $menu->setAttribute() call is utterly ignored, instead passing an array of the children's attributes into:

https://github.com/KnpLabs/KnpMenu/blob/master/src/Knp/Menu/Renderer/ListRenderer.php#L55
or
https://github.com/KnpLabs/KnpMenu/blob/master/src/Knp/Menu/Resources/views/knp_menu.html.twig#L25

I assume this was done to enable root-level setting of child attributes, and it can be overcome fairly easily with a custom renderer or by extending the base twig template.  If this is not the reason why, I am definitely curious to hear it.

This is because attributes are for the <li> tag and childrenAttributes are for the <ul> tag with the list of children. For the root node, there is no <li> tag, but we decided to keep consistency by always using childrenAttributes for the <ul> even at the root.

@stof: Ah excellent!  Thank you for the clarification.  Is this mentioned somewhere in documentation that I missed?

see the note in https://github.com/KnpLabs/KnpMenu/blob/master/doc/01-Basic-Menus.markdown#menu-attributes

Thank you, sorry for commenting on such an old issue.

wether problem was solved? because I have same use case as @dirkluijk mentioned. And from the code I see that is impossible to pass attributes from twig.

@aur1mas knp_menu_render options are renderer options. These options will not alter the menu being rendered in any way (I will keep rejected any PR trying to do this). There are already several way to influence the menu attributes:

changing the children attributes of the root menu from the template:
{% set menu = knp_menu_get('main') %}
{% do menu.setChildrenAttribute('class', 'child-class') %}
{% knp_menu_render(menu, {'activeClass': 'active'}) %}

passing an option to the menu builder and use it in the builder to change the way the menu is built
{% set menu = knp_menu_get('main', options={'root_class': 'child-class'}) %}
{% knp_menu_render(menu, {'activeClass': 'active'}) %}

class MenuBuilder
{
    // ...

    public function buildMainMenu(array $options)
    {
        $menu = $this->factory->createItem('main', array(
            'childrenAttributes' => array(
                'class' => isset($options['root_class']) ? $options['root_class'] : null,
            ),
        ));

        // Continue building the menu

        return $menu;
    }
}

Note that the second way does not work when registering menu themselves as services (which has several drawbacks, including this one). You need to use either the convention-based provider (AppBundle:MenuBuilder:buildMainMenu here) or the new way to register builders as service once https://github.com/KnpLabs/KnpMenuBundle/pull/273 will be merged and released (hint: very soon)

👍 Need an option to override the root class

I gave 2 solutions just above, which don't require new features (and don't require adding new responsibilities in the renderer)

I myself tried to do this just now.
I was setting the attributes option on the root component of my menu(the ul), and kept wondering why its not working.
After looking at the code I discovered that the it doesn't render from attributes, but from childrenAttributes for some reason.
https://github.com/KnpLabs/KnpMenu/blob/e11c6cdfda1d8e5964960d6c3ed517ce68ba5bf5/src/Knp/Menu/Resources/views/knp_menu.html.twig#L17-L29
I have 2 questions:

What are attributes used for?
What does the children part of childrenAttributes refer to? Children of what?

IMO documenting the ItemInterface at least would be a good idea.

{% set menu = knp_menu_get('main') %} {% do menu.setChildrenAttribute('class', 'child-class') %} {% knp_menu_render(menu, {'activeClass': 'active'}) %}
If menu as a service and need to be used with two different ways, by exemple one render with class on  and do nothing on the other render. do menu.setChildrenAttribute() will work on the service and no on only the display.
Actually the only way is to make a template for each render with special need... :(


Feature Request: Unresolved Links as visible as Resolved Links in LP (Editor mode)

Feature Requested
I want to make a request to make unresolved links as visible as resolved links in LP (editor mode). Thanks
Relevant Screenshot
Here is in the editor mode

Here is in the reading mode

Relevance

[X] The feature would be useful to more users than just me.

Style Settings

[X] I have checked that the feature is not already available via the Style Settings plugin.

that's actually a bug :D
Nevertheless, fixed

Thanks!


escape the spaces in dir for shell command

This should fix the issue in https://github.com/nodegit/nodegit/issues/1082

I'm hoping this can be merged in soon. Sadly I need this for my build system to work correctly.

Thanks @iloahz.
Sorry that NodeGit didn't work for you @jacobmarshall. It is pretty big so I can understand it not fitting into your use case.

Sorry that NodeGit didn't work for you @jacobmarshall. It is pretty big so I can understand it not fitting into your use case.

Thanks @johnhaley81, I seriously appreciate your work. nodegit is a beautifully eloquent library. I look forward to using it in the future.


kubernetes/events: introduce k8s event recording

This change introduces a kubernetes event recording
framework. The events framework allows recording
kubernetes events on a specific k8s object using
the events.NewEventRecorder() api, or using a generic
event recorder using events.GenericEventRecorder().
The generic event recorder is a singleton that is
used to record events against the osm-controller pod
resource.
As a start, some of the fatal events in controller's
startup have been updated to use the event recording
api.
Part of #654
Signed-off-by: Shashank Ram<EMAIL_ADDRESS>
New Functionality      [X]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests / CI System      [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

minor: adding a couple lines on devel.md about this would also be nice

minor: adding a couple lines on devel.md about this would also be nice (for debugging purposes)

Do we have an en existing guide to debug controller problems? Can add it this there.


Automatically create new rows and/or columns when pasting data to the BeeTable

### Acceptance criteria
- [ ] Write a nice description for this issue.
- [ ] Add acceptance criteria here.

Thank you for this issue. I think it is important we clarify what are compatible combinations of copy and paste. Few examples below:

COPY in external spr. and PASTE into DMN decision table data cells, probably supported?
COPY in external spr. and PASTE into DMN relation data cells, probably supported?
COPY in external spr. and PASTE into DMN header cell, probably not supported?
COPY in DMN decision table data cells and PASTE into DMN relation data cells, probably supported?
COPY anywhere and PASTE into context keys column, probably not supported?
COPY anywhere and PASTE into context values column, probably supported?
....


Can't compile under windows with VC++

line #255(\include\windows\basic_dir_monitor_service.hpp) throws error under windows as class dir_monitor_event has no constructor accepting 3 parameters.  Deleting the first parameter will let go, but I do not know the impact without further understanding the internals.

I'll need to test in newer VS, one I have doesn't support lambdas. ¯_(ツ)_/¯

It's not related to lambda, but a signature mismatch, there's no such an overloaded ctor. I guess there was one, but refactored out later.

I know. I just cannot compile this code with my VS, so cannot verify and make a fix for you now.

I see. I solved my problem by forcing the match of signature at the caller, which may bring side effects. Before you have time to see to it. It might be useful to others using the code.

Can you post your patch here?

Or make a pull-request.

Git newbie here. Will do after I figure out how.

No pressure.

Should be fixed now, reopen if you still have troubles with compiling.


Ansible-deployment halts while installing python-package

shepherd-core currently ships with zstandard. v0.22 has no arm7l-package prebuilt, so compilation starts and takes forever.
https://github.com/indygreg/python-zstandard
Updates: https://github.com/indygreg/python-zstandard/issues/205
Workaround on BBone:
cd
git clone https://github.com/orgua/shepherd-datalib
cd shepherd-datalib/shepherd-core
nano pyproject.toml
# edit out zstandard line
sudo pip install .[elf] -U

zstandard must be installed anyway - it compiles, but needs ~20min
sudo pip install zstandard

will be fixed in 0.8.0 release


Improve config section of containerization docs page

Describe changes
Implements some left over todos from the docs refactoring:

Give step-level dependencies their own dedicated section
Mention how to do it with config.yaml

Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

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

@htahir1 done


feat(CI): test caller-pays-for-query truffle example in CI

...per title.

merging if tests pass

merging if tests pass

Build is failing

I chuckled. More so when I looked and it's not this new example causing the failure, it's good old pesky url-requests.
I have zero clue now why the computation datasource in that example occasionally returns nothing from the POST, and that fix I pushed to that clearly hasn't solved the intermittent issue. We should probably drop it from the CI (again) and go back to the drawing board re that particular example.
This new Truffle example however should be absolutely fine in the CI, and it passed here with flying colours.

 Better this way then ending up with two commits in master where the second just undoes the first.
:radioactive: Branch forced - please re-pull.
This way this PR can be merged and closed, and I've created a new issue re the mysterious url-requests so we can deal with that separately instead.


Error while creating a new deployment in different namespace

helm install kubeinvaders --set-string config.target_namespace="namespace1,namespace2" \
-n kubeinvaders2 kubeinvaders/kubeinvaders --set ingress.enabled=true --set ingress.hostName=kubeinvaders.local --set deployment.image.tag=v1.9.6
Error: INSTALLATION FAILED: rendered manifests contain a resource that already exists. Unable to continue with install: ClusterRole "kubeinvaders" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-namespace" must equal "kubeinvaders2": current value is "kubeinvaders1"

Hi @rohits-splunk at momento is not supported a namespace for the installation of KubeInvaders different from a namespace named "kubeinvaders"
The error is correct. The clusterrole already exists.


Include (Python) listener

Motivation: I'd like to use our grammar's (Python) listener to facilitate the parser's utility beyond SQL execution.
More concretely, I'm currently playing around with extracting a dependency tree for my SQL-based pipeline based on the listener.
👉 I propose to ship the listener within the package
Changes: Removed arg no-listener for Python target & re-generated grammar

@svengiegerich just a small suggestion, keep the changes in the generating code from the changes in the generated code in different commits. In this case is not critical, but when there are multiple manual changes, particularly big ones, it's hard to distinguish from the autogenerated code.

Do I need to mock the class TSqlParserListener?

You can try, yes. Do you have an idea how you will use it -- from what I see it's just method stubs with pass? Can you provide your implementation of the listener interface upon parsing?


Native felt support and low-level kernel tx API types in Miden SDK

Introduce low-level (core?) types and functions so that their signatures will be close to the existing transaction kernel API.
Introduce the conversion functions between low-level and high-level types.
To expose native Miden VM felts, implement a WIT Felt(f64) type with:

failable constructor from u64;
arithmetic ops recognizable in the compiler;

Check the following implementation options:

WIT-bindgen.
Try Wasm reference types to encode felts.

To expose the native Miden VM word, implement it as a Word(Felt, Felt, Felt, Felt) tuple.

@bitwalker
I've found the interface remapping option in wit-bindgen but not anything related to a single type remapping. I believe we can make use of it with types that we want to remap put in the separate interface(s).
See the test for it at https://github.com/bytecodealliance/wit-bindgen/blob/442f0054fef4a7ff3c7fa59cbce7f9024569c5bc/crates/rust/tests/codegen.rs#L350-L403
and here is Opts::with at https://github.com/bytecodealliance/wit-bindgen/blob/442f0054fef4a7ff3c7fa59cbce7f9024569c5bc/crates/rust/src/lib.rs#L178-L180
To make it work in cargo-component we'd need to parse these mappings from Cargo.toml metadata. cargo-component builds and passes the Opts at https://github.com/bytecodealliance/cargo-component/blob/b34443033f067c3779519fd828c0886664e909fd/src/bindings.rs#L188-L203
I'll be happy to work on a PR to cargo-component implementing it. There is already an issue https://github.com/bytecodealliance/cargo-component/issues/225 for this feature created 7 hours ago (great timing!) by another cargo-component user. I can figure out if the maintainer is interested in this feature and start cracking.
But before that, I'd like to experiment with implementing some methods for the existing types to see how we can propagate remapped interfaces implementation code through dependencies sdk -> basic-wallet -> p2id-note. Currently, each generates Rust bindings from the referenced WIT files for the same "SDK" types, and there are no crate level dependencies between them.

@bitwalker I've found the interface remapping option in wit-bindgen but not anything related to a single type remapping. I believe we can make use of it with types that we want to remap put in the separate interface(s).

I found that type and resource remapping are missing in wit-bindgen compared to wasmtime's bindgen!. See https://github.com/bytecodealliance/wit-bindgen/issues/825#issuecomment-1917541546
So they have to be implemented in wit-bindgen first before they can be used in cargo-component. It'll take more time, but I'll be happy to jump on it. Do you think type remapping is preferable to interface remapping in our case?

@greenhat Ah that's where I remember seeing, in wasmtime's bindgen! macro, nice find!
I think it's worth us porting over the type/interface remapping used in wasmtime's bindgen! (i.e. the with option). That'll be pretty critical for us, much more so than just being able to control which interfaces are generated (which might force us to do gross things like break up interfaces we might not otherwise, just so we can have more granular control over what gets emitted). Type remapping is super useful IMO, so worth doing.

@bitwalker I've tried the wit-bindgen's with option for interface remapping to get a picture of how Miden SDK users would use it in their projects (think wallets, notes, etc).
On top of the existing direct importing of Miden SDK WIT file, they'd also need to specify a type remapping metadata (I assume a section in Cargo.toml) and include a Miden SDK crate with remapped types as a dependency. Although it looks like a lot of hoops to jump through, I hope we can streamline this DX in our cargo extension and package management tooling.

@greenhat Ah that's where I remember seeing, in wasmtime's bindgen! macro, nice find!
I think it's worth us porting over the type/interface remapping used in wasmtime's bindgen! (i.e. the with option). That'll be pretty critical for us, much more so than just being able to control which interfaces are generated (which might force us to do gross things like break up interfaces we might not otherwise, just so we can have more granular control over what gets emitted). Type remapping is super useful IMO, so worth doing.

I made #117 for this to get back to it later. I want to finish WIT low-level and high-level API (this task) and #114, keeping in mind that the type remapping will be there eventually.

I extracted Miden VM native felt support in to the separate tracking issue - #118


Just want to say thank you Plailect!

Finished the steps, everything in the guide is perfect and went flawlessly. With the guide I was able to downgrade from 10.7.0-32 to 9.2.0 then 2.0.1 and back up to the lastest 11.1 with arm9 installed. I was also able to backup NAND before the arm9 installation which was awesome. Thanks Plailect! I would pm you if I knew how.

nice blogpost.
also stop misusing issue you 'tard.


MINOR : Update registry version and fix transaction isolation enum

Updated registry version to "<IP_ADDRESS>.4.0.0-96" to resolve HDF build issues, will update once again after we are done with registry release. (which should happen in couple of days)

Raised a PR (https://github.com/hortonworks/streamline/pull/1336) to fix this.

thanks @raju-saravanan for the PR.
I could build successfully. You can change the version to 0.6.0 once it is released, which will be done in the next few days.


Support for helm's post-renderer flag

Hello! We are using our own post-renderer with helm. It creates resources from JS files. We are basically writing charts using JavaScript. There are currently no easy way to integrate it with helmfile.
I want to add to helmfile ability to specify post-renderer, so I've written proposal how it may look like. There are no difference with how other helm flags implemented. If everything looks good let me know and I'll start working on a PR.
Such functionality was already discussed in #1351, but since it was more about kustomize, not generic post-renderer, I've decided to create a new issue.
Proposal
Ability to specify post-renderers for release
releases:
  - name: vault
    namespace: vault
    postRenderers:
      - ./path/to/post-renderer

Ability to specify post-renderers for all releases
helmDefaults:
  postRenderers:
    - ./path/to/post-renderer

Ability to reset default post-renderers
releases:
  - name: vault
    namespace: vault
    postRenderers: [] # or null

helmDefaults:
  postRenderers:
    - ./path/to/post-renderer

Path to post-renderer can be be

absolute (starts with /) — then it will be passed to helm as-is
relative (starts with ./) — then it will be prepended with helmfile.yaml dir path and then passed to helm
Other — then it will be passed to helm as-is, assuming the executable exists in the PATH

Also it might be nessecary to add some extra validation if post-renderer executable exists.
--post-renderer flag (or flags) will be passed to commands:

helm diff
helm template
helm upgrade

If installed helm version does not support post-renderer, the error will be displayed. The same with helm-diff (when helmfile diff/apply used)

Hi,
I think that you can use args to work around it.
I have another use case for it - automatic upgrade of resources before applying by kubectl convert.
Thanks

@AssafKatz3 Thanks for sharing! --args isn't well-designed so i wont recommend using it as written at https://github.com/roboll/helmfile/issues/1804#issuecomment-1039975686, but if it works... go ahead.
BTW, I'd welcome any contributions towards adding a post-renderer support to helmfile.

Seems like #510 fixed this.


Disjointness annotations for functions

Currently the getter/setter pattern is problematic in Rust because the borrow checker cannot be sure that a given function only accesses a certain set of fields, which means that getter functions are strictly less useful than field accesses. I propose a syntax somehow similar to this:
struct Bar;

struct Foo {
    a: Bar,
    b: Bar,
}

impl Foo {
    fn a_mut(&mut self) -> &mut Bar {
        &mut self.a
    }

    fn b_mut(&mut self) -> &mut Bar {
        &mut self.b
    }

    fn a_mut_2(&mut Self { a }) -> &mut Bar {
        &mut a
    }

    fn b_mut_2(&mut  Self { b }) -> &mut Bar {
        &mut b
    }
}

fn main() {
    // This currently does not work in Rust
    {
        let mut foo = Foo { a: Bar, b: Bar };
        let a = foo.a_mut();
        let b = foo.b_mut();
    }
    // This would work, with disjointness annotations
    {
        let mut foo = Foo { a: Bar, b: Bar };
        let a = foo.a_mut_2();
        let b = foo.b_mut_2();
    }
}

This would also probably work well with the fields in traits proposal.

dupe of #1215

Yes, closing as a duplicate. Thanks!

I asked on IRC if that existed and I couldn't find it when I searched for "disjoint". Is there a better way to find duplicates?


Electron example

Hey,
Can you show me an example of face detection / recognition, from a local file in electron angular 9.
I used this awesome library in node js with typescript and worked fabulously.

@agnoam hey ,do u get it? Could u show me how to use it?


A user can execute a batch of jobs each with its own client-opts and function symbol

Goose Client API to create a batch of jobs that each have different client-opts and function symbol

This is not in the scope of the current implementation.


Add ability to generate script source for manual script injection in hooks

Adds a new function to return the script source, for manual injection into a <script> tag using a hook, as described in https://github.com/sveltejs/kit/issues/10377#issuecomment-2254615976
Adds a new prop to disable head script addition at the component level, if you are using the above function for manual injection.
Also updates setInitialMode to correctly use configured local storage keys.

is there anything I should do to have this considered?

Hey there @fnimick, thanks for this! I need to investigate why the contents of the <script> aren't being minified. I must have introduced a regression in a previous release as at one point it was being minified.

Any updates or anything I can do to move this along?


Release 2.6

@rultor release, tag is 2..6

@rultor release, tag is 2.6

@rultor release, tag is 2.6

@aliceice OK, I will release it now. Please check the progress here

@rultor release, tag is 2.6

@aliceice Done! FYI, the full log is here (took me 3min)


[BUG] Installation on large disks (>2TB) results in small root disk size

Describe the bug
To Reproduce
Steps to reproduce the behavior:

Install Harvester to a disk larger than 2TB
When Harvester is ready, go to the terminal and check the filesystem.

Results
The root disk size is small and it fails to provision a VM because there's no sufficient disk.
$ df -h|head
Filesystem      Size  Used Avail Use% Mounted on
/dev/sda1        19G  9.3G  8.4G  53% /
/dev/loop1       47M   47M     0 100% /usr
none             63G  1.8M   63G   1% /etc
tmpfs            13G  1.9M   13G   1% /run
tmpfs            13G     0   13G   0% /tmp
dev              10M     0   10M   0% /dev
shm              63G     0   63G   0% /dev/shm
cgroup_root      10M     0   10M   0% /sys/fs/cgroup
/dev/loop2      290M  290M     0 100% /usr/src

$ sudo fdisk -l /dev/sda
Disk /dev/sda: 4.9 TiB,<PHONE_NUMBER>280 bytes,<PHONE_NUMBER> sectors
Disk model: PERC H730 Mini
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disklabel type: dos
Disk identifier: 0x66b82500

Device     Boot Start      End  Sectors Size Id Type
/dev/sda1  *     2048 39903231 39901184  19G 83 Linux

Environment:

Harvester ISO version: v0.1.0
Installation Mode: ISO mode
Underlying Infrastructure Baremetal:Baremetal with 4TB disk

Related issue: https://github.com/rancher/k3os/issues/145

This is an MBR limit. It resizes correctly using EFI boot for large disks.


Add circuit-execution.mdx to docs migration

This is a new file since we came up with the original docs migration plan.
left ToC

patterns index page

Reviewers, please confirm that you like where the page shows up in the screenshots in the PR description, and that you like the new URL guides/circuit-execution


Added SDK release to Whats New page

Added a new bullet for API v3 preview SDK release

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

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

#sign-off

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


When object inherits from IEnumerable model doesn't show up.

Have applied a custom schema in: c.SchemaFilter<ApplySchemaVendorExtensions>();
When trying to show a model that has inherited from IEnumerable the model didn't show up correctly in Swagger-UI. Instead it showed:
[
   "string"
]

ReminderGroupBatch : IReminderGroupBatch inherited from its interface of IReminderGroupBatch : ICloneable, IEnumerable<Uri>
Schema code was:
public void Apply(Schema schema, SchemaRegistry schemaRegistry, Type type)
{
             if (type == typeof(IReminderGroupBatch) || type == typeof(ReminderGroupBatch))
            {
                schema.example = new ReminderGroupBatch
                {
                    BatchGuid = Guid.NewGuid(),
                    PatientReminderUris = new List<Uri>()
                    {
                        new Uri("http://localhost:56003/api/v1/reminders/" + Guid.NewGuid())
                    }
                };
            }
}

After removing the IEnumerable<Uri> the model showed correctly as:
{
  "PatientReminderUris": [
    "http://localhost:56003/api/v1/reminders/4c847cd1-f328-4692-a0ad-1dfa3b87989f"
  ],
  "BatchGuid": "d23a5282-8554-417e-9a5b-9110945826aa"
}

Independent of Swashbuckle, the default JSON serializer for WebAPI will serialize/deserialize an instance of your complex type as a JSON array and not an object. But, it sounds like you expect it to be represented as an object instead of an array (it can't be both in JSON). Have you applied some custom serialization rules to make it behave this way at runtime?

Furthermore, if you do explicitly tell the serializer to treat the type as a JSON object instead of a JSON array as follows, then Swashbuckle will honor this and describe it as an object:
[JsonObject]
public class ReminderGroupBatch : IEnumerable<Uri>
{
  ...
}

@domaindrivendev thanks for the response!


Improve BaseVector::setNull()

bits::kNull ? value : !value is unnecessary because bits::kNull is
a constant. In addition, we can use bits::setNull() to replace
bits::setBit(), so that the intention is clearer.
No functional change in this PR.

@mbasmanova  Could you please help to take a look?  Thanks.


Fix:CloudWatch List metrics with dimensions

Fix:CloudWatch List metrics with dimensions
https://github.com/localstack/localstack/issues/3222

Coverage decreased (-39.06%) to 55.425% when pulling 40fbb5d1f33e410fa17431bbf694485f762c4904 on usmangani1:LISSUE3222 into 47dbad291e717059a93f5cf795a9be08cfc872cc on spulec:master.

This is now part of moto >= 1.3.16.dev134


"Added Test cases for logging identity"

@irina-mitrea-luxoft Please review the new PR for relese/5.1.0

@vstratan This looks ok to me. Do the tests pass successfully? (The Travis build seems to fail before the newly added tests have been run)

@vstratan This looks ok to me. Do the added tests pass successfully on your machine? (The Travis build seems to fail before the newly added tests have been run)

@irina-mitrea-luxoft I can confirm both on wsn and wseb - tests are passing on my local machine

@vstratan Hello, Vlad, just some minor feedback. If we address this minor feedback, this can be merged.

Merging PR as Travis failure doesn't seem to have any connection to current PR.


Could sha-3 be used for Checksum algorithm for container image?

is it possbile to create Container image using sha-3?  if so how?  hats the procedure that this can be achieved?

Nice

Deleting this since I don't think there's any specific change needed for image-spec. But feel free to keep the conversation going here if there's more that OCI can do.


Publish Github action for Linkspector

Allow users to run Linkspector as a GitHub action

Created https://github.com/UmbrellaDocs/action-linkspector and released v1 https://github.com/marketplace/actions/run-linkspector-with-reviewdog


alt key modifier  is not reported

when press ctrl + A, following is reported(don't understant \u{1a} means) on windows 10 OS
character='\u{1a}'
_keymods _repeat=KeyMods { shift: false, ctrl: true, alt: false, logo: false } false
when press alt+...  nothing reported
but alt key is ok on linux ubuntu OS.

more information:
on Ubuntu, home end inert delete enter pgdown pgup, arrow up, arrow down, arrow left, arrow right  capslock ...  are not reported...

about linux, after code reading, I think KEYSYMTAB(in keycodes.rs) should be enlarged to support special key event report

about linux, after code reading, I think KEYSYMTAB(in keycodes.rs) should be enlarged to support special key event report

Feel free to open a PR!

OK,  I will open  one PR for not-fl3/miniquad


Include a link pointing to documentation in the output of istioctl analyze

When istioctl analyze finds validation issues, in addition to listing them it should also print a link to the documentation.
Mockup:
Error [IST0101] (VirtualService ratings-bogus-starhost.default) Referenced host not found: "*.ratings"
Error [IST0101] (VirtualService ratings-bogus-starhost.default) Referenced host+subset in destinationrule not found: "*.ratings+v1"
Warn [IST0107] (Deployment details-v1.default) Misplaced annotation: sidecar.istio.io/statsInclusionPrefixes can only be applied to Pod
Error: Analyzer found issues. For more information about these message codes, see https://istio.io/docs/reference/config/analysis/

Note that the docs don't actually exist yet on istio.io, but do appear in preliminary.istio.io.
We should add this message once we have the reference on istio.io and it's fleshed out enough to be basically useful.

Once we have support for YAML/JSON output, we can include links to the specific doc for each message code directly. We don't want to do so for the "regular" output, though, because it would be very cluttered.

Are there tasks to write the https://preliminary.istio.io/docs/reference/config/analysis/ detail links?  I see some messages are linked to a detail page.

@adammil2000 suggested printing doc links for regular output as well (along with using a specially crafted url so we can get insight on what sort of error codes users are usually hitting).

As of today, the doc pages for each analyzer message are live on the Istio docs site here:
https://istio.io/docs/reference/config/analysis/
You can now link directly to the doc page dedicated to each message from inside Istio code. Each message follows this url format:
https://istio.io/docs/reference/config/analysis/
For example:
https://istio.io/docs/reference/config/analysis/ist0001/
Please note some messages were hidden from the autogenerated doc index like NotImplemented, and others. As of this comment, we did not add a dedicated page for the IST0002: Deprecated message because the plan was to hide it, but it's still present. If we need to keep it, let me know and I'll add a page for it.

@adammil2000 Let's go ahead and add a simple boilerplate page for IST0002 like we have for IST0001. Do you have bandwidth to handle that?

https://github.com/istio/istio.io/pull/5885


VTT: add support for TextAlign

My use case is "STL in to VTT out".
The STL parser seems to implement JC -> TextAlignType, but the TextAlignType is not taken into account by the VTT writer...
Thank you!
Nicolas

How broadly is align supported by WebVTT players?

The only one thing I know for sure, is that iOS devices in France do support align and it makes it possible to support SDH which is somewhat required by CSA. I don' know about broader support in browsers or so.

I think we need to make sure that adding align would not break other VTT players.

... or alternatively we put support for align behind a flag.

Indeed, my PR is based on a "text_position" configuration item. Naming is always the biggest thing, so let me know of course...

Closed by https://github.com/sandflow/ttconv/commit/ecec1faf39e15a2a1ce5571ebd66204a16382354


Derived properties in UV-vis spectroscopy measurements

In  this measurement we need to add the following:

absorptance:

A = 1 - T - R

bandgap: from the inflection point of the absorptance
absorption_coefficient: if the thickness (d) is available for that position

α ≈ ln ((1 − R)/T )/d.

one_sun_implied_jsc with the integral of the product of the absorptance and the AM1.5 GT solar spectrum (similar to the calculation in EQE)

You would need the AM1.5 GT spectrum already used in the EQE normalizer for this.

do you have a formula to calculate the band gap?

do you have a formula to calculate the band gap?

We can us the same way that we do with the EQE. We want to treat the absorptance equivalently to the EQE. In this case, we do not derive Urbach energies because we cannot resolve those very low values well with this technique.

import json
import numpy as np
import matplotlib.pyplot as plt
from scipy.interpolate import interp1d
from scipy.signal import argrelextrema, savgol_filter
from scipy.optimize import curve_fit

# Constants
h = 6.62607015e-34  # Planck's constant in Js
c = 3e8  # Speed of light in m/s
eV = 1.60218e-19  # Electron charge in Coulombs

# Set plot defaults
plt.rcParams.update({
    'axes.titlesize': 24,
    'axes.labelsize': 24,
    'xtick.labelsize': 24,
    'ytick.labelsize': 24,
    'legend.fontsize': 18,
    'xtick.direction': 'in',
    'ytick.direction': 'in',
})

def read_data(filename):
    """Read data from a JSON file."""
    with open(filename, 'r') as file:
        return json.load(file)

def interpolate_data(x, y, new_x, kind='linear'):
    """Interpolate y values at new x coordinates."""
    f = interp1d(x, y, kind=kind, fill_value='extrapolate')
    return f(new_x)

def convert_wavelengths_to_energy(wavelengths):
    """Convert wavelengths in nm to photon energies in eV."""
    wavelengths_m = wavelengths * 1e-9
    return (h * c) / (wavelengths_m * eV)

def calculate_absorptance(trans_intensities, reflectance):
    """Calculate absorptance from transmittance and reflectance."""
    return 1.2 - np.array(trans_intensities) - reflectance

def smooth_data(y, window_length=101, polyorder=3):
    """Apply Savitzky-Golay filter to data."""
    return savgol_filter(y, window_length=window_length, polyorder=polyorder)

def find_peaks_and_fit_gaussian(x, y):
    """Detect peaks and fit a Gaussian around the highest peak."""
    peaks = argrelextrema(y, np.greater)[0]
    if peaks.size > 0:
        peak = peaks[np.argmax(y[peaks])]
        peak_energy = x[peak]
        fitting_range = (x > peak_energy - 0.1) & (x < peak_energy + 0.1)
        popt, _ = curve_fit(
            gaussian,
            x[fitting_range],
            y[fitting_range],
            p0=[y[peak], peak_energy, 0.05],
        )
        return popt[1], popt, fitting_range
    return None, None, None

def gaussian(x, amp, cen, wid):
    """Gaussian function used for peak fitting."""
    return amp * np.exp(-((x - cen) ** 2) / (2 * wid**2))

def plot_results(photon_energies, absorptance, fine_photon_energies, smoothed_absorptance, first_derivative, bandgap_energy, popt, fitting_range):
    """Plot the results of the calculations."""
    fig, axs = plt.subplots(1, 2, figsize=(12, 6))
    axs[0].plot(photon_energies, absorptance, 'o', label='Original Absorptance', alpha=0.5)
    axs[0].plot(fine_photon_energies, smoothed_absorptance, '-', color='r', label='Smoothed Absorptance')
    axs[0].set_ylim(-0.5, 1.1)
    axs[0].set_xlabel('Photon Energy (eV)')
    axs[0].set_ylabel('Absorptance')
    axs[1].plot(fine_photon_energies, first_derivative, '-', label='Smoothed First Derivative')
    axs[1].set_xlabel('Photon Energy (eV)')
    axs[1].set_ylabel('First Derivative')
    if bandgap_energy:
        axs[1].plot(
            fine_photon_energies[fitting_range],
            gaussian(fine_photon_energies[fitting_range], *popt),
            'r-',
            label='Gaussian Fit',
        )
        axs[1].axvline(
            x=bandgap_energy,
            color='r',
            linestyle='--',
            label=f'Bandgap ~ {bandgap_energy:.2f} eV',
        )
    axs[0].legend()
    axs[1].legend()
    plt.tight_layout()
    plt.show()
    return fig

def main(transmittance_file, reflectance_file, start_energy=1.25, end_energy=2.8):
    trans_data = read_data(transmittance_file)
    refl_data = read_data(reflectance_file)
    trans_wavelengths = np.array(trans_data['wavelength'])

from pepe


bug: [Ionic React] Page transition animation missing after update to 4.8.0-rc.0

Bug Report
Ionic React version:
[x] 4.8.0-rc.0
Current behavior:
After updating @ionic/react to latest version, all page transition animations are missing, and pages change instantly with a noticeable flicker.
The issues also occurs when creating a fresh app with sidemenu template.
Changing animated prop on IonRouterOutlet or setting it with setupConfig() doesn't change anything.
Expected behavior:
Page transitions should be animated as in previous versions.
Ionic info:

Ionic:

   Ionic CLI       : 5.2.3 (C:\Users\sato\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework : @ionic/react 4.8.0-rc.0

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   NodeJS : v10.15.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.4.1
   OS     : Windows 10

Hi @SatoAV,
If you are using views outside of tabs, try wrapping your IonRouterOutlet with a n imported from @ionic/react-router. This isn't documented well yet but we will get that updated.
I don't believe the sidemenu has any page transitions in the starters though. The pages are supposed to appear instantly when you switch between them from the side menu.

@elylucas,
I'm indeed using views outside of tabs.
I used sidemenu to start my current project, and it had page transition from the start.
There's no IonRouterOutlet exported from @ionic/react-router though.

@SatoAV
Sorry, looks like my last post got messed up somehow. Wrap your IonRouterOutlet with a ViewManager (which is imported from @ionic/react-router).

@elylucas, wrapping it like so doesn't change anything:
<ViewWrapper>
  <IonRouterOutlet />
</ViewWrapper>

Am I doing something wrong?

@elylucas, that solved the problem, thanks a lot!
Is there any kind of changelog for @ionic/react ?
The only bits of information about recent updates I got were from Ionic Show on YouTube =\

@SatoAV,
I started one in the /packages/react folder. When Ionic React goes final, the changelog will be a part of the main Ionic Repo changelog.

I am still seeing this error... and the stackblitz example you referenced @elylucas https://stackblitz.com/edit/ionic-react-480-rc0-vvnday
doesn't work.


Possible EventEmitter memory leak detected

Hi,
Since using v0.1.9 I see the following error occasional in the logs:
[4/8/2020, 12:38:23 PM] [XboxTv] Device: 192.x.x.x, name: Xbox-L, state: Offline
(node:447) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 _on_discovery_response listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
(node:447) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 _on_console_status listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
[4/8/2020, 12:38:33 PM] [XboxTv] Device: 192.x.x.x, name: Xbox-L, state: Offline
(node:447) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 _on_timeout listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
Config as before:
"devices": [
{
"name": "Xbox-L",
"host": "192.x.x.x",
"xboxliveid": "FD0xxx",
"switchInfoMenu": false
}
],
"platform": "XboxTv"

This is from 3rd SDK package (smartglass), I can not fix them in my plugin, we need to wait for fix from the SDK author. Right now can be ignored.

Have been unable to get it setup due to this as well unfortunately.
ReferenceError: error is not defined
at Object. (/usr/lib/node_modules/homebridge-xbox-tv/node_modules/xbox-smartglass-core-node/src/smartglass.js:188:40)
at EventEmitter.emit (events.js:323:22)
at EventEmitter. (/usr/lib/node_modules/homebridge-xbox-tv/node_modules/xbox-smartglass-core-node/src/events.js:116:27)
at EventEmitter.emit (events.js:311:20)
at Object. (/usr/lib/node_modules/homebridge-xbox-tv/node_modules/xbox-smartglass-core-node/src/smartglass.js:264:30)
at Socket.emit (events.js:311:20)
at UDP.onMessage [as onmessage] (dgram.js:924:8)

Fixed in 0.1.12

Hi @grzegorz914, i've just updated and again got an error, here a part of the log after the update:
[4/13/2020, 4:43:28 PM] [XboxTv] Device: <IP_ADDRESS>, name: Xbox-L1, state: Offline
[4/13/2020, 4:43:33 PM] [Config] Running Command: sudo -E -n npm install --save homebridge-xbox-tv@latest
[4/13/2020, 4:43:38 PM] [XboxTv] Device: <IP_ADDRESS>, name: Xbox-L1, state: Offline
[4/13/2020, 4:44:20 PM] [Config] Homebridge restart request received
[4/13/2020, 4:44:20 PM] [Config] UI / Bridge settings have not changed; only restarting Homebridge process
[4/13/2020, 4:44:20 PM] [HB Supervisor] Sending SIGTERM to Homebridge
[4/13/2020, 4:44:20 PM] Got SIGTERM, shutting down Homebridge...
[4/13/2020, 4:44:20 PM] [Sonos] closed http://<IP_ADDRESS>:44011/notify
[4/13/2020, 4:44:25 PM] [HB Supervisor] Homebridge Process Ended. Code: 143, Signal: null
[4/13/2020, 4:44:30 PM] [HB Supervisor] Restarting Homebridge...
[4/13/2020, 4:44:30 PM] [HB Supervisor] Starting Homebridge with extra flags: -P /homebridge/node_modules
[4/13/2020, 4:44:30 PM] [HB Supervisor] Started Homebridge with PID: 2122
[4/13/2020, 4:44:31 PM] Loaded config.json with 0 accessories and 5 platforms.
[4/13/2020, 4:44:31 PM] ---
[4/13/2020, 4:44:32 PM] Loaded plugin: homebridge-camera-ffmpeg
[4/13/2020, 4:44:32 PM] Registering platform 'homebridge-camera-ffmpeg.Camera-ffmpeg'
[4/13/2020, 4:44:32 PM] ---
[4/13/2020, 4:44:32 PM] Loaded plugin: homebridge-sss-platform
[4/13/2020, 4:44:32 PM] Registering platform 'homebridge-sss-platform.sss-platform'
[4/13/2020, 4:44:32 PM] ---
[4/13/2020, 4:44:32 PM] Loaded plugin: homebridge-xbox-tv
[4/13/2020, 4:44:32 PM] Registering platform 'homebridge-xbox-tv.XboxTv'
[4/13/2020, 4:44:32 PM] ---
[4/13/2020, 4:44:32 PM] Loaded plugin: homebridge-zp
[4/13/2020, 4:44:32 PM] Registering platform 'homebridge-lib.Lib'
[4/13/2020, 4:44:32 PM] Registering platform 'homebridge-zp.ZP'
[4/13/2020, 4:44:32 PM] ---
[4/13/2020, 4:44:32 PM] Loaded plugin: homebridge-config-ui-x
[4/13/2020, 4:44:32 PM] Registering platform 'homebridge-config-ui-x.config'
[4/13/2020, 4:44:32 PM] ---
[4/13/2020, 4:44:32 PM] Loading 5 platforms...
[4/13/2020, 4:46:04 PM] [XboxTv] Device: <IP_ADDRESS>, name: Xbox-L1, state: Offline
(node:2122) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 _on_discovery_response listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
(node:2122) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 _on_console_status listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
[4/13/2020, 4:46:14 PM] [XboxTv] Device: <IP_ADDRESS>, name: Xbox-L1, state: Offline
This was an inline update via homebridge-config-ui-x to homebridge-xbox-tv v0.1.12, anything else I should do, can help with?
Thx

Ps. a Docker clear and reinstall also throws error:
[4/13/2020, 9:46:24 PM] [XboxTv] Device: <IP_ADDRESS>, name: Xbox-L1, state: Offline
(node:495) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 _on_discovery_response listeners added to [EventEmitter]. Use
emitter.setMaxListeners() to increase limit
(node:495) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 _on_console_status listeners added to [EventEmitter]. Use emi
tter.setMaxListeners() to increase limit

please check latest ver. 0.1.31

No errors since 12 hrs, docker clear also good, looks good!

OK, i close this, if some happen feel free to open.


aliases.nix: add aliases for removed torch packages

We've removed the abandoned and broken torch project as part of https://github.com/NixOS/nixpkgs/issues/71888
This commit adds aliases for:

https://github.com/NixOS/nixpkgs/pull/81173
https://github.com/NixOS/nixpkgs/pull/83568

CC @worldofpeace @nh2


Request Valiation for Rest Apis

Currently validations on beans do not work. I tried to add @Valid  annotation on CrudController but is not working. Also tried overriding the methods and provide validation implementation in the specific controller but that also not working.
would like to have following to perform validations for the apis
    @PostMapping
    public Long create(@RequestBody **@Valid** DTO dto) {
        return facade.create(dto);
    }

Have you tried using @Validated?
I'm going to start implementing this code tomorrow

I confirm that with @Validated it works fine. It's still a Work In Progress.


feat [remote_config] Support for fetchTimeout

Description
Remote Config exposes settings for configurable fetch timeout, but it's missing in current RemoteConfigSettings implementation. Would be great to have it customizable as default 1 minute doesn't work well in every app.
https://firebase.google.com/docs/reference/ios/firebaseremoteconfig/api/reference/Classes/FIRRemoteConfigSettings?authuser=1#fetchtimeout
Related Issues
None I'm aware of.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@googlebot I signed it!

@googlebot I fixed it.

Thanks for this PR, apologies that it got stale - this was out of our control. I have noticed however that the latest Remote Config plugin now already supports this: https://firebase.flutter.dev/docs/remote-config/usage#settings so i'm going to go ahead and close this PR now. Thanks again.


Escapes table name to fix Mysql2 errors

I'm working on a project where I have inherited weird table names (porting the app from another language/framework to Ruby on Rails).
Currently some tables are named like so table-name which cause the following error:
Mysql2::Error: Unknown column 'table' in 'where clause': SELECT  table-name.* FROM table-nameWHERE (table-name.position IS NOT NULL) ANDtable-name.snapshot_query_id = 1  ORDER BY table-name.position DESC LIMIT 1
Escaping the table name with ` ` helps avoid this problem.

Could you please rebase and use quoted_table_name instead of plain backticks?
That would make code working in other DB as well.

@ychaker, are you still wanting us to progress on this?

@brendon I had totally forgotten about this (and hadn't seen @fabn's comment).
I actually am using my own fork at the moment, so yes I would like to switch back to the main gem if this gets fixed.

Addressed in #205, it also handle quoting for column names.

@ychaker, I've just merged @fabn's commit into master. Can you double check that this is now working for you?

@brendon i use @fabn's branch earlier today and that worked, so I'm sure master should work. I'll test it again tomorrow and let you if it doesn't. But if you don't hear from me (in case I forget) that should mean that it's working fine.
Thank you 😄

No worries :D

@brendon I tested the master branch this morning and it works.
Thanks for getting this done.

You're welcome :D