nuisance is hosted by Hepforge, IPPP Durham
close Warning: Can't synchronize with repository "(default)" ("(default)" is not readable or not a Git repository.). Look in the Trac log for more information.

Opened 8 years ago

Closed 7 years ago

#27 closed defect (fixed)

Sort Covariance/Data Reads

Reported by: Patrick Stowell Owned by: somebody
Priority: major Milestone: Tuning paper, vxry
Component: Cleaning Version:
Keywords: Cc:

Description

There are lots of different functions inside the Measurement1D/2D classes to read in data histograms and covariances, and its a bit confusing which one should be used.

These should be moved to a Utils namespace, so that people always have to initialise the covariance inside their constructor.
Covar = StatUtils::LoadCovarance?(file);
or
Correl = StatUtils::LoadCorrelation?(file);
Covar = StatUtils::MakeCovarFromCorrel?(Correl, fDataHist);

Provided Covariance is setup the SetupDefaultHist? function can always handle filling Decomp and Correl.

Change History (4)

comment:1 Changed 8 years ago by Patrick Stowell

Milestone: milestone1 → milestone2

comment:2 Changed 8 years ago by Clarence Wret

Component: component1 → Cleaning

comment:3 Changed 8 years ago by Clarence Wret

Milestone: milestone2 → Tuning paper, vxry

Milestone renamed

comment:4 Changed 7 years ago by Patrick Stowell

Resolution: → fixed
Status: new → closed

Cleaner functions have been added at the top of Measurement* to address this. The old ones will be phased out.

Note: See TracTickets for help on using tickets.