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

Rename SLES-51194_C5473413 to SLES-51194_C5473413.pnach (Harry Potter and the Chamber of Secrets German Version) #456

Merged
merged 3 commits into from
Dec 18, 2024

Conversation

Snake356
Copy link
Contributor

@Snake356 Snake356 commented Dec 11, 2024

When submitting, I forgot to specify .pnach where it does not recognize the file of course, has now been resubmitted and corrected, plus I added the zoom and Y-FOV and it works without any problems.

When submitting, I forgot to specify .pnach where it does not recognize the file of course, has now been resubmitted and corrected.
When submitting, I forgot to specify .pnach where it does not recognize the file of course, has now been resubmitted and corrected, plus I added the zoom and Y-FOV and it works without any problems.
@pgert
Copy link
Contributor

pgert commented Dec 12, 2024

Zoom/Y-FOV are intended to be use combined, yes - so put them in one hack:

[Widescreen 16:9]
gsaspectratio=16:9
author=arapapa

// Zoom
patch=1,EE,004ca798,word,3c013f24 // 3c013f49
patch=1,EE,004ca79c,word,34210000 // 34210fda

// Y-FoV
patch=1,EE,004e8eb0,word,3c013fe3 // 3c013faa
patch=1,EE,004e8eb4,word,34218e39 // 3421aaab

Pnach edited again and combined zoom and Y-FOV in widescreen 16:9 line. It has also been tested and everything works.
@Snake356
Copy link
Contributor Author

Zoom/Y-FOV are intended to be use combined, yes - so put them in one hack:

[Widescreen 16:9] gsaspectratio=16:9 author=arapapa

// Zoom patch=1,EE,004ca798,word,3c013f24 // 3c013f49 patch=1,EE,004ca79c,word,34210000 // 34210fda

// Y-FoV patch=1,EE,004e8eb0,word,3c013fe3 // 3c013faa patch=1,EE,004e8eb4,word,34218e39 // 3421aaab

I have edited it again and also adopted the change as you said.

@F0bes F0bes merged commit 5cc1f7c into PCSX2:main Dec 18, 2024
3 checks passed
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

Successfully merging this pull request may close these issues.

4 participants