Mirror icon indicating copy to clipboard operation
Mirror copied to clipboard

Draft - Update, Late, Fixed.

Open JesusLuvsYooh opened this issue 1 year ago • 1 comments

Needs renaming, and better comments/tooltip.

Screenshot_2024-07-06_at_19 27 50

JesusLuvsYooh avatar Jul 06 '24 18:07 JesusLuvsYooh

From discord:

  • we don't need Update(), only LateUpdate and FixedUpdate
  • need to change Time.deltaTIme to Time.fixedDeltaTime in case of fixedupdate

miwarnec avatar Jul 09 '24 07:07 miwarnec

@JesusLuvsYooh can we still do this for all the current NTs by any chance? just ldateupdate & fixedupdate?

miwarnec avatar Apr 11 '25 07:04 miwarnec

@JesusLuvsYooh can we still do this for all the current NTs by any chance? just ldateupdate & fixedupdate?

Sure go ahead, I think Gadget wanted to move it to NT Base too, which makes sense. Probably easier to start fresh branch/pr and add it, use this one as inspiration and then we can delete it. (I'd still recommend we add all 3 updates as an option, im sure there's been cases where each one has been needed)

JesusLuvsYooh avatar Apr 11 '25 08:04 JesusLuvsYooh

start fresh branch/pr

Agreed, with the following suggestions:

  • Change the order of the enum to most-to-least likely usage public enum UpdateMethod { Update, FixedUpdate, LateUpdate }

  • Rename UpdateCall to DoUpdate

  • Apply to NTBase instead of the individual NT variants.

NTHybrid already has Use Fixed Update bool and examples use it, so this is a breaking change and examples will need to be updated accordingly.

MrGadget1024 avatar Apr 11 '25 12:04 MrGadget1024