search

udacity / CarND-Capstone

MIT License
190 stars 1.53k forks source link

Update lowpass.py #313

Closed 2sbsbsb closed 3 years ago

2sbsbsb commented 5 years ago

It is a minor change but it is very clear now what this class is going to do.

0<a<1 is a weight multiplied to current measurement and (1-a) multiplied to the last_value where last_value is the function of last_measurement.

I would go ahead and remove b altogether and just use a and (1-a)

UAnjali commented 3 years ago

Hi @2sbsbsb, Thanks for bringing this to our notice. But, we are closing out the issue because there are no enhancements proposed here. Although, feel free to suggest if it is causing any improvement to the code. Thank you.