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

BUG: vertical displacement not taken into account in compute_x_points() method #151

Closed
RemDelaporteMathurin opened this issue Aug 4, 2020 · 3 comments

Comments

@RemDelaporteMathurin
Copy link

lower_x_point = (
1-(1+shift)*triangularity*minor_radius,
(1+shift)*elongation*minor_radius
)
if self.configuration == "double-null":
# upper_x_point is up-down symmetrical
upper_x_point = (
lower_x_point[0],
-lower_x_point[1]
)

The X points should be functions of vertical_displacement attribute

@shimwell
Copy link
Collaborator

shimwell commented Aug 4, 2020

Ah yes, I think the other plasma also ignores vertical_displacement

Perhaps this is just a simple addition or subtraction

@RemDelaporteMathurin
Copy link
Author

It is, just put it there so I remember it :-)

@RemDelaporteMathurin
Copy link
Author

Fixed in #156

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

No branches or pull requests

2 participants