FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Bump FLINT to 3.4.0 by oscarbenjamin · Pull Request #376 · flintlib/python-flint · GitHub

Repository navigation

Bump FLINT to 3.4.0 - #376

Closed
oscarbenjamin wants to merge 1 commit into
flintlib:mainfrom
oscarbenjamin:pr_flint35
Closed

oscarbenjamin wants to merge 1 commit into
flintlib:mainfrom
oscarbenjamin:pr_flint35

Conversation

Copy link
Copy Markdown
Collaborator

See gh-375

Copy link
Copy Markdown
Collaborator Author

The CI job here that runs with FLINT's main branch failed with illegal instruction in test_acb_rmul which is just

def test_acb_rmul() -> None:
x = acb(2, 3)
assert 2 * x == acb(4, 6)
assert acb(5, 7) * x == acb(-11, 29)
assert raises(lambda: object() * x, TypeError) # type: ignore[operator]

This is pretty much just calling acb_mul:
def __rmul__(s, t):
cdef acb_struct tval[1]
cdef int ttype
ttype = acb_set_any_ref(tval, t)
if ttype == FMPZ_UNKNOWN:
return NotImplemented
u = acb.__new__(acb)
acb_mul((<acb>u).val, tval, s.val, getprec())
if ttype == FMPZ_TMP:
acb_clear(tval)
return u

That is possibly a bug in FLINT.

Copy link
Copy Markdown
Collaborator Author

The pyodide job has failed because FLINT 3.4.0 doesn't build in pyodide.

mkoeppe commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

It needs flintlib/flint#2594

Copy link
Copy Markdown
Collaborator Author

Thanks. I'll think about how to handle that. Could just use a lower FLINT version for pyodide or could patch.

oscarbenjamin deleted the pr_flint35 branch June 15, 2026 14:14
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL