Skip to content

ENH implement a getter for normalized counts#79

Merged
BorisMuzellec merged 6 commits intomainfrom
normed_counts_getter
Feb 20, 2023
Merged

ENH implement a getter for normalized counts#79
BorisMuzellec merged 6 commits intomainfrom
normed_counts_getter

Conversation

@BorisMuzellec
Copy link
Copy Markdown
Collaborator

Reference Issue or PRs

Fixes #77.

What does your PR implement? Be specific.

This PR implements a getter for DeseqDataSet's normalized counts.

Thanks to the @property decorator, normalized counts can be obtained as if they were a DeseqDataSet attribute (dds.normed_counts), without having to store them in memory. Instead, they are computed on-the-fly.

@BorisMuzellec BorisMuzellec requested review from a user and maikia February 15, 2023 08:56
Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Thanks @BorisMuzellec ! I have tiny remarks

BorisMuzellec and others added 2 commits February 15, 2023 12:05
Co-authored-by: mandreux-owkin <62643750+mandreux-owkin@users.noreply.github.com>
@BorisMuzellec
Copy link
Copy Markdown
Collaborator Author

BorisMuzellec commented Feb 16, 2023

Actually, I think that with the AnnData integration I think that the most natural way to access normalized counts is by setting a new layer, although this induces a memory overhead

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

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

LGTM! Much simpler now

@BorisMuzellec BorisMuzellec merged commit 58438da into main Feb 20, 2023
@BorisMuzellec BorisMuzellec deleted the normed_counts_getter branch February 20, 2023 08:57
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.

How to get normalized expression values

1 participant