This adds times (in CXC seconds) to states and commands. In looking at real use cases these columns are typically needed and the impact of adding them by default is not so bad.
It also converts columns in the CommandsTable from bytes to unicode. I found that Chandra.Time.date2secs failed for bytes input. This could be fixed, but that failure points to other subtle problems that can be encountered using bytestring values (even though astropy Table does a reasonable job of hiding that with the unicode sandwich).
Testing
[x] Passes unit tests on MacOS (shiny) (no longer targeted for merging to flight before shiny)
Description
This adds times (in CXC seconds) to states and commands. In looking at real use cases these columns are typically needed and the impact of adding them by default is not so bad.
It also converts columns in the
CommandsTablefrom bytes to unicode. I found thatChandra.Time.date2secsfailed for bytes input. This could be fixed, but that failure points to other subtle problems that can be encountered using bytestring values (even though astropy Table does a reasonable job of hiding that with the unicode sandwich).Testing