[BUG] IME problem under wayland

Describe the bug
When I input ASCII characters, as long as the frequency is a little faster, repeated characters will appear.
For example, "hello world" will become: "heeellllllooo wwooorrrlld".
When I input Unicode characters, there is a high probability that the characters will be garbled, and sometimes the input characters will disappear.
The most outrageous thing is that sometimes it triggers the string to "flicker".
https://github.com/fltk-rs/fltk-rs/assets/25324935/b70e5e3b-ca7f-422f-a3d8-195caa1f9f0a
In the video above, it looks like I’m constantly deleting text, but I’m not. I just typed emo.

Note: Only wayland triggers this issue, x11 does not.
This issue has rendered the input method of Wayland basically unusable,  I hope it can be fixed.
To Reproduce
Cargo.toml
[dependencies]
fltk = { version = "1.4.28", features = [
    "use-ninja",
    "use-wayland",
] }

src: https://github.com/fltk-rs/fltk-rs/blob/682a7255df5a887412f5c3f2c228f133a602af74/fltk/examples/editor.rs
Expected behavior
Perfect support for Wayland.
Desktop info

OS: Debian Sid GNU/Linux (x86_64)
Version: 1.4.28
Display: Wayland
Desktop:  KDE Plasma 5.27.10-1+b2

Additional info
(process:160445): Pango-WARNING **: 22:53:29.310: Invalid UTF-8 string passed to pango_layout_set_text()

Hi
Thanks for the report.
Just making sure, are you using IME (input method editor).
I'm not sure how this works, but apparently wayland has several issues with IME.
On my side, I tried typing Arabic in the editor example under wayland, and it seems to work correctly. No flickers, unicode issues or pango warnings.
Since I can't replicate, I'll post to the fltk-general forum and see what the core devs think.

@2moe I would need some more info, please, to try and reproduce the issue.

do you use an input method, such as one to input Chinese or Japanese characters?
if yes, do you activate this input method via a menu on the desktop such as this one:

what do you mean by "input Unicode characters" ?
is the issue that fast typing tends to create repeated characters related or not to the issue about "garbled characters" ?

If you cannot reproduce this issue, I can provide a QEMU virtual machine image with the wayland + fcitx5 environment. Using other wayland applications (e.g., MOZ_ENABLE_WAYLAND=1 firefox) + fcitx does not have this issue.
I encountered this issue only in fltk apps.
fltk is a simple lightweight GUI. Thank you for your efforts!

I've updated the steps for reproduction.

Note: I was able to reproduce the garbled & repeated characters problem 100% in a VM environment.
apt install plasma-desktop plasma-workspace-wayland konsole sddm
apt install fonts-noto-cjk fonts-noto-color-emoji

apt install fcitx5-chinese-addons

If you are currently in a virtual machine nogui environment, you need to run systemctl start sddm
Once on the desktop, use a terminal to run im-config

run plasma-open-settings

run fcitx5-configtool

ShuangPin: Similar to standard Chinese Pinyin, but it only takes two English characters to enter a Chinese character.

Here are my environment variables, which don't really need to be set manually.
LANG=C.UTF-8

GTK_IM_MODULE=fcitx
QT_IM_MODULE=fcitx
XMODIFIERS=@im=fcitx
INPUT_METHOD=fcitx
SDL_IM_MODULE=fcitx

After configuring fcitx5, you need to logout and re-enter the desktop.

@2moe Many thanks for all this detailed information. I didn't succeed immediately in having fcitx IM run here. But I want first to share with you what I have here working very well with FLTK's Wayland backend:
I install these packages for input methods:

chinese:  ibus-libpinyin
japanese: ibus-kkc

and I use the Gnome desktop. I obtain FLTK apps that seem to input correctly Chinese and Japanese characters (I know absolutely nothing about these characters so that's just an impression) as well as the very few emoticons I've been able to have appear in the auxiliary IM window. Here is a screenshot of what I get running the FLTK test/input program and activating "Chinese (Intelligent Pinyin)" in the keyboard menu at top right :

If I turn back to latin IM, I can type text very fast in an FLTK text field and get no problem of repeated letter at all.
Now I see that you use KDE-plasma rather than Gnome and fcitx rather than IBus-based IM. That is unchartered territory for me. FLTK runs well under KDE-plasma until attempting to use an IM.
My new questions:

Is it possible to use IBus-based IM under KDE-Plasma?
FLTK implements support of IM according to what is explained in that web page:
https://dorotac.eu/posts/input_method/
Is fcitx incompatible with that approach?

OK. I've now installed properly the fcitx5 pinyin IM under Debian 12 bookworm using your detailed instructions. I use it with KDE-Plasma. I find that it works nearly OK with FLTK, namely I used the editor and input FLTK test apps.
It's not perfect because after the number key that selects the desired character is pressed, the new text is selected in the FLTK widget, so it's necessary to type the right arrow key once before inputting the next character. I hope to be able to fix that.
When in latin text input mode, I can type text very fast without problem.
So, I can't reproduce the problems you report. What FLTK app did you use exactly? What do you see when you use for example the editor FLTK program?

@ManoloFLTK
I compiled the app via github actions
You can use docker to pull the binary and run it.
docker run --rm -v $PWD/tmp:/host ghcr.io/2moe/testing-app:editor-demo-x64 cp editor-demo /host

./tmp/editor-demo

I am building a VM image of kde + fltk app. Unfortunately, the file size is over 2GB and cannot be uploaded to github releases.
I had to split it.
I need to do more testing to know if the problem is only with sid.

@2moe  Don't try to prepare docker or VM images for me. I won't use them. Also because I use Linux in a VM on an arm64 MacBook Pro.
Please, let me know what you get with a straight FLTK app such as editor or input and the most recent github version of FLTK accessible to you.

@ManoloFLTK
Sorry, I thought you were using the x86_64 architecture.
Can you use the linux arm64 binary?

Src: https://github.com/2moe/testing-vm/tree/main/fltk/editor-demo

Is this program based on fltk-rs? or on pure fltk?

fltk-rs (rust), not fltk (C++).

Could you use a pure fltk(C++) program, to rule out it's a problem with fltk-rs?

Could you give me an example?

Possible complete procedure to do that:
git clone https://github.com/fltk/fltk
cd fltk
mkdir build
cd build
cmake ..
make -j5 editor
bin/test/editor &

and then report here how you see the editor app behave with your IM.

Problems remain.

ioctl(2, TCGETS, {c_iflag=ICRNL|IXON|IXOFF|IUTF8, c_oflag=NL0|CR0|TAB0|BS0|VT0|FF0|OPOST|ONLCR, c_cflag=B38400|CS8|CREAD, c_lflag=ISIG|ICANON|ECHO|ECHOE|ECHOK|IEXTEN|ECHOCTL|ECHOKE, ...}) = 0
getpid()                                = 265627
write(2, "\n(editor:265627): Pango-\33[1;33mW"..., 125
(editor:265627): Pango-WARNING **: 01:39:05.070: Invalid UTF-8 string passed to pango_layout_set_text()
) = 125
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="(\0\0\0\0\0<EMAIL_ADDRESS>iov_len=160}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 160
pselect6(4, [3], [], [], {tv_sec=0, tv_nsec=31358000}, NULL) = 1 (in [3], left {tv_sec=0, tv_nsec=30399184})
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\1\0\0\0\1\0\f\0%\0\0\0\21\0\0\0\5\0\f\0\31\2\0\0", iov_len=3708}, {iov_base="", iov_len=388}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 24
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
pselect6(4, [3], [], [], {tv_sec=0, tv_nsec=30043000}, NULL) = 1 (in [3], left {tv_sec=0, tv_nsec=3563075})
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base=",\0\0\0\0\0\f\0\270\301\346\3\1\0\0\0\1\0\f\0,\0\0\0\33\0\0\0\2\0\24\0"..., iov_len=3684}, {iov_base="", iov_len=412}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 64
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\22\0\0\0\t\0\30\0\0\0\0\0\0\0\0\0<EMAIL_ADDRESS>iov_len=260}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 260
pselect6(4, [3], [], [], {tv_sec=0, tv_nsec=1611000}, NULL) = 1 (in [3], left {tv_sec=0, tv_nsec=1522398})
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\33\0\0\0\2\0\24\0\316\301\346\3\0\302\0\0\0\220\0\0\3\0\0\0\2\0\24\0\316\301\346\3"..., iov_len=3620}, {iov_base="", iov_len=476}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 40
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\20\0\0\0\20\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=176}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 176
pselect6(4, [3], [], [], {tv_sec=0, tv_nsec=826000}, NULL) = 0 (Timeout)
pselect6(4, [3], [], [], {tv_sec=0, tv_nsec=15023000}, NULL) = 1 (in [3], left {tv_sec=0, tv_nsec=7403023})
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="&\0\0\0\0\0\10\0\1\0\0\0\1\0\f\0&\0\0\0\33\0\0\0\2\0\24\0\327\301\346\3"..., iov_len=3580}, {iov_base="", iov_len=516}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 60
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\20\0\0\0\20\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=88}, {iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\20\0\0\0\20\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=88}], msg_iovlen=2, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 176
pselect6(4, [3], [], [], {tv_sec=0, tv_nsec=6692000}, NULL) = 1 (in [3], left {tv_sec=0, tv_nsec=5290997})
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\33\0\0\0\2\0\24\0\331\301\346\3\0\210\0\0\0g\0\0\3\0\0\0\2\0\24\0\331\301\346\3"..., iov_len=3520}, {iov_base="", iov_len=576}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 40
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\20\0\0\0\20\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=176}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 176
pselect6(4, [3], [], [], {tv_sec=0, tv_nsec=4850000}, NULL) = 1 (in [3], left {tv_sec=0, tv_nsec=215072})
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base=",\0\0\0\0\0\f\0\332\301\346\3\1\0\0\0\1\0\f\0,\0\0\0\33\0\0\0\2\0\24\0"..., iov_len=3480}, {iov_base="", iov_len=616}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 64
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\20\0\0\0\20\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=176}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 176
pselect6(4, [3], [], [], NULL, NULL)    = 1 (in [3])
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\33\0\0\0\2\0\24\0\345\301\346\3\0\23\0\0\0003\0\0\3\0\0\0\2\0\24\0\345\301\346\3"..., iov_len=3416}, {iov_base="", iov_len=680}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 40
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\20\0\0\0\20\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=176}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 176
pselect6(4, [3], [], [], NULL, NULL)    = 1 (in [3])
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\33\0\0\0\2\0\24\0\346\301\346\3\0\10\0\0\0001\0\0\3\0\0\0\2\0\24\0\346\301\346\3"..., iov_len=3376}, {iov_base="", iov_len=720}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 40
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\20\0\0\0\20\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=176}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 176
pselect6(4, [3], [], [], NULL, NULL)    = 1 (in [3])
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\33\0\0\0\2\0\24\0\347\301\346\3\0\0\0\0\0.\0\0\3\0\0\0\2\0\24\0\347\301\346\3"..., iov_len=3336}, {iov_base="", iov_len=760}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 40
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\4\0\0\0\4\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=176}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 176
pselect6(4, [3], [], [], NULL, NULL)    = 1 (in [3])
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\33\0\0\0\1\0\20\0<EMAIL_ADDRESS>iov_len=3296}, {iov_base="", iov_len=800}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 40
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\3\0\0\0\0\0\30\0C\254\2\0\17\0\0\0\4\0\0\0\4\0\0\0\17\0\0\0\1\0\24\0"..., iov_len=352}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 352
pselect6(4, [3], [], [], NULL, NULL)    = 1 (in [3])
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\20\0\0\0\2\0\20\0C\255\2\0\22\0\0\0\21\0\0\0\1\0\f\0\22\0\0\0 \0\0\0"..., iov_len=3256}, {iov_base="", iov_len=840}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 60
ioctl(2, TCGETS, {c_iflag=ICRNL|IXON|IXOFF|IUTF8, c_oflag=NL0|CR0|TAB0|BS0|VT0|FF0|OPOST|ONLCR, c_cflag=B38400|CS8|CREAD, c_lflag=ISIG|ICANON|ECHO|ECHOE|ECHOK|IEXTEN|ECHOCTL|ECHOKE, ...}) = 0
getpid()                                = 265627
write(2, "\n(editor:265627): Pango-\33[1;33mW"..., 125
(editor:265627): Pango-WARNING **: 01:39:05.721: Invalid UTF-8 string passed to pango_layout_set_text()
) = 125
ioctl(2, TCGETS, {c_iflag=ICRNL|IXON|IXOFF|IUTF8, c_oflag=NL0|CR0|TAB0|BS0|VT0|FF0|OPOST|ONLCR, c_cflag=B38400|CS8|CREAD, c_lflag=ISIG|ICANON|ECHO|ECHOE|ECHOK|IEXTEN|ECHOCTL|ECHOKE, ...}) = 0
getpid()                                = 265627
write(2, "\n(editor:265627): Pango-\33[1;33mW"..., 125
(editor:265627): Pango-WARNING **: 01:39:05.724: Invalid UTF-8 string passed to pango_layout_set_text()
) = 125
ioctl(2, TCGETS, {c_iflag=ICRNL|IXON|IXOFF|IUTF8, c_oflag=NL0|CR0|TAB0|BS0|VT0|FF0|OPOST|ONLCR, c_cflag=B38400|CS8|CREAD, c_lflag=ISIG|ICANON|ECHO|ECHOE|ECHOK|IEXTEN|ECHOCTL|ECHOKE, ...}) = 0
getpid()                                = 265627
write(2, "\n(editor:265627): Pango-\33[1;33mW"..., 125
(editor:265627): Pango-WARNING **: 01:39:05.727: Invalid UTF-8 string passed to pango_layout_set_text()
) = 125
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
sendmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\21\0\0\0\2\0\10\0\21\0\0\0\7\0\10\0 \0\0\0\10\0\20\0\264\0\0\0}\0\0\0"..., iov_len=160}], msg_iovlen=1, msg_controllen=0, msg_flags=0}, MSG_DONTWAIT|MSG_NOSIGNAL) = 160
pselect6(4, [3], [], [], NULL, NULL)    = 1 (in [3])
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base="\21\0\0\0\5\0\f\0\32\2\0\0", iov_len=3196}, {iov_base="", iov_len=900}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 12
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
pselect6(4, [3], [], [], NULL, NULL)    = 1 (in [3])
poll([{fd=3, events=POLLIN}], 1, -1)    = 1 ([{fd=3, revents=POLLIN}])
recvmsg(3, {msg_name=NULL, msg_namelen=0, msg_iov=[{iov_base=",\0\0\0\0\0\f\0S\304\346\3\1\0\0\0\1\0\f\0,\0\0\0", iov_len=3184}, {iov_base="", iov_len=912}], msg_iovlen=2, msg_controllen=0, msg_flags=MSG_CMSG_CLOEXEC}, MSG_DONTWAIT|MSG_CMSG_CLOEXEC) = 24
poll([{fd=3, events=POLLIN}], 1, 0)     = 0 (Timeout)
pselect6(4, [3], [], [], NULL, NULL^C)    = ? ERESTARTNOHAND (To be restarted if no handler)
strace: Process 265627 detached

Here, I find the fcitx5 IM mostly works and I obtain this

after 2 rounds of IM.
Capture with the auxiliary window at work:

The only problem is see is that entered text is selected so I have to type the right arrow key after.
I am puzzled.

Could you select text with space key?

It's 8pm here, so I'm stopping for today. Good evening.

Have a nice evening.

I was not aware of the use of space to enter the currently selected character(s). I was typing the number shown next to the desired character in the candidate box.
I confirm that doing exactly

When the candidate box appears, you can type "hahaha", press space to select the text, and then press backspace to delete the text.

does trigger a bug: further character insertions don't work.
Can you confirm, on your side, that if you type the right arrow key after each character insertion in the editor window, things run well?
I will try here to fix the state of the text buffer after each character insertion: the newly inserted text should not be selected.

With  →  (right arrow) it is possible to insert text that is not garbled, but I can't continue to type ,  (full-width comma, U+FF0C) with IME.
In fact I can't type the full-width ： (U+FF1A), “ (U+201C) , "”"(U+201D) yet.

When in Chinese Input Method mode, press : (U+003A), input method will automatically convert it to : (U+FF1A)

In most cases, we're typing not just a character, but a complete sentence, so various symbols are definitely used, as well as backspace, space and Enter.
For example: 🐱 is an emoji character.
🗓，🐱🍴🍗❔ is a sentence.

I tried WAYLAND_DEBUG=1 on this editor, it seems at least is has one common text-input-v3 bug, but probably also others too (offset are bytes not unicode character).
So first bug it has is the infinite loop message when using text-input-v3.
This due to every<EMAIL_ADDRESS>will be replied with<EMAIL_ADDRESS>And for every done, it will also carries the current preedit, like:
[3029701.208]  -> zwp_text_input_v3@17.commit()
[3029701.332] zwp_text_input_v3@17.preedit_string("f", 0, 1)
[3029701.382] zwp_text_input_v3@17.done(501)
[3029702.013]  -> zwp_text_input_v3@17.set_cursor_rectangle(13, 33, 5, 14)
[3029702.028]  -> zwp_text_input_v3@17.commit()
[3029702.144] zwp_text_input_v3@17.preedit_string("f", 0, 1)
[3029702.195] zwp_text_input_v3@17.done(502)
[3029702.818]  -> zwp_text_input_v3@17.set_cursor_rectangle(13, 33, 5, 14)
[3029702.832]  -> zwp_text_input_v3@17.commit()
[3029702.946] zwp_text_input_v3@17.preedit_string("f", 0, 1)
[3029702.995] zwp_text_input_v3@17.done(503)
[3029703.669]  -> zwp_text_input_v3@17.set_cursor_rectangle(13, 33, 5, 14)
[3029703.684]  -> zwp_text_input_v3@17.commit()
[3029703.819] zwp_text_input_v3@17.preedit_string("f", 0, 1)
[3029703.867] zwp_text_input_v3@17.done(504)
[3029704.547]  -> zwp_text_input_v3@17.set_cursor_rectangle(13, 33, 5, 14)
[3029704.564]  -> zwp_text_input_v3@17.commit()
[3029704.763] zwp_text_input_v3@17.preedit_string("f", 0, 1)
[3029704.810] zwp_text_input_v3@17.done(505)
[3029705.406]  -> zwp_text_input_v3@17.set_cursor_rectangle(13, 33, 5, 14)
[3029705.420]  -> zwp_text_input_v3@17.commit()

This has happened in other app that write their own text-input-v3 implementation.
You can check the fix that I have done to kitty before:
https://github.com/kovidgoyal/kitty/pull/5369/files
Basically the psuedo code should look like:
handle_done() {
    preedit_changed = (pending_preedit == preedit);
    preedit = pending_preedit;
    pending_preedit = null;  // It is important to reset, because if there is no preedit_string(), but only done(), it means clear preedit
    if (preedit_changed) {
       update_preedit....
    }
    /// ....
    /// e.g. ...update_and_redraw() .....
    /// ....
    if (cursor_rectangle != previous_cursor_rectangle) {
         previous_cursor_rectangle =  cursor_rectangle;
         send_cursor_rectangle()
    }
}

The blinking is also caused by this infinite message loop.
The second one is that preedit_string cursor is not respected.
e.g.
zwp_text_input_v3@17.preedit_string("f a", 0, 1)
Means, "f" should be highlighted, " a" should be in normal format.
Kwrite

Fltk

The third issue can also be demonstrated in the screenshot above:
The cursor rectangle should point to the start of cursor, not end of cursor. or preedit.
preedit_string("f a", 0, 1)  -> means the cursor rectangle, should point to the x axis of left bounding box of f, not the right bounding box of a.

@wengxt Many thanks for this information. I will dive in the code to try and improve it.

@2moe I believe to have improved a bit the FLTK code. Many other changes are still necessary to do what @wengxt described, but this will take time. Would you like to try the modified code, as follows ?
cd <directory where the FLTK git repository was downloaded 2 days ago>
git pull
cd build
make editor
bin/test/editor &

There should be some improvement, hopefully, although many problems remain.

https://github.com/fltk-rs/fltk-rs/assets/25324935/6699ee8e-23e3-4f76-be06-c736bf81a292
If I adjust the window size of the fltk editor while typing, the candidate window of IM will keep blinking.
After the input is complete, pressing ' multiple times will move the cursor to the left.

type "haha", and press space, it's no problem.
However, when I then pressed ！, the fltk-editor does not respond.
[3960164.319] zwp_text_input_v3@17.commit_string("哈哈")
[3960167.126]  -> zwp_text_input_v3@17.commit()
[3960167.175] zwp_text_input_v3@17.done(176)
[3960167.184] zwp_text_input_v3@17.done(176)
[3960170.932]  -> zwp_text_input_v3@17.set_cursor_rectangle(33, 34, 5, 14)
[3960171.025]  -> zwp_text_input_v3@17.commit()
[3960171.178] wl_display@1.delete_id(39)
[3960171.196] wl_callback@39.done(37599569)
[3960174.313]  -> wl_surface@18.damage_buffer(0, 0, 1000000, 1000000)
[3960174.340]  -> wl_surface@18.attach(wl_buffer@36, 0, 0)
[3960174.352]  -> wl_surface@18.set_buffer_scale(2)
[3960174.372]  -> wl_surface@18.frame(new id wl_callback@39)
[3960174.377]  -> wl_surface@18.commit()
[3960174.427] zwp_text_input_v3@17.done(177)
[3960174.434] zwp_text_input_v3@17.done(178)
[3960174.438] zwp_text_input_v3@17.done(179)
[3960205.504] wl_display@1.delete_id(39)
[3960205.536] wl_callback@39.done(37599602)
[3960552.703] wl_keyboard@16.modifiers(53332, 1, 0, 16, 0)
[3960552.927] wl_keyboard@16.key(53334, 37588418, 42, 1)
[3960653.751] zwp_text_input_v3@17.commit_string("！")
[3960653.862]  -> zwp_text_input_v3@17.commit()
[3960653.873] zwp_text_input_v3@17.done(179)
[3960654.060] zwp_text_input_v3@17.done(180)
[3960975.719] zwp_text_input_v3@17.commit_string("！")
[3960975.807]  -> zwp_text_input_v3@17.commit()
[3960975.826] zwp_text_input_v3@17.done(180)
[3960976.168] zwp_text_input_v3@17.done(181)

Type “haha”, press space, then press 】 multiple times, then press backspace, it will trigger the garbled characters bug.

@2moe I have followed what @wengxt suggested and believe FLTK should now work much better with fcitx5. There's still a tendency to repeat some characters when in Latin input mode, and I still have to look at that.
Would you like to test the modified code and report here if it's better and if there remains problems? TIA. The procedure is the same as before :
cd <directory where the FLTK git repository was downloaded 3 days ago>
git pull
cd build
make editor
bin/test/editor &

@ManoloFLTK Yes, now fltk apps work with fcitx5 IM.
Thank you very much.

@2moe I really have to thank you because without a patient and knowledgeable fcitx5 user I could never have come out with a correct FLTK code.
In one of your posts above you gave an example of use of a Japanese IM. Do you think you could help me evaluate whether FLTK handles now such IM correctly?

Great. Many thanks again.
Manolo

Thank you all @ManoloFLTK, @2moe and @wengxt .
@2moe I've pull the updated FLTK sources into master.
If you would like to try it, you can do so by adding the following to your Cargo.toml file:
[patch.crates-io]
fltk = { git = "https://github.com/fltk-rs/fltk-rs" }

The next patch release will most likely be next week.

@MoAlyousef Thank you.


Change the font list

I have this problem guys

Its look bad. It occurs just in the list

Do you have the Roboto font family installed on your system? I had a font related issue myself (#74) and installing the font fixed it.

@Wootness Yes, the font Roboto is installed in my system.


Tauri

Website
https://tauri.app/
Documentation
https://tauri.app/v1/guides/getting-started/setup/sveltekit
Integration summary

Make sure all prerequisites are met: https://tauri.app/v1/guides/getting-started/prerequisites/ (depends on #7)
use adapter static
enable prerendering (modify +layout.js
Create the rust project npm run tauri init

Further details
This composer would only support kit.
Is there a way we can test this composer automatically? Currently I don't see a way as it requires so many and different prerequisites, that differ from os

Project is now using the svelte-add name again. Issue moved: https://github.com/svelte-add/svelte-add/issues/338


Don't set reset observer callback & set CB after world_ is initialized

Description
Fix segfault caused by https://github.com/moveit/moveit2/pull/2858
Checklist

[ ] Required by CI: Code is auto formatted using clang-format
[ ] Extend the tutorials / documentation reference
[ ] Document API changes relevant to the user in the MIGRATION.md notes
[ ] Create tests, which fail without this PR reference
[ ] Include a screenshot if changing a GUI
[ ] While waiting for someone to review your request, please help review another open pull request to support the maintainers

Sorry, I don't use any collision update callback and didn't test this.


[BUG] When import deeplake

🐛🐛 Bug Report
When I am trying to install deeplake and import it on Google colab, its shown that deeplake have error with numpy, please check
!pip install deeplake -> OK
import deeplake -> error
module 'numpy' has no attribute 'float'. np.floatwas a deprecated alias for the builtinfloat. To avoid this error in existing code, use floatby itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, usenp.float64 here.

Thanks @davidlee1102 for reporting the issue, do you mind sharing numpy version?

The aliases was originally deprecated in NumPy 1.20, you can check on that

@davidlee1102 we do not use the np.float alias anywhere in the codebase. Can you post the full traceback?

Thank you for your response; I have rechecked that the error comes from Colab and the library Numpy from Colab; it did not come from your code.
Now I will close the bug; thank you for providing us with the deep library.

thanks a lot for reporting the issue and let us know if we can be of help down the road!


Add ripple effect to buttons in website

I add the ripple effect to Top button in developer page, by using a online lib. I already create a pull request, you can check if this effect meet your expection

Cool ripple button !


Views with an array layout should be indexable with a slice

Currently this breaks (with a nicer message since c649045f354db58e2b7e2ced4b7fbd0de3d71f90 / #1375) but there's no reason this can't work; it should return another ArrayLayout instead:
  File "/home/whitequark/Projects/glasgow/software/test.py", line 35, in elaborate
    m.d.sync += o_reg.eq(o_reg[1:])
                         ~~~~~^^^^
  File "/home/whitequark/Projects/glasgow/software/.venv/lib/python3.12/site-packages/amaranth/lib/data.py", line 796, in __getitem__
    raise TypeError(
TypeError: View cannot be indexed with a slice; did you mean to call `.as_value()` first?

cc @isabelburgos

The workaround for the pattern above is fairly annoying too:
m.d.sync += o_reg.eq(o_reg.as_value()[len(o_reg[0].as_value()):])
# instead of
m.d.sync += o_reg.eq(o_reg[1:])


update deps

Purpose
update to latest dependencies
closes #690
Content

CLIMAParameters -> ClimaParams: This PR touches a lot of files because of the renaming of ClimaParams
ClimaLand v0.11: Albedo struct types were renamed, and BucketModelParameters constructor changed following ClimaLand's switch to using ClimaParams. Note that for some parameters we aren't using the defaults (but probably could)
ClimaCore v0.13: Removed a few deprecated functions, which were only being used in seabreeze

LGTM. Is the static albedo map corrupted for that one run on Buildkite? Maybe it'll work if you trigger a new build?

Yes! The agent was lost while the file was being accessed, so it got corrupted. I'm clearing the depot, and then it should work when I re-build


type mismatches within if statements do not localize the error

For this example:
type alias Model = { i:Int, s:String, x:Int }

foo : String -> Float -> Float -> Model -> Model
foo s f1 f2 m =
    if | m.s == s ->
            { m
            | i <- 0
            , x <- round <| m.i * (f1-f2)
            }
       | otherwise -> m

The following error is not helpful enough:
-- TYPE MISMATCH --------------------------------------------------------------

This expression is triggering a type mismatch.

 4|
 5|     if | m.s == s ->
 6|             { m
 7|             | i <- 0
 8|             , x <- round <| m.i * (f1-f2)
 9|             }
10|        | otherwise -> m
11|

To be more specific, as I infer all the types, I am seeing a conflict between
this type:

    Int

and this type:

    Float

A better message would highlight the fact that the problem is with the expression m.i * (f1-f2) on line 8.

The 0.16 error message is pretty interesting:

So this is the root error, just with more context and stuff. I think it makes sense to close this and maybe make a meta-issue along with #28 about only using information from type annotations after the entire body has been checked. In this case, the body is self-consistent, which is kind of weird!

Things are different now. I think we should update the example in a new issue if the problem is the same.


[QUESTION] PVC not attaching due to problematic iscsid on LXD container

I can not create a simple PVC and pod to use it as defined in https://raw.githubusercontent.com/longhorn/longhorn/v1.1.1/examples/pod_with_pvc.yaml
Environment:

Longhorn version: 1.1.1
Kubernetes version: 1.21.0
Node config

OS type and version: Ubuntu 18.04 (can reproduce this with 20.04 as well since both use iscsid version 2.0-874)

Underlying Infrastructure: LXD

Additional context
This is a 2 node cluser: 1 master, 1 worker. Keeping it small in order to work past this issue.
The iscsid service is running on each node.
Here's the LXD profile
config:
  limits.cpu: "2"
  limits.memory: 4GB
  limits.memory.swap: "false"
  linux.kernel_modules: overlay,nf_nat,ip_tables,ip6_tables,br_netfilter,iscsi_tcp
  raw.lxc: |
    lxc.apparmor.profile = unconfined
    lxc.cgroup.devices.allow = a
    lxc.mount.auto=proc:rw sys:rw
    lxc.cap.drop =
  security.nesting: "true"
  security.privileged: "true"
description: Default LXD profile
devices:
  eth0:
    nictype: bridged
    parent: br0
    type: nic
  root:
    path: /
    pool: default
    type: disk
name: k8s
used_by:
- /1.0/instances/kmaster1
- /1.0/instances/kworker1

And back on the container, we can see that the iscsi_tcp kernel module is loaded
root@kworker1:~# lsmod | grep iscsi
iscsi_tcp              24576  0
libiscsi_tcp           32768  1 iscsi_tcp
libiscsi               57344  2 libiscsi_tcp,iscsi_tcp
scsi_transport_iscsi   110592  3 libiscsi_tcp,iscsi_tcp,libiscsi

Immediately after installing longhorn, I attempt to create a single PVC and pod that uses it with https://raw.githubusercontent.com/longhorn/longhorn/v1.1.1/examples/pod_with_pvc.yaml
I see the following in the logs from the instance-manager pod that attempts to handle the request:
2021/05/12 18:53:05 failed to start up frontend: Failed to execute: nsenter [--mount=/host/proc/1/ns/mnt --net=/host/proc/1/ns/net iscsiadm -m node -T iqn.2019-10.io.longhorn:pvc-83ade826-333d-428b-a119-3098d3f41226 -p <IP_ADDRESS> --login], output Logging in to [iface: default, target: iqn.2019-10.io.longhorn:pvc-83ade826-333d-428b-a119-3098d3f41226, portal: <IP_ADDRESS>,3260] (multiple)
, stderr, iscsiadm: got read error (0/0), daemon died?

From the worker node, I can discover targets provided by the instance-manger pod in question:
# iscsiadm -m discovery -p <IP_ADDRESS>:3260 --type sendtargets
<IP_ADDRESS>:3260,1 iqn.2019-10.io.longhorn:pvc-83ade826-333d-428b-a119-3098d3f41226

However, it seems I can not login?
# iscsiadm -m node -L all
Logging in to [iface: default, target: iqn.2019-10.io.longhorn:pvc-83ade826-333d-428b-a119-3098d3f41226, portal: <IP_ADDRESS>,3260] (multiple)
iscsiadm: got read error (0/0), daemon died?
iscsiadm: Could not login to [iface: default, target: iqn.2019-10.io.longhorn:pvc-83ade826-333d-428b-a119-3098d3f41226, portal: <IP_ADDRESS>,3260].
iscsiadm: initiator reported error (18 - could not communicate to iscsid)
iscsiadm: Could not log into all portals

At this point, the iscsid service is now in a state of restarting over and over.

Hi! I have done some searching since the post in slack. From these tickets in lxc/lxd/forum suggest that the problem is a known issue with lxd support on iscsi and Longhorn required hosts to support iscsi.

https://github.com/lxc/lxd/issues/6536
https://github.com/lxc/lxc/issues/177
https://bugs.launchpad.net/ubuntu/+source/lxc/+bug/1226855
https://discuss.linuxcontainers.org/t/open-iscsi-failed-to-start-in-two-separate-lxc-containers-on-two-different-machines/3166

Thank you very much for the help. I'll review those and continue watching this thread. I'll also post any progress I'm able to make.

HI! Since this is a known issue for LXC/LXD, I will close the issue for now.  Feel free to reopen the issue, if you have any additional feedback!

This could be helpful - basically mount the host's iscsiadm into your LXD container like so:
lxc config device add container-dev-k3s-s1 sdb1 disk source=/sbin path=host/sbin lxc config device add container-dev-k3s-s1 sdb2 disk source=/etc/iscsi path=host/etc/iscsi
Then
ln -s /host/sbin/iscsiadm /sbin/iscsiadm ln -s /host/etc/iscsi /etc/iscsi
Trying to get this work at present time..

This could be helpful - basically mount the host's iscsiadm into your LXD container like so:
lxc config device add container-dev-k3s-s1 sdb1 disk source=/sbin path=host/sbin lxc config device add container-dev-k3s-s1 sdb2 disk source=/etc/iscsi path=host/etc/iscsi
Then
ln -s /host/sbin/iscsiadm /sbin/iscsiadm ln -s /host/etc/iscsi /etc/iscsi
Trying to get this work at present time..

Hi, just wondering if this approach was successful?

Why was this closed?

Why was this closed?

Mistakenly closed while migrating zenhub to github project.


Added aop logging to controller's public method

 Bank: Ensure everything is properly logged

 Bank: Ensure everything is properly logged

 Bank: Ensure everything is properly logged


Draw APIs

This is a partial successor to #16, mapping out how custom draw routines are supported. It should support both:

a medium-level API independent of the toolkit backend, providing things like "draw quad" (provided by kas::draw::Draw already, but needs plenty of refinement)
a low-level API providing direct access to whatever draw API the toolkit uses (I have some ideas, just needs some hacking)

#63 enabled usage of the medium-level API for widget drawing; some other recent work improved this API.


added space to debug message

IzPack project » izpack #717 SUCCESS
This pull request looks good
(what's this?)

Thank you.


Tag names - unique constraint.

Is there a reason why the tags.name column doesn't have a unique constraint?  I searched through the issues and didn't really see any direct discussion, despite there being some previous issues on attempts to prevent duplicates.
I think historically the tag input widget has attempted to enforce uniqueness of tag names but now with the addition of the tag management page, tags can be created/edited to have duplicate names.  Unfortunately there are areas of the code that assume a tag name is unique--the tag suggestion list in the editor, as well as the updateTags method in the post model class, for example.  If you create two tags with the same name things start behaving badly.
I can't really think of a use case for two tags to have the same name?  It seems to me that we ought to add a unique constraint to tags.name in addition to trying to prevent this in the various tag UIs.

Be good to get this change made, thinking about it the reason why the unique constraint is not there, is because post.title and user.name doesn't have it, so it was just an oversight that tag.name should have it.
The main issue we're going to have with adding the constraint now, is that the code doesn't prevent duplicates, so there may be duplicates in the database already. I'm not sure how we can fix that so that the constraint can be added :anguished:
Adding an order integer to posts_tags would also be good as we could start to work towards outputting tags in the order they were added to the post, and allowing that to be re-organised.

Although some time has passed since this was discussed I'd like to reopen it, because in my imho it is an unnecessary constrain to theme and app developers.
To be honest - I'd like to ask the question the other way around: which reason is there to make the tag names unique?

imho for the uniqueness of tags the url assigned to the tag should be enough to identify the tag
as soon as hirachical tags arise it(uniqueness of names) will constrain us from using tags with the same names but different semantic meanings and urls in differentt sub trees/paths of the hirachy, because they have to be unique.
They're unnecessary extra db constraints

As an example: I am currently trying to abuse tags for navigation in a multilingual ghost...where it is nothing but unusual for the tags  to have the same names (in both languages).
To decide which tag in which language to choose in the editor I've added a "language-hash-tag" to the tag (for example "Java & More #de") and removed it in the UI using a custom helper.
I would rather propose an optional or wildcard field (free to use for whatever) to tags. This would enable theme and app developers to do more with tags and the upcoming tag hirachies than the usual blog offers atm.
Like for example implement and offer more complex and/or specialiced tag based navigations implemented using ghost and it's api only.
Also multilingual blogs with the same tag names in n (being the language count)-hirachies would enable ghost to offer sth no other lightweight blogging plattform can offer - being honestly multilingual.
I am aware of the fact that the editior's multiselect textfield's autocomplete would have to be refactored to also show the url or the tag's optional field to securely decide which tag to use.

Been digging through really old issues...
I think all the really bad UI issues with this are gone now that the tag input component has been rebuilt.
Now that we have a concept of visibility, I can start to see more cases where we might not want to enforce unique tags.
This hasn't really come up as causing issues, so I think we can safely close this for now and revisit it if we decide to change to logic around tag names.


Unknown plan node type com.facebook.presto.sql.planner.plan.TableWriterMergeNode when running CREATE AS SELECT

Bug description
I found that simple query such as
create table ivan_test3 as select 1 as a, 2 as b;

fails in Prestissimo with
VeloxUserError:  Unknown plan node type com.facebook.presto.sql.planner.plan.TableWriterMergeNode

I could reproduce it in Presto + Velox setup as well as in a unit test (works fine with Presto Java). It seems we always need to set table_writer_merge_operator_enabled to false to make this work (practically all Prestissimo tests set this flag).
Do you know if Velox inherently cannot support TableWriterMergeNode or has it not simply been added yet? Also, what is the difference between TableWriterMergeNode and TableWriteNode and why does Presto choose the former when running CTAS? I could not find much information on TableWriterMergeNode purpose so I would greatly appreciate more context on this operator. Thanks!
System information
Reproducible on the latest master and seems to be a known issue. I am just trying to collect information on this and see if it is support is simply missing.
Relevant logs
No response

@majetideepak @aditi-pandit Do you know where I can find more context on the difference between TableWriterMergeNode and TableWriteNode? Should we support this operator in Velox? Thank you!

@isadikov  TableWriterMergeNode is to merge the meta outputs from multiple table writer. We are going to add this support soon. Thanks!


Fix callback forwarding in async plan method for OpenAI function agent

The callback argument was missing, preventing me to get callbacks to work properly when using it async

thanks @rogeriochaves!


Git

Describe the bug
after creating new repository from the stackblitz this error can be seen.
Link to the blitz that caused the error
https://stackblitz.com/edit/node-2dttn2?file=bestSum.js
Steps to reproduce
in the project,
click on connect repository.
it creates new repository but unable to push the code into it and keeps on loading.
Expected behavior
should be able to push the code.
Screenshots

Platform
Browser name  = Chrome
Full version  = 96.0.4664.45
Major version = 96
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.45 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 94762484,
  "usedJSHeapSize": 77072552,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = f0f0e35d34e6f38828752db79b041c48b98806b5

Additional context
No response

Duplicate of #431

@gyansatapathy #431 was fixed, so I'm closing this. Please feel free to open a new issue if you still see something breaking.


Reproduce differential analysis with a published dataset

Find a simple dataset from ENCODE with e.g. the same cells in 2 different conditions. See if we can reproduce the differential peak analysis results.
Potential paper to use:

Eder, T., Grebien, F. Comprehensive assessment of differential ChIP-seq tools guides optimal algorithm selection. Genome Biol 23, 119 (2022). https://doi.org/10.1186/s13059-022-02686-y

Instead of the DCSsim/DCSsub datasets, let's use #165

Question: how did the DCS authors run diffbind, since it can use either EdgeR and DESeq2? https://github.com/Edert/DCS_predictions/blob/main/run_pd_diffbind.sh

superseded by https://github.com/CCBR/CHAMPAGNE/issues/165


How do you override models with swagger-core 1.5.3?

Hi -
I'm trying to upgrade to swagger-core 1.5.3 and I've discovered that the OverrideConverter class has been removed.  However, the documentation on the wiki still refers to this class.  What should I be using in the new version to achieve the same functionality?

So yeah, it has changed and not documented yet (sorry about that).
In the meantime, you can have a look at the test - https://github.com/swagger-api/swagger-core/blob/master/modules/swagger-core/src/test/java/io/swagger/model/override/CustomConverterTest.java - and we'll do our best to answer any follow up questions you may have.

Thank you very much!  I used that example along with this: https://github.com/swagger-api/swagger-core/blob/master/modules/swagger-core/src/test/java/io/swagger/model/override/SamplePropertyConverter.java to figure out how to create a property (that example you linked me to just returns null).
I think I have everything set up.  But now I'm wondering, where do I put this code. The old way was to put it in the constructor of a servlet class.  Newer documentation puts a lot of startup type code in the the init function of a servlet class, which is what I did.  But it doesn't appear to be working.  I'm wondering if it's because you have to create a local instance of the ModelConverters class instead of using it statically.  I hope that question makes sense...

Never mind!  I figured it out.  I needed to use the ModelConverters.getInstance() method.  Thanks again for your help!

Glad it worked out. Can we close this issue?

It is still not documented though  :(


Remove logitech-control-center Cask

PR can be leverage, in case the maintainer decision is to delete logitech-control-center Cask
Ref : #30901

Does it mean that there's no more way to make extra buttons on Logitech mice work?

I guess so. when tried a manually install of existing software it redirected me to https://secure.logitech.com/maclcc, but that page is issuing a 404. I believe Logitech discontinues this development

Ugh, what a mess. Particularly funny since this mouse works on Linux out of the box since, like, forever.


Loading a X509Certificate2 causes a CGI exception on Azure App Service

I am trying to load a .p12 file into a X509Certificate2 object.
The code runs fine on my local machine.
Whenever, I run the following line on Azure, it crashes the asp.net process with a CGI error:
var certificate = new X509Certificate2(fileBytes, "notasecret", X509KeyStorageFlags.Exportable);
The following line does not crash:
var cert = new X509Certificate2();
Also, if the password is incorrect, the server gives a normal exception explaining that the password is incorrect:
var certificate = new X509Certificate2(fileBytes, "wrong");
I upgraded to 1.0.1 hoping it would solve the problem, but it occured on both 1.0.0 and 1.0.1.
So, to be clear:

Only crashes on Azure App Site (with CGI error)
Only happens when the cert file password is correct

Here is the full code:
        ```
        var log = "";

        var keyFileServerPath = _env.ContentRootPath + @"\GACert.p12";
        log += "\r\n" + "Got File Path";

        var cert = new X509Certificate2();
        log += "\r\n" + "Created Empty Cert";

        var fileExists = System.IO.File.Exists(keyFileServerPath);
        log += "\r\n" + "File Exists? =" + fileExists;

        var fileBytes = System.IO.File.ReadAllBytes(keyFileServerPath);
        log += "\r\n" + "File Bytes.Length=" + fileBytes.Length;

        //// WRONG PASSWORD - Causes a normal Exception (so password is OK)
        //var certificate = new X509Certificate2(fileBytes, "wrong");
        //log += "\r\n" + "Created Cert Using Bytes";

        //// FAILED - CGI Error (Crashing the Asp.Net Process)
        //var certificate = new X509Certificate2(fileBytes, "notasecret", X509KeyStorageFlags.Exportable);
        //log += "\r\n" + "Created Cert Using Bytes";

Can you check if your cert is expired?  And try with a non-expired cert?   On a desktop/laptop machine, this works fine, but on Azure App Service, this seems to be failing.  See App Service forum entry, or the issue where this was first filed on.
cc @davidebbo

The cert was just generated yesterday, so I don't know how it would be expired.

Have you found any solution. I have the same problem - framework net461?

The problem is the Azure Web Apps restricts access to the machines private key store. Please note that this only works for Basic Tier and above (not Free or Shared tier). Are you in this case?

Hej,
I have the same issue but with Azure Functions. The certificate works perfectly in my local machine but It does not work in Azure Portal. It throws the following exception.
Error in OnTriggerAuditCenterOperationStatus : Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index   at System.Collections.CollectionBase.System.Collections.IList.get_Item(Int32 index)
at System.Security.Cryptography.X509Certificates.X509Certificate2Collection.get_Item(Int32 index)
at qnet.concierge.core.helpers.ManagementCertificateService.GetX509Certificate2BySubjectName(String subjectName)
at qnet.concierge.core.inventory.SystemCenterSubscriptionInventoryHandler.d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at qnet.concierge.core.audit.SystemCenterJobAuditor.d__3.MoveNext()

I solved this issue by generating a new ssl certifiacte using command makecert. Something was missing in my original certificate (I can not remember details now)

This still is not fixed in .Net Core 2.1.2, I have the exact same issue. A certificate generated with OpenSSL crashes the process when it is used to create an instance an X509Certificate2. It works perfectly on premise under Win10 and fails when running in an Azure WebApp.

This issue was moved to dotnet/corefx#28860


Within a docker build, Modules dependant on node-gyp fail (only where npm-shrinkwap in use)

Not sure which is the best place for this,
We are using a number of npm packages which in turn have a dependancy on node-gyp,
i.e. my project does not directly use node-gyp
We run npm-install as part of our docker image build process..
When I put a npm-shrinkwrap.json in place the npm install is failing due to errors relating to python not being in place..
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
gyp ERR! stack     at failNoPython (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:449:14)
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:353:11
gyp ERR! stack     at F (/usr/local/lib/node_modules/npm/node_modules/which/which.js:69:16)
gyp ERR! stack     at E (/usr/local/lib/node_modules/npm/node_modules/which/which.js:81:29)
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/which/which.js:90:16
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/which/node_modules/isexe/index.js:44:5
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/which/node_modules/isexe/access.js:8:5
gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:123:15)
gyp ERR! System Linux 4.4.0-59-generic
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"

Maybe I need to include a python install as part of pre-build
But my confusion is, why is would fail for this reason only when the npm-shrinkwrap is in place?

Maybe there are install scripts that start node-gyp, that's something a quick grep -r through your node_modules directory should be able to tell you.
If that isn't it, you'll probably have more luck getting your question answered at https://github.com/npm/npm/issues.  node-gyp's role is a passive one, it only does what it's told by the process that started it, presumably npm.

node-gyp is being kicked off by a https://github.com/kesla/node-snappy
I was able to get it to work by installing python, make and GCC as part of the docker build process, but it seems strange to me that this is necessary in this case

The solution was to modify my docker image to install the dependancies of node-gyp:
make, GCC and python
So that the node-gyp script could run.
As to why the behaviour is different from the shrinkwrap non skrinkwrapped is still a bit of a mystery to me

@vidhill Can you please provide your dockerfile I am constanly getting Gyp errors - sepecifally around Python paths.

Hi,
I can't share the exact Dockerfile, as it's not an open source project, for now I'll create a gist..
https://gist.github.com/vidhill/0a85dc1848feee4171944dc4d7757895
-this is an untested Dockerfile because I'm off to bed now..
if this doesn't help I can create a proper mini-project


Error in installation

Describe the bug
Can't install
To Reproduce
Just install via manager
Error in console:
Install custom node 'Crystools'
Install: ['https://github.com/crystian/ComfyUI-Crystools']
Download: git clone 'https://github.com/crystian/ComfyUI-Crystools'
Install: pip packages

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', 'deepdiff']
  [ComfyUI-Manager] skip black listed pip installation: 'torch'

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', 'numpy']

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', 'Pillow>=9.5.0']

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', 'pynvml']

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', 'py-cpuinfo>=9.0.0']

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', 'piexif']

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', '--extra-index-url https://test.pypi.org/simple']
[!]
[!] Usage:
[!]   /config/05-comfy-ui/env/bin/python3 -m pip install [options] <requirement specifier> [package-index-options] ...
[!]   /config/05-comfy-ui/env/bin/python3 -m pip install [options] -r <requirements file> [package-index-options] ...
[!]   /config/05-comfy-ui/env/bin/python3 -m pip install [options] [-e] <vcs project url> ...
[!]   /config/05-comfy-ui/env/bin/python3 -m pip install [options] [-e] <local project path> ...
[!]   /config/05-comfy-ui/env/bin/python3 -m pip install [options] <archive url/path> ...
[!]
[!] no such option: --extra-index-url https://test.pypi.org/simple
install script failed: https://github.com/crystian/ComfyUI-Crystools

## ComfyUI-Manager: EXECUTE => ['/config/05-comfy-ui/env/bin/python3', '-m', 'pip', 'install', 'pyrsmi==0.0.6']
[!] ERROR: Could not find a version that satisfies the requirement pyrsmi==0.0.6 (from versions: 1.0.1)
[!] ERROR: No matching distribution found for pyrsmi==0.0.6
install script failed: https://github.com/crystian/ComfyUI-Crystools
Installation was successful.

I will fix it today

Hi, I add the same problem.
You can install pyrmsi by yourself by 'pip install pyrsmi', it seems to work well.

Hi, I had the same problem. You can install pyrmsi by yourself with 'pip install pyrsmi', it seems to work well.

Im not computer wiz, but i try copy paste pip install text into powershell or cmd, just error.....any other more user friendly installlation for that pyrsmi

i am also getting this error. also tried what m-rdev said, but didn't work for me either.

From the root folder run:
python_embeded\python.exe -m pip install pyrsmi
It help me

yes Thanks god bro.....for those who trouble

Crystools Updated! Please try to updated (should be 1.16.0)


Added TwoPointersAlgo.java

This is an algorithm which reduces the time complexity while finding all pairs in an array that sum up to a given number.

Description

Related Issue

This PR closes #
Motivation and Context

Screenshots (if any)
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

Thanks for PR! Merged 👍

@SVijayB Could you please mention one of the topics in your Repo as "hacktoberfest-accepted"

@varunvjha if the pull request is merged into a repository with the label hacktoberfest, it is automatically considered towards progress.


Meaningful errors for non-lattice dispatches #793

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

@FyzHsn For some reason Start Covalent dispatcher server was skipped which made tests/covalent_tests/dispatcher_plugins/local_test.py fail.

@kessler-frost this can be closed too since #1600 was recently merged


Pi Zero W and Pi 3A+ all connect to the cloud but all return "nothing is listening on port 80" errors on the public URL.

I have tried to get the Balena-cam working today on both a Pi 3A+ and a Pi Zero W via the "Deploy with Balena" route but while both connect to my account / the cloud, each one yields the following error when trying to access the public IP address to view the camera:
"We encountered an error when reaching this balena.io device:
(ID)
tunneling socket could not be established: 500
One possible reason is because nothing is listening on port 80 on the device."
I have tried the current builds (as of today, 03/07/22) for the Pi Zero and both 32 and 64 bit versions for the Pi 3. In all cases I am using the official camera module and the hardware in each case works from Pi OS.
The other devices I have currently running other Balena tasks are all still connecting so I don't think it's a connectivity issue my end or a service issue at Balena.
Here is the log from the Pis on startup:
Supervisor starting
Applying configuration change {"SUPERVISOR_POLL_INTERVAL":"900000","SUPERVISOR_DELTA":"1","SUPERVISOR_DELTA_REQUEST_TIMEOUT":"59000","SUPERVISOR_DELTA_VERSION":"3"}
Applying boot config: {"gpu_mem_1024":"448","gpu_mem_256":"192","gpu_mem_512":"256","start_x":"1","avoid_warnings":"1","disable_splash":"1","dtparam":["i2c_arm=on","spi=on","audio=on"],"gpu_mem":"16"}
Applied boot config: {"gpu_mem_1024":"448","gpu_mem_256":"192","gpu_mem_512":"256","start_x":"1","avoid_warnings":"1","disable_splash":"1","dtparam":["i2c_arm=on","spi=on","audio=on"],"gpu_mem":"16"}
Applying boot config: {}
Applied configuration change {"SUPERVISOR_POLL_INTERVAL":"900000","SUPERVISOR_DELTA":"1","SUPERVISOR_DELTA_REQUEST_TIMEOUT":"59000","SUPERVISOR_DELTA_VERSION":"3"}
Applied boot config: {}
Applying configuration change {"SUPERVISOR_VPN_CONTROL":"true"}
Applied configuration change {"SUPERVISOR_VPN_CONTROL":"true"}
Rebooting
Supervisor starting
Creating network 'default'
Supervisor starting
Can someone please offer me some assistance in this ?

Same here! I have tried this with "Deploy with Balena" and ended as you.


robots/configuration docs

What type of change is this?

[x] Doc update,

Checklist

[x] I ran all tests on my computer and it's all green (i.e. invoke test).
[x] I ran lint on my computer and there are no errors (i.e. invoke lint) .

Notes
I'm hosting the sphinx docs generated from these file changes to make reviewing easier.

Done!


Feature request: Allow type declarations (f as Foo) in using block expression.

It would be very useful to be able to write this:
using f as Foo = AcquireAndReturnSubtypeOfFoo():
f.DoFooStuff()
This is currently not allowed by the compiler (the "as Foo" part) but similar constructions are allowed in C# and Visual Basic and it would be nice to have this behavior.

Unfortunately, there's no good way to support this.
using is a macro, and macros can have expressions as arguments, and a Block as the body.  But variable as type = SomeValue is a DeclarationStatement, which is a Statement rather than an Expression, and trying to change that would require changing the parser and the AST and potentially breaking a lot of things.

Maybe one way to achieve this more easily is with the experimental 'var' macro. using var f = Acq... this way 'f = Acq' is an expression while the inner var macro can add the declaration statement to the AST.

@popcatalin81 That still won't parse, because macros are Statements, not Expressions.


V0.14.0

Codecov Report

Merging #152 into master will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
- Coverage    53.5%   53.34%   -0.16%
==========================================
  Files          19       19
  Lines        1284     1284
==========================================
- Hits          687      685       -2
- Misses        597      599       +2

Impacted Files
Coverage Δ

...eleton/modules/autoupdate/assetBundleDownloader.js
92.66% <0%> (-1.84%)
:arrow_down:

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 035ae56...dd13929. Read the comment docs.


Sink Stored Procedure table name is required

Hi all,
I am copying data from a sql table to another one. The sink has a stored procedure which in which an SQL Merge is done. The stored procedure has been tested directly and i can confirm that it works. I also have several different copy-activities with the same logic.
However, for 2 of them, I keep getting the following Factory Validation Output error.

Sink Stored Procedure table name is required

Any idea where this might come from?
Here the type and procedure:
-- create type
create type [sta].[my_type] as table (
	[column1] [nvarchar](255) null,
	[column2] [nvarchar](255) null,
	[column3] [nvarchar](255) null,
	[column4] [nvarchar](255) null,
	[column5] [nvarchar](255) null,
	[column6] [nvarchar](255) null
)

GO

-- create procedure
create procedure [cdw].[sp_load_table] @mytable [sta].[my_type] readonly
as
begin
    merge [cdw].[mytable] as target
	-- handle duplicates
    using (select distinct * from @mytable) as source
    on (
		    target.[column1]=source.[column1]
		and target.[column2]=source.[column2]
		and target.[column3]=source.[column3]
		and target.[column4]=source.[column4]
		and target.[column5]=source.[column5]
		and target.[column6]=source.[column6]
    )
    when matched and (
		   COALESCE(target.[column1],1)<>COALESCE(source.[column1],1)
		or COALESCE(target.[column2],1)<>COALESCE(source.[column2],1)
		or COALESCE(target.[column3],1)<>COALESCE(source.[column3],1)
		or COALESCE(target.[column4],1)<>COALESCE(source.[column4],1)
		or COALESCE(target.[column5],1)<>COALESCE(source.[column5],1)
		or COALESCE(target.[column6],1)<>COALESCE(source.[column6],1)
    ) then
        update set
			 target.[column1]=source.[column1]
			,target.[column2]=source.[column2]
			,target.[column3]=source.[column3]
			,target.[column4]=source.[column4]
			,target.[column5]=source.[column5]
			,target.[column6]=source.[column6]
    when not matched by target
        then insert
            (
             [column1]
			,[column2]
			,[column3]
			,[column4]
			,[column5]
			,[column6]

            )
        values
            (
			     source.[column1]
				,source.[column2]
				,source.[column3]
				,source.[column4]
				,source.[column5]
				,source.[column6]
            )
    when not matched by source
        then delete;
end

And here the ADF activity:

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

ID: a2e7af17-5daf-eebf-92b4-433eca93d328
Version Independent ID: cce235ca-d882-4f70-cacc-b3c2c7feae13
Content: Copy data to and from SQL Server by using Azure Data Factory
Content Source: articles/data-factory/connector-sql-server.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@candidson Thank you for the valuable feedback,we are investigating the issue.

@candidson There appears to be a table missing to allow the stored procedure to function correctly: Azure Data Factory – using a Stored Procedure as a destination (link).
Additionally, can you please identify a document that you are using as a basis for this functionality, as this channel is intended for document feedback. I did find this document: Copy data to and from SQL Server by using Azure Data Factory (link), and will use this as a basis for assigning ownership of the issue.

@Mike-Ubezzi-MSFT Thanks for your feedback. Actually, I do have 6 activities exactly built the same way. They all work except for one.
I did follow the different links you sent - originally...
The stored procedure works fine when tested directly on azure sql db.
Furthermore, the ADF execution doesn't fail... It doesn't even pass validation

@candidson Thank you for the additional feedback, I am assigning to the ADF team for further investigation.

@candidson Additionally, have you been successful with a merge operation in ADF before?

@Mike-Ubezzi-MSFT yes i have. As i was saying, i have several activities built exactly the same way (all with merge tasks). And only two aren't working.

Having exactly the same issue here! The scenario described by @candidson reflects exactly my environment and the message I'm getting in the validation is the same.
I have around 18 other pipelines configured like this and they all work fine....

This is in fact a bug. I'm also having this problem. All my pipelines worked perfectly. Now, I assign a procedure, and the sink can have a dynamic or static table assigned, but this still happens. I have over 30 pipelines working with the same configuration, but now when I try to create new ones (or change the sink dataset of those already created) this message appears.
I've noticed that this is happening with databases that have more than 1 schema.
My older datasets, created when my database didn't have secondary schemas, work. But the new ones, where the Sink configuration asks me to specify a schema, aren't working.
This is critical: I can't deploy my factory to production because of this.

Having exactly the same issue here! The scenario described by @candidson reflects exactly my environment and the message I'm getting in the validation is the same.
I have around 18 other pipelines configured like this and they all work fine....

Did you add another schema on our database? I have a secondary one for logging and temporary data. I'm considering dropping it, create the dataset again just to retry. This is screwing up my deadlines here. Like, it's not like data factory is cheap!

@candidson There appears to be a table missing to allow the stored procedure to function correctly: Azure Data Factory – using a Stored Procedure as a destination (link).
Additionally, can you please identify a document that you are using as a basis for this functionality, as this channel is intended for document feedback. I did find this document: Copy data to and from SQL Server by using Azure Data Factory (link), and will use this as a basis for assigning ownership of the issue.

Mike, this document is for ADFv1. I'm using ADFv2, and like I said, I have 30 pipelines working.

Having exactly the same issue here! The scenario described by @candidson reflects exactly my environment and the message I'm getting in the validation is the same.
I have around 18 other pipelines configured like this and they all work fine....

Did you add another schema on our database? I have a secondary one for logging and temporary data. I'm considering dropping it, create the dataset again just to retry. This is screwing up my deadlines here. Like, it's not like data factory is cheap!

Indeed I did add a new Schema today. I had more than one before, but I created a new one just before trying to set this new pipeline that is not coming through.
Actually the same scenario is giving a different error now. in the "Mapping" tab, if I try to "Import Schemas" it will throw an error:
"Failed to import sink schema. 'Type=System.Data.SqlClient.SqlException,Message=Column, parameter, or variable #1: Cannot find data type #TypeTableName#.  Must declare the table variable "@a".  Parameter or variable '@a' has an invalid data type"
It is really messing up my deadlines too.... how great!

Having exactly the same issue here! The scenario described by @candidson reflects exactly my environment and the message I'm getting in the validation is the same.
I have around 18 other pipelines configured like this and they all work fine....

Did you add another schema on our database? I have a secondary one for logging and temporary data. I'm considering dropping it, create the dataset again just to retry. This is screwing up my deadlines here. Like, it's not like data factory is cheap!

Indeed I did add a new Schema today. I had more than one before, but I created a new one just before trying to set this new pipeline that is not coming through.
Actually the same scenario is giving a different error now. in the "Mapping" tab, if I try to "Import Schemas" it will throw an error:
"Failed to import sink schema. 'Type=System.Data.SqlClient.SqlException,Message=Column, parameter, or variable #1: Cannot find data type #TypeTableName#. Must declare the table variable "@A". Parameter or variable '@A' has an invalid data type"
It is really messing up my deadlines too.... how great!

For me, activities which were working fine, started acting out and do not work anymore.. throwing out the validation error

Facing the exact same validation issue mentioned in this thread. Please help.

We just identified this is a bug of ADF UI. It's caused by a design change of ADF which splits tableName into schema and table, leading the validation failed to pass. Currently we are working on the fix, to work around it, you can append &feature.splitTableName=false to the end of the ADF web's url to temporarily disable this feature and create a new sql dataset. Or you can also click the code button on the error dataset, and manually add a fake tableName properties like "tableName": "fake", beside the schema and table property to bypass the validation.

We just identified this is a bug of ADF UI. It's caused by a design change of ADF which splits tableName into schema and table, leading the validation failed to pass. Currently we are working on the fix, to work around it, you can append &feature.splitTableName=false to the end of the ADF web's url to temporarily disable this feature and create a new sql dataset. Or you can also click the code button on the error dataset, and manually add a fake tableName properties like "tableName": "fake", beside the schema and table property to bypass the validation.

The workaround with the URL parameter works for me! Thanks!

We just identified this is a bug of ADF UI. It's caused by a design change of ADF which splits tableName into schema and table, leading the validation failed to pass. Currently we are working on the fix, to work around it, you can append &feature.splitTableName=false to the end of the ADF web's url to temporarily disable this feature and create a new sql dataset. Or you can also click the code button on the error dataset, and manually add a fake tableName properties like "tableName": "fake", beside the schema and table property to bypass the validation.

The workaround with the URL parameter works for me! Thanks! Please let us know once the problem is fixed. thanks!

The fix has been deployed. Refresh the webpage and the issue should be gone. Thanks.

The fix has been deployed. Refresh the webpage and the issue should be gone. Thanks.

Awesome! Thanks a lot!

We just identified this is a bug of ADF UI. It's caused by a design change of ADF which splits tableName into schema and table, leading the validation failed to pass. Currently we are working on the fix, to work around it, you can append &feature.splitTableName=false to the end of the ADF web's url to temporarily disable this feature and create a new sql dataset. Or you can also click the code button on the error dataset, and manually add a fake tableName properties like "tableName": "fake", beside the schema and table property to bypass the validation.

The workaround with the URL parameter works for me! Thanks! Please let us know once the problem is fixed. thanks!

The fix has been deployed. Refresh the webpage and the issue should be gone. Thanks.

All good here now! Thanks!

Thanks guys for the fast fix... Closing this thread now


Suggestion: merge PR by PR number

hub pr checkout accepts a PR number, whereas hub merge accepts a URL.
Could hub merge also accept a number?

This could be useful. My only concern is that this would make it impossible to merge legit branches that only consists of digits because hub won't know whether git merge 123 means PR #123 or a branch that might legitimately exist called "123". How about the idea proposed in https://github.com/github/hub/issues/1483#issuecomment-396063128?

That would be great!


Implement Request and Responses in mempool storagetower::Service for txmessages from the network

Motivation

Specifications

Designs

Related Work

Is this ticket about sending tx messages to peers when Zebra adds a new transaction to its mempool?
Or is it about handling received tx messages from peers?
(The verify part of the download and verify stream.)
https://developer.bitcoin.org/reference/p2p_networking.html#tx

Closed by PR #2651


[ascon] Fix AscentLint warning

The Ascon hardware block is very much work in progress. Unfortunately, lint warnings and errors inside Ascon cause the whole AscentLint CI job to fail.

Thx @vogelpi for fixing this. I was not aware, that an error or warning inside Ascon causes the whole AscentLint CI to fail...
Should we disable ascnentlint for now, or do we/I need to pay more attention an ascentlint?
I think it should be more relaxed in the future to commit error/warning free code, as there will be less parts that depend on each other.

Thx @vogelpi for fixing this. I was not aware, that an error or warning inside Ascon causes the whole AscentLint CI to fail...
Should we disable ascnentlint for now, or do we/I need to pay more attention an ascentlint? I think it should be more relaxed in the future to commit error/warning free code, as there will be less parts that depend on each other.

I think we should aim to keep it enabled for Ascon and keep Ascon lint clean right from the beginning.

Thanks @vogelpi !


Implement better DLL injection methods

We will use this issue for a discussion on implementing different injection methods for our libraries.

So if I understand correctly, we want a function in RemoteInjector.cs that starts a specified process and then manually maps a specified DLL into the process?

We want to implement a function to load a DLL into the process.
There is an interface for a Module Injector which is used to call the actual injection method.
https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.BinaryInjection/Loader/IModuleInjector.cs
The Module Injector Implementation here: https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.BinaryInjection/Loader/ModuleInjector.cs
The Assembly loading code is accessed through the AssemblyLoader class here:
https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.BinaryInjection/Loader/AssemblyLoader.cs#L21
Here is an example of a module being injected using it's filepath:
https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.BinaryInjection/RemoteInjection/RemoteInjector.cs#L206
The actual process we start is wrapped in a ManagedProcess handle: https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.Memory/ManagedProcess.cs#L7
The current injection method is implemented here:
https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.Memory/Processes/ProcessManager.Windows.cs#L20
So I think that would be the best place to implement the actual injection code, because then we don't have to change anything as far as function calls.
The .NET C# Process class is wrapped and passed to the ProcessManager here in the RemoteInjector: https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.BinaryInjection/RemoteInjection/RemoteInjector.cs#L34
You can modify the process creation code here if necessary: https://github.com/unknownv2/CoreHook/blob/master/src/CoreHook.BinaryInjection/RemoteInjection/RemoteInjector.cs#L85

If that's confusing, I understand as well... I really appreciate the help so I can answer any questions and I am very open to suggestions on how to better document everything or maybe how to better structure the code and projects to make it easier to understand.
Thank you again!

Once we get this DLL injection working I could definitely help you structure the codebase of this project better as well as help you with the documentation of the functions.


[information_schema]MATERIALIZED VIEW  information error

Steps to reproduce the behavior (Required)

CREATE TABLE '...'

CREATE TABLE `duplicate_table_with_null_partition` (
                                        `k1` date,
                                        `k2` datetime,
                                        `k3` char(20),
                                        `k4` varchar(20),
                                        `k5` boolean,
                                        `k6` tinyint,
                                        `k7` smallint,
                                        `k8` int,
                                        `k9` bigint,
                                        `k10` largeint,
                                        `k11` float,
                                        `k12` double,
                                        `k13` decimal(27,9)
                                    ) ENGINE=OLAP
                                    DUPLICATE KEY(`k1`, `k2`, `k3`, `k4`, `k5`)
                                    COMMENT "OLAP"
                                    PARTITION BY RANGE (k1) (
                                        START ("2020-05-01") END ("2021-10-04") EVERY (INTERVAL 15 day)
                                    )
                                    DISTRIBUTED BY HASH(`k1`, `k2`, `k3`, `k4`, `k5`) BUCKETS 3
                                    PROPERTIES (
                                        "replication_num" = "3",
                                        "storage_format" = "v2"
                                    );
curl --location-trusted -u root: -T /home/disk4/lvchenyang/codebak/StarRocksTest/lib/../common/data/basic_types_data_without_null -XPUT -H label:view_d668c55e_61ce_11ed_b633_00163e0e550b http://ip:port/api/test_info_tables_conf_d661acec_61ce_11ed_b633_00163e0e550b/duplicate_table_with_null_partition/_stream_load
CREATE MATERIALIZED VIEW mv1
                                             DISTRIBUTED BY HASH(k1) BUCKETS 10
                                             REFRESH ASYNC
                                             AS SELECT k1, k6, k7, k8, k9, k10
                                             FROM duplicate_table_with_null_partition;

SELECT  '....'

select * from information_schema.tables_config where TABLE_SCHEMA='test_info_tables_conf_d661acec_61ce_11ed_b633_00163e0e550b' and TABLE_NAME='mv1';

Expected behavior (Required)
PRIMARY_KEY=def
PARTITION_KEY=def
PROPERTIES should be equal to reality
Real behavior (Required)

StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

+----------------------+
| current_version()    |
+----------------------+
| MAIN-RELEASE e21cd48 |
+----------------------+

PRIMARY_KEY/PARTITION_KEY/PROPERTIES


⚠ SceneTime has degraded performance

In a086716, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 3754 ms

Resolved: SceneTime performance has improved in bfa92f2 after 6 minutes.


Missing optional dependency Jinja2 when publishing

Since version 0.1.3 has been released, I've encountered packing errors when publishing. After investigation, it seemed to be happen since the release of v0.1.3. The project did not pin the version of viktor_table_view, therefore finding the culprit not difficult.
Here is the error:
  ERROR   packing completed with errors: Could not generate manifest:
           Traceback (most recent call last):
            File "/tmp/wd/dist/site-packages-layer/usr/local/lib/python3.12/site-packages/pandas/compat/_optional.py", line 141, in import_optional_dependency
              module = importlib.import_module(name)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
            File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
              return _bootstrap._gcd_import(name[level:], package, level)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
            File "<frozen importlib._bootstrap>", line 1381, in _gcd_import
            File "<frozen importlib._bootstrap>", line 1354, in _find_and_load
            File "<frozen importlib._bootstrap>", line 1318, in _find_and_load_unlocked
          ModuleNotFoundError: No module named 'jinja2'

          During handling of the above exception, another exception occurred:

          Traceback (most recent call last):
            File "viktor_connector/connector.pyx", line 693, in connector.ManifestHandler.run
            File "/tmp/wd/dist/app-code-layer/home/user/src/app/__init__.py", line 7, in <module>
              from app.app_analysis.controller import AppAnalysisController as AppAnalysis
            File "/tmp/wd/dist/app-code-layer/home/user/src/app/app_analysis/controller.py", line 12, in <module>
              from viktor_table_view import TableView, TableResult
            File "/tmp/wd/dist/site-packages-layer/usr/local/lib/python3.12/site-packages/viktor_table_view/__init__.py", line 2, in <module>
              from viktor_table_view.table_view import TableResult
            File "/tmp/wd/dist/site-packages-layer/usr/local/lib/python3.12/site-packages/viktor_table_view/table_view.py", line 11, in <module>
              from pandas.io.formats.style import Styler
            File "/tmp/wd/dist/site-packages-layer/usr/local/lib/python3.12/site-packages/pandas/io/formats/style.py", line 56, in <module>
              jinja2 = import_optional_dependency("jinja2", extra="DataFrame.style requires jinja2.")
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
            File "/tmp/wd/dist/site-packages-layer/usr/local/lib/python3.12/site-packages/pandas/compat/_optional.py", line 144, in import_optional_dependency
              raise ImportError(msg)
          ImportError: Missing optional dependency 'Jinja2'. DataFrame.style requires jinja2. Use pip or conda to install Jinja2.

          2024-04-08 09:54:57.808 ERROR   : Missing optional dependency 'Jinja2'. DataFrame.style requires jinja2. Use pip or conda to install Jinja2.
          error building image: error building stage: failed to execute command: waiting for process to exit: exit status 1

@MSlootwegVIKTOR thanks for the report! Could you list the version of other packages?

viktor==14.7.1
requests==2.31.0
pandas==2.2.1
plotly==5.20.0
viktor-table-view==0.1.2
python-dotenv==1.0.1


Shipping , get Account sends empty

Hello I tried to call shipping getAccount but got empty object.
SellingPartnerAPI.callAPI({
          operation: "getAccount",
          //@ts-ignore
          endpoint: "shipping",
        }).then((res) => {
          console.log(res);
        });

@Javadebi I think this will also be caused by the issue discussed in #72.


Issues building Android application on macOS: dependencies unclear

Summary
During make build-android; I receive a message telling me I need to install the JDK.
Environment Information

Device Name: MacBook
OS Version: macOS Mojave

Steps to reproduce
Install dependencies (Android Studio + SDKs) as advised in tutorial on a vanilla macOS install
Expected behavior
I am told to I need to install the JDK. I do not have java or javac in my path so it is clear it is not installed. I don't know which version of the JDK is appropriate. Will JDK11 suffice?
Thanks
Sam

JDK 8 will work 100%, can't give any guarantees about newer versions though. Maybe that should be added to the documentation to avoid confusion in the future.

When you run the command there are some files that get moved from `'/Users/samnazarko/Code/Git/mattermost-mobile/android/app/src/main/java/com/mattermost/rnbeta/`` to a folder with your package name, make sure those files are in the above folder before running the build command. you can make sure that is the case by resetting the git branch

git status shows that the files are untouched:
On branch master
Your branch is up to date with 'origin/master'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   ios/Mattermost/Mattermost.entitlements
	modified:   ios/MattermostShare/MattermostShare.entitlements

Untracked files:
  (use "git add <file>..." to include in what will be committed)

	fastlane/env
	osmc_env.sh

 ls -l android/app/src/main/java/com/mattermost/rnbeta/
total 160
-rw-r--r--  1 samnazarko  staff  14985 19 Nov 13:45 CustomPushNotification.java
-rw-r--r--  1 samnazarko  staff   4902 19 Nov 13:45 InitializationModule.java
-rw-r--r--  1 samnazarko  staff   1077 19 Nov 13:45 InitializationPackage.java
-rw-r--r--  1 samnazarko  staff    941 19 Nov 13:45 MainActivity.java
-rw-r--r--  1 samnazarko  staff   4379 19 Nov 13:45 MainApplication.java
-rw-r--r--  1 samnazarko  staff   1921 19 Nov 13:45 MattermostManagedModule.java
-rw-r--r--  1 samnazarko  staff   1040 19 Nov 13:45 MattermostPackage.java
-rw-r--r--  1 samnazarko  staff    952 19 Nov 13:45 NotificationDismissService.java
-rw-r--r--  1 samnazarko  staff   1801 19 Nov 13:45 NotificationPreferences.java
-rw-r--r--  1 samnazarko  staff   4143 19 Nov 13:45 NotificationPreferencesModule.java
-rw-r--r--  1 samnazarko  staff   3293 19 Nov 13:45 NotificationReplyService.java
-rw-r--r--  1 samnazarko  staff   8618 19 Nov 13:45 NotificationsLifecycleFacade.java
```

@samnazarko perhaps you could join us in our community server and there I could help you a bit more, even others can jump in to help you. https://community.mattermost.com best place to talk about this would be the  ~Native Mobile apps channel

@samnazarko is it ok to close this issue now?

Yes. Thank you so much for your help!


Configure LBTrustStore resource and add example

Description of your changes
Configures LBTrustStore.elbv2 resource and add the example
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] ~Added backport release-x.y labels to auto-backport this PR if necessary.~

How has this code been tested
Via uptest: https://github.com/crossplane-contrib/provider-upjet-aws/actions/runs/10586728565

/test-examples="examples/elbv2/v1beta1/lbtruststore.yaml"

/test-examples="examples/elbv2/v1beta1/lbtruststore.yaml"


Front Matter not calling inside code blocks

I am trying to create a page variable in the front matter of a tutorial and pass it inside of a code block, but it is not rendering. The same variable is working outside of the code block as a normal paragraph element.
My guess is that it has to do with the brackets being rendered differently inside of a <pre> element.
Here is my front matter:

Here is what it is rendering:

Moved here: https://github.com/electricjs/electric/issues/42


Don't attempt to remove orphans

Yes, the only possibility I could find is duplicate items + removal with the iterator, which I fixed here.


Receive an Import Error using this libraries on a notebook

ImportError: cannot import name 'cached_property' from 'functools'
This is error I receive. I am using python 3.7.13 on a local PC on a notebook.
The error is causing the cache_properties was introduced in python 3.8.

Thank you for the report. I'll fix it today.

@Rami-Ismael , I fixed the issue. Could you please confirm that the current git works for you?
pip install git+https://github.com/xl0/lovely-tensors

Thank you so much. It looks like it was expected. Furthermore, thank you so much for making a great library. I first heard about it in the Yannic Kilcher discord. I have always wanted to use libraries for a project.   I want to express gratitude for fixing my issues

:two_hearts:


using firebase_core: D8: Program type already present: android.support.v4.media.MediaBrowserCompat$CustomActionCallback

adding to this
error is only there when using both
image_picker:
firebase_core:
both works well separately.

im having exactly the same, image_picker + firebase_core

solved by downgrading to image_picker: ^0.4.12 as per #27194

Duplicate of #27194

For anyone still experiencing it even with all those fixes applied:
I did fix it by updating the Gradle version in <your-app>/android/gradle/wrapper/gradle-wrapper.properties from distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-all.zip to distributionUrl=https\://services.gradle.org/distributions/gradle-5.4.1-all.zip
After that all builds run smoothly


Bugfix/nls fix translation installation, closes #565 #564 #556

There are 2 commits here that address some recent issues:

a146962 addresses a minor issue with some syntax errors in po/pt_BR in the comments at the top of the file closing #564
607fc3d adds AM_INIT_AUTOMAKE to configure.ac fixing a bug where @mkdir_p@ was undefined in po/Makefile causing the failure of .mo files to be installed addressing #565 and #556

I tested this by trying to upgrade the autotools but that failed for me, so I researched a little and found that AM_INIT_AUTOMAKE absence may have been the cause. I duely tried that and this fix works with automake-1.16.2 and automake-1.16.5, both with gettext-0.21.
Thanks @joewing for your consideration of these commits.

@joewing you may also want to consider this - https://github.com/joewing/jwm/pull/432#issuecomment-1126605931
Especially if you decide if this PR isn't warranted.

Thanks! I don't see any reason not to merge these.
As far as cmake is concerned... that branch is a bit out of date and I don't think handles everything. autotools are somewhat confusing, but they're everywhere and JWM already uses them so it might not be worth the effort to convert.

Thanks Joe :smile:


Implement Bls.key_validate

We're missing a KeyValidate implementation in the Bls module.

hey can i work on this?

assigned!

hey after running the test and make fmt a lot of files got formatted to the configured style namely
       modified:   lib/bls.ex         modified:   lib/lambda_ethereum_consensus/execution/rpc.ex         modified:   lib/lambda_ethereum_consensus/state_transition/epoch_processing.ex         modified:   lib/lambda_ethereum_consensus/state_transition/shuffling.ex         modified:   lib/lambda_ethereum_consensus/store/lru_cache.ex         modified:   lib/utils/bit_list.ex         modified:   lib/utils/diff.ex         modified:   native/bls_nif/src/lib.rs         modified:   test/fixtures/block.ex         modified:   test/spec/runners/bls.ex         modified:   test/spec/runners/fork_choice.ex         modified:   test/spec/runners/helpers/process_blocks.ex         modified:   test/spec/runners/operations.ex         modified:   test/spec/runners/ssz_static.ex         modified:   test/spec/runners/sync.ex         modified:   test/unit/req_resp_test.exs         modified:   test/unit/snappy_test.exs
Am I supposed to discard those changes or commit them ?

That's weird 🤔. Can you please share some of those changes? Also, which elixir version are you using? (i.e. elixir --version)
I just tried it locally on main and it doesn't modify any files, so I guess it's some kind of misconfiguration.

I am working with elixir 1.16.2.
One of the changes that happened is

(Sorry , I was unable to take a ss from my laptop)

Since they seem to be style changes, I say we discard them for now to simplify the review process. We can always add them later in a standalone PR 😄


how can i generate apikey

hi.
how can i generate apikey for
var auth = new FirebaseAuthProvider(new FirebaseConfig("api_key"));

This is not a issue.
If you need to find the API key go to:
Project Settings -> General -> look for "web API key".

actually i thought that maybe create doc for this. closing


fix: term vs trunc

Note
This is quite confusing so please check this carefully and make sure I haven't mixed up where to use each one!
What?
Mostly explained in #951. We are mixing up termination and truncation. This is especially prevalent in recurrent systems, but all systems have this issue.
Now very explicit about termination vs truncation, removed all mentions of done.
Done attribute remove from RnnLearnerState and replaced with truncated in the PpoTransition.
How?
Explicitly name variables termination and trunctation instead of done

This is an old attempt at fixing this most recent can be seen by following issue #951


Datapoints missing (charging level, kilometer, reach)

Installed version 2.8.0 after trying all the stable build with no success.
Added my login credentials, connection to BMW established, but it seems not all data points are visible, that should be visible.
For example there is no charge level, total kilometers, reach.

I've tried and installed the adapter a few more times, and  but I can't get it to work. Also I tried adding another instance, without success. Data is still missing.
Is there any way to do this? Is this a known error? Can this be caused by quota limitations as well?

which information are under state?

@HochdruckHummer Might depend on your car. I have a 2015 520d with NBT system. This doesn't support what they call LSC (Last State Call). As I understand it information like kilometer/mileage is send to the ConnectedDrive servers through that LSC. So if the car doesn't support transmission to the ConnectedDrive server you'll have no data in ioBroker either. As I recall they stopped some of the services when 3G was taken offline in Germany. The "old" cars only supported up to 3G and use 2G now :-(


Update test_as_bytes with clippy feedback

Cargo clippy suggests to simplify the code:
called `iter().copied().collect()` on a slice to create a `Vec`. Calling `to_vec()` is both faster and more readable
for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#iter_cloned_collect
`#[warn(clippy::iter_cloned_collect)]` on by default

If iter().copied().collect() is still intended to be used, I could add a comment on why it is preferred in this case and add an ignore for the clippy warning.

Thanks!


feat: drop useless select and simplify code

In the a | b code in the following, the select doesn't matter from what I can tell, and I figured out than rust Simd has saturating_sub which I think is cleaner than the compare select code.
Note: Removing the need for needle_cased_mask and capital_mask here, means that one could choose to keep track of an exact equal (a ≠ A, but a = a, A = A) and an inexact equal (a = A) instead of the way code with capital letters currently work.
Note 2: According to https://godbolt.org/z/8vb7vx39K the saturating_sub code generates the same assembly as the current code, so this is purely a matter of preference. (Whereas I think the code with select on or is truly not needed.)

I think that select does matter since this would turn an "@" into a backtick: String.fromCharCode("@".charCodeAt(0) | 0x20)

The saturating_sub looks great, thanks for finding that!

I think that select does matter since this would turn an "@" into a backtick: String.fromCharCode("@".charCodeAt(0) | 0x20) (JS)

Ah, my bad, I didn't think about special characters, and thought it would be nicer to have an exact match to score based on. I have removed that change.


Feature/add homepage

Initiated React project using Vite, configured project name, and removed default Vite code.
Integrated React Material UI into the project and configured the library.
Created a home page with a header, welcome text, and three buttons for navigation.
Incorporated an About Us section with relevant content and an image.
Introduced a Services section with details about the project's offerings.
Implemented a Contact Us section with a form and an associated image.
Implemented functionality for the header buttons, directing users to their respective sections.
Updated the project's name and read me with instructions on how to set up the project.

Good implementation of a website.


Certain Youtube links not working

https://hastebin.com/rabodevava.sql
Didn't have this issue a while ago, any ideas?

Update your dependencies by running python3.5 -m pip install --upgrade -r requirements.txt

I did this, and then did !restart. didnt work for me, still getting the error.
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/youtube.py", line 1005, in _decrypt_signature
if i - prev == step:
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/youtube.py", line 919, in _extract_signature_function
}
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/common.py", line 517, in _download_webpage
if block_msg:
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/common.py", line 424, in _download_webpage_handle
if fatal:
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/common.py", line 404, in _request_webpage
self.to_screen('%s' % (note,))
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/YoutubeDL.py", line 2000, in urlopen
for lang, formats in subtitles.items()]))
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/utils.py", line 513, in sanitized_Request
"""Sanitizes and normalizes path on Windows"""
File "/usr/lib/python3.5/urllib/request.py", line 268, in init
self.full_url = url
File "/usr/lib/python3.5/urllib/request.py", line 294, in full_url
self._parse()
File "/usr/lib/python3.5/urllib/request.py", line 323, in _parse
raise ValueError("unknown url type: %r" % self.full_url)
ValueError: unknown url type: '/yts/jsbin/player-en_US-vfl8LqiZp/base.js'
(caused by ValueError("unknown url type: '/yts/jsbin/player-en_US-vfl8LqiZp/base.js'",))

!restart may not be enough. You will likely need to hard restart your bot entirely. If it doesn't work, run python3.5 -m pip install -U youtube-dl --force-reinstall first, then hard restart.