search

Ellpeck / Coroutine

A simple implementation of Unity's Coroutines to be used for any C# project
https://www.nuget.org/packages/Coroutine
MIT License
37 stars 3 forks source link

Added function to reset stats. #12

Closed zaafar closed 3 years ago

Ellpeck commented 3 years ago

I don't really see a reason for this to exist?

zaafar commented 3 years ago

oh yes, they are public! I can change them directly. my bad.

zaafar commented 3 years ago

Actually no, the "set" is private.

So basically this will allow me to reset the stats so I can gather more recent stats. Actually thinking about it again, I just want to reset the "MAX" stat.

zaafar commented 3 years ago

@Ellpeck so I was thinking more about this a bit more. To make things more generic, rather than exposing MaxMoveNextTime and AverageMoveNextTime, If we just expose LastMoveNextTime (so basically TotalMoveNextTime, MoveNextCount and LastMoveNextTime ). I can do all the average, rolling average, maximum, min, sd on my side.

sounds good?

zaafar commented 3 years ago

I can create an MR if that sounds good to you.

Ellpeck commented 3 years ago

yea, that sounds good.