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

Audio popping in Shantae for GameBoy colour #1046

Closed
kurayami6 opened this Issue Apr 7, 2018 · 4 comments

Comments

Projects
None yet
4 participants
@kurayami6

There is a loud popping sound every time you enter and leave the save room.

mGBA version 0.7.5071-21d808d0

Windows 7, i5, GTX 970

@TurdPooCharger

This comment has been minimized.

Show comment
Hide comment
@TurdPooCharger

TurdPooCharger Apr 7, 2018

I think there's been a regression. I notice popping sounds, too, on top of graphical glitches for some titles, not necessarily just rom hacks.

mGBA v1.6.2 stable release, for the 3DS, no selected external BIOS:

  • Pokemon Blue & Red Version DX hack - pops at startup. Blank white screen when New Game at professor Oak.

  • Metroid II - Return of Samus DX hack - Missing missing and health bar.

  • Super Mario Land 2 - 6 Golden Coins (v1.2, original GB rom) - Missing bottom status bar.
    Edit 1 - I take it back. The DX hack version also deosn't have the bar.

Edit 2 - I took a look at the other issues open. These appear to be related.
#1040 #1037 #1036

TurdPooCharger commented Apr 7, 2018

I think there's been a regression. I notice popping sounds, too, on top of graphical glitches for some titles, not necessarily just rom hacks.

mGBA v1.6.2 stable release, for the 3DS, no selected external BIOS:

  • Pokemon Blue & Red Version DX hack - pops at startup. Blank white screen when New Game at professor Oak.

  • Metroid II - Return of Samus DX hack - Missing missing and health bar.

  • Super Mario Land 2 - 6 Golden Coins (v1.2, original GB rom) - Missing bottom status bar.
    Edit 1 - I take it back. The DX hack version also deosn't have the bar.

Edit 2 - I took a look at the other issues open. These appear to be related.
#1040 #1037 #1036

@orbea

This comment has been minimized.

Show comment
Hide comment
@orbea

orbea Apr 7, 2018

Contributor

This seems easy to test with Pokemon Crystal where the audio will pop as soon as the game boots. It also seems to have been a regression between 0.6.1 and 0.6.2. I bisected this and found the following commit.

commit fd60a1caff2c80ede9c2acda6986dea09abea5f0
Author: Vicki Pfau <vi@endrift.com>
Date:   Sat Jul 8 21:32:51 2017 -0700

    GB Audio: Make audio unsigned with bias (fixes #749)

:100644 100644 a0a8c6eaf7ababbffaa0880ffbc786a6d7cbc101 4c5c8466d754e4789bbc3b9ce7d359074126ad60 M	CHANGES
:040000 040000 c24e964b0d4208c2cc25036f33063d57a76dd45e 916954518f711c1ff0d97723ae013693f7a3908a M	src

fd60a1c

Contributor

orbea commented Apr 7, 2018

This seems easy to test with Pokemon Crystal where the audio will pop as soon as the game boots. It also seems to have been a regression between 0.6.1 and 0.6.2. I bisected this and found the following commit.

commit fd60a1caff2c80ede9c2acda6986dea09abea5f0
Author: Vicki Pfau <vi@endrift.com>
Date:   Sat Jul 8 21:32:51 2017 -0700

    GB Audio: Make audio unsigned with bias (fixes #749)

:100644 100644 a0a8c6eaf7ababbffaa0880ffbc786a6d7cbc101 4c5c8466d754e4789bbc3b9ce7d359074126ad60 M	CHANGES
:040000 040000 c24e964b0d4208c2cc25036f33063d57a76dd45e 916954518f711c1ff0d97723ae013693f7a3908a M	src

fd60a1c

@orbea

This comment has been minimized.

Show comment
Hide comment
@orbea

orbea Apr 8, 2018

Contributor

Reverting the above commit also works around this.

Contributor

orbea commented Apr 8, 2018

Reverting the above commit also works around this.

@endrift

This comment has been minimized.

Show comment
Hide comment
@endrift

endrift Apr 8, 2018

Member

Dupe of #1036.

Member

endrift commented Apr 8, 2018

Dupe of #1036.

@endrift endrift closed this Apr 8, 2018

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