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

3D shadowcasting bug for weather with sight_penalty != 1.0 #65205

Open
andrei8l opened this issue Apr 20, 2023 · 3 comments
Open

3D shadowcasting bug for weather with sight_penalty != 1.0 #65205

andrei8l opened this issue Apr 20, 2023 · 3 comments
Labels
Mechanics: Weather Rain, snow, portal storms and non-temperature environment (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@andrei8l
Copy link
Contributor

andrei8l commented Apr 20, 2023

Describe the bug

Shadowcasting appears to pass through solid obstacles when the weather's sight_penalty is not 1.0.

Attach save file

Maple Glen-trimmed.tar.gz

Steps to reproduce

  1. Load save file.
  2. Make sure 3D fov is enabled
  3. Change weather to Snowstorm
  4. Look around up then back down
  5. You now have vision past the wall. You can already see some bad shadowcasting in step 3 when you look up.

Expected behavior

Zarro boogs.

Screenshots

Screenshot from 2023-04-20 14-06-07
There must be a floor to the south of the wall for this bug to occur in this test case

Correct shadowcasting, for reference

Screenshot from 2023-04-20 14-05-26

Versions and configuration

  • OS: Linux
    • OS Version: LSB Version: n/a; Distributor ID: Arch; Description: Arch Linux; Release: rolling; Codename: n/a;
  • Game Version: 66c6e8e [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    No Fungal Growth [no_fungal_growth],
    Bionic Professions [package_bionic_professions]
    ]

Additional context

Can be reproduced with 0.F so it's not a recent bug.

@andrei8l andrei8l added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Apr 20, 2023
@NetSysFire NetSysFire added the Mechanics: Weather Rain, snow, portal storms and non-temperature environment label Apr 20, 2023
@andrei8l andrei8l changed the title 3D shadowcasting bug for weather with sight_penalty != 0 3D shadowcasting bug for weather with sight_penalty != 1.0 Apr 21, 2023
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@github-actions github-actions bot added the stale Closed for lack of activity, but still valid. label May 21, 2023
@SurFlurer
Copy link
Contributor

SurFlurer commented Jun 8, 2023

I can /confirm this is still happening on ec7537a . Let's keep it open.

@SurFlurer
Copy link
Contributor

SurFlurer commented Mar 23, 2024

Here's another way to reproduce the bug.

edit: I screwed the code while digging for this bug lol. That's actually my fault.

edit 2: I was going to explain this scenario, but screwed up. At least I could post the original case here, using save file provided in #72508. After giving myself debug night vision and walking around, here you can see some of the pillar's shadows get truncated by the railroad, but the others' are not.

I believe it's because the railroad is considered outside and the concrete floors are not. Thus they have different transparency values when the weather is foggy. This, in combination with shadowcasting's different sweep directions ( shown in the picture ), led to this bug.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mechanics: Weather Rain, snow, portal storms and non-temperature environment (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

No branches or pull requests

3 participants