Resolve "Check momentum between `FROMFILE` distribution and BEAM"

Related to #701 (closed)

added Bug label

added 1 commit

• f313cfed - FROMFILE distribution cross-checks momentum with BEAM command

Compare with previous version

marked this merge request as ready

requested review from @adelmann, @ext-calvo_p, and @kraus

Note that when we merge this, all the regression tests that use FROMFILE and the default P0 will fail, so will also need to be updated.

changed the description

mentioned in issue #701 (closed)

changed milestone to %2022.1

added 1 commit

• 3a747a2b - check BEAM momentum for FROMFILE distribution in OPAL-cycl

Compare with previous version

assigned to @ext-calvo_p

requested review from @snuverink_j and removed review request for @ext-calvo_p

I haven't checked all the comments, but I thought the idea was to remove the need for redundant information? While in the chosen solution the user needs to specify the energy two times.

I don't quickly see a different solution, and you probably gave this a thought, but the fact that there is a different solution for Opal-T and Opal-Cycl indicates bad design.

added 1 commit

• 1a5aa319 - remove comments and fix if statement

Compare with previous version

approved this merge request