Phil Hord

I left EWMA in place, but it runs only in 250ms intervals instead of fast-as-possible. But it should be easy to put it back into "free-running" EWMA just by always...

I refactored it out once and make a dummy analog_tick() with a #define when it wasn't needed. But it got a little ugly and I was already changing other stuff,...

@drf5n Good points. I'm not as concerned about getting the units right, but only because I don't have any reasonable way to determine the correct PWM/degree_error, for example. With regard...

Interesting idea. We do something similar internally with HOME commands. What software issues the commands? Is it on the PC? Phil On Wed, Oct 12, 2016, 2:07 AM Dave3891 [email protected]...

I think M220 takes effect only on following commands. On Wed, Oct 12, 2016, 4:35 PM Dave3891 [email protected] wrote: > Yes, it is on the PC with a custom GUI...

Nice work. This is curious to me: `ACCELERATE_RAMP_LEN` is still there and untouched by this series. Replacing the three calls to it with `acc_ramp_len` and using some specific (but possibly...

I think it could be written more clearly. But also it seems like it could be a bug. What if distance==1 and delta_um[E]==0? Maybe it should say ``` if (distance...

Sometimes I see a bug when I abort a print and restart it, Teacup will try to rewind the extruder to position 0.000. This is bad enough, but sometimes it...

I just dropped a bunch more for you to consider. :boom:

Good catch, @Traumflug ! Also, you can drop the cast from pgm_read_byte since it already returns uint8_t. ``` sendf_P(serial_writechar, PSTR("%su\n"), pgm_read_byte(&init_sequence_2[i])); ```