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

Update sha256 for Wonder Boy in Monster World (USA, Europe) for proper eeprom assignment #1842

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

remutro
Copy link
Contributor

@remutro remutro commented Feb 18, 2025

Looks like the has value we had for Wonder Boy in Monster World was outdated from the GoodGen era and was not current with the currently agreed dump of this game in No-Intro's DB. This resulted in a hang after pressing start at the main menu as EEPROM values were not being found.

This update the sha256 hash with the currently accepted dump of this game. EEPROM values are now being found and assigned correctly and the game is now playable.

@LukeUsher
Copy link
Member

Is there a reason we are using the hash here as opposed to game id like we do for n64?

@remutro
Copy link
Contributor Author

remutro commented Feb 18, 2025

They don't really have a game id code that is consistent like the N64 which you can trust because Nintendo made sure they were unique and valid. Sega cartridges have a serial number field, but a lot of them seem to be garbage and controlled by the developer. For example, the serial for FIFA 96 (32x) is: "Revision1.0". That's not a real serial Id, it's whatever junk they wanted to put in that field. If there were a real Id assigned, it would be preferred, but we don't have that here.

There are also hundreds of entries in this file all based on hash which I would imagine is all for the same reason as well.

@LukeUsher LukeUsher merged commit 274d6eb into ares-emulator:master Feb 19, 2025
8 checks passed
@remutro remutro deleted the wonder-boy-hash branch February 19, 2025 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants