Project

General

Profile

Emulator Issues #9369

Depth-related outline issues in Suzumiya Haruhi no Heiretsu

Added by internetakias over 4 years ago. Updated about 4 years ago.

Status:
Fixed
Priority:
High
Assignee:
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
Yes
Relates to usability:
No
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
Fixed in:
5.0-440

Description

Game Name?

Suzumiya Haruhi no Heiretsu

Game ID? (right click the game in the game list, properties, info tab)

R44J8P

MD5 Hash? (right click the game in the game list, properties, info tab, MD5 Hash: Compute)

c6bf5fd56cb6f44c4365181c4a8d7f6a

What's the problem? Describe what went wrong.

[[Suzumiya Haruhi no Heiretsu|This game has a known issue that causes its models' outlines to act as a wireframe]]

What steps will reproduce the problem?

In the older builds: enabling Fast Depth Calculation. Now, just running the game under any back end with any setting causes this issue.

Which versions of Dolphin did you test on? Does using an older version of Dolphin solve your issue? If yes, which versions of Dolphin used to work?

4.0-8987

What are your PC specifications? (CPU, GPU, Operating System, more)

NVIDIA GTX 760

Intel Core i5 2320 @3.0GHz

Windows 7 64-bit

Is there any other relevant information? (e.g. logs, screenshots,
configuration files)

R44J8P-3.png (1010 KB) R44J8P-3.png The bug in action internetakias, 02/23/2016 03:12 PM
R44J8P-18.png (847 KB) R44J8P-18.png internetakias, 02/25/2016 12:58 PM
5274
5284

History

#2 Updated by JosJuice over 4 years ago

  • Assignee set to kostamarino
  • Milestone set to Current

Assigning to kostamarino for now so we can get the INI updated for 5.0.

#3 Updated by JMC4789 over 4 years ago

  • Assignee deleted (kostamarino)
  • Priority changed from Normal to High

No. We're not changing the INI. We need to fix fast depth. Please provide a 3 frame fifolog.

https://wiki.dolphin-emu.org/index.php?title=Fifoplayer

#4 Updated by JMC4789 over 4 years ago

  • Assignee set to Armada

#6 Updated by JMC4789 over 4 years ago

NEVERMIND. Can reproduce, it's just a lot less severe in this fifolog.

Could you get a fifolog from the area you have the screenshot in? It's much more obvious there.

#7 Updated by JMC4789 over 4 years ago

  • Status changed from New to Accepted

#8 Updated by internetakias over 4 years ago

5284

JMC4789 wrote:

NEVERMIND. Can reproduce, it's just a lot less severe in this fifolog.

Could you get a fifolog from the area you have the screenshot in? It's much more obvious there.

Here, though the outlines look thinner now, for some reason.
https://mega.nz/#!wNVTVbZZ!x6WM686TihESrIV1gZr5vdwJ2Atqx8F5C6taqmq0Nw0

#9 Updated by Fog over 4 years ago

If slow depth fixes the issue, we can update the INI until we have a fix in place. This fix may take time, so INI will be what we will use if no fix is available before 5.0 release.

#10 Updated by Fog over 4 years ago

  • Status changed from Accepted to Fix pending
  • Regression changed from No to Yes

#11 Updated by hk.konpie over 4 years ago

Incidentally should be SafeTextureCacheColorSamples = 0

#12 Updated by Helios over 4 years ago

I'm personally for a INI fix until a real fix hits. Same reason as when I disabled Wiimotes in netplay. The Real Fix is a lot more comprehensive than just turning it off and could delay 5.0 a fair bit if Armada doesn't jump right on this, which I dunno if he will. and it's even less of an issue here as it simply uses slower code, rather than disabling functionality entirely. And isn't that kind of why our gameini system exists anyways?

#13 Updated by JosJuice over 4 years ago

  • Status changed from Fix pending to Accepted

Setting slow depth in the INI doesn't count as "Fix pending"/"Fixed". It can however count as not being Milestone Current anymore if it gets merged.

#14 Updated by Armada over 4 years ago

  • Status changed from Accepted to Fix pending

This issue is fixed in PR 3587: https://github.com/dolphin-emu/dolphin/pull/3587

However since this PR can not yet be merged without side effects I'll merge the INI fix.

#15 Updated by Fog over 4 years ago

#16 Updated by Fog over 4 years ago

#17 Updated by Fog over 4 years ago

  • Milestone deleted (Current)

#18 Updated by theincrediblemastere over 4 years ago

Is this fixed?

Armada's PR is closed, but some of the parts of it are merged. So I'm sorta confused.

#19 Updated by internetakias over 4 years ago

theincrediblemastere wrote:

Is this fixed?

Armada's PR is closed, but some of the parts of it are merged. So I'm sorta confused.

No.
http://i.imgur.com/8I24zdt.jpg

#20 Updated by internetakias over 4 years ago

Using Dolphin 5.0-103, by the way

#21 Updated by Armada over 4 years ago

It definitely should be fixed if you didn't change the per-game settings or opened the video settings during gameplay.

#22 Updated by internetakias over 4 years ago

Armada wrote:

It definitely should be fixed if you didn't change the per-game settings or opened the video settings during gameplay.

I see. Isn't this more of a workaround than a fix, though?

#23 Updated by JMC4789 about 4 years ago

  • Status changed from Fix pending to Fixed
  • Fixed in set to 5.0-440

Fixed completely now.

Also available in: Atom PDF