IAP GITLAB

Skip to content

Implementation of a templated magnetic field interface and UniformMagneticField type.

Remy Prechelt requested to merge rprechelt-magnetic-field-mixins into master

Following the examples in the ngc_demo project, I have implemented a UniformMagneticField mixin that can be added to the C8 environment hierarchy. This is tested in two unit tests in testEnvironment.cc here. This takes a fixed magnetic field vector in the constructor and returns that fixed magnetic field everywhere.

I'd appreciate your feedback @mreininghaus!

The code approval procedure is described in the wiki: Code approval procedure wiki

  • The MR is without WIP (work in progress) status
  • Make sure the most recent CI jobs (config, quality, build, test, example) all run fine with no failures
    • if "check clang-format" failed: the code contributor has to run ./do-clang-format.py --apply eventually with the --all option
    • if "check copyright" failed the code contributor has to run./do-copyright.py --add=20xy
  • Also run all the extra optional jobs "coverage", "release-clang-8", "release-u-18.04" and make sure no problems occur
  • Check in the "coverage" job output that the coverage did not decreases (!). It should always stay, or increase. If it decreased --> ask contributor to add further unit tests, and check coverage report.
  • On the MR page, open the "Open in Web IDE" tool
    • Check if the provided solution solves the Issue, discuss on gitlab
    • Check that all changes are actually related to the issue
    • There are no debug statements left, not even commented out
    • Check all changes for coding rules and guidelines
  • When all above is done
    • Add label "Code Review Finished"
Edited by Maximilian Reininghaus

Merge request reports

Loading