ISAAC ROS 3.1 devcontainer fixes & improvements #293
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Still working on this
Isopod00
changed the title
Skipped more keys in the rosdep command when opening the devcontainer.
Isopod00
commented
Oct 11, 2024
Isopod00
changed the title
Isopod00
added a commit
that referenced
this pull request
Oct 17, 2024
* Initial attempt at isaac ros 3 * More changes for initial build * Changing /home/blixt to $HOME so the enter_container script runs on other computers * Swapped submodules to use isaac 3.1, deleted our docker.zed in favor of nvidias. Need to test still * The container works, building is broken with zed though * Minor changes to ros2_wrapper version, Isaac ros 3.1 builds now!!!! * Removing references to combine_esdf that were breaking nvblox. * Updating to ZED SDK 4.1. Uses our own Docker zed file to do so * Move deepstream dependency into base image * Modify deepstream insatllation to use ngc cli * Update README.md I didn't like the double "first" in the readme so I changed it * Adding Deepstream download file to .gitignore * Using tag 3.1 instead of main branch * Removing Deepstream from image key since we deleted deepstream docker file * Revert "Removing Deepstream from image key since we deleted deepstream docker file" This reverts commit 4dde93b. * Added the deepstream dockerfile back * Delete Dockerfile.ros2_jazzy * Added our missing user Dockerfile * More dockerfile updates * More dockerfile updates * More changes * Updated .gitignore * Updated .gitignore * Updated .gitignore * Small Whitespace Changes * Add version to isaac ros tag * Remove cache file * Remove submodules * First working build on x86_64 * "I didn't delete the line to mount the zed neural depth" - Jonathan * Readd docker_args * Working nvblox reconstruction after changing zed namespace for 3.1 * Fixing a linter fail from bad syntax * Fixing a linter fail from bad syntax * Made zed remappings for nvblox optional * Setting fixed frame back to map * Whitespace / Formatting Fixes * Fixed missing item in .gitignore * .gitignore whitespace fix * Added submodules back for isaac ros packages * Added the missing dependencies back to umn dockerfile * Revert back to the old zed remapping method * Autoformatting * Revert zed namespace change * nvblox works with gazebo sim again! * Autoformatting * Readding deesptream to gitignore * Update submodules * Add asset scripts * Revert "Add asset scripts" and "Update submodules" This reverts commit 637cf6d. This reverts commit ae40418. * Add foxglove dependency * Remove ros-humble-foxglove-msgs from the skipped keys * change to deepstream download, so it downloads arm64 on the jetson * Updating readme to include the new ngc installation stuff * Changing this made it so deepstream didnt download with every build * ISAAC ROS 3.1 devcontainer fixes & improvements (#293) * Update README.md * Updating scripts to mount /rosbags in place of /ssd * Removing old container entry info * uploading can startup script that i got off jetson before wiping * Removed the unused zed sdk install scripts * they call me the docker dude --------- Co-authored-by: alex berg <[email protected]> Co-authored-by: Isopod00 <[email protected]> Co-authored-by: Alex Berg <[email protected]> Co-authored-by: Alex Berg <[email protected]> Co-authored-by: UMN Robotics <[email protected]> Co-authored-by: umnrobotics <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Still working on this