-
Notifications
You must be signed in to change notification settings - Fork 1
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
listDBKeyfile.sh - problem with Tassel v5 format #23
Comments
Hi sorry about the delay – I’ve done this – see before and after below. If you could confirm this fixes AMCC before: ./listDBKeyfile.sh -s SQ2701 -v 5 > SQ2701_old.txt patch: iramohio-01$ git diff
after: ./listDBKeyfile.sh -s SQ2701 -v 5 > SQ2701_new.txt diff of before and after : iramohio-01$ diff SQ2701_old.txt SQ2701_new.txt
|
Just realised that some columns in keyfile have been changed. Is this to comply with the Tassel requirements? I have tried a few keyfiles. To reproduce what I reported here I use an example SQ2708 from the database: julia> keyInfo[11:17] I am OK to adapt to the new changes. But in its current form, the column 4 and 17 are of the same name ("FullSampleName" vs "fullsamplename"). I would suggest use the standard header as suggested in the official documents (Please see the Link), with optional customized information as you currently include. “DNASample” should be the sample names as submitted to the sequencing lab. And the “FullSampleName” is required, but in many case, it may be upon to the uses (like me) do the due diligence, for the specific purpose. I am happy to discuss. |
There’s a slight bug in the listDBKeyfile.sh script when generating T5 key files. T5 is incredibly picky when it comes to the column headers and not having them right causes the program not to run. Could you please amend the script so the header has the following capitalised fields?
Flowcell
Lane
Barcode
FullSampleName
The text was updated successfully, but these errors were encountered: