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

[misc] Prepare for 1.12.0 release #1061

Merged
merged 5 commits into from
Mar 28, 2024
Merged

Conversation

prathapsridharan
Copy link
Contributor

@prathapsridharan prathapsridharan commented Mar 21, 2024

This release of the census package is being done to include an important update for tiledbsoma in 1.9.0 which includes a fix to categorical columns. To ensure that this release goes smoothly, we will:

  1. Run an acceptance test off TileDB-SOMA main branch which contains the fixes that will go into 1.9.0. We can think of this as a pre-release acceptance test
  2. After (1) passes and the tiledb folks officially release tiledbsoma==1.9.0, we will update the dependencies in this PR to point to tiledbsoma==1.9.0 and do the official release of cellxgene-census==1.12.0

@prathapsridharan prathapsridharan marked this pull request as draft March 21, 2024 20:49
Copy link

codecov bot commented Mar 25, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 81.33%. Comparing base (aa1439f) to head (b1a300b).

Files Patch % Lines
...us/src/cellxgene_census/experimental/_embedding.py 0.00% 1 Missing ⚠️
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           
Flag Coverage Δ
unittests 81.33% <50.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -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
Copy link
Contributor

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)

Copy link
Contributor Author

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

Copy link
Contributor

@bkmartinjr bkmartinjr left a 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

@prathapsridharan prathapsridharan marked this pull request as ready for review March 28, 2024 15:29
@prathapsridharan prathapsridharan merged commit a5dbdef into main Mar 28, 2024
15 checks passed
@prathapsridharan prathapsridharan deleted the psridharan/census-rel-1.12.0 branch March 28, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants