AdolfVonKleist / Phonetisaurus

Phonetisaurus G2P
BSD 3-Clause "New" or "Revised" License
449 stars 122 forks source link

remove exp10 from 3rdparty/rnnlm/rnnlmlib.cpp #53

Open esdeboer opened 4 years ago

esdeboer commented 4 years ago

3rdparty/rnnlm/rnnlmlib.cpp contains the function exp10 in some logging statements.

The exp10 function is not available on freeBSD, can you remove those logging lines, or change exp10 with pow(10.0, x)?

The muse project defined their own exp10 function if missing: https://github.com/muse-sequencer/muse/commit/348d849e89de4ec3b859456b6fff3709f9ff414b

AdolfVonKleist commented 4 years ago

If you create a pull request for this, I'm happy to merge it. I don't think this bit is really used much TBH. Unless you are actively using it you could also omit it from the compilation.

esdeboer commented 4 years ago

I created pull request #54 , the rmnllib only defined the exp10 for apple, not for the other systems not having it.