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

Halo 2 skybox fix (& realtime lighting shadow casting in other games) #1644

Conversation

polymetal0
Copy link

@polymetal0 polymetal0 commented May 9, 2024

Clamping depth value to 1 apparently fixes this. Skybox is also affected by the see through issue:

EDIT: In case you are caching shaders in disk, you will need to delete your shader cache file.

Captura3
Captura3mal
Captura4
Captura4mal
Captura
Capturamal
Captura2
Captura2mal

Clamping depth value to 1 apparently fixes this issue.
@Shoegzer
Copy link

Shoegzer commented May 9, 2024

Fantastic! And yeah, the see-through textures issue still exists as per #55 but this is quite a welcome fix.

@polymetal0
Copy link
Author

I noticed that this changes also restore dynamic shadows (tested on Conker and Halo CE).

Some screenshots:

Latest release:
conker
halo

Clamping z to 1:
conker2
halo 2

@polymetal0 polymetal0 changed the title Halo 2 skybox fix Halo 2 skybox fix (& realtime lighting shadow casting in other games) May 16, 2024
This change by itself restores skybox, but previous one also fixes realtime shadows in other games.
@GXTX
Copy link
Contributor

GXTX commented May 16, 2024

Ideally you'd test with https://github.com/abaire/nxdk_pgraph_tests

@polymetal0
Copy link
Author

polymetal0 commented May 17, 2024

Thanks! I don't expect these changes to make it to the main branch anyway, but I hope someone with more understanding of the project can get a better clue about where the issue might be.

@polymetal0 polymetal0 closed this May 21, 2024
@polymetal0 polymetal0 deleted the nv2a-Fix-Halo-2-Skybox-missing branch May 21, 2024 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants