IndexOutOfBounds Exception in Error Streams

Description:
@OnError(action='STREAM')
define stream inputStream (symbol string, price long, volume long);

@info(name = 'query1')
from inputStream
select symbol, str:charAt("WSO2", 10) as charFrequency
insert into outputStream;

@info('errorQuery')
from !inputStream
select *
insert into ErrorStream;

The above siddhi apps produce IndexOutOfBoundsException when error event is published,
java.lang.ArrayIndexOutOfBoundsException: 4

	at io.siddhi.core.event.stream.StreamEvent.setOutputData(StreamEvent.java:164)
	at io.siddhi.core.event.stream.converter.FaultStreamEventConverter.convertData(FaultStreamEventConverter.java:96)
	at io.siddhi.core.event.stream.converter.FaultStreamEventConverter.convertData(FaultStreamEventConverter.java:114)
	at io.siddhi.core.event.stream.converter.FaultStreamEventConverter.convert(FaultStreamEventConverter.java:48)
	at io.siddhi.core.stream.StreamJunction.handleError(StreamJunction.java:412)
	at io.siddhi.core.stream.StreamJunction$Publisher.send(StreamJunction.java:503)
	at io.siddhi.core.stream.input.InputDistributor.send(InputDistributor.java:49)
	at io.siddhi.core.stream.input.InputEntryValve.send(InputEntryValve.java:75)
	at io.siddhi.core.stream.input.InputHandler.send(InputHandler.java:52)

Version
Siddhi 5.1.8

I tried to reproduce the behavior running the following testcase with your Siddhi application. But I couldn't reproduce the issue.
Below is the test-case used.
    @Test
    public void faultStreamTest() throws InterruptedException {
        SiddhiManager siddhiManager = new SiddhiManager();
        String siddhiApp = "" +
                "@OnError(action='STREAM')\n" +
                "define stream inputStream (symbol string, price long, volume long);\n" +
                " \n" +
                "@info(name = 'query1') \n" +
                "from inputStream \n" +
                "select symbol, str:charAt(\"WSO2\", 10) as charFrequency \n" +
                "insert into outputStream;\n" +
                "\n" +
                "@info('errorQuery')\n" +
                "from !inputStream \n" +
                "select * \n" +
                "insert into ErrorStream;" +
                "";
        SiddhiAppRuntime siddhiAppRuntime = siddhiManager.createSiddhiAppRuntime(siddhiApp);
        siddhiAppRuntime.addCallback("!inputStream", new StreamCallback() {
            @Override
            public void receive(Event[] events) {
                EventPrinter.print(events);

            }
        });
        InputHandler inputHandler = siddhiAppRuntime.getInputHandler("inputStream");
        siddhiAppRuntime.start();
        inputHandler.send(new Object[]{"IBM", 100L, 100L});
    }

I'm getting the following output. Which is the intended behavior.
[Event{timestamp=1575007419806, data=[IBM, 100, 100, io.siddhi.core.exception.SiddhiAppRuntimeException: Index argument 10 is negative or not less than the length of the given string WSO2. Exception on class 'io.siddhi.extension.execution.string.CharAtFunctionExtension'], isExpired=false}]

It seems the issue was due to the wrong format of object array was send through input handler.


[FLINK-15356][metric] Add applicationId to flink metrics running on yarn

What is the purpose of the change
When sending metrics to Prometheus, these systems have only the Flink job ID, and the Flink job ID is UUID, which cannot be associated with the application job on the yarn. Therefore, we need to increase the applicationId when running on yarn. This helps us to accurately find the corresponding job on yarn when the metric is abnormal.
Brief change log

DefaultDispatcherResourceManagerComponentFactory.java
JobManagerMetricGroup.java
TaskManagerMetricGroup.java
UnregisteredMetricGroups.java
ScopeFormat.java
MetricUtils.java
TaskManagerRunner.java

Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

This feature needs more discussion; closing the PR for now.


⚠ Bofrost.fr has degraded performance

In 9f8354c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1147 ms

Resolved: Bofrost.fr performance has improved in eedab69 after 6 minutes.


Consider improving code formatting

It seems the tabs are not consistent. Would suggest to improve it and use space tabs.

The standard for GDScript is tab characters as tabs.

@aaronfranke you can change it to space tabs


Fix danger-swift parameters filter

The filter was removing the parameter before the option instead of the one after

merge on green


chore: increase max replica as we have kafka backlog now

Secure Coding Practices Checklist Link

https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Checklist

[ ] Input Validation
[ ] Output Encoding
[ ] Authentication and Password Management
[ ] Session Management
[ ] Access Control
[ ] Cryptographic Practices
[ ] Error Handling and Logging
[ ] Data Protection
[ ] Communication Security
[ ] System Configuration
[ ] Database Security
[ ] File Management
[ ] Memory Management
[x] General Coding Practices

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.


Not functioning

After updating to the latest version, the complete page header and tab bar are gone.
Logs:
ogger: frontend.js.latest.202005191
Source: components/system_log/init.py:209
First occurred: 21:20:40 (22 occurrences)
Last logged: 21:27:48
https://www.gloudemans.info:8123/hacsfiles/custom-header/custom-header.js:1:15443 TypeError: undefined is not an object (evaluating 'e.voice.setAttribute')
https://www.gloudemans.info:8123/frontend_latest/chunk.5ef968208258097f0ba7.js:2:20625 TypeError: undefined is not an object (evaluating 't._leaflet_pos')
https://www.gloudemans.info:8123/hacsfiles/lovelace-swipe-navigation/swipe-navigation.js:10:20 TypeError: null is not an object (evaluating 'root.lovelace')
Logger: frontend.js.latest.202005190
Source: components/system_log/init.py:209
First occurred: 21:19:08 (5 occurrences)
Last logged: 21:19:25
https://www.gloudemans.info:8123/hacsfiles/custom-header/custom-header.js:596:55148 TypeError: null is not an object (near '...ha-panel-lovelace").lovelace,!t)return;...')
https://www.gloudemans.info:8123/hacsfiles/frontend-20200521162326.js:1:17191 NotSupportedError: Cannot define multiple custom elements with the same tag name
HA 0.110.1
Plugin 1.4.3
HACS 1.0

Same here with the same setup

same problem

again all broke

Same issue. My page header and tab bar are gone.
HA 0.110.1
Plugin 1.4.3
HACS 1.0

Same issue. My page header and tab bar are gone.

Can't do with the new HACS

весь ад вырвался на свободу, потерял все мое меню - вернулся обратно в 1.3.8, меню снова отображается -
DOWNGRADE от 1.4.3 парней.

Не могу сделать с новым HACS

same problem

If your issue is like mine then it's because you have conversation: commented out in your HA configuration.yaml. Unquoting it resolves this error for now but I'm sure maykar will address it properly when time permits. You can add voice_hide: true to your CH config to hide the icon for now.

If your issue is like mine then it's because you have conversation: commented out in your HA configuration.yaml. Unquoting it resolves this error for now but I'm sure maykar will address it properly when time permits. You can add voice_hide: true to your CH config to hide the icon for now.

Thank you, adding conversation: into my HA configuration.yaml allows customer header to come back!
Where is the file for the CH config to add voice_hide: true at? No rush, just happy to have tablets on walls working again right now.

Either raw configuration editor or yaml depending on which way you set it up.  Same place you defined all other CH options.

Either raw configuration editor or yaml depending on which way you set it up. Same place you defined all other CH options.

Hmmm, I just have done it all via the web UI -- I just use it to slim down the header across the board and that's it. I don't see a way via web UI to add anything other than exceptions?

Hide Voice Button toggle should do it.

@CorneliousJD

web UI

how to come on web UI?

@beliy1985
add conversation:
into my HA configuration.yaml allows customer header to come back!

Либо необработанный редактор конфигурации, либо yaml, в зависимости от того, как вы его настроили. Там же, где вы определили все остальные параметры CH.

Хммм, я только что сделал все это через веб-интерфейс - я просто использую это, чтобы уменьшить заголовок по всем направлениям и все. Я не вижу способ через веб-интерфейс добавить что-либо, кроме исключений?

How to enter such settings

Same config as everyone else.  Updated to 0.110.1 today.  Everything else is completely updated.  Running on rPi 3B+
Adding conversation: to configuration.yaml restore to header.  Please fix.  Have no need to the conversation integration.

This got fixed so fast I didn’t even notice it.
Thanks!

all hell broke loose, lost all my menu - reverted back to 1.3.8 menu is shown again -
DOWNGRADE from 1.4.3 guys.

Can't do with the new HACS

yes you can - simply click on reinstall on the 3dot icon, then choose a different version.


Remove djelm_src scripts

When creating djelm programs the binding scripts with the hydration code is added to the src/djelm_src directory. These files have the following downsides.

They are not intended to be used by users and create extra "stuff" to shove in to source control.
Changes djelm makes to these files will mean re-generating them by the user, and thus can make them stale.
They are only required when compiling to glue together hydration code and Elm programs.
djelm_src is just a weird name and doesn't really mean anything.

Proposed solution

Generate these files on the fly when compiling.
Shove generated files in to elm-stuff so the user doesn't see them or need to keep track of them. Because other generated djelm files are thrown in to elm-stuff and versioned, we could make changes to how we compile without the user needing to do anything.

Released with 0.16.0


Add an EXPLAIN variant that lists all of the VG terms in a column or the _MIMIR_CONDITION

This is now a server side implementation. I haven't added anything to the parser, the WebAPI handles the request through an alternate route (localhost:9000/vgterms?). Consequently, the feature is not available for the console as of now.


Overhaul picking benchmarks

Objective

Part of #16647.
This PR goes through our ray_cast::ray_mesh_intersection() benchmarks and overhauls them with more comments and better extensibility. The code is also a lot less duplicated!

Solution

Create a Benchmarks enum that describes all of the different kind of scenarios we want to benchmark.
Merge all of our existing benchmark functions into a single one, bench(), which sets up the scenarios all at once.
Add comments to mesh_creation() and ptoxznorm(), and move some lines around to be a bit clearer.
Make the benchmarks use the new bench! macro, as part of #16647.
Rename many functions and benchmarks to be clearer.

For reviewers
I split this PR up into several, easier to digest commits. You might find it easier to review by looking through each commit, instead of the complete file changes.
None of my changes actually modifies the behavior of the benchmarks; they still track the exact same test cases. There shouldn't be significant changes in benchmark performance before and after this PR.
Testing
List all picking benchmarks: cargo bench -p benches --bench picking -- --list
Run the benchmarks once in debug mode: cargo test -p benches --bench picking
Run the benchmarks and analyze their performance: cargo bench -p benches --bench picking

Check out the generated HTML report in ./target/criterion/report/index.html once you're done!

Showcase
List of all picking benchmarks, after having been renamed:

Example report for picking::ray_mesh_intersection::cull_intersect/100_vertices:

cc @aevyrie, since you're the person who I know is most familiar with picking, and most likely to catch any mistakes in the documentation!
cc @mockersf, since you wrote the original version of these benchmarks in https://github.com/aevyrie/bevy_mod_raycast/pull/17.
No pressure if neither of you want to review, just thought you might be interested!

@BD103 what do you think still needs to be done before we merge this? Looks like the CI failure is just cargo fmt.

@BD103 what do you think still needs to be done before we merge this? Looks like the CI failure is just cargo fmt.

I also wanted to add one more test case to verify that the ray does / doesn't intersect the mesh when cargo test --bench picking was run, but it's ready now!

Sometimes I use the mobile app to put the Waiting label on my own PRs so that I can remember to fix a merge conflict or a CI check when I get to my laptop. It looks like they're doing the same thing to me.


SSH connection drops before sending the command

Describe the bug
It seems that it doesn't work through SSH.
To Reproduce
Steps to reproduce the behavior:

In config, opt for ssh connection (authkey or password doesn't matter, same behaviour)

Additional context
Before each remote command, EOF is received and the response is consequently empty
Logs*
2024-10-15 17:00:36 - INFO - Starting resource allocation process...
2024-10-15 17:00:36 - DEBUG - Inside run_command: use_remote_proxmox = True
2024-10-15 17:00:36 - DEBUG - Executing command remotely.
2024-10-15 17:00:36 - DEBUG - Running remote command: pct list | awk 'NR>1 {print $1}'
2024-10-15 17:00:36 - DEBUG - Attempting to connect to Proxmox host via SSH...
[...]
2024-10-15 17:00:36 - INFO - Connected (version 2.0, client OpenSSH_9.2p1)
[...]
2024-10-15 17:00:37 - DEBUG - userauth is OK
2024-10-15 17:00:37 - INFO - Authentication (password) successful!
2024-10-15 17:00:37 - DEBUG - SSH connection established successfully.
[...]
2024-10-15 17:00:37 - DEBUG - [chan 0] EOF received (0)
2024-10-15 17:00:37 - DEBUG - Remote command 'pct list | awk 'NR>1 {print $1}'' executed successfully. Output:
2024-10-15 17:00:37 - DEBUG - [chan 0] EOF sent (0)
2024-10-15 17:00:37 - DEBUG - Dropping user packet because connection is dead.
2024-10-15 17:00:37 - DEBUG - Dropping user packet because connection is dead.

The issue you're encountering seems to stem from the SSH session terminating unexpectedly, leading to an empty response when attempting to execute the command remotely. This could be due to multiple reasons, ranging from session timeout, incorrect command execution, or configuration issues. Here’s how to approach and troubleshoot the issue:
Key Issues from Logs:

EOF received immediately after command execution: This indicates that the SSH connection is being closed or terminated unexpectedly after the remote command runs, causing no output to be returned.

Dropping user packet because connection is dead: This suggests that the SSH connection may be timing out, disconnecting, or otherwise terminating during the command execution.

Troubleshooting Steps:

Verify SSH Configuration:

KeepAlive settings: Check if ClientAliveInterval and ClientAliveCountMax are set in your SSH server configuration (/etc/ssh/sshd_config on the Proxmox host). These control how long the server waits before considering the connection as inactive. For instance:ClientAliveInterval 60
ClientAliveCountMax 3

This ensures that the connection remains alive even during longer commands.

Check for Background Processes: Sometimes, the remote command could spawn a background process that causes the session to close. Run the command locally or in a manual SSH session and confirm it works as expected.

Add Timeout to the Command: The command being executed (pct list | awk 'NR>1 {print $1}') might take longer than expected to execute. Adding a timeout or debugging it by running it manually on the host could help:
timeout 30 pct list | awk 'NR>1 {print $1}'

This will help isolate whether the command itself is taking too long or encountering an issue.

Check SSH Logs: Investigate the Proxmox system's SSH logs for additional information on the disconnection. Look at /var/log/auth.log for any SSH-related messages, which may help determine why the connection was dropped.

Test with Different Commands: Try simpler commands through SSH to see if the issue is with the particular command or the SSH session in general. For example, use echo "Test" or uptime to see if they return correctly.

Inspect Command for Background Process: Ensure that the pct command isn’t unintentionally spawning processes that could lead to session disconnection. Also, try running the command with logging and error redirection to capture any output, e.g.,:
pct list | awk 'NR>1 {print $1}' > /tmp/pct_output.log 2>&1

Then check /tmp/pct_output.log for any clues.

Verify SSH Version Compatibility: Ensure that the OpenSSH client on your local machine and the SSH server on the Proxmox host are fully compatible, as some versions may exhibit odd behavior with particular configurations.

SSH Verbose Output: Enable verbose output for SSH to gather more information on the session:
ssh -v user@hostname "pct list | awk 'NR>1 {print $1}'"

This will provide detailed information about each step in the SSH connection process.

Further Diagnostics:

Test with both password and key-based authentication to rule out any authentication-related issues.
If the issue only occurs with certain commands, look for any resource limits or restrictions imposed by Proxmox, such as CPU, memory, or disk quota, which might cause the session to drop.

⚠  If you can then share logs or output here it will be really useful since I failed into reproduce the issue.


Can’t select/use custom templates

Microsoft PowerToys version
v0.82.1
Installation method
Microsoft Store
Running as admin
No
Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
Basically I can make a custom template on FancyZones Editor no problem but as soon as I try and select it I get an issue

✔ Expected Behavior
To be able to select a custom template
❌ Actual Behavior
An error prompt
Other Software
No response

Fixed by going to C:\Users[your user]\AppData\Local\Microsoft\PowerToys\FancyZones and making sure the file "applied-layouts.json" wasn't hidden in its properties


Fix handling of feedforward acceleration in BipedalLocomotion::TSID::JointTrackingTask::setSetPoint

Fix https://github.com/ami-iit/bipedal-locomotion-framework/issues/718 .

cc @GiulioRomualdi @Gio-DS

Compilation with Eigen in 20.04 is failing with:
 [ 80%] Building CXX object src/TSID/tests/CMakeFiles/DynamicsTaskTSIDUnitTests.dir/DynamicsTaskTest.cpp.o
/home/runner/work/bipedal-locomotion-framework/bipedal-locomotion-framework/src/TSID/tests/JointTrackingTaskTest.cpp: In function ‘void CATCH2_INTERNAL_TEST_0()’:
/home/runner/work/bipedal-locomotion-framework/bipedal-locomotion-framework/src/TSID/tests/JointTrackingTaskTest.cpp:70:34: error: ‘begin’ was not declared in this scope
   70 |             for (auto& element : feedforwardDesiredJointAcc)
      |                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~

CI seems happy, ready for review @GiulioRomualdi .

CI seems happy, ready for review @GiulioRomualdi .

(Please merge with squash as the history is messy)

Thank you @traversaro


How to Set Environment Variables?

Is there a way to pass in environment variables as in the "command" job component as a parameter? Since in SDK V2, the Environment class doesn't have environment_variables as an attribute, how should we pass in environment variables to a spark job?

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

ID: 47e45bd8-edb0-e7a9-e7e0-c338d62062d4
Version Independent ID: d5151ae2-2ee9-ade3-0dcd-319278f42393
Content: Submit Spark jobs in Azure Machine Learning - Azure Machine Learning
Content Source: articles/machine-learning/how-to-submit-spark-jobs.md
Service: machine-learning
Sub-service: mldata
GitHub Login: @ynpandey
Microsoft Alias: yogipandey

@winnieshen96
Thanks for your feedback! We will investigate and update as appropriate.

@winnieshen96
Yes, you can pass environment variables to a Spark job in Azure Machine Learning using the Environment class in the SDK v2. Although the Environment class does not have an environment_variables attribute, you can still set environment variables by using the Environment class in combination with the RunConfiguration class.
Here is an example of how to set environment variables for a Spark job using the Environment class and RunConfiguration class:
from azureml.core import Environment, RunConfiguration
Create an environment
myenv = Environment.from_conda_specification(name="myenv", file_path="./myenv.yml")
Set environment variables
myenv.environment_variables = {"EXAMPLE_ENV_VAR": "EXAMPLE_VALUE"}
Create a run configuration
myrunconfig = RunConfiguration()
Assign the environment to the run configuration
myrunconfig.environment = myenv
Submit the Spark job using the run configuration
In this example, we first create an Environment object from a Conda environment specification file. We then set the environment variables for the Environment object using the environment_variables attribute. Next, we create a RunConfiguration object and assign the Environment object to the environment attribute of the RunConfiguration object. Finally, we submit the Spark job using the RunConfiguration object.
You can modify this example to suit your specific needs. Let me know If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@winnieshen96
Please Note, GitHub forum is dedicated for docs related issues. For any technical queries or clarifications, we encourage to utilise Microsoft Q & A platform. Kindly raise your query on Microsoft Q&A Platform


Feature Requestion: Improved visualization of missing permission errors

First, thanks for working on and releasing Aquascope, I think it's going to have a massive impact on Rust pedagogy.
I was trying it out to see if it could help my students understand one of their most common errors where they attempt to return a mutable reference from an immutable one, and while the information is technically displayed it's not necessarily easy to understand it.
Here's the test program I used:
fn omg<T>(x : & T) -> &mut T {
  &mut x
}

fn main() {
  omg(&mut 5);
}

Aquascope shows (also has a trace which is uninformative here):

The permissions show that *x only has +R, but there's no display showing we want a &mut T (and thus need +R +W +O).
Perhaps one of those permissions boxes could be shown for the result when there's a borrow check error?

Thank you for trying out the tool, we hope your students find it useful!
If I understand your request correctly, it can be summed up as: "in the function omg we can see that *x gains read permission, but there isn't something that directly says we need write permission".
I would argue that this feature is available. In the following screenshot, I've hovered over those two colored dots on line 2 to the left of x.

This represents a permission expectation, saying that in order to take a unique reference, we need both read and write permissions. However, the blue W is hollow showing that this permission is missing. Thus symbolizing a permission violation, in this case, promoting a shared reference to a unique reference.
If you're demonstrating this with the online playground, at the moment you'd have to hover the cursor over the dots to expand them to the appropriate characters (they are also color-coded). If you tried your hand at embedding this into mdBook, you can make the characters expanded by default with a `{boundaries}` annotation on line 2. Which combined would look like this:

```aquascope,interpreter+permissions,boundaries,stepper,shouldFail
fn omg(x : & T) -> &mut T {
  &mut x `{boundaries}`
}

fn main() {
  omg(&mut 5);
}```

Note that we don't currently guarantee any stability on the mdBook interface.
Please let us know if this still doesn't capture the feature you're looking for.

Oh I didn't think of doing that, it's not obvious from the interface alone that there is an error, perhaps this info could be highlighted or otherwise identified to show that there's an error?


Add dw_conv for aarch64 and fix a barrier bug with 1x1 conv

Description
This pull request adds a depthwise convolution for aarch64 and a bug fix for the barrier with 1x1 conv.
Fixes # (github issue)
Checklist
Code-change submissions

[ ] Do all unit and benchdnn tests (make test and make test_benchdnn_*) pass locally?
[ ] Have you formatted the code using clang-format?

New features

[ ] Have you added relevant tests?
[ ] Have you provided motivation for adding a new feature?

Bug fixes

[ ] Have you added relevant regression tests?
[ ] Have you included information on how to reproduce the issue (either in a github issue or in this PR)?

RFC PR

[ ] Does RFC document follow the template?
[ ] Have you added a link to the rendered document?

Please update xbyak_translator_aarch64 and send pull-request again.

Updated xbyak_translator_aarch64.


Point tracker increases player points everytime mvp item is spawned/dropped to the ground

Being rewarded with boss MVP points should happen only once per raid per boss, but picking up and dropping the same MVP item multiple times will result to an incorrect amount of points.
Easy repro in game: Defeat any boss, pick up the MVP item, drop the MVP item, and repeat last two steps.

fixed in eeded2330221108901f95613cb66b9f05898692c


OSX: Expose 3-finger swipe events in browser-window

This adds 4 new events to browser-window for 3-finger trackpad swipes on OS X.

swipe-up
swipe-right
swipe-down
swipe-left

#4181 added support for 2-finger scroll events on OS X, which allow apps to handle some forms of back/forward navigation. However, OS X also exposes a swipeWithEvent method on NSResponder for 3-finger swipes, and it's possible to set these as the "Swipe between pages" gesture. These swipes are only directional, with no velocity, so are easily exposed as one-off events. They will only fire if the option is set in System Preferences to Swipe with two or three fingers or Swipe with three fingers

I think we should probably add just one swipe event and pass deltaX and deltaY as parameters.

Given the deltas will only ever arrive in 4 combinations, should the API just expose a single direction string instead of the raw deltas?

I started rewriting this PR to expose the deltas in the API, then got annoyed with having to reimplement the delta checking at the application level, and thinking about everyone else who would have to do the same. I suspect optimising for a more intuitive API vs exposing the underlying mechanism makes more sense in this case.
That said, I agree with collapsing 4 distinct events down to one event with an enum argument; the main reason I didn't initially was to avoid dealing with passing around arguments in the internals.

I pushed a version with a single direction argument, but I have one with deltas in a separate local branch that I can push instead if you'd prefer.

I'm good with a single direction argument, thanks for this!


Normalization and cleanup

WIP.
I have thus far normalized the exponent test values with a script similar to the following (this is a slightly improved version):
import re

tofix = ["p-mbar", "p-mmHg", "density", "ppn", "rhorhon", "sqrt-rhorhon", "dynamic-viscosity", "kinematic-viscosity", "thermal-conductivity", "specific-weight", "air-number-density", "frequency", "mean-free-path"]

currexp = {}

for fix in tofix:
	currexp[fix] = "0"

filenames = ["iso-2533-1975-table5.yaml", "iso-2533-1975-table6.yaml", "iso-2533-1975-table7.yaml"]

for filename in filenames:
	input = open(filename, "r")
	output = open("fixed-" + filename, "w")
	for line in input:
		noneMatched = True
		for fix in tofix:
			if line.startswith("    " + fix + ": "):
				match = re.search(r'( (\+|-)[0-9]*)$', line)
				if match is not None:
					currexp[fix] = "e" + match.group(0)[1:]
					output.write(line[:-len(currexp[fix])-1] + currexp[fix] + "\n")
				else:
					output.write(line[:-1] + currexp[fix] + "\n")
				noneMatched = False
				break
		if noneMatched:
			output.write(line)
	input.close()
	output.close()

Line 14830 of iso-2533-1975-table5.yaml has bad data (lowercase L character in place of a number: p-mmHg: 9.4785le+0, I noticed during comparison since it tripped up the former version of my script), but I don't know what the correct value should be.
Metanorma PR checklist

[ ] Breaking changes (list related PRs)
[ ] Documentation update required (create task for this)
[ ] External dependency introduced (documentation update need)
[ ] Gem with native library introduced

Suppose it's 1, and that there will be more, albeit more subtle errors to correct anyways... With the aid of the very tool?!? :)

OK, actually, maybe let's get it merged like this? Fixes #15.

@newbthenewbd thank you! Let's get this merged. Next we want to move these values into one large test file... (or split for practicality)


Incorrect use of the name 'Tick' Should be 'Bar'

Nice codebase! One thing: You define a tick as open/high/low/close/volume data, but that is incorrect. please take a look at investopedia.com. A tick is a trade, usually the last trade placed, which means it contains: price, quantity, timestamp, symbol at minimum. What you call a 'tick' is in fact a 'bar'. You should consider renaming to fix that as it creates confusion when trying to use the javadocs.

"Bar" comes with a graphical meaning. And for now I don't want to reduce the tick concept to its graphical part. I agree that the name is not perfect but I did not find another better one. I see it like a clock tick with OHLCV for the preceding period.

I see why you wouldn't want to change it (after all, it's pretty embedded into the codebase) but I do agree with @marshallgs that it is confusing given that "tick" is so widely used across the industry.
Internally to my own system I've used Price to mean the instantaneous prices from the market (actually containing a bid & an offer price) and PeriodicPrice to mean the OHLCV or tick as per the ta4j parlance.
My PeriodicPrice has open, high, low & close fields plus start and end date-times - in effect they reprsent a summary over the period (end minus start).
These are often called Candles (due to their shape on the graph).
Looking at ta4j I think Trade models the real-world tick-prices.

I can see that Bar has a graphical meaning, or reference. However I think this is a minor issue, and I think Tick and Bar are well known and understood.
Here are some alternative terms that you may wish to consider

Quote or PriceChangeEvent
Window - a more tech specific term for a time span of data - as an alternative to Bar. Alternatively OHLC.

Transfered to https://github.com/ta4j/ta4j/issues/9.


SAK-48323 webcomponents. Bundle and remove empathise.

https://sakaiproject.atlassian.net/browse/SAK-48323

@adrianfish @ern I think we should avoid merging big changes before
tickets from Unidigital were merged. Please, take in mind. Thanks!

@jesusmmp @mpellicer Agreed. That's why I've put the PR up early, for transparency.

@mpellicer Are there particular commits in the s2u branch that you are worried about? This PR mainly affects the web components, the lit stuff, so I'm hopeful we won't get a lot of overlap. Is there any chance you can prioritise merging the risky commits into master? Can you point me to those commits so I can take a look?

@mpellicer @bgarciaentornos
This PR's getting a bit painful with the conflicts and I'd like to get it in soonish, if possible. So, if we could get any webcomponents PRs into master soon for review, that would be great.

Well @adrianfish, we have very very limited resources to contribute stuff to Master right now, but we made the weekly meeting today and I've assigned @stetsche to contribute the most critical features, with the help of @bgarciaentornos.
We're going to contribute to Master this two tickets very soon:
https://sakaiproject.atlassian.net/browse/S2U-34
https://sakaiproject.atlassian.net/browse/S2U-5
@bgarciaentornos Do you feel that other tickets need contribution? I don't see any here:
https://github.com/sakaiproject/sakai/commits/s2u-22x/webcomponents
Thanks

You're right, those 2 seem to be the only tickets with related changes (although I'm seeing here other backend changes on tools like assignments but I'm guessing that's because it's still a draft).

Great news, thanks for putting people on this. I know you're on a tight schedule.

Hi @adrianfish,  I can not work on the migration of S2U-5 at the moment because of other tasks. It will take some time until I can get to it, so this PR can go in first, let's not block it further!

@stetsche Great news. I can stop rebasing it now then :)
I'll do a load of testing today and I'll ask you for a review. Obviously, it is a big PR, but if you could just skim over it that would be good.

@stetsche Thanks for the review, btw. Really appreciate it. You spotted some real crap from me :)

@stetsche The ui components are staying in the main Sakai repo for now. It seemed to make sense to just change things gradually. We can get the benefits of a monorepo approach from inside the webcomponents module. We can publish components to the npm repo, all that stuff, but the code still stays together. That may change in the future, but I didn't want to bite too much off at once.

@adrianfish You are welcome, thanks for the info!

Adrian so this will all end up in the sakai-ui repo? and I am guessing this is the a step before that happens? If so I think it would be a good idea to update the webcomponents strategy doc and layout the direction there?
this is very good direction so that we can start publishing webcomponents separately from Sakai, nice!!!

Regarding the package author entries:
https://docs.npmjs.com/cli/v10/configuring-npm/package-json#people-fields-author-contributors
It's recommended to supply the email address and the one I've used is attached to a sakai-ui npm repository at https://www.npmjs.com/~sakai-ui. At the moment we are not publishing to it as we are linking between npm modules in our bundling, but we will be at some point.

The problem is that we already have the sakai-ui namespace on npm, and the login is<EMAIL_ADDRESS>We could use the expanded multiple lines version, but the single line conforms with the way git histories show things.


SIGINT-1197: Expose parameters to generate sarif report and upload in Azure as artifact

Closing this MR as these changes are incorporated as part of https://github.com/synopsys-sig/synopsys-extension/pull/112


Optionally add tags to notifications

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Notification, Other
🔖 Feature description
modification to the existing notifications framework.  adding the option to include tags (maybe per tag, or all tags would be fine) in the notifications that go out.
Currently, we are monitoring 2 systems.  API (production) and API (development).  the (tags) for each monitor are not included in the notification resulting in this:
Uptime Kuma Alert
Message
[API] [:red_circle: Down] Request failed with status code 502
Unfortunately, it doesnt tell me which environment is down, without changing the monitor names.  but the full monitor names show up on the status pages, and i dont really want to append -Production or -Development to all of my monitor names for the public status page, and currently only show the tags on our internal only status page.
✔ Solution
Allow an option when configuring notifications to include the tags in the message.  They can be on their own line, comma separated.
We use the Slack notifications, if you want to test with this one before expanding to the other services :)
I know some notification services may run into length issues due to this request and might need to be addressed or omitted from this request.
❓ Alternatives
Adding the environment suffix to each monitor name so its included in the notification text.  I was hoping to leverage tags in this way without needing to make the monitor names longer than they need to be.
Also, I dont really want/need the environment tags/ or names on the public facing status page, but they are helpful for our internal facing status page.  That is why i started leveraging the tags i the first place, and it works well for that.
📝 Additional Context
No response

would like to work on this !

Would love this too. I have dozens of sites across multiple different servers on multiple different hosts, and while tags are the ideal place to store details of the server and host, I'd really like it if they could be displayed in the notifications so that I'd know instantly which server was down.

Just discovered Kuma ; great tool.
We have several environment across the world. Having the tags in the Teams notification could be a good way to identify quickly the problem's location.

I have been wanting to see this possibility since tags exist 👍

here is our contribution guide: https://github.com/louislam/uptime-kuma/blob/aa676150eb4cc803fe3de10f186a083f44a1c658/CONTRIBUTING.md

Please consider the Alternative method in the OP. Here's my simple use case and unfortunately changing Tags is way more work than pre-pending some unique identifier.
In Uptime Kuma backups it says to backup the entire docker volume folder instead of using the backup feature. Then I take that docker folder onto my test environment. When I do that, the same exact messages for failures are sent to me in Telegram. There is no way to make them unique so that I can tell the difference between Production and Test environments that run the same set of containers for me. I would love for NOTIFICATION settings (rather than tags) to have some prepend or append feature where I can add "T-" for test or "P-" for production and only do that change once after copying my docker folder. Having tags means I have to change 27 items in my case and hence the reason why adding this at the Notification config level.
Thank you very much for this AMAZING software! Way better than anything I've tried so I'm thoroughly impressed.

yes please add mor option to notification text. I need Tags in my Push Notifications


permission for porting to 1.20.1

im wondering if i can have permission to port this over to 1.20.1 my self, im happy to do all the work necessary i just want your permission to use what you've done and work off that

I will happily give you permission, and if you have any questions about the port I'd be happy to help. I just don't have time to setup a test server, eclipse, forge, gradle and all the other required crap.
Please submit any changes on the 1.20.1 branch as a GitHub pull request!

thanks, it may take a bit, but ill get it done

hey question, whats the best way to chat/communicate? i do have questions about how some of you code works (mainly the chat formating and color handeling) ive got the base components working and the luckperms and essentials intergrations working and tested in a development server as well as a server im running for me and some friends

GitHub is good, if you create your own forked repository I will star and watch it. You can freely email me at<EMAIL_ADDRESS>for any in-depth or technical questions you might have.


/servermanager.sh: line 37: ./PalServer.sh: No such file or directory

At first I got the problem from the other issue but that seems to be fixed. Now I'm getting this error: /servermanager.sh: line 37: ./PalServer.sh: No such file or directory
Thanks for creating this Docker container though :)
Software setup

OS: Fedora Server

Edit:
Fixed by running this command: chown -R 1000:1000 game/
Also, change the ports section to this so you can connect:
ports:
- "8211:8211/udp"

maybe fixed by this commit

Same issue here

maybe fixed by this commit

And since I'm using the latest image from hub I guess this commit did not fix it.

https://github.com/jammsen/docker-palworld-dedicated-server/issues/1#issuecomment-1900630342

Fixed by running this command: chown -R 1000:1000 game/
Also, change the ports section to this so you can connect:
ports:

"8211:8211/udp"

Fixed by running this command: chown -R 1000:1000 game/
Also, change the ports section to this so you can connect:
ports:
"8211:8211/udp"

Please see #1
This is dublicate content, im closing this in favor to only keep 1 ticket up-to-date, but thanks for reporting!


sendQuery documentation outdated and I need an example

The documentation for sendQuery is confusing:
First, you can use H.query as usual within the handleExtraQuery function you provide to Formless as input.

Is this up-to-date? I find no function handleExtraQuery in formless. I'm also not sure about H.query.
Also, it would be awesome to have an example of how to use the feature. What I'm trying to do is send the formless component a query so some input fields are then reset (I don't want to do this in the submit button's onClick handler since I want to make a HTTP request first and check the result).

That documentation does look out of date. To clarify, sendQuery is meant for when you have a formless component that has a child component that you want to query; you're sending the query through the formless component.
There is an example of sendQuery used in the "real world" example. In the below code, we're sending a query through the "options form" to a dropdown component located in that form in order to manually reset it:
https://github.com/thomashoneyman/purescript-halogen-formless/blob/2e94bff0d99f48dae004b4fac943abb0ba62b5b2/example/real-world/GroupForm.purs#L186
However, if you just want to send a query to the formless component you've written and have that component reset a few fields, then you don't want sendQuery.
There isn't an example of that, unfortunately, but it's the same idea as writing your own Actions for a formless component. First, you define your query type, similar to how the options form defines its action type:
https://github.com/thomashoneyman/purescript-halogen-formless/blob/3f6ffef29ac13587fea0ee2c608567c93fd867ae/example/real-world/OptionsForm.purs#L106-L107
Then, you write your query handler, similar to how the options form defines its handleAction:
https://github.com/thomashoneyman/purescript-halogen-formless/blob/3f6ffef29ac13587fea0ee2c608567c93fd867ae/example/real-world/OptionsForm.purs#L180-L183
Finally, you add the handler to your form spec with handleQuery = handleQuery:
https://github.com/thomashoneyman/purescript-halogen-formless/blob/3f6ffef29ac13587fea0ee2c608567c93fd867ae/example/real-world/OptionsForm.purs#L123-L127
With this all wired up, you can now send queries to your form component with the ordinary Halogen.query function.

@pmiddend Did this help you move forward, or are some things unclear?

Ah, sorry to let you hanging like this. This actually made it much clearer! However, while browsing through the real-world-halogen code, I discovered that my special use case can apparently be handled more easily using this code:
void $ H.query F._formless unit $ F.asQuery $ F.reset _dropId

Which is in my parent component handleAction method. I'm not really sure what it does, but it seems to work.

With that in mind I'm going to go ahead and close this. Glad things are working for you!


[Bug] Wacatac.H!ml in user32.dll

Before Reporting

[X] I found no existing issues matching my bug
[X] My issue is not caused by the Millennium interface inside Steam (Themes, Plugins & Update Panel). If it is, make a issue report in this repo.
[X] My issue is not caused by a theme

Describe the Bug

Expected Behavior
Not getting a virus popup from windows defender
Steps To Reproduce

Launch steam
get this

Operating System
Windows
Anything else?
how do i uninstall millenium? not because of the popup, just because the only thing i use it for is themes

Millennium just released a brand new version (about 10 minutes ago) so antiviruses haven't had enough time to properly authenticate Millennium, they usually go away after a few days. Windows Defender just sees a file you downloaded that isn't commonly downloaded (because it was just created) and sees it as potentially malicious.
You can uninstall with this guide: https://docs.steambrew.app/users/uninstalling
If you're concerned about the safety of your pc, you can join the discord server and ask around about your concerns :)
Happy patching!

You can also check virus total, its just a false positive:
https://www.virustotal.com/gui/file/cdea5e9969ca9927ee31c15f33e04312b766f1ba3179ed95d776fcb3f7fe8d2c?nocache=1


Support building against bundled libzmq for cython extensions?

I'd like to build a cython extension against pyzmq. In order to do this portably, I'd like to be able to link against the bundled libzmq when it is present. I think that all I need to do this is to have a portable way to get the libzmq library path (like get_includes())  and to keep the bundled libzmq's header files somewhere sensible in pyzmq when it is bundled, adding that path to the output of get_includes.
I'm happy to try implementing this, but I want to make sure there is not a good reason to avoid this.
This comes up for me a lot because pip defaults to installing bundled copies of libzmq, which then conflict with a system built libzmq, but that might be a harder issue to get around. (On my own machine, I can always do pip install --no-use-wheel pyzmq, but I don't always have control over how pyzmq is installed.

You will have to be a little careful, because you are not guaranteed that libzmq is bundled, and pyzmq doesn't bundle libzmq's headers, and further shouldn't if libzmq is not bundled as an extension itself. So you will always have to cover both cases of bundled libzmq and non-bundled libzmq.
But in principle, adding the zmq headers when libzmq is bundled as an extension is sensible.

I will ensure that pyzmq won't bundle libzmq headers when libzmq is not bundled, and I'd prefer to reliably have unbundled versions, so I still think that should be the default. But I'll add the ability to include the libzmq headers when bundled.

Fixed by #905


EnvironmentTunnels have "undefined" in tooltip

Connect to an SSH remote with a LocalForward entry in the config
Resolver returns an entry in environmentTunnels
Find that entry in the remote explorer, hover
Tooltip is like 5901 to localhost:5901 - undefined

Yuck. Thanks for reporting it!


Use containerd cli for CI

Related to https://github.com/containerd/containerd/pull/2131.
Add cmd/test-containerd which is a customized version of containerd for CI.
We added warning:
// NOTE(random-liu): This customized containerd is only for CI test
// of cri plugin. NEVER USE IT IN PRODUCTION!

With this PR, we can get rid of the script to automatically update containerd vendor.

@mikebrow Added the suffix in the version string. And also added warning log.


Add data about package quality

How could we display information about npm package quality ?
We could use data from:

http://packagequality.com/
https://npms.io/

API response:
http://packagequality.com/package/redux
{
	"_id": "54e2cd1ff22796cf67c857ee",
	"name": "redux",
	"source": "npm",
	"created": "2015-03-10T09:57:27+00:00",
	"lastUpdated": "2016-06-17T19:00:04+00:00",
	"nextUpdate": "2015-03-25T10:30:26+00:00",
	"timesUpdated": 2,
	"repoTotalIssues": [0.9994165694282381, 1],
	"repoOpenIssues": [1, 1],
	"repoLongOpenIssues": [1, 1],
	"versions": [0.9811320754716981, 1],
	"downloads": [0.9999997440054393, 1],
	"quality": 0.9961096777810752,
	"description": "Predictable state container for JavaScript apps"
}

https://api.npms.io/search?from=0&size=1&term=redux
{
	"total": 1646,
	"results": [{
		"date": "2016-04-24T12:20:45.108Z",
		"evaluation": {
			"maintenance": {
				"commitsFrequency": 1,
				"recentCommits": 1,
				"issuesDistribution": 0.9995429837308245,
				"openIssues": 1
			},
			"popularity": {
				"downloadsCount":<PHONE_NUMBER>333334,
				"dependentsCount": 548,
				"communityInterest": 22813,
				"downloadsAcceleration": 5542.798896499239
			},
			"quality": {
				"tests": 0.85,
				"branding": 0.85,
				"dependenciesHealth": 1,
				"carefulness": 0.9
			}
		},
		"score": {
			"final": 0.9068110186654271,
			"detail": {
				"popularity": 0.7340079979736842,
				"maintenance": 0.9999999999935061,
				"quality": 0.9996940645897018
			}
		},
		"keywords": ["redux", "reducer", "state", "predictable", "functional", "immutable", "hot", "live", "replay", "flux", "elm"],
		"name": "redux",
		"description": "Predictable state container for JavaScript apps",
		"publisher": {
			"email"<EMAIL_ADDRESS>			"username": "gaearon"
		},
		"links": {
			"bugs": "https://github.com/reactjs/redux/issues",
			"npm": "https://www.npmjs.com/package/redux",
			"repository": "https://github.com/reactjs/redux",
			"homepage": "http://redux.js.org"
		},
		"version": "3.5.2",
		"searchScore": 0.14198856
	}]
}

First iteration
I added data from packagequality.com.
By clicking on the "quality" tab, projects are sorted by packagequality.com score.

Second iteration
Later, I used data from npm.io.
The guys from npms.io were really nice and made some adjusments for me (see this issue: https://github.com/npms-io/npms-api/issues/35)


Game not working

i get to the world creation and I get close to the end of the world creation.  It gets stuck on awaiting character spawn

Can you please post or upload the log file? Which world generator are you using?

City module.  Will post log file when found

I can replicate. Chunk generating threads all die so there's never enough world for the player to spawn.
Seems present in both latest release and latest dev build. I knew @msteiger was working on Cities and may have left the old version alone for last release. Next release I imagine we'll have the new Cities in which doesn't have this issue (I just flew around in the Cities world for quite a while no problem)
So I'd call this a known issue that should be fixed with the pending changes in the dev branch for Cities. @msteiger since it isn't working currently should we just merge Cities develop to master? It works well as I'm testing with the TeraMath update

This should be fixed now. The preview screen is not yet supported, but this will come soon. Close?

Oh, yes, I thought there was another item to follow-up on after the Cities fix, but had just been searching the forum. Thanks for pinging this :-)


Handle negative indices in a simpler way

Maybe change
https://github.com/Continvvm/continuum/blob/a80971f21bcef265192056e775e41633e8161ac2/continuum/scenarios/base.py#L193-L194
to
index = index % total_len

will make it simpler and more efficient when index is very small?

Hello,
Sorry for the delay, I was busy with a submission.
You're right that's much better, I've been dumb.
Thanks for the PR!


Defining data source and mapping columns for a working yesoreyeram-infinity-datasource in a Scenes App

Hi Scenes experts,
Thank you for Scenes, an amazing leap forward in Grafana.
We are not achieving a working  definition of the Infinity json datasource in a queryRunner despite trying various approaches.  We are using a provisioned version of  the fine yesoreyeram-infinity-datasource at https://github.com/grafana/grafana-infinity-datasource
We use as shown below under the following header and it works fine in the base GF instance in which the App itself is running.
--------- standard dashboard working version --------------
A key to solving  this issue seems to be to transform the data by mapping columns. We are not sure how to implement this with the typescript in the Scenes definition.
We have the app-with-scenes example  from https://github.com/grafana/grafana-plugin-examples working as an App in in our existingWorld
In the helloWorldScene.tsx, we try
uid: 'DS_INFINITY'  This is a template variable mapping that works in the standard dashboard in the same GF instance.
The attempt to use ${} instead of bare notation failed with "cannot find datasource".
Our attempt below under this header FAILS:
====== helloWorldScene.tsx  from https://github.com/grafana/grafana-plugin-examples ============
ERROR: Cannot read properties of undefined (reading 'method')
The snippets we have found below do not help us to achieve a working version or give enough understanding to adapt our existing code.
From https://github.com/grafana/scenes/blob/main/docusaurus/docs/advanced-data.tsx
...................................................................
export function getAdvancedDataScene() {
  const data = new SceneQueryRunner({
    datasource: {
      type: 'prometheus',
      uid: 'gdev-prometheus',
    },
    queries: [
      {
        refId: 'A',
        expr: 'rate(prometheus_http_requests_total{}[5m])',
      },
    ],
  });

...................................................................
We did not find enough in the this doc from the scenes repo either:
docusaurus/docs/advanced-data.md
So far the debug logs are not giving me much insight either, so it feels a bit like working blind.
It would be great to receive some proposals to attempt and I will report back here and to the Infinity plugin community here:
https://github.com/grafana/grafana-infinity-datasource/discussions/1098
Best wishes
E
--------- standard dashbord working version --------------
"pluginVersion": "11.4.0",
 "targets": [
   {
     "columns": [
       {
         "selector": "sessionlabel",
         "text": "Session",
         "type": "string"
       },
       {
         "selector": "value",
         "text": "Result",
         "type": "number"
       },
       {
         "selector": "time",
         "text": "Time",
         "type": "timestamp_epoch"
       },
       {
         "selector": "analyte",
         "text": "Test",
         "type": "string"
       },
       {
         "selector": "measurementSeries",
         "text": "Session Date",
         "type": "string"
       }
     ],
     "datasource": {
       "type": "yesoreyeram-infinity-datasource",
       "uid": "${DS_INFINITY}"
     },
     "filters": [],
     "format": "timeseries",
     "global_query_id": "",
     "hide": false,
     "refId": "A",
     "root_selector": "",
     "source": "url",
     "type": "json",
     "url": "/v2/getkrafttimeshift/${__user.email}/${subject}/${lookupsetrecordskraftsingle}/${lookupsetrecordskraftsingle}/aintKetone",
     "url_options": {
       "data": "",
       "method": "GET"
     }
   }

====== helloWorldScene.tsx  from https://github.com/grafana/grafana-plugin-examples ============
NOT WORKING
FAILS WITH Cannot read properties of undefined (reading 'method')
 const datasource: DataSourceRef = {
    type: 'yesoreyeram-infinity-datasource',
    uid: 'DS_INFINITY'
  };

  queryRunner3 = new SceneQueryRunner({
    datasource,
    queries: [
      {
        refId: 'C',
        url: 'https://api.foobar.org/etcetcetc',
        method: 'GET',
        source: 'url',
        type: 'json',
        root_selector: '$',
        columns: [
          {
            selector: "analyte",
            text: "analyte",
            type: "string"
          },
          {
            selector: "value",
            text: "value",
            type: "number"
          },
          {
            selector: "time",
            text: "time",
            timestampFormat: "2006-01-02T15:04:05Z07:00",
            type: "timestamp_epoch"
          }
        ],
      },
    ],
  });

[SOLVED]
We now have a working solution for timeseries as shown below
If it is interesting to others to save the time to get fast to this solution, then happy to contribute a howto in this scenes or infinity projects 🤗
The other graph types or members of PanelBuilders work too.
In file /var/lib/grafana/plugins/myorg-scenes-app/src/pages/HelloWorld/helloWorldScene.tsx
const queryRunner = new SceneQueryRunner({
    datasource: {
    type: 'yesoreyeram-infinity-datasource',
    uid: 'yesoreyeram-infinity-datasource',
    },
    queries: [
      {
        refId: 'C',
        url: '/simplescene',
        columns: [
          {
           selector: "time",
           text: "time",
           type: "time"
         },
         {
           selector: "series1",
           text: "series1",
           type: "number"
         },
         {
           selector: "series2",
           text: "series2",
            type: "number"
         },
        ]
      },
    ],
  });

And here in the SceneFlexLayout:
        new SceneFlexItem({
          width: '60%',
          height: 400,
          body: PanelBuilders.timeseries()
            .setTitle('Infinity timeseries now working 🤸')
            .setData(queryRunner2)
            .build(),
        }),

This produces a timeseries graph of a dataframe like this
  {
    "time":<PHONE_NUMBER>000,
    "series1": 10,
    "series2": 15
  },
  {
    "time":<PHONE_NUMBER>000,
    "series1": 20,
    "series2": 25
  }
]

Here is the provisioned datasource yaml:
apiVersion: 1

datasources:
- id: 3
  name: "infinity"
  uid: "yesoreyeram-infinity-datasource"
  type: "yesoreyeram-infinity-datasource"
  url: "http://foobar...."
  basicAuth: false
  basicAuthUser: ""
  isDefault: false
  jsonData:
    global_queries: []
  readOnly: true
  apiVersion: ""
  overwrite: true

Thank you again to the Grafana Gods, and for Scenes!


Images not found

Two of the images in the script are returning 404 now.
http://img70.imageshack.us/img70/4853/cutesquirrels27rn9.jpg
http://img70.imageshack.us/img70/9615/cutesquirrels15ac7.jpg

Fixed by #1687.

@parkr #1687 wasn't merged though

It is proposed now in non-deprecated repo: https://github.com/hubot-scripts/hubot-shipit/pull/14


new version doesn't work with Seurat tutorial in User Docs

While attempting an integration with scvi I followed the user guide on the website to install and setup. Install with reticulate as per the user guide worked without error following these command:
library(reticulate)
py_install("scanpy")
py_install("scvi")

sc <- import('scanpy', convert = FALSE)
scvi <- import('scvi', convert = FALSE)

however, running any of the scvi commands / setting settings or passing data resulted in the following error (note: the sc function to convert to AnnData worked fine): AttributeError: module 'scvi' has no attribute 'data'
a quick google search found an issue someone had very similar in python and the solution was that data structure had changed of the package and they needed to install the newest version. I am not sure if this error I am seeing is because I need a NEW version of scVI OR if the user docs is for the old one.
I will keep troubleshooting and try to figure out which one, this is a clean new install of scVI so there shouldnt be any old code hiding anywhere that I know of , but conda repo could be pulling old package..  but if someone already knows the answer and wants to give me an idea of where to look, i would be very grateful.
Error in py_get_attr_impl(x, name, silent) :
  AttributeError: module 'scvi' has no attribute 'data'

scvi$settings$progress_bar_style = 'tqdm'

scvi$data$setup_anndata(adata)

Error in py_get_attr_impl(x, name, silent) :
  AttributeError: module 'scvi' has no attribute 'data'

Versions:

VERSION
scvi-0.1.6 ?? according to bioconda
Python 3.8.5

How did you install scvi via conda?
Note that the name is scvi-tools for installation. See the following link:
https://anaconda.org/bioconda/scvi-tools
https://docs.scvi-tools.org/en/stable/installation.html

thank you for the links, I used py_install("scvi") inside R , following this --> https://docs.scvi-tools.org/en/stable/user_guide/notebooks/scvi_in_R.html
will check out your links and try scvi-tools , if this is just cause I installed a different package with a similar name  , I am going to face palm so hard

hmmm I seem to be in a loop, switching conda environments doesnt seem to impact available libraries and conda_remove() isnt uninstalling anything to try and load a different version. I will keep playing with it and hopefully make some progress and will update the thread if I figure anything out.

Thank you for the links, I used py_install("scvi") inside R , following this --> https://docs.scvi-tools.org/en/stable/user_guide/notebooks/scvi_in_R.html

Our package used to be called scvi, but now it's called scvi-tools. Even with the name change it's still import scvi, so here you should py_install("scvi-tools")

Okay can confirm this approach removed my initial issues.  Reticulate did not like (or was pulling from) the wrong sources for scvi-tools for its native conda installation of libraries... using conda directly outside of reticulate solved the problem as you suggested.
I am having a new issue with pyTorch not seeing my 2080 Ti GPU that I would love to throw cores for training on, but that is a separate config issue not related to scVI.  I am closing the issue and thank you for taking the time to help me out.
I might write a step by step blog post on how to configure this for first timers moving forward with R.  At First I was thinking of doing a fork and PR to update the documentation but honestly, the documentation is fine in terms of what to do, its just separated up in a way that until you told me the order I didn't get by default from the traditional reticulate approach... so changing documentation is probably overkill for a small user base of R users with reticulate experience and previous zero scVI use...

Probably some cuda driver issue with your gpu.
If you write a blog post we can link to it from our installation page. Feel free to make a PR when/if you do it.


Bring server up to 2.2.10

If you want to get 2.2.10  version, I can refresh docker hub for images building with all released npm.

Cool. I'm just using a locally built one from that pr right now,.
On Thu, May 26, 2016, 11:30 AM Andrew Chen<EMAIL_ADDRESS>wrote:

If you want to get 2.2.10 version, I can refresh docker hub for images
building with all released npm.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/yongjhih/docker-parse-server/pull/47#issuecomment-221905961

This project is as launcher for parse-server npm, so i think it's unnecessary merged with ParsePlaftfrom/parse-server. But I'll consider about merging if resolved conflicts.

@yongjhih this needs to be closed :)


Use status spec  when creating new instances

Summary

Makes use of the .status field in the instance when creating it for the first time.
Now, after the first reconciliation the controller will use different logic based on the presence of a status condition.
This change is required to wait until everything is ready before we can provision the connection secret in the instance's namespace (future PR).

Checklist

[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] PR contains the label area:operator
[x] Link this PR to related issues
[x] I have not made any changes in the charts/ directory.

For reviewers:
You can run make local-install install-samples && kubectl wait --for condition=Ready postgresqlstandalone/my-instance to provision the instance and wait for it.


Deselecting the last geographic region does not work

When the last selected region for a geographic search is deselected, only the display on the map is updated, but the (now deselected) region is still POSTed to the find_societies API.

I'm not having this issue on my machine or on the dplace test site.

The test site is having this issue:

Select one region,
hit search,
then go back to refine the search,
deselect the region
hit search
=> the results for the (now deselected) region show still up.

@xrotwang Oh, I see now. I'll check this out, thanks.

Fixed in #229


Fetch contributions to sections during compilation

For reference the rationale is becuase we can get easily rate limited by Github...
Failed invoking get with status : 403, body :
{"message":"API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)","documentation_url":"https://developer.github.com/v3/#rate-limiting"}


statically link for arm/v7 32bit

Did we test and make sure this doesn't impact any of the other platforms/builds? Intel Linux? Darwin? Windows? etc.?


Port remainder of patients + lines scss

I rule and have completed some work on Case Manager that's ready for review!
Port the two remaining spare files over.
This involves starting to port over some of the bootstrap config over. To handle stuff that requires reffing bootstrap classes, we throw our bootstrappy config into a single file and import that into the main pack, and ref it in modules as well. (I think webpack is smart enough to just process everything down cleanly.)
This pull request makes the following changes:

port patient.scss contents into webpack
port lines.scss contents into webpack
establish a strategy for reffing webpack components

no view changes
It relates to the following issue #s:

Bumps #1854

For reviewer:

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

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

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

Codecov Report

Merging #2202 (df7fe1f) into main (53ff376) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2202   +/-   ##
=======================================
  Coverage   91.13%   91.13%
=======================================
  Files          84       84
  Lines        1986     1986
=======================================
  Hits         1810     1810
  Misses        176      176

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 53ff376...df7fe1f. Read the comment docs.


Cleaning up content files

Hi!
A nice new feature would be cleaning up content files with a Panel button, as describe on the Kirby Cookbook.
Or do you think it's possible with Custom jobs ?
Thanks!

new job added cleanContent. will be available in next release which comes later today.


add ClassInjector.getResourceAsStream().

#2350

Current coverage is 39.80% (diff: 37.41%)

Merging #2351 into master will decrease coverage by 0.03%

@@             master      #2351   diff @@
==========================================
  Files          1613       1613
  Lines         55936      56024    +88
  Methods       10999      11014    +15
  Messages          0          0
  Branches       7719       7733    +14
==========================================
+ Hits          22288      22302    +14
- Misses        33648      33722    +74
  Partials          0          0

Powered by Codecov. Last update 37bab5f...399598e


Column Reference BPA rule does not work if measure is in common folder

Hello,
I have a question regarding a BPA Rule Column references should be fully qualified which has its definition like:
DependsOn.Any(Key.ObjectType = "Column" and Value.Any(not fullyQualified))
Let's say I have a measure SelectedReport with a definition SelectedReport := SelectedValue([Report UID]);
[Report UID] corresponds to a 'Reports'[ReportUID] column.
If I put this measure into a table Reports, than the rule works fine. It returns an error that measure does not use fully qualified reference to the column. However, if I put this measure into a common measure table, e.g.  _Measures, than this rule stop working. It seems that is  somehow related with Dependency feature in Tabular Editor.
I would like to know your recommendations regarding to this challenge. Did you find similar behaviour? How did you solve it?

It seems like there is some confusion between measure and column references. From your description, it sounds like [Report UID] is a measure. If so, then SelectedValue([Report UID]) should not be fully qualified, as there are no column references in this expression.
Could you show an example of a DAX expression in which you are referencing a column with no table qualifier, in which the rule doesn't work as expected?

Hello Daniel,
my mistake when describing the issue. The fake measure will not work but the logic remains. I am providing you a PBIX file and bim model with the dummy data example. It is packed in a zip file.
In the attached model there are two measures with the simple definition:  Wrong := SUMX(Reports, [UID])
[UID] is referencing a column in table Reports. So, there should be a fully qualified reference Reports[UID] in the measure.
When I put the measure directly into the Reports table, then the rule works fine. However, when I put the measure into dummy table called Measure Table, then the rule does not work. And I assume it is related to a dependency feature in Tabular Editor.
Thank you for your time!
Qualified Columns _ Error.zip

I see, thanks for clarifying.
You’ve encountered one of the limitations of the built-in dependency detection of TE2. It’s not actually performing a full semantic analysis of the DAX expression (there is no open source DAX parser available anywhere to my knowledge). So what TE2 does instead, is just to perform a simple tokenization of the code. If a token has square brackets, but no qualifier, it is assumed to be a column on the same table as the object owning the DAX expression. If no column of that name exists, it is assumed to be a measure. This explains why the rule works while your measure is defined on the 'Reports' table, and why the rule stops working when you move the measure to a different table.
I unfortunately don’t have a good solution for you. TE3 has a built-in DAX parser that can handle this, but it’s proprietary so I can’t include it in TE2. Perhaps the issue could be solved by TE2 using the DISCOVER_CALC_DEPENDENCIES DMV but this would only work while connected to Power BI / Analysis Services.


Return country code in return value

Related to https://github.com/hellodigit/digit/issues/19358 (returning country code will allow us to correctly extract base phone number and send verification SMS successfully)

Can you add some tests?

@sudothinker sure thing, looks like I'll need to update the existing tests to expect a third element in returned array as well (doesn't seem like the tests are automatically run anywhere but I can at least verify everything passes locally)

@sudothinker Updated existing tests to expect the country code returned as last element in array, all tests now passing: https://github.com/hellodigit/node-phone/pull/12/commits/b4f9e67d1f602d3670f9cd3891fdf241c2704322


Should we use  javax.annotation.Nullable and javax.annotation.concurrent.Immutable

Part of JSR-305.
These annotations are part of Java 8, but could be used before by including this dependency
<groupId>com.google.code.findbugs</groupId>
<artifactId>jsr305</artifactId>

Actually the annotations are not part of Java 8, which is good because you can include them from the jsr305 jar and it will work the same everywhere. What changes in Java 8 is that you have type annotations, so you can write things like List<@Nullable String>. Conversely, you can't write that if you want to compile on earlier versions, though you can still write @Nullable String frob(@Nullable String s).
@Nullable in particular can provide a useful service to client code if that code is using FindBugs or another similar tool.

Stale issue, closing out.


Add linter

This pr adds a linter, standard to the typescript generator and ts-standard to the compiler, in an effort to improving the code quality and resilience.
Standard is a well known linter in the js community and other than fixing the code style, it also contains a series of gotchas to avoid falling in "brittle code" situations.
The linter will be executed as CI step as well.
This pr is the first step towards refactoring of the compiler.

@Mpdreamz I don't think so, as run-validations.sh is just for running the validator infrastructure, while the linter is useful only if you are working on the compiler/generator.

@Mpdreamz I don't think so, as run-validations.sh is just for running the validator infrastructure, while the linter is useful only if you are working on the compiler/generator.


[NSE-1108] allow to use different cases in column names

What changes were proposed in this pull request?
This patch fixes the different cases in column names by converting to lower case
How was this patch tested?
pass jenkins

note: this fix relies on lower case table schema, a better solution would be switching to the boundreference:
https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/BoundAttribute.scala#L69


feat: Theme toggling button

A new button which toggles between dark and light theme
I have put the name of the button as button-53 because there was already button-52 on pr list.

Thanks for suggesting a theme toggle button! I've implemented two variations:

A button-based toggle: https://github.com/origin-space/originui/blob/main/components/buttons/button-53.tsx
A checkbox-based toggle: https://github.com/origin-space/originui/blob/main/components/checkboxes/checkbox-19.tsx

Both components are designed to be framework-agnostic and not strictly bound to Next.js, providing flexibility for different use cases.


Genbank fasta template

Coverage remained the same at 93.91% when pulling 2798e767a857d2379c57a98554351ff14fab4839 on genban_fasta_template into 46df58348cf5286f7f80461dcad88d5464dd6fac on master.

Coverage increased (+0.05%) to 93.95% when pulling ef31df58322b0c16dbdb541862558ed50a8b28e4 on genban_fasta_template into 46df58348cf5286f7f80461dcad88d5464dd6fac on master.

Coverage decreased (-0.12%) to 93.76% when pulling 0112617c2fba7756685127d4af4df5b76b61ea5a on genban_fasta_template into 03a7da18c21b6b72298a1757bb8b4ed8b3fb8167 on master.

Coverage decreased (-0.12%) to 93.76% when pulling 0112617c2fba7756685127d4af4df5b76b61ea5a on genban_fasta_template into 03a7da18c21b6b72298a1757bb8b4ed8b3fb8167 on master.

Coverage decreased (-0.12%) to 93.76% when pulling 0112617c2fba7756685127d4af4df5b76b61ea5a on genban_fasta_template into 03a7da18c21b6b72298a1757bb8b4ed8b3fb8167 on master.


Adjoint

Adjoint, transpose and conj specialisations.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 23 (73.91%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 87.387%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/qutip_tensorflow/core/data/tftensor.py
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
97

Relevant Lines:
111

💛  - Coveralls

I reverted the changes to tftensor.py. I will wait for inner to be merged as I will have another conflict with test_mathematics and __init__.


use Swift 5.0-RELEASE

Motivation:
Now that Swift 5 is released, we should use the release version.
Modifications:
switch to Swift 5.0-RELEASE.
Result:
newer, shinier

late LGTM


Different results from serpapi (Google Trends) versus Google Trends site

I'm having an issue (which is causing a serious headache / project issue for us) where the results from serpapi are different versus those returned from Google, when querying the website directly.
Below is a simple code snippet to reproduce:
from serpapi import GoogleSearch
import pandas as pd

PARAMS = {'engine': 'google_trends',
          'data_type' : 'RELATED_QUERIES',
          'q' : "health insurance",
          'geo': "IE",
          'date' : "2022-01-01 2022-12-31",
          'hl' : 'en-GB',
          'csv' : True,
          'api_key' : '[Key]'}

search = GoogleSearch(PARAMS)
results = search.get_dict()

rel = results['related_queries']['top']
df = pd.DataFrame(rel)

df[["query", "value"]]

df.to_csv("serpapi_results.csv")

Below is an image with the difference in results (and also attached are the files)

This is an image of the URL in Google Trends:

serpapi_results.csv
relatedQueries_google_results.csv
Can you let me know if this is a known issue or if I've made some mistake in my API call?
Thanks,
Ronan

Thank you @robrady, I'm closing this as it's documented in our Public Roadmap:
https://github.com/serpapi/public-roadmap/issues/1069


Add ArtifactAccessRequest to this list

Preview :

@kfollis

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close
The changes in this PR have been moved to private repo PR #10693


Update README.md

Added codecov badge

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.27%. Comparing base (b0e6e42) to head (9a98fa6).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   97.27%   97.27%
=======================================
  Files           7        7
  Lines         220      220
=======================================
  Hits          214      214
  Misses          6        6

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


extension  not instaliing

Type: Bug
i installing any extension then pop of my window error while installing extension .please check the log for more details and try manully add extension then reppeat pop my window eperm:operation not permitted,mkdir c/user/shiva/vscode/extension plz solve this issue
VS Code version: Code 1.81.1 (6c3e3dba23e8fadc360aed75ce363ba185c49794, 2023-08-09T22:22:42.175Z)
OS version: Windows_NT x64 10.0.22621
Modes:

Thank you for opening the issue. Sorry to hear you are experiencing the installation issue. It appears to be specific to VS Code editor and tools. It is best to reach out VS Code team at https://github.com/microsoft/vscode/issues.
-VS Marketplace Team

Plz solve this problem and sorry for my English is poor
On Mon, 4 Sep, 2023, 6:25 pm madhurivadaligithub, @.***>
wrote:

Closed #763 https://github.com/microsoft/vsmarketplace/issues/763 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vsmarketplace/issues/763#event-10273799043,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCKXXEH4MXY55FMXWHWWDWTXYXFTHANCNFSM6AAAAAA4KGVDDU
.
You are receiving this because you authored the thread.Message ID:
@.***>

What is the solution?
On Mon, 4 Sep, 2023, 6:40 pm asdghklnvvj, @.***> wrote:

Plz solve this problem and sorry for my English is poor
On Mon, 4 Sep, 2023, 6:25 pm madhurivadaligithub, @.***>
wrote:

Closed #763 https://github.com/microsoft/vsmarketplace/issues/763 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vsmarketplace/issues/763#event-10273799043,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/BCKXXEH4MXY55FMXWHWWDWTXYXFTHANCNFSM6AAAAAA4KGVDDU>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vsmarketplace/issues/763#issuecomment-1705249126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCK3BSTCKJAEYNUVSEKQZW3XYXHLFANCNFSM6AAAAAA4KGVDDU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


S3 bucket signed url is not working

I am using s3 bucket signed url in place of the input path. Its showing 'ffmpeg exited with code 1 no such file or directory'

@ocramot Did you ever find a workaround for this issue? I've just hit this bug but with a different URL (still with an ampersand).

@snobu sorry, I really can't help you. I had a look at our old code and it just seems that we get the signed url using  Google Cloud Storage APIs and ffmpeg seems to like it. I don't remember if and how we worked around it.

Thanks for getting back to me. Fixed my problem. It was never the URL but the Authorization header i had to send along. I simply needed to escape the whitespace for some reason Authorization:\ Bearer \value.


Add HDFS health check

@BenWhitehead , @jsancio please review!

This was also tested in my EA cluster.

I think this is much better than the curl command. :+1:

Supersedes previous PR #89

LGTM.


Request for update

Hello friend, I hope I won't bother you.
I am a Chinese. When I came into contact with this software for the first time, it successfully surprised me.
It really makes the game TLD more fun by at least 80%. But when I learned that you may not update this software again, it is really a pity, I believe many people think the same as me.
So if possible, we sincerely hope that you can slightly modify this software, just to make it read the archive of the current version of the TLD.
It's a really, really good mod and it's a shame it just disappeared.  Maybe you can start crowdfunding online, I believe there will be many people supporting you.
Finally, again a sincere request that you update this software, or just make it read the TLD's archives.
I hope this passage of mine didn't make you feel uncomfortable, and if it did, I sincerely apologize. Because I really need it too much, finally I wish you a happy life!

Hi! Nice to hear you've enjoyed the editor. There's 2 people who have offered to help me in the development so it's likely we'll get something usable ready at some point. I can't really promise any timeline though. Also, one of the biggest reason for not wanting to maintain this project anymore is that since TLD is still in development it means that it will break again in the future and then I need to figure out what has changed.

Finally, again a sincere request that you update this software, or just make it read the TLD's archives.

Out of curiosity what features would you want it to have to "read archives"? At the moment this save editor can already read the new saves (including inventory), but it still needs some work to be usable for normal users.

I hope this passage of mine didn't make you feel uncomfortable, and if it did, I sincerely apologize. Because I really need it too much, finally I wish you a happy life!

No worries it's all good! I wish you a happy life as well! Let's hope we'll see this project released next year :blush:

Hello friends, it is a great honor to receive your reply.  At the same time, I am also sorry, because the major I studied cannot provide you with useful help, and I may not be able to provide you with some useful suggestions.
The editor version I used before is TLD Save Editor 2.18.1, this version is almost perfect in my opinion, it contains almost all the functions that players like me need, the most important thing is that it has  Chinese, this is so surprising to me.
It's just a pity that after the TDL game update, TLD Save Editor 2.18.1 cannot read the game save file, which means that the game save file cannot be edited.  So, if it is not very complicated, I think it is enough to make a slight modification on the basis of the source code of TLD Save Editor 2.18.1, so that it can read the game archive of the new version of TLD and edit it.  As for the items added in the new version of the game, it may not need to be added to the editor.
The above are some of my thoughts on this software, of course, in any case, I and other players will appreciate everything you and all developers have done, because you have made this game twice as fun.
Finally, I wish you good health and all the best!

I'll close this issue to cleanup open issues


Added routes/service relationships

@simon3z Please review

@erinboyd please fix all issues in this PR (travis failure, style warnings) before it gets reviewed.

@erinboyd if this is the PR that we should consider, can you close #3109?
Have you found the travis error?

@simon3z fixed all the travis errors. I don't want to close 3109 until we are happy with this, so the comments don't get lost. Unless you disagree ;)

@erinboyd have you fixed the travis errors locally? I still see the CI failed here.

@erinboyd it seems that there's still a reference to service_name somewhere in save_inventory_helper.rb. At least travis is failing on that.

@erinboyd actually I am not 100% sure that it's in save_inventory_helper.rb, it could be in the parser as well. Try to search it a little and ping me if you can't find it. Thanks!

@simon3z I think I found the issue in openshift.rb...fixing it now

@simon3z I can do it just by name, rather than service_name, but I am not sure that is unique.
Let me know.

@simon3z failing on nonrelated test:
Failed examples:
rspec ./spec/migrations/20140519211930_add_user_current_group_to_miq_groups_spec.rb:18 # AddUserCurrentGroupToMiqGroups#up add current_group to miq_groups if miq_groups not empty
rspec ./spec/migrations/20140519211930_add_user_current_group_to_miq_groups_spec.rb:28 # AddUserCurrentGroupToMiqGroups#up skip if current_group is present in miq_groups
rspec ./spec/migrations/20140519211930_add_user_current_group_to_miq_groups_spec.rb:37 # AddUserCurrentGroupToMiqGroups#up user's current_group is orphaned
rspec ./spec/migrations/20140519211930_add_user_current_group_to_miq_groups_spec.rb:9 # AddUserCurrentGroupToMiqGroups#up add current_group to miq_groups if miq_groups empty
rspec ./spec/migrations/20140519211930_add_user_current_group_to_miq_groups_spec.rb:50 # AddUserCurrentGroupToMiqGroups#up current group is valid but not in miq_groups

@simon3z I added   belongs_to :container_service
has_many :labels, -> { where(:section => "labels") }, :class_name => "CustomAttribute", :as => :resource to /app/models/container_route.rb otherwise the spec test would fail.

@simon3z please review

@erinboyd thanks, there's one comment to address.

:+1: Thanks @erinboyd this is good to be merged @chessbyte


Prevent duplicated rule exception when importing rules and filters

Fixes #85.
Implementation

Skip adding imported rules and rule filters to rule registry.
Split tests/rules/example.py into tests/rules/rules.py and tests/rules/rule_filters.py.
Test Duplicated exception is not raised by adding an imported skim_schemaX rule filter.

Note
Both rules and rule filters can be imported now, but the existing tests cover only rule filters.

I will give @matthieucan the final say as he worked on the rule registry


Adding missing Marshaler array functions.

GetMany uses Marshaler to get various marshal array functions, but they were only set for structs.  Setting it for the other types.
Fixes #629

    internal static readonly Func<T[], object> CreateMarshalerArray;

thanks for catching all these - this stuff is a pain

Refers to: src/WinRT.Runtime/Marshalers.cs:438 in bcbe8d2. [](commit_id = bcbe8d2ad42bd0d1ca510fa6f88af85f0bdd242f, deletion_comment = False)


Mikrotik Identity can contains spaces

If Identity (router name) on mikrotik contain space scrapli raise TimeOut exception
SSH on the router:
[ravenich@ravenich] > /sys identity set name="Test Router"
[ravenich@Test Router] >
Scrapli exception:
scrapli.exceptions.ScrapliTimeout: Timed out determining if session is authenticated
I'm not a regexp expert, but i think a simple space in "comms_prompt_pattern" fix the situation.
I'm just begin my work with python and simply don't know how to add a PR.
Also mikrotik have a safe mode, it enabled by Ctrl-X, but that string matches with regexp:
[ravenich@Test Router] >
[Safe Mode taken]
[ravenich@Test Router] <SAFE>

Hey @ravenich thanks for opening this!
Sounds like you're probably bang on with the space. I can probably get that fixed up this weekend and/or tagging @AlexLardschneider here too if he is interested/has time to take a peak.
What is this safe mode thing all about? I have effectively zero mikrotik experience so if you could ELI5 that'd be dope :)
Thanks!
Carl

@ravenich sorry for delay. added a space to the mikrotik prompt pattern and added a test to make sure we dont regress on that. Given that I dont have a mikrotik router to test im just going to assume it is good to go :) If you could test the develop branch and let us know if its working that would be great.
Regarding the safe mode -- should this be matched or are you saying we would not want to match it? If we dont want to we can just add a negative lookahead like we do for the Arista prompt: pattern=r"^((?!config)[a-z0-9.\-@()/: ]){1,63}#\s?$"
Let us know what ya think!
Carl

@carlmontanari Thanks, now it works.
`with Scrapli(**my_device) as conn:
print(conn.get_prompt())
[ravenich@ravenich router] >`
*If you want you can try mikrotik CHR. It is a virtual image for hypervisors, have speed limitations on free version but no feature limits.
About safe mode - i think this pattern is fine. User can use some kind of expression to verify safe mode by the get_prompt  function.

@ravenich sounds good! I'll go ahead and close this then. Will hopefully merge to master and make a new release soonish as I know there were a few minor mixups that @AlexLardschneider just got pushed in recently as well. Thanks for opening this!
Carl


Change Gina model in ba_security2 to use hologram pushcart model.

See https://github.com/SamVanheer/halflife-unified-sdk/issues/431
The MapUpgrader has been updated to change holo.mdl to holo_cart.mdl in ba_security2.

This needs to apply the custom hull size that player.mdl and holo.mdl get in SetCustomHullForGenericMonstersUpgrade.

I've merged this PR and applied the hull size changes. Testing shows Gina appears as expected.


ASP Net Core vs WPF performance

Hello,
I upgraded a WPF application to be a ASP NET CORE 2 application. I am running  ASP NET CORE 2 application using IIS + Kestrel.   ASP NET CORE 2 application has application insights and authentication middleware with empty handler, while WPF dont.
Excluding all the time it takes to process the business logic in the application, ASP application takes approximatly 150ms more than WPF application. That is, time taken at IIS + kestrel + middlewares.
The processing time I calculated is a average of over 1000 calls.
Does this delay sounds right ? Or am I missing any configuration in IIS or Kestrel to get it better ?

The IIS + Kestrel scenario does have some known latency overhead. In the short term the best alternatives are to use Kestrel or HttpSys directly. In the future we're working on a direct IIS option that will address this.

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


update url of yaml in README

What type of PR is this?
/kind documentation
What this PR does / why we need it:
update url of yaml in README
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

By the way, the image now has been published to swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-dashboard:latest, the karmada-dashboard.yaml should be updated as well.

Hi @zhouqunjie-cs Please sign your commits, and here is a guide for you:
https://github.com/karmada-io/dashboard/pull/9/checks?check_run_id=4772225924
Please cc me after that, thanks.

Hi @RainbowMango  commits have been signed and the image has also been updated.


Excluding Pilotfish NTMThreadPool from Executor instr and fix NPE

Codecov Report

Merging #626 into master will decrease coverage by 0.92%.
The diff coverage is 36.84%.

@@             Coverage Diff              @@
##             master     #626      +/-   ##
============================================
- Coverage     63.34%   62.41%   -0.93%
  Complexity       68       68
============================================
  Files           189      189
  Lines          7428     7112     -316
  Branches        861      821      -40
============================================
- Hits           4705     4439     -266
+ Misses         2444     2407      -37
+ Partials        279      266      -13

Impacted Files
Coverage Δ
Complexity Δ

.../apm/agent/concurrent/ExecutorInstrumentation.java
45.83% <100%> (+1.15%)
0 <0> (ø)
:arrow_down:

...ic/apm/agent/servlet/ServletTransactionHelper.java
76.38% <33.33%> (+0.16%)
0 <0> (ø)
:arrow_down:

...va/co/elastic/apm/agent/impl/transaction/Span.java
66.66% <0%> (-10.46%)
0% <0%> (ø)

...apm/agent/impl/transaction/TraceContextHolder.java
53.19% <0%> (-9.31%)
0% <0%> (ø)

...lastic/apm/agent/impl/transaction/Transaction.java
88.23% <0%> (-5.89%)
0% <0%> (ø)

...c/apm/agent/impl/async/SpanInScopeBaseWrapper.java
58.33% <0%> (-4.63%)
0% <0%> (ø)

...va/co/elastic/apm/agent/impl/ElasticApmTracer.java
80.4% <0%> (-1.1%)
0% <0%> (ø)

.../v6_4/ElasticsearchClientAsyncInstrumentation.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

.../v5_6/ElasticsearchClientAsyncInstrumentation.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...m/agent/impl/async/SpanInScopeRunnableWrapper.java
100% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 5 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 f6781c3...71d9397. Read the comment docs.


Uncaught (in promise) TypeError: Cannot read property 'biography' of null

This happens when using sqlite instead of postgres.

See https://github.com/FredericHeem/starhackit/pull/250


There is no 'X' button on the search box when running search with the option 'All' checked

Storage Explorer Version: 1.29.0-dev
Build Number: 20230217.4
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.2.1 (Apple M1 Pro)
Architecture: ia32/x64
How Found: From running test cases
Regression From: Previous release (1.28.0)
Steps to Reproduce

Launch storage explorer -> Open the search filter dropdown.
Check the option 'All'.
Type a string to 'Search for resources' box -> Run search.
Check whether there is a 'X' button on the search box.

Expected Experience
There is a 'X' button on the search box.

Actual Experience
There is no 'X' button on the search box.

@JasonYeMSFT feel free to assign back to me if this is an issue with the new tree view instead of with the new 'All' option.

Verified this issue on the build 20230219.3. Fixed. So, we are going to close it.


Remove autogeneration of version number from pxf/build.gradle

NOT READY TO MERGE. The purpose of this PR is to solicit feedback on our approach. We'll need to do more work to get the hardcoded version number out of this code.
This enables us to use IntelliJ's code navigation and refactoring
features. Previously, IntelliJ would navigate to / refactor the
generated files rather than the source files since the generated files
were the ones that actually got compiled.
We also upgraded Gradle to 3.0 because we suspected IntelliJ might not have good support for older Gradle versions. We may want to revert the change to the Gradle version if it wasn't the culprit.
Co-authored-by: Lav Jain<EMAIL_ADDRESS>Co-authored-by: Ben Christel<EMAIL_ADDRESS>

Until recently the version number was hardcoded in the code. generatedSources task does seem to be more pain than benefit. I'm fine with merging this change even in the mean time until we figure out an alternate means of fetching the version programatically

Closing this for now, as these changes are out of date and the issue hasn't been a pain point for us recently.


add local_users testing to openbmctool test code

Add automated testing of local_users functions    disableall,enableall,queryenabled of openbmctool.py.

any update on this one there @ssombar

Adding a test of openbmctool's  local_users queryenabled function.  The other local_users tests (namely disableall and enableall) will require LDAP.   Those will be tested under a separate issue where we will test those two functions as part of testing openbmctool's LDAP functions.

Oh yeah.. let's not code it then, it will make thing complicated in the tool..

https://gerrit.openbmc-project.xyz/#/c/openbmc/openbmc-test-automation/+/16580