Skip to content

[WIP] Update helicity and q(r) discretization based on feedback #2

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
Merged
chandrarn merged 2 commits into from
Jan 26, 2026
Merged

[WIP] Update helicity and q(r) discretization based on feedback #2

chandrarn merged 2 commits into from
Jan 26, 2026

Conversation

Copy link
Contributor

Copilot AI commented Jan 26, 2026 •
edited
Loading

  • Add make_smoothing_spline to the import statement in synthwave/magnetic_geometry/equilibrium_field.py
  • Verify the fix by running tests or checking syntax
  • Final code review and security check

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Jan 26, 2026
Merged
@chandrarn
Add missing import for make_smoothing_spline
088fd1a 
Co-authored-by: chandrarn <8441592+chandrarn@users.noreply.github.com>
@chandrarn chandrarn marked this pull request as ready for review January 26, 2026 21:31
@chandrarn chandrarn merged commit 8c5e0e2 into RNC_Filament_Edits Jan 26, 2026
Copilot stopped work on behalf of chandrarn due to an error January 26, 2026 21:31
The session was cancelled by the user.
Copilot AI requested a review from chandrarn January 26, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chandrarn chandrarn Awaiting requested review from chandrarn

Assignees

@chandrarn chandrarn

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chandrarn