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

Better algorithm for extending PsiContour #140

Open
johnomotani opened this issue Dec 5, 2022 · 0 comments
Open

Better algorithm for extending PsiContour #140

johnomotani opened this issue Dec 5, 2022 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@johnomotani
Copy link
Collaborator

In the temporaryExtend() method used to extend a PsiContour

def temporaryExtend(

at the moment the initial guess for new points is made by extrapolating from the positions of nearby points. It would be more accurate and probably more robust to use an ODE integrator to follow the flux surface, similar to how the separatrix is traced here
# Integrate a distance "step" along the leg
solve_result = solve_ivp(
dpos_dl,
(0.0, step),
pos,
rtol=0.0,
atol=self.user_options.leg_trace_atol,
)
newpos = (solve_result.y[0][1], solve_result.y[1][1])

@johnomotani johnomotani added enhancement New feature or request help wanted Extra attention is needed labels Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant