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

Add samples from SRP018247 #13

Closed
6 of 7 tasks
k8hertweck opened this issue Jul 18, 2017 · 7 comments
Closed
6 of 7 tasks

Add samples from SRP018247 #13

k8hertweck opened this issue Jul 18, 2017 · 7 comments
Labels

Comments

@k8hertweck
Copy link
Owner

k8hertweck commented Jul 18, 2017

  • update SRP018247.lst

  • uncomment lines 16-19 in mothur_prep.sh

  • update line 39 in mothur_prep.sh

  • update line 45 in mothur_prep.sh

  • update line 48 in mothur_prep.sh

  • run mothur_prep.sh to test

  • checking parameters in mothur_otu.bat

@aortiz24
Copy link
Contributor

I have updated SRP018247.lst and uncommented lines 16-19 in my new branch called "New_samples".

I am unclear as to the updates I need to make to line 39, 45, and 48 in mothur_prep.sh. Clarification is needed and appreciated.

I ran dataDownload.sh before I tried to run mothur_prep.sh. I ran into a problem:
fastq-dump: command not found, I think this error is popping up because SRATool kit is not installed on the mac mini.

@k8hertweck
Copy link
Owner Author

Great!

Regarding lines 39, 45, and 48: these lines execute the mothur commands to prep the combined/merged files. You need to add the extra samples from SRP018247.

Do you still have the original fastq file downloads? You can manually move them to the data folder. Otherwise, let me know and I'll email you the links to redownload from my Dropbox. I'll also install the software on the mac mini.

@k8hertweck
Copy link
Owner Author

Could you please check and make sure SRP018247.lst matches the files Katrin wanted included in this analysis? If you go to the data overview spreadsheet in Google Drive, the "design" spreadsheet includes which samples she wanted excluded; these should be removed from the list of samples.

k8hertweck added a commit that referenced this issue Aug 2, 2017
Added phylogenetic code and resolved issue #13
@aortiz24 aortiz24 mentioned this issue Sep 11, 2017
@aortiz24
Copy link
Contributor

I have completed the top five tasks this issue #13 and have launched a PR.

@aortiz24
Copy link
Contributor

I have ran mothur_prep.sh to test the analysis and it is running smoothly(6th task on issue #13). I double checked that the samples highlighted in the dataoverview doc were excluded. With your approval, we could be ready to run the mothur_prep.sh for SRP018246 and SRP18247!

@k8hertweck
Copy link
Owner Author

I think there were a few samples assigned incorrectly, but they should be good to go now. I'm merging and doing a test run; could you double check?

@k8hertweck
Copy link
Owner Author

Last checkbox here resolved by #30

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

No branches or pull requests

2 participants