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

cell_census.get_anndata should return obs columns and var columns used for filtering #56

Closed
pablo-gar opened this issue Dec 23, 2022 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@pablo-gar
Copy link
Contributor

Issue

SOMADataFrame.read_as_pandas_all() retrieves all columns used for filtering for example:

image

in this case feature_id was included in the final object as it was used for filtering

However for the cell_census.get_anndata() method, this is not the case:

image

Solution

In cell_census.get_anndata() all columns used for filtering in obs_value_filter and var_value_filter must be included in the corresponding anndata.AnnData.obs and anndata.Anndata.var respectively

@pablo-gar pablo-gar added the bug Something isn't working label Dec 23, 2022
@bkmartinjr bkmartinjr self-assigned this Dec 24, 2022
@atolopko-czi
Copy link
Collaborator

atolopko-czi commented Jan 4, 2023

There is some effort involved with implementing this, as it requires parsing the declarative query string. We do have the BNF grammar for this, but is it worth doing this?

Also, I think there will be cases where a user might not want all of the query columns in the result.

@bkmartinjr
Copy link
Contributor

I think this is a SOMA bug/inconsistency - shouldn't be hard to fix. I will get to it soon.

@bkmartinjr
Copy link
Contributor

Root cause: single-cell-data/TileDB-SOMA#755

@bkmartinjr
Copy link
Contributor

this is fixed in tiledbsoma, and will be available in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants