-
Notifications
You must be signed in to change notification settings - Fork 13
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
Rethink BT SETTL models #125
Comments
Actually, this atmospheric modeling aspect of the package seems "extra" and not needed for the core functionality. Maybe we can just remove the BT SETTL models? Or include a sparse grid and then include documentation about how to add any desired model grids? |
We could also move all the data files to a required download to make the application lighter. We do this with most of STScI's data-heavy applications. |
Since both Jackie and Sherelyn gave the green light to remove them (on Slack), I'm going that route at the moment. I'm just deleting the |
Removed in #126 but keeping this issue open in case we want to re-visit. |
Currently, the BT SETTL models are being distributed as part of this package and resulting in it being quite large. We should figure out a different way of distributing these files.
The text was updated successfully, but these errors were encountered: