Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added navigation dockers #99

Merged
merged 1 commit into from
Mar 6, 2025
Merged

Added navigation dockers #99

merged 1 commit into from
Mar 6, 2025

Conversation

GerardoFJ
Copy link
Member

Added modular navigation dockers

@GerardoFJ GerardoFJ requested a review from a team as a code owner March 1, 2025 10:36
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't it missing the workdir and other configs?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same, isn't it missing workdir and other configs?

# fi

# Setup camera permissions
if [ -e /dev/video0 ]; then
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this might not be necessary, It was for camera permissions for vision, but idk if nav needs it

Copy link
Member

@Ale-Coeto Ale-Coeto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just added a few comments

@GerardoFJ GerardoFJ merged commit bb1061f into main Mar 6, 2025
7 checks passed
@GerardoFJ GerardoFJ deleted the nav_docker branch March 6, 2025 17:11
EmilianoHFlores added a commit that referenced this pull request Mar 6, 2025
* STT microservice with CUDA (#109)

* bug: av install only works inside container

* fixed bug

* fixed requirement versions

* removed unused dependencies

---------

Co-authored-by: GerardoFJ <[email protected]>

* Add docker compose files for jetson and computer (#113)

Add script to download model files, and update gitignore to ignore them

* Moveit config insano (#115)

* insano as fuck

* El diablooo

* Fixed lidar el diablooo

* Fixed xacro insano

* SRDF update but not probe xddd

* Fixeada la chingadera del oscar

* Added zed fixed

* Changed base_link

* created nav_basics

---------

Co-authored-by: lleo-13 <[email protected]>

* Added navigation dockers (#99)

* Nav_2 (#117)

* Changed base_link

* Adding nav_things to nav_develpment (#98)

Added things

* renamed files

* added dualshok and added joint

* Added navigation dockers (#100)

* added packages dependant

* added rclcppcomponents nav

* Fixed nav_basics

* Added slam mapping launch

* Fixed typo

* Added amcl launch

* typo fixed

* typo fixed

* typo fixed

* typo fixed

* typo fixed

* Added navigation and localization launch

* Added setupdds

* Fixed setupdds

* Added fixed urdf real zed

* Deleted trash

* Feature/chroma task (#90)

* insert item updated

* adding json validation

* chroma adapter to recycle methods in multiple nodes

* wrapper and json format working

* adapter and json format added

---------

Co-authored-by: Gilberto Malagamba Montejo <[email protected]>
Co-authored-by: GerardoFJ <[email protected]>
Co-authored-by: Diego Eduardo Hernández Cadena <[email protected]>
Co-authored-by: Gerardo Fregoso Jiménez <[email protected]>
Co-authored-by: lleo-13 <[email protected]>
Co-authored-by: lleo-13 <[email protected]>
EmilianoHFlores added a commit that referenced this pull request Mar 7, 2025
* STT microservice with CUDA (#109)

* bug: av install only works inside container

* fixed bug

* fixed requirement versions

* removed unused dependencies

---------

Co-authored-by: GerardoFJ <[email protected]>

* Add docker compose files for jetson and computer (#113)

Add script to download model files, and update gitignore to ignore them

* Moveit config insano (#115)

* insano as fuck

* El diablooo

* Fixed lidar el diablooo

* Fixed xacro insano

* SRDF update but not probe xddd

* Fixeada la chingadera del oscar

* Added zed fixed

* Changed base_link

* created nav_basics

---------

Co-authored-by: lleo-13 <[email protected]>

* Added navigation dockers (#99)

* Nav_2 (#117)

* Changed base_link

* Adding nav_things to nav_develpment (#98)

Added things

* renamed files

* added dualshok and added joint

* Added navigation dockers (#100)

* added packages dependant

* added rclcppcomponents nav

* Fixed nav_basics

* Added slam mapping launch

* Fixed typo

* Added amcl launch

* typo fixed

* typo fixed

* typo fixed

* typo fixed

* typo fixed

* Added navigation and localization launch

* Added setupdds

* Fixed setupdds

* Added fixed urdf real zed

* Deleted trash

* Feature/chroma task (#90)

* insert item updated

* adding json validation

* chroma adapter to recycle methods in multiple nodes

* wrapper and json format working

* adapter and json format added

---------

Co-authored-by: Gilberto Malagamba Montejo <[email protected]>
Co-authored-by: GerardoFJ <[email protected]>
Co-authored-by: Diego Eduardo Hernández Cadena <[email protected]>
Co-authored-by: Gerardo Fregoso Jiménez <[email protected]>
Co-authored-by: lleo-13 <[email protected]>
Co-authored-by: lleo-13 <[email protected]>
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