Skip to content
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

[Windows] TensorFlow support #189

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gard-n
Copy link

@gard-n gard-n commented Aug 19, 2022

I managed to compile the tensorflow custom_ops on Windows using this workaround and ignoring the C4596 with warning level /W3. I compiled with Visual Studio 2019 (Version 16.7.5, MSVC 19.27.29112), TensorFlow 2.9.1, and CUDA 11.2. The tests_tensorflow scripts seem to work as intended.

This solves #117.

Other changes:

  • Load the dll in pyrender_tensorflow when using Windows.
  • Remove checks that are supposed to ignore tensorflow on Windows.
  • Fix small bug in test_g_buffer: target_depth size was (256,256) not (256,256,1).

Fix compilation errors with workaround from: tensorflow/tensorflow#45823 (comment)
Set warning level w3 to threat C4596 as warning.

Load the dll in pyrender_tensorflow when using Windows.
Remove checks that are supposed to ignore tensorflow on Windows.
Fix small bug in test_g_buffer: target_depth size was (256,256) not (256,256,1).

Tested with Tensorflow 2.9.1, CUDA 11.2, and Visual Studio 2019, Version 16.7.5.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant