Skip to content

big overhaul 2.4#88

Open
stann-co wants to merge 30 commits intomasterfrom
big-overhaul-2.4
Open

big overhaul 2.4#88
stann-co wants to merge 30 commits intomasterfrom
big-overhaul-2.4

Conversation

@stann-co
Copy link
Owner

@stann-co stann-co commented Sep 1, 2025

Some of the draw functions needs to be tested and updated
the zone constraining doesn't have a smooth transition to and from each other but just snaps
get_zoom_x / y have been removed outright and using zoom_amount should be fine regardless if smooth_draw is on or off.

It has been separated from this branch to not get confused with the extra 3d features
https://github.com/stann-co/STANNcam/tree/3d_camera_and_upgrade

A lot of duplicated code has been replaced by internal transform functions like
__room_to_view and __view_to_room
taking a point from room coordinates and translating to view coordinates and vice versa, automatically handling smooth_draw differences as well

would be good to get a look over, even before i try to fix up the last few things

@stann-co stann-co requested a review from bfrymire September 1, 2025 21:06
@stann-co
Copy link
Owner Author

stann-co commented Sep 1, 2025

another branch and pull-request into this branch could be made to try and improve the player following code as well

Copy link
Collaborator

@bfrymire bfrymire left a comment

Choose a reason for hiding this comment

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

looks good!

@stann-co stann-co requested a review from bfrymire October 21, 2025 23:52
bfrymire and others added 5 commits January 7, 2026 01:40
removed speed_threshold, and set it to be 1 always.
The speed variable can then be set to a value between 0 and 1.

1 one will make the camera track the followed instance perfectly, though with smooth_draw that may result in it appearing stuttery, if the moving instance's speed isn't a whole number, and there setting the camera speed to less than 1 may look more smooth
they were originally internal, but in case anyone get's use out of them, we may as well expose them
afaik the old issue of constructor functions showing up everywhere is no longer a thing, and so we don't need to ignore literally every function anymore (i hope)
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