Skip to content

Commit d5e3270

Browse files
committed
Revert "Merge branch 'develop' into feature/lynnt_ccnuefilter"
This reverts commit d93ec8e, reversing changes made to d46ac48.
1 parent bb0bde1 commit d5e3270

24 files changed

Lines changed: 101 additions & 1881 deletions

.github/pull_request_template.md

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,13 @@
11
## Description
22
Please provide a detailed description of the changes this pull request introduces.
33

4-
$${\color{red}!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!}$$
5-
6-
$${\color{red}\bf{\textrm{IMPORTANT UPDATE June 22nd 2025:}}}$$ If you are making a PR which is intended as a patch for the CURRENT production (which started in Spring 2025), you must make two PRs: one for develop and one for the production/v10_06_00 branch.
7-
8-
$${\color{red}!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!}$$
9-
104
## Checklist
115
- [ ] Added at least 1 label from [available labels](https://github.com/SBNSoftware/sbndcode/issues/labels?sort=name-asc).
126
- [ ] Assigned at least 1 reviewer under `Reviewers`,
137
- [ ] Assigned all contributers including yourself under `Assignees`
148
- [ ] Linked any relevant issues under `Developement`
159
- [ ] Does this PR affect CAF data format? If so, please assign a CAF maintainer ([PetrilloAtWork](https://github.com/PetrilloAtWork) or [JosiePaton](https://github.com/JosiePaton)) as additional reviewer.
1610
- [ ] Does this affect the standard workflow?
17-
- [ ] Is this PR a patch for the ongoing production? If so, separate PR must also be made for production/v10_06_00 branch!
1811

1912
### Relevant PR links (optional)
2013
Does this PR require merging another PR in a different repository (such as sbnanobj/sbnobj etc.)?

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616
cmake_minimum_required(VERSION 3.20 FATAL_ERROR)
1717

18-
set(${PROJECT_NAME}_CMAKE_PROJECT_VERSION_STRING 10.06.01)
18+
set(${PROJECT_NAME}_CMAKE_PROJECT_VERSION_STRING 10.06.00.01)
1919
find_package(cetmodules REQUIRED)
2020
project(sbndcode LANGUAGES CXX)
2121

README.md

Lines changed: 0 additions & 4 deletions
This file was deleted.

sbndcode/JobConfigurations/base/cafmakerjob_sbnd_data_base.fcl

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,8 @@ physics:
125125
# change tools in sbn to have 2D TPC sim/sp input
126126
physics.producers.pandoraShowerRazzle.SimChannelLabel: "sptpc2d:gauss"
127127
physics.producers.pandoraTrackDazzle.SimChannelLabel: "sptpc2d:gauss"
128+
128129
physics.producers.cnnid.WireLabel: "sptpc2d:gauss"
129-
# uncomment below lines to use DNN ROI SP for Razzle/Dazzle/CNNID
130-
#physics.producers.pandoraShowerRazzle.SimChannelLabel: "sptpc2d:dnnsp"
131-
#physics.producers.pandoraTrackDazzle.SimChannelLabel: "sptpc2d:dnnsp"
132-
#physics.producers.cnnid.WireLabel: "sptpc2d:dnnsp"
133130

134131
### Calorimetry for data
135132
physics.producers.cnnid.PointIdAlg.CalorimetryAlg: @local::sbnd_calorimetryalgdata

sbndcode/JobConfigurations/standard/caf/cafmakerjob_sbnd.fcl

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,6 @@ physics:
127127
physics.producers.pandoraShowerRazzle.SimChannelLabel: "simtpc2d:simpleSC"
128128
physics.producers.pandoraTrackDazzle.SimChannelLabel: "simtpc2d:simpleSC"
129129
physics.producers.cnnid.WireLabel: "simtpc2d:gauss"
130-
# uncomment below line for to use DNN ROI SP for CNNID scores
131-
#physics.producers.cnnid.WireLabel: "simtpc2d:dnnsp"
132130

133131
physics.producers.vertexCharge.CaloAlg: @local::sbnd_calorimetryalgmc
134132
physics.producers.vertexStub.CaloAlg: @local::sbnd_calorimetryalgmc

sbndcode/JobConfigurations/standard/caf/cafmakerjob_sbnd_data_sce.fcl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ physics.producers.cafmaker.TrackChi2PidLabel: "pandoraSCEPidData"
1111
physics.producers.cafmaker.CRTSpacePointMatchLabel: "crtspacepointmatchingSCE"
1212
physics.producers.cafmaker.SBNDCRTTrackMatchLabel: "crttrackmatchingSCE"
1313
physics.producers.cafmaker.OpT0Label: "opt0finderSCE"
14-
physics.producers.cafmaker.TPCPMTBarycenterMatchLabel: "tpcpmtbarycentermatchingSCE"
1514
physics.producers.cafmaker.CVNLabel: "cvnSCE"
1615

1716
physics.producers.cnnid.ClusterModuleLabel: "pandoraSCE"

sbndcode/JobConfigurations/standard/caf/cafmakerjob_sbnd_sce.fcl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ physics.producers.cafmaker.SBNDCRTTrackMatchLabel: "crttrackmatchingSCE"
1313
physics.producers.cafmaker.CRTHitMatchLabel: "pandoraSCETrackCRTHit"
1414
physics.producers.cafmaker.CRTTrackMatchLabel: "pandoraSCETrackCRTTrack"
1515
physics.producers.cafmaker.OpT0Label: "opt0finderSCE"
16-
physics.producers.cafmaker.TPCPMTBarycenterMatchLabel: "tpcpmtbarycentermatchingSCE"
1716
physics.producers.cafmaker.CVNLabel: "cvnSCE"
1817

1918
physics.producers.cnnid.ClusterModuleLabel: "pandoraSCE"

sbndcode/JobConfigurations/standard/reco/reco1_data.fcl

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,3 @@ physics.ana: [superadata]
3535
outputs.out1.SelectEvents: [ "reco1" ]
3636

3737
physics.producers.gaushit.CalDataModuleLabel: "sptpc2d:gauss"
38-
39-
# uncomment below 4 lines to run DNN ROI finding SP
40-
#physics.producers.sptpc2d.wcls_main.outputers: ["wclsFrameSaver:spsaver" , "wclsFrameSaver:dnnsaver"]
41-
#physics.producers.sptpc2d.wcls_main.structs.use_dnnroi: true
42-
#physics.producers.sptpc2d.wcls_main.structs.nchunks: 2 # should match training config
43-
#physics.producers.sptpc2d.wcls_main.structs.tick_per_slice: 4 # should match training config
44-
45-
# uncomment below line to run DNN ROI finding SP for reco
46-
#physics.producers.gaushit.CalDataModuleLabel: "sptpc2d:dnnsp"

sbndcode/JobConfigurations/standard/standard_detsim_sbnd.fcl

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,3 @@ outputs:
9292
]
9393
}
9494
}
95-
96-
# uncomment below 4 lines to run DNN ROI finding SP
97-
# physics.producers.simtpc2d.wcls_main.outputers: ["wclsDepoFluxWriter:postdrift", "wclsFrameSaver:spsaver", "wclsFrameSaver:dnnsaver"] # "wclsFrameSaver:simdigits" <- by default, do not save RawDigits. Uncomment this line to save RawDigits and set
98-
# physics.producers.simtpc2d.wcls_main.structs.use_dnnroi: true
99-
# physics.producers.simtpc2d.wcls_main.structs.nchunks: 2 # should match training config
100-
# physics.producers.simtpc2d.wcls_main.structs.tick_per_slice: 4 # should match training config

sbndcode/JobConfigurations/standard/standard_reco1_sbnd.fcl

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,3 @@ physics.end_paths: [ @sequence::physics.end_paths, ana ]
3535
#outputs table overrides
3636
outputs.out1.dataTier: "reconstructed"
3737
outputs.out1.outputCommands: [@sequence::outputs.out1.outputCommands, @sequence::sbnd_reco1_drops]
38-
39-
# uncomment below line to run DNN ROI finding SP for reco -- must have run DNN ROI finding in detsim stage
40-
#physics.producers.gaushit.CalDataModuleLabel: "simtpc2d:dnnsp"

0 commit comments

Comments
 (0)