search

sot / proseco

Probabilistic star evaluation and catalog optimization
https://sot.github.io/proseco
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

Get other useful bits from the OR into the proseco ACACatalogTable? #262

Open jeanconn opened 5 years ago

jeanconn commented 5 years ago

I'd be a fan of pushing for the API change on both sides to get the other good bits from the OR and schedule in there:

  1. a proper obsid (already discussed) [DONE]
  2. observation duration or tstop
  3. target offsets (code supports but not being passed by MATLAB) (See https://github.com/sot/sparkles/issues/141) for more details. 4.presence of monitor window and/or monitor window target coords [DONE]
  4. rad zone indicator (LOW PRIORITY)

2 and 3 would help for nailing down attitudes and allowed rolls for aca_preview even if small effect. 4 would be great while we're in there to help with future checks.

taldcroft commented 5 years ago

cc: @jskrist

taldcroft commented 5 years ago

Took this off the 4.4 list, not realistic for this release.

jeanconn commented 5 years ago

Per discussion at the 7-Mar-2019 matlab meeting, we should also pass a parameter to indicate if an observation is in the rad zone (with the thought being that non-radzone-ERs don't need 3 bright stars etc).

jeanconn commented 5 years ago

And would it be possible to pass the allowed roll range?

jskrist commented 5 years ago

I'm knee-deep in infrastructure changes, but I will look into this (likely during the testing phase for the next MATLAB Tools release).