Log of /trunk/tmva/src/BinaryTree.cxx
Parent Directory
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: 8507 byte(s)
Diff to
previous 40005
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
37399 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Modified
Wed Dec 8 15:22:07 2010 UTC (4 years, 1 month ago) by
evt
File length: 8368 byte(s)
Diff to
previous 36966
several bug fixes to TMVA copied from dev
fix to multiple reader problem, ROOT-bug 76076, make static variable in
event non-static
fix valgrind error due to uninitialized values
fix 5 more valgrind error detected in unit tests
fix to sub-optimal BDT settings
fix bug fix in Likelihood with VarTransforms, fixed unit tests breaks
LikelihoodD and LikelihoodPCA
Revision
21630 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Modified
Thu Jan 10 19:40:44 2008 UTC (7 years ago) by
brun
File length: 7551 byte(s)
Diff to
previous 21218
From Joerg Stelzer:
* documentation of all classes ( I hope I caught all 17 of them, but I didn't know how to check except by looking through all the files)
* plugin capabilities for user developed multivariate classifiers
* An improved GUI where the user can print the significance curves for adjustable signal and background yields
* A fix to a compiler complaint that Axel told me about
Revision
21218 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Modified
Thu Dec 6 06:22:43 2007 UTC (7 years, 1 month ago) by
brun
File length: 6456 byte(s)
Diff to
previous 21079
From Joerg Stelzer:
fix for the coding rule violation (missing docu) and a bug fix in a macro in the tmva/test directory (we had hardcoded a filename there).
Revision
20882 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Modified
Mon Nov 19 11:31:26 2007 UTC (7 years, 2 months ago) by
rdm
File length: 5658 byte(s)
Diff to
previous 20226
Set property svn:eol-style LF on all source and Makefiles. This should avoid
problems with Win32 line endings ending up in the repository. All MS tools
support LF eols fine.
Revision
20226 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Modified
Fri Oct 5 11:14:06 2007 UTC (7 years, 3 months ago) by
brun
File length: 5658 byte(s)
Diff to
previous 19826
From Joerg Stelzer:
New features:
- Cuts can now be applied independently for signal and background in the PrepareTrainingAndTestTree phase.
- Previously, the input variables used by the Fisher classifier were always normalised to [-1,1] by default. This has been removed, so that it is now in the hand of the user to decide whether or not normalisation is applied. Choose "Normalise" ("!Normalise") for normalisation (no normalisation), default is "!Normalise".
- Significant speed improvements for PDERS. For the options to benefit from this, see the example "PDERSkNN" in macros/TMVAnalysis.C or examples/TMVAnalysis.cxx.
Thanks to Kamil Kraszewski and friends from Cracow for implementing this.
- Re-established backward compatibility of TMVA code down to ROOT version 4.02/00.
- Shortened BDT weight-file and standalone C++ reader class by 20% and 50%, respectively.
- Weight expressions can now be set individually for signal and background via
the calls factory->SetSignalWeightExpression( "<signal-expression>" ) and factory->SetBackgroundWeightExpression( "<background-expression>" ). The former call is still supported.
- Overtraining test: a new GUI button (corresponding to an extension of the macro "mvas.C") is available to plot a comparison of the classifier response distributions for the training and independent test data sets. The results of a Kolmogorov-Smirnov compatibility test are printed on stdout and plots.
- The cuts corresponding to a given signal efficiency can be retrieved via the reader. An example for this is implemented in "macros/TMVApplication.C". Briefly,
retrieve the cuts classifier object as follows:
TMVA::MethodCuts* mcuts = (TMVA::MethodCuts*)reader->FindMVA( "CutsGA method" );,
define cut vectors (a vector of pairs can also be retrieved via overloaded
GetCuts function): std::vector<Double_t> cutsMin; std::vector<Double_t> cutsMax;
and fill them via: mcuts->GetCuts( wantedSignalEfficiency, cutsMin, cutsMax );
- Clean up of code and include headers to improve forward declaration.
- Bug fixes:
- Memory leaks in the Reader class are removed: the Reader is now properly destructed (deletion of all handled classifiers). Thereby, pointer problems in the destructors of Fisher and SVM have been found and fixed.
- The macro TMVApplication.C produced a segmentation fault when run from the ROOT prompt
(the compiled version in the examples directory worked fine). This problem is now solved.
- The color selection has been adapted to the new color palette that was introduced in ROOT 5.16. The macros should now look alike with all ROOT versions (above 4.02/00).
- Very important bug fix: the application of cuts in the PrepareTrainingAndTestTree
call in conjunction with the use of several trees (ie, several consecutive calls
of factory->AddSignalTree(...) or factory->AddBackgroundTree(...)), lead to a wrong application of the cut to all trees but the first one in the signal and background chains. More details can be provided if requested - please contact
the authors. We wish to thank Manfred Groh for spotting and analysing the problem!
- Some compilers complained about a missing #include "TMVA/Configurable.h" in the Reader
class. This has been fixed. - Fixed problem in RuleFit's standalone class when using integer input variables.
- Fixed compilation problem when using decorrelation preprocessing of input variables in C++ standalone reader classes.
- Fixed bug in number-of-plots calculation in correlation script. - Fixed bug in printing of number of events in case of several trees (no impact on results).
- Fixed inconsistency between cut optimisation and cut reading: the aligned definition of min and max cuts is: a variable passes a cut if: min < var <= max.
(This inconsistency may have affected your results if you used cut optimisation together with integer variables. Please check with the new version.)
- Fixed macro path in TMVAGui.C to fix problem when running the GUI in the ROOT/TMVA distribution.
Also: TMVA Style moved from TMVAlogon into tmvaglob to fix style problem when running in the ROOT/TMVA distribution.
- Fixed typos in weight file names in MLP and BDT macros - Fixed "MinMax" and "RMS" options of PDERS (thanks to Junpei Maeda for spotting this)
- Fixed compilation problem in MetricEuler class on some platforms
Revision
16805 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Modified
Fri Nov 17 16:00:03 2006 UTC (8 years, 2 months ago) by
brun
File length: 5221 byte(s)
Diff to
previous 16768
New version of TMVA fixing many coding conventions violations.
New version of the tmva test suite. To execute it run the script
TMVAnalysis.C
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.
