-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updating Tensorflow 1.14 to 2 #32
Comments
No specific reason. The project was initially developed before Tensorflow 2 was available. A side story: you might have noticed we had to deal with the static graph in tensorflow even though our problem inherently has dynamic input graph size (since the number of jobs is not pre-determined). Indeed, our best choice was pytorch, but it did not support certain sparse matrix operations that we needed. It was an unfortunate decision that we had to live with Tensorflow 1.x at that time. Later on, we actually implemented a pytorch version of the core graph neural network: https://github.com/hongzimao/gcn_pytorch |
Hi, |
This might be helpful: #31 (comment) |
Hi, Mao.
There's a specific reason why you didn't update Decima to work with Tensorflow 2? Despite using the tf.contrib...
The text was updated successfully, but these errors were encountered: