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

Jazzy - General code maintenance #123

Merged
merged 52 commits into from
Dec 11, 2024
Merged

Jazzy - General code maintenance #123

merged 52 commits into from
Dec 11, 2024

Conversation

rafal-gorecki
Copy link
Contributor

@rafal-gorecki rafal-gorecki commented Nov 29, 2024

Description:

  • Shutdown whole if spawner failed
  • Use bringup.launch.py instead combined.launch.py (add microros.launch.py)
  • Fix simulation (changes releated with naming ignition -> gazebo and Twist -> TwistaStamped)
  • Fix some tests
  • Cleaner flashing script logs
  • Add laser scan filter
  • Clean up repository: delete unused args (use_gpu, simulation-engine)
  • Use specific hash in vcs dependencies
  • Remove tools folder

DominikN and others added 30 commits September 18, 2024 23:08
* Fix controller namespace

* spellcheck fix
* added docker setup for local build

* ros2 humble works without custom controllers
* Clean up output of flash script

* Pre commit

* Cleaner

* Specify concrete commands

* Simplify

* pre-commit fix

* Clean up usb firmware
* Add scan filter

* Clean up

* undo

* added docker setup for local build

* broken ros2 controller - previous worked fine

* Clean up unused arguments

* Fix workflow: Remove spellcheck (redundant), fix integration check

* Fix multiple robot spawn

* fixes

* Delete tools folder

* Pre-commit update and remove redundant flake8

* Fix some tests and set hash in vcs

* Clean up

* Move controller test to bringup and reduce test time

* Fix lidar in simulation

* Rename test files

* Fix tests

* Add scan_filtered test and improve workpace tree

---------

Co-authored-by: Dominik Nowak <[email protected]>
* applied package.xml template

Signed-off-by: Jakub Delicat <[email protected]>

* chronology

Signed-off-by: Jakub Delicat <[email protected]>

* Support

Signed-off-by: Jakub Delicat <[email protected]>

* Removed unnecessary dep

Signed-off-by: Jakub Delicat <[email protected]>

* DDS xml lint

---------

Signed-off-by: Jakub Delicat <[email protected]>
Co-authored-by: rafal-gorecki <[email protected]>
* Add hardware and sim Docker

* Fix hardware docker

* Fixes

* Use packages-up-to

* Add description to compose
* Add hardware and sim Docker

* Fix hardware docker

* Fixes

* Fixes

* Simulation gz_gui fixes + use TwistStamped

* Use packages-up-to
@rafal-gorecki rafal-gorecki changed the title Jazzy devel Jazzy - General code maintenance Dec 5, 2024
@rafal-gorecki rafal-gorecki marked this pull request as ready for review December 11, 2024 13:28
@DominikN DominikN merged commit 9ffafaa into jazzy Dec 11, 2024
2 of 3 checks passed
@DominikN DominikN deleted the jazzy-devel branch December 11, 2024 14:27
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.

3 participants