search

lava / matplotlib-cpp

Extremely simple yet powerful header-only C++ plotting library built on the popular matplotlib
MIT License
4.28k stars 1.12k forks source link

Fix #221 and #225 #230

Closed williamleong closed 3 years ago

williamleong commented 3 years ago

The C API for Python 2 (https://docs.python.org/2/c-api/init.html#c.PySys_SetArgv) expects char ** as the second argument.

void PySys_SetArgv(int argc, char **argv)

while the C API for Python 3 (https://docs.python.org/3.6/c-api/init.html#c.PySys_SetArgv) expects wchar_t **argv as the second argument.

void PySys_SetArgv(int argc, wchar_t **argv)

Fixed using the solution suggested in #225 and is probably the same problem for #221.

lava commented 3 years ago

Thanks for contributing, and sorry for the long delay with merging this!