Repository navigation
Identifier "M_PI" is undefined #6
Description
Not sure if this issue is important yet, but I figure you may have written it in Linux/macOS as many compilers (especially VS) seem to hide it during build due to it being a 'POSIX extension'. Ends up causing some problems as seen below (The image), then below that I just provided the text from the terminal. The build completed, but those errors were just shown during that process. I simplified the messages and removed my personal parts, but I presume it may be understandable. (I'm not use to writing these issues, so if I did this wrong and you need more info, let me know)
parakeet.cpp\src\fft.cpp(28,29): error C2065: 'M_PI': undeclared identifier \parakeet.cpp\build\parakeet.vcxproj]
mel_gpu.cpp
\parakeet.cpp\src\mel_gpu.cpp(24,33): error C2065: 'M_PI': undeclared identifier \parakeet.cpp\build\parakeet.vcxproj]
\parakeet.cpp\src\mel_gpu.cpp(24,18): error C2737: 'two_pi': const object must be initialized \parakeet.cpp\build\parakeet.vcxproj]
Activity
orijin-tian commentedon Jun 2, 2026
I was able to fix it with placing #define _USE_MATH_DEFINES before then adding "_USE_MATH_DEFINES" to C/C++: Edit Configurations (JSON) in the command pallet (or just c_cpp_properities.json). Could likely be also fixed with declaring this after cmath:
#ifndef M_PI
#define M_PI 3.14159265358979323846
#endif
mvanhorn commentedon Jun 7, 2026
Opened #18. Defines
_USE_MATH_DEFINESbefore<cmath>(plus an#ifndef M_PIfallback) in fft.cpp/mel_gpu.cpp, and adds_USE_MATH_DEFINESas a PUBLIC MSVC compile definition on theparakeettarget so the test executables that also useM_PIbuild. Non-MSVC builds are unaffected.