-
Notifications
You must be signed in to change notification settings - Fork 302
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
6 ➡️ 7 (main) #1293
6 ➡️ 7 (main) #1293
Conversation
Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Jorge Perez <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: ahcorde <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
* Support battery draining start via topics Signed-off-by: Nate Koenig <[email protected]> * Added more documentation and a todo item Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]>
* Make tests run as fast as possible (#1194) Signed-off-by: Louise Poubel <[email protected]> * [Dome] Make tests run as fast as possible (#1250) Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Martin Pecka <[email protected]>
Signed-off-by: William Lew <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
…linear and angular separately. (#1229) Signed-off-by: Roni Kreinin <[email protected]> Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: harshmahesheka <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
* Surface plugin. Signed-off-by: Carlos Agüero <[email protected]> Co-authored-by: Ian Chen <[email protected]>
* Adding angular acceleration. Signed-off-by: Carlos Agüero <[email protected]> * add test Signed-off-by: Ian Chen <[email protected]> * update link api tutorial Signed-off-by: Ian Chen <[email protected]> Co-authored-by: Ian Chen <[email protected]>
Signed-off-by: Louise Poubel <[email protected]> Co-authored-by: Jorge Perez <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #1293 +/- ##
==========================================
+ Coverage 62.01% 62.32% +0.31%
==========================================
Files 276 278 +2
Lines 23020 23242 +222
==========================================
+ Hits 14276 14486 +210
- Misses 8744 8756 +12
Continue to review full report at Codecov.
|
Signed-off-by: Tomoya.Fujita <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
The buoyancy engine should stop working when it crosses the water surface. This PR adds a <surface> tag which stops the buoyancy engine from exerting an upward force when the vehicle is surfacing. Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]> Signed-off-by: Louise Poubel <[email protected]> Co-authored-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
fb7778c
to
633f687
Compare
This typo appears to be an artifact of some earlier copy-pastes. It sometimes causes the SceneBroadcaster to crash in unoptimized builds. Signed-off-by: sicongw <[email protected]>
I discovered that the `<topic>` parameter has not been documented in the docstring. This mini-PR adds the docs for it. Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
633f687
to
4e5faf1
Compare
CMake warnings in Ubuntu and Windows, probably out of the scope of this PR. I think that the failing test in Ubuntu is also unrelated. |
Yeah the Ubuntu ones should be fixed by ign-gui releases. Windows is in the works, i.e. #1299
Yup, that's been flaky |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can not detect problems in the merg or missing parts. If CI is happy, I'm happy too.
➡️ Forward port
ign-gazebo6
tomain
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)