Added data density-aware zoom selection for requesting bigwig data for trackster via byte-range.

[msauria]

I have limited data calls to between 1000-4000 data points per canvas and created a promise chain for resolving which zoom level ultimately returns data. This should fix issue #3854 without breaking for high-density data.

[martenson]

Hi @msauria - sorry for taking this long, we have just had a discussion about this PR yesterday and we concluded there is not a person (in the meeting) that understands/uses trackster enough to review this.

Hopefully we will do better for the 18.01.

[jxtx]

@jgoecks should be able to review this. He's the only one who has touched this specific code.

[martenson]

The review of this should probably follow the merge of https://github.com/galaxyproject/galaxy/pull/5261

Therefore I am pushing it to 18.05.

[dannon]

After I wrap up #5261, I'll also have touched this code and should be able to get this reviewed/merged.

[jxtx]

Thank you!

On Fri, Jan 5, 2018 at 5:57 PM Dannon Baker [email protected] wrote:

After I wrap up #5261 https://github.com/galaxyproject/galaxy/pull/5261, I'll also have touched this code and should be able to get this reviewed/merged.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/galaxyproject/galaxy/pull/4246#issuecomment-355688257, or mute the thread https://github.com/notifications/unsubscribe-auth/AAE4ZYDzDFHrWjNydXwAfzTjweVb-1-Bks5tHqjfgaJpZM4OGl6n .