Skip to content

Conversation

fchapoton
Copy link
Contributor

fix #38430

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.

Copy link

github-actions bot commented Oct 13, 2025

Documentation preview for this PR (built with commit d68e75f; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@maxale
Copy link
Contributor

maxale commented Oct 14, 2025

@fchapoton The fix looks good to me. However, in addition to fixing the error message, it should be also mentioned in the documentation for magma_free that in the case of timeout the TimeoutError exception is raised. Currently, it just says:

The code must evaluate in at most 120 seconds and there is a limitation on the amount of RAM.

without saying what would happen when time is out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Call to magma_free results in ExpatError: syntax error: line 1, column 49

2 participants