Make IN_PLACE flag even if it's the real mpi4py IN_PLACE

rmjarvis / MockMPI

BSD 2-Clause "Simplified" License

4 stars 1 forks source link

Closed joezuntz closed 2 years ago

joezuntz commented 2 years ago

Recent versions of mpi4py have MPI.IN_PLACE as a subclass of int, not just as a raw int. This breaks the use of that in the mock version.

This fixes it.

joezuntz commented 2 years ago

I didn't mean to merge that, I have no idea wha I clicked. But it does work now!

rmjarvis commented 2 years ago

Haha. It's fine. I'll cut a release. 0.8.1 I guess? This is basically just a bugfix.

Or should we jump to 1.0 now? It seems pretty stable to me. We could signal that we think the API is stable now.

joezuntz commented 2 years ago

It’s basically a bug fix, yeah. No strong opinion though!

rmjarvis commented 2 years ago

OK. v0.8.1 released. But maybe next time we add a new feature, we could think about bumping to 1.0.