Server initialization failure.

I am trying to connect Monaco editor to vscode-html-languageserver-bin, and I am having some issue.
My setup: Ubuntu/16.04.
It fails when the editor sends an initialization request to the server. The request sent over the wire looks like:
{"jsonrpc":"2.0","id":0,"method":"initialize","params":{"rootPath":null,"rootUri":"file:///home","capabilities":{"workspace":{"applyEdit":true,"workspaceEdit":{"documentChanges":true},"didChangeConfiguration":{"dynamicRegistration":false},"didChangeWatchedFiles":{"dynamicRegistration":false},"symbol":{"dynamicRegistration":true},"executeCommand":{"dynamicRegistration":true}},"textDocument":{"synchronization":{"dynamicRegistration":true},"completion":{"completionItem":{"snippetSupport":true},"dynamicRegistration":true},"hover":{"dynamicRegistration":true},"signatureHelp":{"dynamicRegistration":true},"references":{"dynamicRegistration":true},"documentHighlight":{"dynamicRegistration":true},"documentSymbol":{"dynamicRegistration":true},"formatting":{"dynamicRegistration":true},"rangeFormatting":{"dynamicRegistration":true},"onTypeFormatting":{"dynamicRegistration":true},"definition":{"dynamicRegistration":true},"codeAction":{"dynamicRegistration":true},"codeLens":{"dynamicRegistration":true},"documentLink":{"dynamicRegistration":true},"rename":{"dynamicRegistration":true}}},"initializationOptions":{"storagePath":"/","gocodeCompletionEnabled":true},"trace":"off"}}
The server responds with:
{"jsonrpc":"2.0","id":0,"error":{"code":-32603,"message":"Request initialize failed with message: Cannot read property 'css' of undefined"}}
Any idea why this happens, and how I can fix it?
Thanks!

I don't really now what it's going on because it's not my code :sweat_smile:, but if we can have more information on the error, I can try to help. A full stack trace should be printed as log notification or in the stderr of the vscode-html-languageserver-bin process

Have you solved your issue ?

I have same issue.
I'm using NeoVim with vim-lsp plugin on Mac .
vim-lsp's logs:
["<---", 4, "html-languageserver", {"response": {"id": 1, "jsonrpc": "2.0", "error": {"code": -32603, "message": "Request initialize failed with message: Cannot read property 'css' of undefined"}}, "request": {"method": "initialize", "jsonrpc": "2.0", "id": 1, "params": {"rootUri": "file:///Users/me", "initializationOptions": {}, "capabilities": {"workspace": {"applyEdit ": true}}, "rootPath": "/Users/me", "trace": "off"}}}]

A full stack trace should be printed as log notification or in the stderr of the vscode-html-languageserver-bin process

Sorry, I couldn't get these log or stderr.. How could I get these?
Thanks.

I do get the same issue with vim-lsp
if executable('html-languageserver')
  au User lsp_setup call lsp#register_server({
      \ 'name': 'html-languageserver',
      \ 'cmd': {server_info->[&shell, &shellcmdflag, 'html-languageserver --stdio']},
      \ 'whitelist': ['html', 'javascript', 'typescript'],
      \ })
endif

Had to debug this with kakoune, so I thought I'd leave my findings.
The version of html-languageserver this package depends requires either:

embeddedLanguages property to be set on initializationOptions or
initializationOptions to be absent

if you don't, it'll crash.
Source


[BUG] Elastic.Serilog.Sinks/README.md is outdated

I was trying out your nuget package (currently latest version 8.6.1) for Serilog with the example provided at: https://www.nuget.org/packages/Elastic.Serilog.Sinks
I wouldn't compile because there is no ConcurrentConsumers property anymore in the BufferOptions class.
So I guess following line should be updated in the docs:
https://github.com/elastic/ecs-dotnet/blob/eac06c3df532d33c1889cca7dca388216c945dc4/src/Elastic.Serilog.Sinks/README.md?plain=1#L28
Also a small typo how wer are communicating on this line: https://github.com/elastic/ecs-dotnet/blob/eac06c3df532d33c1889cca7dca388216c945dc4/src/Elastic.Serilog.Sinks/README.md?plain=1#L43

Property was renamed to ExportMaxConcurrency for anyone else, looking for a solution without digging through the commit history...


Clarifications of the instructions in the specification

Overall, I think this is a good proposition. However, the actual specification needs some additional work, from a pedagogical view.
Remember that the specification will be read by many people with varied background. Hence, it must include things that you and I might consider obvious. Concretely, an absolute majority of the readers will be people using assembly -- only a fraction of the readers will be hardware implementors, so it should be written from their point of new.
Background: I'm a compiler engineer at IAR and the author of the IAR compiler for RISC-V. I also participated in the definition of the RISC-V Zce (code size) extension. I have read instruction specifications for 35+ years.
The syntax of the instructions are not defined
A good specification of an assembly instruction should start with the syntax -- the name of the instruction and the placement of the operands.
Currently, the only place where the syntax of, say, ld is it described is indirectly as the the result of expanding the compressed instructions.
The description is hardware-centric
From the point of view of an assembly programmer the following sentence doesn't really make sense:

It computes an effective address by adding the zero-extended offset, scaled by 8, to the stack pointer, x2.

How the offset is stored in the opcode is an encoding issue, and the steps the hardware takes to extract it is irrelevant for programmers.
A better wording would be "It computes the effective address by adding the offset to the stack pointer, x2".
The range of offset is not specified
Presumably, this is the same range as the corresponding RV64 instructions. However, a reader should not need to look up that in another part of document -- in fact, most users will not know that that would be the place to look.
Example

c.ld dst, offset(src)
Load a 64 bit value into the registers dst and dst+1 from the memory location offset + src. offset must be a multiple of eight in the range 0 .. 248.

I believe the most confusion is coming from the fact that the encodings are not actually included.
The reason for not including them is that we are reusing existing encodings.
There is precedent for this in Zfinx, where the F instructions are also not reintroduced, but it is just stated that the existing instructions will use the x registers.
I am open to adding the encodings, if there is a consensus to have this.
The concrete wording around address calculation is directly mapping to a wording from the C extension. See e.g.

C.LWSP loads a 32-bit value from memory into register rd. It computes an effective address by adding
the zero-extended offset, scaled by 4, to the stack pointer, x2

As I cannot change that wording, consistency trumps here.


Cache rust dependencies

Currently when building the app Cargo will try to update it's index and fetch all dependencies, which can take some time (10 minutes) as the index is really big.
We could make use of a cache to reduce this step to a few seconds when rust/Cargo.lock's hash hasn't changed (no dependency change). It's not common to change our dependencies so we would hit the cache fairly often.

@becominginsane Can you tell me where the generated files are located? I mean, the ones we should cache.

rust/app/.cargo is Cargo's home directory for the project and it should be caching everything there. If we are still not hitting the cache completely then we either need to change the Makefiles a bit or cache $HOME/.cargo  too


Open Manipulator Fails BringUP

Hi everyone, I'm having this errors (exception) when trying to bringup the turtlebot3 with the open manipulator:
$ ros2 launch turtlebot3_manipulation_bringup hardware.launch.py
[INFO] [launch]: All log files can be found below /home/dali/.ros/log/2023-07-01-20-56-48-983432-dali-turtlebot-10821
[INFO] [launch]: Default logging verbosity is set to INFO
[ERROR] [launch]: Caught exception in launch (see debug for traceback): executable
'[<launch.substitutions.text_substitution.TextSubstitution object at 0xffff8f854100>]' not found on the PATH
Data: Turtlebo3 with OpenManipulator X, Humble, Ubuntu 22.04, Raspberry Pi 4.

Hi, Could you try below command on 'turtlebot3_ws' using Terminal?
$ source install/setup.sh

I have the same issue too.
@yeongkyuyoon Sourcing the install/setup.bash did not fix the issue.
ROS2 Humble / Ubuntu 22.04 Server / Raspberry Pi 4 / Turtlebot3 Waffle Pi with Manipulator.
ubuntu@wafflepi:~/turtlebot3_ws$ source install/setup.bash ubuntu@wafflepi:~/turtlebot3_ws$ ros2 launch turtlebot3_manipulation_bringup hardware.launch.py [INFO] [launch]: All log files can be found below /home/ubuntu/.ros/log/2023-07-13-16-58-41-633216-wafflepi-1236 [INFO] [launch]: Default logging verbosity is set to INFO [ERROR] [launch]: Caught exception in launch (see debug for traceback): executable '[<launch.substitutions.text_substitution.TextSubstitution object at 0xffffbafb6c50>]' not found on the PATH ubuntu@wafflepi:~/turtlebot3_ws$

@ozanaki
Perhaps some dependency packages are not installed, resulting in errors when executing the launch file. I identified all the missing packages in the ROS2 Foxy version and conducted tests accordingly.
It is assumed that the Humble version also encounters a similar problem. Please install the following dependency packages from the Humble distro.
sudo apt install ros-foxy-hardware-interface
sudo apt install ros-foxy-xacro
sudo apt install ros-foxy-imu-sensor-broadcaster
sudo apt install ros-foxy-diff-drive-controller
sudo apt install ros-foxy-position-controllers
sudo apt install ros-foxy-gripper-controllers
sudo apt install ros-foxy-joint-state-broadcaster
sudo apt install ros-foxy-joint-trajectory-controller
sudo apt install ros-foxy-controller-manager
and run sudo apt-get update  && sudo apt-get dist-upgrade.

@yeongkyuyoon
I have installed these packages for Humble as you suggested. This error message has gone, the manipulator has gone to the start position while brought up.
sudo apt install ros-humble-hardware-interface
sudo apt install ros-humble-xacro
sudo apt install ros-humble-imu-sensor-broadcaster
sudo apt install ros-humble-diff-drive-controller
sudo apt install ros-humble-position-controllers
sudo apt install ros-humble-gripper-controllers
sudo apt install ros-humble-joint-state-broadcaster
sudo apt install ros-humble-joint-trajectory-controller
sudo apt install ros-humble-controller-manager
But this time some different error messages appeared on the terminal as
[ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
and
[ros2_control_node-1] *** stack smashing detected ***: terminated
seems not related to this issue. I just proceed to another issue :)
Thank you for your help

I was able to solve the original issue but typing (at tb3) sudo apt install ros-humble-launch-*. But then another problem came through bring up:
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!

@ozanaki  @gandradeng
Hi, I guess it's a problem with OpenCR's firmware. What firmware are you using? Have you installed it according to the guidelines in the E-manual?

[TurtleBot3 SBC] Download the OpenCR firmware file on Raspberry Pi (SBC) and upload the correct firmware with the following commands.
$ export OPENCR_PORT=/dev/ttyACM0
$ export OPENCR_MODEL=turtlebot3_manipulation
$ rm -rf ./opencr_update.tar.bz2
$ wget https://github.com/ROBOTIS-GIT/OpenCR-Binaries/raw/master/turtlebot3/ROS2/latest/opencr_update.tar.bz2
$ tar -xvf opencr_update.tar.bz2
$ cd ./opencr_update
$ ./update.sh $OPENCR_PORT $OPENCR_MODEL.opencr

Hi, @yeongkyuyoon! I did exactly as in E-manual and until now the open manipulator is not working.

@yeongkyuyoon
I am exactly following the instructions in the manual including setting up the OpenCR. I installed the manipulation package on freshly installed ROS2 Humble on Ubuntu 22.04 on Raspberry Pi 4 (waffle pi with manipulator) described as in the manual  https://emanual.robotis.com/docs/en/platform/turtlebot3/manipulation/#software-setup.
Here is the ssh terminal output screen after applying the bringup command
$ ros2 launch turtlebot3_manipulation_bringup hardware.launch.py:

[INFO] [launch]: All log files can be found below /home/ubuntu/.ros/log/2023-07-19-09-35-37-701805-wafflepi-1953
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [ros2_control_node-1]: process started with pid [1956]
[INFO] [robot_state_publisher-2]: process started with pid [1958]
[INFO] [spawner-3]: process started with pid [1960]
[INFO] [ld08_driver-4]: process started with pid [1962]
[robot_state_publisher-2] [INFO] [1689748540.081126855] [robot_state_publisher]: got segment base_footprint
[robot_state_publisher-2] [INFO] [1689748540.084080957] [robot_state_publisher]: got segment base_link
[robot_state_publisher-2] [INFO] [1689748540.086333806] [robot_state_publisher]: got segment base_scan
[robot_state_publisher-2] [INFO] [1689748540.088819302] [robot_state_publisher]: got segment camera_link
[robot_state_publisher-2] [INFO] [1689748540.090887115] [robot_state_publisher]: got segment camera_rgb_frame
[robot_state_publisher-2] [INFO] [1689748540.092974003] [robot_state_publisher]: got segment camera_rgb_optical_frame
[robot_state_publisher-2] [INFO] [1689748540.093049058] [robot_state_publisher]: got segment caster_back_left_link
[robot_state_publisher-2] [INFO] [1689748540.093081150] [robot_state_publisher]: got segment caster_back_right_link
[robot_state_publisher-2] [INFO] [1689748540.093108428] [robot_state_publisher]: got segment end_effector_link
[robot_state_publisher-2] [INFO] [1689748540.093135131] [robot_state_publisher]: got segment gripper_left_link
[robot_state_publisher-2] [INFO] [1689748540.093160668] [robot_state_publisher]: got segment gripper_right_link
[robot_state_publisher-2] [INFO] [1689748540.093186483] [robot_state_publisher]: got segment imu_link
[robot_state_publisher-2] [INFO] [1689748540.093211742] [robot_state_publisher]: got segment link1
[robot_state_publisher-2] [INFO] [1689748540.093237075] [robot_state_publisher]: got segment link2
[robot_state_publisher-2] [INFO] [1689748540.093261408] [robot_state_publisher]: got segment link3
[robot_state_publisher-2] [INFO] [1689748540.093287037] [robot_state_publisher]: got segment link4
[robot_state_publisher-2] [INFO] [1689748540.093311648] [robot_state_publisher]: got segment link5
[robot_state_publisher-2] [INFO] [1689748540.093335852] [robot_state_publisher]: got segment wheel_left_link
[robot_state_publisher-2] [INFO] [1689748540.093360444] [robot_state_publisher]: got segment wheel_right_link
[ros2_control_node-1] [INFO] [1689748540.103276592] [resource_manager]: Loading hardware 'TurtleBot3ManipulationSystem'
[ros2_control_node-1] [INFO] [1689748540.107125038] [resource_manager]: Initialize hardware 'TurtleBot3ManipulationSystem'
[ros2_control_node-1] [INFO] [1689748540.113484645] [turtlebot3_manipulation]: Succeeded to open port
[ros2_control_node-1] [INFO] [1689748540.121159295] [turtlebot3_manipulation]: Succeeded to set baudrate
[ros2_control_node-1] [INFO] [1689748540.150026914] [turtlebot3_manipulation]: OpenCR Model Number 20480
[ros2_control_node-1] [INFO] [1689748540.178978772] [turtlebot3_manipulation]: Connected manipulator
[ros2_control_node-1] [INFO] [1689748540.207992482] [turtlebot3_manipulation]: Connected wheels
[ros2_control_node-1] [INFO] [1689748540.208380904] [resource_manager]: Successful initialization of hardware 'TurtleBot3ManipulationSystem'
[ros2_control_node-1] [INFO] [1689748540.209037639] [resource_manager]: 'configure' hardware 'TurtleBot3ManipulationSystem'
[ros2_control_node-1] [INFO] [1689748540.209114601] [resource_manager]: Successful 'configure' of hardware 'TurtleBot3ManipulationSystem'
[ros2_control_node-1] [INFO] [1689748540.209157434] [resource_manager]: 'activate' hardware 'TurtleBot3ManipulationSystem'
[ros2_control_node-1] [INFO] [1689748540.209181397] [turtlebot3_manipulation]: Ready for start
[ros2_control_node-1] [INFO] [1689748540.236964525] [turtlebot3_manipulation]: Wait for IMU re-calibration
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ld08_driver-4] /dev/ttyUSB0    CP2102 USB to UART Bridge Controller
[ld08_driver-4] /dev/ttyACM0    OpenCR Virtual ComPort in FS Mode
[ld08_driver-4] FOUND LDS-02
[ld08_driver-4] LDS-02 started successfully
[ros2_control_node-1] [INFO] [1689748543.271573655] [turtlebot3_manipulation]: Joints and wheels torque ON
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [INFO] [1689748543.351609203] [turtlebot3_manipulation]: Set profile acceleration and velocity to joints
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [INFO] [1689748543.422589409] [turtlebot3_manipulation]: Set profile acceleration and velocity to gripper
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [INFO] [1689748543.493650207] [turtlebot3_manipulation]: Set goal current value to gripper
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [INFO] [1689748543.528636065] [turtlebot3_manipulation]: System starting
[ros2_control_node-1] [INFO] [1689748543.530428400] [resource_manager]: Successful 'activate' of hardware 'TurtleBot3ManipulationSystem'
[ros2_control_node-1] [INFO] [1689748543.550493230] [controller_manager]: update rate is 100 Hz
[ros2_control_node-1] [INFO] [1689748543.552310232] [controller_manager]: RT kernel is recommended for better performance
[ros2_control_node-1] [INFO] [1689748543.552507008] [turtlebot3_manipulation]: Start to read wheels and manipulator states
[ros2_control_node-1] [INFO] [1689748543.566558895] [turtlebot3_manipulation]: Start to write wheels and manipulator commands
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [INFO] [1689748543.604358856] [controller_manager]: Loading controller 'joint_state_broadcaster'
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [ERROR] [1689748543.638808497] [turtlebot3_manipulation]: Can't control joints
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [ERROR] [1689748543.674483737] [turtlebot3_manipulation]: Can't control gripper
[spawner-3] [INFO] [1689748543.713645593] [spawner_joint_state_broadcaster]: Loaded joint_state_broadcaster
[ros2_control_node-1] [INFO] [1689748543.717025005] [controller_manager]: Configuring controller 'joint_state_broadcaster'
[ros2_control_node-1] [INFO] [1689748543.717379613] [joint_state_broadcaster]: 'joints' or 'interfaces' parameter is empty. All available state interfaces will be published
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!
[ros2_control_node-1] *** stack smashing detected ***: terminated
[ERROR] [ros2_control_node-1]: process has died [pid 1956, exit code -11, cmd '/opt/ros/humble/lib/controller_manager/ros2_control_node --ros-args --params-file /tmp/launch_params_5fpdjqzo --params-file /home/ubuntu/turtlebot3_ws/install/turtlebot3_manipulation_bringup/share/turtlebot3_manipulation_bringup/config/hardware_controller_manager.yaml -r ~/cmd_vel_unstamped:=cmd_vel -r ~/odom:=odom'].

Hi, I've got the same problem as @ozanaki. Did anybody manage to solve it? I suppose it is something connected with OpenCR software.

Hi, I've got the same problem as @ozanaki. Did anybody manage to solve it?

I am also getting the same problem as @ozanaki , fail to bring up TB3 with Openmanipuator-X in ROS 2 Humble, getting error “[ros2_control_node-1] [ERROR] [DynamixelSDKWrapper] [TxRxResult] There is no status packet!”...
Any solution?

This issue occurs with the latest version of Dynamixel, and it has been confirmed that it can be resolved by using the Dynamixel Wizard to change the Return Delay Time of all motors from 250 ms to 0 ms.


Mark perf tests as flaky

Mark linux_animated_placeholder_perf and linux_multi_widget_construction_perf__timeline_summary as flaky until https://github.com/flutter/flutter/issues/73873 is resolved.

Merging to green the build.

Merging to green the build.


by default price attribute is selected in category

Bagisto Version

[ ] master branch

Steps To Reproduce

go to category

add a new category

Actual Result

It's not a by default, it is the first item that is selected and the attribute is a required field.


feature request: can you addy cachybrowser?

Hi, would it be possible to add cachy browser? https://github.com/CachyOS/CachyOS-Browser-Settings
aur package here: https://aur.archlinux.org/packages/cachy-browser
bonus bonus request: would be interesting to see mercury browser too: https://thorium.rocks/mercury

Hi -- thank you for the suggestion. I'm currently focused on adding more tests, but I will add your suggestions to our list on issue #66.

@arthuredelstein btw you could also close #147

@arthuredelstein I updated it and I think found all these issues (or almost all)

Can you add Floorp and Thorium browsers?

@Tuscann
bad issue - #66 is the right issue
thorium is already in the list for new browsers


More expressive invoice descriptions

This pull request adds more descriptive invoice messages that resolve #58

Tested ... this is a vast improvement over erroneous liquid swap on all CLN invoices prior.
Unfortunately this is bothering me ...
My swap-out is a swap-in from the partner's POV.
My swap-in is a swap-out from the partner's POV.
Invoice labels swap-out or swap-in are generated from the POV ... of the requester? (confused about this)
Invoice label: swap-out lbtc fee fbd2c20f5461a567909d19a0dd742db0ad91efbcbddc4ac0b2b88f07517dbfd8
It bothers me that these invoice labels show swap-out or swap-in to both sides when that isn't what it is from the POV of one side.
      "type": "swap-out",
      "role": "receiver",

Within listswaps and the statistics for each peer in listpeers it seems we get this POV right by differentiating the type and the role?
Since the invoice text is identical to both sides, how about this instead?
Invoice label: peerswap lbtc fee 699350x125x0 fbd2c20f5461a567909d19a0dd742db0ad91efbcbddc4ac0b2b88f07517dbfd8
Invoice label: peerswap lbtc claim 699350x125x0 fbd2c20f5461a567909d19a0dd742db0ad91efbcbddc4ac0b2b88f07517dbfd8
It was peerwap ... the short-id is often more useful at a glance ... and they can lookup the swap id if they want to.
I think after unifying the types in the invoice labels this is ready for merge.

After this is merged please rebase #83 and #97

Requested changes are done. Self Ack 6e22105723617df2fff6515c4087863094789998


Showcase: Use standalone Smiles Drawer in a typescript project

https://github.com/Acylation/obsidian-chem
Run npm install smiles-drawer, and then declare it in the global.d.ts using the line declare module 'smiles-drawer';, then you'll have the smiles-drawer included in your project.
To call the module, you can use this to import import SmilesDrawer from 'smiles-drawer';

Thanks! Can you please post a minimal example that shows how to use this in React.js?

I did try combining React with my project, https://github.com/Acylation/obsidian-chem-indev, which successful identify the package while no actual rendering example had I implemented.
Maybe you can try this https://github.com/reymond-group/smilesDrawer/issues/157#issuecomment-1484825796
The quoted example calls SVGDrawer API, however, a rather wrapped-up class SmiDrawer would be easier to use. Here's the rough type defs.
declare module 'smiles-drawer' {
	class SmiDrawer {
		constructor(moleculeOptions?: object, reactionOptions?: object);
		static apply(
			moleculeOptions?: object,
			reactionOptions?: object,
			attribute?: string | undefined,
			theme?: string,
			successCallback?: CallableFunction,
			errorCallback?: CallableFunction
		): void;
		apply(
			attribute?: string,
			theme?: string,
			successCallback?: CallableFunction,
			errorCallback?: CallableFunction
		): void;
		draw(
			smiles: string,
			target:
				| HTMLElement
				| SVGElement
				| HTMLImageElement
				| string
				| 'svg'
				| 'canvas'
				| 'img'
				| null,
			theme?: string,
			successCallback?: CallableFunction,
			errorCallback?: CallableFunction,
			weights?: number[] | object
		): void;
		drawMolecule(
			smiles: string,
			target:
				| HTMLElement
				| SVGElement
				| HTMLImageElement
				| string
				| 'svg'
				| 'canvas'
				| 'img'
				| null,
			theme: string,
			weights: number[] | object,
			callback: CallableFunction
		): void;
		drawReaction(
			smiles: string,
			target:
				| HTMLElement
				| SVGElement
				| HTMLImageElement
				| string
				| 'svg'
				| 'canvas'
				| 'img'
				| null,
			theme: string,
			settings: object, // reaction settings
			weights: number[] | object,
			callback: CallableFunction
		): void;
		svgToCanvas(
			svg: SVGElement,
			canvas?: HTMLCanvasElement
		): HTMLCanvasElement;
		svgToImg(svg: SVGElement, img?: HTMLImageElement): HTMLImageElement;
		getDimensions(
			element: HTMLImageElement | HTMLCanvasElement | SVGElement,
			svg: SVGElement
		): { w: number; h: number };
	}
}


Use camera color matrix behavior

If you do not set the imgdata.params.use_camera_matrix option in LibRaw, it defaults to be the same as imgdata.params.use_camera_wb. Conditional defaults seem confusing to me.
In keeping with the mantra of "have reasonable defaults", should we just set this to "True" by default, or is it just me that thinks conditional defaults are confusing and following the camera wb is okay?
In all fairness, if you use the camera settings for one you probably also want to use it for the other, but I still don't like it.

@campaul agrees. Maybe swap back if we discover there's a good reason LibRaw did it that way.


Substituted string concatenation as argument of StringBuilder.append()

(chores) Substituted string concatenations
Substituted string concatenation as argument of StringBuilder.append() with chained append() calls - every concatenation is transformed by compiler to temp StringBuilder itself (additional garbage). Implemented templated var in instanceof check.
This issues are highlighted in static analysis check

Can you update the PR

I updated the PR


Automatic robots.txt / webmanifest generation

What kind of changes does this PR include?

Automatic robots.txt generation
Automatic webmanifest / icon generation

Description

Since a change of domain can result in many manual edits, it can increase the risk of wrong domains in some files. This is especially important for a template theme. So robots.txt is generated using astro.config.site now.
User can change the icon.svg and forget about icons now. It's automatically handled by astro-webmanifest. Multiple sizes are generated including Apple touch icons.
Also, webmanifest config takes it inputs largely from siteConfig which makes it robust to the manual edits. Change the site config and you're ready.
Nothing visual has changed.

Thanks @MelihDarcanxyz for doing this. I've added back some static links just to speed-up builds, particularly page heavy sites, and to show a favicon in dev


rez-pip package tar.gz install & {root}

I am developing a python library for in-house tools. I'm using python to build and then install via rez-pip (see build.py below). However, once installed the package {root} does not point to the correct location.
Once installed via rez-pip, {root} is reported to be
c:\programdata\rez\packages\nymus\0.1.0\platform-windows\arch-AMD64\python-3.9.13
which is incorrect, as the library is actually installed to
C:\ProgramData\rez\packages\nymus\0.1.0\e7b64aebe77c11047a71e73bebf05e90cbb6361d
Consequently the library cannot be found, as the incorrect path is added to {PYTHONPATH}
 package.py & build.py
# package..py

name = "nymus"
version = "0.1.0"
build_requires = ["python-3"]
build_command = "python {root}/build.py"
requires = ["python_dotenv", "pandas-1.5.3", "click-8.1.3"]
uuid = "d72bcf23-5a7b-4448-ab8d-b9a5a23973ed"

@early()
def variants():
    """Generate variant for current platform"""
    from rez.package_py_utils import expand_requires

    requires = ["platform-**", "arch-**", "python-**"]
    return [expand_requires(*requires)]

def commands():
    info("Root path {root}")
    env.PYTHONPATH.append('{root}/python')
    env.PATH.append('{root}/bin')

# build.py

import os
import sys
from pathlib import Path
from subprocess import run

from rez.exceptions import BuildError

if __name__ == "__main__":
    sourcePath = Path(os.environ["REZ_BUILD_SOURCE_PATH"])

    if os.environ["REZ_BUILD_INSTALL"] == "1":

        binPath = Path(os.environ["REZ_BUILD_SOURCE_PATH"]) / "bin"
        installPath = Path(os.environ["REZ_BUILD_INSTALL_PATH"])
        buildPath = Path(os.environ["REZ_BUILD_PATH"])

        if sys.platform == "win32":
            proc = run(
                args=[
                    "python",
                    "setup.py",
                    "egg_info",
                    "--egg-base",
                    buildPath,
                    "sdist",
                    "--dist-dir",
                    str(buildPath),
                ],
                cwd=sourcePath / "python_libraries",
            )
            if proc.returncode != 0:
                raise BuildError(str(proc))

            proc = run(
                args=[
                    "rez-pip",
                    "--install",
                    buildPath
                    / f"{os.environ['REZ_BUILD_PROJECT_NAME']}-{os.environ['REZ_BUILD_PROJECT_VERSION']}.tar.gz",
                ],
            )
            if proc.returncode != 0:
                raise BuildError(str(proc))
        else:
            raise NotImplementedError(sys.platform)

Environment

Windows
Rez version: 2.112.0
Rez python version: 3.9.13

To Reproduce
Build and install any python library
Expected behavior
{root} should be set correctly.
Actual behavior
{root} does not point to the install location of the package.

Without diving into the details why it happens this seems to be a case of hashed variants attribute not being set in the package.py though used for creating the variant.
I am a bit confused by the setup as a whole though. In a rez-pip workflow i would not expect a package.py to be present. Just a setup.py, build to bdist_wheel and rez-pip it.
Could you share why you are setting up the workflow like this? And also if there is a specific reason not to build as a wheel instead?

I have the same comments/questions as @instinct-vfx. Additionally, I'm not convinced that expand_requires works. Also, why do you required the arch and platform in the variant? Shouldn't these be requirements of pandas?

Without diving into the details why it happens this seems to be a case of hashed variants attribute not being set in the package.py though used for creating the variant.
I am a bit confused by the setup as a whole though. In a rez-pip workflow i would not expect a package.py to be present. Just a setup.py, build to bdist_wheel and rez-pip it.

You're right, this the rez-build was then calling rez-pip which was daft. Obviously so, now that you pointed it out.
Unless there's a good way to wrap this all up in a single build call, I'll have to build the tar.gz and then rez-pip install it.

Could you share why you are setting up the workflow like this? And also if there is a specific reason not to build as a wheel instead?

Just that my experience is entirely with using setuptools in this way. If building as a wheel is the way to go, I'll look into it.

Also, why do you required the arch and platform in the variant? Shouldn't these be requirements of pandas?

This was purely to circumvent this error:
ERROR    ResourceError: Attempting to install a package without variants (MemoryVariantResource({'location': 'memory{0x1d9b9fd5cc0}', 'name': 'nymus', 'version': '0.1.0', 'index': None, 'repository_type': 'memory'})) into an existing package with variants (FileSystemPackageResource({'location': 'c:\\programdata\\rez\\packages', 'name': 'nymus', 'version': '0.1.0', 'repository_type': 'filesystem'}))


Broken link in README.md

Describe the bug
A link is broken in the README.md
To Reproduce
Steps to reproduce the behavior:

Open the README.md
Go to line 168
Click on the link
Nothing happens :)

Additional context
https://github.com/meilisearch/meilisearch/blob/main/README.md?plain=1#L168

It seems like there is no target to link to for #demo
In order to fix it you can either tweak line 31 by adding a name attribute to the  tag, so it would look like this:
<p align="center" name="demo">
Or you could also set a "demo" header above the trumen-show gif (aka line 30)

that's "p" tag on the end of the second, I've wrote so that it  models it, jaja sorry

Thanks @eskombro and @lucas-cauhe 😄
Related to https://github.com/meilisearch/meilisearch/issues/2266

Hello @lucas-cauhe I tried your fix here but it did not work: https://github.com/meilisearch/meilisearch/pull/2504/files
I might understand something! If you have time to go to the PR, your review is more than welcome ❤
In any case, I close this issue since it's not something we really consider as a bug regarding the Meilisearch repo: we will add the link to the official demo once it's officially released!
The link is not broken, there is just no link to share at the moment.


TSW-63205: update PoC deployment script

I have merged into my 'bd' branch and will be testing with some other changes. So I'll close this PR but it will get merged to master with the rest.


Can I update openai/baseline?

If I update the openai/baseline, what files that you created and/or modified that will I need to create and/or modify?
modify this file: baselines/common/cmd_util.py
create these files:
run_multiagent_unity.py
run_unity.py
baselines\common\vec_env\unity_vec_env.py
baselines\ddpg\main_unity.py

my code is a bit messy! - I think it is just these:
baselines/run_unity.py
baselines/run_multiagent_unity.py
baselines/common/vec_env/unity_vec_env.py - this is the vectorized enviroment wrapper for ml-agents
baselines/common/cmd_util.py - i added make_multi_unity_vec_env
re - baselines\ddpg\main_unity.py - i'm not sure i got this working

Thank you.


Problem with GetOrAdd() and null values from value factory

Hi,
I'm running into a big problem using the GetOrAdd() method on ICacheManager<object>.
Sometimes a value factory supplied to this method will return a null reference, because the underlying data has been removed, for example. This crashes the code with an ArgumentNullException because the CacheItem constructor won't allow a null value.
Would it be possible for the underlying implementation to invoke the value factory and only set the cache item if the value from the factory is not null, otherwise just return null so that the calling code can just continue?
I understand why we don't want to set a null value in the underlying cache, but it should be possible to handle null from a value factory as this can easily happen.
Otherwise I guess I would have to look at a special value to represent null which is also cacheable, but that's not ideal.
If there's a better solution that I'm missing, happy to hear it.
Thanks,
Sam

@sam-piper  Good question. The problem is, that the intention of the GetOrAdd method is to ensure a value is stored in the cache at the end (and gets returned).
If the method would return null, it could mean different things and you'd have to always react on it.
If the value factory for some reason returns null, that's basically an invalid application state, right? And should either not happen, or, if you expect it to happen, you'd have to handle it.
Unfortunately there is no simple way to cancel the operation and do something reasonable ;)
What about you add a try/catch for ArgumentNull around it. Make it an extension e.g. TryGetOrAdd if you don't want to write that code every time.
Does that make sense? Let me know what you think.

Thanks for the reply!
I think that a value factory should be able to return null without causing exceptions - I don't think that the underlying caching layer should have to constrain its implementation either, it's just a black box.
For example say I'm processing a web request that references a cached entity graph which is not currently in the cache and has just been deleted by another request on another server - the code uses GetOrAdd as a way of saying, look in the cache, if it's not there then invoke the function that loads it from the database, cache that value and return it. The database will return a null reference if the entity no longer exists, and the code expects this value to be passed through the caching layer regardless.
This isn't exceptional in my opinion, it's just standard data access, but becomes difficult to handle with the current implementation here.
Using try/catch would work around the issue I'm having, but to me it feels a bit hacky - I'd like to see the underlying framework be more flexible here, ideally.
Sent from my iPhone
On 18 Oct 2016, at 6:49 PM, MichaC <notifications<EMAIL_ADDRESS>wrote:
@sam-piperhttps://github.com/sam-piper Good question. The problem is, that the intention of the GetOrAdd method is to ensure a value is stored in the cache at the end (and gets returned).
If the method would return null, it could mean different things and you'd have to always react on it.
If the value factory for some reason returns null, that's basically an invalid application state, right? And should either not happen, or, if you expect it to happen, you'd have to handle it.
Unfortunately there is no simple way to cancel the operation and do something reasonable ;)
What about you add a try/catch for ArgumentNull around it. Make it an extension e.g. TryGetOrAdd if you don't want to write that code every time.
Does that make sense? Let me know what you think.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MichaCo/CacheManager/issues/99#issuecomment-254432730, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AN_IiPP0EmUhUDlVcryBzoofEYb23Jejks5q1HoNgaJpZM4KZbxc.

I agree that an exception is less simple to deal with than null. But you'd have to check for null in that case.
My concern was that returning null could mean different things, which would be misleading in what failed or what the result actually means.
In the case of GetOrAdd, returning null would mean that the key doesn't exist and add did not work for whatever reason. That's kind of fine, but if you don't expect null and want to know why, the debug process would be kind of annoying. I usually prefer in a more explicit API which is easier to understand why something happens...
Update works similar, although there are some catches. If the update valueFactory returns null, it will throw an error. In case they key didn't exist at all, it will return null. I already have mixed feelings about that returning null...
If I would also return null in case valueFactory returns null, it would be even more confusing to figure out what happend.
So, for Update I will not change it for sure, but then the API should also be kind of consistent...

For now I'll use try/catch - I'm not anticipating that this problem will occur very often, so hopefully the number of exceptions generated is small, but again, handling a control flow issue by catching exceptions is not ideal.
I understand the difficulties around what null means. I personally think that GetOrAdd could be modified without needing to be consistent with other methods, as it works in a specific way and I think is very likely to be used orthogonally to other methods - just add a null check before creating and adding the internal cache item and the issue as I experience it is solved. This still stays in line with the rule that null values are not cached, and the calling code checks for null only if it needs to.
It's just a suggestion, I understand if this change conflicts with the way you've designed the API, but for me it is a bit dissonant with the way my own code is working, and I imagine others will encounter the same problem as well.

@sam-piper I thought about it a little bit more and the result is that I will add another method TryGetOrAdd to compensate. So, if you don't want to handle exceptions, use that one and it will return false in case your factory returns null.
The method has an out parameter for the value (which in that case would also be null or default(T)).
The implementation doesn't throw/catch exceptions.
I always disliked the behavior of Update returning null in case the key doesn't exist. I also changed that to throw an exception now. There is already TryUpdate and AddOrUpdate in case you want to implement a certain app flow without exception handling.
So, those two methods are in line now in terms of behavior I think.

@sam-piper
Version 0.9.2 is out and and changes mentioned above are in. Let me know if that solves your concerns or what you think. Thanks!

In my case it was necessary to cache null values. It was indicating "Not found" result.
And I found a simple solution:
public class NullableObject<T>
{
    public T Value { get; }

    private NullableObject(T value)
    {
        Value = value;
    }

    public static implicit operator T(NullableObject<T> nullableObject)
    {
        return nullableObject.Value;
    }

    public static implicit operator NullableObject<T>(T objectValue)
    {
        return new NullableObject<T>(objectValue);
    }
}

Code updates:
Throws ArgumentNullException exception on null values:
private ICacheManager<ObjectToCache> _cache;
Doesn't throw ArgumentNullException exception and returns back cached null values (without calling valueFactory method):
private ICacheManager<NullableObject<ObjectToCache>> _cache;


What If There are Dead In Results???

I think that is normal I believe it is 2 wilds colliding during a scan


[Security Issue] JavaScript Execution using link

When we enter:

I am able to execute alert(1) in Javascript, which may lead to cross site scripting. Any Fix!!

Have you diabled the enablePreviewContentClick option?

This issue will be closed. If need, please reopen it.

What if  the Javascript:URI is right clicked and open in another tab. It executes XSS

Yeah, also works for handlers like


Improve tests in Clojure Robot name exercise

Moved from https://github.com/exercism/exercism.io/issues/1994 -- original issue by @pdej

Seems like these tests could be improved. The execution of (robot-name/reset-name robbie) is done before any tests are run, which would make all tests pass by simply hard coding a name when initializing a new robot, and hard coding another one for reset.
A better way would be to create a robot, save the original name, reset it and then test against a new robot or the original name, all in the same test. Instead of having a global definition of the robots. Or have I misunderstood the tests?

I agree with the suggested improvement. The robbie and clutz vars feel weird.


Protocol Designer: Use React-Router

As a protocol designer developer, I would like to use react router to manage pages
Acceptance Criteria

Protocol designer uses react router to manage file pages

Implementation Notes

Note we should hold on this until IA design discussion Tuesday 2/2/2018

Duplicate of #906


Uamp isn't restarting playback after pause and resume

Testing with the benchmark variant, allowing speaker playback.
Download a playlist.
Play, wait for sound.
Pause and then hit play again.  Music doesn't start playing.

Just tested again on a different device and working.

At the time, seemed like an issue on device.


WIP: a first neighbor sampling loader

We're still missing many functions to implement an efficient neighbor sampling, but here are some trials on the implementation.

Closing as it's carried over in #112


fix for issue #72

two commits cause I accidentally omitted a line and misnamed the issue.

@Astromations Just to clarify your fix ill request @kyrie25 to review this fix ok?

@kyrie25 can you share your opinion regarding this pr before merge

@kyrie25 can you share your opinion regarding this pr before merge

Have not tested but making the icon disappear doesn't really count as fixing the issue but rather a lazy (no offense) workaround to omit a Spotify feature, of which will raise questions among users.
Would be nice if you can find a way to realign the icons instead.

@Astromations Can you work regarding @kyrie25 s review please?
@kyrie25 Thank you for sharing your idea

It could be fixed the same way as I did in #71.

@kyrie25 can you share your opinion regarding this pr before merge

Have not tested but making the icon disappear doesn't really count as fixing the issue but rather a lazy (no offense) workaround to omit a Spotify feature, of which will raise questions among users. Would be nice if you can find a way to realign the icons instead.

If I did keep the icon it'd prevent users from clicking on the center of the playlist.

The icon is marginally small, plus that problem does not arise from the users since it's not the only place they can click to select the playlist.
Removing the feature on the other hand, would arise among users.

@kyrie25 can you share your opinion regarding this pr before merge

Have not tested but making the icon disappear doesn't really count as fixing the issue but rather a lazy (no offense) workaround to omit a Spotify feature, of which will raise questions among users. Would be nice if you can find a way to realign the icons instead.

If I did keep the icon it'd prevent users from clicking on the center of the playlist.

It's the same case with the playing icon.

The icon is marginally small (taking up less than 1/9th of the playlist icon), plus that problem does not arise from the users since it's not the only place they can click to select the playlist. Removing the feature on the other hand, would arise questions among users.

So should I keep the icon and just resize the playlist image

This will suffice, as it'd only realign the icon and nothing more. Had some free time so I looked into the issue.

This will suffice, as it'd only realign the icon and nothing more. Had some free time so I looked into the issue.

That works. I'll keep the icon removed on my end though cause I find it kind of ugly

@kyrie25 @Astromations  so we are ready to marge right?

All good.

copy that,
merging now

@Astromations @kyrie25 @MGPlayzYT thank you


Enable wiki or discussions for people to share examples and related useful libraries?

I've been spending a bit of time looking at client-side ratelimiting libraries, and would like to share them. Wiki or Discussions might be a good place for people to share that stuff.

Today I learned GitHub added discussions, looks neat. I've enabled them, would love to discuss rate limiting solutions as I've been looking for a good library for a while without success.


Ethereum A1 test network tap can not receive test coins

Ethereum A1 test network tap can not receive test coins

Its because the Twitter API is hitting a rate limit. Because of this, we can't recognize the Ethereum address in your tweet. Next steps: You will need to wait and try again. If you continue having problems, you can also send your address in our Discord's "🪙︱manual-faucet" channel and ask the community to send you testnet funds.


txn: disable follower read for max ts point get read

Ref: https://github.com/pingcap/tidb/issues/43583
What's changed:
When autocommit point get is used, the linearizability could be broken.Disable follower-read path for autocommit point get read.
Side effects:
There would be read hotspot issue if follower read is used and most of the reads are autocommit point get read. Should it be a configurable option?

Try to use another solution described in https://github.com/pingcap/tidb/issues/43583#issuecomment-1539580909.


apple2driver draws 100% cpu

What happened?
Hi,
apple2driver draws 100% CPU which makes the single core zero extremely slow. and heats the zero and some 74xx.
this is when Apple2-IO-Rpi is not connected to the zero:
Read block 0101 in slot 0, drive 1...failed can't write byte while byte is incoming
Read block 0101 in slot 0, drive 1...failed timed out writing byte -- read stuck low
... repeated over and over

and this is when working fine, but still drawing 100% cpu
Read block 0002 in slot 5, drive 1...succeeded
Sending date/time...7F 2E 26 0D
Read block 0002 in slot 5, drive 1...succeeded
Sending date/time...7F 2E 26 0D
Read block 0002 in slot 5, drive 1...succeeded
Sending date/time...7F 2E 26 0D
Read block 0002 in slot 5, drive 1...succeeded
Read block 0003 in slot 5, drive 1...succeeded
Read block 0004 in slot 5, drive 1...succeeded
Read block 0005 in slot 5, drive 1...succeeded

could this be caused by using HCT instead of LS? Wasn't able to get LS for some of the chips.
from Apple point of view it works fine, but SSH and Wifi totally suck on single core Zero.
driver version is 29, not listed..
firmware version F
shell version - no idea which shell and how to find out, sorry.
thank you very much.
ps. tried other Apple II+ and Apple IIe, looks like the same problem.
Hardware Version
Sixth Prototype (Self-assembled)
Firmware Version
000F
Driver Version
0028 (add live drive loading and regeneration)
Shell Version
000D (added version info)
Where are you seeing this problem?
Other
What computer is this running on?
Apple II+

The high CPU usage is due to polling the GPIO ports. There is a new version of the hardware in the works that will change this.

I should also say, this has nothing to do with the 74LS chips... It's how polling the GPIO works for now. See #127

Great!
Tested on Zero 2 and Zero 1.
Zero 2 25% of one CPU core.
Zero 1 50% CPU.
The biggest problem was that SSH just didn't work well with 100% busy on a single core CPU. So I think this is gonna work fine.
Not sure which Raspberry I used last time, because I cannot connect to my Zero 1. Maybe it's one without wifi. So cannot confirm that it really works better now. Sorry :(
ps. you have to iterate through the update process https://github.com/tjboldt/Apple2-IO-RPi/discussions/65 before you'll see any changes.

Unfortunately I noticed it made keyboard very sluggish and unusable in the shell... So I made another tweak that will go in shortly. It will consume more CPU but not as bad as before the first fix.


How to add an imprint

First, thank you so much for this beautiful theme. It's a lot of fun to create a website based on this theme and Hugo.
Question
In Germany it is mandatory, because of §5 of the TMG, to add an imprint. I don't understand how I can add an additional page to hold this information. How can I create an extra page? I'm new to Hugo, so please excuse this newbie question.

Duplicate of https://github.com/hugo-toha/toha/issues/83

The solution is explained in this PR


Rewrite database model

This is a refactor of the DB to support new requirements. Migration and models have been updated.
THIS IS INCOMPATIBLE WITH PREVIOUS DB MODELS - EXPECT THINGS TO BREAK!

Shelving for now due to change in requirements.


Change submodule to fork

Hi, and thanks for the pull request!  As per our private discussion, I've decided to update to the latest upstream release (which uses meson/ninja) instead.  Closing.


WW-5334 Misc VelocityManager code cleanup

WW-5334

Hmm we need to rethink the Maven module setup to avoid cyclic dependencies

Hmm we need to rethink the Maven module setup to avoid cyclic dependencies. I'll create a separate PR addressing this.

What do you mean by that? I was planning to drop support for Portlets in Struts 7.x

What do you mean by that?

Since struts2-junit-plugin depends on other plugins (such as struts2-convention-plugin, struts2-portlet-plugin, struts2-velocity-plugin), it's impossible to use struts2-junit-plugin within those plugins to write unit tests (it creates a cyclic dependency). We either need to move the unit tests for these affected plugins into their own modules (quite odd), or what would be better is to not have struts2-junit-plugin depend on any other plugins.

I was planning to drop support for Portlets in Struts 7.x

That's fine, I'll try reorganise the modules slightly until then - otherwise I can't unit test struts2-velocity-plugin!

Will rebase and reopen after review and merge of #732


Update zero-hour-auto-purge.md for HC Phish actions

Customers were requesting clarity on the ZAP action for Phish vs. High Confidence Phish

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks


add MCP server badge

This PR adds a badge for the mcp-server-duckdb server listing in Glama MCP server directory.

Glama performs regular codebase and documentation scans to:

Confirm that the MCP server is working as expected
Confirm that there are no obvious security issues with dependencies of the server
Extract server characteristics such as tools, resources, prompts, and required parameters.

This badge helps your users to quickly asses that the MCP server is safe, server capabilities, and instructions for installing the server.

Thank you for your contribution! The addition of the MCP server badge looks great.


Gem swap with zero value

Hey I'm seeing a transaction with a value of 0 even though it shouldn't be, not clear what the problem is
tx 0x091fb66ab1c522b613cc06e9baf870951c71a220d013af8eb9227d98c6f8cdd8
will try to figure this out on my own if I can but the repo has changed so much I'm not sure where its failing.

Hey I'm seeing a transaction with a value of 0 even though it shouldn't be, not clear what the problem is
tx 0x091fb66ab1c522b613cc06e9baf870951c71a220d013af8eb9227d98c6f8cdd8
will try to figure this out on my own if I can but the repo has changed so much I'm not sure where its failing.

This error is caused by the Sudoswap event in the Gem transaction.
I am parsing Sudoswap events by decoding the transaction's input data function.
However, the Gem transaction relies on decoding log events.
I can't think of a way to decode Sudoswap log events right now. 🫠


undefined reference to 'gEfiLoadedImageProtocolGuid'

