Skip to content

IRSA-8042: Fixed: a votable with a .fits extension being recognized wrong on linux - #2041

Merged
robyww merged 1 commit into
devfrom
IRSA-8042-detection
Oct 10, 2026
Merged

robyww merged 1 commit into
devfrom
IRSA-8042-detection

Conversation

@robyww

@robyww robyww commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

IRSA-8042: Fixes a votable with a .fits extension being recognized wrong on linux

  • FormatUtil.getMimeType() test for votable
  • removed Files.probeContentType(Path.of(inFile)) since it is platform specific
  • better use of tile extensions
  • fix a couple of small bugs and added some test

Testing

  • download and drop the test-datalink.xml
  • filter the table column local_semantics to coadded-spectrum
  • the table and chart should show up in the data product viewer (FYI- it service seems to take a moment)

test-datalink.xml

@robyww robyww added this to the 2026.4 milestone Oct 9, 2026
@robyww
robyww requested a review from loitly October 9, 2026 19:32
@robyww robyww self-assigned this Oct 9, 2026
@robyww robyww added the bug label Oct 9, 2026
@robyww
robyww force-pushed the IRSA-8042-detection branch from 16a32fa to aac9a9f Compare October 9, 2026 19:46
@robyww robyww changed the title IRSA-8042: Fixes a votable with a fix extension being recognized wrong on linux IRSA-8042: Fixed: a votable with a .fits extension being recognized wrong on linux Oct 9, 2026

@loitly loitly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. Worked as described.

…g on linux

 - FormatUtil.getMimeType() test for votable
 - removed Files.probeContentType(Path.of(inFile)) since it is platform specific
 - better use of tile extentions
 - added some test
 - SpectrumMetaInspector will recognize HOPS / spitzer
@robyww
robyww force-pushed the IRSA-8042-detection branch from aac9a9f to 630c6b1 Compare October 10, 2026 00:29
@robyww
robyww merged commit bfd6f87 into dev Oct 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants