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

Unable to run test worlds for underwater simulation #2344

Closed
sauk2 opened this issue Mar 27, 2024 · 6 comments · Fixed by #2373
Closed

Unable to run test worlds for underwater simulation #2344

sauk2 opened this issue Mar 27, 2024 · 6 comments · Fixed by #2373
Labels
documentation Improvements or additions to documentation wontfix This will not be worked on

Comments

@sauk2
Copy link
Contributor

sauk2 commented Mar 27, 2024

Environment

  • OS Version: Ubuntu 22.04
  • Build: Source build
  • Branch: gz-sim8 (commit: 5944188)

Description

The test worlds for DopplerVelocityLogSystem plugin show a blank screen after launching. Specifically these files:

Steps to reproduce

The gz command is run with verbosity 2

  gz sim underwater_currents.sdf --verbose 2

Output

This is the screen that is seen. No change occurs after this.

Screenshot from 2024-03-28 01-10-14

Following is the generated log

[Wrn] [SdfEntityCreator.cc:800] Gazebo does not support Ogre material scripts. See https://gazebosim.org/api/sim/8/migrationsdf.html#:~:text=Materials for details.
[Wrn] [SdfEntityCreator.cc:805] Custom material scripts are not supported.
[Wrn] [SdfEntityCreator.cc:800] Gazebo does not support Ogre material scripts. See https://gazebosim.org/api/sim/8/migrationsdf.html#:~:text=Materials for details.
[Wrn] [SdfEntityCreator.cc:805] Custom material scripts are not supported.
[Wrn] [SdfEntityCreator.cc:800] Gazebo does not support Ogre material scripts. See https://gazebosim.org/api/sim/8/migrationsdf.html#:~:text=Materials for details.
[Wrn] [SdfEntityCreator.cc:805] Custom material scripts are not supported.
[Wrn] [SdfEntityCreator.cc:800] Gazebo does not support Ogre material scripts. See https://gazebosim.org/api/sim/8/migrationsdf.html#:~:text=Materials for details.
[Wrn] [SdfEntityCreator.cc:805] Custom material scripts are not supported.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:52:3: QML RenderWindowOverlay: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:67:3: QML EntityContextMenu: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:52:3: QML RenderWindowOverlay: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:67:3: QML EntityContextMenu: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:52:3: QML RenderWindowOverlay: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:67:3: QML EntityContextMenu: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:52:3: QML RenderWindowOverlay: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:67:3: QML EntityContextMenu: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:52:3: QML RenderWindowOverlay: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:67:3: QML EntityContextMenu: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:52:3: QML RenderWindowOverlay: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:67:3: QML EntityContextMenu: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityTree/EntityTree.qml:148:7: QML ToolButton: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityTree/EntityTree.qml:148:7: QML ToolButton: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/WorldStats/WorldStats.qml:53:3: QML RowLayout: Binding loop detected for property "x"
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:52:3: QML RenderWindowOverlay: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
[GUI] [Wrn] [Application.cc:908] [QT] file::/EntityContextMenuPlugin/EntityContextMenuPlugin.qml:67:3: QML EntityContextMenu: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
libEGL warning: egl: failed to create dri2 screen
libEGL warning: egl: failed to create dri2 screen
libEGL warning: egl: failed to create dri2 screen
libEGL warning: egl: failed to create dri2 screen
[Wrn] [Ogre2Camera.cc:421] Ogre2Camera::SetVisibilityMask: Mask bits c0000000 are set but will be ignored as they conflict with the reserved bits used internally by the ogre2 backend.
@sauk2 sauk2 added the bug Something isn't working label Mar 27, 2024
@sauk2
Copy link
Contributor Author

sauk2 commented Mar 27, 2024

@azeey I noticed this while working on issue #2289. I have since found that adding SceneBroadcaster plugin to the SDF resolves this issue.

If you could confirm this issue is valid, I can raise a fix PR for the same.

@sauk2 sauk2 changed the title Unable to run test world for underwater simulation Unable to run test worlds for underwater simulation Mar 27, 2024
@azeey
Copy link
Contributor

azeey commented Mar 28, 2024

Hi @sauk2 , those files are meant to be used for integration tests. They don't need the SceneBroadcaster because the tests don't need the GUI. Adding it will unnecessarily increase test time. Most of the SDFormat files there intentionally don't have the SceenBroadcaster or other systems that are normally essential. Instead of your fix in #2346, what we need is a README in the test/worlds directory to tell users not to use these files, but instead refer to the examples directory.

@sauk2
Copy link
Contributor Author

sauk2 commented Mar 29, 2024

Hi @sauk2 , those files are meant to be used for integration tests. They don't need the SceneBroadcaster because the tests don't need the GUI. Adding it will unnecessarily increase test time. Most of the SDFormat files there intentionally don't have the SceenBroadcaster or other systems that are normally essential. Instead of your fix in #2346, what we need is a README in the test/worlds directory to tell users not to use these files, but instead refer to the examples directory.

I was not aware of that. Thanks for the clarification.

To my knowledge, there is no example usage of DopplerVelocityLogSystem present in the examples directory. Do you think it would be a good idea to provide one?

@arjo129
Copy link
Contributor

arjo129 commented Apr 1, 2024

Yes @sauk2. Rather than adding it to the integration tests (which dont need visuals) create a new doppler velocity log example file.

@arjo129 arjo129 added documentation Improvements or additions to documentation wontfix This will not be worked on and removed bug Something isn't working labels Apr 1, 2024
@sauk2
Copy link
Contributor Author

sauk2 commented Apr 1, 2024

Yes @sauk2. Rather than adding it to the integration tests (which dont need visuals) create a new doppler velocity log example file.

@arjo129 Understood. Will go ahead with that.

Should I add that README too as a part of that PR? If yes, then is there any additional information it should contain other than what is mentioned in the following comment?

Hi @sauk2 , those files are meant to be used for integration tests. They don't need the SceneBroadcaster because the tests don't need the GUI. Adding it will unnecessarily increase test time. Most of the SDFormat files there intentionally don't have the SceenBroadcaster or other systems that are normally essential. Instead of your fix in #2346, what we need is a README in the test/worlds directory to tell users not to use these files, but instead refer to the examples directory.

@arjo129
Copy link
Contributor

arjo129 commented Apr 1, 2024

Yes thatd be great!!

arjo129 pushed a commit that referenced this issue Apr 18, 2024
# Summary
The examples/worlds directory does not contain any example world for DopplerVelocityLogSystem plugin. This PR adds the following:

Example world based on test/worlds/flat_seabed.sdf but with a SceneBroadcaster.
README.md in test/worlds directory to inform users not to use those world files.
More information can be found in Issue #2344 and PR #2346.

# Test it
This world can be tested (after building and sourcing the workspace) using the following command
```
  gz sim flat_seabed.sdf
```
---------

Signed-off-by: Saurabh Kamat <[email protected]>
@github-project-automation github-project-automation bot moved this from Inbox to Done in Core development Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation wontfix This will not be worked on
Projects
Archived in project
3 participants