-
Notifications
You must be signed in to change notification settings - Fork 15
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
alr transformation method? #24
Comments
I get the same error message even when I specify |
Supplying a vector of indices also didn't work and the same error message appears: |
Hi. I have tried to reproduce this with the selex dataset and the current version of ALDEx2 from bioconductur (1.20.0), which should be the same as 1.19.5. The following code runs fine: library(ALDEx2) can you provide me with a reproducible example? sessionInfo() Matrix products: default locale: attached base packages: other attached packages: loaded via a namespace (and not attached): |
The problem seems to occur when a model matrix is provided to
R version 4.0.2 (2020-06-22) Matrix products: default locale: attached base packages: other attached packages: loaded via a namespace (and not attached): |
the aldex.effect function is now fully compliant with an ALR denominator. |
Hi, I am still having the issue mentioned by @aimirza. When using a model matrix, the only denominator that works for aldex.clr() is "all". I would like to use "iqlr" and "lvha", but this results in the error about supplying a vector of indices. This error can be replicated by running the code in section 4.6 (Complex study designs and the aldex.glm module) of the Bioconductor vignette for the selex dataset. Any suggestions appreciated! |
@deniando Did you manage to figure this out? I am running into the same issues with aldex.clr() and aldex.glm() |
Hi @roshonda-jones, no unfortunately I could not get this working. |
I'm sorry for this, I have a short-term fix and can push a real fix to the next release. The issue is that the value passed must be an integer. So you can do the following: |
if you want to use the LVHA or IQLR, then you will need to choose the indices of the denominator manually via pairwise comparisons. The indices of the denominator will be in the aldex.clr object (x) as x@denom This is something I will fix as well (again) in the next release |
Thanks for the advice @ggloor and the intended update. |
Thanks this worked for me. |
I would like to use additive log ratio (alr) transformation. Does ALDEx2 support this? I tried setting
denom
to an index of the feature I wanted to use as the denominator but the error saysError in aldex.clr.function(reads, conds, mc.samples, denom, verbose, : please supply a vector of indices for the denominator
. I had previously installed aldex2 from a different source and I was able to specific a single index for the parameterdenom
. After uninstalling and then installing usingdevtools::install_github("ggloor/ALDEx_bioc")
, I can no longer specify a single feature index.I am currently using aldex version 1.19.5
The text was updated successfully, but these errors were encountered: