--replace-in-metadata is not working with unicode characters

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

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

Provide a description that is worded well enough to be understood
This issue is related to the https://github.com/yt-dlp/yt-dlp/issues/4547 in which some of the special characters (such as question mark)  from the metadata are being incorrectly changed into Unicode look-alikes and saved in file names. So far there have been 5 cases encountered:

Should use | instead of ｜.
Should use : instead of ：.
Should use ? instead of ？.
Should use * instead of ＊.
Should use " instead of ＂.

I tried to use the --replace-in-metadata to fix the problem temporarily, but encountered new bugs - hence the current issue.
Bug 1
--replace-in-metadata "title,uploader,channel,uploader_id,channel_id" "[：]" ":" does not work as the yt-dlp does not find the character.
Instead of "[：]", I also tried "：", "[\uff1a]", "\uff1a"  but to no avail.
Additional info encountered earlier by another person in this comment: https://github.com/yt-dlp/yt-dlp/issues/5341#issuecomment-1575638499
Bug 2
Even if the first bug is going to be fixed (or the underlying issue #4547), that does not solve the problem because the normal->unicode translation is happening even with characters directly mentioned in the output filename template.
Therefore

By %(uploader_id,channel_id)s[%(webpage_url_domain)s%(id&:)s%(id|)s] becomes
By @AlexanderBromley[youtube.com：rZEh4kcpAAc] (instead of squared brackets I would prefer to use parenthesis, but escaping them is unsupported)

And since it is not part of meta-data there is no way to substitute incorrect characters using --replace-in-metadata. Unless I don't know of a way of applying the replacement on the whole name as a whole (instead of individually on metadata fields)?
Test command for both bugs
Test command: yt-dlp -vU -f "bv*[height<=480]+ba/b[height<=480]" --merge-output-format mkv --embed-thumbnail --embed-metadata --embed-info-json --embed-chapters --embed-subs --sub-langs all --no-write-auto-subs --sponsorblock-mark all --no-mtime --progress --concurrent-fragments 10 -R infinite -o '~/Downloads/Videos/%(webpage_url_domain)s/%(uploader_id,channel_id)s%(uploader,channel& [|)s%(uploader,channel|)s%(uploader,channel&]|)s/%(release_date>%Y-%m-%d,upload_date>%Y-%m-%d)s_%(title|).180B%(title.181&…|)s_By %(uploader_id,channel_id)s[%(webpage_url_domain)s%(id&:)s%(id|)s]_%(height|480)sp.%(ext)s' https://www.youtube.com/watch\?v\=rZEh4kcpAAc --replace-in-metadata "title,uploader,channel,uploader_id,channel_id" "[：]" ":"
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '-f', 'bv*[height<=480]+ba/b[height<=480]', '--merge-output-format', 'mkv', '--embed-thumbnail', '--embed-metadata', '--embed-info-json', '--embed-chapters', '--embed-subs', '--sub-langs', 'all', '--no-write-auto-subs', '--sponsorblock-mark', 'all', '--no-mtime', '--progress', '--concurrent-fragments', '10', '-R', 'infinite', '-o', '~/Downloads/Videos/%(webpage_url_domain)s/%(uploader_id,channel_id)s%(uploader,channel& [|)s%(uploader,channel|)s%(uploader,channel&]|)s/%(release_date>%Y-%m-%d,upload_date>%Y-%m-%d)s_%(title|).180B%(title.181&…|)s_By %(uploader_id,channel_id)s[%(webpage_url_domain)s%(id&:)s%(id|)s]_%(height|480)sp.%(ext)s', 'https://www.youtube.com/watch?v=rZEh4kcpAAc', '--replace-in-metadata', 'title,uploader,channel,uploader_id,channel_id', '[：]', ':']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[812cdfa06] (linux_exe)
[debug] Python 3.10.11 (CPython x86_64 64bit) - Linux-5.19.0-43-generic-x86_64-with-glibc2.35 (OpenSSL 3.1.1 30 May 2023, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, phantomjs broken, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.05.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1851 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 18e328f92e9e172f5757a8084c7c6ef950d846ffbef57da3785aeb601e16cdc8
yt-dlp is up to date<EMAIL_ADDRESS>[youtube] Extracting URL: https://www.youtube.com/watch?v=rZEh4kcpAAc
[youtube] rZEh4kcpAAc: Downloading webpage
[youtube] rZEh4kcpAAc: Downloading ios player API JSON
[youtube] rZEh4kcpAAc: Downloading android player API JSON
[youtube] rZEh4kcpAAc: Downloading m3u8 information
[youtube] rZEh4kcpAAc: Downloading MPD manifest
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Replacing all '[：]' in title with ':'
[MetadataParser] Did not find '[：]' in title
[debug] Replacing all '[：]' in uploader with ':'
[MetadataParser] Did not find '[：]' in uploader
[debug] Replacing all '[：]' in channel with ':'
[MetadataParser] Did not find '[：]' in channel
[debug] Replacing all '[：]' in uploader_id with ':'
[MetadataParser] Did not find '[：]' in uploader_id
[debug] Replacing all '[：]' in channel_id with ':'
[MetadataParser] Did not find '[：]' in channel_id
[SponsorBlock] Fetching SponsorBlock segments
[debug] SponsorBlock query: https://sponsor.ajay.app/api/skipSegments/c84e?service=YouTube&categories=%5B%22preview%22%2C+%22outro%22%2C+%22poi_highlight%22%2C+%22interaction%22%2C+%22chapter%22%2C+%22sponsor%22%2C+%22music_offtopic%22%2C+%22intro%22%2C+%22filler%22%2C+%22selfpromo%22%5D&actionTypes=%5B%22skip%22%2C+%22poi%22%2C+%22chapter%22%5D
[SponsorBlock] No matching segments were found in the SponsorBlock database
[info] rZEh4kcpAAc: Downloading 1 format(s): 244+251
[info] There are no subtitles for the requested languages
[info] Downloading video thumbnail 41 ...
[info] Video Thumbnail 41 does not exist
[info] Downloading video thumbnail 40 ...
[info] Video Thumbnail 40 does not exist
[info] Downloading video thumbnail 39 ...
[info] Video Thumbnail 39 does not exist
[info] Downloading video thumbnail 38 ...
[info] Video Thumbnail 38 does not exist
[info] Downloading video thumbnail 37 ...
[info] Writing video thumbnail 37 to: /home/mikko/Downloads/Videos/youtube.com/@AlexanderBromley [Alexander Bromley]/2019-04-01_Part I：  What is Anterior Pelvic Tilt, How it Can Affect Your Lifts and How to Know if You Have It_By @AlexanderBromley[youtube.com：rZEh4kcpAAc]_480p.webp
[debug] Invoking dashsegments downloader on "https://manifest.googlevideo.com/api/manifest/dash/expire/1688355492/ei/RO6hZJeDGsflyAW64IjQBw/ip/2001%3A14ba%3Aa403%3A1f00%3A4f38%3Afaad%3A551b%3A5c21/id/ad9121e247290007/source/youtube/requiressl/yes/playback_host/rr1---sn-qo5-2vgs.googlevideo.com/mh/0d/mm/31%2C29/mn/sn-qo5-2vgs%2Csn-ixh7yn7d/ms/au%2Crdu/mv/m/mvi/1/pl/40/hfr/all/as/fmp4_audio_clear%2Cwebm_audio_clear%2Cwebm2_audio_clear%2Cfmp4_sd_hd_clear%2Cwebm2_sd_hd_clear/initcwndbps/1650000/spc/Ul2Sq5IBkPC1RmIGSdXn0ilCdWfRoOJcj2mNDbM4Pw/vprv/1/mt/1688333380/fvip/2/keepalive/yes/fexp/24007246/beids/24350017/itag/0/sparams/expire%2Cei%2Cip%2Cid%2Csource%2Crequiressl%2Chfr%2Cas%2Cspc%2Cvprv%2Citag/sig/AOq0QJ8wRQIhAK1GwBcwGx50Bgcg3WLG_Z4DGs30mt8HFjaw35fVY076AiBlnulZVv-6qazoS-hzQnLH80sSpquBAmsVRZLflqQt-w%3D%3D/lsparams/playback_host%2Cmh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps/lsig/AG3C_xAwRAIgaoN5DCH6Bpxu6-sRvph6iGDYOjhew0rsVNTU27aGAUACIEm8vK4wLKjUSJbVe4m5et9Byn6dhmdSOiit9tBG88rM"
[dashsegments] Total fragments: 142
[download] Destination: /home/mikko/Downloads/Videos/youtube.com/@AlexanderBromley [Alexander Bromley]/2019-04-01_Part I：  What is Anterior Pelvic Tilt, How it Can Affect Your Lifts and How to Know if You Have It_By @AlexanderBromley[youtube.com：rZEh4kcpAAc]_480p.f244.webm
WARNING: The download speed shown is only of one thread.  Many athletes suffer from bad habits, one of the most common is the anterior pelvic tilt. When the hips tilt anteriorly, the abdominals, glutes, and hamstrings are stretched, and the hip flexors and erectors tightened. This causes serious problems with  performance and can even lead to chronic back and neck pain.

Avoiding these issues and optimizing performance means knowing how to diagnose the anterior pelvic tilt early and fixing it asap.

In this video, we go over proper posture and alignment and compare that to anterior pelvic tilt posture. We discuss the muscles that contribute to bad posture, and how to lengthen and strengthen them.' -metadata 'synopsis=FREE 50pg eBook "Breathing and Bracing"
https://empirebarbell.clickfunnels.com/optin26667388

All athletics require stability and coordination, and that means a sound base of posture and bracing. Many athletes suffer from bad habits, one of the most common is the anterior pelvic tilt. When the hips tilt anteriorly, the abdominals, glutes, and hamstrings are stretched, and the hip flexors and erectors tightened. This causes serious problems with  performance and can even lead to chronic back and neck pain.

Avoiding these issues and optimizing performance means knowing how to diagnose the anterior pelvic tilt early and fixing it asap.

In this video, we go over proper posture and alignment and compare that to anterior pelvic tilt posture.  The character in title etc is already just ASCII : and not Unicode ：. They are only replaced inside the filename
How is "Bug 2" different from #4547?

"Bug 1" is invalid. The character in title etc is already just ASCII : and not Unicode ：. They are only replaced inside the filename

I don't understand...
Let's break it down.

This youtube video's title is Part I: What is Anterior Pelvic Tilt, How it Can Affect Your Lifts and How to Know if You Have It. It has the ASCII : in it.
When saving the file with the meta-field title in it, ASCII : becomes Unicode ：. This is what #4547 is about.
Additionally:
The current issue's Bug1 shows that --replace-in-metadata flag is buggy and not working when trying to fix the unnecessary ASCII->Unicode renaming in meta-fields.
The current issue's Bug2 also demonstrates that the renaming problem not only concerns meta-fields, but also the non-meta-field input - the characters that were directly written in the output filename template (as in Title: %(title)s).

So I disagree with the duplication categorization for both sub-bugs of this current issue.

When saving the file with the meta-field title in it, ASCII : becomes Unicode ：

No, the : is only "sanitized" when used as part of the filename. The value of the field itself is unchanged. You can check this in he info-json or using --print

The current issue's Bug1 shows that --replace-in-metadata flag is buggy and not working when trying to fix the unnecessary ASCII->Unicode renaming in meta-fields.

Therefore, when you ask yt-dlp to replace the character in the metadata field, there i nothing to replace.

The current issue's Bug2 also demonstrates that the renaming problem not only concerns meta-fields, but also the non-meta-field input - the characters that were directly written in the output filename template (as in Title: %(title)s).

Again, it is the filename that is sanitized, not the field. So this will be automatically fixed when https://github.com/yt-dlp/yt-dlp/issues/4547 is


It is not possible to subscribe to one topic with many addresses

sub1, _ := b.ws.Subscribe.Status("VBRPREMRDLANIF2WT2WRFXNLC5GDFGZYBVFSSAGK")
sub2, _ := b.ws.Subscribe.Status("VDYWAEJF7ZNGEOPAVGMSX6KDAX6PHVRENJVKX2OM")

Sub2 overrides sub1 programmatically, while websocket channel is still sending messages to sdk, which can't handle them

in your example do you use a single websocket connection for two accounts?

fixed up https://github.com/proximax-storage/nem2-sdk-go/pull/96/commits/6983f92a9a5bfc85602f3a619c3835248ff894d5


ExpandedControlsActivity leaks every time it is dismissed

Once a video is casting and ExpandedControlsActivity is showing, each time I dismiss the controls by tapping the arrow in the top-left, the activity is leaked. I see these messages if I turn on StrictMode for the app:
12-08 15:37:04.980 18037-18037/? E/StrictMode: class com.google.sample.cast.refplayer.expandedcontrols.ExpandedControlsActivity; instances=4; limit=1
                                               android.os.StrictMode$InstanceCountViolation: class com.google.sample.cast.refplayer.expandedcontrols.ExpandedControlsActivity; instances=4; limit=1
                                                   at android.os.StrictMode.setClassInstanceLimit(StrictMode.java:1)

If you believe this is a bug in the Cast SDK, please file an issue: https://code.google.com/p/google-cast-sdk/issues/list


fix(plugin-cloud-storage): encode filename to remove unsupported chars

Description
I've used an encodeURIComponent to encode incoming filename from DB that is broken when unsupported characters are included in string.
This fixes #6985

[ ] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[ ] Chore (non-breaking change which does not add functionality)
[x] 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 not work as expected)
[ ] Change to the templates directory (does not affect core functionality)
[ ] Change to the examples directory (does not affect core functionality)
[ ] This change requires a documentation update

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

At first glance, I think this would be a breaking change with images that have spaces in them. We'd need to make this safely backwards-compatible.

Correct me if I'm wrong, but I believe the filename args for each adapter are all generated here. If that's the case, a backwards-compatible solution would be to expose a config field for sanitize-filename options.
That would also solve this issue for local uploads, and would enable nested filepaths which currently don't work.


fix: added placeholder to field

Pull Request Template
Issue Overview
This PR addresses #1301

[x] This change addresses the issue in full
[ ] This change addresses only certain aspects of the issue
[ ] This change is a dependency for another issue
[ ] This change has a dependency from another issue

Description
Since error messaging around invalid urls is already implemented, this PR just updates the placeholder based on discussions with @kathyccheng.
How Can This Be Tested/Reviewed?
This can be tested by going to Detroit Partners, signing in, editing an application, going to the 'Application Process' tab, and emptying the company website field to see the new placeholder.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have added QA notes to the issue with applicable URLs
[x] I have performed a self-review of my own code
[x] I have reviewed the changes in a desktop view
[ ] I have reviewed the changes in a mobile view
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have assigned reviewers
[ ] I have run yarn generate:client and/or created a migration if I made backend changes that require them
[ ] I have exported any new pieces added to ui-components
[x] My commit message(s) is/are polished, and any breaking changes are indicated in the message and are well-described
[ ] Commits made across packages purposefully have the same commit message/version change, else are separated into different commits

Reviewer Notes:
Steps to review a PR:

Read and understand the issue, and ensure the author has added QA notes
Review the code itself from a style point of view
Pull the changes down locally and test that the acceptance criteria is met
Also review the acceptance criteria on the Netlify deploy preview (noting that these do not yet include any backend changes made in the PR)
Either explicitly ask a clarifying question, request changes, or approve the PR if there are small remaining changes but the PR is otherwise good to go

On Merge:
If you have one commit and message, squash. If you need each message to be applied, rebase and merge.

@ColinBuyck Looking at the issue, it doesn't seem like the issue was targeting placeholder text. To be honest I'm not actually sure what the issue was looking for, but to me they don't seem related. I may be missing some offline context?

@emilyjablonski @kathyccheng You're correct about the offline context, and sorry for not reflecting that here! Kathy and I discussed this since the error below is already in place. As an alternative, she suggested updating the placeholder text. I can update the issue now!


Fix eqs rendering in SolveMathProblems

This PR fixes the eqs. rendering, e.g., things like the attached screenshot with $ dangling around.

@lmoresi, do you know if there is a way to see how the online version of the PR will look like?

[https://tr.cloudmagic.com/h/v6/emailtag/tag/2.0/1618462758/e0463a85314d353cd65b915287a67da9/5/56e0546f227d5dd9cd487b5bee1a3fe2/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d/newton.gif]
To preview, install jupyterbook and build it from the Jupyterbook directory
jupyter-book build .
Prof Louis Moresi
<EMAIL_ADDRESS>(m) +61 4 0333 1413tel:+61403331413
(us) +1<PHONE_NUMBER>tel:+15053494425
www.moresi.infohttps://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403327-225ad204-871d-a517-57dc-9766b7b2c2f0/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=http://www.moresi.info/
geo-down-under.geoscience.educationhttps://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403339-3c047c70-1902-7ebf-c739-ea6202cf2397/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=https://geo-down-under.geoscience.education/
@LouisMoresihttps://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403345-e8032d81-ce21-860a-c7b2-396b5df7ccad/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=https://twitter.com/LouisMoresi
On Thu, Apr 15, 2021 at 14:50, Navid C. Constantinou<EMAIL_ADDRESS>wrote:
This PR fixes the eqs. rendering, e.g., things like the attached screenshot with $ dangling around.
[Screen Shot 2021-04-15 at 2 49 07 pm]https://user-images.githubusercontent.com/7112768/114815568-cb68e980-9df9-11eb-8baa-2b66e8eabf6c.png
@lmoresihttps://github.com/lmoresi, do you know if there is a way to see how the online version of the PR will look like?

You can view, comment on, or merge this pull request online at:
https://github.com/ANU-RSES-Education/EMSC-4033/pull/17https://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403349-fc9ad6f7-cd19-e56a-3350-391d7799d2b2/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=https://github.com/ANU-RSES-Education/EMSC-4033/pull/17
Commit Summary

fix eqs rendering in SolveMathProblems

File Changes

M Notebooks/Themes/SolveMathProblems/0-IntroductionToNumericalSolutions.mdhttps://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403355-6132ffba-c68a-0383-cf0d-8ea63f8dc75a/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=https://github.com/ANU-RSES-Education/EMSC-4033/pull/17/files#diff-485cd87bfeeb82f122a45178c456d150c38e72e171bbe59d2f5b26acf9c77e14 (42)

Patch Links:

https://github.com/ANU-RSES-Education/EMSC-4033/pull/17.patchhttps://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403361-72c6d1a5-4c5a-134a-db18-15537277b460/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=https://github.com/ANU-RSES-Education/EMSC-4033/pull/17.patch
https://github.com/ANU-RSES-Education/EMSC-4033/pull/17.diffhttps://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403366-22301163-3b4e-9b94-308b-0e470f91274a/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=https://github.com/ANU-RSES-Education/EMSC-4033/pull/17.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ANU-RSES-Education/EMSC-4033/pull/17, or unsubscribehttps://tr.cloudmagic.com/h/v6/link-track/1.0/1618462775403370-b5f5b7c8-2129-aa61-03bd-8844763cf293/1618462758/e0463a85314d353cd65b915287a67da9/5250f4c7ec865783b4effb79dc50600f/28b39f8600e15803408c6a1afc4b3f4d?redirect_uri=https://github.com/notifications/unsubscribe-auth/ADABPI5UOI7R44NFS4DQHSTTIZWAHANCNFSM426VXB6A.


Tweet action buttons displayed above image preview when using large previews

This can't be changed due to the way large previews work in TweetDeck.
The tweet content and buttons are merged, while the image preview goes outside the box containing the tweet.
The only way to fix it would be to use some horrifyingly disgusting and messy hack, but I'm not doing that.


Social media integration

We should discuss and (if desired) implement the following features:

Social media buttons
Twitter feed integration
... (More ideas?)

Homepage is main source for information.
Social media buttons linking to Facebook, Twitter, GitHub, Xing are integrated on the "Mitmachen" page


Turn db_handler to a class (and other small things)

The main difference is the db_handler is now a class. Had to make some changes to the telescope model as well, but they were quite small.
The problem, as usual, is that I can't test those changes. The db_handler tests pass (both Yaml and MongoDB), but I can't run the telescope model part. I am trying to do it on the wgs, but that turned out to be a nightmare (ran out of quota in the middle of creating the conda environment and now my conda is completely screwed up).
@RaulRPrado, can you please run the other tests before you approve the merge?

Fixed my conda installation on the server and will run the tests myself (already found issues). Will fix and update this PR.

Looks good. I'll review it carefully soon, but probably not today.

Ok, everything is fine. You may merge it.


python310Packages.bravado-core: disable failing tests

Description of changes
Fix build (https://hydra.nixos.org/build/178181709)
ZHF: #172160

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 174952 run on x86_64-linux 1

3 packages built:

powerdns-admin
python310Packages.bravado-core
python39Packages.bravado-core

Result of nixpkgs-review pr 174952 run on x86_64-linux 1

3 packages built:

powerdns-admin
python310Packages.bravado-core
python39Packages.bravado-core

candidate backport: #177017


Add advanced “TEMPLATE_ENGINE.md” documentation.

The point of this document is to explain (to folks that are already experts in web development) how html template engines conceptually work.
It walks the reader through a series of iterations which add increasing sophistication to a toy template engine library.
Closes #187.

@klebba — Took some time (maybe more than I’d like to admit) trying to update these docs as you requested.
As discussed, these are aimed at folks who are already experts with JS / DOM-APIs. Rather than be a high-level explainer (like the last one was) — it goes through a series of examples (each building off the last) to introduce the key problems that template engine has to solve.

Iterated on nearly all the comments you left in here @klebba — the one main thing that I am going to put to later is adding additional code comments to the example code. I think that will actually take quite a bit more time to thoughtfully do and I’d like to pause on this for now.


Added interactive tests

Hey there! Thanks for using Gluegun. Your CLI looks rad.
I decided to implement tests on your CLI to brush up on my interactive CLI testing skills. I learned quite a few things (including some things I know I'm still not good on), so it was worth the effort.
You get a free interactive test out of it. :-) Enjoy.

I actually have planned on adding tests, but because I'm really new in gluegun or even this tests things, I just don't know where to start. So this is really helpful. Thanks 😄

Feel free to upgrade to Gluegun 4.0.0, which I just released. It should be compatible.


[stable/prometheus] Fix node exporter args

fixes #3519
NodeExporter supports passing extraArgs as single key
please merge after https://github.com/kubernetes/charts/pull/3522  (version 5.0.3), this is 5.0.4

/assign @mgoodness

/ok-to-test

/lgtm

@unguiculus  you need to put 5.1.2 I guess...

/retest

/lgtm


BrowserModule has already been loaded (on lazy load module)

When I try to import FancyImageUploaderModule on a lazy module, I got the following error:
Error: BrowserModule has already been loaded. If you need access to common directives such as NgIf and NgFor from a lazy loaded module, import CommonModule instead.
My main module (app.module.ts):
@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    BrowserAnimationsModule,
    HttpClientModule,
    appRouter
  ],
  providers: [
  ],
  bootstrap: [AppComponent]
})

Lazy module with FancyImageUploaderModule:
@NgModule({
  imports: [
    CommonModule,
    PainelRouter,
    FancyImageUploaderModule
  ],
  declarations: [PainelComponent]
})

Thanks for reporting! I guess I use wrong module (BrowserModule). Will make a fix this evening.


Release v1.0.0

Coverage remained the same at 100.0% when pulling 6e57e865d342aa73e50932ba3a93143205ac14d1 on release-v1.0.0 into 568921ff14803fef34476eb936b507a1faa714f5 on master.


chore(release): 1.107.1

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-fkVQbXRiQi6A
Commit ID: 1cf96b1cd0969c28dfb41ad07b995a8f121cce6f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


chore: Update .codegen.json with commit hash of codegen and openapi spec

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.435%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
10557

Relevant Lines:
11421

💛  - Coveralls


[Bug]: Error 400 when something is wrong with the connection string

What happened?
My connection string happened to have a random generated password that contains a semicolon (;) and the static web app would return an error 400 but no other data that could help with the problem. I've solved the issue by adding single quotes on the password field at the Link your database step but I think this can be addressed with code so that the single quotes are added by the code itself.
Version
v0.5.34
What database are you using?
Azure SQL
What hosting model are you using?
Static Web Apps (SWA)
Which API approach are you accessing DAB through?
GraphQL
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

hi @alexandruilea,
thanks for reporting the issue. @thomasgauvin, is this something that can be fixed on the SWA portal side?

An error message that would suggest the error better would be
highly appreciated as error 400 in this case is pretty useless. Returning
error 500 would already be an improvement over 400 as I was looking at what
could be wrong with my request while the error came from the configuration
of the backend.
On Sat, Jul 8, 2023, 00:36 Aniruddh Munde @.***> wrote:

hi @alexandruilea https://github.com/alexandruilea,
thanks for reporting the issue. @thomasgauvin
https://github.com/thomasgauvin, is this something that can be fixed on
the SWA portal side?
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/data-api-builder/issues/1521#issuecomment-1626177639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQP4Y2NDMDEBCHVDDCYM3LLXPB6N7ANCNFSM6AAAAAAYP3KEBI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Aniruddh25, thanks for tagging, I'll coordinate with Portal to get this fixed to escape characters in the connection string


A1111 1.6.0 - Rect mode not working

Once again, Rect mode is broken

Hello and thanks for maintaining latent couple!
However it seems that rect mode is once again buggy, it works however after doing the same workaround of creating any image on mask mode and clicking "I've finished my sketch"

Here be the error

@MaxLanc  Yeah... I forgot to remove check when mode is Rect :notlikethis:
I'll fix this within 2 hours, thanks for the report!

@MaxLanc The fix should fix the issue, please check it!

@aria1th
Tested and can confirm it werks, closing issue!
Thanks a lot!


docs: links to @fuels-ui/css and @fuels/react are broken

The links in example/app to the design system are broken (404).  It also may be a good idea to put these links in the root readme as well.

Closed since we don't have more examples inside the repository


Unable to select CheckBox in ValueHelp Dialog after updated UI5 lib to 1.120.3

Describe the bug
We are trying to click a check box on a ValueHelp dialog

We copy the press from the test recorder as it is :
iPressTheControl: async () => {
    await browser.asControl({
    	selector: {
            id: "grantAccess--userId-mInput-valueHelpDialog-table",
            searchOpenDialogs: true,
            interaction: {
                    idSuffix: "rowsel0"
            }
    }}).press();
}

and we add it in Javascript accordingly
static async iPressTheFirstRowCheckBox(){
  await browser.asControl({
    forceSelect:true,
    selector: {
          id: "grantAccess--userId-mInput-valueHelpDialog-table",
          searchOpenDialogs: true,
          interaction: {
                  idSuffix: "rowsel0"
          }
  }}).press();
}

To Reproduce
Steps to reproduce the behavior:

Run the test
Click on checkBox of the row in the value help

Expected behavior
The checkBox is not actually clicked but there is no thrown exception.
The test execution fails on the next step

Related to https://github.com/ui5-community/wdi5/issues/564 ?

It could be, but i couldn`t find any solution of the other issue as well


Use systemlinkadmin.0.13.4 & systemlink.0.13.32

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
The 2023-05 release uses the systemlink.0.13.32 and systemlinkadmin.0.13.4 helm charts. The previous 2023-05 SBOM and notice files, incorrectly used the systemlink.0.12.75 and systemlinkadmin.0.12.9 helm charts (which correspond to the 2023-04 release).
This PR updates the notice files and the SBOMs to the correct ones.

Partial outage with GitHub Actions is blocking this PR - https://www.githubstatus.com/incidents/pr3498h3qkfy


ci: Run rustfmt on x86_64-linux only

Also removing obsolete PATH="$HOME/rust/bin:$PATH".

I would say this doesn't not say noticeable build time. The rustfmt
on Linux builds are same program (same executable, same architecture).
So I personally don't want to run it multiple time. I agree with you on
keeping rustfmt running on Windows.

Okay, so perhaps one rustfmt on linux, one on macos, and one on windows?


Configuration api updates

updates to the configuration so that it can put configuration options of enable search and browse in the configuration.

👍

Why was this needed?

👍


Probs with connection between Mqtt and NodeRed

I have my problems with the share of mqtt. Whenever I have everything running, with 5 lighting units, they lose their connection to the mqtt brocker after a certain amount of time (about 10 minutes). But otherwise I find this software code already fantastic.

Does each McLighting have unique hostname?

Jeep all ESP8266 have their own name and via the IP address are still all reachable. Only via my NodeRed site they are not reachable after some time. If I leave one of the ESP's on the serial port of the Adruino IDE I see that the MQTT connection disappears and the ESP after the program lines from the "definitions.h" here: #ifdef ENABLE_MQTT
#define MQTT_MAX_PACKET_SIZE 512
#define MQTT_MAX_RECONNECT_TRIES 4
4 attempts to reestablish the connection, and if that fails to take an unclear state.

I'm suspecting the MQTT server is having a hard time keeping up, does this issue happen if only one McLighting is connected. Do you have #define ENABLE_AMQTT enabled? Or are you using #define ENABLE_MQTT?

I use ENABLE_MQTT and now have about 7 weather ESP sensors that send via MQTT and now the Garden Lights ESP's which are to be controlled via Mqtt. And I still use the old code for the mc lighting.

// Neopixel
#define PIN 2 // PIN (2 / D4) where neopixel / WS2811 strip is attached
#define NUMLEDS 8 // Number of leds in the strip
// #define BUILTIN_LED 2 // ESP-12F has the built in LED on GPIO2, see https://github.com/esp8266/Arduino/issues/2192
// #define BUTTON 4 // Input pin (4 / D2) for switching the LED strip on / off, connect this PIN to ground to trigger button.
const char HOSTNAME [] = "Garden Light5"; // Friedly hostname
// #define HTTP_OTA // If defined, enable Added ESP8266HTTPUpdateServer
// # define ENABLE_OTA // If defined, enable Arduino OTA code.
#define ENABLE_MQTT // If defined, enable MQTT client code, see: https://github.com/toblum/McLighting/wiki/MQTT-API
// # define ENABLE_HOMEASSISTANT // If defined, enable Homeassistant integration, ENABLE_MQTT must be active
// #define ENABLE_BUTTON // If defined, enable button handl

Cant help you there, could you update to the latest code? The old code had an error that was introduced by me (i think) where the mqtt_clientid was manually set.


Winston: Update Backend to serve agreed upon structure

[ ] Update DB and API with the id, name, tagline, location, image, and price fields for listings (see #20):
{
id: 2,
name: "property-2",
tagline: "this is property 2",
location: "Houston, TX",
image:
"https://a0.muscache.com/im/pictures/2f53e928-a91e-464c-b69f-14eff28a3b85.jpg?im_w=960",
price: 263,
}

@unnamedsunshine are you good with the above requirements for the listings fields?

Yes. looks good!


x/vgo: wrong error when x/go.mod tag v2.0.0 does not say module "x/v2"

I am developing a program which uses russross/blackfriday and of course it does not support vgo yet, so I tried fork and add go.mod myself for fun.
Because blackfriday reaches v2.0.0 and developed in v2 branch, I had to checkout to v2 branch.
I added go.mod file there but I didn't put version to it's module path by mistake.
So the file started with module "github.com/kybin/blackfriday" not module "github.com/kybin/blackfriday/v2".
Then I tagged it as v2.0.1.
When I ran vgo get -u from my repository, it panicked with vgo: github.com/kybin/blackfriday/v2 v2.0.1: missing go.mod.
It confuses me because I know the file exists.
I checked vgo source and found modfetch.codeRepo.findDir returns wrong error when the module path does not have version info.
It would be better the error message is something like go.mod's module path is not matched with it's tag version.
Thank you.

I agree, the error should be fixed.

Thanks for the great explanation. I sent https://golang.org/cl/122399 to clear up this situation.


[Doc Improvement][Teams 2.1 update][3448498]

Added images from sbs-messagingextension-searchcommand

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/toolkit-v4/sbs-v4/sbs-gs-bot-v4.yml
88
49
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/toolkit-v4/sbs-v4/sbs-gs-bot-v4.yml
88
49
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/toolkit-v4/sbs-v4/sbs-gs-bot-v4.yml
88
49
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Request: Show more options to transform the text, when opening the context menu

Show more options to transform the text, when opening the context menu. If you copied a text you should not only be able to "use it as prompt", "explain meaning" and "generate an image": Summarize, translate and correct would be very useful too. It would also be useful to just give a personal instruction what to do with this text. The guys from Sider AI have already created a similar version for chrome (to help you with the prompts).

Due to a long inactivity and a lot of new features this issue will be closed.


Update sbt-avrohugger to 2.5.1

About this PR
📦 Updates com.julianpeeters:sbt-avrohugger from 2.0.0-RC22 to 2.5.1
📜 GitHub Release Notes
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.julianpeeters", artifactId = "sbt-avrohugger" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.julianpeeters", artifactId = "sbt-avrohugger" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-minor, version-scheme:semver-spec, commit-count:1

Superseded by #126.

Superseded by #130.


[swift2objc] Support Parsing Generic Types and Type Constraints

Generic Types in Swift are similar to generics in Dart, and are used for similar purposes in structs, classes, typealiases and functions.
Type Constraints are a Swift feature where the type in a generic function or class can be restricted based on whether it implements either a given protocol or a given class:
func myFunc<T: Equatable>(param: T) {
  // function body
}

Dart has a similar feature where a generic is constrained to extend a given class.
This pull request adds support for not only parsing type generics but also for parsing type constraints by

[x] Adding support for parsing type generics in Swift
[ ] Being able to easily access generics used in compound declarations from the methods, properties and nested declarations in it
[x] Including fields for the constraints applied on the given generic type
[x] Adding support for parsing such constraints from functions
[x] Adding support for parsing such constraints from compound declarations
[ ] Adding support for getting type constraints used in Swift Extensions

More information can be found here: #1814

At the moment, Type Constraint tests do not succeed completely because support for protocols have not been implemented, so either of #1828 or #1827 have to be implemented to make successful tests for these


nodejs8 function is not deploying with private repo reference

Is this a BUG REPORT or FEATURE REQUEST?:
BUG REPORT
What happened:
I was trying to deploy a nodejs8 http function with a dependency file and few environment variables. One among the environment variable is setting the private npm repo url. When I ran the deploy command, it started deploying by accepting all the flags. But during the Install container phase, it goes under crashloop with the following error.
npm ERR! path /.npmrc
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall open
npm ERR! Error: EACCES: permission denied, open '/.npmrc'
npm ERR!  { Error: EACCES: permission denied, open '/.npmrc'
npm ERR!   stack: 'Error: EACCES: permission denied, open \'/.npmrc\'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'open',
npm ERR!   path: '/.npmrc' }
npm ERR!
npm ERR! Please try running this command again as root/Administrator.

┌──────────────────────────────────────────────┐
│           npm update check failed            │
│     Try running with sudo or get access      │
│     to the local update config store via     │
│ sudo chown -R $USER:$(id -gn $USER) /.config │
└──────────────────────────────────────────────┘

My deploy command looks like the below,
kubeless function deploy elastic-writer --runtime nodejs8 \
					--handler elastic-writer.writer \
					--from-file elastic-writer.js \
					--dependencies package.json \
					--env http_proxy:<http-proxy>  \
					--env https_proxy:<https-proxy> \
					--env HTTPS_PROXY:<https-proxy> \
					--env HTTP_PROXY:<http-proxy> \
					--env no_proxy:<no-proxies> \
					--env NPM_REGISTRY=<internal-private-repo> \
					--env NPM_SCOPE=@internal

What you expected to happen:
To set the private repository in the .npmrc and run seamlessly.
How to reproduce it (as minimally and precisely as possible):
Deploy a nodejs function with a private repository configuration passed as the environmental variable.
Anything else we need to know?:
I guess, the install container doesn't have enough permissions. Attached the contents of the kubectl describe po elastic-writer-2651850065-qpbdf command below.
events.txt
Environment:
oc v1.5.1+7b451fc
kubernetes v1.5.2+43a9be4
features: Basic-Auth

Server <Internal-Cluster>
openshift v3.7.9
kubernetes v1.7.6

Kubernetes version (use kubectl version): 1.7
Kubeless version (use kubeless version): v0.5.0
Cloud provider or physical cluster: OpenShift v3.7

Hi @gkarthiks,
In Openshift, the pods are forced to be run as a user different than root. We have changed (https://github.com/kubeless/kubeless/pull/681) the way our containers are running to avoid issues like that but the change has not been released yet. To check if that solves your problem can you edit the deployment to change the securityContext and use the UID 1000?
kubectl edit deployment elastic-writer

The change should be:
-      securityContext: {}
+      securityContext:
+        fsGroup: 1000
+        runAsUser: 1000

Hi @andresmgot , I patched the deployment and the rolling update happened for the Deployment. But still same error in the same insall container.

Hi @gkarthiks
Unfortunately I am not able to reproduce your issue: when I set the private registry and scope and I use a securityContext I am able to install some basic dependencies. Could be the case that some npm module tries to write to /.npmrc without taking into account the running user?
Regarding npmrc documentation is should be using a path within the HOME of the running user (that should be /home/node/.npmrc) or the path to the project (/kubeless/.npmrc). In both of those paths the user 1000 have write access.
Could you try to isolate the module that causes the issue? Does it happen with any module?
Also, to verify that the user running npm is the correct one: Can you edit the Deployment again and, in the command that install npm dependencies, can you prepend npm config ls -l? That should be printing some information about npm before installing dependencies. The args field that you need to modify should look like:
      - args:
        - npm config ls -l && npm config set registry https://registry.npmjs.org && npm install --production
          --prefix=/kubeless

@gkarthiks we are on the kubernetes slack #kubeless channel. It would be nice to coordinate a live debugging session, I think it will go faster than github :)

@sebgoa  Unfortunately slack is not accessible from our office network. 😞

just email andres at bitnami dot com to set a meeting to sort this out.

Now I am running v0.6.0 and I am getting securityContext issue as given below. The Deployments are created but no pods are created. In the events, I am getting this error:
Error creating: pods "get-python-3153575416-" is forbidden: unable to validate against any security context constraint: [fsGroup: Invalid value: []int64{1000}: 1000 is not an allowed group securityContext.runAsUser: Invalid value: 1000: UID on container prepare does not match required range. Found 1000, required min:<PHONE_NUMBER> max:<PHONE_NUMBER> securityContext.runAsUser: Invalid value: 1000: UID on container get-python does not match required range. Found 1000, required min:<PHONE_NUMBER> max:<PHONE_NUMBER>]

@sebgoa yes, will look into this ASAP

just a note to let you know that I was able to reproduce the issue using minishift. First it seems that the user 1000 is not valid for runAsUser so to use the default you can execute:
kubectl patch configmap -n kubeless kubeless-config \
 -p '{"data":{"deployment":"{\"spec\":{\"template\":{\"spec\":{\"securityContext\":{}}}}}"}}'
kubectl delete pod -n kubeless -l kubeless=controller

that will override the changes in 0.6.0 and will use the default securityContext that Openshift provides. The problem now is that the user<PHONE_NUMBER> which is the one used by default doesn't have a HOME defined with permissions to write on it so it tries to use /.  I was able to install dependencies
setting the env var HOME to /tmp in the install container:
      - args:
        - echo '1bccd6e12e112c6482a6fb8ba2fd996c3d839ad547535f611b35d4c9cd0385cd  /kubeless/package.json'
          > /tmp/deps.sha256 && sha256sum -c /tmp/deps.sha256 && npm config set registry https://registry.npmjs.org && npm
          install --production --prefix=/kubeless
        command:
        - sh
        - -c
        env:
        - name: HOME
          value: /tmp

The problem is that this also affects to NodeJS but also Golang and PHP runtimes which also present errors. I will let you know when a patch is ready and @mhausenblas if you have any suggestions it will be highly appreciated.

@andresmgot
But why would you add /tmp as HOME, because when I run the npm config ls command; it shows the userconfig=/.npmrc. and globalconfig = "/usr/local/etc/npmrc"
That is because as per npm documentation here, the userconfig takes $HOME/.npmrc.
And if you do echo $HOME inside the container, it gives / as the value. So per npm doc, this is the reason userconfig sets to /.npmrc
So I've added /kubeless as mu HOME and not in the YAML just in the kubeless function deploy command as I've updated in slack, this is what I did.

kubeless function deploy <function_name>
--runtime nodejs8
--handler fileName.func
--from-file fromFileName
--dependencies package.json
--env NPM_REGISTRY=
--env NPM_SCOPE=@<npm_redirector>
--env HOME=/kubeless

The above writes the **private repo** perfectly into the `/kubeless/.npmrc` file and pulls the package perfectly.
This adds the dependency HOME=/kubeless in init containers and actual container as well.
So that solved the problem.
Also one more question.. Does kubeless has hardcoded the UID for the kafka/ZK ?

True, that's a valid option too. I am glad you found it.

Also one more question.. Does kubeless has hardcoded the UID for the kafka/ZK ?

It is not, you shouldn't have any problem with the kafka-zookeeper-openshift.yaml manifest that we provide.

What happened?
I am trying using a few modules from the private repo in my function. I don't know how to push my .npmrc file when I deploy the function. I tried the way shown above by setting env variable like this.(Deployment command)
*function deploy vincent --runtime nodejs6 --from-file vincentKubelessCronJob.js --handler vincentKubelessCronJob.postRequest --dependencies package.json  --namespace kubeless --schedule " /1 * * "
--env NPM_REGISTRY=
--env _auth=
--env email=
--env always-auth=
What is expected?
The function should pull the package from the private repo.


Ubuntu Launcher treats Mapbox Studio as an Atom window

I'm guessing this is because Mapbox Studio is also running on atom-shell, but if Atom is installed and you launch Mapbox Studio, it opens as a new Atom window instead of as a separate application.
Steps to reproduce:

Install Atom
Launch Mapbox Studio by double-clicking the atom executable in the Linux download
Mapbox Studio will launch as an Atom window.

Mapbox Studio 0.25.0
Ubuntu 14.04

Possibly related to https://github.com/atom/atom/pull/6438#issuecomment-94522788 (Mapbox Studio's executable is called atom right now)


Initial proposition for problems refactor.

This is just an initial proposition of how we could handle various problems.
I moved the logic that @alexjuda created for the graph partition problem to a separate file. I'm not sure whether this is the right way of handling it though, open to suggestions.
Another way of handling it is to put all the problems derived from qiskit in one file, treat it as private and expose a public interface in __init__.py?
Possible further improvements:

Refactor get_random_maxcut_hamiltonians to get_random_hamiltonian_for_problem
Refactor get_solution_cut_size to get_solution_cost_for_problem
Refactor solve_maxcut_by_exhaustive_search to solve_problem_by_exhaustive_search
Obviously we need tests ;)

Potential issues:

We might want to have 2 different implementations of the same problem – e.g. form MaxCut we have one from qiskit and our own. For those simple problems, it's not that big of a deal, but for problems like TSP or Vehicle Routing it matters a lot.
It looks like qiskit.aqua is getting depreciated so perhaps it's worth figuring out a different way to handle it.

@alexjuda @simonwa7
I have addressed some of the issues we talked about.
It seems like qiskit.aqua depreciation is not a problem yet – according to the migration guide, we'll just need to change imports once it's done.


Add vep_assembly as input to cle_somatic_exome

With #572 merged, vep_assembly is now required input. Add it to cle_somatic_exome.cwl and gathered_cle_somatic_exome.cwl

+1

Merged this to fix the broken master.


Deprecate Exec.getExecResult which is replaced by getExecutionResult

https://github.com/gradle/gradle/issues/15681

@bot-gradle test and merge

Your PR is queued because #16734 is being built.

OK, I've already triggered a build for you.

OK, I've already triggered a build for you.

Error while scheduling pre-tested commit, please contact the administrator.

Well the conflict needs to be resolved...


Add a help tab

Add a tab to the settings off to the right called "Help"
Fill it with basic text about the plugin. What the plugin does, and that there is the auth0 settings dashboard too.
Make sure to link to the Auth0 help channels like chat, the website, blog and so on.
Do we have a technical writer to draft this page?

We should include a feedback section in the help tab too, pretty prominent.
I'm not sure what system(s) Auth0 already uses, but some very simple way for the user to submit feedback like "What one thing would you change about this plugin?" or something like that.

@mgonto who can work on the copy for this page? also, we can add the uservoice widget right? (I think we dont use deligted anymore)

We can add a widget. For copy, if @SteveC can help with it, perfect, otherwise ping Rich about it!

@glena :point_up:

@SteveC can you?

Fill it with:

basic text about the plugin
What the plugin does
there is the auth0 dashboard (manage), what you can do with it and that the rest of the configurations should be done there
add support channels, website and blog
add uservoice widget

text:
Thank you for installing the Auth0 WordPress Plugin[link to plugin page].
This plugin allows you to connect your WP instance to many login solutions. If you have many users commenting or buying products from you, then our social connectors will help. They will let users log in using things like Twitter, Google or Facebook. We also support many enterprise login systems like Active Directory.
Auth0 is a powerful solution and besides the options you see here on your WordPress instance, there are many more things you can do on your Auth0 dashboard[link to dashboard]. The dashboard allows you to enable more authentication providers and activate advanced features like running javascript snippets on a login event to do things like record activity or send an email. But don't worry, if you just want to enable social logins for your site, you can safely stay within the options here.
If you're having any issues - please contact us. We have a variety of channels to help you:

Our chat room[link to chat room] is great for online support
We have a lot of documentation at our help website[link to docs]
For more information on Auth0, see our blog[link to blog]

[uservoice widget]

Let's see how that looks (screenshot please german?) and iterate. In particular, I have no idea what the UserVoice widget looks like, so it may be better to integrate it with the text differently.

Sure

Checking on the feedback widget


Patch CVE 2023 45853 for rust

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Fix CVE-2023-45853 for rust
Change Log

Fix CVE-2023-45853

Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2023-45853

Test Methodology

Pipeline build id: 441173

Regarding Github PR checks: Everything is passing except for ptest of clamav, mozjs, rpm-ostree, rust which are known failures.


[stable/nginx-ingress] Prometheus metrics, why is the statsExporter sidecar needed?

Is this a request for help?: Yes
We are working on moving the deployment of our nginx ingress controller to use the stable Helm chart, but I have some doubts about the stats/metrics.
AFAIK (and according to docs) the controller already exposes the vts metrics on :10254/metrics if the enable-vts-status: "true" config option is provided in the ConfigMap and in fact we have been using it thus far to scrape the metrics with Prometheus.
However, the Chart is deploying an additional sidecar (stats-exporter) that queries the vts status page and exposes the metrics in Prometheus format using the nginx-vts-exporter.
Why is the above required? why not simply scrape the metrics that are exposed by the controller itself?

This would really clean things up, now that 0.9.0 has embedded metrics. +1.

When the chart was added, there was no built in support for prometheus. it is supoprted now, so I agree the sidecar should be dropped. I don't think anyone is running 0.8 anymore? It was fairly buggy. :(
also, though, the :10254/metrics endpoint is not available to prometheus if it isn't listed in the pods ports list, so it is currently inaccessable without patching. Can that be added please?

Seems the sidecar is away in current version (helm chart 0.14.1)?
At least its not started anymore.
Unfortunately with the included endpoint my dashboard (https://grafana.com/dashboards/2949) does not work anymore.

The sidecar is no longer necessary and was removed. The ingres controller itself exports Prometheus metrics.
@monotek You'll have to update the dashboard.

I know. This one works: https://grafana.com/dashboards/3050

Sorry for reviving old issue, @monotek Dashboard 3050 is not working for me.
Because nginx ingress is exposing only those metrics for me, I'm not seeing any nginx_* metrics. Am I doing anything wrong?

You have to enable the metrics in nginx helm chart: https://github.com/helm/charts/blob/master/stable/nginx-ingress/values.yaml#L238-L263
But i guess the 3050 chart will not work anyway as the metric names changed...

I've enabled metrics on helm yaml here is the part:
metrics:
    enabled: true
    service:
      annotations: {}
      prometheus.io/scrape: "true"
      prometheus.io/port: "10254"

and if I access my nginx ingress /metrics endpoint it returns prometheus metrics, thus resulting Prometheus reports nginx is UP and I'm seeing those exposed metrics on nginx job.
But those exposed metrics are not familiar to me, can you confirm that those metrics are really the right ones?

Ah I see, I've just enabled
  stats:
      enabled: true

and upgraded helm chart.
But prometheus didn't get any new metrics from nginx..
This is my prometheus.yaml job for nginx:
- job_name: nginx-ingress
  metrics_path: /metrics
  static_configs:
    - targets:
      - nginx-ingress-controller:80

If I go to my nginx ingress IP address' /metrics endpoint I can check prometheus exposed metrics.

@shinebayar-g probably this too late but I believe you are not configuring the annotations properly. So instead of:
service:
  annotations: {}
   prometheus.io/scrape: "true"
   prometheus.io/port: "10254"

Use:
service:
  annotations:
    prometheus.io/scrape: "true"
    prometheus.io/port: "10254"

I've got that, thank you :+1:


Unable to retrieve or update shard-sync _local documents

Using CouchDB 2.3.0:
According to the docs here - https://docs.couchdb.org/en/stable/api/local.html
It states that we can use the key/id to retrieve and update _local documents.
When I call for /{db}/_local_docs I get two sets of documents returned, one is view related (key contains "purge-mrview") the other type of document contains "shard-sync"
The first type of document containing "purge-mrview" can be retrieved via their id value, but the second type always return 404.
Is there any way to obtain and modify these local documents? (the ones with "shard-sync" in the key)
(I am interested in modifying these documents to work around the problem documented here - https://github.com/apache/couchdb/issues/1987

Closing this cos it turned out that some of the docs do exist and are editable, I just had a load of missing ones in my DB

shard-sync documents are used by the internal replicator to sync shard content between shard copies they are not written at the cluster level so don't follow the cluster document hashing rules where the document id is hashed and sent to a particular shard range based on that. That explains why when you make request you might see different subsets of them depending on which shard copies respond to your requests.

@nickva the trick is that I think if someone re-names or replaces a node during a shard split/sync, there needs to at least be some sort of doc that explains how to clean-up after the fact.


Add django-google-optimize

Used the 'Testing' section, got a proposal for a different one or is it fine?

@adinhodovic thank you!


fix(sso.py): import ThumbprintMismatchException from pyVmomi.Security

Signed-off-by: Anthony Brown<EMAIL_ADDRESS>

Hi Antony
Thanks for the fix.
The SDK is not yet ready with pyVmomi <IP_ADDRESS>. There are more incompatibility issues. Currently, We are working on pyVmomi patch till then SDK will be compatible only with pyVmomi  version upto 7.0.3
Regards
Kunal


Snakemake: secondary predictions

add rule to perform secondary predictions (PSIPRED, DIOPRED, MEMSAT-SVM) for each gene

obsolete with DMPfold


Add support for tr

This is for #288

This looks good. Do you want to make the changes I listed? After that I can merge it.

I was going back and forth on that one myself. I updated everything to sue row instead or tr.


Mobs spawn

Introduction
With our new Mapgen, we need to tweak the spawn nodes of many monsters/passive animals.
We also need to change the spawn rate of many monsters/passive animals always because of the new Mapgen.
Spawning node - Animals

[x] Bee => group:flowers
[x] Bunny => dirt_with_grass
[x] Chicken => dirt_with_grass
[x] Cow => dirt_with_grass
[x] Goat => dirt_with_grass
[x] Kitten => dirt_with_grass
[x] NPC => dirt_with_grass + dirt + junglegrass + sand (beach)
[x] Rat => stone + sandstone
[x] Sheep => dirt_with_grass
[x] Warthog => dirt + junglegrass

Spawning Rate - Animals

[x] Bee => 5 000
[x] Bunny => 10 000
[x] Chicken => 10 000
[x] Cow => 10 000
[x] Goat => 10 000
[x] Kitten => 10 000
[x] NPC => 50 000
[x] Rat => 10 000
[x] Sheep => 10 000
[x] Warthog => 10 000

Spawning node - Monsters

[x] Creeper => dirt_with_grass
[x] Dirt Monster => dirt_with_grass + drygrass (new biome)
[x] Dungeon Master => stone + sandstone + netherrack (nether)
[x] Green Slime => acid_source + acid_flowing
[x] Lava Slime => lava_source + lava_flowing
[x] Mese Monster => stone + sandstone
[x] Minotaur => drygrass (new biome)
[x] Oerkki => stone + sandstone
[x] Sand Monster => desert_sand + sand (beach)
[x] Spider => jungleleaves + jungletree
[x] Stone Monster => stone + sandstone
[x] Tree Monster => leaves + beech_leaves
[x] Wolf => dirt_with_grass
[x] Yeti => dirt_with_snow + snow + snowblock
[x] Zombie => dirt_top (nether)

Spawning Rate - Monsters

[x] Creeper => 20 000
[x] Dirt Monster => 10 000
[x] Dungeon Master => 7 000
[x] Green Slime => 2 000 (1/4000 + 1/4000 = 1/2000)
[x] Lava Slime => 2 000 (1/4000 + 1/4000 = 1/2000)
[x] Mese Monster => 7 000
[x] Minotaur => 25 000
[x] Oerkki => 7 000
[x] Sand Monster => 20 000
[x] Spider => 7 000
[x] Stone Monster => 7 000
[x] Tree Monster => 8 000
[x] Wolf => 10 000
[x] Yeti => 30 000
[x] Zombie => 6600

Other information(s)
The lesser spawn number is, the more monsters/animals will spawn.
If you have suggestions, do not hesitate ! :)

I thinck it's ok like this, need to update the spreadsheet now ! :)

Spreadsheet updated https://github.com/Ombridride/minetest-minetestforfun-server/commit/885cfdc524ec4118cf750f8f13af17bf110f7f01


[SPARK-42166][K8S] Make docker-image-tool.sh usage message up-to-date

What changes were proposed in this pull request?
This PR aims to make docker-image-tool.sh usage message up-to-date for Apache Spark 3.4.0.
Why are the changes needed?

Simplify multi-arch publishing example by removing java tag in order to keep one concept. In addition, Java 17 is better than Java 11 in terms of multi-arch support.
Use v3.4.0 instead of all EOL versions (v2.3.0, v3.0.0)
Use the smallest multi-arch image tag example, 11-jre, instead of 11-jre-focal.

$ docker images | grep eclipse-temurin | sort -k7
eclipse-temurin    11-jre           87a0df44b0d1   6 weeks ago     239MB
eclipse-temurin    11-jre-focal     0cb88201ea2b   6 weeks ago     245MB
eclipse-temurin    17-jre           096f7591e8eb   6 weeks ago     260MB
eclipse-temurin    11               a20b0d225174   6 weeks ago     435MB
eclipse-temurin    17               2ed42194c6b0   6 weeks ago     450MB

Does this PR introduce any user-facing change?
Yes, but this is a usage message.
How was this patch tested?
Manually review.

Could you review this, @Yikun ?

Thank you, @HyukjinKwon !

Merged to master for Apache Spark 3.4.0.

late LGTM.

Use the smallest multi-arch image tag example, 11-jre, instead of 11-jre-focal.

FYI, According to https://github.com/docker-library/official-images/blob/master/library/eclipse-temurin#L84-L85 .

11-jre == 11-jre-jammy (Ubuntu 22.04)
11-jre-facal (Ubuntu 20.04)

I think it's fine.


create a fetchpypi function thats using blake2b for the crypto hash (python)

the modern pypi cdn hashes files using blake2b, which is also part of their url
so it would be very practical to reuse those hashes for file checking and full url building

Unfortunately those hashes are checked by nix itself, and we'd need to add blake2b to nix.

@RonnyPfannschmidt Do you have a link to documentation of how the hashes are computed?

I have some nix expressions for python packages from a custom index, this uses pip with --require-hashes to handle the downloads. I could probably generalise this a bit to create a fetchpypi function that only needs a sha and pypi index.

im closing this one as unsustainable since sha256 are available as well as name based redirects

As far as I know the name-based redirects don't work with wheels though, and I think we're going more in the direction of using wheels.

@FRidh wheels have urls like https://files.pythonhosted.org/packages/py2.py3/m/matlab_kernel/matlab_kernel-0.9.5-py2.py3-none-any.whl

This is what I've been using to download python packages from a custom index https://gist.github.com/LnL7/e87c76d1bf4217dcdc8226bcec1e71c1. Removing --no-binary :all: would make this use wheels by default.

@FRidh that being said, using wheels as source package implies killing the ability to run tests in many ways
but it could be valuable to set up a bootstrap tool-chain using downloaded wheels in the pythonpath

@FRidh that being said, using wheels as source package implies killing the ability to run tests in many ways

Indeed, we would still have to fetch the tests elsewhere then. I'm not a big fan of it, but as it is likely setuptools will be used fewer in the future, developers will sometimes only upload wheels.

This is what I've been using to download python packages from a custom index https://gist.github.com/LnL7/e87c76d1bf4217dcdc8226bcec1e71c1.
Removing --no-binary :all: would make this use wheels by default.

That's a good idea, using pip as fetcher.

I'll create a pr that integrates it.

@LnL7 If we can just use fetchurl that would be even better #22256

That only works for pypi.python.org, I created that expression so I could use our internal pypi server at work.

That I understand, having a generic function is nice to have. For use in Nixpkgs I would prefer we use a lighter-weight fetchurl though.

Sure, perhaps we could swap out the implementation depending on what index is used? I would be nice if I could use some of the existing packages in nixpkgs.

Since fetchpypi and fetchurl are both fixed output drvs, the implementation should not matter for the output.

@LnL7 most pypi servers have some form of simple redirect as far as i recall, perhaps you can get by with a sinmple fetchurl wrapper as well
an pip using function would be still nice as building block for complex setups

Ronny Pfannschmidt<EMAIL_ADDRESS>writes:

im closing this one as unsustainable since sha256 are available as well as name based redirects

Also, name-based redirects I believe are intended as a stop-gap
migration strategy.


HDKeyDerivation.deriveChildKeyWithNoParent is not necessary anymore in DeterministicKeyChainTest

…we can use deriveChildKey directly.

HDKeyDerivation.deriveChildKeyWithNoParent is no longer useful.
This will be closed because other updates have been made to resolve this issue.


[FEATURE] Adjust the speed and Import the GIFs

I have ideas for 2 features:

Adjust the speed for each frame.
or
Import the GIFs (edited from photoshop) in the project. After that, combine the GIFs and images together, and export the GIFs in output file.
Thanks so much !

In case of second, I have found a project can combine GIFs together. I don't know C++ language, so I don't know how to run it. https://github.com/Goldo02/gif_art_engine_for_nft.git.


Create dataset loader for Indo_MultiModal_PMD_ID

NusaCatalogue: https://indonlp.github.io/nusa-catalogue/card.html?id_mm_pmd

Dataset
id_mm_pmd

Description
Introduced in the FLAVA paper, Public Multimodal Dataset (PMD) is a collection of publicly-available image-text pair datasets. PMD contains 70M image-text pairs in total with 68M unique images. The dataset contains pairs from Conceptual Captions, Conceptual Captions 12M, WIT, Localized Narratives, RedCaps, COCO, SBU Captions, Visual Genome and a subset of YFCC100M dataset. Indo_MultiModal_PMD_Indonesia is the Indonesian language version.

License
License refers to the individual datasets that compose PMD_Indonesia

#self-assign


Add Various Comment Methods to Table class

Adds methods for listing comments in a record, updating comments, creating new comments, and deleting existing comments.
Part of a number of Airtable API updates as referenced in #208

Hey @BAPCon
Thanks for submission - this looks great.
Would you be able to add some unit tests to go along with these changes?
Thanks

@gtalarico
Yeah no problem

@gtalarico I'm new to writing tests, does this look correct to you?

Codecov Report

Merging #238 (26cdd60) into main (a97ca26) will decrease coverage by 3.09%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main     #238      +/-   ##
==========================================
- Coverage   87.46%   84.37%   -3.09%
==========================================
  Files          15       14       -1
  Lines         638      672      +34
==========================================
+ Hits          558      567       +9
- Misses         80      105      +25

Impacted Files
Coverage Δ

pyairtable/api/api.py
76.31% <ø> (ø)

pyairtable/api/base.py
78.37% <ø> (ø)

pyairtable/formulas.py
85.71% <ø> (ø)

pyairtable/api/abstract.py
91.71% <29.41%> (-8.29%)
:arrow_down:

pyairtable/api/table.py
80.70% <30.76%> (-19.30%)
:arrow_down:

pyairtable/orm/fields.py
75.78% <70.00%> (-0.75%)
:arrow_down:

... and 5 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add options openWith and showStore.

I've added an option to show and Open With dialog for when the user might have multiple apps that could handle the mimetype of the file.

This is only for Android, and by no means am I an Android developer.

The alternative would be to create a new method FileViewer.openWith which calls the native open method with the openWith param set. Happy to change.

Nice idea @rickysullivan!
I have just changed some naming and moved everything into a single options param.


Unable to Join GameRoom after migrating to 0.15.3 Version  Unity

Hi Team,
After migrating Colyseus to 0.15.3 Version in Unity, I am not able to join Game room.
Our Colyseus server(0.15.0) is running on multiple instances with a unique publicAddress.
Stack Trace:
JoinGameRoom Error - Unable to connect to the remote server  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter1[TResult].GetResult () [0x00000] in <9577ac7a62ef43179789031239ba8798>:0    at Colyseus.ColyseusClient+<ConsumeSeatReservation>d__251[T].MoveNext () [0x00200] in /Users/phonato-user/All Data/Bit Bucket/CamelCashCasino/Assets/Colyseus/Runtime/Colyseus/Models/ColyseusClient.cs:369
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter1[TResult].GetResult () [0x00000] in <9577ac7a62ef43179789031239ba8798>:0    at Colyseus.ColyseusClient+<CreateMatchMakeRequest>d__261[T].MoveNext () [0x001c7] in /Users/phonato-user/All Data/Bit Bucket/CamelCashCasino/Assets/Colyseus/Runtime/Colyseus/Models/ColyseusClient.cs:416
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter1[TResult].GetResult () [0x00000] in <9577ac7a62ef43179789031239ba8798>:0    at Colyseus.ColyseusClient+<JoinOrCreate>d__131[T].MoveNext () [0x00037] in /Users/phonato-user/All Data/Bit Bucket/CamelCashCasino/Assets/Colyseus/Runtime/Colyseus/Models/ColyseusClient.cs:113
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <9577ac7a62ef43179789031239ba8798>:0
at System.Runtime.CompilerServices.TaskAwaiter1[TResult].GetResult () [0x00000] in <9577ac7a62ef43179789031239ba8798>:0    at SwRoomController+<JoinGameRoom>d__13.MoveNext () [0x00045] in /Users/phonato-user/All Data/Bit Bucket/CamelCashCasino/Assets/Scripts/SlotsWar/Controllers/SwRoomController.cs:38  UnityEngine.Debug:Log (object) SwRoomController/<JoinGameRoom>d__13:MoveNext () (at Assets/Scripts/SlotsWar/Controllers/SwRoomController.cs:42) System.Runtime.CompilerServices.AsyncTaskMethodBuilder1<Colyseus.ColyseusRoom1<SlotsWarGameState>>:SetException (System.Exception) Colyseus.ColyseusClient/<JoinOrCreate>d__131:MoveNext () (at Assets/Colyseus/Runtime/Colyseus/Models/ColyseusClient.cs:113)
System.Runtime.CompilerServices.AsyncTaskMethodBuilder1<Colyseus.ColyseusRoom1>:SetException (System.Exception)
Colyseus.ColyseusClient/d__261<SlotsWarGameState>:MoveNext () (at Assets/Colyseus/Runtime/Colyseus/Models/ColyseusClient.cs:416) System.Runtime.CompilerServices.AsyncTaskMethodBuilder1<Colyseus.ColyseusRoom1<SlotsWarGameState>>:SetException (System.Exception) Colyseus.ColyseusClient/<ConsumeSeatReservation>d__251:MoveNext () (at Assets/Colyseus/Runtime/Colyseus/Models/ColyseusClient.cs:369)
System.Threading.Tasks.TaskCompletionSource1<Colyseus.ColyseusRoom1>:SetException (System.Exception)
Colyseus.ColyseusClient/<>c__DisplayClass25_01<SlotsWarGameState>:<ConsumeSeatReservation>g__OnError|1 (int,string) (at Assets/Colyseus/Runtime/Colyseus/Models/ColyseusClient.cs:351) Colyseus.ColyseusRoom1/<>c__DisplayClass31_0:b__1 (string) (at Assets/Colyseus/Runtime/Colyseus/Room/ColyseusRoom.cs:232)
NativeWebSocket.WebSocket/d__24:MoveNext () (at Assets/Colyseus/Runtime/WebSocket/WebSocket.cs:403)
UnityEngine.UnitySynchronizationContext:ExecuteTasks () (at /Users/bokken/buildslave/unity/build/Runtime/Export/Scripting/UnitySynchronizationContext.cs:107)

Hi @VkDevPh, I've spent a few hours and couldn't reproduce this issue.
I see you also reported an error "error CS0173: Type of conditional expression cannot be determined because there is no implicit conversion between 'method group' and '<null>'" (screenshot), which I also failed to reproduce.
Can you provide a reproduction scenario for these issues?

Hi @endel ,
Thanks for your time.
For the issue I reported error CS0173: Type of conditional expression cannot be determined because there is no implicit conversion between 'method group' and null on Discord, Please check the video I've sent to your e-mail (endel@colyseus.io).
https://drive.google.com/file/d/1fzbvYBC6_P7KfTqBJkYrMKgfZ855U3hF/view?usp=sharing.
Simply create a new project In Unity (2020.2.2f1) and import the Colyseus package (0.15.3). The error appears in the console.
Also for the above issue, reported here, I have shared server details over email.
Please check and try to connect to the given room, I think an error should come. (Unable to connect to remote server ..)
Let me know if you need any other details.
Thanks

Thanks a lot @endel. :)


Test and potential fix for issue #722

One tiny lint error, can you fix it real quick?
/react-redux-form/test/model-actions-spec.js
error  Missing trailing comma  comma-dangle


Where is state stored?

Where on the hard drive is the state stored? It seems to persist even if A1111 is fully deleted and re-downloaded.

IndexedDB, in the browser.  If you're using chrome you can see it in Developer Tools  in the Application tab.


Default value for enqueue_after_transaction_commit

I'm wondering about the current default value of SolidQueue.enqueue_after_transaction_commit = true (as added in #209). I'd like to suggest that false would be a better default.
1. Risk of a future unexpected behavior change
On a DB-backed queue (SolidQueue, GoodJob, etc), Rails 7.1 and prior already behave as if this is false. (Assuming the queue and app both use the same DB, which I believe is the most common case, by far.)
By defaulting SolidQueue to true, apps switching from Rails 7.1 to Rails 7.2 will suddenly change behavior--without a deprecation warning or other heads up. This seems like a lurking gotcha for developers.
2. Rails docs encourage false
As the present time, the Rails edge docs say, "Active Job backends that use the same database as Active Record as a queue, should generally prevent the deferring, and others should allow it." (ref)
"Prevent the deferring" would be a default of false.
I suspect this encouragement is precisely because it avoids the sudden behavior change described in (1).
--
I believe that most small to medium sized apps will benefit from false all the time, especially as the default across all types of jobs. I'd expect wanting a global setting of true to be limited mostly to larger apps where a separate DB for SolidQueue is used.
However, if a default of true is really wanted long term, that transition should be made explicit. One way to do this would be with a major new release of SolidQueue after Rails 7.2 is released, such that developers would be drawn to look at a "breaking changes" section of the release notes where the behavior change would be documented.
Another approach would be to start writing deprecation warnings for a period of time before switching the default. Again, after the Rails 7.2 release so that apps can set their desired behavior before the default behavior is changed.
I emphasize the need to do any of this after the Rails 7.2 release because there is no setting in Rails 7.1 (apart from deliberately configuring SQ to use a separate DB connection) to accomplish the equivalent behavior to true.

@zarqman, those are great points, especially the second one. I hadn't seen the Rails Edge docs before I chose the default of true.
I agree with you. Let's change that back to false. If you'd like to contribute a PR to that, of course feel free to do so! Otherwise I'll get it done shortly.

Done!

Thanks so much for the quick response and for your work on SolidQueue!

We'll change both the Rails guide and the sq docs to reflect this.

The Rails guide change is in https://github.com/rails/rails/pull/52659. The change in Solid Queue (the default + docs) will be added for v1.0.


WIP: host-disk: remove SELinux categories during migration

What this PR does / why we need it:
Migrating a VMI with a RWX disk requires both the source and target virt-launchers to have access to the disk.
That is not possible when SELinux categories are used inside the PVC.
This PR removes the categories at the beginning of the migration, then sets them to the correct values at the end of the migration.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
WIP

Needs finalizeMigration to be called only once the migration has finished
Needs functional and unit tests

Release note:

Fixed migration issue for VMIs that have RWX disks backed by filesystem storage classes.

/retest

/retest

/retest-required


How to get the output of the previous iteration ,and use them in the current epoch?, thanks

🐛 Bug
Dear author,
How can I  get the output of the previous iteration or epoch ,and the  I  can use them in the current epoch?, thanks
I want to use the output of the previous epoch as one of the features of the input of the current epoch (in the order of the index of the same dataloader).
Best regards.

PyTorch Lightning Version (e.g., 1.5.0):
PyTorch Version (e.g., 1.10):
Python version (e.g., 3.9):
OS (e.g., Linux):
CUDA/cuDNN version:
GPU models and configuration:
How you installed PyTorch (conda, pip, source):
If compiling from source, the output of torch.__config__.show():
Any other relevant information:

Additional context

I'm not sure I'm understanding your question (and I'm not a Lightning maintainer), but have you looked into the many hooks that LightningModule provides, such as on_train_start, on_train_epoch_start, on_train_epoch_end and on_train_end?
For example, you can use on_train_start to create two instance variables, one to store the outputs of the previous epoch, another to store the outputs of the current epoch. In train_step you can read the variable of the previous epoch and update the variable of the current epoch. In on_train_epoch_end you can move the current epoch variable into the previous epoch variable and reset the current epoch variable to empty. In on_train_end you can cleanup both variables to avoid consuming memory unnecessarily.
If this is not enough, maybe you can provide more details about what you are trying to do?


The 'Refresh' event isn't generated on Windows

There is the 'WM_PAINT' window message which Windows sends to an application when its window needs to be redrawn. Logically this message should be processed and the 'winit::WindowEvent::Refresh' event should be emitted as a consequence.

I'm going to try to tackle this.

The actual snippet to handle WM_PAINT is really simple:
winapi::WM_PAINT => {
    use events::WindowEvent::Refresh;
    send_event(window, Refresh);
    0
},

However, this event still isn't ever raised because winit doesn't invalidate the render surface. After talking with WindowsBunny on #rust-gamedev, responding to WM_PAINT in game-like applications is unusual.
In order to get Windows to raise WM_PAINT (and WM_NCPAINT), we would need to probably implement an API to explicitly invalidate the window contents, analogous to Win32 InvalidateRect.

WM_PAINT is not meant to be used by a game-like application, rather by GUI applications that use something like GDI to paint their window.
The fact that WM_PAINT is rarely, if ever, sent is a good thing - the OS tries to reduce the need to repaint an application. Windows uses a lot of tricks to reduce the amount of times WM_PAINT is sent.
On older versions, WM_PAINT was sent each time the window was hidden and then reshown. Nowadays Windows saves the painted window and repaints it later.
I do believe @LPGhatguy implementation of the Refresh event is good, since it works exactly as Windows intended it.
For example, on X11, Refresh is caused by the Expose which is called only when the window is created or the surface of the window has been lost - which, on modern computers, means very rarely.

I think that's how Refresh was designed. In other words, receiving a Refresh event means that you are strongly encouraged to redraw. However you shouldn't draw only when you receive these events.


Doesn't work with Sylius

Hello,
We are currently using Sylius 1.0.4 with your bundle for our project, but are having a few problems. These are not problems directly linked to your bundle, but after implementation into our solution. We tried using the dev-symfony3 branch which fixes problems related to the composer.json, but are left with these :

Annotations: @ParamConverter("article", class="ED\BlogBundle\Interfaces\Model\ArticleInterface", converter="abstract_converter") This is an examplTThis is an example, but every ParamConverter crashes the application.
User (BlogUserInterface): we are using the Sylius User class and are having a problem with the parameter’s type. If we remove the consraint in your bundle it works fine.
The only solution I have found at the moment is to modify your bundle directly, but I’m not satisfied with that solution for many reasons...

Do you have a better solution ?
I didn't know where to post this, but if you could help me I would be very grateful
Have a good day.

Hi davidou66,
Thanks for using EDBlogBundle. Can you explain me what error do you see after ParamConverter crashes the application? Can you paste it here?
Also I am curious to see how does your Sylius User class looks.

Hi,
Error from the ParamConverter:
Controller "ED\BlogBundle\Controller\Frontend\BlogController::singleArticleAction()" requires that you provide a value for the "$article" argument. Either the argument is nullable and no null value has been provided, no default value has been provided or because there is a non optional argument.
User class:
`<?php
namespace AppBundle\Entity;
use ED\BlogBundle\Interfaces\Model\BlogUserInterface;
use ED\BlogBundle\Interfaces\Model\ArticleCommenterInterface;
use FOS\UserBundle\Model\User as BaseUser;
use Sylius\Component\Core\Model\AdminUser as SyliusUser;
use Doctrine\ORM\Mapping as ORM;
use FOS\UserBundle\Model\UserInterface;
/**

@ORM\Entity

@ORM\Table(name="user")

@ORM\Entity(repositoryClass="AppBundle\Repository\UserRepository")
/
class User extends SyliusUser implements BlogUserInterface, ArticleCommenterInterface
{
/*

Required by BlogUserInterface

@ORM\Column(name="blog_display_name", type="string")
*/
protected $blogDisplayName;

public function getBlogDisplayName()
{
return $this->blogDisplayName;
}
public function setBlogDisplayName($blogDisplayName)
{
$this->blogDisplayName = $blogDisplayName;
 return $this;

}
public function getCommenterDisplayName()
{
return $this->blogDisplayName;
}
}`

Sorry for the late response, we found another bundle wich  extends the Sylius e-commerce platform.
By the way, great job on this bundle. It's a shame we couldn't use it.