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

6 ➡️ 7 (main) #1293

Merged
merged 43 commits into from
Jan 25, 2022
Merged

6 ➡️ 7 (main) #1293

merged 43 commits into from
Jan 25, 2022

Conversation

chapulina
Copy link
Contributor

➡️ Forward port

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

azeey and others added 30 commits November 15, 2021 18:03
Signed-off-by: Addisu Z. Taddese <[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]>
…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: 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]>
@chapulina chapulina added the needs upstream release Blocked by a release of an upstream library label Jan 14, 2022
@github-actions github-actions bot added the 🌱 garden Ignition Garden label Jan 14, 2022
@codecov
Copy link

codecov bot commented Jan 14, 2022

Codecov Report

Merging #1293 (65efda9) into main (a40d428) will increase coverage by 0.31%.
The diff coverage is 76.51%.

❗ Current head 65efda9 differs from pull request most recent head 4e5faf1. Consider uploading reports for the commit 4e5faf1 to get more accurate results
Impacted file tree graph

@@            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     
Impacted Files Coverage Δ
include/ignition/gazebo/EntityComponentManager.hh 100.00% <ø> (ø)
include/ignition/gazebo/Link.hh 100.00% <ø> (ø)
include/ignition/gazebo/Server.hh 100.00% <ø> (ø)
include/ignition/gazebo/System.hh 100.00% <ø> (ø)
...nition/gazebo/components/JointTransmittedWrench.hh 100.00% <ø> (ø)
include/ignition/gazebo/detail/View.hh 100.00% <ø> (ø)
include/ignition/gazebo/rendering/SceneManager.hh 100.00% <ø> (ø)
src/systems/buoyancy/Buoyancy.hh 100.00% <ø> (ø)
src/systems/diff_drive/DiffDrive.hh 100.00% <ø> (ø)
...int_position_controller/JointPositionController.hh 100.00% <ø> (ø)
... and 17 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a40d428...4e5faf1. Read the comment docs.

fujitatomoya and others added 5 commits January 14, 2022 13:57
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]>
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]>
arjo129 and others added 3 commits January 25, 2022 15:39
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]>
@j-rivero
Copy link
Contributor

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.

@chapulina
Copy link
Contributor Author

CMake warnings in Ubuntu and Windows, probably out of the scope of this PR.

Yeah the Ubuntu ones should be fixed by ign-gui releases.

Windows is in the works, i.e. #1299

I think that the failing test in Ubuntu is also unrelated.

Yup, that's been flaky

Copy link
Contributor

@j-rivero j-rivero left a 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.

@chapulina chapulina removed the needs upstream release Blocked by a release of an upstream library label Jan 25, 2022
@chapulina chapulina merged commit faaa3ec into main Jan 25, 2022
@chapulina chapulina deleted the chapulina/6_to_7 branch January 25, 2022 19:31
@j-rivero j-rivero mentioned this pull request Sep 16, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱 garden Ignition Garden
Projects
None yet
Development

Successfully merging this pull request may close these issues.