uf-mil / SubjuGator

SubjuGator 8 on-board software
31 stars 43 forks source link

Self Check Improvements #208

Open duggerdufl opened 7 years ago

duggerdufl commented 7 years ago
DSsoto commented 7 years ago

I would like to have a tool to automatically spin each thruster in sequence to confirm that they are indexed correctly and spinning in the right direction.

I'm currently looking at @jpanikulam's attempts at something like this.

duggerdufl commented 7 years ago

That's a good idea. Would save time when we have problems or have to change out a thruster.

sentree commented 7 years ago

It would be fairly straight forward to write an RQT plugin for this.

duggerdufl commented 7 years ago

What's the advantage of a RQT plugin over just a text based tool?

sentree commented 7 years ago

I didn't realize this was just a request to the existing system, sorry. It should be pretty simple to set ROS parameters for the path to each device in /dev and check if the file exists from Python.

sentree commented 7 years ago

@DSsoto suggested creating alarms so that we are notified of any changes.

DSsoto commented 7 years ago

I addressed the 2 first points @duggerd made and also created the thruster_spin_test utility, Some of these diagnostic tests should be reported to the alarm server though, I will put that on the back burner.

BTW, should we make the self_check run automatically when we launch the sub? Thoughts?

sentree commented 7 years ago

As long as the sub is still in a totally inert and killed state for the test, this is fine. I don't want Ken getting any thruster surprises if someone forgets that this is a default launch behavior.

DSsoto commented 7 years ago

We should also add to the self check:

Modified @duggerd's initial post so we can track progress from the issues tab