search

yandex / faster-rnnlm

Faster Recurrent Neural Network Language Modeling Toolkit with Noise Contrastive Estimation and Hierarchical Softmax
Other
562 stars 138 forks source link

Addition of a build system generator #9

Open elfring opened 9 years ago

elfring commented 9 years ago

I suggest to reuse a higher level build system than your current small make file so that powerful checks for software features will become easier.

akhti commented 9 years ago

Currently only CUDA test is required. Does it fail for you?

elfring commented 9 years ago
akhti commented 9 years ago

I'm trying to follow KISS principle. At the moment the only non-trivial (and optional) dependency is cuda. A higher level build system would add one more. Of course, the makefile performs only naive checks for software features. And once I discover it fails in some cases, I'll use cmake.

Build system generator is a neat feature. Hovewer, the current version of the toolkit cannot be build under windows out of the box as pthreads support is required. Therefore VS project generators are useless. What's more, most of the other IDEs are able to import Makefile projects directly. Please correct me if I am wrong.

To conclude, I'm not sure that an andvanced build system would make building process simpler. And that's the main reason that keeps me from using cmake.