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

[r] use new SOMATileDBContext$context() accessor #357

Merged
merged 2 commits into from
Apr 6, 2023

Conversation

mlin
Copy link
Contributor

@mlin mlin commented Apr 5, 2023

cf. single-cell-data/TileDB-SOMA@8691ce0

also incidentally fix deferred FIXME regarding UTF-8 error in value_filter

cf. single-cell-data/TileDB-SOMA@8691ce0

also incidentally fix deferred FIXME regarding UTF-8 error in value_filter
@mlin mlin requested a review from atolopko-czi April 5, 2023 20:25
@@ -1,6 +1,6 @@
# A known-good Cell Census version. This may need updating if the version used
# is withdrawn for any reason.
KNOWN_CENSUS_VERSION <- "2023-02-13"
KNOWN_CENSUS_VERSION <- "2023-04-04"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! This will be less of a problem once have a LTS (5 year) release.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not just designate one build to be a "long term test build", give it a tag, and not delete it? I don't think we need a "real" LTS release for this purpose....

Copy link
Collaborator

@atolopko-czi atolopko-czi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mlin mlin merged commit e379871 into main Apr 6, 2023
@mlin mlin deleted the mlin/r-use-context-accessor branch April 6, 2023 01:05
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.

3 participants