arq5x / scurgen

A tool for detecting patterns in genomic data with space filling curves
9 stars 4 forks source link

GUI updates, bigWig support, docs, and more #33

Closed daler closed 11 years ago

daler commented 11 years ago

Big chunk o' commits, but I think the end result is a substantial improvement. The bigWig support speeds things up nicely since data are essentially pre-binned.

Summary of changes:

Example of GUI using bigWig input, with chrom boundaries: example screenshot

arq5x commented 11 years ago

Looks like a fantastic addition! I merged but can't run because I don't have bx.bbi.bigwig_file. Do you know what should be added to setup.py to auto-install this?

arq5x commented 11 years ago

I added bx-python to setup.py and am in business. Pushed to repo.

arq5x commented 11 years ago

It looks like precomputed.npz is not installed - I get a IOError: [Errno 2] No such file or directory: '/Library/Frameworks/Python.framework/Versions/7.3/lib/python2.7/site-packages/scurgen-0.1.0-py2.7.egg/scurgen/precomputed.npz' message. Is this a static file that should be installed with the package?

arq5x commented 11 years ago

It looks like there is an expectation that the chrom2rc method is available in all HilbertMatrix* objects. I ran the GUI with a BED file as an input, but it failed when calling chrom2rc, which is present in the BigWig subclass. At first glance it seems that this method should be in the base class?

daler commented 11 years ago

I haven't figured out the best way to make precomputed.npz . . . I think ideally it would be when running setup.py upon first install. For now though, run precompute.py to make the .npz file.

daler commented 11 years ago

Ah right, I forgot that I used chrom2rc when making the chrom outlines. You're right, it's no longer bigWig-specific and should be in a base class