Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[SM64] Fix SM64 binary geolayout exports #423

Merged
merged 5 commits into from
Jan 1, 2025

Conversation

Lilaa3
Copy link
Collaborator

@Lilaa3 Lilaa3 commented Aug 15, 2024

Don't pass in dl reference when not an object empty (need confirmation this is fine)
Update size and export binary functions for transform nodes
Check for export tag in material binary saves (fixes bleed exporting unused data)

There seems to be something wrong with reverts still, will investigate
@Lilaa3 Lilaa3 marked this pull request as draft August 15, 2024 18:07
@Lilaa3
Copy link
Collaborator Author

Lilaa3 commented Aug 15, 2024

There seems to be something wrong with reverts still, will investigate before actually having up to review

@Lilaa3 Lilaa3 force-pushed the fix-binary-geolayouts branch from d5e18ea to d37ce93 Compare August 15, 2024 18:20
@Lilaa3
Copy link
Collaborator Author

Lilaa3 commented Aug 15, 2024

Last commit accidentaly fixed an issue with bleed in binary!

@Lilaa3
Copy link
Collaborator Author

Lilaa3 commented Aug 15, 2024

It turns out my last issue was just from world defaults being wonky in the file I was testing, everything is fixed sorry for the wait

@Lilaa3 Lilaa3 marked this pull request as ready for review August 15, 2024 20:14
@SonicTheHedgeHog64
Copy link
Contributor

LET'S FUCKING GO

@Lilaa3 Lilaa3 added bug Something isn't working sm64 Has to do with the Super Mario 64 side f3d Has to do with the "f3d" code common to all games labels Aug 16, 2024
Copy link
Collaborator

@jesusyoshi54 jesusyoshi54 left a comment

Choose a reason for hiding this comment

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

exported successfully Mario in RM test ROM.
worked with and without bleed.
also exported to decomp and it still worked
image

@jesusyoshi54 jesusyoshi54 added the merge soon Will be merged in a few days at most if nothing else comes up label Dec 28, 2024
@jesusyoshi54 jesusyoshi54 merged commit 98520f1 into Fast-64:main Jan 1, 2025
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working f3d Has to do with the "f3d" code common to all games merge soon Will be merged in a few days at most if nothing else comes up sm64 Has to do with the Super Mario 64 side
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants