search

spacetelescope / romanisim

Nancy Grace Roman Space Telescope WFI Data Simulator
https://romanisim.readthedocs.io
Other
15 stars 13 forks source link

Truncate some too-long dummy metadata strings. #111

Closed schlafly closed 6 months ago

schlafly commented 6 months ago

There are a handful of dummy strings in from the roman_datamodels maker utilities that are too long to be parsed by the archive. We truncate those here to be be shorter.

We could consider making the defaults for these strings fit within the archive schema lengths in roman_datamodels or having validation code check these string lengths. That would avoid this sort of issue.

schlafly commented 6 months ago

Is it clear to you that you would want to do this in RDM? Ideally validate would do it, but I don't know if we have special RDM code in validate or if that just gets passed straight down to asdf, which doesn't know about these archive keywords? You could just update the default values in RDM which would be easier, I guess.

PaulHuwe commented 6 months ago

RDM wraps validate, so this could be added there.