Skip to content
Snippets Groups Projects

ISF_Fatras: fix status values for secondaries produced by ISF_Fatras

Merged John Derek Chapman requested to merge jchapman/athena:ISF_FatrasSecondaryStatus_main into main
All threads resolved!

Cleaned-up the setting of initial properties of ISFParticles created for secondary particles created by ISF_Fatras. Mostly this is related to setting the status correctly and (for now) explicitly setting the ISFParticle::id to HepMC::UNDEFINED_ID. (In future this will be set by the ISFTruthIncident - see !71407 (merged)).

This MR also adds some FIXME comments to be followed up in future MRs.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Apart from the compilation errors, the changes look good. @jchapman, can you please have a look at the errors?

    L1 shifter

  • John Derek Chapman marked this merge request as draft

    marked this merge request as draft

  • added 1 commit

    • 291f8013 - ISF_Fatras: fix status values for secondaries produced by ISF_Fatras

    Compare with previous version

  • John Derek Chapman resolved all threads

    resolved all threads

  • John Derek Chapman marked this merge request as ready

    marked this merge request as ready

  • :pencil: :pushpin: build area is cleaned as the previous build was for the MR labeled as full-build

  • This merge request affects 2 packages:

    • Simulation/ISF/ISF_Fatras/ISF_FatrasTools
    • Simulation/ISF/ISF_Fatras/ISF_FatrasToolsG4

    Affected files list will not be printed in this case

    Adding @lmijovic ,@jchapman ,@tcuhadar as watchers

  • :white_check_mark: CI Result SUCCESS (hash 291f8013)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9639] (remote access info)

  • Looks good, approving --L1

  • merged

  • Tadej Novak mentioned in commit c2b9658e

    mentioned in commit c2b9658e

  • Please register or sign in to reply
    Loading