Skip to content

Addition of the Photon Filter#1860

Open
iffatzarif wants to merge 1 commit into
Mu2e:mainfrom
iffatzarif:PhotonFilter
Open

Addition of the Photon Filter#1860
iffatzarif wants to merge 1 commit into
Mu2e:mainfrom
iffatzarif:PhotonFilter

Conversation

@iffatzarif

Copy link
Copy Markdown
Contributor

We added a Photon ID filter that uses box cuts as well as helix based vetoing. This is from my Yale research project focusing on online photon identification and is complementary to machine learning based approaches. This PR includes the histograms that I used for analysis and a future PR will move this into a diagnostic plug-in.

@michaelmackenzie michaelmackenzie self-requested a review June 16, 2026 19:04
@FNALbuild

Copy link
Copy Markdown
Collaborator

Hi @iffatzarif,
You have proposed changes to files in these packages:

  • Filters

which require these tests: build.

@Mu2e/fnalbuild-users, @Mu2e/write have access to CI actions on main.

⌛ The following tests have been triggered for b1503e3: build (Build queue - API unavailable)

About FNALbuild. Code review on Mu2e/Offline.

@FNALbuild

Copy link
Copy Markdown
Collaborator

☔ The build is failing at b1503e3.

scons: *** [build/al9-prof-e29-p101/Offline/tmp/Filters/src/PhotonFilter_module.os] Error 1
Test Result Details
test with Command did not list any other PRs to include
merge Merged b1503e3 at 2b0f51a
build (prof) Log file.
ceSimReco 〰️ Log file.
g4test_03MT 〰️ Log file.
transportOnly 〰️ Log file.
POT 〰️ Log file.
g4study 〰️ Log file.
cosmicSimReco 〰️ Log file.
cosmicOffSpill 〰️ Log file.
ceSteps 〰️ Log file.
ceDigi 〰️ Log file.
muDauSteps 〰️ Log file.
ceMix 〰️ Log file.
rootOverlaps 〰️ Log file.
g4surfaceCheck 〰️ Log file.
trigger Log file.
check_cmake 〰️ Log file.
FIXME, TODO TODO (0) FIXME (0) in 1 files
clang-tidy ➡️ 3 errors 137 warnings
whitespace check no whitespace errors found

N.B. These results were obtained from a build of this Pull Request at b1503e3 after being merged into the base branch at 2b0f51a.

For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants