TensorflowGNN transformer

[post2web]

Description

Contribution for issue #350

Pull request type

Please delete options that are not relevant.

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring with functional or API changes
• Refactoring without functional or API changes
• Build or packaging related changes
• Documentation content changes
• Other (please describe):

Related issues

Closes #350

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

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

Reviewer checklist (the reviewer checks this part)

• Core feature implementation
• Tests
• Code documentation
• Documentation on gqlalchemy/docs

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

[CLAassistant]

All committers have signed the CLA.

[katarinasupe]

Change the order of imports + add blank lines (1. Standard library imports, 2. Third-party library imports, 3. Local application/library imports; all grouped and alphabetized within each block)

Suggested change

	from typing import List, Dict, Any, Set, Tuple
	import tensorflow as tf
	import tensorflow_gnn as tfgnn
	from collections import defaultdict
	from gqlalchemy.transformations.constants import DEFAULT_NODE_LABEL, TFGNN_ID
	from gqlalchemy.transformations.translators.translator import Translator
	import numpy as np
	from gqlalchemy.memgraph_constants import (
	MG_HOST,
	MG_PORT,
	MG_USERNAME,
	MG_PASSWORD,
	MG_ENCRYPTED,
	MG_CLIENT_NAME,
	MG_LAZY,
	)
	from collections import defaultdict
	from typing import Any, Dict, List, Set, Tuple
	import numpy as np
	import tensorflow as tf
	import tensorflow_gnn as tfgnn
	from gqlalchemy.memgraph_constants import (
	MG_CLIENT_NAME,
	MG_ENCRYPTED,
	MG_HOST,
	MG_LAZY,
	MG_PASSWORD,
	MG_PORT,
	MG_USERNAME,
	)
	from gqlalchemy.transformations.constants import DEFAULT_NODE_LABEL, TFGNN_ID
	from gqlalchemy.transformations.translators.translator import Translator

[katarinasupe]

Suggested change

	Convertion rules:
	Conversion rules:

[katarinasupe]

What happens with temporal, Enum or Point types from Memgraph?

[katarinasupe]

Suggested change

	"""Produce cypher queries for data saved as part of the TFGNN graph. If the graph is homogeneous, a default TFGNN's labels will be used.
	_N as a node label and _E as edge label. The method converts 1D as well as multidimensional features. If there are some isolated nodes inside TFGNN graph, they won't get transferred. Nodes and edges
	created in Memgraph DB will, for the consistency reasons, have property `dgl_id` set to the id they have as part of the TFGNN graph. Note that this method doesn't insert anything inside the database,
	it just creates cypher queries. To insert queries the following code can be used:
	>>> memgraph = Memgraph()
	graph_tensor = TFGNNTranslator(...)
	for query in TFGNNTranslator().to_cypher_queries(graph_tensor):
	memgraph.execute(query)
	"""
	"""Produce Cypher queries for data saved as part of the TFGNN graph. If the graph is homogeneous, a default TFGNN's labels will be used (_N as a node label and _E as edge label). The method converts 1D as well as multidimensional features. If there are some isolated nodes inside TFGNN graph, they won't get transferred. Nodes and edges
	created in Memgraph DB will, for the consistency reasons, have property `tfgnn_id` set to the id they have as part of the TFGNN graph. Note that this method doesn't insert anything inside the database,
	it just creates Cypher queries. To insert queries the following code can be used:
	memgraph = Memgraph()
	graph_tensor = TFGNNTranslator(...)
	for query in TFGNNTranslator().to_cypher_queries(graph_tensor):
	memgraph.execute(query)
	"""

[katarinasupe]

• cypher -> Cypher
• put default node and edge labels in ()
• dgl_id -> tfgnn_id
• removed >>> in code example to avoid incorrect indentation

[katarinasupe]

Suggested change

	Creates cypher index queries for the graph.
	Creates Cypher index queries for the graph.

[katarinasupe]

homogeneous graph (not homoginious). Fix in test name and docstring

[katarinasupe]

heterogeneous graph (not heteroghinious). Fix in test name and docstring

[katarinasupe]

heterogeneous graph (not heteroghinious). Fix in test name and docstring

[katarinasupe]

Hi @post2web, I left some comments, if you can take a look.
Also, I added changes made with black formatter locally (poetry run black .), so I recommend you do that before the next commit 🙏
My colleague @antejavor will also take a look at the PR and leave some comments if needed.

Also, it would be great if you could contribute to the translators documentation, which refers to import and export guides.

[antejavor]

Just two small comments, the rest looks good IMO.

EDIT: I have deleted my comments, leaving to @katarinasupe to approve.