While linking SELoader I get the following error:
Efi/Lib/libSELoader.a(Image.o): In function LoadImage':Image.c:(.text+0x26): undefined reference to gEfiLoadedImageProtocolGuid'
x86_64-poky-linux-ld:Efi/Lib/libSELoader.a(Image.o): relocation R_X86_64_PC32 against undefined hidden symbol `gEfiLoadedImageProtocolGuid' can not be used when making a shared object
x86_64-poky-linux-ld: final link failed: Bad value
I can see why this happens:
In Src/Efi/Lib/Image.c EfiLoadedImageProtocolGuid declaration is dependent on:
#if GNU_EFI_VERSION <= 303
Then in LoadImage(), EfiLoadedImageProtocolGuid is accessed unconditionally:
Status = EfiProtocolOpen(gThisImage, &gEfiLoadedImageProtocolGuid,
(VOID **)&LoadedImage);
Causing the undefined symbol.
Therefore if the GNU_EFI_VERSION is > 303, linking will fail.
Is this the correct behaviour? Should versions of gnu-efi above 3.0.3 be supported with some alternative? Or is it intended, that only versions below 3.0.3 are supported (despite Mak.env having a default of 3.0.5)?

I definitely miss this for a while :(
gEfiLoadedImageProtocolGuid is defined since gnu-efi 3.0.4 rather than 3.0.3. I guess you are using gnu-efi 3.0.3. This issue is fixed by https://github.com/jiazhang0/SELoader/commit/9c2723afae0d61f3a1bd6f11e81b5855ec466c6a.

gEfiLoadedImageProtocolGuid is defined since gnu-efi 3.0.4 rather than 3.0.3.

Ah, that makes sense. At the time I wasn't very familiar with the codebase, and was unsure why the define would only be for versions < 3.0.3.

I guess you are using gnu-efi 3.0.3

Yes, for now. I'm building this through Yocto, and I'm not sure when we'll be upgrading.

This issue is fixed by 9c2723a

Thanks!

I believe this issue has been resolved. Plz reopen this if you have further issues. Thanks.


Upcoming events section [Improvements]

Hello,
Just letting you know that the "Upcoming events" section of Polkassembly is tricky to navigate as the dates are all mixed up and not displayed chronologically.

It would be good to see upcoming events only, and not past events.
Thanks!  🤝🏿

Added in https://github.com/Premiurly/polkassembly/pull/208


Refactor queries to include order direction in ListAgents, ListComponents, and ListServices

Proposed changes

Update order direction to desc for Components, Agents, Services.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

down arrow maybe

good catch, I hadn't noticed this on other chains. will see what's causing this.
currently ticket is just to sort by 'desc', happy to make this dynamic so user can click on the header, thoughts here @oaksprout?


Attribute admin_rpamodifiedon is a String, but a DateTime type was specified

Hi,
While importing the latest version (CenterofExcellenceCoreComponents_1_70_2_managed) of CoE core components, I am getting error as Attribute admin_rpamodifiedon is a String, but a DateTime type was specified. My current version of core components installed is 1.39 which is a managed solution.
Could you help me to resolve the issue?
Thanks.
Devendra

Hello. Can you please use the interim solution provided in #491 ?
You will install that version, which will remove the field. And then install 70_2.
Thanks for using CoE
Duplicate of #491

Hi @JeneferM-MSFT!
I am experiencing the same issue. My current version is 1.60, I tried installing 1.70 and 1.70.2 with same error message.
Unfortunately I cannot install the version from #491 since it is an earleir version from the one I have.
Many thanks for your help !
Victor

Victor would you mind creating a separate github issue for that one? I'll create a fix for you and that version there.

Hi Jennifer !
Just did it. Issue #515
Many thanks again


Distributed Tests: Test run 'XXX' is in 'InProgress' state for 'Run Functional Tests' task

Hi, I have a build running coded UI tests in a distributed environment.Tests were running perfectly, once we upgraded to TFS 2017 Update3 RTM we have weird issue I see that 'Run Functional Tests task' is not able to run the tests. In the console log I see only this message "Distributed Tests: Test run 'XXX' is in 'InProgress' state", I see the same message for hours and hours until I cancel the build.But some times tests are running perfectly, but some times are not executing. I have attached the screen shot.
Please provide your  inputs on how to triage this issue?.

Hi.. Me too facing the same issue.
-- Chary

@msathya20 can you please provide the DTAExecutionHost logs and the build logs with system.debug set to true and also log into the test machines when this happens and see if an error dialog or something is blocking the tests from continuing on the machine.

@bangaruchary please follow the above steps and share the logs, we'll look into this.

Hi Shreyas
Please find DTAExecution log attached for your analysis.
We have TFS2017.Up3 server (Recently migrated this collection from TFS2015.3 to latest, since then we are observing this issue.)
Test agent and build agent are different machines configured with same account.
Test Agent has VS2013 and TestAgent 13 installed.
Lab Network Service and Labe Agent Service both are running on Test Agent.
DTAExecutionHost.exe.log
-Chary

@bangaruchary we have deprecated DTA RFT task and are asking people to move to the VsTest v2 task.
In tfs 2017 Update 3, the VsTest task has the same distribution capability as DTA in the Release pipeline.
And all our bug fixes and improvements after 2017 Update 3 went into only the VsTest task as we deprecated DTA RFT as there were known limitations with it..
By looking at your build definition it seems like it'll be easy to convert it to run the same tests in the Release pipeline wherein you can use the VsTest v2 task. The VsTest V2 task is more resilient and has better logging and should fix your issue (if not fix will atleast help us get to the RCA). Our hands are tied when it comes to fixing bugs in RFT DTA as it is deprecated and the codebase is frozen.
Try running the tests in the release pipeline and if you need any help in this let me know and I can assist you.

Hi All,
Sorry for the delay..
I have configured VsTest V2 task in my build definition. builds are succeeded, but tests are not executing in VM. I have tried command line execution by using vstest.console.exe, tests are executing perfectly in local Machine. While executing build in  VMs i am getting below  warning message.
"No test assembiles found matching the pattern:**\WinEmbedded.dll"
I have changed couple of patterns but no luck.
Can you please provide the resolution for this issue??

Best Regards,
Sathya

@msathya20 can you send screen shots of your build definition showing clearly all the options in the vstest task and also the copy files tasks that copy your dlls.
After this set system.debug=true (if you haven't already) and send us the build logs zip file.
I believe its a simple matter of your dlls being in a different folder from where the vstest task is looking for it.
Your search folder input should point to the folder in which your dlls are present on the VM.
The debug logs zip should enable me to point out the search folder and pattern you should be using to run your tests.

Hi Shreyas,
I have attached build definition step by step screen shots and log files.
VSTest.zip
Can you please check and provide the solution for avoiding this issue??
Best Regards,
Sathya

Hi @msathya20
So couple of things you need to know/do.

With VsTest task you no longer need another test agent machine. Tests run on the same machine on which you have your build agent.

Remove all the copy tasks you have.

Change the search folder input in the vstest task to $(System.DefaultWorkingDirectory)

This should get your tests running.

@msathya20 okay here are the steps:
Your build definition will simply build the dlls and then upload them as artifacts. (you can use the default template for building .NET apps it already has the build and upload artifact steps)
You can remove the vstest task from the build definition
Setup vsts-build agents on the VMs you want to run the tests on.
Create a new Release definition that downloads the artifacts (binaries) uploaded by the build and uses the VsTest task to run the tests. This release definition should use the agent pool that contains the agents running on the VMs you want to run the tests on.
You build definition now simply builds binaries and uploads -> this should run on the agent pool containing the agent where you want to build.
https://blogs.msdn.microsoft.com/devops/2017/03/26/vstest-task-dons-a-new-avatar-testing-with-unified-agents-and-phases/
But may I know why the tests need to run on those VMs specifically. Also why don't you simply build the binaries that VM and run the tests there itself. That should be waaay simpler. All you'd need to do is setup a vsts-agent on your VM and use that to run the same definition as described in previous to previous post. (Unless you don't want to install VS on those VMs in which case you won't be able to build there.)

@msathya20 can we have a quick call? Send a mail to<EMAIL_ADDRESS>with you skype id and I'll reply with a convenient time to have the call. We can resolve this speedily.

Hi Shreyas,
I sent mail to specified mail address,,but it seems mail was not delivered. Please find the Skype Id below and let us know your free time for a discussion.
ID-<EMAIL_ADDRESS>Regards,
Sathya.M

Had a call and was able to get the test running.

Closing this thread. Please reopen if you are still facing issues.


Clean command for stack

Right now the lib folder is not cleaned before building and as a files are removed / renamed it leaves behind the old files in the lib folder.
This may need a quick conversation to talk through how this interacts with file watches and we may need to clean the public folder as well. For development I don't think we see the webpack files but when do a prod build it may not hurt to have it ensure those folders are deleted and recreated.

@lxndrdagreat do you see an issue with this? I'm thinking we do something similar to the unthink-cli and bring in shx package rm lib/public on each build.

Yeah, I think shx is a good way to go. Since we're bringing in shx, can we look at replacing the copyfiles lib (used to copy content/public folders) with cp via shx?

Yeah it should be able to replace that easily and no need to have both.


Clean up a bunch of stuff

I ignored node_modules, so you'll have to add a build command to heroku (it's way better to add a build command instead of versioning all of node_modules, which is like 10000 files)

Thanks, I still have a lot of refactoring to do haha.

Thanks, I still have a lot of refactoring to do haha.

This pull request is objectively worse than the current iteration of the legendary KannaKamuiBot

This pull request is objectively worse than the current iteration of the legendary KannaKamuiBot

This pull request is objectively worse than the current iteration of the legendary KannaKamuiBot

How dare you

This pull request is objectively worse than the current iteration of the legendary KannaKamuiBot

How dare you

Thanks, I still have a lot of refactoring to do haha.

The point of the pull request is that I forked the repository and already did all the refactoring, so all you have to do is accept the pull request and my commits will be merged into your repository

Thanks, I still have a lot of refactoring to do haha.

The point of the pull request is that I forked the repository and already did all the refactoring, so all you have to do is accept the pull request and my commits will be merged into your repository

Yea I will, I just wanted to look at the changes first.

Yea I will, I just wanted to look at the changes first.


[Hot Fix][Tock Studio] : Faq creation fix

Fixes a bug introduced by the merge of two recent PRs

Ping @assouktim


fix: cancel background stream job if table fragment not persist

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Resolve #14104
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Sadly it's not the root cause of #9712 , still need to investigate.

Sadly it's not the root cause of #9712 , still need to investigate.

Yeah, I was just about to ask why this can cause a freeze of CN🤣


Initial implementation of IngressPlugin support

What about have a storage properties accessible from "plugin master class" that enables to storage plugin state on kafka? Like schema registry stored on a kafka topic in my case? Sure I can do it from the plugin impl. Just wanted your opinion on a generic way for ingresses plugins to store a state.

IngressPlugins do not require that Kafka be present as they can be deployed on the Standalone version, so this is something that only specific plugins should handle.

Got it. Thanks.

Would it make sense to chain several plugins instead of restricting to only one?

If daisy chaining plugins is needed, the idea is to have a daisy chaining plugin, but from the Ingress point of view, only a single plugin is known. The daisy chaining plugin could decide to implement various strategies for deciding if processing should continue or not for example. The general principle is to keep Ingress simple and delegate any advanced stuff to plugins.

Good point.


metric_learning的损失函数如何应用在检索中，目前实现中都存在传labels给到度量损失函数

https://github.com/tensorflow/tensorflow/blob/r1.10/tensorflow/contrib/losses/python/metric_learning/metric_loss_ops_test.py

Hi!@frostjsy!We see that the issue template has not been filled, could you please do so as it helps us analyse the issue [tf version, steps followed before you ran into this error or stand alone code/colab gist to reproduce the issue faced].Thanks


Bug: React 19 - Removed Export

A secret internal export has been removed from React 19. Importing it causes builds to fail.
https://github.com/LegendApp/legend-state/blob/faa7595016d7859cba87f01ec3ae2995b88d650d/src/config/enableReactTracking.ts#L13
I haven't tested it, but this should fix things, i think.
import React from 'react';
// ...
// @ts-expect-error Internals
const dispatcher = React.__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED?.ReactCurrentDispatcher.current;

build failed after upgrade next.js to v15
enableReactTracking({ auto: true })

Sorry I haven't looked into this yet! The enableReactTracking feature depends on a hack using that secret internal import, which is why it's not recommended for production. I may be able to find a different hack to make 19 work, but since this method is an unreliable hack it really shouldn't be used anyway. I'll update the docs to discourage usage and mention it doesn't work in 19.
And then I'll see if I can find a new hack for 19. For now I'd suggest not using this method and using the observer pattern instead.


browserHistory/Memory history is deprecated, replace with createBrowerHistory from utils

passes the tests

also added some types

the history isn't required in tests at all, at the PR in upstream shows, but in out case, if we're to make it optional, it would require adding non-null assention or an "if" check, so I say we leave it as is, perhaps there will be some tests later that use it


Travis CI to deploy to AWS

Needs to only deploy to AWS if new release/tag...

Almost done (see: https://travis-ci.org/opensyllabus/osp-api/builds/265057851)! Just need to handle running new task to get new image since the deploy script only pushes to docker image repo.


Update the onnx-tensorrt submodule - CI to TRT7

Description
This PR updates the onnx_tensorrt submodule to the latest commit.

@mxnet-bot run ci [unix-gpu]

Thanks, Serge. How is this tested at the moment?

Thanks, Serge. How is this tested at the moment?

It was supposed to be tested in the tensorrt tests deleted here in this PR https://github.com/apache/incubator-mxnet/pull/18579 ( related to https://github.com/apache/incubator-mxnet/issues/18729 ). I don't understand why it was deleted.
Also, I am getting this error in the CI
[2020-07-17T18:52:46.759Z] ninja: error: unknown target 'onnx/onnx.proto', did you mean 'onnx/onnx-ml.proto'?

not sure of what needs to be changed in ninja

cc @leezu on the build.
I think the tests were removed as we are deprecating NDArray in 2.0 and there's additional work needed for supporting numpy operator for tensorrt. cc @eric-haibin-lin in case there is more context.

@mxnet-bot run ci [centos-cpu]

@leezu thanks, I missed those!
I reverted them back


feat: add htmlLanguageIds configuration

Description
Add htmlLanguageIds configuration
Additional context
I would like to use this extension on ERB template.

Thanks!


Notifications with new messages when I'm logged out

Summary
App is sending notifications with new messages, although I'm logged out.
Environment Information

Device Name: iPhone SE 2020
OS Version: 13.5.1
Mattermost App Version: v1.32.2

Steps to reproduce

Log in
Log out
Wait for new messages

Expected behavior
App can't showing notifications with new messages when I'm logged out.
Observed behavior
App is showing notifications with new messages when I'm logged out.

@danoxide when logging out probably there was a failure, thus the app logged you out the session remain active in the server.
This is intentional behavior to not have you be unable to switch servers and watnot...
To remove the session you can login to the webapp go to Account Settings -> Security -> View active sessions, and then force the logout for the device

Okay, but IMO when I want to log out I'd expect that I'll be logged out and the notifications were stopped. I think "active session" should only provide possibility to close session remotely.

True, but to ensure that is the case we will need to block the “logout” until the server acknowledge which can be a problem as well cause of connectivity. Basically in your case you would’ve been “stuck” and would’ve needed to try and logout again and again until successful.
If we weight the two I think that the frustration of not being able to logout is grater than continue receiving notifications, specially cause sharing your device with another person is somewhat an edge case in my opinion.

Closing some older issues. Please re-open this if the issue still reproduces on the latest version (v2.10),


Fixing the Multiplier Option

One of them had the old input multiplier instead of the drop down menu.
The other one had a misspelling that needed to be fixed.

One of them had the old input multiplier instead of the drop down menu.
The other one had a misspelling that needed to be fixed.


OS X - test tcp_connections::test::test_multiple_nodes_small_stream ... FAILED

Travis OS X - test tcp_connections::test::test_multiple_nodes_small_stream ... FAILED
$ rustc --version

rustc 1.3.0-nightly (4dfe7a16c 2015-07-30)

$ cargo --version

cargo 0.4.0-nightly (e0a82d6 2015-07-29) (built 2015-07-30)

For example > https://travis-ci.org/maidsafe/crust/jobs/73546406
test tcp_connections::test::test_multiple_nodes_small_stream ... FAILED

test tcp_connections::test::test_small_stream ... ok

test test::check_rust_unit_testing_is_not_parallel ... ok

test transport::test::test_ord ... ok

failures:

---- tcp_connections::test::test_multiple_nodes_small_stream stdout ----

	thread 'tcp_connections::test::test_multiple_nodes_small_stream' panicked at 'called `Result::unwrap()` on an `Err` value: Error { repr: Os { code: 24, message: "Too many open files" } }', ../src/libcore/result.rs:732

failures:

    tcp_connections::test::test_multiple_nodes_small_stream

test result: FAILED. 16 passed; 1 failed; 1 ignored; 0 measured

thread '<main>' panicked at 'Some tests failed', ../src/libtest/lib.rs:253

@rossmuir: we open many file descriptors in tests to increase the chance to detect some race (Rust can detect data races in compile time, but there are other kind of concurrency problems). Just increase the OS X limits for open file descriptors:
ulimit -n 1024

Thanks @vinipsmaker but I would rather find a fix that does not alter system-wide settings

What about,
If we are on osx we reduce the number of connections. As many folk will download and do cargo run, cargo test etc. then this is an issue.
However we add a CI ENV_VAR called CI or similar (I am sure there is one) where on the CI script we do increase this count and set the var, in the test check for this var and osx and if set run the increased connections.

I'm not sure who (and how) fixed the issue, but seems fine in the latest code.
Should I close the issue?

Yes I think this is OK now.
On Fri, Aug 28, 2015 at 11:16 PM, Vinícius dos Santos Oliveira <
<EMAIL_ADDRESS>wrote:

I'm not sure who (and how) fixed the issue, but seems fine in the latest
code.
Should I close the issue?
—
Reply to this email directly or view it on GitHub
https://github.com/maidsafe/crust/issues/228#issuecomment-135900217.

--
David Irvine
twitter: @metaquestions
blog:  http://metaquestions.me


rename posix to more generic name

As angr grows to emulate more environments, we need to genericize the ability for a program to interact with its environment.
Two options:

rename posix to supervisor and allow the plugin to implement its own logic for the things like file read/write, the program break, etc
rename posix to environment and move all the logic into syscallls, making the plugin a pure storage mechanism. This would involve making library functions that currently call directly into posix for syscall functionality instead of performing a syscall directly, something that I would kind of like to happen anyway.

other options would be appreciated :)

Why not breaking supervisor into filesystem, networking, etc.?

I'm not sure if it makes much sense to separate filesystem and networking (in terms of sockets and so forth), but I'm also not a big fan of supervisor and environment. What about os or system?

@zardus There are some systems that model file systems and networks differently. Personally I prefer to have them separate.

@zardus What in particular do you dislike about the two options in the OP and how would os and system be different?

Additional consideration for a name would be system, I would guess that at some point in angr's development this was the goal, since the class of the de-facto posix plugin is SimSystemPosix

Though my preference remains environment -- it works better even in the cases where you're emulating e.g. a kernel, or some firmware. We're interested in storing data about that which exists outside our analysis context, the environment in which this program lives.

If we go with environment, under which there are filesystem, networking, and other stuff, it might be difficult for users to replace individual components with their own implementation (since filesystem and networking and such are not state plugins anymore, but rather "environment plugins"). What do you think?

If we go for environment, then the logic for handling all those individual components will be removed from environment and pushed into syscalls. Users looking to customize environment-handling would do so on a syscall basis, so we would also want to improve our interface for hooking or replacing syscalls. Do you forsee any usability problems with that approach?

the logic for handling all those individual components will be removed from environment and pushed into syscalls.

Does that prevent us from reusing code (e.g. file system logics) between different platforms, like Linux and Windows?


Multi-channel support

This issue tracks the big picture of multi-channel support.  I'll update this later with a better summary.
The main items are:

API design
implementation for each backend

WASAPI
AudioUnit
PulseAudio
OpenSL
others (minimum changes to retain existing functionality, actual support will need to be contributed)

deal with code duplicating between cubeb and Gecko's AudioConverter
work out how this interacts with device switching (discussed in issue #167)

Right now, there's some discussion in pull #171 covering this that needs to be moved into this issue.

As a independent cross-platform audio library, it's better to handle the downmix/upmix inside cubeb, rather than Gecko's AudioConverter, to tackle the incompatibility of the channel number. However, it has to figure out what capability of downmix/upmix needed in cubeb. Is it need a general purpose conversion from m channels to n channels, where m≠n, or just some specific conversion like k channels to stereo, where 2 < k < 9, and stereo to 5.1 ?

Beyond the implementation for each backend, I think the general design should follow the below comments.

Multiple Channel Configuration
To support multiple channels, we need to know the following information.

Channel Layout
The different channel layouts may have same channel count. For example, 2.2 and 4.0 both are use 4 channel,  but 2.2 use front left, front right, side left, side right and 4.0 use front left, front right, front center, back center. Thus, it needs a way to distinguish which speakers should be used.

Channel order
Some platform requires channel order to setup the speakers, e.g., kAudioOutputUnitProperty_ChannelMap for AudioUnit on OS X, so we should use a vector to define the channel orders for different channel layouts.

The simplest solution is to use SMPTE channel layout(order) as the input format. By this, the order is fixed, so we can only handle the layout part.

Downmix and Upmix
Convert the audio data from  m channels input to n channels output, where m≠n. The stereo, 5.1, 7.1 are most common use cases, so we can only focus on m = 2, 6, 8 or n = 2, 6, 8 in current phase. MythTV is a useful reference.

This feature could be divided into several phases:

phase 1

Implement basic support for multiple channel on each platform.
Implement downmix for multiple channels. (There is a spec for audio 5.1 downmix, while I can not find the standard for upmix. Thus, the design of upmix could stay same in this phase.)

phase 2

Integrate with audio device switching.

phase 3

Add some fancy features, e.g., stereo-to-5.1 simulation.

Downmix/Upmix module
The downmix/upmix code should be separated into a independent module from WASAPI for following reasons:

It can be reused for all backends. The downmix/upmix is refered to some standards, so it should be same for each backend.
It's easier for testing. We can ignore the internal process of backends and just focus on the correctness of supported layout conversion.
It's flexible to extend. If we want to add some fancy features like stereo-to-5.1 simulation, then only this module needs to be modified, and it will be applied to all backends.

Agreed, we should separate it.

I think we could design three mechanisms for mixing:

Specific conversion
mapping channel data
bypassing data by channel index

Each time when we try to upmix or downmix, we need to try converting data with the above order. That is, we can try specific conversion first. If it wroks, then the job is done. Otherwise, we next try mixing by mapping the channel. If it still doesn't work, then we try mixing by bypassing the channel data. The final mechanism should be our fallback plan and it should always work.
Specific conversion
Some conversion has its own definition, so we need to implement this. For example, Table 2 in ITU-R BS.775-3 define the downmix equations from 3F2 to 1F, 2F, 3F, 2F1, 3F1 and 2F2.
Mixing by mapping channel data
In most cases, the input and output data can be mapped by its layout setting. For example, if we try downmixing from 3F(L, R, C) to Stereo(L, R), we only need to pass the first two input channel data to output.
Mixing by bypassing channel data
There is some case the above mechanisms don't cover. The downmix from stereo(L, R) to mono(M) is an example. There is no spec and there is no matched channel for this conversion. Especially, WASAPI can support some unmatched speaker settings like 6 channels with stereo layout(stereo should only has 2 channels). In such case, we don't know the mixing policy should follow the layout or channel number.
The simplest plan is to follow its channel numbers. If the input has 2 channels and output has 1 channel, then we just need to pass the first data to the output. We just need to pass the channel data by channel index.
An alternative way is to define some matrices to compress/expand the audio data. However, the combination is not a small number.

For testing, I am wondering if it's feasible to fake an audio device and programmatically register it as default audio output. Then we can intercept and verify the output through the faked device. Maybe this issue should be discussed in #193.


UISlider and containerView problem

Hi, i am using ButtonBarPagerTabStripViewController which has 3 VCs. Each of them has table view and some cells have UISlider inside them. The problem is if i try to swipe faster on the slider thumb the containerView is scrolling and not respecting the slider. It works correctly only if press the slider thumb and wait a little and then move it. I tried many ways like
containerView.delaysContentTouches = false containerView.panGestureRecognizer.delaysTouchesBegan = containerView.delaysContentTouches containerView.canCancelContentTouches = false
and other stuff that i found on SO but still doesnt work correctly. Do you have any idea how to fix this ?

Anyway, i fixed it by using custom scroll view and overriding gestureRecognizerShouldBegin() method, where if the touch is inside slider dont begin scrolling.


Aborted key derivation returns PSA_ERROR_NOT_PERMITTED instead of PSA_ERROR_BAD_STATE

psa_key_derivation_output_key() returns PSA_ERROR_NOT_PERMITTED instead of
PSA_ERROR_BAD_STATE when called after the operation was aborted.
This causes a failure in test_c021 of the PSA compliance test suite.

psa_key_derivation_output_key returns NOT_PERMITTED unless a key has already been passed as input. I agree that as long as no algorithm has been set, BAD_STATE would be a more appropriate error code. This would mean adding
if( operation->alg == PSA_ALG_NONE )
    return( PSA_ERROR_BAD_STATE );

before the can_output_key check.
If psa_key_derivation_setup has been called but no SECRET input has been passed yet, it's still unknown whether the operation will permit outputting a key. So arguably BAD_STATE would still be a better error code in that case. However, I don't think it's worth the added complexity to change this case.


Chunked uploads for WeedCOCO zip files with Uppy + tusd

Replaced the WeedCOCO zip uploader with Uppt/tusd which use the tus protocol for chunked uploads.

Closes #584

This review also deserves a little attention @spikelynch

This CI failure is a different issue with Set Form that we have seen elsewhere but I don't understand. Set Form appears to be clicked, but then the dialog does not close. I think we introduced wait(10000) (which is a very long wait) precisely because the Set Form button was not effectively closing the dialog. Argh! I implemented the form dialog when I was pretty naive about react, so it's possible that, for instance, there are concurrency issues and I should have used useEffect somewhere..?

congrats @spikelynch, this is a big one :)


Composer 1 vs 2: Adds extra sites/ directory for Drupal 7 installation

Thanks in advance for any effort at all on this. :-)
I use composer-installers to put Drupal stuff where it belongs. See my minimal composer.json below for an example of how this works.
Using Composer 1, this works as expected: Drupal core is placed in docroot/ without any problem.
Using Composer 2, everything works as expected again, except for one extra bit of path is inserted to docroot/sites/sites/. It should only be docroot/sites/.
This doesn't make a lot of sense because the whole package should just be placed within the custom directory. This is the only directory that has any special cases... Other directories within drupal/drupal are not misplaced.
This issue only happens with custom path configurations for both drupal-core and drupal-module. I would expect that if there were a problem, it would be with one overwriting the other, not adding a bonus directory.
% composer show composer/installers | grep versions
versions : * v1.10.0
% composer --version
Composer version 2.0.11 2021-02-24 14:57:23

composer.json:
{
    "name": "foo/bar",
    "type": "project",
    "config": {
        "sort-packages": true,
        "platform": {
            "php": "7.3"
        }
    },
    "require": {
        "composer/installers": "@stable",
        "drupal/ctools": "1.17",
        "drupal/drupal": "7.78"
    },
    "extra": {
        "installer-paths": {
            "docroot": [
                "type:drupal-core"
            ],
            "docroot/sites/all/modules/contrib/{$name}": [
                "type:drupal-module"
            ]
        }
    },
    "minimum-stability": "dev",
    "prefer-stable": true,
    "repositories": {
        "drupal": {
            "type": "composer",
            "url": "https://packages.drupal.org/7"
        }
    }
}

Can confirm: composer selfupdate --snapshot cures this.
I hope 2.0.12 gets released soon. :-)

It's out now


HamburgerMenu and custom style

I'm binding HamburgerMenu.NavButtonCheckedIndicatorBrush in xaml but it doesn't have effect because of HamburgerMenu.RefreshStyles which overrides it. What about setting properties in RefreshStyles only if ReadLocalValue is DependencyProperty.UnsetValue?

Another great PR. Test it carefully though. :)


Return a clone of the DynamicIndex after inserting a new item into a factory

I just hit a use case where I need to store dynamic indices of new items in the models of other items in the factory, but there is no way to obtain a DynamicIndex right now, except through a connected event. I would like to suggest to return a clone of the dynamic index in all insert calls. For example, a clone of the dyn_index here:
https://github.com/Relm4/Relm4/blob/ae761b14064ead777137637db51a60158dbf1433/relm4/src/factory/collections/vec_deque.rs#L189-L220

Sounds good. This would be a good first issue, but if nobody wants to pick it up, I can create a PR for it too.


Docs: Add remark regarding pnp support

Closes https://github.com/storybookjs/storybook/issues/23654

What I did
This PR adds a remark regarding usage of the test-runner and PnP mode, assisting users to fix issues that might arise
Checklist for Contributors
Manual testing
This section is mandatory for all contributions. If you believe no manual test is necessary, please state so explicitly. Thanks!
Documentation

[ ] Add or update documentation reflecting your changes in this repository
[ ] Request documentation updates in the test-runner docs website

Checklist for Maintainers

[ ] Make sure this PR contains one of the labels below:

Available labels

skip-release: Skip any releases, e.g., documentation only changes, CI config etc.
patch: Upgrade patch version (e.g. 0.0.x)
minor: Upgrade patch version (e.g. 0.x.0)
major: Upgrade patch version (e.g. x.0.0)

@jonniebigodes you might be interested in this


Mod just...isnt working?

have both the fabric launcher and API installed. i downloaded the file directly into my mods folder like i usually do (if this could be the cause let me know? but i dont think it is? could be wrong, i have before).
The menu just...wont show up. lol. ive tried restarting the game multiple times and the menu is nowhere to be found. I'm assuming this means that it just somehow isn't working correctly? It just ... is refusing to play nice with the launcher or something for me. if anyone knows what's wrong and would walk me through lmk.

What do you mean? There's no button?

Adding a comment as I was having this issue, and resolved it.  If your mods folder is linked fabric does not seem to load the mods.  for example I am on linux and soft-linked ~/.minecraft/mods to ~/.minecraft/mods-fabric-1.14.4 the modmenu  mod never loaded, removing the link and just copying the folder fixed the issue


added timur's blog

Thanks for your help
However, please check if the PR fulfills these requirements

[x] Links are correct

[x] Github handle of user is specify, if not use LinkedIn link

[x] Entry is according to standard
- [Name](URL) - Resources description [@github-username](https://github.com/github-username)

Hi @timurgaleev, please fix a content, as Markdown-lint does not pass.


How to reproduce the HED-owt-ucm results for voc2012

Hi, I download the HED and gPb-owt-ucm code for reproducing the voc2012 segmentation segments. But I found that the gPb produce the 8-orientation boundary probabilities, which is needed in owt, while HED not. So I wonder how can I input the HED result into owt algorithms.
Thanks!

Hi Dingry,
Since there hasn't been a single codebase that does the job, I used two toolboxes:

To generate oriented contours.  Piotr's Matlab Toolbox:  https://github.com/pdollar/toolbox
To generate owt-ucm.  MCG:  https://github.com/jponttuset/mcg
Here's a snippet of code I put together (in Matlab) to generate HED-owt-ucm from HED.

  % Add Piotr's toolbox path.
  addpath(genpath('./toolbox/'));

  % Add MCG path.
  addpath(genpath('./mcg-2.0/full/src/ucms/'));
  addpath(genpath('./mcg-2.0/full/lib/'));

  % Load HED contours as variable E.
  ...

  % Calculate orientations of contours.
  [Ox,Oy] = gradient2(convTri(E,4));
  [Oxx,~] = gradient2(Ox);
  [Oxy,Oyy] = gradient2(Oy);
  O = mod(atan(Oyy.*sign(-Oxy)./(Oxx+1e-5)),pi);

  % Perform oriented watershed transform.
  [owt2, superpixels] = contours2OWT(E, O);

  % Perform spectral globalization.
  sPb = spectralPb_fast(owt2);
  sPb = (sPb - min(sPb(:))) / (max(sPb(:)) - min(sPb(:)));

  % Calculate ucm2.
  ucm2 = double(ucm_mean_pb(owt2 + sPb, superpixels));

  % Calculate the final segmentation given threshold th.
  labels2 = bwlabel(ucm2 <= th);
  labels = labels2(2:2:end, 2:2:end);

Hope this helps.
Best,
Jyh-Jing

Thanks! It works!

"% Load HED contours as variable E."
I am generating the HED contours with my own pytorch model. What format is needed for your algorithm to work? E.g. HxW uint8? Or WxH float?

Hi @ifeherva ,
It's the same as the input images, which are HxW uint8.
Jyh-Jing

Thanks a lot, I got it to work as well. (Had to convert E to single otherwise conv did not work).
What was the threshold you used (th)? I vaguely remember 0.3, but I might be wrong.

Glad to know it worked for you!  (Yes, conversion to single would be required for some platforms.)
We used 0.95 for the ucm threshold.

I still have troubles reproducing the cluster labels. I am using matlab2020a and I do not really understand what is going on, but I definitely dont get meaningful results using the code above.
In fact 'ucm2' has a bunch of zeros where the clusters should be and the 'edges' are superhigh values like 53 or 191 so the check against .9 does not make any difference. Thus bwlabel creates hundreds of clusters. I think the thresholding is not working properly so I get too many edges, do you have any idea where the bug might be? maybe the values in ucm2 are not normalized (in my matlab version)? Thanks a lot!

Update: owt2 needs to be normalized before calling ucm_mean_pb(). Fixing that seems to be producing the same output as published.

% Load HED contours as variable E.
I am using another contour detector instead of HED. I am loading these images as H X W float (uint8 gave me an error in convTri(E,4))
However, upon running this script with various threshold values (0.5, 0.4, 0.2, 0.1, 0.05), I always end up with very few labels (mostly 1 with very few 2 and 3).
Here are few sample images I am loading in E. Could you tell me what could be going wrong?
I tried normalizing owt2 as @ifeherva suggested but that doesn't help too.

% Load HED contours as variable E.
I am using another contour detector instead of HED. I am loading these images as H X W float (uint8 gave me an error in convTri(E,4))
However, upon running this script with various threshold values (0.5, 0.4, 0.2, 0.1, 0.05), I always end up with very few labels (mostly 1 with very few 2 and 3).
Here are few sample images I am loading in E. Could you tell me what could be going wrong?
I tried normalizing owt2 as @ifeherva suggested but that doesn't help too.

Hi @ashwinipokle,
It's hard to know what went wrong by looking at these two images--they seem totally fine.
I'd suggest you print out the histogram of E, normalized owt2, and normalized sPb.  I guess lots of contour intensities are cluttered at a very low range?
Regards,
Jyh-Jing

Hi @ashwinipokle,
It's hard to know what went wrong by looking at these two images--they seem totally fine.
I'd suggest you print out the histogram of E, normalized owt2, and normalized sPb.  I guess lots of contour intensities are cluttered at a very low range?
Regards,
Jyh-Jing


Add gazebo msgs: Capsule and Cone

Desired behavior
Related with this PR https://github.com/ignitionrobotics/ign-rendering/pull/289.
Capsule and Cone are defined in ignition::msgs but not in gazebo::msgs https://github.com/osrf/gazebo/tree/gazebo11/gazebo/msgs
The objective is to add these msgs to ignition::gazebo and uncomment the code in the PR.

I don't think we can change the message definitions in Gazebo classic at this point, because it would break ABI. I don't think we should add capsule and ellipsoid support to Gazebo classic at this point.


Please allow setting of pod-max-pids via UI

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Request:
The Rancher UI currently provides UI support for project- and workload-level resource limits for CPU and memory. But it's not possible through the Rancher UI (except by editing cluster.yaml directly) to set the --pod-max-pids option for the kubelet, which limits the number of PIDs that each pod can utilize concurrently.
As this is an important limit to set in multi-tenant clusters, to prevent fork-bomb pods from wreaking havoc in the cluster, we propose that the flag be promoted to a first-class UI widget in the cluster creation and templating system.

Note that if Rancher makes this a first-class configuration option, that this will also require some additional configuration updates in the nginx ingress controller.  By default, the nginx ingress controller creates 64 threads per worker, and one worker per core.  Every thread consumes one PID.  So even for very generous values of --pod-max-pids (e.g. 1000), it does not take many CPUs before nginx consumes all available PIDs.  For example, on a system with 40 cores, nginx requires over 2,500 PIDs.  Setting the nginx worker-processes option in the ConfigMap to some reasonable value (such as 4 or 8) by default will ensure that the ingress controller won't fail to start.


Prevent possible infinite loop in setting active snippets

yankpad-set-active-snippets can enter an infinite loop if there are categories that contain no snippets.
By adding a private function for appending snippets from a category, it can be prevented.

Thanks!