Leo Lahti

Results 463 comments of Leo Lahti

Use the "@inheritParams" roxygen functionality for this.

Please close after the PR is merged.

Perhaps my mistake - I reopened it.

@Insaynoah what's the status of this PR? Can we get it fixed.

@nuorenarra could you briefly describe the issue here?

We could consider adding calculateDistance; or it could be optional to return that as part of runMDS output. But that is a separate issue. I suggest we open an issue...

@nuorenarra when time allows, could _you_ make the suggested PR to the documentation in mia and OMA on this?

Interestingly, the [qiime2R](https://github.com/jbisanz/qiime2R) package seems to work. Something to learn from there? ``` remotes::install_github("jbisanz/qiime2R") library(qiime2R) d

We are now adding TSE support for qiime2R; perhaps this solves the issue and we could switch to using qiime2R functions instead of maintaining parallel mia implementation? This is currently...

Support for TreeSE has been added now in [qiime2R PR 64](https://github.com/jbisanz/qiime2R/pull/64). However that pkg is [not yet in Bioc or CRAN](https://github.com/jbisanz/qiime2R/issues/3)