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 tutorial.md #5602

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shrutikhare-git
Copy link

TODO:

  1. Step "3. Rename the datasets" can be updated to specify why this is required.
  2. "Warning: Not all browsers work!" can be updated to suggest some browsers that do work.

@@ -63,10 +63,10 @@ In this tutorial will use the following datasets:
## [A] Marker Genes:

We'll start with two input datasets of marker genes (Study sets):
* **Marker genes per cell cluster:** This dataset lists the genes that are significantly different in each cell cluster.
* **Marker genes per condition (wt and ko):** This dataset lists the genes that are significantly different between the wild-type (wt) and knockout (ko) conditions.
* **Marker genes per cell cluster:** This dataset lists the genes that are differentially enriched different in each cell cluster.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* **Marker genes per cell cluster:** This dataset lists the genes that are differentially enriched different in each cell cluster.
* **Marker genes per cell cluster:** This dataset lists the genes that are differentially enriched in each cell cluster.

@@ -134,7 +134,7 @@ To perform GO enrichment analysis on each cell cluster individually, we need to
> 1. {% tool [Split file](toolshed.g2.bx.psu.edu/repos/bgruening/split_file_on_column/tp_split_on_column/0.4) %} with the following parameters:
> - {% icon param-file %} *"File to select"*: `Markers_cluster` (Input dataset)
> - *"on column"*: `c1`
> - *"Include the header in all splitted files?"*: `Yes`
> - *"Include the header in all split files?"*: `Yes`
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
> - *"Include the header in all split files?"*: `Yes`
> - *"Include the header in all splitted files?"*: `Yes`

Copy link
Collaborator

Choose a reason for hiding this comment

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

Unfortunately, the tool itself says 'splitted', so we have to make sure the tutorial text matches the tool text in the Graphical User Interface / Galaxy interface, otherwise people can get confused.

@nomadscientist
Copy link
Collaborator

Some great catches here, thanks! Check the suggestions above and commit or change, and we should be able to review and merge this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants