Skip to content

Pandora CI cluster metrics#909

Open
AndyChappell wants to merge 2 commits intodevelopfrom
feature/chappell_ci_pandora_clstr
Open

Pandora CI cluster metrics#909
AndyChappell wants to merge 2 commits intodevelopfrom
feature/chappell_ci_pandora_clstr

Conversation

@AndyChappell
Copy link
Contributor

Description

This pull request adds new fcl and Pandora XML configurations to support direct production of Pandora metrics within the continuous integration system. Provisionally, these contain a number of basic cluster metrics (ARI, purity, completeness, etc), but could in principle be extended in the future.

The ci_reco2_sbnd.fcl inherits from standard_reco2_sbnd.fcl and makes the minimal changes to the Pandora configuration for this workflow to run. The PandoraSettings_CI_SBND.xml mirrors the current PandoraSettings_Master_SBND.xml. Unfortunately, because this can't inherit a workflow, so it will be necessary to ensure suitable syncrhonisation between the CI XML and the standard Master XML if the Master XML changes (the CI XML removes the visualisation algorithms and adds the cluster validation algorithm). However, the Master XML is the most stable part of the configuration, and so changes here should be relatively rare.

Checklist

  • Added at least 1 label from available labels.
  • Assigned at least 1 reviewer under Reviewers,
  • Assigned all contributers including yourself under Assignees
  • Linked any relevant issues under Developement
  • Does this PR affect CAF data format? If so, please assign a CAF maintainer (PetrilloAtWork or JosiePaton) as additional reviewer.
  • Does this affect the standard workflow?
  • Is this PR a patch for the ongoing production? If so, separate PR must also be made for production/v10_06_00 branch!

@AndyChappell AndyChappell self-assigned this Feb 5, 2026
@AndyChappell AndyChappell added enhancement New feature or request test GitHub or related test reco1/reco2 Reconstruction labels Feb 5, 2026
Copy link
Member

@henrylay97 henrylay97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this so quickly @AndyChappell!

Looks great to me, I have one request - could you put a comment line in the top of the Master XML that directs people editing it to make the same edits to the CI XML? Might prevent the future divergence you mention 😄

@AndyChappell
Copy link
Contributor Author

Done

@nathanielerowe
Copy link
Contributor

Tagging @SabrinaBrickner and @vitodb since it is a CI related PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request reco1/reco2 Reconstruction test GitHub or related test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants