-
Notifications
You must be signed in to change notification settings - Fork 45
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
Flipping coordinates for biplot ordination #258
Comments
Wouldn't it make more sense for the visualizers to do the flipping as appropriate? Otherwise I worry that this could enable "illicit" actions, e.g., if passing those ordination results to a method that should not operate on flipped samples/loadings. I see that this makes more sense for feature-specific stats, but keep in mind that rhapsody is currently making use of the |
Good point @nbokulich . One option along those lines is to have an I'm not sure how I feel about defining a |
@thermokarst @nbokulich moving this to emperor and eventually qurro, since it is probably a better place to put this |
Addition Description
One thing that could become increasing helpful with the introduction of feature-specific stats (i.e. songbird, aldex2, rhapsody, qurro) is the ability to flip ordinations.
For example, if someone wants to analyze their metabolites in a qurro rank plot, they would need to first flip their ordination coordinates (i.e. swap sample coordinates with feature coordinates) so that they can have metabolites as "features".
Another use case is if someone wants to visualize the microbe co-occurrences learned from DEICODE - it'll require the features to be plotted instead of the samples.
Given that this will likely be a recurring pattern, it may make sense to have this "flipping" operation further upstream.
Current Behavior
Doesn't exist at the moment.
Proposed Behavior
I've written a simple CLI script that can do this at the moment - but if there is interest, I'll be happy to make a contribution to q2-diversity
References
https://github.com/biocore/qurro
https://github.com/biocore/DEICODE
https://msystems.asm.org/content/4/1/e00016-19
CC @fedarko @cameronmartino @ElDeveloper
The text was updated successfully, but these errors were encountered: