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

Extend FloorPlot #50

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Extend FloorPlot #50

wants to merge 15 commits into from

Conversation

giadarol
Copy link
Member

Note that this PR can be merged only after xtrack v0.73.1 is released.

The following modifications and extensions are introduced:

  • Plotting the XY projection is now supported.
  • The FloorPlot is now generated from the survey table alone without requiring access to the line. This allows plotting the survey after the corresponding line has been modified or deleted. For this purpose additional columns (length, isthick and element_type have been added to the survey table in xtrack v0.73.1.
  • The type of magnet is inferred from the element type and not from the strengths. This is to support the common use case in beam-line design, in which one defines a lattice geometry and wants to visualize the positions of the elements before defining the magnetic strengths. A side effect of this modification is that thin elements are all shown as multipoles and in the same color (we could think of doing something more sophisticated in the future, if required by users).

Examples including horizontal and vertical bends are available at https://github.com/xsuite/xplt/blob/extend_floorplot/examples/line_cases.ipynb

image image image image image image

@giadarol giadarol marked this pull request as draft January 18, 2025 21:18
@giadarol giadarol requested a review from eltos January 18, 2025 21:18
@giadarol giadarol marked this pull request as ready for review January 19, 2025 16:11
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.

1 participant