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

Move conda packaging CI to conda-forge #2665

Open
3 of 4 tasks
richafrank opened this issue Mar 10, 2020 · 4 comments
Open
3 of 4 tasks

Move conda packaging CI to conda-forge #2665

richafrank opened this issue Mar 10, 2020 · 4 comments

Comments

@richafrank
Copy link
Member

richafrank commented Mar 10, 2020

A while back, we added conda packaging steps to zipline's CI for itself and its dependencies that were missing in the default anaconda channel. This work added time to CI, but enabled us to more easily release conda packages with each zipline release. The windows CI that was added ensured both that we could offer windows packages and that the tests passed on windows.

In more recent times, 1) conda-forge was built out and 2) we've released zipline less often. We've also found that our CI has rarely found packaging issues, except for missing dependencies.

We'd like to move our packaging automation to conda-forge to save our CI time and complexity. We'll keep our windows tests as part of zipline's and other repos' CI.

Some details:

  • PR and merge conda-forge feedstocks for zipline and its dependencies. A branch is in progress.
  • One dependency is an old version of bcolz. Either we should remove the upper bound that limits us to an old version and release a new version of zipline, or we should maintain a bcolz feedstock as well. I vote for new zipline release.
  • Another packaged-only-by-Quantopian dependency is lru-dict. We might remove this dependency instead of including it, since it requires a native build for each version of python.
  • Remove packaging from zipline's CI. This also requires changing the install steps not to rely on conda. Our GH actions migration would help with this.
@richafrank
Copy link
Member Author

@richafrank
Copy link
Member Author

zipline branch draft PR: #2678

@richafrank
Copy link
Member Author

Updated empyrical on conda-forge: conda-forge/empyrical-feedstock#8
PR to add zipline: conda-forge/staged-recipes#12553

@richafrank
Copy link
Member Author

ta-lib added to conda-forge too: conda-forge/staged-recipes#12733

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

1 participant