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

Globally sync Water Wheel rotation #5032

Open
wants to merge 4 commits into
base: 1.18.1
Choose a base branch
from

Conversation

MalkContent
Copy link
Contributor

synchronizes all waterwheels that are going at the same speed by setting their rotation to level.getGameTime()*perTick % 1

changing perTick by changing the waterflow starts a grace period where the rotation is not synced to the game time so players still perceive smooth transitions in waterwheel speed/rotation instead of janky jumping of the wheel. after the grace period the wheel makes a single jump, hopefully when noone is looking anymore.

MalkContent and others added 4 commits February 23, 2022 18:34
syncs all waterwheels rotation depending on gameTime and turning speed
has grace period, so players changing the wheel speed by altering the liquid flow don't look at janky animation jumps
"Adding the check to the tick method definitly isn't the solution" -malte
after the grace period transitions smoothly into the synced state instead of snapping to it
@MalkContent MalkContent force-pushed the globalSyncWaterWheels branch from 1706c92 to 58891dd Compare February 24, 2022 00:00
@MalkContent
Copy link
Contributor Author

if you've got the time, check out the new smooth transition into the synced state
just wanted to see if I could do that like I had in my head
if it's to your liking, I'll clean it up, otherwise, I'll just make it so it syncs rotation when the BE is loaded and not "live", like was discussed some 5(?) months ago
sorry this took so long

@MalkContent
Copy link
Contributor Author

also: the huge list of commits makes me think that the rebasing didn't go as planned, at least on the remote side..
I'll look into that tomorrow

@TeamSpen210
Copy link
Contributor

You're trying to merge into 1.17, so it's including all the commits since then. Change the target branch for the PR?

@malte0811 malte0811 changed the base branch from 1.17.1 to 1.18.1 February 24, 2022 08:26
@MalkContent
Copy link
Contributor Author

ty for that
didn't notice yesterday late

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