You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@andreasfelix@PaulGoslawski
We should include many more unit tests, so that if I change anything I am sure I do not break something. I lost half a day checking if previous things still work and I am still not sure if I now broke something or not (on my branch).
Testing is crucial if you are working as a team on the same code.
The text was updated successfully, but these errors were encountered:
do you still think this latticejson is a good idea? I heavily used it for the https://lattice-summaries.netlify.app/ website, but the code is somewhat alpha quality at the moment.
if you would be interested, we would could sit down for 3-4 days sometime next year and push this more to be production ready?
How i would move forward:
Only support a limited subset of elegant and MAD-X language, where only basic variable assignments and sequence repetition/reflection is allowed. As these are turning complete scripting language it is impossible to support every corner case. For example control flow statements like if and for do not belong in a lattice file in my opinion.
Write down a specification of these limited subsets of the elegant and MAD-X language.
Write down a 1:1 mapping between these limited subsets and LatticeJSON (e.g. "Dipole" <--> "csbend" for elegant)
@andreasfelix @PaulGoslawski
We should include many more unit tests, so that if I change anything I am sure I do not break something. I lost half a day checking if previous things still work and I am still not sure if I now broke something or not (on my branch).
Testing is crucial if you are working as a team on the same code.
The text was updated successfully, but these errors were encountered: