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

Implement the PPU (poorly) #3

Merged
merged 22 commits into from
Oct 17, 2024
Merged

Implement the PPU (poorly) #3

merged 22 commits into from
Oct 17, 2024

Conversation

maxfierke
Copy link
Owner

@maxfierke maxfierke commented Jun 28, 2024

Basic idea is to get rendering somewhat working. Not looking for cycle accuracy or matching the behavior of the pixel FIFOs, but hopefully getting parts of dmg_acid2 passing.

Very much still going off of DMG-01 & pandocs, but this is about where DMG-01 hits incomplete, so filling in some of the other details as I go

Current state

Screenshot 2024-10-16 at 9 05 00 PM

Background, window, and object rendering are all implemented, but I suspect there's something broken with windowing, something also not quite right with object priority, and probably more.

@maxfierke maxfierke marked this pull request as draft June 28, 2024 03:21
Seems to fix the endless scrolling on dmg-acid2
the clock rate ticker being defined and started outside of the loop made for some interesting problems w/ the debugger, as the ticker kept on ticking while the debugger was active. Instead, we can use time.After w/ the nice new automatic GC-ing in Go 1.23+

This also gets the Nintendo logo scrolling down nicely when running w/ the boot rom 😄
@maxfierke maxfierke marked this pull request as ready for review October 17, 2024 02:08
@maxfierke maxfierke changed the title Implement PPU Implement the PPU (poorly) Oct 17, 2024
@maxfierke
Copy link
Owner Author

This is getting large enough where it's time to just merge it in its broken state and iterate on, rather than keeping this long-running branch going. Scrolling Nintendo logo is good enough for now!

@maxfierke maxfierke merged commit 03806e8 into main Oct 17, 2024
2 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.

1 participant