Skip to content
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

Update tutorials #30

Merged
merged 9 commits into from
Jul 21, 2020
Merged

Update tutorials #30

merged 9 commits into from
Jul 21, 2020

Conversation

cameronmartino
Copy link
Collaborator

Added three tutorials on the exact same time series IBD dataset but using:

  • Python API
  • QIIME2 API
  • QIIME2 CLI

@cameronmartino
Copy link
Collaborator Author

@gwarmstrong Okay, this should be good to go now.

Comment on lines +71 to +79
**Output:**
```bash
Saved DistanceMatrix % Properties('phylogenetic') to: IBD-2538/core-metric-output/unweighted-unifrac-distance.qza
Saved DistanceMatrix % Properties('phylogenetic') to: IBD-2538/core-metric-output/weighted-unifrac-distance.qza
Saved PCoAResults to: IBD-2538/core-metric-output/unweighted-unifrac-distance-pcoa.qza
Saved PCoAResults to: IBD-2538/core-metric-output/weighted-unifrac-distance-pcoa.qza
Saved Visualization to: IBD-2538/core-metric-output/unweighted-unifrac-distance-pcoa.qzv
Saved Visualization to: IBD-2538/core-metric-output/weighted-unifrac-distance-pcoa.qzv
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was confused by this at first. Maybe this is holdover from the jupyter notebook, where all of these bash commands could have been run in the same cell?
Can you put each output with the corresponding bash command? A la Q2 docs: https://docs.qiime2.org/2020.2/tutorials/moving-pictures/


# Compositional Tensor Factorization (CTF) Introduction

In order to account for the correlation from a subject to thierself we will compositional tensor factorization (CTF). CTF builds on the ability to account for compositionality and sparsity using the robust center log-ratio transform covered in the RPCA tutorial (found [here](https://forum.qiime2.org/t/robust-aitchison-pca-beta-diversity-with-deicode)) but restructures and factors the data as a tensor. Here we will run CTF through _gemelli_ and explore/interpret the different results.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change "thierself" to "theirself" here, again.

ipynb/tutorials/IBD-Tutorial-QIIME2-CLI.md Outdated Show resolved Hide resolved
Saved FeatureData[FeatureTrajectory] to: IBD-2538/ctf-results/state_feature_ordination.qza
```

We will now cover the output files being:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
We will now cover the output files being:
We will now cover the following output files:

mf.to_csv('IBD-2538/data/subject-metadata.tsv', sep='\t')
```

With out `subject-metadata` table build we are not ready to plot with emperor.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
With out `subject-metadata` table build we are not ready to plot with emperor.
With our `subject-metadata` table build we are now ready to plot with emperor.

* state_subject_ordination
* state_feature_ordination

First, we will explore the `state_subject_ordination`. The subject trajectory has PC axes like a conventional ordination (i.e. PCoA) but with time as the second axis. This can be visualized through the existing q2-longitudinal plugin.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe for each of the out put files, you could do sub-headings in like H4 or something? To make each artifact a little easier to find.

E.g.

state_subject_ordination

First, we will explore the state_subject_ordination. ...

@gwarmstrong gwarmstrong merged commit 825beea into master Jul 21, 2020
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.

None yet

3 participants