What do I need to override to remove the add button from the grid view

I just want the quick search and refresh visible.

If you're using Saltarelle, in your MaterialCodeGrid.cs in the script project:
protected override List<ToolButton> GetButtons()         {             var buttons = base.GetButtons();             buttons.RemoveAt(0);             return buttons;         }

(I made an assumption your table was called MaterialCode)

And this question asked 10th time, even though there is a howto for it :/


Add build status image

Use shields.io to show the Advertising Library build status in its README

This is how it looks btw:

Is UID code available for eddystone? I was able to run url , but not uid or tlm

@kartikguptadce it is not. I might be able to add support to it by next week.

Thats great! Can you help me what all enhancements can be made in Arduino implementation side of Eddystone to enhance the protocol. Any suggestions on top of the head?
I know on the app side it could be use case based.

What do you mean by Arduino implementation? What protocol do you want to enhance?

I meant, I wish to enhance the the implementation available at https://developer.mbed.org/users/roywant/code/mbed_EddystoneURL_Beacon/

Ah. I think we are talking about two different things. I was talking about the Eddystone Advertising Library. You seem to be talking about the Embed Implementation of Eddystone. I can't help you with the Embed implementation.

Thanks for prompt response, Yes, I got that, Any finalized dates with  Embed Implementation of Eddysone of UID/TLM ?

There is an mbed version that broadcasts all three packet types but it's a bit primitive. We're working with ARM to get a better version, should be posting something in a week or so.

Thanks Scott, That version worked. I am now setup to enhance Eddystone code.


RC test: Graphing Quadratics 1.0.0-rc.3

@KatieWoe, @ariel-phet, @amanda-phet, Graphing Quadratics 1.0.0-rc.3 is ready for RC testing.
This RC is for brand=phet only (no phet-io, no a11y), and will not be shared with a client.
Please document issues in https://github.com/phetsims/graphing-quadratics/issues and link to this issue.
Assigning @ariel-phet for prioritization.

General RC Test

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test all query parameters on all platforms. (See QA Book for a list of query parameters.)
Download HTML on Chrome and iOS.
Make sure the iFrame version of the simulation is working as intended on all platforms.
Make sure the XHTML version of the simulation is working as intended on all platforms.
Complete the test matrix.
Don't forget to make sure the sim works with Legends of Learning.
Check this LoL spreadsheet and notify AR if it not there.
If this is rc.1 please do a memory test.

Issues to Verify

[x] Improve drag behavior of "Point on Parabola"
[x] Overlap of eye button and "point on parabola" coordinates
[x] pointerAreas for eye button and "Point on Parabola" overlap
[x] width of the "Quadratic Terms" accordion box

These issues should have the "status:fixed-pending-testing" label. Check these issues off and close them if they are fixed. Otherwise, post a comment in the issue saying that it wasn't fixed.

Link(s)

Simulation
iFrame
XHTML
Test Matrix
Legends of Learning Harness

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

@KatieWoe As with the previous RC test, the issues that were fixed (identified in "Issues to Verify" above) were all very isolated changes. So I'm confident that no new problems have crept in as the result of these changes, and I'd be comfortable just doing a "spot check" of those specific issues (vs a full test matrix on all platforms).   The list of platforms that you identified in https://github.com/phetsims/QA/issues/233#issuecomment-444528698 should be sufficient.

[ ] Win 10 chrome Full
[ ] Win 10 firefox
[ ] Win 10 edge Full

[ ] Mac 10.14 chrome
[ ] Mac 10.14 firefox Full
[ ] Mac 10.14 safari Full

[ ] iOS 12 Full
[ ] iOS 9 Full

[ ] Chrome OS

[ ] Win 7 IE Full
[ ] Win 7 chrome
[ ] Mac 10.11 safari
[ ] Mac 10.11 firefox

iOS 12 still unavailable.  Tested iOS 11 instead.

QA is done

Thanks @KatieWoe and QA team!

RC testing is completed, Graphing Quadratics 1.0.0 is published, closing.


Add support for running tests in parallel

Fixes #2

The way rayon splits arrays you get results in a very mangled order sadly. It's slightly faster

@LukasKalbertodt This is ready for review, when you find a moment :)

Wait a second... I do have CI! Why didn't it run for this PR?! :smile:
It even broke (minor style). But don't worry, I will fix it on master soon(TM).

Released as 0.3.0!

Yay, thanks!


[influxdb-cxx] Update to latest commit on 2022-08-20

Describe the pull request

What does your PR fix?
Fixes #21590, since the upstream has not released a release version for more than a year, so update influxdb-cxx to latest commit. Remove the old patch, all feature test pass on x64-windows.

Thanks!


The C Compiler identification is unknown

I followed the tutorials and successfully installed ROS Melodic on Windows 10. I also created the shortcut terminal as per the tutorials. But when i created a workspace and ran catkin_make, I got the following error:-
Base path: C:\catkin_ws
Source space: C:\catkin_ws\src
Build space: C:\catkin_ws\build
Devel space: C:\catkin_ws\devel
Install space: C:\catkin_ws\install

Running command: "cmake C:\catkin_ws\src -DCMAKE_BUILD_TYPE=RelWithDebInfo -DCATKIN_DEVEL_PREFIX=C:\catkin_ws\devel -DCMAKE_INSTALL_PREFIX=C:\catkin_ws\install -G NMake Makefiles" in "C:\catkin_ws\build"

-- The C compiler identification is unknown
-- The CXX compiler identification is unknown
CMake Error at CMakeLists.txt:6 (project):
The CMAKE_C_COMPILER:
cl

is not a full path and was not found in the PATH.
To use the NMake generator with Visual C++, cmake must be run from a shell
that can use the compiler cl from the command line.  This environment is
unable to invoke the cl compiler.  To fix this problem, run cmake from the
Visual Studio Command Prompt (vcvarsall.bat).
Tell CMake where to find the compiler by setting either the environment
variable "CC" or the CMake cache entry CMAKE_C_COMPILER to the full path to
the compiler, or to the compiler name if it is in the PATH.
CMake Error at CMakeLists.txt:6 (project):
The CMAKE_CXX_COMPILER:
cl

is not a full path and was not found in the PATH.
To use the NMake generator with Visual C++, cmake must be run from a shell
that can use the compiler cl from the command line.  This environment is
unable to invoke the cl compiler.  To fix this problem, run cmake from the
Visual Studio Command Prompt (vcvarsall.bat).
Tell CMake where to find the compiler by setting either the environment
variable "CXX" or the CMake cache entry CMAKE_CXX_COMPILER to the full path
to the compiler, or to the compiler name if it is in the PATH.
-- Configuring incomplete, errors occurred!
See also "C:/catkin_ws/build/CMakeFiles/CMakeOutput.log".
See also "C:/catkin_ws/build/CMakeFiles/CMakeError.log".
Invoking "cmake" failed
How can I solve this issue?

Please launch from a fully sourced developer command prompt: http://wiki.ros.org/Installation/Windows


Add output_file variable when using layouts

The output_file is relevant when there's a need to create a link to the current file from the layout.

@frm since you don't have a lot of time and this project doesn't have a lot of visibility, I think I'll just start merging these ok?

Can you give me an example where this is needed?

Yap, for instance, you have a layout for every blog post and want to create a full link for people to copy

@frm bump


Addon installation fails on docker Hassio

Not possible to install through addon store. Gives following error message on HASS.io running in docker on Synology NAS:
"Failed to install addon
The command '/bin/ash -o pipefail -c apk add --no-cache npm libusb-dev bluez-dev linux-headers eudev-dev build-base python git' returned a non-zero code: 61"
From Supervisor log:
"20-11-22 09:59:17 INFO (SyncWorker_2) [supervisor.docker.addon] Starting build for 5069d20e/amd64-addon-ruuvitag-discovery:0.1.13
20-11-22 09:59:25 ERROR (SyncWorker_2) [supervisor.docker.addon] Can't build 5069d20e/amd64-addon-ruuvitag-discovery:0.1.13: The command '/bin/ash -o pipefail -c apk add --no-cache npm libusb-dev bluez-dev linux-headers eudev-dev build-base python git' returned a non-zero code: 61
20-11-22 09:59:25 ERROR (SyncWorker_2) [supervisor.docker.addon] Build log:
Step 1/23 : ARG BUILD_FROM
Step 2/23 : FROM ${BUILD_FROM}
---> e3299fbb2f75
Step 3/23 : ENV LANG C.UTF-8
---> Using cache
---> d3b551c3dcb1
Step 4/23 : RUN apk add --no-cache npm libusb-dev bluez-dev linux-headers eudev-dev build-base python git
---> Running in 61938c4db919
fetch http://dl-cdn.alpinelinux.org/alpine/v3.11/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.11/community/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/community/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/testing/x86_64/APKINDEX.tar.gz
ERROR: unsatisfiable constraints:
musl-1.1.24-r2:
breaks: musl-dev-1.1.24-r3[musl=1.1.24-r3]
satisfies: world[musl=1.1.24-r2]
apk-tools-2.10.4-r3[so:libc.musl-x86_64.so.1]
kmod-26-r0[so:libc.musl-x86_64.so.1]
pkgconf-1.6.3-r0[so:libc.musl-x86_64.so.1]
libblkid-2.34-r1[so:libc.musl-x86_64.so.1]
gcc-9.3.0-r0[so:libc.musl-x86_64.so.1]
g++-9.3.0-r0[so:libc.musl-x86_64.so.1]
tzdata-2019c-r0[so:libc.musl-x86_64.so.1]
bash-5.0.11-r1[so:libc.musl-x86_64.so.1]
ca-certificates-20191127-r1[so:libc.musl-x86_64.so.1]
eudev-3.2.9-r1[so:libc.musl-x86_64.so.1]
libusb-1.0.23-r0[so:libc.musl-x86_64.so.1]
busybox-1.31.1-r9[so:libc.musl-x86_64.so.1]
binutils-2.33.1-r0[so:libc.musl-x86_64.so.1]
sqlite-libs-3.30.1-r2[so:libc.musl-x86_64.so.1]
curl-7.67.0-r0[so:libc.musl-x86_64.so.1]
eudev-libs-3.2.9-r1[so:libc.musl-x86_64.so.1]
libuv-1.34.0-r0[so:libc.musl-x86_64.so.1]
ncurses-libs-6.1_p20200118-r2[so:libc.musl-x86_64.so.1]
expat-2.2.9-r1[so:libc.musl-x86_64.so.1]
git-2.24.3-r0[so:libc.musl-x86_64.so.1]
musl-utils-1.1.24-r2[so:libc.musl-x86_64.so.1]
nghttp2-libs-1.40.0-r0[so:libc.musl-x86_64.so.1]
bluez-libs-5.52-r1[so:libc.musl-x86_64.so.1]
nodejs-12.15.0-r1[so:libc.musl-x86_64.so.1]
gdbm-1.13-r1[so:libc.musl-x86_64.so.1]
python2-2.7.18-r0[so:libc.musl-x86_64.so.1]
libstdc++-9.3.0-r0[so:libc.musl-x86_64.so.1]
libbz2-1.0.8-r1[so:libc.musl-x86_64.so.1]
file-5.37-r1[so:libc.musl-x86_64.so.1]
isl-0.18-r0[so:libc.musl-x86_64.so.1]
libcurl-7.67.0-r0[so:libc.musl-x86_64.so.1]
mpc1-1.1.0-r1[so:libc.musl-x86_64.so.1]
libmagic-5.37-r1[so:libc.musl-x86_64.so.1]
oniguruma-6.9.4-r0[so:libc.musl-x86_64.so.1]
libffi-3.2.1-r6[so:libc.musl-x86_64.so.1]
zlib-1.2.11-r3[so:libc.musl-x86_64.so.1]
libgomp-9.3.0-r0[so:libc.musl-x86_64.so.1]
readline-8.0.1-r0[so:libc.musl-x86_64.so.1]
make-4.2.1-r2[so:libc.musl-x86_64.so.1]
libgcc-9.3.0-r0[so:libc.musl-x86_64.so.1]
libssl1.1-1.1.1d-r3[so:libc.musl-x86_64.so.1]
alpine-baselayout-3.2.0-r3[so:libc.musl-x86_64.so.1]
libcrypto1.1-1.1.1d-r3[so:libc.musl-x86_64.so.1]
libatomic-9.3.0-r0[so:libc.musl-x86_64.so.1]
xz-libs-5.2.4-r0[so:libc.musl-x86_64.so.1]
pcre2-10.34-r1[so:libc.musl-x86_64.so.1]
c-ares-1.15.0-r0[so:libc.musl-x86_64.so.1]
scanelf-1.2.4-r0[so:libc.musl-x86_64.so.1]
gmp-6.1.2-r1[so:libc.musl-x86_64.so.1]
jq-1.6-r0[so:libc.musl-x86_64.so.1]
mpfr4-4.0.2-r1[so:libc.musl-x86_64.so.1]
Removing intermediate container 61938c4db919"

By quick googling I found this: https://stackoverflow.com/questions/48892448/error-unsatisfiable-constraints-using-apk-in-dockerfile
So problem is probably that "testing" repository. It's not counted as "stable" so it causes problems. Would it be possible to not to use it?

Hi petait,
I had same problem. Solution was to install RuuviTag Discovery manually.
I followed these instructions,
First get NodeJS,: https://www.makersupplies.sg/blogs/tutorials/how-to-install-node-js-and-npm-on-the-raspberry-pi
then followed this
https://github.com/balda/ruuvitag-discovery/issues/3
Now I got it running. Just need to learn, how to use it.

Hi petait,
I had same problem. Solution was to install RuuviTag Discovery manually.
I followed these instructions,
First get NodeJS,: https://www.makersupplies.sg/blogs/tutorials/how-to-install-node-js-and-npm-on-the-raspberry-pi
then followed this
#3
Now I got it running. Just need to learn, how to use it.

Hi Nuuo90,
Unfortunately that is not a solution for me as I'm using Hass.io on Synology in Docker. Only way would be to change that repository to a stable one.

Thanks for reporting this bug. I'll try to take time this week-end to fix the build.
Home Assistant or docker image seems to have break something.

Release 0.1.14 should fix the bug ;)
(python package was removed from alpine docker image, replaced by python3)
Build is working locally, but I havent tested it in Home Assistant. @petait I hope to have good news :)

Thank you @balda ! It works like a charm now!
This add on is even better than I could have hoped for. So much thanks for it :)


Add a CHANGELOG

Good for people interested in your package.

Thanks for reminding this. Currently this project is solely for personal use, and only executable binary is built. So Changelog doesn't really matter (notes from releases page serve the same purpose).
I have plan to refactor this into a library, so that others can put this proxy server in front of arbitrary wai application. Hope at that time, this project will become more useful to others.


is it possible that there is no transfer of parameters to the script?

do you mean call extern script without parameters?  you can just leave it blank.


fix: Fix pathRewrite in k8s Proxy for WebSocket requests

Hey, I just made a Pull Request!
WebSocket requests path were not being rewritten by the proxy properly, now they do.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Having issues doing a changeset, getting this error with yarn Type Error: Cannot read properties of undefined (reading 'toUpperCase')

Hi @mclarke47 keep seeing an aws-sdk error in a couple E2E tests, is it another botched release from them? or still the same release and AWS has yet to fix it?

I'm not very knowledgable about the aws-sdk tbh, did you raise the issue in discord?

I did it in the vmware-spotify channel, should I do it in a specific one?

Hi @clareliguori , sorry pinging you directly, but just in case you have an idea.

There was an issue that seems to match what you're seeing - a fixed version v3.378.0 was released on Jul 26:
https://github.com/aws/aws-sdk-js-v3/issues/5014
I'm not sure why the tests aren't respecting the version in the lockfile though (v3.370.0) and instead seem to be pulling in the broken version v3.377.0.  In any case, re-running the tests might fix it.

The e2e's that are failing here don't use the lockfile that we have in the monorepo, as they're meant to test new installations of npx @backstage/create-app which will basically pull in all the latest deps and run typechecking etc with those. Looks like this might just need a retrigger now though if the issues has been fixed :tada: thanks @clareliguori! :pray:

All checks are green, looks like we still need an approval from a maintainer -- @Rugvip or @benjdlambert?


Fix loading indicator in night mode.

First commit is for extracting the code for checking night mode setting.
Second commit fixes the loading indicator in night mode for message edit button.
Third commit fixes the loading indicator in night mode for confirm dialog.
Fourth commit is for disabling cancel button while loading in confirm dialog.

GIFs or screenshots:

Nice, merged, thanks @sahil839!


Manual installation refers to the wrong link

In the readme, there is a manual installation option, but this link did not work:
https://github.com/FichteFoll/discordrp-mpris
but when i used the zip of the latest release:
https://github.com/FichteFoll/discordrp-mpris/archive/refs/tags/v0.3.2.zip
it worked as intended. I also wonder why systemctl --user enable discordrp-mpris.service --now does not work when I do it that way. Right now i open a terminal to run the rich presence, but I would like it to be automatic. Should I set up a cron job to do it or is there a better way?

Thanks for the heads-up, I fixed the installation commands in a1374a80dcfb33a297cda85155dc126ce84ff113.
When you use manual installation, the systemd unit file isn't copied to the proper location where systemd expects it. You will need to copy/create it yourself at ~/.config/systemd/user first. Then you can control it like any other unit file installed to your system. Note that you will most likely also need to update the executable path to /home/<your_user>/.local/bin/discordrp-mpris, unless you installed as root.


Consider relaxing the SPDX licenseInfoInFiles check

We currently do
https://github.com/oss-review-toolkit/ort/blob/4629bd7b84c1d7db037faba52f930b01089e373a/utils/spdx/src/main/kotlin/model/SpdxFile.kt#L189-L198
which results in the SPDX report not to be written out at all if a license is not an SPDX expression with non-deprecated SPDX IDs and / or LicenseRef- "exceptions". This can be problematic if e.g. a scanner emits deprecated SPDX IDs, or declared licenses contain some free text license name.
My proposal is to not fail to write the report at all in this case, but just to log an error / warning in this case. This would allow the issue to become visible to the consumers of the SPDX report, instead of forcing them to get everything right even before the report gets written.
What do @mnonnenmacher and @fviernau think here?

I would like if there was a reporter option like failOnInvalidLicense which defaults to false, then users that definitely do not want to have an SPDX file with invalid licenses could still keep the behavior to fail hard. The reason is that we currently do not have a good way to propagate issues from the reporters and any warning or error log is very likely not getting noticed.

Just to be sure we're on the same page. We're talking about strings which do not match the SPDX expression grammar, right? Like whether it should fail for any random string like e.g.  "my random string"?

My goal would be to accept any strings that from the pure grammar syntax represents a valid SPDX expression. I.e. "my random string" should not be valid, but "foo AND bar" should.
However, I'm right now quite confused that "my random string".toSpdx(ALLOW_ANY) actually passes, although I though it'd not be valid grammar syntax according to our ANTLR rules. Any idea whether that's intended @mnonnenmacher?

I find it useful to let the user running ORT to make the choice.
Therefore implementing this a configuration value (with lenient behavior as the default) would IMHO the best way to go

Just a note on the implementation: The validation happens in the SPDX model, not in the reporter. So exposing the validation level as an SPDX reporter specific option requires some refactoring of the SPDX model first to make the validation configurable.

Maybe something like https://github.com/nesk/akkurate could be used to first separate the SPDX validation from the data class constructors, and then only call the validators if configured to do so.

Maybe something like https://github.com/nesk/akkurate could be used to first separate the SPDX validation from the data class constructors, and then only call the validators if configured to do so.

Another alternative validation library is https://github.com/konform-kt/konform which is what ORT Server already uses.


Support enum and set in test generation

What problem does this PR solve?
Close #946

/run-all-tests

@birdstorm It will be awesome if you can state how do you implement it.

@birdstorm It will be awesome if you can state how do you implement it.

@zhexuany description updated.

/run-all-tests

/run-all-tests pd=v3.0.8 tikv=v3.0.8 tidb=v3.0.8

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests


errors with react-native 0.44.0

setup a clean project
{
"name": "NatLoc",
"version": "0.0.1",
"private": true,
"scripts": {
"start": "node node_modules/react-native/local-cli/cli.js start",
"test": "jest"
},
"dependencies": {
"react": "16.0.0-alpha.6",
"react-native": "0.44.0",
"react-native-location": "^0.27.0"
},
"devDependencies": {
"babel-jest": "20.0.1",
"babel-preset-react-native": "1.9.2",
"jest": "20.0.1",
"react-test-renderer": "16.0.0-alpha.6"
},
"jest": {
"preset": "react-native"
}
}
installed react-native-location
linked
and compile in xcode
gives errors (just showing a few)
../react-native/React/Base/RCTBridge.h:66:12: note: previous definition is here
@interface RCTBridge : NSObject
^
^
@property (nonatomic, copy, readonly) NSArray *moduleClasses;
^
: error: property has a previous declaration
@property (nonatomic, strong, readonly) NSURL *bundleURL;
^
error: property has a previous declaration
@property (nonatomic, strong) Class executorClass;
^
In file included from /Users/pierrejean/Dropbox/dev-app/NatLoc/node_modules/react-native-location/RNLocation.m:3:
../react-native/React/Base/RCTBridge.h:166:37: note: property declared here
@property (nonatomic, strong) Class executorClass;
^
error: property has a previous declaration
@property (nonatomic, weak, readonly) id delegate;
^
error: property has a previous declaration
@property (nonatomic, copy, readonly) NSDictionary *launchOptions;
^
error: property has a previous declaration
@property (nonatomic, readonly, getter=isLoading) BOOL loading;
^
error: property has a previous declaration
@property (nonatomic, readonly, getter=isValid) BOOL valid;
^
error: property has a previous declaration
@property (nonatomic, readonly, weak) JSContext *jsContext;
^
error: property has a previous declaration
@property (nonatomic, readonly, strong) RCTPerformanceLogger *performanceLogger;
^
^
4 warnings and 10 errors generated.
Build failed    2017-05-14, 12:50 PM

I have exactly the same error with these "dependencies": {
"react": "16.0.0-alpha.6",
"react-native": "0.44.0",
"react-native-location": "^0.27.0",
}

replace #import <React/RCTBridge.h> instead of #import "RCTBridge.h" in project.
I can build successful, but throw out 'undefined is not an Object (evaluating DeviceEventEmitter.addListener)'...
My React-native version is 0.42.3

same here, cannot read property of any method :(

any updates on this issue?


SurfaceControl setDisplayPowerMode failed

[ ] I have read the FAQ.
[ ] I have searched in existing issues.

Environment

OS: Android
scrcpy version: 2.3.1
installation method: [e.g. manual build, apt, snap, brew, Windows release...]
device model: GOOGLE-PIXEL 5
Android version: 14

Describe the bug
2023-12-13 11:11:10,868 ERROR [scrcpy] Could not invoke method
java.lang.NoSuchMethodException: android.view.SurfaceControl.getInternalDisplayToken []
at java.lang.Class.getMethod(Class.java:2937)
at java.lang.Class.getMethod(Class.java:2449)
at d.b.a.g.f.d(Unknown Source:33)
at d.b.a.g.f.c(Unknown Source:1)
at d.c.a.a.f.b(:2)
at d.c.a.a.n$a.run(:31)
2023-12-13 11:11:10,872 ERROR [scrcpy] Could not invoke method
java.lang.reflect.InvocationTargetException
at java.lang.reflect.Method.invoke(Native Method)
at d.b.a.g.f.g(Unknown Source:18)
at d.c.a.a.f.b(:2)
at d.c.a.a.n$a.run(:31)
Caused by: java.lang.IllegalArgumentException: displayToken must not be null
at android.view.SurfaceControl.setDisplayPowerMode(SurfaceControl.java:1813)
... 4 more
system-framework-framework.jar-SurfaceControl.txt

scrcpy version: 2.3.1

Are you sure?

Sorry, I made a mistake!

scrcpy version: 2.3.1

Are you sure?

Sorry, I made a mistake!


Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.

I extended catalog table with custom field. I noticed this error from past few days. on troubleshot, if i remove following code
from columns.tsx in my custom component.
<OverflowTooltip
        text={entity.metadata.description}
        placement="bottom-start"
      />

it works perfectly without tooltip.

Alright, thanks for reporting - fixed in https://github.com/backstage/backstage/pull/8563


Update organisations to use STI

whilst code looks pretty straightfoward change, could you add description as i dont what its supposed to acheive?


Add -[rR] support

This issue was originally created at: 2002-12-06 01:14:07.
This issue was reported by: stevenknight.
stevenknight said at 2002-12-06 01:14:07

Clear out (don't use) default environment/objects. These two  meant different things to Gnu Make, but they're probably the same for us. Or should we separate them somehow?

issues@scons said at 2002-12-06 01:14:07

Converted from SourceForge tracker item 649400

gregnoel said at 2008-04-02 13:15:18

Release team triage.

gregnoel said at 2008-04-18 18:47:28

We shouldn't have UNCONFIRMED issues; changing them to NEW.

gregnoel said at 2008-12-26 13:20:21

Adjust triage of issues.

stevenknight said at 2009-11-10 18:00:19

stevenknight => issues@scons

Make -r, --no-builtin-rules - eliminate use of built-in implicit rules. Also clear out the default list of suffixes for suffix rules.
Make -R, --no-builtin-variables - Don't define any built-in variables.
Why would we want these?

Yeah. 20 years noone has missed them. Closing.


New version: Box.Box version 2.26.330

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

I get this with a 1603 when running in Sandbox; Probably just a sandbox issue though; I'll wait for someone else to test on VM
Calling custom action Box.Desktop.Installer.CustomActions!CustomActions.CustomActions.CreateFallbackDeviceIDKey
Begin CreateFallbackDeviceIDKey
  setting:e92f1a52-778d-404b-8f25-74615e4b3264
MSI (s) (F0:1C) [15:20:42:643]: Product: Box -- Error 1935. An error occurred during the installation of assembly component {22D75DAD-658C-3109-91F7-4CB1A0729109}. HRESULT: 0x80070003. assembly interface: IAssemblyCache, function: CreateAssemblyCacheItem, assembly name: Microsoft.VC90.CRT,version="9.0.30729.6161",publicKeyToken="1fc8b3b9a1e18e3b",processorArchitecture="amd64",type="win32"

Error 1935. An error occurred during the installation of assembly component {22D75DAD-658C-3109-91F7-4CB1A0729109}. HRESULT: 0x80070003. assembly interface: IAssemblyCache, function: CreateAssemblyCacheItem, assembly name: Microsoft.VC90.CRT,version="9.0.30729.6161",publicKeyToken="1fc8b3b9a1e18e3b",processorArchitecture="amd64",type="win32"
Action ended 15:20:42: InstallExecute. Return value 3.
Info 1935. An error occurred during the installation of assembly component {22D75DAD-658C-3109-91F7-4CB1A0729109}. HRESULT: 0x80070422. assembly interface: IAssemblyCache, function: UninstallAssembly, assembly name: Microsoft.VC90.CRT,version="9.0.30729.6161",publicKeyToken="1fc8b3b9a1e18e3b",processorArchitecture="amd64",type="win32"
Action ended 15:20:43: INSTALL. Return value 3.

Looks more like error 1935?
https://support.microsoft.com/en-us/topic/fix-an-installation-may-fail-with-error-1935-when-an-msi-file-tries-to-install-many-policy-files-on-a-computer-that-has-the-net-framework-2-0-installed-a0b0439b-0d6e-f149-f550-e869442fcd9d

Looks more like error 1935? https://support.microsoft.com/en-us/topic/fix-an-installation-may-fail-with-error-1935-when-an-msi-file-tries-to-install-many-policy-files-on-a-computer-that-has-the-net-framework-2-0-installed-a0b0439b-0d6e-f149-f550-e869442fcd9d

Possibly; I just took the first error I saw; At the end of the install it returns a 1603


Datadog or Papertrail

Hi,
Sort of a question more than an issue.. how does datadog fit into this mix? Is this more than just shipping cloudwatch lots to papertrail?
Thanks.

Hi actually yes, if the log entry is formatted in certain way we treat it as a metric entry and push it to datadog. See our heroku to datadog log drain for more details. https://github.com/apiaryio/heroku-datadog-drain-golang#custom-metrics I guess we should maybe document the whole thing nicely so it makes sense to everybody not just us.

Thanks for the quick answer! The documentation is welcome, as it appears this tool is exactly what we need. I might be asking you a few more :) Is there a better place to ask or is github issues fine?

Github issues is fine.


NuGet outdated?

Hi, why is the latest version in NuGet gallery 1.0.0-beta-484 while in MyGet is 1.0.0-beta-532?

we decided to stop publishing unstable releases to NuGet, and nightly builds are automatically published to MyGet for easier public testing.
I will try to unpublish all unstable versions from NuGet, once publishing the first stable version.

Please give some visible info (on first sight) to readme and doc, because I missed it and was developing some time with the old version from nuget.

it's mentioned in the documentation site, I will make it more clearer, thanks for the feedback


chip-tool CLI command to compute manul pairing code cheksum

Test Case TC-DD-1.2 Step 3 requires us to compute checksum of the manual pairing code and validate, Can a CLI command be added into chip-tool for computing this checksum for test scripts to use.
Test plan screenshot for reference

Closing issue, It looks like this was achieved with PR #13048, @kvikrambhat please re-open if not.


Unable to install howdy under Ubuntu 20.04

Hi
I have trying to install howdy under Ubuntu 20.4 but each time I run
sudo apt install howdy
my machine power off
How can I diagnose the problem?
Best regards and thanks for your help.

I've searched for similar issues already, and my issue has not been reported yet.
Linux distribution (if applicable):
Howdy version (sudo howdy version):

I tried again to install howdy and now I could install it but when I tried
sudo howdy add
I get an import error
I am attaching the screenshot

Thanks you very much and best regards

I was also having issues after installing Howdy yesterday on ZorinOS (Ubuntu 20.04) - got the  "command not found".
Tried purging the app, cleaning/autoremoving apt, then finally re-installed successfully (where I saw it downloading Numpy packages), so I guess it was just a blip in my initial install process.
I'd say try the following, then see if you get a different error (cause it seems like some missing/improperly installed dependencies somewhere)
sudo apt purge howdy
sudo apt autoremove
sudo apt update
sudo apt install howdy

A spontaneous shutdown is new behavior for sure. The installer runs a c compile that can take quite a lot of power, maybe your power supply is not up to the task?
The second issue is a corrupted install, the steps from @ntfrueh will help there


Support EIP-712 structured signing

Overview
See https://github.com/shapeshift/web/issues/5138 for original issue.
We want to be able to support actual EIP-712 signing i.e eth_signTypedData_v4 scheme. We currently only support personal_sign i.e good old regular unstructured signing.
The obvious first consumer of this will be CoW swapper, for which signing requests will be able to be displayed in a structured way vs. the hashed digest currently.
References and additional details
MM signing schemes docs: https://docs.metamask.io/wallet/how-to/sign-data/
CoW reference PR with a fallback to personal_sign: https://github.com/gnosis/cowswap/pull/576
Acceptance Criteria

CoW swapper uses eth_signTypedData_v4 if available in the current hdwallet (note, this isn't implemented yet and the guts of this will be hdwallet implementation)
personal_sign is still working as before, and this is brought as a non-breaking change, with the possibility to choose a signing scheme, but no runtime change for current consumooors, except CoW swapper
This should be implemented in hdwallet for all wallets in which eth_signTypedData_v4 works (possibly all?), and properly have feature capabilities as off for others.
eth_signTypedData_v4 is not the default option, rather consumers are able to select a signing scheme. personal_sign is still the default, in case no signing scheme is selected
When trying to do an eth_signTypedData_v4 call, failures are properly handled with a fallback to personal_sign

For MetaMask, eth_signTypedData_v4 should be intented but may fail, because Ledger and Trezor through MM do not support eth_signTypedData_v4, see https://docs.metamask.io/wallet/how-to/sign-data/

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Will implement when cowswap volume goes parabolic.

@gomes to take a look at this for portals.


Git Backend makes first request to repo without credentials

Describe the bug
We are running Spring Cloud Config with the Git backend and we've identified that when we call /actuator/health it's first making a call to the git repo and receiving a 401 (Unauthorized), and right away a call that succeeds. It appears as if the first request is not sending the credentials, but the second is.
This doesn't seem right, am I missing something? I'm using a version a bit old (2.1.2.RELEASE), I looked through the issues but couldn't find any issue related to this.
Here is a screenshot of the outgoing calls that /actuator/health makes depicting the issue:

Notice how this is not an isolated incident, but rather, each failed attempt is followed by a successful one:

I'd say it's all up to your configuration. Can you share?

Thanks for the quick reply. We are running it inside a container, here are the environment variables we are setting:
SPRING_CLOUD_CONFIG_SERVER_GIT_USERNAME=some-username
SPRING_CLOUD_CONFIG_SERVER_GIT_PASSWORD=some-password
<EMAIL_ADDRESS>SPRING_CLOUD_CONFIG_SERVER_GIT_FORCE-PULL=true
SPRING_CLOUD_CONFIG_SERVER_GIT_BASEDIR=/opt/app-config
ENCRYPT_KEY=some-key
MANAGEMENT_ENDPOINTS_WEB_EXPOSURE_INCLUDE=health

what version?
Seems odd that use have the username in the uri and env vars

what version?

2.1.2.RELEASE

Seems odd that you have the username in the uri and env vars

Indeed. I haven't noticed that. Let me check if there is a reason for it and remove it to see if it makes a difference.

That's an old version and I would suggest updating to Hoxton.SR8

Updated the git url to exclude the username, it did not make a difference. I'm in the process of updating the app to the latest version. I'll share the results.

We upgraded to the latest versions, but no luck:
spring-boot-starter-parent from 2.1.5 to 2.3.4
spring-cloud-config-server from 2.1.2 to 2.2.5
Here is the whole pom file.
It does not seem to be a problem with just the /actuator/health endpoint, it also happens when requesting the configuration like /some-app/some-profile/some-label

@OlgaMaciaszek and @spencergibb any progress on this issue- I still see it happening 2020.0.4. any progress on that?

@spencergibb any update on that?

On springboot 3.3.1 SpringCloud 2023.0.2 Java21
cloud:     config:       server:         bootstrap: ${CONFIGSERVER_BOOTSTRAP:true}         encrypt:           enabled: ${CONFIGSERVER_ENCRYPTENABLED:true}         git:           uri: https://gitlab.com/arctravel/arc-settlement/config/default-config.git           basedir: ${CONFIGSERVER_BASEDIR}/default           clone-on-start: ${CONFIGSERVER_CLONEONSTART}           force-pull: ${CONFIGSERVER_FORCEPULL}           username: ${GIT_USER}           password: ${GIT_TOKEN}

@spencergibb is there anymore research I can provide to help in getting a fix for this one?

This does limit our able to monitor for true production issues


Please Update Regional availability

Please update the regional availability as, at least, I confirmed that ZRS can be deployed in Japan East though the document describes only LRS is available in the region.

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

ID: 00e43ae5-d548-0d4d-570d-845b41b495c9
Version Independent ID: 6519e9c8-5e17-c9f9-c348-e9e8e73c9ffd
Content: Premium tier for Azure Data Lake Storage
Content Source: articles/storage/blobs/premium-tier-for-data-lake-storage.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@maku0119, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@maku0119  Firstly, apologies for the delay in responding here and any inconvenience this issue may have caused.  Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

Thanks for raising @maku0119 . We've removed the regional availability info from this article as it's redundant to the information presented in this article - https://docs.microsoft.com/en-us/azure/storage/common/storage-redundancy#zone-redundant-storage. That article should reflect accurate regional availability info for premium.
#please-close


Update strings.xml

Ajout de traductions manquantes

So many mistakes in the translation here... Very bad. 👎


Fix user search bug causing errors if no results are found

I rule and have completed some work on Case Manager that's ready for review!
(brief, plain english overview of your changes here)
Made the user search nil response friendly. Based on Sentry alert:

Exception

NoMethodError: undefined method `any?' for nil:NilClass
  app/views/users/search.js.erb:1:in `_app_views_users_search_js_erb__235396535284795989_968060'
    <% if @results.any? %>
...
(133 additional frame(s) were not displayed)

ActionView::Template::Error: undefined method `any?' for nil:NilClass

This pull request makes the following changes:

Cleaner nil handling for if you search for users with only whitespace

You can recreate the error:

checkout main
navigate to user management
search for a space, boom, error

Checkout this branch and do it again - no error!
For reviewer:

Adjust the title to explain what it does for the notification email to the listserv.
Tag this PR:

feature if it contains a feature, fix, or similar. This is anything that contains a user-facing fix in some way, such as frontend changes, alterations to backend behavior, or bug fixes.
dependencies if it contains library upgrades or similar. This is anything that upgrades any dependency, such as a Gemfile update or npm package upgrade.

If it contains neither, no need to tag this PR.

Codecov Report

Merging #2243 (078114e) into main (bd81368) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2243   +/-   ##
=======================================
  Coverage   65.51%   65.51%
=======================================
  Files          93       93
  Lines        2421     2421
=======================================
  Hits         1586     1586
  Misses        835      835

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 bd81368...078114e. Read the comment docs.


How can I make validation on the cell when it is editing, not after editing?  [Help]

During the updating on the content of the cell, I want to make validation, How Can I do That?

i want the same thing,
have you found any solution in pluto_grid?
like SfDataGrid has canSumbitCell function to validate before submitting.

During the updating on the content of the cell, I want to make validation, How Can I do That?

i have added canSubmitCell function (default null) and shouldResetOnFailure flag (default false) in PlutoColumn to achieve what i needed, ping me if you still need this

Hi @Infinity-1729
Can you give me your solution, please?


RTL support

It adds basic RTL support for go-epub

@Monirzadeh correct, it looks like one of the remote resources for one of the tests is returning a 404, as per #66
Regarding this PR, I'm having trouble understanding what it's for. We already have a method to set page-progression-direction. Isn't that enough for RTL support?
Thanks!

Ah, I might've misunderstood, maybe page-progression-direction is just the direction that the pages turn, and not the way the characters are ordered.
At any rate, I want to get this fixed but I'd like to better understand the problem first.

When you say it doesn't "work", what does that mean?
Do you have a sample RTL EPUB I could take a look at? Preferably one with dir (and PPD if possible) set. I'm curious to see for myself what the different options do.
What Epub reader are you using?

Thanks!

Ah ok, I think I understand the problem now. The characters of the words are all in correct RTL order, but without dir, the text is improperly justified and even out of order.
I was going through the EPUB 3.3 spec, and it looks like dir can also be set in the package document file. Did you try that? https://www.w3.org/TR/epub-33/#attrdef-dir

I don't like the idea of merging something that will break the tests, but my understanding is that it is unrelated to this change.
Therefore, I am ok to merge.

I don't like the idea of merging something that will break the tests

Agreed. @Monirzadeh if you can merge or rebase this off the main branch, we can merge it.
I won't be available for the rest of the week but @owulveryck feel free to merge it if the tests are passing.
Thanks!


Latest version not available in maven repo

Our team is using this library in our Java project.  Unfortunately, we are stuck using the 0.2.1 version, because that's the latest available in the maven repository: https://mvnrepository.com/artifact/org.wiztools/xsd-gen/
Are there any plans to publish the latest version?  We would like to be able to use newer features introduced post 0.2.1.
Thank you

Any possibility of using Jitpack:
https://jitpack.io/#wiztools/xsd-gen/

Any possibility of using Jitpack:

Unfortunately, this isn't a possibility due to corporate rules.


how is this different from...

https://github.com/suda/tool-bar

This one is uglier but doesn't require any plugins.  You can select a command, or file, or web page with a few clicks and then type in the name.  There are no icons.  See the readme image.


Remove the IEKS

It's badly implemented and noone uses it so removing it simplifies the code and documentation. I also suspect that the IEKS will be easier to implement at a later point in time, if the codebase becomes a bit more filter-like.

Codecov Report

Merging #106 (908d1f5) into main (7fe479d) will increase coverage by 0.50%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   94.57%   95.08%   +0.50%
==========================================
  Files          25       24       -1
  Lines         996      977      -19
==========================================
- Hits          942      929      -13
+ Misses         54       48       -6

Impacted Files
Coverage Δ

src/ProbNumDiffEq.jl
76.47% <ø> (ø)

src/perform_step.jl
98.31% <100.00%> (-0.03%)
: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 7fe479d...908d1f5. Read the comment docs.


images not aligned at all (break out of editor) as of 1.2.22 again

Hi,
it seems that with 1.2.22 the alignment issue of images returned. The previous version had this fixed as images would appear right where the text is/below it. With the new version the images appear to be aligned most-left when used on new lines.
See this example with 1.2.22:

and the same document with the 1.2.15 which I had backed up before the alignment changes:

Seems to be related to: https://github.com/xRyul/obsidian-image-converter/issues/133
What theme is this? Do you use any snippets or other plugins which modify images?
Currently I struggle to replicate on clean vault.


Fix: retrieval stats

Closes https://github.com/embeddings-benchmark/mteb/issues/1475
We now have statistics for (almost) all tasks. However, I couldn't retrieve them for FilipinoHateSpeechClassification because attempting to download it results in a DatasetGenerationError
Checklist

[X] Run tests locally to make sure nothing is broken using make test.
[X] Run the formatter to format the code using make lint.

@Samoed will you create an issue on FilipinoHateSpeechClassification otherwise I believe this is fine


Flextype Admin Panel: Welcome page

Welcome page should be added - dashboard

moved to https://github.com/flextype-plugins/admin/issues/86


Use a TURN server for proxying traffic when peer-to-peer fails

We should use a TURN server in "the cloud" (not on either peer's local network) to proxy traffic if needed by WebRTC.  Without doing this, WebRTC connections will fail for ~15% of pairs of users.

The long term ideal would be to use a P2P distributed set of TURN servers operated by uProxy instances with sufficiently open NATs or public IPs (resilient overlay style, perhaps using a kaleidoscope like protocol).

Closing as this is not a priority right now


Missing drugs when updating to ChEMBL 29

Context
The comparison of the numbers between the drug index based on ChEMBL 28 and ChEMBL 29 shows a decrease in the number of unique drugs (13 185 vs 12594). ChEMBL 29 represents an overall increase over the previous version, which indicates that there is something wrong with the way we generate the drug index.
There are 984 ChEMBL IDs that went missing between versions. Out of these, 968 of them are present in the dataset we produce out of the ChEMBL molecule dump (gs://open-targets-pre-data-releases/21.09.1/input/annotation-files/chembl/chembl_29_molecule-2021-09-16.jsonl). The other 16 have been removed by ChEMBL.
Here is the dataset with the offending molecules:
https://app.zenhub.com/files/143733948/8fa001d8-0826-42a6-80ca-327acdfb59b0/download
Is it a pipeline issue?
The drug step has suffered minor changes since the last release:

There were some changes that needed to be made in PIS to generate the ChEMBL target dataset. I don't think this is the root cause as both versions of the datasets are identical.
Some minor improvements in the Drug step that I don't see how they could cause the difference.

Is it a data issue?
As @JarrodBaker pointed out, the requisites for a drug to be included are:

have an associated indication
have the MoA annotated
have a cross-reference to DrugBank.

I have compared them by creating a hasIndication, hadIndication, hasMoa, hadMoa columns.

There are only 5 records whose indications have gone missing:

[Row(id='CHEMBL288376'),
 Row(id='CHEMBL4594270'),
 Row(id='CHEMBL4594358'),
 Row(id='CHEMBL253144'),
 Row(id='CHEMBL4298194')]

And one molecule when MoAs are considered:

[Row(id='CHEMBL4298088')]

The DrugBank xrefs have not changed as these have been collected from the same input file(gs://open-targets-pre-data-releases/21.09.1/input/annotation-files/drugbank-2020-06-01.csv.gz).
I don't think we have changed how MoA and indications are inherited since 21.06, so I don't see how the definition of drug could account for the change.
I'll keep digging, but any suggestion is welcome  @d0choa @DSuveges @mkarmona  @JarrodBaker @cmalangone

I think the lack of molecules is explained by the work done to align with ChEMBL parent/child propagation of indications and MoAs #1590
This change (which I don’t think went live in 21.06) implied fewer indications in child molecules as a result of not propagating from parents to children anymore. This change will imply that a lot of the child molecules that had an indication in the past, don't have it anymore. As a result, many molecules won't fulfil the rule for inclusion in the platform (indication | MoA | Drugbank ID).
The next confirms that most of the molecules lost in the above list were indeed child molecules:
❯ grep -c parent problematic_chembl_ids.json
963
❯ grep -c id problematic_chembl_ids.json
968

@ireneisdoomed really well described issue :clap:

@d0choa I kind of assumed these changes went live in 21.06 as they were merged mid-June and the indexes were uploaded to the release buckets on June 27th. It would be very nice to confirm it as that would definitely explain things.

I retract my own words. I think there might be some bug related to #1590 that might be causing the disappearance of some MoAs and therefore some molecules.  I'll dig a bit further

@d0choa these are good examples to see what is failing here.

Looking closely at the two examples @d0choa wrote above CHEMBL3094198 and CHEMBL2104871 it seems their parent molecules are the ones holding the MOA data.
The case CHEMBL3094198 does not hold any MOA but its parent does as I show below
❱ cat chembl_29_molecule-2021-09-16.jsonl | grep CHEMBL3094198 | jqless
{
  "molecule_chembl_id": "CHEMBL3094198",
  "molecule_hierarchy": {
    "parent_chembl_id": "CHEMBL1431",
    "molecule_chembl_id": "CHEMBL3094198"
  },
  "molecule_type": "Small molecule",
...

Let's check the parent's MOAs
◆ cat chembl_29_mechanism-2021-08-20.jsonl | grep 'CHEMBL1431'
{"molecule_chembl_id": "CHEMBL1703", "target_chembl_id": "CHEMBL3391681", "action_type": "INHIBITOR", "mechanism_of_action": "Mitochondrial glycerol-3-phosphate dehydrogenase inhibitor", "mechanism_refs": [{"ref_id": "24847880", "ref_url": "http://europepmc.org/abstract/MED/24847880", "ref_type": "PubMed"}], "record_id": 1344532, "parent_molecule_chembl_id": "CHEMBL1431", "_metadata": {"parent_molecule_chembl_id": "CHEMBL1431", "all_molecule_chembl_ids": ["CHEMBL1703", "CHEMBL1431"], "should_appear_in_browser": false}}
{"molecule_chembl_id": "CHEMBL1703", "target_chembl_id": "CHEMBL2363065", "action_type": "INHIBITOR", "mechanism_of_action": "Mitochondrial complex I (NADH dehydrogenase) inhibitor", "mechanism_refs": [{"ref_id": "23277337", "ref_url": "http://europepmc.org/abstract/MED/23277337", "ref_type": "PubMed"}, {"ref_id": "23358356", "ref_url": "http://europepmc.org/abstract/MED/23358356", "ref_type": "PubMed"}, {"ref_id": "25017630", "ref_url": "http://europepmc.org/abstract/MED/25017630", "ref_type": "PubMed"}], "record_id": 1344532, "parent_molecule_chembl_id": "CHEMBL1431", "_metadata": {"parent_molecule_chembl_id": "CHEMBL1431", "all_molecule_chembl_ids": ["CHEMBL1703", "CHEMBL1431"], "should_appear_in_browser": false}}

Checking these target Chembl IDs
◆ cat chembl_29_target-2021-09-16.jsonl | grep 'CHEMBL3391681'|jq -r '.target_components[].accession'
P43304

And a further look at our target index we can double-check if the accession is correct
@ T.filter($"symbols" === "P43304").show
+---------------+--------------+-------+
|             id|approvedSymbol|symbols|
+---------------+--------------+-------+
|ENSG00000115159|          GPD2| P43304|
+---------------+--------------+-------+

And taking some random accession from the other target and resolving back from our index
@ T.filter($"symbols" === "P03923").show
+---------------+--------------+-------+
|             id|approvedSymbol|symbols|
+---------------+--------------+-------+
|ENSG00000198695|        MT-ND6| P03923|
+---------------+--------------+-------+

Repeating the same procedure with the other molecule shows that the accession id is not human therefore it is contained neither in the UniProt human file nor the target index
@ T.filter($"symbols" === "P54300").show
+---+--------------+-------+
| id|approvedSymbol|symbols|
+---+--------------+-------+
+---+--------------+-------+

So the most probable hypothesis to pose is whether Chembl is projecting some parents' MOAs onto their children molecules @ireneisdoomed @JarrodBaker. A further question is whether this is a systematic approach from them or an uncovered bug as you can see in their JSON a field called "should_appear_in_browser": false which could mean nothing tbh.

In conclusion:

Our codebase is working as specified. MoAs and Indications are only propagated child -> parent (#1590) and molecules are only included if (Indication | MoA | DrugBank)
The majority of missing molecules are child molecules before (#1590) kicked in. Because they lost the inherited MoA they now don't qualify as valid molecules.
ChEMBL web does seem to propagate MoAs in the wrong direction parent -> child in the offending molecules. This propagation happens at their API level and it goes against the propagation rule they recommended.

Actions:

@ireneisdoomed to confirm with ChEMBL why their site does not comply with the rules discussed on the 3/6/21 meeting

What a ticket! I was heading in the same direction as you @d0choa . When I look at mechanisms of action in 21_06 data the missing molecules are found in the chemblIds field along with their parent, but in 21_09 they are disappearing.

After a conversation with Juan and Patricia from ChEMBL, everything looks like we are safe and this drop makes sense.

For the example we were discussing, CHEMBL3094198, it is not that the MoA is being inherited from the parent, but from another salt.
This is a bug in ChEMBL's logic that they will be resolving in their UI. When we had the discussion in June (#1590), the approach they recommended was not entirely implemented on their side. These molecules that we are dropping now shouldn't have been there in the first place.
I have provided them with a list of these compounds for them to test.
For the other 5 molecules which are not children and hence do not follow this pattern, they are gone because they used to be in the MoA/indication dataset. Patricia said that this could be caused by that same molecule being represented with another CHEMBLID. I have also provided these and she'll look everything makes sense.

Related to the last point, one interesting thing I learnt was that every InChiKey is represented by a single CHEMBLID. Therefore, for a given molecule, if the known molecular information changes they will create another ID to represent that molecule (and hopefully deprecate the obsolete one).
I consider this ticket as closed with no actions pending to be done on our side.


update readme with info on accessing validation errors and extending ajv

Thank you ! Out of the country currently!


Roomba 671 - Lights do not turn off

I've managed to configure hassbian access with the help of your code. Everything is working correctly, except for the lights on the device not turning off.
I have continuous: false set in my configuration.yaml. Is there anything else I need to do, please?
Many thanks.

Same issue on Roomba 890 both in continuous or not.

Hello, have you figured this out? I think I have the same problem. I am using hassio component and I have Roomba 676.
Thanks!

Hello, same issue here, also for Roomba 676.

Hello, same isseu here but i have a Roomba E5

What lights are you talking about?

When you connect to the roomba from the original app the ligt from clean button will turn on and the display will show you the battery percentage and WiFi status
When i use the iRobot intergration these lights keeps on 24/7

Here is a picture

Same issue here. Running the iRobot configuration with home assistant. Issue is present in both continuous and non-continuous mode.

Same thing happening with Roomba 675

Same problem here, any updates available?

Same on 676, seems like a limitation on the roomba. As soon and as long as an mqtt connection is active, the LEDs are on. Can we submit feature requests to iRobot somewhere? :clown_face:

Same here, this is pretty problematic as it seems to drain the battery quite a bit, and using continuous: off in home-assistant (in my case), just make the state-sync very unreliable.

If anyone knows how to turn the lights off, I’ll be happy to implement it.
Roomba do not accept user suggestions by the way (one of their policies) - I assume it’s some legal thing.
The power drain from the lights is minuscule compared to the drive/vacuum motors. Probably about the same as the WiFi connection.

the same issue on roomba 694

same issue with my roomba e5, lights will not turn off while having the roomba integration enabled.

For what's worth it - same here, Roomba 675. Especially annoying as it sleeps in the same room as some guests. Green light can be spooky thus acceptable for Halloween, but the other 364 nights have no use 😓

Same here with 671, can't turn off the button lights :/

Same problem with mine:
R671040 iRobot
Firmware: 3.5.62+95

Came across this issue while trying to solve the same problem so I figured I might share the outcome.
All I actually needed was the ability to start cleaning sessions every now and then. I did not really care about continuous updates on battery levels and the likes,  so simply turning the integration off/on was an acceptable solution to me as long as it could be done automatically (so that I do not have to actually do it manually in the HA integrations settings before and after each cleaning session).
If that approach is of interest to anyone passing by, I installed Spook (available through HACS) which among many other things adds actions (as in stuff you can use in automations) to enable or disable integrations.
This means a cleaning automation with a Roomba vacuum may now look like so:
Enable Roomba integration -> Wait a bit -> Run a usual cleaning session -> Disable Roomba integration
(the lights turn off after a minute or so with my Roomba 6XX)
This of course is not a perfect solution (a perfect one would probably need to involve Roomba firmware developers and therefore likely require some kidnapping) but is an ok solution for my own scenario, which ultimately is just about cleaning the floor without turning my house into a matrix-inspired nightclub.
(cross posting from a home-assistant issue as I also ended up here while searching for a solution)

Thanks Adrien for sharing your approach,
this is such a simple and somewhat elegant solution! Much appreciated!
Happy new year!
On Thursday, 4 January 2024 at 11:36 PM, Adrien Coppens @.***> wrote:

Came across this issue while trying to solve the same problem so I figured I might share the outcome.
All I actually needed was the ability to start cleaning sessions every now and then. I did not really care about continuous updates on battery levels and the likes, so simply turning the integration off/on was an acceptable solution to me as long as it could be done automatically (so that I do not have to actually do it manually in the HA integrations settings before and after each cleaning session).
If that approach is of interest to anyone passing by, I installed Spook (available through HACS) which among many other things adds actions (as in stuff you can use in automations) to enable or disable integrations.
This means a cleaning automation with a Roomba vacuum may now look like so:
Enable Roomba integration -> Wait a bit -> Run a usual cleaning session -> Disable Roomba integration
(the lights turn off after a minute or so with my Roomba 6XX)
This of course is not a perfect solution (a perfect one would probably need to involve Roomba firmware developers and therefore likely require some kidnapping) but is an ok solution for my own scenario, which ultimately is just about cleaning the floor without turning my house into a matrix-inspired nightclub.
(cross posting from a home-assistant issue as I also ended up here while searching for a solution)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


feat: SearchInput component with debounce functionality

Description
Component: SearchInput

Modified the component with new props to manage debounce functionality.
Added a test file
Added required mock files for running jest test
Added a documentation file describing the component

Why
New Feature: Enhanced the SearchInput component with debounce functionality.
Issue
#222
Checklist

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[x] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas

Hi @charmi-v  could you please resolve the conflicts? I'll merge the pr afterwards. Sorry for the inconvenience.

Hi @charmi-v could you please resolve the conflicts? I'll merge the pr afterwards. Sorry for the inconvenience.

Hello @evegufy ,
Conflicts have been resolved, and the package version has been updated to 3.0.32.


mTLS and traffic split

Background
I am preparing a traffic shift from a web API in a Swarm cluster (let's call it swarm_api) to the same API in a Kubernetes cluster (k8s_api). The two clusters are on distinct private networks. The API's clients are on the same private network as the Swarm cluster, making requests over HTTP. However, the Kubernetes cluster is isolated (and fully managed by a cloud provider; the nodes' network is abstracted); the connection must therefore be secured.
My plan is to add an edge envoy in the Swarm cluster (swarm_proxy) to split traffic between swarm_api and a second edge envoy in the Kubernetes cluster (k8s_proxy), which would route traffic to k8s_api. The connection between swarm_proxy and k8s_proxy must be secured with mutual TLS as it goes through the Internet.
Private network 1
                      (swarm_proxy)---HTTP--->(swarm_api)
____________________________|__________________________
Internet                    |
                          HTTPS
____________________________|__________________________
Private network 2           |
                            ⌄
                       (k8s_proxy)----HTTP---->(k8s_api)

What I have so far

One the one hand, I have successfully prototyped a traffic split over HTTP, without mTLS:Private network 1
                      (swarm_proxy)---HTTP--->(swarm_api)
____________________________|__________________________
Internet                    |
                          HTTP
____________________________|__________________________
Private network 2           |
                            ⌄
                       (k8s_proxy)----HTTP---->(k8s_api)

swarm_proxy uses an http_connection_manager to split traffic between swarm_api and k8s_proxy; k8s_proxy simply routes traffic to k8s_api (see configurations below).
On the other hand, I have successfully prototyped simple routing with mTLS:Private network 1
                      (swarm_proxy)
____________________________|__________________________
Internet                    |
                          HTTPS
____________________________|__________________________
Private network 2           |
                            ⌄
                       (k8s_proxy)----HTTP---->(k8s_api)

swarm_proxy uses a tcp_proxy to route traffic to k8s_proxy; the envoy cluster adds a tls_context including a client certificate; k8s_proxy requires TLS with client certificate, then routes traffic to k8s_api (see configurations below).

Issue
Now I'm struggling to make both traffic split and mTLS work at the same time: if I try to combine the two approaches in a third prototype, swarm_proxy returns 301 Moved Permanently when it routes traffic to k8s_proxy (see configurations below).
Is there something I'm missing / don't understand, or is this a bug?
Envoy Configurations
Prototype 1: traffic split
# swarm_proxy
static_resources:
  listeners:
  - address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 80
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        config:
          stat_prefix: ingress
          route_config:
            virtual_hosts:
            - name: api
              domains:
              - "*"
              routes:
              - match:
                  prefix: "/myapi/v1"
                route:
                  weighted_clusters:
                    runtime_key_prefix: routing.traffic_split.api
                    clusters:
                    - name: swarm_api
                      weight: 50
                    - name: k8s_proxy
                      weight: 50
          http_filters:
          - name: envoy.router
          access_log:
          - name: envoy.file_access_log
            config:
              path: "/dev/stdout"
  clusters:
  - name: swarm_api
    type: strict_dns
    hosts:
    - socket_address:
        address: swarm_api
        port_value: 80
    connect_timeout: 1s
  - name: k8s_proxy
    type: strict_dns
    hosts:
    - socket_address:
        address: k8s_proxy # TODO replace with public DNS when implemented
        port_value: 80
    connect_timeout: 1s
admin:
  access_log_path: "/dev/stdout"
  address:
    socket_address:
      address: <IP_ADDRESS>
      port_value: 9901

# k8s_proxy
static_resources:
  listeners:
  - address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 80
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        config:
          stat_prefix: ingress
          route_config:
            virtual_hosts:
            - name: k8s_api
              domains:
              - "*"
              routes:
              - match:
                  prefix: "/myapi/v1"
                route:
                  cluster: k8s_api
          http_filters:
          - name: envoy.router
          access_log:
          - name: envoy.file_access_log
            config:
              path: "/dev/stdout"
  clusters:
  - name: k8s_api
    type: strict_dns
    hosts:
    - socket_address:
        address: k8s_api
        port_value: 80
    connect_timeout: 1s
admin: {...}

Prototype 2: mutual TLS
# swarm_proxy
static_resources:
  listeners:
  - address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 80
    filter_chains:
    - filters:
      - name: envoy.tcp_proxy
        config:
          stat_prefix: ingress
          cluster: k8s_proxy
          access_log:
          - name: envoy.file_access_log
            config:
              path: "/dev/stdout"
  clusters:
  - name: k8s_proxy
    type: strict_dns
    hosts:
    - socket_address:
        address: k8s_proxy # TODO replace with public DNS when implemented
        port_value: 443
    connect_timeout: 1s
    tls_context:
      common_tls_context:
        validation_context:
          trusted_ca:
            filename: /etc/certs/ca.crt.pem
        tls_certificates:
        - certificate_chain:
            filename: /etc/certs/swarm_proxy.crt.pem
          private_key:
            filename: /etc/certs/swarm_proxy.key.pem
admin: {...}

# k8s_proxy
static_resources:
  listeners:
  - address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 443
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        config:
          stat_prefix: ingress
          route_config:
            virtual_hosts:
            - name: k8s_api
              domains:
              - "*"
              require_tls: ALL
              routes:
              - match:
                  prefix: "/myapi/v1"
                route:
                  cluster: k8s_api
          http_filters:
          - name: envoy.router
          access_log:
          - name: envoy.file_access_log
            config:
              path: "/dev/stdout"
      tls_context:
        require_client_certificate: true
        common_tls_context:
          validation_context:
            trusted_ca:
              filename: /etc/certs/ca.crt.pem
          tls_certificates:
          - certificate_chain:
              filename: /etc/certs/k8s_proxy.crt.pem
            private_key:
              filename: /etc/certs/k8s_proxy.key.pem
  clusters:
  - name: k8s_api
    type: strict_dns
    hosts:
    - socket_address:
        address: k8s_api
        port_value: 80
    connect_timeout: 1s
admin: {...}

Prototype 3: traffic split and mutual TLS (NOT WORKING)
# swarm_proxy
static_resources:
  listeners:
  - address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 80
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        config:
          stat_prefix: ingress
          route_config:
            virtual_hosts:
            - name: api
              domains:
              - "*"
              routes:
              - match:
                  prefix: "/myapi/v1"
                route:
                  weighted_clusters:
                    runtime_key_prefix: routing.traffic_split.api
                    clusters:
                    - name: swarm_api
                      weight: 50
                    - name: k8s_proxy
                      weight: 50
          http_filters:
          - name: envoy.router
          access_log:
          - name: envoy.file_access_log
            config:
              path: "/dev/stdout"
  clusters:
  - name: swarm_api
    type: strict_dns
    hosts:
    - socket_address:
        address: swarm_api
        port_value: 80
    connect_timeout: 1s
  - name: k8s_proxy
    type: strict_dns
    hosts:
    - socket_address:
        address: k8s_proxy # TODO replace with public DNS when implemented
        port_value: 443
    connect_timeout: 1s
    tls_context:
      common_tls_context:
        validation_context:
          trusted_ca:
            filename: /etc/certs/ca.crt.pem
        tls_certificates:
        - certificate_chain:
            filename: /etc/certs/swarm_proxy.crt.pem
          private_key:
            filename: /etc/certs/swarm_proxy.key.pem
admin: {...}

# k8s_proxy : same as prototype 2

You need to either add use_remote_address: true to k8s_proxy or remove require_tls: ALL from it.
If you're using use_remote_address: false (default), then k8s_proxy is going to receive and accept X-Forwarded-Proto: http from swarm_proxy and reject client's HTTP request, since it doesn't fulfill the require_tls restriction, which applies to client's HTTP request and not to the connection to k8s_proxy, require_client_certificate: true is enough to enforce mTLS between the proxies.
Also, you should add use_remote_address: true to swarm_proxy, since it's acting as an edge proxy.

Thanks for the solution and explanations @PiotrSikora! That worked.


fix(AbstractSite.check_sanity): improve future date guard

Solves #1187
Tests for dates in the future compared only by year, and had a hardcoded 2025 value.
Making the tolerance lower (a  week) and dynamic
(from the day of scraping)

Thinking that this should actually be a logger.error, and not an InsanityException; since it would break scrapers and not ingest any data, even if it is a single record that has a bad date (which we could easily correct manually)
Currently happening on coloctapp:


Upgrade our issue templates to use GitHub issue forms ✍

Introduction
GitHub has recently rolled out a public beta for their issue forms feature. This would allow you to create interactive issue templates and validate them 🤯.
Appwrite currently uses the older issue template format. Your task is to create GitHub issue forms for this repository. Please use Appwrite's issue templates as a reference for this PR.
Tasks summary:

[ ] Fork & clone this repository
[ ] Prepare bug report issue form in .github/ISSUE_TEMPLATE/bug.yaml
[ ] Prepare documentation issue form in .github/ISSUE_TEMPLATE/documentation.yaml
[ ] Prepare feature request issue form in .github/ISSUE_TEMPLATE/feature.yaml
[ ] Push changes to master and test issue forms on your fork
[ ] Submit pull request

If you need any help, reach out to us on our Discord server.
Are you ready to work on this issue? 🤔 Let us know, and we will assign it to you 😊
Happy Appwriting!

Hi @Meldiron I would like to work on this issue

can i work on this issue?

@KKVANONYMOUS Assigned it to you! All the best 🚀

@vendz We would love you to check out some of our other open issues and join us on Discord if you need help finding some.


Improve documentation examples

Hi, I'm trying to follow along with the examples but copy/pasting them doesn't produce the displayed result.

Hey @reubano
Maybe something is off in how you're importing axis? Can you try this Codepen sandbox and reproduce any issues?
http://codepen.io/dbox/pen/pbOYwP

Hmmm, this example makes things a lot clearer. I was putting the stylus code into my jade templates. I think that's one area that should be clarified in the docs. For example coming from Bootstrap, its hard to understand what goes where in Axis.

It does say in the repo description that it's a stylus library, and we have very extensive docs on how to use and configure it. Do you think there are any ways other than these to make it more clear?

It does say in the repo description that it's a stylus library

Similarly, Bootstrap is a LESS library but the examples are designed to go into your template file.

and we have very extensive docs on how to use and configure it

I'm trying out roots so the configuration part was already taken care of

Do you think there are any ways other than these to make it more clear?

Be more like Bootstrap docs. If you compare their button example:
Link
<button class="btn btn-default" type="submit">Button</button>
<input class="btn btn-default" type="button" value="Input">
<input class="btn btn-default" type="submit" value="Submit">

to yours:
.selector
  button()

.selector
  button: green 'large'

vs what you supplied in the codepen:
Im a button

.btn
  button()

You can see that the axis docs assume the user already has the appropriate template setup and intuitively knows the code presented should go in a stylus file. This is complicated by the fact the roots uses jade templates which has syntax similar to that of stylus. Whereas with Bootstrap, I can copy and paste the example as is and it just works.

@reubano This makes a lot of sense, thanks for providing a clear example of what you were talking about here. I definitely see where you are coming from. Would you be willing to take a crack at a pull request to introduce this type of change into axis' docs?
Here's the repo: https://github.com/jescalan/axis-www

Would you be willing to take a crack at a pull request to introduce this type of change into axis' docs?

Can't make any promises just yet. But if I'm able to decipher enough to get this website built, I'll be happy to contribute my learnings.

@reubano awesome, looking forward to it!


Null check operator used on a null value

Hi, sometimes I get this error when loading screen with AuthButtons

Null check operator used on a null value, #0 RenderBox.size (package:flutter/src/rendering/box.dart:2023) #1 _ButtonContentState.getTextWidgetWidth (package:auth_buttons/src/shared/core/widgets/button_content.dart:95) #2 _ButtonContentState.initState. (package:auth_buttons/src/shared/core/widgets/button_content.dart:50) #3 SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1175) #4 SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1113) #5 SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:1015) #6 _rootRun (dart:async/zone.dart:1391) #7 _CustomZone.run (dart:async/zone.dart:1293) #8 _CustomZone.runGuarded (dart:async/zone.dart:1201) #9 _invoke (dart:ui/hooks.dart:150) #10 PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:318) #11 _drawFrame (dart:ui/hooks.dart:115)

Hi there,
Thanks for your time,
This happened because the AuthButton is not able to get the RenderBox. This is not really an issue, but I will provide some failback value.
Thanks an good coding!

@elbeicktalat I think this is a Flutter related issue https://github.com/flutter/flutter/issues/107397
A fallback value may not work due this bug.

I think no body facing this issue anymore, by the way it was caused to flutter.


Explictly list GH repos that can write to ECR.

We were using GitHub Topics (labels that can be applied to a repo) to determine which repos should be permitted to write to our ECR container registry.
Turns out this is inadequate, because anyone in the alphagov GitHub org who can create repos (roughly 700 people, most of whom don't work on the GOV.UK web CMS) can create a repo and set whatever topics they like on it, then get write access to our private ECR.
For now, let's just go back to maintaining a list of authorised repos. That shouldn't be too toily in the short term, since we don't create new repos all that often.

dammit 😭


sub-menu在非弹出模式下同样支持 selectable 属性。

Types of changes

[ ] New feature
[ ] Bug fix
[X] Enhancement
[ ] Component style change
[ ] Typescript definition change
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[ ] Test cases
[ ] Continuous integration
[ ] Breaking change
[ ] Others

How is the change tested?

Changelog

Component
Changelog(CN)
Changelog(EN)
Related issues

Menu
sub-menu 在非弹出模式下同样支持 selectable 属性。
sub-menu also supports selectable property in non-popup mode.

Checklist:

[] Test suite passes (npm run test)
[] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others
should be submitted to main branch)

Other information

@heyingliang 你好，贡献前先同步main分支再切分支修改Bug提交PR🫶🏻

已调整并重新发起，麻烦处理一下
---- 回复的原邮件 ----
| 发件人 | @.> |
| 日期 | 2024年10月12日 16:23 |
| 收件人 | @.> |
| 抄送至 |<EMAIL_ADDRESS>|
| 主题 | Re: [arco-design/arco-design-vue] sub-menu在非弹出模式下同样支持 selectable 属性。 (PR #3321) |
Closed #3321.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


fix(plugins/json): fix quoting and escaping in string-based json.value

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area plugins
What this PR does / why we need it:
This fixes the issue of #56. The json plugin relied on the fastjson.MarshalTo function, which unwillingly added the " quotes around strings and caused the Falco rules to not match string values properly.
Which issue(s) this PR fixes:
Fixes #56
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jasondellaluce
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


Build fails with @metamask/sdk dependency in wagmi/connectors

Check existing issues

[X] I checked there isn't already an issue for the bug I encountered.

Describe the bug
When using wagmi version 2.13.2 and @rainbow-me/rainbowkit version 2.2.0, the production build (pnpm run build) fails due to issues with the transitive dependency<EMAIL_ADDRESS>which is pulled in via @wagmi/connectors. The error is related to .d.ts.map files from @metamask/sdk that Webpack cannot process.
However, pinning @metamask/sdk to version 0.30.0 resolves the issue, allowing the build to succeed. This suggests a potential regression in<EMAIL_ADDRESS>or an incompatibility with @wagmi/connectors.
The build failed with the following error:
Module parse failed: Unexpected token
You may need an appropriate loader to handle this file type...
Import trace for requested module:
<EMAIL_ADDRESS>
Link to Minimal Reproducible Example
No response
Steps To Reproduce
Package
{
  "@tanstack/react-query": "^5.61.5"
  "viem": "^2.21.51",
  "wagmi": "^2.13.2",
  "@rainbow-me/rainbowkit": "2.2.0"
}

Run
pnpm install
pnpm run build

What Wagmi package(s) are you using?
wagmi, @wagmi/connectors
Wagmi Package(s) Version(s)
<EMAIL_ADDRESS><EMAIL_ADDRESS>Viem Version
^2.21.51
TypeScript Version
No response
Anything else?
No response

+1

Here is how I fix it temporary
install<EMAIL_ADDRESS>and override the it in wagmi
example (I'm using the pnpm)
{
   "dependencies": {
       "@metamask/sdk": "0.30.0"
   } ,
   "pnpm": {
      "overrides": {
         "@metamask/sdk": "0.30.0"
      }
   }
}

+1

+1

for yarn users, you can add the following to your package.json
  "resolutions": {
    "@metamask/sdk": "0.30.0"
  },

Even I am using the latest wagmi version ^2.13.3 but I am still seeing some warnings.
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
 ✓ Compiled /favicon.ico in 8.5s (4398 modules)
 GET /favicon.ico 200 in 9191ms
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (132kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (132kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (131kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)


Fix compilation for wasm: env WASI_SYSROOT should be optional

On ubuntu-latest github action runner, it used to work without the environment variable.
Fixed #1109 #1113

cc @Niederb @Kuuuube Can you try this PR please?
cc @antaalt

I'm still able to replicate the issue using this pr.

Noticed that #1105 had this diff
-                } else if target == "wasm32-wasi"
-                    || target == "wasm32-unknown-wasi"
-                    || target == "wasm32-unknown-unknown"
-                {

And these targets were not added to the targets list. Adding them to the list along with your changes in this PR does fix the issue.
Threw in a PR targeting this PR's branch. #1115
If theres anything wrong with my changes in that PR feel free to close it or do whatever.


crystal 0.23 reports old version number

I installed crystal 0.23 from source. I've the following requirements:

pre-compiled crystal-0.21.1-1-linux_x86-64
gc 7.6
llvm 3.9.1

In the source dir, I ran command 'make' and 'make spec'. All passed. But it seems crystal 0.23 is reporting 0.21.1 for me. See below.
$ ./crystal-0.21.1-1/bin/crystal -v
Crystal 0.21.1 [3c6c75e] (2017-03-06) LLVM 3.5.0
$ ./crystal-0.23.0-1/bin/crystal -v
Using compiled compiler at `.build/crystal'
Crystal 0.21.1 (2017-06-29) LLVM 3.9.1

Crystal grabs version number from git or environment variable on compile time. I guess @need47 builds crystal from tar ball source.

@MakeNowJust  Indeed, I used tar.gz

Moving out of 0.23.1 release since its not a regression - was happening from before.
It would be nice to fix it, anyway 👍

@MakeNowJust And it just took the 0.21.1 from the original compiler?

It seems so - I've downloaded the 0.22.0 zip, built it with 0.23.0, and got it to report 0.23.0


test: E2E test refactoring

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[x] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I've signed the CLA.
[x] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
[x] My builds are green. Try syncing with master if they are not.
[x] My organization is added to USERS.md.

Changes

Fixes various minor bugs in tests.
Remove unused Sonar code.
Remove code designed for old CircleCI builds -> simplify.
Everything is now logged to stdout/stderr - but because we go test now instead of go test -v - it is only output on test failure.

codegen - ./hack/generate-proto.sh: line 7: GOPATH: unbound variable
test-e2e  - panic: test timed out after 10m0s

Re-run - flaky?

@sarabala1979 finally ready for review!
You could rubber-stamp if you like.

@sarabala1979 to approve.


Support Confluent version 6.2.5

First of all, thank you for creating this. I found it on this thread and it's saved so much time and effort with developing on M1
I am currently still on the 6.x major version of Confluent and wanted to use 6.2.5, but noticed that an image wasn't available from your registry. Would it be okay to update .gitmodules in a 6.x branch in this repo to build that image? If not, is there any other way I can help to update the pipeline to support parallel major versions?

Hi @anuragagarwal561994 and @bm1549.
First of all, thank you @anuragagarwal561994 for this project. In the company I work for, we use Confluent and because many of us received new M1 macs, your project is a life saver!!
I also want to offer my help and I could try to create a PR for version 7.2.0 . Please let me know if this would be ok and if there is another way I could help.

@bm1549 let me give it a try

@bm1549 I checked it, the versions as listed by the official confluent repo has some vulnerabilities. I guess support for version 6.2.5 can be provided but as earlier stated please make sure that the image is used only in local and that it would expose some vulnerabilities.

@bm1549 let me know if there are other good images that you can think I can work upon.
@sjucovschi I tried 7.2.0 there are some issues happening will get back on this.

@anuragagarwal561994 let me know if there's anything I can help with


bad option --allow-superuser in ci

Currently trying to update an app from 1.8.1 to 1.10.1 and are getting the error reported at the meteor repository
https://github.com/meteor/meteor/issues/10894#issuecomment-581227188
error:   exec: /builder/home/.meteor/packages/meteor-tool/.<IP_ADDRESS>mtsmz6.qnno++os.linux.x86_64+web.browser+web.browser.legacy+web.cordova/mt-os.linux.x86_64/dev_bundle/bin/node: bad option: --allow-superuser

Any idea what the problem could be?
I cannot reproduce it locally.

The workaround is to pass --node-version and --npm-version in v1.2.0 until we know why bad-option error throws from meteor >= 1.9


Make stand alone pre-processing pipeline

After exome refactoring #270

Not needed or requested at the moment.


feat: Add explicit differences between body and footer in specs

#37

@stevemao re-review

@bcoe  @stevemao please review 😄
Let's merge this one 😄


chore: merge master from getredash/redash (ENG-3127)

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] New Query Runner (Data Source)
[ ] New Alert Destination
[ ] Other

Description
Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings (if there are UI changes)

@tvansteenburgh I've been deploying to my sandbox and monitoring the redash dashboard in cloudwatch, running queries and looking for/fixing any tracebacks i see in the server, worker, scheduled worker, and installer tasks

also @tvansteenburgh here's a diff of this branch vs getredash/redash's master branch: https://github.com/getredash/redash/compare/master...stacklet:redash:sonny/stacklet/integration
this should be a better diff to look at


[Amazon AWS] Add support for credentials from credentials file

Extension
https://www.raycast.com/Falcon/aws
Description
I want to read short-lived credentials directly from the credentials file, including AWS Session Token
Who will benefit from this feature?
I can start using Leapp and switch profile from them
Anything else?
No response

@andreacavagna01 This extension is already supposed to support reading from the credentials file. Can you share how this is not working for you? It might also be useful to share your config and credentials file (redacted, of course), so that I can reproduce the issue.


[BUG] - The Tabs component often fails to click when using tap to click on the macOS touchpad.

NextUI Version
2.1.7
Describe the bug
https://github.com/nextui-org/nextui/assets/20062482/ce13c1b3-8e02-4434-9e51-58a7ef512af5
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
https://nextui.org/docs/components/tabs#usage

Expected behavior
Every time you tap the Tab, it should work.
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Chrome

Hey @u3u try again using this codesandbox:
https://codesandbox.io/p/sandbox/dwvtlf?file=%2FApp.jsx&utm_medium=sandpack
It works fine for me btw:
https://github.com/nextui-org/nextui/assets/30373425/6a6a22c2-c66d-4338-b993-000da735b496

Hi, it still occurs on my end, only happening when lightly tapping the touchpad (no issue when pressing down firmly). I've tried multiple times and occasionally there is a problem where clicking doesn't switch tabs. I wonder if there is any special handling of click events internally? I suspect it might be an issue with React Aria because it seems similar to #1349.
I think if it only adds onClick event to the Tab button to update the state, this problem is unlikely to occur. There must be other additional processing for clicks or touches that may cause this problem.
If allowed to add additional onClick event to the Tab button, it can be debugged, and should be able to force update the Tab in controlled mode.
https://github.com/nextui-org/nextui/assets/20062482/3518c0d3-d294-4727-a428-ab243853e8a0

I have confirmed that onSelectionChange does not always trigger when a tab is clicked.
I fixed it by delegating the page click event.
useEventListener('click', (e) => {
  const el = e.target as HTMLElement | undefined;
  const button = el?.closest?.<HTMLButtonElement>('button[data-slot="tab"]');

  if (button) {
    const key = button?.dataset.key as TabKey;

    setSelectedTab(key);
  }
});

I also have this issue sometimes with tap to click on macOS with Accordion.

Hey @u3u try again using this codesandbox: https://codesandbox.io/p/sandbox/dwvtlf?file=%2FApp.jsx&utm_medium=sandpack
It works fine for me btw: https://github.com/nextui-org/nextui/assets/30373425/6a6a22c2-c66d-4338-b993-000da735b496

I have the same issue with the provided code in codesandbox. Tapping does not effect a change from photos to music to videos.

I had the same issue when running playwright e2e tests - it cannot click the tabs. Adding the event listener as described by u3u fixed it.

I'm having this issue with Accordion, Dropdown menu (only the items, not the button), and Tabs.  The NextUI website itself has tons of tabs (Preview - Code) scattered throughout, and this issue keeps happening there as well.  It's inconsistent though, I'd say it fails to engage the tab (or accordion etc) about 70% of the time with a tap on the mac trackpad.  The fix above by u3u fixed it for me as well.  I suspect a problem with usePress.

The @u3u work around works good for single tabs usage, in my case i have two in the same page and the listener is called twice.
+1

I'm also having this issue - a light click on mac fails to register quite often. It makes the tabs borderline unusable.

Is this fixed in the latest version, @jrgarciadev ?

@aecsar In the version that is on CodeSandbox that @jrgarciadev created, the problem persists, the problem only occurs using the "Touch click" of a Mackbook.

@aecsar In the version that is on CodeSandbox that @jrgarciadev created, the problem persists, the problem only occurs using the "Touch click" of a Mackbook.

A lot of people just use a MacBook, without any mouse :) Me included. Even at home with a big screen. So you might have an idea how annoying this bug is — and it affects a lot of components (tabs, drop-downs, accordion etc). I wish I had time to dig into this, but in this case I need to get familiar with the whole codebase and this step pushes me back for now. Was hoping core devs have an idea :)
P.S. I had problems with my wrist while using a mouse a long time ago and it was gone when I switched to trackpads permanently.

Really hoping to see a fix in next versions. It's a very bad UX actually.

Not just bad UX, this is a blocker from adopting NextUI in our project.
@nextui-org/use-aria-press is handling mouse/touch events and doing event propagation for multiple components including Accordion. I wonder if the bug is here.

import { Tabs as NextTabs, TabsProps } from "@nextui-org/react";
import { ReactElement } from "react";

export const Tabs: <T = object>(props: TabsProps<T>) => ReactElement = (props) => {
  return (
    <NextTabs
      onClick={(e) => {
        const target = e.target as HTMLElement;
        const selected = target.parentElement!.getAttribute("data-selected");
        const key = target.parentElement!.getAttribute("data-key")! as string;
        if (!selected && key) {
          props.onSelectionChange!(key);
        }
      }}
      {...props}
    />
  );
};

Same issue here...

Confirming this issue still persists in the latest version in Dropdown and in Buttons.

Actually, for me it's mostly been issues with dropdown items, and I was able to get it to work much better by moving my logic to the onClick of each item instead of in onAction.

Also seeing this issue when using the Switch component.

Even though the website of NextUI says that the Tabs section is updated as of my writing, it still has bugs. I can't click  component when im using responsive design mode in firefox even when i tap lightly and hard. Please fix this issue. im 75% in my project and implemented a lot from NextUI.

im having te same issue. go here https://nextui.org/docs/components/tabs and if you are using web browser in pc/mac activate the reponsive mode and turn on the touch. i didnt work too. tabs dont get selected. how can we fix this?

Hey @u3u try again using this codesandbox: https://codesandbox.io/p/sandbox/dwvtlf?file=%2FApp.jsx&utm_medium=sandpack
It works fine for me btw: https://github.com/nextui-org/nextui/assets/30373425/6a6a22c2-c66d-4338-b993-000da735b496

The Code sandbox you provided and even the entire docs website does not work with a tap to click on the MacOS trackpad. It works if I double-tap or press to click. It used to work fine on a Windows laptop trackpad.
Also, the react-aria component works perfectly fine for me - https://react-spectrum.adobe.com/react-aria/Tabs.html

I found a workaround. It's a very hacky way to fix this, but it works... We need to use controlled tabs and the tab title must be the same as the tab key.
function TabsWrapper() {
	const [selectedTab, setSelectedTab] = useState('First');
	return (
		<Tabs
			aria-label='Tabs'
			selectedKey={selectedTab}
			// @ts-expect-error
			onClick={e => setSelectedTab(e.target.innerText)}
		>
			<Tab key='First' title='First'>
				First Tab Content
			</Tab>
			<Tab key='Second' title='Second'>
				Second Tab Content
			</Tab>
		</Tabs>
	);
}


Error on attempting services logs on TTY container

Signed-off-by: Drew Erny<EMAIL_ADDRESS>
Errors are better than undefined behavior.
- What I did
Error on attempting to do docker service logs on a service with TTY-attached containers.
Right now getting service logs from a container with an attached TTY does not work. The behavior was undefined and caused the command to hang and strange messages to occur in the daemon logs.
- How I did it
This returns errors, both deep in the swarmkit adapter (to guard against undefined behavior, which is Bad) and in the daemon (to tell users that the thing they're asking for is not possible) by checking if the requested service/container is TTY-attached and returning an error if it is.
- How to verify it
Create a service with a TTY attached, try to do logs, see how errors are returned
- A picture of a cute animal (not mandatory but encouraged)

Can you elaborate a bit on why retrieving logs from a service using a TTY involves undefined behavior, and what would be involved in fixing this?
I'm not familiar with this area but it seems like it shouldn't be inherently impossible to retrieve logs when the output is going to a TTY. There might be control characters in there, but that shouldn't be a fundamental problem. We should be escaping control characters in logs anyway, and if we aren't, that's a potential security bug because people could use them to forge or overwrite log messages (cc @diogomonica).
But if there's some design reason why we can't fetch logs when a TTY is in use, this might be the right course of action.

The ultimate reason is that I don't understand the implications of a TTY log, I don't know why they're treated differently (done with io.Copy intead of multiplexed through StdCopy) and the current behavior is broken.

Tentatively targeting 17.04.0

To be more clear, at the very least, logs with a TTY are not multiplexed with stdcopy, and so the code we have which does demultiplexing does binary math on some bytes that results in values that do not make sense, and cause issues.

When tty is enabled the data out of the container is on a single stream, there is no differentiator for stdout and stderr.

Ok, I understand that now. Deeper problem is that we're relying on the stdcopy format to provide us a message length, so that we can always find and parse the timestamp for each message line. Without that message length provided, we cannot reliably parse out the timestamp from the log lines.
Basically, we're relying on a broken, fragile API, which we're going to need to change down the road, and which excludes the possibility of doing TTY logs for now.

LGTM

Addressed concerns @aluzzardi @aaronlehmann