Skip to content

Conversation

@user202729
Copy link
Contributor

fix #41127

📝 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

@cxzhong
Copy link
Contributor

cxzhong commented Nov 15, 2025

I tested it is fixed

sage: SR.cleanup_var('x')
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[1], line 1
----> 1 SR.cleanup_var('x')

File sage/symbolic/ring.pyx:738, in sage.symbolic.ring.SymbolicRing.cleanup_var()

File sage/symbolic/ring.pyx:937, in sage.symbolic.ring.SymbolicRing._repr_element_()

File sage/symbolic/expression.pyx:13534, in sage.symbolic.expression._repr_Expression()

TypeError: Cannot convert str to sage.symbolic.expression.Expression

vbraun pushed a commit to vbraun/sage that referenced this pull request Nov 16, 2025
fix sagemath#41127

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [ ] 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

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->

URL: sagemath#41176
Reported by: user202729
Reviewer(s):
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.

segfault on cleanup_var()

2 participants