Skip to content

ENH add fit size factors type as an argument of DeseqDataSet#345

Merged
BorisMuzellec merged 5 commits intoscverse:mainfrom
regloeb:regis/fit_size_factors_type_argument
Dec 2, 2024
Merged

ENH add fit size factors type as an argument of DeseqDataSet#345
BorisMuzellec merged 5 commits intoscverse:mainfrom
regloeb:regis/fit_size_factors_type_argument

Conversation

@regloeb
Copy link
Contributor

@regloeb regloeb commented Dec 2, 2024

Reference Issue or PRs

What does your PR implement? Be specific.

@BorisMuzellec
Copy link
Collaborator

LGTM, thanks @regloeb!

A minor comment: for consistency wrt vst_fit_type, could you rename fit_size_factors_type as size_factors_fit_type?

@BorisMuzellec BorisMuzellec changed the title add fit size factors type as an argument of DeseqDataSet ENH add fit size factors type as an argument of DeseqDataSet Dec 2, 2024
@regloeb
Copy link
Contributor Author

regloeb commented Dec 2, 2024

done !
thanks for the quick review @BorisMuzellec

@BorisMuzellec BorisMuzellec merged commit 4b9d19a into scverse:main Dec 2, 2024
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