Log of /trunk/roofit/roostats/src/HypoTestInverter.cxx

---

Revision 44366 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed May 30 15:32:27 2012 UTC (2 years, 7 months ago) by axel
File length: 47716 byte(s)
Diff to previous 44126

Remove #include "Riostream.h" from roofitcore/inc/RooCacheManager.h, which pulls in using namespace std.
Deal with the consequences.

---

Revision 44126 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri May 4 16:55:47 2012 UTC (2 years, 8 months ago) by moneta
File length: 47695 byte(s)
Diff to previous 44096

- fix in AsymptoticCalculator the formulae for the two-sided test statistics for both observed and expected p values
- improve search for upper/lower limit in InverterResult
  add also a flag to indicate in the InverterResult if it comes from using a two sided test statistic.  
 This is needed to use the right formula for the asymptotic case. 

- Add also possibility to skip and to not draw the points where there is a failing fit or any other error

- Add in Profile likelihood possibility to query if the test statistic is one or two sided

---

Revision 44096 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu May 3 15:37:10 2012 UTC (2 years, 8 months ago) by moneta
File length: 47082 byte(s)
Diff to previous 44095

fix compilation error from previous commit

---

Revision 44095 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu May 3 15:27:39 2012 UTC (2 years, 8 months ago) by moneta
File length: 47083 byte(s)
Diff to previous 44038

fixes for Coverity (division by zero, 42512,3)

---

Revision 44038 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Apr 30 15:21:37 2012 UTC (2 years, 8 months ago) by moneta
File length: 46601 byte(s)
Diff to previous 44037

fix a bug in constructor taking two models

---

Revision 44037 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Apr 30 15:18:34 2012 UTC (2 years, 8 months ago) by moneta
File length: 46601 byte(s)
Diff to previous 44034

from Sven: add a fixed scan in log scale. 
It is enables by setting the option in RunFixedScan or SetFixedScan

---

Revision 44034 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Apr 30 14:54:15 2012 UTC (2 years, 8 months ago) by moneta
File length: 46303 byte(s)
Diff to previous 42591

Verious fixes for outstanding Coverity issues (mainly add check of returned pointers to avoid  to dereference them)

---

Revision 42591 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Jan 11 10:16:55 2012 UTC (3 years ago) by moneta
File length: 45645 byte(s)
Diff to previous 41455

fix a bug reported by Cristoph Naegeli in the auto-scan

---

Revision 41455 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Oct 18 13:23:21 2011 UTC (3 years, 3 months ago) by moneta
File length: 45670 byte(s)
Diff to previous 40959

merge from development branch r41278 through r41454 
 - add new AsymptoticCalculator and its support in HypoTestInverter
 - add utility function to make the nuisance pdf
   + various small improvements

---

Revision 40959 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Sep 20 16:33:21 2011 UTC (3 years, 4 months ago) by moneta
File length: 44208 byte(s)
Diff to previous 40293

- add in MCMCCalculator SetChainParameters to define the parameters to be put in the chain

- fix a bug  of computing  p values in SamplingDistribution introduced in 40920

- ToyMCSampler - clear the cached GenContext and pdf lists  when setting a new PDF

- add a unique name for teh SamplingDist objects created in the Inverter
- fix also the shading in the plots of SamplingDist objects and the plotting in  the log scale

---

Revision 40293 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Jul 19 08:50:28 2011 UTC (3 years, 6 months ago) by moneta
File length: 43653 byte(s)
Diff to previous 40292

fix an error message

---

Revision 40292 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Jul 19 08:38:08 2011 UTC (3 years, 6 months ago) by moneta
File length: 43492 byte(s)
Diff to previous 40252

fix for bug https://savannah.cern.ch/bugs/?84542

---

Revision 40252 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Jul 14 21:51:58 2011 UTC (3 years, 6 months ago) by moneta
File length: 43492 byte(s)
Diff to previous 40095

HypoTestInverter: fix run of fixed scan if min max are outside variable bounds
HypoTestResult: return -1 instead of infinity (which crashes the plots) for Cls error when Clb=0
(CLs is = -1 in that case)

LikelihoodTestStats:  add option ReuseNLL on by default

---

Revision 40095 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Jul 1 10:04:25 2011 UTC (3 years, 6 months ago) by moneta
File length: 42664 byte(s)
Diff to previous 40092

- fix bug in scanning introduced in 40092
- improve in the Result the treatment when only one point is present and better diagnostic and comments in Add method

---

Revision 40092 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Jul 1 07:45:56 2011 UTC (3 years, 6 months ago) by moneta
File length: 42631 byte(s)
Diff to previous 40056

apply some bug fixes reported by Tim Adye in the HypoTestInverter:

 - fix  RunFixedScan to work with only one bin
 - return result when used RunOnePoint()

HypoTestInverteResult: 

 - fix a problem  in streaming by storing in the class a copy of the scanned variable, since it is not needed 
to store all the linked objects (they are stored anyway in the workspace) 
 The original RoORealVar  has links to the test statistics and when used an optimized test statistics the streaming 
failed

---

Revision 40056 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Jun 28 07:36:22 2011 UTC (3 years, 6 months ago) by moneta
File length: 42571 byte(s)
Diff to previous 40049

fix some Coverity reports

---

Revision 40049 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Jun 27 20:52:57 2011 UTC (3 years, 6 months ago) by moneta
File length: 42503 byte(s)
Diff to previous 40006

add possibility to close proof for every run to avoid massive mem leaks

---

Revision 40006 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Jun 27 15:29:22 2011 UTC (3 years, 6 months ago) by moneta
File length: 42249 byte(s)
Diff to previous 39975

apply some memory leak  fixes in the inverter and in the plot class 
make HTIResult::Add public

---

Revision 39975 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Sun Jun 26 22:49:46 2011 UTC (3 years, 6 months ago) by moneta
File length: 41035 byte(s)
Diff to previous 39928

merge from roostats dev branch : 

 - from Sven: new version of ToyMCSampler
 - from Kyle: update tutorial StandardTestStatDistributionDemo
 -from LM : new version of Inverter with rebuilding pValues distributions using toys
            + update comments

---

Revision 39928 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Jun 24 08:18:59 2011 UTC (3 years, 7 months ago) by moneta
File length: 33342 byte(s)
Diff to previous 39865

merge latest fixes in development branch of roostats in the trunk from revision r39839 through r39927

 see http://root.cern.ch/viewvc/branches/dev/roostats/roofit/?view=log

---

Revision 39865 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Jun 21 20:58:17 2011 UTC (3 years, 7 months ago) by moneta
File length: 30802 byte(s)
Diff to previous 39719

merge roostats and histfactory from development branch up to revision 39830

 - improvements in HypoTestInverter (better interpolation + plot with bands)  + Coverity fixes 
 -  new version of HistFactory from Kyle adding FlexibleInterpvar

---

Revision 39719 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Jun 14 15:17:29 2011 UTC (3 years, 7 months ago) by moneta
File length: 27720 byte(s)
Diff to previous 39513

merge from roostats development branch (from revision r39531 through r39718 )

---

Revision 39513 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue May 31 17:49:43 2011 UTC (3 years, 7 months ago) by moneta
File length: 23581 byte(s)
Copied from: branches/dev/roostats/roofit/roostats/src/HypoTestInverter.cxx revision 39512
Diff to previous 39511

merge from the trunk development in dev roostats branch from 39465 up to current revision
- new FrequentistCalculator class from Sven 
 - new HypoTestInverter class and rename old one HypoTestInverterOriginal 
- some fixes in ProofConfig from Sven

---

Revision 39511 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue May 31 17:33:27 2011 UTC (3 years, 7 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverter.cxx
File length: 23581 byte(s)
Copied from: branches/dev/roostats/roofit/roostats/src/HypoTestInverterNew.cxx revision 39509
Diff to previous 39509

rename HypoTestInverterNew HypoTestInverter

---

Revision 39509 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue May 31 17:21:33 2011 UTC (3 years, 7 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterNew.cxx
File length: 23675 byte(s)
Diff to previous 39505

make some clean up of the class
- add separate constructor for hybrid or frequentist calculators

---

Revision 39505 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue May 31 14:14:34 2011 UTC (3 years, 7 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterNew.cxx
File length: 33405 byte(s)
Diff to previous 39498

new version of Inverter which does the automatic scan using the code from Giovanni
 - add also some improvements in the Result classes

- now also GetInterval run automatically the scan for the limit

---

Revision 39498 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue May 31 08:47:47 2011 UTC (3 years, 7 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterNew.cxx
File length: 22155 byte(s)
Diff to previous 39494

fix a problem in HypoTestInverterResult. Downcast to the righst class

---

Revision 39494 - (view) (download) (as text) (annotate) - [select for diffs]
Added Mon May 30 21:15:17 2011 UTC (3 years, 7 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterNew.cxx
File length: 21652 byte(s)

add new version (from Annapaola) of HypoTestInverter using the new Hybrid or FrequentistCalculator