SNOW-1665766: Add support for VECTOR_L1_DISTANCE

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
<!---
In this section, please add a Snowflake Jira issue number.
<p>Note that if a corresponding GitHub issue exists, you should still include
the Snowflake Jira issue number. For example, for GitHub issue
https://github.com/snowflakedb/snowpark-python/issues/1400, you should
add "SNOW-1335071" here.
--->
Fixes SNOW-1665766

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.
[ ] I acknowledge that I have ensured my changes to be thread-safe. Follow the link for more information: Thread-safe Developer Guidelines

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

I have read the CLA Document and I hereby sign the CLA


Node doesn't display

Topic doesn't visible after added this line;
rcl_node_options_t node_ops = rcl_node_get_default_options();
node_ops.domain_id = 30;
 RCCHECK(rclc_node_init_with_options(&node, "micro_ros_arduino_node", "", &support, &node_ops));
in the ardiuno code
I ran this line
ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/serial/by-id/usb-Arduino_LLC_Arduino_Due-if00 -v6
ROS_DOMAIN_ID=30
then re-upload to ardiuno and I get no luck. Restart the laptop doesn't help either :P

The top sourced in microros_ws along with the bottom left side. The bottom right side sourced in /opt/ros/foxy just to make sure if it can view the node.
Followed this and I did the same thing, not sure where to take a look now even clear the ros domain in .bashrc.
Is it a new bug or is it something that I missed?

Hardware description: Ardiuno DUE
RTOS: from the tutorial list
Version or commit hash: Foxy

Expected behavior
Display "micro_ros_arduino_node"" node
Actual behavior
Doesn't display the node

If you don't change the domain id everything works correctly?

Nope, even if I didn't change the domain id.
I recently commented this line out and comment the ros_domain in .bashrc out as well too then open a new terminal/reupload to ardiuno due. No luck.

Try to not use Serial in lines like:
  Serial.begin(9600); // Starts the serial communication

This serial port is being used by the micro-ROS client to communicate with the agent. Let me know if deleting all Serial from your code the agent receives data.

Unfortunately,
I deleted that line and this too as well;
Serial.print("Distance: ");
Serial.println(distance);
No node displaying either. Agent nor ros2 were able to receive data.
Question, was it supposed to be
ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/serial/by-id/usb-Arduino_LLC_Arduino_Due-if00 -v6
or should I run at
ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyACM0 -v6 (As long as the port match, that is)
I tried both and no luck so far

Both agent command should work. Have you tried to use one of the basic examples provided with this library? Does the basic publisher work?

Actually since you mentioned that now, it didn't work ever.
It worked with this but I know this tutorial isn't intend for this library.
I will start over on micro ros and see if the result is same. I will let you know by tomorrow.
Truly appreciate your time and efforts!

Update:
So, I went to release list to download the latest. Apparently, I was on .02v instead of v1.
Before I download the zip, I went ahead and deleted the folders for micro ros ardiuno in ardiuno by going to library and delete them. After that, I added the ZIP to ardiuno and tested the example. Unfortunately, it's not working as I hoped.
So, I started to wonder if it was the agent issue so I tested the agent. It worked

So, as you can see that the right bottom corner isn't picking anything up from the agent due to source /opt/ros/foxy/setup.bash
That leads me to wonder if I need to source in micro ros ardiuno?

I ran colcon build in micro_ros_ardiuno folder then source install/setup.bash inside the folder.
Still no topic visible

I'm kinda out of ideas to troubleshoot now. I set ros domain to 10 due to the code:

Still no luck.
Any new idea or suggestion? @pablogs9

Paste here the ouput of the agent but using the -v6 flag

Like this?
I'm not familiar nor skilled with agent yet

yes, but with the micro-ROS client being connected

hm, I don't think I have that one in any of my steps. I thought it was just agent. Can you tell me the command to run that?
Apparently, I only have micro_ros_agent as in my package.

the agent but with your board connected an communicating as in https://github.com/micro-ROS/micro_ros_arduino/issues/223#issuecomment-823108560

Like this? (the left top corner isn't being used like other 3.
How do you connect to the board using udp4 or rather serial? I tried with the serial and it wouldn't run
Such as this
bwuk@robots:~/microros_ws$ ros2 run micro_ros_agent micro_ros_agent udp4 --dev dev/serial/by-id/usb Arduino_LLC_Arduino_Due-if00 -v6
Warning: '--port <value>' is required
Usage: 'micro_ros_agent <udp4|udp6|tcp4|tpc6|serial|pseudoterminal> <<args>>'
For a more detailed description about all the available arguments, please execute the agent with '-h/--help' option.

Bingo! Found! Here is the output

The error part is from CPU reset on ardiuno due. We can see it doesn't connect to ros2

I'm new to micro-ros and was having the same issue as OP where ros2 node list and ros2 topic list showed nothing, and then discovered that rqt_graph does show a  /micro_ros_arduino_node even when ros2 node list shows nothing. I'm not sure why this is the case, but I thought I'd mention this in case it's helpful.
Agent OS: Ubuntu 20.04
ROS2 distribution: Foxy
Hardware: Teensy 4.1
Teensy client code: micro-ros_publisher.ino
Steps to reproduce:

Load client code onto Teensy
unplug USB cable between Teensy and Linux box running the Agent
plug USB cable back in
Startup agent using the following command within 2 seconds of plugging in USB cable: ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyACM0 -b 115200  -v 6
In another terminal
-- source install/setup.bash
-- rqt_graph
-- select Nodes only and click Refresh ROS graph button

Also, while writing this report I discovered that if I killed all background ros2 processes, and then ran the agent and rqt_graph, I was able to get ros2 node list and ros2 topic list to show expected output. While perhaps there is a workaround here, it does not seem like expected behavior....

The steps you provide isn't working for me, unfortunately.
I had to re-install everything on this part. Here is what I've done so far
Between Teensy and Arduino:

Plug the micro USB to Teensy 4.1 (the board should have an orange blinking)
Press the button on the board.(The blinking should stop)
Open terminal and follow this list:

wget https://downloads.arduino.cc/arduino-1.8.13-linux64.tar.xz
wget https://www.pjrc.com/teensy/td_153/TeensyduinoInstall.linux64
wget https://www.pjrc.com/teensy/00-teensy.rules
sudo cp 00-teensy.rules /etc/udev/rules.d/
tar -xf arduino-1.8.13-linux64.tar.xz
chmod 755 TeensyduinoInstall.linux64
./TeensyduinoInstall.linux64 --dir=arduino-1.8.13
cd arduino-1.8.13/hardware/teensy/avr/cores/teensy4
make
cd ~/arduino-1.8.13/
sudo ./install

Ardiuno and Micro-ROS:

Download latest zip from (Micro-ROS)[https://github.com/micro-ROS/micro_ros_arduino/releases/tag/v1.0.0]
Open Arduino IDE
Click Sketch > Include library > Add Zip > [the zip you downloaded]
cd [Ardiuno PATH]/hardware/teensy/avr/
curl https://raw.githubusercontent.com/micro-ROS/micro_ros_arduino/foxy/extras/patching_boards/platform_teensy.txt > platform.txt
Open a file called "Micro-ros_publisher.ino" from example in Micro-ros library
Click Download in Arduino IDE

WIth the agent:

Download the code into Teensy
Unplug the board then plug and run it in terminal after source install/setup.bash in micro-ros directory
ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyACM0 -b 9600 -v 6
Open another terminal and source install/setup.bash then run ros2 topic list
Open another terminal and source install/setup.bash then rqt_graph
Click refresh node on it.

No luck :(

Also, while writing this report I discovered that if I killed all background ros2 processes, and then ran the agent and rqt_graph, I was able to get ros2 node list and ros2 topic list to show expected output. While perhaps there is a workaround here, it does not seem like expected behavior....

@KohlhardtC  Question, how do you kill ros processes?
For me, I used pkill ros since aux was able to detects micro_ros running in background however it doesn't display the topic to ros2 visibly.
Here is the picture

How are you running the micro-ROS Agent?
Where have you set the baud rate to 9600? By default it is 115200

1. How are you running the micro-ROS Agent?

2. Where have you set the baud rate to 9600? By default it is 115200

#1 answer:

I ran the agent by go in the micro-ros-ardiuno
source install/setup.bash
once the I plugged the Teensy, I run this
ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyACM0 -b 115200 -v 6

That's it.
Answer #2:
I actually changed it to 9600 since 115200 didn't work for me. The result is same. I was hoping 9600 will, magically, works :P

I just used the kill command. I don't know if the kill was the action that
helped make it possible to see topics/nodes or if something else is going
on. I'm guessing something else is going on since I haven't had any issues
with missing topics/nodes recently.
On Wed, May 12, 2021 at 7:42 AM Kakcalu13 @.***> wrote:

How are you running the micro-ROS Agent?

Where have you set the baud rate to 9600? By default it is 115200

#1 https://github.com/micro-ROS/micro_ros_arduino/pull/1 answer:

I ran the agent by go in the micro-ros-ardiuno
source install/setup.bash
once the I plugged the Teensy, I run this
ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyACM0 -b
115200 -v 6

That's it.
Answer #2 https://github.com/micro-ROS/micro_ros_arduino/pull/2:
I actually changed it to 9600 since 115200 didn't work for me. The result
is same. I was hoping 9600 will, magically, works :P
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/micro-ROS/micro_ros_arduino/issues/223#issuecomment-839829407,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOTLLOMVXB7ST3D7OLKD3TNKHV5ANCNFSM43FZUR5A
.

I was thinking about this some more, and I starting to wonder if maybe the problem is ros2 specific and not micro-ros specific?
For example, I googled "ros2 node not discovered node list" and found something like this can happen if there are network configuration issues or the ROS_DOMAIN_ID environment variable is set.

I was thinking about this some more, and I starting to wonder if maybe the problem is ros2 specific and not micro-ros specific?
the ROS_DOMAIN_ID environment variable is set.

This made me REALIZED...Since I re-install arduino due to multiple paths. I forgot to add this part since my old file was modified with this part. The new didn't have this!
rcl_node_options_t node_ops = rcl_node_get_default_options();
node_ops.domain_id = 10;
RCCHECK(rclc_node_init_with_options(&node, "my_node_name", "", &support, &node_ops));

from #21
then I inserted the code above and bam! It's working!!!!!

Thank you so much for your time!   @KohlhardtC @pablogs9
Since this issue is now resolved.
To anyone who is trying to get this working, feel free to @ me in a new issue!


chore: prerelease

Description
Problem*
Resolves
Summary*
Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Can you point to the issue that this is trying to resolve and put a more descriptive title?

Resolves https://github.com/noir-lang/noir/issues/3316


RoundedRect EyeShape (and PupilShape) with custom corner radius

Hello,
First, the package is great-- thank you!
I'm aware the RoundedRect eye or pupil shape doesn't support defining a custom corner radius, although the pixel shape does. I'm wondering if this is a decoding limitation, or if it's possible with a custom generator?
Thanks again.

@matthew-dubois Hi! And thanks for the kind words -- glad to hear its been useful for you.
There is no technical limitation for this, either in this library or in the qr code spec.  Just remember that the more stylish the code the less readable it is.
Theres nothing to stop you from creating your own custom eye/pupil generators that define a customizable corner radius.
Cheers and thanks once again!

@matthew-dubois Hey mate,
I've just pushed 23.2.0 which includes a 'cornerRadiusFraction' on both the QRCode.EyeShape.RoundedRect and the QRCode.PupilShape.RoundedRect.
Let me know how you go.
Cheers!

Hello,
I just tried it, and it's exactly what I was looking for.
Again, thank you for the quick response and update!


[Fix] Metafile.yml: Fix typo of all config file path.

Motivation
While I look thorough the model zoo using mim, I found that there are some models failed to download because of the wrong config file path in metafile.yml (all weights links are OK btw.)
Related: https://github.com/open-mmlab/mim/pull/192
Modification
Fix all possible typos in metafile.yml. Now mim should be able to download all model configs without error!
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Delete account should show spinner

background disappears when pressing delete
should show spinner instead of trash icon while deleting

Checked on Dev, QA, and eWallet environments
Work as per requirements
Env:
Dev - https://www.screencast.com/t/I3UNDYGe
QA - https://www.screencast.com/t/DfujVKWZ3O
eWallet  - https://www.screencast.com/t/7S1QoyoT


Validate YAML on Bitbucket Server & Refactor the Validation

Changes

The main goal of this pull request is to better surface error logs to the pipeline as code developer on Bitbucket Server: https://issues.redhat.com/browse/SRVKP-6767.
Essentially, this pull request does three things in three commits:

Validates the YAML and prints a friendly message for better debugging purposes on bitbucket server as well. Bitbucket Server was not included in https://github.com/openshift-pipelines/pipelines-as-code/pull/1578.
Refactores the whole YAML validation to its own function and uses it in all providers.
Adds the bad YAML test for gitea as well so that we have tested it on all providers.

Submitter Checklist

[x] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[x] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[x] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

/ok-to-test

thanks for the contribution!
cc @zakisk

/test go-testing

/retest

@zakisk I don´t know why the e2e tests failed (I have not changed them). I rebased on main to get the latest commits and pushed again to run the tests.

@zakisk I don´t know why the e2e tests failed (I have not changed them). I rebased on main to get the latest commits and pushed again to run the tests.

it happens sometimes, let's wait for this new run!

@tricktron oh, I missed @chmouel 's comment.

@zakisk @chmouel Alright, I also had to fix the test/testdata/TestGithubPullRequestSecondBadYaml.golden file. Now all the e2e tests pass as well.

Thank you for your contribution ! @tricktron


Verify via tool

Aha.... Perhaps.
I'll dig tomorrow
On Mon, 29 Nov 2021, 22:01 Boris Pelakh, @.***> wrote:

@.**** commented on this pull request.
In onto_tool/onto_tool.py
https://github.com/semanticarts/ontology-toolkit/pull/92#discussion_r758769627
:

@@ -1181,7 +1183,7 @@ def format_value(v):
# Extend the width of each column to contain the longest value.
max_length = [max(a, b) for a, b in zip(max_length, [len(s) for s in as_text])]
rows.append(as_text)

row_format = " ".join(f"{{:{length}.{length}}}" for length in max_length) + "\n"

row_format = "||" + "| ".join(f"{{:{length}.{length}}}" for length in max_length) + "||\n"
rows.sort(key=lambda x: x[0])

@pwin https://github.com/pwin Could this sort be the reason your ORDER
BY didn't work for the table?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/semanticarts/ontology-toolkit/pull/92#pullrequestreview-818356643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIFYTELLAKVXNFLOYFFOS3UOPZ43ANCNFSM5IEMP66A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Slide upload functionality is missing from talk pages

Speakers need to be able to upload their slides to the EP2019 website, just like it was in previous year. This functionality is no longer available, it seems, e.g.
https://ep2019.europython.eu/talks/hYAEdqK-europython-2020-help-us-build-the-next-edition/
We will need to readd this, since we ideally want to have all slides available on our website:
https://ep2019.europython.eu/events/call-for-proposals/#Release-agreement-for-submissions

Should the link to the slides become available on the public talk page as soon as they're uploaded?


Add tests

Codeception tests will help ensure functionality for future updates.

Considering #7, we shouldn't invest time into adding tests for the StringUtility service, as it will be removed anyways.

Resolved by #9


Error handling

When using the fs reading mechanism with pipes, an error is thrown in strict mode if one of the rows does not match the length of the header. However, a try-catch block around the entire chain won't catch the error.  Here's an example:
try {
	fs.createReadStream(file_path)
	.pipe(csv({
		separator: ",",
		strict: true
	}))
	.on('headers', function (headers) {
		console.log("headers: ", headers)
	})
	.on('data', function (data) {
		// a row was parsed
		results.push(data)
	})
	.on('end', function () {
		//
		console.log(results);
	});
} catch (err) {
    // error parsing csv - it never gets here
}

The issues here aren't for support. Please read about error handling with streams, which is what you're using when you pipe.


[fix][docs] Remove redudant steps in docs gen

Motivation
There are some redundant information in current pulsar docs, as pointed out by @Anonymitaet. This PR aims to fix those.
Modifications

Remove redundant markdown list in CLI docs gen steps
Use markdown file for reference-configuration instead of links
Remove unused leading warning on some pages

Verifying this change

[x] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

Documentation

[x] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: SignorMercurio#5

LGTM
PTAL @urfreespace @tisonkun thanks!

@SignorMercurio feel free to ping me to merge if all tests pass.

Codecov Report

Merging #18132 (4bce553) into master (6c65ca0) will increase coverage by 35.92%.
The diff coverage is 50.00%.

@@              Coverage Diff              @@
##             master   #18132       +/-   ##
=============================================
+ Coverage     34.91%   70.83%   +35.92%
+ Complexity     5707      438     -5269
=============================================
  Files           607       26      -581
  Lines         53396     2246    -51150
  Branches       5712      245     -5467
=============================================
- Hits          18644     1591    -17053
+ Misses        32119      483    -31636
+ Partials       2633      172     -2461

Flag
Coverage Δ

unittests
70.83% <50.00%> (+35.92%)
:arrow_up:

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

Impacted Files
Coverage Δ

...apache/pulsar/proxy/server/DirectProxyHandler.java
63.63% <50.00%> (ø)

...pache/pulsar/proxy/server/ProxyServiceStarter.java
60.66% <0.00%> (-1.34%)
:arrow_down:

...n/java/org/apache/pulsar/broker/PulsarService.java

...pulsar/broker/admin/impl/PersistentTopicsBase.java

.../org/apache/pulsar/broker/admin/v2/Namespaces.java

...broker/delayed/InMemoryDelayedDeliveryTracker.java

.../pulsar/broker/service/AbstractBaseDispatcher.java

...rg/apache/pulsar/broker/service/BrokerService.java

...ache/pulsar/broker/service/EntryFilterSupport.java

.../service/SystemTopicBasedTopicPoliciesService.java

... and 574 more

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


An error occurred attempting to determine the process id of the DNX process hosting your application

I'm not sure where the best place is to post this, so if this isn't the correct place please point me in the right direction.
When attempting to run the Asp.net 5 starter projects from Visual Studio 2015 Community edition update 1, using Asp.Net 5 RC1 Update 1 via the F5 key, I receive the following error "An error occurred attempting to determine the process id of the DNX process hosting your application"
Turns out that the error can be triggered by a bug  which occurs if the installation is being used via a windows login name that has a space in it. e.g. "Tim Smith".
I was not able to find a patch but there is a work around. Just create another user account on the machine that doesn't have a space in it (e.g. "Tim") and use visual studio under that account. In my case this solved the issue. I can now create web projects in Visual Studio 2015 Community Edition Update 1 via the Asp.Net 5 templates and can run them via F5 and Ctl+F5 from within visual studio and I no longer receive the error.
However if I switch to the original user account on the machine that has a space in it, I still receive the error when trying to run a project generated from the same template. Same code ran from two different windows user accounts producing two very different results. Unfortunately it took me 7 days of work to find the source of this error.
Does anyone know if this issue will be fixed prior to Asp.Net 5 RC2?

I've found that my user account doesn't have a space in it, but it does have an underscore in it, which I'm maybe guessing is causing a similar issue.

I'm getting the same error on my new desktop PC with Windows 10 Pro and Visual Studio 2015 Community Edition. My user account ID has neither spaces nor underscores, but I'm getting the error repeatedly. I created four separate projects with the ASP.NET 5 Web Application template, and each of them fails with the same error. When I open any of the four projects in Visual Studio, I can see a dnx.exe process appear in the Task Manager. But for whatever reason, Visual Studio is not able to recognize the process.

This error has nothing to do with spaces in the user name. It's a generic error that happens when the Http Platform handler cannot launch the configured executable (in this case dnx). That can be for any number of reasons.

@davidfowl I didn't mean to imply that having a space in the profile name was the only thing that could trigger this error.  As you mentioned it can be caused by lots of things.  I was only pointing out that in RC1 Update 1 a space in the user name will cause this error.  This bug took me 7 days of effort to track down exactly because the error can be triggered by so many things.  For the case I'm pointing out, the underlying error caused by the space in the user name is a file not found error when launching the DNX.  But that error happens silently behind the scenes and what the developer sees is the "An error occurred attempting to determine the process id of the DNX process hosting your application" error.  As I mentioned on my machine I have my main user account which has a space in the name that I still can't make work, but I created a second user account that has no space and it works every time.  Same code, ran on the same dev box from two different accounts.

Hi, @davidfowl! I'm having the same problem and no idea on how to fix it.

Have you tried replacing the two files in C:\Program Files\Microsoft DNX\Dnvm with the latest versions from here: https://github.com/aspnet/dnvm/tree/dev/src
Also are you running with web or IISExpress when using VS 2015, should be using the former afaik

@chaddoncooper  is your comment directed to @kirby900 , @rjperes or myself?  @kirby900 and @rjperes are receiving the same error as the issue I was logging but the root cause of their issues are different than mine (as noted by their comments).

Make sure you have the right processPath in web.config. I changed mine to test something, several days later reopened this proj but forgot to change it back to processPath="%DNX_PATH%" and landed on this general exception when it couldn't find the runtime.

I have the same problem. It worked before, but since two days I have the issue. (I think I only updated VS 2015 to Update 1). Even when creating a new solution from the templates, I receive the same error. No idea where to look or what to do. Starting the app with the "web" configuration works. However trying to start with IIS Express gives this error.
Any suggestions?

when i get this problem, i just change the port on the applicationUrl inside launchSettings.json. not sure though why this happens.

Same problem. In launchSettings.json I have configured "applicationUrl": "http://localhost:26503/", but dnx listens on http://<IP_ADDRESS>:31750...

@krispaks Thanks, that did the trick for me as well.

I got this error because I misunderstood how launchSettings.json worked when enabling SSL.  I mistakenly believed the applicationUrl and the sslPort needed to match.  i.e. I set my applicationUrl to https://localhost:43000 and then my sslPort to 44300.
I understand now that applicationUrl is probably the hostname(?) and sslPort just uses that same hostname with the ssl port tagged on.

I ran across a solution on StackOverflow.
Steps
Close VS, Delete project.lock.json, I also did a dotnet restore, reopen VS and rerun.

@TJSoftware
Thanks, this solved this error for me.

I am using VS 2015 Update 2, Windows 10 and getting the same error. None of the above solutions helped to resolve it. I posted a question on StackOverFlow.
Is it a bug in VS 2015 Update 2 when running on Windows 10?

The original error I reported when I opened this issue has now been resolved by the ."NET Core SDK Installer (Preview 1)" found here: https://www.microsoft.com/net/download
I can now create and run MVC Core projects under windows login accounts that have a space in them like "Tim Smith"
A big THANK YOU for fixing the issue.
This issue can now be closed.


How do you set the min or max to be today

Is it possible to set the max="today()"?

only-valid="{before: 'today', inclusive: true}"


3lang: tfgrid: define actions and do the implementation

Issue: https://github.com/threefoldtech/web3_proxy/issues/94

@brandonpille i think we can do both, customizable and simple, and it should still be easy to use.
for the actions part, if this is the users gateway to deploying stuff, then the options are many. so i need to know which solutions would be exposed.
right now we have solutions for deploying:

machines
kubernetes clusters
gateways
discourse instance
funkwhale instance
peertube
presearch
zdbs
taiga
machine with a mounted qsfs

@robvanmieghem what do you mean by have main as default instead of tgrid.main?

@brandonpille Don't call the action tfgrid.main to indicate mainnet but a grid property for example that deploys to mainnet by default if not specified.

Ah yes makes sense! Or I would introduce an action load or keys where you put your mnemonic or seed and where you can add the grid property too.

i added some code, it should be a bit more clear now how parsing works.

i added some code, it should be a bit more clear now how parsing works.

Can you first define the actions in a markdown file? It's not a good idea to start the implementation if the input is not yet known. Let's define the input first, provide some examples. After that we can start the implementation.

i added some code, it should be a bit more clear now how parsing works.

Can you first define the actions in a markdown file? It's not a good idea to start the implementation if the input is not yet known. Let's define the input first, provide some examples. After that we can start the implementation.

@Omarabdul3ziz  and i added tfgrid actions' docs here

@Omarabdul3ziz and i were discussing some specifications about 3lang, and we had some questions:

Will the md files contain more than one action, or just one action per document?
If there are multiple actions per file, should the actions be processed in any particular order?

Currently for the tfgrid, only actions per project are processed in order, but there is no guarantee which project will be processed first

For the 3lang parser, how should the output be formatted?

If there is only one action per file, then we could just return the result of this action
If there are multiple actions, then a format should be documented since there could also be multiple modules performing these actions.

i resolved some comments, commented on some others.
i'll just work on the machines with gateways solution to allow updates, then add the appropriate documentation for it

i resolved some comments, commented on some others. i'll just work on the machines with gateways solution to allow updates, then add the appropriate documentation for it

Alright thanks! Could you also check why CI is red?

@Omarabdul3ziz and i did the required refactoring for the tfgrid actions. we had some remarks, which may or may not be important, but here they are anyway:

in the proposed version of the parser here , the actions are run in a specific order defined only by us. i don't know if this a desired behavior, but at least we need to know which actions to do before which. and as @brandonpille suggested, we believe it's best to execute them one by one, and let the user be responsible of the ordering of the actions.
in the proposed version of the parser here, arguments are not used, and instead parameters with 'yes' or 'no' values are used. we didn't really get why, but if arguments are well documented, shouldn't a flag, or an argument in our case, be enough?
in the proposed version of the parser here, a vm name is provided while creating a set of machines in the same network. if the desired behavior here is to deploy a number of vms with the same name, this would be prone to error as there is no guarantee that each vm will be deployed on a distinct node. if this is a problem, maybe we should randomly create vm names, or else restrict users to deploying single machines at a time and let them provide names as they want?

regardless of the remarks above, we did the refactoring as proposed for most of the basic tfgrid, but i think it would be useful to get some feedback on this before continuing

resolved @brandonpille latest comments in https://github.com/threefoldtech/web3_proxy/pull/132

solutions

They are no... Look under manual/src/threelang/tfgrid

Do want all solution structs and definitions like peertube and funkwhale to be directly in the threefoldtech.threebot.tfgrid module in the v client? Same for the serverside go package.

What do you mean? Do you want all structs and definitions under lib/tfgrid?

solutions

I think they are. Look under manual/src/threelang/tfgrid

actually these are the docs for the old examples, they're there but they still need to be updated. will update them.

Do want all solution structs and definitions like peertube and funkwhale to be directly in the threefoldtech.threebot.tfgrid module in the v client? Same for the serverside go package.

What do you mean? Do you want all structs and definitions under lib/tfgrid?

It was a question, I misformulated it ( should have been "Do we want"). I actually don't like it all flat in the root tfgrid module/package.
Off course, this does not mean that 3lang ( which is an easy to use intermediate language) should follow the same naming as the modules/packages do.

Do want all solution structs and definitions like peertube and funkwhale to be directly in the threefoldtech.threebot.tfgrid module in the v client? Same for the serverside go package.

What do you mean? Do you want all structs and definitions under lib/tfgrid?

It was a question, I misformulated it ( should have been "Do we want"). I actually don't like it all flat in the root tfgrid module/package. Off course, this does not mean that 3lang ( which is an easy to use intermediate language) should follow the same naming as the modules/packages do.

Where else woudl you put it?

Oh I see. @MarioBassem can you make sure that the documentation on the tfgrid examples is up to date?


For organization read it is not returning all site codes

When a read of an organization is done it does not return all site codes if there is more than one site code, we need to check that this is represented in the demonstrator back end and if so that it is represented in the returned resource.
It would be worth checking the organization search as well.

No longer an issue as site code has been removed from orgnization.


Turn off autoComplete in the AutoComplete component

Current behaviour
The Autocomplete allows the navigator's autocomplete which doesn't return the code and is confusing for the user.
Expected behaviour
The attribute 'autoComplete="off"' should be added by default to the composent so that the only clickable options come from the focus composent.
Versions
Focus-core: lastest
Focus-components: lastest

@bnau Small issue to begin with.
Basically, it is adding a props on the input, so web navigator do not output an autocomplete.
See https://developer.mozilla.org/fr/docs/Web/HTML/Element/Input#attr-autocomplete for detail.
Be wary of React "camelcasing" attribute on html element.

Pull request #1461

Delivered in 2.2.0


New sequence length error in LoRA script

Originally when I added the LIMA dataset, I had a max_len setting to set the maximum sequence length. We removed it in favor of the override_max_len argument in e.g., finetune/lora.py
Now, when preparing LIMA:
python scripts/prepare_lima.py --checkpoint_dir checkpoints/meta-llama/Llama-2-7b-hf/

and then finetuning with LoRA (after setting override_max_seq_length=2048) :
python finetune/lora.py --checkpoint_dir checkpoints/meta-llama/Llama-2-7b-hf/ --data_dir data/lima/

the following error occurs:
ValueError: Cannot forward sequence of length 2431, max seq length is only 2048.

I suspect this is maybe due to some recent changes. It's also probably not LIMA but affects all datasets.

Yes, this is due to my recent changes which touched how this is handled. I think it's best that we add back a configurable max_seq_length to the prepare_* scripts for the case where the model's max_sequence_length is limited at runtime.


feat: added support for browserified builds (closes #176)

I have added support for #176 in this pull request, and have also updated the README's browser usage section appropriately (specifically dropping wzrd.in in favor of unpkg.com).
Additional comments:

Note that I added a default "unpkg" version in package.json which gets consumed by unpkg.com in order to serve as the default version when loaded via <script src="https://unpkg.com/uuid"></script>
I added husky in order for commitlint to actually get used by contributors automatically
I used npm-run-all to keep the package.json tidier
I added eslint-plugin-compat and caniuse-lite with a custom .dist.eslintrc file that gets used to lint the dist folder for browser compatibility specified in .browserlistrc
I used babelify to ensure that the builds are built with babel, and @babel/preset-env which lets us target browsers automatically in .browserlistrc
I kept source maps turned off for generated browserify/babelify builds to reduce file size
I used tinyify to attempt to minify the dist outputs (see https://github.com/browserify/tinyify)

Screenshot proving it works in browser console:

Final note is that the purpose for me doing this in addition to #176 is to support development of Cabin and Lad among other projects.

I'm curious why you went with unpkg instead of jsdelivr? Especially after @broofa's comment.
Is unpkg suggesting production usage now?

I use unpkg for development, but I suppose it'd work for prod.  Never had any issues with it.  jsdelivr might be better approach though.

@styfle I have updated the PR to support jsdelivr

cc @broofa

When I run npm run prepare it automatically updates it.

My apologies for letting this gather dust, and just generally being unresponsive.  By way of excuse/rationalization, the reason for this has been that this PR speaks to an issue I feel increasingly strongly about - the extent to which modules should/shouldn't bend over to deal with packager-related issues.
As this PR demonstrates, supporting a particular form of packaging adds substantial dependencies, and introduces significant complexity.
At the end of the day, dealing with such issues is more trouble than it's worth.  I feel bad about that - I know it makes some users lives more difficult - but having to support the never-ending parade of packaging tools... AMD, UMD, CommonJS, ES6, Webpack, Browserify, Bower, Angular, Typescript, Rollup, and so on... it's just painful.
I'm done with it.  Sorry, I know that's not exactly "friendly", but I don't have the time/attention needed to deal with this class of problems.
This is a Node/CommonJS-style module.  Packagers either support that or they don't.  If they don't, then that's an issue for the packager project.


PeerNotFound error that comes from Crust - Jim's results

Log output that we got:
thread 'main' panicked at '
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!   unwrap! called on Result::Err                                              !
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
examples/client.rs:1263,21 in client
Err(Crust(PeerNotFound))
', /home/nikita/.cargo/registry/src/github.com-1ecc6299db9ec823/unwrap-1.2.1/src/lib.rs:67:25

No details of what was done, if anything, to resolve this - @nbaksalyar could you update with what was done here?

could you update with what was done here?
Sure @S-Coyle. We just don't panic in this case: it's a rare occasion when Crust reports that a direct connection was established but while we're trying to get a peer's IP address it's already disconnected at that point in time (hence PeerNotFound). Now we just ignore the result and don't report it as a successful connection.

Thanks Nikita, closing issue.


slack: Map attachments.

Currently, the uploaded files are being added in the dir: uploads/<realm_id>/<attachment_id>/<file_name>.
In zulip export data those are added in : uploads/realm_id/<number>/<hash>/<file_name>

I'll make the changes after #8 has been merged

@rheaparekh rebase reminder from rht/python-zulip-api slack. I just rebased from zulip/python-zulip-api master

Merged!


⛳add help for xcode 10 error

I see that's listed here - https://github.com/tensorflow/swift/blob/master/Usage.md
Would be best to collapse these instructions.


Problem with the module

I have installed the release 2.0.1 to prestashop <IP_ADDRESS> and when a customer make an order to my website write his name in Greek. The module complete the customer informatio automated and this has the result to appear the name as html code. This is a problem .
In addition how I can change the translation text in the frontpage of the module ?
Thanks a lot for your reply

Hey @Yanmar2200 we are currently working on fixing this and will be releasing it with one of the next versions of Prestashop.
If the release does not fix your issue, could you please raise this through the bank's support team - we'll be able to track your request better. We are going to be closing the issues tab here as we can't track these questions & queries as well as through the regular support route.


config.reload configuration for logstash

Problem description
Cant make my logstash to auto reload changed configuration.
Extra information
I have tried to add reload command to my compose file but failed to do so.
Compose setup
So this is my Logstash section:
logstash:
    image: docker.elastic.co/logstash/logstash:6.5.4
    command: bin/logstash –f apache.config --config.reload.automatic --config.reload.interval 10
    ports:
      - "5000:5000"
      - "5044:5044"
      - "9600:9600"
.
.
.

Docker logs
Logstash service logs:
elk_logstash.1.ek1c7vid194o@ELK2-DEV    | ERROR: Unknown command '–f'
elk_logstash.1.ek1c7vid194o@ELK2-DEV    |
elk_logstash.1.ek1c7vid194o@ELK2-DEV    | See: 'bin/logstash --help'
elk_logstash.1.ek1c7vid194o@ELK2-DEV    | [ERROR] 2019-01-30 12:11:40.150 [main] Logstash - java.lang.IllegalStateException: Logstash stopped processing because of an error: (SystemExit) exit

Any idea why? how can i make my logstash reconfigurable?
Thanks alot!

This error is due to a misconfiguration of Metricbeat, as pointed out in #352


Subscription match: Single level wildcard token should not matching parent

According to latest 3.1.1 specification, section <IP_ADDRESS> Single level wildcard:
The plus sign (‘+’ U+002B) is a wildcard character that matches only one topic level.
For example,  “sport/+” does not match “sport”  but it does match "sport/".

However, Current subscription matching code:
org.eclipse.moquette.spi.impl.subscriptions.TreeNode.matches() will matching parent even it's a Single level wildcard token:
void matches(Queue<Token> tokens, List<Subscription> matchingSubs) {
        Token t = tokens.poll();

        //check if t is null <=> tokens finished
        if (t == null) {
            matchingSubs.addAll(m_subscriptions);
            //check if it has got a MULTI child and add its subscriptions
            for (TreeNode n : m_children) {
                if (n.getToken() == Token.MULTI || n.getToken() == Token.SINGLE) {
                    matchingSubs.addAll(n.subscriptions());
                }
            }

            return;
        }
    // others....
}

it probably should change to:
    //check if it has got a MULTI child and add its subscriptions
    for (TreeNode n : m_children) {
          if (n.getToken() == Token.MULTI) {
              matchingSubs.addAll(n.subscriptions());
          }
    }

BTW: what's purpose of The field m_inflightStore in MemoryStorageService/MapDBPersistentStore?
it's a unused field and can be safely removed.

And, what's purpose of adding an empty subscription to client during client connect?
org.eclipse.moquette.spi.impl.ProtocolProcessor.processConnect, Line 253:
 LOG.info("Create persistent session for clientID <{}>", msg.getClientID());
 m_sessionsStore.addNewSubscription(Subscription.createEmptySubscription(msg.getClientID(), true)); //null means EmptySubscription

For me, this empty subscription seems meaningless.

For the first question I've added a unit test ti validate it, and it work as it's with out any change (look at this https://github.com/andsel/moquette/blob/master/broker/src/test/java/org/eclipse/moquette/spi/impl/subscriptions/SubscriptionsStoreTest.java#L316)
For the second, it's related to issue #86 and essentially the "in flight" at the moment is poorly implemented.


New token: WETH on Metis

New things

Contract address for WETH token on Metis
nETH LP token (in src/swappools.ts) for Metis
Test cases

Coverage increased (+0.0005%) to 99.483% when pulling 36f0b957db518b7d79c6a8ca68da731d37a335ad on feature-token/metis-eth into 822316e938d0fc1782bcc09e3535c075a6e812e5 on dev.


Feature/plugins

API surface exposed is equivalent to the Rust API, including plugin callbacks (exposed as pyrage.plugin.Callbacks, which must be subclassed for anything interesting to happen).
Non-Cloneability of plugin instances is resolved by putting them behind an Arc. Perhaps a more elegant solution could be produced later, by someone with more knowledge of PyO3 internals.
This was tested with rage's example age-unencrypted-plugin which happens to exercise the display_message callback, and additionally with age-plugin-tpm to exercise actual encryption functionality.
Typing stubs are amended to cover the new API surface.
Resolves #55.
EDIT 2024-03-29: now based on #63, which should be merged first.

Nice work! I'll do another round of review tonight.

This is looking pretty good, thank you @vikanezrimaya! If you could give age_plugin a try that would be fantastic (and if it doesn't work we can go ahead with the current approach).
Finally, it'd be good to have 1-2 tests in this PR before merge. Nothing huge; just something small like the sample you posted above.

Sorry for the hassle here -- I think you'll need to deconflict now that pyo3 has updated to use the Bound<...> APIs.

P.S. let me know if you'd like a release with these new APIs. Otherwise I'll likely do it sometime later this month.


Idea: Global fragment map

In our app, we have a fair number of commonly used fragments (especially for queries triggered via non-UI code).  It'd be pretty handy to register fragments globally with an apollo-client instance to cut down on verbosity when querying
Maybe as a fragments argument when constructing the client?

We thought about this when designing the createFragment concept. At the time, we came to the conclusion that handling global state like this might actually be somewhat complicated to reason about and we went with a more local approach.
However, from the discussion in #480, it is evident that we will end up implementing this in some capacity since we want to be able to reference fragments by names only rather than by the references returned by createFragment.

I'll go ahead and close this since this suggestion is pretty much a part of #480. We can re-open if this is not the case.


updated 'getting started' link

The last link in the concourse guide was broken. Updated it to http://concourse.ci/using-concourse.html

Thank you!


Integrations: Replace account IDs with display names in Jira comments.

This PR enhances the Jira integration by fetching account display names for mentions in comments using
the Jira API. Account IDs are replaced with their corresponding display names to make Zulip notifications
easier to read.
Changes:

Update view.py to fetch account display names via the Jira API and replace [~accountid: id] with
the respective display names.
Document the process for generating JIRA_API_TOKEN in /integrations/doc/jira.
Add a new test case in tests.py to validate this functionality.
Add a new fixture comment_created_with_account_mention.json for testing account mentions.

Fixes: #32646
Screenshots and screen captures:

Jira

Zulip

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Hi @PieterCK, this PR is up for review. Please take a look when you have a moment. Thanks!

Oh and the commit summary is a bit too long, I'd rephrase it to:
integrations: Improve Jira's comment-event notification.

Hi @PieterCK,
Just a gentle reminder to review this PR when you have a chance. Your feedback would be greatly appreciated!
Thank you!

@PieterCK , I made the changes and also rephrased doc a bit for config_options, please take a look.
The reason for double URL encoding was because URLSearchParams in integration_url_modal.ts already encodes URL one time   then JIRA API call also encodes it one more time. so @ becomes %40 in integration URL then becomes %2540 during request.
In this case I think better would be that we'll make integration_url's query params (config options) without any URL encoding e.g. @ input as config option will be same as @ in URL to be copied. I can put up separate PR for that, as it's not related to Jira integration. please let me know your thoughts, thanks!


How to do robust ??

I am not getting good results!!! with input IMG and generated video is there some other settings which i am missing?, for robust does you have to just write nothing in prompt section right??? Please correct me

https://github.com/dreamoving/dreamoving-project/assets/155391363/1874643d-9ea8-4fc4-8a5d-45ea4472c74a

https://github.com/dreamoving/dreamoving-project/assets/155391363/f022829b-bf37-4da5-b5af-84c06bb8926a

I think it's work great only for input face and prompt clothes & background ,but also it's cool tool

@Anandyyyt the videos shown are always cherry-picked, you have to carefully choose a good input picture, then hope to have good results after a few hours of tweaking, true story. I spent hours to install dreamtalk (https://github.com/ali-vilab/dreamtalk) to finally get results worse than FOMM (https://github.com/AliaksandrSiarohin/first-order-model)  released 4 years ago !


\setmathfontface does not respects Style key in lulatex

Description
\setmathfontface does not recognize Style= key in font options if luatex engine is used
Add info or delete as appropriate:

Relevant for LuaTeX
Issue tracker has been searched for similar issues

Minimal example demonstrating the issue
\documentclass{article}
\usepackage{unicode-math}
\setmathfontface\symbsw{Arsenal-Regular.otf}[Style=Swash]
\begin{document}
$\symbsw{ABCD}$
\end{document}

Further details
This example defines a swash alphabet in XeLaTeX, but LuaLaTeX drops the swashes.  Rawfeatures=swsh has the same effect

\setmathfontface does not seem to set mode=base which is required in LuaTeX for fonts used in Math mode.

The corresponding fontspec/unicode-math font option would be Renderer=Basic.


feat(extra7195): Added check for dependency confusion in codeartifact

Context
A recent feature has been released to mitigate against dependency confusion in AWS CodeArtifact. More can be read here:

https://aws.amazon.com/blogs/devops/tighten-your-package-security-with-codeartifact-package-origin-control-toolkit/
https://zego.engineering/dependency-confusion-in-aws-codeartifact-86b9ff68963d

Description
I've included this check in extras as I don't expect most prowler users to want to run it only people specifically interested in checking for this issue.
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Please let me know if any changes are required to improve this check. I'm not a very experienced prowler user so apologies in advanced if I'm not following any best practices

Hi @congon4tor, thank you for your contribution! The check looks great, we will review it shortly and let you know if some changes are necessary.

Hi @congon4tor, I added some changes to your check, thank you for your contribution. However, there are some packages that returns ORIGIN_TYPE=None. Please, contemplate this case, if you need help let us know.

Hi @sergargar thanks for your changes. What kind of packages are you experiencing ORIGIN_TYPE=None? From what I see in AWS docs it should always be INTERNAL or EXTERNAL.

originType -> (string)
Describes how the package version was originally added to the domain. An INTERNAL origin type means the package version was published directly to a repository in the domain. An EXTERNAL origin type means the package version was ingested from an external connection.

https://docs.aws.amazon.com/cli/latest/reference/codeartifact/describe-package-version.html
It could be a mess up on my code. If you can provide some more details I'll try to fix it

Hi @congon4tor, I created a package in CodeArtifact to test this check and in some cases the Origin information is not returned using the CLI. This is what I'm getting for a package I created:

As you can see, there is no origin info, that is what I was referring to.

I see, how did you create that package I'll try to replicate it. I've tested this in an org with over 50k package versions (pip, mvn and npm) and I've never seen this happen before

I created a pypi package and uploaded it using twine. In this documentation you can see in steps 5 and 6 how I did it.
It's strange that you did not face this issue with all of that packages though.

I found a package that also has no origin info this is how it looks like in the AWS console.

It is slightly different from your example

I think the only option is either to default to considering unknown origins as internal or external. Currently they are getting treated as external which could miss vulnerable packages. Defaulting to internal might raise false positives

My python test packages were pushed to codeartifact in the same way you did it. I might be able to get in touch with someone from AWS's code artifact development team to see if they can help figure this out

I just heard back from AWS:
There are two things here. The fact that originType is UNKNOWN is expected for packages that were added to CodeArtifact before we released the feature in July. This is because the service was not tracking the origin of packages prior to this.
On the 2nd point where the output of describe-package-version is not returning any origin information is weird and looks like a bug. I will create a ticket for my team and ask the relevant engineers about it.
In the unkown case should we raise it as a vulnerable package? Or is there a textwarning we can use for that kind of situation?
For the no origin info I think we should give AWS some time to look into it

Thank you for this information @congon4tor, it is really helpful!!
I would treat unknown package as internal, but let's see what AWS says about this bug because in my package there is no PUBLISH or UPSTREAM information.

AWS believe this is probably due to you AWS CLI version you will need to be using at least v2.7.16. Let me know if that fixes it for you and I'll add a warning in the code

They are right, I tried it again with the latest version and it works! Thank you.
However, there is no need to verify the AWS CLI version since in Prowler's README we indicate that the latest version of the CLI is required.


Node Classification

Hi, thanks for organizing the code so neatly, I am wondering if the code applicable for node classification task. If yes, can you please point out which part should be changed?

Hi! In our paper and implementation, we do not evaluate our method on node-level tasks, but it should be easy to transfer GSAT to node-level tasks. A typical way to apply perturbation-based methods on node-level tasks is to utilize local computation graphs. Take a look at Algorithm 1 from Appendix A of PGExplainer, where a detailed description is provided.
Thank you for your interest in our work. Let me know if you have any other questions.


TGI change inference result

Hello everyone,
I implement a TGI server with Rope scaling and my inference result are completely different with TGI and without TGI. do you know if it's a common problem and how can I solve it ?

Without further information it's impossible to help you.
What are you doing, what's your reference, which is better ?
Also just fill in the classic template it really helps us understand your environment your are running on.
I will go ahead and close this, but please re-open an issue with the more complete information and we can look at what's going on.


Unable to install scenario_simulator_v2 because rosdep fails to resolve dependencies

Describe the bug
I've tried to install the simulator as described here: https://autowarefoundation.github.io/autoware-documentation/main/tutorials/scenario-simulation/planning-simulation/installation/)
To Reproduce
Steps to reproduce the behaviour:

Go to autoware workspace
cd autoware
Import Simulator dependencies:
vcs import src < simulator.repos
Install dependent ROS packages:

source /opt/ros/galactic/setup.bash
rosdep install -y --from-paths src --ignore-src --rosdistro $ROS_DISTRO

Note: in the documentaion, they are using ros humble instead of galactic
Expected behavior
rosdep is supposed to install the needed dependencies, but rosdep keys could not be resolved:
ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
tracetools_launch: Cannot locate rosdep definition for [ament_mypy]
behavior_tree_plugin: Cannot locate rosdep definition for [quaternion_operation]
autoware_auto_geometry: Cannot locate rosdep definition for [osrf_testing_tools_cpp]
concealer: Cannot locate rosdep definition for [ament_cmake_clang_format]
random_test_runner: Cannot locate rosdep definition for [ament_cmake_clang_format]
tracetools_test: Cannot locate rosdep definition for [ament_cmake_mypy]
tracetools_trace: Cannot locate rosdep definition for [ament_mypy]
traffic_simulator_msgs: Cannot locate rosdep definition for [ament_cmake_clang_format]
simulation_interface: Cannot locate rosdep definition for [zmqpp_vendor]
ros2trace: Cannot locate rosdep definition for [ament_mypy]
tum_trajectory_planner_emulator: Cannot locate rosdep definition for [matplotlib_cpp]
traffic_simulator: Cannot locate rosdep definition for [quaternion_operation]
geometry: Cannot locate rosdep definition for [quaternion_operation]
user_defined_value_condition_example: Cannot locate rosdep definition for [ament_cmake_clang_format]
simple_junit: Cannot locate rosdep definition for [ament_cmake_clang_format]
openscenario_preprocessor: Cannot locate rosdep definition for [ament_cmake_clang_format]
kashiwanoha_map: Cannot locate rosdep definition for [ament_cmake_clang_format]
openscenario_interpreter: Cannot locate rosdep definition for [ament_cmake_clang_format]
openscenario_visualization: Cannot locate rosdep definition for [color_names]
radar_threshold_filter: Cannot locate rosdep definition for [ament_clang_format]
tracetools_read: Cannot locate rosdep definition for [ament_mypy]
openscenario_interpreter_example: Cannot locate rosdep definition for [ament_cmake_clang_format]
tum_control_visualizer: Cannot locate rosdep definition for [matplotlib_cpp]
scenario_simulator_exception: Cannot locate rosdep definition for [ament_cmake_clang_format]
cpp_mock_scenarios: Cannot locate rosdep definition for [ament_cmake_clang_format]
simple_sensor_simulator: Cannot locate rosdep definition for [quaternion_operation]

Because of the issues with rosdep, I'm not able to build and use the scenario_test_runner
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu 20.04
ROS2 galactic
autoware.universe, scenario_simulator_v2

@AndriiChumak145
Thank you for reporting!
The error seems to occur due to the EOL of Galactic.
Would you mind trying to execute below commands instead of step 3?
source /opt/ros/galactic/setup.bash
rosdep update --include-eol-distros
rosdep install -y --from-paths src --ignore-src --rosdistro $ROS_DISTRO

Thanks, your suggestion helped to resolve the rosdep issue. After that, I ran colcon build --symlink-install --cmake-args -DCMAKE_BUILD_TYPE=Release --packages-up-to scenario_test_runner  to build the needed packages and got the following error:
--- stderr: geometry
make[2]: Warning: File 'gtest/CMakeFiles/gtest_main.dir/depend.make' has modification time 1233 s in the future
make[2]: warning:  Clock skew detected.  Your build may be incomplete.
make[2]: Warning: File 'gtest/CMakeFiles/gtest_main.dir/depend.make' has modification time 1233 s in the future
make[2]: warning:  Clock skew detected.  Your build may be incomplete.
make[2]: Warning: File 'CMakeFiles/geometry.dir/src/intersection/intersection.cpp.o' has modification time 1235 s in the future
/home/andrii/autoware/src/simulator/scenario_simulator/common/math/geometry/src/polygon/polygon.cpp:15:10: fatal error: quaternion_operation/quaternion_operation.h: No such file or directory
   15 | #include <quaternion_operation/quaternion_operation.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

I tried to solve the problem and installed quaternion_operation package myself. This solved the problem with geometry but after that I got:
--- stderr: simulation_interface
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
Protobuf_LIBRARY
    linked by target "simulation_interface" in directory /home/andrii/autoware/src/simulator/scenario_simulator/simulation/simulation_interface

---

As I understood, I need the protobuf library. I tried to install it from https://www.datascienceblog.net/post/programming/essential-protobuf-guide-python/  but the installation wasn't successful:
protoc
protoc: error while loading shared libraries: libprotoc.so.23: cannot open shared object file: No such file or directory

I also found a line in CMake include(FindProtobuf REQUIRED). Does the project need only FindProtobuf file?

FindProtobuf should be provided by CMake as standard.
ref: https://github.com/Kitware/CMake/blob/master/Modules/FindProtobuf.cmake
Can you check if the packages below are installed?
They should be resolved and installed by rosdep, but they seem to be missing.

libprotobuf-dev
protobuf-compiler
libprotoc-dev
libprotobuf17

I've checked the packages and only the last one was installed, so I've also installed the three above. I've also run rosdep install explicitly on src/simulator and it installed all the required rosdeps.
Now, the build fails with a different error:
--- stderr: simulation_interface
CMake Error at /usr/share/cmake-3.16/Modules/FindProtobuf.cmake:469 (file):
  file STRINGS file "/usr/local/include/google/protobuf/stubs/common.h"
  cannot be read.
Call Stack (most recent call first):
  CMakeLists.txt:22 (find_package)

CMake Error at /usr/share/cmake-3.16/Modules/FindProtobuf.cmake:475 (math):
  math cannot parse the expression: " / 1000000": syntax error, unexpected
  exp_DIVIDE (2).
Call Stack (most recent call first):
  CMakeLists.txt:22 (find_package)

CMake Error at /usr/share/cmake-3.16/Modules/FindProtobuf.cmake:476 (math):
  math cannot parse the expression: " / 1000 % 1000": syntax error,
  unexpected exp_DIVIDE (2).
Call Stack (most recent call first):
  CMakeLists.txt:22 (find_package)

CMake Error at /usr/share/cmake-3.16/Modules/FindProtobuf.cmake:477 (math):
  math cannot parse the expression: " % 1000": syntax error, unexpected
  exp_MOD (2).
Call Stack (most recent call first):
  CMakeLists.txt:22 (find_package)

This is the first error I have seen.
Can you give me a version of protobuf?
Also, I would appreciate it if you could try a clean build.

My protobuf version is 3.6.1
I've cleaned the build and install space, reinstalled dependencies, and tried to rebuild everything. Now, it fails on following:
--- stderr: lanelet2_matching
CMake Error at /opt/ros/galactic/share/mrt_cmake_modules/cmake/Modules/GatherDeps.cmake:24 (message):
  Gather depenencies failed: Traceback (most recent call last):

    File "/opt/ros/galactic/share/mrt_cmake_modules/cmake/../scripts/generate_cmake_dependency_file.py", line 366, in <module>
      main(sys.argv[1], sys.argv[2], sys.argv[3])
    File "/opt/ros/galactic/share/mrt_cmake_modules/cmake/../scripts/generate_cmake_dependency_file.py", line 293, in main
      catkin_packages = getCatkinPackages(workspaceRoot)
    File "/opt/ros/galactic/share/mrt_cmake_modules/cmake/../scripts/generate_cmake_dependency_file.py", line 268, in getCatkinPackages
      getPackagesInPath(packages, path)
    File "/opt/ros/galactic/share/mrt_cmake_modules/cmake/../scripts/generate_cmake_dependency_file.py", line 234, in getPackagesInPath
      package = ET.parse(os.path.join(root, manifest)).getroot()
    File "/usr/lib/python3.8/xml/etree/ElementTree.py", line 1202, in parse
      tree.parse(source, parser)
    File "/usr/lib/python3.8/xml/etree/ElementTree.py", line 584, in parse
      source = open(source, "rb")

  FileNotFoundError: [Errno 2] No such file or directory:
  '/home/andrii/autoware/install/simple_planning_simulator/share/simple_planning_simulator/package.xml'

Call Stack (most recent call first):
  CMakeLists.txt:11 (include)

line in the CMakeLists.txt: include(GatherDeps)

@AndriiChumak145 What rosdistro and Linux distribution are you using?

@hakuturu583
To his credit, he already wrote it.
Please see his first post.

@HansRobo Can you check protoc version?

My version of protobuf is also 3.12.4 because I'm using Ubuntu 22.04.
And 3.6.1 is fomal version for Ubuntu 20.04.

Humm..
It seems a problem with protoc version.
But I think this tool is still work with galactic.
I will check the protoc version in github actions environment.

Has there been any updates on this front? I too am trying to compile this package on ROS2 galactic with Autoware.Universe. I've resolved the quaternion_operation dependency by compiling it from source and adding it to the package path. Now zeromq is said to be missing while compiling, specifically from the simulation_interface package. But when running rosdep for simulation_interface I get that all dependencies are installed. The manifest lists "zmqpp_vendor" as the dependency and rosdep reports that it is installed but build returns a message saying "zmqpp/zmqpp.hpp" is missing.
Can I get some insight regarding this?

@stevenbrills Galactic support is already dropped and zmqpp_vendor package is released on humble.

I think this issue is no longer active, if necessary, please re-open issue.


Debugtron appears to hold open .asar file

I reported signalapp/Signal-Desktop#5426, but the issue turned out to be with Debugtron.
Basically, it looks like Debugtron is opening the Signal app.asar even though I've never used Debugtron with Signal. This interferes with Signal's ability to update. I understand that Debugtron wants to open this file in order to populate the installed apps list, but it should close the file once it's been read. The current behavior is quite counterintuitive, since I have no reason to think that simply having Debugtron open (without ever having it interact with Signal) would affect Signal's behavior.

@cg505 Is this Windows-OS related?

This is happening on Windows 10. I'm not sure if that answers your question.


Text not updating

When I use state to change the content of <Text>, it doesn't update. As you can see in the screenshot, the first number is using React Native's Text and the second using RNSvg's, both with the same state value, but the latter is not updating.

I'm having this issue too :(

I have another app using 4.0.0-rc of this, and the text updates based on state, I think the combination of react-native 1.5.2 and react-native-svg 4.1.1 is problematic...or something.

For me it's not working on android only, can someone check that? @magicismight

Similar issue but it's not updating with G element's props in Android.
the RN vision is 0.38
the RNSVG vision is 4.40
when I change the props to rotated G element,it doesn't work in Android
<G
    rotate={`${this.props.value}`}
    width={SIZE}
    height={SIZE}
    origin={`${HALFSIZE},${HALFSIZE}`}
    id="hand">
     <!--other elements-->
</G>


uriLinkStream does not bring any event

Describe the bug
I am using app_links: ^3.4.3 version.
Trying to listen to all new events via appLinks.uriLinkStream, but receive nothing.
At the same time the stream allUriLinkStream brings me all the clicked events.
Does it related to

[ ] App Links (Android)
[ ] Deep Links (Android)
[x] Universal Links (iOS)
[ ] or Custom URL schemes? (iOS)

Does the example project work?

[ ] Yes
[ ] No
[x] Irrelevant here   - I dint get how I can test it since example.com is not your domain and the browser is opened

Did you fully read the instructions for the targeted platform before submitting this issue?
Uploaded your files to webserver, HTTPS, direct connection, scheme pattern setup, ...

[x] Yes
[ ] No
[ ] Irrelevant here

I am experiencing something similar, but only when users are using Chrome browser on iOS. If they are using Safari then everything works good and the app receives the events.

I am using the simulator for tests and Safari is used there.

allUriLinkStream wraps uriLinkStream with getInitialAppLink at preamble. This the same stream.
You can't consume twice an event.

Sorry I didn't read carefully. You're issue is not on iOS but on web.

I am sorry, but I know which platform I am using and where I have a problem.
Can you please reopen the issue and investigate it?
If you need some more details from me - I can provide them.

I am using the simulator for tests and Safari is used there.

? This is a bit misleading

By that in let you know which device I was using and which defaul browser is there.
But the application is flutter iOS one.

Ok, so as I said before allUriLinkStream wraps uriLinkStream with getInitialAppLink at preamble. Finally, this the same stream and more globally, there's only one single stream despites the four getters available.
https://github.com/llfbandit/app_links/blob/53798e5d67ebb0508e9681c8ecc3af4165a26063/lib/src/app_links_method_channel.dart#L63
But I don't understand for which use case separate streams would be useful?


Fixed compilation of OpenCL code

Fixed compilation of OpenCL code. I was surprised, but it still works.

Can one of the admins verify this patch?


Fixed a pyTorch warning

Fixed Pytorch warning "Argument interpolation should be of type InterpolationMode instead of int."

Sorry，this change will be error in lower pytorch version, so just keep it.


Hooks fired with no arguments

e.g. my PRIVMSG hook still gets fired on a CTCP; however with no arguments because the 'handler' returned nothing.
I think the fix is to not call callbacks/hooks if the handler returns nothing?

Ah yeah, sounds right, thanks.

@mirrexagon thanks for quick merge and release!

@daurnimator No problem!


arm/qemu: Add mounting of tmpfs

Summary
No
Impact
No
Testing
No

[Experimental Bot, please feedback here]
The provided PR information does not meet the NuttX requirements.
Here's why:

Missing Essential Information:  The "Summary" section lacks details about the actual changes being made. It only states "No" without any context.
Incomplete Impact Assessment:  Simply stating "No" for each impact category is insufficient.  The PR needs to explicitly address how the change might affect users, builds, hardware, documentation, security, compatibility, etc.  If there's truly no impact in a particular area, a brief explanation is still helpful (e.g., "Impact on build: NO - This change only affects application code and does not modify the build system").
Absent Testing Data: The "Testing" section needs to provide concrete evidence that the changes work as expected.  This includes:

Specifying the build host and target environments used for testing.
Including relevant testing logs from before and after the changes.

In short, a PR needs to clearly articulate what's being changed, why it's necessary, how it was tested, and what potential impact it might have on the NuttX ecosystem.


Add demo documentation site and unpin book theme

This is a small update to the documentation after @rabernat nerd-sniped me by showing an image on twitter of an outdated version of the book theme 😅
It does a couple things:

Fixes the CSS import of the book theme CSS (you needed to include the styles/ prefix, closes #7)
Adds a default logo (closes #2)
This fixes the weird duplication bug (the issue was the non-imported CSS above. closes #8)
Adds a little demo documentation site to be able to quickly preview things
Adds a noxfile.py and some mini docs in the README just to help people quickly build/preview the docs

Amazing Chris! Thank you so much! I will test this with the Pangeo Forge docs and get back to you asap.

I finally got around to trying this with pangeo forge recipes, and it worked great (once I remembered to explicitly upgrade sphinx-book-theme--for some reason I was on 0.1.7 even after pip installing your branch.)


Add documentation for doxygen

Fixed in https://github.com/endail/pico-scale/commit/17a67947537c4128a6f1a36c66aa4df7dcc98ff8


Question: Converting circular structure to JSON

Hi there,
I am very new to NestJS and something I did wrong. If you can help me I will be very thankful :)
Module:
import { Global, Module } from '@nestjs/common';
import KeycloakService from './keycloak.service';
import { HttpModule } from 'nestjs-http-promise';

@Global()
@Module({
  imports: [
    HttpModule.register({
      timeout: 10000,
      retries: 3,
      retryDelay: () => {
        return 500;
      },
      baseURL: 'https://my-local-domain.local',
      headers: {
        Accept: 'application/json',
        'Content-type': 'application/json',
      },
    }),
  ],
  providers: [LocalService],
  exports: [LocalService],
})
class LocalModule {}
export default LocalModule;

Service:
import { Injectable } from '@nestjs/common';
import { HttpService } from 'nestjs-http-promise';

@Injectable()
class LocalService {
  constructor(private httpService: HttpService) {}

  public async callSomeServer(): Promise<any> {
    return await this.httpService.get('/users');
  }
}

export default LocalService;

Controller:
import { Controller, Get } from '@nestjs/common';
import LocalService from './local.service';

@Controller()
export class LocalController {
  constructor(private readonly localService: LocalService) {}

  @Get('/user')
  public async userCall(): Promise<any> {
    const ret = await this.localService.callSomeServer();

    console.log(ret);

    return ret;
  }
}

In the console, I get a response from the console.log(ret) but after that I have an error:
ERROR [ExceptionsHandler] Converting circular structure to JSON
    --> starting at object with constructor 'ClientRequest'
    |     property 'socket' -> object with constructor 'TLSSocket'
    --- property '_httpMessage' closes the circle

I have found a solution (It was fast :) )
In controller return have to be:
return ret.data;

Hi, the response of axios request contains complex data structure with circular dependency in it which cause the problem.
Instead of logging the whole response I think you may want to log only the response: console.log(ret.data).
Also you can check this library which can flat circular json response.


acs conversion

Codecov Report

Merging #275 (cd0d111) into master (85ed052) will decrease coverage by 0.27%.
The diff coverage is 61.15%.

@@            Coverage Diff             @@
##           master     #275      +/-   ##
==========================================
- Coverage   68.22%   67.94%   -0.28%
==========================================
  Files          19       20       +1
  Lines        6564     6792     +228
==========================================
+ Hits         4478     4615     +137
- Misses       2086     2177      +91

Impacted Files
Coverage Δ

stix2elevator/__init__.py
84.09% <ø> (ø)

stix2elevator/cli.py
0.00% <0.00%> (ø)

stix2elevator/convert_pattern.py
74.15% <50.00%> (-0.42%)
:arrow_down:

stix2elevator/convert_to_acs.py
52.14% <52.14%> (ø)

stix2elevator/options.py
77.22% <60.00%> (-0.50%)
:arrow_down:

stix2elevator/utils.py
89.90% <83.33%> (-0.25%)
:arrow_down:

stix2elevator/convert_stix.py
81.91% <89.58%> (+0.43%)
:arrow_up:

stix2elevator/missing_policy.py
25.09% <100.00%> (ø)

stix2elevator/test/test_idioms.py
74.60% <100.00%> (+2.02%)
:arrow_up:

stix2elevator/test/test_main.py
100.00% <100.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


SQL logic error: view logbook_view already exists (1)

Hi!
First of all, thanks for your amazing work! :)
I've updated to v2.44.0 and after running docker run --rm -it -p 4000:4000 -v /root/data:/data vsimakhin/web-logbook:v2.44.0 I get this error:
SQL logic error: no such table: metadata (1)
ERROR	2024/12/13 17:47:27 main.go:162: SQL logic error: view logbook_view already exists (1)

Hey!
Wow, that looks like a nice glitch because in theory there cannot (or shouldn't) be these two errors in a row :)
Does the error still appear if you restart the app? And what version did you use before?

It's a clean install on Debian 12 using Docker. I actually wanted to start with v2.44.0, and was getting another error Error: Unable to open database file: out of memory (14). Then I tried v2.43.3 with no errors and after that I tried running v2.44.0 again and got ERROR	2024/12/13 17:47:27 main.go:162: SQL logic error: view logbook_view already exists (1)

Ok, I tried to reproduce it...
So the error Error: Unable to open database file: out of memory (14) usually occurs when you set the data path to the directory where you have no write rights, for example in your case /root/data
$ mkdir -p /tmp/logbook/data
$ cdmod 0400 /tmp/logbook/data
$ docker run --rm -it -p 4000:4000 -v /tmp/logbook/data:/data vsimakhin/web-logbook:v2.44.0
ERROR	2024/12/14 20:19:38 main.go:162: unable to open database file: out of memory (14)

the /tmp/logbook/data directory was empty.
Then I tried to run 2.43.3 version
$ docker run --rm -it -p 4000:4000 -v /tmp/logbook/data:/data vsimakhin/web-logbook:v2.43.3
INFO	2024/12/14 20:21:26 Web Logbook v2.43.3 is ready on http://localhost:4000

after that run the /tmp/logbook/data contains the web-logbook.sql, but without proper permissions. It happens because previous versions of the docker image didn't change the user and everything was running under root. I updated the Dockerfile recently to use a new approach (more "cloud native" style) and also to build for multiarch - https://github.com/vsimakhin/web-logbook/blob/main/docker/build.Dockerfile#L23
So if I'm right the owner of your /root/data/web-logbook.sql is root now. And to fix it you just need to run sudo chown $(whoami):$(whoami) /root/data/web-logbook.sql  (basically change the owner to current user)

I've updated readme and changelog with BREAKING statement based on your findings. Thanks!

Thanks for the reply! I only have one user on my cloud VM which is root.
-rw-r--r-- 1 root root 1650688 Dec 14 22:33 web-logbook.sql

Weird that Docker doesn't have write access to the file. What should I change in my case then?

Hm, that's weird. Can you try to run chmod 0666 ./web-logbook.sql. It will add read/write access for all users and groups

I tried setting chmod 0666 data/web-logbook.sql and the error remains:
web-logbook-1  | SQL logic error: no such table: metadata (1)
web-logbook-1  | ERROR	2024/12/15 20:10:12 main.go:162: SQL logic error: view logbook_view already exists (1)
web-logbook-1 exited with code 1

When I also set chmod 0666 data as well I get this error:
web-logbook-1  | ERROR	2024/12/15 20:11:30 main.go:162: unable to open database file: out of memory (14)

It seems to be working now after running chmod -R a+rwx ./data. Thanks for your help! :)

Ah, my bad, forgot about the rights for the directory. Glad it's working now!


unable to find version 1.2.0

I get this error :(
when I try to use.
here
Error: Unable to resolve action<EMAIL_ADDRESS>unable to find version 1.2.0

Howdy!
I'm still working on this repo, I should have left up the warning in the readme, sorry!!
Not sure why that's happening, right now I've only tried @main and not specific versions.
Everything should work on main, though...

This was a documentation issue, you either want @v1 or @v1.3.0 (needs the v) — but again, things are moving fast, so might be better to wait a couple days...


non common trend of readout pattern in each channel

I observed a non-common trend of the readout pattern in some order:
See https://github.com/prvjapan/pyird/issues/5#issuecomment-1000947088.
This results in artifact in one-d raw spectrum for a faint object (can find it by performing REACH.py)
To remove this, some preparations are needed

[x] RSD (raw spectral detector) matrix (rsdmat.py in spec) to see the effect (by taking median axis=1)
[ ] allocate channel position in a 1D raw spec

After that, consider how to remove it. probably GP?

The following is an example of the artificial dip due to the non-common trend of the readout pattern (REACH.py):

To make this figure, I implemented "RSD matrix" (spec/rsdmat.py), which is a matrix of (npix x norder) aligned to the detector 2D matrix. Here is the example of the conversion to use (for the above figure):
from pyird.spec.rsdmat import multiorder_to_rsd
rsd=multiorder_to_rsd(rawspec, pixcoord)

fig=plt.figure()
ax=fig.add_subplot(111)
for i,esp in enumerate(rawspec):
    plt.plot(pixcoord[i],esp,alpha=0.3, color="gray") #multi-order raw spec form for reference
plt.plot(np.nanmedian(rsd[:,-6:-1],axis=1),alpha=1.0, color="C1",label="median (-6:-1)") #can perform np.nanmedian for the axis of the orders
plt.plot(np.nanmedian(rsd[:,:],axis=1),alpha=1.0, color="C0",label="median")

Fitting residuals to GP2Dcross model, anyway I was able to correct the non-common channel trend.
Use rm_nct (remove noncommon trend) option in median_XY_profile(calim, rm_nct=True):

This is a schematic picutre of the RSD matrix:

I think this was done anyway.


fix fix spill data usage counter is not update correctly

Proposed changes
Issue Number: close #xxx
If QueryContext is destructed earlier than PipelineFragmentContext, spill_dir_ will be already moved to spill_gc directory, which result int spill data disk usage is not decreased.

run buildall


Both move and leave triggered for removed transition-group item if transition-group has display: flex

Version
2.5.17
Reproduction link
https://jsfiddle.net/pcb17vq4/
Steps to reproduce
See Fiddle.
What is expected?
Only a 'leave' is triggered when removing any item from the list displayed with display: flex.
What is actually happening?
Both a 'leave' and a 'move' are triggered

May I take this one?

Go ahead!

Great! Tonight I'm working on it.

This should not be a bug, It's just the difference between display: inline-block + position: absolute and display: flex + position: absolute.
In your fiddle there is,
.list-complete-leave-active {
  transition: all 1s;
  position: absolute;
}

Which sets the element to position: absolute when animating.
See the difference when position: absolute is added on element '4':

Try it in this fiddle: https://jsfiddle.net/pcb17vq4/1/

See @jkzing's fiddle above. Just remove position: absolute during transition and it will be working as expected.

Thanks, @jkzing! I found out that, but was trying to find a workaround.

Thanks for your efforts!


Terminal Extension pty terminals should respect dimension overrides

https://monacotools.visualstudio.com/DefaultCollection/Monaco/_build/results?buildId=67143&view=logs&j=2d2b3007-3c5c-5840-9bb0-2b1ea49925f3&t=384d930d-6775-58a0-3817-a8714909e89a&l=398
  1) window namespace tests Terminal Extension pty terminals should respect dimension overrides:

      AssertionError [ERR_ASSERTION] [ERR_ASSERTION]: 105 == 10
      + expected - actual

      -105
      +10

      at extensions\vscode-api-tests\src\singlefolder-tests\terminal.test.ts:395:8
      at l.fire (d:\a\1\vscode-reh-web-win32-x64\out\vs\server\remoteExtensionHostProcess.js:48:652)
      at y.$acceptTerminalDimensions (d:\a\1\vscode-reh-web-win32-x64\out\vs\server\remoteExtensionHostProcess.js:662:970)
      at processTicksAndRejections (internal/process/task_queues.js:89:5)

I was disabling it in https://github.com/microsoft/vscode/pull/90101, was just waiting on a test pass before merging.

/duplicate https://github.com/microsoft/vscode/issues/90064


Steal the McGuffin Thief Objective

About the PR

New Thief objective to steal a McGuffin
Why / Balance

YOU NEED THIS ITEM TO FINISH YOUR QUEST!!!
Technical details

adds new thief objective
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: New Thief Objective: Steal the McGuffin

Won't this turn into "break into kitchen and cook" in like 90% of rounds?

or simply ask the chef for it :godo:

There is already the objective to “steal” a warden hat, which is the difficulty of walking up to a lathe and is less likely to get eaten. Either way, you could steal ingredients, maybe need to steal a pig for bacon or something.

its funny but it makes the problem of half of thief objectives not being theft bigger

while the name is funny i agree that this isn't really something you can "steal." it adds onto the problem many thief objectives already have. As such, I'm closing this PR.


Optimize broadcast implementation in kadcast

fixes #1043

Codecov Report

Merging #1057 (695c552) into master (828364b) will decrease coverage by 1.47%.
The diff coverage is 59.45%.

@@            Coverage Diff             @@
##           master    #1057      +/-   ##
==========================================
- Coverage   50.51%   49.03%   -1.48%
==========================================
  Files         145      154       +9
  Lines        7544     8069     +525
==========================================
+ Hits         3811     3957     +146
- Misses       3066     3421     +355
- Partials      667      691      +24

Impacted Files
Coverage Δ

pkg/config/consts.go
0.00% <ø> (ø)

pkg/config/genesis/config.go
0.00% <0.00%> (ø)

pkg/config/genesis/presets.go
50.00% <ø> (ø)

pkg/core/candidate/validator.go
57.14% <ø> (ø)

pkg/core/chain/legacy.go
26.08% <0.00%> (-2.49%)
:arrow_down:

pkg/core/chain/mock.go
8.33% <ø> (ø)

pkg/core/consensus/agreement/accumulator.go
83.01% <ø> (ø)

pkg/core/consensus/agreement/handler.go
56.14% <ø> (ø)

pkg/core/consensus/agreement/store.go
62.12% <ø> (ø)

pkg/core/consensus/agreement/testutil.go
100.00% <ø> (ø)

... and 141 more

Continue to review full report at Codecov.

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


LESSPlugin lost live reloading of @imported files in 2.2.2

As discussed in this similar issue: https://github.com/fuse-box/fuse-box/issues/763#issuecomment-326933765
After updating from 2.1.0-beta.2 to 2.2.2 (and setting target: browser to fix an unrelated issue), I lost the live CSS reloading when changing nested .less files.
I tried the<EMAIL_ADDRESS>and it didn't fix the issue, I'm on windows but it seems unrelated (the LESS plugin do not send back the dependencies like the SASS one).

Some super nice update here<EMAIL_ADDRESS>You will get error reports in the browser, please test

Hi thanks for looking into it. Here's what I did and the results:

install<EMAIL_ADDRESS>clear fusebox cache
start my usual dev/watch task
my app is loading fine
changing something in a @imported .less file
node/fusebox process output this error (nothing in browser)

SyntaxError: Unexpected token o in JSON at position 1
    at JSON.parse (<anonymous>)
    at File.generateCorrectSourceMap (C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\fuse-box\dist\commonjs\core\File.js:381:39)
    at CSSPluginClass.transform (C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\fuse-box\dist\commonjs\plugins\stylesheet\CSSplugin.js:129:35)
    at tasks.push (C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\fuse-box\dist\commonjs\core\File.js:126:63)
    at context.queue.realm_utils_1.each.promise (C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\fuse-box\dist\commonjs\core\File.js:137:76)
    at iterate (C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\realm-utils\dist\commonjs\each.js:79:34)
    at iterate (C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\realm-utils\dist\commonjs\each.js:87:25)
    at C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\realm-utils\dist\commonjs\each.js:92:16
    at Promise (<anonymous>)
    at Object.exports.Each [as each] (C:\Data\Projects\Hrweb-git-tfs\HrWeb.WebClient\node_modules\realm-utils\dist\commonjs\each.js:9:12)

I have to clear the cache to be able to start my app again (fusebox has the same error on first build if I don't). I can debug if you need some values to help.

@guillaume86 can you reproduce your case in a repository? I am sure we will get it all fixed

Here's the minimal case I think:
https://github.com/guillaume86/fusebox-issue-780
If you uncomment the sourceMaps option in the gulpfile the error also comes up.

@guillaume86 discovered a bug there... that wasn't just related to less
<EMAIL_ADDRESS>tested your repo, all working just fine 👍

@nchanged great thanks for the fix

The live reloading is working fine now, but setting sourceMaps: true still triggers the same error, not really important for me but that's probably also worth a look.

Oh man, forgot about that one, I will check

I am now getting this error after upgrading to 2.3.2

this repo still works? https://github.com/guillaume86/fusebox-issue-780
i mean i can reproduce the issue there?

Sorry, I haven't checked out the version in the link yet. I just noticed after I upgraded from 2.2.31 -> 2.3.2 that when I build clean (after removing my .fusebox folder) that everything builds successfully. If I make a change to a source file or try rebuilding again it throws the exact same exception and stack trace that was described in this issue.

I checked out the example project and it seems to work fine. I will see if I can reproduce it.