search

yaq-project / yaqd-control

Command line tools for inspecting and controlling yaq daemons.
https://control.yaq.fyi
GNU Lesser General Public License v3.0
2 stars 3 forks source link

tested list format of yaqd status for REPL and related scripting #8

Closed kameyer226 closed 2 years ago

kameyer226 commented 2 years ago

Some syntax/naming could be adjusted but this is what I had in mind.

Closes #3

kameyer226 commented 2 years ago

oops forgot to mention it is list instead of dict now

ksunden commented 2 years ago

I am inclined to say that this is way over complicated...

kameyer226 commented 2 years ago

Live tables are painful to deal with but it makes more sense to work with their populated outputs than to fill a separate list from scratch. Unfortunately the columns for busy and offline have to be converted to string and it was done in a separate loop.

This allows for a script to check if all the appropriate daemons we want working are in fact working prior to launching other scripts and/or bluesky-cmds. Here we can specialize which ones need to work, since some of them don't really have to be online for many reasons (repair, disconnected, forced deactivation, etc).

ksunden commented 2 years ago

And live tables have no place being generated when used as a python library, which is why we have been hesitant to suggest usage of yaqd-control as a library other than in very particular circumstances.

kameyer226 commented 2 years ago

Please suggest alternative that is not as complex

kameyer226 commented 2 years ago

Suggest alternative, please, thank you

ksunden commented 2 years ago

The alternative is to not do it because it is out of scope for this package.