Skip to content
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.

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

New magnetic field class - mirror machine #325

Merged
merged 5 commits into from
Jun 19, 2023
Merged

New magnetic field class - mirror machine #325

merged 5 commits into from
Jun 19, 2023

Conversation

rogeriojorge
Copy link
Contributor

This PR introduces the mirror machine model from https://arxiv.org/abs/2305.06372
It expands the set of magnetic field models available to SIMSOPT to include a magnetic mirror model for fusion applications. The default input parameters follow the ones used to model WHAM (https://wippl.wisc.edu/wisconsin-hts-axisymmetric-mirror/)

@rogeriojorge rogeriojorge marked this pull request as ready for review June 15, 2023 18:39
@rogeriojorge
Copy link
Contributor Author

Ready for review. However, for some reason, I don't obtain a symmetric grad B tensor. Still checking out why.

@rogeriojorge
Copy link
Contributor Author

Actually the field is not curl-free so the gradB tensor is expected to not be symmetric

@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.32 ⚠️

Comparison is base (c0529b8) 91.35% compared to head (40d27e9) 91.04%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #325      +/-   ##
==========================================
- Coverage   91.35%   91.04%   -0.32%     
==========================================
  Files          68       68              
  Lines       12042    12110      +68     
==========================================
+ Hits        11001    11025      +24     
- Misses       1041     1085      +44     
Flag Coverage Δ
unittests 91.04% <100.00%> (-0.32%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/simsopt/field/magneticfieldclasses.py 97.35% <100.00%> (+0.42%) ⬆️

... and 10 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

mbkumar
mbkumar previously approved these changes Jun 17, 2023
Copy link
Collaborator

@mbkumar mbkumar left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@mbkumar mbkumar merged commit 6d66c97 into master Jun 19, 2023
@mbkumar mbkumar deleted the rj/mirror branch June 19, 2023 17:04
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.

3 participants