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.

Sign up for GitHub

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

Jump to bottom

[builder] all python dependencies should be pinned #581

Closed
bkmartinjr opened this issue Jun 27, 2023 · 0 comments · Fixed by #594
Closed

[builder] all python dependencies should be pinned #581

bkmartinjr opened this issue Jun 27, 2023 · 0 comments · Fixed by #594
Assignees
@bkmartinjr
Labels
census builder Related to the census builder tool P1 Priority 1 - Improvement with wide impact, fix within 1 week

Comments

@bkmartinjr
Copy link
Contributor

to prevent dependency surprises (e.g., the recent scipy/pyarrow issue) all dependencies for the Builder should be pinned. This requires a periodic chore to update them, but at least that will be a controlled process, and is already an existing chore given that we must pin at least tiledbsoma.
@pablo-gar pablo-gar added P1 Priority 1 - Improvement with wide impact, fix within 1 week census builder Related to the census builder tool labels Jun 27, 2023
@bkmartinjr bkmartinjr self-assigned this Jun 30, 2023
@bkmartinjr bkmartinjr mentioned this issue Jun 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bkmartinjr bkmartinjr

Labels
census builder Related to the census builder tool P1 Priority 1 - Improvement with wide impact, fix within 1 week
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

pin builder dependencies chanzuckerberg/cellxgene-census
2 participants
@bkmartinjr @pablo-gar