[FLINK-7378][core]Create a fix size (non rebalancing) buffer pool type for the floating buffers

What is the purpose of the change
Currently the number of network buffers in LocalBufferPool for SingleInputGate is limited by a * <number of channels> + b, where a is the number of exclusive buffers for each channel and b is the number of floating buffers shared by all channels.
Considering the credit-based flow control feature, we want to create a fix size buffer pool used to manage the floating buffers for SingleInputGate. And the exclusive buffers are assigned to InputChannel directly.
Brief change log

Add the isCreditBased property in ResultPartitionType
Create a fix size buffer pool for `SingleInputGate
Assign the exclusive buffers for input channels directly
Provide the methods of requesting and recycling batch of segments in NetworkBufferPool

Verifying this change
This change is already covered by existing tests, such as NetworkBufferPoolTest and LocalBufferPoolTest.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

@NicoK , Thank you for reviews!
Considering the difference between FixedBufferPool and LocalBufferPool, I think you are right. I also think of this question when implement this new type. Maybe I misunderstood the stephan's meaning from google doc.
I proposed the new BufferPoolListener for interaction easily between RemoteInputChannel and BufferPool, because RemoteInputChannel may request more floating buffers and wants to be notified more than once, and the boolean return can decide the behavior easily. And it can replace the current EventListener provided in BufferProvider interface. In order not to affect the current process, I did not replace that in this PR.
I referred to the existing code format when implementation and found many cases keep the space after . Thank you for letting me know the new rule. I may modify it next week based on your comments.

@NicoK , I have submitted the updates:

Create the fix size LocalBufferPool for floating buffers
Assign the exclusive buffers for InputChannel directly
The proposed BufferPoolListener  will be included in next PR

Reviewed 9 of 13 files at r2.
Review status: all files reviewed at latest revision, 7 unresolved discussions, some commit checks failed.

a discussion (no related file):
Depending on how you build on this in the other PRs, what do you think about using a fixed-size LocalBufferPool (or a customized sub-class) per RemoteInputChannel instead? This would solve potential issues with recycling and would also be a lot less code. Additionally, you will gain the buffer availability listener feature so that you will be notified when the buffer is released (which may be deep inside other code with no access to the RemoteInputChannel anymore.
FYI: This change of commits in the PR actually would qualify for a separate PR

flink-runtime/src/main/java/org/apache/flink/runtime/io/network/NetworkEnvironment.java, line 216 at r2 (raw file):

					if (gate.getConsumedPartitionType().isCreditBased()) {
						// Create a fix size buffer pool for floating buffers and assign exclusive buffers to input channels directly
						bufferPool = networkBufferPool.createBufferPool(extraNetworkBuffersPerGate, extraNetworkBuffersPerGate);

we still need to call gate.setBufferPool(bufferPool) in order for the gate to be aware (this call is common to both paths of the if)

flink-runtime/src/main/java/org/apache/flink/runtime/io/network/buffer/NetworkBufferPool.java, line 164 at r2 (raw file):

			}

			redistributeBuffers();

now here, you may need to add the try-catch releasing any already added segments back (see my comments in SingleInputGate

flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/ResultPartitionType.java, line 38 at r2 (raw file):

	 * no checkpoint barriers.
	 */
	PIPELINED_BOUNDED(true, true, true, false);

Does it make sense, to already add an PIPELINE_CREDIT_BASED(true, true, true, true)? I guess, credit-based can be considered bounded as well

flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/ResultPartitionType.java, line 82 at r2 (raw file):

		return isBounded;
	}

please add a (simple) javadoc similar to the isBounded()method

flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/SingleInputGate.java, line 315 at r2 (raw file):

			return segments;
		} catch (Throwable t) {
			if (segments != null && segments.size() > 0) {

Unfortunately, the cleanup will not work as documented - if networkBufferPool.requestMemorySegments(networkBuffersPerChannel); throws an exception, segments will be null. In order to handle all cases, e.g. successfully requested some and afterwards an exception was thrown, you need to handle this inside NetworkBufferPool#requestMemorySegments().
I guess, after changing this, this method will not be required anymore

flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/SingleInputGate.java, line 319 at r2 (raw file):

			}

			if (t instanceof IOException) {

please use ExceptionUtils#rethrowIOException for this pattern

Comments from Reviewable

@NicoK , thank you for giving this discussion and comments!
Actually I proposed the same way of using fixed-size LocalBufferPool for managing exclusive buffers for per RemoteInputChannel with stephan before implementation. I attached the dialogue below:

Maybe I did not catch stephan's meaning from the above dialogue and took the current way to implement.  I also agree with the way you mentioned and the fixed-size buffer pool for RemoteInputChannel can be submitted in an separate PR.

For ResultPartitionType comment, I expand to add the 'isCreditBased' field temporarily in order not to affect the current process. My initial idea is to remove this field after the whole feature is verified to enable. If you approve this as a formal way, I will add the new mode PIPELINE_CREDIT_BASED(true, true, true, true) and javadoc for it.
For gate.setBufferPool(bufferPool), it is my carelessness to not add it.
For releasing segments when exception, it is actually better to process inside the method of NetworkBufferPool.
I will submit the modifications later today.

Hi @zhijiangW,
regarding the buffer pool implementation, I was just curious about why it was done that way. But it is fine to keep the logic in RemoteInputChannel if you make sure, that a recycler puts these buffers right (back) into the buffer queue (I guess, that's in one of the follow-up PRs). This way, we avoid an additional intermediate component (and the need to interact with it). To conclude, on a second thought, it is fine as it is.
The thing with ResultPartitionType is that without an (intermediate) way to set isCreditBased to true, we are not really able to test this code path on higher levels such as the NetworkEnvironment (or maybe I'll see that in the follow-up PRs as well).
Speaking of tests...I understand that with the switch to credit-based flow control, some parts will be covered by existing tests, but we also change the behaviour at some points and the current tests are already a bit sparse. Can you also add tests for

the NetworkEnvironment changes (into NetworkEnvironmentTest),
NetworkBufferPool#requestMemorySegments, NetworkBufferPool#recycleMemorySegments (into NetworkBufferPoolTest which currently is a bit sparse though)
the changes in SingleInputGate (into SingleInputGateTest)

Yes, this way also has some advantages, and recycling these exclusive buffers would be covered in next PR with some additional tests.
I will consider your suggestions to supplement some tests in this PR and submit the modifications based on all the above comments.

@NicoK , I have submitted the updates based on the above comments. :(

@NicoK , thank you for so detail and helpful comments!
I will be on team outing tomorrow and come back on Sunday. I would consider you concerns carefully and may submit the updates next week.

@NicoK , I have submitted the updates based on last comments.
Your concerns of interaction between requesting memory segments and creating buffer pool is really necessary. I also noticed that after reviewed the process.

@NicoK , thank you for always helpful reviews!
I am very busy improving the runtime for our singles day these days. I will submit the updates for this PR later today. And I also plan to update the next PR which is based on this PR during weekends.

@NicoK , I have submitted the updates.

@NicoK , sorry for the typos. I have submitted the updates.

@NicoK , I missed that message of verifyAllBuffersReturned() issue before.
I have submitted the modifications of it. :)

merging.


Unable to build UE4 docker image

Output of the ue4-docker info command:
ue4-docker version:         0.0.105 (latest available version is 0.0.105)
Operating system:           Linux (Ubuntu 20.04.5 LTS, 5.15.0-67-generic)
Docker daemon version:      23.0.1
NVIDIA Docker supported:    Yes
Maximum image size:         No limit detected
Available disk space:       332.37 GiB
Total system memory:        125.43 GiB physical, 30.52 GiB virtual
CPU:                        12 physical, 12 logical (x86_64)

Additional details:

Are you accessing the network through a proxy server? No

I created issue #301 on Friday, last week, because I was unable to build Unreal Engine using the command ue4-docker build 4.24.3 . You seem to to have fixed that issue and introduced another one in the process. Now, I'm getting the following error:
Traceback (most recent call last):
  File "/home/ram/.local/bin/ue4-docker", line 8, in <module>
    sys.exit(main())
  File "/home/ram/.local/lib/python3.8/site-packages/ue4docker/main.py", line 109, in main
    COMMANDS[command]["function"]()
  File "/home/ram/.local/lib/python3.8/site-packages/ue4docker/build.py", line 52, in build
    config = BuildConfiguration(parser, sys.argv[1:], logger)
  File "/home/ram/.local/lib/python3.8/site-packages/ue4docker/infrastructure/BuildConfiguration.py", line 410, in __init__
    raise Exception()
Exception

Aaargh!

@slonopotamus Can you please push the changes to PyPI, thanks.
Also, I'm actually trying to build the UE editor for Carla Simulator. According the instructions on their page, the command to build is ue4-docker build 4.24.3 --no-engine --no-minimal but when I run this I get an error saying no-engine is not a valid argument. Was support for this removed? What does it actually do?

Sorry for inconvenience, looks like I undertested some of my recent refactorings. Pushed in 0.0.106.

Was support for this removed?

Yep, just drop it. There was ue4-engine image that nobody used: https://github.com/adamrehn/ue4-docker/pull/273
WRT docs - we're in the process of migrating docs to this repo (mainly because docs get outdated). You can take a look at a more-up-to-date version here: https://ue4-docker.docs.adamrehn.com/#ue4-docker-build

Was support for this removed?

Yep, just drop it. There was ue4-engine image that nobody used: #273
WRT docs - we're in the process of migrating docs to this repo (mainly because docs get outdated). You can take a look at a more-up-to-date version here: https://ue4-docker.docs.adamrehn.com/#ue4-docker-build

Thanks for the information. Will create another issue if I run into any other problem :sweat_smile:


Create Splunk accounts for dev team

We should have access, but routing rules are preventing us from reaching the Splunk instance.


Split all the new UI changes into options

"Should" just be a case of .click() etc. etc

Done and/or now ambiguous


correct procedure in case of more conditioning/control

Hi, thank you for the code, it works flawlessly.
I have one question though... I am trying to add more control to the network so it can be controlled with two images instead of one. I am doing it in a way that I add 3 new channels to the control image and then dstack two images going to the training. Is it the right way or do you have any better approach in mind?

should be correct. but training some complicated tasks will need relative large batchsize. so perhaps consider rent some a100 from clouds

thanks for the answer. And if I want to add more like class conditions instead of whole image, how should I proceed? For example ... I want to generate image based on the rotation of object and control is an angle of the rotation.

draw bounding box

i will try it, thanks

i will try it, thanks

Hi, did the training by stack work for you? Thanks

I would like to enhance the image's control further. However, when I replace the text CLIP encoder with a small fully connected network to embed the time sequence, represented by (T, value) pairs, the quality of the generated image significantly deteriorates. Do you have any advice on how to restore the full power of Controlnet for better results?

Hi, thank you for the code, it works flawlessly. I have one question though... I am trying to add more control to the network so it can be controlled with two images instead of one. I am doing it in a way that I add 3 new channels to the control image and then dstack two images going to the training. Is it the right way or do you have any better approach in mind?

I also want to train on two control images, could you share where and how you add the code?


Syncing with AppGateway Configured Through Terraform

I create App Gateway with Terraform.  Initially, it has default rules, because it requires at least one rule for most of the resource types.  The subsequent rules, backends, etc... that are added to App Gateway come from AGIC as applications are deployed.
What I found, which is expected, when I redeploy my Terraform project, it wipes out all of the AGIC controlled rules.   It also never syncs them back, unless, which is the only way I found to make it sync, is to delete a release (e.g. helm delete), where they almost instantly get added back, minus the one of course that I've deleted.  So my workaround if I need to redeploy the Terraform project is to delete an app release and redeploy.
I'm sure others have run into this.  I guess I would like to see Terraform handle these differently, not sure how, I know I've seen at least one request asking to be able to separate them out of the main azurerm_application_gateway resource block.  But this isn't the Terraform project, so I can't ask for that!
Has anyone run into this and come up with an elegant way of handling it?  Really the only thing that I can think of is somehow having AGIC update my Terraform project, which is not elegant, and likely more destructive.  I am also looking at pulling out the App Gateway from my core Terraform project so I can re-run the Terraform project without worrying about AppGateway, but there are some dependencies between Vnet, Roles, AKS, that I'd have to get right and then sequence running them on any initial create, which I suppose isn't terrible, but in the end it doesn't solve the problem, just separates it.
What I'd like to see, at least, is re-syncing the AGIC rules back into App Gateway if they don't exist, without forcing it by deleting a release.  This is still destructive.   Outside of this, it's more of a Terraform thing, which I will throw some comments in the repo also.

@davejhahn we added the ability to reconcile (beta) in the last release. Can you give that shot and provide feedback ?

@akshaysngupta Absolutely!  Will give it a try and let you know.

@akshaysngupta I was able to test--blew away an application gateway, added back, no rules, then it refreshed everything.  So, that is a great capability.
I would add though, it is great for non-production, when it comes to production, it's still a huge problem but I think that is something that the Terraform resource will need to handle though (and don't blame it on the AGIC at all).
I did create an issue on this https://github.com/terraform-providers/terraform-provider-azurerm/issues/7935 for reference if anyone else would care to upvote it.
Thanks!

Hi @davejhahn ,
This is documented in terraform docs but is not explicit... The problem is that the real state of the agw doesn't match with the desired state in the terraform files so terraform will "fix" the differences.
When you deploy agw with terraform, you have to add some ignores in the terraform files in order to skip the lifecycle manage. You should add this in you azurerm_application_gateway:
resource "azurerm_application_gateway" "agw" {
  name                = ...
  ...

  lifecycle {
    ignore_changes = [
      tags,
      ssl_certificate,
      trusted_root_certificate,
      frontend_port,
      backend_address_pool,
      backend_http_settings,
      http_listener,
      url_path_map,
      request_routing_rule,
      probe,
      redirect_configuration,
      ssl_policy,
    ]
  }
}

With this ignores, terraform will skip consolidation of the differences in subresources during the terraform apply

@JorTurFer I will try it out, I think I misunderstood how the ignore_changes works...based on what you are presenting, all blocks listed will be ignored for updates entirely, which is exactly what I want.

@davejhahn
When you add a ignore to any item, all changes over that item will be ignored after creation. I use the ignores that I pasted in my deployments with agic. In the same way for example, if your cluster has autoscale for the nodes, you should add the properly ignore to the node count

@JorTurFer I was able to get it working wonderfully, thanks for your help, appreciate it.


command to register a topic

The bundle does a lot, including creating/subscribing topic by itself.
It only recognizes topics it registered itself and only registers topics (insert into DB) it confirmed subscriptions for, and only confirms ... ...
For people doing most of this from different channels (different infrastructure/automation solution), this is not only unneeded but also blocking.
So let's provide a simple command that insert a topic which would already exist in AWS inside the database and returns after having listed registered topic.
Note: provide Topic setters would allow manual topic updating.

Hello @drzraf, first of all, thank you for your commit.
When I created the bundle the first time, it was my precise intention to not allow a developer to use external topics with the bundle: the idea was to provide a complete solution that manages all by itself and provides all the tools required.
Can you explain, please, your use case? What is the flow you follow creating topics? Why do you need to register external ones?

Topic creation are at the border between Symfony application configuration and infrastructure provisioning. I'm of those who rather consider the second option where ansible modules are best fit for the task.


Error _reactDnd.DragDropContext thrown after importing

I have
"devDependencies": {
"@babel/preset-react": "^7.0.0",
"axios": "^0.19",
"cross-env": "^7.0.2",
"laravel-mix": "^5.0.4",
"lodash": "^4.17.13",
"react": "^16.13.1",
"react-dom": "^16.13.1",
"resolve-url-loader": "^3.1.0"
},
"dependencies": {
"@babel/preset-env": "^7.9.6",
"react-dnd": "^10.0.2",
"react-dnd-html5-backend": "^10.0.2",
"react-router-dom": "^5.1.2",
"react-router-guards": "^1.0.2",
"react-select": "^3.1.0",
"react-spinners": "^0.8.1",
"react-tag-input": "^6.4.2"
}
but import { WithContext as ReactTags } from 'react-tag-input';
gives error below
app.js:48562 Uncaught TypeError: (0 , _reactDnd.DragDropContext) is not a function
at Object../node_modules/react-tag-input/dist-modules/components/ReactTags.js (app.js:48562)

Same Problem

Same issue!

This library requires:
react-dnd: ^5.0.0 react-dnd-html5-backend: ^3.0.2
You are using react-dnd: 10.0.2 so it's not compatible with react-tags. You can downgrade react-dnd to 5.0.0 and it will work.

Same issue here! I want to use some cool features on updated react-dnd (context hooks) but I can't because this package throws errors when react-dnd is updated 😞

Same here 😢

same here

Same here, It's nice if dependencies packages are compatible with the latest versions.

I am not sure when I will be able to pick this up.  Please feel free to open PR for the same.

I had the same issue so I changed my package.json as :
"react": "^16.13.1",
"react-dnd": "^5.0.0",
"react-dnd-html5-backend": "^3.0.2",
"react-dom": "^16.13.1",

And that fixed the issue


feat: export payout splits to csv

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3722

PR #3719

PR #3720

PR #3723

PR #3724

PR #3725

PR #3727   👈

PR #3728

This comment was auto-generated by Graphite.

@wraeth-eth started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@wraeth-eth merged this pull request with Graphite.


Added badges to the readme

Coverage remained the same at 100.0% when pulling 68c9fabcbec5e0427fb21f1e4417a7cf761e9b3d on badges into e500a95bec61200fcb41309d55693eda59e0b5da on master.

Coverage remained the same at 100.0% when pulling 68c9fabcbec5e0427fb21f1e4417a7cf761e9b3d on badges into e500a95bec61200fcb41309d55693eda59e0b5da on master.


Add MIT LICENSE file to the repository.

About issue-
Add MIT LICENSE file to the repository and add its link to the repo README.
Refer to the documentation for help.

@Pranav016  Hi, I came across this and would like to take this up, I'm new to Open Source so any help/guidance is more than welcome. Please assign this to me and also let me know of any relevant information. Thanks

@Pranav016  Hi, I came across this and would like to take this up, I'm new to Open Source so any help/guidance is more than welcome. Please assign this to me and also let me know of any relevant information. Thanks

Created in PR #20

Created in PR #20

I would like to work on it

I would like to work on it

I would like to work on this as a mentee under JWOC.

I would like to work on this as a mentee under JWOC.


toolbar element gap missing?

focus is cut off?

The server sends layout: "FlowLayout,1,1,1,1,1,1,0,0,0,3,false" so the gap between the elements is only one px, the focus-rect ~3px thats probably why it looks like that. Will the server change that or should I try to increase the gap with messing with the layout

No the layout is fixed... In this case the style should be changed. It would be good enough if focus is "over" the other component or increase the gap - but use styling for this - otherwise it's not possible to create themes


Example programs hang looking for Tau Camera hardware when here is an additional, non-responsive serial port on the computer

Python Library versions

Run pip list | grep TauLidar in the command-line and paste the output. Include some details on your system as well

TauLidarCamera 0.0.4
TauLidarCommon 0.0.2
Running on a Mac
Camera Info

If possible, paste the output of camera.info() from Python (after camera has been successfully opened)

    model:      4.0
    firmware:   3.3
    uid:        450225
    resolution: 160x60
    port:       /dev/tty.usbmodem00000000001A1

Describe the bug
Example programs hang on Looking for connected Tau LiDAR Camera hardware ... when there is an additional, non-responsive serial port on the computer
Steps to Reproduce

There are several serial ports on my Mac:

The Tau Camera is on /dev/tty.usbmodem00000000001A1
Not really sure where /dev/tty.usbmodem104NTZNH11332 comes from or what it is

Run distance.py example program
Hangs on Looking for connected Tau LiDAR Camera hardware ... message indefinitely

What is expected?
Expecting the example program to:

scan the serial ports
establish connection to the connected Tau camera
initialize the camera
start sending depth frames to the computer

What is actually happening?
The program hangs on the step where it's Looking for connected Tau LiDAR Camera hardware
Cannot be killed with ctrl+c, can only be stopped by killing the process
Any additional comments?
The program seems to get stuck in the ports = Camera.scan() call
I've updated the example programs to have a command-line option to specify the serial port (see commit a86c52f4a193aa918f50561c7080237fa2079047).
The example programs work fine when the serial port is specified and the scan function is avoided, so the Camera itself is not the issue:

@vsemi could you please look into this?
Perhaps we can add a timeout when attempting to connect to each port in the scan function?

I'm not able reproduce this issue, but by reading the code, that could be problem of waiting for (problematic) serial port response, as normally there is just very limited number of available serial ports to be looped, the hanging only possible happens in a specific loop while trying communicating to a (problematic) serial port which somehow does not respond and be more weird that it hanging in pyserial library, and we do have one mini-second timeout setting for pyserial, not sure what happens it failed to give up in such case after one mini-second. I do not have a Mac machine, not sure if it happens on Mac only, but I think it might happens only when there is a fake serial port or problematic serial port. Suggested user to use device manager of the computer to investigate that problem serial port, to fix or disable it.

@vsemi It's not a feasible solution to have users disable serial ports on their machines. Most serial ports have some other purpose on the machine.
For example /dev/cu.Bluetooth-Incoming-Port on Macs has something to do with Bluetooth. Disabling it is not an option since that would likely cause issues with how bluetooth work.
On top of that, our goal is to make the Tau Camera easy to use out of the box and this issue is definitely fixable in our code.

I took a deeper dive and found where the issue lies. Here is a trace of what happens with the /dev/cu.Bluetooth-Incoming-Port serial port on my machine:

In the scan function we loop through all available serial ports and call getIdentification on each of them
In getIdentification, we _sendCommandWithoutData to the serial port, which then uses _sendCommand
In _sendCommand, the _write function works properly, but it's the _read function where the problem happens
In _read, the issue happens in the while loop:

_ser.read works properly - hence no timeout - but what's returned is a b''
This turns into a 0-length byte-array and gets added to array, which doesn't change the length of array
Then, when the while loop re-evaluates the loop condition, len(array) is always 0 resulting in an infinite loop

Same thing happens with the /dev/tty.usbmodem104NTZNH11332 serial port on my machine.
Can you please look into updating the Communication._read function to have some sort of infinite loop protection?

@greenbreakfast line 156 in Communication._read, limited the maximum attempts to be 100, is not an infinite loop:
if count > 100 : break
Could you help to debug how it does not beak the loop over there?

@vsemi looks like the max attempt limit was taken out in your fix for #14 (commit 3e0036d31f49cde28154f0d2ff96468c27307bbc)
There's a count variable but nothing to break the loop: https://github.com/OnionIoT/tau-lidar-camera/blob/master/TauLidarCamera/communication.py#L136

@greenbreakfast my bad, how comes I made such a mistake, in my vsemi repo where I investigate the issue I still have that 2 lines, must be a mistake when I copy and paste that block over. Added back in branch issue_17. Let's see if it can break the loop on your Mac.

@vsemi thanks, the update in issue_17 fixes the bug I was seeing. I'll merge into master and make a new release.
How was the 100 loop limit chosen? If it's an arbitrary number, how can we be sure that's enough loop attempts?

@greenbreakfast  1) read timeout setting is one mini-second, 100 attempts means 100 mini-seconds, the slowest frame-rate (10 fps) expected in very low edge devices. 1) tests on different platforms, includes edge device (Omega), observed normally it can be done in attempts.

@vsemi ok, sounds reasonable. Thanks again!
Fix released in v0.0.5


Replace all uses of std::round with std::rint and std::lrint

The round function doesn't actually round numbers correctly. Switch to rint/lrint for IEEE-754 compliance.

What's the point of lrint when you can just cast the output of rint?


Add OnChaos mode to pod-delete experiment

I have a k8sprobe which mode is "OnChaos" in pod-delete experiment, it's not able to run. Seems the OnChaos mode is not ready  on pod-delete.
Ideally, the "OnChaos" mode should be enabled for all generic experiments.

cc: @ispeakc0de

OnChaos Mode is available across all generic experiments from 1.13.1.


Fix future-incompatibilities warning

This PR fixes a warning about future-incompatibility.
Details of the warning:

warning: reference to packed field is unaligned
--> /home/felfert/.cargo/registry/src/github.com-1ecc6299db9ec823/uhid-virt-0.0.5/src/codec.rs:187:35
|
187 |                     let payload = &event.u.start;
|                                   ^^^^^^^^^^^^^^
|
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
= note: for more information, see issue #82523 https://github.com/rust-lang/rust/issues/82523
= note: fields of packed structs are not properly aligned, and creating a misaligned reference is undefined behavior (even if that reference is never dereferenced)
= help: copy the field contents to a local variable, or replace the reference with a raw pointer and use read_unaligned/write_unaligned (loads and stores via *p must be properly aligned even when using raw pointers)
= note: #[allow(unaligned_references)] on by default

Duplicate of #4

oops. However it begs the question, if this repo is abandoned. After all, #4 is from last year and the other PRs are even older and none of them has been merged.
Also: The Cargo.toml in this repo has version 0.0.4, while on crates.io the crate has 0.0.5. Strange.

Yeah, not really sure whats going on. Mentioned that in the other PR and a linked commit. Weird stuff.

Ha! just compared the repo against the source from crates.io. Turns out, that the Cargo.toml was modified by cargo during upload, so obviously @flukejones just forgot to bump the version before uploading, See for yourself:
diff -uw /home/felfert/.cargo/git/checkouts/uhid-virt-913c9a01cae84a47/9cb5f6f/Cargo.toml /home/felfert/.cargo/registry/src/github.com-1ecc6299db9ec823/uhid-virt-0.0.5/Cargo.toml
--- /home/felfert/.cargo/git/checkouts/uhid-virt-913c9a01cae84a47/9cb5f6f/Cargo.toml	2023-04-25 23:47:05.610510969 +0200
+++ /home/felfert/.cargo/registry/src/github.com-1ecc6299db9ec823/uhid-virt-0.0.5/Cargo.toml	2020-06-30 10:39:10.000000000 +0200
@@ -1,24 +1,36 @@
+# THIS FILE IS AUTOMATICALLY GENERATED BY CARGO
+#
+# When uploading crates to the registry Cargo will automatically
+# "normalize" Cargo.toml files for maximal compatibility
+# with all versions of Cargo and also rewrite `path` dependencies
+# to registry (e.g., crates.io) dependencies
+#
+# If you believe there's an error in this file please file an
+# issue against the rust-lang/cargo repository. If you're
+# editing this file be aware that the upstream Cargo.toml
+# will likely look very different (and much more reasonable)
+
 [package]
+edition = "2018"
 name = "uhid-virt"
-version = "0.0.4"
+version = "0.0.5"
 authors = ["Luke Jones<EMAIL_ADDRESS>"Sameer Puri<EMAIL_ADDRESS>"Daniel Stiner<EMAIL_ADDRESS>-edition = "2018"
+exclude = [".travis.yml", "example/"]
 description = "Interface to Linux UHID (user-space HID transport drivers)"
-repository = "https://github.com/flukejones/uhid-virt"
-license = "MIT OR Apache-2.0"
 readme = "README.md"
 keywords = ["hid", "uhid", "linux", "userspace", "transport"]
 categories = ["os", "os::unix-apis"]
-exclude = [
-    ".travis.yml",
-    "example/"
-]
+license = "MIT OR Apache-2.0"
+repository = "https://github.com/flukejones/uhid-virt"

 [lib]
-doc = true
 test = true
+doc = true
+[dependencies.enumflags2]
+version = "^0.6.4"
+
+[dependencies.libc]
+version = "^0.2.0"

-[dependencies]
-uhidrs-sys = "1.0.0"
-enumflags2 = "^0.6.4"
-libc = "^0.2.0"
+[dependencies.uhidrs-sys]
+version = "^1.0.1"

One of the reasons I never used github is because it never had good email notifications. Microsoft must have changed something because this is the first time I've ever received emails. Hell I didn't even know I had outstanding PR's on this.
I've merged #4 so I'll close this.


Debugger view variables without pausing application.

This might be a feature I just can't find.
Is it possible to do or possible to add. I'd like to run the debugger so I can view the variable values etc. However I'd like for it not to stop at each breakpoint.
If that makes any sense.

Can you give a specific example of what you want to do? I'm not exactly sure

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add tracking tag for analytics

Create in a PR, make a deploy preview, have a look at Datadog ingress.

Tracking to introduce:
<script
    src="https://www.datadoghq-browser-agent.com/us1/v4/datadog-rum.js"
    type="text/javascript">
</script>
<script>
    window.DD_RUM && window.DD_RUM.init({
      clientToken: 'pub4c8b1a9ef4c424ef4d0ce30adaedc6d0',
      applicationId: '3937f727-75df-40e8-bed7-84fd4ef7bcf6',
      site: 'datadoghq.com',
      service: 'tbd.website',
      env: '<ENV_NAME>',
      // Specify a version number to identify the deployed version of your application in Datadog
      // version: '1.0.0',
      sessionSampleRate: 100,
      sessionReplaySampleRate: 20,
      trackUserInteractions: true,
      trackResources: true,
      trackLongTasks: true,
      defaultPrivacyLevel: 'mask-user-input',
    });

    window.DD_RUM &&
    window.DD_RUM.startSessionReplayRecording();
</script>

@tlongwell-block suggests:
We can try setting env to devtest for testing

PR in #570
Before closing this issue, open a new one after PR is submitted to make for env vars to set the env variable in static/scripts/dd-analytics.js to prod for Production environments so we capture testing and prod as different environments. We don't wanna muddy the real metrics with local dev and branch deploys :) @dayhaysoos got ideas for that? Use Netlify Env Vars to replace the devtest default value now in that file?

@ALRubinger can this issue be closed yet?

Nope, we've got to separate env vars between production and dev per comment above. Awaiting DataDog rollout from Infra ENG.


Update to Go 1.18

Codecov Report

Merging #271 (ccefc6f) into main (2630e55) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #271   +/-   ##
=======================================
  Coverage   75.55%   75.55%
=======================================
  Files          17       17
  Lines        1489     1489
=======================================
  Hits         1125     1125
  Misses        286      286
  Partials       78       78

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 2630e55...ccefc6f. Read the comment docs.

//@#Cocaiine#@\//@#LexZii`มายืนตีเขียด.,#@\//@#fOrma๋riin๊z.,#@\//@#Melo๋diied#@\//
@.***:6969#@\//@#www.giithub.com#@\//@#SOUNEDT-TRAACK-OffMelo
๋diiedffO#@\\\\\
ในวันที่ อ. 22 มี.ค. 2022 11:52 น. Xabier Larrakoetxea Gallego <
@.***> เขียนว่า:

Merged #271 https://github.com/slok/sloth/pull/271 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/slok/sloth/pull/271#event-6280602571, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYI5CT4QRM2REKAABVIRCKLVBFG2LANCNFSM5RJWMCEA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Fix Undo/Redo UI

As per UI review, move the undo/redo buttons inline.

Can you elaborate on what this means?

Is it related to the case where the app is in Landscape orientation and the screen is smaller thus hiding the undo button beneath the top view?

Ah, the UI Lab at UC 2017 recommended the undo-redo buttons be moved inline into the Status view at the top. I'm not sure I agree (hard to reach). But certainly, there's some conflict in Landscape mode that needs resolving.

Thanks for the clarification.


Confusing/wrong remarks section of SslStream.SslProtocol property

The remarks section of the property SslStream.SslProtocol is confusing or wrong.

It says that a security protocol can be passed to the constructor of the SslStream. But I see no constructor that takes such an argument.
It says that SslProtocols.Default is the default value. But the documentation of SslProtocols.Default explicitly states "Despite the name of this field, SslStream does not use it as a default except under special circumstances.".

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

ID: 33872e47-b50c-0594-b58f-b6778becac5d
Version Independent ID: 43f10566-772d-6751-860a-f94bd4d75734
Content: SslStream.SslProtocol Property (System.Net.Security)
Content Source: xml/System.Net.Security/SslStream.xml
Service: unspecified
Product: .net
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@dotnet/docs The current documentation is still the same as when I opened this issue, so the same problems remain. Please reopen.

@dotnet/docs The current documentation is still the same as when I opened this issue, so the same problems remain. Please reopen.

Reopening and moving to dotnet-api-docs.

Reopening and moving to dotnet-api-docs.

Issue moved to dotnet/dotnet-api-docs #5280 via ZenHub

Issue moved to dotnet/dotnet-api-docs #5280 via ZenHub


Aggregator

Add aggregator with test

Codecov Report

Merging #10 (474df4e) into main (541045a) will decrease coverage by 0.48%.
The diff coverage is 84.81%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   90.96%   90.47%   -0.49%
==========================================
  Files          10       11       +1
  Lines        1195     1386     +191
==========================================
+ Hits         1087     1254     +167
- Misses        108      132      +24

Impacted Files
Coverage Δ

src/hdx/scraper/configurable/aggregator.py
84.81% <84.81%> (ø)

src/hdx/scraper/runner.py
90.12% <0.00%> (+2.14%)
:arrow_up:

Continue to review full report at Codecov.

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


API documentation removed after v2?

Zombie 2.5.1 had a pretty good and comprehensive API documentation (https://github.com/assaf/zombie/tree/v2.5.1/doc), but in v3 it was since removed and not readded. Why was it removed?

3.x has different API from 2.x. The major version bump indicates they're not backward compatible.

Any chance that new API documentation can or will be produced for v3 and/or v4? It's kind of a pain to have to look in the source to find everything.

:+1: Did the major version bump really invalidate all those docs?  Sucks to lose solid documentation, especially given that basic operations are currently missing (e.g. no mention of browser.click).

No, but there's a few (non backwards compatible) API changes, that it warranted a major version bump and the old docs are "use at your own risk".

Duplicate of #799


must-gather: separate collection of rbd_vol and snap info

This commit separates the collection of rbd vol info and
rbd snap ls as the single file was unreadable and
difficult to debug.
Signed-off-by: yati1998<EMAIL_ADDRESS>

@idryomov @crombus please do review the changes again.

@idryomov please review the PR, now the output is getting collected as required.

@Madhu-1 @nixpanic

@idryomov waiting for your review in this PR

@idryomov do review the PR, Done with the changes.

@agarwal-mudit would like to get your review on this. I have addressed all the review comments and is now in ready to merge state.


Issue while Implementing this code with UIScrollview

Hi,
I have used your code in my application & it works pretty good. Only it shows issue while i am using the UIScrollView,
My Code structure is like below:

There is one view controller which contains a scrollView. Inside that scroll view i have loaded 4 another views.
The scrolling of pages are only horizontally & all the view contains a collection view.
So when i scroll the collection view to up, the navigation bar is get hide as per requirement but while i scroll the UIScrollView horizontally the next view shows a white screen at a top (in place of navigation bar).

Thanks for using the library, but unfortunately I think this issue is out of scope.


Modify Autoware_AI Docker image and save it for not loose the content when building the container again. How?

Hi I need to modify the a docker image from Autoware_AI repository after build it. The problem is:
A) I build the image running a .sh file:
cd $WORKING_DIRECTORY/docker/generic ./run.sh -t 1.14.0
It is specifically from Autoware: https://www.svlsimulator.com/docs/system-under-test/autoware-instructions/
B) I modify the scripts contained inside the packages contained in Autoware folder
C) When I exit the container, and later enter again the modifications are not there anymore, of coure, because the image is built from Dockerfile from scratch...
To find a solution I have tried 2 different approachs:

1)To modify the container and save it as described here: https://www.scalyr.com/blog/create-docker-image/

Issue: When using other terminal, trying to add .txt file for Autoware_AI running container, to modify the container, Autoware_AI container does not appear as active (but it is). Just other container are avaialable when I try to copy a file to Autoware_AI:

Commit Changes To a Docker image: https://phoenixnap.com/kb/how-to-commit-changes-to-docker-image

Issue: Problem to connect to Autoware_AI server and run the ros packages. This problem does not happen when building the original Docker file with .sh
The complete description of my problem as well as output of terminal attempts are better described here:
https://answers.ros.org/question/376512/fork-autoware_ai-repository-and-create-docker-image/?answer=376583#post-id-376583
https://get-help.robotigniteacademy.com/t/fork-autoware-ai-repository-and-create-docker-image/9533/4
I am kind of new in forking,changing docker images. I do not understand how to fix this, find a solution for create my custom docker image and make it functional.
I am sorry to post this question here as I am not sure if it is a confirmed bug. I have read:
the communication rules: https://github.com/Autoware-AI/autoware.ai/wiki/Contributing-to-Autoware
as well as support guideline: https://github.com/Autoware-AI/autoware.ai/wiki/Support-guidelines
For this reason I did not submit the question here at first. However I still was not answered in ROS, Stack Overflow and other forums.
And I am almost sure an Autoware Developer can assist me quickly in this case with  certain facility to help me develop my custom simulation based on my launch files and scripts. And unfortunately I am not able in this moment to save the modifications in the Autoware_AI container as I do easily in Autoware_Auto project. I am really stuck here.
Thanks very much in advance!

I have solved here. In my own answer:
https://answers.ros.org/question/376512/fork-autoware_ai-repository-and-create-docker-image/
I believe it is missing to update the documentation:
https://github.com/Autoware-AI/autoware.ai/wiki/Generic-x86-Docker#run-an-autoware-docker-container
In case 2:


User Feedback Changes

Based on user feedback, add how we want to present the project in a more meaningful way.

Added this to the document in #6, please review this once we do more of the background to make sure it lines up to our goal.


Dockerfile for demo server

Wanting to test out RESTler with a simple demo server, I created a Dockerfile so this demo server can be easily run.
Since FLASK_RUN_HOST didn't work, I've added it to the run command. It will keep the default localhost binding, but allows setting it as <IP_ADDRESS> for when run in a container.

Thanks for adding this!

Just FYI, there was a CI failure which I re-ran and it passed - looks like a rare flakiness issue.  (Opened #424 to track it.)

In reply to: 987401537


[BUG] Error getting the autofilter range when loading the file, before any import or parsing

Please prefix your issue with one of the following: [BUG] [PROPOSAL] [QUESTION].
Package version, Laravel version
2.1.20. 5.3.31
Expected behaviour
When loading an Excel file, it hould reach the handle() method in the handler in order to start processing the imported data.
Actual behaviour
Beforer eaching the handle(), the library throws an error due to trying to access a "$" as one of the Excel columns when trying to get the range boundaries, as it sets the autofilter range as an absolute columns and rows range (f.ex., "$B$17:$AS$18" was my case). When the cell tries to parse those columns and rows, it interprets the $ as a column/row id and throws an error trying to access it in the static array.
Strangely enough, that range I gave as a real example isn't even right, as the Excel starts at A17, as you can see in the screenshot.
Exception stack trace

ErrorException: Undefined index: $ in C:\httpd\www\project-backoffice\vendor\phpoffice\phpexcel\Classes\PHPExcel\Cell.php:809
Stack trace:
#0 C:\httpd\www\project-backoffice\vendor\phpoffice\phpexcel\Classes\PHPExcel\Cell.php(809): Illuminate\Foundation\Bootstrap\HandleExceptions->handleError(8, 'Undefined index...', 'C:\httpd\www\de...', 809, Array)
#1 C:\httpd\www\project-backoffice\vendor\phpoffice\phpexcel\Classes\PHPExcel\Cell.php(729): PHPExcel_Cell::columnIndexFromString('$A')
#2 C:\httpd\www\project-backoffice\vendor\phpoffice\phpexcel\Classes\PHPExcel\Worksheet\AutoFilter.php(131): PHPExcel_Cell::rangeBoundaries('$A$9:$AJP$238')
#3 C:\httpd\www\project-backoffice\vendor\phpoffice\phpexcel\Classes\PHPExcel\Reader\Excel2007.php(1011): PHPExcel_Worksheet_AutoFilter->setRange('$A$9:$AJP$238')
#4 C:\httpd\www\project-backoffice\vendor\maatwebsite\excel\src\Maatwebsite\Excel\Readers\LaravelExcelReader.php(287): PHPExcel_Reader_Excel2007->load('C:\httpd\www\de...')
#5 C:\httpd\www\project-backoffice\vendor\maatwebsite\excel\src\Maatwebsite\Excel\Excel.php(119): Maatwebsite\Excel\Readers\LaravelExcelReader->load('C:\httpd\www\de...', NULL, false, NULL)
#6 C:\httpd\www\project-backoffice\vendor\maatwebsite\excel\src\Maatwebsite\Excel\Files\ExcelFile.php(93): Maatwebsite\Excel\Excel->load('C:\httpd\www\de...', NULL, NULL)
#7 C:\httpd\www\project-backoffice\vendor\maatwebsite\excel\src\Maatwebsite\Excel\Files\ExcelFile.php(31): Maatwebsite\Excel\Files\ExcelFile->loadFile()
#8 [internal function]: Maatwebsite\Excel\Files\ExcelFile->__construct(Object(Illuminate\Foundation\Application), Object(Maatwebsite\Excel\Excel))
#9 C:\httpd\www\project-backoffice\vendor\laravel\framework\src\Illuminate\Container\Container.php(794): ReflectionClass->newInstanceArgs(Array)
#10 C:\httpd\www\project-backoffice\vendor\laravel\framework\src\Illuminate\Container\Container.php(644): Illuminate\Container\Container->build('App\Http\Import...', Array)
#11 C:\httpd\www\project-backoffice\vendor\laravel\framework\src\Illuminate\Foundation\Application.php(709): Illuminate\Container\Container->make('App\Http\Import...', Array)
#12 C:\httpd\www\project-backoffice\vendor\laravel\framework\src\Illuminate\Routing\RouteDependencyResolverTrait.php(85): Illuminate\Foundation\Application->make('App\Http\Import...')

Screenshot of Excel file

Steps to reproduce the behaviour
I have tried with multiple Excel files, included from previous, already finished and working, importer implementations, to see if it got to the handle(). Some reached it without generating the error, others that I knew that previously worked don't. I haven't found yet a way to differentiate the Excels that fail from those which don't. Those failed throwed the error before even reaching my code, throwing it only during the internal load() function.

I have exactly the same problem, is there any way to prevent it from happening?


Typescript error 6.20.1: OidcProvider Property 'fetch' is missing in type

Hi,
I'm getting this error on latest version. Using React and Typescript.
Shouldn't the property 'fetch' be optional? I can't find any documentation on this 'fetch' props in any of your examples.
Property 'fetch' is missing in type '{ children: Element; configuration: { client_id: string; authority: string; redirect_uri: string; silent_redirect_uri: string; scope: string; service_worker_relative_url: string; service_worker_only: boolean; }; loadingComponent: () => Element; authenticatingErrorComponent: () => Element; authenticatingComponent: ()...' but required in type 'OidcProviderProps'.ts(2741)

Hi just publish a fix @rasa2k , very sorry for the problem !
checkou the version 6.22.7

I'm still having the same issue after update the package to 6.22.7:
{
  "name": "@axa-fr/react-oidc",
  "version": "6.22.7",
  "private": false,
  "main": "dist/index.js",
  "jsnext:main": "dist/index.js",
  "types": "dist/index.d.ts",
  "description": "OpenID Connect & OAuth authentication using react",
  "repository": {
    "type": "git",
    "url": "https://github.com/AxaGuilDEv/react-oidc.git"
  },

I don't see any change in the commit referring the issue file OidcProvider.d.ts.

Property 'fetch' is missing in type '{ children: Element; configuration: { client_id: string; redirect_uri: string; scope: string; authority: string; service_worker_relative_url: string; service_worker_only: boolean; }; }' but required in type 'OidcProviderProps'.ts(2741)
OidcProvider.d.ts(24, 5): 'fetch' is declared here.

export declare type OidcProviderProps = {
    callbackSuccessComponent?: ComponentType<any>;
    sessionLostComponent?: ComponentType<any>;
    authenticatingComponent?: ComponentType<any>;
    authenticatingErrorComponent?: ComponentType<any>;
    loadingComponent?: ComponentType<any>;
    serviceWorkerNotSupportedComponent?: ComponentType<any>;
    configurationName?: string;
    configuration?: OidcConfiguration;
    children: any;
    onSessionLost?: () => void;
    onLogoutFromAnotherTab?: () => void;
    onLogoutFromSameTab?: () => void;
    withCustomHistory?: () => CustomHistory;
    onEvent?: (configuration: string, name: string, data: any) => void;
    fetch: Fetch;
};

Fixed in v6.22.9. Thanks for quick response.

hi @dlaraf ,
All demos works with lastest exceot nextjs. Are you using nextJS?

No I'm using React v.18.2.0

Im having the same issue. Im on react 18.2.0 too:
`TS2741: Property 'getFetch' is missing in type '{ children: Element; configuration: OidcConfiguration; }' but required in type 'OidcProviderProps'.
91 |
92 |     ) : (

93 |       `

Oh, i just understood, i fix it rigth now thank you @dlaraf

6.22.16 fixed this problem for me:
https://github.com/AxaFrance/react-oidc/releases/tag/v6.22.16

Hi @NicerDicerPro , thank you very much for the feedback. Very sorry for the issue.

@guillaume-chervet we thank you for the quick response and fix of the problem. Keep up the good work!

6.22.16 fixed this problem for me too. Thanks! Have a great weekend!


[BUG] Deploy don't end

Current Behavior
When deploying  dataExtension, automation, dataExtract, script and fileTransfer the function does not end (was 20 min saying "Completed download")
https://copadonull71743980.lightning.force.com/lightning/r/copado__Result__c/a0z09000003FacKAAS/view
Expected Behavior
Deploy the things, or at least throw some error because of timeout

Commited files:

filetrasnfer, script and dataExtract -> work

dataextension and automation -> work

if all 5 in the same time -> we get this bug

pls add links to result/jobExecution

Now it seems that it is working: https://copadonull71743980.lightning.force.com/lightning/r/copado__Result__c/a0z09000003Fai0AAC/view

I guess this was an one time issue due to copado servers, I think we can close this. @JoernBerkefeld


#114 Trying to fix rspec differ issue

@Sutto My test past locally but not on the Travis server is there something I need to do?
Also do you mind reviewing this and tell me if this change helps


fix js runtime error of getNlpResult

forgot remove old code about Caps
Change-Id: I72d32c49e3d8ccb2ce301571c5f62c500e0c3fae

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added

Seems CI failed on unrelated problems, merge this is acceptable for now :)


feat(content): slayer skill

Some initial scripts related to the slayer skill, for whenever we get there...
Demo: https://streamable.com/9djx45
Things that are done:

Slayer tasks up to around june 2006
Slayer introduction chat
Generic slayer master chat
Slayer gem chat

There are still things that need to be done:

The slayer skill does not exist, it currently gives runecraft exp
The slayer master shop
The current slayer task 'table' is just a placeholder and there's only one
The task extra checks are placeholders as the quest varp/varbits do not exist yet
Npc/Obj types
Slayer npcs combat scripts
Slayer npcs spawns

This PR only works if you're running the server with BUILD_VERIFY=false as it's technically adding 'custom' content

This looks great, but I'm not 100% sure on if this is something we would necessarily ever merge in this repository. As of right now, it'll most likely be another half year, if not more, before we even leave the 225 revision, and another good chunk of time before we're at January 2005 when Slayer is introduced.
This might be more suitable to close, and re-open when the time comes with focus being on content we have in the current revision, even more so as it's a bit unclear how the Github structure will be once we consider 225 complete.


DOC: Clarify argument documentation for solve

What does this implement/fix?
There was a minor styling error for the transpose keyword.
While fixing this, I took the opportunity to make the documentation on the other keywords a bit clearer and added some lines to document the default values.
Additional information
Since this is my first PR to scipy I highly appreciate your feedback on the Docstring/Commit-Messages/PR/...

The whole transposed keyword, obviously in hindsight, was a mistake. We should have thought it a bit better. Other than that this looks good to me.

@Kaniee would you like to finish this up? If you need help resolving merge conflicts, please let me know.


add the ability to select system fonts in the interface settings

Please add the ability to select system fonts in the interface settings.

As of v0.9.0, you can select the font for all menus.  You cannot select the font for the panels, as that one is hard.

Thank you very much.


DEFERRED RENDERER

Billboards and fog isn't work well

Billboards fixed in 67c449f7dc592c74c0cef0e06e75ca0e67f8995b
Next steps: shadows and fog

Fixed using a multi-target blend state


Feature/dependabot

Bump commons-compress to 1.26.1
Bump selenium-java to 4.19.1
Bump netty from 4.1.107 to 4.1.108
Bump CXF version from 4.0.3 to 4.0.4

https://jenkins.fdp.osgp.cloud/view/SMHE-cucumber/job/gxf-smhe-cucumber-build/223/

https://jenkins.fdp.osgp.cloud/view/SMHE-cucumber/job/gxf-smhe-cucumber-build/224/

https://jenkins.fdp.osgp.cloud/view/SMHE-cucumber/job/gxf-smhe-cucumber-build/233/


added tarrant county college

@roshanpal This domain is blacklisted because the college allows to register an email address there even if you are not a student of the college. If you are the student of this college, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


Added Default Codes Generator to MP5

I like the system here but I think it would be better if we had all of dolphins code toggable with a default setting. In other words a page where u can choose what codes you want similar to the built in emulator tools and some are checked by default


Implement #63 deadline of projects funding

@jduha Merge conflict now with the home view. I merged #97 which made a lot of changes to the home page.


improve syntax highlighting

right now the way this is done is very naively implemented and is still rough around the edges. solidify this so it works properly and is more polished, then look into optimizing

as of writing this the main issues are:

handle conflicting syntax highlights - mostly with regular expressions: #include <foo/bar.h> the / somehow triggers the comment regexp mixed with the includes regexp highlighting

This is mostly fixed. Everything is in place for it to be nicely extended. The lexer is still in it's infancy but it's a good start and most of the bugs that raised this issue have been resolved.


[EKS] Encrypt Volumes: EBS and EFS

EBS
aws_ebs_encryption_by_default
aws_ebs_default_kms_key | Resources | hashicorp/aws | Terraform | Terraform Registry
Create Key like https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/v4.31.0/examples/stateful/main.tf#L375-L394
See also aws_ebs_default_kms_key | Resources | hashicorp/aws | Terraform | Terraform Registry

It is implemented in v2.0.0


Add cfg(test) to mod tests in mem_table.rs.

The tempfile dependency causes build failure due to being in dev-dependencies. Adding cfg(test) fixes the isseue.

I would prefer adding it before “mod tests”. Would you please fix it (both in starter code and reference solution)? Thanks!

Looks like it was only missing from the mod tests; in the starter directory.

Thanks!


Add more tests to pm_list.js.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!


Export LG functions to all actions

Fixes #4730
Refs https://github.com/microsoft/BotFramework-Composer/issues/2413
Refs https://github.com/microsoft/BotFramework-Composer/issues/4958
Refs https://github.com/microsoft/botbuilder-dotnet/issues/3652
Refs #5309
Description
There has been a long ask that to be able to call LG functions in all actions, like SetProperty, IfCondition, ChoiceInput, etc, basically every action. While today the workaround is to use the notions of namespace+exports to manually export LG functions to actions. It requires a manual process and it doesn't auto language fallback, where most users are expecting.
In this change, we inject all LG functions to the dialog's expression engine, thus, it can be called the same way as any other built-in functions.  And we do control the injecting scope in a fine-grain manner, that only the functions in the LG file referred by this particular dialog are visible to this dialog's actions. This will have less chance for a naming conflicts.
If in a case that the LG function name is conflict with built-in functions, we will register lg.name instead of name to be able to refer. This is an approach that aligns with how in current LG system, the naming conflicts are been handled.
Scenarios
With this change, you will be able use LG functions in any actions, for example
// in an IfCondition
{
   "$type": "Microsoft.IfCondition",
   "condition": "=MyTemplateA()"
}
// in a SetProperty
{
  "$type": "Microsoft.SetProperty",
  "value": "=MyTemplateB()"
}
// in ChoiceInput
{
  "$kind": "Microsoft.ChoiceInput",
  "choices": "=MyChoices()"
}

Note for Choices in ChoiceInput
With #5309 (which has made LG into choices), we've already exposing LG functions to choices, with a syntax like "choices":"${MyChoices()}", like the way how SendActivity is been written, which works but actually don't align with the underlying schema that declaring choices is an expression(not an LG field).
In this PR, we support a more aligned approach as "choices":"=MyChoices()", which is the standard expression syntax, and have better align with schema and Composer UX.
Note: this is NOT a breaking change, even to #5039, this is additive.
Specific Changes

Inject LG as functions to all actions
Control the injection scope
Handle the naming conflicts

Testing

Like the overall approach, just not sure you need to put in the dialog scope and it might be worth thinking about a bit in light of the namespace proposal.

Like the overall approach, just not sure you need to put in the dialog scope and it might be worth thinking about a bit in light of the namespace proposal.

Thanks, make sense, replied in context.

        var interrupted = false;

This isn't related to this PR is it?

Refers to: libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/AdaptiveDialog.cs:662 in 08d3ea2. [](commit_id = 08d3ea2714c82a17da554cd482ed4a6933d5ac23, deletion_comment = False)

Pull Request Test Coverage Report for Build 240902

0 of 0   changed or added relevant lines in 0 files are covered.
146 unchanged lines in 11 files lost coverage.
Overall coverage increased (+0.07%) to 78.992%

Files with Coverage Reduction
New Missed Lines
%

/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/Skills/SkillHttpClient.cs
1
95.0%

/libraries/Microsoft.Bot.Builder.LanguageGeneration/Analyzer.cs
3
96.1%

/libraries/Microsoft.Bot.Builder.LanguageGeneration/StaticChecker.cs
3
98.11%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/EmitEvent.cs
6
70.97%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/AdaptiveDialogBot.cs
6
80.33%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Slack/SlackHelper.cs
8
86.31%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/TemplateEngineLanguageGenerator.cs
9
72.6%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Input/OAuthInput.cs
9
52.68%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SignOutUser.cs
10
54.55%

/libraries/AdaptiveExpressions/FunctionUtils.cs
13
95.03%

Totals

Change from base Build 234300:
0.07%

Covered Lines:
18304

Relevant Lines:
23172

💛  - Coveralls

Closing this one. Related issue is closed too


Adding Codecov support

Can not be completed until #30 is solved.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@a4c41dc). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             main     #29   +/-   ##
======================================
  Coverage        ?   92.5%
======================================
  Files           ?      17
  Lines           ?     621
  Branches        ?       0
======================================
  Hits            ?     575
  Misses          ?      46
  Partials        ?       0

Flag
Coverage Δ

binding
90.8% <0.0%> (?)

python
93.1% <0.0%> (?)

vroom
∅ <0.0%> (?)

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


BLAZOR - "Find All References" Feature in VS 2019.

Just like any CS file that we can right click and select "Find All References", to give us all other files that are referencing this file, it would be very useful to get a reference list of components/files that use a specific component.
Thanks!
..Ben

Agreed. We have this for C# code (and it will work for the @functions block in VS 2019) - however we don't have this support right now for component types and parameters used from markup.
Sadly, we won't get to this in the 3.0 release.


Missing InnoUnp-1 Struct6210u.pas from Sources folder

Above file  (Struct6210u.pas) is missing from InnoUnpacker-Windows-GUI/innounp-1/sources folder. Compilation halts as Delphi complains that the file is missing.
Any ideas???
Regards

Fixed


Remove ProTxs from mempool that refer to a ProRegTx for which the collateral was spent

See commits.
I observed TXs never being removed even though the collaterals were spent already, causing getblocktemplate and generate to fail. This PR fixes this.

Turned out this needs a refactoring for another fix, please wait with reviews


[ISTHMUS] Add support for set operations

Add support for set operations UNION and MINUS in SQL -> Substrait Proto -> Substrait POJO. This would involve adding implementation of methods in SubstraitRelVisitor, RelCopyOnWriteVisitor, ProtoRelConverter, and RelProtoConverter. Queries such as these should roundtrip between SQL, Substrait Proto, and Substrait POJO:
select p_partkey as partkey from part where p_partkey > cast(100 as bigint)
union select l_partkey as partkey from lineitem where l_orderkey > cast(100 as bigint);
select p_partkey as partkey from part where p_partkey > cast(100 as bigint)
minus select l_partkey as partkey from lineitem where l_orderkey > cast(100 as bigint);

I am working on a draft PR, will submit it later today or tomorrow


switch to using styled-jsx/css

OK, let's try this again after #16 😬

Oh hey, the now bot covers it pretty well! I don't think it should fail the build though.

Nice! I was quite confused there for a bit 😅

Oh OK, I was wondering if that was expected or not. Thanks!


Styling fix for simulator buttons#1219

Fixes #1219
Describe the changes you have made in this PR -
I added width:100% attribute to '.objectPropertyAttributeChecked.btn'
Also added a margin-bottom to them so that they don't stick to each other.
Screenshots of the changes (If any) -
Before :

Now :

Coverage remained the same at 84.266% when pulling c60a77b009ce314baab1c0188986743e5cf0db64 on angad-k:styling-fix-for-simulator-buttons#1219 into 1145c3acff926c1f7f309f6a581c18bac0e3c070 on CircuitVerse:master.

Please add margins between the two buttons too.

I have increased the margin-bottom to 5px.

Please update the screenshot as well.

Please update the screenshot as well.

Done.


Guests Tenant change inconvenience

As far as I experimented, whenever you are invited to an organization as a guest, and you already have an organization, yo need to change the Tenant organization, that is not convenient, notifications are not showing as they normally do, and having to change your Tentant EVERY time you need to interact at different organizations is time consuming and pretty annoying. Is there any plan to completely integrate one Team for more than one organization? Thanks

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

ID: 048930a4-fdf2-9a90-3b47-46af66db3db4
Version Independent ID: 8d6a2c84-94e9-b17e-f5c1-91191187ce4e
Content: What the guest experience is like - Microsoft Teams
Content Source: Teams/guest-experience.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@pablovera100  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @pablovera100, thank you for your feedback.
This is all the information I have:
https://microsoftteams.uservoice.com/forums/555103-public/suggestions/17750851-i-want-to-use-multiple-teams-accounts-at-the-same
If you want some else, you can add a new user voice item.

@pablovera100 Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Arduino IDE

It works in Arduino IDE?

I test it using Arduino IDE, you have to flash your chip with bootloader first to accept Arduino codes,
I'll try to update the README file and the wiki with all details soon.
Anyway, this is tutorial on Arduino website regarding this
https://create.arduino.cc/projecthub/akarsh98/using-a-stm32-like-an-arduino-tutorial-stm32f103c8-1073cb

I test it using Arduino IDE, you have to flash your chip with bootloader first to accept Arduino codes,
I'll try to update the README file and the wiki with all details soon.
Anyway, this is tutorial on Arduino website regarding this

Can you tell me what's that didn't work? Is pcm-stm32f103.ino compiled/uploaded correctly ?
BTW, you have to put hex.h on the same directory with  pcm-stm32f103.ino files , and you can create hex.h using Audacity or FFmpeg to create PCM unsigned 8-bit
on Audacity [ref]

Open the audio file
File -> Export ...
In the window that pops up, select the file format as "Other uncompressed files"
Select "Options"
Select the "Header" as "RAW (header-less)"
or  FFmpeg
 ffmpeg 'source.mp3' -t 10 -acodec   pcm_u8 -ac 1 -ar 8000 song.wav
and convert it to hex here http://tomeko.net/online_tools/file_to_hex.php?lang=en
But first you need to use my hex.h hex file first to make sure it will work
pin number will be PA8

Your hex.h file is too big and causing compile error, I reduced the file but PA8 output is only 32kHz PWM signal. only hiss in the speaker

I reduced the file

After you cut off the hex file, you should change quack_wav_len value according to number of hex digit on that file (e.g use notepad++ to count it )

something I have a problem with converting to HEX, the sound file is 4 seconds long, after conversion it has a large size of 302884 bytes

I have a problem with converting to HEX,

Ok, first have you test the file I provided, does it work, to making it work for new sound file you should use Audacity as I mentioned above

I figured out Audacity, everything worked fine, but another problem appeared, it does not work simultaneously with the SSD1306 display, separately everything works, but together only information is displayed on the display, but there is no sound
void loop() {         display.clearDisplay();   display.setTextSize(1);   display.setTextColor(WHITE);   display.setCursor(10,0);   display.println("OK");   display.display();   sound();  }   void sound(){   char sample = pgm_read_byte(&quack_wav[p++]);   pwmWrite(PA8, sample) ;     if (p == quack_wav_len) {p=0;}     delayMicroseconds(120); }

separately everything works, but together only information is displayed on the display, but there is no sound

Please upload the .ino .hex files you're using, and tell me by detail what did work and didnt work .
Anyway this code were made precisely for specified clock cycle and uses
pwmWrite , I think using any other function in the loop will cause it to have unpredicted delays and the wave will be out of sync
you can either debug it by changing  pwmtimer2.setPrescaleFactor(   )  to lower value

I redid the function a little and everything worked, I will continue to test
void sound() {
  for (int i = 0; i <= sizeof(quack_wav); i++) {
    char sample = pgm_read_byte(&quack_wav[p++]);
    pwmWrite(PA8, sample);
    if (p == sizeof(quack_wav))
    {
      p = 0;
    }
    delayMicroseconds(120);
  }
}

I redid the function a little and everything worked, I will continue to test

Good to hear that, BTW I create this project to test/learn about composite video on the STM32F103c8t6 black pill  ,
Even there already been projects doing it, but I want it to be learning experience for me and others, if you are interested about this please let me know to contact each other, it's not necessary to start soon, anytime you have free time .

yes, it's interesting, still, increase the volume, on one transistor it's still not enough volume


Show Realtime Schedule.

Inserting Realtime Schedule is too heavy for this system.
This system is not for team. Setting Schedule will be inserted on Next Project.


error 500 on OPTIONS preflight

I've allowed all the origin, and also the PUT and OPTIONS method for the request, but the client still got the error on preflight. I tried the API from 2 different postman, and got two different response header. The client is using AngularJS $http

please paste more information, e.g. your current config

@adamchainz Sorry, this is the current config, anything else do you need?

Hmm I can't see anything obvious. Please find the stack trace from your 500 - in your logs or with Sentry or w/e. It's going to be very hard to debug without it.

@adamchainz sorry to ask, but sentry for the angluar client app or the API?

API

API
On Wednesday, 26 October 2016, Kenan Christian<EMAIL_ADDRESS>wrote:

@adamchainz https://github.com/adamchainz sorry to ask, but sentry for
the angluar client app or the API?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ottoyiu/django-cors-headers/issues/157#issuecomment-256316673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0WCd5CcyrfrdVRSK1JLZDnXz9vUzoRks5q3zS4gaJpZM4Kg9fz
.

--
Adam

Fixed after disabling App Enlight. It seems that app enlight disable OPTIONS request.


Add vue-template-es2015-compiler

Using it to transpile the render code generated allows to remove with
statements, making the generated code strict mode compatible

Any idea when you'll be able to release this? Any doubts I can help with?

Travelling 🗺
Published a new release just now.

Thanks!


Asset named "' + name + '" was overwritten because of naming collision

I see you are checking @ this line
https://github.com/halt-hammerzeit/webpack-isomorphic-tools/blob/48533b5db3d6864dc1f26d4d9fce67e9bd28aa07/source/helpers.js#L2
for the existence of a parsed asset and you print out an error message if it is already exists. Now if the parsed asset was an empty object (i.e {}) this would be a problem and the message would be misleading.
Not sure if it should check for the emptiness of the object before displaying the message or even before assigning the parsed asset to set[name] @
https://github.com/halt-hammerzeit/webpack-isomorphic-tools/blob/c20c11456be2dcc3d80a2ce88796de406d6f879d/source/plugin/write assets.js#L199

@kikoanis That error message basically means that either the naming function or the filter function are not specific enough. Technically an empty object doesn't have any meaning and rewriting it could be ok but I think it would be better to let a developer know that his functions aren't specific enough.

Also see this discussion
https://github.com/halt-hammerzeit/webpack-isomorphic-tools/issues/10

Totally makes sense Actually I was getting this when trying to build react-redux-universal-hot-example and I only get the error messages on parsing "less" files (font-awsome files) so it might be an issue with webpack-isomorphic-tools.js config file. Will look more and see how it goes. Also, you might consider adding the file name (m.name) to the error message if the parsed file was empty for the message to be more informative.

@kikoanis that's a good idea and actually i've included some info already: you might try the debug: true flag

I see you figured out the real issue in this https://github.com/erikras/react-redux-universal-hot-example/pull/423
Thanks. closing...

FYI
I'm posting this in every issue and PR to notify whoever may be interested:
today I've released an alternative helper library called universal-webpack.
It takes a different approach than webpack-ismorphic-tools and instead of hacking Node.js require() calls it just compiles all code with target: 'node' webpack configuration option.
As a result, all Webpack plugins and features are supported.
If you think you might need that here's an example project:
https://github.com/halt-hammerzeit/webpack-react-redux-isomorphic-render-example


HELP!  0 Employees identified

Regardless of the domain, I do not get any results on a scan.  I have tried generating a second API key to see if it would address, to no avail.
python InSpy.py  COMPANYNAME  --domain company.com  --email first.last
Regardless of company and domain I receive response "0 Employees identified"

Hi, I met the same problem with you.
I found the reason is the request is blocked by Linkedin.
in http.py line 19:
        r = requests.get(url, timeout=3, headers=random_header())

	if r.status_code == 200:
		if "linkedin.com" in url:
			return {"status": r.status_code, "response": r.text}
		else:
			return {"status": r.status_code, "response": r.json()}
	else:
		return {"status": r.status_code, "response": ""}

I got the response status  "999"
So, in workbench.py line 55, search_linkedin, the response is : {'response': '', 'status': 999}
also I found someone else met the same problem:
https://support.archive-it.org/hc/en-us/community/posts/115011883686-What-is-a-HTTP-999-error
But I still don't know how to fix it.
Can anyone help on this ? thanks a lot.

@gojhonny  could you help on this ?

Hey guys, thanks for reporting this. Will look into it!

我也是，运行了代码没有受到一点反馈的数据，提示错误

I was working on this for the last couple of months in my fork of this project. I found that to scrape or to view the LinkedIn pages one must login to their accounts or use LinkedIn API. If you try to scrape directly without login into your LinkedIn account it throws a register/login page to you resulting in HTTP error - 999. Looking into all possible solutions without adding an API or login


Webhooks

This could use some tests, but webhooks + charges are well on its way.

Still got some things to tweak on this, but just wanted to push this up so you could check it out so far.


Compiling help: i need help with this error

Which target are you using?
[ ] Windows
Explain your issue
Please check first if your issue haven't already been reported yet, and make sure you ran the update.bat file before building.
source/funkin/game/PlayState.hx:20: characters 25-27 : Expected ) or , or :

what did you edit in the source, since it has to be because of that

what did you edit in the source, since it has to be because of that

what did you edit in the source, since it has to be because of that

you can't for sure do that in haxe
for helping you though i need more context of what were you trying to do

(closing this since no response back from a while


framework/core/Core.php not found in version 4

We decided to just do our own Behat things with SilverStripe 3.x as the silverstripe-behat-extension tied us into old version of Behat etc. and we wanted to be able to do our Behating consistently with how we do it in our Symfony projects.
To access the core stuff from SilverStripe 3.x so we did...
    require_once(__DIR__ . '/../../../../framework/core/Core.php');

...so we could do things like this...
    /**
     * @BeforeScenario
     * @return void
     */
    public function loadFixtures()
    {
        global $databaseConfig;
        require_once(__DIR__ . '/../../../../framework/core/Core.php');
        $fixtureFactory = Injector::inst()->create(BehatFixtureFactory::class);
        DB::connect($databaseConfig);
        DB::get_conn()->clearAllData();
        FixtureLoader::loadInto($fixtureFactory);
    }

Unsurprisingly, things have changed and this doesn't work with version 4.
Can anyone shed light on how we would approach this now?

It may be best to raise this on the behat module but I suspect you'll need to make use of the new CoreKernel we have in SS 4.

Thanks, we're not using the behat module, so not sure that would help. I'll get onto the new CoreKernel as time allows.

OK - cool, I'll close this as it's a support request

We decided to just do our own Behat things with SilverStripe 3.x as the silverstripe-behat-extension tied us into old version of Behat etc. and we wanted to be able to do our Behating consistently with how we do it in our Symfony projects.

We've updated behat to the latest version, so would you consider trying the current behat module again?
Otherwise, you can upgrade your behat code the same way we upgrader this module, with the following change:
https://github.com/silverstripe/silverstripe-behat-extension/commit/f768c952dccf1bbb19a3032e6413953ac147fe8d

Thanks @tractorcow, that got that sorted.


[QUESTION] DeltaTable delete with spark and scala no databricks

libs
spark 3.3.2
scala 2.12.14
hadoop 3.3.2
delta 2.2.0

delta table config
deltaTable logRetentionDuration 4 hours
deltaTable deleteFileRetentionDuration 4 hours

environment
environment: kubernetes
1 driver 46G, 14cores + 16 executors 46G, 14cores
cloud: aws s3
no databricks used

spark config
"spark.sql.shuffle.partitions": "2000"
"spark.default.parallelism": "2000"
"spark.sql.extensions": "io.delta.sql.DeltaSparkSessionExtension"
"spark.sql.catalog.spark_catalog": "org.apache.spark.sql.delta.catalog.DeltaCatalog"
"spark.databricks.delta.vacuum.parallelDelete.enabled": "true"
"spark.databricks.delta.retentionDurationCheck.enabled": "false"
"spark.sql.sources.parallelPartitionDiscovery.parallelism": "250000"
"spark.sql.adaptive.enabled": "true"
"spark.sql.adaptive.optimizeSkewedJoin.enabled": "true"

code
import io.delta.tables.DeltaTable
val deltaTable = DeltaTable.forPath(spark, "s3://BUCKET/TABLE_PATH)
deltaTable.delete("date < '2023-02-24'")
deltaTable.optimize()
deltaTable.vacuum(retentionHours = 0)

Question1:
I see only three executors doing DeltaFileOperations: Listing
INFO DeltaFileOperations: Listing s3a://BUCKET/TABLE/date=...
Why only 3 executors are involved in listing and it is not parallelized more?
Question2:
I do not see partitioned files being deleted when I run it in aws s3
Question3:
All works well when I run it in Azure: partitions are deleted.
Question4:
When I run it in azure then I see also:
23/02/24 07:58:55 WARN AzureFileSystemThreadPoolExecutor: Disabling threads for Delete operation as thread count 0 is <= 1
23/02/24 07:58:55 INFO AzureFileSystemThreadPoolExecutor: Time taken for Delete operation is: 0 ms with threads: 0
23/02/24 07:58:55 WARN AzureFileSystemThreadPoolExecutor: Disabling threads for Delete operation as thread count 0 is <= 1
23/02/24 07:58:55 INFO AzureFileSystemThreadPoolExecutor: Time taken for Delete operation is: 0 ms with threads: 0
23/02/24 07:58:55 WARN AzureFileSystemThreadPoolExecutor: Disabling threads for Delete operation as thread count 0 is <= 1
23/02/24 07:58:55 INFO AzureFileSystemThreadPoolExecutor: Time taken for Delete operation is: 0 ms with threads: 0

Regarding question 2 and 3: in S3, are you saying you don't see the files being deleted? or the partitions being deleted? If you run VACUUM and delete all the files in a partition, we don't delete the partition folder right away. We would do that on the next VACUUM run.

Does "spark.databricks.delta.vacuum.parallelDelete.enabled" work in kubernetes?


Make breaking API changes to align with docs

When I was writing the docs, I decided to write them for the API I want, not the one I have :P
So we should make some changes to the library itself to bring it up to date:

[ ] Make ApolloClient a default export of the package
[ ] Make the arguments to ApolloClient an object instead of positional
[ ] Make sure ApolloClient#query doesn't take a returnPartialResults option
[ ] Add a few more functions to WatchedQueryHandle

@jbaxleyiii @johnthepink heads up, these are definitely breaking changes. Let me know if you disagree with any of them. You can see the docs for examples of how it would look: https://apollostack.gitbooks.io/apollo-client/content/

@stubailo cool I'll be thinking about it, but on first glance looks good to me!

@stubailo I'm always a fan of documentation driven development :+1:
Those look good to me, espeically tmaking the client the default export. I'd also personally prefer if the default for forceFetch was false. It seems like defaulting to cached data is the more often desired action, rather than forcing entirely new data. I can see how returnPartialData's default makes a lot of sense though due to the UI concerns.

@jbaxleyiii yeah, I think you're right. My only concern is that to make forceFetch false the default, we need a lot more testing about how it interacts with different servers that may or may not support certain features.

Perhaps by default, we should just assume the server doesn't support the Relay Node spec, and just check if we have all of the query results on the client. Then we can set forceFetch to false, and it will help a lot for stuff like navigating to a route you just visited.

Perhaps by default, we should just assume the server doesn't support the Relay Node spec, and just check if we have all of the query results on the client.

@stubailo I think this sounds great


Add Irena's code to recipes

We should add Irena's code for computing displacement time series of layers and the bed:
https://github.com/irenavankova/ApRES_processing/tree/main

closed with #6


SSH_Host_Script added

Description
SSH_Host_Script is added
#2534
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[ ] I have added the project meta data in the PR template.

Project Metadata
none
Category:

[ ] Calculators
[ ] AI/ML
[ ] Scrappers
[ ] Social_Media
[ ] PDF
[ ] Image_Processing
[ ] Video_Processing
[ ] Games
[ ] Networking
[ ] OS_Utilities
[x] Automation
[ ] Cryptography
[ ] Computer_Vision
[ ] Fun
[ ] Others

please review at once @avinashkranjan @1e9abhi1e10


ensure DEFINE-TYPE defined constructors are lexical

right now we use DEFINE-GLOBAL-VAR*. Fixing this will require some understanding of compile-time vs load-time etc.

Also see unit in early-types.lisp.

(fixed)


Also run helm init when using local binary

Also when using the local binary the command helm init --client-only should be called to ensure that helm is properly initialized as otherwise package will fail if not
There is no harm when running command on an already initialized helm
~# helm init --client-only
$HELM_HOME has been configured at /home/jenkins/.helm.
Not installing Tiller due to 'client-only' flag having been set
Happy Helming!

/assign @axdotl

Thanks @axdotl


Update [Jira] tests to include color validation

Augmenting the tests for Jira that were added in #2541 to also validate color scheme

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against d2db92b1986ad935efc2e9fffd4eac1b1bb5330c


[Suggestion] Plausible deniability feature?

This is a really useful and simple tool. Thank you!
In regards to usability while traveling, sometimes authorities at an authoritarian country may coerce the visitors to decrypt the files on on a USB drive. It would be nice to have a plausible deniability feature linked to a second passphrase where the second passphrase would decrypt some dummy data of the user's choosing (e.g., another PDF file). Just a thought.

There has been a lot of talk about plausible deniability over the last tens of years and the outcome seems quite clear:
Plausible deniability works well only and only if the whole device (especially the HW) and ideally the whole world is made to contain only plausible deniability functionality.
So I think in this project it would not make much sense. But YMMV.

@dumblob sorry I'm not as well versed in cryptography and such. I wasn't aware that implementing a second passphrase for plausible deniability is not effective. Could you provide some references? Thanks.


Append tests failing with xarray 0.19.0

With xarray 0.19.0, the following two tests are failing:
DatasetWriterTest.test_append_overlapping_append_newer
>       np.testing.assert_equal(expected, ds3.t.data)
E       AssertionError:
E       Arrays are not equal
E
E       (shapes (5,), (3,) mismatch)
E        x: array(['2001-01-01T00:00:00.000000000', '2001-01-02T00:00:00.000000000',
E              '2001-01-03T00:00:00.000000000', '2001-01-04T00:00:00.000000000',
E              '2001-02-05T00:00:00.000000000'], dtype='datetime64[ns]')
E        y: array(['2001-01-01T00:00:00.000000000', '2001-01-02T00:00:00.000000000',
E              '2001-01-03T00:00:00.000000000'], dtype='datetime64[ns]')

DatasetWriterTest.test_append_overlapping_retain
E       AssertionError:
E       Arrays are not equal
E
E       (shapes (6,), (7,) mismatch)
E        x: array(['2001-01-01T00:00:00.000000000', '2001-01-02T00:00:00.000000000',
E              '2001-01-03T00:00:00.000000000', '2001-01-04T00:00:00.000000000',
E              '2001-01-05T00:00:00.000000000', '2001-01-06T00:00:00.000000000'],
E             dtype='datetime64[ns]')
E        y: array(['2001-01-01T00:00:00.000000000', '2001-01-02T00:00:00.000000000',
E              '2001-01-03T00:00:00.000000000', '2001-01-04T00:00:00.000000000',
E              '2001-01-05T00:00:00.000000000', '2001-01-05T00:00:00.000000000',
E              '2001-01-06T00:00:00.000000000'], dtype='datetime64[ns]')

The failures seem to be connected with the introduction of consolidated=True as the default for xarray.open_zarr.

Analysis: for test_append_overlapping_append_newer, the test just needs to be adapted, since it was relying on the old default when creating the initial zarr, resulting in mixed consolidated / unconsolidated access.
For test_append_overlapping_retain, there seems to be a bug in the way the append code handles consolidated zarrs, which had previously gone undetected since the test only made use of unconsolidated zarrs.


Conflicting with Excalidraw plugin embeds

Hello,
Unfortunately, the excalidraw plugin and this plugin conflict.
It is not possible to open an embedded excalidraw canvas when this plugin is active:

Is there any way to work around this?
Thanks!

+1. Might look into fixing this myself as it completely impedes on the excalidraw workflow

private isImageElement = (imgEl: HTMLImageElement): boolean => {
        return (
            imgEl &&
            'IMG' === imgEl.tagName &&
            imgEl.className !==
                'excalidraw-svg excalidraw-embedded-img excalidraw-canvas-immersive'
        )
    }

That fixes the issue and excalidraw now reverts to requiring a double click to open, but I wonder actually if keeping the svg image capabilities are worthwhile, for example you can zoom infinitely for clarity because it's an SVG. It just needs some tweaks to make it work correctly (render in the gallery view, and open at a larger real size).
It isn't the most sound piece of code as the class could change and I've only tested a few things, but I'm definitely thinking about opening a PR and seeing if @sissilab can help us out 👍

@lucmos It seems there is not any conflict now and I'll close this thread now. Please feel free to give feedback if you still got any trouble.

Good spot @sissilab. Confirmed it's working now on the latest Excalidraw version 🎉
I think it was fixed in this version, but eh, can't be bothered figuring out for sure!

Good spot @sissilab. Confirmed it's working now on the latest Excalidraw version 🎉
I think it was fixed in this version, but eh, can't be bothered figuring out for sure!

Thank you for providing this clue. I'll keep an key on it if encountering any conflict again.


Menu window

I notice on my machine, sometimes the menu window gets lost. For most people this won't be a big issue, but we may want to consider if there is a way to bring the window to the front somehow. A menu link or something?

This is a real problem, and it is also a problem in one of my inspirations (GIMP). I have to see how they are working that out.

Potentially this? http://pyqt.sourceforge.net/Docs/PyQt4/qwidget.html#raise

I tried the above with no luck (possibly due to my use of MacOS). Seems to be an outstanding problem and after a bit of web searching it looks to be OS dependent as well.

Added the raise_() method to the menu component. And to test on FileNavigator. On my mac I see it momentarily come to the foreground, but then become buried again.

@nguy as this is system dependent, could you make a video showing this problem in your system? I think I know what you mean, but I would like to be sure that your experence is the same as mine.

I shared a short video in your email.

Interesting how IOS externalize the menu bar.
If you pay attention the menu does blink for a moment in the upper corner where it should be. That mean it's IOS that is interfering in the Qt work, we have to avoid that.

Yep, you are correct. This is why I indicated that it is an OS-dependent issue. I don't know if there is a solution for this behavior.

have you seen this page "http://doc.qt.io/qt-5/osx-issues.html"?

in script
app.setAttribute(QtCore.Qt.AA_MacPluginApplication, True)

Think this could be put in at startup as a check with a conditional statement identifying MacOSX?

first of all, did it work?

Yes, this does work to put the menu on the menu window. To implement, we'd have to install an OS check. I feel like this should be automatic in the background.
The issue remains how to bring the menu window to the front for operation.

To test and internet search returns the following code:
import os
import platform

if os.name == 'posix':
    if platform.system().lower() == 'darwin':
        print("You are using MacOSX")

My guess would be that we could put a script in artview.core that returns isMac boolean. If true then the code above is run. We then just need to keep this in every script. Unless you see an easier way.

I see you already created it, but I wouldn't make such a function.

We already have a function to start the app, in artview/view.py start. I would add the test in that function and start using it in the scripts.

also, I don't understand the test (what does darwin has to do with it?). Do you have any explanation why it works?

what do you mean with "The issue remains how to bring the menu window to the front for operation."

This PR doesn't need to be implemented. Just a first shot.
I like the idea of having this test somewhere other than core.py.
As for the test: os.name returns posix for any unix-based system, therefore a secondary test is required to see what the system platform is called. 'Darwin' is returned for the Mac unix system, 'linux' is returned for linux unix systems.
Updated PR code.

This remaining issue is that displays still cover the menu mostly. Having the File, Modes on the Menu window helps, but it can still be buried inadvertently pretty easily. This is the part I'm not sure can be fixed easily.

ou yes, this is a difficult one. The answer to that would be imploring the tab system so we don't need separated windows.

I'm still on the fence of whether to move the displays toward tabs. You successfully did this already with the "simple plot". One advantage of this would be that ARTview might be able to be used more easily on mobile devices then...

I'm thinking of doing something similar to kate (KDE "simple" text editor). It has Tabs and at the same time the option successively divide the display horizontally or vertically, so that multiple tabs can be displayed at the same time.

I'll be interested to see this. Split windows can be really nice to work with.

The original scope does not seem possible at this time. PR #136 takes care of menu on MacOSX now appearing in the MainMenu window.


Consider adding IntersectionObserver

Now that the dropzone attribute has been removed from HTML5, assuming it is removed from the test, 1 point is freed up.
A good potential replacement is IntersectionObserver. It isn't widely supported yet (Chrome only so far and in development for Edge) and is incredibly important for performance. Currently ad providers do things like check 60 times a second (or more) if an ad is in view to report back to advertisers if the ad was viewed (they only pay if the ad is visible). This happens for every ad on the page, which is one of the major reasons ads slow down the browser.
IntersectionObserver is feature designed to combat this so that ads can be performant without resorting to these hacks.

Spec https://wicg.github.io/IntersectionObserver/

IntersectionObserver was added in commit cc2054745e44d0180d51c1a8d90311a6c91f6536


#9490: Replace tt_dnn op's usage in C++ with TTNN

Ticket
Link to git hub issue #9490
Problem description
Replace tt_dnn op's usage in C++ with TTNN for unary op migration for following ops:

ABS
ACOS
ASIN
COS
ATAN
COS
ELU
EQZ
ERF
ERFC
EXP
EXP2
EXMP1
GELU
GEZ
GTZ
HEAVISIDE
I0
ISFINITE
ISINF
ISNAN
ISNEGINF
ISPOSINF
LEAKY_RELU
LEZ
LOG
LOG10
LOG2
LOGICAL_NOT_UNARY
LTZ
NEG
NEZ
RECIP
RELU
LEAKY_RELU
SIGMOID
SIGN
SIGNBIT
SILU
SIN
SQRT
SQUARE
TAN
TANH

Checklist

[ ] Post commit CI passes
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes
[ ] [post-commit] models tests
[ ] Device perf regressions and output report
[ ] Model perf regressions and output report
[ ] Nightly fast dispatch model regression CI

lgtm! let us know when pipelines are green. thank you!

Assuming we move the tests in a separate PR.

Yes. It will be raised as a separate PR


use xenon.blockstack.org as bootstrap node

Description
This PR changes the bootstrap node from testnet-miner.blockstack.org to xenon.blockstack.org

[x] @agraebe

@friedger Just need to fix the linting issue then should be good to go!


feat: add wallet migration to askar

Codecov Report
Attention: Patch coverage is 0% with 163 lines in your changes are missing coverage. Please review.

Project coverage is 0.05%.  Consider uploading reports for the commit 23fe486 to get more accurate results

Files
Patch %
Lines

...s/aries_vcx_core/src/wallet/base_wallet/migrate.rs
0.00%
28 Missing :warning:

...ries/misc/wallet_migrator/src/migrate2askar/mod.rs
0.00%
25 Missing :warning:

...s_vcx_core/src/wallet/base_wallet/base64_string.rs
0.00%
18 Missing :warning:

...s_vcx_core/src/wallet/base_wallet/base58_string.rs
0.00%
17 Missing :warning:

...ies_vcx_core/src/wallet/indy/indy_wallet_config.rs
0.00%
15 Missing :warning:

aries/aries_vcx_core/src/wallet/indy/mod.rs
0.00%
15 Missing :warning:

aries/aries_vcx_core/src/wallet/utils.rs
0.00%
11 Missing :warning:

...aries_vcx_core/src/wallet/base_wallet/key_value.rs
0.00%
10 Missing :warning:

.../aries_vcx_core/src/wallet/agency_client_wallet.rs
0.00%
6 Missing :warning:

...es/aries_vcx_core/src/wallet/base_wallet/record.rs
0.00%
6 Missing :warning:

... and 3 more

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1144   +/-   ##
=====================================
  Coverage   0.05%   0.05%
=====================================
  Files        506     509    +3
  Lines      25477   25452   -25
  Branches    4641    4614   -27
=====================================
  Hits          13      13
+ Misses     25464   25439   -25

Flag
Coverage Δ

unittests-aries-vcx
0.05% <0.00%> (+<0.01%)
:arrow_up:

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

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

@xprazak2 needs some rebasing now, should be prolly easier now to simplify feature flags?

Rebased, looks much cleaner now.

well done 👍