-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Created and added the query to obtain the values of the ECAL attributes. #22668
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22668/4027 |
A new Pull Request was created by @coder-amey (Amey Noolkar) for master. It involves the following packages: CondTools/RunInfo @ggovi, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@coder-amey can you provide the link of local sqlite which you may have produced with this code? You can take an example of a stable fill from last year. I would suggest to copy paste the dump of payload here as well. |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
The sqlite file generated using this package (fills 6300 - 6310) can be found here: A sample output (for fill 6300) from this modules is as follows: LHCInfoPopConAnalyzer:##################################################################### %MSG lhcinfo_pop_test.db:LHCInfoESAnalyzer:##################################################################### |
+1 |
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. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Hello, @franzoni, @arunhep, @lpernie and @ggovi.
@ravindkv and I have created and inserted a query to the BEAM_PHASE view of the CMS_DCS_ENV_PVSS_COND schema, in order to obtain the values of the ECAL attributes.
This query completes the LHCInfo package that was merged through this pull request and replaces the earlier dummy values with actual values from the OMDS.
I request you to kindly scrutinize our work and provide us with your valuable suggestions so that we can improve this pull request and get it merged.
Thank you for your time.
Sincerely,
Amey & Ravindra.