Skip to content
Snippets Groups Projects

purge unstable particles from Hijing

Merged Petr Balek requested to merge pbalek/athena:pbalek-atlhi-483-end-vtx into main
All threads resolved!

Occasionally, Hijing produces unstable particles (i.e. with status=2) without end vertex. That is disliked by simulation that throws errors because of that.

It was attempted to fix it by filtering such particles, see !65417 (merged). Unfortunately, FixHepMC can also delete particles, and if it removes the last outgoing particle from the vertex, it also deletes the vertex. This leaves the mother particle without the end vertex, leading to problems in simulation.

This MR moves the filtering to FixHepMC, removing all particles without end vertex regardless if created by Hijing or by FixHepMC. It also switche this filtering on for Hijing; behaviour of all other generators is unchanged.

tagging @stapiaar @ewelina @tbold

related JIRA ticket: ATLHI-483

Merge request reports

Pipeline #10363724 passed

Pipeline passed for c423acdd on pbalek:pbalek-atlhi-483-end-vtx

Approval is optional

Merged by Adam Edward BartonAdam Edward Barton 1 month ago (Feb 14, 2025 9:54am UTC)

Merge details

  • Changes merged into main with 04c13651 (commits were squashed).
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • It looks great! Thanks @pbalek

  • :white_check_mark: CI Result SUCCESS (hash c06479cb)

    Athena AthGeneration
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    tests :white_check_mark: :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
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output (remote access info)

  • Petr Balek added 1 commit

    added 1 commit

    • c423acdd - purge unstable particles from Hijing

    Compare with previous version

  • Tomasz Bold resolved all threads

    resolved all threads

  • Petr Balek marked this merge request as ready

    marked this merge request as ready

  • This merge request affects 3 packages:

    • Generators/EvgenJobTransforms
    • Generators/EvgenProdTools
    • Generators/Hijing_i

    Affected files list will not be printed in this case

  • :white_check_mark: CI Result SUCCESS (hash c423acdd)

    Athena AthGeneration
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    tests :white_check_mark: :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
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output (remote access info)

  • mentioned in commit 04c13651

  • Please register or sign in to reply
    Loading