Execute commands when interpreter is not running

We have an issue regarding command execution when the interpreter is not running, In this situation, only engine commands such as Start/Pause can be executed. Other commands, for instance Uod commands can be added to the engine's command queue, but are not executed until the interpreter is started by clicking Start.

The behavior of executing such commands only after the user starts the method is correct. But we need to visualize that commands are pending in a way very similar to the runlog. We can't technically add them to the runlog (which only exists when a run is active) but we need to show these pending commands in a similar way.


Pre 3.1 upgrade playbook to adjust cinder for multi-backend support.

This playbook will be leveraged to meet the ursula 3.1.0 upgrade pre-conditions for cinder service. Following cinder configurations will be adjusted to able to support cinder multi-backend in 3.1.0.

Some of the previous versions of ursula configured cinder service based on host it was running on. Since we are using Ceph for cinder backend, cinder could be configured to unique host (i.e host=ceph) to make cinder volume services HA.

Adjust cinder backend name from rbd_volumes to new name based on the ceph backend type.
- rbd_ssd  --> For SSD Ceph Cluster
- rbd_hybrid  --> For Hybrid Ceph Clutser

3: Create a default cinder volume_type based on the backend name(set as extra_specs). Also adjust existing volumes, snapshots to this default volume_type.
- CEPH_SSD
- CEPH_HYBRID

retest

retest

I think there's only a few changes needed here. I don't care about CI pass/fail as this is outside the CI code path.

retest

retest

retest

retest

retest

retest

I'm going to merge this in. We can refine as testing happens.


Cherry-pick PR #8879 into release-1.4: [reconfiguration] fix bug caused by #8484

This cherry-pick was triggerd by a request on #8879
Please review the diff to ensure there are not any unexpected changes.

Motivation
The bug is that we expect a compute result for every transaction in the block,
without considering the fact that if a block is suffix of a reconfiguration block, its
compute result is carried from its parent and should be viewed as empty.
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
ENABLE_FAILPOINTS=1 ./scripts/cti --pr 8879 --run reconfiguration
Reconfiguration: total epoch: 101 finished in 131 seconds
Related PRs
(If this PR adds or changes functionality, please take some time to update or suggest changes to the docs at https://developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

cc @zekun000

:broken_heart: Test Failed - ci-test

/land


Platformio doesn't fetch samd core files for Servo Library

Configuration
Operating system:
Ubuntu 14.04
PlatformIO Version (platformio --version):
version 2.11.2.dev2
Description of problem
I am trying to build some code with the Servo library for Sparkfun's samd21 mini breakout board.
However, the code fails to compile. The errors I get are listed below:
Actual Results
In file included from src/blink_main.cpp:2:0:
.pioenvs/zero/Servo/Servo.h:67:2: error: #error "This library only supports boards with an AVR or SAM processor."
#error "This library only supports boards with an AVR or SAM processor."
^
scons: *** [.pioenvs/zero/src/blink_main.o] Error 1
===================================================== [ ERROR ] Took 1.46 seconds =====================================================

The content of platformio.ini:
[env:zero]
platform=atmelsam
framework=arduino
board=sparkfun_samd21_mini_usb
build_flags=-I include/ -I include/ros_lib
lib_use=Wire, Servo

Source file to reproduce issue:
#include <Arduino.h>
#include  <Servo.h>
void setup() {
}
void loop() {
}

Additional info
The contents of the Servo directory for my environment are:
Servo/
├── avr
│   └── ServoTimers.h
├── sam
│   └── ServoTimers.h
└── Servo.h
2 directories, 3 files

I know  that this is not correct since arduino 1.6.9 libraries contains a samd/ folder as well.
Is this an issue with my workspace or the tool?

@valeros please re-pack Arduino framework for SAM and fix this library framework-arduinosam/libraries/Servo. Need to remove Servo/src/avr folder.
@bhavyangupta this issue is fixed in PlatformIO 3.0. It has C Processor behaviour and understands macros. Temporary solution for you is to remove manually ~/.platformio/packages/framework-arduinosam/libraries/Servo/src/avr folder.

I removed the avr folder as directed, but the code doesn't compile and gives the same error. Are there any other things that I can try?

Sorry, that is not a bug.
@valeros please ignore my post above. Code compiles very well and AVR code is under #ifdef.

The issue is linked with wrong board manifest.
Temporary solution is to add SAM define. We will fix it in the next release:
[env:sparkfun_samd21_mini_usb]
platform = atmelsam
framework = arduino
board = sparkfun_samd21_mini_usb
build_flags = -DARDUINO_ARCH_SAM

@valeros should we add -DARDUINO_ARCH_SAM for all SAMD based boards?

Thank you for responding so promptly!  I really appreciate your help. I added the build switch, it now gives me the following errors:
[Thu Jul 28 15:04:26 2016] Processing zero (platform: atmelsam, framework: arduino, board: sparkfun_samd21_mini_usb, lib_use: Wire, Servo, build_flags: -I include/ -I include/ros_lib -DARDUINO_ARCH_SAM)
---------------------------------------------------------------------------------------------------------------------------------------
....
.pioenvs/zero/Servo/sam/ServoTimers.h:52:29: error: 'TC1' was not declared in this scope
......
.pioenvs/zero/Servo/sam/ServoTimers.h:59:29: error: 'TC1' was not declared in this scope
......
.pioenvs/zero/Servo/sam/ServoTimers.h:66:29: error: 'TC1' was not declared in this scope
......
.pioenvs/zero/Servo/sam/ServoTimers.h:73:29: error: 'TC0' was not declared in this scope
......
.pioenvs/zero/Servo/sam/ServoTimers.h:80:29: error: 'TC0' was not declared in this scope
......
.pioenvs/zero/Servo/sam/Servo.cpp:79:9: error: 'union Tc' has no member named 'TC_CHANNEL'
..
.pioenvs/zero/Servo/sam/Servo.cpp:81:13: error: 'union Tc' has no member named 'TC_CHANNEL'
..
.pioenvs/zero/Servo/sam/Servo.cpp:81:43: error: 'TC_CCR_SWTRG' was not declared in this scope
..
.pioenvs/zero/Servo/sam/Servo.cpp:90:13: error: 'union Tc' has no member named 'TC_CHANNEL'
..

@bhavyangupta please remove this lib_use = Wire, Servo

I did that and there is no change in the build output. Same errors.

Could you try my platformio.ini above? https://github.com/platformio/platformio/issues/737#issuecomment-236006485

HI @bhavyangupta !
Looks like the implementation for samd MCUs is missed in this library.
Could you please try to use this version? Just unzip it to the lib folder in your project. Does it work now?

@ivankravets
Here's the platformio.ini I tried ( no changes to output):
[env:sparkfun_samd21_mini_usb]
platform=atmelsam
framework=arduino
board=sparkfun_samd21_mini_usb
# build_flags=-I include/ -I include/ros_lib -DSERIAL_BUFFER_SIZE=10000
# build_flags=-I include/ -I include/ros_lib
build_flags=-I include/ -I include/ros_lib -DARDUINO_ARCH_SAM
# lib_use=Wire, Servo

@valeros
I agree with your observation. However, when I put the Servo folder from the repo in the lib folder, the errors are still there. I can see that the tool has downloaded the Servo library and ignored the one the lib folder. Here's the directory tree for your reference:
.
├── include
│   └── ros_lib
├── lib
│   ├── readme.txt
│   └── Servo
├── lo
├── log
├── main.cpp
├── platformio.ini
└── src
    ├── blink_main.cpp
    ├── duration.cpp
    └── time.cpp
5 directories, 8 files

sparkfun_samd21_mini_usb/
├── FrameworkArduino
├── FrameworkArduinoVariant
├── FrameworkCMSISInc
├── FrameworkDeviceInc
├── libFrameworkArduino.a
├── libFrameworkArduinoVariant.a
├── Servo
└── src
6 directories, 2 files

Hm, OK let's replace the old Servo lib with new one here ~/.platformio/packages/framework-arduinosam/libraries/Servo/

Tried that. No change in the output. Is there a way to disable the downloading of Servo library? I can see that it gets the library from somewhere after I remove the Servo from the .pioenvs/sparkfun_samd21_mini_usb/ folder.

@bhavyangupta please read warning block http://docs.platformio.org/en/stable/projectconf.html#envs-dir
@bhavyangupta Could you share somewhere full project that doesn't work?

@ivankravets  Here you go.
I was deleting the folder to "check" if it actually fetches a new Servo library if one is already there in the ~/.platformio/packages/framework-arduinosam/libraries/Servo/ path.

Sorry, that was my mistake. Don't need to use build_flags= -DARDUINO_ARCH_SAM. The Servo library contains own SAMD implementation.
Just place Servo library to lib folder of the project.
platformio.ini
[env:sparkfun_samd21_mini_usb]
platform=atmelsam
framework=arduino
board=sparkfun_samd21_mini_usb
# build_flags=-I include/ -I include/ros_lib -DSERIAL_BUFFER_SIZE=10000
# build_flags=-I include/ -I include/ros_lib
#build_flags= -DARDUINO_ARCH_SAM
# lib_use=Wire, Servo
src_filter = +<blink_main.cpp>

P.S: Be careful, I filtered source files just to the one file (because I don't have ros_lib headers)
Structure
tree -L 3 .
├── lib
│   ├── Servo-master
│   │   ├── README.adoc
│   │   ├── examples
│   │   ├── keywords.txt
│   │   ├── library.properties
│   │   └── src
│   └── readme.txt
├── platformio.ini
└── src
    ├── blink_main.cpp
    ├── duration.cpp
    └── time.cpp

5 directories, 8 files

@valeros please update Servo library in frameworksam package. It should contain SAMD implementation.

It works! Thanks a lot!

Thanks a lot that reported.
Please don't close this issue. @valeros will update library and close it.

Updated!


Updated README File

Enhanced the README file for the Random Disco Light Simulator project by adding detailed sections on features, inputs, outputs, and contribution guidelines. Improved the overall structure and readability for a better user experience.
Thankyou!

#164

Thank you for your feedback and for assigning the level.


feat: update grad_sync_done

What this PR does / why we need it:
enable grad_sync_done for a list of np.ndarrays
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @your-reviewer

Release note:

NONE

/approve
/lgtm


Change subscriptions to be a multiple of $1, instead of 1¢

Right now stripe's default receipts for e.g. a $75 subscription looks like:
7500 x monthly
Because the subscription is defined as 7500 cents each month.
The receipt would be cleaner if the subscription was based on a $1 amount to make it:
75 x month

Fixed by moving to stripe products instead of plans


Allow merging into non-master branches

We need this to sync stable branches.

/cc @mdbooth

If I understand your change correctly, the merge-bot will now fetch from upstream (the "source" remote) a branch with the same name of the downstream (the "dest" remote).
Do you think we should make the upstream branch name configurable, so that you can merge, say, "upstream/v1.25" into "dest/v4.12"? Or do you think that we're fine with using the same name for the use cases we currently have?

If I understand your change correctly, the merge-bot will now fetch from upstream (the "source" remote) a branch with the same name of the downstream (the "dest" remote).

That's not what the change does, or at least not what I intended.
The head for the destination branch was mistakenly hardcoded to master which I'm changing here to follow the value passed via the --merge option.

Do you think we should make the upstream branch name configurable, so that you can merge, say, "upstream/v1.25" into "dest/v4.12"? Or do you think that we're fine with using the same name for the use cases we currently have?

We already have that with the --source and --merge options.

OK my Python is rusty... (eh eh)


Changing postgres port to 5432

Codecov Report
Patch and project coverage have no change.

Comparison is base (2f177e9) 89.47% compared to head (dc3bba9) 89.47%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   89.47%   89.47%
=======================================
  Files           3        3
  Lines          38       38
=======================================
  Hits           34       34
  Misses          4        4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Inconsistency between selection behaviour for polygons and circles without fill.

As a result of https://github.com/openlayers/openlayers/pull/7750 if, for example, a large box is drawn over a small one the small box can be selected if fill is not defined, but if a large circle is drawn over a small one the small circle cannot be selected even if the CircleStyle has no fill.  Should the polygon fill hit detection logic also be applied CircleStyle for consistency?

Yes.

Bump

@mike-000 - can you provide a simple example that demonstrates this problem?  If so, it would probably be straightforward for someone to debug and fix.  If not, this issue will eventually be closed by the stale bot.

https://codesandbox.io/s/polygonstyles-p3pyb
From top to bottom in the codesandbox:

polygons
circle geometries using same style as the polygon
point geometries styled with circle style using same fill as the above.

Left column: overlapping geometries are styled with no fill
Right column: overlapping geometries are styled with transparent fill
Test clicking on areas A,B,C and on stroke in positions D and E

Polygons with no fill
A, B, C no effect
D and E The polygon with that stroke is selected
Polygons with transparent fill
A lower left polygon is selected
B,C,D,E top right polygon is selected
Circles with no fill and with transparent fill,, and points using circle style with no fill
A lower left circle is selected
B,C,D,E top right circle is selected
Points using circle style with transparent fill
A, B, C no effect
D and E The circle with that stroke is selected
For all three geometry types I would have expected no fill and transparent fill to behave the same way as they do for polygons

I believe, that the behaviour from openlayers 4.6.5 where polygons without fillstyle are also detected is not bad too.

Fixing a year old inconsistency probably isn't a priority over getting a major release out but I'll keep it alive until someone has time to look at it.

The solution for Circle geometry is simple, as one line appears to have been left the wrong side of an if statement in #7750 https://codesandbox.io/s/polygon-styles-wtorl
#7750 was never applied to CircleStyle so some reworking of ol/style/RegularShape will be needed to apply the same logic and should be a separate PR


merge from arlac77/npm-package-template-esm-only

.travis.yml

chore(travis): add /^mkpr/.*$/ to branches.only
chore(travis): add /^npm-template-sync/.*$/ to branches.only

:tada: This PR is included in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Tracking lifetime of file in summary and adding telemetry

Fixes #6551
This is only a draft PR and I will complete when I return.
We are trying to add 3 new info to summary: 1. time stamp of creating the document 2. runtime version when the document was created. 3. summary counter. And we also need to log them.
I have tried to add the runtime version to metadata of container runtime.
I haven't found where the summary counter is.
For the timestamp, we will use timestamp in IConnected class, which should be accessible in the driver layer (documentDeltaConnection). I'm trying to see how to access this from the runtime layer where the metadata is.

to access IConnected.timestamp in runtime layer, I found this chain:
p.s. timestamp is in the new version of protocol-definitions, which hasn't been consumed, so I'm tracking the other property of IConnected
documentDeltaConnection.innitialMessages -> deltaManager.setupNewSuccessfulConnection -> deltaManager.connectCore -> deltaManager.connect -> container.connectToDeltaStream -> Container.load -> loader.loadContainer -> loader.resolveCore -> loader.request -> RelativeLoader.request -> dataStoreHelper.requestFluidObject
@agarwal-navin is this the right path I should follow?

Can you please add tests to validate that these properties are logged correctly and are preserved across loads.


fix(table): update element finding function

Updated using of find_element to find_elements to avoid the NoSuchElementException when the element isn't found in the page.

:tada: This PR is included in version 3.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fabric API missing from fabric.mod.json, but game crashes without it

Just missing one line here:
https://github.com/Tectato/BetterCommandBlockUI/blob/9e68f833a229ac3033f3f817f633eb6e358e10db/src/main/resources/fabric.mod.json#L28-L31

Fixed in latest release


How to zoom with Aperture without a computer touchscreen? As it requires two fingers.

LineageOS/android_packages_apps_Aperture

[ ] I have checked that a similar feature request does not already exist.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

scrcpy/blob/21e2e2606e683326d9171afb106a5473f16d8bb9/doc/video.md does not mention zoom.


Cilium not compatible with XDP SYNPROXY  K8S nodeport service protection

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Sorry this issue is not a bug from what I understand, and not important enough to be a CFP I think. I had ported in kernel version of XDP SYNPROXY to xdp-projects bpf-examples repo to run XDP SYNPROXY as DaemonSet to protect each k8s node from SYN flood attack xdp-synproxy , for example to protect k8s nodeport service from SYN flood attack, it works as intended to prevent SYN flood attack.
the issue is the XDP SYNPROXY  breaks normal nodeport service connection,  when node received the SYN, XDP SYNPROXY responded with SYN+ACK, and client responded with ACK, node send RST to the final ACK because the SYN is not seen by node, tcpdump on the node interface shown below:
11:19:12.350335 IP <IP_ADDRESS>.51014 > <IP_ADDRESS>.32506: Flags [.], ack<PHONE_NUMBER>, win 502, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 0
11:19:12.350538 IP <IP_ADDRESS>.32506 > <IP_ADDRESS>.51014: Flags [R], seq<PHONE_NUMBER>, win 0, length 0

from what I understand, XDP hooks at driver layer, Cilium datapath mainly works at TC layer, the SYN packet processed by XDP SYNPROXY and Cilium datapath has never seen the SYN. XDP SYNPROXY also works with netfilter SYNPROXY module and netfilter connection tracking, but Cilium datapath mostly bypass netfilter and has its own connection tracking, so it appears to me this is expected result, I could be missing something. I wonder though if there is possibility let Cilium nodeport datapath accommodate this scenario?
Cilium Version
cilium-cli: v0.14.0 compiled with go1.20.2 on linux/amd64
cilium image (default): v1.13.2
cilium image (stable): v1.14.2
cilium image (running): v1.13.0
Kernel Version
Linux cilium-dev 6.6.0-rc2+
Kubernetes Version
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.1", GitCommit:"5e58841cce77d4bc13713ad2b91fa0d961e69192", GitTreeState:"clean", BuildDate:"2021-05-12T14:18:45Z", GoVersion:"go1.16.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.9", GitCommit:"b631974d68ac5045e076c86a5c66fba6f128dc72", GitTreeState:"clean", BuildDate:"2022-01-19T17:45:53Z", GoVersion:"go1.16.12", Compiler:"gc", Platform:"linux/amd64"}
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

by the way, XDP SYNPROXY works with k8s with flannel CNI fine with nodeport service

that would be nice to have, currently it's hard/impossible to apply any DDoS protections to Cilium in form of iptables rules, with the workaround being using a non-Kubernetes node as a frontend

@ninja- SYNPROXY is stateful connection so it requires connection tracking, if you just want to rate limit tcp/udp request per IP, if you do not have Cilium XDP program attached to node, you can attach your own XDP program to rate limit request per IP, for example to rate limit DNS request per IP DNS RRL , and I have ported it here  libbpf-bootstrap xdppacket


Change volume when toggling pause/unpause

Is it possible to change Volume when User pause/unpause during recording ?
currently it gives me error

Could you provide your problem more specifically? It makes me hard to imagine your situation.

Hi, What I am trying to do is when user click pause record, I want to add 1 second of no sound record. That's why when user click I want to  set volume to 0 record 1sec and change back volume to 1.
Is this a possible?

Thanks for coming back. I now get what you mean. I have to think about how this would be possible someday :thinking:

Hello, I have another question
Is it possible to record without a sound ?
I need to create a 1-2sec mp4

Hello, I have another question
Is it possible to record without a sound ?
I need to create a 1-2sec mp4 that has no sound

I think it is good to open up another issue for that. Currently, I have no idea.


Add method to get testresults count by launch item id

Coverage increased (+0.1%) to 85.456% when pulling 82585054edabbcfd019658cbe6eb02dc15580fa1 on results-count-by-launch-item into cac98015f97a947333617a97214e198711014f1e on master.


added 3d dp question

Paths in matrix whose sum is divisible by k is added to the Leetcode DP folder

@all-contributors please add @ashwath462 for his code


ensure bytes for blosc name

This failed for me on Python 3.  There might be more cases where this is necessary

Thanks.

On a deeper look these changes shouldn't be necessary, blosc.compname_to_compcode already has an internal check to convert text to bytes. The error was probably also due to Zarr versions being out of date on some workers. I'm going to back out these changes but let me know if you still hit problems..


bugfix: remove out large samples from the multi pack batch sampler

Fix for #51
larger entries, longer than max_batch_len, may be present in the output of the sampler which gets filtered out in the collator
in rare instances, this can lead to empty batches which can be problematic for the training loop

Great catch! This should be processed in the data processing step. Can you add a warning in red or yellow 033 notation so the user knows?

You can just add it when there's a sample larger than Max batch lenght


[FEA] make deployed artifact pom self-contained without parent poms

As of 23.12 and before we rely on separately deployed intermediate parent poms such as
rapids-4-spark-parent/
rapids-4-spark-private-parent/
Moreover there are separate Scala 2.13 parent poms https://oss.sonatype.org/content/repositories/snapshots/com/nvidia/
rapids-4-spark-private-parent_2.12/
rapids-4-spark-private-parent_2.13/
rapids-4-spark-private_2.12/
rapids-4-spark-private_2.13/
There is no shared parent pom to avoid dependencies between pipelines
There is an additional complexity from the fact that we do not deploy:deploy the artifacts with all related projects but selectively deploy artifacts with deploy:deploy-file which makes the process susceptible to accidental omissions of intermediate dependencies.
This issues proposes to fold parent pom definitions into the deployed leaf poms with the effect that there is no implicit dependency on any internal pom. The self-contained pom should contain just enough details for the artifact to be used as a dependency for external projects and for dependency:get not require any intermediate poms. This may be achievable by flatten:flatten or some similar custom logic whose output will be used with deploy:deploy-file.
Note that this issue is only about the poms that are published externally to Central for other projects to consume spark-rapids as a dependency, which we already massage before deploying.
We should feel free to maintain any form most suitable for building spark-rapids.

@gerashegalov  I'm adding dependency check for the pre-merge and nightly build jobs, according to our discuss method
1, Create a fresh local dir such as /tmp/local-release-repo
2, mvn deploy:deploy-file -Durl=file:/tmp/local-release-repo  xxx \
3, Create another fresh local directory for /tmp/test-get-dest
4, Create a fresh empty dir /tmp/m2-cache every iteration such that there are no cached side-effects from one
 depenendcy:get to another
5, execute depenency:get xxx
    -Dmaven.repo.local=/tmp/m2-cache

The dependency check still failed on jdk-profiles on current branch-24.02, are you going to fix it first?

Hi @NvTimLiu

The dependency check still failed on jdk-profiles on current branch-24.02, are you going to fix it first?

The test should mimic the release deploy logic irrespective whether this issue (eliminating parent and intermediate poms) is fixed.
If the test is failing it means it still does not reflect that currently deploy-file for jdk-profiles is required.
Ideally the test should share the same script
https://github.com/NVIDIA/spark-rapids/blob/ed1fa9f0c92f8c7c1a4cbdedf2b4d61db6d1beca/jenkins/deploy.sh#L97-L107
just with different parameters. It would be helpful to create a file with the list of the artifacts required for the release. Then we could use it as the input in the release/test script.
Once this issue is fixed we will drop the jdk-profiles module from this list, but the test remains valid.

Hi @NvTimLiu

The dependency check still failed on jdk-profiles on current branch-24.02, are you going to fix it first?

The test should mimic the release deploy logic irrespective whether this issue (eliminating parent and intermediate poms) is fixed.
If the test is failing it means it still does not reflect that currently deploy-file for jdk-profiles is required.
Ideally the test should share the same script
https://github.com/NVIDIA/spark-rapids/blob/ed1fa9f0c92f8c7c1a4cbdedf2b4d61db6d1beca/jenkins/deploy.sh#L97-L107
just with different parameters. It would be helpful to create a file with the list of the artifacts required for the release. Then we could use it as the input in the release/test script.
Once this issue is fixed we will drop the jdk-profiles module from this list, but the test remains valid.

@gerashegalov Thanks for the explanation.
I agree we can reuse the  deploy script [spark-rapids/jenkins/deploy.sh](https://github.com/NVIDIA/spark-rapids/blob/ed1fa9f0c92f8c7c1a4cbdedf2b4d61db6d1beca/jenkins/deploy.sh#L97-L107) to do dependency check for the to-be-released aritfacts.
A file list for deploy artifacts would be complex for the deploy script to work with, e.g.
file list:  pom.xml, dist/pom.xml, rapids.jar, rapids-cuda11.jar,  then deploy.sh should parse the parent pom, dist pom, classifier jar, main jar, javadocs.jar, sources.jar, etc.
More over, we need to refactor  deploy.sh [spark-rapids/jenkins/deploy.sh](https://github.com/NVIDIA/spark-rapids/blob/ed1fa9f0c92f8c7c1a4cbdedf2b4d61db6d1beca/jenkins/deploy.sh#L97-L107) to parse deploy list file.
Let me check if we should break this issue into diff tasks.

@NvTimLiu I'll file some other issue for discussing a reproducible deploy test. To reiterate, we will need this test regardless of what our poms look like.
This issues is about a potential simplification of published poms


Add oaiSource, itemOaiId, metadataPrefix to exception message

Please add oaiSource, itemOaiId, metadataPrefix to exception message for easier troubleshooting
https://github.com/DSpace/DSpace/blob/a250efd99cb41b430d6637f39a698dc615f78601/dspace-api/src/main/java/org/dspace/harvest/OAIHarvester.java#L885
We get this message only:
Asunto: AUSJAL: Harvesting Error

Collection 66 failed on harvest:

Date:           16/06/21 2:25
Status Flag:    3

OAI server returned the following errors during getDescMD execution: [idDoesNotExist]

Exception:
org.dspace.harvest.OAIHarvester$HarvestingException: OAI server returned the following errors during getDescMD execution: [idDoesNotExist]
        at org.dspace.harvest.OAIHarvester.getMDrecord(OAIHarvester.java:887)
        at org.dspace.harvest.OAIHarvester.processRecord(OAIHarvester.java:451)
        at org.dspace.harvest.OAIHarvester.runHarvest(OAIHarvester.java:354)
        at org.dspace.harvest.OAIHarvester$HarvestThread.runHarvest(OAIHarvester.java:1380)
        at org.dspace.harvest.OAIHarvester$HarvestThread.run(OAIHarvester.java:1371)
        at java.base/java.lang.Thread.run(Thread.java:834)

Something like this:
        "OAI server returned the following errors during getDescMD execution: " + errorSet.toString()
        + " on source:" + oaiSource + ", item: " + itemOaiId + ", metadataprefix: " + metadataPrefix);

Moving this to "unscheduled backlog" for now.  I agree this could be improved, but it's low priority for main development at this time (cause of DSpace 7 work).  That said, if someone can get to this sooner, PRs are always welcome!


[probleme while running bloxstrap]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What settings do you have enabled?

Did you read https://github.com/pizzaboxer/bloxstrap/wiki/Roblox-crashes-or-does-not-launch?

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


SD FIA review (mandatory requirements)

Review of SD for FIA requirements that are mandatory.

Move FMT_SMF.1 "Set authorization failure parameters for FIA_AFL_EXT.1" to the selection for consistency.
Rest of the PR is fine.

I have made the change and added it to this pull request. Please approve and I will merge.


add signing as part of the build job

add the secureboot signing of kernel modules by converting the build process into a multistage process
This:

Adds a ...Build.Sign sub section to the CRD for defining the certificate and key secret information for signing.

It modifies the job manager to loop through a list of jobs in order, waiting for each to finish before it starts the next one, and waiting for all of them to finish before it reports the build as complete.

It adds a ShouldRun() function to the Maker interface so the job manager loop can determine if a given job is required to run (so the build does not run if Build.Dockerfile is not defined, and signing doesn't run if Build.Sign is not defined in the CR)

If this is not the last job to run it  changes the name of the image produced by the job by appending the job name, and feeds that into the next job with the final job producing containerImage

Finally:   It adds a new (signing.go) k8s job definition that actually runs the signing process itself

A similar set of changes will need to be added to buildconfig, shipwright or any other future build processes we consider, but that would be a different PR...

/ok-to-test

superceeded


Ansible version dependancy problem stops the install

I am getting this error
SpectrumScale_base: Error:
  SpectrumScale_base:  Problem: conflicting requests
  SpectrumScale_base:   - nothing provides (ansible-core >= 2.12.2 with ansible-core < 2.13) needed by ansible-5.4.0-3.el8.noarch
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

I have tried with both
Spectrum_Scale_Developer-<IP_ADDRESS>-x86_64-Linux-install
and
Spectrum_Scale_Developer-<IP_ADDRESS>-x86_64-Linux-install
The problem occurs with both.

Hi Daniel,
thank you for reporting this issue.
The last Spectrum Scale version I successfully tested was <IP_ADDRESS> mid of July 2022. At that time, Ansible 2.9.27 was available on EPEL.
In the meantime this "Ansible classic" version got pulled from EPEL, and was replaced with version 5.4.0 that requires ansible-core. ansible-core is only available on CentOS Stream, but not on CentOS 8.
When re-running the tests today, I get the same error.
This is essentially an Ansible packaging issue like also pointed out here.
An option might be to pull in ansible-core from CentOS Stream, but there is risk that doing so might cause dependency issues.

Switching to pip-based Ansible installation seems to solve the issue. Also now disabling EPEL after installing Ansible. Disabling EPEL is recommended by latest Spectrum Scale installer versions to prevent (Object) package dependency conflicts.
diff --git a/shared/Vagrantfile8.rpms b/shared/Vagrantfile8.rpms
index 6103d92..747615c 100644
--- a/shared/Vagrantfile8.rpms
+++ b/shared/Vagrantfile8.rpms
@@ -108,11 +108,10 @@ Vagrant.configure('2') do |config|
       /bin/sh -c \"sed -i 's/metalink/#metalink/g' /etc/yum.repos.d/epel*\" && /bin/sh -c \"sed -i 's|#baseurl=https://download.example/pub/|baseurl=https://mirror.init7.net/fedora/|g' /etc/yum.repos.d/epel*\"
     "

-  # Install Ansible as required by Spectrum Scale 5.1.1+ installer
-    config.vm.provision "shell",
-    name:   "Install additional RPMs required by Spectrum Scale",
+  config.vm.provision "shell",
+    name:   "Install Ansible as required by Spectrum Scale 5.1.1+ installer",
     inline: "
-      /usr/bin/dnf makecache && /usr/bin/dnf install -y ansible
+      /usr/bin/dnf -y install rust libsodium python3-paramiko python3-pynacl && pip3 install setuptools_rust && pip3 install ansible==2.9.27
     "

   # Install RPM to prevent CLI errors due to locale for German users
@@ -129,4 +128,11 @@ Vagrant.configure('2') do |config|
     inline: "
       /usr/bin/dnf upgrade -y
     "
+
+  # Disable EPEL
+  config.vm.provision "shell",
+    name:   "Disable EPEL to prevent package conflicts during Spectrum Scale installation",
+    inline: "
+      /bin/sh -c \"sed -i 's/enabled=1/enabled=0/g' /etc/yum.repos.d/epel*\"
+    "
 end

@dannyk96 I have merged the potential fix for your issue, please test and close this issue in case it is working for you as well.
I verified that the latest code works with the libvirt backend using Spectrum Scale <IP_ADDRESS> and <IP_ADDRESS>.

Thnaks Harold!
I get much further now
The ./spectruynmscale toolkit get extarcted, but won't run as /usr/bin/pthion3 is missing
I am working in it now

Hi Daniel, the screenshot proves that you can ssh into a fully-deployed VM so I'd consider the original bug as fixed.
I supposed the new issue you're seeing now is due to the way you changed to the root user. Can you try a sudo -s instead?
BTW: What you are trying to do (modifying the Scale installation) is not in-scope for this project (yet).

Hi Harold,
I have no intension of running ./spectrumscale to install anything.
I was running it with argument 'version' as a pre-check, as I had a problem on a customer system where ./spectrumscale would not run because of Python dependancy issues : and this was proved by getting them to run just  ./spectrumscale version as the reproducer

Hi Daniel,
got it. This is what I just successfully tried on my VM instance:
$ vagrant ssh
Last login: Mon Oct 10 15:36:52 2022 from <IP_ADDRESS>
[vagrant@m1 ~]$ sudo -s
[root@m1 vagrant]# cd /usr/lpp/mmfs/<IP_ADDRESS>/ansible-toolkit/
[root@m1 ansible-toolkit]# ./spectrumscale version
usage: spectrumscale [-h] [-v] [--version]
                     {setup,node,config,nsd,filesystem,callhome,fileauditlogging,enable,disable,install,deploy,upgrade}
                     ...
spectrumscale: error: invalid choice: 'version' (choose from 'setup', 'node', 'config', 'nsd', 'filesystem', 'callhome', 'fileauditlogging', 'enable', 'disable', 'install', 'deploy', 'upgrade')
[root@m1 ansible-toolkit]# ./spectrumscale --version
IBM Spectrum Scale Ansible Install Toolkit release: <IP_ADDRESS>

But yes this ticket can be closed.
I confirtn that Ansible is being installed via pip

see new ticket re. .ansible-playbook not being on the default path (it is now in usr//local/bin)


feat(obstacle_cruise_planner): implement slow down planner

Description
Implemented slow down planner, inserting slow down point in the trajectory where the point is close to the dynamic/static obstacles.

Related links
launcher PR: https://github.com/autowarefoundation/autoware_launch/pull/288
Tests performed
Planning simulator works well.

TODO

[x] scenario sim: https://evaluation.tier4.jp/evaluation/reports/50ce7861-d8b3-5ef9-87b1-f68421f860a8?project_id=prd_jt

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

slow down virtual wall


Expose thrift connection_timeout setting

Thrift is extremely sensitive to connection timeouts, and defaults to just 100ms... We should expose the configuration option and set a slightly higher default, to prevent unexpected and badly explained 'No live servers' errors.

Thank you!


Failed build of mapbox on Android. Installation configuration

Environment

Dev OS: macOS 12.5.1

RN: 0.69.4
"@rnmapbox/maps": "rnmapbox/maps#main"
Followed this manual for v10:
https://github.com/rnmapbox/maps/blob/main/android/install.md#mapbox-maps-sdk-v10
import org.apache.tools.ant.taskdefs.condition.Os

// Top-level build file where you can add configuration options common to all sub-projects/modules.

buildscript {
    ext {
        buildToolsVersion = "31.0.0"
        minSdkVersion = 21
        compileSdkVersion = 31
        targetSdkVersion = 31

        if (System.properties['os.arch'] == "aarch64") {
            // For M1 Users we need to use the NDK 24 which added support for aarch64
            ndkVersion = "24.0.8215888"
        } else {
            // Otherwise we default to the side-by-side NDK version from AGP.
            ndkVersion = "21.4.7075529"
        }

        // START - rnmapbox
        RNMapboxMapsImpl = "mapbox"
        // END - rnmapbox

    }
    repositories {
        google()
        mavenCentral()
    }
    dependencies {
        classpath("com.android.tools.build:gradle:7.1.1")
        classpath("com.facebook.react:react-native-gradle-plugin")
        classpath("de.undercouch:gradle-download-task:5.0.1")
        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

allprojects {
    repositories {
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url("$rootDir/../node_modules/react-native/android")
        }
        maven {
            // Android JSC is installed from npm
            url("$rootDir/../node_modules/jsc-android/dist")
        }
        mavenCentral {
            // We don't want to fetch react-native from Maven Central as there are
            // older versions over there.
            content {
                excludeGroup "com.facebook.react"
            }
        }
        google()
        maven { url 'https://www.jitpack.io' }
    }
}

Error I got:
Could not determine the dependencies of task ':app:mergeDebugAssets'.
> Could not resolve all task dependencies for configuration ':app:debugRuntimeClasspath'.
   > Could not find com.mapbox.maps:android:10.7.0.
     Searched in the following locations:
       - file:/path/node_modules/react-native/android/com/mapbox/maps/android/10.7.0/android-10.7.0.pom
       - file:/path/node_modules/jsc-android/dist/com/mapbox/maps/android/10.7.0/android-10.7.0.pom
       - https://repo.maven.apache.org/maven2/com/mapbox/maps/android/10.7.0/android-10.7.0.pom
       - https://dl.google.com/dl/android/maven2/com/mapbox/maps/android/10.7.0/android-10.7.0.pom
       - https://www.jitpack.io/com/mapbox/maps/android/10.7.0/android-10.7.0.pom
     Required by:
         project :app > project :rnmapbox_maps
   > Could not find com.mapbox.mapboxsdk:mapbox-sdk-turf:6.4.0.
     Searched in the following locations:
       - file:/path/node_modules/react-native/android/com/mapbox/mapboxsdk/mapbox-sdk-turf/6.4.0/mapbox-sdk-turf-6.4.0.pom
       - file:/path/node_modules/jsc-android/dist/com/mapbox/mapboxsdk/mapbox-sdk-turf/6.4.0/mapbox-sdk-turf-6.4.0.pom
       - https://repo.maven.apache.org/maven2/com/mapbox/mapboxsdk/mapbox-sdk-turf/6.4.0/mapbox-sdk-turf-6.4.0.pom
       - https://dl.google.com/dl/android/maven2/com/mapbox/mapboxsdk/mapbox-sdk-turf/6.4.0/mapbox-sdk-turf-6.4.0.pom
       - https://www.jitpack.io/com/mapbox/mapboxsdk/mapbox-sdk-turf/6.4.0/mapbox-sdk-turf-6.4.0.pom
     Required by:
         project :app > project :rnmapbox_maps

Is this config not full?

Should I follow this manual?
https://docs.mapbox.com/android/navigation/guides/get-started/install/#add-the-dependency

If configure according to https://docs.mapbox.com/android/navigation/guides/get-started/install/#add-the-dependency
then next error occurs:
2 files found with path 'lib/arm64-v8a/libc++_shared.so' from inputs:
 - /Users/user/.gradle/caches/transforms-3/18b53a92cc290c862f00c3b0cd13f0b8/transformed/jetified-react-native-0.69.4-debug/jni/arm64-v8a/libc++_shared.so
 - /Users/user/.gradle/caches/transforms-3/767a7e73575d58e1c01420f9f9c2f252/transformed/jetified-common-22.1.0/jni/arm64-v8a/libc++_shared.so
If you are using jniLibs and CMake IMPORTED targets, see
https://developer.android.com/r/tools/jniLibs-vs-imported-targets

@rusakovic   do you have the access token configured to download mapbox ?!
https://docs.mapbox.com/android/navigation/guides/get-started/install/#configure-credentials
For libc++_shared.so have you tried this:
https://github.com/rnmapbox/maps/blob/main/README-v10.md
# add the following:
android {
    packagingOptions {
        pickFirst 'lib/x86/libc++_shared.so'
        pickFirst 'lib/x86_64/libc++_shared.so'
        pickFirst 'lib/arm64-v8a/libc++_shared.so'
        pickFirst 'lib/armeabi-v7a/libc++_shared.so'
    }
}

@mfazekas
Thank you for your help!
Maybe this link could be added to main Readme?

to here?
https://github.com/rnmapbox/maps/blob/main/android/install.md#mapbox-maps-sdk-v10
Because my personal navigation flow started from this link.

@rusakovic yes I agree, we should move that to Android install guides.


Multiple instances of same mesh

Hi there,
I am having a hard time figuring out if it is possible to have several instances of the same mesh with different transforms.
This is important for rendering speed purposes when you have a LOT of instances of the same mesh (nuts and bolts for example)
Thank you in advance,
Pierre

Hi Marco,
Thank you for the answer.
Unfortunately, I need a different transform matrix for each instance, so the Glyph will not work.
As for the mesh input sharing, it does not speed up the rendering, which is what I am looking for.
It would be great to be able to add a mesh with a list of transforms and let the system understand that it can create instances
Regards,
Pierre

On 1 Feb 2021, at 14:32, Marco Musy<EMAIL_ADDRESS>wrote:
Hi,
how many such instances do you have?
One way to save memory I guess is to force the same underlying data to the actors (first method)
which is still slow in rendering. The second is to use Glyph which is way faster:
from vedo import *
from numpy import random
cone = Cone(r=0.5, res=50)
first method
cone_data = cone.inputdata()
meshes = []
for i in range(5000):
x,y = random.rand(2)*100
cn = Cone().color(i).pos(x,y)
cn._update(cone_data)
meshes.append(cn)
show(meshes, axes=1)
second method
positions = random.rand(5000, 2)*100
gl = Glyph(positions, cone, c=range(len(positions)))
show(gl, axes=1)
https://user-images.githubusercontent.com/32848391/106508181-56f5fe00-64cc-11eb-9d91-730cf08ee350.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/marcomusy/vedo/issues/305#issuecomment-771102037, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHQPJSXERV5Q7LB4YXD6E3S436TJANCNFSM4W5HIZBA.

@theponpon  this is weird.. I posted yesterday a solution and it looks like it disappeared.. i post it again:
from vedo import *
from numpy import random

cone = Mesh(datadir+'shuttle.obj')

positions = random.rand(5000, 2)*1000
oarr = random.randn(5000, 3)

gl = Glyph(positions, cone, orientationArray=oarr, c=range(len(positions)))
show(gl, axes=1)


Rename Facebook.Get to Facebook.Request and add HTTP method as parameter

Dynamic contents as parameter depending on HTTP method selected

Implemented with latest instructions.


Clarify label prefix guidance

I just found in
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/labels.md
that:
Valid label keys have two segments: an optional prefix and name, separated by a slash (/). (...)
If the prefix is omitted, the label key is presumed to be private to the user. System components which use labels must specify a prefix. The kubernetes.io/ prefix is reserved for kubernetes core components.
I found that cluster addons specify "k8s-app" and "version" label with no prefix:
    k8s-app: heapster
    version: v2
    kubernetes.io/cluster-service: "true"

I wonder if we should rename it to:
    kubernetes.io/k8s-app: heapster
    kubernetes.io/version: v2
    kubernetes.io/cluster-service: "true"

or rather update the doc about labels.

Does #9901 address this?

#9901 only fixed heapster service labels, there are other services. I can work on it now.

I were wrong, #9901 fix the last one.

The guidance should specify that system components attaching labels to user objects must be prefixed.
In the case of cluster addons, we're the "user".
The real problem is that the cluster addons are in the default namespace -- #9356, so our object names and labels pollute users' object names and labels.


Rabbit MQ connections needs to be managed in a separate module

RabbitMQ connections should not be created parallel as the connections returned within the same period will be using only a single underlying connection. This has been thourougly tested . No connection creation to any host should be parallel but should be done in a sequential way. The connection creation has to be created in a different thread so that we can save time and effort from the application.
I am using RabbitMQConnectionManager class which basically abstracts out all the connection and channel creation part to a separate class . This takes care of recreating connections in the background if connections are down.

Hi,
Can you squash your commits, remove any formatting changes (keeping everything in line with existing formatting) and re-submit please? It's quite to hard to follow what the actual changes are with all the formatting changes. Also please remove the Eclipse (or any other IDE) project files from git repository.

Actually, I'm going to have to reject this pull request due to:

There appears to be an external dependency added. While I'm quite comfortable with feature improvement that require addition of external dependencies, I'd like to limit external dependencies to ones that have been vetted by some large or small open source community (by way of adoption).
The changes made are very hard to follow due to lot of unnecessary formatting changes submitted along side.
The improvements made are not being communicated in enough of a detail to determine whether their functionality is line with the goals of this project.


[added] wraptag property for Link

Since it is a very common use case to want the active class of a Link to be applied to the parent tag for Bootstrap compatibility, this PR adds a new optional wraptag prop to Link. If supplied, the link will be wrapped in the given tag, and the class names applied to the wrapper instead of the anchor tag.
That means I can now do this:
<Link to="foo" wraptag="li">Foo</Link>

instead of having to make my own wrapper component, looking something like this:
var Navlink = React.createClass({
  mixins: [ State ],
  render: function() {
    var p = this.props;
    var className = (p.linkClasses || '')+(this.isActive(p.to, p.params, p.query) ? ' active' : '');
    return
{Link(p)}
;
  }
});

Have you considered using react-router-bootstrap with react-bootstrap instead?

My personal opinion is that Link should stay as simple as possible. As you said yourself, it's easy to create a wrapper component, which is the model encouraged by React. Rather than make existing components do many things, wrap them and add functionality using composition.

@krawaller I agree with @gaearon. I'd like to keep <Link> as simple as possible and the wrapper + mixin approach is the one currently encouraged by React.

Yeah, I guess you guys are right. It just feels a bit... inelegant to have Link do the active class work, only for me to do the same thing in a wrapping component to cover the bootstrap use case.
My reasoning was that since using a property is cognitively much cheaper than creating a component + importing a mixin + making the isActive call + adding the classes, the difference in cost warranted the API bloat.
But I might be overestimating how common the wrapping need actually is, and either way, you are probably right to demand an even higher price for increasing API complexity. Well done, you passed the test! ;)

Personally I don't see adding a component in the consuming app as a cost. I wrap most of the components from other libraries, whether it's a Link, a DocumentTitle or YoutubePlayer, because more often than not I need some customization.
This also gives you greater flexibility when changing tools. For example, I had a custom Link before migrating to React Router, when I still used Backbone. I also had UserLink, ArticleLink and other app-specific links that accepted my models. When I moved from Backbone to React Router, I didn't need to modify the consuming components that used UserLink or ArticleLink—only these helper components themselves.

I meant it as a cognitive cost, i.e. how hard it is to figure out. But that was a really silly reasoning to begin with, as the ability to create components is definitely something you can assume in a user of this library. :P

What about allowing the link component itself to be customized like Ember does it:
<Link component="li" to="..." />

This would allow us to do whatever we want.

This is deliberately a WONTFIX here, because those are going to have very poor a11y. If you need compatibility with e.g. React-Bootstrap, use React-Router-Bootstrap.

ok, sure. I'm not looking for compatibility with React-Bootstrap but I do need the same functionality as krawaller was asking for.
It seems like I have to create my own wrapper component using a mixin which is fine however, that's a tough thing for a new comer to wrap their head around. Especially without knowing there's a state mixin which can help with that.
I think an adding an example to the repo showing how accomplish this would help a lot of people, myself included.

You can quite possibly just use the React-Bootstrap wrapper. It's fairly generic.

You can quite possibly just use the React-Bootstrap wrapper. It's fairly generic.

I'm not interested in using that library, I just want to keep things simple. After browsing through the react-router code base it appears that the State mixin isn't a thing anymore.
Anyone willing to provide a gist on how to create a simple wrapper for Link which applies an active class as discussed in this thread? I'm using ES2015 (ES6)

React-Router-Bootstrap is literally two wrapper components and has no dependency on React-Bootstrap at all.
You can reinvent as much or as little as you'd like. I'm not interested in continuing this conversation.

@taion Thanks for the help. Maybe you didn't realize that your second last comment referred me to react-bootstrap, not react-router-bootstrap. I missed that in one of the initial comments as well.
No need to respond to this one...


Insert translation after the original text

Original message by Noah

For me, however, the plugin would be even more useful if it would insert the translation of the text directly after the original text. I would therefore like to have an option in the settings that could give you the choice of whether the text selected for translation should be deleted and replaced with the translated text or whether the translation should be appended directly to the original text. Perhaps this could even be developed into two separate commands so that the workflow can be even faster and you can decide on a case-by-case basis how the text should be translated.

Thank you, Noah, for submitting your request and for presenting your suggestion in a clear and concise manner. Currently, I am occupied, but I will consider finding an effective solution and implementing it if I determine that there is a widespread need for it. I will keep this issue open and will take action if it receives 10 likes.

I have opened a pull request (https://github.com/friebetill/obsidian-deepl/pull/4) that includes this feature, among others.
I have implemented it in such a way that the translation is inserted directly (with a space) after the selection. Does this work for you?

Closed by #4


Font overrides

After discussing with @GarethOrmerod it seems like there may be occasions on which we need font sizes (and probably spacings) that don't follow the default scale.
The first of these is the UserInitiatedDialog, the styles for which essentially remains the same from the smallest to largest breakpoints.
This PR introduces a fontSizeOverrides utility, that allows us to output whichever of the 10 possible font sizes that exist in the system at each of the small, medium and large breakpoints.

Are we overriding a set of font sizes, or allowing you to override whatever you like?

We're allowing (but definitely not encouraging) you to override whatever you like – you can pick from any of the 10 possible font sizes to display at each of the three breakpoints.
We have a strict system that we want to adhere to wherever possible (nearly everywhere), but in the instances where we need to deviate from it, it felt like it made sense to make that deviation as flexible as possible.


Code release

Hello, thank you for great work!
When will you upload code?

@NikitaVasilevN
Hello,  Nikita Vasilev
I'm sorry for the late reply.
We are internally organizing our code and making modifications to make it easier for users to use.
We appreciate your patience as we will release it soon!
Thank you.


Spatial awareness demo scene doesn't do anything in editor view

Describe the bug
I'm running the Spatial Awareness demo scene from the examples  and nothing happens
To reproduce
Open a new empty project
import nuget
installed MRTK foundatoin, extentions, tools, and examples.
Opened the spatial awareness scene
in the holographic emulator i connected to my hololens.
i hit play
nothing happens.
Expected behavior
I expected to see a wireframe mesh of my environemtn as the plague suggest i do.

There is another MeshObserver class that will render a mesh object file as a spatial mesh.
Updating this bug to revamp the example scene. If in editor, show a game object. If on device, use the actual spatial mesh.
Even with this change, spatial mesh will not show up in emulator by default

Actually going to close this as dupe of #6262


Read marker: crash when turning device just after hitting "Jump to first unread msg" banner

To reproduce:

Hit "Jump to first unread msg" banner
Quickly turn the device in landscape or portrait mode

Crash. Try again if didn't crashed the first time.
I filled a crash report about it.

See https://github.com/matrix-org/riot-android-rageshakes/issues/334

Validated


Support multiple microphones

For multi-devices scenarios (e.g., interviews or calls with two USB microphones connected to the same Mac) it would be great to have the ability to use PTT on multiple devices, with separate shortcuts for each device

@yulrizka With this approach, would it be possible to make the software also remember the last microphone that was selected? Or even switch automatically to a microphone once it's online (example, disconnecting/connecting headphones).

That is possible but it's not implemented yet. I'll be happy to review if someone wants to work on it


Update rabbitmq-server-3.7 package

This PR updates the version of rabbitmq-server-3.7 to 3.7.17

@cf-london Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


fix(react): Re-export makeAllQueryParamsHook

:tada: This PR is included in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Extend the Learning Rate Monitor callback to log the initial lr for optimizers

🚀 Feature
Log the initial learning rate of each optimizer
Motivation
Currently the LR monitor callback only takes effect if the user specifies LR schedulers for training. However, one could want to record what the initial learning rate used for their optimizer's parameter groups even without any LR schedulers.

Pitch
We could log this once on train start following the logic here: https://github.com/PyTorchLightning/pytorch-lightning/blob/f6fe715e73cc87bfc106329f9103564097515379/pytorch_lightning/callbacks/lr_monitor.py#L138-L156
cc @simran2905

Alternatives

Additional context

@ananthsub @edenlightning I would like to work on this. Can you please assign this task to me?

@ananthsub @edenlightning I would like to work on this. Can you please assign this task to me?


add Brigade v2.0.0-alpha.4 post

Currently has a date of 5/6/21... can adjust as needed.

Looks great. My only remaining thought is that maybe we should wrap with a brief paragraph the invites people to kick the tires, provide feedback, etc.

Good call @krancour.  Updated!

@vdice love the last paragraph!


Base msdata running hotspot

hi team
can do msdata to running hotspot ? I tried it but only shows it in all sample layers, it can't be displayed  slice, do u have recommended way?
so thanks

Do you want to run hotspot on each sample or on the sample merged from all samples?

I want to run all  samples to find a common module

You can follow the tutorial to run those methods on MSData object.
Example codes:
import stereo as st
from stereo.core.ms_data import MSData

"""
load samples and create MSData object, let's say ms_data
"""

# Preprocessing
ms_data.tl.cal_qc()

#Filtering
ms_data.tl.filter_cells(min_counts=200, min_genes=3, max_genes=2500, pct_counts_mt=5, inplace=True)

#Normalization
ms_data.tl.raw_checkpoint()
ms_data.tl.normalize_total(target_sum=10000)
ms_data.tl.log1p()

#Highly variable genes
ms_data.tl.highly_variable_genes(min_mean=0.0125, max_mean=3,min_disp=0.5, n_top_genes=2000, res_key='highly_variable_genes')
ms_data.tl.scale()

#  analysis of spatial hotspot
ms_data.tl.spatial_hotspot(
                    use_highly_genes=True,
                    use_raw=True,
                    hvg_res_key='highly_variable_genes',
                    model='normal',
                    n_neighbors=30,
                    n_jobs=20,
                    fdr_threshold=0.05,
                    min_gene_threshold=10,
                    res_key='spatial_hotspot',
                    )


Standardize underscore names in updateHostsFile.py

Some of my PR's accidentally introduced underscore naming into a file that is largely camel case.  Generally, it is good (stylistically-speaking) to have a consistent naming convention.
Looking through PEP8 conventions, it seems that underscore is preferred, so I removed a lot of camel casing in updateHostsFile.py as a result.

I'm glad you did this, @gfyoung.
On balance I would have preferred mixedCase over lower_case_with_underscores because that's most common in node/javascript where I spend most of my time.  That said, I couldn't find a compelling proscription, one way or another, with Python.
Basically, I'm glad to standardize on this based on your initiative.
Tested, works great in all facets I tried.

Thanks @StevenBlack !  I took my decision from the following link here for future reference.

Yes, I saw that.  It's not authoritatively stated, one way or another.  I'm happy it's standardized on something.  Thanks for that.
Next thing would be, pick a Python linter, and add a linter config file to flag code standards violations in the .py files.  Not a big priority, something nice to have in the future.


spirv-fuzz: TransformationMoveBlockDown invalidates the module

Consider the following file.
shader.txt
It contains a perfectly valid SPIR-V (i.e. the following test succeeds).
TEST(A, a) {
  std::string shader; // code from the file

  const auto env = SPV_ENV_UNIVERSAL_1_3;
  const auto consumer = nullptr;
  const auto context = BuildModule(env, consumer, shader, kFuzzAssembleOption);
  ASSERT_TRUE(IsValid(env, context.get()));
}

However, if we try to move a basic block with id 370 one block down, the transformation will be applied correctly (i.e. TransformationMoveBlockDown::IsApplicable returns true) but the spirv-val will fail with with a somewhat ambiguous message:
error: line 121: Case construct that targets 310[%310] has invalid branch to block 414[%414] (not another case construct, corresponding merge, outer loop merge or outer loop continue)
  %310 = OpLabel

Commit id is 7afbc0c8.

@alan-baker Would you be able to take a look at the following files?
bad.asm.txt
good.asm.txt
They differ only in the syntactic positions of blocks 370 and 402, and there is no dominance relationship between these blocks so I don't think that swapping them should change whether the module is considered valid - but "bad" is considered invalid while "good" is considered valid.
Might this be a validator bug?

It does look like a validator bug. The reachability information is incorrect. Because of that, it is miscalculating the contents of the case construct.


Fails to run in Windows 10 pro

From Event Viewer Application log:
Faulting application name: SourceGen.exe, version: <IP_ADDRESS>, time stamp: 0x5bae6108
Faulting module name: comctl32.dll, version: 6.10.17134.285, time stamp: 0xa16245d4
Exception code: 0xc00000fd
Fault offset: 0x00053fd7
Faulting process id: 0xcac4
Faulting application start time: 0x01d45757732bb412
Faulting application path: C:\Users\charl\Downloads\6502bench100a1_debug\SourceGen.exe
Faulting module path: C:\WINDOWS\WinSxS\x86_microsoft.windows.common-controls_6595b64144ccf1df_6.0.17134.285_none_42efceba44e1907b\comctl32.dll
Report Id: db5c6643-df38-481d-988f-cfe8d548530f
Faulting package full name:
Faulting package-relative application ID:

Ugh. Can you try the Release version in case that makes a difference?
.NET is not working out as well as I'd hoped. :-(

Additional info:
The Release version does essentially the same thing:
Note they are both complaining about the comctl32.dll.
Faulting application name: SourceGen.exe, version: <IP_ADDRESS>, time stamp: 0x5bae610f
Faulting module name: comctl32.dll, version: 6.10.17134.285, time stamp: 0xa16245d4
Exception code: 0xc00000fd
Fault offset: 0x00053fd7
Faulting process id: 0x106ac
Faulting application start time: 0x01d45757334d43e8
Faulting application path: C:\Users\charl\Downloads\6502bench100a1\SourceGen.exe
Faulting module path: C:\WINDOWS\WinSxS\x86_microsoft.windows.common-controls_6595b64144ccf1df_6.0.17134.285_none_42efceba44e1907b\comctl32.dll
Report Id: 3727f89b-5964-4b14-8219-059ba55564a1
Faulting package full name:
Faulting package-relative application ID:
running 'dotnet --version' from the command prompt shows 2.1.103
Attempting to install '.NET Framework 4.7.2' says it is already installed.

The 'comctl32.dll' thing has been nagging my memory.  I think I had this problem with a project a while ago.
I had to add Comctl32.lib; to the list of Additional Dependencies for the Linker.
That's Configuration Properties - Linker - Input - Additional Dependencies in Visual Studio 2017.

I posted on stackoverflow and somebody found the problem (https://github.com/fadden/6502bench/issues/3).  I've posted 1.0.0-alpha2 in releases; please give that a try.

Okay, both the release and debug version (1.0.0-alpha2) now run.


Can not verify modal window is present

Hello.
I have a modal window which opens by clicking on the button. I need to verify that the window is shown so I tried to verify window is present by css. Also I tried to verify title of window. It does not work as well

        <select-client-modal _nghost-c5="">

Choose Client

                    <select _ngcontent-c5="" class="form-control choose-client-modal_select ng-untouched ng-pristine ng-valid" ng-reflect-model="Test">

                        <option _ngcontent-c5="" value="test" ng-reflect-value="test"> CCNB </option>
                    </select>

                    <button _ngcontent-c5="" class="btn btn-primary btn-lg" type="button">Select</button>

        </select-client-modal>

How can I verify this window is present and it's a "Choose Client" window?

I get this error
Failures:

Login page should show select client dialog
Message:
Failed: script timeout: result was not received in 11 seconds
(Session info: chrome=70.0.3538.77)
(Driver info: chromedriver=2.43.600210 (68dcf5eebde37173d4027fa8635e332711d2874a),platform=Windows NT 10.0.17134 x86_64)


Fix address computation for symbols loaded from PDB files

Symbol addresses in PDB files are stored as so called "relative virtual
addresses", which is the offset from the start of the corresponding
.text section. Symbol addresses in ELF files however are stored as
"virtual addresses" which include the load bias and the executable
segment offset. To make all our address computations work consistently,
we pass in the load bias (ImageBase for COFF) and the executable
segment offset when loading the PDB file to adjust symbol address.
If the relative virtual addresses are desired when opening a PDB file,
one can simply pass 0, 0 as load bias and exec offset, respectively.
Bug: http://b/194497774
Tested: Unit test.

Thanks for the quick review @antonrohr


Difference between the 'within_alignment' and the 'between_alignment' methods

Dear authors,
I'm writing to understand better the difference between the 'within_alignment' method and the 'between_alignment' one applied in Assemblytics, especially in the case of deletion.
From what I have seen looking at an alingment between two assembled genome, the 'within_alignment' deletion results in a portion where a sequence is well aligned over another one. On the other side, the 'between_alignment' deletions are always on a portion where the sequence aligned presents a deletion, a soft-clipped region or a gap.
So my question is: why should we consider the first method as correct for deletion detection?
Thank you,
Leo

The within-alignment deletions are equivalent to what you would see in the CIGAR string of a BAM file, so they are the easiest to detect since they happen within a single alignment. For example, a small deletion like this:
ACATGCTGATCG
ACATG--GATCG

(This is a toy example where the deletion and sequences are both much smaller than you would detect with Assemblytics. In this example, two bases are deleted, which was detected within an alignment.)
If you want to visualize a within-alignment deletion in your own dataset, MUMmer's show-aligns command will let you do that.

Okay, I will try also with MUMmer.
In another way, I tried to visualize the alignments (BAM file) with the Integrated Genome Viewer. What I have seen is that sometimes, independently from the 'method' assigned, the bed file of Assemblytics presents Deletions in regions where it does not seem to be present. I report below two examples, the first of a within_alignment, and the second of a between_alignment.

I checked the corresponding CIGAR string and they do not show any D.
How could this be explained?

Okay, I will try also with MUMmer.
In another way, I tried to visualize the alignments (BAM file) with the Integrated Genome Viewer. What I have seen is that sometimes, independently from the 'method' assigned, the bad file of Assemblytics presents Deletions in regions where it does not seem to be present. I report below first an example of a 'within_alignment' deletion and secondly a 'between_alignemnt' deletion.

I also checked the CIGAR string of these regions but they don't show any D.
How could this be explained?

Is the BAM file also coming from MUMmer? Assemblytics is simply looking in the delta file for these within-alignment deletions, so it's very possible that if the alignments are different between what you show in IGV and what is in the delta file, that's the source of the difference. That's why I suggest using MUMmer show-aligns to visualize this part of the delta file so you can see the deletion in the delta file visually.
The between-alignment deletion in the second example looks like it falls between the alignments, so if those alignments are of the same contig, then that would make sense and be exactly what we expect to see.
As a general note though, Assemblytics was made 10 years ago when there were no other variant callers for assemblies. Today you have more options for assembly-based variant callers, so I recommend you read some papers to find the newer variant callers and see if one of those works better for your purposes. Genome assembly technologies have also changed in 10 years, so I just wouldn't expect Assemblytics to be the best method anymore.


Im in vue but its throwing a react error at me?

These are the only two lines I have added:

This is the error its throwing. For the record Im using Nuxt 3 but ive turned off SSR just to be sure thats not causing the issue.

Heres my setup:

Seconding this issue.
I'm having a similar problem -
<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'react' in '~\node_modules@theatre\studio\dist'

This problem seems to be related to Vite (used as the main bundler in nuxt) which does not support ENV variables.
It uses import.meta.env.XXX instead of process.env.XXX .
For Nuxt there are two solutions I believe: switch to webpack bundle (because it supports env variables natively) or add this to your nuxt.config.js and keep vite bundler :
export default defineNuxtConfig({ 	vite: { 		define: { 			"process.env.TESS_ENV": process.env.ENV, 		}, 	}  })
It works for me with "nuxt": "3.0.0-rc.12".

Same error but latest temporary solution doesn't work

Same error but latest temporary solution doesn't work

same issue here, I'm on Nuxt 2.15.7 (so webpack) and I get the same issue

Any clue on this?

nope, sorry :(

@plcdnl  Well, I made the dumbest thing ever, I've added react to my deps yarn add react and it started working again.
I'm controlling the studio init manually so it won't affect the bundle size
 store.isDebug && this._initDebug()
  .....

  async _initDebug(){
    // // THEATRE
    const studio = (await import('@theatre/studio')).default
    studio.initialize()
    const project = getProject(labels.projectName, {state})
    const sheet = project.sheet(labels.sheets.hero)

    // STATS
    const Stats = (await import('stats.js')).default
    this._stats = new Stats()
    document.body.appendChild(this._stats.dom)
  }


Generates too many executables

Given
...

executable cachix-server
  import:         defaults
  main-is:        Main.hs
  other-modules:
    CollectMetrics
    DeleteNars
    ExpireTrials
    GarbageCollection
    GenElm
    Migrate
    SyncGitHub

  ...

It generates
cradle:
  stack:
    - path: "./exe/cachix-server/Main.hs"
      component: "cachix-server:exe:cachix-server"

    - path: "./exe/cachix-server/CollectMetrics.hs"
      component: "cachix-server:exe:cachix-server"

    - path: "./exe/cachix-server/DeleteNars.hs"
      component: "cachix-server:exe:cachix-server"

    - path: "./exe/cachix-server/ExpireTrials.hs"
      component: "cachix-server:exe:cachix-server"

    - path: "./exe/cachix-server/GarbageCollection.hs"
      component: "cachix-server:exe:cachix-server"

    - path: "./exe/cachix-server/GenElm.hs"
      component: "cachix-server:exe:cachix-server"

    - path: "./exe/cachix-server/Migrate.hs"
      component: "cachix-server:exe:cachix-server"

    - path: "./exe/cachix-server/SyncGitHub.hs"
      component: "cachix-server:exe:cachix-server"

This is an intentional design. It's required to support other-modules.
Are you experiencing any issues when using the generated config?

I do, but that's an upsteam ghcide issue then.

Not necessarily if you remove the extra options do the issues go away?

https://github.com/haskell/ghcide/issues/910


[ALLUXIO-1875] Add keyCount to PartitionInfo in key-value store

https://alluxio.atlassian.net/browse/ALLUXIO-1875

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9243/
Test PASSed.

@saltylin Thanks for fixing the TODO! Left some comments.

@uronce-cc Thanks for your review! I left a question in comments.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9269/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9291/
Test PASSed.

Merged build finished. Test PASSed.

@saltylin Thanks for fixing the TODO!
LGTM

@saltylin Overall this looks good. Thanks!
I just have one question. What is the reason for changing the count from a long to an int?

@gpang Because Index#keyCount() returns an int. One KeyValuePartitionWriter is associated with one index, thus the types of the two keyCounts should be same.

@saltylin That makes sense.
Do you think it would be better to make everything a long then? I ask because you use an int32 for the thrift definition, and I'm wondering if it would be better to just make that thrift definition int64?

@gpang Well, I think this is unnecessary. Let's assume the number of keys reaches the max value of int, which is 1 << 31 - 1. Every key-value pair is associated with a keyLength (int) and a valueLength (int), and just the length information will be about 1 << 34, that's 16GB, which should be far smaller than the real size since keys and values also consume bytes. Thus I think int is ok for the keyCount in a partition.

@apc999 can you take a look at the key count?
This PR looks good after that. Could you please resolve the merge conflicts? Thanks!

@gpang Updated!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9420/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9440/
Test PASSed.

@apc999 could you take a look?

LGTM

LGTM


ND4J: Nd4j and INDArray API cleanup

This PR identified a number of methods that are unused and/or untested...
https://github.com/eclipse/deeplearning4j/pull/7997
Some of them (setParams method for example) should probably simply be removed, to clean up the API. If we decide to keep them, they should be tested.
INDArray should probably be reviewed for this also.

We have talked about moving to namespace style APIs as well for Nd4j (#7970).  It might be worth setting up a pass to go over all of the Nd4j api stuff and set up reflection tests (like SkymindIO/deeplearning4j#42), all at once.

@rnett sounds good to me, a thorough pass to clean this up all at once would be a good idea.

Both were recently cleaned up by @RobAltena


Add CXX compiler to SCL-F Windows build

The swift-corelibs-foundation build now needs a CXX compiler because it implicitly builds swift-foundation-icu which is a C++ project. Adding the CXX compiler here ensures the build uses the appropriate compiler and resolves issues found during a toolchain build with swift-foundation enabled. Testing with this change and swift-foundation enabled shows that it actually begins the cmake build (unlike currently where cmake fails to detect a valid CXX compiler)

@swift-ci please smoke test

@swift-ci please build toolchain Windows platform


string-ends-with-failed-output.eo:29-31: Enable the...

The puzzle 218-e738c098 from #218 has to be resolved:
https://github.com/objectionary/eo-hamcrest/blob/298bd4884878fbdcd1bac2a2aa35b22603bf18e4/src/test/eo/org/eolang/hamcrest/failed-output/string-ends-with-failed-output.eo#L29-L31
The puzzle was created by levBagryansky on 17-Oct-23.
Estimate: 90 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@graur please pay attention to this new issue.


MySQL text field unique constraint fix

Fixing the problem of the last added migration to place a unique key constraint on key and device type. Converted key field into a varchar. Based on various information sources, key size could be up to 4096 bytes. However, MySQL only allows limited size on varchars on InnoDB. So this change was made based on an observation of the size of keys in my database which is around 160 chars.
Fixes #35

Out of 13957725 keys in my DB, the average length of the key was: 161 characters.