Skip to content
GitLab
Select Git revision
  • zonalstats_histogram default
Find file BlameHistoryPermalink
  • Manuel Grizonnet's avatar
    ENH: rename files with txx extension to hxx · 3a1fd1fc
    Manuel Grizonnet authored 
    OTB followed since the beginning the ITK convention and use .txx extension for all
template classes. Nevertheless, some development tools do not recognize .txx
file extension. Other tool like GitHub can't do in-browser syntax highlighting for
txx files I think.

The root problem is the use of the txx which should be changed to hxx (or hpp).

In 2011, after an in-depth discussion near April 20, 2011 on the
Insight-Developers mailing list, ITK rename all txx files to hxx (and event
prevent the push of .txx files with a pre-commit hook). It happens is major release v4.

You can find some arguments in the discussion about the change and also in other
projects related to ITK which applied the same modification, see for instance VXL:

https://github.com/vxl/vxl/issues/209

This commit apply now the same modification for OTB.

I understand that it will change some habit for developers and don't bring new
features but I think that in general it is better to stay align with ITK guidelines.

In my opinion, it always facilitate the use of OTB and ITK together if we share
when we can the same code architecture, directory organization, naming
conventions...
    3a1fd1fc
otbTrainNormalBayes.hxx 2.06 KiB
1 
/*
2 
 * Copyright (C) 2005-2017 Centre National d'Etudes Spatiales (CNES)
3 
 *
4 
 * This file is part of Orfeo Toolbox
5 
 *
6 
 *     https://www.orfeo-toolbox.org/
7 
 *
8 
 * Licensed under the Apache License, Version 2.0 (the "License");
9 
 * you may not use this file except in compliance with the License.
10 
 * You may obtain a copy of the License at
11 
 *
12 
 *     http://www.apache.org/licenses/LICENSE-2.0
13 
 *
14 
 * Unless required by applicable law or agreed to in writing, software
15 
 * distributed under the License is distributed on an "AS IS" BASIS,
16 
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
17 
 * See the License for the specific language governing permissions and
18 
 * limitations under the License.
19 
 */
20
21 
#ifndef otbTrainNormalBayes_txx
22 
#define otbTrainNormalBayes_txx
23 
#include "otbLearningApplicationBase.h"
24 
#include "otbNormalBayesMachineLearningModel.h"
25
26 
namespace otb
27 
{
28 
namespace Wrapper
29 
{
30
31 
  template <class TInputValue, class TOutputValue>
32 
  void
33 
  LearningApplicationBase<TInputValue,TOutputValue>
34 
  ::InitNormalBayesParams()
35 
  {
36 
    AddChoice("classifier.bayes", "Normal Bayes classifier");
37 
    SetParameterDescription("classifier.bayes", "Use a Normal Bayes Classifier. "
38 
        "See complete documentation here \\url{http://docs.opencv.org/modules/ml/doc/normal_bayes_classifier.html}.");
39
40 
  }
41
42 
  template <class TInputValue, class TOutputValue>
43 
  void
44 
  LearningApplicationBase<TInputValue,TOutputValue>
45 
  ::TrainNormalBayes(typename ListSampleType::Pointer trainingListSample,
46 
                     typename TargetListSampleType::Pointer trainingLabeledListSample,
47 
                     std::string modelPath)
48 
  {
49 
    typedef otb::NormalBayesMachineLearningModel<InputValueType, OutputValueType> NormalBayesType;
50 
    typename NormalBayesType::Pointer classifier = NormalBayesType::New();
51 
    classifier->SetRegressionMode(this->m_RegressionFlag);
52 
    classifier->SetInputListSample(trainingListSample);
53 
    classifier->SetTargetListSample(trainingLabeledListSample);
54 
    classifier->Train();
55 
    classifier->Save(modelPath);
56 
  }
57
58 
} //end namespace wrapper
59 
} //end namespace otb
60
61 
#endif
62