search

arq5x / scurgen

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

Yet another GUI refactor #32

Closed daler closed 11 years ago

daler commented 11 years ago

Chroms split across multiple axes, checkboxes for toggling visibility, and a general design overhaul to make it much easier to add new stuff on a chrom-specific and/or data file-specifc basis

arq5x commented 11 years ago

Thanks Ryan, I merged this manually because of a few small conflicts. Quick question, when running `scurgen gui config.yaml', I see the checkboxes for toggling display, but in my hands, they seem to affect the display of the color scales, not the data. Do you see otherwise?

daler commented 11 years ago

Hmm, it works for me, using your merged version.

But I realized the CpG islands, using "Blues" is difficult to see even when it's on. Toggling the 3rd (phastcons, "Spectral_r") makes a really obvious difference when it's toggled off.

Upon clicking a checkbox to toggle off, does the alpha slider for that data set go down to zero -- and then reset to its previous value when it's toggled back on again?

arq5x commented 11 years ago

Odd. Yeah, the alpha sliders respond as expected, but I see no change in the plot itself. Instead, the color scales toggle to all white (off) and then back to the original color scale. It's as if the scale, not the data, is being toggled.

On Oct 1, 2012, at 10:14 AM, Ryan Dale notifications@github.com wrote:

Hmm, it works for me, using your merged version.

But I realized the CpG islands, using "Blues" is difficult to see even when it's on. Toggling the 3rd (phastcons, "Spectral_r") makes a really obvious difference when it's toggled off.

Upon clicking a checkbox to toggle off, does the alpha slider for that data set go down to zero -- and then reset to its previous value when it's toggled back on again?

— Reply to this email directly or view it on GitHub.

daler commented 11 years ago

Does this work any better?: https://github.com/daler/scurgen/commit/dbaee35db70fac9a460b94f716e08a7df099ed81

arq5x commented 11 years ago

Strangely, no. I can get it to trigger the plot refresh by toggling between log and linear. But if I just use the checkboxes, the plots are not updated even though they call plt.draw() as well.

arq5x commented 11 years ago

Clearly this seems to be a platform difference and will be difficult for you to replicate. Once I get a paper out that I am obsessing over, I will will get back to this and try to isolate the problem. Thanks for the nice improvement to the GUI.

daler commented 11 years ago

Hmm, that's frustrating. Similarly, when I get some time I'll work on testing both scurgen & pbt on multiple platforms as well.