Modernize deterministic mixer code

• replace uint32_t state and free functions with a small struct
• define a single overload set for mixing
• be paranoid about implicit conversions of the 'extra' state
• use the above mentioned struct everywhere, and remove copies of the code
• add a unit test (which contains a copy of the original code to verify the results are the same)
• replace use of boost::integer_traits with std::numeric_limits

added cleanup modernisation labels

/ci-test --merge

added ci-test-triggered label

• [2020-10-03 00:42] Validation started with lhcb-master-mr#1400
• [2020-10-03 17:24] Validation started with lhcb-master-mr#1403

added 1 commit

• f8765a37 - Move DeterministicMixer to LHCbMath, re-use in LoKiHlt

Compare with previous version

added 2 commits

• d7ad6f75 - Modernize deterministic mixer code
• ff6f9700 - Move DeterministicMixer to LHCbMath, re-use in LoKiHlt

Compare with previous version

added 1 commit

• 28d10c8a - Move DeterministicMixer to LHCbMath, re-use in LoKiHlt

Compare with previous version

added 1 commit

• a6bc2758 - Move DeterministicMixer to LHCbMath, re-use in LoKiHlt

Compare with previous version

added 1 commit

• a3d106fb - Move DeterministicMixer to LHCbMath, re-use in LoKiHlt

Compare with previous version

added 1 commit

• afa02aea - prefer std::numeric_limits over boost::integer_traits

Compare with previous version

added 1 commit

• a2bab979 - add unit test

Compare with previous version

added 1 commit

• abed7902 - add unit test

Compare with previous version

added 1 commit

• 02e5eb9c - add unit test

Compare with previous version

added 1 commit

• f5389223 - add unit test

Compare with previous version

assigned to @ascarabo

resolved all threads

assigned to @graven and unassigned @ascarabo

marked as a Work In Progress

assigned to @ascarabo and unassigned @graven

assigned to @graven and unassigned @ascarabo

added 1 commit

• 9eb70134 - add unit test

Compare with previous version

added 2 commits

• 83e71334 - prefer std::numeric_limits over boost::integer_traits
• 363f65e1 - add unit test

Compare with previous version

unmarked as a Work In Progress

assigned to @ascarabo

added 6 commits

• 363f65e1...b862fdb8 - 2 commits from branch master
• 215be9db - Modernize deterministic mixer code
• aedec86c - Move DeterministicMixer to LHCbMath, re-use in LoKiHlt
• 9c53e9e3 - prefer std::numeric_limits over boost::integer_traits
• dc3a3248 - add unit test

Compare with previous version

• [2020-10-04 12:57] Automatic merge failed in lhcb-master-mr#1404
• [2020-10-04 19:33] Validation started with lhcb-master-mr#1405
• [2020-10-05 09:24] Validation started with lhcb-master-mr#1407

unassigned @graven

added 1 commit

• 71e19e3f - fan-out multiple arguments

Compare with previous version

added 1 commit

• eec794df - fan-out multiple arguments

Compare with previous version

changed the description

assigned to @raaij

mentioned in issue Moore#213 (closed)

unassigned @raaij and @ascarabo

resolved all threads

merged

mentioned in commit 94f8e5a4

@graven, this doesn't seem to work with the latest gsl. See http://lhcb-nightlies.web.cern.ch/logs/build/nightly/lhcb-lcg-dev4/1428/x86_64-centos7-gcc9-opt/LHCb/#show_Kernel_LHCbMath/Kernel_LHCbMath_128

see !2796 (merged)

added Core label