Skip to content

Conversation

@ashwinvkNV
Copy link

No description provided.

@huihuaNvidia2023
Copy link
Collaborator

Hi @ashwinvkNV, just to check do you still plan to merge this MR? If yes, we can start the review cycle. It looks good to me and this feature can be useful. Thanks!

@ashwinvkNV
Copy link
Author

ashwinvkNV commented May 27, 2025

Yes, LGTM.
I was able to run inference using generated onnx file with isaac_ros_tensor_rt, but did not "functionally" test it.

@huihuaNvidia2023
Copy link
Collaborator

I just realized that the #31 already covers this part by using the isaaclab.rsl_rl. I think we can wait a little bit more, and potentially can close this PR once #31 is merged.

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.

2 participants