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

Vertical sprite mosaic bug #1008

Closed
CaptainCaffeine opened this issue Mar 9, 2018 · 2 comments

Comments

@CaptainCaffeine
Copy link

commented Mar 9, 2018

When comparing mosaic output between mGBA and my own emulator, I noticed that the bottom block of vertical mosaic sprites would sometimes render incorrectly in mGBA. Below are pictures of Tonc's mos_demo.gba running on hardware and in mGBA, with a horizontal mosaic of 2 and a vertical mosaic of 15. The yellow fangs of the metroid on the left should be half the length of the blue fangs of the metroid on the right, but on mGBA they're the same length.
mgba_tonc_mos_demo_bug
mos_demo_hardware

The demo can be downloaded from www.coranac.com/files/tonc-bin.zip. You can use the d-pad to adjust the mosaic of the sprite, and A+d-pad to adjust the mosaic of the background.

@endrift

This comment has been minimized.

Copy link
Member

commented Mar 9, 2018

This appears to affect the intro cutscene in Aria of Sorrow:

a2ce-8 (old behavior) vs a2ce-7 (fixed behavior)

Compared against hardware:
screen shot 2018-03-08 at 10 51 35 pm

Unfortunately my fix appears to break the Minish Cap test, so I'll need to take a closer look at it when I have more time.

@endrift endrift added this to the mGBA 0.6.2 milestone Mar 9, 2018
@endrift endrift modified the milestones: mGBA 0.6.2, mGBA 0.7.0 Apr 14, 2018
@endrift endrift modified the milestones: mGBA 0.7.0, mGBA 0.8.0 Sep 2, 2018
@endrift endrift modified the milestones: mGBA 0.8.0, mGBA 0.7.2 May 21, 2019
@endrift endrift closed this in 3056655 May 21, 2019
@endrift

This comment has been minimized.

Copy link
Member

commented May 21, 2019

Finally figured out how to fix this properly. Thanks for finding it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.