cmssw icon indicating copy to clipboard operation
cmssw copied to clipboard

Update of the Phase-2 GMT Emulator

Open zhenbinwu opened this issue 1 year ago • 7 comments

PR description:

Major ukpdate of the Phase-2 GMT Emulator:

  1. Add the Phase-2 KMTF alogirthm for barrel muon
  2. Get the Phase-2 OMTF and EMTF to the Phase-2 Standalone Muon ghost busting algorithm
  3. Refined the Tracker Muon matching and quality bits for low pt muon
  4. Restructure the TPS code into header and source files.
  5. Update the Tracker Muon output to GT, sending quality score and isolation sum

zhenbinwu avatar Mar 05 '24 16:03 zhenbinwu

cms-bot internal usage

cmsbuild avatar Mar 05 '24 16:03 cmsbuild

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44319/39345

ERROR: Unable to merge PR.

See log https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44319/39345/cms-checkout-topic.log

cmsbuild avatar Mar 05 '24 16:03 cmsbuild

This PR requires input from the Phase-2 OMTF #43687 and EMTF #43766. It seems OMTF was merged while EMTF is work in progress. I will wait for that PR and rebase.

zhenbinwu avatar Mar 05 '24 17:03 zhenbinwu

@zhenbinwu Could you please merge https://github.com/zhenbinwu/cmssw/pull/2? This fixes a small configuration bug of the GMT->GT inputs.

aloeliger avatar Mar 11 '24 13:03 aloeliger

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44319/39417

ERROR: Unable to merge PR.

See log https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44319/39417/cms-checkout-topic.log

cmsbuild avatar Mar 11 '24 17:03 cmsbuild

@zhenbinwu Both of the required PRs for this PR have been merged. Could you please rebase this PR ontop of the current state of CMSSW_14_1?

aloeliger avatar Mar 15 '24 10:03 aloeliger

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44319/39501

ERROR: Unable to merge PR.

See log https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44319/39501/cms-checkout-topic.log

cmsbuild avatar Mar 15 '24 16:03 cmsbuild

@zhenbinwu Both of the required PRs for this PR have been merged. Could you please rebase this PR ontop of the current state of CMSSW_14_1?

@zhenbinwu just a ping on the rebase.

aloeliger avatar Mar 18 '24 11:03 aloeliger

@zhenbinwu Both of the required PRs for this PR have been merged. Could you please rebase this PR ontop of the current state of CMSSW_14_1?

@zhenbinwu just a ping on the rebase.

Thanks for the ping. The first rebase seems not working. I will try again tomorrow. I also have one patch will go in as well.

zhenbinwu avatar Mar 19 '24 02:03 zhenbinwu

This PR contains too many commits (401 >= 240) and will not be processed. Please ensure you have selected the correct target branch and consider squashing unnecessary commits. The processing of this PR will resume once the commit count drops below the limit.

cmsbuild avatar Mar 20 '24 00:03 cmsbuild

Mm.. this is messed up

zhenbinwu avatar Mar 20 '24 00:03 zhenbinwu

Closing this PR while making another fresh PR

zhenbinwu avatar Mar 20 '24 20:03 zhenbinwu