Add option to render all connected components at once #93

fedarko · 2016-09-27T15:39:37Z

I guess this would be done in the viewer. We could do this by taking the bounding boxes for each component and "concatenating" them, with some margins in between. (For some reason, laying out components individually seems to be faster than laying out the entire graph at once—so we'd still generate layouts on a component-by-component basis, but just concatenate the resulting layouts to produce something basically equivalent.

This probably shouldn't be the default behavior (will wreck viewer performance for huge graphs that have sizable non-largest components) but having it as an option would be useful in some cases (e.g. graphs composed of a lot of tiny component fragments).

The scaling applies to whatever's being drawn, so it adjusts as different components/etc. are drawn. When we implement the ability to draw multiple components at once (see #93), we can adjust this to scale accordingly to the increased size. This results in a pretty huge speedup, which is great! I think the slightly choppy progress bar animations are more than an alright cost, and we can probably look into making that a bit smoother in the future (but that's super low priority relative to tons of other stuff).

fedarko · 2017-08-09T21:41:57Z

This issue was moved to marbl/MetagenomeScope#16

fedarko added the lowpriorityfeature label Sep 27, 2016

fedarko self-assigned this Sep 27, 2016

fedarko changed the title ~~Add option to render all components at once~~ Add option to render all connected components at once Feb 25, 2017

fedarko mentioned this issue Aug 9, 2017

Add option to render multiple connected components at once marbl/MetagenomeScope#16

Closed

fedarko closed this as completed Aug 9, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add option to render all connected components at once #93

Add option to render all connected components at once #93

fedarko commented Sep 27, 2016 •

edited

fedarko commented Aug 9, 2017

Add option to render all connected components at once #93

Add option to render all connected components at once #93

Comments

fedarko commented Sep 27, 2016 • edited

fedarko commented Aug 9, 2017

fedarko commented Sep 27, 2016 •

edited