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

[BUG] - OpenKh.Command.IdxImg Critical error in DDS Files #613

Open
gledson999 opened this issue Jun 23, 2022 · 2 comments
Open

[BUG] - OpenKh.Command.IdxImg Critical error in DDS Files #613

gledson999 opened this issue Jun 23, 2022 · 2 comments

Comments

@gledson999
Copy link

openkh-build1859 and openkh-build1876
I'm using this two build to make teste and compare and the error is the same

OpenKh.Command.IdxImg
In what program within this project does this bug occur?

Describe the Bug
I was doing more tests on the tool, I discovered another critical error, this time when you are do replacements in the background textures, the tool inserts the textures and sort wrong.

In the tests below I will give an example of 3 specific scenarios and details of each one:

First Screenshot:
hb13.map
I just leave this DDS image because the tool can keep the missing DDS with original source file, just replacing the translated texture from folder:
image

This was a result:
image

hb10.map
I leave only this image here:
image

image

The result was this too:
image

But looking in detail, I noticed that the accessories shop remain with the correct textures, but with this is a image directly from the *.map (untranslated low resolution) instead of DDS file (translated hi-res):
image

hb05.map
I leave this files:
image
and the result was this:
image

Here is a curious detail, the DDS texture was inserted correctly at this point, I don't know if it was just luck or if it was influenced by the tool, but anyway I thought it was interesting to report too.
image

NOTE
All replaced DDS are the same size and format as the original.
(Example:)
If the original DDS was DXT5, the modified DDS was DXT5 also with the same settings.

@TopazTK
Copy link
Collaborator

TopazTK commented Jun 23, 2022

Basically the same as #571 but on somewhere else.

@Vladabdf
Copy link
Collaborator

Vladabdf commented Aug 4, 2023

#571 was closed due to being fixed. Does that mean this is as well?

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

3 participants