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

Not clear from the readme, that samplenames MUST be different! #14

Open
serge2016 opened this issue Mar 10, 2017 · 4 comments
Open

Not clear from the readme, that samplenames MUST be different! #14

serge2016 opened this issue Mar 10, 2017 · 4 comments

Comments

@serge2016
Copy link

The program crashes if there are identical samplenames in the input file (2nd column).
It is very strange, because as I understand, this software does support PE reads.

@slzhao
Copy link
Owner

slzhao commented Mar 11, 2017

Did you mean in fastq QC or bam QC?

@serge2016
Copy link
Author

I got this error with fastq QC.

@slzhao
Copy link
Owner

slzhao commented Mar 14, 2017 via email

@serge2016
Copy link
Author

Hello, Slzhao!
In the same readme there is:

	#An example of input file for fastq QC. The labels are optional.
	#Column1	    Column2
	sample1Pair1File	labelForThisFile
	sample1Pair2File	labelForThisFile
	Sample2Pair1File	labelForThisFile

Add, please, something unique to the labels, and mention it in the text... It would be more clear, as I think.

Also the readme now is very long. Maybe it would be better to extract the changelog from it to separate file?

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

No branches or pull requests

2 participants