[root] Log of /trunk/roofit/roostats/src/HypoTestInverterResult.cxx

Revision 48842 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Mar 7 10:59:49 2013 UTC (22 months, 2 weeks ago) by moneta
File length: 39790 byte(s)
Diff to previous 44366

fix a bug in computing the upper limit when the range of plot was different than range of variables. 
See http://root.cern.ch/phpBB3/viewtopic.php?f=15&t=16164

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: 39734 byte(s)
Diff to previous 44171

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

Revision 44171 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue May 8 10:19:13 2012 UTC (2 years, 8 months ago) by moneta
File length: 39713 byte(s)
Diff to previous 44156

fixes for Coverity and a bug in finding interpolated limit

Revision 44156 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon May 7 16:37:38 2012 UTC (2 years, 8 months ago) by moneta
File length: 39255 byte(s)
Diff to previous 44145

- AsymptoticCalculator:: do a fix for bins where pdf is zero when generating Asimov data sets
- HypoTestInverterResult: another fix for lower/upper limit search

Revision 44145 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Sun May 6 22:02:39 2012 UTC (2 years, 8 months ago) by moneta
File length: 36530 byte(s)
Diff to previous 44133

fix a bug in doing interpolation for upper/lower limits when there the points are not crossing the line

Revision 44133 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Sat May 5 13:48:49 2012 UTC (2 years, 8 months ago) by moneta
File length: 35879 byte(s)
Diff to previous 44130

fix a compilation warning

Revision 44130 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Sat May 5 10:08:31 2012 UTC (2 years, 8 months ago) by moneta
File length: 35811 byte(s)
Diff to previous 44126

fixes for Coverity and improve upper/lowerlimit search in InverterResult

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: 35948 byte(s)
Diff to previous 43992

- 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 43992 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Apr 27 16:11:41 2012 UTC (2 years, 8 months ago) by moneta
File length: 31293 byte(s)
Diff to previous 43231

fix ~ 15 high impact coverity issues (Resource leaks in NumberCountingPdfFactory and Uninitilized variables  in other classes)

Revision 43231 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Mar 2 16:38:19 2012 UTC (2 years, 10 months ago) by moneta
File length: 31170 byte(s)
Diff to previous 43199

remove a log message

Revision 43199 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Mar 1 20:17:42 2012 UTC (2 years, 10 months ago) by moneta
File length: 31166 byte(s)
Diff to previous 41810

merge from the roostats development branch the new developments from Sven Kreiss:

 new ToyMCImportanceSampler 
 new  version of the ToyMCSampler that can use multiple test statistics and "detailed output" of those test statistics.
 extension to the FrequentistCalculator that allows the user to manually specify the values for the signal nuisance parameters using SetConditionalMLEsNull/Alt().
 
 make  HypoTestResult managing now the  SamplingDIstribution for null and alt

Revision 41810 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Nov 8 14:16:59 2011 UTC (3 years, 2 months ago) by moneta
File length: 31037 byte(s)
Diff to previous 41725

- fix a problem in computing expected limits when number of toys was different at every point.
 (add also two option to compute the expected limits)

- fix missing generation for RooStats::MakeNuisancePdf function

Revision 41725 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Nov 2 16:12:07 2011 UTC (3 years, 2 months ago) by moneta
File length: 28727 byte(s)
Diff to previous 41684

add fixes suggested by Tim Adye: 
 - fix Add when  this is empty
 - fix FindIndex
 - add a new method Add passing just an HypoTestResult

Revision 41684 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Nov 1 16:27:08 2011 UTC (3 years, 2 months ago) by wouter
File length: 28191 byte(s)
Diff to previous 41455

  roofit/histfactory

    - Import changes from dev-roostats branch

  roofit/roostats
 
    - Import update on HypoTestInverterResult and HypoTestInverterPlot
      from dev branch (only fixes for compiler warnings)

  roofit/roofit

    - Import new classes Roo1DMomentMorphFunction and Roo2DMomentMorphFunction
      from dev-roostats branch (contribution by Max Baak)

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: 28185 byte(s)
Diff to previous 41068

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 41068 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Sep 30 08:25:53 2011 UTC (3 years, 3 months ago) by moneta
File length: 26323 byte(s)
Diff to previous 40959

fix computation of expected limit in GetExpectedLimit() 
 in case of CLs use the full limit distribution instead of getting quantiles from the p-values at each tested point

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: 25626 byte(s)
Diff to previous 40294

- 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 40294 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Jul 19 10:41:32 2011 UTC (3 years, 6 months ago) by moneta
File length: 25411 byte(s)
Diff to previous 40132

fix a bug in computation of interpolated limit. For some conditions an extrapolation was done instead of interpolating

Revision 40132 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Jul 4 21:20:11 2011 UTC (3 years, 6 months ago) by moneta
File length: 25172 byte(s)
Diff to previous 40095

fix a bug in Add reported by Tim Adye

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: 25173 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: 23456 byte(s)
Diff to previous 40006

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 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: 23160 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: 20282 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: 19401 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: 16874 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: 11007 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: 7819 byte(s)
Diff to previous 36230

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 36230 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Sat Oct 9 20:21:02 2010 UTC (4 years, 3 months ago) by wouter
File length: 7406 byte(s)
Diff to previous 34109

  And yet more Coverity fixes

Revision 34109 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Jun 24 15:00:16 2010 UTC (4 years, 7 months ago) by moneta
File length: 7399 byte(s)
Diff to previous 31798

merge roostats development branch from revision 33228 to 34108 included

Revision 31798 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Dec 10 14:57:15 2009 UTC (5 years, 1 month ago) by moneta
File length: 7397 byte(s)
Diff to previous 31742

merge latest changes in https://root.cern.ch/svn/root/branches/dev/roostats/roofit/roostats 
from  r31742 through r31794

Revision 31742 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Dec 9 17:34:15 2009 UTC (5 years, 1 month ago) by moneta
File length: 7371 byte(s)
Diff to previous 31741

fix warning coming and an error from previous commit

Revision 31741 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Dec 9 17:27:53 2009 UTC (5 years, 1 month ago) by moneta
File length: 7096 byte(s)
Diff to previous 31322

import changes from roostats dev branch r31494 through r31740  of 
https://root.cern.ch/svn/root/branches/dev/roostats/roofit/roostats
made by Gregory

Revision 31322 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 19 16:47:43 2009 UTC (5 years, 2 months ago) by moneta
File length: 6675 byte(s)
Diff to previous 31297

merge latest changes from Gregory in https://root.cern.ch/svn/root/branches/dev/roostats

URL: http://root.cern.ch/viewcvs?rev=31318&root=root&view=rev
Log:
bring consistency with UpperLimit - set upper bound instead of limit and compute both lower and upper bound

Revision 31297 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Nov 18 20:44:22 2009 UTC (5 years, 2 months ago) by moneta
File length: 5656 byte(s)
Diff to previous 31285

merge in the trunk the latest fix from Gregory in the HypoTestInverter:
 http://root.cern.ch/viewcvs?rev=31291&root=root&view=rev
LOG:
fixed UL search (find the object with the smallest error that is < 1 sigma from the target)

Revision 31285 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Nov 18 16:38:36 2009 UTC (5 years, 2 months ago) by moneta
File length: 5010 byte(s)
Diff to previous 31276

merge latest changes in https://root.cern.ch/svn/root/branches/dev/roostats from 31273 to 31284

Revision 31276 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Nov 18 15:06:42 2009 UTC (5 years, 2 months ago) by moneta
File length: 4617 byte(s)
Copied from: branches/dev/roostats/roofit/roostats/src/HypoTestInverterResult.cxx revision 31268
Diff to previous 31178

merge developments in https://root.cern.ch/svn/root/branches/dev/roostats/roofit/roostats from revision 30833 to revision 31273 


fix also the tutorials due to the new changes (copy from branch)

Revision 31178 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Nov 16 09:07:03 2009 UTC (5 years, 2 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterResult.cxx
File length: 4670 byte(s)
Diff to previous 31167

- avoid deleting elements in TList if TLisyt is set to own the elements

Revision 31167 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Nov 13 18:11:15 2009 UTC (5 years, 2 months ago) by schott
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterResult.cxx
File length: 4673 byte(s)
Diff to previous 31131

added estimated error on the upper limit

Revision 31131 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 12 15:16:14 2009 UTC (5 years, 2 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterResult.cxx
File length: 3164 byte(s)
Diff to previous 31085

- use consistent signature for constructing result classes
  have a default constructor and one passing the name + parameters
  and forget about the title

Revision 31085 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed Nov 11 13:26:24 2009 UTC (5 years, 2 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterResult.cxx
File length: 3216 byte(s)
Diff to previous 31074

- add more comments in interval classes and do some cleanup: 
  - make IsInInterval a const method
  - pass when possible reference instead of pointer and use RooArgSet by value

Revision 31074 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Nov 10 18:46:30 2009 UTC (5 years, 2 months ago) by schott
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterResult.cxx
File length: 3173 byte(s)
Diff to previous 31051

add error on HypoTestInverterPlot

Revision 31051 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Nov 9 15:54:02 2009 UTC (5 years, 2 months ago) by schott
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInverterResult.cxx
File length: 2931 byte(s)
Copied from: branches/dev/roostats/roofit/roostats/src/HypoTestInvertorResult.cxx revision 31049
Diff to previous 31048

finished renaming HypoTestIntertor->HypoTestInverter

Revision 31048 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Nov 9 15:37:27 2009 UTC (5 years, 2 months ago) by schott
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInvertorResult.cxx
File length: 2931 byte(s)
Diff to previous 30655

renamed HypoTestInvertor* to HypoTestInverter*

Revision 30484 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Sat Sep 26 10:17:28 2009 UTC (5 years, 3 months ago) by moneta
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInvertorResult.cxx
File length: 2847 byte(s)
Diff to previous 30479

code cleanup

Revision 30479 - (view) (download) (as text) (annotate) - [select for diffs]
Added Fri Sep 25 19:42:59 2009 UTC (5 years, 3 months ago) by schott
Original Path: branches/dev/roostats/roofit/roostats/src/HypoTestInvertorResult.cxx
File length: 2203 byte(s)

new HypoTestInvertor class

This form allows you to request diffs between any two revisions of this file. For each of the two "sides" of the diff, enter a numeric revision.

Links to HEAD:	(view) (download) (as text) (annotate)
Sticky Revision:

	Diffs between and
	Type of Diff should be a