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

Add auto-generated index.json file (#74) #76

Merged
merged 4 commits into from
Jan 2, 2025
Merged

Add auto-generated index.json file (#74) #76

merged 4 commits into from
Jan 2, 2025

Conversation

tomlogic
Copy link
Owner

Tool to generate a root-level index.json file as the first step in resolving #74.

Python implementation has support for using the index in its dev branch.

New feature in preparation for moving maps into a subdirectory.
Consumers of the maps should use the index to look up the appropriate
map for a given PinMAME ROM name.
Also adds a table summarizing which properties apply to each encoding.
(The table includes a new "dipsw" encoding currently under development
in a separate branch.)
@tomlogic tomlogic mentioned this pull request Dec 31, 2024
Copy link
Contributor

@francisdb francisdb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice improvement. Will the index stay in the root or move to the maps folder? And if it stays will the paths contain maps/?

@tomlogic
Copy link
Owner Author

tomlogic commented Jan 2, 2025

I think that keeping it at the top level is a good idea, and will allow for any directory structure we might want.

For example, maybe there are multiple directories at the root level that hold maps. Instead of maps/williams, maps/stern, and maps/gottlieb, we just have williams, stern, and gottlieb at the top level.

I'll merge this, and then get started on sorting maps for another PR.

@tomlogic tomlogic merged commit 25b360f into master Jan 2, 2025
1 check passed
@tomlogic tomlogic deleted the feature/index branch January 2, 2025 11:52
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

Successfully merging this pull request may close these issues.

2 participants