Skip to content

Commit 7da557d

Browse files
committed
make CSCSegmentBuilder produce a cfi file including algo_psets
1 parent 2bfcf1a commit 7da557d

File tree

1 file changed

+4
-19
lines changed

1 file changed

+4
-19
lines changed

RecoLocalMuon/CSCSegment/src/CSCSegmentBuilder.cc

Lines changed: 4 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -106,25 +106,10 @@ void CSCSegmentBuilder::fillPSetDescription(edm::ParameterSetDescription& desc)
106106
/// Top-level parameters
107107
desc.add<int>("algo_type", 5)->setComment("Choice of the building algo: 1 SK, 2 TC, 3 DF, 4 ST, 5 RU, ...");
108108

109-
// Define the nested PSet for individual configurations
110-
edm::ParameterSetDescription algoPSet;
111-
algoPSet.add<std::vector<int>>("parameters_per_chamber_type", {1, 2, 3, 4, 5, 6, 5, 6, 5, 6});
112-
113-
// Define the VPSet inside algo_psets
109+
// Define the validator for entries in algo_psets
114110
edm::ParameterSetDescription innerPSet;
115-
// Allow any parameter in the innerPSet
116-
innerPSet.setAllowAnything();
117-
118-
// Fill the VPSet for algo_psets
119-
edm::ParameterSetDescription innerAlgoPSet;
120-
innerAlgoPSet.addVPSet("algo_psets", innerPSet);
121-
122-
// Additional top-level keys
123-
algoPSet.addVPSet("algo_psets", innerPSet);
124-
algoPSet.add<std::string>("algo_name", "CSCSegAlgoRU");
125-
algoPSet.add<std::vector<std::string>>(
126-
"chamber_types", {"ME1/a", "ME1/b", "ME1/2", "ME1/3", "ME2/1", "ME2/2", "ME3/1", "ME3/2", "ME4/1", "ME4/2"});
111+
innerPSet.setAllowAnything(); // Allow any parameters in the inner PSet
127112

128-
// Add to the top-level VPSet
129-
desc.addVPSet("algo_psets", algoPSet);
113+
// Add algo_psets as a VPSet with an empty default value
114+
desc.addVPSet("algo_psets", innerPSet, {})->setComment("Default empty VPSet, can contain anything");
130115
}

0 commit comments

Comments
 (0)