-
Notifications
You must be signed in to change notification settings - Fork 75
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
[F3D] Fix RDP world defaults #377
Conversation
This is the ucode default, so having this set to NOISE is a weird choice that also breaks SM64 (please confirm this doesn´t affect OOT please)
Draft until a good solution for game mode world defaults is implemented aha |
…te based on game mode
assume every geo mode is off for oot formatting and grammar fix Make world defaults draw into one function, add better text and homebrew mode warning two cycle in oot
This reverts commit c66f4b6.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know much about this in general but here's a code review
where is |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OoT seems to work properly
T̶h̶i̶s̶ ̶i̶s̶ ̶t̶h̶e̶ ̶u̶c̶o̶d̶e̶ ̶d̶e̶f̶a̶u̶l̶t̶,̶ ̶s̶o̶ ̶h̶a̶v̶i̶n̶g̶ ̶t̶h̶i̶s̶ ̶s̶e̶t̶ ̶t̶o̶ ̶N̶O̶I̶S̶E̶ ̶i̶s̶ ̶a̶ ̶w̶e̶i̶r̶d̶ ̶c̶h̶o̶i̶c̶e̶ ̶t̶h̶a̶t̶ ̶a̶l̶s̶o̶ ̶b̶r̶e̶a̶k̶s̶ ̶S̶M̶6̶4̶ ̶(̶p̶l̶e̶a̶s̶e̶ ̶c̶o̶n̶f̶i̶r̶m̶ ̶t̶h̶i̶s̶ ̶d̶o̶e̶s̶n̶´̶t̶ ̶a̶f̶f̶e̶c̶t̶ ̶O̶O̶T̶ ̶p̶l̶e̶a̶s̶e̶)̶
World defaults need to vary across game modes, this pr will try to focus on that from now