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

Removed FWCore/ParameterSet dependecies from *Formats packages #33897

Merged
merged 6 commits into from
Jun 3, 2021
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 0 additions & 1 deletion CondFormats/CSCObjects/BuildFile.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<use name="DataFormats/MuonDetId"/>
<use name="FWCore/MessageLogger"/>
<use name="FWCore/ParameterSet"/>
<use name="CondFormats/Serialization"/>
<use name="boost_serialization"/>
<export>
Expand Down
11 changes: 1 addition & 10 deletions CondFormats/CSCObjects/interface/CSCReadoutMapping.h
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,10 @@

#include "CondFormats/Serialization/interface/Serializable.h"

#include <DataFormats/MuonDetId/interface/CSCDetId.h>
#include "DataFormats/MuonDetId/interface/CSCDetId.h"
#include <vector>
#include <map>

namespace edm {
class ParameterSet;
}

class CSCReadoutMapping {
public:
/// Default constructor
Expand Down Expand Up @@ -90,11 +86,6 @@ class CSCReadoutMapping {
*/
int chamber(int endcap, int station, int vmecrate, int dmb, int tmb) const;

/**
* Fill mapping store
*/
virtual void fill(const edm::ParameterSet&) = 0;

///returns hardware ids given chamber id
CSCLabel findHardwareId(const CSCDetId&) const;
///returns vmecrate given CSCDetId
Expand Down
7 changes: 3 additions & 4 deletions CondFormats/CSCObjects/interface/CSCReadoutMappingFromFile.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,21 +8,20 @@
* Find file from FileInPath of ParameterSet passed from calling E_Producer.
*/

#include <CondFormats/CSCObjects/interface/CSCReadoutMappingForSliceTest.h>
#include <FWCore/ParameterSet/interface/ParameterSet.h>
#include "CondFormats/CSCObjects/interface/CSCReadoutMappingForSliceTest.h"
#include <string>

class CSCReadoutMappingFromFile : public CSCReadoutMappingForSliceTest {
public:
/// Constructor
explicit CSCReadoutMappingFromFile(const edm::ParameterSet& ps);
explicit CSCReadoutMappingFromFile(std::string iFullPathFileName);
CSCReadoutMappingFromFile() {}

/// Destructor
~CSCReadoutMappingFromFile() override;

/// Fill mapping store
void fill(const edm::ParameterSet& ps) override;
void fill(std::string iFullPathFileName);

private:
std::string theMappingFile;
Expand Down
14 changes: 7 additions & 7 deletions CondFormats/CSCObjects/src/CSCReadoutMappingFromFile.cc
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
#include <CondFormats/CSCObjects/interface/CSCReadoutMappingFromFile.h>
#include <FWCore/MessageLogger/interface/MessageLogger.h>
#include <FWCore/ParameterSet/interface/FileInPath.h>
#include "CondFormats/CSCObjects/interface/CSCReadoutMappingFromFile.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
#include <iostream>
#include <fstream>
#include <sstream>

CSCReadoutMappingFromFile::CSCReadoutMappingFromFile(const edm::ParameterSet& ps) { fill(ps); }
CSCReadoutMappingFromFile::CSCReadoutMappingFromFile(std::string iName) { fill(std::move(iName)); }

CSCReadoutMappingFromFile::~CSCReadoutMappingFromFile() {}

void CSCReadoutMappingFromFile::fill(const edm::ParameterSet& ps) {
edm::FileInPath fp = ps.getParameter<edm::FileInPath>("theMappingFile");
theMappingFile = fp.fullPath();
void CSCReadoutMappingFromFile::fill(std::string fileName) {
theMappingFile = std::move(fileName);
// edm::FileInPath fp = ps.getParameter<edm::FileInPath>("theMappingFile");
//theMappingFile = fp.fullPath();
std::ifstream in(theMappingFile.c_str());
std::string line;
const std::string commentFlag = "#";
Expand Down
2 changes: 0 additions & 2 deletions CondFormats/CSCObjects/test/BuildFile.xml
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,10 @@
</library>

<bin name="testCSCMapping" file="testCSCMapping.cpp">
<use name="FWCore/ParameterSetReader"/>
<use name="cppunit"/>
</bin>

<bin name="testCSCTriggerMapping" file="testCSCTriggerMapping.cpp">
<use name="FWCore/ParameterSetReader"/>
<use name="cppunit"/>
</bin>

Expand Down
15 changes: 5 additions & 10 deletions CondFormats/CSCObjects/test/testCSCMapping.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,10 @@
*/

#include <cppunit/extensions/HelperMacros.h>
#include <FWCore/Utilities/interface/Exception.h>
#include <FWCore/PluginManager/interface/ProblemTracker.h>
#include <FWCore/ParameterSet/interface/ParameterSet.h>
#include "FWCore/ParameterSetReader/interface/ParameterSetReader.h"
#include <FWCore/ParameterSet/interface/FileInPath.h>
#include <CondFormats/CSCObjects/interface/CSCReadoutMappingFromFile.h>
#include <DataFormats/MuonDetId/interface/CSCDetId.h>
#include "FWCore/Utilities/interface/Exception.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "CondFormats/CSCObjects/interface/CSCReadoutMappingFromFile.h"
#include "DataFormats/MuonDetId/interface/CSCDetId.h"
#include "Utilities/Testing/interface/CppUnit_testdriver.icpp"
#include <iostream>
#include <cstdlib>
Expand Down Expand Up @@ -60,13 +57,11 @@ class testCSCMapping : public CppUnit::TestFixture {
void testCSCMapping::testRead() {
edm::FileInPath fip("CondFormats/CSCObjects/data/csc_slice_test_map.txt");
std::cout << "Attempt to set FileInPath to " << fip.fullPath() << std::endl;
edm::ParameterSet ps;
ps.addParameter<edm::FileInPath>("theMappingFile", fip);

std::cout << myName_ << ": --- t e s t C S C M a p p i n g ---" << std::endl;
std::cout << "start " << dashedLine << std::endl;

CSCReadoutMappingFromFile theMapping(ps);
CSCReadoutMappingFromFile theMapping(fip.fullPath());

// The following labels are irrelevant to hardware in slice test
int tmb = -1;
Expand Down
2 changes: 0 additions & 2 deletions CondFormats/CSCObjects/test/testCSCTriggerMapping.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,6 @@
#include <CondFormats/CSCObjects/interface/CSCTriggerMappingFromFile.h>
#include <DataFormats/MuonDetId/interface/CSCDetId.h>
#include "Utilities/Testing/interface/CppUnit_testdriver.icpp"
#include "FWCore/ParameterSet/interface/ParameterSet.h"
#include "FWCore/ParameterSetReader/interface/ParameterSetReader.h"
#include <iostream>
#include <cstdlib>

Expand Down
2 changes: 1 addition & 1 deletion CondFormats/JetMETObjects/test/BuildFile.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
</bin>

<bin name="TestCondFormatsJetMETObjectsJetCorrectorParameters" file="JetCorrectorParameters_t.cpp">
<use name="FWCore/ParameterSet"/>
<use name="FWCore/Utilities"/>
<use name="CondFormats/JetMETObjects"/>
<use name="cppunit"/>
</bin>
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#include "Utilities/Testing/interface/CppUnit_testdriver.icpp"
#include "cppunit/extensions/HelperMacros.h"

#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "CondFormats/JetMETObjects/interface/JetCorrectorParameters.h"
#include "CondFormats/JetMETObjects/interface/FactorizedJetCorrector.h"

Expand Down
2 changes: 1 addition & 1 deletion CondFormats/L1TObjects/src/L1MuDTEtaPatternLut.cc
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
// Collaborating Class Headers --
//-------------------------------

#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "CondFormats/L1TObjects/interface/L1TriggerLutFile.h"

using namespace std;
Expand Down
2 changes: 1 addition & 1 deletion CondFormats/L1TObjects/src/L1MuDTExtLut.cc
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
// Collaborating Class Headers --
//-------------------------------

#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "CondFormats/L1TObjects/interface/DTTFBitArray.h"
#include "CondFormats/L1TObjects/interface/L1MuDTExtParam.h"
#include "CondFormats/L1TObjects/interface/L1TriggerLutFile.h"
Expand Down
2 changes: 1 addition & 1 deletion CondFormats/L1TObjects/src/L1MuDTPhiLut.cc
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
// Collaborating Class Headers --
//-------------------------------

#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "CondFormats/L1TObjects/interface/DTTFBitArray.h"
#include "CondFormats/L1TObjects/interface/L1TriggerLutFile.h"

Expand Down
2 changes: 1 addition & 1 deletion CondFormats/L1TObjects/src/L1MuDTPtaLut.cc
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
// Collaborating Class Headers --
//-------------------------------

#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "CondFormats/L1TObjects/interface/DTTFBitArray.h"
#include "CondFormats/L1TObjects/interface/L1MuDTAssParam.h"
#include "CondFormats/L1TObjects/interface/L1TriggerLutFile.h"
Expand Down
2 changes: 1 addition & 1 deletion CondFormats/L1TObjects/src/L1MuDTQualPatternLut.cc
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
// Collaborating Class Headers --
//-------------------------------

#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "CondFormats/L1TObjects/interface/L1TriggerLutFile.h"

using namespace std;
Expand Down
4 changes: 2 additions & 2 deletions CondFormats/PPSObjects/src/PPSDirectSimulationData.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
#include "DataFormats/CTPPSDetId/interface/CTPPSPixelDetId.h"
#include "DataFormats/CTPPSDetId/interface/CTPPSDiamondDetId.h"
#include "FWCore/Utilities/interface/Exception.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"

#include "TFile.h"

Expand Down Expand Up @@ -107,4 +107,4 @@ std::string PPSDirectSimulationData::replace(std::string input, const std::strin
start_pos += to.length();
}
return input;
}
}
4 changes: 2 additions & 2 deletions CondFormats/PhysicsToolsObjects/test/SiStripDeDx2DBuilder.cc
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/ParameterSet/interface/ParameterSet.h"
#include "CommonTools/ConditionDBWriter/interface/ConditionDBWriter.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
//#include "FWCore/Utilities/interface/FileInPath.h"

#include "CondFormats/PhysicsToolsObjects/interface/Histogram2D.h"

Expand All @@ -24,7 +24,7 @@ class SiStripDeDx2DBuilder : public edm::EDAnalyzer {
virtual void analyze(const edm::Event&, const edm::EventSetup&);

private:
edm::FileInPath fp_;
//edm::FileInPath fp_;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOTE: this was left intentionally commented out as the code that uses fp_ was already commented out.

bool printdebug_;
};

Expand Down
4 changes: 2 additions & 2 deletions CondFormats/PhysicsToolsObjects/test/SiStripDeDx3DBuilder.cc
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/ParameterSet/interface/ParameterSet.h"
#include "CommonTools/ConditionDBWriter/interface/ConditionDBWriter.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
//#include "FWCore/Utilities/interface/FileInPath.h"

#include "CondFormats/PhysicsToolsObjects/interface/Histogram3D.h"

Expand All @@ -24,7 +24,7 @@ class SiStripDeDx3DBuilder : public edm::EDAnalyzer {
virtual void analyze(const edm::Event&, const edm::EventSetup&);

private:
edm::FileInPath fp_;
//edm::FileInPath fp_;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOTE: this was left intentionally commented out as the code that uses fp_ was already commented out.

bool printdebug_;
};

Expand Down
4 changes: 2 additions & 2 deletions CondFormats/PhysicsToolsObjects/test/SiStripDeDxMipBuilder.h
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/ParameterSet/interface/ParameterSet.h"
#include "CommonTools/ConditionDBWriter/interface/ConditionDBWriter.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
//#include "FWCore/Utilities/interface/FileInPath.h"

#include "CondFormats/PhysicsToolsObjects/interface/Histogram2D.h"

Expand All @@ -22,7 +22,7 @@ class SiStripDeDxMipBuilder : public edm::EDAnalyzer {
virtual void analyze(const edm::Event&, const edm::EventSetup&);

private:
edm::FileInPath fp_;
//edm::FileInPath fp_;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOTE: this was left intentionally commented out as the code that uses fp_ was already commented out.

bool printdebug_;
};

Expand Down
5 changes: 1 addition & 4 deletions CondFormats/SiPixelObjects/BuildFile.xml
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,14 @@
<use name="CondFormats/Serialization"/>
<use name="CondFormats/DataRecord"/>
<use name="CondFormats/SiStripObjects"/>
<use name="root"/>
<use name="Geometry/TrackerGeometryBuilder"/>
<use name="CondFormats/External"/>
<use name="DataFormats/Common"/>
<use name="DataFormats/GeometryVector"/>
<use name="FWCore/Framework"/>
<use name="FWCore/MessageLogger"/>
<use name="FWCore/ParameterSet"/>
<use name="Geometry/CommonDetUnit"/>
<use name="Geometry/CommonTopologies"/>
<use name="Geometry/Records"/>
<export>
<lib name="1"/>
</export>

Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
#include "CondFormats/SiPixelObjects/interface/SiPixelCPEGenericErrorParm.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
#include <fstream>

void SiPixelCPEGenericErrorParm::fillCPEGenericErrorParm(double version, std::string file) {
Expand Down
7 changes: 0 additions & 7 deletions CondFormats/SiPixelObjects/src/SiPixelQuality.cc
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,12 @@
#include "CondFormats/SiPixelObjects/interface/SiPixelQuality.h"
#include "CondFormats/SiPixelObjects/interface/SiPixelFrameReverter.h"
#include "CondFormats/SiPixelObjects/interface/SiPixelFrameConverter.h"
#include "DataFormats/Common/interface/Handle.h"
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/Framework/interface/ESWatcher.h"
#include "CondFormats/DataRecord/interface/SiPixelFedCablingMapRcd.h"
#include "CondFormats/SiPixelObjects/interface/SiPixelFedCablingMap.h"
#include "CondFormats/SiPixelObjects/interface/SiPixelFedCablingTree.h"
#include "CondFormats/SiPixelObjects/interface/PixelROC.h"
#include "CondFormats/SiPixelObjects/interface/LocalPixel.h"
#include "Geometry/CommonDetUnit/interface/PixelGeomDetUnit.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "Geometry/TrackerGeometryBuilder/interface/TrackerGeometry.h"

#include <algorithm>
Expand Down Expand Up @@ -146,9 +142,6 @@ const std::vector<LocalPoint> SiPixelQuality::getBadRocPositions(const uint32_t&
if (myroc->idInDetUnit() == i) {
LocalPixel::RocRowCol local = {39, 25}; //corresponding to center of ROC row, col
GlobalPixel global = myroc->toGlobal(LocalPixel(local));
// edm::ESHandle<TrackerGeometry> geom;
// es.get<TrackerDigiGeometryRecord>().get( geom );
// const TrackerGeometry& theTracker(*geom);
const PixelGeomDetUnit* theGeomDet = dynamic_cast<const PixelGeomDetUnit*>(theTracker.idToDet(detid));

PixelTopology const* topology = &(theGeomDet->specificTopology());
Expand Down
1 change: 0 additions & 1 deletion CondFormats/SiPixelTransient/BuildFile.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
<use name="FWCore/MessageLogger"/>
<use name="FWCore/ParameterSet"/>
<use name="FWCore/Utilities"/>
<use name="boost"/>
<export>
Expand Down
2 changes: 1 addition & 1 deletion CondFormats/SiPixelTransient/src/SiPixelGenError.cc
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@

#ifndef SI_PIXEL_TEMPLATE_STANDALONE
#include "CondFormats/SiPixelTransient/interface/SiPixelGenError.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
#define LOGERROR(x) LogError(x)
#define LOGWARNING(x) LogWarning(x)
Expand Down
2 changes: 1 addition & 1 deletion CondFormats/SiPixelTransient/src/SiPixelTemplate.cc
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@
#ifndef SI_PIXEL_TEMPLATE_STANDALONE
#include "CondFormats/SiPixelTransient/interface/SiPixelTemplate.h"
#include "CondFormats/SiPixelTransient/interface/SimplePixel.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
#define LOGERROR(x) LogError(x)
#define LOGINFO(x) LogInfo(x)
Expand Down
2 changes: 1 addition & 1 deletion CondFormats/SiPixelTransient/src/SiPixelTemplate2D.cc
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@

#ifndef SI_PIXEL_TEMPLATE_STANDALONE
#include "CondFormats/SiPixelTransient/interface/SiPixelTemplate2D.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
#define LOGERROR(x) LogError(x)
#define LOGINFO(x) LogInfo(x)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
#include "DataFormats/Math/interface/liblogintpack.h"
#include "DataFormats/Math/interface/libminifloat.h"
#include "DataFormats/PatCandidates/interface/CovarianceParameterization.h"
#include "FWCore/ParameterSet/interface/FileInPath.h"
#include "FWCore/Utilities/interface/FileInPath.h"

uint16_t CompressionElement::pack(float value, float ref) const {
float toCompress = 0;
Expand Down
1 change: 0 additions & 1 deletion EventFilter/CSCRawToDigi/interface/CSCDigiToRaw.h
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
#include "FWCore/ParameterSet/interface/ParameterSet.h"

class FEDRawDataCollection;
class CSCReadoutMappingFromFile;
class CSCChamberMap;

class CSCDigiToRaw {
Expand Down