[root] / trunk / roofit / roofitcore / inc / RooVectorDataStore.h Repository:
ViewVC logotype

Log of /trunk/roofit/roofitcore/inc/RooVectorDataStore.h

Parent Directory Parent Directory


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

Revision 46464 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Oct 11 14:00:00 2012 UTC (2 years, 3 months ago) by wouter
File length: 20422 byte(s)
Diff to previous 46156
  *** Assorted bug fixes (Savannah and otherwise) ***

  o RooDataHist

     - When importing TH1s omit 'average bin density' correction factor as that
       proves to be very uninuitive. (Savannah #96153)


  o RooGlobalFunc

     - Set importDensity by default to kFALSE in Import(TH1&) following
       new convention (Savannah #96153)


  o RooGenFitStudy

     - Save errors of fitted parameters in summary data


  o RooStreamParser

     - Fix parsing issue for variables starting with an 'i' (Savannah #82458)


  o RooPlot

     - Add static interface to control directory auto-attach to allow importing in workspaces
       (Savannah #94239)


  o RooRealBinding

     - In saveXvec()/restoreXvec() also save and restore all buffers of components of
       the bound function to make sure it works fine if components are cached in
       in the likelihood and operate in clean mode.


  o RooVectorDataStore

     - In cacheArgs() and recalculateCache() skip calculation of elements where the
       event weight is zero

     - In the streamer explicit convert stl::vector pointers to an empty collection
       to null pointers to make vector stores with uncertainty work fine after
       persistence (ROOT persistence turns null pointers in to pointers to null
       collections as part of the read-back process) (Savannah #94908) 


  o RooRealSumPdf

     - Elaborate one of the debug print statements


  o RooProduct

     - Change internal storage from set to list and change constructor input argument 
       from set to list to support products of identical terms (Savannah #94925)


  o RooAddition

     - Change constructor input argument from set to list to support products
       of identical terms (Savannah #94925)


  o RooAbsPdf

     - In fitTo() only invoke SUMW2 afterburner if there are >0 free parameters
       (Savannah #92332)


  o RooWorkspace

     - In generic object import() disable auto-attach of RooPlots to gDirectory
       (Savannah #94239)


  o RooListProxy

     - In add() check if _owner is set before calling addServer on it (allows
       operation in schema evolution) (Needed for Savannah #94925)


  o RooFormulaVar

     - Fix documentation (Savannah #93498)


  o LinkDef2

     - Add schema evolution for RooProduct and RooAddition (for Savannah #94925)

Revision 46156 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Sep 25 08:40:08 2012 UTC (2 years, 3 months ago) by wouter
File length: 20421 byte(s)
Diff to previous 44507
 *** Speedup of caching of parameterized integrals ***

  o RooAbsCachedReal

    - Add interface to activate option of RooAbsCachedReal to keep clone of source object in cache for faster operation
      (the downside of this is that future structural changes of the source object will not be detected)

  o RooCachedReal

    - Implement source cloning and persistence (based on code originally in RooAbsReal::fillDataHist)


  o RooAbsPdf

    - in syncNormalization() activate source caching feature for parameterized normalization integrals
 




 *** Improve robustness of level-2 likelihood optimization 'cache-and-track' ***

  o RooVectorDataStore

    - in cachArgs() detect cache-and-track object labeled with a CATCondSet or CATNormSet string label
      and reconstitute the corresponding set of conditional observables / normalization observables
      from the RooNameSet transcript in the label. Fill initial content of cache-and-track elements
      with these labels with the appropriate normalization set and associate the set with the RealVector
      element that is used for future cache updates.

    - In recalculateCache() pick up custom normalization set for conditional product terms as specified
      in cacheArgs


  o RooAbsOptTestStatistic

    - in optimizeConstantTerms() check for non-default normalization specifications of RooProdPdf components
      and if found, transcribe the specification to the component object in a CATCondSet or CATNormSet label
      so that the cache-and-track optimizer can evaluate these according to the RooProdPdf specs.


  o RooProdPdf

    - Make findPdfNSet() public so that RooAbsOptTestStatistic can use it.

 
  o RooNameSet

    - Add content() method as accessor to the payload.





 *** Miscellaneous ***

  o RooAbsGenContext

    - Fix compiler warning


  o RooDataSet

    - In ctor import weight variable of imported dataset if no weight variable is specified
      (fixes Savannah #95641)

Revision 44507 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Mon Jun 4 12:30:41 2012 UTC (2 years, 7 months ago) by axel
File length: 20264 byte(s)
Diff to previous 44077
Remove
  using namespace std;
from Riostream.h, which has huge consequences for all of ROOT.
Riostream.h is now a simple wrapper for fstream, iostream, iomanip for backward compatibility; Riosfwd.h simply wraps iosfwd.

Because of templates and their inline functions, Riostream.h needed to be included in headers, too (e.g. TParameter.h), which violated the assumption that Riostream.h is not exposing its using namespace std to headers.
ROOT now requires R__ANSISTREAM, R__SSTREAM, which does not change the set of supported compilers.

Without "using namespace std", several identifiers are now prefixed by std::; e.g. roofit/* source files now have a using namespace std to keep their coding style.
TFile::MakeProject() now generates "using namespace std" to convert the CINT-style class names into C++ ones.

Revision 44077 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Wed May 2 17:10:21 2012 UTC (2 years, 8 months ago) by moneta
File length: 20244 byte(s)
Diff to previous 42950
remove "using namespace std" from the header files

 see https://savannah.cern.ch/bugs/?86347 and 
https://savannah.cern.ch/bugs/?94227

Revision 42950 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Feb 9 21:21:53 2012 UTC (2 years, 11 months ago) by wouter
File length: 20192 byte(s)
Diff to previous 42864
   Assorted bug fixes

   o RooChi2Var, RooXYChi2Var, RooDataWeightedAverage, RooNLLVar

     - Deploy upgraded interface of recalculateCache()


   o RooAbsDataStore

      - Extend interface for recalculateCache() to take needed
        info for smart recalculation when using NCPU>1 and to
        pass needed info for proper handling of normalization
        observables


   o RooCompositeDataStore

      - Forward recalculateCache() to components (should not
        be needed but just in case)


   o RooVectorDataStore

     - Treat conditional observables properly in the normalization
       set passed to cached pdfs

     - Only update elements that are actually being recalculated
       when using NumCPU>1


   o RooAbsAnaConv

     - Persist _model so that generation works after persistence

 
   o RooAbsReal

     - Disable cache-and-track when making plot projections
       with DataWeightedAverage, as this cannto work because
       the needed information for normalization is not available

     - Reinstate cloning of the function in fillDataHist(). Without
       it is is possible to trigger as accidental 'self-destruct'
       of cached normalization objects that call fillDataHist()


   o RooCachedReal

     - Make evaluate() return _func instead of null to make
       disableCache mode of base class functional

     - Initialize data member _useCdfBoundaries in all ctors

Revision 42864 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Feb 3 14:05:24 2012 UTC (2 years, 11 months ago) by wouter
File length: 20143 byte(s)
Diff to previous 41885

 == Fix bug in handling of cached conditional pdfs ==

  o RooVectorDataStore

    - Add argument to recalculateCache() that indicates conditional observables
      When passing normalization set to cached nodes removed conditional observables

  o RooNLLVar

    - Pass conditional observables to RooAbsDataStore::recalculateCache() 

  o RooAbsDataStore

    - Add argument to method recalculateCache()

Revision 41885 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 10 17:13:30 2011 UTC (3 years, 2 months ago) by wouter
File length: 20116 byte(s)
Diff to previous 41862
   o RooVectorDataStore

     - In itialize all pointers to zero in RealVector ctor

Revision 41862 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 10 09:44:32 2011 UTC (3 years, 2 months ago) by wouter
File length: 20038 byte(s)
Diff to previous 41768
   o PiecewiseInterpolation, RooAbsCachedReal, RooAbsCachedPdf

     - Follow adjustment in RooCacheManager


   o RooConvCoefVar, RooEffProd, RooResolutionModel, RooErrorVar, RooProjectedPdf, RooAbsHiddenReal

     - Migrate to getValV


   o RooProdPdf

     - Follow adjustment in RooCacheManager     
     - Migrate to getValV


   o RooVectorDataStore

     - Various small fixes

     - Allow storage of non-constant 'track-and-change' cache entries.
       These are cached PDF nodes that are not constant, but need
       to be recalculated when selected parameters change. A 
       RooChangeTracker data member is added to the vectors to
       do this dependency checking, and recalculation (as needed)
       is performed in method recalculateCache() to be called from
       the test stat before looping of the data.
       

   o RooAbsReal

     - Introduce new non-virtual inline getVal() methods that handles
       clean-cache state cases and delegates all other cases to a 
       virtual method getValV() that corresponds to the original
       getVal() method


     - Add 'ignore' more to eval error logging protocal


   o RooAddPdf

     - Follow adjustment in RooCacheManager 
     - Optimize control structure in evaluate() to be more efficient
       (move conditional outside loop)

   o RooAbsTestStatistic

     - Add extra argument to constOptimizeTestStatistic to control
       activation of track-and-change optimization

   o RooRealMPFE

     - Migrate to getValV()

     - Add support for new track-and-change option argument in 
       transmission of const optimization request to servers
       

   o RooRealVar

     - Exploit new _fast flag of RooAbsReal to choose non-virtual
       handling of getVal()

     - Migrate to getValV() [ but still dummy ]

     - Small fix in attachToVStore()


   o RooNLLVar

      - Add data member tracking first use

      - After first use, trigger wiring of all caches in RooCacheManagers
        to speed up trivial cache selection decisions (i.e those where
        only a single cache entry exists)

      - Add call to RooAbsDataStore::recalculateCache() to implement
        track-and-change update prior to likelihood loop


   o RooConstVar

      - Migrate to getValV()


   o RooAbsArg

     - Eliminate flipACLean() protocol since no longer used

     - Simplify method opermode() now that flipAClean is abandoned.

     - In findConstantNodes() add support for change tracking
       and include nodes with "ChangeTracking" atttribute in
       list of constants. Mark all truly constant expressions
       with attribute "ConstantExpression"

     - Add extra argument to constOptimizeTestStatistic to control
       activation of track-and-change optimization  

     - In printCompactTree() also print valueDirty state when in Auto mode


     - Add method wireAllCaches() which propagates cache wiring request
       to all registered cache nodes on all branches nodes

     - Add transient data member with namereg pointer to string with
       current object name. Overload SetName() and SetName() and title
       to clear pointer whenever name changes.


   o RooAbsPdf

     - Migrate to getValV(). Eliminate handling of clean-cache cases
       as this is now done in RooAbsReal::getVal()

     - Disable use of RooAbsPdf::raiseEvalError() protocol as this
       is now communicated through RooAbsReal::logEvalError()


   o RooRealIntegral

     - Migrate to getValV()

     - Use setInhibitDirty() instead of flipAClean() in evaluate()
       when performing numeric integrations


   o RooTreeDataStore

     - In cachArgs() add code to select only cache items with
       label "ConstantExpression" as tree store is unable to
       cache track-and-change items

   o RooAbsCategory

     - When printing also show index associated to label


   o RooAbsOptTestStatistic

     - Add extra argument to constOptimizeTestStatistic to control
       activation of track-and-change optimization

     - In optimizeConstantTerms() when trackChanges option is active
       request change tracking for all component nodes of RooAddPdf
       and RooRealSumPdf, unless these components are RooProdPdf/RooProducts
       respectively in which case the components of these products are tracked
       
   o RooRealProxy

     - Use _fast attribute of RooAbsArg to directly return value cache
       of RooAbsReals if that is allowed


   o RooAbsCache

     - Add virtual method wireCache()


   o RooCacheManager

     - Add new method getObj() that takes isetRangeName as 'const char*'
       instead of 'const TNamed *' to be able to eliminate the conversion
       when in wired mode

     - Implement cache wiring - If at moment of the request only a single
       cached entry exists - always return that one from here one.


   o RooAbsDataStore

     - Implement virtual method calculateCache()

Revision 41768 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 3 17:56:00 2011 UTC (3 years, 2 months ago) by pcanal
File length: 19044 byte(s)
Diff to previous 41759
Undo 41639/41759 which was superseeded by 41754

Revision 41759 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 3 16:54:34 2011 UTC (3 years, 2 months ago) by pcanal
File length: 19125 byte(s)
Diff to previous 41755
Reapply revision 41639 from the trunk:
Fix compilation on gcc 3.4.3

Revision 41755 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 3 16:33:56 2011 UTC (3 years, 2 months ago) by rdm
File length: 19044 byte(s)
Diff to previous 41754
re-order initializer list.

Revision 41754 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Thu Nov 3 16:21:13 2011 UTC (3 years, 2 months ago) by rdm
File length: 19045 byte(s)
Diff to previous 41703
fix issues causing compilation failures on Solaris with CC5.

Revision 41703 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Nov 1 22:47:22 2011 UTC (3 years, 2 months ago) by wouter
File length: 19029 byte(s)
Diff to previous 41676
   o RooVectorDataStore

     - Correct version this time (with proper support for persistent)


   o RooAbsOptTestStatistics

     - Initialize all pointer data members in all ctors


   o RooCompositeDataStore

     - Implement sumEntries() as forward to components


   o RooAbsArg

     - Another fix for windows static linking

Revision 41676 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Tue Nov 1 15:44:30 2011 UTC (3 years, 2 months ago) by wouter
File length: 18705 byte(s)
Diff to previous 41639
   *** Miscellaneous speed improvements ***
  
    o PiecewiseInterpolation

      - Import changes from Max baak (new interpolation options)

      - Deploy fast iterators


    o RooGaussian

      - Bug fix in analytical integral over 'mean' parameter (Lorenzo Moneta)


    o RooBernstein

      - Deploy fast iterators


    o RooProdPdf

      - Reorganize code for improved speed


    o RooDataHist

      - Add support for vector-based data storage


    o RooGenContext

      - Improve generation speed by putting internal pdf clone to ADirty mode


    o RooRealBinding

      - Minimize number of calls to min() and max() for efficiency reasons


    o RooAbsRealLValue

      - In randomize() Minimize number of calls to min() and max() for efficiency reasons


    o RooDataSet

      - Make addFast() even faster by disable state propagation


    o RooAbsGenContext

      - Disable state propagation in output dataset for efficiency reasons
      - Call initGenerator() only on the first generation sequence

 
    o RooVectorDataStore

      - Add support for native vs current buffers, to support the functioning
        of addColumn(s)() and cacheArgs() when the datastore is attached
        to a pdf in RooAbsOptTestStatistics

   
    o RooAbsCategoryLValue

      - Support change in RooAbsCategory to operate nominally by index rather than by label


    o RooXYChi2Var

      - Change code to support new pdf/data attachment scheme in RooAbsOptTestStatistics
        where only the store buffers are reattached and the variable objects themselves


    o RooFFTConvPdf

      - Bug fix in getActualObservables() - functionality of setCacheObs() is now restored


    o RooAbsReal

      - Small adjustments in getVal() to make it faster in most cases


    o RooAbsCollection

      - Add datamember to keep track contents is exclusively RooRealVars. For those that
        do implement a faster version of assignFast()


    o RooAbsData

      - Add methods attachBuffers() and resetBuffers() - forwarded to datastore - that
        facilite new 'fast-attach' methods used in RooAbsOptTestStatistic


    o RooCompositeDataStore

      - Fix copy constructors so that they clone and own the subsidiary datasets

      - Operate index by value rather than by string label for efficiency reasons


    o RooRealVar

      - Bug fix in attachToVStore() to properly handle RRVs with StoreError tags


    o RooAbsPdf

      - Adjustments in getVal() to make it faster for most cases
      - Modify generate(GenSpec*) to initialize specInit flag that
        triggers one-time call of initGenerator()
      - Modify generate() to not repeatedly call initGenerator() for the same gencontext

 
    o RooAbsArg

      - Deploy fast iterators in more places
      - Add various new inline methods performing multiple ops (e.g retrieving and
        clearing one or more dirty flags) for improbed RooAbsReal/RooAbsPdf::getVal()
        operations
   

    o RooBanner

      - Increment version tag to 3.48


    o RooRealIntegral

      - Add overloaded getVal() method that also checks for shapeDirty flag,
        which was removed from RooAbsReal::getVal() as it is not needed for
        any class other than RooRealIntegral


    o RooAddGenContext

      - Disable state propagation in output dataset for efficiency reasons
      - Replace ROOT container classes with STL versions for efficiency reasons


    o RooMultiCatIter

      - Adjust code to new RooCatType behaviour that by default only tracks the
        integer representation of type


    o RooProdGenContext

      - Replace ROOT container classes with STL versions for efficiency reasons


    o RooTreeDataStore

      - Adjust code so that it works with the new fast-attach mechanism used
        in RooAbsOptTestStatistic


    o RooRealProxy

      - In lvptr() remove explicit check for lvalue


    o RooSimGenContext

      - Disable state propagation in output dataset for efficiency reasons
      - Replace ROOT container classes with STL versions for efficiency reasons
      - In createDataSet() create a prototype dataset in the first call, and
        clone that for subsequent calls rather than calling the expensive named
        argument constructor each time


    o RooTrace

      - Add (dummy) method callgrind_zero() and callgrind_dump() as hook functions
        to facilitate improved callgrind profiling


    o RooAbsCategory

      - Adjust class so that it works with the integer representation of types only
        and lookup associated string label on the fly, whenever it is requested


    o RooFormular

      - Eliminate overloaded getVal() as its functionality is now provided in 
        RooAbsReal::getVal()


    o RooAbsOptTestStatistic

      - Implement new fast-attach mechanisms for pdf and datasets: only readjust the
        buffer pointers of the dataset to the pdf observables, rather than reattaching
        the pdf to the dataset observables. This results in a _much_ faster 
        operation of setData() and no longer entangles the test statistics with observables
        of datasets that may be deleted during the lifetime of the test statistic

 
    o RooSimultaneous
 
       - Overload fillDataHist() with new version with correct normalization (from Scott Snyder)


    o RooCategory

       - Adjust getLabel() so that it retries labels associated with the current index on the fly


    o RooCatType

       - Make label contents 'transient', i.e. it is not copied on assignment for faster 
         category operations. RooAbsCategory will load the associated labels on demand from now on.

Revision 41639 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Fri Oct 28 17:46:00 2011 UTC (3 years, 2 months ago) by pcanal
File length: 18034 byte(s)
Diff to previous 41536
Fix compilation on gcc 3.4.3 and update white space

Revision 41536 - (view) (download) (as text) (annotate) - [select for diffs]
Added Fri Oct 21 20:30:29 2011 UTC (3 years, 3 months ago) by wouter
File length: 16530 byte(s)
   *** Deployment of vector datastore & miscellaneous speed improvements ***  

   o RooProduct,RooProdPdf,RooRealSumPdf,RooConstraintSum,RooAddition

     - Deploy minimal iterator


   o RooDataHist

     - Add sumEntries() 
     

   o RooDataSet

     - In ctor, allow choice of vector and tree data store.
     - Change calls to attachToTree() to attachToStore()     
     - Add sumEntries() forwarding call to stre


   o RooVectorDataStore

     - New RooAbsDataStore implementation, based on STL vectors
       rather than TTrees


   o RooAbsReal

     - Adjust getVal() so that clean-cache elements are processed more efficiently
     - Add attachToVStore() 
    

   o RooAbsCollection,RooLinkedList

     - Add minimalIterator() returning by valye a minimistic iterator (forward step only)
       with a non-virtual step function


   o RooAddPdf

     - Deploy minimal iterator
     - Skip evaluation of supplemental normalization factors if the're all one


   o RooAbsData

     - Add static member encoding default storage type (vector vs tree)
     - Add method convertToVectorStore() to convert datasets with a tree-based store
       to a vector-based store


   o RooCompositeDataStore

     - Implement dummy loadValues() method


   o RooRealVar

     - Implement attachToVStore()


   o RooAbsPdf

     - Deploy minimal iterators
     - In getVal() also check for negative integrals as problematic value


   o RooAbsArg
 
     - Implement default version of isDerived() that always returns kTRUE
     - Add pure virtual method attachToVStore()
     - Deploy minimal iterators
     - Implement attachToStore() which forwards to attachToTree() or attachToVStore()
       depending on the storage type


   o RooTreeDataStore

     - Use setOperMode() with recursion where needed instead of looping manually
       over branches
     - Implement sumEntries()


   o RooNameReg

     - Add special fast handling of null pointers for conversions both ways


   o RooAbsCategory

     - Implement attachToVstore()


   o RooRealVar

     - Implement isDerived() method that was orignally in RooAbsArg
     - Implement attachToVStore() that optionally attaches error buffers too


   o LinkDef3, Module.mk

     - Add class RooVectorDataStore


   o RooAbsString

     - Implement dummy attachToStore()


   o RooConstVar, RooCategory

     - Implement isDerived() that always returns kFALSE 


   o RooAbsDataStore

     - Add pure virtual method sumEntries()
     - Add pure virtual method loadValues() 


   o RooLinkedListIter

     - Add class RooMinimalLinkedListIterator for forward-only 
       iteration without virtual function calls 
     - Add typedef to RMLLI
     - Add non-virtual NextNV() to RooLinkedListIter

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.

  Diffs between and
  Type of Diff should be a

Sort log by:

Subversion Admin
ViewVC Help
Powered by ViewVC 1.0.9