DQM/Integration package added back to CMSSW #4363
Conversation
|
A new Pull Request was created by @deguio for CMSSW_7_1_X. DQM/Integration package added back to CMSSW It involves the following packages: DQM/Integration The following packages do not have a category, yet: DQM/Integration @cmsbuild, @Degano, @ktf, @nclopezo can you please review it and eventually sign? Thanks. |
|
Do you really need this kind of layout? The whole script directory will be copied in path... |
|
It's not clear what we could remove right now for this July GR. We can trim down once we have everything running stably as a whole system, but many of these files correspond to services/scripts/configurations that are specific to online functionalities we either need right away or anticipate needing in the short term, so re-doing the layout would make most sense once we can make a more final call on these files. |
| @@ -0,0 +1,124 @@ | |||
| import FWCore.ParameterSet.Config as cms | |||
There was a problem hiding this comment.
why the strange filename?
There was a problem hiding this comment.
This was used as a rollback version to the fu_pp.py configuration in the same directory. However now the difference with fu_pp.py is only in the file list, so it could be removed.
There was a problem hiding this comment.
Sounds like it would be best to remove (maybe also the cvs setup files with passwords?)
On Jun 26, 2014, at 7:28 AM, "danduggan" <notifications@github.commailto:notifications@github.com> wrote:
In DQM/Integration/SMPlayback/cfg/~fu_pp.py:
@@ -0,0 +1,124 @@
+import FWCore.ParameterSet.Config as cms
This was used as a rollback version to the fu_pp.py configuration in the same directory. However now the difference with fu_pp.py is only in the file list, so it could be removed.
—
Reply to this email directly or view it on GitHubhttps://github.com//pull/4363/files#r14225803.
|
+1 |
|
this new package should to be assigned to DQM |
|
merging - but perhaps we can also get rid of the csh version of the cvs setup as as followup PR.. |
DQM/Integration package added back to CMSSW
|
thanks, |
all the added files are only used in the online. they don't affect the offline workflows. needed in 71X for the July GR. will be automatically forward ported to 72X.