-
Notifications
You must be signed in to change notification settings - Fork 14
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
max chrom size too small #66
Comments
To make it a bit more concrete. |
Honestly, I'm not sure what the repercussions are. I would try querying the pairs file directly with pairix using out-of-bounds coordinates and see what it does or wait to hear back from the pairix maintainers. I'm curious, would plain old cload pairs (not pairix) not work in your case? It does a two-pass ingestion using mergesort without requiring an index or even sorted pairs. Or do you run into performance issues? |
Meant to post that in the cooler issue, but whatevs... |
Yes, I already thought about the out-of-bounds querying and wanted to try it today, but didn't manage to. However, I will do so tomorrow and let you know. As far as I can remember I never tried to use cload pairs because we decided to go with pairix due to the nice querying with pypairix, but I will also look into this. Thanks anyway |
I'll have to modify pairix to add an option to increase max chrom size. Just curious, what specie is it? |
Thanks for moving so quickly. It's axolotl and it defies every heuristic you have learned for processing genomic data ^^ |
That's cool! I'll take a look at it next couple of days and let you know. |
I also leave this here: |
Hi there,
I am using pairix in conjunction with cooler to generate cool files for my Hi-C data. Pairix generation seemed to work without an issue using
cooler csort pairix
. However, I am now generating the base resolution withcooler cload pairix
and getting maximum chromosome size warnings since some of chromosomes are larger than 2^30.Does this have an effect on my results and can this be resolved in your implementation to also support larger chromosomes?
These are the sizes:
Thanks
The text was updated successfully, but these errors were encountered: