Youtube: UnicodeDecodeError: 'ascii' codec can't decode byte 0xb9 in position 21: ordinal not in range(128)

Hello, I tried using youtube-dl installed from chocolatey to download a playlist from Youtube.
I am constantly getting errors thrown while downloading. Impossible to leave the program on for a longer while as it crashes within an hour of starting.
Youtube-dl's version in this report is 2015-03-28, but I tried doing this again in the 2015-04-03. It continued the download on the first episode and finished, downloaded the second episode without error, and crashed while downloading the third, with the same error message at 6.1% into the file.
Logs included with a couple download retries, as well as ffmpeg --version, also installed from Chocolatey.
Sorry for the awful line wrapping, but the Windows console doesn't want to work well with longer lines of text.
youtube-dl "https://www.youtube.com/playlist?list=PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham" -o "D:\Youtube\%(uploader)s\%(upload_date)s %(title)s - %(id)s\%(title)s [%(forma
t)s].%(ext)s" --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://www.youtube.com/playlist?list=PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham', u'-o', u'D:\\Youtube\\%(uploader)s\\%(upload_date)s %(title)s
- %(id)s\\%(title)s [%(format)s].%(ext)s', u'--verbose']
[debug] Encodings: locale cp1250, fs mbcs, out cp852, pref cp1250
[debug] youtube-dl version 2015.03.28
[debug] Python version 2.7.8 - Windows-8-6.2.9200
[debug] exe versions: ffmpeg 2.5.2, ffprobe 2.5.2
[debug] Proxy map: {}
[youtube:playlist] PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham: Downloading webpage
[download] Downloading playlist: Cities Skylines: Lets Play With Xisuma
[youtube:playlist] playlist Cities Skylines: Lets Play With Xisuma: Collected 32 video ids (downloading 32 of them)
[download] Downloading video 1 of 32
[youtube] p8SwN4L12m0: Downloading webpage
[youtube] p8SwN4L12m0: Extracting video information
[youtube] {22} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {43} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {18} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {5} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {36} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {17} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {298} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {302} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {135} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {244} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {134} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {243} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {133} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {242} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {160} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {140} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {171} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] p8SwN4L12m0: Downloading DASH manifest
[debug] Invoking downloader on u'https://r5---sn-5uh5o-f5fz.googlevideo.com/videoplayback?requiressl=yes&mime=video%2Fmp4&ip=<IP_ADDRESS>&ratebypass=yes&ipbits=0&spara
ms=dur%2Cgcr%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Cmime%2Cmm%2Cms%2Cmv%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&id=o-AE1VFwjmpLvzEnTMT6t4cWSjwaXLXFbN
72jWzVLD3o1W&ms=au&gcr=pl&mv=m&dur=1226.849&key=yt5&sver=3&expire=1428092295&source=youtube&pl=20&fexp=900720%2C905652%2C907263%2C924637%2C934954%2C9407565%2C9407663%2
C9407758%2C9408102%2C9408192%2C947243%2C948124%2C948605%2C948703%2C951703%2C952612%2C952634%2C952635%2C957201%2C961404%2C961406%2C966201&upn=0PUoXLAewoc&initcwndbps=10
25000&itag=22&mm=31&mt=1428070594&signature=3555A4150B277760499F90348DC7426583B727D9.C2E5B435CF2CA4A9DDDD2830695C69EBA2909397'
[download] Destination: D:\Youtube\xisumavoid\20150303 Cities Skylines 01 Mayor Xisuma - p8SwN4L12m0\Cities Skylines 01 Mayor Xisuma [22 - 1280x720].mp4
[download]  17.1% of 213.91MiB at 46.31KiB/s ETA 01:05:21Traceback (most recent call last):
  File "__main__.py", line 19, in <module>
  File "youtube_dl\__init__.pyo", line 408, in main
  File "youtube_dl\__init__.pyo", line 398, in _real_main
  File "youtube_dl\YoutubeDL.pyo", line 1446, in download
  File "youtube_dl\YoutubeDL.pyo", line 662, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 821, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 715, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 662, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 708, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 1150, in process_video_result
  File "youtube_dl\YoutubeDL.pyo", line 1381, in process_info
UnicodeDecodeError: 'ascii' codec can't decode byte 0xb9 in position 21: ordinal not in range(128)

C:\Users\Mateusz>youtube-dl "https://www.youtube.com/playlist?list=PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham" -o "D:\Youtube\%(uploader)s\%(upload_date)s %(title)s - %(id)s\%
(title)s [%(format)s].%(ext)s" --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://www.youtube.com/playlist?list=PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham', u'-o', u'D:\\Youtube\\%(uploader)s\\%(upload_date)s %(title)s
- %(id)s\\%(title)s [%(format)s].%(ext)s', u'--verbose']
[debug] Encodings: locale cp1250, fs mbcs, out cp852, pref cp1250
[debug] youtube-dl version 2015.03.28
[debug] Python version 2.7.8 - Windows-8-6.2.9200
[debug] exe versions: ffmpeg 2.5.2, ffprobe 2.5.2
[debug] Proxy map: {}
[youtube:playlist] PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham: Downloading webpage
[download] Downloading playlist: Cities Skylines: Lets Play With Xisuma
[youtube:playlist] playlist Cities Skylines: Lets Play With Xisuma: Collected 32 video ids (downloading 32 of them)
[download] Downloading video 1 of 32
[youtube] p8SwN4L12m0: Downloading webpage
[youtube] p8SwN4L12m0: Extracting video information
[youtube] {22} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {43} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {18} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {5} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {36} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {17} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {298} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {302} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {135} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {244} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {134} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {243} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {133} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {242} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {160} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {140} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {171} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] p8SwN4L12m0: Downloading DASH manifest
[debug] Invoking downloader on u'https://r5---sn-5uh5o-f5fz.googlevideo.com/videoplayback?initcwndbps=1056250&gcr=pl&upn=bRIZmCq46C4&source=youtube&sver=3&dur=1226.849
&ipbits=0&expire=1428093130&ratebypass=yes&requiressl=yes&ip=<IP_ADDRESS>&ms=au&mime=video%2Fmp4&sparams=dur%2Cgcr%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Cmime%2Cmm%
2Cms%2Cmv%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&mv=m&mt=1428071434&pl=20&key=yt5&id=o-AACaaCZd-oERN2_38M9MVE2_qxedItWtl_-iEq0DqTdE&mm=31&itag=22&fexp=
900720%2C907263%2C934954%2C937407%2C9406089%2C9406921%2C9407751%2C9407873%2C9408101%2C947243%2C948124%2C948703%2C951703%2C952612%2C957201%2C961404%2C961406&signature=4
6F20AF7C3F6DAED68B3D60681E1D5BAACBF3496.18404837E8F76C046064A12C5D21B049B6ED5D6F'
[download] Resuming download at byte 38343073
[download] Destination: D:\Youtube\xisumavoid\20150303 Cities Skylines 01 Mayor Xisuma - p8SwN4L12m0\Cities Skylines 01 Mayor Xisuma [22 - 1280x720].mp4
[download]  29.7% of 213.91MiB at 243.36KiB/s ETA 10:32  Traceback (most recent call last):
  File "__main__.py", line 19, in <module>
  File "youtube_dl\__init__.pyo", line 408, in main
  File "youtube_dl\__init__.pyo", line 398, in _real_main
  File "youtube_dl\YoutubeDL.pyo", line 1446, in download
  File "youtube_dl\YoutubeDL.pyo", line 662, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 821, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 715, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 662, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 708, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 1150, in process_video_result
  File "youtube_dl\YoutubeDL.pyo", line 1381, in process_info
UnicodeDecodeError: 'ascii' codec can't decode byte 0xb9 in position 21: ordinal not in range(128)

C:\Users\Mateusz>youtube-dl "https://www.youtube.com/playlist?list=PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham" -o "D:\Youtube\%(uploader)s\%(upload_date)s %(title)s - %(id)s\%
(title)s [%(format)s].%(ext)s" --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://www.youtube.com/playlist?list=PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham', u'-o', u'D:\\Youtube\\%(uploader)s\\%(upload_date)s %(title)s
- %(id)s\\%(title)s [%(format)s].%(ext)s', u'--verbose']
[debug] Encodings: locale cp1250, fs mbcs, out cp852, pref cp1250
[debug] youtube-dl version 2015.03.28
[debug] Python version 2.7.8 - Windows-8-6.2.9200
[debug] exe versions: ffmpeg 2.5.2, ffprobe 2.5.2
[debug] Proxy map: {}
[youtube:playlist] PL7VmhWGNRxKgwY_fiNKeleGxVlZOG6ham: Downloading webpage
[download] Downloading playlist: Cities Skylines: Lets Play With Xisuma
[youtube:playlist] playlist Cities Skylines: Lets Play With Xisuma: Collected 32 video ids (downloading 32 of them)
[download] Downloading video 1 of 32
[youtube] p8SwN4L12m0: Downloading webpage
[youtube] p8SwN4L12m0: Extracting video information
[youtube] {22} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {43} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {18} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {5} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {36} signature length 42.38, html5 player en_US-vfldudhuW
[youtube] p8SwN4L12m0: Downloading player https://s.ytimg.com/yts/jsbin/html5player-en_US-vfldudhuW/html5player.js
[youtube] {17} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {298} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {302} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {135} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {244} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {134} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {243} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {133} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {242} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {160} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {140} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] {171} signature length 42.40, html5 player en_US-vfldudhuW
[youtube] p8SwN4L12m0: Downloading DASH manifest
[debug] Invoking downloader on u'https://r5---sn-5uh5o-f5fz.googlevideo.com/videoplayback?sparams=dur%2Cgcr%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Cmime%2Cmm%2Cms%2C
mv%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&gcr=pl&mm=31&ip=<IP_ADDRESS>&ms=au&requiressl=yes&pl=20&itag=22&ratebypass=yes&mt=1428071936&dur=1226.849&sou
rce=youtube&mime=video%2Fmp4&id=o-AOhCN4bSBE0QoSEAoRUxzp3TjJo7iD2p7eDpX-SfO5Bw&initcwndbps=918750&ipbits=0&expire=1428093604&fexp=900720%2C902543%2C907263%2C934954%2C9
36118%2C9405627%2C9406795%2C9406923%2C9408093%2C9408101%2C9408260%2C9408344%2C947243%2C948124%2C948703%2C951703%2C952612%2C957201%2C961404%2C961406%2C966201&sver=3&upn
=xCuwXgcypis&key=yt5&mv=m&signature=AFBC7D6DE990F241FBC81D8247E0639AA5E1F94A.AB0A89029684FA0039590A117ED8A099CAE5D484'
[download] Resuming download at byte 66562353
[download] Destination: D:\Youtube\xisumavoid\20150303 Cities Skylines 01 Mayor Xisuma - p8SwN4L12m0\Cities Skylines 01 Mayor Xisuma [22 - 1280x720].mp4
[download]  93.8% of 213.91MiB at 74.50KiB/s ETA 03:02   Traceback (most recent call last):
  File "__main__.py", line 19, in <module>
  File "youtube_dl\__init__.pyo", line 408, in main
  File "youtube_dl\__init__.pyo", line 398, in _real_main
  File "youtube_dl\YoutubeDL.pyo", line 1446, in download
  File "youtube_dl\YoutubeDL.pyo", line 662, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 821, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 715, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 662, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 708, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 1150, in process_video_result
  File "youtube_dl\YoutubeDL.pyo", line 1381, in process_info
UnicodeDecodeError: 'ascii' codec can't decode byte 0xb9 in position 21: ordinal not in range(128)

C:\Users\Mateusz>

C:\Users\Mateusz>ffmpeg --version
ffmpeg version 2.5.2 Copyright (c) 2000-2014 the FFmpeg developers
  built on Dec 30 2014 17:18:09 with gcc 4.9.2 (GCC)
  configuration: --disable-static --enable-shared --enable-gpl --enable-version3 --disable-w32threads --enable-avisynth --enable-bzlib --ena
ble-fontconfig --enable-frei0r --enable-gnutls --enable-iconv --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-
libfreetype --enable-libgme --enable-libgsm --enable-libilbc --enable-libmodplug --enable-libmp3lame --enable-libopencore-amrnb --enable-lib
opencore-amrwb --enable-libopenjpeg --enable-libopus --enable-librtmp --enable-libschroedinger --enable-libsoxr --enable-libspeex --enable-l
ibtheora --enable-libtwolame --enable-libvidstab --enable-libvo-aacenc --enable-libvo-amrwbenc --enable-libvorbis --enable-libvpx --enable-l
ibwavpack --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxavs --enable-libxvid --enable-lzma --enable-decklink --enable-zlib

  libavutil      54. 15.100 / 54. 15.100
  libavcodec     56. 13.100 / 56. 13.100
  libavformat    56. 15.102 / 56. 15.102
  libavdevice    56.  3.100 / 56.  3.100
  libavfilter     5.  2.103 /  5.  2.103
  libswscale      3.  1.101 /  3.  1.101
  libswresample   1.  1.100 /  1.  1.100
  libpostproc    53.  3.100 / 53.  3.100
Unrecognized option '-version'.
Error splitting the argument list: Option not found

Oops, did --version instead of -version, but the output is basically the same, apart from the last two lines.

I have more information on this issue. I believe this is caused by the internet cutting out, or just being unstable. Instead of stopping and retrying the download it crashes (with the same message). I might try to reproduce the bug again if that's needed when I have the time.

This issue has been fixed quite some time ago.


Add FFI for sgx_status_t

Generates the FFI bindings for the sgx_status_t. This binding uses the
newtype_enum to ensure an enum per possible value while preventing
undefined behavior if an unknown return type is provided.

attaching bindings for the sgx_status_t value
bindings.txt

We can dump the mc-sgx-core-sys crate, otherwise looks good---we'll want to add additional types to this crate, but this is a good start.

I'll drop the mc-sgx-core-sys crate, think i was just copying the other layouts, but it is just noise here.
As for the additional types my thought was to add them on an as needed bases, similar to https://github.com/mobilecoinfoundation/sgx/pull/2/files#diff-8c75269a3c404dce7baf3463401473b295282e8fb022b384144bb350e256b3a1

Also, you need to set the C-types location, it's putting std::os::raw::c_uint into sgx_status_t


Fix Vonage::Voice::Ncco class method definition

Hello here 🙂
Still in the process of trying to upgrade the gem to the latest version in my company, I detected an issue introduced in the 7.4.0 release.
Since the Vonage::Voice::Ncco was introduced, it could lead with a conflict with any other class implementing one of these method names
https://github.com/Vonage/vonage-ruby-sdk/blob/ecb5dbd9d288dbb6f50f8fb97f2f56603b185916/lib/vonage/voice/ncco.rb#L6-L14
The issue is located here
https://github.com/Vonage/vonage-ruby-sdk/blob/ecb5dbd9d288dbb6f50f8fb97f2f56603b185916/lib/vonage/voice/ncco.rb#L17
Since this code is called at the class level, self.class is Class (and not the expected Vonage::Voice::Ncco class, which is self), which means methods are defined in the native ruby class Class.
Since all classes are a Class, this methods could conflict with any other class (from another library, or from the project) that are implementing one of these class method names depending on the load order.
Links

Pull request #206

Thanks @superchilled 🙏

Could have caused some nasty bugs... 😬

Indeed, our CI just "exploded" 😅

Hopefully should be able to include this as part of the same release for the ServiceError PR.

🙇♂ 👌

Thanks @superchilled 🙏

Could have caused some nasty bugs... 😬

Indeed, our CI just "exploded" 😅

Hopefully should be able to include this as part of the same release for the ServiceError PR.

🙇♂ 👌

Fixed in https://github.com/Vonage/vonage-ruby-sdk/releases/tag/v7.5.0


Add fuzz and scanning logic for checking failed Azure CI tests

We should add args to the test suite run in Azure which permit some diffs in the outputs of tests but still be considered a pass. Once we have that working, we should add pass/fail status to test suite run for Azure CI.

This was basically completed in #20005


Move the documentation website source code into the master branch

Currently, the documentation website source code is sitting in the gh-pages branch. I want to change this so it sits in a docs/ directory in the master branch, then automatically deploy to the gh-pages branch. I also want to utilise Nuxt 3 rather than raw HTML, CSS & JS.

Blocked for now as waiting for static builds to be supported by Nuxt 3.


[qt5-base] build failure on macOS 10.15

Host Environment

OS: macOS 10.15 from GitHub Actions
Compiler: whichever XCode version runs on the GitHub Actions runner

To Reproduce
I reproduced this in release-only build, but from the failure logs, it does not look like that has any impact.
./vcpkg install qt5-base:x64-osx-release --clean-after-build --overlay-triplets=triplets_overlay
Contents of triplets_overlay/x64-osx-release.cmake:
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE static)

set(VCPKG_CMAKE_SYSTEM_NAME Darwin)
set(VCPKG_BUILD_TYPE release)

Failure logs
https://github.com/FranciscoPombal/qBittorrent/runs/1499053061?check_suite_focus=true
Starting package 26/26: qt5-base:x64-osx-release
Building package qt5-base[core]:x64-osx-release...
-- [OVERLAY] Loading triplet configuration from: /Users/runner/work/qBittorrent/qBittorrent/triplets_overlay/x64-osx-release.cmake
-- Figuring out qt target mkspec. Target arch x64
-- Target mkspec set to: macx-clang
-- Host mkspec not set. Qt's own buildsystem will try to figure out the host system
-- Downloading http://download.qt.io/official_releases/qt/5.15/5.15.1/submodules/qtbase-everywhere-src-5.15.1.tar.xz...
-- Extracting source /Users/runner/qbt_tools/vcpkg/downloads/qtbase-everywhere-src-5.15.1.tar.xz
-- Applying patch patches/winmain_pro.patch
-- Applying patch patches/windows_prf.patch
-- Applying patch patches/qt_app.patch
-- Applying patch patches/gui_configure.patch
-- Applying patch patches/icu.patch
-- Applying patch patches/xlib.patch
-- Applying patch patches/egl.patch
-- Applying patch patches/zstdd.patch
-- Applying patch patches/Qt5BasicConfig.patch
-- Applying patch patches/Qt5PluginTarget.patch
-- Applying patch patches/create_cmake.patch
-- Applying patch patches/Qt5GuiConfigExtras.patch
-- Using source at /Users/runner/qbt_tools/vcpkg/buildtrees/qt5-base/src/5.15.1-63a9328040.clean
xcodebuild: error: SDK "/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk" cannot be located.
xcrun: error: unable to lookup item 'SDKVersion' in SDK '/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk'
-- Detected OSX SDK Version:
CMake Error at ports/qt5-base/portfile.cmake:264 (string):
  string sub-command REGEX, mode MATCH needs at least 5 arguments total to
  command.
Call Stack (most recent call first):
  scripts/ports.cmake:136 (include)

Error: Building package qt5-base:x64-osx-release failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: qt5-base:x64-osx-release
  Vcpkg version: 2020.11.12-unknownhash

Additionally, attach any relevant sections from the log files above.

Additional context
I don't have a macOS system so I can't test this further on my own.

@Neumann-A This is a follow-up to https://github.com/microsoft/vcpkg/issues/14941, as mentioned in https://github.com/microsoft/vcpkg/issues/14941#issuecomment-738806392.

that is due to xcrun --show-sdk-version failing you could try setting VCPKG_OSX_DEPLOYMENT_TARGET to 10.15 in the triplet to skip the detection but I assume it has something to do with an env variable being missing for xcrun to succeed (probably SDKROOT and/or MACOSX_DEPLOYMENT_TARGET)

@Neumann-A

that is due to xcrun --show-sdk-version failing you could try setting VCPKG_OSX_DEPLOYMENT_TARGET to 10.15 in the triplet to skip the detection

OK, I'm trying that now: https://github.com/FranciscoPombal/qBittorrent/runs/1499371354?check_suite_focus=true

but I assume it has something to do with an env variable being missing for xcrun to succeed (probably SDKROOT and/or MACOSX_DEPLOYMENT_TARGET)

If the triplet method does not work, do you known what I should set those variables to?

Nope don't have access to macosx myself. I am just cleverly googling error messages and that error does not have a clever solution yet.
https://stackoverflow.com/questions/60536169/xcrun-show-sdk-version-error-cant-build-with-xcode-on-mojave
maybe you can run xcrun --show-sdk-version --verbose or use --log to get more info from what is happening and why it is not working (https://www.real-world-systems.com/docs/xcrun.1.html). maybe xcode is simply not setup correctly.

@Neumann-A
I ran:
xcrun --show-sdk-path --verbose --log
echo "**********"
xcrun --show-sdk-version --verbose --log

Here is the output:
/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk
**********
xcrun: note: looking up SDK with '/Applications/Xcode_12.2.app/Contents/Developer/usr/bin/xcodebuild -sdk /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk -version SDKVersion'
xcrun: note: PATH = '/usr/local/opt/pipx_bin:/Users/runner/.cargo/bin:/usr/local/lib/ruby/gems/2.7.0/bin:/usr/local/opt/ruby/bin:/usr/local/opt/curl/bin:/usr/local/bin:/usr/local/sbin:/Users/runner/bin:/Users/runner/.yarn/bin:/usr/local/go/bin:/Users/runner/Library/Android/sdk/tools:/Users/runner/Library/Android/sdk/platform-tools:/Users/runner/Library/Android/sdk/ndk-bundle:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/usr/bin:/bin:/usr/sbin:/sbin:/Users/runner/.dotnet/tools:/Users/runner/.ghcup/bin:/Users/runner/hostedtoolcache/stack/2.5.1/x64'
xcrun: note: SDKROOT = '/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk'
xcrun: note: TOOLCHAINS = ''
xcrun: note: DEVELOPER_DIR = '/Applications/Xcode_12.2.app/Contents/Developer'
xcrun: note: XCODE_DEVELOPER_USR_PATH = ''
xcrun: note: xcrun_db = '/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/xcrun_db'
xcodebuild: error: SDK "/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk" cannot be located.
xcrun: error: unable to lookup item 'SDKVersion' in SDK '/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk'

Looks like the same "incoherence" mentioned in https://stackoverflow.com/questions/60536169/xcrun-show-sdk-version-error-cant-build-with-xcode-on-mojave.
The output from xcrun --sdk macosx10.15 --show-sdk-path --verbose --log is:
xcrun: note: looking up SDK with '/Applications/Xcode_12.2.app/Contents/Developer/usr/bin/xcodebuild -sdk macosx10.15 -version Path'
xcrun: note: PATH = '/usr/local/opt/pipx_bin:/Users/runner/.cargo/bin:/usr/local/lib/ruby/gems/2.7.0/bin:/usr/local/opt/ruby/bin:/usr/local/opt/curl/bin:/usr/local/bin:/usr/local/sbin:/Users/runner/bin:/Users/runner/.yarn/bin:/usr/local/go/bin:/Users/runner/Library/Android/sdk/tools:/Users/runner/Library/Android/sdk/platform-tools:/Users/runner/Library/Android/sdk/ndk-bundle:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/usr/bin:/bin:/usr/sbin:/sbin:/Users/runner/.dotnet/tools:/Users/runner/.ghcup/bin:/Users/runner/hostedtoolcache/stack/2.5.1/x64'
xcrun: note: SDKROOT = 'macosx10.15'
xcrun: note: TOOLCHAINS = ''
xcrun: note: DEVELOPER_DIR = '/Applications/Xcode_12.2.app/Contents/Developer'
xcrun: note: XCODE_DEVELOPER_USR_PATH = ''
xcrun: note: xcrun_db = '/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/xcrun_db'
xcodebuild: error: SDK "macosx10.15" cannot be located.
xcrun: note: looking up SDK with '/Applications/Xcode_12.2.app/Contents/Developer/usr/bin/xcodebuild -sdk macosx10.15 -version Path'
xcrun: note: PATH = '/usr/local/opt/pipx_bin:/Users/runner/.cargo/bin:/usr/local/lib/ruby/gems/2.7.0/bin:/usr/local/opt/ruby/bin:/usr/local/opt/curl/bin:/usr/local/bin:/usr/local/sbin:/Users/runner/bin:/Users/runner/.yarn/bin:/usr/local/go/bin:/Users/runner/Library/Android/sdk/tools:/Users/runner/Library/Android/sdk/platform-tools:/Users/runner/Library/Android/sdk/ndk-bundle:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/usr/bin:/bin:/usr/sbin:/sbin:/Users/runner/.dotnet/tools:/Users/runner/.ghcup/bin:/Users/runner/hostedtoolcache/stack/2.5.1/x64'
xcrun: note: SDKROOT = 'macosx10.15'
xcrun: note: TOOLCHAINS = ''
xcrun: note: DEVELOPER_DIR = '/Applications/Xcode_12.2.app/Contents/Developer'
xcrun: note: XCODE_DEVELOPER_USR_PATH = ''
xcrun: note: xcrun_db = '/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/xcrun_db'
xcodebuild: error: SDK "macosx10.15" cannot be located.
xcrun: error: unable to lookup item 'Path' in SDK 'macosx10.15'

There might be a problem with the runner setup I guess.

that is due to xcrun --show-sdk-version failing you could try setting VCPKG_OSX_DEPLOYMENT_TARGET to 10.15 in the triplet to skip the detection
OK, I'm trying that now: https://github.com/FranciscoPombal/qBittorrent/runs/1499371354?check_suite_focus=true

It builds OK with this workaround (adding set(VCPKG_OSX_DEPLOYMENT_TARGET 10.15) to the triplet file).

hmm maybe it needs to be xcrun --sdk macosx --show-sdk-path

I experienced this issue to when trying to build qt5-base via vcpkg on github's mac runner:
  Starting package 32/32: qt5-base:x64-osx
  Building package qt5-base[core]:x64-osx...
  -- Figuring out qt target mkspec. Target arch x64
  -- Target mkspec set to: macx-clang
  -- Host mkspec not set. Qt's own buildsystem will try to figure out the host system
  -- Downloading https://download.qt.io/official_releases/qt/5.15/5.15.2/submodules/qtbase-everywhere-src-5.15.2.tar.xz;https://mirrors.ocf.berkeley.edu/qt/official_releases/qt/5.15/5.15.2/submodules/qtbase-everywhere-src-5.15.2.tar.xz -> qtbase-everywhere-src-5.15.2.tar.xz...
  -- Extracting source /Users/runner/work/SerialAudio/SerialAudio/vcpkg/downloads/qtbase-everywhere-src-5.15.2.tar.xz
  -- Applying patch patches/winmain_pro.patch
  -- Applying patch patches/windows_prf.patch
  -- Applying patch patches/qt_app.patch
  -- Applying patch patches/gui_configure.patch
  -- Applying patch patches/icu.patch
  -- Applying patch patches/xlib.patch
  -- Applying patch patches/egl.patch
  -- Applying patch patches/zstdd.patch
  -- Applying patch patches/mysql_plugin_include.patch
  -- Applying patch patches/mysql-configure.patch
  -- Applying patch patches/Qt5BasicConfig.patch
  -- Applying patch patches/Qt5PluginTarget.patch
  -- Applying patch patches/create_cmake.patch
  -- Applying patch patches/Qt5GuiConfigExtras.patch
  -- Applying patch patches/limits_include.patch
  -- Using source at /Users/runner/work/SerialAudio/SerialAudio/vcpkg/buildtrees/qt5-base/src/5.15.2-fa8de595e1.clean
  xcodebuild: error: SDK "/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk" cannot be located.
  xcrun: error: unable to lookup item 'SDKVersion' in SDK '/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk'
  -- Detected OSX SDK Version:
  CMake Error at ports/qt5-base/portfile.cmake:323 (string):
  Error:   string sub-command REGEX, mode MATCH needs at least 5 arguments total to
    command.
  Call Stack (most recent call first):
    scripts/ports.cmake:142 (include)

  Error: Building package qt5-base:x64-osx failed with: BUILD_FAILED
  Please ensure you're using the latest portfiles with `git pull` and `./vcpkg update`, then
  submit an issue at https://github.com/Microsoft/vcpkg/issues including:
    package: qt5-base[core]:x64-osx -> 5.15.2#13
    vcpkg version: 2021-11-02-af04ebf6274fd6f7a941bff4662b3955c64f6f42
    vcpkg-tool version: 369878e 2021-11-10 (2 hours ago)

  Additionally, attach any relevant sections from the log files above.

  [LogCollection][End]File:'/Users/runner/work/SerialAudio/SerialAudio/builds/ninja-multi-vcpkg/vcpkg-manifest-install.log'.
  CMake Error at vcpkg/scripts/buildsystems/vcpkg.cmake:831 (message):
  Error:   vcpkg install failed.  See logs for more information:
    /Users/runner/work/SerialAudio/SerialAudio/builds/ninja-multi-vcpkg/vcpkg-manifest-install.log
  Call Stack (most recent call first):
    /Users/runner/work/_temp/-1442738397/cmake-3.21.2-macos-universal/CMake.app/Contents/share/cmake-3.21/Modules/CMakeDetermineSystem.cmake:124 (include)
    CMakeLists.txt:3 (project)

  CMake Error: CMake was unable to find a build program corresponding to "Ninja Multi-Config".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
  CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
  CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
  -- Configuring incomplete, errors occurred!
⏱ elapsed: 2846.265 seconds
Error: "CMake failed with error code: '1'.
    at CMakeRunner.<anonymous> (/Users/runner/work/_actions/lukka/run-cmake/v10/dist/index.js:4164:23)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/runner/work/_actions/lukka/run-cmake/v10/dist/index.js:4007:58)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
Error: run-cmake action execution failed: 'Error: "CMake failed with error code: '1'.'

(I should note the windows build was successfull)

That's means VCPKG_OSX_DEPLOYMENT_TARGET is not defined or empty.

@FranciscoPombal #21486 fixed this issue, please get the latest version and try again.


Exploring what's possible with Custom Shaders

Custom Shaders (issue #9518 and initial implementation #9725) allows applying user-defined vertex and fragment shaders to glTF models as well as 3D Tiles. This is new behavior, as the old materials system was not compatible with 3D Tiles.
While the implementation is still a work in progress, we still want to gather ideas of what exciting things will be possible with this new feature.
Here are a few ideas we've seen already, though more are welcome!

glTF supports storing application-specific semantics (custom vertex attributes). For example, a mesh could have _TEMPERATURE. Custom shaders provides a method to access these values and use them in the shader
(requires glTF EXT_feature_metadata) Make a tileset that includes simple polygons with metadata labeling them as "grass" or "building" etc. Then use custom shaders to procedurally apply textures
A user could apply a mask/tint to areas of imagery data in more nuanced ways than with styling. This will be even more useful once EXT_feature_metadata is available and the shader can use metadata to mask where the shading should be applied.
If we include cartographic WGS84 coordinates (caveat: 32-bit precision limitations on the GPU) you could easily overlay a lat/lon grid over 3D tiles.
@srothst1 had a couple interesting ideas for visualization:

Weather visualizations on a tileset with terrain/imagery. For example, maybe visualize wildfires or other extreme weather events in the news. This would probably be a good use case for EXT_feature_metadata feature textures or feature ID textures.
We talked about how you might represent The Valley Curtain in CesiumJS. While certainly you could do something simple with a glTF and animations alone, with a custom vertex shader you could simulate the curtain flowing in the breeze (perhaps with procedural noise or something)

Thanks to @mramato and @srothst1 and others for some of the above ideas.
@mramato mentioned a couple more ideas in slack, could you expand on them in more detail here?

A while back there was a PR to add uniforms to the styling language and one of the sandcastles I put together was flying a plane around a mountain and shading based on distance.

Here's some random questions/ideas:

Just to confirm my understanding, with these new capabilities, I could have a 3D Tiles roof marked as "grass" and have a custom grass shader used for drawing now?
Should/could there be a way to extend the 3D Tiles styling language so that you can provide an easily transportable way to share these visual enhancements without them being app-specific?  For example, if I have a bunch of custom shaders to render Cesium OSM Buildings based on their material metadata (glass, brick, grass, etc...) it would be great to have a way to include that with the tileset.
Can 3D Tiles/glTF ultimately replace (or heavily augment) the Geometry and Appearances system?  Would generating 3D Tiles (or glTF models) on the fly and then attaching materials to them be cleaner more uniform solution?
Does this mean we now have a clean way to do video on models or video on 3D Tiles? Similar to what we can already do in G&A.

Does this get us any closer to being able to drape imagery layers over 3D Tiles?

Random thought that came to mind after talking with @srothst1  - would it be possible to make certain built-in uniforms available to custom shaders like node id and primitive id? That way someone could color specific nodes of a model like in https://community.cesium.com/t/how-to-color-specific-part-of-model/14898. Of course, the better solution would be to use EXT_feature_metadata where this type of functionality would be built in.

@ptrgags Given we've done the initial work on the custom shader implementation, is there anything actionable left in this issue that we'd plan to address, or can it be closed?


spec: put links into old files

To finish moving the specifications #561 we have to put links into the old files that point to the new location in the tendermint/spec repository.

#599 resolved it


Initialise parser in arbitrary state

Cucumber will need to get a step's keyword in order to generate a stepdef snippet with the right keyword.
Pickles don't contain the step keyword because they can originate from non-gherkin documents (such as Markdown, which doesn't have step keywords).
Pickles contain the path of the source file.
It's possible to parse the source file with Gherkin, iterate over scenarios and find the step for a certain line.
But that's very cumbersome.
It would be better to ask the parser to return a Step AST node directly. This would require initialising the parser in the right state, so that parsing a step doesn't cause a parse error.
Something like this (pseudoish code):
var parser = new Parser();
parser.state = Parser.ScenarioState;
var step = parser.parse('    Given I have 4 cukes');
var keyword = step.keyword;

Thoughts?

Or am I trying too hard to decouple Gherkin from pickles here? Is it more pragmatic to put the keyword back on PickleStep? Will we regret that later when people feed Cucumber from all sorts of source formats?

Since this makes things so much more complex I would vote for one of following simpler solutions.

Snippets are no longer defined with a keyword. I like this as it reinforces that cucumber doesn't differentiate between "Given", "When", "Then" step definitions and also removes the need to support i18n in snippets/step definitions See cucumber-js#248

Just add the keyword to the pickled steps. It will be undefined when coming from other sources, but the snippet generator is going to have to deal with this anyway.

I'm not at all in favour of 1, since it removes valuable semantic context from a step definition. Context that is valuable to a human.
I'm fine with 2 I think. @gasparnagy what are your thoughts about this? You've also mentioned a use case for initialising the parser in a different state. What's that use case again?

You cannot assume that it is the keyword of the step that should be used in the snippet. For "And", "But" and "*" Cucumber-Ruby uses keyword of the preceding step with a "Given", "When" or "Then".
The general problem of making lookup from locations in pickles to Gherkin/Markdown source need to be solved sooner or later - for reporting if not sooner. Currently Cucumber-Ruby solves it by using classes for both the AST and the test cases(/pickles) and having references from the test case/step classes to the AST classes. In potential use cases like sending pickles files off into a computing grid and getting back results json files, that approach will not do of course.
A somewhat related lookup problem is to run the right test cases for command line arguments like "path/name.feature:23". Line 23 may not be the line of a pickle, if it is the line of an outline step the all test cases for that Scenario Outline is executed. This particular lookup is not implemented very efficiently in Cucumber-Ruby today (see cucumber/cucumber-ruby#901).

@brasmusson could we solve this problem by putting this logic in the compiler? When the compiler sees a Given  followed by a Then , it will create pickle steps with two Given  keywords?

Parsing just the step line might not be the solution to finding the keyword for the snippet, however the feature can be useful in other cases. Cucumber-Ruby is (ab)using the Gherkin3 parser to parse nested steps (feature), by setting up the parser in specific parser state, not exactly the kind of dependencies you really want (as I wrote in the PR description).

@aslakhellesoy Yes, using the compiler to calculate the proper snippet keyword would work. The the attribute of the pickle should rather be named snippet_keyword than keyword since it would be aimed at the snippets and not for instance the pretty formatter output. The intentions, as I understand it, is then to be able to produce both the new raw result json file and the snippets, without using the Gherkin source. The Gherkin source would only be needed to produce the other (more final) reports.

Hello there,
I'm maintaining a Python Gherkin library and this is also something I would like to see implemented. Currently to parse a step by itself I'm recreating a complete feature and then extracting the steps, so an interface to do this will be nice:
https://github.com/koterpillar/aloe/blob/master/aloe/parser.py#L217

Yep. I'm having to do the same 'wrap the thing that I want to parse up in a fake feature file and extract it afterward' trick in my library as well. It was easier than trying to monkey patch this gem back when I first started using it.
Being able parse fractions of a feature file would be a nice feature to have right out of the box.

I feel like this should be as easy as tweaking the grammar to say 'here are some more valid starting rules'. Is this not the case?

@aslakhellesoy , @brasmusson I have again found myself in need of this kind of ability to parse arbitrary portions of Gherkin. Is it something that is still desired on your end?

@enkessler I can see some places where the ability to parse arbitrary portions of Gherkin, would remove the need for the "'wrap the thing that I want to parse up in a fake feature file and extract it afterward' trick" (Cucumber-Ruby's StepsParser and DataTableParser). It would not help in the case of finding the correct keyword for a snippet though, to be sure to get that right the whole feature file is needed.
All in all, I view this a rather low priority feature, compared to other work item within the Cucumber open-source project like:

Using cucumber-expressions in the Cucumber implementations.
Upgrade Cucumber-JVM to use the new Gherkin parser and compiler.
Killing the legacy parsers of Cucumber-Ruby
Implementing command line formatters (Pretty and HTML).

@aslakhellesoy
I would proceed with @charlierudolph suggestion 2 (adding the keyword as optional attribute to pickles). If this optional information is lacking for the snippet generator (meaning keyword is missing), it should use some like * I have 4 cukes (to indicate a generic-step; if it is not already implemented that way).
If your first solution is still desirable, I would suggest a minor "simplification" (and hiding how the parser gets into the correct state). Add another parser entry point (or a specific StepParser/StepsParser), like:
var parser = new Parser();
var step = parser.parse_step('Given I have 4 cukes');  //< State machine init hidden from the user.
...
// BETTER: parser.parse_steps(text)
// ==> Overcomes keyword ambiguities mentioned by @brasmusson (And, But, ...)

Yeah I agree the keyword should be in the pickle steps. Happy to take a pull request! Even for just a single language or two.

If the new optional attribute is intended for snippets, it should be named snippet_keyword (or something), or both keyword (the actual keyword) and snippet_keyword should be added as optional attributes.
I have implemented snippet generation with the correct keywords from pickles in cucumber/cucumber-jvm#1035. It is a two step process where the runner (which only knows about pickles), generated snippets with **KEYWORD** as the keyword and stores them in the result object, and then the front end replaces **KEYWORD** with the correct keyword - after having reparsed the feature.


Add CLI option for watcher polling interval

In some environments the fs watcher falls back to polling for
changes. Both our current watcher and the proposed new watcher
both had support for user-defined polling intervals. I see no
reason not to add this for users who want extra control.
Fixes #801.

Feels @saper ?

Everybody should have noticed by now that I avoid watcher issues like hell. Our watcher tests are still wrong and we need to fix them.
On a serious note, I happen to debug NFS performance issues for living and I'd like to know more about #801.

Doesn't feel like --interval is a right way to do it.

Could you elaborate? Poll interval control is pretty common to FS watchers.
I personally think the default 500ms is perfectly reasonable. I'm convinced we need to ability to change it.

To support that feeling, https://github.com/sass/node-sass/issues/801 been around for about 6 months with other supports. Also Ruby Sass does not support this via their CLI.

I wonder what kind of system is this when 500ms is inducing too much load. Maybe we are ineffective in some other, nasty way and we can avoid that. Like we don't use stat() where we should.

I don't think it's load. More likely it's simply an annoying delay. We frontend developers are very touch about things feeling instant. Considering we can get sub-second compilations, wait 500ms for the watcher to kick in can feel like an eternity. Speaking from experience.

#801 says something about CPU load. So in reality you are eager to consider node-sass --ultra-productive-developer, shortened to node-sass -U ?

I think I'll abstain on this one. I know something about kernel polling mechanisms like kqueue and epoll, I've heard there are issues with Mac OS X watcher API (the main reason so many 'watcher' packages exist in node I think) but I have little clue how what we are doing right now in JavaScript translates to a real something being done on the system that might affect system performance if polling too often. Until I study the APIs and carry out some NFS tests the answer from me will be: "I don't know".

Closing this. It appears to be an isolated case.

:tada


getLightAgentConfiguration() in conversation.

Description
Using getLightAgentConfiguration() in postUserMessage() and editUserMessage()

Risk

Deploy Plan

cc @spolu as a follow up of https://github.com/dust-tt/dust/pull/5303


Unsupported output is not properly caught

When using an unsupported output, the error verification is not properly done and result in escaping the error message display.
https://github.com/vegaprotocol/vegawallet/blob/a32cd1838438bef69e70ea0138e993049783a307/cmd/cmd.go#L33-L35
For this condition to work, the InvalidOutputError should contain any unpredictable value such as the wrapped error message.

Tested OK on latest wallet
Muthus-MBP:repos muthupandianvadivelu$ vegawallet -o muthu network list
unsupported output


refactor: using test classes in test_inline tests

Description
As the title.
Using:
hyperfine --warmup 2 --show-output "pytest tests/test_inline_functions/*"

Performance
Before
  Time (mean ± σ):     53.528 s ±  3.060 s    [User: 3.689 s, System: 1.806 s]
  Range (min … max):   48.600 s … 58.802 s    10 runs

After
  Time (mean ± σ):     43.933 s ±  4.528 s    [User: 3.644 s, System: 1.557 s]
  Range (min … max):   39.058 s … 52.305 s    10 runs

Issue linked
NA
Checklist

[x] I have visited and read Develop PyMAPDL.
[x] I have tested my changes locally
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the PyMAPDL coding style guidelines.
[x] I have added appropriate unit tests that also ensure the minimal coverage criteria.
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved to the PR if any.
[x] I have assigned this PR to myself.
[x] I have marked this PR as draft if it is not ready to be reviewed yet.
[x] I have made sure that the title of my PR follows Commit naming conventions (e.g. feat: adding new MAPDL command)

@pyansys-ci-bot LGTM.


add CPU percentage to prometheus info/output

Is your feature request related to a problem? Please describe.
Currently there only are values for podman_container_cpu_seconds_total and podman_container_started_seconds. While these can be used to convert to %cpu, this however requires additional computation and is not ideal for direct ingesting or graphing the data.
Describe the solution you'd like
Having an an additional value for %cpu (percentage) similiar or identical to what podman stats does.
-> convert podman_container_started_seconds to unix time -> calculate seconds since container start (runtime seconds) -> divide podman_container_cpu_seconds_total  by runtime seconds is currently needed unless I am missing something obvious?

never mind, I learned something new about rate() regarding prometheus today.


changed favicon to GDSC official logo

changed the favicon which closes #23

Aswm ! @ayushsaha1018 :)

FIXES #23


MTU should be configurable

Different setups will have different specifications, having the MTU configurable via a config file(or a given parameter) could help the end user.
https://github.com/openvswitch/ovn-kubernetes/blob/ae4ecfb2d323fc3eb2e0b5f0611d19dbbe2630c0/bin/ovn-k8s-overlay#L284
https://github.com/openvswitch/ovn-kubernetes/blob/a16e91fa3f9f26d8413e55a28500cbd572970f37/bin/ovn-k8s-cni-overlay#L106

If you are using overlay networks with Geneve, the last time I calculated, it came to around 1443. But if someone adds more Geneve options in OVN, it will decrease. But I see your point. Some people may have underlay at 9000 and in that case, you don't need this. I guess, we can default to 1400, but people can pass it as an option.

This is configurable now.


File viewer does not work

As you can see in the image:
Request to URL like http://<IP_ADDRESS>:8080/20201125095309-coding.html?token=null will return 404.

Configuration:
(setq org-roam-server-host "<IP_ADDRESS>"
      org-roam-server-port 8080
      org-roam-server-authenticate nil
      org-roam-server-export-inline-images t
      org-roam-server-serve-files nil
      org-roam-server-served-file-extensions '("pdf" "mp4" "ogv")
      org-roam-server-network-poll t
      org-roam-server-network-arrows nil
      org-roam-server-network-label-truncate t
      org-roam-server-network-label-truncate-length 60
      org-roam-server-network-label-wrap-length 20)

Can you try to export a document in your database to html using org-mode export function. C-c C-e h o default key presses.

Yes, as it should be.

Can you try restarting your computer: https://github.com/org-roam/org-roam-server/issues/107

Yes, I will try latter. Meanwhile, this could be useful for debugging.
org-roam-server-root is a variable defined in org-roam-server.el.

Value
"/path/to/.emacs.d/.local/straight/build-27.1/org-roam-server/."

ls /path/to/.emacs.d/.local/straight/build-27.1/org-roam-server/.
assets                       org-roam-server.el
index.html                   org-roam-server.elc
org-roam-server-autoloads.el

It still does not work.
I notice the following error reported in the minibuffer when enabling org_roam-server-mode
-contains?: Wrong type argument: listp, sp-text-mode-emoticon-p

Hi, I find this link https://www.gitmemory.com/issue/org-roam/org-roam-server/75/687602053.
When I had the 404 issue, I tried to enable org-roam-server in a daemon Emacs, which started by brew services. Now, if I tried to enable org-roam-server in non-deamon Emacs, the file viewer now.
With this information, could you figure out where the bug could come from?

As far as I remember it was related to smartparens https://github.com/org-roam/org-roam-server/issues/115

I had the same issue after a fresh installation of doom emacs, and this 115 indeed solved it for me.

This issue has reappeared for me.
I am using doom emacs, and I am not sure why it happens now. I still have the configuration given in 115, but it does not help anymore...

What error are you getting?

Yes, exactly.
However, I see the content of the backlinks...

For me, deleting org-roam.db and recreating it fixed the problem. Can you try it?

I have the same issue whenever I rename filename of an oram item.
The viewer would say that the renamed item is not found.
Maybe, some hook could be added after file is renamed.
发自我的iPad

在 2021年3月21日，11:39，Göktuğ Karakaşlı @.***> 写道：

For me, deleting org-roam.db and recreating it fixed the problem. Can you try it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

For me, deleting org-roam.db and recreating it fixed the problem. Can you try it?

I tried it, but it doesn't help.. I still get the same error.

@goktug97 this is the http request I see in the emacs log
(request
 (date "Sun, 21 Mar 2021 09:56:52 GMT")
 (address "<IP_ADDRESS>")
 (get "/org-roam-buffer")
 (headers
  ("GET" "/org-roam-buffer?path=/Users/roiho/Dropbox%20(Weizmann%20Institute)/PHD/org-roam/20200924192709-rotator_get_different_geometric_phases_by_an_auxiliary_term.org&label=Rotator:%20get%20different%20geometric%20phases%20by%20an%20auxiliary%20term&token=null" "HTTP/1.1")
  ("Host" "localhost:8080")
  ("User-Agent" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:86.0) Gecko/20100101 Firefox/86.0")
  ("Accept" "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8")
  ("Accept-Language" "en-US,en;q=0.5")
  ("Accept-Encoding" "gzip, deflate")
  ("Connection" "keep-alive")
  ("Referer" "http://localhost:8080/")
  ("Upgrade-Insecure-Requests" "1")
  ("Content" "")))

in the beginning it tries  (get "/org-roam-buffer"). is this ok?
also, later it asks for an org file and not an html file. it this ok?

@goktug97 this is the http request I see in the emacs log
(request
 (date "Sun, 21 Mar 2021 09:56:52 GMT")
 (address "<IP_ADDRESS>")
 (get "/org-roam-buffer")
 (headers
  ("GET" "/org-roam-buffer?path=/Users/roiho/Dropbox%20(Weizmann%20Institute)/PHD/org-roam/20200924192709-rotator_get_different_geometric_phases_by_an_auxiliary_term.org&label=Rotator:%20get%20different%20geometric%20phases%20by%20an%20auxiliary%20term&token=null" "HTTP/1.1")
  ("Host" "localhost:8080")
  ("User-Agent" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:86.0) Gecko/20100101 Firefox/86.0")
  ("Accept" "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8")
  ("Accept-Language" "en-US,en;q=0.5")
  ("Accept-Encoding" "gzip, deflate")
  ("Connection" "keep-alive")
  ("Referer" "http://localhost:8080/")
  ("Upgrade-Insecure-Requests" "1")
  ("Content" "")))

in the beginning it tries  (get "/org-roam-buffer"). is this ok?
also, later it asks for an org file and not an html file. it this ok?

Is this a new issue for you? Did viewer work in the past? /Users/roiho/Dropbox%20(Weizmann%20Institute) Maybe the encoded path causing the problem. Can you try again by copying your roam folder to a local folder (not cloud) without any spaces and changing the path variable accordingly in emacs?

I don't know what solved the issue, but it is now working great! (among else I updated my doom emacs)

@goktug97 this is the http request I see in the emacs log
(request
 (date "Sun, 21 Mar 2021 09:56:52 GMT")
 (address "<IP_ADDRESS>")
 (get "/org-roam-buffer")
 (headers
  ("GET" "/org-roam-buffer?path=/Users/roiho/Dropbox%20(Weizmann%20Institute)/PHD/org-roam/20200924192709-rotator_get_different_geometric_phases_by_an_auxiliary_term.org&label=Rotator:%20get%20different%20geometric%20phases%20by%20an%20auxiliary%20term&token=null" "HTTP/1.1")
  ("Host" "localhost:8080")
  ("User-Agent" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:86.0) Gecko/20100101 Firefox/86.0")
  ("Accept" "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8")
  ("Accept-Language" "en-US,en;q=0.5")
  ("Accept-Encoding" "gzip, deflate")
  ("Connection" "keep-alive")
  ("Referer" "http://localhost:8080/")
  ("Upgrade-Insecure-Requests" "1")
  ("Content" "")))

in the beginning it tries  (get "/org-roam-buffer"). is this ok?
also, later it asks for an org file and not an html file. it this ok?

Is this a new issue for you? Did viewer work in the past? /Users/roiho/Dropbox%20(Weizmann%20Institute) Maybe the encoded path causing the problem. Can you try again by copying your roam folder to a local folder (not cloud) without any spaces and changing the path variable accordingly in emacs?

I don't know what solved the issue, but it is now working great! (among else I updated my doom emacs)

For some reason, it stopped working again. Now the request is empty:
(request
 (date "Sun, 11 Apr 2021 10:22:56 GMT")
 (address "<IP_ADDRESS>")
 (get "/roam-data")
 (headers
  ("GET" "/roam-data?token=null" "HTTP/1.1")
  ("Host" "localhost:8080")
  ("User-Agent" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:86.0) Gecko/20100101 Firefox/86.0")
  ("Accept" "text/event-stream")
  ("Accept-Language" "en-US,en;q=0.5")
  ("Accept-Encoding" "gzip, deflate")
  ("Connection" "keep-alive")
  ("Referer" "http://localhost:8080/")
  ("Cookie" "_xsrf=2|6bb1fce4|b5bc1f8bd8d658827fd5a7db71a66952|1616329630; username-localhost-8888=\"2|1:0|10:1616329634|23:username-localhost-8888|44:YTUyZjA2MzQxNGUyNGZlMGFiMTliNTM1MzRlODZiYjc=|b9b2f7b1adb43f02c92534eed7954e431b3d72bfe7796992063c759b35a0cb15\"")
  ("Pragma" "no-cache")
  ("Cache-Control" "no-cache")
  ("Content" "")))


change defaults

Why's this necessary? What defaults would someone intend to set on a client level?

timeout. I've been playing in a repl and when the C# server is breakpointed in the middle of a request the client times out relatively quickly.

K, makes sense. In that case I'd like to do a breaking release that switches to using a single options argument where base is an option.

makes sense, you want me to do that?

Sure!

how's that?

looks good, i'll double check and merge


attacking the squiggles

Okay, so there are 4 things I haven't figured out how to handle yet.

what is the point of the round_half_up function at the top of processing.rs? looking at the description it seems like it behaves exactly like normal rounding(could be misinterpreting)
we have multiple options for handling fft.rfft that don't involve writing it ourself: rustfft,ndrustfft, and another lib called arrayfire. alternatively we could implement it natively, but I have a feeling ffts are probably difficult to optimize.
both the functions in function.rs involve numpy functions that don't have rust equivalents, namely logical_and and np.exp, looking these up they seem feasible to implement but they involve switching gears a bit
lastly tile doesn't seem to have a rust equivalent currently, but depending on why they were using it, we may want to use something else in its place anyway(an equivalent to np.broadcast?)

I completely agree, and actually went to check that both Python and Rust round up .5 and they both do. I don't know why actually this, maybe some old Python version had a  bug with this. I'd propose that we ignore this for the time being. Since we aren't sure I would like to keep the old version commented.
The old mfcc in Rust used fftw and rustfft and AFAIK rusfft was fast enough, coming pretty close to fftw (said to be the fastest around). ndrustfft escapes my understanding, and doesn't include a reverse operation, which we need regardless. And regarding arrayfire, it is really interesting, but to me it's more like a beast of it's own, and this comment https://discourse.julialang.org/t/arrayfire-vs-abstractarray-performance-and-future-in-julia/60533/5 makes me think that for CPU we are better served with using  some BLAS/LAPACK interface. Now, for GPU (and maybe TPUs/FFPGAs) this could be very valid given that we don't have anything like that in the Rust ecosystem, but this might need some consideration.
For np.exp(array)  we can do array.map(|v|v.exp()) while for np.logical_and(array1, array2) we could do azip!((a in &mut array1, &b in &array2) *a = a && b);, the second one might need some touches as I haven't tested it.
From what I see they seem to be using tile correctly, however, I would like a more math enlightened mind to look at it. @AmateurAcademic could you see if in https://github.com/astorfi/speechpy/blob/master/speechpy/processing.py where they are  using np.tile they mean to use broadcasting, like  np.broadcast_arrays, np.broadcast_to or np.broadcast_arrays?

@AmateurAcademic A shame! Regarding SonoPy, I would say no, there's no need to use the same mfcc. In my (limited) testing a model feded with SonoPy and that same model feded with SpeechPy, returned the exact same value, more testing might be needed though. Regardless, if what I found holds up our implementation should be fine as long the numbers are the same.
The only problem could be the remaining vectorizer, as it is using the SonoPy function "mel_spec" instead of mfcc. SpeechPy should have an equivalent in there, but I can't say for certain. Anyway, this is not a big issue, I haven't seen any model using this vectorizer.


localStorage is undefined && doesn't ignore node_modules

localStorage.setItem('token', res.data.token)

This is giving me an error "'localStorage' is not defined. (no-undef)"
It's also checking files inside node_modules for some reason.

window.localStorage

Correct answer! Also https://npmjs.com/global if you want to get modular about it.

You could also add this comment to the top of the file, which would also work in web workers.
/* eslint-global localStorage */

see #619 for more details :)


Addition of Kahn's Algorithm

💥 Proposal
This is for the addition of Kahn's Algorithm to the graphs folder in c++ section.
I think it would be a good addition as it is a very important algorithm which is helpful in competitive programming.
May I make a PR for this?

I want to do  it sir

I want to do it sir

Sorry, but the task is completed now

Add proper description of the algorithm so that contributors can have a clear idea if they wanna take up this issue or not.

Should I do it?

Add proper description of the algorithm so that contributors can have a clear idea if they wanna take up this issue or not.

Description:
Topological sort gives a linear ordering of vertices in a directed acyclic graph such that, for every directed edge a -> b, vertex ‘a’ comes before vertex ‘b’. Remember, a directed acyclic graph has at least one vertex with an in-degree of 0 and one vertex with an out-degree of 0.
Kahn’s Topological Sort Algorithm works by finding vertices which have no incoming edges and removing all outgoing edges from these vertices.
So, it would be a good addition to the repo if you want.


An unhandled exception when calling gurobi solver

Hi,
I am solving my convex optimization problem with cvxpy via calling gurobi solver. However, I come across the following exception. I do not name the variable as "y". Does anyone know where results_dict["y"] comes from? Or, how to fix it?
In addition, I also received a warning about the failure of inaccurate barrier solution. Does anybody have any suggestion for improving it?
Thanks.
Here are outputs I got after called prob.solve(solver='GUROBI', verbose=True):

Barrier solved model in 22 iterations and 0.09 seconds
Optimal objective 4.18131886e+02
Warning: failed to compute QCP dual solution due to inaccurate barrier solution
Try decreasing BarQCPConvTol for more accuracy

Traceback (most recent call last):
File "label_infer.py", line 187, in
main(args)
File "label_infer.py", line 184, in main
g.learning_via_opt(args.norm, args.out)
File "label_infer.py", line 143, in weight_learing_via_opt
prob.solve(solver='GUROBI', verbose=True)
File "/usr/local/lib/python2.7/site-packages/cvxpy/problems/problem.py", line 195, in solve
return self._solve(_args, *_kwargs)
File "/usr/local/lib/python2.7/site-packages/cvxpy/problems/problem.py", line 321, in _solve
kwargs)
File "/usr/local/lib/python2.7/site-packages/cvxpy/problems/solvers/gurobi_intf.py", line 295, in solve
return self.format_results(results_dict, data, cached_data)
File "/usr/local/lib/python2.7/site-packages/cvxpy/problems/solvers/gurobi_intf.py", line 443, in format_results
new_results[s.EQ_DUAL] = results_dict["y"][0:dims[s.EQ_DIM]]
KeyError: 'y'

Can you post a minimal working example that triggers this error?

I think this error is input dependent.
My problem is simple, Minimize(sum_squares(L-YW)+anorm(W, 2.0)). When I randomly generate "L", "Y", and "a", it works. However, it raises the above exception when solving the problem on my dataset.

Ok, can you run this under the debugger and print out "results_dict" and also "model.status" in the solve method?

I ran it under the debugger, and find some error in /usr/local/lib/python2.7/site-packages/cvxpy/problems/solvers/gurobi_intf.py.
There is no key of 'y' in results_dict. The reason is, it raises GurobiError: GurobiError() when it tries to access lc.QCPi at line 280, which is before the assignment statement for results_dict['y']. Currently, there is no handler for this exception. However, results_dict["status"] is 'optimal' at line 293. Do you have any idea about the exception when accessing lc.QCPi at line 280?

267         try:
268             model.optimize()
269             results_dict["primal objective"] = model.ObjVal
270             results_dict["x"] = np.array([v.X for v in variables])
271
272             # Only add duals if not a MIP.
273             # Not sure why we need to negate the following,
274             # but need to in order to be consistent with other solvers.
275             if not self.is_mip(data):
276                 vals = []
277                 for lc in gur_constrs:
278                     if lc is not None:
279                         if isinstance(lc, gurobipy.QConstr):
280                             vals.append(lc.QCPi)
281                         else:
282                             vals.append(lc.Pi)
283                     else:
284                         vals.append(0)
285                 results_dict["y"] = -np.array(vals)
286         except:
287             pass
289         results_dict["model"] = model
290         results_dict["variables"] = variables
291         results_dict["gur_constrs"] = gur_constrs
292         results_dict["status"] = self.STATUS_MAP.get(model.Status,
293                                                      s.SOLVER_ERROR)

I need to see the output of model.Status. results_dict["status"] stores the output of self.STATUS_MAP.get(model.Status, s.SOLVER_ERROR), but I need to know the input.

The output of model.Status is 2. I have two inputs, where Y is a matrix with the size of 100012, and L is a matrix with the size of 10001. Shall I send them to you via email?

Try running it with the latest code on master. If that doesn't work, you should email me the matrix data.

I update it to 0.4.5. It returns "solver_error" now.
BTW, do you have any idea about the warning:
Warning: failed to compute QCP dual solution due to inaccurate barrier solution
         Try decreasing BarQCPConvTol for more accuracy

Try decreasing BarQCPConvTol :)? The warning means that Gurobi had trouble computing the dual values. You probably don't need dual values so just set BarQCPConvTol or whatever to a very high value.

Got it. Thanks!

I update it to 0.4.5. It returns "solver_error" now.
BTW, do you have any idea about the warning:
Warning: failed to compute QCP dual solution due to inaccurate barrier solution
         Try decreasing BarQCPConvTol for more accuracy

I am getting the same error, how to fix this? I need dual variables.


Request to Added temp-throttle package

Description of changes
I am the author of temp-throttle. This is a request to add package. Service module also coming- I wanted to test pull request of package first.
It is a Linux shell script for throttling system CPU frequency based on a desired maximum temperature. Prevent system overheating due to bad thermal/fan drivers, or passive cooling, or broken fans.

Homepage https://github.com/Sepero/temp-throttle
Great Reviews http://seperohacker.blogspot.com/2012/10/linux-keep-your-cpu-cool-with-frequency.html

Things done
It is written entirely in Bash and GNU utils. Compatible with all architectures of Linux. Linux only.

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I'm seeing "rebuild darwin". This package will Only run on systems with Linux kernel.

I'm seeing "rebuild darwin". This package will Only run on systems with Linux kernel.

rebuild-darwing: 0 means none of Darwin packages were updated.

I don't know how to resolve this error. It builds without issue on my machine.
error: undefined variable 'Sepero'

maintainers = with lib.maintainers; [ Sepero ];

Also, you should split your changes into three commits: one for adding yourself to the maintainers list, one for adding the package and one for adding the module.

I'm not really sure what you mean "split". Do you want me to squash the commits?

Do you want me to squash the commits?

Yes, sorry

three commits: one for adding yourself to the maintainers list, one for adding the package and one for adding the module.

Into which of these three should I put adding the module to services list? (Or should it be a fourth commit?)

This appears to be a bad rebase?

This appears to be a bad rebase?

That is an accurate assessment. I was requested to squash commits on a per file basis.
I spent the better part of a day trying to do so before giving up, and enlisting help. My help was also unable to accomplish the task.
Now my repository is in a questionable state. Any suggestions on what to do from here?

I’d probably create a new branch off of an updated master and manually recreate the commits. 🤷♂

three commits: one for adding yourself to the maintainers list, one for adding the package and one for adding the module.

Into which of these three should I put adding the commit for module to services list? (Or should it be a fourth commit?)

Before I do anything else, I believe I should await an answer for this as well.

three commits: one for adding yourself to the maintainers list, one for adding the package and one for adding the module.

Into which of these three should I put adding the commit for module to services list? (Or should it be a fourth commit?)

I'm still waiting for an answer to this question.

Into which of these three should I put adding the commit for module to services list? (Or should it be a fourth commit?)

Create a commit for each logical unit, see the relevant documentation:
https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions
https://github.com/NixOS/nixpkgs/blob/master/nixos/README.md#commit-conventions
https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#commit-conventions
There's also a ton of examples you can learn from:
https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+label%3A"8.has%3A+module+(new)"+is%3Amerged


Consent screen (EXPOSUREAPP-11910)

Spacing are not adjusted  when name is long
Figma :

PR:

I think we need different icons for dark them , see other info screens

In Turkish legal text the button "label" has to match the actual button label as in other languages , we have got some bugs regarding this before

See https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-11109 for reference


Feature request

Ability to specify maximum size of disk cache (fixed size and not using til disk out of space)
Ability to re-adjust this size on migration.

wait for it


clean up: finish unit test for config-api

What is the purpose of the change
finish unit test for config-api module, remove useless classes and move class into right packages.
Brief changelog
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #1795 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1795      +/-   ##
============================================
+ Coverage     39.28%   39.34%   +0.05%
- Complexity     4209     4216       +7
============================================
  Files           618      618
  Lines         29742    29742
  Branches       5257     5257
============================================
+ Hits          11684    11701      +17
+ Misses        16190    16176      -14
+ Partials       1868     1865       -3

Impacted Files
Coverage Δ
Complexity Δ

...ba/dubbo/remoting/transport/netty/NettyServer.java
67.85% <0%> (-3.58%)
8% <0%> (-1%)

...java/com/alibaba/dubbo/config/ReferenceConfig.java
48.33% <0%> (-0.37%)
36% <0%> (-1%)

...bo/remoting/transport/netty/NettyCodecAdapter.java
54.68% <0%> (+1.56%)
3% <0%> (ø)
:arrow_down:

...rpc/protocol/dubbo/telnet/InvokeTelnetHandler.java
55.69% <0%> (+3.79%)
13% <0%> (+2%)
:arrow_up:

...om/alibaba/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (+5.55%)
6% <0%> (+1%)
:arrow_up:

...libaba/com/caucho/hessian/util/IdentityIntMap.java
30.95% <0%> (+5.95%)
7% <0%> (+2%)
:arrow_up:

...a/dubbo/remoting/transport/netty/NettyChannel.java
66.25% <0%> (+8.75%)
21% <0%> (+2%)
:arrow_up:

...onfig/invoker/DelegateProviderMetaDataInvoker.java
100% <0%> (+18.18%)
7% <0%> (+2%)
: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 9c4017e...f24a410. Read the comment docs.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
11 seconds
sanity-inspect-image

✅ Succeeded
10 seconds
deprecated-base-image-check

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
58 seconds
clamav-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
17 seconds
sanity-label-check

✅ Succeeded
17 seconds
sanity-optional-label-check

✅ Succeeded
5 seconds
show-summary


Add an entry of Kibabii university

The university official website URL: https://kibu.ac.ke/

A URL of a page on the official website where a long-term (>1 year) IT related course is offered by the university: https://kibu.ac.ke/academic-programs/undergraduate-programmes/bachelor-of-science-in-information-technology/

@Joakim-animate90 First stage of verification passed, official domain is kibu.ac.ke as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Joakim-animate90 Pull request merged. Thank you.


Eframe unusable when rendering with WGPU backend on Wayland

Describe the bug

When running eframe with the wgpu and wayland features enabled, the result window is unresponsive to any mouse or keyboard events. Running with glow works perfectly fine as intended.
To Reproduce
Steps to reproduce the behavior:

Clone emilk/eframe_template repo.
Open Cargo.toml
Change glow in eframe features to wgpu
Add wayland feature to eframe
Run with cargo run
Try clicking or typing anything.

Expected behavior
App should work as expected like when using the Glow backend.
Desktop (please complete the following information):

OS: EndeavourOS 6.7.2-arch-1

Are you on NVIDIA, by chance? I presume this is https://github.com/gfx-rs/wgpu/issues/4775. Will most likely be fixed in the next major wgpu release (0.20), or when NVIDIA decides to fix their drivers.

@Friz64 yes, I have an Nvidia GTX 1060. I'll test it once the new release of wgpu comes out. Maybe that'll solve it.

@anantnrg it's been a few updates by now, did you try again?

@Wumpf sorry im late, but no i did not try again.

@anantnrg mind giving it another spin with the latest egui version? :)
Also if this still doesn't work, can you please provide any log output generated by RUST_LOG=trace

@Wumpf sadly, im on windows right now (ue5 works better), and dont have a working linux install. So i wont be able to test it again. sorry :(


test: add spec for blog, get-blog-post.$slug, get-all-front-matter

test: add spec for get-all-front-matter
test: add spec for get-blog-post.$slug
test: add spec for blog.
This is my first cut at adding some tests here. The guiding philosophy for this effort is summarized in the Remix discussions. It appears to be the closest thing to an official stance I can find and I personally agree with the approach.
I've stopped here in order to keep the PR at a reasonable size and to start a convo if folks disagree with the above strategy.
Note: this is also my first time using Graphite's stack approach to PRs.

Current dependencies on/for this PR:

main

PR #111

PR #112   👈

This comment was auto-generated by Graphite.

This looks nice!
The only thing I would mention, not really a criticism or a suggested change, is that Remix has something called unstable_createRemixStub in @remix-run/testing. I would bet money that stability is coming in v2 if not sooner. We may want to watch for that function since it could allow us to test our routes more easily when it's released.

This looks nice!
The only thing I would mention, not really a criticism or a suggested change, is that Remix has something called unstable_createRemixStub in @remix-run/testing. I would bet money that stability is coming in v2 if not sooner. We may want to watch for that function since it could allow us to test our routes more easily when it's released.

Speaking of, getting our website v2.0 ready is very much on my radar, really excited for an opportunity to dig into some of the new features!

Remix has something called unstable_createRemixStub in @remix-run/testing. I would bet money that stability is coming in v2 if not sooner. We may want to watch for that function since it could allow us to test our routes more easily when it's released.

i took some time to look at that method. it's not documented via remix.run. the @remix-run/testing pkg isn't listed there either. but that makes sense for something unstable.
but i dug into it anyway via the source code. if you look at the tests in particular, it's meant to mock out a whole route and child routes. that's actually really awesome. and i think that changes the game for point 3 of the philosophy i was following.

1. Unit tests loaders and action (see https://sergiodxa.com/articles/test-remix-loaders-and-actions)
2. Test with RTL any component not depending on Remix hooks or components (they need a context that you don’t have access to work)
3. Use Cypress to do E2E tests to test whole routes

you probably don't need cypress anymore to test the routes, or at least far less of it. and you could even combine the testing of loaders and the routes via this func. that's really cool.
to get back on track for the pr tho, it doesn't seem applicable for resource routes from what i can tell. you can provide a loader into unstable_createRemixStub, but the return is a JSX.Element, meaning you're expected to pass that into render and then make assertions. in this resource route context, where there is no JSX.Element to do assertions on, this stub method isn't really all that helpful unfortunately. i'd be curious how the remix team doucments this func in the future and what cases they explicitly recommend using it on. my guess is that resource routes are better off tested by j/ testing the loader method directly like i've done here. but that's j/ a guess.
so for this pr, i'm going to keep it as is. but i'll experiment w/ using that func on future tests that test a more conventional route.

@matt-hernandez, i was playing w/ unstable_createRemixStub and unfortunately i couldn't get it to work. (could use some doc on it, but ya know, it's a future thing so i can't get too upset 😅 ).
i started backing out anything remix austin specific in debugging it. i ended up with this minimal stackblitz example that my intuition says should pass, but alas it doesn't. since you have more experience with it, take a look to see if you can see what i'm doing wrong. if not, i'll pass on getting this func to work until it has some doc.

@matt-hernandez, i was playing w/ unstable_createRemixStub and unfortunately i couldn't get it to work. (could use some doc on it, but ya know, it's a future thing so i can't get too upset 😅 ).
i started backing out anything remix austin specific in debugging it. i ended up with this minimal stackblitz example that my intuition says should pass, but alas it doesn't. since you have more experience with it, take a look to see if you can see what i'm doing wrong. if not, i'll pass on getting this func to work until it has some doc.

@colbywhite I removed the loader functions from your stubbed route and the tests started passing. I looked in further and I found that the stubbed app that returns from unstable_createRemixStub has a hydrationData prop that takes in mocked loader and action data for the initial route that is supposed to render. I did this in your stackblitz and it also passed.
<RouteComponent
        initialEntries={['/blog/foo']}
        hydrationData={{ loaderData: { '/blog/foo': 'some content' } }}
      />

I can't tell if this is intentional or a mistake. My first guess says it's a mistake. hydrationData is a prop that is used in the React Router layer of Remix, and judging by this Github issue, it's also unstable.
For resource routes, I know that loader functions are called even in the stub. Here's a snippet from that Kent C. Dodds workshop I took a week ago.
const RemixStub = createRemixStub([
	{
		id: 'root',
		path: '/',
		element: <App />,
	},
	{
		id: 'resources-city-combobox',
		path: '/resources/city-combobox',
		loader,
	},
])

I know this loader is invoked in a test. If the hydrationData can get you the tests you want, use it. If not, I say we hold off until this feature becomes fully supported.


EnhancementImplement compatibility with existing maxcompute catalogs from previous versions.

before pr #40225
Proposed changes
Issue Number: close #xxx

run buildall

run buildall


Flutter V2 migration issue

There are several issue while migrating to flutter 2.0.0

error: The named parameter 'child' isn't defined. (undefined_named_parameter at [rutorrentflutter] lib/screens/torrent_details_screen.dart:139)

Flutter toast needs to be upgraded: In file included from /Users/pratikbaid/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/fluttertoast-4.0.1/ios/Classes/FluttertoastPlugin.m:2:  /Users/pratikbaid/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/fluttertoast-4.0.1/ios/Classes/UIView+Toast.h:328:19: warning: empty paragraph passed to '@param' command [-Wdocumentation]

Modal bottom sheet version outdated
../../../../Developer/flutter/.pub-cache/hosted/pub.dartlang.org/modal_bottom_sheet-0.1.6+1/lib/src/material_with_modal_page_route.dart:21:16: Error: Getter not found: 'opaque'

Shared pref outdated
pub-cache/hosted/pub.dartlang.org/shared_preferences-0.5.8/android/src/main/java/io/flutter/plugins/sharedpreferences/SharedPreferencesPlugin.java:18: warning: [deprecation] Registrar in PluginRegistry has been deprecated

5 Raised Button
'RaisedButton' is deprecated and shouldn't be used. Use ElevatedButton instead. See the migration guide in flutter.dev/go/material-button-migration-guide). This feature was deprecated after v1.25.0-8.1.pre. • lib/components/add_url_bottom_sheet.dart:101:22 • deprecated_member_use
@harchani-ritik I'd like to fix this issue

Also, the RaisedButton and FlatButton is deprecated too. It's better to use ElevatedButton or creating a custom button for our app.

I Think for migrate to flutter V2 this link might help you @pratikbaid3 .

Thank you @dhruv-kabariya. I'll have a look.


[coverage] prevent stack overflow in xtest's run of language/ir-testsuite's tests

Motivation
prevent stack overflow in xtest's run of language/ir-testsuite's tests
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
CI, the old coverage shell script.
Related PRs
None.

/land


Add ability to mount drives

This would not only be a new feature, but there a times when docker node simply crashes seemingly without any reason, but I found that it is a known issue due tue containers not started with mount option: https://github.com/SeleniumHQ/docker-selenium/issues/392
As a workaround user can send --disable-dev-shm-usage capability to prevent random container crashes

Closed via https://github.com/RationaleEmotions/just-ask/pull/26


DocumentNode.SelectNodes returns null

DocumentNode.SelectNodes returns null instead of empty collection when it doesn't find any nodes.
What is the current state on this issue? Did it end up as "won't fix"?
Currently the best workaround is this extension method but it's easy to forget to put it everywhere
public static IEnumerable<T> NullGuard<T>(this IEnumerable<T> collection)
{
	return collection ?? new T[0];
}

original discussion

Hello @poma ,
Unfortunately, we currently don't have the time to dig in the old issue tracker. We currently only fix problems that is reported in this new issue tracker.
We will try to add a global options to fix this issue.
Unfortunately, we cannot change the default behavior since some people have now code that depends on this null value returned. However, we can an option that if enabled, return an empty collection instead of null.
This issue should be completed by the start of next week.
Best Regards,
Jonathan

Thanks! I think adding it as a global option it the best choice.
Most issues in the old tracker are ancient anyway but it may worth to look at top 10 most voted ones

Yup, this is what I have already done even if I have said we didn't support the old Issue Tracker ;)
We are currently working on CSS Selector Query Engine
Best Regards,
Jonathan

Hello @poma ,
We added a new option for the HtmlDocument.OptionEmptyCollection which return an empty collection instead of null when SelectNodes return nothing.
Let me know if this new option is working correctly for you.
Best Regards,
Jonathan

Yep works perfectly

While I understand changing the behavior would end up breaking a ton of old projects that check for null instead of empty, it still fills me with rage that this wasn't the default behavior in the first place seeing as this is how C# collection objects are supposed to work by default.

I understand your frustration.
Just to let you known, I'm not the original author of the library, just the one who support it ;)
We will for sure redo this library in the future and make sure this becomes the default behavior.
Best Regards,
Jonathan


What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Fortinet.FortiClientVPN
Package Version
<IP_ADDRESS>8
Please describe the changes you would like to see
There is a new release available (<IP_ADDRESS>8).
https://links.fortinet.com/forticlient/win/vpnagent

The link provided goes to the online installer which doesn't seem to have any silent switches. I can't seem to find the links to the offline zip installers that are used for previous versions

I'm not sure if this is helpful but, once you start the online installer,
you can grab the .MSI from C:\ProgramData\Applications\Cache{GUID of
installer}{version number}
On Fri, Sep 15, 2023 at 12:03 PM Kaleb Luedtke @.***>
wrote:

The link provided goes to the online installer which doesn't seem to have
any silent switches. I can't seem to find the links to the offline zip
installers that are used for previous versions
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/winget-pkgs/issues/119570#issuecomment-1721588504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCR5ZOCMRYCYOHL5DYJP62DX2SC4LANCNFSM6AAAAAA4XBMVOI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Alternatively it's in appdata during the install wizard.
https://www.reddit.com/r/fortinet/comments/rfzq8c/msi_file_for_forticlient_vpn/
On Fri, Sep 15, 2023 at 12:49 PM Austin Bailey @.***>
wrote:

I'm not sure if this is helpful but, once you start the online installer,
you can grab the .MSI from C:\ProgramData\Applications\Cache{GUID of
installer}{version number}
On Fri, Sep 15, 2023 at 12:03 PM Kaleb Luedtke @.***>
wrote:

The link provided goes to the online installer which doesn't seem to have
any silent switches. I can't seem to find the links to the offline zip
installers that are used for previous versions
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/winget-pkgs/issues/119570#issuecomment-1721588504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCR5ZOCMRYCYOHL5DYJP62DX2SC4LANCNFSM6AAAAAA4XBMVOI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm not sure if this is helpful but, once you start the online installer, you can grab the .MSI from C:\ProgramData\Applications\Cache{GUID of installer}{version number}

Unfortunately that doesn't help, since Winget doesn't support changing which executable it is running. The downloaded installer must be able to run unattended

Check out that last reddit comment. I suspect this is how it's configured
today (using the https://filestore.fortinet.com/forticlient/ link)
On Fri, Sep 15, 2023 at 12:52 PM Kaleb Luedtke @.***>
wrote:

I'm not sure if this is helpful but, once you start the online installer,
you can grab the .MSI from C:\ProgramData\Applications\Cache{GUID of
installer}{version number}
Unfortunately that doesn't help, since Winget doesn't support changing
which executable it is running. The downloaded installer must be able to
run unattended
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/winget-pkgs/issues/119570#issuecomment-1721641475,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCR5ZOBULKZJLQFN2VOWMRLX2SIUTANCNFSM6AAAAAA4XBMVOI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Did some digging and this is the only version that I could find that isn't ancient and is at least more compatible with the later firmwares for FortiGate routers.
https://filestore.fortinet.com/forticlient/FortiClientSetup_<IP_ADDRESS>3_x64.zip


update: drop go.uber.org/multierr dependency

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area plugin-sdk
What this PR does / why we need it:
The go.uber.org/multierr dependency made our SDK require Go version >= 1.19. Since we don't really need it, it's best for us to just drop the dependency and stick to an older Go version requirement.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
update: drop go.uber.org/multierr dependency

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 132342f6f6f379011f752ef7f7960dcd9edb95da

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Update README.md to fix a typo

Pull request description
Provide description for the PR
Checklist

[ ] I have added unit tests
[ ] I have applied the change to all applicable products
[ ] I have added the change description to the changelog.md and Chart.yaml files
[ ] (Atlassian only) I have run the E2E test (if applicable)
[ ] (Atlassian only) Internal Bamboo CI is passing

@axpraka thank you! Merged


Build instructions with meson fail

Instructions in README for meson fail because there is no meson.build in the utils directory

Fixed #21


Update deps and resolve ci error.

https://circleci.com/gh/deep-c/react-redux-notify/87?utm_campaign=build-failed&utm_medium=email&utm_source=notification
https://github.com/jsdom/jsdom/issues/2304
https://github.com/facebook/jest/issues/6766
Also various devDependency's are raising security issues which should be able to be resolved by updating the version used.

closing this which fixes the bigger jsdom issue https://github.com/deep-c/react-redux-notify/pull/34 . Other vulnerability issues only pertain to dev dependencies used which we can fix separately.


[#3550] Port CloudAdapter & MSI to TypeScript samples (1/2)

Addresses # 3550
Description
This PR updates the TypeScript samples, adding support for CloudAdapter and MSI.
Detailed Changes

Updated the botbuilder dependency to the RC0 version.
Updated index.js to:

Replace BotFrameworkAdapter with CloudAdapter.
Added an instance of ConfigurationServiceClientCredentialFactory to manage credentials.
Pass the new parameters AppType and AppTenantId to the ConfigurationServiceClientCredentialFactory instance.

Added the new settings MicrosoftAppType and MicrosoftAppTenantId to the env file.
Added missing env files.
Removed devDependency @types/node from echo and core bots as it was causing issues when running npm install.

This PR updates the following samples:

00.empty-bot
02.echo-bot
03.welcome-users
05.multi-turn-prompt
06.using-cards
13.core-bot

Testing
The following images show the 13.core-bot sample properly configured and working on Azure

Promoted to MS in PR# 3573


bug: first argument in the params is not binary

GORM Playground Link

https://github.com/go-gorm/playground/pull/698
Description

I'm having an issue in my code when switching to the new gorm.io packages where I pass in a few arguments that are binary, but the first argument is not interpreted that way. I tried to recreate this as best as I could in the playground. This is the output of the query in my actual code with me changing out some variable names for privacy.
SELECT  * FROM objects WHERE  id IN (94,108,231,39,177,211,74,232,190,173,75,172,220,205,107,73,'<binary>','<binary>')  AND  user_id = '<binary>' ;

these should all be seen as <binary>

https://github.com/go-gorm/gorm/blob/5e599a07ec6aacd85b7806805908c4a78fa0e5ce/logger/sql.go#L103
From the source code, your first few parameters are printable, so they are printed out, while other parameters are not printable, so they are printed out .


increase number of domains for async state hashing

Problem
Four is too small a default for the number of domains to setup.
Solution
Let's use 8.

Newbie question: What is the purpose of domain? is it like the number of deku node that we allow to run?


Hystrix timeout + rx retry() = deadlock

We seem to have gotten into a situation where we're stuck in a deadlock. The use case:
Setup:
[] Convert the HystrixCommand into an Observable
[] Wrap this observable with a retry(x times)
When:
[] HystrixCommand times out
Then:
[] Instead of retrying the same command x times, a deadlock occurs.
Below you can see the code that exhibits this deadlock.

io.reactivex:rxjava:1.1.1
com.netflix.hystrix:hystrix-core:1.4.23

public class Command extends HystrixCommand<String> {

    public Command() {
        super(Setter.withGroupKey(asKey("test"))
                .andThreadPoolPropertiesDefaults(
                        HystrixThreadPoolProperties.Setter()
                                .withCoreSize(10)
                )
                .andCommandPropertiesDefaults(
                        HystrixCommandProperties.Setter()
                                .withExecutionTimeoutInMilliseconds(10)
                ));
    }

    @Override
    protected String run() throws Exception {
        Thread.sleep(200);
        return "response";
    }
}

public class CommandWithRetryTest {

    public static void main(String[] Args) {
        String result = Observable.just(1)
                .flatMap(i -> new Command()
                                .toObservable()
                                .retry(2) // This triggers the deadlock.
                )
                .toBlocking()
                .first(); // ===> deadlock
    }
}

@costimuraru You shouldn't use retry() on an Observable that Hystrix produces.  The Hystrix model is that each command object is single-use.  Therefore, if a command fails once, it will never produce anything different.
That said, a deadlock is pretty harsh, so I'll see why this is happening and if there's anything that Hystrix can do to produce an Observable that makes this error more tolerable.

I believe that this got fixed by #1396.  The subscribe-after-unsubscribe case was not handled well before.  In #1370, it manifested as a memory leak.  In this case, as a deadlock.  I added a unit test in #1406 to show that this is fixed in master.