search

technobly / VEXTREME

VEXTREME - Vectrex Multicart
GNU General Public License v3.0
61 stars 12 forks source link

Y-Music crashes / plays incorrect tune #38

Open cparsonsuk opened 4 years ago

cparsonsuk commented 4 years ago

Bug Report

Everything except Monty on the Run and Robocop crash the Vextreme requiring reset. The two songs that don't crash play the incorrect music and display the incorrect image.

Expected Behaviour

All songs play the correct music and display the correct image when selected

Observed Behaviour

Screen goes blank when most songs selected. Monty and Robocop do not crash but play the incorrect tune

Steps to Reproduce

Vextreme 0.2

References

Links to a video, or more info

technobly commented 4 years ago

Thanks for the report @cparsonsuk ! I can confirm this is also not fixed by the new reset mechanism #31

technobly commented 4 years ago

I just realized YMusic.bin is 64KB and VEXTREME must not deal with that bank switching correctly. Is the bank switching algorithm that YMusic.bin uses known? @cparsonsuk I know you are the other Chris, but maybe you know?

technobly commented 4 years ago

Karl_Quappe_64.bin runs on VEXTREME, and I'm pretty sure it's accessing the upper 32KB because the high score text MAL is shown, which lives in that bank.

cparsonsuk commented 4 years ago

Hey Brett, I'm not sure which bank switching method Chris Malcolm uses, it might be one other than the one provided via VIDE which is, presumably, the way Malban does Karl Quappe. Chris

On Sat, 18 Apr 2020 at 22:35, Brett Walach notifications@github.com wrote:

I just realized YMusic.bin is 64KB and VEXTREME must not deal with that bank switching correctly. Is the bank switching algorithm that YMusic.bin uses known? @cparsonsuk https://github.com/cparsonsuk I know you are the other Chris, but maybe you know?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/technobly/vextreme/issues/38#issuecomment-615953619, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNSA7W6WOV3GCCUXKBEUHTRNIMJZANCNFSM4KFYKXMQ .