Skip to content

Propagate exception from forked subprocess #41177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Propagate exception from forked subprocess #41177

wants to merge 1 commit into from
+71 −5

Conversation

@user202729
Copy link
Contributor

@user202729 user202729 commented Nov 15, 2025

If you try to execute the following

sage: H.<x, y> = AffinePlaneCurveArrangements(QQ)
sage: A = H(y^2 + x, y + x - 1, x)
sage: A.fundamental_group()

without sirocco, previously, a mysterious exception is raised. Now you get the (mostly) correct exception.

📝 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.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

@user202729 user202729 force-pushed the fork-propagate-exception branch from 966dd4e to 0f99e6e Compare November 15, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

s: needs review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@user202729