This makes it more explicit where scan ids are converted to mission phases. This is a bit fuzzy around reactivation year boundaries, but is FAR more accurate.
This has been tested by loading every single field of view through all phases through 2023 december, verifying that it returns the correct phase except near year boundaries where it may be off by a year, but still return "Reactivation_".
It is precisely correct for non-reactivation phases.
I think we should leave it as is, there are so many edge cases where the user can put in invalid scan-ids. Attempting to enumerate them all would make this code even more complicated.
This makes it more explicit where scan ids are converted to mission phases. This is a bit fuzzy around reactivation year boundaries, but is FAR more accurate.
This has been tested by loading every single field of view through all phases through 2023 december, verifying that it returns the correct phase except near year boundaries where it may be off by a year, but still return "Reactivation_".
It is precisely correct for non-reactivation phases.