EFI variant not building correctly

On aarch64 macOS:
$ make EFI=1
cargo build --release --features efi
    Finished release [optimized] target(s) in 0.03s
cp target/release/libkrun-efi.dylib target/release/libkrun-efi.1.7.2.dylib
cp: target/release/libkrun-efi.dylib: No such file or directory
make: *** [target/release/libkrun-efi.1.7.2.dylib] Error 1

$ ls -l target/release
...
libkrun.dylib
...

It seems that the generated dylib is not being named properly.
When manually copying the file over:
$ cp target/release/libkrun.dylib target/release/libkrun-efi.dylib
$ make
gcc -O2 -static -Wall  -o init/init init/init.c
init/init.c:23:10: fatal error: 'linux/vm_sockets.h' file not found
#include <linux/vm_sockets.h>
         ^~~~~~~~~~~~~~~~~~~~
1 error generated.
make: *** [init/init] Error 1

The init binary is being compiled, which should not be as the EFI feature explicitly sets BUILD_INIT = 0. This is strange, we see that the EFI feature is found because of cargo build's --features efi feature, yet some other features (like BUILD_INIT) may be overwritten?

Good catch, #170 should fix this. As for the second call, you seem to be missing EFI=1 on make there, which would explain why it's trying to build init/init.

@slp Yes, you're right. Thanks for submitting the fix.


Viosock: Take Care of Recv Requests Stucks After Pocket Shutdown

When an other end (peer) of the socket reports a shutdown, the recv request queue is not processed. This means that some recv calls may get stuck forever. This pull request changes the behavior -- when a shutdown from its peer is received, the recv queue gets processed and all requests that would get stuck are completed with STATUS_SUCCESS (which usually ends in receiving zero bytes).
This should fix #1020. I tested with

client in WIndows VM, server on the host,

both client and server within the same Windows VM.

@MartinDrab Thanks!
Waiting for CI to run before merging


Provide more accurate error prompts when the import path does not exist

When I try to compile a project with a non-existent path in import, rspack will display an error message like this:
error[internal]: Resolve error
  ┌─ src/main/index.ts:1:1
  │
1 │ ╭ import './non-existent/path'
2 │ │
  │ ╰^ Failed to resolve ./non-existent/path in typescript|<my_entry_path>

This error is emphasized on typescript, but typescript has absolutely nothing to do with the cause of this problem.
Especially in a non-code resource, such an error message will make people think that it is a loader problem rather than a path problem. For example:
error[internal]: Resolve error
  ┌─ src/main/index.ts:1:1
  │
1 │ ╭ import './non-existent/path/foo.node'
2 │ │
  │ ╰^ Failed to resolve ./non-existent/foo.node in typescript|<my_entry_path>

System:
OS: macOS 13.6.1 (22G313)
CPU: 2.6 GHz 6-Core Intel Core i7
Memory: 32 GB 2667 MHz DDR4
npmPackages:
@rsbuild/cli: 0.3.14

Source code:
// src/main/index.ts
import './non-existent/path/foo.node'

// rspack.config.js
const path = require('path');

/**
 * @type {import ('@rspack/cli').Configuration}
 */
const config = {
    entry: {
        index: path.join(__dirname, 'src/main/index.ts'),
    },
    output: {
        path: path.resolve(__dirname, 'dist'),
    }
}

May I be responsible for fulfilling this requirement? @Boshen

You may upgrade to the latest version ;-). It's been fixed.


Change timers process callback to physics

Changed all timer process callbacks to physical frames so that they are framerate independent. This prevents dialogue from causing problems with certain game features that require precise timing. (e.g. replay system that can accelerate/decelerate)
This is based on a recent PR to SceneTreeTimer: https://github.com/godotengine/godot/pull/63026

I think it makes sense to offer this as a feature, but you don't always want the timers to be based on frames, some times you need to sync with time, so an option should be added to select which one you want.
Do you think you can make that change to the PR?
It would require a new setting and check that setting for the timers to be set either by frame or time.

I think it makes sense to offer this as a feature, but you don't always want the timers to be based on frames, some times you need to sync with time, so an option should be added to select which one you want.
Do you think you can make that change to the PR? It would require a new setting and check that setting for the timers to be set either by frame or time.

Even if the timer process callback is changed to physics frame, it still be sync with time in seconds (based on delta time of physics frame), So it shouldn't cause any problems to regular users.
But I still added the corresponding setting for some special cases, or for someone doesn't like timing in physical frames. (Idle process callback is the default value)
We may also be able to instructing users to enable physical frame timing in the documentation to solve issues caused by erratic idle timing?


docker confuses nvm's detection of npm prefix  conflict

I'm having trouble using nvm in a Docker container. Every time I run nvm, it complains that the npm config prefix option is set.  Deleting any npm prefix option doesn't help, and I'm 99.9% sure that I'm not setting it anywhere.
The problem seems to be that the actual path of the home directory (and thus .nvm directory) is /home.host/griscom, but there's a symlink from /home to /home.host, and the home directory is actually configured as /home (which is the value of $HOME). (Setting HOME to /home.host/griscom doesn't help.)

Operating system and version:

Ubuntu 18.04 under Docker version 18.09.0, build 4d60db4; hosted on macOS 10.13.6. nvm version 0.33.11 .

nvm debug output:

nvm --version: v0.33.11
$SHELL: /bin/sh
$SHLVL: 1
$HOME: /home/griscom
$NVM_DIR: '$HOME/.nvm'
$PATH: /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/opt/local/Library/Frameworks/Python.framework/Versions/3.4/bin:$HOME/bin
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
/bin/sh: 0: Illegal option --
shell version: ''
uname -a: 'Linux 4.9.125-linuxkit #1 SMP Fri Sep 7 08:20:28 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux'
OS version: Ubuntu 18.04.1 LTS
curl: /usr/bin/curl, curl 7.58.0 (x86_64-pc-linux-gnu) libcurl/7.58.0 OpenSSL/1.1.0g zlib/1.2.11 libidn2/2.0.4 libpsl/0.19.1 (+libidn2/2.0.4) nghttp2/1.30.0 librtmp/2.3
wget: /usr/bin/wget, GNU Wget 1.19.4 built on linux-gnu.
git: /usr/bin/git, git version 2.17.1
grep: /bin/grep, grep (GNU grep) 3.1
awk: /usr/bin/awk, GNU Awk 4.1.4, API: 1.1 (GNU MPFR 4.0.1, GNU MP 6.1.2)
sed: /bin/sed, sed (GNU sed) 4.4
cut: /usr/bin/cut, cut (GNU coreutils) 8.28
basename: /usr/bin/basename, basename (GNU coreutils) 8.28
rm: /bin/rm, rm (GNU coreutils) 8.28
mkdir: /bin/mkdir, mkdir (GNU coreutils) 8.28
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.7.0-git
nvm current: system
which node: /usr/bin/node
which iojs:
which npm:
npm config get prefix: bash: npm: command not found
npm root -g: bash: npm: command not found

nvm ls output:

        v8.12.0
->       system
node -> stable (-> v8.12.0) (default)
stable -> 8.12 (-> v8.12.0) (default)
iojs -> N/A (default)
lts/* -> lts/dubnium (-> N/A)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.16.0 (-> N/A)
lts/carbon -> v8.15.0 (-> N/A)
lts/dubnium -> v10.15.0 (-> N/A)

How did you install nvm? (e.g. install script in readme, Homebrew):

curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.33.11/install.sh | bash

What steps did you perform?

nvm use 8.12.0

What happened?

nvm output the following error message and then failed:
nvm is not compatible with the npm config "prefix" option: currently set to "/home.host/griscom/.nvm/versions/node/v8.12.0"
Run `nvm use --delete-prefix v8.12.0` to unset it.

What did you expect to happen?

for "nvm use" to execute properly

Is there anything in any of your profile files (.bashrc, .bash_profile, .zshrc, etc) that modifies the PATH?

No.
The problem seems to be confusion between /home/griscom and /home.host/griscom. My Docker startup command is:
docker run  \
		--hostname dev-ubuntu-18.04 \
		--mount src=/Users/griscom,target=/home.host/griscom,type=bind \
		--mount src=dev-ubuntu-18.04,target=/data,type=volume \
		-ti dev-ubuntu-18.04:latest griscom 501 staff 20 host

This maps my home directory to the Docker-internal directory /home.host/griscom. Then, inside Docker, /home is linked to /home.host.
Somehow, npm and nvm have different ideas of which is being used: /home.host or /home.
Below is some more info, all generated inside the Docker instance. The biggest key is that echo ~ yields /home/griscom, while npm config get prefix yields /home.host/griscom/.nvm/versions/node/v8.12.0, which has /home.host rather than /home.
griscom@dev-ubuntu-18 $ nvm exec 8.12.0 npm install --production
nvm is not compatible with the npm config "prefix" option: currently set to "/home.host/griscom/.nvm/versions/node/v8.12.0"
Run `nvm use --delete-prefix v8.12.0 --silent` to unset it.
Running node v8.12.0
nvm is not compatible with the npm config "prefix" option: currently set to "/home.host/griscom/.nvm/versions/node/v8.12.0"
Run `nvm use --delete-prefix v8.12.0` to unset it.
griscom@dev-ubuntu-18 $ echo $?
127
griscom@dev-ubuntu-18 $ echo $PATH
/home/griscom/.nvm/versions/node/v8.12.0/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/opt/local/Library/Frameworks/Python.framework/Versions/3.4/bin:/home/griscom/bin
griscom@dev-ubuntu-18 $ echo ~
/home/griscom
griscom@dev-ubuntu-18 $ npm config get prefix
/home.host/griscom/.nvm/versions/node/v8.12.0
griscom@dev-ubuntu-18 $ npm config delete prefix
griscom@dev-ubuntu-18 $ npm config get prefix
/home.host/griscom/.nvm/versions/node/v8.12.0
griscom@dev-ubuntu-18 $ ls -l /
total 72
drwxr-xr-x   1 root root 4096 Jan  4 17:57 bin/
drwxr-xr-x   2 root root 4096 Apr 24  2018 boot/
drwxr-xr-x   4 root root 4096 Jan  4 18:18 data/
drwxr-xr-x   5 root root  360 Jan  4 20:26 dev/
-rwxr-xr-x   1 root root 1880 Jan  4 18:18 docker-entrypoint.sh*
drwxr-xr-x   1 root root 4096 Jan  4 20:26 etc/
lrwxrwxrwx   1 root root   10 Jan  4 20:26 home -> /home.host/
drwxr-xr-x   3 root root 4096 Jan  4 20:26 home.host/
drwxr-xr-x   1 root root 4096 Jan  4 17:57 lib/
drwxr-xr-x   2 root root 4096 Dec  4 17:11 lib64/
drwxr-xr-x   2 root root 4096 Dec  4 17:11 media/
drwxr-xr-x   2 root root 4096 Dec  4 17:11 mnt/
drwxr-xr-x   2 root root 4096 Dec  4 17:11 opt/
dr-xr-xr-x 215 root root    0 Jan  4 20:26 proc/
drwx------   2 root root 4096 Dec  4 17:12 root/
drwxr-xr-x   1 root root 4096 Dec 28 23:22 run/
drwxr-xr-x   1 root root 4096 Dec 28 23:22 sbin/
drwxr-xr-x   2 root root 4096 Dec  4 17:11 srv/
dr-xr-xr-x  13 root root    0 Jan  4 20:26 sys/
drwxrwxrwt   1 root root 4096 Jan  4 21:42 tmp/
drwxr-xr-x   1 root root 4096 Dec  4 17:11 usr/
drwxr-xr-x   1 root root 4096 Dec  4 17:12 var/
griscom@dev-ubuntu-18 $  df /home.host/
Filesystem     1K-blocks    Used Available Use% Mounted on
/dev/sda1       61252420 9157944  48953308  16% /home.host
griscom@dev-ubuntu-18 $ cat ~/.npmrc
init.author.name=Dan Griscom
<EMAIL_ADDRESS>init.author.url=http://suitable.com
//registry.npmjs.org/:_authToken=4e6a3c35-dfae-4293-9bdd-93099c4f8ae1
griscom@dev-ubuntu-18 $ echo $SHELL
/bin/sh
griscom@dev-ubuntu-18 $ nvm exec 8.12.0 npm install --production
nvm is not compatible with the npm config "prefix" option: currently set to "/home.host/griscom/.nvm/versions/node/v8.12.0"
Run `nvm use --delete-prefix v8.12.0 --silent` to unset it.
Running node v8.12.0
nvm is not compatible with the npm config "prefix" option: currently set to "/home.host/griscom/.nvm/versions/node/v8.12.0"
Run `nvm use --delete-prefix v8.12.0` to unset it.
griscom@dev-ubuntu-18 $

Hi, this issue seems like it could be closed.

Indeed; duplicate of #617.


Included hyperlinks to the references

Thanks!
Send along any cool art you make,
or make a pr to the art database at
https://github.com/kylemath/p5.eegedu.art
and it will automatically load int othe site on the dropdown menu for others to see

sweet, thanks kyle!


problam with checkbox input-field position

Hi,
In this file at line 4792 the value of left attribute of  checkbox input-fields (which have absolute position) is set to -9999px.
Why is that?
It makes my screen unnecessarily wide.
Thanks!

That's the question to https://github.com/Dogfalo/materialize
But probably you do something wrong or use too old browser.

OK! BTW - There is an open issue for this problem...


Beta

Codecov Report

Merging #266 (c53b7bd) into develop (1026dbe) will decrease coverage by 0.55%.
The diff coverage is 82.89%.

@@             Coverage Diff             @@
##           develop     #266      +/-   ##
===========================================
- Coverage    85.68%   85.13%   -0.56%
===========================================
  Files           17       17
  Lines          517      565      +48
  Branches       132      143      +11
===========================================
+ Hits           443      481      +38
- Misses          17       22       +5
- Partials        57       62       +5

Impacted Files
Coverage Δ

src/interfaces/listr.interface.ts
100.00% <ø> (ø)

src/listr.ts
88.46% <0.00%> (ø)

src/renderer/verbose.renderer.ts
72.72% <0.00%> (-4.70%)
:arrow_down:

src/lib/task.ts
80.73% <73.68%> (-2.43%)
:arrow_down:

src/utils/logger.ts
86.27% <80.00%> (-0.69%)
:arrow_down:

src/renderer/default.renderer.ts
86.98% <90.00%> (+0.77%)
:arrow_up:

src/interfaces/state.constants.ts
100.00% <100.00%> (ø)

src/lib/task-wrapper.ts
90.00% <100.00%> (ø)

src/utils/logger.constants.ts
100.00% <100.00%> (ø)

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 1026dbe...1b048c2. Read the comment docs.

Codecov Report

Merging #266 (c53b7bd) into develop (1026dbe) will decrease coverage by 0.55%.
The diff coverage is 82.89%.

@@             Coverage Diff             @@
##           develop     #266      +/-   ##
===========================================
- Coverage    85.68%   85.13%   -0.56%
===========================================
  Files           17       17
  Lines          517      565      +48
  Branches       132      143      +11
===========================================
+ Hits           443      481      +38
- Misses          17       22       +5
- Partials        57       62       +5

Impacted Files
Coverage Δ

src/interfaces/listr.interface.ts
100.00% <ø> (ø)

src/listr.ts
88.46% <0.00%> (ø)

src/renderer/verbose.renderer.ts
72.72% <0.00%> (-4.70%)
:arrow_down:

src/lib/task.ts
80.73% <73.68%> (-2.43%)
:arrow_down:

src/utils/logger.ts
86.27% <80.00%> (-0.69%)
:arrow_down:

src/renderer/default.renderer.ts
86.98% <90.00%> (+0.77%)
:arrow_up:

src/interfaces/state.constants.ts
100.00% <100.00%> (ø)

src/lib/task-wrapper.ts
90.00% <100.00%> (ø)

src/utils/logger.constants.ts
100.00% <100.00%> (ø)

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 1026dbe...1b048c2. Read the comment docs.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How does an admin create an app password?

This document focuses on a user generating their own app password, but surely an Admin should be able to create an app password for, say, a mailbox that is used to authenticate a printer for smtp?  Is the only way really to log in as that user and create the app password?  If so, it should be stated, else, info should be added to say how the admin can do it. :-)

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

ID: 8751595d-4839-d2ee-1e2c-16d07ef84a2d
Version Independent ID: e66f96b3-5b88-9304-1ec0-9e3bb8a2cfbe
Content: How to manage app passwords - Azure Active Directory
Content Source: articles/active-directory/user-help/multi-factor-authentication-end-user-app-passwords.md
Service: active-directory
Sub-service: user-help
GitHub Login: @curtand
Microsoft Alias: curtand

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

I'm also interested in the answer to this.

Hey @bcraigie
Thanks for providing feedback that helps improve our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response.  We are currently updating all the end-user content and will be migrating to the Microsoft Support site. We are closing the issue as we are migrating the content to a new site. #please-close


Orion check not working anymore

The check inside train.py, to see if we are using Orion, does not work anymore.
it should be: orion.client.cli.IS_ORION_ON (inside the train method)

solved in #21

solved in #21


Unicode characters got broken

With sourcegraph chrome extension, Unicode characters on GitHub source files are broken.
Expected

Actual

Link to above sample:
https://github.com/open-korean-text/open-korean-text-wrapper-node-2/blob/master/test/tokens.spec.js

Thanks for reporting, @rokoroku ! We are reverting a change that caused this tomorrow.

Hey @rokoroku , just double checked, and the latest deployment of the Chrome extension fixed this. Thanks again for reporting!


[orf.at] Unsupported URL and using generic extractor instead of orf.py

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
all countries
Provide a description that is worded well enough to be understood
Hello guys. Got two links for test:
https://oe1.orf.at/programm/20240916/769302/Playgrounds
https://sound.orf.at/collection/2/72427/rattle-dirigiert-6-mahler
Both fail to download. yt-dlp will switch to generic extractor after that giving unsupported url error.
however audio files can be located manually by this links:
https://loopstream01.apa.at/?channel=oe1&shoutcast=0&player=radiothek_item_v1&referer=oe1.orf.at&id=2024-09-16_1405_tl_51_7DaysMon20_2491805.mp3&_=1726498267358
https://loopstream01.apa.at/playlist.m3u8?channel=oe1&id=2024-09-15_1103_tl_51_7DaysSun14_2491007.mp3&offset=0&offsetende=5271949&shoutcast=0&player=sound_v1&referer=sound.orf.at&userid=9da0d70b-9d2f-415d-9607-38bc356523d8&cb=1726499239020
Provide verbose output that clearly demonstrates the problem

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

Complete Verbose Output
./yt-dlp -Uv -F "https://sound.orf.at/collection/2/72427/rattle-dirigiert-6-mahler"
/Library/Frameworks/Python.framework/Versions/3.12/Resources/Python.app/Contents/MacOS/Python: can't open file './yt-dlp': [Errno 1] Operation not permitted
sy@SYs-MacBook-Air yt-dlp % ./yt-dlp -vU -F "https://sound.orf.at/collection/2/72427/rattle-dirigiert-6-mahler"
[debug] Command-line config: ['-vU', '-F', 'https://sound.orf.at/collection/2/72427/rattle-dirigiert-6-mahler']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [4d9231208] (zip)
[debug] Python 3.12.2 (CPython arm64 64bit) - macOS-14.6.1-arm64-arm-64bit (OpenSSL 3.0.13 30 Jan 2024)
[debug] exe versions: phantomjs 2.1.1, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, curl_cffi-0.7.1, mutagen-1.47.0, requests-2.32.3, sqlite3-3.45.1, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1835 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[generic] Extracting URL: https://sound.orf.at/collection/2/72427/rattle-dirigiert-6-mahler
[generic] rattle-dirigiert-6-mahler: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] rattle-dirigiert-6-mahler: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://sound.orf.at/collection/2/72427/rattle-dirigiert-6-mahler
Traceback (most recent call last):
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1626, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1761, in __extract_info
    ie_result = ie.extract(url)
                ^^^^^^^^^^^^^^^
  File "./yt-dlp/yt_dlp/extractor/common.py", line 741, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "./yt-dlp/yt_dlp/extractor/generic.py", line 2526, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://sound.orf.at/collection/2/72427/rattle-dirigiert-6-mahler

sound.orf.at is a duplicate of https://github.com/yt-dlp/yt-dlp/issues/9581
For oe1.orf.at it looks like the regex needs to be expanded
diff --git a/yt_dlp/extractor/orf.py b/yt_dlp/extractor/orf.py
index 9c37a54d6..f156e96fb 100644
--- a/yt_dlp/extractor/orf.py
+++ b/yt_dlp/extractor/orf.py
@@ -44,7 +44,7 @@ class ORFRadioIE(InfoExtractor):

     _VALID_URL = rf'''(?x)
         https?://(?:
-            (?P<station>{_STATION_RE})\.orf\.at/player|
+            (?P<station>{_STATION_RE})\.orf\.at/(?:player|programm)|
             radiothek\.orf\.at/(?P<station2>{_STATION_RE})
         )/(?P<date>[0-9]+)/(?P<show>\w+)'''

In the upstream version of the extractor, the sound.orf.at URL is already handled, and the oe1.orf.at/programm URL works after changing /player/' in the URL pattern to /(?:player|programm)/`. As a work-around you can use  https://oe1.orf.at/player/20240916/769302/Playgrounds which appears also to work with yt-dlp.
Where are links like oe1.orf.at/programm found?


RSS template should use PublishDate

<pubDate>{{ .Date.Format "Mon, 02 Jan 2006 15:04:05 -0700" | safeHTML }}</pubDate>

I'd like to pick this up but is there a specific template that this should be applied to or is this just generic updating all themes rss templates?  Mostly asking since I can't find an RSS template in the examples and from searching the hugo codebase.

@wwsean08 I think we will have to wait until we decide what to do with #3977 -- this isn't as clear as I thought when I opened this issue.

Hey @bep, any thoughts since your last comment? Hate to let it go stale :smile:


SSR issues (4.3.0)

Hi,
I have SSR issues in a Gatsby (theme novela) due to this lib.
I saw the SSR fixes, tried versions like 4.0 or 4.3 and it didn't improve my issue.
You can see it live in production at https://sebastienlorber.com/atomic-css-in-js
Repro is my own Gatsby site: https://github.com/slorber/sebastienlorber.com
Before hydration, the page looks like:

After hydration, the page looks like:

I know there is no image in the screenshot, but it's definitively the images that are the cause of this problem.
My Gatsby theme uses this lib here: https://github.com/narative/gatsby-theme-novela/blob/master/%40narative/gatsby-theme-novela/src/components/Image/Image.Zoom.tsx
If I tweak the source code, and make "Controlled.js" return "children", my markup issues get fixed.
Not sure exactly what's the problem, but that seems related to https://joshwcomeau.com/react/the-perils-of-rehydration

@slorber You're in luck! I've got some time off work coming up after tomorrow, so I'll have time to put towards updating things here.
If you really want to help me resolve this issue ASAP, can you 🙏  create an example repository that only exposes this issue? That'll help me narrow in and resolve it so much faster

Hi,
Will try to get a better repro, but for now what I have is this minimal MDX on my site:
---
title: Atomic CSS-in-JS
slug: atomic-css-in-js
author: Sébastien Lorber
date: 2020-04-27
hero: ./images/hero.png
excerpt: ...or how to scale your CSS-in-JS.
secret: false
---

![](./images/rnw-rules-order.png)

```jsx
import {t} from 'react-native-tailwindcss';

<View style={[t.absolute, t.inset0, t.p4, t.bgBlue500]} />

Note: This predictable behavior is only possible thanks to using the most strict atomic CSS.

It seems I have to add both the image zoom + the code to reproduce this issue. Will try to see how to isolate that more but it could take time.

FYI I'm almost done completely rewriting this library into:

a vanilla JS package that is ~3kb (minified & gzipped)
a react hook that uses 👆  and is maybe 100-150 bytes

This and all the other issues should be closed out by it. I'm almost there (working out some API stuff).
Thanks for your patience

There is an alpha release out (no documentation nor tests yet): 5.0.0-alpha.1
This has condensed the "react" nature of this to nothing more than a react hook wrapping<EMAIL_ADDRESS>For gatsby, it looks like their gatsby-image might not accept ref (let me know if it does), which sucks, but good news: you can do this:
const { ref } = useImageZoom({ /* image zoom options here */ })

return (
  <div ref={ref}>
    <Image ... />
  </div>

However, it is not guaranteed to have all the refined parts that using it directly on an img tag would.
I'm considering doing a check to see if the current target is NOT an img, but the child IS an img, then just attach it to that -- maybe through a type flag with the value 'find-first-img' or something, but I haven't thought it out fully.

This looks great! The github repo link on image-zoom's npm page gives a 404. Can you please link the repo here?

The homepage should now work for it; some life stuff came up, and I have a lot of build work to do on this and a couple of issues to resolve before I can get everything in sync.

thanks @rpearce
I'm closing because actually updating my gatsby / novela did fix the SSR issue I had, so it was probably not related to this lib finally...
(fixed while still on 4.3.1)


closed: received a video packet, but track is not set up

Which version are you using?
1.9
Which operating system are you using?
Linux amd64 standard
Describe how to replicate the issue

start the server
publish with  gliveT80 rtmp
read with

Server logs
closed: received a video packet, but track is not set up，Sometimes it succeeds, using nginx-rtmp will definitely succeed
Network dump
No response

Hello, this compatibility issue can be solved if you provide a network dump, that can be generated in this way:

Download wireshark (https://www.wireshark.org/)
Start capturing on the interface used for exchanging packets

If the server and the external hardware or software are both installed on your pc, the interface is probably "loopback", otherwise it's the one of your network card.

Start the server and replicate the issue
Stop capturing, save the result in .pcap format
Attach


8497.1 Add a migration in system codes table for status - timeout.

https://github.com/Shippable/pm/issues/8497

This is done


added support for Bruker TSF format

added support for Bruker TSF format
fixed timsTOF instrument type not being recorded in instrumentConfiguration

updated PSI-MS CV to 4.1.99

/squash

@brendanx67 What do you think about including this in 22.2?


REPL parse failure

Welcome to Scala 3.3.1-RC1-bin-SNAPSHOT-git-5262680 (20.0.1, Java OpenJDK 64-Bit Server VM).
Type in expressions for evaluation. Or try :help.

scala> final val x = 42
-- [E088] Syntax Error: ------------------------------------------------------------------------------------------------
1 |final val x = 42
  |      ^^^
  |      Expected start of definition
  |
  | longer explanation available when compiling with `-explain`

Expectation
That seems like a natural thing to do in the REPL.

Scala 2:
$ scala-cli repl -S 2.13
Welcome to Scala 2.13.14 (OpenJDK 64-Bit Server VM, Java 17.0.8).
Type in expressions for evaluation. Or try :help.

scala> final val x = 42
val x: Int(42) = 42

scala> val x = 42
val x: Int = 42

Scala 3:
$ scala-cli repl -S 3.4
Welcome to Scala 3.4.2 (17.0.8, Java OpenJDK 64-Bit Server VM).
Type in expressions for evaluation. Or try :help.

scala> final val x = 42
-- [E088] Syntax Error: --------------------------------------------------------
1 |final val x = 42
  |      ^^^
  |      Expected start of definition
  |
  | longer explanation available when compiling with `-explain`
scala> final def x = 42
-- [E088] Syntax Error: --------------------------------------------------------
1 |final def x = 42
  |      ^^^
  |      Expected start of definition
  |
  | longer explanation available when compiling with `-explain`

Why should this work; shouldn't final only be available inside classes?
What is the use-case for this?

Note: you get the same error when trying to define a final val in a method for example:
➜  ~/scala-snippets-5 cat finalVal.scala
package finalVal

@main def Main =
  final val x = 42
  ()
➜  ~/scala-snippets-5 scala-cli run finalVal.scala
Compiling project (Scala 3.4.1, JVM (17))
[error] ./finalVal.scala:4:9
[error] Expected start of definition
[error]   final val x = 42
[error]         ^^^
Error compiling project (Scala 3.4.1, JVM (17))
Compilation failed

Note from @dwijnand: -explain actually doesn't show anything useful here.

This issue was picked for the Scala Issue Spree of today, June 11th. @aherlihy, @mbovel, @bracevac, @EugeneFlesselle will be working on it.

Note from @dwijnand: -explain actually doesn't show anything useful here.

Scratch that, I wasn't compiling it correctly (getting other compile errors on what I was working on).

Ah, indeed:
➜  ~/scala-snippets-5 scala-cli run -O -explain finalVal.scala
Compiling project (Scala 3.4.1, JVM (17))
[error] ./finalVal.scala:4:9
[error] Expected start of definition
[error]
[error] Explanation
[error] ===========
[error] You have to provide either class, trait, object, or enum definitions after qualifiers
[error]   final val x = 42
[error]         ^^^
Error compiling project (Scala 3.4.1, JVM (17))
Compilation failed

However, this message also doesn't seem particularly helpful.


Clear selection w/out showing the drop-down

Using the single select option, which allows me to clear the current selection, is it possible to allow the user to select the clear button (x) but not show the drop-down when it is selected?  Otherwise, to clear the selection, I must click the (x), then click out of the control to hide the drop-down.

can $event.stopPropagation() be implemented to prevent this behaviour?

Is there a workaround for this?


[Event Request] Codeunit 353 Item Availability Forms Mgt - function CalculateNeed

Dear,
I request for below Integration

Definition of Integration as below

thanks & regards
Ashwini Tripathi

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Management JVM support

Fixes #1564

Shouldn't we make it more explicit (e.g. in limitations.adoc) that this extension will stay JVM only due to limitations inherent to GraalVM?

Shouldn't we make it more explicit (e.g. in limitations.adoc) that this extension will stay JVM only due to limitations inherent to GraalVM?

Yes, I was thinking along similar lines so I can add some notes about that.

Latest commit adds limitations.adoc.


Document "ROI_sets Surface_schemes" and where to find them into the readthedocs

Document all the ABCD-BIDS ROI_sets Surface_schemes into the readthedocs DBP section(s).

planned for later release


EPIC: NFC/RFID integration

In order to track spools (and to prepare for a Spoolman integration) additional spool information such as vendor, material, color etc could be stored on an RFID/NFC sticker that is added to the spool and read once the spool is weighted. Requires some changes to the JSON de/serialization, MQTT client and 3D models

writing: https://github.com/pcbreflux/espressif/blob/master/esp32/arduino/sketchbook/ESP32_RFID_Write/ESP32_RFID_Write.ino

information that needs to be stored on the tag:

spool id
manufacturer (name or id) - for UI
material (name or id) - for UI
spool weight in grams

information that could be stored on the tag

name (needs to be size limited!) - for UI
color
purchase date / first scan date?

considerations

spool id and spool weight aren't changing -> tag sector 0, block 1 & 2
material and manufacturer  -> tag sector 1, all data blocks
name -> tag sector 2, all data blocks
color as hex ->tag sector 3, block 0
date -> tag sector 3, block 1
with a simple REST interface (or a connection to Spoolman) all non-essential data could be pulled via spoolid/tagid

closed with 83105cf


Update minor spelling change in useExportData.md

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow reference images to be added to test report, even if passed

Hey 👋
We automatically generate test reports using XCTestHTMLReport and distribute this to members of our team (people who may not necessarily have quick access to our repository), all as part of our app release process.
We wanted to add the ability to always add reference images (and string diffs) to the test report, even if the test passes. This is so we can visually verify what "success" actually means.
This is not a breaking change, and reports will only increase in size if users add attachReferenceImages: true to their assertion calls.
I've also added the unique "name" (which has already been passed in previously) to the attachment activity. This is because we have multiple assertions in a single test which means it was difficult to distinguish between the different activities. -- This will be added automatically for any use who provides a name.
For those interested, this is how it looks using the report tool linked above 😄

Noticing a failure on setting XCTAttachment.name - is this a linux-only issue that I just need to wrap?

Confirmed this is now working for us. Good from my side 👍

Thanks for the PR, this sounds interesting!
Just so I understand, when you use this do you really add attachReferenceImages: true to all of your assertSnapshot invocations? Or do you create your own assert helper that always passes true for the argument?
I'm mostly wondering if this is something you tend to turn on for all tests, or do you really need to be able to opt in and out of it depending on the test.

Our test suite has a small wrapper over this library to help with running each test in a large number of configurations - so yeah we just have one place that we update.
Happy to move to using environment variables if you'd rather that? (I personally find them a little less readable, especially as they're not documented anywhere currently!)

Rebased to get it up to date 👍 If you'd like it moving to ENV vars just give me a shout and I can make that change 😄

Would this be able to get merged soon?

I'd loved to have seen this merged, in fact we use it internally as our build since this is such an important feature to us!
If Brandon/Stephen confirm what changes they want I'm happy to make them to get it in 👍


chat archiver

The chat archiver saves chat logs in JSON "batch" files under /CHANNEL/chat/HOUR/.
It uses backfiller to sync batch files between machines, and has a "merge" process where two
batches for the same time range are turned into one. In this way machines converge on a single
canonical log, even if one of them misses something.
See the documentation in the PR for more.

@HubbeKing any idea why building the chat archiver is failing with denied: permission_denied: write_package on push? I've given github actions write permission to the repo:

Since it's only chat_archiver that's failing, there's probably something fucky with specifically that package's permissions? Try comparing the settings of https://github.com/ekimekim/wubloader/pkgs/container/wubloader-chat_archiver to https://github.com/ekimekim/wubloader/pkgs/container/wubloader-cutter ? Might be something subtly different and/or weird

That was definitely the issue before, but I fixed it and still got the same error. Maybe I need to force an entirely new build before the credentials will update?

Nope, still broken.

I don't get it. According to the docs, the GITHUB_TOKEN should be granted the permissions noted in the workflow file, and it clearly is, since it's successfully pushing the other component images. Could be something with specifically PRs? https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/enabling-features-for-your-repository/managing-github-actions-settings-for-a-repository#preventing-github-actions-from-creating-or-approving-pull-requests

I'm going to try merging to see if it works on master?


Create Docker Box

It would be amazing if we create a Docker Box with all the ready to run Optimus. Spark, Optimus,  Jupiter Notebook, etc. It seems the best way to start https://github.com/jupyter/docker-stacks/tree/master/all-spark-notebook

Seems great. Waiting for the PR 👍


getRawValue returns the incomplete value

When I type the value (85) 99999-8877 the value returned by getRawValue removes the last character:<PHONE_NUMBER>
Other example:

Nativebase: ^2.12.1
react-native-masked-text: ^1.9.2 (already tested with the latest version, the same problem occurs)
This problem was not happening, it happened after I updated the version of nativebase. This happened to occur with all the masks.
Has anyone ever had this problem?

Hey man, the problem is because you are getting the raw value before setting the state.
To avoid this behavior, use this helper:
https://github.com/benhurott/react-native-masked-text#including-the-rawtext-in-onchangetext-1120
If you still have a problem, feel free to reopen the issue, ok?

This suggestion solved my problem. Thank you

Component
<TextInputMask         includeRawValueInChangeText={true}         type={type}         value={value}         options={options}         onEndEditing={onEndEditing}         onChangeText={onChangeText}         style={{fontSize: 20, paddingLeft: 10, width: '100%'}}         placeholder={placeholder}         onChange={onChange}         ref={ref}       />
Screen
<InputMask
        value={price}
        onChangeText={(masked, unmasked) => setPrice(unmasked)}
        placeholder="Valor total"
        type="money"
      />


Autoloading, Default Mapping implementation

This commit adds an Vim-autoloading file for the provided function, changes the
implementation of default mapping from global variable to if statements, remove
injection of global variable if not added by the user with get(g:,'', default).
Changes some double quotes to single quotes as preferred by vint.

Hey @CantoroMC thanks for the update! I'll want to poke at this just a bit locally to make sure things still work, but wanted to let you know I'd seen it and assuming everything looks good I'll likely merge in a few days.

Thanks @christoomey, your Youtube videos got me into Vim.
Additionally, this plugin is very simple and useful and i've thinked to make my contribution rather than add it locally.
I already tested the plugin before and after the changes and works as expected, especially since there are not changes to the operator itself but only to structure of the plugin.
Bye

So glad you found value in the videos! I've tested out locally and all looks good so I've now merged this. Thanks for the tweak!


Make Lombok and Javadoc get along

Some Maven profile work left in both delombok and documentation profiles of parent POM.

Fixed w/ 0a6ddb1f7d5e84b71dbcc7a349d474cdcbbf2b7c.


DisableLeaveMessages does not disable leave messages.

Upon leaving the server a message is still displayed.
'Foxikle left the game.'
No join messages are displayed.

This issue was resolved in v.16


pass RV.sample(*args, **kwargs) to tfp.distribution.sample(*args, **kwargs)

https://github.com/pymc-devs/pymc4/blob/1a30106869f57b545bcb67bd9b10d4aff996c233/pymc4/random_variables/random_variable.py#L129
why would you want to limit yourself to only sample once per call?

Not sure of design choice here. @twiecki and @junpenglao?

What value is pymc adding anyways to distributions? arithmetic ops?

In the long-run, a hopefully more user-friendly API than the TFP API. The pm3 API was superb; at least for myself, it's facilitated learning and solving problems, formulating models, and modelling in general.

tfp distributions methods resemble pm3 RVs. Of course, the magic is in how you put the RVs together.

tfp distributions methods resemble pm3 RVs

Yes, the only difference being that TFP has distribution parameterizations that are more programmer-friendly, and less statistician-friendly.

Of course, the magic is in how you put the RVs together.

Agreed! Two days ago, was trying to implement a zero-inflated Beta distribution. Colleague and I sorta got it working, probably could be better; pm3's API was superb for doing this.

why would you want to limit yourself to only sample once per call?

This is actually the safest choice to avoid incorrect shape broadcasting. Take the model in https://docs.pymc.io/notebooks/hierarchical_partial_pooling.html as an example, if you were to write a sample function for this model in pure TFP, you might put it down as such:
def custom_forward(nmc, n_obs):
  phi = tfd.Uniform(low=a, high=b).sample(nmc)
  kappa_log = tfd.Exponential(rate=tf.cast(1.5, dtype)).sample(nmc)
  thetas = tfd.Beta(concentration1=tf.tile((tf.exp(kappa_log)*phi)[:, None], (1, n_obs)),
                    concentration0=tf.tile((tf.exp(kappa_log)*(1.0-phi))[:, None], (1, n_obs))).sample(nmc)
  y = tfd.Binomial(
      total_count=tf.cast(at_bats, dtype),
      probs=thetas
  ).sample(nmc) <== This wont work as Binomial does not have sample_n yet
  return phi, kappa_log, thetas, y

Note the number of MC sample is defined as nmc. However, if you run the function like custom_forward(4, 18), you would actually get incorrect (or rather, unintuitive) tensors:
(<tf.Tensor 'Uniform/sample/Reshape:0' shape=(4,) dtype=float64>,
 <tf.Tensor 'Exponential/sample/Reshape:0' shape=(4,) dtype=float64>,
 <tf.Tensor 'Beta/sample/Reshape:0' shape=(4, 4, 18) dtype=float64>,
 <tf.Tensor 'Binomial/sample/Reshape:0' shape=(4, 4, 4, 18) dtype=float64>,

More problematic, the result is not the same as when you run:
for i in range(4):
    ... = custom_forward(1, n_obs)

The (stacked) result will have a different but now correct shape:
(<tf.Tensor 'Uniform/sample/Reshape:0' shape=(4,) dtype=float64>,
 <tf.Tensor 'Exponential/sample/Reshape:0' shape=(4,) dtype=float64>,
 <tf.Tensor 'Beta/sample/Reshape:0' shape=(4, 18) dtype=float64>,
 <tf.Tensor 'Binomial/sample/Reshape:0' shape=(4, 18) dtype=float64>,

the reason is that where in the context of Bayesian model checking, what you actually want is that the last two RV does not do additional boardcasting.
So at the end, the safest way is to do sample() (or sample(1) to make sure you dont end up accidentally broadcasting something). Moreover, sample() dont require user to always think about batch shape (which could be tricky or even impossible in some cases).

tfp distributions methods resemble pm3 RVs. Of course, the magic is in how you put the RVs together.

TFP distribution does not resemble RV, that's why you need Edward2 inside TFP to actually do that.

Close for now, but feel free to keep discussing here!

TFP distribution does not resemble RV, that's why you need Edward2 inside TFP to actually do that.

Ah yes, @junpenglao is correct! I was not thinking straight when I responded. :smile:

@junpenglao your code style has become Googlified!

@ericmjl If you meant the 2 space yep I am guilty as charged...


validator on prop change

What problem does this feature solve?
The validator on props are executed against the passed-in props on creation, but if it was binding, the value of the prop can be updated to bypass validation.
What does the proposed API look like?
No API change. Just validation checks on value change.

Validators are executed on prop change. If not, please provide a repro.

Ah, my mistake. I encountered the case of passing in an array as a prop, and manipulating an element via arr[0] = 'str'; where the validator of the prop enforces that all elements are numbers. I learned that the issue is with reactivity and that it must be updated via this.$set(arr, '0', 'str') for the component to acknowledge the change.
If object properties that are declared on data-creation can be reactive, why can't array indices be?

That's the limitation of ES5. Converting Array indices with Object.defineProperty would lead to big performance loss and is not really worth it.


Support any license key  in build_spdx_license_expression

>>> from licensedcode.cache import build_spdx_license_expression
>>> build_spdx_license_expression("aaaa")
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "python3.10/site-packages/licensedcode/cache.py", line 513, in build_spdx_license_expression
    validate_spdx_license_keys(license_expression=license_expression, licensing=licensing)
  File "python3.10/site-packages/licensedcode/cache.py", line 552, in validate_spdx_license_keys
    raise InvalidLicenseKeyError(messages)
licensedcode.cache.InvalidLicenseKeyError: ['License key: aaaa is not a valid license key from LicenseDB', 'License key: aaaa is not a valid license key from LicenseDB']

Related to https://github.com/nexB/scancode.io/issues/749


Regarding calculation of Homography matrix

Hi,
Currently, i have 2 image (lets say A and B) and i am using SIFT and BestOf2NearestMatcher() for feature detection and matching (And i am planning to use my own feature matching method in the future my school project, i need help for H matrix estimation similar to OpenCV).
the output pairwise_matches in file feature_matcher.py (code snippet below)
pairwise_matches = self.matcher.apply2(features, *args, **kwargs)
pairwise_matches is of size 4 with datatype detail_MatchesInfo where, index [1] and [2] contains valid H matrix for image A & B respectively.
So, could any one please provide more details on how exactly homography [2] is generated??
i think following is the high level  flow of  BestOf2NearestMatcher (but i want to know more details on Homography is estimated for  image B)
BestOf2NearestMatcher::match()
|
|- CpuMatcher::operator()
| |
| |- CpuMatcher::match()
| |
| |- FlannBasedMatcher::knnMatch()
|
|- (Convert the matched points to coordinates whose center is zero)
|
|- findHomography()
|
|- (Check the results and decide whether to press “No” and calculate again)
|
|- findHomography() (executed only in inliers)
in particular, how matches between image B and A are extracted and how its used for estimating the Homography matrix .??
Thank you in advance.

You can search for the code how the matrix is calculated within https://github.com/opencv/opencv/


Minor Fixes to the Blackrock reader

This pull request is equivalent to this PR that I closed: https://github.com/NeuralEnsemble/python-neo/pull/325 but with a cleaner commit history.

Coverage decreased (-0.01%) to 50.331% when pulling<PHONE_NUMBER>b05df5239acf2978087a0b5ba8b800 on Blackfynn:blackrock/minor-fixes into 4ee20e620ab9892335fda1b01d23aff72f0a81aa on NeuralEnsemble:master.

Hi rohanshah, I just had a look at your fixes, thanks a lot, looks all good! I tested against our data files, and it works well.
@apdavison I think this is fine to merge.

@apdavison do you mind merging this when you have some time? I have another pull request I'd like to make on top of these changes as well.

@rohanshah Hi, actually, I also have another PR with a lot of changes to BlackrockIO waiting. Would you mind if I go ahead first? Mine is at:
https://github.com/INM-6/python-neo/tree/fix/blackrockio_issues

@mdenker sure thing! could you also let me know when your changes go out so that I can fix any merge conflicts my branch (https://github.com/NeuralEnsemble/python-neo/pull/334)?

@rohanshah ok, thanks, I'll try to match my branch with your previous fixes and make the PR by the end of tomorrow -- I'll post it here then. This PR will include a lot of little fixes that accumulated from various people, but no principle major changes (except for a new keyword argument to allow to load LFPs either as discrete sampled integer values or as voltages).

@rohanshah Sorry, took a bit longer than expected, since I found another bug in the unit tests. I made a PR now at #335


Documentation

Can you update the documentation for ASPNet. I am trying to find usage for         .AddMicrophoneKeyValueStore() and have not been successful. Is this to update and read from key value store in consul? how do i store my current configuration to the microphone key value store when the app boots up?

I cannot invest much time here right now as I am moving and selling my house.
So I will get on this as soon as things clear up a bit.
Sorry for the delay


⚠ Thumbnails API Endpoint has degraded performance

In f5bbcc5, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 2497 ms

Resolved: Thumbnails API Endpoint performance has improved in c009145.


[POM-292] update priv policies | missing RU

This PR doesn't seem needed after #227 landed; closing.


For label placements, enable configurable pixel size threshold

The label generation logic should probably have a separately configurable pixel size threshold generically per layer, and specifically per feature selection in the YAML.
For instance, if landuse layer polygons are 1 pixel threshold, then we might want 10 pixel threshold for label generation in general (queries.yaml), but maybe it should be different for national parks only (either in landuse.yaml or in the TileStache transform?).
Connects to: https://github.com/mapzen/vector-datasource/issues/202

Connects to #583.

Talked a bit about this IRL, assigning to @rmarianski.

If further work is needed here, we'll make a new issue in a new milestone.


Project deletion doesn't work the second time around

I ran into the following issue on prod and in local deployment where I couldn't delete a project of the same name as a previously deleted project.
Steps I did to encounter this issue:

Create a project, 'A'
Delete that proejct
Create a project, with same name as previous ('A')
Clicking delete a second time has no effect.

Update: okay, it deletes but requires a page refresh to actually go away (or maybe was taking a long time to delete?)

Issue is that the delete link is using project name not UUID. A fix is coming shortly

merged


add command line option for printing algo on stdout, default is false

Because after the first couple times it gets boring as hell.

Coverage increased (+0.0%) to 86.97% when pulling 5ac7d101a1d9140303b6a8038edc32be0dc23606 on anddam:no-print-algo into 3b56a62660dabaa36a68e94dd54baf970434461e on quantopian:master.

Coverage increased (+0.0%) to 86.97% when pulling 5ac7d101a1d9140303b6a8038edc32be0dc23606 on anddam:no-print-algo into 3b56a62660dabaa36a68e94dd54baf970434461e on quantopian:master.

Coverage increased (+0.0%) to 86.97% when pulling 5ac7d101a1d9140303b6a8038edc32be0dc23606 on anddam:no-print-algo into 3b56a62660dabaa36a68e94dd54baf970434461e on quantopian:master.

Looks good, thanks!

@twiecki Is there a reason we didn't merge this before? If there was no reason, can we rebase and merge?

@llllllllll No reason, we can merge this.

We need to rebase and fix the conflicts

Committed with https://github.com/quantopian/zipline/commit/65448cc8d163bafab2aad9bdda3c5075f597c4e1


Introducing error reporter.

This PR introduces an error reporting mechanism so that the user is aware of:

feedback for the user from specific actions. e.g if a file is not mapped correctly, let the user know that it requires it's attention.
https://i.imgur.com/wbkEp88.png

errors or bugs that are not intended behavior and were the user can take action e.g a file that could not be opened because it does not exist anymore
https://i.imgur.com/2DeqrnN.png

For error/bugs the user is prompted to file a bug in github.
Usage will be introduced in a following PR.
Fixes #85

I have a couple of nits and a proposal:
I'm wondering if we should add telemetry reporting as part of the ErrorReporter class. A couple of places we report errors after launching an error code. It might make sense to do that as part of the ErrorReporter? The only think is then it would need to be aware of the telemetryReporter instance, so that might prove to be too complicated to make them combine, since this is currently a static class. I'm fine keeping them separate for now

I actually am leaning towards keeping telemetry out of this, so I'm going to update to approved with some nits


Improve email change DX

current email change email is sent to user

[ ] add template to supabase dashboard
[ ] gotrue confirmation url should be a get request to users endpoint with the email_change_token as a query param, which is confirmed and returns a new valid login session (hence invalidating older refresh_tokens that may have been opened on the old email) this is current implemented here

Taking the liberty to close but feel free to reopen if there's more to be done w.r.t templating.
Will follow up with the sub-issues like #67 when a slot frees up. Have replicated the bug but need to discuss the fix
cc: @kangmingtay @hf


Error when installing package

Hi,
When installing your package I receive following error:
[Composer\Repository\RepositorySecurityException]
The contents of https://packagist.org/p/provider-latest%240cbfb40ab72a881d2
1b70f78286d39cd72e3b0eb8704c13e79dc49624e549973.json do not match its signa
ture. This should indicate a man-in-the-middle attack. Try running composer
again and report this if you think it is a mistake.
I can't seem to move on after this point :(

I just tested this again and it seems to work again!


Text numberOfLines={2} and ellipsizeMode="middle" breaks on Android

[x] I have reviewed the documentation
[x] I have searched existing issues
[ ] I am using the latest React Native version

Line restriction with ellipsizeMode set in a Text componenet fails to preform any truncation on Android when numberOfLines={2} is set.
This fails: <Text numberOfLines={2} ellipsizeMode="middle" >Some long text • 10</Text>
This works: <Text numberOfLines={1} ellipsizeMode="middle" >Some long text • 10</Text>
Both of the above snippets work as expected on iOS.
Environment
Environment:
OS: macOS Sierra 10.12.6
Node: 7.0.0
Yarn: Not Found
npm: 3.10.8
Watchman: 4.9.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 3.0 AI-171.4443003
Packages: (wanted => installed)
react: 16.0.0 => 16.0.0
react-native: 0.53.0 => 0.53.0
Steps to Reproduce

On Android, combination of numberOfLines={2} and ellipsizeMode="middle" on a Text component fails to truncate the text and no ellipses are visible. When I remove ellipsizeMode it defaults to tail and it truncates the text without issue.
Code Snippet
//...
<Text
  style={styles.text_name}
  numberOfLines={2}
  ellipsizeMode="middle"
>
  {`${this.props.listItem.name} • ${this.props.listItem.quantity}`}
</Text>
//...
const styles = StyleSheet.create({
  //...
  text_name: {
    color: ColorPalette.GREY.BLACK,
    fontSize: 18,
    fontFamily: fonts.proxima.semibold
  },
  //...
});

Check out my Snack reproducing this: FAILING WITH 2 LINES
Here's a Snack of it working with one line: WORKING WITH 1 LINE
Expected Behavior

I expect Android should have the same functionality as iOS
Correct iOS Screenshot

Actual Behavior

I have been working on this within my app and below is a screenshot of the section of my app where this is used.
Incorrect Android Screenshot

Extra Info
The reason I need ellipsizeMode="middle" is because I have a title and a number that I append to the title and want to always ensure that the number at the end is shown.
I have read through many of the issues about numberOfLines on this repo and I have not found anything that proposed a solution.
Any ideas?

The Snack examples I posted reproduce this issue on the latest release.

nobody care this ?

Still present on React Native 0.57.1… :-/

Still present on 57.7 ...

This issue appears to be solved in RN 0.59:

Using this snippet:
import React, { Component } from 'react';
import { StyleSheet, View, Text } from 'react-native';

export default class App extends Component<> {
  render() {
    return (
      <View style={styles.container}>
        <Text numberOfLines={2} ellipsizeMode="middle">
          Lorem Ipsum is simply dummy text of the printing and typesetting
          industry. Lorem Ipsum has been the industry's standard dummy text ever
          since the 1500s, when an unknown printer took a galley of type and
          scrambled it to make a type specimen book. It has survived not only
          five centuries, but also the leap into electronic typesetting,
          remaining essentially unchanged. It was popularised in the 1960s with
          the release of Letraset sheets containing Lorem Ipsum passages, and
          more recently with desktop publishing software like Aldus PageMaker
          including versions of Lorem Ipsum.
        </Text>
        <Text
          numberOfLines={1}
          ellipsizeMode="middle"
          style={{ marginTop: 20 }}
        >
          Some long text • 10
        </Text>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: 'white',
    justifyContent: 'center',
    paddingTop: 40,
    paddingHorizontal: 50
  }
});

Closing.

same isuee.
numberOfLines=1 works fine, but not for numberOfLine=2, 3, 4 ...

@kelset Wait your example shows it it's broken on the Android.
This is still happening on 0.61.5.


Use the same structure for the Windows builds across repositories

Structure all the Windows (aka MSVC) builds as we did for -pubsub and -common, namely:

A single build.bat script loads the necessary environment variables and starts the build.ps1 driver.

Life is too short to write a lot of .bat code, but we need to load the vcvars64.bat file.

The build.ps1 script determines the build to run by:

Checking the environment variables set by Kokoro to determine the build name.
Developers can provide the build name as the first argument.
In effect, this makes the script work the same as the build.sh scripts for Linux.

The script then calls a specific driver for Bazel and/or CMake, passing any configuration parameters to them.
All the Kokoro .cfg files are in the same directory as these scripts, analogous to the ci/kokoro/docker/*.cfg files.

[x] -common
[ ] -cpp
[ ] -spanner
[x] -pubsub
[x] -bigquery (N/A too early for Windows builds on that project)

Working on the -cpp changes, I need to:

[x] Create the new build scripts on GitHub ci/kokoro/windows, using -new when a file conflicts with the existing scripts.
[ ] Configure Kokoro to use the -new files.
[ ] Remove the old files, copying or moving the -new files when needed.

Working on the -spanner changes, I need to:

[x] Create the new build scripts on GitHub, using -new when a file conflicts with the existing scripts.
[ ] Configure Kokoro to use the -new files for the continuous builds.
[ ] Troubleshoot any problems
[ ] Configure Kokoro to use the new -new files for the presubmit builds, removing the old Kokoro configs
[ ] Rename the -new files on GitHub, change the GitHub Kokoro configs at the same time.


[gardening] Eliminate dead variable.

Just a dead variable that snuck in.

@swift-ci Please smoke test and merge


spider.request is not awaitable

I just start to use Ruia and I most say that I love it.
But I find my self a little confused with some code in the documentation not working.
The link of the documentation page is https://howie6879.github.io/ruia/en/tutorials/spider.html.
I am trying to replicate:
class MySpider(Spider):
    async def parse(self, response):
        for i in range(10):
            response = await self.request(f'https://some.site/{i}')
            yield self.parse_next(response)

    async def parse_next(self, response):
        print(response.html)

and when give an error when calling self.request and say it's not awaitable
I take a look to the source (installed on my computer) and in did spider.request now returns a Response object.
My question is how to proceed now to get the same result of this line response = await self.request(f'https://some.site/{i}') and be able to manipulate the response

After experimenting a little I find the solution (call the fetch method of Request object)


Material Design Input

Describe the feature you'd like
App Should Material Design Text Input Layout theme currently the app is using the material design layout box but not its theme
We should use the outlined box
https://material.io/design/components/text-fields.html#outlined-text-field
Would you like to work on the issue?

[x] Yes
[ ] No
Other:

Will be handled once we integrate material components


Fixes update streaming for newly built buildings

can you try if the built/destroy works well ? @cwastche

can you try if the built/destroy works well ? @cwastche

Fixed with latest commit


Helm tab in tech preview should be unrelated to helm repository endpoints

In the tech preview, a Helm tab appears when the user adds a helm repository in the endpoints page. However, the newly revealed helm tab contains integration with the kube endpoints' helm releases which may be entirely unrelated to the repository endpoint added. For example, even if you add a repository with no charts, all installed releases (whether installed from local files or from other repositories) will still be shown, and can be managed. Since much of the functionality here is independent of whether a helm repository endpoint has been added, it may be useful to always display this tab when there's a connected kube/caasp cluster, and charts can be displayed as well after repositories have been added

This was fixed in version 3.0.0 with the new Workloads view


[Bug]: cannot append third party scripts into html by using html tags while set config html inject: false

Version
System:
    OS: macOS 14.6.1
    CPU: (11) arm64 Apple M3 Pro
    Memory: 90.97 MB / 18.00 GB
    Shell: 5.9 - /bin/zsh
  Browsers:
    Chrome: 130.0.6723.117
    Safari: 17.6

Details
when we set config html inject: false, build scripts will not append into html;
but we also want to append third party scripts, and cannot work well while set config html inject: false;
Reproduce link
https://github.com/sbyps/rspack-demo/tree/test
Reproduce Steps
pnpm i
pnpm build
cat dist/index.html
cannot append eruda script file

This is as expected, I have updated the documentation to make it clearer: https://github.com/web-infra-dev/rsbuild/pull/4049


Missing artisan commands

Hi, I updated some of the missing artisan commands.
Still missing if this PR is validated:
module:publish
module:install
module:setup
module:dump

Thanks! 👍


Added HA 2023.1

In absence of a comment section or a HA community forum topic - thanks for this latest release! This really is one of the most useful EPEX pricing based integrations around! The additional dynamic entities make it even better.
Did you consider showcasing the integration on the HA community forum?

Thanks for the positive feedback. I have three feature requests in the issue list and a forth not listed that I plan to implement. After that, it could be a good idea to showcase it in the HA community forum.


PP updates for new-main branch

Creating this to keep track of how fucked the conflicts are going to be. new-main needs to be renamed & set as the new default branch eventually.

i thought github would be smart enough to ignore cherry-picked commits and not pretend there is conflicts but i guess i'm asking for too much


readBytes method

Hi, i'm trying to read temperature from a sensor (sht21) with a raspberry pi, here is my working code:
var i2c = require('i2c');
var address = 0x40;
var wire = new i2c(address, {device: '/dev/i2c-1',debug: false});
wire.scan(function(err, data) {
// result contains an array of addresses
if (err !== null) {
console.log('error scan: ' + err);
} else {
console.log("scan data: "+data);
}
});
wire.writeByte(0xF3, function(err) {
if (err !== null) {
console.log("error write: " + err);
} else {
console.log("write ok");
}
});
wire.readByte(function(err, res) {
if (err !== null) {
console.log("error readByte: " + err);
} else {
console.log(res);
}
})
this works flawless, but i need to read the first 3 bytes to get all data, so i will use readbytes method with this code:
wire.readBytes(0x00, 3, function(err, res) {
// result contains a buffer of bytes
//var value = res.readUInt8(0);
if (err !== null) {
console.log("readBytes: " + err);
} else {
console.log(res);
}
});
but it gives me an error. Probably i didn't understand what i have to pass as command parameter.

I think you want to do wire.readBytes(0xF3, 3, function ....
The initial write that you're doing is the command byte. That's telling your device that you want to receive 3 bytes of data.

Hi @wakatanka I'm also using the sh21t sensor with a raspberry pi. Have a look at pull request #35 from @bbx10 which contains a fix to read bytes without sending the command/register parameter. This fix also works for sh21t sensor.

I closed pull request #35 because the same functionality is available using plain read. For example, wire.read(3, function...). My fork is obsolete since @kib357 added plain I2C read and write.


[Feature Request] Override variables from test files when calling maestro test

Is your feature request related to a problem? Please describe.
I would like to define variables in the env section of the YAML file and then be able to override those when calling maestro run ... via both the command line and the environment.
Describe the solution you'd like
I think two things are needed:

The precedence order should be changed. Currently the order is YAML > command-line > environment (where > indicated "overrides"). The order should be command-line > environment > YAML.

Variables defined as MAESTRO_SOMEVAR should be made available as simply SOMEVAR (without the MAESTRO_) prefix. Combined with (1), this would allow you to use the environment to override any variable without having to enforce all variables starting with MAESTRO_ in your tests.

Describe alternatives you've considered
My current workaround is to comment out the environment variables in the YAML file before committing to version control so that CI can inject them via the command line. I would rather use environment variables to inject, but I don't want to prefix all of my variables with MAESTRO_ in the tests.
Additional context
I believe (1) would be a backward-incompatible change if it were just implemented outright. Assuming that's not desirable, the new loading order could be enabled via a new command line flag (--override-yaml-env maybe?). Then maybe the new order could become the default in maestro 2.x and the flag could be flipped (--no-override-yaml-env).
(2) Could be done in a (very nearly) backward compatible way by exposing MAESTRO_* variables from the environment both with and without the MAESTRO_ prefix. In my opinion, the prefixed version could be deprecated and removed in 2.x.
Here's a test case:
# test.yaml

appId: ""
env:
  YAML: from yaml
  COMMAND_LINE: from yaml
  ENVIRONMENT: from yaml
---
- evalScript: |
    ${console.log('COMMAND_LINE: ' + COMMAND_LINE)};
    ${console.log('ENVIRONMENT: ' + ENVIRONMENT)};
    ${console.log('YAML: ' + YAML)};

Then run via:
MAESTRO_COMMAND_LINE='from env' \
  MAESTRO_ENVIRONMENT='from env' \
  maestro test \
  --env="MAESTRO_COMMAND_LINE=from command line" \
  test.yaml

This currently prints:
 ║         Log messages:
 ║           COMMAND_LINE: from yaml
 ║           ENVIRONMENT: from yaml
 ║           YAML: from yaml

But should print:
 ║         Log messages:
 ║           COMMAND_LINE: from command line
 ║           ENVIRONMENT: from env
 ║           YAML: from yaml

Would love this too, it would be useful when calling them as sub flows as well

+1 for a need to define default env vars which can be overriden via cli params.

@cvetojevichbojan You can achieve that via this:
env:
  APP_ID: ${APP_ID || "default"}

...where if APP_ID is passed in via env vars, it will use that, else if it's null/not passed in then it will default to "default"

Awesome, then i suppose it would be good to add this example to the documentation as well: https://maestro.mobile.dev/advanced/parameters-and-constants


Add Trails Page

Trails will be one of the most involved aspects of the site, and hopefully one of the most used. As a result, prior to coming up with requirements we should first explore use-cases for people using this page.
Uses cases for a page dedicated exclusively to trails are:

As a new mountain biker, where can I find specific trails or routes that I will have fun and comfortable on?
As an experienced mountain biker, what trails are new or have new features that I can go ride?
As a person new to the city and comfortable biking, what parks are around that I can mountain bike at?
As a person just visiting to the city, given my skill level where should I travel and what trails should I ride?

Trail-building related use-cases are:

As a part-time trail-builder, what trails exist that could use trimming, raking, etc.?
As someone who would like to learn how to trail-build, or contribute in a group format, when are the next trail-building days?
As someone on the fence about trail-building when is SIMBS going to work on the trails I love (and when should I come to trail maintenance days)?

Yeah, we were also talking about having trail forks connected.

Also maybe some recommended loops

Any notices about trails

Complete


Let fetchMock.mock() return scoped mock

First of all thanks for your awesome library. It makes life much easier when testing my react components that perform API calls.
While experimenting with asserting one question came up. Would it be possible to let fetchMock.mock() return a new mock that only tracks fetch calls where the given filter and options apply so I can assert like that? :
// arrange
const customerAPIMock = fetchMock.mock('path:/customer/');
const productAPIMock = fetchMock.mock('path:/product/');

// act
// Do something that only hits the customer api
// Do something that POST's to the product api

// assert
expect(customerAPIMock).toHaveFetchedTimes(1);
expect(productAPIMock).not.toHaveFetched();
expect(productAPIMock).toHavePostedWith({body: {name: 'Unicorn Milk', price: '$42.00'}});

As far as I understood this can currently be achieved like that using a name...
// arrange
fetchMock.mock({url: 'path:/customer/', name: 'customerAPI'});
fetchMock.mock({url: 'path:/product/', name: 'productAPI'});

// act
// Do something that only hits the customer api
// Do something that POST's to the product api

// assert
expect(fetchMock).toHaveFetchedTimes(1, {name: 'customerAPI'});
expect(fetchMock).not.toHaveFetched({name: 'productAPI'});
expect(productAPIMock).toHavePostedWith({name: 'productAPI'}, {body: {name: 'Unicorn Milk', price: '$42.00'}});

...but it doesn't feel as natural to me as I need to use strings to reference a certain fetch calls. Also this is error prone and can make my tests fail because of typos in the name or accidentally reusing a name and it is lacking auto-completion.
What do you think about the proposed api and do you think it is hard to implement?

Sorry for the slow response.
I like the API idea, but it feels like a massive departure from the current implementation and probably difficult to achieve. Would require massive amounts of writing new tests for the library (or, rather, for fetch-mock) too.
So realistically I can't see it happening, but will keep it open in case I think of a way.
For fetch-mock there is an easier naming API coming soon probably
fetch.mock(url, response, 'my-name');
expect(fetchMock).toHaveFetchedTimes('my-name');

In fact, even now, you should be able to use 'my-name' instead of {name: 'my-name'} in assertions anyway


VLC: Missing File

Hey!
Really excited about this project, I can really see it coming in handy!
It looks like there could be a file missing from the VLC entry.
appFiles+=("/Users/$loggedInUser/Library/Saved Application State/org.videolan.vlc.savedState")
I usually use App Eraser when I'm removing things, as it seems to do a fantastic job of finding files that have been hidden all over the place. Using this, it appears there was a file entry missing from VLC.

I can see that you're working on seperate_labels like Installomator, so I'll create a pull request there against VLC if that's alright?

Thanks for the addition


Article/ProvidingSource/source GA event has invalid title

https://github.com/cofacts/rumors-line-bot/pull/135/files#diff-f8051c184f469280912e9748671986e6R24
https://docs.google.com/drawings/d/e/2PACX-1vQhBB1Ie_Lk3GPhqSkWTVOfUFHeZEB2vnkn9eM_7IQQtVspZebI7Ne58YfWKTAd3yshfnVuFlx8KpJx/pub?w=1629&h=948

No need for this because ProvidingSource state has been removed


slack.py error

Rasa Core version: 0.12.1
Python version: 3.7
Operating system (windows, osx, ...): MacOS 10.13.6
Issue:
The recent commit made in 0.12.1 release in channels/slack.py:
"out_channel = SlackBot(self.slack_token, self.slack_channel)"
I am unable to connect to slack after this change. The bot wont respond if I enter any message.
If I remove "self.slack_channel" and revert to old code which is:
"out_channel = SlackBot(self.slack_token)", then the bot works fine and is able to respond in slack.
There is no error printed on the console and logs. It just silently fails.
FYI, I am using docker and talking directly to the bot in Slack.
Docker Command used:
docker run -p 5002:5002 --rm --net slack --name mainbot -v pwd/models/dialogue:/app/out -v pwd/models/nlu/default/current:/app/nlu -v pwd/config/credentials.yml:/app/credentials.yml -v pwd/domain/endpoints.yml:/app/endpoints.yml -v pwd/logs:/app/logs -e RASA_DUCKLING_HTTP_URL="http://duckling:8000" core:0.12.1 run python -m rasa_core.run -d /app/out -u /app/nlu --port 5002  --endpoints /app/endpoints.yml --credentials /app/credentials.yml --log_file /app/logs/core_run_bot_log.txt
Content of domain file (if used & relevant): NA

Thanks for raising this issue, @ricwo will get back to you about it soon.

@adityac07 what does your credentials.yml look like? Can you make sure the slack_channel property is filled?
https://rasa.com/docs/core/connectors/#id4

@adityac07 To receive a message back to yourself, either specify your handle or slack ID in slack_channel, or supply an empty string:
slack:
  slack_token: "xoxb-286425452756-tbyMIhoB8uxyFgX7smAltKasdf"
  slack_channel: ""

Thank you. That worked.


Persist containers create

When a custom Container is created using containers.create(), it is saved as a JSON container within the file specified by environment variable CONTAINERS_FILE.
If no name = is specified when using containers.create(), it will not be saved to the file.

Coverage increased (+0.03%) to 95.805% when pulling cb0a8d9abaf1162d68be1d989a7a585eb486d84b on persist-containers-create into 3fc83f921e076b7807112534bd95697fdc923775 on master.


new alloc counter tests

Motivation:
Previously, writing an allocation test was quite hard, the allocation tests
were hard to debug and also the allocation testing framework and the
allocations tests were mangled together. That was bad and therefore
we never started writing allocation tests for other NIO repositories such
as swift-nio-http2.
Modifications:

Make it easy to write alloc tests (just drop a test_...swift file in a folder).
separate the alloc counter framework from the tests

Result:

easier to write allocation tests
'productise' the alloc counter framework

Looks like we have some test failures here. :grin:

@Lukasa ha, it expects package id to be the same as package basename :P:
18:57:02 Completed resolution in 0.62s
18:57:02 error: package at '/code' is swift-nio but was expecting code

but we check the code out into /code... Will fix


jcabi-log 0.24.0 with "%[file]s" decor

Let's use recently released jcabi-log version 0.24.0. There is a new "file" decor, which allows us to show file name in a log line, in a compressed format:
Logger.info("The file was created: %[file]s", path);

This log line will not print absolute path of the file, but only its relative to the current directory part. Thus, our logs will be much more readable, comparing to now.

duplicate of #381


create_fim_lib step allow creation of FIM library and rating curves db in metric units with different precisions

Some concerns:

There is still no avoiding that the HEC-RAS ecosystem in the US mostly uses English units, so Ripple has to take all the load of unit management, which it is kind of already doing.
The flow increment and depth increment are based on feet units, and they round to whole numbers, hence the final feet outputs are clean whole numbers or half of foot. The metric outputs won't be the same. How should we manage that?
Currently it is hardcoded that the length numbers are rounded to 1 decimal digit, but with metric units, we have to do two decimal places to account for cm accuracy.

NWM generally does metric units, but at some places for frontend it does English

Going from depth in float32 feet to uint16 cm will reduce data size roughly about 55%


[OAP-1631][Doc]reorg docs for OAP 0.9

What changes were proposed in this pull request?
How was this patch tested?

cc @yao531441
Please help this , thanks

LGTM!

LGTM


Check if a layer has vector mask before rendering strokes

This PR is to fix #299.
There is sometimes a layer in PSD with stroke but without vector_mask, which result in an error when attributes of layer.vector_mask are accessed in layer.composite(). We just skip rendering strokes when the layer has no vector mask.

Now the PSD file attached in #299 is rendered without errors


tensorboard logging added

Thanks Maksim, could you add some lines to the readme.md how you add the tensorboard logger ?:)

Thanks Maksim, could you add some lines to the readme.md how you add the tensorboard logger ?:)

done


Issue with PHP substr() and empty strings

Description
PHP's implementation of substr() returns an empty string under the following conditions, not false:
substr("", 0);
substr("", 0, 0);

Locutus's implementation of substr() returns false for these arguments.

I've pushed a tiny patch that fixes just the reported scenario, but there are more nuances in PHP that potentially need to be covered, which I'll do in separate patches.


Memory Leak

When doing simple operations or even just parsing an XML theres memory being allocated and not being freed until the whole process is stopped. This has a big impact as we are using this library in production in our backend service that we have to restart every now and then as it runs out of memory quite fast.
I created a script that creates a csv file from the memory usage after each iteration. I attached a diagram of the result.

The script made 3510 parsing and simple text replacement operations (as you can see in the GitHub repo) before I stopped it.
I created a GitHub repo to reproduce this issue here.
I hope this issue can be resolved ASAP.

@rchipka Any idea what this could be due to? I would take a look myself, I just don't know anything about C. I have already tried the one or other "xmlMemFree" or "xmlFree*" method, but this has no real effect.
In libxmljs2 this error cannot be reproduced.

One of the issues seems to be in encode:
https://github.com/libxmljs/libxmljs/blob/3121013c0e447d267bf4514317c52ad81670f2f5/lib/document.ts#L345
When just returning the data it will decrease the used RAM from 3.4GB to 1.5GB in the above example.
May xmlFree has to be used as it is in libxmljs2:
https://github.com/marudor/libxmljs2/blob/e0eb378267b1eba552c30451a7c22ea036ae5357/src/xml_text.cc#L189C18-L189C19

Is there any progress on the issue ?

It seems to me that there is also a memory leak with documents...
a simple loop like this sends the memory usage to 600MB, while with libxmljs2 stays always at 30MB
for (let i = 0; i < 10000000; i++)
{
   let doc = libxmljs.parseXml("<root/>");
}


22khz

Just a quick follow up,
I've been trying to incorporate the alternate 16kHz configuration using
spleeter separate -i audio_example.mp3 -o audio_output -p spleeter:4stems-16kHz
which raises the following error:
ERROR:spleeter:No embedded configuration 2stems-16kHz found
And
spleeter separate -i my_audio_file.mp3 -c spleeter:4stems-16kHz -o /output/path
which raises the following error:
spleeter separate: error: argument -c/--codec: invalid choice:
I'm currently running a Mac OSX, Python 3.7.3, I also attempted to reinstall spleeter with the same results. Any idea whats going on here?
Thanks,
Originally posted by @khaleelyo in https://github.com/deezer/spleeter/issues/15#issuecomment-625981182

When i separate at 16 kHz, is very good but cannot separate at 22khz. Can you tell me â good cmd to do on Windows

When i separate at 16 kHz, is very good but cannot separate at 22khz. Can you tell me â good cmd to do on Windows

The error:

spleeter separate: error: argument -c/--codec: invalid choice:

is due to the improper use of the -c option instead of the -p option.
I can't reproduce the other error:

ERROR:spleeter:No embedded configuration 2stems-16kHz found

so I'm closing the issue.

I ran into this problem but realized that by looking at the source code in the spleeter/resources folder there is no pre-written 22khz json files.  Comparing the default to the 16khz files, there's only one line different, which seems to be the filter cutoff frequency. So you can just make you're own edit. This example is for 2 channel split.
The line I changed was this:
  "F":2048, // did simple math here double 1024 (11k default)

Here's my new file which I named 2stems-22khz.json
To use it just pass in the path to your json file in the -p argument.
{
    "train_csv": "path/to/train.csv",
    "validation_csv": "path/to/test.csv",
    "model_dir": "2stems",
    "mix_name": "mix",
    "instrument_list": ["vocals", "accompaniment"],
    "sample_rate":44100,
    "frame_length":4096,
    "frame_step":1024,
    "T":512,
    "F":2048,
    "n_channels":2,
    "separation_exponent":2,
    "mask_extension":"zeros",
    "learning_rate": 1e-4,
    "batch_size":4,
    "training_cache":"training_cache",
    "validation_cache":"validation_cache",
    "train_max_steps": 1000000,
    "throttle_secs":300,
    "random_seed":0,
    "save_checkpoints_steps":150,
    "save_summary_steps":5,
    "model":{
            "type":"unet.unet",
            "params":{}
            }
}


Update operators.decimal[^t]-function-[fsharp].md

Link to implementation: https://github.com/Microsoft/visualfsharp/blob/master/src/fsharp/FSharp.Core/prim-types.fs#L3941

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/conceptual/seq.tail['t]-function-[fsharp].md
:warning:Warning

Details

docs/conceptual/operators.decimal[^t]-function-[fsharp].md
:white_check_mark:Succeeded

docs/conceptual/seq.tail['t]-function-[fsharp].md

Line 48: [Warning] Invalid file link:(~/seq.head['t%5D-function-%5Bfsharp%5D.md).
Line 77: [Warning] Invalid file link:(~/seq.head['t%5D-function-%5Bfsharp%5D.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[WIP] Feature/http upgrades

Implemented an http client for fully customizable http requests and tests. Labeled as WIP because errors still need to be tested and http errors reinplemented

Next step is to add HTTPError, I tried to by simply adding a const in the http.go file

Need to resolve merge conflicts

Needs restructuring, as 'send' is a global method

The latest commit has comments in the class methods

@ear7h It seems that this PR is almost completed, what do we still lack for now?

@ear7h I think we still need some test cases for error situations to keep the test coverage. See here

@st0012 At the moment, the client class uses the default Go client with no way of changing it. I need to implement cookies and other settings

@ear7h I think we can merge this first and open a new PR for the things you talk about. How do you think?

@st0012 I think that's good idea, I have no idea how i'm going to attack these changes

@ear7h Sorry that I didn't ask until now, but does this PR also covers #331?


Allow bindings in the textField component

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Use case: I want to update some values of a datagrid through a form. So I expect the values of selected row should prefill in the form. The ideal component inside a form for this function is TextField, but I can't bind a value to it. It seems to only take input.
Default value property also doesn't serve the purpose.
App link
Examples 🌈
No response
Motivation 🔦
No response

Did a little benchmark in other tools on how they handle this. The benchmark was:

create two inputs

make the second input default value equal to the first ones value

type in the first input, then type in the second input, then type in the first one again

retool, appsmith, openblocks and dynaboard all behave the same. When the default value changes, the input value is reset to the default value.

I'm fine with adopting the same behavior


how ?

command + F1 F2 F3 ti's not  work  , please give me  some help or tips. tkx

Hm, do you see any it in edit menu? If not, then plugin isn't loaded.

There is usually a problem with xcode DVTPlugInCompatibilityUUID.  If alcatraz doesn't work, just please clone the repository and hit run, and restart xcode, and it should work.
Xcode has a really stupid think with those IDs, but I've set it up so that when you build the project, it will immediately update the plist with DVTPlugInCompatibilityUUID of the xcode that is used to build it.
Hope this helps.

ye, thx for help.


Add back mergemock into CI and small fixes

See https://github.com/protolambda/mergemock/pull/20 for the bulk of the changes to get this working


Drop missing values from raw data in featureVector.py

While reading raw data from the input .csv file in Featurizer.readRawData, drop the rows with missing values.
Fixes #25

@sechkova I don't think we want to drop all rows when there is value missing in one of the columns. Data will typically not have all columns filled out since we're collecting data for the time PR was merged, reviews, and comments. If we drop all rows with NaN, then we would be dropping data for PRs that did not merge, do not contain any comments, and etc.
In the case of https://github.com/vmware-labs/ml-conversational-analytic-tool/issues/25, it looks like the body value is missing. Even with a missing body, the code looks at comments and reviews to calculate other metrics. What do you think about returning 0 for body related metrics like pull_setiment and pull_code_blocks and letting the code run through comments and reviews?
You might run into the same issue if comments or reviews are empty? Not sure how that's handled but would be good to look into that too!

I see, we don't want to drop full rows and reduce the dataset. It is not very clear to me the idea of featureVector.py though. Which fields (columns) are useful input for it and which ones are ignored? Why doesn't it work with the output of github_data.py and why does it provide a different format of data which on its turn does not fit into the expected input of run.py
As a whole featureVector.py could benefit from a better documentation but I already opened #32 related to it. That will help sanitising the input and fixing the crash for the general case of missing fields and not only for missing body.

I think the documentation needs to be improved here. The original project used a feature vector at first to define a set of features that can be used to answer "constructiveness" in data instead of using manual annotation.
However, it was realized that labeled data might be more appropriate in this case which is why there is github_data.py to preprocess the data to be used for annotation purposes.
Annotation is not part of the project and must be done by the user who leverages this framework.
Once there is annotated data, then they can leverage run.py and there is no support for model creation using feature vector.
I agree that it actually causes more confusion than an alternative solution to manual annotation since the training doesn't support the output of the feature vector. How about we completely remove the file? wdyt?

Unless there are efforts planned to integrate feature vector with run.py (which seems not the case) let's remove it ... or move it as a side utility with some brief documentation? I noticed that feature vector uses commentAnalysis.py, the only component with some tests implemented 👀 I do not want to suggest removing the single existing test 😁

So maybe what we can do is apply the fixes that @difince suggests but then I still want to try updating the readme and/or move the file to make it clear that it is not directly related to run.py and the main usage of mcat.
I'd also happily delete the whole feature vector thing as otherwise it has to be maintained but I don't want to get rid of someone's work that easily. If you think keeping it will be of value in future I'll apply the fixes.

Thanks for the advice, I'll rework the fix in trying to save the feature vector from deletion :)

Closing to clean up for deprecation


rancher-2.2.3 HA安装后,访问UI提示连接已重置?

访问 https://host,提示无法访问此网站, 连接已重置。

pod也都是正常的
NAMESPACE       NAME                                      READY   STATUS      RESTARTS   AGE
cattle-system   cattle-cluster-agent-fbb998bf-4d266       1/1     Running     0          96m
cattle-system   cattle-node-agent-4pkf4                   1/1     Running     0          96m
cattle-system   cattle-node-agent-6q5p4                   1/1     Running     0          96m
cattle-system   cattle-node-agent-bdk5k                   1/1     Running     0          96m
cattle-system   cattle-node-agent-bl4gg                   1/1     Running     0          96m
cattle-system   rancher-567c6fd5dc-b4l7p                  1/1     Running     1          114m
cattle-system   rancher-567c6fd5dc-vvkrf                  1/1     Running     1          114m
cattle-system   rancher-567c6fd5dc-w878n                  1/1     Running     1          114m
ingress-nginx   default-http-backend-78fccfc5d9-9t7g6     1/1     Running     0          178m
ingress-nginx   nginx-ingress-controller-bzxdb            1/1     Running     0          178m
ingress-nginx   nginx-ingress-controller-fvps8            1/1     Running     0          178m
ingress-nginx   nginx-ingress-controller-q8nn8            1/1     Running     0          178m
ingress-nginx   nginx-ingress-controller-szs5d            1/1     Running     0          178m
kube-system     canal-kdb66                               2/2     Running     0          178m
kube-system     canal-lfflx                               2/2     Running     0          178m
kube-system     canal-sxjwr                               2/2     Running     0          178m
kube-system     canal-wxf86                               2/2     Running     0          178m
kube-system     kube-dns-58bd5b8dd7-xwq5s                 3/3     Running     0          178m
kube-system     kube-dns-autoscaler-77bc5fd84-bqkg2       1/1     Running     0          178m
kube-system     metrics-server-58bd5dd8d7-2cpvt           1/1     Running     0          178m
kube-system     rke-ingress-controller-deploy-job-2mmtb   0/1     Completed   0          178m
kube-system     rke-kube-dns-addon-deploy-job-6rhwz       0/1     Completed   0          178m
kube-system     rke-metrics-addon-deploy-job-hcswt        0/1     Completed   0          178m
kube-system     rke-network-plugin-deploy-job-7wdd4       0/1     Completed   0          179m
kube-system     tiller-deploy-7f8586cdf-mghqx             1/1     Running     0          170m

组件也是健康的
$ kubectl get cs
NAME                 STATUS    MESSAGE              ERROR
controller-manager   Healthy   ok
scheduler            Healthy   ok
etcd-2               Healthy   {"health": "true"}
etcd-1               Healthy   {"health": "true"}
etcd-0               Healthy   {"health": "true"}

Useful
Info

Versions
Rancher v2.2.3 UI: v2.2.41

Route
/

HA里面
NAMESPACE       NAME                                      READY   STATUS      RESTARTS   AGE
cattle-system   rancher-234c6fd3dc-b4l6p                  1/1     Running     1065   134h

Can you supply more info on your setup (basically copy paste from this and fill in: https://raw.githubusercontent.com/rancher/rancher/master/.github/ISSUE_TEMPLATE.md)
Or you can find us via WeChat by searching rancher2


spec: minor bug relating to changes made for shifts

Does this issue reproduce with the latest release?
Yes (Go 1.13).
What did you do?
Read the latest go language specification at https://golang.org/ref/spec#Arithmetic_operators
What did you expect to see?
The shift operators shift the left operand by the shift count specified by the right operand, which must be non-negative.
What did you see instead?
The shift operators shift the left operand by the shift count specified by the right operand, which must be positive.
This language was added as part of a10b4cff91cb5d26e2049f6efc20349aa4d50d20, which modified the spec to reflect the shift changes made in go 1.13. It is trivially wrong: it disallows shifts by zero.

Fair enough. It appears that whether zero is considered positive depends on where you're coming from...
"non-negative" avoids the ambiguity. Will fix.


Validate scopes against user roles during token generation

We only have role based scope validation in token validation step. If scope validator is configured in the application, validation should happen during token generation as well.

Fixed with https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/919


Add Kotlin support

Jeka should accept build classes to be written in Kotlin.
Beside, Jeka should propose a convenient way to build Kotlin project.

Hi, I just wanted to give my opinion here, as someone who actually feels like Jeka can be a promising new build system for Java!
The main thing that attracted me to give this project a go is that it seems to be very lightweight and it feels like a Java library, not a heavy build system like Gradle.
It's really excellent that I can write build logic in Java, as it feels like it integrates well with my Java projects without introducing the complexity and maintenance issues with having a second language, like Groovy or Kotlin, to keep up with.
I actually love both Groovy and Kotlin, but over the years, I have suffered as a consequence as it's not trivial to keep up with the pace of changes in Java, which is evolving pretty fast these days , as well as Groovy and Kotlin (and in my case, Gradle, Spock, Kotest and all related stuff around that). Supporting Kotlin is much harder than it seems... Gradle, for example, has a long running issue due to plugins being written in one Kotlin version, the user wanting to use another, and Gradle itself internally using yet another version.
Having a build system that only requires me to have a modern enough version of Java would be a dream!
So , while I think it's great that you want to add support for Kotlin, I would like to suggest that the basic runtime do not include that (not knowing whether or not that's already what you plan to do).
The current distribution seems to be in the order of 6MB? Which is pretty great... adding a Kotlin compiler would triple that, probably? Anyway, just wanted to raise my concern because I would love to have a really lightweight build tool for Java, and I'm afraid including Kotlin by default could make it heavy (imagine having a build system so heavy you need a daemon to make it fast enough!).

Thank for your feedback. Always good to hear.
Don't worry, Jeka won't get bloated by this feature and should even get lighter than it is now.
Indeed, the Kotlin specific libs will be downloaded on the fly, only if needed.
In  the future I will generalize this approach to get a far smaller distribution by removing things like ivy, classgraph, bouncycastle out of the main jar.
Last but not least, the next release will feature a completely reworked execution engine, leading in a quicker startup and a cleaner & lighter overall design.
I'm currently focussed on it

Great to hear that! This project is looking incredible!

Kotlon JVM plugin has been release in 0.10.4


respect system proxy settings

New updater doesn't respect the windows system proxy.

When you say "new updater" do you mean in a recent build? I don't think the updater code has changed recently, it likely never has respected the proxy.

I misinterpreted the latest email and assumed the out of band installer included new update code.
In any case, today was the first time I noticed a failure reaching the update server due to proxy issues.

No worries, thanks for clarifying.
Because our core has to function without any QT libraries (in the case of headless operation), we can't rely on their proxy support unfortunately and we don't want to write and maintain cross-platform proxy detection code.
That said, when we implement our hidden preferences / advanced preferences capability for more obscure settings, we'll include a way to manually specify proxies.
Blocked by #126

Fixed in build 1.1.1229-dev.


How to stream the video to a file?

I tried added this line
videoconvert ! x264enc tune=zerolatency  ! mp4mux ! queue ! filesink location=v.mp4 sync=false

to the pipeline but it looks like that the EOS event is not sent to the pipeline and therefore the file is not streamable as reported by ubuntu movie player.

I'm not sure - this is more a general gstreamer issue than coral-specific. I'd recommend asking in that community for help.
How are you closing the capture ? Ctrl-C ?