-
Notifications
You must be signed in to change notification settings - Fork 7
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
Change of default legacy
in edgeR::glmQLFit()
with edgeR v4.0.7 changes results in cluster-abundance.Rmd
#16
Comments
legacy=FALSE
in edgeR::glmQLFit.default with edgeR v4.0.7 changes results in cluster-abundance.Rmdlegacy
in edgeR::glmQLFit.default with edgeR v4.0.7 changes results in cluster-abundance.Rmd
legacy
in edgeR::glmQLFit.default with edgeR v4.0.7 changes results in cluster-abundance.Rmdlegacy
in edgeR::glmQLFit()` with edgeR v4.0.7 changes results in cluster-abundance.Rmd
legacy
in edgeR::glmQLFit()` with edgeR v4.0.7 changes results in cluster-abundance.Rmdlegacy
in edgeR::glmQLFit()
with edgeR v4.0.7 changes results in cluster-abundance.Rmd
Ah, thanks for alerting me. Absolutely it was not my intention to change any edgeR defaults in the release version of edgeR. Six days ago I was fixing a bug in the I have now repaired my mistake and reset the default back to |
@LTLA @alanocallaghan If you're able to wait for the edgeR 4.0.10 to percolate through, the issue will be fixed and there is no need to make any changes to OSCA.multisample. |
Thank you very much for the prompt reply, Gordon. We'll still need to decide what to do in BioC 3.19. |
In 9a50f0d I just stuck with |
From edgeR news file:
Recently, the default changed from
legacy=TRUE
tolegacy=FALSE
in the release branch (BioC 3.18): https://code.bioconductor.org/browse/edgeR/commit/1f0de5e1fa24e436315e13fad517e1bdac502fdd@gksmyth: It's a bit unexpected for the default value to change in the release version, so I'd like to confirm that this change was intended for the release version (BioC 3.18) and not just the devel version (BioC 3.19) of edgeR?
@LTLA, @alanocallaghan:
With regards to
inst/book/cluster-abundance.Rmd
in OSCA.multisample, we can revert to the previous behaviour just by addinglegacy=TRUE
in the appropriate places which seems the best solution for BioC 3.18 (I'll make a PR).For BioC 3.19, we can either adapt the text to the new results (i.e. use the default
legacy=FALSE
) or stick with the old results and text (i.e. also makelegacy=TRUE
).Example based on
inst/book/cluster-abundance.Rmd
Extract example and setup DGEList object
Performing the DA analysis
Assuming most labels do not change
Removing the offending labels
Testing against a log-fold change threshold
Session info
The text was updated successfully, but these errors were encountered: