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

Break up this package into smaller pieces? #117

Open
pllim opened this issue Apr 3, 2020 · 2 comments
Open

Break up this package into smaller pieces? #117

pllim opened this issue Apr 3, 2020 · 2 comments

Comments

@pllim
Copy link
Contributor

pllim commented Apr 3, 2020

This package is a collection of unrelated things. If you just need one thing from here, you also get all these other things that you will never need. Maybe consider breaking up the existing modules into several distinct groups ("do one thing but do it well"), each group to be its new package (that is not a namespace package).

The obvious ones (this list is not exhaustive):

  • TEAL can be its own thing. And if eventually only drizzlepac uses it, maybe just move it into drizzlepac. (Personally, I think this is the big one. If you take this out, I think half of those modules would naturally go with it. Also see MNT: Deprecate and eventually remove TEAL? #94 )
  • FITS related stuff that we still use can be their own thing (stpyfits, asnutil).
  • Computation algorithm used by drizzlepac can be their own thing, maybe, or moved to drizzlepac if no one else uses it.

For the really obscure stuff or the things we don't really use anymore (GEIS, anyone?), then maybe those can just stay here or be retired.

@olebole
Copy link
Contributor

olebole commented Aug 6, 2021

I am not sure about the IRAF related stuff here. If you plan to remove it, please ping me so that I can migrate it to pyraf.

@pllim
Copy link
Contributor Author

pllim commented Aug 6, 2021

I am not sure which team manages this now. Maybe @nden or @larrybradley knows. Feel free to port anything that you like. This package is open source with BSD license. 😸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants