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

GSdx: Don't modify the Y offset value when feedback write is enabled #2991

Closed
wants to merge 1 commit into from

Conversation

MrCK1
Copy link
Member

@MrCK1 MrCK1 commented Jun 13, 2019

Fixes the frame output height when viewing security cameras in The Suffering series.

The output height is now reported correctly as 640x448 instead of 640x494 in the title bar as well.

Master: (HW)
ts_master

PR: (HW - ignore flickering, it's another bug with output blending)
ts_pr

Master: (SW)
pcsx2_2019_06_13_12_05_00_287

PR: (SW)
pcsx2_2019_06_13_12_06_04_100

… Fixes the frame output height when viewing security cameras in The Suffering series.

The output height is now reported correctly as 640x448 instead of 640x494 in the title bar as well.
@MrCK1 MrCK1 added this to the Release 1.8 milestone Jun 13, 2019
@lightningterror
Copy link
Contributor

What is the status on this?

@MrCK1
Copy link
Member Author

MrCK1 commented Jun 17, 2019

It is a hack, so it will not get merged

@MrCK1 MrCK1 closed this Jun 17, 2019
@ssakash
Copy link
Member

ssakash commented Jun 18, 2019

Maybe make a report of it in the issue tracker if you're not planning on working on it?

@ssakash
Copy link
Member

ssakash commented Jun 18, 2019

Or reopen the pull request to show that you're trying to find a solution so potential duplicate work is avoided.

@MrCK1
Copy link
Member Author

MrCK1 commented Jun 18, 2019

I guess you're right, I'll leave it for now. Don't merge this obviously :P

@MrCK1 MrCK1 reopened this Jun 18, 2019
@ssakash
Copy link
Member

ssakash commented Jun 19, 2019

rect

@lightningterror
Copy link
Contributor

Xenosaga dream sequence looks good on my end.

@lightningterror
Copy link
Contributor

As discussed on discord, closing this one since it is a hack. Someone can implement a better alternative in the future.

@MrCK1 MrCK1 deleted the feedbackmerge_yoffset branch January 6, 2021 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants