flintlib / python-flint

Python bindings for Flint and Arb
MIT License
132 stars 27 forks source link

Fix build error in fmpz_mod_mpoly when using flint HEAD #233

Closed Jake-Moss closed 1 month ago

Jake-Moss commented 1 month ago

Apologies for the delay on this, I've had this fix sitting around since mentioned but haven't been able to test against HEAD due to nix build issues I was determined to fix.

I've added a compat.pxd for small compatibility fixes like this. I noticed that there was a fmpz_mod_mat_compat.pxd but I don't feel like this deserved it's own file.

This will break when using Flint 3.2 below (or at) https://github.com/flintlib/flint/commit/02d4e5dde173124f1fdfb90c158791d2a4fbd3cd.

Closes #232

oscarbenjamin commented 1 month ago

Looks good. Thanks