-
-
Notifications
You must be signed in to change notification settings - Fork 467
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
[OpenGL] Failing to load DXT1 textures with OpenGL renderer #1777
Comments
Tried to play around with same texture - converted it / inspected. Seems like missing 20 bytes are part of header10 data struct which is marked as null in anomaly textures. Data counting (layers, minmaps etc) is correct, so this part is OK Converting from D3D to DXGI fixed missing data problem, so I am not sure whether
Anomaly/CoC just do not care about such issue because OpenGL renderer is not used and this assertion does not cause any problems |
DDS format (especially DX9 one, but DX10 also, in it's earliest times) has many quirks because different vendors/implementations could make their changes to the format. |
This issue is also somewhat related with #545. |
@Xottab-DUTY |
Yep, the idea is to use DirectXTex for OpenGL too. At this moment, it's only an idea. I didn't tested if it works practically. |
Describe the bug
Assertion from GLI library is failing when try to load some textures from anomaly mod. Seems like particles/weapons are using DXT1 without alpha channel for optimal space usage (wpn_gauss.dds 682kb instead of original 1366kb).
Everything is OK with other renderers.
Original GLI is not updated for ages, unfortunately externals fork does not seem alive too so reporting it here.
To Reproduce
Expected behavior
Screenshots, videos
24kb are lost somewhere
Since header/header10 parts are read and game works on assertion comment, It feels like something is missing in this calculation approximation, but I am not familiar with renderer logics:
this->GlobalMemorySize = Storage.layer_size(BaseFace, MaxFace, BaseLevel, MaxLevel) * Layers;
wpn_gauss.zip
The text was updated successfully, but these errors were encountered: