L515 map_device_descriptor Cannot open /dev/video Last Error No such device

Required Info

Camera Model
L515

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu18.04

Kernel Version (Linux Only)
5.4.0-113-generic

Platform
PC

SDK Version
2.50.0-0~realsense0.6127

Language
C++

Segment
Robot

Issue Description
I was trying to retrieve color image and imu data to do camera-imu calibration. I wrote my code based on TUM-basalt, you can find the code from here:
https://github.com/rancheng/basalt-mirror/blob/master/src/rs_l515_record.cpp
When run the program, it always stuck at first frame and prompt the following error:
 08/06 11:43:00,485 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,487 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,488 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,588 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,588 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,588 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,688 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,688 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,688 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,789 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,789 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,789 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,889 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,889 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,889 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,989 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,989 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,989 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,090 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,090 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,090 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,190 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory

I tried to use INFRARED only, and made sure the config for streams are all valid.
you can follow the basalt official build instruction and install gcc9 to build the project:
sudo apt-get update && apt-get install -y wget gnupg lsb-release software-properties-common && apt-get clean all
sudo apt-get update && sudo apt-get install -y wget gnupg lsb-release software-properties-common && apt-get clean all
sudo apt-get update && sudo apt-get install -y wget gnupg lsb-release software-properties-common
wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key| apt-key add -
sudo wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key| apt-key add -
sudo wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key| sudo apt-key add -
echo "deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-11 main" > /etc/apt/sources.list.d/llvm11.list
sudo echo "deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-11 main" > /etc/apt/sources.list.d/llvm11.list
sudo echo "deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-12 main" > /etc/apt/sources.list.d/llvm12.list
sudo echo "deb http://apt.llvm.org/bionic/llvm-toolchain-bionic-11 main" > /etc/apt/sources.list.d/llvm11.list
sudo echo "deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-11 main" >> /etc/apt/sources.list.d/llvm11.list
sudo gedit /etc/apt/sources.list.d/llvm11.list
sudo add-apt-repository ppa:ubuntu-toolchain-r/test
sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE
sudo add-apt-repository "deb https://librealsense.intel.com/Debian/apt-repo $(lsb_release -cs) main" -u
sudo apt-get install -y cmake git libtbb-dev libeigen3-dev libglew-dev ccache libjpeg-dev libpng-dev openssh-client liblz4-dev libbz2-dev libboost-regex-dev libboost-filesystem-dev libboost-date-time-dev libboost-program-options-dev libopencv-dev libpython2.7-dev libgtest-dev lsb-core gcovr ggcov lcov librealsense2-dev librealsense2-gl-dev librealsense2-dkms librealsense2-utils doxygen graphviz libsuitesparse-dev clang-11 clang-format-11 clang-tidy-11 clang-12 clang-format-12 clang-tidy-12 g++-9 libfmt-dev && apt-get clean all
sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-9 60 --slave /usr/bin/g++ g++ /usr/bin/g++-9
gcc --version
g++ --version

Then build the project
git clone --recursive https://github.com/rancheng/basalt-mirror.git
cd basalt-mirror
./scripts/install_deps.sh
mkdir build
cd build
cmake .. -DCMAKE_BUILD_TYPE=RelWithDebInfo
make -j8

To run the program please do the following:
basalt_rs_l515_record --dataset-path ~/l515_calib_data/ --manual-exposure

This is the full running log of the program and a gui attached:

/home/ran/CLionProjects/basalt-mirror/cmake-build-debug/basalt_rs_l515_record --dataset-path /home/ran/basalt_ds/ --manual-exposure
Sensor L500 Depth Sensor. Supported options:
	Visual Preset
	Frames Queue Size
	Error Polling Enabled
	Depth Units
	Inter Cam Sync Mode
	LDD temperature
	Mc Temperature
	Ma Temperature
	Global Time Enabled
	Apd Temperature
	Depth Offset
	Freefall Detection Enabled
	Sensor Mode
	Host Performance
	Humidity Temperature
	Enable Max Usable Range
	Alternate IR
	Noise Estimation
	Enable IR Reflectivity
	Digital Gain
	Laser Power
	Confidence Threshold
	Min Distance
	Receiver Gain
	Post Processing Sharpening
	Pre Processing Sharpening
	Noise Filtering
	Invalidation Bypass
Sensor RGB Camera. Supported options:
	Backlight Compensation
	Brightness
	Contrast
	Exposure
	Gain
	Hue
	Saturation
	Sharpness
	White Balance
	Enable Auto Exposure
	Enable Auto White Balance
	Frames Queue Size
	Power Line Frequency
	Auto Exposure Priority
	Global Time Enabled
	Host Performance
Sensor Motion Module. Supported options:
	Frames Queue Size
	Enable Motion Correction
	Global Time Enabled
Device Intel RealSense L515 connected
Enabling manual exposure control
got calib
Skipping gyro data. Timestamp before the first accel measurement.Skipping gyro data. Timestamp before the first accel measurement.Skipping gyro data. Timestamp before the first accel measurement.frame set size: 1
frame count: 1
Timestamp / exposure 0:<PHONE_NUMBER>00 / 499ms
done img data reading ...
 08/06 11:43:00,485 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,487 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,488 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,588 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,588 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,588 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,688 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,688 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,688 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,789 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,789 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,789 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,889 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,889 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,889 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,989 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,989 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:00,989 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,090 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,090 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,090 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such device
 08/06 11:43:01,190 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,190 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,190 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,290 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,290 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,290 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,390 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,390 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,390 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,490 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,490 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,490 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,590 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,591 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,591 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,691 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,691 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,691 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,791 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,791 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,791 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,891 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,891 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,891 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,991 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,991 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:01,991 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,091 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,091 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,091 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,191 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,191 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,191 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,292 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,292 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,292 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,392 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,392 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,392 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,492 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,492 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,492 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,592 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,592 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,592 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,692 ERROR [140013064918784] (librealsense-exception.h:52) map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,692 ERROR [140013064918784] (sensor.cpp:572) acquire_power failed: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory
 08/06 11:43:02,692 ERROR [140013064918784] (global_timestamp_reader.cpp:239) Error during time_diff_keeper polling: map_device_descriptor Cannot open '/dev/video0 Last Error: No such file or directory

Hi @rancheng  Could you please first try unplugging the micro-sized end of the USB cable from the base of your L515 camera, turning the connector around the opposite way and re-inserting it into the base of the camera (USB-C cables are two-way insertion at the micro-sized end).  Do you still experience the errors with the connector reversed, please?

I have also commented out the Pangolin side to diagnose the issue part.
The problem happens immediately after first color frame was read.

I also tried to do nothing in callback just print the strings it also prompt this error:
else if (frame.get_profile().stream_type() == RS2_STREAM_COLOR) {
      std::cout << "color frame ..." << std::endl;
      /*
      ...
      */
}

The strange thing is that I can access the exposure and sensor data from python script with pyrealsense2 library without any problem.
def callback(frame):
    global color_frame_data, depth_frame_data, gyro_frame_data, accel_frame_data
    pr = frame.get_profile()
    if pr.stream_type() == rs.stream.gyro:
        # print("got gyro frame ...")
        pass
    if pr.stream_type() == rs.stream.accel:
        # print("got accel frame ...")
        pass
    if pr.stream_type() == rs.stream.color:
        print("got color frame ...")
    if frame.is_frameset():
        frameset = frame.as_frameset()
        f1 = frameset.get_color_frame().as_video_frame()
        f2 = frameset.get_depth_frame().as_video_frame()
        color_data = np.asanyarray(f1.get_data())
        print("color_data.shape: {}".format(color_data.shape))
        depth_data = np.asanyarray(f2.get_data())
        print("depth_data.shape: {}".format(depth_data.shape))
        actual_exposure = f1.get_frame_metadata(rs.frame_metadata_value.actual_exposure) * 1e-6
        print("actual_exposure: {}".format(actual_exposure))

The output is expected:
color_data.shape: (720, 1280)
depth_data.shape: (480, 640)
actual_exposure: 0.149205
color_data.shape: (720, 1280)
depth_data.shape: (480, 640)
actual_exposure: 0.149205
color_data.shape: (720, 1280)
depth_data.shape: (480, 640)
actual_exposure: 0.149205
color_data.shape: (720, 1280)
depth_data.shape: (480, 640)
actual_exposure: 0.149205
color_data.shape: (720, 1280)
depth_data.shape: (480, 640)
actual_exposure: 0.149205

Hi @MartyG-RealSense , Thanks for your advice, I closed the depth and only enabled IMU and color sensors streams, it works!
Also I followed the advice from #issue50.
Removing the device.hardware_reset() line in my code seems to fix this bug.
I am going to close this issue as it is solved.
Thanks again @MartyG-RealSense !


MAINT: Update python-interface version

This update eliminates some DeprecationWarnings on newer versions of
Python.

@fwagner-quantopian Looks like python-interface pulled in funcsigs==1.0.2 and typing==3.6.2, do we want to pin those here as well?


Upgrade nodejs to 16

https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

Workflows using this action are now generating this warning:
The following actions uses node12 which is deprecated and will be forced to run on node16: r0adkll/sign-android-release@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
 ```


Added custom context name

Changes
Added a variable called cluster_context to replace hardcoded cluster context name
Linked Issues

#313

@vrsarin Please address the lint errors. Otherwise LGTM.

@vrsarin Please address the lint errors. Otherwise LGTM.

Fixed the linitng issues. Please approve.

@dereknola removed the line. Please approve.


Peggy can't parse superfluous semicolons

Steps to reproduce: Parse the following grammar:
Start = Part*; ;
Part = .

Expected result: It is a valid Peggy grammar, since the separation of its rules is not ambiguous.
Actual result: It is not a valid Peggy grammar.
This happens because Peggy does not accept duplicated semicolons. I suppose not fixing this could be justified with the reasoning that duplicated semicolons are almost certainly a mistake on the part of the programmer, but I wanted to at least point out this limitation.

This one I might be open to changing.  However, I want to make sure I can capture the extra semis in peggy-eslint-parser first.  Will leave this open to discussion.

After looking at peggy-eslint-parser, this is still an easy fix.  Including in v3.0.1.

Thanks for pointing this out!


Automatically determine num_actions and num_chance_outcomes in stochastic_muzero_policy

It is possible to automatically determine the num_actions and num_chance_outcomes parameters to stochastic_muzero_policy from the root, decision_recurrent_fn, and chance_recurrent_fn parameters, via ChanceRecurrentFnOutput.action_logits.shape and DecisionRecurrentFnOutput.chance_logits.shape. That would reduce the number of parameters that users need to pass in by two.
I suggest allowing num_actions and num_chance_outcomes to be None, making them None by default, and determining their values automatically if they're None. This preserves backward compatibility. Thoughts?

Thanks for the suggestions. It sounds good.
We do not need to preserve the backward compatibility with the None defaults. The stochastic_muzero_policy is still experimental.
Will you send a pull request or should I do that?

Thanks for the improvements.


ZTEST Docs Page

The Test Framework documents mix new and old APIs which is a bit confusing, it would be great to have a page that shows how to do things with samples and such with the new API, keeping the old docs as Legacy or something
https://docs.zephyrproject.org/latest/develop/test/ztest.html?highlight=tests

@yperess Feel free to assign to me unless you think you can get this done by end of next week.

this is completed, closing.


Catch up latest docs: from 5.1 to 5.4

This is an automatic catch up PR to align our non-translated documentation with the latest official documentation.

=> #220


Could not resolve project :react-native-gpay

Could not determine the dependencies of task ':app:compileDebugJavaWithJavac'.

Could not resolve all task dependencies for configuration ':app:debugCompileClasspath'.
Could not resolve project :react-native-gpay.
Required by:
project :app
Unable to find a matching configuration of project :react-native-gpay: None of the consumable configurations have attributes.

Same Here. @57912130  did you rectify it ??

同样在这里。@ 57912130 你纠正了吗？

Didn't solve

Please follow proper setup configuration step. You can't face this issue.

Please follow proper setup configuration step. You can't face this issue.

I follow all the steps and I ran into the same error!

Okay I'll check it again and update change publish very soon, i'll also update feature of AndroidX.
Thank You

Same issue. Any solution?

I get the same exact error using react-native: 0.58.6

Same issue here, using react-native: 0.59.5

AM also facing this issue, any help ?

same issue. Any update on this?

Same issue. Posted the question again because I didn't check "Closed" issues.
https://github.com/JadavChirag/react-native-GPay/issues/5#issue-489382312

Same here. Tested on React Native 0.58.6, can we get any update on why this is happening?

I get the same exact error using expo SDK33

Same issue. RN 0.61.5

Just landed into this same issue just now.
RN: 0.58.9
Did the Linking as specified on the Doc.
Also added the Meta tag on my AndroidManifest.
why is this beautifu module not working ?


Remove unnecessary javascript code and replace with hugo template code

Supersede #95

Replace JS with Hugo template code, this time animation is preserved when clicking navigation links ;-)

@susnux conflicts emerged after merging the CSS variables
Won't removal of  data-fnav-title somehow break links to specific sections? I'll have to try that

@susnux can you pull from master and resolve conflicts? Or should I have a look at that?

Won't removal of  data-fnav-title somehow break links to specific sections? We don't want to break #102

No links, like #foo work by using the id property and the cover buttons work by using the data-title-anchor prop, so I do not think this breaks anything.

@susnux can you pull from master and resolve conflicts?

Done :)

@susnux Thanks. Just one more thing I noticed: When I click the cover (landing area) button, the scrolling to content is now somewhat initiallly slow. At the beginning it hardly scrolls at all and then it speeds up a lot. Is there any way to optimize it for a more smooth transition from the cover image to the content sections?
You can compare
https://deploy-preview-132--hugo-scroll.netlify.app/
and the master branch
https://zjedi.github.io/hugo-scroll/#contact

@zjedi I think I resolved that issue now :)


Minimum time for deprecation cycles?

Feature request
This is more of a meta request, but I wanted to ask if it's possible to have longer deprecation cycles for API-breaking changes.
This request is motivated by the 0.50 release dropping the shim for decorators.  It's great that you communicated this clearly in the release notes and warnings, but as far as I can tell, the entire deprecation cycle here was less than two months (warning first appeared in 0.49.0 on 2020-04-16, breaking change implemented in 0.50.0 on 2020-06-10).
Since we're not on strict semantic versioning territory here, these kinds of changes are impossible to anticipate by matching version numbers, so the best we (as downstream package maintainers) can do is watch for warnings and react as quickly as possible.  The "as quickly as possible" is the catch here, since many of us are time- and resource-constrained, and can't operate on fast release schedules.
I don't have any strong recommendations here, but I wanted to raise the issue because it's causing a lot of trouble for end-users whose installations have suddenly broken because downstream maintainers could not react quickly enough.  In my specific case, this deprecation cycle landed on the busiest time of the year (end of the academic year), though I expect in general that downstream maintainers will have widely varying schedule conflicts.  Having a documented, longer cycle for handling deprecations (e.g., at least 6 months) could go a long way toward supporting downstream devs and end-users.

@bmcfee, sorry about the short deprecation cycle. Part of the reason that we moved so quickly is that we assumed downstream projects can pin to the older versions that they support. Would that be possible for your case?

Part of the reason that we moved so quickly is that we assumed downstream projects can pin to the older versions that they support. Would that be possible for your case?

Yeah, that makes sense, but as a library (not an end-user application), pinning to exact versions could easily result in unsatisfiable configurations.  Pinning to wildcards is possible, if you know what the wildcards should be in advance, but without semantic versioning that's impossible to predict.
I have been telling end-users to pin while we sort things out, but I don't think it's a feasible solution in general.

As another core developer, I'd also like to offer my apologies for this and the impact it had on librosa. We tried to contact all the projects that we knew about that were using internal APIs ahead of deciding how fast to move (https://github.com/numba/numba/issues/5214). The general feedback, from various user groups using Numba's internal APIs, was that they were tracking Numba latest release/Numba master branch. As a result it was decided that a quick deprecation cycle would be fine. This is the first time in about 5 years the project has seen a refactor of this scale, should this be needed again (and I really hope not for a long time!) I think doing a more typical length deprecation cycle (http://numba.pydata.org/numba-doc/latest/reference/deprecation.html, some are running for over a year!) would be something to consider.
I also think it was just a bit unfortunate that librosa was using something from Numba's internals, when it probably didn't need to be https://github.com/librosa/librosa/pull/1164, I hope that patch fixed the project and also makes it less susceptible to any future changes to Numba internals. If there's anything else we can do to help out with this please shout! Thanks :)

No worries, these things happen!  It's true that we never should have been using an internal API in the first place (and the usage had long been deprecated anyway), but thanks for the correction.

If there's anything else we can do to help out with this please shout! Thanks :)

Nope, I think we're all good!  No major harm done, I just wanted to raise the point that maybe a little documentation about expected cycle length could be helpful.  But it sounds like you're all well on top of that anyway, so I'm happy for this issue to be closed out.  Thanks again!


Error 500 when downloading deck as a "Text file" for certain decks

I get 500 when i try do download the following deck - https://netrunnerdb.com/en/decklist/54886/stabby-foxx-1st-at-bristol-sc as a "Text file".
I DO NOT get the error when doing the same for https://netrunnerdb.com/en/decklist/55245/stabby-maxx-4th-gent-gnk

checking out the log shows
Call to a member function getCards() on null {"exception":"[object] (Symfony\Component\Debug\Exception\FatalThrowableError(code: 0): Call to a member function getCards() on null at .../netrunnerdb/src/AppBundle/Controller/SocialController.php:681)"} []

Also happens for these decks if you save to your decks and try and "text file" them
https://netrunnerdb.com/en/decklist/54141/recurring-punishment-core-19-nisei-tournament-1st-place
https://netrunnerdb.com/en/decklist/54142/recursive-puncher-core19-nisei-tournament-1st-place

ok, it looks like there are 2 places this fails and it is the same error both times.  If the MWL version is not set for the deck this error will happen.
As a workaround until we push a patch you should be able to set the target MWL version and save the deck.

Also, for the record, "Casual Play" for the MWL setting is the same as not being set to the database (NULL).

Ok, we have a fix submitted for this.  We'll push it live after Euros are over.

There is a slightly different issue happening for the spags core tournament decks.
The error there looks like:
[2019-06-03 13:59:40] request.INFO: Matched route "decklist_export_text". {"route":"decklist_export_text","route_parameters":{"_controller":"AppBundle\Controller\SocialController::textexportAction","decklist_id":"54141","_route":"decklist_export_text"},"request_uri":"https://netrunnerdb.com/decklist/export/text/54141","method":"GET"} []
[2019-06-03 13:59:40] security.INFO: Populated the TokenStorage with an anonymous Token. [] []
[2019-06-03 13:59:40] php.CRITICAL: Call to undefined method AppBundle\Entity\Decklist::getDeckSize() {"exception":"[object] (Symfony\Component\Debug\Exception\FatalThrowableError(code: 0): Call to undefined method AppBundle\Entity\Decklist::getDeckSize() at /home/alsciende/netrunnerdb/src/AppBundle/Controller/SocialController.php:742)"} []
[2019-06-03 13:59:40] request.CRITICAL: Uncaught PHP Exception Symfony\Component\Debug\Exception\UndefinedMethodException: "Attempted to call an undefined method named "getDeckSize" of class "AppBundle\Entity\Decklist"." at /home/alsciende/netrunnerdb/src/AppBundle/Controller/SocialController.php line 742 {"exception":"[object] (Symfony\Component\Debug\Exception\UndefinedMethodException(code: 0): Attempted to call an undefined method named "getDeckSize" of class "AppBundle\Entity\Decklist". at /home/alsciende/netrunnerdb/src/AppBundle/Controller/SocialController.php:742)"} []
I'll patch this up and do a bit more testing tonight.  Thanks for bearing with us and for reporting issues!

ok, i found the issue with published decklist exports.


How to select only string values?

Hi!
Is it possible to get comma separated string values only?
for examle:
a,b,c,d
I need this to store into database SET field.

No, that is not possible. But you can easily create them yourself.
The value you get back is either an empty string (if nothing is selected), or an array with strings (the values of the selected options). You can easily transform the list of strings to a comma-separated string with the join function.
// let's say you have the value is stored in selectValue.

let commaSeparated = selectValue === '' ? '' : selectValue.join(',')


 I managed to get it running on the third or fourth try. Two discoveries:

The process crashes if the log directory doesn't exist, so we need to make sure we create it first
Sending Unicode input that can't be decoded crashes the server. (Yes, I ran Hypothesis against it to see what happens. I can send you the test if you're interested.) This patch catches the error, and just logs it rather than crashing – I'm unable to crash the server with Hypothesis now.

Thanks @alexwlchan! Looks good, will merge shortly :grinning:


Why is parcel-bundler included?

I cloned this starter & made several changes removing pieces I don't need.
As part of this process, I am uninstalling packages I don't use.
I can't find any reference to parcel-bundler outside of the package.lock & package.json.
Is it used for something I can't see or can it be safely removed.

Probably can be dropped - if you found some dead weight we'd definitely take a PR to remove it 👍

Ah, just saw the other issue about parcel bundler bringing a vulnerable dependency, I'll go ahead and push an update for that.


Defaulting the line number to UID rather than null

@manoj-fsbtech
Can you also check the unit tests, they are breaking now

Is there any update on when this branch will be merged & made available for general use. This null line issue is blocking me on the next phase of my automation & CI/CD integration

@Nemesischild mind raising a new PR so we can move forward with this?

Closing due to inactivity. Please re-raise the PR if there is still interest.


Fastembed hangs when used behind firewall even though local model is present and cache_dir is enforced

What
This is an issue which also affects the langchain FastEmbedEmbeddings abstraction.
Models can be used offline if the model is already present in cache_dir.  However, FastEmbedEmbeddings and the underlying TextEmbeddings still makes a network query even though the model files are already present. This causes it to hang and eventually timeout, making it unusable behind a firewall.
This behavior seems new, it did not happen with v1.2
Why does this happen
In short, its cause fastembed calls the huggingface library which makes an API call to check the repo first, before checking if models exist which causes this hang to occur.
Long version:
Im guessing the default is using onyx embedding, and in fastembed/text/onnx_embedding.py .
The init statement calls download model:
self._model_dir = self.download_model(self._model_description, self._cache_dir)
Onnx embedding inherits this method from TextEmbeddingBase which inherits from ModelManagement class in common/model_management.py,.
download_model calls download_files_from_huggingface with no other kwargs, which calls snapshot_download.
this is defined in huggingface_hub/_snapshot_download.py, which has these  Lines 189-191
 # if we have internet connection we retrieve the correct folder name from the huggingface api
    api = HfApi(library_name=library_name, library_version=library_version, user_agent=user_agent, endpoint=endpoint)
    repo_info = api.repo_info(repo_id=repo_id, repo_type=repo_type, revision=revision, token=token)
    assert repo_info.sha is not None, "Repo info returned from server must have a revision sha."

Proof
Tried it with internet disabled and even though local cache_dir is specified it still makes a network request

You can force local_files_only=True within  huggingface_hub/_snapshot_download.py and this error disappears.
How to Replicate
Have the models saved in BASE_PATH then run the following code. it will hang.
from langchain_community.embeddings import FastEmbedEmbeddings
user= ''
BASE_PATH = f"/home/{user}/.cache/torch/sentence_transformers/"
embeddings = FastEmbedEmbeddings(model_name="BAAI/bge-base-en-v1.5",cache_dir=BASE_PATH)

Suggested fix
Have fastembed check if the models are present before going further

However, I think it is still worth adding this parameter, thanks for pointing it out

Actually, if you don't have an internet connection, then it works, it raises errors only when you have SSLError or ProxyError
And SSLError is exactly what you have in your traceback
except (requests.exceptions.SSLError, requests.exceptions.ProxyError):
    # Actually raise for those subclasses of ConnectionError
    raise

If I shut down internet and run
from fastembed import TextEmbedding

embedding = TextEmbedding(model_name="BAAI/bge-small-en-v1.5", cache_dir='./_model_cache')
print(list(embedding.embed(["qwe"])))

then it works

Its not working for me. Its still trying to connect internet. and giving me HTTPConnectionError. I turned off internet.
And I believe it checks for model.onnx in ./_model_cache

Hi @visheshgitrepo
_model_cache is my own cache folder I am using in my environment, you need to set your own model or keep the default cache folder.
You need to run your code with an internet connection first (or put your files into the cache dir), and then you'll be able to use it without internet.
If you need more help, you can describe what you've done and share a code-snippet

Fastembed requires not only a model file, but also tokenizer path, and some other files, providing just a model file path won't work since it is not sufficient, you need to provide a dir path

Fixed by https://github.com/qdrant/fastembed/pull/223

Hey, I have an error today like this about SSL.
Is there a way to disable SSL check or how can I manually download the model to put it in the right folder ?
bm25 = SparseTextEmbedding(model_name="Qdrant/bm25")

discussed here https://discord.com/channels/907569970500743200/1260529217423147058


Move docker username to .env file

Done in https://github.com/Dankoy/jforwarder/commit/f44cafcae471f5ce29971c01772d5eff0b5a5eed


[DOC] Typos and unclear content

Description
The docs directory contains many rst files with typos and unclear content.
Screenshots

Additional information (CMS/Python/Django versions)

Do you want to help fix this documentation issue?

[ ] Yes, I want to help fix this issue and I will join the channel #documentation on our Discord Server to confirm with the team that a PR is welcome.
[ ] No, I only want to report the issue.

@sourcery-ai develop


LIVERY REQUEST: Philippine Airlines B787-9 Oneworld Star Alliance & Sky Team Livery

OneWorld Version

Skyteam Version

Star Alliance Version

Philippine Airlines isn’t in any airline alliance, so these liveries are kinda strange

SHUT UP!!!😡😡😡😡

SHUT UP!!!😡😡😡😡GITHUBUSER589!!!


[Bug]: Trying to sign up with a 64 character password fails

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Trying to register with a too-long password fails and you don't even get an indication about it.
Steps To Reproduce

Install Appsmith using the recommended docker-compose setup
Click the Get Started button
Enter your user information and make sure to use a 64 character password (well, or any password longer than 48 characters)
The page reloads, you temporarily end up on /setup/welcome?error=Password+length+should+be+between+6+and+48, but you don't see anything about this error on the page itself
A few seconds later, page reloads again and you're back on /setup/welcome

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Self Hosted - latest

we show an appropriate error now


BrowserAuthError Interaction in_progress on page idle

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.23.0
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
1.4.4
Public or Confidential Client?
Public
Description
Idling on our react browser for around 20-30 minute's causes MSAL to throw a BrowserAuthError saying that there is currently an interaction in progress. I've checked the code flow on our react app and it doesn't call the login flow at any point before this attempt to login, so I don't believe we're attempting a login and not redirecting the user.  Once we refresh the page it allows us to log in correctly.

Does the PublicClientApplication attempt a silent login after idling for a while? And is there a correct way to handle this error to reset the current login state?
MSAL Configuration
No response
Relevant Code Snippets
No response
Identity Provider
Azure B2C Custom Policy
Source
External (Customer)

@StevenMedeot Can you please share if you have collected any MSAL logs around the time where you see this? We do not trigger a login call afaik but logs can tell you if that is the case.

  However can you please confirm if this bit is stored in localStorage or sessionStorage? It is supposed to be in sessionStorage which is not shared across tabs.
Also if you are trying to create separate sessions in each tab, can you also confirm your cache isn't set to localStorage?


expand all by default

Does it possible to make it expand all by default?
Thanks

It would be great to have this feature - expand/collapse all nodes at initialization. Any ideas for a workaround until this is made available as a part of this plugin?

@simon1tan: It would be a JS nuisance if the tree is long and deep.

Had expected a solution to this enhancement instead of getting closed without a comment


Hotlink

Why is the hotlink and its js equivalent not working?

It was removed in v3, you can find downloads to the CSS here


feat: Add OpenTelemetry extension to allow auto-instrumentation

Summary
Add the open telemetry php extension via pecl
Use Cases
Enable observability for PHP apps built with the CNB
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Closes: #785

@sophiewigmore - I'm not sure how to set the labels 🤔

Hi @cedricziel thanks for the PR. This change seems reasonable, however you'll need to add it to the extension lists in the files here as well: https://github.com/paketo-buildpacks/php-dist/tree/f0fcf6a87f5304092f826e192a287951d5ed3684/dependency/actions/compile/extensions-manifests. Make sure that the extension version is compatible with the version of PHP that corresponds to the extension file

@sophiewigmore done 👍 ext-opentelemetry is compatible with php >= 8 so I added it to all 3 manifests.

@cedricziel cool! thanks. can you please edit the PR to point to this new branch I've made? https://github.com/paketo-buildpacks/php-dist/tree/opentelemetry
Since this affects the dependency compilation code, I want to push it to a branch on this repository so we can check it works in our dependency update Github action first. We can do that by triggering it off that branch and seeing all of our tests go green with the resulting dependencies

In thinking about doing that, I just realized we also test the compiled PHP dependency and check the extensions, so you may need to update the tests/fixtures here: https://github.com/paketo-buildpacks/php-dist/tree/95ca8b063cc3511e4dcb319d83dd802ec4997ef8/dependency/test

@sophiewigmore added them to the test apps and retargeted the branch

Testing here - https://github.com/paketo-buildpacks/php-dist/actions/runs/10852265782

@cedricziel re-testing here with an automation fix - https://github.com/paketo-buildpacks/php-dist/actions/runs/10905342607


fix: lock down the pipeline catalog

/retest

/test all

/retest

@jstrachan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
c9cf9efbd2df1de325c1e3b6800481a73d8d5220
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan 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

@jstrachan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
6d74fc25ff33f06ab375ed4774e5604a0b99b222
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
6d74fc25ff33f06ab375ed4774e5604a0b99b222
link
/test kube

gsm
6d74fc25ff33f06ab375ed4774e5604a0b99b222
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jstrachan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
6d74fc25ff33f06ab375ed4774e5604a0b99b222
link
/test kube

gsm
6d74fc25ff33f06ab375ed4774e5604a0b99b222
link
/test gsm

vault
6d74fc25ff33f06ab375ed4774e5604a0b99b222
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


add missing url

Describe your changes

Fixes Issue

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

Going to sleep 😴.


Cannot compile source in a subdirectory

I have a directory with iproj.json at the root.
{
    "version": "0.0.1",
    "description": "Compile tools",
    "objlib": "&BUILDLIB",
    "curlib": "&BUILDLIB"
}

Then I have a subdirectory called compile-tools with various source files and a Rules.mk.
C_STDAUT.CMD: c_stdaut.cmdsrc
C_STDAUT.CMD: private PGM := C_STDAUT
C_STDAUT.CMD: private HLPPNLGRP := C_STDAUT

If I set the current directory to the parent and execute a compile with either of
makei compile -f c_stdaut.cmdsrc -e BUILDLIB=AJENKS
makei compile -f compile-tools/c_stdaut.cmdsrc -e BUILDLIB=AJENKS
I get errors:
Set variable <BUILDLIB> to 'AJENKS'
Traceback (most recent call last):
  File "/QOpenSys/pkgs/bin/makei", line 272, in <module>
    cli()
  File "/QOpenSys/pkgs/bin/makei", line 49, in cli
    args.handle(args)
  File "/QOpenSys/pkgs/bin/makei", line 207, in handle_compile
    build_env = BuildEnv(targets, args.make_options, get_override_vars(args))
  File "/QOpenSys/pkgs/lib/bob/makei/build.py", line 66, in __init__
    self._create_build_vars()
  File "/QOpenSys/pkgs/lib/bob/makei/build.py", line 101, in _create_build_vars
    list(map(map_ibmi_json_var, subdirs))
  File "/QOpenSys/pkgs/lib/bob/makei/build.py", line 99, in map_ibmi_json_var
    dir_var_map[path] = IBMiJson.from_file(path / ".ibmi.json", dir_var_map[path.parents[0]])
KeyError: PosixPath('builds')

If I change into the subdirectory, the compile (in the first form) works.
Furthermore, I tried to compile the entire subdirectory using build from the parent directory.
makei build -d compile-tools -e BUILDLIB=AJENKS
...and this failed...
Set variable <BUILDLIB> to 'AJENKS'
Traceback (most recent call last):
  File "/QOpenSys/pkgs/bin/makei", line 272, in <module>
    cli()
  File "/QOpenSys/pkgs/bin/makei", line 49, in cli
    args.handle(args)
  File "/QOpenSys/pkgs/bin/makei", line 224, in handle_build
    build_env = BuildEnv([target], args.make_options, get_override_vars(args))
  File "/QOpenSys/pkgs/lib/bob/makei/build.py", line 66, in __init__
    self._create_build_vars()
  File "/QOpenSys/pkgs/lib/bob/makei/build.py", line 101, in _create_build_vars
    list(map(map_ibmi_json_var, subdirs))
  File "/QOpenSys/pkgs/lib/bob/makei/build.py", line 99, in map_ibmi_json_var
    dir_var_map[path] = IBMiJson.from_file(path / ".ibmi.json", dir_var_map[path.parents[0]])
KeyError: PosixPath('builds')

Similarly, I tried the identical (according to documentation) compile variation.
makei compile -f compile-tools -e BUILDLIB=AJENKS
...and this fails, but in a different way.
Set variable <BUILDLIB> to 'AJENKS'
Traceback (most recent call last):
  File "/QOpenSys/pkgs/bin/makei", line 272, in <module>
    cli()
  File "/QOpenSys/pkgs/bin/makei", line 49, in cli
    args.handle(args)
  File "/QOpenSys/pkgs/bin/makei", line 206, in handle_compile
    targets = get_compile_targets_from_filenames(filenames)
  File "/QOpenSys/pkgs/lib/bob/makei/utils.py", line 228, in get_compile_targets_from_filenames
    result.append(get_target_from_filename(filename))
  File "/QOpenSys/pkgs/lib/bob/makei/utils.py", line 211, in get_target_from_filename
    name, _, ext, _ = decompose_filename(filename)
  File "/QOpenSys/pkgs/lib/bob/makei/utils.py", line 185, in decompose_filename
    raise ValueError(f"Cannot decomposite filename: {filename} as {ext} is not a recognized file extension")
ValueError: Cannot decomposite filename: compile-tools as COMPILE-TOOLS is not a recognized file extension

I'm trying to find a workable set of processes to develop small utilities without needing a git repository for each one (many of which will just be one or two source files). Splitting into subdirectories seems the logical way to go, but I am fighting at every turn.
bob has specific support for subdirectories, but it seems they don't work very well except in a full build (though I have not tested that!)

Do you also have a Rules.mk file in your top level directory.  This file should list all the sub directories that have source needing compiled.

Thanks, I created a root Rules.mk
SUBDIRS = compile-tools

However, both commands fail in the same ways still.

An update. I wondered whether a fresh deploy directory might make a difference. It does. There must have been something in the prior one that was upsetting things. I now have a working compile.


Use editor selection when Navie is invoked from the tool window

:tada: This issue has been resolved in version 0.66.0 :tada:
The release is available on:

v0.66.0
GitHub release

Your semantic-release bot :package::rocket:


Struct fields are not enforced

Hi, thanks for this library. 🙂
It works well but it does not enforce fields that are declared in the struct to be enforced.
Imagine this struct:
defmodule Person do
  @enforce_keys [:name]
  defstruct name: nil,
            age: nil,
            happy?: true,
            phone: nil

  @type t() :: %__MODULE__{
          name: String.t(),
          age: non_neg_integer() | nil,
          happy?: boolean(),
          phone: String.t() | nil
        }
end

Now if you evaluate
data = %{"age" => 42, "happy?" => true, "phone" => "(206) 342-8631"}
Forma.parse(data, Person)

you get
{:ok, %Person{age: 42, happy?: true, name: nil, phone: "(206) 342-8631"}}

Notice that the call was successful and :name is set to nil even though it is in @enforced_keys and its type is not nullable.
Instead, if you evaluate %Person{age: 42, happy?: true, phone: "(206) 342-8631"}, you get this error: (ArgumentError) the following keys must also be given when building struct Person: [:name].
I think Forma should enforce :name being present in data as well. What do you think?
I think it could be achieved by changing the struct clause for Forma.Parser.parse!/3 from this:
  def parse!(input, parsers, {:struct, name, fields}) do
    case input do
      input when is_map(input) -> map_exact_fields!(input, parsers, struct(name), fields)
      _ -> raise "not a map #{inspect(input)}"
    end
  end

to something like this:
  def parse!(input, parsers, {:struct, name, fields}) do
    case input do
      input when is_map(input) -> map_exact_fields!(input, parsers, %{}, fields) |> then(&struct!(name, &1))
      _ -> raise "not a map #{inspect(input)}"
    end
  end

Hi @xxdavid!
I agree that would be nice indeed.
We're not really using this library (nor Elixir) anymore so it's unlikely that I'll have time to fix it, but PRs and contributions are very welcome! 🙏

Hi @frekw, thanks for replying even though you aren't interested in this library anymore (unfortunately, such behaviour is not very common).
If I find some time, I might come up with a PR. 🙂


Testing how this works by adding a meme

Testing how robust the script is by merging a PR with a file containing spaces.


Client-side validator not registered when using inheritance and/or generics

System Details

FluentValidation version: 8.2.2
Web Framework version: ASP.NET Core 2.1

Issue Description
Hi again (see #1087)
I am having some trouble registering client-side validators. This time I am trying to create a custom validator. This is my client-side validator:
public class RequiredIfClientValidator : ClientValidatorBase
{
    public RequiredIfClientValidator(PropertyRule rule, IPropertyValidator validator) : base(rule, validator)
    {
    }

    public override void AddValidation(ClientModelValidationContext context)
    {
        if (context == null)
        {
            throw new ArgumentNullException(nameof(context));
        }

        context.Attributes.Add("data-val", "true");
        context.Attributes.Add("requiredif", "requiredif");
    }
}

I would like to create a property validator related to this client-side validator. Let's start with a simple working example.
public class MyViewModel
{
    public string MyProperty { get; set; }
}

public class RequiredIfValidator : PropertyValidator
{
    public RequiredIfValidator() : base("Message") { }

    protected override bool IsValid(PropertyValidatorContext context)
    {
        return true;
    }
}

public class MyViewModelValidator : AbstractValidator<MyViewModel>
{
    public MyViewModelValidator()
    {
        this.RuleFor(x => x.MyProperty).SetValidator(new RequiredIfValidator());
    }
}

// Startup
fv.ConfigureClientsideValidation(x =>
{
    x.Add(typeof(RequiredIfValidator), (context, rule, validator) => new RequiredIfClientValidator(rule, validator));
}

I put a breakpoint inside RequiredIfClientValidator.AddValidation, it got hit and client-side validation was generated.
My design is more complex, though. I need view model inheritance, so I set up the following:
public class MyViewModelBase
{
    public string MyPropertyBase { get; set; }
}

public class MyViewModelDerived
{
    public string MyPropertyDerived { get; set }
}

public class RequiredIfValidator : PropertyValidator
{
    public RequiredIfValidator() : base("Message") { }

    protected override bool IsValid(PropertyValidatorContext context)
    {
        return true;
    }
}

public class MyViewModelBaseValidator<T> : AbstractValidator<T> where T : MyViewModelBase
{
    public MyViewModelBaseValidator()
    {
        this.RuleFor(x => x.MyPropertyBase).SetValidator(new RequiredIfValidator());
    }
}

public class MyViewModelDerivedValidator : MyViewModelBaseValidator<MyViewModelDerived>
{
    public MyViewModelDerivedValidator()
    {
        this.RuleFor(x => x.MyPropertyDerived).NotEmpty();
    }
}

// Startup
fv.ConfigureClientsideValidation(x =>
{
    x.Add(typeof(RequiredIfValidator), (context, rule, validator) => new RequiredIfClientValidator(rule, validator));
}

This doesn't work. RequiredIfClientValidator.AddValidation method is not called by the framework. The breakpoint is not hit.
What's more, I would like to make my RequiredIfValidator generic, like so (I am omitting boilerplate code, focusing on generics):
public class RequiredIfValidator<TModel, TProperty> : PropertyValidator
{
    public RequiredIfValidator(Func<TModel, TProperty> condition) : base("Message") { }

    protected override bool IsValid(PropertyValidatorContext context)
    {
        return true;
    }
}

This will of course require proper set up in my validator, which I will omit. This is the new code in Startup:
fv.ConfigureClientsideValidation(x =>
{
    x.Add(typeof(RequiredIfValidator<,>), (context, rule, validator) => new RequiredIfClientValidator(rule, validator));
}

This obviously doesn't work when I have view model inheritance because inheritance without generics didn't work either. The interesting thing is that I experience the problem of my breakpoint not being hit even when I use the generic validator without inheritance.
All in all, I think I have two issues here:

Client-side validators for generic validators not being registered.
Client-side validators for validators used on base-class properties not being registered.

Client-side validators for generic validators not being registered

That's correct. Internally, the match is done by using type.IsAssignableFrom which will of course fail for an open generic type. You'll need to create a non-generic interface, implement that on your  RequiredIfValidator and then register with the interface type as the key instead.

Client-side validators for validators used on base-class properties not being registered

As far as I can see there is nothing that would prevent a property validator defined in a base class from being included for clientside validation. Can I suggest you check that again once you've solved your generics issue? If you're still having issues you'll need to send me a small, minimal sample project that reproduces it.

Thank you for a quick reply :-)
I solved the generics issue using a marker interface, just as you suggested. Unfortunately the problem with client-side validation for base class properties still exists. To clarify, the constructor of RequiredIfValidator is getting called, but for RequiredIfClientValidator, neither the constructor not AddValidation are executed.

Please put together a sample project that reproduces the issue. Thanks.

Closing as there's been no further resposne. If you still want me to look into this further then please provide a sample project that reproduces teh issue.

Hi @JeremySkinner, I'm sorry for the late reply. It turns out that generating client-side validation works for base class properties. There was another issue that prevented this from working. Thanks for all the help!


Add axis validations for tf.experimental.numpy.stack

Added validation to check axis is within the range [-Rank(array),Rank(array)) as per the issue #55217

@rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain Can you please review this PR ? Thank you!

@Yashashree304 Can you please check @JW1992's comments and keep us posted ? Thank you!

@Yashashree304 , Could you please refer to the above comment and do needful corrections in order to get PR approved. Thanks!

Hi @Yashashree304 Can you please resolve conflicts? Thank you!

Hi @Yashashree304 Can you please resolve conflicts? Thank you!


v21.1.0-beta.4 release notes

Fixes #8938

This change is

Nice - just had a few grammar/style nits.

I think I got all of these.

Hey @ianjevans, apologies for the late notice on this but the SHA for this release is being updated to abc4eb5a. Please include the additional notes whenever you get the chance. Thank you!


(wip)tests: add univ3 svg generation test

Time spent on this PR: 1d
Adds Mock UniswapV3 contracts that allow querying a dynamically generated on-chain SVG representing a position.
The values are mocked, and due to some issues, the base64 encoding of the image crashes with a stack underflow error (tbd).
However, the encoding of the rest takes ~27M steps
Also: Because NFTDescriptor is a library with an external function, it must be deployed and called with its address. When referring to this library, the compiler will add a placehold in form __$HASH_OF_PATH$__ in the compiled bytecode. Thus, I implemented a way of linking a library in the deployments scripts - we can deploy the contract providing a set of required libraries, that are deployed, hashed, and their address replaces the hash in the compiled bytecode.
Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Resolves #
What is the new behavior?

This change is

good to merge, can ignore the trunk check


Authentication support

First version with:

Login form processing (manual for now since https://jira.spring.io/browse/SPR-15144 is not fixed yet)
Password hash stored in Mongo User JSON
WebSession based authentication

[ ] authenth via Twitter
[ ] authenth via Github
[ ] authenth via Google
[ ] old account importation

We have a simple process without oauth. A token is send by email to the user. If we want to do better we will open a new issue


Combine support!

Fixes #1871

Warnings

:warning:

Any changes to library code should be reflected in the Changelog.
Please consider adding a note there and adhere to the Changelog Guidelines.

:warning:
PR is classed as Work in Progress

:warning:
The library files were changed, but the tests remained unmodified. Consider updating or adding to the tests to match the library changes.

Messages

:book:
iOS: 	 Executed 278 tests, with 0 failures (0 unexpected) in 13.561 (13.712) seconds

:book:
tvOS: 	 Executed 278 tests, with 0 failures (0 unexpected) in 13.782 (13.936) seconds

:book:
macOS: 	 Executed 278 tests, with 0 failures (0 unexpected) in 14.597 (14.729) seconds

  Generated by :no_entry_sign: Danger Swift against b8414db82863e4f542ac2b5c6422858eb3728b72

Codecov Report

Merging #1904 into development will increase coverage by 1.18%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development    #1904      +/-   ##
===============================================
+ Coverage        92.48%   93.66%   +1.18%
===============================================
  Files               26       26
  Lines              932      932
===============================================
+ Hits               862      873      +11
+ Misses              70       59      -11

Impacted Files
Coverage Δ

Sources/Moya/Response.swift
95.4% <0%> (+4.59%)
:arrow_up:

Sources/RxMoya/Observable+Response.swift
61.53% <0%> (+7.69%)
:arrow_up:

Sources/ReactiveMoya/SignalProducer+Response.swift
93.93% <0%> (+9.09%)
:arrow_up:

Sources/RxMoya/Single+Response.swift
100% <0%> (+12.5%)
:arrow_up:

Continue to review full report at Codecov.

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

@Moya/contributors anyone able to make a review for this one? 🙏

Seems like tests on macOS are crashing because the CircleCI environment has macOS Mojave, not Catalina? There is this @available(OSX 10.15...) so it shouldn't run but let me check this one.

Okay it built, but Danger seems to have troubles (currently xcode-summary plugin). cc @f-meloni

:O today I will have a look

@f-meloni no rush!

Could be Xcode 11.0 the problem 🤔

Found it!
<unknown>:0: error: invalid value '5.1' in '-swift-version 5.1'
<unknown>:0: note: valid arguments to '-swift-version' are '4', '4.2', '5'

Marathon sets 5.1 as version, but apparently that is not recognised as valid argument from swiftc yet.
Now this is the problem... Will have a thinking on how solve it, before proposing SPM + Rocket, that is a possibility and would remove the need of using Marathon

@f-meloni I think 5.1 is just specified as 5 but depending on Xcode it will use either 5.0 or 5.1 so they probably won't add 5.1 as a valid argument. 5.1 seems to be valid argument only for SPM tools version for the moment.
I'm gonna merge this PR for now so we have one valid cache for the PRs. If anyone finds something in this PR don't hesitate to comment/open a PR/open an issue!

@sunshinejr It accepts  '4', '4.2', then I would expect it to accept '5', '5.1' too :thinking:

@f-meloni but 4.1 was still 4 in terms of swift version, I think the 4.2 is because there was a lot of breaking changes and it should’ve been named 5.0 but they couldnt’t because the API wasn’t stable. So in a sense these are all just major releases.

@sunshinejr Yeah you probably are right!

Added a fix to Marathon that should solve the problem
https://github.com/JohnSundell/Marathon/pull/209

@sunshinejr is there any extra steps requires to see it when you do pod install of 14.0.0-beta.3 version?  those classes not exposed that introduced in the app.


Racing issue with NFS file backend

Hi, First I'd like to thank you guys creating jug, which is very convenient to use for embarrassingly parallel problems.
However I have found a bug using jug on a cluster (PBS). I used file backend and I did find racing problems. The simple task I used for testing is this:
#====================
myID = 'procid.%s.%s' % (str(os.getpid()) , os.environ['HOSTNAME'])
def createdir(d):
if not os.path.isdir(d):
os.mkdir(d)
os.mkdir('%s/%s' % (d,myID))
else:
os.mkdir(d + '.' + myID)
@TaskGenerator
def runjob(cmd):
hash = hashlib.md5(str(cmd)).hexdigest()
createdir(hash)
cmds = [c.strip() for  c in open(joblistfile,'rt').read().splitlines() if c.strip() and not c.strip().startswith('#')]
for cmd in range(1024):
runjob(str(cmd))
#==============================
I submitted two instances in two different nodes.
I found in the directory created by the task, there were dir names with ".procid." which means racing condition did occur.
The reason I think is "O_EXCL is broken on NFS file systems," as described by GNU/Linux open(2) manpage. I would suggest using flufl.lock, which is safe over nfs:
http://pythonhosted.org/flufl.lock/docs/using.html
Thank you very much for your attention to the problem.
-Zheng Qu

Jug is now more careful about calling fsync() in all the right places, so this (and other) issues is now probably no longer relevant. Please open a new bug report (with a newer version of jug) if you encounter any recent problems.


(doc) Add documentation for Commercial Validation Rule

This is basic documentation for the upcoming rule that validates if packages make use of commercial cmdlets in their automation scripts or not.
The cmdlets is not called out specifically as these can change between versions.
ref: https://github.com/chocolatey/home/issues/76

Awesome work @AdmiringWorm! Thanks for your contribution!


Advise to overwrite additional_dependencies

The hook can still run with the latest version of whichever external tool it uses, but we should probably advise to pin the additional requirement

Codecov Report

Merging #430 (2ccde64) into master (ad47257) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #430   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          607       607
=========================================
  Hits           607       607

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 ad47257...2ccde64. Read the comment docs.

Hmmm maybe mention that this leads to the best reproducibility while using the unpinned version uses the most recent version but might lead to different versions used.
To ensure the most recent versions you need to use pre-commit clean to delete the cached envs and the next time pre-commit runs it reinstalls them.

Thanks - I've added a note about how this leads to best reproducibility.

To ensure the most recent versions you need to use pre-commit clean to delete the cached envs and the next time pre-commit runs it reinstalls them.

Not sure we need to document this if all the examples we show have pinned dependencies (after all, that's how people usually use tools, by copy-and-pasting from READMEs / docs)

merging, if anything else needs adding it can be done later


http2: Flip the upstream H2 frame flood and abuse checks to ON by default

Additional Description:
This PR switches the upstream flood and abuse checks to on by default. It can be disabled through a runtime flag.
Risk Level: High (upstream H2 codec behavior)
Testing: Unit, Integration tests
Docs Changes: N/A
Release Notes: Yes
Platform Specific Features: N/A
Runtime guard: envoy.reloadable_features.upstream_http2_flood_checks
Fixes #12281
Signed-off-by: Yan Avlasov<EMAIL_ADDRESS>

I agree with not wanting to merge high-risk right now. But another factor is that the 1.17 release is scheduled for 12/31, so if we want this to be included in 1.17, it should probably go in now instead of right before we cut the release (which I'm sure won't actually happen on NYE; first or second week of january is likely).

Yeah, 1.17 is my point - if we cut the release in the first week (which per
our release guidance we're supposed to try harder to do than we used to =P)
I think it wouldn't get burn time in prod.
So do we want it in (high risk, but runtime flippable) or wait until after
the release?
On Thu, Dec 17, 2020 at 12:50 PM Greg Greenway<EMAIL_ADDRESS>wrote:

I agree with not wanting to merge high-risk right now. But another factor
is that the 1.17 release is scheduled for 12/31, so if we want this to be
included in 1.17, it should probably go in now instead of right before we
cut the release (which I'm sure won't actually happen on NYE; first or
second week of january is likely).
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/14443#issuecomment-747596705,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AELALPJTFOPETBRUVDYSS43SVJAGVANCNFSM4U6MNSQQ
.

I would vote for holding this until after the release, but I don't feel strongly about it if there is a strong interest in merging now.

I'd say wait until after the release. Even if it went in now, but I bet very few people are about to run master in production over a long break, so it won't have much burn-in time before the release.


Web surfer enhancements.

Why are these changes needed?
Enhancements to WebSurfer including ability to register custom page-to-text renderers, and built-in support for YouTube (including transcripts).
Used as the basis for web surfing in the complex_tasks branch.
Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: 217 lines in your changes are missing coverage. Please review.

Comparison is base (d01063d) 37.79% compared to head (44f1618) 19.76%.

Files
Patch %
Lines

autogen/browser_utils.py
0.00%
216 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1650       +/-   ##
===========================================
- Coverage   37.79%   19.76%   -18.04%
===========================================
  Files          50       50
  Lines        5704     5860      +156
  Branches     1295     1428      +133
===========================================
- Hits         2156     1158      -998
- Misses       3377     4552     +1175
+ Partials      171      150       -21

Flag
Coverage Δ

unittests
19.76% <0.00%> (-18.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Endpoints requiring authZ serve HTTP 302 instead of JSON response when user is not logged in

Steps to reproduce:
@Endpoint
public class RandomEndpoint {
	@PermitAll
	public String permitAllEndpoint() {
		return "permitAll";
	}
}

Now, calling RandomEndpoint.permitAllEndpoint(), while being non-logged in, results in HTTP 302: redirect to /login. This causes SyntaxError: Unexpected token < in JSON at position 0 on the client side, instead of proper Error: Endpoint 'RandomEndpoint' method 'permitAllEndpoint' request cannot be accessed, reason: 'Access denied'

Could change the Spring security config super class so that for unauthenticated endpoint calls,don't redirect to login, but respond with 401.

duplicate to https://github.com/vaadin/spring/pull/925


Create winsrv

@robinharwood
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @KhallsaBraim, unless I'm mistaken this PR seems to contain a new file with only a single character. I'm guessing something isn't right with the PR, closing it for the moment. Please can you reopen a new PR with just the commit you wish to submit. You can use our contributor guide if you need further guidance.
Thanks,
Robin


11331.1 Admiral Add ircCreds systemIntegrations to UI

Shippable/pm#11331
Update the Admiral UI to display the ircCreds integrations and allow the integrations to be deleted and new ones added.

Verified.


Why extends Node?

This is more like a question rather than an actual issue. Is there any particular reason the script extends Node? Correct me if I'm wrong please, but wouldn't it be enough to just not extend anything at all (which implicitly extends Object)?
It's for sure not a performance killer or something. Though I think, where we don't need any node features it makes more sense to just not extend it, right?
Long story short, it might be no problem to simply delete line 3 from the script.
By the way, thanks for the implementation. I am considering using it for a tool I'd like to build in combination with an SQLite database. 😘

You can add a node to a scene tree but not an object.
So for those who wants to use it as a global instance (singleton) they can.

Okay, yes. I just also played around with it a bit (before seeing your reply). It totally works fine when omitting that extends Node line from the script if I (pre)load it like explained in the README:
const uuid_util = preload('res://uuid.gd')
# everything fine in case of uuid.gd doesn't extend Node

But when loading the script as a singleton via the Autoload feature in the project settings (as you just said) then it doesn't work. In that case it only does work when extending it from Node. Makes totally sense.
And I can understand that people prefer to add the script to the autoloaded classes. So, everything fine. Thank you for your reply.
Let me close that issue. Thanks again. ❤


Add 4.0 development changelog

This changelog is missing some changes that were made, but it contains most essential items already.
This changelog intentionally does not include:

Changes backported to Godot 3.3 and 3.4 (these are listed in the current 3.x changelog already).
Changes backported to the 3.x branch (which will become Godot 3.5 in the future).
Future changes that will be made after 4.0alpha1's release.

Thanks!


Replace clip.test.js with new tests

Transforms the old rendering test for layer clipping to the new rendering test approach.

Adapted the code and force pushed. Seems to be good now.


Fix warning from JRuby

Thanks for this super handy little gem.  Here's a quick fix for a warning JRuby throws.
Here's what the warning looks like :
jruby-<IP_ADDRESS> :003 > require 'php_serialize'
/home/nehresman/.rvm/gems/jruby-<IP_ADDRESS>@sentry_adt_poc/gems/php_serialize-1.2/lib/php_serialize.rb:102: warning: `:' after local variable or literal is interpreted as binary operator
/home/nehresman/.rvm/gems/jruby-<IP_ADDRESS>@sentry_adt_poc/gems/php_serialize-1.2/lib/php_serialize.rb:102: warning: even though it seems like symbol literal

Awesome, thanks for the fix!

@jqr Any chance you would have time to release a new version of the gem with this fix?  It would remove this warning from all my users.

I would love to, but this is a fork-with-improvements and I don't actually have author status on the rubygems.org version.

In the mean time you can work around this how most people are using bundler to get the gem directly out of this repo.
Alternatively: it's probably time to rename this gem and release a new version :)


Support linux

Please support linux to install agora sdk
i have this issue: #9

Hi @SeyyedKomeil. Thanks for your request. Due to the difficulty mentioned in #1, I can not tell you an accurate time we support Linux. For any follow-up discussion, please turn to #1.


routing

indirect paths, eg. with /foo/../page/
static files should be accessible only with /pub/
normal pages should be accessible only with /page/{page}, ie. not with /{page}
rethink handling of special branches, ie. /dev/, /page/, /pub/, and special pages, ie. /about, /dev, /pages

all but the first 2 points fixed.
the second was actually an instance of the first.
indirect paths are tolerated, as other sites seem to allow them as well, and are perhaps a client-side concern that has no bearing on server-side routing semantics once the indirection is resolved.
requests outside the scope seem to get dropped (eg. root/..).


[TC-470] Profile&parameters Tenancy

We have recently added "tenancy" to the project.
With tenancy, every resource have a tenant, where resource can be a delivery-service, a server (future) and even a user.
We are now starting to enforce access-control based on the resource tenancy. A user can manage a resource only if the resource is under the user tenancy.
This JIRA deals with  "profiles&parameters as a resource" - adding the tenancy to these objects and enforcing access control for their management. It is required for completeing the DS tenancy phase, as the DS points to a profile that we would not like out of tenant users to modify.
We still need to finalize the specification of this access control, and it would not be included in the first phase of tenancy introduction.
Author: Nir Sopher
JIRA Link: https://issues.apache.org/jira/browse/TC-470

We can potentially extend tenancy to profiles of type=DS profiles but extending it to all profiles/parameters goes against the design of TC where servers/profiles/parameters are intended to be used across the entire system. thoughts @nir-sopher ?

Hi Jeremy,
This is indeed a delicate issue.
The final target I had in mind is that everything has tenancy. There are
tenants that deals with DSes and Tenants that deal with infra. For now the
infra does not need tenancy.
I would try to have 2 different type of profiles/parameters: ds and infra.
DS profiles/parameters tenancy is enforced while infra profiles/parameters
tenancy isn't.
On Tue, Nov 17, 2020, 18:35 Jeremy Mitchell<EMAIL_ADDRESS>wrote:

We can potentially extend tenancy to profiles of type=DS profiles but
extending it to all profiles/parameters goes against the design of TC where
servers/profiles/parameters are intended to be used across the entire
system. thoughts @nir-sopher https://github.com/nir-sopher ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/trafficcontrol/issues/935#issuecomment-729047734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFY2IYEKKAIDJKNQL6BE6RTSQKQ3RANCNFSM4DYXOXPQ
.


Im trying to use grammy on my next.js project but whatever i do i get this

`npx ts-node scripts/build.ts
TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for /Users/mateokappa/Desktop/www/scripts/build.ts
at Object.getFileProtocolModuleFormat [as file:] (node:internal/modules/esm/get_format:160:9)
at defaultGetFormat (node:internal/modules/esm/get_format:203:36)
at defaultLoad (node:internal/modules/esm/load:141:22)
at async nextLoad (node:internal/modules/esm/hooks:865:22)
at async nextLoad (node:internal/modules/esm/hooks:865:22)
at async Hooks.load (node:internal/modules/esm/hooks:448:20)
at async MessagePort.handleMessage (node:internal/modules/esm/worker:196:18) {
code: 'ERR_UNKNOWN_FILE_EXTENSION'
}`

How is this related to grammY?

The build script is from the next.js example from Grammy and I can't run it on my next JS project

AFAIK, you don't have to run this command anyway. It will be run automatically during build.
@PonomareVlad is there a way to run these things locally? Should we document it, given that people try it anyway?

It is also failing on vercel , basically I'm trying to fix it locally so it fixes while running on vercel as well

ts-node doesn't support .ts extension ? 🌚

I had the same error, and switched from ts-node to tsx.
But if you want to use ts-node, see this issue: https://github.com/TypeStrong/ts-node/issues/2100


Support overriding the next_backoff value between retries with an Error.

Some protocols return a value when a request might be served after they return a 429 Too Many Requests response. This is effectively a transient error where we know the exact time when we should retry our request.
It would be nice if we could check our response and if it contains such info override the next_backoff value with a backoff error, for example:
Error::TooManyRequests(e, Duration::from_secs(the_suggested_value))

Hello @poljar, thanks for suggesting this idea. This could be part of a 0.4.0 release.


ESM + CommonJS

This library should support both, ideally written as ESM

This isn't important since using this library as an imported module would be a super rare use case. As long as Node supports cjs, this lib can be cjs.


[AIRFLOW-6105] [AIP-21] Rename Bigtable operators

Rename Bigtable operators
Jira

[ ] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-6105
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.
In case you are proposing a fundamental code change, you need to create an Airflow Improvement Proposal (AIP).
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does
If you implement backwards incompatible changes, please leave a note in the Updating.md so we can assign it to a appropriate release

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1d5b850). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #553   +/-   ##
=========================================
  Coverage          ?   84.22%
=========================================
  Files             ?      680
  Lines             ?    38425
  Branches          ?        0
=========================================
  Hits              ?    32364
  Misses            ?     6061
  Partials          ?        0

Impacted Files
Coverage Δ

airflow/contrib/operators/gcp_bigtable_operator.py
100% <100%> (ø)

airflow/gcp/example_dags/example_bigtable.py
100% <100%> (ø)

airflow/gcp/operators/bigtable.py
100% <100%> (ø)

airflow/gcp/sensors/bigtable.py
100% <100%> (ø)

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 1d5b850...c6f8e60. Read the comment docs.

Done?


FunctionBody: fix compiler error when doing implicit cast

Clang is not happy about implicit casts to void*:
FunctionBody.h:1482:33: error: assigning to 'void *' from incompatible type 'Js::JavascriptMethod' (aka 'void ()(Js::RecyclableObject *, Js::CallInfo, ...)')
entryPoint->address = address;
^ ~~~~~~~
We only want the address of the function here so this is safe, add a
C-style cast.

Hi @xanlpz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@xanlpz not sure what the build failures are here, @dilijev can you see if there's an issue here please?

Now that master is merged into Linux, re-running the tests should work.

@dotnet-bot test this please


kernel: pipe: add a return value check.

Add a ASSERT to check tpipe_put_small_size return
value.
fix issue: https://github.com/zephyrproject-rtos/zephyr/issues/33827
Signed-off-by: Ningx Zhao<EMAIL_ADDRESS>

fixed: pipe: add a return value check.

please fix commit message, should not start with 'fixed'

Thanks @nashif , I have updated this.


[Cypress fix] Wait URL changes to saved map id before open maps listing

Signed-off-by: Junqiu Lei<EMAIL_ADDRESS>Description
Wait URL changes to saved map id before redirect to maps listing page, so that void not able to find saved map in listing page. This issue only happens in GH action cypress CI window OS.
Issues Resolved
https://github.com/opensearch-project/dashboards-maps/issues/210
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #218 (25e907d) into main (b03ac14) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #218   +/-   ##
=======================================
  Coverage   87.42%   87.42%
=======================================
  Files           6        6
  Lines         175      175
  Branches       24       24
=======================================
  Hits          153      153
  Misses         16       16
  Partials        6        6

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

This fix code also passed test in another PR: https://github.com/junqiu-lei/dashboards-maps/actions/runs/3970639890/jobs/6806596016


Automate coobkook version

See here:
https://github.com/shogun-toolbox/shogun/blob/develop/doc/cookbook/source/conf.py#L66
This should not be hard-written in there (we don't want to manually update this every release) but rather be filled in via cmake.
Nice little entrance task to get familiar with the cookbook build mechanics

Done by @lisitsyn in 6.0 release


Eliminate unnecessary expensive size() calls, use ConcurrentLinkedQueue instead of ConcurrentLinkedDeque

perf: eliminate unnecessary expensive size() calls, use ConcurrentLinkedQueue instead of ConcurrentLinkedDeque)
refer to https://github.com/lettuce-io/lettuce-core/issues/2602

[Y] You have read the contribution guidelines.
[Y] You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
[Y] You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
[Y] You submit test cases (unit or integration tests) that back your changes.

Thank you for your contribution. That's slightly altered because of the many unrelated changes and merged as 18750c1c952ff85f376fc105f9d1dbd23eb237fa.


Add Microsoft Teams notification

What
I added microsoftteams field for Microsoft Teams notification configuration which was added in the Spinnaker 1.23.0 release.
The fields are same as the halconfig described  here.
Thanks in advance.
Why
It was missing.
Described in Microsoft Teams vs Slack: Which Collaboration App Is Better?, there are many Microsoft Teams adoption in the world. We should add them soon.
Ref

Document

Spinnaker Adds Microsoft Teams Notification Support, Armory
Version 1.23, Spinnaker

Related PR

feat(notification): Add Microsoft Teams notification support, spinnaker/deck

@KeisukeYamashita I think we need to also update the echo message and translate the microsoftteams block from the halconfig  to the echo config, right?

@maggieneterval Thanks for your quick review🙇♂ Yeah, I totally missed that point, I will raise another PR🙇♂


color mode is not working on nav bar custom component during first load

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Custom button on nav bar doesn't use the right color mode on first loading. After clicking on light/dark mode button on nav bar, it shows the dark mode button.
Reproducible demo
No response
Steps to reproduce

Add a custom button following: https://github.com/facebook/docusaurus/issues/7227
Check the custom button on nav bar.

Expected behavior
Show dark mode on custom image button when first loading the page.
Actual behavior
Custom button on nav bar doesn't use the right color mode when first loading the pages. I confirm that color mode is under "dark".
It shows light mode like this:

Your environment

Public source code: N/A
Public site URL: N/A
Docusaurus version used: 3.5.2
Environment name and version (e.g. Chrome 89, Node.js 16.4): Microsoft Edge for Business Version 130.0.2849.52
Operating system and version (e.g. Ubuntu 20.04.2 LTS): Windows 11

Self-service

[ ] I'd be willing to fix this bug myself.

Sharing the custom button code is not optional 🤷
Most likely this is a duplicate of other well known issues related to React hydration and use of useColorMode()
Please do a search


chore(main): release 1.0.5

:robot: I have created a release beep boop
1.0.5 (2024-07-11)
🐛 Bugfixes

move to googleapis/release-please-action (5dfc88b)

🧰 Maintenance

deps-dev: bump eslint-plugin-jest from 27.9.0 to 28.2.0 (ae263cf)
deps-dev: bump the npm-development group across 1 directory with 3 updates (86662c6)
deps-dev: bump the npm-development group across 1 directory with 4 updates (92fb379)
deps-dev: bump the npm-development group with 5 updates (5fc6de3)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.5
:sunflower:


Concrete 5.8 Support

Hi Just letting you know that this plugin does not work for <IP_ADDRESS> :(
On installation it has the normal issue which is fine after you click back to home for concrete 5.7
However once navigating to the dashboard after installation of this in 5.8 the whole site breaks down and displays the error for : Cannot redeclare class Concrete\Core\Page\Theme\Theme
On logout: Argument 2 passed to Illuminate\Container\Container::make() must be of the type array, object given, called in C:\xampp\htdocs\concrete_58_SASS\packages\asset_pipeline\src\Service\Assets.php on line 200 and defined
Hope you could work around it when you get the time so that this is compatible with 5.8. I would be extremely appreciative!
Thanks.

I think that should be up to the C5 core team, as they stated to be backwards compatible this time...

Thanks for the report!
@krebbi
Good point but this is not particularly necessarily an issue with core backwards compatibility. We do some heavy lifting in this package (and few others as well) regarding core overrides. The core team doesn't suggest to do such things and they probably don't consider these as backwards compatibility issues. However, these are necessary overrides if we want to make everything work correctly for this package, such as having the theme customizable variables available for the preprocessing languages. Not everything in the core can be overridden in a "clean" way, unfortunately.
So, we'll be looking into this once we find the time. Hopefully that would be before the 8.0.0 release.
I highly doubt the core team would give any notice to these issues.
I would expect they have changed the order of how autoloading is handled by the core, or moved some internal logic related to that to a new place.

IF there is anything I could do to help I would (sadly being a front end developer and all) because I think this feature is almost a complete necessity for concrete 5 and i still don't understand why they wouldn't implement scss preprocessing due to bootstrap 4 (they run bootstrap 3) coming out soon!
Thanks for such the quick reply by the way!

@drago2308 If this is important to you, I would highly suggest commenting this discussion in the concrete5 repository:
https://github.com/concrete5/concrete5/issues/2061
I might be willing to bring this into core if they would realize that more and more people would need it. The main problem is I think is that there currently doens't seem to be enough need for it, so they don't give it much attention.

#3
Makes package compatible with version 8.0.0 and higher.


Merge buyrole into rolemanagement

Do this so that purchasing of roles can be done with awareness of Rolemanagement's settings for people who need that.

you forgot to close this 0eafed4