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

refactor executors into standalone package #64

Closed
rabernat opened this issue Jan 27, 2021 · 1 comment
Closed

refactor executors into standalone package #64

rabernat opened this issue Jan 27, 2021 · 1 comment
Labels
executors Related to executors and pipelines

Comments

@rabernat
Copy link
Contributor

Currently our executors are just imported from rechunker:

https://github.com/pangeo-forge/pangeo-forge/blob/5e2362acdd77a5c4441e8ab57deca6577bbd4dff/pangeo_forge/executors.py#L1-L3

We probably want to move the executors into a standalone package.

This means moving these three executor classes:

out of rechunker, along with their tests:

This is a bit of a chore (setting up new repo, CI, docs, packaging, etc.) But in the long term I think it is really the right way to go. This way they can be used by both packages without messy interdependencies.

@rabernat rabernat added the executors Related to executors and pipelines label Jan 27, 2021
@rabernat
Copy link
Contributor Author

We are not going to do this anymore. This is superceded by #160.

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

No branches or pull requests

1 participant