Skip to content

Conversation

@bmerkle
Copy link
Contributor

@bmerkle bmerkle commented Jan 5, 2026

  • enhance variable renaming with comments during notebook conversion
  • comment for modified or redefined variable is attached for better understanding

📝 Summary

Enhanced variable renaming with contextual comments during Jupyter → marimo conversion.
When variables are renamed, modified, or redefined as part of the conversion process, explanatory comments are automatically attached. This improves traceability, preserves intent, and makes the resulting marimo notebook easier to understand and review—especially for non-trivial refactorings introduced during conversion.

🔍 Description of Changes

  • for modified or redefined variable an additional python comment is generated

📋 Checklist

  • I have read the contributor guidelines.
  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • I have added tests for the changes made.
  • I have run the code and verified that it works as expected.

- enhance variable renaming with comments during notebook conversion
- comment for modified or redefined variable is attached for better understanding
@bmerkle bmerkle requested a review from dmadisetti as a code owner January 5, 2026 22:04
@vercel
Copy link

vercel bot commented Jan 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
marimo-docs Ready Ready Preview, Comment Jan 5, 2026 10:51pm

@bmerkle
Copy link
Contributor Author

bmerkle commented Jan 5, 2026

I have read the CLA Document and I hereby sign the CLA

Copy link
Collaborator

@dmadisetti dmadisetti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this.
Noticed a slight discrepancy in the tests.

If this is for reconstruction, then we need to account for in place operations

# ie
x += 1
# vs
x = x + 1

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