[main < T350-MAGE] Implement graphml import and export

[ind1xa]

Description

Implemented import_util.graphml and export_util.graphml

Pull request type

• Bugfix
• Algorithm/Module
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Other (please describe):

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

• Core algorithm/module implementation
• Query module implementation
• Unit tests
• End-to-end tests
• Code documentation
• README short description
• Documentation on memgraph/docs
• Update GQLALchemy signatures in query builder using query module signature generator

######################################

[antoniofilipovic]

Suggested change

	if graphML:
	properties = {
	key: convert_to_isoformat_graphML(vertex.properties.get(key))
	for key in vertex.properties.keys()
	}
	else:
	properties = {
	key: convert_to_isoformat(vertex.properties.get(key))
	for key in vertex.properties.keys()
	}
	if graphML:
	properties = {
	key: convert_to_isoformat_graphML(vertex.properties.get(key))
	for key in vertex.properties.keys()
	}
	continue
	properties = {
	key: convert_to_isoformat(vertex.properties.get(key))
	for key in vertex.properties.keys()
	}

[ind1xa]

but wont continue go to the next vertex in ctx.graph.vertices and lines below will be skipped?

[antoniofilipovic]

yes but isn't that okay?

[ind1xa]

not really, i want those properties to be added to the node in both cases, just in different format (depending if its for graphml or not)

[antoniofilipovic]

Good work, left few comments to do

[antepusic]

Left a few comments, I'll take a more detailed look later too :)

[antoniofilipovic]

Looks good to me