MPL: KnownElementsList.plot_survey()
#1040
Merged
Conversation
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
ax3l
force-pushed
the
topic-plot-lattice
branch
2 times, most recently
from
ee5ab6d to
b9bde5e
Compare
ax3l
commented
Jul 14, 2025
ax3l
commented
Jul 15, 2025
ax3l
commented
Jul 15, 2025
ax3l
changed the title
KnownElementsList.plot()KnownElementsList.plot_survey()
ax3l
force-pushed
the
topic-plot-lattice
branch
3 times, most recently
from
3c83b75 to
524fd84
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Comment on lines
+89
to
+94
| # if el_type == "Kicker": | ||
| # height = copysign(0.8, el_dict["xkick"]) |
Check notice
Code scanning / CodeQL
Commented-out code Note
ax3l
commented
Jul 21, 2025
Comment on lines
+17
to
+31
| "Quad": "tab:blue", | ||
| "Multipole": "tab:orange", | ||
| "Sbend": "tab:green", | ||
| "CFbend": "tab:olive", # TODO: improve and plot as two on top of each other | ||
| "ConstF": "tab:red", | ||
| "ChrPlasmaLens": "tab:red", | ||
| "SoftSolenoid": "tab:red", | ||
| "TaperedPL": "tab:red", | ||
| "RFCavity": "tab:brown", | ||
| "ShortRF": "tab:brown", | ||
| "Buncher": "tab:purple", | ||
| "Aperture": "black", | ||
| "Kicker": "tab:pink", | ||
| # 'tab:cyan' | ||
| "other": "tab:gray", |
Member
Author
There was a problem hiding this comment.
To double check, I asked again about the colors according to ChatGPT:
- sbend (main dipole) – blue
- quadrupole – silver-grey stainless steel (white for the IR triplets)
- multipole corrector (sextupole, octupole, …) – blue
- final-focus / triplet quadrupole (generic “focusing element”) – white
- RF accelerating cavity (400 MHz) – white
- RF buncher cavity (800 MHz) – white
- aperture / collimator – beige-ivory (sometimes bare stainless steel)
- kicker – black
Member
Author
There was a problem hiding this comment.
slac-lcls2:
(visual appearance in the LCLS-II tunnel / gallery)
- sbend (dipole) – teal-green
- quadrupole – teal-green
- multipole (sextupole, octupole, corrector) – teal-green
- other focusing magnets (doublets, triplets) – teal-green
- 1.3 GHz SRF cryomodule – light-grey / metallic stainless-steel
- 3.9 GHz harmonic (bunch-length control) cryomodule – light-grey / metallic stainless-steel
- warm 2.856 GHz copper buncher cavity – copper-orange
- aperture / collimator / slit – bare stainless-steel (silver-grey)
- kicker (fast deflector) – teal-green
Member
Author
There was a problem hiding this comment.
lbnl-als (ALS-U)
- sbend (dipole) – turquoise / teal-green
- quadrupole – turquoise / teal-green
- multipole (sextupole, octupole, corrector) – turquoise / teal-green
- other focusing elements (doublets, triplets) – turquoise / teal-green
- 500 MHz RF accelerating cavity – bare copper (reddish-orange)
- higher-harmonic / bunching cavity – bare copper (reddish-orange)
- aperture / collimator / absorber – bare stainless-steel (silver-grey)
- kicker (injection / extraction) – turquoise / teal-green
Member
Author
There was a problem hiding this comment.
cornell-cesr:
- sbend (dipole) – blue
- quadrupole – green
- multipole (sextupole / octupole / corrector) – red
- other focusing elements (doublets / triplets) – green
- RF accelerating cavity – bare copper (reddish-orange)
- RF buncher / harmonic cavity – bare copper (reddish-orange)
- aperture / collimator / absorber – bare stainless-steel (silver-grey)
- kicker (injection / extraction) – yellow
Member
Author
There was a problem hiding this comment.
fnal-pip-ii (PIP-II linac / transfer line):
- sbend (dipole) – teal-green
- quadrupole – teal-green
- multipole / corrector (sextupole, dipole corrector, etc.) – teal-green
- other warm focusing elements (doublets, triplets) – teal-green
- superconducting RF cryomodules (HWR, SSR1/2, LB650, HB650) – light-grey brushed stainless-steel
- 162.5 MHz RFQ – light-grey brushed stainless-steel
- 325 MHz room-temperature buncher cavity – copper-orange
- aperture / collimator / absorber – bare stainless-steel (silver-grey)
- kicker / chopper – teal-green
Member
Author
There was a problem hiding this comment.
I will add such palettes in a follow-up PR where we can do a careful review and detailed docs.
Member
Author
There was a problem hiding this comment.
I did some checks on the SPS/LHC complex and there are def some hallucinations in the above responses :)
ax3l
commented
Jul 22, 2025
Example: ```py import matplotlib.pyplot as plt sim.lattice.plot_survey(ref) plt.show() ```
cemitch99
reviewed
Jul 22, 2025
cemitch99
approved these changes
Jul 22, 2025
Member
cemitch99
left a comment
cemitch99
left a comment
There was a problem hiding this comment.
Great! I tried this and it worked for me. I just left a suggestion regarding the documentation.
ax3l
commented
Jul 22, 2025
Co-authored-by: Chad Mitchell <[email protected]>
ax3l
commented
Jul 22, 2025
ax3l
commented
Jul 22, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Quick plot of a lattice over
s.Implement #887 (comment)
Example
To Do
rcplot_surveyFollow-Up
refparticle if the lattice is crated fromsim.lattice