Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
S src
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 61
    • Issues 61
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Code Review
    • Issue
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • OPAL
  • src
  • Issues
  • #701

Closed
Open
Created Dec 20, 2021 by albajacas_a@albajacas_aDeveloper3 of 3 tasks completed3/3 tasks

Check momentum between `FROMFILE` distribution and BEAM

Summary

In the MR !513 (merged) we made that the BEAM command checks if energy or pc or gamma have been given, since the default P0 is not used anymore.

But this means that in the case where the distribution is FROMFILE, an energy needs to be given even though it should be read from the distribution file. For example this input file doesn't work anymore because there is no energy or gamma or pc in the BEAM command. It used to work because the default P0 kicked in, even though it was immediately replaced when teh distirbution file was read in.

Of course a work-around is to give any dummy energy, and it will be over-written when the distribution file is read in. But this will confuse users since they can give an energy different from the energy in the distribution file. The momentum specified in the BEAM command and in FROMFILE distribution type must be verified so that both values are consistent with each other.

Also, the regression tests that use FROMFILE still have PC in the BEAM command, that is why no regression tests had failed after this change.

TODO list:

  • Add cross-check between BEAM command energy and FROMFILE energy (done in !562 (merged) )
  • Update broken regression tests that used FROMFILE and the default P0 ( regression-tests#109 (closed))
  • Update manual section 15.3.1 regarding FROMFILE type distribution. (OPAL/documentation/manual#76 (closed))
Edited Feb 18, 2022 by albajacas_a
Assignee
Assign to
2022.1
Milestone
2022.1 (Past due)
Assign milestone
Time tracking