search

fscarlier / xsequence

Package to manage particle accelerator sequences.
1 stars 2 forks source link

kn/knl setter in quadrupole broken #1

Closed mihofer closed 3 years ago

mihofer commented 3 years ago

Currently, one test breaks because a quadrupole initialized with a knl will not propagate this to kn, thus the two elements in the test differ

fscarlier commented 3 years ago

I removed the faulty knl tests. This is a wrong and outdated use of the Quadrupole() class. If you want to defines things with knl one should use the Multipole() class instead. Still, this should be reviewed to define the set priorities of k1, kn, and knl..