AlpyneDreams / d8vk

Direct3D 8 to Vulkan translation for DXVK! Merged into dxvk: https://github.com/doitsujin/dxvk
zlib License
485 stars 8 forks source link

House of the Dead III - Lisa's hair detaches from her head #193

Closed SeongGino closed 4 months ago

SeongGino commented 1 year ago

Software information

The House of the Dead III (PC)

System information

Apitrace file(s)

hod3pc.5.trace.tar.xz

Log files

n/a

The first stage that the character appears (which is stage 2... which is the first real stage, but semantics aside), her hair detaches from her head - almost like it's frozen where it first appears in the shot. Examples for visual aid, both from Stage 2: 2023_11-22 195839 2023_11-22 195852

NOTE: Though the apitrace is attached with WineD3D as instructed, some facial models are broken under it and I'm not sure if the resulting trace is reproduced perfectly: (subtitles glitch here happens on D8VK too) 2023_11-22 202805-THE HOUSE OF THE DEAD 3 2023_11-22 202841-THE HOUSE OF THE DEAD 3 These issues aren't reproduced in dgvoodoo2-through-DXVK.

WinterSnowfall commented 1 year ago

@SeongGino Thank you for the very detailed report and the trace. Can confirm I'm seeing the same problem with missing/detached hair.

NOTE: Though the apitrace is attached with WineD3D as instructed, some facial models are broken under it and I'm not sure if the resulting trace is reproduced perfectly

The trace is good as far as I can tell, it's WineD3D that is even more broken than d8vk in this case:

WineD3D

I've looked at the scene and haven't noticed anything off in terms of calls. At least not too odd... the game queries device capabilities and supported texture formats multiple times on each frame, which is certainly... an interesting design.

There's no ProcessVertices involved at least, which would have been my first guess. I guess this needs the RenderDoc treatment. The game seems to be entirely reliant on fixed function, so perhaps there are some inaccuracies in dxvk's shaders.

mrdeathjr28 commented 4 months ago

In my case resolve heavy stutter and now is more playable

SeongGino commented 4 months ago

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

mrdeathjr28 commented 4 months ago

In my case appear this:

thod3a

WinterSnowfall commented 4 months ago

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

Lisa's hair is still broken, so no, the rebase and merge with dxvk didn't improve the situation here. But no worries, we'll get to it eventually.

SeongGino commented 4 months ago

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

Lisa's hair is still broken, so no, the rebase and merge with dxvk didn't improve the situation here. But no worries, we'll get to it eventually.

Which is exactly what I was referring to, thank you. I will continue to pray for the recovery of Lisa's hair in silence in the meantime. :)

mrdeathjr28 commented 4 months ago

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

Lisa's hair is still broken, so no, the rebase and merge with dxvk didn't improve the situation here. But no worries, we'll get to it eventually.

well in my case appear this:

thod3oha

thod3ohb

thod3ohc

WinterSnowfall commented 4 months ago

We don't have the game, so only checked the trace. Probably worth rechecking in-game also at this point with latest dxvk master to see if it's fixed, as it appears to be based on @mrdeathjr28's screenshots.

Blisto91 commented 4 months ago

Yea apparently the missing hair is baked in to the trace and shows also on native Windows. Wasn't checked before now

SeongGino commented 4 months ago

Checked with the artifact from doitsujin/dxvk 4b0c9c6: no change, still yikes material. 2024_07-09 014406

That said, if it's merged, should we transfer this issue there or keep it to this repo? And not sure if that would necessitate another trace capture or not.

WinterSnowfall commented 4 months ago

That said, if it's merged, should we transfer this issue there or keep it to this repo? And not sure if that would necessitate another trace capture or not.

If you have time, please move it over and close this one, yes. You don't need a new trace, attaching the existing one is fine.

SeongGino commented 4 months ago

Fair enough. Self-closing in favor of doitsujin/dxvk#4130