airpartners / logger

Code and documentation associated with the air quality data logger/sensor box.
https://github.com/airpartners/logger/wiki
0 stars 0 forks source link

Increase robustness on homing three-way valve motor #46

Closed anushadatar closed 3 years ago

anushadatar commented 3 years ago

An issue present on a recent deployment involved "homing" the three way valve such that it begins switching to the expected positions on startup - this has caused issues in deployment (probably due to some issue with the limit switch?) the valve seems to get its relative position from its starting point instead of homing with reference to the limit switches. Fixing this probably requires taking a look at the reset function and making sure that is using absolute coordinates as expected.

anushadatar commented 3 years ago

Interestingly, I can get the valve to move to a specified position (i.e. if i do goto(1000) it goes to 1000 on the other side of the valve, but then it automaticlly defaults back to hitting the limit switch on the -1000 side.

The issue that seems the most likely here is that there's a limit switch failure? or something

anushadatar commented 3 years ago

It works fine with just the valve switching - likely an issue with disconnected MCPC?

anushadatar commented 3 years ago

I think that this should probably be a workflow thing, not an automated thing, fwiw

anushadatar commented 3 years ago

I think ultimately this is taking place because of what's outlined more granularly in Issue #47 - closing this for now to open that one up.