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

Turkey hunting, remove unused code. #348

Open
dsentinel opened this issue Sep 13, 2019 · 2 comments
Open

Turkey hunting, remove unused code. #348

dsentinel opened this issue Sep 13, 2019 · 2 comments

Comments

@dsentinel
Copy link
Contributor

dsentinel commented Sep 13, 2019

There are unused modules, cli's, and functions. Some don't even run. Some create dependencies that may no longer be needed.

There are some functions written to handle specific experiments/datasets. These should be dropped, made for general use, or places somewhere appropriate.

The low hanging fruit is to remove the modules; files/cli tools/entry-points that are not being used, and dependencies unique to them.

The DG at PIC is going to compile a list of CL tools (entry points) that they don't use.
When that reaches a consensus the list can be posted here and the axing can begin.

Ideally, this should happen before unit tests get expanded like #347
We don't want to write tests for code that's going to disappear.

@dsentinel
Copy link
Contributor Author

The turkey:

class Turkey (object):

@ascire-pic
Copy link
Collaborator

Issue is being closed due to shifting development efforts to GeoHDF.

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

4 participants