search

MKLab-ITI / pygrank

Recommendation algorithms for large graphs
Apache License 2.0
29 stars 4 forks source link

Potential issue in the GNN demonstrator example with tensorflow backend #19

Closed maniospas closed 3 months ago

maniospas commented 1 year ago

During the review process of the library's paper, a reviewer pointed out that the following error occurs in their local system with TensorFlow 3.9.2 and Python 3.10.6. 

TypeError: Sequential.call() got multiple values for argument 'training'

This occurs when running the code of the APPNP example. The issue lies fully with the example and not with any additional library functionality - it will not motivate a hotfix.

Investigate whether this issue is unique to the version of TensorFlow or whether the latter has yet again updated something that will break the example in all future versions. At the very least, this error should not occur in github actions.

If this is not the case, investigate whether this issue is platform-dependent.

maniospas commented 3 months ago

Tensorflow interface inconsistencies have been patched in examples.