Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • DaVinci DaVinci
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 18
    • Issues 18
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 13
    • Merge requests 13
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCb
  • DaVinciDaVinci
  • Merge requests
  • !350

Closed
Created Nov 05, 2019 by Carlos Vazquez Sierra@cvazquez👁‍🗨Developer
  • Report abuse
Report abuse

WIP: Update Run 1 Stripping cache

  • Overview 11
  • Commits 5
  • Pipelines 1
  • Changes 4

Follow up discussion from !345 (closed).

  • Update StrippingCache for future Run 1 incremental re-strippings.
  • Add LokiAddParticleHeader as a post-config option to include additional needed headers for some CorePhysics functors cache, !346 (merged).

Quoting @lsestini :

The error posted by Carlos is generated by the StdJets reconstruction. The merge request for HLTJets jet energy correction shouldn't affect StdJets.

I checked the last version of ParamFiles and the JEC root files for StdJets are there: $PARAMFILESROOT/data/JetEnergyCorrections_R05_v3.root/JECSYS_PV1_ETA0_CPF0_PHI0

It is simply looking in the wrong path 'JEC14R05/JECSYS_PV1_ETA0_CPF0_PHI0'

In JetMaker_Config.py JEC14R05 is an alias for the path:

if self.JetEnergyCorrection:
           algo.ApplyJEC = True
           if self.R == 0.5:
           algo.HistoPath = 'JEC14R05/'

But then it is correctly interpreted with

hsvc = HistogramSvc ( 'HistogramDataSvc' );
        alreadySet = False
        if self.R == 0.5:
            for file in hsvc.Input:
                if file == "JEC14R05 DATAFILE='$PARAMFILESROOT/data/JetEnergyCorrections_R05_v3.root' TYP='ROOT'": alreadySet = True
            if not alreadySet: hsvc.Input += [ "JEC14R05 DATAFILE='$PARAMFILESROOT/data/JetEnergyCorrections_R05_v3.root' TYP='ROOT'" ]

Is it possible that this syntax does not work anymore in this patch? Because locally it still works for me.

Edited Oct 26, 2020 by Nicole Skidmore
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: updateS21cache