Skip to content

corrected paths + minor embellishments#101

Open
evezeyl wants to merge 1 commit into
masterfrom
evfi-data_pre_processing
Open

corrected paths + minor embellishments#101
evezeyl wants to merge 1 commit into
masterfrom
evfi-data_pre_processing

Conversation

@evezeyl

@evezeyl evezeyl commented Jan 16, 2019

Copy link
Copy Markdown
Contributor

corrected paths (new organization)

corrected paths (new organization)
Comment thread data_pre_processing.md
@@ -11,7 +11,7 @@ Dataset used during this session can be found in the following location within a

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should probably move the test data to /projects too. Also, maybe remove the Dec14 tag? That would make it more independent.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok - we can remove tag, then we have to rename the files on Abel. - has to be consistent. Can rename when done.
So: pending

Comment thread data_pre_processing.md
We wll use [Trimmomatic](http://www.usadellab.org/cms/index.php?page=trimmomatic) to trim/remove adapter and low quality reads.
This tool is NOT available via _module load_ in abel but available at _/work/projects/nn9305k/bin/_. Make sure you know where the adapter sequences are available.
This tool is NOT available via _module load_ in abel but available at _`/work/projects/nn9305k/bin/`_. Make sure you know where the adapter sequences are available.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We have gone over to use conda. Could you check if trimmomatic is conda available, and if so, update the text?

@karinlag

Copy link
Copy Markdown
Member

Looks mostly good, just a few suggestions that relate to the bigger picture 🙂

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.

2 participants