isis2fits - crashes with 'failed to convert string'
No negative impact is expected as a result of this fix.
Reported by Dale Noss:
Stuart confirmed that this is the known 'strings' error that needs to be fixed as it is encountered by applications.
#1 Updated by Tammy Becker over 3 years ago
External Post details:
I've tried to write FITS files from camera cubes of MRO/CTX, Dawn/FC and Mercury/MDIS.
In each case, I'm able to write a simple (NOMINAL) FITS file. However, when I use the
"info=all" option, isis2fits produces an error that it "Failed to convert string [InstrumentId]
to an integer". It's unclear why " InstrumentId = MDIS-NAC" needs conversion at all,
as FITS supports strings in the header.
isis2fits from=/tmp/EN1053312704M.cal.cub to=/tmp/EN1053312704M.cal.fits info=all
#6 Updated by David Miller over 3 years ago
- % Done changed from 10 to 30
Found other test data that caused error. Made fixes, but would still like to check against other data, ie the file that cause the error reported in the ticket. Waiting to contact reporter to ask about test data mentioned in original ticket before marking resolved. Additional testing is needed.
#8 Updated by David Miller over 3 years ago
- % Done changed from 30 to 90
When this bug happens, InstrumentId, SpacecraftName, TargetName, StartTime, StopTime, and ExposureDuration are not given fits labels because the application halts. With this fix, all those labels should now be available. Will run some tests and recommend new tests be added to strengthen this correction before resolving.