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

Y/C output severly downgraded since merging MikeS cores #353

Closed
mattd121 opened this issue Apr 17, 2023 · 5 comments
Closed

Y/C output severly downgraded since merging MikeS cores #353

mattd121 opened this issue Apr 17, 2023 · 5 comments

Comments

@mattd121
Copy link

The NES core has severely downgraded performance with the integrated Y/C output through a passive MikeS-design adapter compared to the separate Y/C cores. Integrated Y/C shows extreme dot crawl and flickering of lines like the black outlines of the Super Mario Bros hills. I have tested with the last separate Y/C MikeS cores and it runs great with those. Something happened since the merging of the cores that downgraded its performance. I do not have an active MikeS design adapter to test with for comparison.

@sorgelig
Copy link
Member

@MikeS11

@MikeS11
Copy link
Member

MikeS11 commented Apr 18, 2023

If this is related to dot crawl the original NES has dot crawl so the core is defaulted with it on. Open the yc.txt file and change to the optional value I included for the nes core that removes it.

@mattd121
Copy link
Author

Thank you, that worked. My OG unmodded front loader does not seem to have the dot crawl I was seeing in the core default (when stationary, not scrolling) but I know there are many revisions to the hardware and mine is one of many.

@paulb-nl
Copy link
Contributor

I see in the yc.txt file that the phase value for the NES is the same as the SNES. Normally this would be correct but the NES core has been changed to not have the skipped dot every other frame that original hardware has specifically to improve composite dot crawl.

The SNES core still has the skipped dot.

IMO now that we have composite out the skipped dot removal should be optional.

@MikeS11
Copy link
Member

MikeS11 commented Apr 18, 2023

Yeah I did have a look at changing that as well and through testing I didn't see too much of a difference. But I wasn't focusing on dot crawl at the time. Maybe I'll revisit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants