Update EC number for children of GO:0008373 sialyltransferase activity

[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>
[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>
[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>
[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>
[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>  GO:0008118
[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>
[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>
[ ] EC:<IP_ADDRESS> Transferred entry: EC:<IP_ADDRESS>
[ ] <IP_ADDRESS> Transferred entry: <IP_ADDRESS>, itself transferred EC:<IP_ADDRESS>

EC:<IP_ADDRESS> is xref of GO:0004513
EC:<IP_ADDRESS> is xref of  GO:0008118
both terms have RHEA xrefs
Add EC:<IP_ADDRESS> as broadMatch to both

[ ] EC:<IP_ADDRESS> Transferred to EC:<IP_ADDRESS> and then to EC:<IP_ADDRESS>
EC:<IP_ADDRESS> is GO:0047287 'lactosylceramide alpha-2,6-N-sialyltransferase activity'
and EC:<IP_ADDRESS>  is mapped to GO:0003835 'beta-galactoside alpha-2,6-sialyltransferase activity'
GO:0047287 'lactosylceramide alpha-2,6-N-sialyltransferase activity' has no annotations, I suggest we obsolete with replaced by 'GO:0003835 'beta-galactoside alpha-2,6-sialyltransferase activity'

[ ] parent class "GO:0008373 sialyltransferase activity":  add partial EC:2.4.3.-

Hi @edwong57
I made some documentation for updating EC numbers here: https://wiki.geneontology.org/Updating_EC_mappings
Let me know if the documentation has enough information to update these, or how it could be improved.
Thanks, Pascale

Hi @edwong57 I've run into some issues computing metabolic pathways in BioCyc because of these out-dated sialyltransferase EC xrefs.  Would you be able to fix them in the GO sometime soon?  Many thanks, Steven.

@sjm41 Apologies, I was out of town. I will take care of it this week.

I am also going to obsoleted GO:0004513 'neolactotetraosylceramide alpha-2,3-sialyltransferase activity' since the <IP_ADDRESS> Transferred entry: <IP_ADDRESS>, itself transferred EC:<IP_ADDRESS> and then it became a duplicate of GO:0008118 'N-acetyllactosaminide alpha-2,3-sialyltransferase activity'. They both have no annotations, but GO:0008118 has the accepted name in Expasy.

Thanks @edwong57 !

Hi @edwong57
Just FYI I- the dbxrefs should be removed from the obsoleted terms - see https://wiki.geneontology.org/Obsoleting_an_Existing_Ontology_Term#5._Cross-references (I fixed this one)
Thanks, Pascale

Okay. Thank you!

From: pgaudet @.>
Sent: Tuesday, May 9, 2023 2:19 AM
To: geneontology/go-ontology @.>
Cc: Edith Wong @.>; Mention @.>
Subject: Re: [geneontology/go-ontology] Update EC number for children of GO:0008373 sialyltransferase activity (Issue #25189)
Hi @edwong57https://github.com/edwong57
Just FYI I- the dbxrefs should be removed from the obsoleted terms - see https://wiki.geneontology.org/Obsoleting_an_Existing_Ontology_Term#5._Cross-references (I fixed this one)
Thanks, Pascale
—
Reply to this email directly, view it on GitHubhttps://github.com/geneontology/go-ontology/issues/25189#issuecomment-1539495530, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEAQXF73NDDG5ZGP56N4ODXFIDYNANCNFSM6AAAAAAWE7PBUA.
You are receiving this because you were mentioned.Message ID: @.***>


Action required: Migrate to supported version 6  or New of Google Play Billing Library

Use case
please solve this and we got mail from google console update new billing librey  6 or new before august. please make it a new version otherwise remove it from google play store.

Proposal
please solve this and we got mail from google console update new billing librey  6 or new before august. please make it a new version otherwise remove it from google play store.

The only code controlled by the Flutter team that uses the Google Play Billing Library is in_app_purchase, which updated to 6.0 last year. If you are using an old version of in_app_purchase_android, you'll need to flutter pub upgrade. If you are using some other plugin with a dependency on that library, you'll need to contact the developers of that plugin.


Migrate to actix-web 4

Is there a reason this is a Draft, beyond just waiting for stability of actix-web 4? Considering maybe already using this, if you think there's not some potential problem with the migration?

@Ploppz Hey! This is indeed the reason: actix-web 4 is still in beta and might break things until it is stable (already has). So I try to keep this branch updated. I use it on several projects (I specified the git repo/rev in my Cargo.toml) and everything seems OK from my point of view!

As I did a lot of improvements of the overall API/lib, I am going to merge this branch and release a beta. This way the documentation will be available online without the need to build it from source. I will try to follow actix-web releases (beta and stable) by publishing new versions of the lib when necessary.


7.0rc.1 Update-Help for Microsoft.PowerShell.Utility pull content for PS 6 help

I noticed that I am getting PS 6 help for commands on PS7
Steps to reproduce
Update-Help -Module Microsoft.PowerShell.Utility -Force
Get-Help Invoke-RestMethod
Get-Help Invoke-RestMethod -Online

Expected behavior
Related links of help should point to the PS 7.0 uri of  https://docs.microsoft.com/powershell/module/microsoft.powershell.utility/invoke-restmethod?view=powershell-7&WT.mc_id=ps-gethelp
The browser should open to the PowerShell 7 version of the help documentation
Actual behavior
Related links of help point to the PS 6.0 uri of  https://docs.microsoft.com/powershell/module/microsoft.powershell.utility/invoke-restmethod?view=powershell-6&WT.mc_id=ps-gethelp
The browser opens to the PowerShell 6 version of the help documentation
Additional Info
Calling Update-Help -Verbose shows it pulling PS 6 details.
PS> Update-Help -Module Microsoft.PowerShell.Utility -Force -Verbose
VERBOSE: Resolving URI: "https://go.microsoft.com/fwlink/?linkid=855960"
VERBOSE: Your connection has been redirected to the following URI: "https://pshelp.blob.core.windows.net/powershell/help/6/Microsoft.PowerShell.Utility/"
VERBOSE: Performing the operation "Update-Help" on target "Microsoft.PowerShell.Utility, Current Version: <IP_ADDRESS>, Available Version: <IP_ADDRESS>, UICulture: en-US".
VERBOSE: Microsoft.PowerShell.Utility: Updated C:\Users\kemarqu\Documents\PowerShell\Help\en-US\Microsoft.PowerShell.Commands.Utility.dll-Help.xml. Culture en-US Version <IP_ADDRESS>
VERBOSE: Microsoft.PowerShell.Utility: Updated C:\Users\kemarqu\Documents\PowerShell\Help\en-US\Microsoft.PowerShell.Utility-help.xml. Culture en-US Version <IP_ADDRESS>

PS> get-module Microsoft.PowerShell.Utility -ListAvailable

    Directory: C:\program files\powershell\7-preview\Modules

ModuleType Version    PreRelease Name                                PSEdition ExportedCommands
---------- -------    ---------- ----                                --------- ----------------
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Utility        Core      {Export-Alias, Get-Alias, Import-Alias, New-Alias…}

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-rc.1
PSEdition                      Core
GitCommitId                    7.0.0-rc.1
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I found this merged PR for RC1 that I think should have addressed this.
https://github.com/PowerShell/PowerShell/pull/11071
But I cannot identify why I am not getting the PS 7.0 help.
Possibly related historical issues:
https://github.com/PowerShell/PowerShell/pull/5820
https://github.com/PowerShell/PowerShell/issues/6187
https://github.com/PowerShell/PowerShell/issues/2616
https://github.com/PowerShell/PowerShell/issues/10705

I think 7.0 help content is still not published.
/cc @sdwheeler

@iSazonov The content is published. This was verified by @adityapatwardhan. The issue is that the HelpInfoUri in the v7-preview modules points to v6 downloads. This will be fixed for release.


score functionality to leverage cross_val_score from sklearn

Include the necessary functions in the sklearn base classes for easier cross-validation and hyperparameter optimization.

Only to be done for the SklearnLSS model to return Negative Log-Likelihood


Why use a BiFunction when the desired interface was a BiPredicate?

Describe the feature request
In the Unleash interface, there are some methods with a BiFunction<String, UnleashContext, Boolean> argument. But the availables implementations for Unleash (DefaultUnleash, FakeUnleash) both have no safe measures against this function returning a null. Bot will yield a NullPointerException:
DefaultUnleash.java#L123-L127
        boolean enabled;
        UnleashContext enhancedContext = context.applyStaticFields(config);

        if (featureToggle == null) {
            enabled = fallbackAction.apply(toggleName, enhancedContext);

FakeUnleash.java#L43-L46:
    public boolean isEnabled(
            String toggleName, BiFunction<String, UnleashContext, Boolean> fallbackAction) {
        if (!features.containsKey(toggleName)) {
            return fallbackAction.apply(toggleName, UnleashContext.builder().build());

Also, it is more semantic to use Predicate<T> or similar instead of Function<T, Boolean> or similar.
Background
Using Predicate<T> instead of Function<T, Boolean> is a lot more semantic, and also avoids (some) random null pointer exceptions that the java type system allows.
If the null pointer exception happens for some reason, it will be a violation of contract of the API consumer, not of the API provider, as it is now.
Solution suggestions
Change the API to accept a BiPredicate instead of a BiFunction.
It is wildly expected that it shall be ok with the use of lambdas. It will break compatibility if someone passes directly an instance of BiFunction<String, UnleashContext, Boolean>, as the tests within here have broken.
Trying to accept both BiPredicate<String, UnleashContext> and BiFunction<String, UnleashContext, Boolean> will cause the compiler to get confused, as both will satisfy the lambda signature. For example, in DefaultUnleash will yield a compiler error:
    @Override
    public boolean isEnabled(
            final String toggleName, final UnleashContext context, final boolean defaultSetting) {
        return isEnabled(
                toggleName,
                context,
                (n, c) -> defaultSetting);
    }

This will require a manual action of the maintainer to specify that it is a BiPredicate that it is wanted:
    @Override
    public boolean isEnabled(
            final String toggleName, final UnleashContext context, final boolean defaultSetting) {
        return isEnabled(
                toggleName,
                context,
                (BiPredicate<String, UnleashContext>) (n, c) -> defaultSetting);
    }

Hey, @jeffque 👋🏼 Thanks for opening the issue and for the thorough explanation 🙌🏼  From what I can tell, this seems like a very sensible change. However, as you mention in the PR description, this "might break some API consumers if they are using directly the interface BiFunction instead of a lambda". That means this would be a breaking change require a major version bump, so it requires a little more thought than it otherwise would.
I'm gonna take this and discuss it with some of the more Java-oriented of our devs and get back to you. Thanks again!


doc: adding treemacs readme

This is a stub I need to finish but I wanted to upload it for... I don't really know. Also its pertinent to #1166

I am handing in my work for marking

Thanks for the fixes! It seems that the file https://github.com/hlissner/doom-emacs/blob/develop/modules/ui/treemacs/README.org is already existing. You have additions however so it would be great if you could rebase your PR!

I am aware. Thanks for the reminder! Ill probably rebase it later today /tomorrow

Ok that was a bit of a mess but it should be rebased properly now

Thanks!


Unable to restore nuget packages in visual studio 2017 and 2019 enterprise editions

I have downloaded code from TFS and did package restore, - showing everything was restored, but checking the references under projects, a lot references are missing. see screen shot:

Build of course fails:

I had to go into each project and manual uninstall the packages and then re-install them. This is very error prone and time consuming. All the solutions have the same issue.
I followed https://docs.microsoft.com/en-us/nuget/consume-packages/package-restore-troubleshooting  steps and none work.
Help please. Thanks!

Ok, after all things done and deleted the code from local and re-clone code from TFS, it works now. Not sure if all attributed to the solution


Make h incr optional

Description
What the title says ...
Issue(s) addressed

fixes #795
fixes #794

Testing
Removed the hocn increment in :

3dvar_soca.yml , same results.
3dvarlowres_soca.yml ... not the same results as develop ... Needs attention.

Dependencies

waiting on JCSDA/ufo/pull/2190

Dear reviewers,
FYI: The UFO PR was merged ... If that was stopping you from reviewing :)

Dear reviewers,
FYI: The UFO PR was merged ... If that was stopping you from reviewing :)

Okay, okay, I am building now... :joy:


Update dependencies

Typescript warnings are displayed when installing dependencies.

Updated dependencies
Added "resolutions" in package.json to solve warning

done

i changed the confirmation message and resolved conflicts


Nodejs12.x is deprecated

I get this error when deploying the stack
An error occurred: ProjectEntryLambdaFunction - Resource handler returned message: "The runtime parameter of nodejs12.x is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (nodejs18.x) while creating or updating functions. (Service: Lambda, Status Code: 400, Request ID: 4a16bef5-bbed-45d3-bb54-67d4f54e3923)" (RequestToken: 6c49bc82-40b1-87c0-18cc-979c70bbb8c1, HandlerErrorCode: InvalidRequest).
Could you update node version?

@theburningmonk any news on this ?

@aleguern-azivko unfortunately, no


Feature request: project-level diagnosis errors

I might be mistaken, but seeing https://github.com/Microsoft/language-server-protocol/blob/30cd9e13bfb68ecd711837482b92eda1c65b5ab2/protocol.md#publishdiagnostics-notification, it seems that we cannot define an error that doesn't belong to a file? The use-case is for reporting errors like "your build configuration is wrong" or "some dependencies aren't installed correctly".
cc @jchavarri

Duplicate of https://github.com/Microsoft/language-server-protocol/issues/256

Yes, dups #256.


Fix openvino

1、在代码与文档中对ModuleNotFoundError: No module named 'mo'错误进行了提示

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: wangsiyuan06:x: syyxsxxwangsiyuan06 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: wangsiyuan06:x: syyxsxxwangsiyuan06 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Contributors

Site about us!

Looks sane, tho tests are missing

@snapshotpl
The page looks a bit empty at the moment:

Do you think it would make sense - other than repeating GitHub's contributors page - to show a randomized list of contributor avatars?

@localheinz there should be pretty nice list of 36 avatars. I don't know what happend

Thanks @localheinz for help!

@snapshotpl
You're welcome! But wait, there'll be more in a second!

@snapshotpl
Dang, https://github.com/snapshotpl/modules.zendframework.com/pull/8, left the PR in an unmergeable state, probably because of the changes in https://github.com/zendframework/modules.zendframework.com/pull/445.

@Ocramius
Since this includes contributions from me, do you want to take a look again?
@snapshotpl
If @Ocramius has no objections, I would be happy to merge and deploy, we can still re-iterate!

@localheinz it needs a rebase (with --preserve-merges, careful!)

@Ocramius
About the rebase, shall I do it before merge or ask @snapshotpl to do it?

Your decision :-)
On Mar 4, 2015 8:33 AM, "Andreas Möller"<EMAIL_ADDRESS>wrote:

@Ocramius https://github.com/Ocramius
About the rebase, shall I do it before merge or ask @snapshotpl
https://github.com/snapshotpl to do it?
—
Reply to this email directly or view it on GitHub
https://github.com/zendframework/modules.zendframework.com/pull/447#issuecomment-77116250
.

@snapshotpl
Rebasing and merging this in a minute!

Merged with 5a770735957184dd0db9aef73abf5d5ef090875e.

And, thank you very much, @snapshotpl!


perf(metarepos): avoid copy overhead by removing unnecessary converting from byte slice to string

What this PR does
This change removes copy overhead caused by unnecessary type casting from byte slice to string. Refer to this blog for performance.

Current dependencies on/for this PR:

main

PR #532   👈

This comment was auto-generated by Graphite.

@ijsong started a stack merge that includes this pull request via Graphite.

@ijsong merged this pull request with Graphite.


Enabled smart colors

Context
This enables smart colors by default.
Testing Instructions
This PR can be tested by following these steps:

Add a text preset or text set to various backgrounds and see the elements added in proper contrasting color (potentially with a background/scrim).
Observe that this does not cause a "serious" lag before insertion.

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #8238

For now this will be delayed due to performance reasons until we have an opt-in UX for users.


Roll engine af42b6dc95bd..eb973861069b (6 commits)

https://github.com/flutter/engine/compare/af42b6dc95bd...eb973861069b
git log af42b6dc95bd..eb973861069b --no-merges --oneline
eb9738610 Roll src/third_party/skia 0fdc53f9ffc6..f125111ca733 (10 commits) (flutter/engine#6100)
06b8fa4ed Roll src/third_party/skia dd04bec39cea..0fdc53f9ffc6 (13 commits) (flutter/engine#6098)
bf078042d Roll ICU to c56c671998902fcc4fc9ace88c83daa99f980793 (flutter/engine#6097)
c867305c3 Unconditionally set resource id to empty string (flutter/engine#6096)
b4b1d0578 Roll src/third_party/skia e00fe30b8fe7..dd04bec39cea (4 commits) (flutter/engine#6095)
8694036a6 Roll src/third_party/skia 420696961c8c..e00fe30b8fe7 (1 commits) (flutter/engine#6094)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

AutoRoller is stopped; closing the active roll.


Suggestion: Have a Button to switch the sides of the Players

When running an event with a Camera, often times the players sit in a different order than expected, having a single button switch the sides of the players would be very helpful. As an example, going from this:

to this:

currently requires you to retype everything and reset characters and score, and as this is something that I as a TO think is quite frequent having a single button swap them would be great.

Yeah this is a bad design choice im not happy about. Theres actually a swap button, it's on the mid bar separating both characters, but theres nothing there to indicate it exists aside from the icon when hovering.

Oh wow,


Parse statements with or_continue and or_break

It seems that or_else and or_return are supported, but not or_continue and or_break.
(no semantic tokens/go to definition/hover)
Which is understandable as they were added recently.
I'm guessing it should be a simple change to parse them.

Are we talking about the formatter?

no, about ols, is it really not in ast?
From looking at the ast package there is Or_Branch_Expr node and Token_Kind.Or_Break and Token_Kind.Or_Continue

It seems it is. Fixed.


fix: When modify the node name, circular dependence bug #4417

Purpose of the pull request
When modify the node name, circular dependence bug
#4417

+1


https://bugs.kde.org/show_bug.cgi?id=374218

https://bugs.kde.org/show_bug.cgi?id=374218

working dir of appimage is not the dir from which the appimage is started

@boud
AppRun does chdir() to $APPDIR/usr so that we can binary patch hardcoded /usr paths to ././ and have it work, effectively making non-relocateable software relocateable. It is kind of a hack. Rewriting the AppRun file (e.g., in bash) should solve this.
Also, linuxdeployqt does away with an AppRun wrapper entirely and replaces it with just a symlink to the executable.


RTVSLO fails on some links

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

Region
United States
Provide a description that is worded well enough to be understood
Yt-dlp works with this link:
https://365.rtvslo.si/arhiv/razred-zase/174376609
Fails with this link:
https://365.rtvslo.si/arhiv/razred-zase/148350750
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: ['-v', 'https://365.rtvslo.si/arhiv/razred-zase/148350750']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.8.6 (CPython AMD64 64bit) - Windows-10-10.0.19041-SP0 (OpenSSL 1.1.1g  21 Apr 2020)
[debug] exe versions: ffmpeg 2023-08-23-git-5f5a1ccd04-full_build-www.gyan.dev (setts), ffprobe 2023-08-23-git-5f5a1ccd04-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.9.8, brotli-1.0.9, certifi-2019.11.28, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.1
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Using fake IP <IP_ADDRESS> (SI) as X-Forwarded-For
[rtvslo.si] Extracting URL: https://365.rtvslo.si/arhiv/razred-zase/148350750
[rtvslo.si] 148350750: Downloading JSON metadata
[rtvslo.si] 148350750: Downloading JSON metadata
ERROR: [rtvslo.si] 148350750: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\YoutubeDL.py", line 1560, in wrapper
    return func(self, *args, **kwargs)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\YoutubeDL.py", line 1709, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\YoutubeDL.py", line 1768, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\YoutubeDL.py", line 2732, in process_video_result
    self.raise_no_formats(info_dict)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\YoutubeDL.py", line 1073, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [rtvslo.si] 148350750: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

Needs something like this:
diff --git a/yt_dlp/extractor/rtvslo.py b/yt_dlp/extractor/rtvslo.py
index 05942b6b4..054174fe2 100644
--- a/yt_dlp/extractor/rtvslo.py
+++ b/yt_dlp/extractor/rtvslo.py
@@ -1,6 +1,7 @@
 from .common import InfoExtractor
 from ..utils import (
     ExtractorError,
+    int_or_none,
     parse_duration,
     traverse_obj,
     unified_timestamp,
@@ -114,16 +115,16 @@ def _real_extract(self, url):
                         else f.get('language'))
                 })

-        formats.extend(
-            {
-                'url': f['streams'][strm],
-                'ext': traverse_obj(f, 'mediaType', expected_type=str.lower),
-                'width': f.get('width'),
-                'height': f.get('height'),
-                'tbr': f.get('bitrate'),
-                'filesize': f.get('filesize'),
-            }
-            for strm in ('http', 'https')
-            for f in media.get('mediaFiles') or []
-            if traverse_obj(f, ('streams', strm))
-        )
+        for mediafile in traverse_obj(media, ('mediaFiles', lambda _, v: url_or_none(v['streams']['https']))):
+            formats.append(traverse_obj(mediafile, {
+                'url': ('streams', 'https'),
+                'ext': ('mediaType', {str.lower}),
+                'width': ('width', {int_or_none}),
+                'height': ('height', {int_or_none}),
+                'tbr': ('bitrate', {int_or_none}),
+                'filesize': ('filesize', {int_or_none}),
+            }))
+
+        for mediafile in traverse_obj(media, ('mediaFiles', lambda _, v: url_or_none(v['streams']['hls_sec']))):
+            formats.extend(self._extract_wowza_formats(
+                mediafile['streams']['hls_sec'], v_id, skip_protocols=['smil']))


docs(installing_deis): add HAProxy docs for load balancing

See discussion in https://github.com/deis/deis/pull/1352

Deis v1 LTS support ends February 13, 2017, and v1.13.4 is intended to be the last official release, so I'm closing this.
Please see Deis Workflow, the documentation / hub repository for the next-generation PaaS.


cleanup from changes to import_block

What was wrong?
fixes https://github.com/ethereum/py-evm/issues/733
https://github.com/ethereum/py-evm/pull/1151 did most of the heavy lifting but there were some things that needed cleaning up.

the test case was written using loops which made it a bit hard to parse.
the return value for block importing was inconsistent, sometimes returning the imported block in the new blocks section even if it wasn't part of the new chain.

How was it fixed?

rewrote the test to involve fewer loops and ensured that it used all fork rules for the mainnet VM
changed Chain.import_block to return a 3-tuple

(imported_block, new_canonical_headers, orphaned_canonical_headers)

I also added logging output to the regular chain syncer which fixes #733
Cute Animal Picture

100% nitpicking: we all know what "orphan" means in this context, and that it's (semantically) incorrect; how about new_{uncle,ommer}_blocks instead of orphaned_canonical_blocks?..
(It was introduced in PR #1151.)
This might not be a problem now, but might become one come Shasper, where "orphan" may correctly refer to X(n) for which a parent X(n-1) is not known.
See PR #257 for only other mention of term "orphan" in this repo (referencing a now-obsolete sharding implementation).

we all know what "orphan" means in this context, and that it's (semantically) incorrect

Ah, very interesting point. I had the same semantic concern, but orphan is used so commonly in other blockchain contexts to mean a non-canonical block, that it seemed reasonable to follow that trend.

Shasper, where "orphan" may correctly refer to X(n) for which a parent X(n-1) is not known

Agree that since the semantically-appropriate name orphan could reasonably be used there, we should reserve the name for that use case.

how about new_{uncle,ommer}_blocks instead of orphaned_canonical_blocks?

One problem is that uncle is used so heavily when the uncle header is included via GHOST. But at this point, we only know that the block is no longer on the canonical chain.
Maybe:

decanonicalized_blocks (mouthful, and spelling problems?)
non_canonical_blocks
ejected_blocks
dropped_blocks
loser_blocks

One problem is that uncle is used so heavily when the uncle header is included via GHOST. But at this point, we only know that the block is no longer on the canonical chain.

Ah, right. A block becoming non-canonical doesn't mean it's been referenced as an uncle.
There's also the option of {old,prev}_canonical_block, similar to how it was called before...

I'm gonna go with non_canonical_blocks, it's explicit and shouldn't require much of any mental overhead to know what it is.

And then I ended up going with old because non and new were too visually similar

new_canonical_blocks
non_canonical_blocks
old_canonical_blocks


i2c adapter does not start anymore after updates / node 14 and js-controller 3.3 / Sktiptausführung 5.2.21

Describe the bug
i2C adapter won't start anymore (stays red) with latest js-controller

javascript adapter update broke it maybe, because I think after that it stopped working.
because I did not do "iobroker update self" before it stopped working, just adapter updates. Or maybe it even was i2c adapter update?
Versions:

Adapter version: latest NPM beta or latest github: 1.2.1
JS-Controller version: <IP_ADDRESS>
Skriptausführung: 5.2.21
Node version: 14.8.2
Operating system: linux / buanet docker image with node 14

After a NodeJS update one needs to rebuild the dependencies. I believe the command is iobroker rebuild.

I actually did not do the nodejs update now, it was weeks ago and worked until now, but for some reason it helped

stopped iobroker
iobroker rebuild i2c

npm rebuild --loglevel error (System call1) in "/opt/iobroker/node_modules/iobroker.i2c"
Rebuild i2c done

start iobroker again -> similar errors again

BUT after the iobroker rebuild i2c --install
with the install suffix, IT WORKED!
thank you!


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.2.7

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.2.6 -> v1.2.7

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper
v1.2.7
Compare Source
Bug Fixes

deps: update module github.com/ibm/platform-services-go-sdk to v0.29.5 (#172) (642eeb3)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Enforcer Caching for remaining APIs

Earlier we introduced Enforcer Caching on Dashboard List APIs which has improved the Dashboard load time but there are other areas that are also impacted by Enforcer.

Here I am attaching Test cases file link for everyone’s reference.  Link


stories hyperlink on correct part of text

puts the hyperlink on the correct part of the intro text

👍


[Enhancement] Water Egg Crafting

If the water eggs were registered with the ore dictionary as itemAnywater then they could be used in recipes that require water buckets without first having to place the water, scoop it with a bucket, and then craft one at a time.

possibly fixed by #124

I've just spent some time doing some testing.  I've never used industrialcraft, so I did not do any experimentation with that mod.  I was unable to use the lava or water eggs to do anything other than place a lava or water source block in the world.  I could not fill tanks or barrels, they did not work as a lava bucket in a stirling generator, or a water bucket in combustion generator.  All versions are the most recent available on the Curse/Twitch launcher as of this date.  JEI, Waila, and Hwyla are my basics in testing.  I added ceramics (which has a barrel), Ender IO, and tinker's to experiment with these fluid eggs.  In all my experiments, I was able to use water bucket, lava bucket, or both normally.  Lava and Water eggs either placed a source block in the world, or did nothing/was not usable.
  With my installed mods for this test, the only thing craftable with water bucket is feather pulp (I couldn't see anything for lava bucket).  Crafted successfully with water bucket.  Could not craft with water egg.


Remote specific protocol settings (configuration, feature request)

Would it be possible to have a default protocol specific to different commands?
Specifically the use case I have in mind is having ssh access to all of my personal repositories, but only https access to remote repositories.
This acts as a safety measure, as I can easily push to my local repositories over ssh, but need to go through additional steps to push on remote repositories.
↬@Carreau for introducing this method to me.

Will this be addressed if we add a .hubconfig file that you can put in the working copy of an individual repository? Because that is planned.

Not quite. This would involve being able to specify a ssh vs https connection for different remotes on the same repository, specifically upstream vs. origin. In practice they are both remote they are just associated with my github repo (e.g., https://github.com/mpacer/nbconvert) vs. the upstream (https://github.com/jupyter/nbconvert).
Failure on my part to use language precisely originally.
Local and remote should have been origin and upstream, changing that now.

Ah, I see now.
Where do you envision these configuration options be kept? Would this global configuration encompass all same-named remotes, e.g. special casing for "upstream" remote (if configured) would be applied to every repo on your machine?

I guess using something like conditional includes, and/or pushInsteadOf, though these two options do now allow to match on the name of the remote but only on the urls/gitdir. I don't know of any config that allow a per-remote-name configuration but I at least tend to name my remote consistently so a global config would indeed be awesome.


Added strings evaluation functionality to rest_client.ContainerClient.get

Added strings evaluation functionality to rest_client.ContainerClient.get to allow types parsing.
Description in function code.
Added requirement: dateparser

Lint report for commit<PHONE_NUMBER>7a0490cea8656dc1c6576c16b8d8ee:
Lint OK! :cake: :punch: :cookie: .
No commit flaws detected.

CFME QE Bot

Cool - in progress...

Lint report for commit b281dfcd36aa09878bb52ba5dbb7617b59957bbc:
Lint OK! :cake: :punch: :cookie: .
No commit flaws detected.

CFME QE Bot

Done + Tested.

i dont think a rich json decoder should be controlled by boolean parameters to singluar api functions

OK, personally I think it worth to keep it but I can remove this if you want (i.e. revert mgmsystem/rest_client.py), @psav what do you think?

Lint report for commit d4e100451391a90674c9d658d44eef2a47f6d4fe:
Lint OK! :cake: :punch: :cookie: .
No commit flaws detected.

CFME QE Bot


MM-15080 Use event emitter to scroll to bottom when sending post

As I mentioned before, inferring this from the redux state is really, really difficult, so we're just going to use an event to trigger the post list to scroll to the bottom because it's very straightforward to do that.
Ticket Link
https://mattermost.atlassian.net/browse/MM-15080

we're just going to use an event to trigger the post list to scroll to the bottom because it's very straightforward to do that
@hmhealey that is fine i think for IE11 i have to add similar logic anyway


Fix IME position in the single window mode sub-windows.

Fixes https://github.com/godotengine/godot/issues/74156

Thanks!

Cherry-picked for 4.0.2.


test: declare hotrestart_test as enormous to hopefully avoid timeouts.

Commit Message: we gave been seeing timeouts in CI on this test, which I find takes 220 seconds on my workstation. In CI it might take longer, so declare the test enormous and hopefully no timeouts.
Additional Description:
Risk Level: low
Testing: just that test (in asan mode).
Docs Changes: n/a
Release Notes: n/a

Something must have changed here. Is this possibly related to @zuercher recent change? I can't imagine the test should take this long?

I tested this. Before @zuercher 's commit, the test was taking 172s (on my dev box). After, it's taking 176s. I did two runs of the test with each build, and they were very consistent. So the change did slightly increase the test time, but not by a lot. It may have pushed it past the timeout in CI though.

As you can see this fails tsan, even declared as enormous. It was a functional failure though, on the hot-restart generation stat.

Well, this PR is still timing out for that test in TSAN, so something is definitely broken. :(

As you can see this fails tsan, even declared as enormous. It was a functional failure though, on the hot-restart generation stat.

Ah OK, maybe there is a bug/race there also that needs to be fixed?

Do we think this bug might have been introduced by https://github.com/envoyproxy/envoy/pull/11357 ? Or was it pre-existing?

No idea.


Contribution To This Json Project

Hello,
I want to contribute to this project and improve my coding skills. I am a beginner-intermediate level programmer. Can anyone please guide me from where to start in order to contribute this project.
Thank You
Pawandeep Singh

Hi @pawandeep2155 and thanks for your interest in the project. If you want to contribute, the best way would be to have a look at the open issues and see if you can help there. You may also want to have a look at the "WANTED" list where long-term goals of the project are tracked. Finally, the documentation can certainly be improved.
If you need any help, please feel free to ask here or via email.

@pawandeep2155, do you need any assistance?

@nlohmann Thank you for your support. But currently I am contributing to another project. I will come back to your project after I finish that.


chore: remove repeated license in utils file

References
LUI-929
Reported by Chris in tickets channel: https://cim.slack.com/archives/C044CH7REP2/p1689603261093309

Sanity Tests Execution Passed.

Sanity Tests Execution Passed.

Sanity Tests Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Performance improvement: no waiting to check capability overwriting

The current implementation uses rd also as a source register in order to check whether it is a capability to update refcounts accordingly. We can instead just do that when a physical register gets unmapped.

Examples:
ldc ca0, ca1
movc ca0, ca2

The second instruction movc does not need to wait for the value of ca0 to be ready. Instead, when the result of ldc is ready, we can just check whether its value of ca0 has already been overwritten (mapped to a different physical register), and update (or skip updating) the refcounts accordingly.

Initial changes submitted in: 790d74e00dae462bf82aa6c1a3a80cd2d705703a
This version needs to be updated to also actually update the refcounts

Initial changes submitted in: 790d74e This version needs to be updated to also actually update the refcounts

Coool. BTW you can do fancy stuff like using #n to reference issue number n in the commit message.


PROMPT_COMMAND (with current time) breaks cursor position

Environment
Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.19041.0 Microsoft Windows NT 10.0.19041.0

Windows Terminal
Version: 1.4.3243.0

WSL2 Ubuntu 20.x

Steps to reproduce
Set your PROMPT_COMMAND to pretty much anything.
export PROMPT_COMMAND="echo -n \[\$(date +%H:%M:%S)\]\ "

Now use the arrow keys or ctrl+r to go back in history to some earlier command, preferably a long one.
Navigate your cursor left with the home key. (Or arrow up two times+)
Your cursor should now be somewhere before the actual prompt space, somewhere in your path or pc name.
Expected behavior
The cursor should be right after the $.
Actual behavior
Described above. Cursor is not aligned with the visible content.

Weird so I'm trying this just by running a quick
PROMPT_COMMAND="echo -n \[\$(date +%H:%M:%S)\]\ " bash

but I can't get this to repro. I do seem to get some weird behavior with the ctrl+r reverse search, but not with the arrow/home navigation. It's almost like bash is just ignoring the PROMPT_COMMAND part of the prompt when it's redrawing:

But you know, the same thing happens with gnome-terminal

So I doubt that what I'm seeing is unique to the Terminal

Weird so I'm trying this just by running a quick
PROMPT_COMMAND="echo -n \[\$(date +%H:%M:%S)\]\ " bash

but I can't get this to repro. I do seem to get some weird behavior with the ctrl+r reverse search, but not with the arrow/home navigation. It's almost like bash is just ignoring the PROMPT_COMMAND part of the prompt when it's redrawing:

But you know, the same thing happens with gnome-terminal

So I doubt that what I'm seeing is unique to the Terminal

@zadjii-msft thanks for the quick reply. This does, however, not happen with the bash exe. It's just a minore issue but quite annoying from time to time.

@zadjii-msft thanks for the quick reply. This does, however, not happen with the bash exe. It's just a minore issue but quite annoying from time to time.

This does, however, not happen with the bash exe.

Wait, if this isn't happening with bash, then which shell are you using inside WSL?

This does, however, not happen with the bash exe.

Wait, if this isn't happening with bash, then which shell are you using inside WSL?

I was talking about C:\Windows\System32\bash.exe

I was talking about C:\Windows\System32\bash.exe

Oh, that's actually just an alias for starting up bash in WSL. bash is typically the default shell for most distros, so I'm assuming that's the one you're using. It looks to me like you might be running afoul of the following: https://unix.stackexchange.com/a/31643

Oh, that's actually just an alias for starting up bash in WSL. bash is typically the default shell for most distros, so I'm assuming that's the one you're using. It looks to me like you might be running afoul of the following: https://unix.stackexchange.com/a/31643

Yes, that is exactly it. Bash tries to count how long the prompt is, and when you run a command before the prompt is printed it cannot count that command's characters. :smile:

Yes, that is exactly it. Bash tries to count how long the prompt is, and when you run a command before the prompt is printed it cannot count that command's characters. :smile:

Yup, indeed. Thanks!
Moved my PROMPT_COMMAND stuff into PS1.

Yup, indeed. Thanks!
Moved my PROMPT_COMMAND stuff into PS1.


Multiple whatsapp sessions

I need a hint of the best way to handle multiple whatsapp sessions (each one for a different whatsapp identifier), in other words, we have to work with a inconstant amount of whatsapp sessions.
Does this lib work fine with multiple whatsapps? Would it be necessary to create an instance for each one (running on different server ports)?

I have not testet that, but it should be totally fine to create multiple connections to whatsapp with different phone numbers. I can't determine the best way to keep track of them, because it depends on your use case.
A quick suggestion on top of my head would be to use struct embedding:
type waWrapper struct {
	*whatsapp.Conn
	identifier string
}

This way the waWrapper is a *whatsapp.Conn. Now you could define the Handler interface on the waWrapper and add it to itself like:
wac, err := whatsapp.NewConn(5 * time.Second)
if err != nil {...}
wrapper := &waWrapper{Conn: wac, identifier : "MyName"}
wrapper.AddHandler(wrapper)

Now you can access the whatsapp connection and the identifier from inside the handler. That way you should be able to process incoming messages and react to them. And for sending messages yourself that do not react on incoming messages I would simply create a map like map[string]*waWrapper.
Always use pointers to the waWrapper to not get in trouble with weird copies and incosistent state.

Already implemented in dimaskiddo/go-whatsapp-rest you can see how it works in the helper file mostly it's the same like Schulte said, but this also still need some test :)
Hope this helps you also

thanks @dimaskiddo for sharing your repository anyway can you provide for documentation how to run it ? for your repository

sorry if my repo still doesn't have readme, prequisites:

Make sure already install make and dep
Create a workspace directory and export it as one of GOPATH directory
Pull the repo under workspace directory in src/github.com/dimaskiddo/go-whatsapp-rest

How to run it:

make go-dep-ensure
make go-run

I will close the issue now. For further questions regarding the use of dimaskiddo/go-whatsapp-rest please open an issue over there.


Improve customizability of object mapper

In DefaultOpenApiObjectMapperSupplier a dedifcated ObjectMapper is built which is used at various places when the library generates the Open Api spec.
Consider if one should offer customizers for this ObjectMapper, have a look how Spring Web is doing it (see JacksonAutoConfiguration)

I think this issue can be closed as customizability is already given by overriding either OpenApiYamlMapper or OpenApiObjectMapperSupplier beans.


How to trigger submit event on keyboard Enter button hit in React Native?

Hi, I have a question regarding virtual keyaboard and submitting a form on Android.
I have a form and I want it to submit when user hits Enter key after typing. But virtual keyboard has only such button https://c2n.me/3TjN0X8.jpg (with tick icon), after hitting which keyboard just hides and no submit. In some apps I saw such button https://c2n.me/3TjN35e.jpg (with arrow button), after pressing which submit event occurs.  What do I have to do to make virtual keyboard have such "submit" button, instead of 'OK' button?
Here is the code:
class SearchBenefitsForm extends React.Component<IProps> {
  renderTextInput = ({ input, autoFocus }: { input: any, autoFocus: boolean }) => {
    return (
      <TextInput
        {...input}
        autoFocus={autoFocus}
        underlineColorAndroid="transparent"
      />
    );
  }

  render() {
    const {
      handleSubmit,
      submitting,
      invalid,
      onCancel,
    } = this.props;

    return (
      <View style={styles.container}>
        <TouchableOpacity onPress={handleSubmit} disabled={submitting || invalid}>
          <Image style={styles.searchIcon} source={searchIcon}/>
        </TouchableOpacity>
        <View style={styles.searchExpression}>
          <Field
            name={SEARCH_BENEFITS_EXPRESSION}
            autoFocus={true}
            component={this.renderTextInput}
          />
        </View>
        <TouchableOpacity onPress={onCancel} disabled={submitting || invalid}>
          <Image style={styles.closeIcon} source={closeIcon}/>
        </TouchableOpacity>
      </View>
    );
  }
}

export default reduxForm({
  form: SEARCH_BENEFITS_FORM,
})(SearchBenefitsForm);

I've found the following solution: onSubmitEditing handler from TextInput handles event when user hits Enter key on mobile, so it is possible to pass function from Field component to TextInput for execution on Enter.
One question from me: when TextInput gets input parameter from Field, this input has onFocus, onBlur and other event handler https://redux-form.com/6.4.3/docs/api/field.md/#meta-props ,but it doesn't have  onSubmitEditing prop, so I have to pass it as another parameter. Why so?
class SearchBenefitsForm extends React.Component<IProps> {
  renderTextInput = ({ input, autoFocus, onSubmitEditing }: { input: any, autoFocus: boolean, onSubmitEditing: Function }) => {
    return (
      <TextInput
        {...input}
        autoFocus={autoFocus}
        underlineColorAndroid="transparent"
       onSubmitEditing={onSubmitEditing}
      />
    );
  }

  render() {
    const {
      handleSubmit,
      submitting,
      invalid,
      onCancel,
    } = this.props;

    return (
      <View style={styles.container}>
        <TouchableOpacity onPress={handleSubmit} disabled={submitting || invalid}>
          <Image style={styles.searchIcon} source={searchIcon}/>
        </TouchableOpacity>
        <View style={styles.searchExpression}>
          <Field
            name={SEARCH_BENEFITS_EXPRESSION}
            autoFocus={true}
            component={this.renderTextInput}
            onSubmitEditing={handleSubmit}
          />
        </View>
        <TouchableOpacity onPress={onCancel} disabled={submitting || invalid}>
          <Image style={styles.closeIcon} source={closeIcon}/>
        </TouchableOpacity>
      </View>
    );
  }
}

export default reduxForm({
  form: SEARCH_BENEFITS_FORM,
})(SearchBenefitsForm);

Please use <Control> and not <Field>, and let me know if your problem disappears.

I did not find Control in RN Components, did you mean TextInput?


[FR]: 'Delete all before publish' option

Issue validation

[X] I checked the issue to prevent duplicate
[X] My issue concern doesn't concern the Mkdocs template

Is your feature related to a problem ?
No
What solution do you want to see ?
Proposal
I imagine that it would be better if there is an option 'Delete all before publish' applicable to some or all of the following:

Upload all shared notes
Refresh all published notes
Refresh published and upload new notes

Motivation
A usecase is that I renamed an already published note and publish it again, then there are two files, 'original' published file and 'renamed' published file. I want a repository to have only one file with the latest name.
How does the proposal satisfy the motivation?
By deleting all files on the remote before publishing, the remote starts with a fresh state. Then actual publish command upload all public notes. If the pull request is merged after then, it has a rename effect virtually.
Describe the alternative you've considered
No response
OS
IOS, Android, MacOS, Windows, Linux
Anything else?
No response
Plugin version
7.0.2
Obsidian version & debug log
1.5.12
I don't think debug log is required since it is not a bug report

Use the autoclean function.


Start recommending Python 3

In the docs the section The Status of Python 3 states that you strongly recommend using Python 2.6 and 2.7.
Well, isn't it about time to officially recommend to use Python 3 instead? Has this been discussed before (recently)?
I myself at least never ran in any problems using Python 3 with Flask. Though, admittedly, I don't have any large projects or unusual dependencies - or maybe I just instinctively avoid anything that's not Python 3 compatible and thus never noticed problems.
Still, it seems to me that a recommendation to Python 3 is reasonable, unless you have special dependencies, and it doesn't look to me that there are many (active) Flask-* packages that require Python 2.

strongly recommend using Python 2.6 and 2.7.

FYI, Python 2.6 recommendation has been dropped already.
https://github.com/mitsuhiko/flask/commit/add59a91cf45b567397db134d3e4ed4ce3ecd6da

I must have missed that in the link I posted. I copied the text from the current online docs. That's at least something...

@methane the online docs do not reflect the current docs in the repo. Thanks for brining this up.


Update json_ref

Update based on https://github.com/devcontainers/spec/pull/122, https://github.com/devcontainers/spec/pull/124, and a variety of other changes in appears weren't aligned between containers.dev and devcontainers/spec
@Chuxel I just noticed https://github.com/devcontainers/devcontainers.github.io/pull/51 so I was going to delete this branch, but as I found and addressed some other ordering and wording inconsistencies in this branch, maybe we merge this PR instead if it looks good to you?
I also noticed a section missing from devcontainers/spec as I was diffing these two files, so opened this PR to update: https://github.com/devcontainers/spec/pull/126.

This one looks good!  I can close my PR.

Thank you both! 🚀


Old and new command act differently

I'm sure they shouldn't act differently, and I doubt whether just typing kommit should even work anymore. Perhaps give a little deprecation notice for a few versions and then stop working completely...?
tom > ~/dev/schooling > brew install kommit                                                                                  master
==> Downloading https://homebrew.bintray.com/bottles/kommit-1.1.0.el_capitan.bottle.tar.gz
Already downloaded: /Library/Caches/Homebrew/kommit-1.1.0.el_capitan.bottle.tar.gz
==> Pouring kommit-1.1.0.el_capitan.bottle.tar.gz
🍺  /usr/local/Cellar/kommit/1.1.0: 4 files, 11.2K
 tom > ~/dev/schooling > kommit                                                                                  master
Invalid usage...
usage: kommit [-m <msg> | -c | -e | -s | -i | -r | -h]

    -m <msg>        append message
    -c              check if hook installed ?
    -e              edit messages
    -s              show current message(s)
    -i              install kommit hook to .git/hooks/prepare-commit-msg
    -r              remove kommit hook, delete .git/hooks/prepare-commit-msg
    -h              print this message

 tom > ~/dev/schooling > git kommit                                                                                  master
Alias tip: g kommit
Invalid usage...
usage: git-kommit [-m <msg> | -t <msg> | -c | -e | -s | -i | -r | -h]

    -m <msg>        append message
    -t <msg>        append message with timestamp
    -c              check if hook installed ?
    -e              edit messages
    -s              show current message(s)
    -i              install kommit hook to .git/hooks/prepare-commit-msg
    -r              remove kommit hook, delete .git/hooks/prepare-commit-msg
    -h              print this message

 tom > ~/dev/schooling >

yes, all changed... when you install from brew, command name becomes git-kommit which is good for git integration. now you can use;
git kommit -h
git kommit -s
git kommit -m "Hello"

ok ?

Yes, but why is the old command still there? And not listing -t?

it only works under git repo... go to any git repository. that try git kommit -h

I know that. Please read the first post properly!

you are free to delete /usr/bin/kommit  from now on, brew will install git-kommit not kommit. Due to name change, file left orphan there in /usr/local/bin/... please remove it by your self. I did not imagine that...

Ah, ok. Gotcha.


relationships are between elements

the whole idea of the Relationship is that it's between two elements. For example, CONTAINS is to be used when SPDXRef-A contains SPDXRef-B.
I fail to see how the RDF parser or the TagValue parser actually get this information...
Maybe we should discuss each one separately.

@zvr ,

for rdf parser
in rdf format, the spdxElementId which in the above example given by you is SPDXRef-A can be found in subject of namespace['relationship'], whereas relatedSpdxElement can be found in the object.
this code might explain:

import rdflib
from rdflib import Graph
import six
from rdflib import Namespace
from rdflib import RDF
from rdflib import RDFS
import sys
import pprint

file = sys.argv[1]
graph = Graph()
graph.parse(file)
spdx_namespace = Namespace("http://spdx.org/rdf/terms#")
for s, _p, r_term in graph.triples((None, spdx_namespace['relationship'], None)):
        print('spdxElementId {}'.format(s))

        for su, pr, ty in graph.triples((
                r_term, spdx_namespace['relationshipType'], None)):
                    print('relationshipType {}'.format(ty))
        for sub, pre, typ in graph.triples((
                r_term, spdx_namespace['relatedSpdxElement'], None)):
                    print('relatedSpdxElementId {}'.format(typ))
         ```
if you run the above code with SPDXRdfExample (containing relationship), you will get the results

OK, RDF case:
As you say, the Relationship has

subject (the A element)
type ("contains")
object (the B element)
an optional comment.

These should be reflected in the object (class) definition -- well, not the subject, since it is the one containing the object.
You shouldn't be concatenating elements in a string and then splitting it again when you want to get one of them.

I'm concatenating it because, in tag value format, the relationship is in this form:
Relationship: SPDXRef-DOCUMENT DESCRIBES SPDXRef-File
so, I think it would be better to maintain consistency among all formats.

Please don't. The tag-value format is the worst of all the formats and it makes no sense to transform the structures ones (like RDF, JSON, YAML) to the lowest common denominator.
As you can see, the tag-value does not even connect the RelatioshipComment with the Relationship it applies to (but relies on it being somewhere near in the file).

@zvr sorry, I didn't quite get it.
presently it is something like this:
class Relationship(object):
    def __init__(self,relationship,relationship_comment):
        self.relationship = relationship
        self.relationship_comment = relationship_comment

it follows Relationship btw spdx elements


DeprecationWarning: _get_shared_lib_location is deprecated

Describe the bug
functime/feature_extractors.py:36: DeprecationWarning: `_get_shared_lib_location` is deprecated. It will be removed in the next breaking release. The new `register_plugin_function` function has this functionality built in. Use `from polars.plugins import register_plugin_function` to import that function. Check the user guide for the currently-recommended way to register a plugin: https://docs.pola.rs/user-guide/expressions/plugins
    lib = _get_shared_lib_location(__file__)

To Reproduce
File functime_test.py
from functime.feature_extractors import FeatureExtractor

In terminal:
pytest functime_test.py

Expected behavior
No DeprecationWarning is reported.
Desktop (please complete the following information):

MacOSX 14.5
Python 3.12.2
functime==0.9.5

Ciao @khrapovs, and thank you very much for your thorough reports. I had originally and hastily written this down here #194 (which I am closing). I would like to address this but I don't know where to start (invoking our Plugin lord @mgorelli).
PRs are welcome!


Unminified browser.js?

Hi, is an unminified version of lib/browser.js available or can it be created?
Thanks!

fork and run npm install && npm build:dev
Do let me know the use case im interested

I'm interested in adding public/private key generation for NEO to http://cryptostorage.com (the site is still pre-launch).
It would be ideal to have a standalone javascript file capable of generating public/private key pairs in the browser.  The source code needs to be unminified so users can audit it.  Do you have any suggestions?  Perhaps your ansy wallet generator is better to use?

Hmm even when unminified it is quite a chore to assess if the code is truly safe. The easy way out is providing a standalone release that can be downloaded and used with an offline computer. This way only the hash needs to be checked before reading the unpacked source code to ensure it is safe.
If you are merely generating private /public key pairs I suggest just understanding ansy code and implementing it yourself.

Do you have any recommendations for how I can get a standalone release that can be used with an offline computer to generate key pairs?  It's fine if the code includes more than key pair generation - it simply won't be used.
Thanks!

When I meant standalone release, I meant releasing your website on github as a standalone release. Do check out how other offline wallet generators work and how they do stuff.
Like I mentioned, you can either use this or ansy code.

Ah, yes.  It will be provided as a standalone application to download and run on an offline computer.  Thanks for your help!


add HMR to server

https://github.com/ericclemmons/hot-module

first work here https://github.com/entria/entria-fullstack/pull/30
improve here https://github.com/entria/entria-fullstack/pull/30#issuecomment-441205395


Code jumping support by dumbjump

Thank you for your configuration, it is very helpful for me, Sometimes I write C programming language in Emacs, I find that it is hard for me to find the definition of the function. I think the
dumbjum can be a suitable choice for it. Can I create the PR for supporting this feature ?

Hi! Thanks for getting in touch.

Can I create the PR for supporting this feature ?

No thanks. I might try dumb-jump myself at some point (I've been thinking about it recently, actually), but in general I won't add anything to the config that I'm not using myself. So, you might see support arrive at some point, but in the meantime it should be easy to add it to your local config.

Lan<EMAIL_ADDRESS>writes:

Thank you for your configuration, it is very helpful for me, Sometimes I write C programming language in Emacs, I find that it is hard for me to find the definition of the function. I think the
dumbjum can be a suitable choice for it. Can I create the PR for supporting this feature ?

This feature is built-in. Jumping to definition and back. It works for
elisp, c, c++, java and almost all programming languages.
I use these two key combinations: M-, M-.

@jangid Cool, It is a very useful feature. Another question, If we want to use the build in tags system we must generate the TAGS file manually first. Is there any plugins to make this process automatic ?

Lan<EMAIL_ADDRESS>writes:

want to use the build in tags system we must generate the TAGS file
manually first. Is there any plugins to make this process automatic ?

[[info:emacs#Xref][info:emacs#Xref]]
As per the above info page, etags is shipped with Emacs. And this is
automatically activated by major-modes of various programming
languages. I didn't have to manually generate TAGS.

@jangid Oh, let me try it, thank you very much.


:sparkles: Integrating release workflow - initial build, test, package, and upload artifacts to the ci runs.

This workflow builds the package, runs tests, and packages the extension. It also uploads it to the GH ci runs. It runs on merge and can be manually triggered on branches main and release-*. Supports linux, macos, windows.
ref issue: https://github.com/konveyor/editor-extensions/issues/11
todo:

[ ] Add Konveyor-analyzer binaries
[ ] Auto Publish it to the marketplace

Sample run - https://github.com/savitharaghunathan/editor-extensions/actions/runs/11111525940
I had to update the Readme as the vsce package fails if we use template generated readme file.
Note: I think the lint is spitting out too many warnings. Let me know if you all want to keep or remove the step.

Ref issue: https://github.com/konveyor/editor-extensions/issues/11

Looks good! One minor comment (see below) and one question: how long does the build take? If it's quick then we should just replace the current ci-repo workflow (which is running only Linux and finishes on package step). If it takes long (windows and macos!) then I would run it in nightly-ci-repo workflow (once a day).

@rszwajko it takes about ~2 mins to run all 3 builds. Sometimes the delay is because it is waiting for the runners.

@savitharaghunathan

it takes about ~2 mins to run all 3 builds. Sometimes the delay is because it is waiting for the runners.

Then I would go with extending our ci-repo script (we will re-use those cool status badges then :) )

@savitharaghunathan

it takes about ~2 mins to run all 3 builds. Sometimes the delay is because it is waiting for the runners.

Then I would go with extending our ci-repo script (we will re-use those cool status badges then :) )

@rszwajko I can do that. We can always separate it out later if things get complex.

@djzager ptal when you get some time. I am not able to test the release step on my fork. do you have any thoughts? Also, what do you mean by this -

Nice! A few formatting comments that may be impactful.

comments for steps?


Using "em-tab-panel" inside an each block

Hi!!! I'm trying to use ember-idx-tabs on my app.
My associations are as follow:
Practice ->* PracticePart -> Part
On the Practice show template I would like to show a tab for each PracticePart.
I came up with
{{#em-tabs selected-idx=tab_idx}}
    {{#em-tab-list}}
    	{{#each practicePart in practiceParts}}
	  {{#em-tab}}{{practicePart.part.symbol}}{{/em-tab}}
	{{/each}}
    {{/em-tab-list}}

    {{#each practicePart in practiceParts}}
      {{#em-tab-panel}}
        {{practicePart.part.symbol}}
      {{/em-tab-panel}}
    {{/each}}
{{/em-tabs}}

The problem is I'm getting only the tabs, the panels are all hidden (display: none).
If I dont iterate, but instead some em-tab-panels manually they are shown correctly and behave as expected.
I'm guessing its something I'm not doing well, cause I'm just learning ember.js, I'd appreciate any help!!!

My guess is the following.
ember-idx-tabs rely upon the view.parentView and view.parentView.parentView properties. When you create an intermediate view between tabs and tab list, the property chain breaks.

Hi lolmaus!! Thx for the answer!!! I dont understand very much how it works on the inside, but what I could find out is that in line https://github.com/indexiatech/ember-idx-tabs/blob/master/addon/tab-panel.js#L56, this.get('selected') is returning undefined.
Maybe it has to do with what you say, do you have any recomendation on how to work around or fix this? It looks like a pretty common usecase in my opinion.

Hi,
i think the source of the problem is the same as i had with adding and removing panels dynamically. I don't know exactly why this happens but with the each block the the panels and tabs are added then removed and readded again. And the panel removal doesen't work properly.
See pull request #10

I am having similar problem, using {{em-tab}} inside {{each}} helper. Adding new instance causes no problem, however, deleting an item from the array passed to the {{each}} helper causes:
return this.get('tabList.tab_instances').indexOf(this); //  Cannot read property 'indexOf' of null


Still actively supported?

Is it still advisable to try this out since it's last update a year back?

Recently updated to install NC26
Needs some testing. Been struggling with health issues, but expect updates, and an "upgrade" script soon.


Cannot pause any thread

If I click pause on one of the threads - only one concrete thread gets paused:

mono --version
Mono JIT compiler version <IP_ADDRESS>

.NET core debugger pauses all threads


Hide 'dismiss' button

Hi, it would be great to be able to hide the 'dismiss' button on the far left of Pock (the button with the 'cross' in it). I'm constantly hitting it, instead of the Esc button, as it's in the position where the Esc button is normally.
Thanks in advance!

If ControlStrip is visible that dismiss button can't be removed.
You can hide ControlStrip in Pock preferences to have the esc button on the far left of TouchBar.


error: ‘int MyCoolClass::secretX’ is private within this context

Hello,
I'm trying to create some complex object which would be cached from disk if the previous run of the program was successful.  The basic idea is implemented in the two enclosed file.
The idea: check if some file exists in the ctor. If yes, load the object from it; otherwise construct and dump the object. This is achieved by the call "archive (*this)"
Compilation with c++ (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0 as
c++ -g -c -o toto.o toto.cpp -std=c++11
results in
toto.cpp:15:28: error: ‘int MyCoolClass::secretX’ is private within this context
archive(m.x, m.y, m.z, m.secretX, m.enclosed);
The sources and full error messages are enclosed in a zip file. Any hint ?
Regards
Pascal
toto.zip

Code for other's reference:
#include <time.h>
#include <cereal/access.hpp>

class SimpleClass
{
public:
  explicit SimpleClass(struct tm const &);

private:
  struct tm date;
  SimpleClass();
  friend class MyCoolClass;
  friend class cereal::access;
};

class MyCoolClass
{
public:
  MyCoolClass(int x, int y, int z): x(x), y(y), z(z), secretX(-1) {};
  int x, y, z;

private:
  MyCoolClass();
  int secretX;
  SimpleClass enclosed;

  friend class cereal::access;
};

#include <cereal/archives/json.hpp>

#include "toto.hpp"

MyCoolClass::MyCoolClass(): secretX(0), enclosed()
{
  cereal::JSONOutputArchive archive(std::cout);
  archive(*this);
};

template<class Archive>
void save(Archive & archive,
          MyCoolClass const & m)
{
  archive(m.x, m.y, m.z, m.secretX, m.enclosed);
};

template<class Archive>
void load(Archive & archive,
          MyCoolClass & m)
{
  archive(m.x, m.y, m.z, m.secretX, m.enclosed);
};

template<class Archive>
void save(Archive & archive,
          SimpleClass const & m)
{
  archive(m.date);
};

template<class Archive>
void load(Archive & archive,
          SimpleClass & m)
{
  archive(m.date);
};

You've given a specific class within cereal (cereal::access) access to your member variables with the friend class cereal:;access, but you try to access the member variables directly in the non-member save function, which does not have access to the data. If you wanted this function to access those private members, you would need to have it explicitly listed as a friend as well. Check out the documentation of friend functions for more information.


[Bug] When TouchEffect.PressedScale is defined with a Command on a button, the command fire twice the first time.

Description
When TouchEffect.PressedScale is defined with a Command on a Button, the command fire twice the first time.
Steps to Reproduce

Define a button as such:
                <Button
                    xct:TouchEffect.PressedScale="1.1"
                    Command="{Binding ClickCommand}"
                    CommandParameter="{Binding ClickCommandParameter}"
                />

Click the button
The command fires 2 times the first time the app runs, subsequent clicks only fires once.

Expected Behavior

Version with issue:

Last known good version: Unknown

Platform Target Frameworks:

iOS:  14.5.1

I am using 1.3.0 pre-2
Workaround
Use xct:TouchEffect.Command instead.  Do not mix native Command with any TouchEffect functions
@AndreiMisiukevich

@mphill Hi! Thanks for your report
Does this issue happen on iOS only? Does it work fine on Android?

Yeah, confirm this bug. But unfortunately, it's impossible to fix that.
I propose using a workaround for Buttons (the same issue https://github.com/xamarin/XamarinCommunityToolkit/issues/1428)


Implement CHIP platform lock for Android.

Problem
CHIP platform lock is needed to guard chip stack against racing. But device layer is not implemented for Android.
Possible Solution
Implement device layer for android.

We have a lock: https://github.com/project-chip/connectedhomeip/blob/15669b66d32369462d0079649b8edc698e327941/src/controller/java/JniReferences.h#L81


[Candidate 1v1 Battleship Bay ZH v1] Mirrored house near Battleship has 2 less slots on one side

Observed behavior
Mirrored house near Battleship has 2 less slots on one side.
Expected behavior
Mirrored houses have equal amount of slots for fairness.
Screenshots

tried changing the house but as the textures have already been done under and to the side of it, its quite a mission to re-do it all. As its minor, leaving this.

fixed.


Error on build

I am getting the following error on build:
Android_HCE_Emulate_A_CreditCard/app/src/main/java/de/androidcrypto/android_hce_emulate_a_creditcard/HceCcEmulationServiceBasic.java:45: Error: The intent action HceCcEmulatorService (used to send a broadcast) matches the intent filter of a non-exported receiver, registered via a call to Context.registerReceiver, or similar. If you are trying to invoke this specific receiver via the action then you should use Intent.setPackage(<APPLICATION_ID>). [UnsafeImplicitIntentLaunch]
Intent intent = new Intent("HceCcEmulatorService");
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

It can be fixed by adding this line under Intent intent = new Intent("HceCcEmulatorService"):
intent.setPackage(getPackageName());


Sec/NLA: Support passwordless (blank password) login with NLA.

It was supported in freerdp 1.0.2 but not supported in lastest master.
We should take empty password if it is explicitly specified with /p option.
If a password is not specified, we could first try SAM file. If the user entry does not exist, prompt for password.
Fixes #2783

@realjiangms you mean if it is specified with /p:"" ?

Yeah should be /p: or /p:''

First prepare an Win7 with rdp enabled and empty password allowed.
Then make /etc/winpr/SAM with wrong password.
winpr/tools/hash/winpr-hash -f sam -u [username] -p 'non-empty' -v 1 > /etc/winpr/SAM
Tested with following cases:

Specify empty password in command line
xfreerdp /cert-ignore /u:[username] /p:'' /v:[server ip]
or
xfreerdp /cert-ignore /u:[username] /p: /v:[server ip]

Connected successfully.

Not specify password in command line:
xfreerdp /cert-ignore /u:[username] /v:[server ip]
It failed with ERRCONNECT_CONNECT_TRANSPORT_FAILED. This is expected

Not specify password in command line. Fix SAM with empty password
winpr/tools/hash/winpr-hash -f sam -u [username] -p '' -v 1 > /etc/winpr/SAM
xfreerdp /cert-ignore /u:[username] /v:[server ip]

Connected successfully.

Remove /etc/winpr/SAM file, not specify password in command line:
rm  /etc/winpr/SAM
xfreerdp /cert-ignore /u:[username] /v:[server ip]

It report 'Could not open SAM file!'
It then prompt for 'password:'.
Press entry directly and then connected successfully.

@realjiangms looks you are our hero on this issue !


Refactor 2SV sign in routes

Moves the 2SV sign in routes into a new controller and makes them more
like Devise, with the concept of a 2SV session. two_step_verification_session#new and
#create are now used for displaying the 2SV code entry screen
and validating the code respectively.
This frees up the two_step_verification resource itself to allow #update
to be used to change the device used for 2SV.
/cc @jamiecobbett @benilovj @fofr @benlovell

LGTM! :shipit:

:top: refactoring :+1:


drivers/reset: Add MEC172x peripheral reset driver

Individual IP blocks in MEC172x SoC can be reset independently with an internal reset controller in the PCR block.  Resets may not be required for any particular block but this driver allows this functionality to be included easily if necessary.

@wwang22 @pmcfall2

we prefer the clock control driver owning the resetting of the peripherals. This draft PR has the example 56044
This way the bit positions and register indexes for the peripherals come from the DTS instead of soc headers as in this PR

we prefer the clock control driver owning the resetting of the peripherals. This draft PR has the example 56044
This way the bit positions and register indexes for the peripherals come from the DTS instead of soc headers as in this PR

the intention is to have the bits/registers to be enumerated via the DTS.  the 2nd part of this change adds this mechanism to the SoC DTS as such (dts/arm/microchip/mec172xnsz.dsti):
i2c0_smb_0: i2c@40004000 {
compatible = "microchip, xec-i2c-v2";
reg = <0x40004000 0x80>;
....
resets = <&rctl MEC172X_RESET_I2C0>;    this is where the connection to the rctl node for resets are made
};


SecondAppBarLayoutBehavior improvement

I re-developed the SecondAppBarLayoutBehavior to mimic the exact Google behavior. I'll clean the code and push it as soon as possible. Here is a preview.

wow nice! let me know if you need something :D, but like I said in a previous post I a little bussy this week.


Clarifies how to view messages from the web view

Not entirely sure if this is the best place to put it but this is the file where I went hunting for sending messages.

that's a good enough place :)


If media file format is WEBM and no audio OR video format is selected, then --merge-output-format is ignored and --embed-thumbnail FAILS

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I 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.01.06 (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 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
If media file format is WEBM and no audio OR video format is selected, --merge-output-format is ignored and --embed-thumbnail FAILS
Command:
yt-dlp --verbose --embed-thumbnail --format "598" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"
Downloads thumbnail as .webp, downloads video as .webm, converts .webp to .png, then FAILS with ExitCode=1:

ERROR: Postprocessing: Supported filetypes for thumbnail embedding are: mp3, mkv/mka, ogg/opus/flac, m4a/mp4/mov

(Verbose Output see "Complete Verbose Output")
It also fails, if only one audio format and no video is selected, e.g. --format "bestaudio"
It also fails, if --merge-output-format="mp4" is set:
yt-dlp --verbose --embed-thumbnail --format "598" --merge-output-format="mp4" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"

ERROR: Postprocessing: Supported filetypes for thumbnail embedding are: mp3, mkv/mka, ogg/opus/flac, m4a/mp4/mov

However, it works as expected when video AND audio format is selected. yt-dlc then shows a warning and selects .mkv:
yt-dlp --verbose --embed-thumbnail --format "bestvideo+bestaudio" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"
Downloads thumbnail as .webp, downloads video and audio as .webm, converts .webp to .png, then muxes everything into .mkv:

WARNING: webm doesn't support embedding a thumbnail, mkv will be used

When selecting MP4 as media file format --embed-thumbnail and --merge-output-format works as expected (

Additional Tests:

Fails (--merge-output-format="mp4" is ignored):
yt-dlp --verbose --embed-thumbnail --format "bestvideo" --merge-output-format="mp4" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"

ERROR: Postprocessing: Supported filetypes for thumbnail embedding are: mp3, mkv/mka, ogg/opus/flac, m4a/mp4/mov

Fails (downloads video, BUT ignores --merge-output-format="mkv" and outputs .webm, no error/warning):
yt-dlp --verbose --format "315" --merge-output-format="mkv" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"

[download] Destination: _lyIjUrJ4f0.f-315.webm

Works (downloads video only, output is .webm):
yt-dlp --verbose --format "bestvideo" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"

Works (downloads video and audio, merges into .mkv):
yt-dlp --verbose --format "bestvideo+worstaudio" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"

Works (output is .mp4):
yt-dlp --verbose --format "bestvideo+worstaudio" --merge-output-format="mp4" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"

Format List for https://www.youtube.com/watch?v=_lyIjUrJ4f0:
====================================================================
| Electric Sheep Flame Fractals [Generation:248 ID:00227 to        |
| 00253] [8K 4K HD 60FPS EDGE]                                     |
| ---------------------------------------------------------------- |
| Uploader: Aquaritek                                              |
| Upload Date: 2022-09-25                                          |
| Duration: 00:00:10                                               |
| Views:7/Like:0/Dislike:0/                                        |
| https://www.youtube.com/watch?v=_lyIjUrJ4f0                      |
====================================================================
| sb0 | storyboard |      48x27 |           |              | mhtml |
| 251 |     medium | audio only |   4,71 KB |     3 kBit/s |  webm |
| 598 |       144p |    256x144 |  29,85 KB |    24 kBit/s |  webm |
| 597 |       144p |    256x144 |  36,24 KB |    29 kBit/s |   mp4 |
| 599 |   ultralow | audio only |  38,80 KB |    31 kBit/s |   m4a |
| 139 |        low | audio only |  61,05 KB |    49 kBit/s |   m4a |
| 278 |       144p |    256x144 |  74,87 KB |    61 kBit/s |  webm |
| 160 |       144p |    256x144 |  97,75 KB |    80 kBit/s |   mp4 |
| 242 |       240p |    426x240 | 152,33 KB |   124 kBit/s |  webm |
| 140 |     medium | audio only | 159,66 KB |   130 kBit/s |   m4a |
| 133 |       240p |    426x240 | 219,52 KB |   179 kBit/s |   mp4 |
| 243 |       360p |    640x360 | 311,82 KB |   255 kBit/s |  webm |
| 134 |       360p |    640x360 | 514,60 KB |   421 kBit/s |   mp4 |
|  18 |       360p |    640x360 |           |   551 kBit/s |   mp4 |
| 244 |       480p |    854x480 | 680,60 KB |   557 kBit/s |  webm |
| 135 |       480p |    854x480 |   1,05 MB |   878 kBit/s |   mp4 |
| 247 |       720p |   1280x720 |   2,07 MB |  1736 kBit/s |  webm |
| 136 |       720p |   1280x720 |   2,42 MB |  2026 kBit/s |   mp4 |
|  22 |       720p |   1280x720 |           |  2155 kBit/s |   mp4 |
| 302 |     720p60 |   1280x720 |   3,16 MB |  2652 kBit/s |  webm |
| 298 |     720p60 |   1280x720 |   3,69 MB |  3093 kBit/s |   mp4 |
| 303 |    1080p60 |  1920x1080 |   6,46 MB |  5418 kBit/s |  webm |
| 299 |    1080p60 |  1920x1080 |   7,18 MB |  6026 kBit/s |   mp4 |
| 308 |    1440p60 |  2560x1440 |  15,57 MB | 13062 kBit/s |  webm |
| 315 |    2160p60 |  3840x2160 |  32,16 MB | 26981 kBit/s |  webm |
====================================================================

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp --verbose --embed-thumbnail --format "598" -o "%(id)s.f-%(format_id)s.%(ext)s" "https://www.youtube.com/watch?v=_lyIjUrJ4f0"
[debug] Command-line config: ['--verbose', '--embed-thumbnail', '--format', '598', '-o', '%(id)s.f-%(format_id)s.%(ext)s', 'https://www.youtube.com/watch?v=_lyIjUrJ4f0']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2023.01.06 [6becd25] (zip)
[debug] Python 3.10.7 (CPython aarch64 64bit) - Linux-5.19.0-1011-raspi-aarch64-with-glibc2.36 (OpenSSL 3.0.5 5 Jul 2022, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.1 (setts), ffprobe 5.1.1
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1760 extractors
[youtube] Extracting URL: https://www.youtube.com/watch?v=_lyIjUrJ4f0
[youtube] _lyIjUrJ4f0: Downloading webpage
[youtube] _lyIjUrJ4f0: Downloading android player API JSON
[debug] Loading youtube-nsig.4248d311 from cache
[debug] [youtube] Decrypted nsig cbx3qOnByjDdXQZ => AUdD5oe5i0kbcA
[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, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[info] _lyIjUrJ4f0: Downloading 1 format(s): 598
[info] Downloading video thumbnail 46 ...
[info] Writing video thumbnail 46 to: _lyIjUrJ4f0.f-598.webp
[debug] Invoking http downloader on "https://rr2---sn-5hne6nzd.googlevideo.com/videoplayback?expire=1674951252&ei=9GXVY6bfOs6FgAfgpq6IDg&ip=2a02%3A908%3Ae39%3A3a60%3A%3A84&id=o-AOKcqFufnQkeVMq5xPFTwq6H0T5jl1fm_ib7Wa4_x-7q&itag=598&source=youtube&requiressl=yes&mh=ZV&mm=31%2C29&mn=sn-5hne6nzd%2Csn-5hnekn7z&ms=au%2Crdu&mv=m&mvi=2&pl=44&initcwndbps=1698750&spc=H3gIhhX2JkzP6sV6y76Z5pYzGrPTaxA&vprv=1&svpuc=1&mime=video%2Fwebm&gir=yes&clen=30569&dur=10.000&lmt=1664090359206141&mt=1674929610&fvip=1&keepalive=yes&fexp=24007246&c=ANDROID&txp=6219224&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhAM8UiEI_lbiN_Qz1dYry-NipslfWoh-fQD6uNEtxEBCuAiA_0yAfLt06kGQZsDOCQYQMbPp4kk0e_NMnbujOBqgBcg%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIgA9DlKZ6r7BaFpo_CUQhqpRbqrxCSgm2ZsfTdnTYF7oUCIQDc8jxH0yO_M9s_fez_MFlqu8fexjy5gF8IL7BRYAtERw%3D%3D"
[download] Destination: _lyIjUrJ4f0.f-598.webm
[download] 100% of   29.85KiB in 00:00:00 at 61.90KiB/s
[ThumbnailsConvertor] Converting thumbnail "_lyIjUrJ4f0.f-598.webp" to png
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -f image2 -pattern_type none -i file:_lyIjUrJ4f0.f-598.webp -update 1 -movflags +faststart file:_lyIjUrJ4f0.f-598.png
ERROR: Postprocessing: Supported filetypes for thumbnail embedding are: mp3, mkv/mka, ogg/opus/flac, m4a/mp4/mov
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3312, in process_info
    replace_info_dict(self.post_process(dl_filename, info_dict, files_to_move))
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3491, in post_process
    info = self.run_all_pps('post_process', info, additional_pps=info.get('__postprocessors'))
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3473, in run_all_pps
    info = self.run_pp(pp, info)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3451, in run_pp
    files_to_delete, infodict = pp.run(infodict)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/common.py", line 24, in run
    ret = func(self, info, *args, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/common.py", line 129, in wrapper
    return func(self, info)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/embedthumbnail.py", line 216, in run
    raise EmbedThumbnailPPError('Supported filetypes for thumbnail embedding are: mp3, mkv/mka, ogg/opus/flac, m4a/mp4/mov')
yt_dlp.postprocessor.embedthumbnail.EmbedThumbnailPPError: Supported filetypes for thumbnail embedding are: mp3, mkv/mka, ogg/opus/flac, m4a/mp4/mov

yt-dlp -vU
[debug] Command-line config: ['-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2023.01.06 [6becd25] (zip)
[debug] Python 3.10.7 (CPython aarch64 64bit) - Linux-5.19.0-1011-raspi-aarch64-with-glibc2.36 (OpenSSL 3.0.5 5 Jul 2022, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.1 (setts), ffprobe 5.1.1
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1760 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2023.01.06, Current version: 2023.01.06
yt-dlp is up to date (2023.01.06)

from the readme:
--merge-output-format FORMAT    Containers that may be used when merging
                                formats, separated by "/", e.g. "mp4/mkv".
                     -------->  Ignored if no merge is required. (currently
                                supported: avi, flv, mkv, mov, mp4, webm)

so the option behaves as expected, and the error message given is appropriate:

ERROR: Postprocessing: Supported filetypes for thumbnail embedding are: mp3, mkv/mka, ogg/opus/flac, m4a/mp4/mov

solution: you can use --remux-video
--remux-video FORMAT            Remux the video into another container if
                                necessary (currently supported: avi, flv,
                                gif, mkv, mov, mp4, webm, aac, aiff, alac,
                                flac, m4a, mka, mp3, ogg, opus, vorbis,
                                wav). If target container does not support
                                the video/audio codec, remuxing will fail.
                                You can specify multiple rules; e.g.
                                "aac>m4a/mov>mp4/mkv" will remux aac to m4a,
                                mov to mp4 and anything else to mkv


Expiration Bug

Hi
Thanks for this good example, helped a lot.
I think I found a bug with the expiration date at
https://github.com/EricKit/nest-user-auth/blob/cf33a7743af18ecf8def2d4810b3d915e25d2650/src/auth/auth.service.ts#L108
setSeconds does only set the second portion of a Date object. This would result in an unexpected date.
See explanation about the same with setMilliseconds here: https://stackoverflow.com/a/12795802
Also https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/setSeconds
I think the simplest would be to use getTime() and setTime():
expiration.setTime(expiration.getTime() + expiresIn * 1000);

or
expiration = new Date(expiration.getTime() + expiresIn * 1000));

Great catch, thanks!


Check and fix any incorrect key bindings

Shift-J/K are documented as being used to navigate between cells but are not working (at least on a Mac).
We need to go through all the documented key bindings and check which ones don't work and fix them or the documentation.

It's the Ctrl-J/K documentation that maybe incorrect, on the dropdown KeyBoard Shortcuts.


How to switch color theme in examples?

I want to see inline notification examples using other color themes. I don't find theme switch in component itself https://www.carbondesignsystem.com/components/notification/code
, in themes and colors description, in code pen, in frequently asked questions.
Are there any ways to see examples with light theme without installing carbon environment?

Hi @NtsDK this is definitely something we are looking at adding to the website in the future!  In the meantime you can view the components with different themes here, this is just a live demo of our dev testing environment. https://the-carbon-components.netlify.com/?nav=accordion

@alisonjoseph Thank you for help :) This link is what I looked for :)


Localization crash due to reflection on net 8

Describe the bug
The following crash occurs on net 8-rc 1.
 ---> System.InvalidOperationException: Reflection-based serialization has been disabled for this application. Either use the source generator APIs or explicitly configure the 'JsonSerializerOptions.TypeInfoResolver' property.
   at System.Text.Json.ThrowHelper.ThrowInvalidOperationException_JsonSerializerIsReflectionDisabled()
   at System.Text.Json.JsonSerializerOptions.ConfigureForJsonSerializer()
   at System.Text.Json.JsonSerializer.GetTypeInfo(JsonSerializerOptions options, Type inputType)
   at System.Text.Json.JsonSerializer.GetTypeInfo[T](JsonSerializerOptions options)
   at System.Text.Json.JsonSerializer.Deserialize[TValue](Stream utf8Json, JsonSerializerOptions options)
   at FluentAvalonia.UI.FALocalizationHelper..ctor()
   at FluentAvalonia.UI.FALocalizationHelper..cctor()
   --- End of inner exception stack trace ---

This can be triggered on loading a NavigationView.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop/Platform (please complete the following information):

OS: Windows 11
FluentAvalonia Version 2.0.1
Avalonia Version 11.0.0

Additional context
Using dotnet 8.0 RC1

Workaround for now is set <JsonSerializerIsReflectionEnabledByDefault>true</JsonSerializerIsReflectionEnabledByDefault>

It looks like as soon as .NEt 8 is released, adding net8.0 to the TargetFrameworks would solve this problem. The other workaround might if the app explicitly PackageReference System.Text.Json and force explicitly to use the newer  STJ instead of using STJ 6.0.x


Don't clean away headers generated by Configure

Yep, seems to be working. Merge it! ;)

+1

merged.  Thanks!


Data types are no longer highlighted

Problem description
Java data type keywords are no longer highlighted correctly.

This applies even for void functions.

As this is the grammar seemingly used by Linguist, this affects all Java repositories on GitHub.
Expected parse tree output (optional)
N/A.
Actual parse tree output (optional)
N/A.

Hello? Anyone?

this is also a keyword and is not highlighted

Seems like a potential editor issue (if it is, that's on their queries, not here)


Small fixes

Let upper limit of framewise displacement axis scale with data
Allow numeric condition names without failing
Avoid deprecation warning about boolean indices

Codecov Report

Merging #186 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #186   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     10
  Lines        2179   2179
=====================================
  Hits         2179   2179

Impacted Files
Coverage Δ

lyman/visualizations.py
100% <100%> (ø)
:arrow_up:

lyman/glm.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add assertion for required option 'code' on ol.proj.Projection

When using proj4js together with ol3 and instantiation of
ol.proj.Projection is made, the code property is required.
An empty string is enough though.
The reason can be found in the class ol.proj.Projection, starting in line 145:
https://github.com/openlayers/ol3/blob/master/src/ol/proj/proj.js#L145-L147
var code = options.code;
  if (ol.ENABLE_PROJ4JS && typeof proj4 == 'function' &&
      !goog.isDef(projections[code])) {
...

Maybe an additional check if code is defined would make sense?
Problem can be observed in
http://openlayers.org/en/master/examples/wms-image-custom-proj.html
by entering
new ol.proj.Projection({
    units: 'pixels'
});

in the console.

This note should not be necessary, because "code" is not optional. Otherwise the type would be noted as {string|undefined}.

I disagree. We should have a note in the docs stating it is not optional.
For example on http://openlayers.org/en/v3.8.2/examples/attributions.html (no proj4js)
new ol.proj.Projection({});

works fine, while on http://openlayers.org/en/v3.8.2/examples/wms-image-custom-proj.html (with proj4js)
new ol.proj.Projection({});

yields an Error:
Uncaught TypeError: Cannot use 'in' operator to search for 'EPSG' in undefined
    at Function.d [as defs] (http://cdnjs.cloudflare.com/ajax/libs/proj4js/2.3.6/proj4.js:1:15574)
    at new ye (http://openlayers.org/en/v3.8.2/build/ol.js:64:350)
    at <anonymous>:2:1
    at Object.InjectedScript._evaluateOn (<anonymous>:905:140)
    at Object.InjectedScript._evaluateAndWrap (<anonymous>:838:34)
    at Object.InjectedScript.evaluate (<anonymous>:694:21)

because we don't check for undefined before passing it to proj4.defs(code).
We may change the wording, but a note should be there IMO.

When a constructor option is optional, it has the "undefined" type. This is not the case for "code". This is consistent across all our type definition in the API docs.

I know that. But when we pass undefined to proj4.defs the error arises.
I cannot see why adding this documentation line causes so much discussion.

should we not just protect against passing off undefined to proj4.defs?

+1 on @bartvde's suggestion. I'm going to create a pull request.

I'd go even further and add an assertion, if the code is not optional as the type definition states:
goog.asserts.assert(code !== undefined,
    'Option "code" is required for constructing instance');

mayxbe the OP can create the PR? So more people play this game?

Happy with that. It's just about adding goog.asserts.assert... in 1 or 2 places.

Note to @weskamm: If you consider @marcjansen's snippet above, it should be goog.isDef(code), not code !== undefined.

I added the assertion as suggested.
@ahocevar: will you have another look?
Thanks everybody.

Thanks @weskamm. I meant to add the assertion and to remove the change to olx.js. The reason is that I don't know where we would stop when we start adding "Required" to all options in olx.js that do not have "undefined" in their type.

Agreed on the removal of the 'required' in the docs (but only for consistency).
I think (but this is out of the scope of this PR) we should have the required options be marked as required in prose everywhere.
Thanks to @weskamm for the multiple stabs he does at getting this better, and to @ahocevar for pushing  his idea of consistency.

Thanks! Will merge as soon as Travis says ok.

I removed the comment from the doc. Although i think it would be more userfriednly having it in..

We can change the apidoc template to mark all required options as required automatically. No need to do this in prose for every required option.

Nice, @ahocevar. I was thinking the same. Will you do it or shall I try?

If you try @marcjansen it would be great.