Reminded him again and he is aware of it and has plan to do so. We use "Projects" and if meanwhile anyone has a non-updated VM will update the folder name manually for now.
Don't use fabs() on integers. I don't think you want absolute values of individual positions - just absolute value of the difference - and that might not even be right. The PID may overshoot your 1,000 count advancement target - sometimes by a lot if spring is helping to push. If we overshoot the target by more than max deviation, we should consider that as having reached the target (and then some) and proceed to next advancement target. So absolute value does not seem appropriate here even for the delta.
This is the most important item to be initialized. The rest will be initialized according to what state the valve is. For instance, if homing is commanded, edge counter, number of failed homings and other variables pertaining to homing will be initialized first.