Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change FEStatus flag labels in ECAL DQM #44402

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

kyungminparkdrums
Copy link

@kyungminparkdrums kyungminparkdrums commented Mar 14, 2024

PR description:

This PR changes the flag labels of the FE Status plots in the ECAL DQM plot

  • FIFOFULL -> FORCEDFULLSUPP
  • FULLL1ADESYNC -> FIFOFULL

PR validation:

PR is validated by running the ECAL online DQM configuration and verifying the plots on a test DQM GUI.

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

This is the Master PR. Backports are made to 14_0_X in the PR #44404

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 14, 2024

cms-bot internal usage

@kyungminparkdrums kyungminparkdrums changed the title Change FIFOFULL flag to FORCEDFULLSUPP and FULLL1ADESYNC to FIFOFULL Change FEstatus flag labels in ECAL DQM Mar 14, 2024
@kyungminparkdrums kyungminparkdrums changed the title Change FEstatus flag labels in ECAL DQM Change FIFOFULL flag to FORCEDFULLSUPP and FULLL1ADESYNC to FIFOFULL Mar 14, 2024
@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44402/39476

  • This PR adds an extra 12KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @kyungminparkdrums for master.

It involves the following packages:

  • DQM/EcalCommon (dqm)
  • DQM/EcalMonitorClient (dqm)
  • DQM/EcalMonitorTasks (dqm)

@nothingface0, @tjavaid, @syuvivida, @antoniovagnerini, @rvenditti, @cmsbuild can you please review it and eventually sign? Thanks.
@thomreis, @ReyerBand, @rchatter, @wang0jin, @argiro this is something you requested to watch as well.
@sextonkennedy, @antoniovilela, @rappoccio you are the release manager for this.

cms-bot commands are listed here

@kyungminparkdrums kyungminparkdrums marked this pull request as ready for review March 14, 2024 14:20
@kyungminparkdrums kyungminparkdrums changed the title Change FIFOFULL flag to FORCEDFULLSUPP and FULLL1ADESYNC to FIFOFULL Change FEStatus flag labels in ECAL DQM Mar 14, 2024
@tjavaid
Copy link

tjavaid commented Mar 14, 2024

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: Build HeaderConsistency ClangBuild
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-faa370/38140/summary.html
COMMIT: 233a0ae
CMSSW: CMSSW_14_1_X_2024-03-13-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/44402/38140/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-faa370/38140/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-faa370/38140/git-merge-result

Build

I found compilation error when building:

>> Leaving Package PhysicsTools/ONNXRuntime
>> Package PhysicsTools/ONNXRuntime built
Entering library rule at PhysicsTools/ONNXRuntime
>> Compiling  src/PhysicsTools/ONNXRuntime/src/ONNXRuntime.cc
In file included from src/PhysicsTools/ONNXRuntime/src/ONNXRuntime.cc:8:
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_14_1_X_2024-03-13-1100/src/PhysicsTools/ONNXRuntime/interface/ONNXRuntime.h:19:10: fatal error: onnxruntime/onnxruntime_cxx_api.h: No such file or directory
   19 | #include "onnxruntime/onnxruntime_cxx_api.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
In file included from src/PhysicsTools/ONNXRuntime/src/ONNXRuntime.cc:8:
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_14_1_X_2024-03-13-1100/src/PhysicsTools/ONNXRuntime/interface/ONNXRuntime.h:19:10: fatal error: onnxruntime/onnxruntime_cxx_api.h: No such file or directory


Clang Build

I found compilation error while trying to compile with clang. Command used:

USER_CUDA_FLAGS='--expt-relaxed-constexpr' USER_CXXFLAGS='-Wno-register -fsyntax-only' scram build -k -j 32 COMPILER='llvm compile'

>> Entering Package DQM/EcalMonitorClient
>> Entering Package DQM/EcalMonitorTasks
>> Entering Package PhysicsTools/NanoAOD
>> Entering Package PhysicsTools/ONNXRuntime
>> Compile sequence completed for CMSSW CMSSW_14_1_X_2024-03-13-1100
gmake: *** [There are compilation/build errors. Please see the detail log above.] Error 1
+ eval scram build outputlog '&&' '(python3' /data/cmsbld/jenkins/workspace/ib-run-pr-tests/cms-bot/buildLogAnalyzer.py --logDir /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_14_1_X_2024-03-13-1100/tmp/el8_amd64_gcc12/cache/log/src '||' 'true)'
++ scram build outputlog
>> Entering Package Alignment/OfflineValidation
>> Compiling  src/Alignment/OfflineValidation/bin/DMRmerge.cc
>> Compiling  src/Alignment/OfflineValidation/bin/Options.cc


@kyungminparkdrums
Copy link
Author

Hi, the error is unrelated to the changes I've made in the scripts inside the DQM for the PR.

@antoniovilela
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-faa370/38311/summary.html
COMMIT: 233a0ae
CMSSW: CMSSW_14_1_X_2024-03-20-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44402/38311/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

@tjavaid
Copy link

tjavaid commented Mar 21, 2024

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @rappoccio, @sextonkennedy, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2)

@antoniovilela
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit 4ce99ab into cms-sw:master Mar 21, 2024
11 checks passed
artlbv added a commit to artlbv/cmssw that referenced this pull request Mar 25, 2024
commit b3970a8
Author: Artur Lobanov <[email protected]>
Date:   Fri Mar 22 17:21:00 2024 +0100

    Cleaning up AXO model from L1Global

commit ccb11e8
Author: Artur Lobanov <[email protected]>
Date:   Fri Mar 22 12:31:05 2024 +0100

    Remove spurious comments

commit 261c747
Author: Artur Lobanov <[email protected]>
Date:   Fri Mar 22 12:30:04 2024 +0100

    Apply suggestions from code review

    Co-authored-by: Marco Musich <[email protected]>

commit 1530330
Author: Artur Lobanov <[email protected]>
Date:   Fri Mar 22 11:10:42 2024 +0100

    Remove AXO model in customize HLT

commit c691834
Author: Artur Lobanov <[email protected]>
Date:   Fri Mar 22 10:37:44 2024 +0100

    Remove debug print

commit bbd270a
Author: Artur Lobanov <[email protected]>
Date:   Thu Mar 21 23:36:47 2024 +0100

    Apply code format

commit 215cb70
Author: Artur Lobanov <[email protected]>
Date:   Thu Mar 21 23:29:08 2024 +0100

    Set AXO condition model from template

commit 1dec93d
Author: Artur Lobanov <[email protected]>
Date:   Thu Mar 21 23:29:08 2024 +0100

    Set AXO condition model from template

commit 3e82bc4
Author: Artur Lobanov <[email protected]>
Date:   Thu Mar 21 23:28:48 2024 +0100

    Remove Axo model setting from L1 Global

commit 2c51218
Author: Artur Lobanov <[email protected]>
Date:   Thu Mar 21 22:26:54 2024 +0100

    Add backwards compatibility to AXO condition in TM parser

commit 12c3ddb
Author: Artur Lobanov <[email protected]>
Date:   Fri Mar 22 10:17:34 2024 +0100

    Original AXO emu changes from Melissa quinnanm

commit 4ce99ab
Merge: 7911d8c 233a0ae
Author: cmsbuild <[email protected]>
Date:   Thu Mar 21 20:29:32 2024 +0100

    Merge pull request cms-sw#44402 from kyungminparkdrums/fifoflagchange_141X

    Change FEStatus flag labels in ECAL DQM

commit 7911d8c
Merge: 86d37c3 70494ae
Author: cmsbuild <[email protected]>
Date:   Thu Mar 21 18:19:23 2024 +0100

    Merge pull request cms-sw#44458 from fwyzard/improve_alpaka_Pixel_memory_usage_141x

    Improve the memory usage in the alpaka pixel reconstruction

commit 70494ae
Author: Andrea Bocci <[email protected]>
Date:   Tue Mar 19 18:43:46 2024 +0100

    Update OneToManyAssocRandomAccess to use the preferred warp size

commit a7047fd
Author: Andrea Bocci <[email protected]>
Date:   Mon Mar 18 23:01:38 2024 +0100

    Use cached memory buffers in WordFedAppender

    Co-authored-by: Matti Kortelainen <[email protected]>

commit 5a2feb1
Author: Andrea Bocci <[email protected]>
Date:   Mon Mar 18 22:41:49 2024 +0100

    Improve memory usage in SiPixelRawToCluster::acquire()

    Allocate device memory only when actually used.

    Co-authored-by: Matti Kortelainen <[email protected]>

commit 4bf2fb9
Author: Andrea Bocci <[email protected]>
Date:   Mon Mar 18 22:29:46 2024 +0100

    Remove unused variable from SiPixelRawToCluster::acquire()

    Co-authored-by: Matti Kortelainen <[email protected]>

commit 418f6e2
Author: Andrea Bocci <[email protected]>
Date:   Mon Mar 18 22:24:17 2024 +0100

    Fix the TrackingRecHitsDevice constructor

    Fix the hitsModuleStart copy to use the correct source (device-to-device
    instead of host-to-device).

    Improve the offsetBPIX2 copy and use the data member as source, to guarantee
    the lifetime of the source.

    Remove the queue synchronisation.

    Update the unit test accordingly.

commit 233a0ae
Author: Kyungmin Park <[email protected]>
Date:   Thu Mar 14 13:02:17 2024 +0100

    Change FIFOFULL flag to FORCEDFULLSUPP and FULLL1ADESYNC to FIFOFULL

Co-authored-by: Andrea Bocci <[email protected]>
Co-authored-by: Artur Lobanov <[email protected]>
Co-authored-by: cmsbuild <[email protected]>
Co-authored-by: Kyungmin Park <[email protected]>
Co-authored-by: Marco Musich <[email protected]>
Co-authored-by: Matti Kortelainen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants