desihub / desimeter

DESI coordinates and transformations
BSD 3-Clause "New" or "Revised" License
2 stars 4 forks source link

Cumulative passing / failing positioners #63

Closed joesilber closed 4 years ago

joesilber commented 4 years ago

Bin the fit errors vs time for results from the fit_posparams run. Generate a single overview plot saying how many robots were working on each given day. Probably output some table file giving details on which positioners failed when, format TBD.

sbailey commented 4 years ago

This looks good, thanks, with a minor comment that the plot_posparams help string mentions "fit_params" which I think is "fit_posparams", right?

Please add a section to README.md or to a new file in doc/posparams.md (or similar) with examples of how to use this chain. I've lost the big picture context.

Working backwards from here:

I'm sure the answers are in a combination of PRs and emails and slack messages, but let's get a basic end-to-end example documented here for the context of how the pieces fit together and what they do. That could happen before or after merging this PR, but let's not let it linger. i.e. if you don't document it before meging, please open an issue about adding those docs. If the answer to all of this is already in a single place like a DocDB entry, ok to just reference that instead in the README.

Thanks.

joesilber commented 4 years ago

Good point -- I added an issue capturing this need for README, with a link to your comment here. Suggest proceeding with merging this particular branch.