Ambiguous systems using with_default_system_setup(false)

Hi friends, I'm running into a small issue trying to use the system sets with a custom schedule.  Specifically, I'm running into problems using it with GGRS.  However, I can't reproduce this issue by running the custom_system_setup examples, so I'm a bit confused.
2 pairs of systems with conflicting data access have indeterminate execution order. Consider adding `before`, `after`, or `ambiguous_with` relationships between these:
 -- sync_simple_transforms (in sets RapierTransformPropagateSet, SyncBackend) and propagate_transforms (in sets PropagateTransformsSet, RapierTransformPropagateSet, SyncBackend)
    conflict on: ["bevy_transform::components::global_transform::GlobalTransform"]
 -- init_joints (in set SyncBackend) and apply_initial_rigid_body_impulses (in set SyncBackend)
    conflict on: ["bevy_rapier2d::plugin::context::RapierContext"]

The suggested fix seems pretty easy to declare some ambiguous_with in SyncBackend.  Since the comment for PropagateTransformsSet indicates it is meant to mark it to be ambiguous, I didn't want to rush to create a PR.  However, the second ambiguity with init_joints doesn't have a similar market set, so I'm confused how either approach is meant to work.  I also found similar code in the bevy engine, which its use is somewhat different and uses an ambiguous_with
Should/can both of these just be switched to ambiguous_with?  I am happy to work on the next PR towards this.
Are there any hints towards me just using this wrong?  My current code can be found here.
cc @devil-ira from original implementation

I believe this option is why: https://github.com/gschup/bevy_ggrs/blob/main/src/lib.rs#L187
I have submitted the aforementioned PR, which runs for me within the GGRSSchedule, the boxes examples, and the custom setup examples.

Thanks for getting back @devil-ira!  I think this tip may be exactly what's needed here and I'll see about toggling this in GGRS back to the default.


[Event Request] Codeunit::"Inventory Profile Offsetting" - Function SyncTransferDemandWithReqLine - OnAfterSetTransferReqLineFilters

We like to request the following Event:
Object: Codeunit::"Inventory Profile Offsetting"
Function: SyncTransferDemandWithReqLine
Publisher Event: OnAfterSetTransferReqLineFilters
Reason: To implement custom business logic
Where is the event placed:
local procedure SyncTransferDemandWithReqLine(var InventoryProfile: Record "Inventory Profile"; LocationCode: Code[10])
var
TransferReqLine: Record "Requisition Line";
begin
with TransferReqLine do begin
SetRange("Ref. Order Type", "Ref. Order Type"::Transfer);
SetRange("Ref. Order No.", InventoryProfile."Source ID");
SetRange("Ref. Line No.", InventoryProfile."Source Ref. No.");
SetRange("Transfer-from Code", InventoryProfile."Location Code");
SetRange("Location Code", LocationCode);
SetFilter("Action Message", '<>%1', "Action Message"::New);
OnAfterSetTransferReqLineFilters(TransferReqLine, InventoryProfile, LocationCode, CurrTemplateName, CurrWorksheetName);
if FindFirst then
TransferReqLineToInvProfiles(InventoryProfile, TransferReqLine);
end;
end;
Event Code:
[IntegrationEvent(false, false)]
local procedure OnAfterSetTransferReqLineFilters(var TransferReqLine: Record "Requisition Line"; var InventoryProfile: Record "Inventory Profile"; LocationCode: Code[10]; CurrTemplateName: Code[10]; CurrWorksheetName:

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Explicitly support promclient >= 0.4. Closes #39.

@TvoroG if you have a minute, could you please verify that this fixes your issue? Thank you :)

Yes, it will work! Thanks!


Investigate migration to GitHub Projects Beta for sprint management

Things to try out:

[ ] Separate views (eng/product/design)
[ ] Automation of ticket movement (in staging/closed/ready states/ in progress)
[ ] General Automation (ready for review/connecting PR + Issue/slack notifications)

https://docs.google.com/document/d/1tRyXg3LvBx3kjbnY-Rp4rILDLOC1e2oKU9rCHIHk7QE/edit?usp=sharing


Known DLL restriction is incorrect

The information on known dlls not being subject to dot-local redirection is not correct.  A test indicates that these too are dot-local redirected:
C:\Samples\dotlocal>dir /b
advapi32.dll
GdiPlus.dll
kernel32.dll
msvcrt.dll
oleaut32.dll
shell32.dll
ucrtbase.dll
user32.dll
write.exe
write.exe.local
C:\Samples\dotlocal>windbg.exe write.exe
...
00007ff9b3730000 00007ff9b3dfa000   SHELL32  C:\Samples\dotlocal\SHELL32.dll
00007ff9ceb60000 00007ff9cecf9000   USER32   C:\Samples\dotlocal\USER32.dll
00007ff9d2bf0000 00007ff9d2ced000   ucrtbase C:\Samples\dotlocal\ucrtbase.dll
00007ff9d7790000 00007ff9d7857000   OLEAUT32 C:\Samples\dotlocal\OLEAUT32.dll
00007ff9ddaf0000 00007ff9ddbab000   KERNEL32 C:\Samples\dotlocal\KERNEL32.DLL
00007ffa010a0000 00007ffa0114a000   advapi32 C:\Samples\dotlocal\advapi32.dll
00007ffa07f90000 00007ffa0802e000   msvcrt   C:\Samples\dotlocal\msvcrt.dll

@Scottj1s : Thanks for your contribution! The author(s) have been notified to review your proposed change.


feat: decouple callbacks from the train function

Decoupling callbacks preparation from the train function would help use train function without having callbacks being enforced by default (for instance aim callback) when using fms-hf-tuning repo as a module

@kmehant this can also be closed now....


Add /static/ to gitignore

What's this PR do?
Adds /static/ to gitignore for the server
Where should the reviewer start?
.gitignore
How should this be manually tested?
Any background context you want to provide?
This template was adapted ~stolen~ from Quickleft/Sprint.ly

Codecov Report

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

@@            Coverage Diff             @@
##             develop      #55   +/-   ##
==========================================
  Coverage           ?   96.36%
==========================================
  Files              ?       91
  Lines              ?     1649
  Branches           ?        0
==========================================
  Hits               ?     1589
  Misses             ?       60
  Partials           ?        0

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 d393e7b...54a9633. Read the comment docs.


feat(config): add config file for Homeseer HSM200

Same as Express Controls EZMultiPli (0x001e - ezmultipli.json) but with the Homeseer Technologies manufacturer ID 0x000c
fixes #1745

@zwave-js-bot rename commit
@blhoward2 that is a prime candidate for $import. Want to do it now or update later in one go?

I'd rather do it once otherwise I'll have to contend with the fact that a template file already exists there.


Possible bug with calculating lines.

Hi. We've have seen an issue with calculating lines for a dataset. In the code below, we tries to draw isolines around the area with value 15. But when running the code, we get lines around the wrong area. In most cases contourpy have worked fine for our rendering of maps, but in this case there seems to be something wrong. Any suggestions about why?
Here is some test code that prints the numpy arrays for the generated lines.
def test_contourpy():
    import numpy
    from PIL import Image, ImageDraw
    from contourpy import contour_generator

    z = numpy.array(
        [[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0],
         [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0],
         [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0],
         [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0],
         [0, 200, 15, 15, 15, 15, 15, 15, 15, 0, 0],
         [0, 200, 20, 15, 15, 15, 15, 15, 15, 0, 0],
         [0, 200, 200, 15, 15, 15, 15, 15, 0, 0, 0],
         [0, 0, 200, 200, 15, 15, 15, 15, 0, 0, 0],
         [0, 0, 0, 0, 15, 15, 15, 15, 0, 0, 0],
         [0, 0, 0, 0, 0, 15, 15, 0, 0, 0, 0],
         [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]]
        , dtype=numpy.float
    )

    im = Image.new('L', z.shape)
    draw = ImageDraw.Draw(im)
    cont_gen = contour_generator(z=z)
    lines = cont_gen.lines(15.0)
    xy = lines[0].flatten().tolist()
    draw.line(xy, fill=1)
    pix = numpy.array(im.getdata()).reshape(im.size[0], im.size[1])
    print("data:")
    print(z)
    print(pix)

And the output of the code (that generates lines around the area with 200, rather than 15):
data:
[[  0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.]
 [  0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.]
 [  0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.]
 [  0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.]
 [  0. 200.  15.  15.  15.  15.  15.  15.  15.   0.   0.]
 [  0. 200.  20.  15.  15.  15.  15.  15.  15.   0.   0.]
 [  0. 200. 200.  15.  15.  15.  15.  15.   0.   0.   0.]
 [  0.   0. 200. 200.  15.  15.  15.  15.   0.   0.   0.]
 [  0.   0.   0.   0.  15.  15.  15.  15.   0.   0.   0.]
 [  0.   0.   0.   0.   0.  15.  15.   0.   0.   0.   0.]
 [  0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.]]
[[0 0 0 0 0 0 0 0 0 0 0]
 [0 0 0 0 0 0 0 0 0 0 0]
 [0 0 0 0 0 0 0 0 0 0 0]
 [0 1 0 0 0 0 0 0 0 0 0]
 [1 0 1 0 0 0 0 0 0 0 0]
 [1 0 0 1 0 0 0 0 0 0 0]
 [1 1 0 1 0 0 0 0 0 0 0]
 [0 1 1 1 1 0 0 0 0 0 0]
 [0 0 0 0 0 0 0 0 0 0 0]
 [0 0 0 0 0 0 0 0 0 0 0]
 [0 0 0 0 0 0 0 0 0 0 0]]

There is no bug here. You have a number of grid points at the same z-level and you are asking for a contour line at exactly that z-level. What is expected to happen here? To show where the z-field exactly matches the contour level the algorithm would have to produce a filled polygon including all of those grid points, but you have asked for a contour line so producing a polygon is unacceptable.
What the algorithm does is classify each grid point as being above or below the contour level, and contour lines are produced for edges between grid points for which one end is above and one below the contour level. The full explanation is at https://contourpy.readthedocs.io/en/v1.0.7/description.html. Exactly matching the contour level has to be classified as either above or below the contour level, the convention is the latter. So in your example values from minus infinity up to and including 15.0 are classified as below the contour level, everything else is above. The output is consistent with this.
Another way of thinking about this is that as you cannot produce a polygon that spans the whole z=15 area then the contour line at that level needs to be either on the high side of the z=15 area (where z is increasing into adjacent grid points) or the low side (z decreasing into adjacent grid points). The convention is the former. (It could be "in the middle" of the z=15 area but "middle" is not well-defined in general).
In your example you can obtain what you would like to see using a contour level of e.g. 14.999.
The idea to take away here is that contouring is an interpolation algorithm for spatially-discretised continuous-z data. If your z-data is discretised too so that you have flat areas then you should never calculate contours at those z-levels or you will be disappointed. Probably if your z-data is discretised than contouring is a bad choice overall and you would be better using a polygonize algorithm such as in xarray-spatial or gdal.

Thank you for your feedback and explanation, much appreciated!
When trying I saw that cont_gen.lines(15 - sys.float_info.epsilon*5) seem to make it find the contour below 15. We will have to rethink a bit on how to handle this. Contourpy have worked really well for our other grib files that we have rendered, thank you for your effort in implementing this library.

I am glad you've found an acceptable workaround. I will close this issue as no action is required.


Intermittent FTP connection issues

Bug Report
System Information

Ubuntu 18.04 LTS
v<IP_ADDRESS>, built 06/03/2020 14:05
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

The FTP client returns authentication failed intermittently, It may connect and work several times then suddenly start failing, the AMP logs display the following
[19:31:28] [RemoteAMPAuth Activity] : Authentication attempt for user ftptest from <IP_ADDRESS>
[19:31:28] [Core Error]           : NullReferenceException
[19:31:28] [Core Error]           : [0] (NullReferenceException) : Object reference not set to an instance of an object
[19:31:28] [Core Error]           :   at GSMyAdmin.Authentication.RemoteAMPAuth.Authenticate (m.Http.IHttpRequest Request, String Username, String Password, String Token, Boolean TokenRequested, Collections.Generic.List`1[String]& Permissions, String& NewToken, GSMyAdmin.Authentication.UserInfoSummary& UserInfo)
[19:31:33] [RemoteAMPAuth Activity] : Authentication attempt for user ftptest from <IP_ADDRESS>

Reproduction

Setup a new AMP instance, enable FTP in FileManager.kvp.  Repeatedly connect and disconnect with FileZilla, I can't find any specific circumstances for when this happens, just that it eventually will.

This has been addressed in the latest update.


Add Licensing

I think it would be good to add licensing to project if someone would like implement your source code.

I put MIT licensing in the package.json, but I'll be sure to update it in README file and close this out.


Why is props.children kept always as an array when using overrides?

React usually keeps props.cihldren as a single string and not as an array if it contains just one element. Is there a reason? Problem is code consistency: You need to have a condition for components to render within and outside of <Markdown>'s overrides if you do sophisticated stuff with props.children.
Check the sandbox' console: https://codesandbox.io/s/reverent-bas-fp4m6

I solved it with the option below. Now, also props that previously came as arrays are also fixed to strings. Also rendering of MD within JSX keeps working.
Still wondering why an array is returned with the default implementation or is it a bug?
      ...
      option={{
        createElement: (type, props, children) =>
          React.createElement(
            type,
            props,
              children.length === 1 ? children[0] : children
            ),
      ...

I have the same issue. Trying to override native <a> link with next/link component but got the following error:

Warning: Failed prop type: Invalid prop children of type array supplied to NextLink, expected object.


cache ci slow steps

brings builds from ~4mins down to <1m for routine stuff, can be tweaked to be a bit more refined if needed

Love this improvments ❤


ModuleNotFoundError: No module named 'pyrealsense2' on running python script with Raspberry Pi 4B

| Camera Model                       | Intel RealSense depth camera D435i |
| Operating System & Version | Raspberry Pi OS 64-bit "Buster" |
| Platform                                 | Raspberry Pi 4B, 8GB, (SD card 32GB)  |
| Language                               | python 3.7 |
| RealSense-Viewer                  | 2.54.2 |
Hi,
I am trying to run a python script to get the depth data for object detection task with YOLO v8 using Raspberry Pi 4B and Intel RealSense depth camera D435i.
The realsense-viewer is working perfectly fine.
But when I am running a python script, I am getting an error:  ModuleNotFoundError: No module named 'pyrealsense2'
For installation, I have followed the following repository:
[https://github.com/datasith/Ai_Demos_RPi/wiki/Raspberry-Pi-4-and-Intel-RealSense-D435]
For Raspberry Pi OS 64-bit "Buster", I used this link:
[https://support.pishop.us/article/137-official-links-to-raspberry-pi-os-buster]
For given error, I checked the following links to resolve it but unfortunately it is still unresolved:

[https://github.com/IntelRealSense/librealsense/issues/12404]
[https://github.com/IntelRealSense/librealsense/issues/9865]
[https://support.intelrealsense.com/hc/en-us/community/posts/4402953910291-Help-Error-no-module-named-pyrealsense2]
[https://support.intelrealsense.com/hc/en-us/community/posts/360048396854-Raspberry-Pi-ModuleNotFoundError-No-module-named-pyrealsense2]

Actually I am not getting any .so file starting with  'pyrealsense2'.
I checked the folder librealsense > build > wrappers > python and other folders also.
I tried with Raspberry Pi OS 64-bit "Buster" and Raspberry Pi OS 32-bit "Buster" both but getting same error.
Kindly suggest me where is the problem??
Or, If you can tell what is the content of those .so file then I can create those files inside the same folder of my python script.
Thanks in advance!!
Saurabh

Hi @Saurabh-Sac  Please first try replacing import pyrealsense2 as rs in your Python script with the instruction below to see whether it enables the module to be found.
import pyrealsense2.pyrealsense2 as rs
If that does not work, you could try building the pyrealsense2 wrapper from a package instead of source code using the command pip install pyrealsense2 as pip install has had an Arm package for pyrealsense2 since SDK 2.54.1.

Hello @MartyG-RealSense,
I tried with both OS: Raspberry Pi OS 64-bit "Buster" and Raspberry Pi OS 32-bit "Buster" and python 3.7
The command import pyrealsense2.pyrealsense2 as rs is not working and getting the same error (ModuleNotFoundError: No module named 'pyrealsense2')
And the command pip install pyrealsense2, I am  getting error of "Could not find a version that satisfies the requirement pyrealsense2 (from versions: )
No matching distribution found for pyrealsense2"
And when I tried pip install pyrealsense this gave the error as shown in the attached screen shot.

If you have the folder librealsense > build > wrappers > python then this suggests that you were able to successfully build the pyrealsense2 wrapper from source code.
The .so files are not needed though if you instead set the PYTHONPATH with the command below in the terminal if you have not done so already.
export PYTHONPATH=$PYTHONPATH:/usr/local/lib

I tried this and still it is giving same error.
Should I go for Ubuntu OS? If YES, Kindly let me know the compatible Ubuntu and PYTHON version for Raspberry Pi 4B and Intel RealSense depth camera D435i.
And please send me all the related links for installation.
I am using Raspberry Pi Imager v1.8.5

Hello, I am currently having the same issue with RealSense lidar camera L515 and a raspberry pi 5, I also tried:
python3 -m pip install pyrealsense2-aarch64 --break-system-packages
and compiling from source as suggested in :
https://github.com/IntelRealSense/librealsense/blob/development/doc/installation_raspbian.md
but no luck with that either even though I can see in the output of make:
[ 65%] Built target pyrealsense2
Any idea on how to fix this?

@Saurabh-Sac  https://github.com/IntelRealSense/librealsense/issues/4375 has information about installing both the SDK and pyrealsense2 wrapper, although it is complicated.
An Ubuntu installation on Pi is likely to be easier than a Raspberry PI OS installation, though still potentially difficult. An Ubuntu install method that has a good history of success on Pi is the libuvc backend procedure at the link below.  However, the build script at this link does not include the Python wrapper flags in its CMake build instruction, so the script would have to bed edited to include them.
https://github.com/IntelRealSense/librealsense/blob/master/doc/libuvc_installation.md
https://github.com/IntelRealSense/librealsense/blob/master/scripts/libuvc_installation.sh

@TheSav1101  We have only had a couple of cases of RealSense users using Pi 5 at the time of writing this and both of them had difficulties so it is hard to say whether or not a build failure is due to a difference between Pi 4 and Pi 5, unfortunately.
It is possible (though often difficult) to install librealsense successfully on a Pi (the libuvc method has the best chance) but adding the Python wrapper further increases the difficulty of achieving a successful build.

@MartyG-RealSense The thing is that the c++ library actually works, I can take all the data I need with that, but to me it seems strange that even if the python wrapper compiles with no error it is not installed correctly :)

@TheSav1101  Raspberry Pi installation can often provide challenges for RealSense software unfortunately, whether it is the main SDK or one of its wrappers such as pyrealsense2 or ROS.

Hi @Saurabh-Sac and @TheSav1101  Do either of you require further assistance with this case, please?  Thanks!

Hello @MartyG-RealSense thank you for the help, but I think I will try and use the ros2 wrapper (should less of a hustle in my case) :)

Thanks very much @TheSav1101 for the update about your situation!

Hello @MartyG-RealSense,
Now I am able to run successfully with 64-bit "Buster".
But the processing of Intel RealSense depth camera D435i is very slow. I am using it for gesture recognition and object detection with MediaPipe and YOLOv8.
How can we speed up the processing of depth camera?

Hi @Saurabh-Sac  Another RealSense user with YOLOv8 also reported slow performance at https://github.com/IntelRealSense/librealsense/issues/12136#issuecomment-1708943802 using a different type of single-board computer.
The kind of processing-intensive recognition and detection activities that are being performed are probably primarily responsible for the slowdown.
As mentioned at https://github.com/IntelRealSense/librealsense/issues/12136#issuecomment-1709591007 you may be able to achieve some extra performance if librealsense was built from source code with support for use of multiple CPU cores enabled.  If you already have a working librealsense installation with Buster then it may not be worth the risk of installing librealsense again though in case it becomes broken.

By the way @MartyG-RealSense I think that I might have figured out the problem I had.
Do you know if there is an "upper limit" to the python version when building from source? Because I was using python3.11, but maybe I could get better results using an older version...
I am posting this in case somebody has my same issue :smile:

@TheSav1101  I am not aware of an upper limit when building the pyrealsense2 wrapper from source.

Hello @MartyG-RealSense,
Now I am able to run successfully with 64-bit "Buster".
But the processing of Intel RealSense depth camera D435i is very slow. I am using it for gesture recognition and object detection with MediaPipe and YOLOv8.
How can we speed up the processing of depth camera?

Hi @Saurabh-Sac
We have the same problem. How did you proceed in solving this problem? Can you explain it detailly.
Thanks

@TheSav1101  Do you require further assistance with this case, please?  Thanks!
Hi @erayak  Were you able to achieve a solution?

Thank you @MartyG-RealSense but I resolved all my issues :)

Hello @MartyG-RealSense,
Now I am able to run successfully with 64-bit "Buster".
But the processing of Intel RealSense depth camera D435i is very slow. I am using it for gesture recognition and object detection with MediaPipe and YOLOv8.
How can we speed up the processing of depth camera?

Hi @Saurabh-Sac
We have the same problem with raspbian we couldn't install pyrealsense2. How did you proceed in solving this problem? Can you explain it detailly.
Thanks

Hello @erayak
Actually It was just by trial and error. It was not straight forward.
I got one path related error and one cmake (package missing) error during pyrealsense2 build.
For installation, I have followed the following repository:
[https://github.com/datasith/Ai_Demos_RPi/wiki/Raspberry-Pi-4-and-Intel-RealSense-D435]
For Raspberry Pi OS 64-bit "Buster", I used this link:
[https://support.pishop.us/article/137-official-links-to-raspberry-pi-os-buster]

Hello @MartyG-RealSense
Can I use laptop (in place of Raspberry Pi) to process the Intel RealSense depth camera D435i data (means object position (x, y, z)) and Raspberry Pi to only control (send PWM signal for servo motors) a robotic arm to pick an object using its inverse kinematics simultaneously ?

@Saurabh-Sac   Something like the RealSense inverse-kinematics robot arm project at https://github.com/silvery107/silvery-armlab-f22 ?

Hi
yes...something like this. I want to pick the different objects and for object detection I am using YOLO algorithm.
Actually I want to run....

RealSense object detection data .py file using laptop (not Raspberry Pi)  [due to slow processing speed of depth camera with Raspberry Pi]
and
After getting that data (object position), I want to use Raspberry Pi to run the servo motors of robotic arm.

Can we do these two things simultaneously?

I belive you can do all of this with ROS @

Hi
yes...something like this. I want to pick the different objects and for object detection I am using YOLO algorithm.
Actually I want to run....

RealSense object detection data .py file using laptop (not Raspberry Pi)  [due to slow processing speed of depth camera with Raspberry Pi]
and
After getting that data (object position), I want to use Raspberry Pi to run the servo motors of robotic arm.

Can we do these two things simultaneously?

I belive you can do all of this with ROS @Saurabh-Sac

Hello @TheSav1101
But I want to directly implement on hardware setup for robotic arm (without any simulator like ROS & Gazebo)

I mean, you can use ros to implement the lan communication between the two devices using messages and have the raspberry node handle the movement with your own code without using simulations, If I understood correctly that's exactly what you need, isn't it?

yes...Okay.
Is there any other interface or platform for communication? (Like MATLAB tool or other protocol)

Hi @Saurabh-Sac  Have you been able to solve your issue or do you require further assistance, please?  Thanks!

Hi @MartyG-RealSense I shifted to Jetson Nano now. After resolving many issues it is working fine now.
I believe there should be a single line command to install pyrealsense2 for single-board computer.
Because the provided installation document generate issues in any kind of single-board computer whether it is Arduino, Raspberry Pi or Jetson.
Thanks!!

The pip install pyrealsense2 single line installation command covers most situations for x86/x64 and Arm architecture computers, though it admittedly does not work for every single-board computer (such as Raspberry Pi) and it also takes time to add support to the pip packages for the newest Python 3 versions.  For situations where pip install does not work or a recent Python version is being used, source-code install can be used.

Hi @Saurabh-Sac  Do you require further assistance with this case, please?  Thanks!

Hello @MartyG-RealSense We can close this issue.
Thanks everyone...!!

Thanks very much @Saurabh-Sac for the update!


[BACKEND] Migrate new Keycloak Portal URL

As a TraceX Admin,
I want to migrate to the new keycloak portal url,
so that the application is running again.
Hints / Details

Mail from Julia Jeroch:
Dear all,
please note that we started to reset the Integration Env. to get ready for the 24.05. E2E tests.
As a result, access to the portal and connected applications will be extremely limited until the E2E tests begin.
I understand that this may have a significant impact on many of you. To mitigate the inconvenience, we enabled a special support function. If you still need/want to perform actions such as EDC communications (version < 0.7 RC1), test runs, etc. using previously used users (both technical and functional), you can proceed by switching your connector and/or app instances to the following Keycloak token URL:
Token URL: https://centralidp-v2.int.demo.catena-x.net/auth/realms/CX-Central/protocol/openid-connect/token
Please note that this IDP instance is our previous INT Keycloak instance, which is now available under a new endpoint path. All user accounts and service accounts are still accessible with your existing credentials.
Greetings,
Julia
Acceptance Criteria

all environments are using the intermediate keycloak url
all environments are working again

Out of Scope

...

Duplicate of https://github.com/eclipse-tractusx/traceability-foss/issues/855


[MOB-132] BE - POST Login phase 2

Pull Request Checklist

[x] Title
[x] MOB US' Link
[ ] Link PR to US
[x] Add Tests
[x] Add ENV Variables and Validations
[ ] Documentation Updates
[x] Swagger Updates

Testing Guide
Add LOGIN_ATTEMPTS_TTL to .env.
Try login with an unverified user and an active user 5 times to see if it is blocked.
curl --location 'http://localhost:3000/v1/account/login' \
--header 'Content-Type: application/json' \
--data-raw '{
    "email"<EMAIL_ADDRESS>    "password": "SuperSecret1"
}'

It could be fine to document the new errors thrown in the login service


add --outSuffix compiler option

Search Terms
out, suffix
Suggestion
add --outSuffix compiler option
Use Cases
tsc --module es6 --outSuffix mjs
Examples
Imagine I have a package with file tree:
my_package/
├── a.ts
├── b.ts
├── index.ts
├── package.json
└── tsconfig.json

then I run:
> tsc --module es6 --outSuffix mjs
> tsc --module commonjs --outSuffix js

then the file tree will be:
my_package/
├── a.d.ts
├── a.js
├── a.mjs
├── a.ts
├── b.d.ts
├── b.js
├── b.mjs
├── b.ts
├── index.d.ts
├── index.js
├── index.mjs
├── index.ts
├── package.json
└── tsconfig.json

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Oh, sorry, before I didn't search keyword mjs... Now I find these issues:
https://github.com/microsoft/TypeScript/issues/30076
https://github.com/Microsoft/TypeScript/issues/18442


working interrupts on windows

This should enable interrupts on Windows that don't just kill the process.
I'd be grateful if someone could test this (on Linux/OS X too, so that I know I didn't mess anything up with that part).
Installation instructions can be found here, though only the atom-julia-client part is relevant.
@one-more-minute Sending the interrupt to the powershell-wrapper isn't ideal yet for a couple of reasons:

It should be possible to jsut send it via node.js' internal functions without spawning another powershell instance, but I haven't figured that out yet.
The port used for communication is hardcoded. Could use something like this to find a free port, though.

@Varanas Just tested on OS X. Works fine!

This is a long-standing issue and I think lots of people will appreciate the improvement ;) As far as the first bullet goes I think it should be pretty easy to connect and send the signal with the net functions, and it might make maintaining this stuff a bit easier. For the second I'm not too worried; we can fix that easily enough if it causes problems in future.
We might need to investigate the problems that happen when interrupting an idle client, but this is a strict improvement even without that fixed.

I tried working with the net-functions (that's why they are imported), but something like
client = net.connect(port: 26992, ->
            console.log('connected to server!')
            client.write("SIGINT"))

doesn't work. I get the connected to server, but my powershell script doesn't receive the SIGINT.

At a guess there's some buffering going on. There doesn't seem to be an explicit flush method, but you could try end() or perhaps some incantation of setNoDelay.

Indeed. Thanks for the pointer, @one-more-minute, the implementation is a bit nicer now.
@nilshg, could you try this new commit if you find the time? A bit of testing by someone else than me would be nice ;)

Will try to give it a go tomorrow if possible. I'm running a lot of hand-coded minimum distance estimators at the moment which often take hours to converge, and usually I find a stupid typo in the code after I've started running it, so interrupt would come in handy :)

Alright, I've just tested this on another Windows machine, squashed some bugs, refactored a bit and got rid of the debug echos. I hope this doesn't break anything under Linux ;)

Sweet!
How about a news item to show off your work? ;)

Heh, why not ;)
Where'd that go? NEWS.md?

Changelog.md will do it

ah, I hadn't noticed that version 0.2 wasn't tagged yet


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2020-10-28T22:34:34+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.8
7408204624284ff542c538de92f5eeed597d8d43
link
/test unit-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


fix some typos in comments/code for improving clarity and consistency

This commit provides some minor edits in the cl-waffe2 code and docs. It should help to enhance the clarity and consistency throughout the code base as well as the documentation.

Since I am known as a Typo Expert, this PR is very helpful! Thanks!


NaN occurs when building with GaussianNoise, GaussianNoise and Dense

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):a GNU/Linux system with Linux kernel 4.15.0 on 1 6-core 3.60GHz Intel Core CPU i7-6850K with 64 GB RAM equipped with a NVIDIA Corporation GP102 GPUs
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):source
TensorFlow version (use command below):tensorflow2.1.0-GPU
Python version:3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I got the NaN as loss when I tried to train my model with GaussianNoise, GaussianNoise and Dense
Describe the expected behavior
The loss should be a number
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
batch_size = 110
epochs = 128
num_classes = 10
import os
model_name = 'model.h5'
import tensorflow.keras as keras
(x_train, y_train), (x_test, y_test) = keras.datasets.mnist.load_data()
img_rows, img_cols = x_train.shape[1], x_train.shape[2]

x_train = x_train.reshape(x_train.shape[0], img_rows, img_cols, 1)
x_test = x_test.reshape(x_test.shape[0], img_rows, img_cols, 1)
input_shape = (img_rows, img_cols, 1)
x_train = x_train.astype('float32')
x_test = x_test.astype('float32')
x_train /= 255
x_test /= 255
y_train = keras.utils.to_categorical(y_train, num_classes)
y_test = keras.utils.to_categorical(y_test, num_classes)
model = keras.models.Sequential()
model.add(keras.layers.GaussianNoise(stddev=0.7498748441096037))

model.add(keras.layers.Flatten())
model.add(keras.layers.Dense(num_classes, activation='relu'))
model.compile(loss=keras.losses.categorical_crossentropy,optimizer=keras.optimizers.Adadelta(), metrics=['accuracy'])
model.fit(x_train, y_train, batch_size=batch_size, epochs=epochs, verbose=1, validation_data=(x_test, y_test))
model.save(model_name)

Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
Epoch 1/128

  110/60000 [..............................] - ETA: 43:48 - loss: 9.9702 - accuracy: 0.0636
  660/60000 [..............................] - ETA: 7:18 - loss: 9.6680 - accuracy: 0.0848
 1100/60000 [..............................] - ETA: 4:24 - loss: 9.4182 - accuracy: 0.0991
 1540/60000 [..............................] - ETA: 3:10 - loss: nan - accuracy: 0.1019
 1760/60000 [..............................] - ETA: 2:47 - loss: nan - accuracy: 0.1000
 2200/60000 [>.............................] - ETA: 2:14 - loss: nan - accuracy: 0.0995
 2530/60000 [>.............................] - ETA: 1:58 - loss: nan - accuracy: 0.0992
 3080/60000 [>.............................] - ETA: 1:37 - loss: nan - accuracy: 0.1042
 3630/60000 [>.............................] - ETA: 1:23 - loss: nan - accuracy: 0.1047

Was able to reproduce the issue with TF v2.3 and TF-nightly. Please find the gist of it here. Thanks!

Hi @bugreporter450, I think this is because you're using relu as the activation on the output layer. Try this instead:
model.add(keras.layers.Dense(num_classes, activation='softmax'))


feat(samples): add samples for analyzeIamPolicy and analyzeIamPolicyLongrunning

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Question about linting:
I tried "npm run fix" and "npm run lint -- --fix", both of them returned error "missing script: fix". Where can I get the fix script?

@donghez-google it sounds like you're potentially running npm run fix in the wrong folder, I'm guessing you're in ./samples, and you want to run int he top level folder.
If you run npm run lint in the root folder, it will install and link dependencies for you.
You can then run npm run fix.

@donghez-google it sounds like you're potentially running npm run fix in the wrong folder, I'm guessing you're in ./samples, and you want to run int he top level folder.
If you run npm run lint in the root folder, it will install and link dependencies for you.
You can then run npm run fix.

I ran 'npm run lint' in the root folder but got another error(https://paste.googleplex.com/5000366669168640). I think there's some issues in package.json script, but no idea how to fix it.


fix(build): resolve bugs change crates and binary name

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

src/main.rs
0.00% <0.00%> (ø)

:loudspeaker: Thoughts on this report? Let us know!.


Common init base classes (Swift)

The Swift side of #23.
@vokal/ios-developers variations of this code have been used in at least a couple client projects, but hadn't made it back here yet. I still need to update the Obj-C template to add the same functionality.

Forgot I already had a PR on the branch when I pushed to it. Hope I can sneak that gem update in.


libkb: keep unlocked paper key in memory for rekeying purposes

make a cleaner process to clear it out after an hour of inactivity
introduce a global fake clock for testing purposes
unfortunately, make LoginState more complicated so our tests can know
when cleanup completes.

r? @patrickxb

Took your suggestions and simplified the test lots.  Ready for another look. Thanks @patrickxb

Looks good.
Any thoughts about the Now() and After() in GlobalContext?

Forgot about that.  Will fix it too.
On Friday, January 8, 2016, Patrick Crosby<EMAIL_ADDRESS>wrote:

Looks good.
Any thoughts about the Now() and After() in GlobalContext?
—
Reply to this email directly or view it on GitHub
https://github.com/keybase/client/pull/1654#issuecomment-170039045.

Just GetClock(), is that ok?

LGTM


Don't throw StateError when listing core devices during tool shutdown

The previous attempt at this fix was assuming that the tool's file system was a LocalFileSystem, but in reality it's a LocalFileSystem wrapped in an ErrorHandlingFileSystem. This change takes this into account.
Fixes https://github.com/flutter/flutter/issues/160082
Fixes https://github.com/flutter/flutter/issues/156962

This is failing:
00:15 +2067 ~2 -1: test/general.shard/ios/core_devices_test.dart: Core Device Control list devices Handles file system disposal [E]
  PathNotFoundException: Creation of temporary directory failed, path = '/.tmp_rand0/core_devices.' (OS Error: No such file or directory, errno = 2)
  dart:io                                                       _Directory.createTempSync
  package:file/src/forwarding/forwarding_directory.dart 28:21   ForwardingDirectory.createTempSync
  package:flutter_tools/src/base/error_handling_io.dart 459:27  ErrorHandlingDirectory.createTempSync.<fn>
  package:flutter_tools/src/base/error_handling_io.dart 599:14  _runSync
  package:flutter_tools/src/base/error_handling_io.dart 458:12  ErrorHandlingDirectory.createTempSync
  package:flutter_tools/src/ios/core_devices.dart 65:69         IOSCoreDeviceControl._listCoreDevices
  package:flutter_tools/src/ios/core_devices.dart 136:48        IOSCoreDeviceControl.getCoreDevices
  test/general.shard/ios/core_devices_test.dart 1448:69         main.<fn>.<fn>.<fn>
  dart:async                                                    runZoned
  test/src/common.dart 214:14                                   testWithoutContext.<fn>
  test/src/common.dart 183:18                                   test.<fn>

though I can't immediately understand why

Must have just pushed the fix as you left your comment :-)

This was failing with
00:57 +2513 ~2 -1: test/general.shard\ios\core_devices_test.dart: Core Device Control list devices Handles file system disposal [E]
  Expected: [
              'xcrun',
              'devicectl',
              'list',
              'devices',
              '--timeout',
              '5',
              '--json-output',
              '/.tmp_rand0/core_devices.rand0/core_device_list.json'
            ]
    Actual: [
              'xcrun',
              'devicectl',
              'list',
              'devices',
              '--timeout',
              '5',
              '--json-output',
              '/.tmp_rand0/core_devices.rand0\\core_device_list.json'
            ]
     Which: at location [7] is '/.tmp_rand0/core_devices.rand0\\core_device_list.json' instead of '/.tmp_rand0/core_devices.rand0/core_device_list.json'

One solution is to use a regex pattern to match, but I instead just pushed
fake LocaleFileSystem implementation rather than extending it


Huge html report size

We have an issue when run xcodebuild command with flag -resultBundlePath (xcodebuild does not generate xcactivitylog without flag). Xcode log each step, create folder for example. After we call
xclogparser parse --project <path_to_proj> --reporter html
xclogparser generate html file for each of this steps. As a result xclogparser create 19741 files in directory each has 15kb size. Result report folder has 1.3Gb. If we build project usign Xcode we get small size report.
Could you please tell which flags we can use to reduce xcactivitylog or ignore some steps by xclogparser htm report?
Steps to reproduce:
Build using xcodebuild:
xcodebuild -workspace MyProject.xcworkspace -scheme MyScheme -configuration Debug -destination 'generic/platform=iOS' -resultBundlePath 'resultbundle.result' clean build
Run parse:
xclogparser parse --project MyProject --reporter html --rootOutput Builds/reports
As a result xclogparser create 19741 files in report directory which has 1.3Gb size
Context:
MacOS: Sonoma 14.1.2
Device: MacbookPro M1
Xcode: 15.1
XCLogParser installation: 0.2.38 (Homebrew)

As i can see in build time line Xcode logs a lot of small operations - make directory. Does xclogparcer has options to skip it and do not create step report file for it?


Update travis setup to fix CI runs

Issue
It looks like the bin/setup script is causing travis to switch to using system ruby instead of rvm's version. Possibly set -e is what's causing the issue.
$ rvm use 2.5.5 --install --binary --fuzzy
...
$ ruby --version
ruby 2.5.5p157 (2019-03-15 revision 67260) [x86_64-linux]
...
$ bin/setup
Fetching gem metadata from https://rubygems.org/............
zeitwerk-2.2.2 requires ruby version >= 2.4.4, which is incompatible with the
current version, ruby 2.3.1p112
The command "bin/setup" failed and exited with 5 during .

https://github.com/customink/lamby/blob/master/bin/setup#L1-L6
Fix
This fix is a hack to try and get the travis setup run properly. It's duplicating the required setup commands in bin/setup in the .travis.yml.
The alternative would be to add conditionals in the bin/setup script to handle setup in the CI environment vs. a dev or deployed environment. I don't currently know enough about travis/rvm to offer that solution.
This fix is untested. Hopefully it will just work :tm:.

This is wrong!
It seems like bundler is the issue. Is it using the system bundler instead of the rvm version?

Hmmm, not sure. From a high level, I can share that I want to move this project over to GitHub actions using the LambCI containers (that AWS SAM) uses. I've not done a blog post on this yet but I've been using those containers (which mimic the Lambda runtime) "build" variants (https://github.com/lambci/docker-lambda) as the basis for a development and test env runner. Doing so means removing all the rbenv, pipenv, and nodenv bootstrap/setup struggles and puts everyone on the same platform. Essentially making CI/CD a commodity that can run on any platoform... GitHub actions being the easiest and most integrated for me lately.
All that said, please do look into exploring making Travis better in the meantime :) and when it goes green I'll merge it in.

The culprit seems to have been the sudo gem install bundler command in the travis script. Removing that allowed travis to get to the bin/test step, which failed for Rails related reasons (missing manifest.js).
Hopefully adding one will allow the tests to run successfully. 🤞

I think the current test failures are caused by missing credentials (maybe from the CI). I'd have to take a longer look and get tests running locally to figure out what's missing.

@stevendaniels So work done in https://github.com/customink/lamby/pull/47 any thoughts?


new version tries to require node modules...

I migrated my working node.js app to a new server today. The app spawns casperjs processes to do stuff.
Problem :
It doesn't run anymore
Why?
output casperjs --version from any folder but the one where my node app is installed :
root@xxxxx:/var# casperjs --version
1.1.0-beta3

now from the folder where my node app is installed :
root@xxxxx:/var/nodeapp# casperjs --version
Error: Cannot find module 'sys'

  phantomjs://bootstrap.js:289
  phantomjs://bootstrap.js:254 in require
  /var/nodeapp/node_modules/system/engines/node/system.js:8
  /var/nodeapp/node_modules/system/engines/node/system.js:45
1.1.0-beta3

Isn't that weird? :D Thanks for any insights
Edit : Tried uninstalling casper, then reinstalling from npm (I used github before). Same error.

Seems that this needs some re-ordering of the bootstrap code. Maybe it could help to not require in the outer part (bootstrap.js, last line) of the init code, but use the patched require after it is set, to nail down the path preferences.
Investigating along that line.

Archiving issue using ARCHIVE label as CasperJS, like PhantomJS, is not actively maintained for now.


WebPack issue

I use color picker with webpack + the following code in my js code:
import AColorPicker from 'a-color-picker'
But I get an error in my console:
AColorPicker is not defined

I got the same error, however it does work if you import it like this:
import * as AColorPicker from "a-color-picker";


Fixes #268

If there is a subscriber, but they do not have a language and the subscription does have a
language we assign that language to the user.

lgtm


Pressing spacebar does not select focused checkbox in Firefox

What is your Scenario?
Couple of our tests have started to fail recently due to the following issue. We updated testcafe from 1.15.0 to 1.18.4 just a few days ago so it might be related.
What is the Current behavior?
In Firefox when you use pressKey('space') method it will not select/unselect checkbox with the focus. It seems to behave correctly in other browsers.
What is the Expected behavior?
pressKey('space') method should work correctly regardless the browser.
What is your public website URL? (or attach your complete example)
https://devexpress.github.io/testcafe/example/
What is your TestCafe test code?
fixture`Testcafe Firefox Issue`
    .page`https://devexpress.github.io/testcafe/example/`;
test('Should check/uncheck checkbox on space key press', async () => {
    const testCheckbox = Selector('#tried-test-cafe');
    await t
        .click(testCheckbox)
        .expect(testCheckbox.checked).ok()
        .expect(testCheckbox.focused).ok()
        .pressKey('space')
        .expect(testCheckbox.checked).notOk();
});

Your complete configuration file
No response
Your complete test report
No response
Screenshots
No response
Steps to Reproduce

Set focus to checkbox type input.
call pressKey('space') method.
Checkbox value does not change.

TestCafe version
1.18.4
Node.js version
16.14.2
Command-line arguments
testcafe firefox
Browser name(s) and version(s)
Firefox 99.0
Platform(s) and version(s)
No response
Other
No response

Thank you for your feedback. I reproduced this behavior. We will research it and update this thread once we have any news.


vlc rtsp on nixos does not work

On the same machine I can use /usr/bin/vlc (Ubuntu 14.04) to access the following URL (and see a camera feed), but using ~/.nix-profile/bin/vlc I get the bottom message.
<EMAIL_ADDRESS>
main input error: open of<EMAIL_ADDRESS>failed
Ubuntu version which works: VLC media player 2.1.4 Rincewind (revision 2.1.4-0-g2a072be)

NixOS version which is broken: VLC media player 2.1.5 Rincewind (revision 2.1.4-49-gdab6cb5)
The version on my native NixOS is the same: VLC media player 2.1.5 Rincewind (revision 2.1.4-49-gdab6cb5)

Can you run strace on that? I'm guessing there is a missing library that needs to be given to vlc at build time.

@bjornfor
strace was mostly useless, but the -l option proved useful:
diff a b
2a3,5
>   access_realrtsp        Real RTSP
>   live555                RTP/RTSP/SDP demuxer (using Live555)
>   live555                RTSP/RTP access and demux

TLDR; the live555 library is missing and I don't know what access_realrtsp does.

live555 isn't in nixpkgs (yet). Maybe you could package it?

@bjornfor I packaged live555 (at least, something ended up in the nix store) (with that name in some other nixpkgs root R), and I modified the expression for vlc (default.nix) to also depend on live555, but how do I actually make it such that this ties in with the other expression which depends on vlc. I want that vlc to really be the modified vlc.
I'd prefer you to be very specific.

In particular, I don't get why the following doesn't work in configuration.nix:
  packageOverrides = pkgs: rec {
         # instead of the standard vlc, use mine
         vlc = unstable.vlc;
         };

Maybe unstable isn't what you think it is? Other than that it looks correct.
Try this:
    packageOverrides = pkgs: {
      vlc = (import /path/to/modified-nixpkgs/default.nix { }).vlc;
    };

@bjornfor It says that packageOverrides is not an attribute of the body in configuration.nix. Why? I have no idea. I did manage to start a build of vlc, which probably means that unstable.vlc (which has my modifications) is running.

Building vlc goes at glacial speed, btw. Why doesn't it use all of my cores?

Ah, you need to put the packageOverrides thing in the nixpkgs.config option. Paste this snippet into configuration.nix:
  nixpkgs.config = {
    packageOverrides = pkgs: {
      vlc = (import /path/to/modified-nixpkgs/default.nix { }).vlc;
    };
  };

Ref. http://nixos.org/nixos/manual/ch-options.html#opt-nixpkgs.config

Per-package building is single-threaded by default, because so many upstream packages fail randomly with parallel builds. Once you build more packages in parallell, nix will use all your cores.
If you want to build vlc itself with all your cores, set enableParallelBuilding = true; in the vlc expression.
There is a ML thread or github issue (don't remember) about making that flag a global option too, so that adventurous users can get max speed (at the expense of possibly random failures).

The expression already contains enableParallelBuilding = true (the 14.12 one already did).

Then I don't know. Try adding "set -x" in a preConfigure phase or something and look at what commands are being run.

/etc/nix/nix.conf must contain a setting like
build-cores = 0  # auto-detect number of cores

for parallel building to take any effect.

This information is already contained in hardware-configuration.nix (via nix.maxJobs). Why does build-cores even exist?

You can also override this per-build by the --cores command-line parameter.

Ah, now I learned something new! For one, I thought the default was auto-detection of build-cores, but it is 1 by default, according to "man configuration.nix". The documentation doesn't mention the special '0' value though.

The docs for --cores and build-cores do mention what 0 means (buildCores doesn't). Perhaps such parts of docs should be linked somehow, so we don't have to duplicate the information.

I've extended the docs in ea94a6d.

By the way, can we close this ticket? Or is there still an open question?

I think we should close it when live555 support is in. (I think I have a working version, but I don't have access to it for some time. )

Thank you for the cooperation.


Load balancer does not have available server for client

I am evaluating Spring cloud for my current project.
Currently I am trying my hands at eureka with zuul using Spring.
code repo: https://github.com/bhargodevarya/SpringEurekaServer
I created a simple spring boot application that creates an HTTP url that returns some data.
I registered this application in eureka.
I can see that my service is registered properly in the eureka discovery server.
Also, I can see the zuul service registered there.
In my Zuul logs I can see the following:
2018-09-17 00:45:35.690  INFO 23421 --- [nio-8762-exec-1] o.s.c.n.zuul.web.ZuulHandlerMapping      : Mapped URL path [/userservice/**] onto handler of type [class org.springframework.cloud.netflix.zuul.web.ZuulController]
but when it tries to get the information I get the following log:
2018-09-17 00:45:36.266  INFO 23421 --- [nio-8762-exec-1] c.netflix.loadbalancer.BaseLoadBalancer  : Client: userservice instantiated a LoadBalancer: DynamicServerListLoadBalancer:{NFLoadBalancer:name=userservice,current list of Servers=[],Load balancer stats=Zone stats: {},Server stats: []}ServerList:null
2018-09-17 00:45:36.279  INFO 23421 --- [nio-8762-exec-1] c.n.l.DynamicServerListLoadBalancer      : Using serverListUpdater PollingServerListUpdater
2018-09-17 00:45:36.291  INFO 23421 --- [nio-8762-exec-1] c.n.l.DynamicServerListLoadBalancer      : DynamicServerListLoadBalancer for client userservice initialized: DynamicServerListLoadBalancer:{NFLoadBalancer:name=userservice,current list of Servers=[],Load balancer stats=Zone stats: {},Server stats: []}ServerList:org.springframework.cloud.netflix.ribbon.eureka.DomainExtractingServerList@56e94d19

but when I hit the url http://localhost:8762/userservice/user/setup in my browser, I get the below exception
com.netflix.zuul.exception.ZuulException: Forwarding error
at org.springframework.cloud.netflix.zuul.filters.route.RibbonRoutingFilter.handleException(RibbonRoutingFilter.java:189) ~[spring-cloud-netflix-zuul-2.0.1.RELEASE.jar:2.0.1.RELEASE]
at org.springframework.cloud.netflix.zuul.filters.route.RibbonRoutingFilter.forward(RibbonRoutingFilter.java:164) ~[spring-cloud-netflix-zuul-2.0.1.RELEASE.jar:2.0.1.RELEASE]
at org.springframework.cloud.netflix.zuul.filters.route.RibbonRoutingFilter.run(RibbonRoutingFilter.java:112) ~[spring-cloud-netflix-zuul-2.0.1.RELEASE.jar:2.0.1.RELEASE]
......
......
Caused by: com.netflix.client.ClientException: Load balancer does not have available server for client: userservice
at com.netflix.loadbalancer.LoadBalancerContext.getServerFromLoadBalancer(LoadBalancerContext.java:483) ~[ribbon-loadbalancer-2.2.5.jar:2.2.5]
I have given my code repo link above.
Can some one please have a look ?

Please learn how to format code on GitHub.
Your "user service" application does not have spring.application.name set.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.

Apologies for the late comment, but what Ryan mentioned fixed my issue. Thanks a lot.


Sixel Termination Places Cursor in the Wrong Spot

Windows Terminal version
1.22.2912.0
Windows build number
10.0.19045.5011
Other Software
Ordinary windows CMD prompt, but I can reproduce with mingw and powershell as well.
Steps to reproduce
Display the following output from a file using type or cat
        #0;2;0;0;0#1;2;100;100;0
        #1~~@@vv@@~~@@!20~#0!20@#1!30~--\a a a

This displays some text, followed by a wide band of pixels with two pixel "new lines", followed by some text.
Display the following output from a file using type or cat
        #0;2;0;0;0#1;2;100;100;0
        #1~~@@vv@@~~@@!20~#0!20@#1!30~\a a a

This is the same display with no pixel new lines.
Note that in the first example, the text begins on the following line.  In the second example, the text covers most of the image.
Expected Behavior
In the VT100.net explanation of sixels, it says the following:

When sixel display mode is enabled[...] When sixel mode is exited, the text cursor is set to the current sixel cursor position.
[...]
When sixel scrolling is disabled[...] When sixel mode is exited, the text cursor does not change from the position it was in when sixel mode was entered.

Actual Behavior
You can see from the examples above, that the windows terminal does not display either of these two behaviors.  Instead, the text cursor seems to end up at where the sixel cursor was at the last time it experienced a carriage return. Personally, I would expect the text cursor in the second example to end up at the far right edge of the image instead of the far left edge.

FYI private mode 8452 has no effect on this behavior

Based on information @j4james posted here: https://github.com/hackerb9/vt340test/blob/main/j4james/cursor_position.png
I think that this report is erroneous and that the behavior here is correct. I think maybe what I actually want is private mode 8452 implemented

Actually I take it back- because the extra newlines at the end shouldn't matter, should they?  THey don't seem to matter in the above image and they don't matter on wezterm, for instance.  So I think there's a bug but it's the opposite of what I thought:
The first sixel image should act the same way as the second sixel image.

Actually I take it back- because the extra newlines at the end shouldn't matter, should they?

They do matter. In much the same way that a newline takes up space in a text file, even if there is no text on that line, a graphic newline will take up space in a Sixel image, even if there are no pixels on that line.
It's complicated by the fact that the Sixel lines are a fraction of a text line, so the final text cursor position is often not aligned with the final Sixel position. This means you can sometimes add a graphic newline which appear to have no effect, because it's "rounding down" to the same text row as before.

Okay, got it


[oracle] Improve cache handling

What does this PR do?
Improves cache handling.
Motivation
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1025635f6]

goroutine 527 [running]:
github.com/DataDog/datadog-agent/pkg/collector/corechecks/oracle-dbm.(*Check).StatementMetrics(0xc000c56900)
  /Users/nenad.noveljic/go/src/github.com/DataDog/datadog-agent/pkg/collector/corechecks/oracle-dbm/statements.go:741 +0x2e96
github.com/DataDog/datadog-agent/pkg/collector/corechecks/oracle-dbm.(*Check).Run(0xc000c56900)
  /Users/nenad.noveljic/go/src/github.com/DataDog/datadog-agent/pkg/collector/corechecks/oracle-dbm/oracle.go:218 +0x5d4
github.com/DataDog/datadog-agent/pkg/collector/internal/middleware.(*CheckWrapper).Run(0xc000194960?)
  /Users/nenad.noveljic/go/src/github.com/DataDog/datadog-agent/pkg/collector/internal/middleware/check_wrapper.go:48 +0xc5
github.com/DataDog/datadog-agent/pkg/collector/worker.(*Worker).Run(0xc000b65400)
  /Users/nenad.noveljic/go/src/github.com/DataDog/datadog-agent/pkg/collector/worker/worker.go:152 +0x36a
github.com/DataDog/datadog-agent/pkg/collector/runner.(*Runner).newWorker.func1()
  /Users/nenad.noveljic/go/src/github.com/DataDog/datadog-agent/pkg/collector/runner/runner.go:133 +0x6d
created by github.com/DataDog/datadog-agent/pkg/collector/runner.(*Runner).newWorker
  /Users/nenad.noveljic/go/src/github.com/DataDog/datadog-agent/pkg/collector/runner/runner.go:130 +0x1da

Describe how to test/QA your changes
Set collection interval of query metrics to 1 and check that you don't get repetitive FQT and plan events.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

It looks like there was a potential for a panic before this fix. Shouldn't we have this as a changelog to say we fixed panics in some situations?

Added now, thx!


feat(service): add service tech email support

Add support for service tech emails

Btw please make sure that existing list can be purged. It is sufficient to test the behaviour on a local cluster, because the test might be complicated a bit.

Btw please make sure that existing list can be purged. It is sufficient to test the behaviour on a local cluster, because the test might be complicated a bit.

Test case added in PR handles checking that emails can be deleted


Manual builds do not correctly select docker-bake.preview.hcl when running daily/preview builds

I forgot to add logic into the build-manual.yaml workflow to select docker-bake.preview.hcl or docker-bake.hcl for the bake action. We need to ensure the docker-bake.preview.hcl is correctly selected or else preview builds could be updated to the production registries.

Fixed by https://github.com/rstudio/rstudio-docker-products/pull/765


Working example with saving encrypted data to file?

I have the examples working, however I'm saving the encrypted file using fopen() and fwrite(). When I attempt to decrypt the file using the private key, I get errors saying the data contains certificates and can't be decrypted or verified.
Here's my encryption process:
$m = 'Test Text';
$pathToYourAsciiArmoredPublicKey = dirname(__FILE__).'/keys/pub.asc';
$key = \OpenPGP_Message::parse(
                                \OpenPGP::unarmor(file_get_contents($pathToYourAsciiArmoredPublicKey), 'PGP PUBLIC KEY BLOCK')
                            );
$data = new \OpenPGP_LiteralDataPacket($m);
$encrypted = \OpenPGP_Crypt_Symmetric::encrypt(
                                $key,
                                new \OpenPGP_Message(array($data))
                            );

I'm not sure what I should be writing to the new file... $encrypted['0']->encrypted_data, $encrypted['1']->encrypted_data, $encrypted['2']->data or all 3? Everything I try doesn't seem to work.

I'm not sure what I should be writing to the new file... $encrypted['0']->encrypted_data, $encrypted['1']->encrypted_data, $encrypted['2']->data or all 3? Everything I try doesn't seem to work.

Probably $encrypted->to_bytes()

Probably $encrypted->to_bytes()

That worked, thank you!


fix: do not map from indexed properties

Do not map from indexed properties and add a diagnostic instead.
Rel. https://github.com/riok/mapperly/issues/226

Codecov Report

Merging #232 (88a5db0) into main (d39ace7) will increase coverage by 1.35%.
The diff coverage is 94.75%.

@@            Coverage Diff             @@
##             main     #232      +/-   ##
==========================================
+ Coverage   89.85%   91.21%   +1.35%
==========================================
  Files          76       97      +21
  Lines        2475     3074     +599
  Branches      308      403      +95
==========================================
+ Hits         2224     2804     +580
- Misses        181      190       +9
- Partials       70       80      +10

Impacted Files
Coverage Δ

...y/Descriptors/MappingBuilder/CtorMappingBuilder.cs
71.42% <0.00%> (-11.91%)
:arrow_down:

...scriptors/MappingBuilder/NullableMappingBuilder.cs
95.45% <ø> (ø)

...ppings/PropertyNullAssignmentInitializerMapping.cs
66.66% <ø> (ø)

.../Riok.Mapperly/Descriptors/Mappings/TypeMapping.cs
100.00% <ø> (ø)

src/Riok.Mapperly/Helpers/SymbolExtensions.cs
84.21% <58.82%> (-10.79%)
:arrow_down:

...riptors/MappingBuilder/DictionaryMappingBuilder.cs
64.00% <78.57%> (ø)

...Riok.Mapperly/Descriptors/MappingBuilderContext.cs
97.61% <80.00%> (+0.11%)
:arrow_up:

...scriptors/Mappings/UserImplementedMethodMapping.cs
85.71% <80.00%> (-7.15%)
:arrow_down:

...tFactories/GenericTargetObjectFactoryWithSource.cs
80.00% <80.00%> (ø)

...s/ObjectFactories/SimpleObjectFactoryWithSource.cs
80.00% <80.00%> (ø)

... and 73 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[🐞] unresolved type QwikManifest in Basic app

Which component is affected?
Qwik Runtime
Describe the bug

Reproduction
https://stackblitz.com/edit/qwik-starter?file=src%2Fentry.ssr.tsx
Steps to reproduce

pnpm create qwik@latest

🐰 Let's create a Qwik app 🐇   v0.16.2

✔ Where would you like to create your new project? … npm-demo

✔ Select a starter › Basic App (QwikCity)

✔ Would you like to install pnpm dependencies? … no

cd npm-demo && npm install

You can see that  manifest in entry.ssr.tsx resolved to  type any

System Info
System:
    OS: macOS 11.6.8
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 752.60 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: 1.22.19 - ~/Library/pnpm/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
  Browsers:
    Chrome: 108.0.5359.124
    Safari: 16.2
  npmPackages:
    @builder.io/qwik: 0.16.2 => 0.16.2
    @builder.io/qwik-city: 0.0.128 => 0.0.128
    undici: 5.14.0 => 5.14.0
    vite: 4.0.3 => 4.0.3

Additional Information
No response

Hi @await-ovo 👋
Thanks for mentioning this 🙏  The mentioned issue is also present on<EMAIL_ADDRESS>and<EMAIL_ADDRESS>


dist not included in bower package

Distribution files in dist directory should probably be included in the bower package - it was included in previous versions.

Why don't you just make a pull request?


Criteria match stripped down (still not compiling)

Codecov Report
Patch and project coverage have no change.

Comparison is base (5275c80) 100.00% compared to head (3230cfe) 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##               1.5     #1090     +/-   ##
===========================================
  Coverage   100.00%   100.00%
===========================================
  Files           46        25     -21
  Lines         2652      1184   -1468
  Branches       396       134    -262
===========================================
- Hits          2652      1184   -1468

Flag
Coverage Δ

foundry
?

production
100.00% <ø> (ø)

reference
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 40 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Can't findOne, update, delete on composite primary keys

create table variables (
  name text not null,
  key text not null,
  value text not null,
  email text not null,
  primary key(name, key)
);
insert into variables (name, key, value, email) values ('a', 'b', 'c', 'd');

// typescript error
client.variables.findOne({ where: { variables_name_key_key: { key: 'b', name: 'a' } } })
client.variables.update({ where: { variables_name_key_key: { key: 'b', name: 'a' } } })
client.variables.delete({ where: { variables_name_key_key: { key: 'b', name: 'a' } } })

Related: https://github.com/prisma/prisma-client-js/issues/489

Note: Reproduce again.

I can't reproduce this with<EMAIL_ADDRESS>binary version: f608db3b9e4f50f9a384356e532357b0a525969a. Initially when I marked this as a confirmed bug I assumed that @matthewmueller referred to implementation that client will have @@id. Right now no such functionality is exposed via the client.

Introspecting a Postgres DB that has the above SQL generates this schema:
model variables {
  email String
  key   String
  name  String
  value String

  @@id([name, key])
}

Afterwards the following index.ts (I commented in and out the individual lines to run them one by one) works:
import { PrismaClient } from '@prisma/client'

const client = new PrismaClient()

client.variables.create({ data: { name: 'a', key: 'b', email<EMAIL_ADDRESS>value: 'foo' } }).then(res => { console.log('create', res) })
client.variables.findOne({ where: { name_key: { key: 'b', name: 'a' } } }).then(res => { console.log('findOne', res) })
client.variables.update({ where: { name_key: { key: 'b', name: 'a' } }, data: { value: 'bar' } }).then(res => { console.log('update', res) })
client.variables.delete({ where: { name_key: { key: 'b', name: 'a' } } }).then(res => { console.log('delete', res) })

<EMAIL_ADDRESS>Can you please confirm that this is the wanted behavior @matthewmueller?

Can I request that composite keys be documented in the find/update/delete sections of the docs (or even a separate section that gives a hint how it works)?  I was stuck on them for days and didn't think they worked properly because the schema was showing my where options as including something like <key1>_<key2>which didn't make sense to me and nothing I tried to supply there worked.
Thanks to @janpio 's post, above, now I see how it works and I'm unstuck.  Documentation of this point would have saved a tremendous amount of time.

Hey @dpankros 👋 yes, I just created an issue in the docs repo to persist this, our docs team will look into this soon! 🙏


Dijkstra

This is a dijkstra algorithme implemented in french commit because it is a work that have do in university

@FehimMahfoud : Thanks for Contributing

We're going to go ahead and merge this PR this time. But please make sure to only send one PR after you've finished implementing the whole algorithm.
Thank you for contributing!


MacOs Version

Hi, I want to know if exist any option to run this program on MacOs. I see that in the code exist some logic about Mac OSX but I don't know how should i compile it.
Thanks and best regards!

Ok, I will release the MacOS version again. I'll let you know once it's done.

Hi
You can download v1.6.1 for mac.
Thanks
Ishi

Released CallFlowVisualizer-Mac-1.6.1.zip
https://github.com/tishige/CallFlowVisualizer/releases/download/1.6.1/CallFlowVisualizer-Mac-1.6.1.zip


[fix] branch page's static file issue

Coverage decreased (-0.03%) to 83.915% when pulling bf84489d7790b60c7682f3a46a896bacefccfcef on itowtips:fix-branch into 3647cea10d534055e274382337f9c2dc66ee6d27 on shirasagi:master.

Coverage decreased (-0.03%) to 83.912% when pulling 0693eb6b712fae442e17893252ed56901646f8f9 on itowtips:fix-branch into 5734a99c020f1722720faa70a30bd9030a7b4d15 on shirasagi:master.

Coverage decreased (-0.03%) to 83.918% when pulling 0693eb6b712fae442e17893252ed56901646f8f9 on itowtips:fix-branch into 5734a99c020f1722720faa70a30bd9030a7b4d15 on shirasagi:master.


system services feature: solve import cycle between systemservices & api pkgs

There is still the import cycle, you've just moved the deploy function to deployer package, but the deploy is still done by API.
To fix this issue:

create deploy package with following funcs/methods: deploy.FromPath, deploy.FromGit, deploy.FromGzippedTar
use those funcs/methods inside sytemservices
use those func/methods inside api
repeat the same for service start* functions

In such approach, systemservices won't be importing api package.

There is no import cycle anymore, can you show me the import lines from cycling packages?
The deploy pkg you're proposing is similar to pkg that I proposed on from but with the name native.
You answered as "how many layers API of should be there?" and we went with the mapping idea of yours.
It's late to do this change now, we can discuss about it later and maybe re-consider changing the architecture after we merge waiting PRs that depends on system services.

I added one commit with what I was thinking with this system services deployer and the system services that contains just the list of the different services that the deployer inject into it.
This solves the cyclic dependency and I feel like that we have 2 system services packages that have specific responsibilities (one is the "database", the other is the one that fill this "database").
With that we can also have different ways to fill this system services. Here we deploy based on directories but if we find a way to release the version with a seed of a database with some already deployed system services we could just register these services based on the service database.
This commit can be reverted if it's really doesn't make sense, I just wanted to expose my solution with code because it seems I cannot explain it correctly during meetings.

Are we merging this one? @mesg-foundation/core please review. Before taking any action, I'd like to have conclusion about this https://github.com/mesg-foundation/core/pull/559#issuecomment-438644633 one as well. Because it might effect the solution here.

This one for sure remove import cycle but I dont like the fact that there is deploying inside main. But I'm ok to approve it jsut to move on. But still I think the deploying should be done inside systemservices (wihtout api usage)


increased loading time and crash when using xfrog model with MPI

Hi,
I am running a small standalone ospray application using MPI on a Cray system. The application loads the xfrog-forest model and renders a frame (using ospray_commandline, ospray_cpp etc.).
Now the problem is that the loading time of the xfrog model increases rapidly with the number of mpi workers. On the single node the time spend during loading (done with parseWithDefaultParsers):
1 node 4 ppm (3 mpi workers) -> 20s
1 node 6ppm (5 mpi workers) -> 29s
1 node 12ppm (11 mpi workers) -> 33s
1 node 24 ppn (23 mpi workers) -> 54s
When different nodes are involved, the application segfaults.
2 node 4 ppn (7 mpi workers) ->
_pmiu_daemon(SIGCHLD): [NID 04572] [c11-1c2s7n0] [Wed Aug 31 12:30:24 2016] PE RANK 1 exit signal Segmentation fault
[NID 04572] 2016-08-31 12:30:24 Apid 4989096: initiated application termination
Application 4989096 exit codes: 139
Application 4989096 exit signals: Killed
The increase in loading time appears to come from this loop in TriangleMeshSceneParser::finalize:
https://github.com/ospray/OSPRay/blob/master/apps/common/commandline/SceneParser/trianglemesh/TriangleMeshSceneParser.cpp#L362-L369
As I run this on a single node with 4 workers, it iterates the loop rather quickly. Once the number of workers increases, the loop takes more and more time to iterate. The same is true when multiple nodes are involved. The first iterations go through swiftly and each successive iteration takes increasingly longer. Around the i=318000/377266 mark it crashes when I use multiple nodes.
I thought I run this issue by you guys first before I dig deeper on my own. Maybe there is some communication involved for each added instance which takes a hit with additional workers. If thats the case, maybe some sort of batch instantiation could help. Do you guys have an idea where this could come from and why the application crashes on multiple nodes?
Thanks in advance.
Best,
David

hi David,
thanks for the feedback!  You are right, the initial way we had api calls broadcast over mpi was very inefficient, and it gets worse the more mpi ranks you have.  We currently have an implementation in a development branch that does indeed batch commands together among other changes that make the mpi comm layer more efficient.  One of our developers is currently looking into merging some of those changes with our devel branch (and possible next release) which should help significantly.  I will try to remember to update this issue when that gets put in, hopefully in the next two weeks or so.
I'm not sure why the scene is segfaulting for you on multiple nodes.  We recently ran that model across 22 nodes, so I'll need some help narrowing the seg fault down.  Was this with the "--osp:mpi" flag?  Can you run the same test using our glutViewer app and see if you can replicate the issue?  Can you also test with 1 ppn and see if that fixes the seg fault?
Thanks!
Carson

on another note, are you running multiple mpi processes per node for the sake of your application code or for testing distributed?  It's always better to run a single process per node with OSPRay when possible.

I am running multiple mpi workers per node just for testing distributed. I'll try a single process per node. Are osprays mpi workers single threaded?

No, ospray_mpi_worker spawns multiple threads.

Hi Johannes, thanks for the answer. How is each worker determining the number of threads it will use? I just want to make sure that the number of PEs per node matches what the workers will actually use...

Per default it is up to the used tasking system to decide the optimal number of threads. This usually works fine, e.g. all PEs are used and also CPU groups / partitions are adhered. If this for some reason does not work you might use the parameter --osp:numthreads <n> to override the automatic behaviour.

ah ok thanks.
Just out of curiosity: how does the worker know how many PE's he can use for multithreading? As far as my limited understanding goes, there might be other PE's in use on that node by other users/applications, right?

TBB will auto detect the number of virtual cores to run on a system, but it's up to the user to avoid thread over-utilization by setting the number of threads as Johannes described when running multiple MPI processes on a single node or other resource heavy applications.

When I kick of ospray on the cray, I have to specify the amount of PEs per node which I want to allocate using the aprun command. Doing that results in ospray detecting every single PE as seperate MPI instance, which---as it sounds---will internally be multithreaded.
How can I allocate resources (using aprun) and at the same time make sure that a single mpi instance is running per node with all other PEs on that node being used/available for multithreading?
Thanks for your patience and for bearing with all my questions! Much appreciated!

I'm not familiar with aprun but on the TACC machines (and with mpirun) there's usually some argument (or environment variable) to set the number of cores per node or cores per rank that are assigned. So there should be some way you can specify to use N nodes with 1 ospray process per node and each process assigned all M cores on its node.

David,
I am not familiar w/ aprun, but here is something that might be useful:
http://www.nersc.gov/users/computational-systems/retired-systems/hopper/running-jobs/using-openmp-with-mpi/
Of course we use TBB rather than OpenMP, but the problem is exactly the
same: you want fewer MPI tasks per node than there are PEs, and want
those MPI tasks to fill the PEs using threads. Unlike tihs document we
suggest is a single MPI task per node (this document suggest four); but
this should allow you to adapt the parameters accordingly.
Hope this helps - if it does, feel free to put a little page onto the
github wiki ;-)
Cheers,
Ingo
On 09/21/2016 08:28 AM, David Koerner wrote:

When I kick of ospray on the cray, I have to specify the amount of PEs
per node which I want to allocate using the aprun command. Doing that
results in ospray detecting every single PE as seperate MPI instance,
which---as it sounds---will internally be multithreaded.
How can I allocate resources (using aprun) and at the same time make
sure that a single mpi instance is running per node with all other PEs
on that node being used/available for multithreading?
Thanks for your patience and for bearing with all my questions! Much
appreciated!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ospray/OSPRay/issues/80#issuecomment-248629361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwZtU6Llhk0XHMIl4JDG5-H2CTTthFhks5qsT8HgaJpZM4JxejT.

@Twinklebear what's the status of merging in the work unit batching with devel?

@carsonbrownlee I've got the crash when exiting fixed but only with a hack, I need to add in the management class you mentioned in the comment in mpi/MPICommon.cpp. The other stuff left to do is have work units indicate if the master should run them or not in master/worker mode (currently the master runs everything). You can currently run the ospGlutViewer with the batching but volumes crash currently (my guess is b/c of this issue)

while this can still be optimized, message batching (now called buffered write/read streams) was merged into OSPRay proper a while ago which should mitigate load times over MPI.


[Hub Generated] Review request for Microsoft.Insights to add version stable/2022-06-15

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

Hi @avjerafi, Your PR has some issues. Please fix the CI sequentially by following the order of Avocado, semantic validation, model validation, breaking change, lintDiff. If you have any questions, please post your questions in this channel https://aka.ms/swaggersupport.
Task	How to fix	Priority
Avocado	Fix-Avocado	High
Semantic validation	Fix-SemanticValidation-Error	High
Model validation	Fix-ModelValidation-Error	High
LintDiff	Fix-LintDiff	high
If you need further help, please feedback via swagger feedback.

@changlong-liu Please advise regarding the apparent validation errors. I believe they are unjustified:

I've executed all these validations locally for our resource type ("ScheuledQueryRules") prior to submitting this PR.
Following these validation errors I've re-executed them locally again for the entire service ("monitor") and reviewed the results:

Avocado + Semantic + Model validations: Local executions return 0 errors which are actually related to the submitted changes (the ScheduledQueryRules resource type).
Prettier: Executed for the entire monitor service and got 0 changes.

@changlong-liu  Please note: this new api version introduces no changes to the spec itself.
Beginning with this api version, our service will require that all Action Groups which are references in Log Search Alert rules must already be provisioned whenever a rule is added/updated - a restriction which was not imposed until now.
This is necessary in order to comply with the Schrems II requirement of looking up and tracking the location of all referenced Action Groups.
This new constraint has the potential to break existing deployments for some customers. The introduction of this new api version is intended to let customers opt-in to this.

@changlong-liu Please note: this new api version introduces no changes to the spec itself.
Beginning with this api version, our service will require that all Action Groups which are references in Log Search Alert rules must already be provisioned whenever a rule is added/updated - a restriction which was not imposed until now.
This is necessary in order to comply with the Schrems II requirement of looking up and tracking the location of all referenced Action Groups.
This new constraint has the potential to break existing deployments for some customers. The introduction of this new api version is intended to let customers opt-in to this.

Hi @avjerafi , the new api version need arm review first, and then we will go to SDK breaking change review.

Hi @changlong-liu,
I'm still waiting for ARM review. Do you know can I ping in order to expedite this? We would like to have this api version approved within this week if possible.

Hi @avjerafi , would you also taking a look at the avocado issues: https://github.com/Azure/azure-rest-api-specs/pull/19652/checks?check_run_id=7210901264.
For arm review, you may ping Azure Resource Manager RP API Review<EMAIL_ADDRESS>if in urgent.

Hi @ruowan , the LintDiff error seems come from the common-types v2 which is referenced by this PR: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/common-types/resource-management/v2/types.json#L17 .
Would you help to confirm whether it blocks this PR?

Hi @changlong-liu,
Can you please add clarity to this new MISSING_APIS_IN_DEFAULT_TAG validation error in avocado?
Do I understand correctly that the api version which is missing is 2018-04-16? This is the api version I'm replacing in the default tag in this PR. Shouldn't it be missing?
Please advise. Thanks!

Hi @changlong-liu, Can you please add clarity to this new MISSING_APIS_IN_DEFAULT_TAG validation error in avocado? Do I understand correctly that the api version which is missing is 2018-04-16? This is the api version I'm replacing in the default tag in this PR. Shouldn't it be missing? Please advise. Thanks!

Hi @avjerafi , the full description can be found at https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1704911&view=logs&j=22cb5f76-5058-5a9d-2377-d04e5451a5f2&t=75f8d99b-764d-5b43-90d5-e9f774389f08.
It's complaining about the path '/subscriptions/{}/resourcegroups/{}/providers/Microsoft.Insights/scheduledQueryRules' once existed in 2018-04-16 no longer exist in the latest swagger.
Would you confirm whether it's expected or not?

Hi @changlong-liu,
Thanks for the clarification.
I verified that all 3 paths in the specs of both api versions 2018-04-16 and the new 2022-06-15 remained exactly the same:
/subscriptions/{subscriptionId}/providers/Microsoft.Insights/scheduledQueryRules
/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Insights/scheduledQueryRules
/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Insights/scheduledQueryRules/{ruleName}

Hi @changlong-liu, Thanks for the clarification. I verified that all 3 paths in the specs of both api versions 2018-04-16 and the new 2022-06-15 remained exactly the same: /subscriptions/{subscriptionId}/providers/Microsoft.Insights/scheduledQueryRules /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Insights/scheduledQueryRules /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Insights/scheduledQueryRules/{ruleName}

Hi @ruowan , would you help to take a look whether this is a false alarm?

Thanks for feedback, it's a false alarm. We will fix avocado issue. For this PR, I added approved-Avocado, it won't block your PR process.

Thank you @ruowan for your swift response!
Regarding the LintDiff validation errors:
As far as I can tell, these were not triggered by the changes in this PR. Is it possible to unblock this api version from being released?

Hi @avjerafi , for the "Swagger LintDiff" and "SDK Track2 Validation" Error, both of them are complaining about incompatible Schema.
They are about defining/referencing multiple models which has same name, and they are incompatible,
Would you please check?

Hi @avjerafi , for the "Swagger LintDiff" and "SDK Track2 Validation" Error, both of them are complaining about incompatible Schema. They are about defining/referencing multiple models which has same name, and they are incompatible, Would you please check?

Hi @changlong-liu,
Yes, I can see the cause: there are duplicate type definitions which are not identical. But these involve several resource types, which are not handled by my team and I cannot modify. The error messages include a suggestion to use the "modelerfour.lenient-model-deduplication" setting to temporarily suppress these comparisons. How do I set this? I could not find this setting being used anywhere the swagger spec.

hi @qiaozha , @ArcturusZhang , would you help review the breaking changes in JS/GoLang? Thanks!

Hi @avjerafi , I see "DoNotMerge" label is added. Ping me please if you want to merge this, thanks!

@changlong-liu The ARM manifest was fully deployed and I've removed the DoNotMerge flag. Please merge this PR. Thanks!

/azp run

@changlong-liu The ARM manifest was fully deployed and I've removed the DoNotMerge flag. Please merge this PR. Thanks!

Adding @ArcturusZhang as the new assignee.

@ArcturusZhang Requesting again to please merge this PR. Thank you.

/azp run

@ArcturusZhang Kind reminder: please merge this PR. Thanks!


fix: add osmosis token send support

Description

adds support Osmosis network token sends
depends on: https://github.com/shapeshift/lib/pull/1201

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Medium risk - this modifies the send code path for all cosmos-sdk sends.
Testing

Engineering

Operations

verify that non-OSMO sends function as expected
verify that there are no regressions with native Osmosis and Cosmos sends

Screenshots (if applicable)

@pastaghost Please test that the Tx value is actually matching the intended one and don't rely on the confirmed toast, this can be verified through Mintscan or with the Keplr popup, which is showing amounts using again the wrong precision for Osmosis LP tokens sends.
I've checked on both Keplr and Osmosis and it doesn't seem to be possible to send LP tokens, which would explain why Mintscan shows such 0-balance amount, if that's so the send button should be disabled for those

No, that's not quite right. Cosmos send amounts are denominated in base units. (0.631788)^18 =<PHONE_NUMBER>00000000, so the send amount shown in the Keplr confirmation is correct. That also matches the Osmosis network representation of LP asset balances. See here.
If you take another look at the error message you posted, you'll see that it's complaining about insufficient funds, in fact a zero balance for the token you tried to send. That doesn't match your Pool 1 LP token balance as shown in the app, so there must be something else going on.
The problem is actually the denom. The correct denom for the Pool 1 LP token is 'gamm/pool/1' not 'ibc/gamm/pool/1' as shown in Keplr and on Mintscan for your test transaction. I fixed this by passing the raw denom through in the case that we're trying to send an LP asset and prepending ibc/ otherwise.
The Osmosis network doesn't make any distinction between LP tokens and native/IBC tokens as far as sends are concerned, and you can send them just like any other.

Keplr wallet has separate names for the base-unit and decimal-precision units for each recognized asset. For example:
    currencies: [
      {
        coinDenom: "OSMO",
        coinMinimalDenom: "uosmo",
        coinDecimals: 6,
        coinGeckoId: "osmosis",
      },
      {
        coinDenom: "ION",
        coinMinimalDenom: "uion",
        coinDecimals: 6,
        coinGeckoId: "ion",
      },
    ],

The asset data is hardcoded in the extension source code. If an asset entry exists, Keplr uses the asset data to format the transaction amounts in decimal-precision format before displaying the send confirmation to the user. If no asset entry exists, the transaction data is passed through as-is. That's why the amount is shown in base units on the Keplr send confirmation screen for Osmosis LP assets but not others.

This is a UI problem on the Keplr side that we can't control, but there is something that we can (should) do about it on our side. Since the network lists asset balances in base units along with the base unit denom, we shouldn't be using gamm/pool/1 when showing the amount in decimal-precision units. Mintscan apparently encountered the same problem and decided to use GAMM-N for LP token amounts in decimal-precision and gamm/pool/n for LP token amounts in base units - I think we should follow suit. This will resolve any ambiguity in the web app and keep the UI consistent with Mintscan. CC: @DiggyDiggy2 @reallybeard