Skip to content

Conversation

@jjomier
Copy link
Contributor

@jjomier jjomier commented Jun 14, 2025

Added i4h CLI so that holoscan applications can be run as follows:

./i4h run realsense
./i4h run clarius_cast
./i4h run clarius_solum

@jjomier jjomier requested review from Nic-Ma and wendell-hom June 14, 2025 00:52
@wendell-hom
Copy link
Contributor

/build

@mingxin-zheng mingxin-zheng self-requested a review June 18, 2025 06:33
@mingxin-zheng
Copy link
Contributor

I have a question about the overall communication. When i4h run clarius_cast, does it stream the ultrasound image from the clarius probe to i4h/isaacsim via dds ipc communcation @wendell-hom @jjomier ?

@jjomier
Copy link
Contributor Author

jjomier commented Jun 18, 2025

/build

@jjomier
Copy link
Contributor Author

jjomier commented Jun 18, 2025

I have a question about the overall communication. When i4h run clarius_cast, does it stream the ultrasound image from the clarius probe to i4h/isaacsim via dds ipc communcation @wendell-hom @jjomier ?

It does. @wendell-hom to confirm

@wendell-hom
Copy link
Contributor

I have a question about the overall communication. When i4h run clarius_cast, does it stream the ultrasound image from the clarius probe to i4h/isaacsim via dds ipc communcation @wendell-hom @jjomier ?

It does. @wendell-hom to confirm

It will publish the stream via dds with default topic name topic_ultrasound_data. If isaacsim subscribes to this topic, then yes it will receive the ultrasound stream.

@wendell-hom
Copy link
Contributor

@jjomier

I'm seeing an error when running ./i4h run realsense. It looks like holoscan sdk base image is using python3.12, but pyrealsense2 is not available with 3.12. The i4h repo is using python3.10 so this wasn't an issue.

➜  ./i4h run realsense                                                                                                                                             
...                                                                                                                                                                           
 > [2/3] RUN pip install pyrealsense2 rti.connext:
0.472 ERROR: Could not find a version that satisfies the requirement pyrealsense2 (from versions: none)
0.491 ERROR: No matching distribution found for pyrealsense2
------

 1 warning found (use docker --debug to expand):
 - InvalidDefaultArgInFrom: Default value for ARG ${BASE_IMAGE} results in empty or invalid base image name (line 20)
Dockerfile:24
--------------------
  22 |     ARG DEBIAN_FRONTEND=noninteractive
  23 |     
  24 | >>> RUN pip install pyrealsense2 rti.connext
  25 |     
  26 |     # Install librealsense.
--------------------
ERROR: failed to solve: process "/bin/sh -c pip install pyrealsense2 rti.connext" did not complete successfully: exit code: 1

@wendell-hom
Copy link
Contributor

/build

2 similar comments
@wendell-hom
Copy link
Contributor

/build

@jjomier
Copy link
Contributor Author

jjomier commented Jun 27, 2025

/build

@jjomier
Copy link
Contributor Author

jjomier commented Jun 30, 2025

@wendell-hom can you review?

@wendell-hom
Copy link
Contributor

@jjomier the applications in i4h use dds and require an RTI_LICENSE_FILE.
Are we passing in this env variable or setting it somewhere in the docker container?

jjomier added 3 commits August 1, 2025 02:11
…ENSE_FILE environment variable to Dockerfiles for Clarius Solum and Realsense applications.
@jjomier jjomier closed this Aug 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants