Resolve "Momentum tolerance in Opal-t"

Closes #765 (closed)

changed milestone to %2024.1

added Feature request OPAL-T labels

added 1 commit

• be03e836 - momentum tolerance in opal-t

Compare with previous version

added 2 commits

• ad4a6a42 - add license header and clean up
• 7dd7955a - fix momentum tolernace in pyopal and tests

Compare with previous version

added 1 commit

• 142f1fbc - fix attribute name

Compare with previous version

marked this merge request as ready

@ext-rogers_c: I've had to modify pyopal tests to adapt the implementation of momentum tolerance from distribution command. I hope I managed to get it right... Please, check it, and send me any modification you think is convenient

added 1 commit

• 0e1c1b72 - disable check momentum if parameter is negative

Compare with previous version

requested review from @carl_j, @ext-rogers_c, and @adelmann

approved this merge request

resolved all threads

added Add/modify Doc Cleanup labels

approved this merge request

added 1 commit

• ce9a6032 - move back MOMENTUMTOLERNACE into BEAM command

Compare with previous version

added 1 commit

• c4c32379 - remove comment and fix space

Compare with previous version

added 1 commit

• 51cbf15b - fix space

Compare with previous version

requested review from @snuverink_j

added 2 commits

• efd7b23a - fix variable initialisation
• 8f9611bb - fixing coding style in PartData for member variables

Compare with previous version

approved this merge request

approved this merge request

resolved all threads

added 52 commits

• 8f9611bb...654699f4 - 51 commits from branch master
• c0da8dd2 - Merge remote-tracking branch 'upstream/master' into 765-momentum-tolerance-in-opal-t

Compare with previous version

@snuverink_j, @adelmann: Could you approve it again? I had to update the branch to resolve some merge conflicts.

approved this merge request

approved this merge request

merged

mentioned in commit 8a76dc83

mentioned in issue #727 (closed)