desihub / desimeter

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

xytest analysis #119

Closed joesilber closed 4 years ago

joesilber commented 4 years ago

This script reads in data gathered from the online DB using get_posmoves. It can accept a data set comprising one or many exposure ids. In other words, it can analyze one "xytest", yes, but will just as happily analyze performance data for example from one or many nights of observing. Then it calculates the positioning errors and generates several outputs.

CSV outputs:

Plot outputs:

This is a good feature set, for me to start anticollision testing and get rapid feedback on the results. I therefore think this is a good version to push to master. Some future features it could include are:

Additionally, the code is reasonably well factored into functions, but for simplicity right now I kept all the functions in one script file. Future versions could fairly easily push these functions into separate modules --- if we see any value in their re-use.

julienguy commented 4 years ago

code requires numpy version >= 1.15 because of the use of numpy.quantile. I had numpy 1.14 and updated to 1.18 to be able to run this.

I guess it's fine, but I'd like to give first @sbailey and/or @weaverba137 a chance to comment.

sbailey commented 4 years ago

I'm fine with numpy>=1.15 requirement; current desiconda release at NERSC uses numpy 1.19.1. Functionality sounds great.

julienguy commented 4 years ago

do we have to edit the requirements.txt file? If yes, is numpy>=1.15 the correct format?

weaverba137 commented 4 years ago

Yes that should be the correct format.