---

Todo List

page package CommonWithDsp

page package SCTPixelDAQ

Remove delay loop in writeBlockToFlashHpi when timing is understood

Get true block VME operations working in mdspBlockRead/Write.

Remove C-style casts - use polymorphic calls to VmeInterface

Have status() return an XML file

Test text buffer wrap-around

Test multiblock VME transfers in mdspBlockRead and mdspBlockWrite

page package RodUtils

page package VmeInterface

Global SctAnalysis::AnalysisService::ipcAnalyzeModule (AnalysisServiceIStatus *status, char *testname, char *modulename)

This seems broken to me.

This seems broken to me.

Global SctAnalysis::AnalysisWorkerGroup::work (const std::string)

put description of work() here.

Global SctData::CalChargeVariable::setChipActualPoint (double actualPt, ModuleConfiguration &config, unsigned int chip) const

Check parameters here?

Global SctData::ChipNOResult::getNoise () const

More checks.

Global SctCalibrationController::ConfigUpdater::getMID (const SctData::TestResult &t) const

remove the nasty const_cast below when proper CORBA implementation arrives.

Global SctData::ConfigurationVariableIOHelper::getFromTypeRep (unsigned short typeRep)

Think: Would this be more intuitive if it was done explicitly??

Global SctData::IO::ConfigurationVariableStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

throw Sct::LogicError("Unsupported operation in ConfigurationVariableIS", MRS_ERROR, "libSctDataIS");

Class SctData::DefaultVariable

Think: Is this necessary? Does it imply public virtual inheritance of ConfigurationVariable ...are there any problems?

Global SctData::IO::ErfcFitObjectStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

: Needed to get round gcc295 bug

Global SctData::IO::ErfFitObjectStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

: Needed to get round gcc295 bug

Class SctFitter::FitAlgorithmMap

Add an iterator for this class

Make this thread-safe

Global SctFitter::FitAlgorithmMap::setAlgorithm (string variableName, auto_ptr< FitAlgorithm >)

Think: Should this return a ptr to the previous alg?

Global SctData::IO::FitObjectStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

Think about this- should we set vars that are already initilized?? For now, yes

Global SctData::IO::FitScanResultStreamer::write (OStream &out, const Streamable &ob, const IOManager &manager) const

modify when bug is fixed in IS

Global SctFitter::Fitter::go (unsigned nWorker)

proper RegExp

Global SctData::IO::GausFitObjectStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

: Needed to get round gcc295 bug

Class Sct::IsException

Add meaningful information here.

Global SctFitter::MinuitFitStrategy::fitTGraph (const TGraph &graph, TF1 &fit) const

implement

Global SctFitter::MinuitFitStrategy::fitTGraphErrors (const TGraphErrors &graph, TF1 &fit) const

implement

Global SctFitter::MinuitFitStrategy::fitTH1 (const TH1 &hist, TF1 &fit) const

implement

Class SctData::ModuleDefect

Is this enough? Do we need to add more classes here? Should we be able to add other ModuleDefect s anywhere in our code or is having a central repository here enough? Should this class be renamed ChannelDefect?

Global SctData::ModuleDefect::DEAD

add all the different types of defect and check the descriptions

Global SctData::ModuleDefect::ModuleDefect (int id, bool severe, const string &name, const char *d, double parameter)

rethink this

Global SctData::ModuleDefect::isPrototype () const

is this necessary?

Global SctAnalysis::NoiseOccupancyAlgorithm::analyzeChip (const unsigned ichip, const SctData::OccupancyProjector &, const SctData::ChipConfiguration &, SctData::ModuleDefectList &defects)

modify for hybrids

Global SctAnalysis::NoiseOccupancyAlgorithm::canAddFitScanResult (const string &name)

document

Global SctAnalysis::NoiseOccupancyAlgorithm::canAddRawScanResult (const string &name)

document

Global SctData::RawScanResultIOHelper::set (const RawScanResult &ob)

Const cast for no great reason - this all needs to be looked at some more

Global SctCalibrationController::RunControl::enterRunning ()

Check status

Global SctData::IO::ScanPointsStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

Exception here

Exception here

Exception here

Global SCTAPI::publishRawDataObject (unsigned int runNumber, unsigned int scanNumber, const ConfigurationVariable &scan_variable, ModuleConfiguration *moduleconfiguration=0)

fix nasty kludge for FullBypass scan

Global Sct::Streamable::getClassName () const=0

We could use typeid instead here, but that would probably mean removing the namespace stuff.

Global SctData::StrobeDelayVariable::getActualPoint (double logicalPt) const

: Fix this!!! To get ns from DAC ???

Global SctData::StrobeDelayVariable::setChipActualPoint (double actualPt, ModuleConfiguration &config, unsigned int chip) const

Check parameters here?

Class SctData::TestResult

Think: Do we want a TestHeader class? Such that a Test is a collection of TestHeaders

Global SctData::IO::TGraphAsymmErrorsStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

sort this out - unscalable casts needed for RootStreamableAdapter

Global SctData::IO::TGraphAsymmErrorsStreamer::write (OStream &out, const Streamable &ob, const IOManager &manager) const

sort this out - unscalable casts needed for RootStreamableAdapter

Global SctData::ThresholdVariable::setChipActualPoint (double actualPt, ModuleConfiguration &config, unsigned int chip) const

Check parameters here?

Global SctData::IO::TopHatFitObjectStreamer::read (IStream &in, Streamable &ob, const IOManager &manager) const

: Needed to get round gcc295 bug

Global SctAnalysis::TrimRangeAlgorithm::canAddFitScanResult (const string &name)

document algorithm.

Namespace SctData

DefaultVariable used

Namespace SctData

check that ST_COMPRESSION is really the right variable!

Namespace SctData

check that ST_COMPRESSION is really the right variable!

Namespace SctService

Exception specifications

const correctness

---