Disabling hierarchical layout and re-enabling it causes errors

[Tyler-Maclachlan]

Hi there,

I am using the hierarchical layout with edges that can be 'split?', when splitting the edges I disable the layout so that none of the nodes move (it's really frustrating having the node you were looking at suddenly move), however when I re-enable the layout afterwards I get the following error:

Uncaught Error: To use the hierarchical layout, nodes require either no predefined levels or levels have to be defined for all nodes.

I have predefined levels manually on every node, and looping through the DataSet shows that each node does indeed have a level.

Here's how I do it:

this.options.layout.hierarchical.enabled = false;
this.network.setOptions(this.options);

// split the edges

// loop over the data set and log the node level - every node has a level
this.graphNodes.forEach(n => {
     console.log(n.level);
})
this.options.layout.hierarchical.enabled = true;
this.network.setOptions(this.options);

[Thomaash]

Hi @Tyler-Maclachlan,

could you provide an MWE on JSFiddle or somewhere, please?

Thanks.

[Tyler-Maclachlan]

Hi @Thomaash

Here is a codepen example.

It always gives the error on split, and every now and then when collapsing the edges back together

[Thomaash]

This has me puzzled. When hierarchical layout is disabled some new hidden nodes are added. Not sure why though.

[Tyler-Maclachlan]

I believe that happens when you have smoothing enabled on edges. Those support nodes shouldn't be taken into account in the check for node levels.

[Tyler-Maclachlan]

Disabling smoothing for edges stops the error from showing up, but doesn't work for this use case as the edges need to be curved to show each edge.

[Thomaash]

Okay, thanks for investigating this. There are already two (#171 and #177) PRs messing with hierarchical layout that I'd like to merge before addressing this. So after @mojoaxel finds the time to review them I'll have a look if the support edges can be safely ignored in hierarchical layout.

[robsonn]

Recently I had the same error.

Error spam in second change, first remove hierarchical(ok)

const optionsRemoveHierarchy = { ...this.options, layout: { hierarchical: { enabled: false } }, edges: { physics: false }, nodes: { physics: false } }

this.network.storePositions()
this.network.setOptions(optionsRemoveHierarchy)

second time change back hierarchical span error

const optionsHierarchy = { ...this.options, layout: { hierarchical: { enabled: true } }, nodes: { physics: true }, edges: { physics: true } }
this.network.setOptions(optionsHierarchy)

@Thomaash @Tyler-Maclachlan I solved changing the smooth type for any value different of dynamic
ex: edges: { smooth: { enabled: false }} or edges: { smooth: { enabled: true, type: 'discrete'(for example)}

Note: vis-network version - 9.1.4