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

Update bgd to bw conversion in callpeak_macs2.bds #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

florianduclot
Copy link

The bedgraph to BigWig conversion step throws an error on my system for all my files:
(...)pooled.tagAlign.fc.signal.srt.bedgraph is not case-sensitive sorted at line 1879. Please use "sort -k1,1 -k2,2n" with LC_COLLATE=C, or bedSort and try again.
As mentioned in the error message and at https://gist.github.com/taoliu/2469050, using LC_COLLATE=C fixes the issue on my system when the command is ran separately (outside of chipseq.bds pipeline).

Would this addition cause conflict with the rest of the code?

Thanks,

The bedgraph to BigWig conversion step throws an error on my system for all my files:
"(...)pooled.tagAlign.fc.signal.srt.bedgraph is not case-sensitive sorted at line 1879.  Please use "sort -k1,1 -k2,2n" with LC_COLLATE=C,  or bedSort and try again."
As mentioned in the error message and at https://gist.github.com/taoliu/2469050, using "LC_COLLATE=C" fixes the issue on my system when the command is ran separately (outside of chipseq.bds pipeline).

Would this addition cause conflict with the rest of the code?

Thanks,
@leepc12
Copy link
Contributor

leepc12 commented Jun 9, 2016

The funciton already has LC_COLLATE=C in line 67 of https://github.com/florianduclot/TF_chipseq_pipeline/blob/ca31df6fb4f484156eb7c098be3f938158141498/modules/callpeak_macs2.bds

But it was not exported so I modified it to export LC_COLLATE=C (in the commit 082c4a7).

@florianduclot
Copy link
Author

Sorry I didn't notice this line before.

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants