Skip to content

Split game data between fusion/shared#180

Merged
Miepee merged 1 commit intomainfrom
zm6-split-game-data
Sep 10, 2025
Merged

Split game data between fusion/shared#180
Miepee merged 1 commit intomainfrom
zm6-split-game-data

Conversation

@biosp4rk
Copy link
Copy Markdown
Collaborator

@biosp4rk biosp4rk commented Sep 9, 2025

Data that both games have is in constants/game_data.py, and data unique to fusion is in mf/constants/game_data.py

return 0xF


def starting_equipment(rom: Rom) -> int:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

does zm not have starting equipment?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It does, but in a different format. I could put it with the other shared data, but we probably won't have shared code for setting the equipment

raise ValueError(rom.region)


def file_screen_text_ptrs(rom: Rom) -> int:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what exactly is meant with "text pointers" here? pointers to any text on the file select (e.g Energy, Time, Sound, Exit etc.) or just the samus data at the top?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It's the list of pointers to text on the title screen. I think you can see the different text entries in mage. I believe currently we only edit the entry that says Samus data. In ZM it uses a custom graphic at the top instead of using text

from mars_patcher.rom import Game, Region, Rom


def area_room_entry_ptrs(rom: Rom) -> int:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

so we decided on not moving these into a json?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

We could if we want. I don't feel strongly either way

Copy link
Copy Markdown
Contributor

@Miepee Miepee left a comment

Choose a reason for hiding this comment

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

Fine with this for now, but may need a second look regarding starting_equipment and title_screen_text in the future.

@Miepee Miepee merged commit aefb854 into main Sep 10, 2025
8 checks passed
@biosp4rk biosp4rk deleted the zm6-split-game-data branch September 10, 2025 17:16
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