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

Consolidate scene3D with ign-gui #798

Closed
wants to merge 2 commits into from
Closed

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Apr 29, 2021

Signed-off-by: ahcorde ahcorde@gmail.com

🎉 New feature

This PR is part of the consolidation of Scene3D with ign-gazebo's GzScene3D gazebosim/gz-gui#137

Related to:

Summary

This new plugin makes use of the Scene3DInterface provided in the ign-gui PR

Test it

replace the GZScene3D with this:

<plugin filename="Scene3DNew" name="3D View">
  <ignition-gui>
    <title>3D View</title>
    <property type="bool" key="showTitleBar">false</property>
    <property type="string" key="state">docked</property>
  </ignition-gui>

  <engine>ogre2</engine>
  <scene>scene</scene>
  <ambient_light>0.4 0.4 0.4</ambient_light>
  <background_color>0.8 0.8 0.8</background_color>
  <camera_pose>6 0 6 0 0.5 3.14</camera_pose>
  <fullscreen>0</fullscreen>
</plugin>

and run

ign gazebo -r -v 4 diff_drive.sdf --gui-config gui.config

You will see:

  • The scene with the model
  • sky enabled
  • try moveto
  • try follow

Checklist

Note to maintainers: Remember to use Squash-Merge

Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde ahcorde force-pushed the ahcorde/consolidate_scene3d branch from c4d848a to a48adfa Compare May 5, 2021 15:29
@ahcorde
Copy link
Contributor Author

ahcorde commented Jun 22, 2021

Closing this PR in favour of this other PR #813

@ahcorde ahcorde closed this Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant