Skip to content

Commit

Permalink
add clangd config and fix include warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
YanzhaoW committed Feb 25, 2025
1 parent 12f271f commit 421fba1
Show file tree
Hide file tree
Showing 9 changed files with 82 additions and 17 deletions.
2 changes: 2 additions & 0 deletions .clangd
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Diagnostics:
MissingIncludes: Strict
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,19 @@
******************************************************************************/

#include "R3BNeulandLSQREngineAdaptor.h"
#include "R3BDataMonitor.h"
#include "R3BNeulandHitPar.h"

#include <FairRuntimeDb.h>
#include <R3BException.h>
#include <R3BLogger.h>
#include <R3BNeulandCalData2.h>
#include <R3BNeulandCalToHitPar.h>
#include <R3BNeulandCommon.h>
#include <R3BNeulandHitModulePar.h>
#include <R3BNeulandHitPar.h>
#include <R3BNeulandParDirCreator.h>
#include <R3BShared.h>
#include <algorithm>
#include <cmath>
#include <vector>

namespace
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,13 @@
#pragma once

#include "R3BNeulandCosmicEngine.h"

#include <R3BNeulandCalData2.h>
#include <R3BNeulandCalToHitPar.h>
#include <R3BNeulandCosmicTracker.h>
#include <R3BNeulandHitCalibrationEngine.h>
#include <R3BShared.h>
#include <vector>

namespace R3B::Neuland::Calibration
{
Expand Down
Original file line number Diff line number Diff line change
@@ -1,10 +1,17 @@
#include "R3BNeulandMilleCalDataProcessor.h"
#include "R3BLogger.h"
#include "R3BNeulandCalData2.h"
#include <Fit/BinData.h>
#include <Math/WrappedMultiTF1.h>
#include <R3BNeulandCommon.h>
#include <TError.h>
#include <algorithm>
#include <fmt/core.h>
#include <fmt/ranges.h>

Check warning on line 10 in neuland/calibration/cal_to_hit/engine/R3BNeulandMilleCalDataProcessor.cxx

View workflow job for this annotation

GitHub Actions / clang-tidy (default, debian12)

included header ranges.h is not used directly [misc-include-cleaner]
#include <numeric>
#include <range/v3/algorithm.hpp>

Check warning on line 12 in neuland/calibration/cal_to_hit/engine/R3BNeulandMilleCalDataProcessor.cxx

View workflow job for this annotation

GitHub Actions / clang-tidy (default, debian12)

included header algorithm.hpp is not used directly [misc-include-cleaner]
#include <range/v3/algorithm/find_if.hpp>
#include <vector>

namespace R3B::Neuland::Calibration
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@
#include <R3BNeulandCalData2.h>
#include <R3BNeulandCommon.h>
#include <TF1.h>
#include <fmt/core.h>
#include <unordered_map>
#include <vector>

namespace R3B::Neuland::Calibration
{
Expand Down
40 changes: 26 additions & 14 deletions neuland/calibration/cal_to_hit/engine/R3BNeulandMillepede.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -12,26 +12,46 @@
******************************************************************************/

#include "R3BNeulandMillepede.h"
#include "ParResultReader.h"
#include "R3BDataMonitor.h"
#include "R3BLogger.h"
#include "R3BNeulandCalData2.h"
#include "R3BNeulandCalToHitPar.h"
#include "R3BNeulandMilleCalDataProcessor.h"
#include <R3BException.h>
#include <R3BNeulandCalToHitParTask.h>
#include <R3BNeulandCommon.h>
#include <SteerWriter.h>

#include <TGraphErrors.h>
#include <TH1.h>
#include <algorithm>
#include <array>
#include <fmt/core.h>
#include <iterator>
#include <memory>
#include <numeric>
#include <range/v3/algorithm/all_of.hpp>
#include <range/v3/iterator/operations.hpp>
#include <range/v3/numeric/accumulate.hpp>
#include <range/v3/view.hpp>

#include <cstdlib>
#include <optional>
#include <range/v3/view/all.hpp>
#include <range/v3/view/filter.hpp>
#include <range/v3/view/sliding.hpp>
#include <range/v3/view/transform.hpp>
#include <stdexcept>
#include <string>
#include <utility>
#include <vector>

namespace rng = ranges;

constexpr auto DEFAULT_RES_FILENAME = "millepede.res";
constexpr auto SCALE_FACTOR = 10.F;
constexpr auto REFERENCE_BAR_NUM = 25;
// constexpr auto REFERENCE_BAR_NUM = 25;
constexpr auto MILLE_BUFFER_SIZE = std::size_t{ 100000 };
constexpr auto DEFAULT_T_ERROR = 2; // ns

Expand Down Expand Up @@ -332,7 +352,7 @@ namespace R3B::Neuland::Calibration

auto MillepedeEngine::select_t_diff_signal(const std::vector<MilleCalData>& plane_data)
{
if (plane_data.size() < 1)
if (plane_data.empty())
{
return plane_data.end();
}
Expand Down Expand Up @@ -463,21 +483,10 @@ namespace R3B::Neuland::Calibration

void MillepedeEngine::init_parameter()
{
auto num_of_modules = GetModuleSize();

if (cal_to_hit_par_ == nullptr)
{
throw R3B::runtime_error("Pointer to cal_to_hit_par is nullptr!");
}

// auto& module_pars = cal_to_hit_par_->GetListOfModuleParRef();
// module_pars.clear();

// for (unsigned int module_num{ 1 }; module_num <= num_of_modules; ++module_num)
// {
// auto module_par_iter = module_pars.try_emplace(module_num).first;
// module_par_iter->second.effective_speed.value = init_effective_c_;
// }
}

void MillepedeEngine::init_steer_writer()
Expand All @@ -486,7 +495,10 @@ namespace R3B::Neuland::Calibration
steer_writer.set_filepath(pede_steer_filename_);
steer_writer.set_parameter_file(parameter_filename_);
steer_writer.set_data_filepath(input_data_filename_);
steer_writer.add_method(SteerWriter::Method::inversion, std::make_pair(3.F, 0.001F));
static constexpr auto NUMBER_OF_ITERARTION = 3.F;
static constexpr auto CONVERGENCE_RECOGNITION = 0.001F;
steer_writer.add_method(SteerWriter::Method::inversion,
std::make_pair(NUMBER_OF_ITERARTION, CONVERGENCE_RECOGNITION));
steer_writer.add_other_options(std::vector<std::string>{ "hugecut", "50000" });
steer_writer.add_other_options(std::vector<std::string>{ "outlierdownweighting", "4" });

Expand Down
12 changes: 12 additions & 0 deletions neuland/calibration/cal_to_hit/engine/R3BNeulandMillepede.h
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,25 @@
******************************************************************************/
#pragma once

#include "MilleEntry.h"
#include "R3BDataMonitor.h"
#include "R3BLogger.h"
#include "R3BNeulandCalData2.h"
#include "R3BNeulandCalToHitPar.h"
#include "R3BNeulandCosmicEngine.h"
#include "R3BNeulandMilleCalDataProcessor.h"
#include <Mille.h>
#include <ParResultReader.h>
#include <PedeLauncher.h>
#include <R3BNeulandCommon.h>
#include <TH1.h>
#include <cstdint>
#include <fmt/core.h>
#include <memory>
#include <optional>
#include <string>
#include <utility>
#include <vector>
// #include <RankChecker.h>

class TGraphErrors;
Expand Down
11 changes: 11 additions & 0 deletions neuland/calibration/cal_to_hit/engine/R3BNeulandPredecessor.cxx
Original file line number Diff line number Diff line change
@@ -1,8 +1,19 @@
#include "R3BNeulandPredecessor.h"
#include "R3BDataMonitor.h"
#include "R3BLogger.h"
#include "R3BNeulandCalData2.h"
#include "R3BNeulandCalToHitPar.h"
#include "R3BNeulandCommon.h"
#include "R3BValueError.h"
#include <R3BNeulandCalToHitParTask.h>
#include <TF1.h>
#include <TFitResult.h>
#include <TFitResultPtr.h>
#include <TH1.h>
#include <TH2.h>
#include <array>
#include <fmt/core.h>
#include <vector>

namespace R3B::Neuland::Calibration
{
Expand Down
8 changes: 8 additions & 0 deletions neuland/calibration/cal_to_hit/engine/R3BNeulandPredecessor.h
Original file line number Diff line number Diff line change
@@ -1,7 +1,15 @@
#pragma once

#include "R3BDataMonitor.h"
#include "R3BLogger.h"
#include "R3BNeulandCalData2.h"
#include "R3BNeulandCalToHitPar.h"
#include "R3BNeulandCosmicEngine.h"

#include <TH2.h>
#include <fmt/core.h>
#include <vector>

namespace R3B::Neuland::Calibration
{
class Predecessor : public CosmicEngineInterface
Expand Down

0 comments on commit 421fba1

Please sign in to comment.