search

datacarpentry / organization-genomics

Project Organization and Management for Genomics
https://datacarpentry.org/organization-genomics
Other
23 stars 76 forks source link

Added alternative first entry for SRA accession SRP064605 #120

Closed laninsky closed 5 years ago

laninsky commented 5 years ago

REL4541B was not the first entry in the SRA Run Selector Table when I viewed the link (it was actually the last), so also added REL762B (which was the first run). Think it will confuse folks if the run they are told to look for isn't there.

hoytpr commented 5 years ago

Hi @laninsky and thanks very much for catching this! It looks like this happened because the entire SRA is moved to "the cloud". This was bound to happen anyway, because more data will be added as the experiment continues!

Rather than look for "REL4541B" or "REL762B", maybe we should simply say "the first" Run Number as a longer term solution otherwise this will keep happening. Could you make that change to your PR? Then we could incorporate it. I appreciate your help with this, and we should leave this open for comments.

Comments from @raynamharris , or @mkuzak welcome (Other maintainers names may have changed?)

laninsky commented 5 years ago

Cool, all done @hoytpr! @rltillett 's solution could be a good solution too

hoytpr commented 5 years ago

Hi @laninsky , and thanks. After going through the process and performing the search as @rltillett suggests in #121 , I like the search idea. It works and maintains the consistency of the lesson for downstream use. Could you and @rltillett make this happen? It would be a nice contribution!

Peter

laninsky commented 5 years ago

Hi Peter - definitely keen to help out on this but probably won't be for a little while due to my work commitments. In the meantime, might we make this change just to reduce confusion in the short-term?