search

desihub / desimeter

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

Fix get posmoves merge #161

Closed joesilber closed 3 years ago

joesilber commented 3 years ago

Fix for get_posmoves --merged feature. For when vstack on tables that have either no data or slightly different data types. Astropy is quite intolerant of such situations, so this addresses.

joesilber commented 3 years ago

Example call: get_posmoves --host beyonce.lbl.gov --port 5432 --password xxx --exposure-ids 3522 --petal-ids 0 -c -t -o ~/jhsilber/posmovedata/expid_3522 -m

julienguy commented 3 years ago

Does not work without option -c. I tried at NERSC on KPNO data: get_posmoves --exposure-id 66575,66576 --recent-rehome 66575 -o test --merge

joesilber commented 3 years ago

Merge now works without the -c option.

Note I merged latest master code into this branch, though I probably didn't need to. It was just stale so I wanted to make sure I was testing against all the latest.