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

Add tutorial + example SDF for shadow texture size #2597

Merged
merged 13 commits into from
Sep 10, 2024

Conversation

athenaz2
Copy link
Contributor

@athenaz2 athenaz2 commented Sep 5, 2024

🎉 New feature

Add tutorial + example SDF. Related to gazebosim/gz-rendering#1034

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Signed-off-by: Athena Z <athenaz@google.com>
Signed-off-by: Athena Z <athenaz@google.com>
<engine>ogre2</engine>
<camera_pose>-10 0 7 0 0.5 0</camera_pose>
<shadows>
<texture_size light_type="directional">16384</texture_size>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

set this to 8192 as I see that's what the tutorial tells the user to do?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tutorials/shadow_texture_size.md Outdated Show resolved Hide resolved
tutorials/shadow_texture_size.md Outdated Show resolved Hide resolved

### Example usage for the GUI

Let's change the shadow texture size for directional light with the SDF file below. (The finished SDF file can be viewed [here](https://github.com/gazebosim/gz-sim/blob/main/examples/worlds/shadow_texture_size.sdf).)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you mention that we currently only support changing shadow texture size for directional lights?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup = done here b48725b

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh can you also change the link to point to gz-sim9 branch? i.e. https://github.com/gazebosim/gz-sim/tree/gz-sim9/examples/worlds/shadow_texture_size.sdf

tutorials/shadow_texture_size.md Outdated Show resolved Hide resolved
athenaz2 and others added 6 commits September 6, 2024 14:07
Co-authored-by: Ian Chen <ichen@openrobotics.org>
Signed-off-by: Athena Z. <athenaz@google.com>
Signed-off-by: Athena Z. <athenaz@google.com>
Signed-off-by: Athena Z. <athenaz@google.com>
Signed-off-by: Athena Z. <athenaz@google.com>
@ahcorde ahcorde enabled auto-merge (squash) September 9, 2024 08:27
@ahcorde ahcorde merged commit b1f919b into gazebosim:gz-sim9 Sep 10, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏛️ ionic Gazebo Ionic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants