dasmoth / dalliance

Interactive web-based genome browser.
http://www.biodalliance.org/
BSD 2-Clause "Simplified" License
226 stars 68 forks source link

Workaround / better error message for "Bumping limit exceeded" #145

Closed ryan-williams closed 9 years ago

ryan-williams commented 9 years ago

Ported from cycledash #546

I see that error message frequently, don't know what it means, and don't know what if anything I can do to fix/work around it.

ymen commented 9 years ago

cc @dasmoth

From what I understand, the "bumping limit" is a limit on the number of "rows" that the track is allowed to display. I think this was just to prevent the unlikely case that a BAM file has too many alignments and the browser crash when trying to render all of them. But when you see this error message, there are features in the BAM track that aren't rendered, which can be quite problematic!

In terms of getting around the problems:

The newer version has an option in the track editor, which can be used to configure this interactively. (ref https://github.com/dasmoth/dalliance/issues/65#issuecomment-49027714)

Before screenshot 2015-03-18 at 9 11 27 am

After changing to bump limit to 1000 screenshot 2015-03-18 at 9 12 17 am

From experience though, I usually just set subtierMax property when adding a track programmatically to a really large number, or overwrite the default subtierMax here (ref https://github.com/dasmoth/dalliance/issues/65#issuecomment-47210549)

Hope this helps, Thomas can correct me if I'm wrong :)

dasmoth commented 9 years ago

Thanks @ymen, that's all correct. I'll just mention that the "Bump overlaps" option in the track editor is simply a UI for setting subtierMax.

keithcallenberg commented 8 years ago

@ymen called the situation in which this would occur "unlikely", but imagine that you are displaying targeted sequencing data and regularly have 2000x coverage? In this context, it is not practical to increase the subtierMax high enough that this warning/error message goes away; if we want to visualize individual reads, we must be happy with downsampling. Is there any way we could either expose a config parameter to disable this "Bumping limit" message, or make it somehow less obtrusive (it covers the top 1-2 reads). I am not well versed in js, but if I was able to make that change, do you think it could be merged @dasmoth?

For reference, IGV downsamples reads by default and does not tell the user (http://www.broadinstitute.org/igv/Preferences#Alignments).