Skip to content

Bump FLINT to 3.4.0#376

Open
oscarbenjamin wants to merge 1 commit intoflintlib:mainfrom
oscarbenjamin:pr_flint35
Open

Bump FLINT to 3.4.0#376
oscarbenjamin wants to merge 1 commit intoflintlib:mainfrom
oscarbenjamin:pr_flint35

Conversation

@oscarbenjamin
Copy link
Collaborator

See gh-375

@oscarbenjamin
Copy link
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.

@oscarbenjamin
Copy link
Collaborator Author

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

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 10, 2026

It needs flintlib/flint#2594

@oscarbenjamin
Copy link
Collaborator Author

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

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