winter-telescope / winterapi

API interactions for WINTER
MIT License
1 stars 1 forks source link

t_exp parameter confusion #31

Closed tahumada closed 9 months ago

tahumada commented 9 months ago

It is not clear that the t_exp parameter in the WinterRaDecToO class is the total expusure and not the exposure time for one of the frames that go in the coadd.

I suggest

  1. adding a message in the api_res 'msg' saying something like : 'Each exposure is %single_exp and the coadd total expusure is %t_exp' , where single_exp = t_exp/n_dither
  2. Displaying the forementioned single_exp in the table produced by build_schedule_locally. I suggest to put the column before visitExpTime and maybe rename visitExpTime to CoaddExpTime
  3. (not needed, but useful) Adding a single_exp variable that check that single_exp * n_dither = t_exp