update params

yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( conductivity sensors ).

Fixed

added Todo comment

They were added while debugging fp_test_configs and why it's never being updated.
Removed.

tip: use the logger and set the statement to error level and you can see it in runtime

pressure limit will be unsigned int not signed int with LDT-473..beta CR
Add a TODO to update once the 473 is ready to merge

Are these print statements add for debugging?
If not, is this allowed

typo, PUMP_ON_STATE

removed

Remove this extra line

This class member is a way for dialin to present all the valve states in a list like so:

>>> fp = FP()
>>> fp.valves.valve_states_enum
[0, 0, 0, 0, 0, 0, 0]


It is organized in the order of the enum that was used to iterate. This is purely for presentation / Quality of life feature for the dialin user. We can remove it if we know users are no longer using it for monitoring

Can you review this with Michael G and find out if this is even needed? Can we remove? If not, what is it doing and why?

I see it as a list, don't know why it is named enum or what the original purpose of it was.
It was never updated, so I added the update part at least, so it will at least reflect the current states instead of always stating zeros.