EdgeConfig such as labelpos is not respected during Dagre layout

[elvin-hwang]

I'm attempting to pass in EdgeConfig to my Dagre graph during layout, but I found passing in { labelpos: 'c' | 'l' | 'r' } does not change the output of the label position and always uses the default 'r' configuration. Same can be said about other EdgeConfig properties, such as labeloffset, minlen, weight. (also labeloffset is misspelled as labeloffest in EdgeConfig)

Here is how I'm layouting the dagre graph

import Dagre, { Edge, layout } from '@dagrejs/dagre'

const dagreGraph = new Dagre.graphlib.Graph({ multigraph: true, compound: true })
    .setDefaultEdgeLabel(() => ({}))
    .setDefaultNodeLabel(() => ({}))

 dagreGraph.setGraph({
    rankdir: graphOptions.rankdir,
    ranksep: graphOptions.ranksep,
    edgesep: graphOptions.edgesep,
    nodesep: graphOptions.nodesep,
  })

...
add nodes
...
add edges
...
    dagreGraph.setEdge(
      edge.source,
      edge.target,
      { label: edge.displayLabel, width: labelWidth, height: labelHeight },
      edgeId,
    )
...

layout(dagreGraph, { labelpos: 'c', labeloffest: 1000, minlen: 1000, weight: 1000 })

Calling layout with and without EdgeConfig option returns the same coordinates for the Edge label

[elvin-hwang]

Here's the type interface that suggests layout to be passed with config
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/dagre/index.d.ts#L100

However, passing in EdgeConfig to layout() does not actually respect the config, and instead needs to be set during setEdge like below, which is not documented on the wiki nor in the type declarations for setEdge.

Ideally this internal function to buildLayoutGraph should also pass in the config option so that each edge has the proper config passed in from the layout() arguments. Otherwise, if this is an intentional design choice, then it should be documented as such.

dagreGraph.setEdge(
      edge.source,
      edge.target,
      { label: edge.displayLabel, width: labelWidth, height: labelHeight, labelpos: 'c' },
      edgeId,
    )