Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SDSS module includes data on install #26

Open
djperrefort opened this issue Jan 24, 2020 · 0 comments
Open

SDSS module includes data on install #26

djperrefort opened this issue Jan 24, 2020 · 0 comments
Labels
Bug Something isn't working

Comments

@djperrefort
Copy link
Collaborator

Problem

The SDSS spectroscopic data release sako18_spec is published using .fits files in IRAF format. I've yet to figure out how to parse the wavelength values from these files reliably in Python. At present the sako18_spec includes compressed .txt versions of these files that have been pre-parsed using IRAF. Including survey data in the installation process is against the overall design of the package.

Expected Behavior

No SDSS data should be included with the package on install. Data should be downloaded using the download_module_data function.

Potential Solutions

  1. Host the pre-parsed files online. This isn't ideal as it opens the door for having to host other data, which is an unwanted responsibility.
  2. Download the SDSS spectra as fits files and figure out how to parse the IRAF format.
@djperrefort djperrefort added the Bug Something isn't working label Feb 24, 2020
@djperrefort djperrefort modified the milestone: Version 1.0 Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant