Stepper configs have suboptimal P term setting

PnPConf created stepper configs use a initial P value of 50

This is much too low for stepper configs and will result in following errors (50 is ok as a starting point for servos)

For stepper configs, P should be 1/Servo-thread-period so P=1000 for the standard 1ms servo thread period

Ok I will see what I can do

I pushed a better fix into 2.7

steppers get a calculated P
servos get 50

This could still be screwed up if the user set the axis as a servo then changes to a stepper but its a little better.
Thanks for the heads up

Chris M

I think you may have the calculation wrong,
P should be 1/ servo-period-in-seconds, not servo period/1000

so P=1000 at 1 ms, 500 at 2 ms, 4000 at 250 usec

servo-period-in-ns/1000 is correct for 1 ms but wrong everywhere else

damn ok ill fix

ok pushed a fix again.
Thanks for checking!

Chris M