-
Notifications
You must be signed in to change notification settings - Fork 23
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
[misc] Prepare for 1.12.0 release #1061
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1061 +/- ##
=======================================
Coverage 81.33% 81.33%
=======================================
Files 73 73
Lines 5566 5566
=======================================
Hits 4527 4527
Misses 1039 1039
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
f97b2c5
to
1cf85aa
Compare
1cf85aa
to
3f734a4
Compare
@@ -31,7 +31,8 @@ dependencies= [ | |||
# NOTE: the tiledbsoma version must be >= to the version used in the Census builder, to | |||
# ensure that the assets are readable (tiledbsoma supports backward compatible reading). | |||
# Make sure this version does not fall behind the builder's tiledbsoma version. | |||
"tiledbsoma~=1.8.0", | |||
# TODO: When ready to release 1.12.0 update tiledbsoma version to 1.9.x and delete this temporary comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
time to delete the comment?
BTW, I believe ~=1.9.1
means anything >1.9.1 and <1.10.0, so this pin should be fine (1.9.0 never existed in the wild)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will remove comment.
I believe ~=1.9.1
Yea that is my understanding too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor point/nit, but otherwise LGTM
This release of the census package is being done to include an important update for
tiledbsoma
in1.9.0
which includes a fix to categorical columns. To ensure that this release goes smoothly, we will:TileDB-SOMA
main
branch which contains the fixes that will go into1.9.0
. We can think of this as a pre-release acceptance testtiledbsoma==1.9.0
, we will update the dependencies in this PR to point totiledbsoma==1.9.0
and do the official release ofcellxgene-census==1.12.0