BFC options for FWD productions #759
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -1707,6 +1707,8 @@ Bfc_st BFC[] = { // standard chains | |||||
| "StFcsPointMaker","StFcsPointMaker,libMinuit","Fill FCS points", kFALSE}, | ||||||
| {"fcsPi0Libs","", "", "MuDst", "", "StFcsPi0FinderForEcal", "Libs for FCS Pi0 Finder", kFALSE}, | ||||||
| // FTT | ||||||
| {"fttSim","fttChain","","StEvent,fttDb", | ||||||
| "StFttFastSimulatorMaker","StFttFastSimulatorMaker","Ftt Fast Simulator", kFALSE}, | ||||||
| {"ftt","fttChain","","FttDat,FttHitCalib,FttClu,FttPoint", "StMaker","StChain","FST chain" ,kFALSE}, | ||||||
| {"FttDat","","fttChain","StEvent","StFttRawHitMaker","StFttRawHitMaker,StEvent", | ||||||
| "sTGC Raw hit maker", kFALSE}, | ||||||
|
|
@@ -1721,6 +1723,9 @@ Bfc_st BFC[] = { // standard chains | |||||
| {"FwdTrack","","","fcsDb","StFwdTrackMaker", | ||||||
| "XMLIO,genfit2,KiTrack,StarGeneratorUtil,libMathMore,StEventUtilities,StEpdUtil,StFwdTrackMaker", | ||||||
|
Contributor
There was a problem hiding this comment. [minor] [off topic]. Could you comment on why the tracker depends on event generator utilities? |
||||||
| "Forward Track Maker", kFALSE}, | ||||||
| {"FcsTrackMatch","","","FwdTrack","StFcsTrackMatchMaker", | ||||||
| "StFcsTrackMatchMaker", | ||||||
|
Comment on lines
+1726
to
+1727
There was a problem hiding this comment. [nitpick] The formatting is inconsistent with other entries. The second line should be aligned with the opening quote of the first line to maintain consistent indentation style throughout the file. |
||||||
| "Forward Track to FCS Match Maker", kFALSE}, | ||||||
|
|
||||||
| #if 0 | ||||||
| {"fpd" ,"fpd","","", "StFpdMaker","StFpdMaker","FPD/BBC Data base chain",kFALSE}, | ||||||
|
|
@@ -1949,6 +1954,7 @@ Bfc_st BFC[] = { // standard chains | |||||
| {"PicoVtxVpdOrDefault","","","-PicoVtxDefault" ,"","","pico Vtx cut on Tof and VPD or default",kFALSE}, | ||||||
| {"PicoVtxFXT" ,"","","-PicoVtxDefault" ,"" ,"","pico Vtx constraint on FXT [198,202] mode",kFALSE}, | ||||||
| {"PicoVtxMtd" ,"","","-PicoVtxDefault" ,"" ,"","pico Vtx using MTD matching mode",kFALSE}, | ||||||
| {"PicoVtxless" ,"","","-PicoVtxDefault" ,"" ,"","pico Vtx NOT required for FWD" ,kFALSE}, | ||||||
|
Contributor
There was a problem hiding this comment.
Suggested change
Contributor
There was a problem hiding this comment. Actually, don't we need a concurrent change in ...near line 698 of StBFChain.cxx. |
||||||
| {"PicoCovMtxSkip" ,"","","" ,"" ,"","Do not write covariance matrices to picoDst (default)",kFALSE}, | ||||||
| {"PicoCovMtxWrite","","","-PicoCovMtxSkip" ,"" ,"","Write track covariance matrices to picoDst",kFALSE}, | ||||||
| {"PicoBEmcSmdSkip" ,"","","" ,"" ,"","Do not write BSMD to picoDst (default)",kFALSE}, | ||||||
|
|
||||||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The description 'Ftt Fast Simulator' should use consistent capitalization with other entries. Consider 'FTT Fast Simulator' to match the naming convention used elsewhere in the file.