Skip to content

GetJESSF, SystematicHelper, etc..#50

Merged
choij1589 merged 57 commits intoCMSSNU:mainfrom
kyj519:main
Jul 31, 2025
Merged

GetJESSF, SystematicHelper, etc..#50
choij1589 merged 57 commits intoCMSSNU:mainfrom
kyj519:main

Conversation

@kyj519
Copy link
Contributor

@kyj519 kyj519 commented Jul 31, 2025

  1. Ignore data/Run3_v12_Run2_v9.
    There have been many changes under this path, but since this version is being deprecated, you can safely ignore it.

  2. Drop SKFlat.py entirely from the codebase.

  3. Many systematics are now decorrelated by era.
    Added a decorrelated_by_era feature in SystematicHelper.cc—you must now supply a DataEra in the constructor to use this.

  4. Constructor signature changed in MyCorrection.cc.
    The b-tagging ROOT file selection can now vary per baseline selection; default initialization remains unchanged for backward compatibility.

  5. Updated template/run.sh & setup.sh for micromamba paths.
    They now handle cases where micromamba isn’t installed under /opt. Please review to ensure alternative install locations work correctly.

  6. Modify setup script’s SKNANO_HOME.
    It’s currently fixed to a custom directory—confirm if this was intentional or should be parameterized.

  7. Added phi parameter to GetJESSF.
    Jet-φ–dependent corrections were introduced in the 2023 BPix era, so we now pass phi into GetJESSF.

@choij1589
Copy link
Collaborator

choij1589 commented Jul 31, 2025

Hi @yjk519 , micromamba works okay within singularity environment.

DiLepton.cc missing systHelper update, but I will update it in the next PR.

@choij1589 choij1589 merged commit 4a3a41b into CMSSNU:main Jul 31, 2025
1 check failed
kyj519 added a commit that referenced this pull request Aug 1, 2025
Following PR from #50 - JES tag updates + etc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants