Fix FITS-IDI mount type codes #2

Merged
verkout merged 1 commits from kettenis/jive-toolchain-verify:mntsta into master 4 months ago
There is no content yet.
Poster

Without this some tests in the jive-toolchain-regressions fail.
/me wonders how you didn't run into this yourself...

Without this some tests in the jive-toolchain-regressions fail. /me wonders how you didn't run into this yourself...
Owner

Actually, I did run into this (it's even in the READMEs iirc), but I had been looking for the cause of the bug in the wrong place ...
I thought the FITS files were wrong (although I couldn't imagine this to be the case) and did check the tConvert code and saw that was right and didn't think of checking the check code :-) [Quis custodiet ipsos custodes? ...] and then I gave up/postponed as "fix later"

Thanks for finding this!

Actually, I _did_ run into this (it's even in the READMEs iirc), but I had been looking for the cause of the bug in the wrong place ... I thought the FITS files were wrong (although I couldn't imagine this to be the case) and did check the tConvert code and saw that was right and didn't think of checking the check code :-) [Quis custodiet ipsos custodes? ...] and then I gave up/postponed as "fix later" Thanks for finding this!
verkout merged commit a21efada2a into master 4 months ago
The pull request has been merged as a21efada2a.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.