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

shadow on propeller (fgfs next as of 19.11.2021) #470

Open
hbeni opened this issue Dec 19, 2021 · 16 comments · Fixed by #499
Open

shadow on propeller (fgfs next as of 19.11.2021) #470

hbeni opened this issue Dec 19, 2021 · 16 comments · Fixed by #499

Comments

@hbeni
Copy link
Collaborator

hbeni commented Dec 19, 2021

On yesterday next I got this by chance.
In the propeller I have seen the interiour shadow of the plane. Is that intentional?
Sun was in my back, fair weather setting, afternoon.

grafik


Steps to reproduce:

  • Fair weather
  • Time=Afternoon (noon works too)
  • Start in air (approach or so, and use the cruising preset)
  • Fly so, that the sun casts a shadow to the cockpit
  • Try to move the shadow onto the disc and observe shadow casts there

SF FGFS Issue ticket: https://sourceforge.net/p/flightgear/codetickets/2713/

@HHS81
Copy link
Owner

HHS81 commented Dec 20, 2021

that`s new behavior and not intended. Probably a graphic glitch some where in the hardcode.

@hbeni
Copy link
Collaborator Author

hbeni commented Dec 20, 2021

Ok, I think I remember others having similar problems and it’s been worked on. There was some traffic in the forum and mailing list i think:
https://forum.flightgear.org/viewtopic.php?f=37&t=39632&hilit=Clipping#p391662

there it was the DDS cache corrupted.

@hbeni
Copy link
Collaborator Author

hbeni commented Dec 20, 2021

I mark this as WONTFIX for now, but keep it open, so we don't forget that we test once in a while...

@hbeni hbeni added the wontfix label Dec 20, 2021
@hbeni
Copy link
Collaborator Author

hbeni commented Dec 20, 2021

C172p has the same issue.

@hbeni
Copy link
Collaborator Author

hbeni commented Feb 10, 2022

asked the mailing list.

@hbeni
Copy link
Collaborator Author

hbeni commented Feb 14, 2022

Opened an FGFS issue ticket: https://sourceforge.net/p/flightgear/codetickets/2713/

@hbeni
Copy link
Collaborator Author

hbeni commented Jan 9, 2023

asked again on the list....

@hbeni
Copy link
Collaborator Author

hbeni commented Jan 10, 2023

On the Mailing List James Turner wrote:

A quick searching: model-interior-ALS defines a 'shadow-mapping-enabled' flag, which eg the C172 sets in a few places, eg: ./Aircraft/c172p/Models/Effects/interior/c172p-interior.eff: true

And the compositor shadows use the uniform 'shadows_enabled' which is controlled by modef-default.eff,

<shadows_enabled>
<use>/sim/rendering/shadows/enabled</use>
</shadows_enabled>

What you might need is to subclacc model-default.eff and make a model-unshadowd.eff for your prop disc? I don't know if this is something which makes sense to include in FGData effects.

@TheFGFSEagle Hi Eagle, does that make sense to you? Do you have some spare time to assist?

@TheFGFSEagle
Copy link
Contributor

Yeah, I think too what you're seeing on the prop is not the shadow from the shadow map from the interior effects, but simply the compositor shadow - in which case James' idea should work fine.

@hbeni
Copy link
Collaborator Author

hbeni commented Jan 10, 2023

Hm what still puzzles me a bit - when we have no shadow on the fast propeller disc, where does the light come from, when the sun is behind the plane.
Shouldn't there be a shadow in the disc, even for fast rotating blades?

I think we probably need to do real life investigation.

@hbeni
Copy link
Collaborator Author

hbeni commented Jan 10, 2023

Asked on the forums: https://forum.flightgear.org/viewtopic.php?f=34&t=41161

@TheFGFSEagle
Copy link
Contributor

IMO the shadow should on the prop disc too, but faded by the same amount as the prop - which is just what we have right now, right ?

hbeni pushed a commit that referenced this issue Jan 11, 2023
Disable shadows on FastProp

Fix #470
@hbeni
Copy link
Collaborator Author

hbeni commented Jan 11, 2023

So - I decided to merge Eagles fix.
The reasoning is that this restores the years-old previous behaviour which sounds more sensible currently.

The shadowing was definitely wrong, and if we get better details on how a real shadow would look like, we now can model that properly trough the new separated effect.

@hbeni hbeni reopened this Jan 26, 2024
@hbeni
Copy link
Collaborator Author

hbeni commented Jan 26, 2024

Looks like its again here....
Tried by spawning on Keflavik RW02, Noon time, so the sun is in the back. Idling on the threshold, the shadow is to be seen on the disc.

@TheFGFSEagle
Copy link
Contributor

Hmmmmm, that's weird … I think I should be able to take a look later in the afternoon / evening.

@TheFGFSEagle
Copy link
Contributor

With FGFS from 17.10., it is still working - just noting that here to facilitate the search for the offending commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants