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

4 to 5 #938

Merged
merged 16 commits into from
Jul 28, 2021
Merged

4 to 5 #938

merged 16 commits into from
Jul 28, 2021

Conversation

adlarkin
Copy link
Contributor

@adlarkin adlarkin commented Jul 28, 2021

➡️ Forward port

Port ign-gazebo4 to ign-gazebo5

Branch comparison: ign-gazebo5...ign-gazebo4

Requires ign-gui 5.2.0 release to be made (see gazebosim/gz-gui#261)

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

chapulina and others added 15 commits July 15, 2021 10:47
Signed-off-by: Franco Cipollone <[email protected]>

Co-authored-by: RafiAbdullah <[email protected]>
Co-authored-by: Louise Poubel <[email protected]>
Signed-off-by: Marcos Wagner <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Franco Cipollone <[email protected]>

Co-authored-by: Franco Cipollone <[email protected]>
Co-authored-by: Louise Poubel <[email protected]>
Signed-off-by: addy1997 <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>

Co-authored-by: Jose Luis Rivero <[email protected]>
Co-authored-by: Louise Poubel <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>

Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: ahcorde <[email protected]>

Removed changes to diff-drive command when backporting to ign-gazebo3

Signed-off-by: Louise Poubel <[email protected]>
Extend test to have two logical cameras observing the same box.

Signed-off-by: Jaldert Rombouts <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>

Co-authored-by: Nate Koenig <[email protected]>
* Added world exporter

Signed-off-by: Gonzalo de Pedro <[email protected]>

* Fixed CMake

Signed-off-by: Gonzalo de Pedro <[email protected]>

* Changes based on review

Signed-off-by: Gonzalo de Pedro <[email protected]>

* Added example world

Signed-off-by: Nate Koenig <[email protected]>

* bump required ign-common version

Signed-off-by: Louise Poubel <[email protected]>

* PR updates

Signed-off-by: Nate Koenig <[email protected]>

* Update documentation to have simulation self terminate

Signed-off-by: Nate Koenig <[email protected]>

* Rename world_export to collada_world_exporter

Signed-off-by: Nate Koenig <[email protected]>

* Finish world exporter renaming

Signed-off-by: Nate Koenig <[email protected]>

* Revert change

Signed-off-by: Nate Koenig <[email protected]>

* Added a tutorial

Signed-off-by: Nate Koenig <[email protected]>

* Update transform

Signed-off-by: Nate Koenig <[email protected]>

* Added message

Signed-off-by: Nate Koenig <[email protected]>

* Added a test

Signed-off-by: Nate Koenig <[email protected]>

* Added more shapes

Signed-off-by: Nate Koenig <[email protected]>

* Cleanup in two locations

Signed-off-by: Nate Koenig <[email protected]>

* Fix build

Signed-off-by: Nate Koenig <[email protected]>

* Apply scale, and fix codecheck

Signed-off-by: Nate Koenig <[email protected]>

Co-authored-by: Louise Poubel <[email protected]>
Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Ashton Larkin <[email protected]>
@adlarkin adlarkin added the needs upstream release Blocked by a release of an upstream library label Jul 28, 2021
@adlarkin adlarkin requested a review from chapulina as a code owner July 28, 2021 03:57
@github-actions github-actions bot added the 🏢 edifice Ignition Edifice label Jul 28, 2021
@adlarkin adlarkin mentioned this pull request Jul 28, 2021
7 tasks
Signed-off-by: Ashton Larkin <[email protected]>
@adlarkin
Copy link
Contributor Author

The ign-gui 5.2.0 release is now complete, so I just re-triggered CI to make sure that builds succeed and that checks pass.

@adlarkin adlarkin merged commit 24be610 into ign-gazebo5 Jul 28, 2021
@adlarkin adlarkin deleted the adlarkin/4_to_5 branch July 28, 2021 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏢 edifice Ignition Edifice needs upstream release Blocked by a release of an upstream library
Projects
None yet
Development

Successfully merging this pull request may close these issues.