graphonline icon indicating copy to clipboard operation
graphonline copied to clipboard

non-standard GraphML syntax for directed edges

Open nichtich opened this issue 1 year ago • 1 comments

GraphML edges are directed depending on edge attribute directed set to graph attribute edgedefault by default. Graphonline uses edge attribute isDirect instead and ignores graph attribute edgedefault. To maintain backwards compatibility, I'd recommend to support all three:

  • directed (primary)
  • isDirect (fallback)
  • edgedefault (else)

P.S: I'm surprised the source code emits proper GraphML but the "export" feature at https://graphonline.ru/en/ returns a slightly different format with isDirect XML attribute?!

P.P.S: I think this code is doing the wrong conversion from internal graph data to GraphML.

nichtich avatar Apr 17 '24 12:04 nichtich

Thanks. We will look it. Indeed we have 2 exports for graphoffline utility and for user export.

UnickSoft avatar Apr 17 '24 16:04 UnickSoft

Fixed.

UnickSoft avatar Aug 11 '24 11:08 UnickSoft