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

Rendering error in OpenGL with landscapes that are set to “normal” transfer mode #484

Open
aaronfreed opened this issue May 4, 2024 · 2 comments

Comments

@aaronfreed
Copy link

aaronfreed commented May 4, 2024

A case could be made that this should never happen, so who cares. I counter with “All Roads Lead to Sol…”

image

This is what the same area looks like in software mode.

image

Which is almost impossible to notice and at least an order of magnitude less egregious. This may be undefined behaviour, but I posit that it should nonetheless be fixed to avoid breaking people’s immersion.

(There are also ceilings like this in “Come and Take Your Medicine” and “Son of Grendel”, but they’re much smaller and thus much harder to notice. Also, I think this happens with literally any transfer mode except “landscape”.)

@treellama
Copy link
Member

You need to enable NPOT textures in order to render landscapes as normal, since the built-in landscapes are NPOT

@aaronfreed
Copy link
Author

That makes sense. Is there any reason not to have that as a default option now? How many modern computers/GPUs are incompatible with NPOT textures, and if there are a significant amount, is this something Aleph One would be able to auto-detect the first time it loads? I assume replicating the software mode behavior with NPOT textures off would be extremely challenging.

(I wouldn't say the behavior with NPOT enabled is precisely "as normal", but it's at least much less annoying and in fact looks much better than the vanilla behavior.)

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

No branches or pull requests

2 participants