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

[Ogre1.x] Point and spot lights don't cast shadows #32

Open
osrf-migration opened this issue Oct 30, 2018 · 7 comments
Open

[Ogre1.x] Point and spot lights don't cast shadows #32

osrf-migration opened this issue Oct 30, 2018 · 7 comments
Labels
enhancement New feature or request

Comments

@osrf-migration
Copy link

Original report (archived issue) by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Summary

All lights have the Light::SetCastShadow function, but it only works if the light type is directional.

Motivation

I noticed this while reviewing this Ign-Gazebo PR. The issue is analogous to Gazebo issue 2083.

Describe alternatives you've considered

  • Implement shadows for all light types
  • Print a warning if the function is called for a light that doesn't support shadows

Additional context

This blue spot light had shadows on:

lightshadow.png

@osrf-migration
Copy link
Author

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


  • changed title from "Point and spot lights don't cast shadows" to "[Ogre1.x] Point and spot lights don't cast shadows"

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


  • Edited issue description

@osrf-migration osrf-migration added major enhancement New feature or request labels Apr 15, 2020
@chapulina chapulina removed the major label May 25, 2020
@WilliamLewww
Copy link
Contributor

WilliamLewww commented Jul 26, 2021

If the engines are similar enough, I could try to implement point light shadows like how I did here: gazebosim/gazebo-classic#3051.

@chapulina
Copy link
Contributor

That would be nice! ign-rendering uses the same Ogre 1 version as Gazebo classic, so I think that it should be relatively straight-forward to port that implementation.

@iche033
Copy link
Contributor

iche033 commented Jul 27, 2021

changes to the core classes should be straight forward to port. The demo world will likely have to live in ign-gazebo because ign-rendering does not support parsing sdf files.

@WilliamLewww
Copy link
Contributor

I'll also see if I can get it working for Ogre2.

@iche033
Copy link
Contributor

iche033 commented Jul 27, 2021

directional light and spot light shadows should be working in ogre2. However, there is an issue with having directional light and point light shadows enabled at the same time as found in gazebosim/gz-sim#587 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants