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

Enhance tests for GBC #2

Closed
pinobatch opened this issue May 21, 2018 · 1 comment
Closed

Enhance tests for GBC #2

pinobatch opened this issue May 21, 2018 · 1 comment

Comments

@pinobatch
Copy link
Owner

@pinobatch pinobatch commented May 21, 2018

Now that the help screen is enhanced for Game Boy Color, several existing tests in the Game Boy suite could benefit from enhancement as well.

  • Gray ramp (32 actual steps instead of 12 quarter steps)
  • Solid color screen (custom RGB color)
  • Motion blur (change shade 0-3 to 0-31 and it'll substitute for 100 IRE)
  • Shadow sprite (colorize the Gus portrait)
  • Scroll test
  • Vertical scroll test
  • Lame boy demo (make it look like nrom-template)
  • Grid (make border tiles red for better contrast)
@pinobatch
Copy link
Owner Author

@pinobatch pinobatch commented May 29, 2018

For Scroll test, the following changes are needed that can't be accomplished with the attribute plane alone:

  • The sky behind the clouds should be blue on GBC or white on monochrome, but the clouds' interior should be white on both.
  • The behind the grass should be blue on GBC or light gray , but the light gray portion of the grass should be medium green on GBC.

Neither of these crosses any scanline that currently uses color 3. Therefore, we don't need to include separate tile data for as we currently do with the Gus portrait in Shadow sprite. Instead, we can draw tiles with the same shade on mono but differently on GBC by changing rBGP in addition to rSCX in .waitandwrite in scrolltest.s. Fortunately, most of these change points reflect existing split points.

  • 0-15: %00100100 (white, light gray, dark gray, white in cloud)
  • 16-111: %11100100 (normal)
  • 112-119: %01100100 white, light gray, dark gray, light gray in water)
  • 120-143: %11100100 (normal; new split point)

Adding the new split point above 120 is expected to extend the runtime of set_hillzone_scroll_pos by 8 scanlines, leaving only 24 for input processing logic. We'll have to see whether this causes a problem for the other use of this routine in the Shadow sprite test.

@pinobatch pinobatch closed this in 84922a9 May 30, 2018
@pinobatch pinobatch mentioned this issue May 30, 2018
3 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant