Failed Complie in Win Envrionment about returnn HOT 4 CLOSED

I tried to fix it. Can you try again? Anyway, this faulthandler support is optional, so you can remove that. But patches to fix it are also welcome.

from returnn.

The TF benchmark uses also our native LSTM op, which it tries to compile on-the-fly.

It (the native LSTM op and other native ops) expects that it finds the TF header files. They are usually part of the PIP package of TF. It gets the path via tf.sysconfig.get_include(). Maybe you can check if that function returns a valid path for you and if you have the header files in there. Otherwise, it's a bit strange that the Windows PIP package does not contain the headers. Maybe you can report that upstream here.

The next thing is, it expects that there is a compiler. Currently there is g++/nvcc hardcoded and also the flags are probably partly specific for those. You will find related code in TFUtil.OpCodeCompiler. You would need to see how you can call the Windows default compiler and the relevant settings. See the TF documentation how to compile a custom op on Windows.

You don't need the native ops, though, if you don't use them. They are faster, esp the LSTM implementation is faster, but you can also use LSTMBlockFused for example. In that case it should work (or maybe you will hit another bug as Windows is not really tested by us). For example, you can try:

rnn.py demos/demo-tf-contribrnn-lstm.12ax.config

Or:

rnn.py demos/demo-tf-vanilla-lstm.12ax.config

You can also run the TF benchmark without the native op, e.g.:

demos/demo-tf-benchmark.py --selected "BasicLSTM,StandardLSTM,LSTMBlock,LSTMBlockFused,CudnnLSTM"

from returnn.

@albertz Thank you very much for the fast reply. It solves the issue!

However, when runing the demos/demo-tf-benchmark.py, there is another issue raised,

fatal error: tensorflow/core/framework/op.h: No such file or directory

Windows environment seems not suitable for the open source projects.

Thank you again for providing such awesome codes.

from returnn.

@albertz Great, everything works smooth now. I also created a PR to fix the print in config file to support python 3.5. 😄 #12
Thanks for your patient help.

from returnn.