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

[SM64] Fixed Bug In Water Box Exports #358

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

jesusyoshi54
Copy link
Collaborator

bug introduced in #339 woops lol.
I just did something simple here, the swap between quats and euler objects should be isomorphic (I can't actually prove this it has been a while since maths) so there is no loss in info.

@jesusyoshi54
Copy link
Collaborator Author

Fixes #355

Copy link
Collaborator

@Lilaa3 Lilaa3 left a comment

Choose a reason for hiding this comment

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

Don´t say oops scutte its on me for assuming that was correct lol

@jesusyoshi54 jesusyoshi54 added the merge soon Will be merged in a few days at most if nothing else comes up label Jun 10, 2024
@Lilaa3 Lilaa3 merged commit 8767823 into Fast-64:main Jun 11, 2024
1 check passed
@jesusyoshi54 jesusyoshi54 deleted the waterbox_fix branch July 28, 2024 23:42
Lilaa3 added a commit to Lilaa3/fast64 that referenced this pull request Aug 22, 2024
The previous fix was incomplete but the issue was never noticed from how uncommon camera volumes are.
Previous fix was a fix to Fast-64#339
Lilaa3 added a commit to Lilaa3/fast64 that referenced this pull request Aug 22, 2024
The previous fix was incomplete but the issue was never noticed from how uncommon camera volumes are.
Previous fix was a fix to Fast-64#339
Lilaa3 added a commit that referenced this pull request Aug 23, 2024
The previous fix was incomplete but the issue was never noticed from how uncommon camera volumes are.
Previous fix was a fix to #339
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge soon Will be merged in a few days at most if nothing else comes up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants