-
Notifications
You must be signed in to change notification settings - Fork 52
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
Coil forces #450
Coil forces #450
Conversation
Ml/coil forces
Update branch to have FramedCurve for coil orientation
…rently working, but the autodifferentiation and VJPs still all need to be implemented for dJ.
modified: src/simsopt/field/force_2.py
…hout Exceptions in the commit. Additionally, a Taylor test has been coded up, though it is currently failing. Next step is to see what is wrong with either the test or the Optimizable (presumably the Optimizable).
… accordancce with the fixed math for dJ.
…ives w.r.t. the current, see issue #441
…into coil_forces
…h critical_current
…o coil_forces merge latest commits
Coil_forces
In |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #450 +/- ##
==========================================
+ Coverage 92.61% 92.68% +0.06%
==========================================
Files 75 77 +2
Lines 13837 13982 +145
==========================================
+ Hits 12815 12959 +144
- Misses 1022 1023 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Hearing no objections, I'll merge. |
This is to pull all the force optimization work into master.