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

No match for digital nes games #1403

Open
fekir opened this issue Nov 4, 2023 · 10 comments
Open

No match for digital nes games #1403

fekir opened this issue Nov 4, 2023 · 10 comments

Comments

@fekir
Copy link

fekir commented Nov 4, 2023

For example, from below is not recognized by retroarch, even if it has an entry on no-intro.

https://mhughson.itch.io/from-below

As pointed out in the comments, the original file has a given nes header, while no-intro replaced it with a nes 2.0 header before hashing, as it is done for all nes roms.

As no-intro also provides the hashes for the roms without header, it might make more sense that retroarch calculates the hash for nes files without the header if it does not found any match, ie ignore the first 16 bytes

@RobLoach
Copy link
Member

RobLoach commented Nov 4, 2023

@fekir
Copy link
Author

fekir commented Nov 4, 2023

@RobLoach what should I submit? The games are already upstream (nointro in this case)

In this particular case: https://datomatic.no-intro.org/index.php?page=show_record&s=45&n=3375

@RobLoach
Copy link
Member

RobLoach commented Nov 4, 2023

If you have the full No-Intro set, it would be great to update the Headered .dat file for NES:
https://github.com/libretro/libretro-database/blob/master/dat/Nintendo%20-%20Nintendo%20Entertainment%20System.dat

Makes sense that From Below isn't showing up, because that was dumped to No-Intro after that .dat file was updated. Clrmamepro's Dir2Dat or similar will do it for you.

@fekir
Copy link
Author

fekir commented Nov 4, 2023

There is a misunderstanding.

I have to modify the downloaded rom, otherwise retroarch does not recognize it.

Nes roms have an optional header.
No-intro overrides this header.

Thus the hashes do not match.

If I adapt the header, then the rom is recognized.

As no-intro also has headerless hashes, a more robust approach would be to calculate a headerless hash.

@RobLoach
Copy link
Member

RobLoach commented Nov 4, 2023

Correct. Some people have exported some set with the headers over at https://github.com/libretro/libretro-database/blob/master/dat/Nintendo%20-%20Nintendo%20Entertainment%20System.dat

Otherwise, Manual Scan is the best option.

@fekir
Copy link
Author

fekir commented Nov 5, 2023

I see, but wouldn't it be better to use the data already available at no-intro instead of duplicating it?

@RobLoach
Copy link
Member

RobLoach commented Nov 6, 2023

The data provided from No-Intro lives at https://github.com/libretro/libretro-database/blob/master/metadat/no-intro/Nintendo%20-%20Nintendo%20Entertainment%20System.dat

RetroArch doesn't currently support skipping the NES header when scanning, hence our need to have a seperate one for with a common set of headered CRCs. The CRCs end up being different, so it's not duplicate.

@fekir
Copy link
Author

fekir commented Nov 6, 2023

RetroArch doesn't currently support skipping the NES header when scanning, hence our need [...]

My proposal/feature request was to add that support ;)

The CRCs end up being different, so it's not duplicate

Yes, but it is already possible to recognize those games, it's a duplicated effort.

But maybe this is the wrong repo for adding such feature?

@RobLoach
Copy link
Member

Found out what's wrong...

When downloading the sets, we need to check toggle "Pirate, Homebrew, Aftermarket" 🤦

@RobLoach
Copy link
Member

But even with that, I'm unsure the hashes are the same for the release vs what's in No-Intro.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants