search

ornladios / ADIOS

The old ADIOS 1.x code repository. Look for ADIOS2 for new repo
https://csmd.ornl.gov/adios
Other
54 stars 40 forks source link

adios_mpi.cpp doesn't work with Python 3.7 #202

Open rgmiller opened 4 years ago

rgmiller commented 4 years ago

The current version of wrappers/numpy/adios_mpi.cpp is generated by Cython 0.28.0. This code is not compatible with the C API in Python 3.7. Attempting to compile it results in lots of errors along the lines of 'PyThreadState' {aka 'struct _ts'} has no member named 'exc_type' and similar. This is a known problem with Cython that has been fixed as of Cython 0.29. (See the first entry under "Bugs Fixed" at https://github.com/cython/cython/blob/0.29.x/CHANGES.rst)

I believe simply re-generating adios_mpi.cpp using the new Cython will result in code that compiles against Python 3.7 (and earlier).

jychoi-hpc commented 4 years ago

I have just recompiled with the recent Cython 0.29 and uploaded to the repository. Please let us know if there is still a problem.

ax3l commented 2 years ago

X-ref: https://github.com/ornladios/ADIOS/issues/216#issuecomment-999011323