Added alternative first entry for SRA accession SRP064605 #120
Conversation
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](https://trace.ncbi.nlm.nih.gov/Traces/sra/?run=SRR2584403) (which was the first run). Think it will confuse folks if the run they are told to look for isn't there.
|
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?) |
Modified to just clicking on the first run in response to pull request comment
added a little more context on *what* table we are referring to
|
Cool, all done @hoytpr! @rltillett 's solution could be a good solution too |
|
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 |
|
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? |
Added alternative first entry for SRA accession SRP064605
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.