-
Notifications
You must be signed in to change notification settings - Fork 3
Major refactoring for JAX-style classes. #29
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
Draft
EstevaoMGomes
wants to merge
65
commits into
main
Choose a base branch
from
eg/analysis
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
871d169
Add new integrator scripts for field tracing and guiding center dynamics
EstevaoMGomes f86f5ee
Improve loss functions speed
EstevaoMGomes 408c2c5
Refine integrator analysis
EstevaoMGomes 6ff79db
Optimize rotation matrix computation in RotatedCurve function
EstevaoMGomes fdb1508
Edit fo and gc integration analysis plots
EstevaoMGomes 415bbae
Improve loss functions computational efficiency
EstevaoMGomes f38fdea
Feature: gradient analysis
EstevaoMGomes 079cdb7
Add block_until_ready to integrators analysis
EstevaoMGomes fe5d81d
Minor tweaks to gradient analysis plot
EstevaoMGomes 9595ab4
Create Poincare Plots analysis
EstevaoMGomes 108123e
Refactor integrators analysis: add cyclotron frequency calculation an…
EstevaoMGomes c80ff2c
Change loss functions to be quadratic & implement separation loss
EstevaoMGomes ea0af03
Minor improvements in coils class
EstevaoMGomes 5617e7f
Add join method to Particles class and optimize tracing parameters in…
EstevaoMGomes 0bb4462
Simplify curve appending logic in apply_symmetries_to_curves
EstevaoMGomes 91f8cd0
Update cyclotron frequency calculation and adjust tracing parameters …
EstevaoMGomes 549a43f
Enhance Particles class with phase angle parameter and update Tracing…
EstevaoMGomes 23bf271
Refactor coil separation loss function and optimize tracing parameter…
EstevaoMGomes dcef625
Tentative to incorporate pytree methods for BiotSavart class
EstevaoMGomes 3c96714
Refactor Tracing class initialization and improve parameter handling …
EstevaoMGomes b1a227e
Add gc_vs_fo.py for particle tracing and visualization; adjust parame…
EstevaoMGomes 8332e1e
Add output directory creation and update file saving paths in analysi…
EstevaoMGomes c64a9c1
Add comparison_coils.py for BiotSavart field analysis and performance…
EstevaoMGomes f867857
Fix errors derived from dynamics refactoring
EstevaoMGomes 7d7be44
Enhance plotting in gc_integrators.py: adjust figure sizes, add toler…
EstevaoMGomes 53682b6
Change dynamics to accept Integrator name
EstevaoMGomes 093f11d
Add comparison_gc.py for comparing gc trajectories between SIMSOPT an…
EstevaoMGomes 1c6584c
Fixed energy calculation for fo trajectories
EstevaoMGomes 32b84ad
Finalize gc analysis ESSOS vs SIMSOPT
EstevaoMGomes f902e22
Add comparison script for fo tracing & improve gc script plots
EstevaoMGomes 823c6db
Fixed error in dynamics when tracing fieldlines
EstevaoMGomes 32c36b3
Based on work from PR #19
EstevaoMGomes 798731d
Fix energy calls in integrators analysis
EstevaoMGomes c3134b9
Fixed loss functions and enhanced coil separation logic
EstevaoMGomes 5a728c0
Fixed coils&surface opt example
EstevaoMGomes 43dd5a0
Added extra simsopt compilation runs
EstevaoMGomes e6cb9fa
Added field line comparisons & minor improvements to fo and gc compar…
EstevaoMGomes 97f2985
Improve length calculation & refactor gammas to lazy initialization &…
EstevaoMGomes 312dab3
Fix coils.from_simsopt imports
EstevaoMGomes 32fa067
Add loss function comparison with simsopt
EstevaoMGomes d4bb387
Add surface comparison with simsopt
EstevaoMGomes 00f2160
Refactor surfaces gamma to lazy initialization & add SquaredFlux func…
EstevaoMGomes 9c2407b
Fix Coils.from_ imports
EstevaoMGomes 8975934
Finished surface comparison
EstevaoMGomes 70dcaa6
Finish simsopt comparison & improve analysis plots
EstevaoMGomes bbf6cc4
Update Ubuntu for workflows
EstevaoMGomes e844b67
Merge branch 'main' into eg/analysis
EstevaoMGomes 7d807ff
Fixed near-axis & surfaces examples
EstevaoMGomes 9853449
Fixed merge changes
EstevaoMGomes 1395650
Deleting comparison_simsopt folder in examples
EstevaoMGomes ddb2282
Changing surfaces.py to correct the number of modes used, added optio…
eduardolneto a0374cc
Fix(coils, fields): turned field BiotSavart into a PyTree and modifie…
EstevaoMGomes c7378bb
Fix: minor fixes
EstevaoMGomes d3b073c
Merge branch 'en/surface_modes_hotfix' into eg/analysis
EstevaoMGomes d40fa9c
Fix(coils): assertion removal in Coils class; Perf(coils): vmap & sep…
EstevaoMGomes 37252ef
Fix(analysis): precompile coil gammas for comparison with simsopt
EstevaoMGomes df5a525
Fix(surfaces): PyTree able Surfaces
EstevaoMGomes a4fad74
Fix(fields,surfaces): fixed mpol in vmec import
EstevaoMGomes ce7caf7
Fix(surfaces): jit and pjit fix
EstevaoMGomes 43fd404
Feat (losses): Create losses files
EstevaoMGomes ac5c7bc
Fix (losses): Allow for grad usage during optimization
EstevaoMGomes c9efcf7
Clean (losses): Move example to correct file
EstevaoMGomes 41a0c1a
Refactor (example): Simplify coils_from_vmec_surface example with new…
EstevaoMGomes 2324cfe
Refactor (coils): Decache gamma calculation
EstevaoMGomes a2af497
refactored examples folder
rogeriojorge File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Import of 'model' is not used.