-
Notifications
You must be signed in to change notification settings - Fork 632
Free Camera #337
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
Free Camera #337
Conversation
Fetch latest
|
Think I've found an issue here. If you're in free camera mode and start climbing a ladder, vine, etc., re-centering the camera with Z doesn't work. You're stuck in free camera mode until you reach a platform. Pure guesswork: in the base game, the camera gets locked behind you when you're climbing, so I suspect it's this camera lock that's preventing the player from escaping free camera mode. EDIT: Not an issue, but options to invert the axes would be welcome. EDIT2: An option to default back to the original camera when entering a new map might be warranted? I can see people wanting it both ways. I think the HD remakes on Wii U reset the camera on each map change but that shouldn't be only behavior. |
Good ideas! Will implement those and fix the ladder bug |
# Conflicts: # libultraship/libultraship/Controller.cpp # libultraship/libultraship/Controller.h # libultraship/libultraship/Lib/Fast3D/U64/PR/ultra64/controller.h # libultraship/libultraship/UltraController.h
|
Trying it out right now. It works well, apart from the camera clipping through the floor when it pans down. Also noticed that the 'ladder' glitch also happens when charging up for a spin attack |
Fix compilation issues
dcvz
left a comment
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.
Looks great! Only big issue is that we seem to have removed some of the dg strings again. Otherwise just some small nitpicks. Great work here! ✨
dcvz
left a comment
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.
LGTM 🚀 🏆
briaguya0
left a comment
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.
couple small comments, but nothing that makes me think we shouldn't ![]()
| Conf[ConfSection][STR(BTN_CRIGHT)] = std::to_string(SDL_CONTROLLER_BUTTON_RIGHTSHOULDER); | ||
| Conf[ConfSection][STR(BTN_CLEFT)] = std::to_string(SDL_CONTROLLER_BUTTON_Y); | ||
| Conf[ConfSection][STR(BTN_CDOWN)] = std::to_string(SDL_CONTROLLER_BUTTON_X); | ||
| Conf[ConfSection][STR(BTN_CUP)] = std::to_string(SDL_CONTROLLER_BUTTON_RIGHTSTICK); |
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 like this
| Normal1* norm1 = (Normal1*)camera->paramData; | ||
|
|
||
| f32 playerHeight = Player_GetHeight(camera->player); | ||
| f32 sp94; |
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.
it's not clear what sp94 means, since it's a local var only used in here, it'd be nice to have a more descriptive name
* wip free cam * Almost done, needs collision still * Added free cam behind cvar * added WIP collision * Fixed & implemented "Manual mode" from WW & TP * Fixed camera not rotating when Link is moving * fixed initialized camera rotation * Fixed camera getting stuck + made it smoother * reduced deadzone * fixed epona camera height + added WW z-target free camera * Adjusted player camera height & fixed fov * Fixed camera roll * fixed fov when moving the camera while in z-target * Camera resets to Auto when going through doors or changing maps * Fixed building * touch * more touch work * Added WIP mouse support to the free cam * gui stuff * fixed building * fixed building error * ok fixed building for real this time * oops * Fix compilation issues * removed mouse stuff that magically appeared in this branch * smoothed out stick values & removed remains of mouse support * re-added manual camera when pressing Z * reduced minimum Y position of camera * Addressed dcsv's nitpicks * part 2 * oops Co-authored-by: David Chavez <[email protected]>
This reverts commit 0a24a07.

Has a toggle in the experimental im-gui menu. C-Buttons need to be changed from the right stick to other buttons manually in the shipofharkinian.ini file. Here's what I use with a Pro Controller: