Ralf Kliemt authored 8 months ago and Florian Uhlig committed 8 months ago

Add generation of Jupyter kernelspec files to support Root & CbmRoot classes in Python3.
Generation is performed by cmake and the files are placed at <build/install>/share/jupyter/.
Some documentation is provided in docs/jupyter.

Pierre-Alain Loizeau authored 10 months ago and Florian Uhlig committed 9 months ago

- Add flag (cmake option) to disable the L1 optimization
- Auto-set this flag if a Profiling build with gcc 10.2.1 is detected
=> Redmine Refs #2590

Pierre-Alain Loizeau authored 1 year ago and Florian Uhlig committed 1 year ago

Add missing iostreams and system requirement and dependency

Add special case for macosx
When using libboost_iostreams below 1.80.0 libzstd is linked explicitly
but not found even if installed in /usr/local/lib so we need to add this
path to the target properties.

Pierre-Alain Loizeau authored 1 year ago and Volker Friese committed 1 year ago

Runs and pop the server but:
- Input interface not defined! Something need to be invented to replace FairMQ
- Commands do not work as the static boolean accessed by CLING do not seem to be share to the main threads (worked in FairTask world)
- Due to this, no mechanism to stop the main loop apart from Ctrl+C
- Not tested with histos, neither with canvases

Add the sytem compiler include paths needed for our static standalone
clang-tidy binary to the list of include paths.
The paths are only added to the compilation units in the generated file
compile_commands.json which is used as input for clang-tidy. Without the
paths clang-tidy can't find the standard C and C++ header files.

The geometries, parameters and input data are not needed when running
clang-tidy, so don't waste time downloading them.
The still downloaded projects include header files which are used by CbmRoot
classes. If these header files are not found by clang-tidy the check of some
compilation units fail.

Use VC in scalar mode, otherwise system and hardware related header files are
needed which couldn't be found.
Fix an issue with an missing header file in KFParticle when using the scalar
VC version.

Administrator authored 1 year ago and Florian Uhlig committed 1 year ago

The remaining code in the mvd directory was not used after all the changes.
Move the 2 PDF files to core/detectors/mvd as temporary solution. Finally the
files need a better place.
Remove the complete mvd directory and the usage from the main CMakeLists.txt

Administrator authored 2 years ago and Florian Uhlig committed 1 year ago

Add build system files for new mvd code structure
Move code from mvd directory to core/detectors/mvd, sim/detectors/mvd,
reco/detectors/mvd and analysis/detectors/mvd.
Create the libraries CbmMvdCore, CbmMvdSim, CbmMvdReco and CbmMvdAna.

This commit will not compile.

Eoin Clerkin authored 2 years ago and Florian Uhlig committed 2 years ago

Record the hash and date of the build and report these at build and configuration times.

Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

Remove obsolete files containing macros which were needed before the build
system was moderinsed. The only macro which is still needed was added in
the file CbmMacros.cmake.

Felix Weiglhofer authored 2 years ago and Florian Uhlig committed 2 years ago

Replace usage of deprecated LIBRARY_OUTPUT_PATH variable with newer CMAKE_LIBRARY_OUTPUT_DIRECTORY.
The new variable has the advantage that it populates the LIBRARY_OUTPUT_DIRECTORY property of targets.
This property is required by xpu to figure out where the generated cuda/hip libraries should be placed.

Administrator authored 2 years ago and Florian Uhlig committed 2 years ago

Update NicaFemto version which include now all patches.

Omveer Singh authored 2 years ago and Florian Uhlig committed 2 years ago

print_info() macro added to CbmMacro.cmake module
unnecessary variables removed

There were problems reported when a system version of GTest exist which
should be solved with this fix.

Files moved to NO_DICT_SRC:
L1Algo/L1Event.cxx
L1Algo/L1EventMatch.cxx

Files are commented for time being in L1 target:
OffLineInterface/CbmL1RichENNRingFinder.cxx
OffLineInterface/CbmL1RichENNRingFinderParallel.cxx

xpu git tag version modified
Vc package finding moved to top level CMakeLists.txt

Administrator authored 2 years ago and Florian Uhlig committed 2 years ago

If using a non FairSoft installation of the dependencies problems were
reported after the last changes to the build system. Instead of hardcoding
the SIMPATH use the the information from ROOT about its binary directory.
Remove some old usage of the root-config script, use CMake variables provided
by ROOT instead.

Linking the MQ TSA  binaries on Debian11 need an additional boost library as
well as a reordering of the link libraries.

Use cmake FindPackage config files instead of the self written FindPackage
module files.
The FindPackage config files are created by the developers themself and
installed with external dependencies.

Add missing CMake variables which are not defined any longer by the self
written FindPackage module files.

Administrator authored 2 years ago and Florian Uhlig committed 2 years ago

Do some Minor fixes.

Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

Restructure code in the main CMakeLists.txt

Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

Move code which does some basic checks into separate CMake file.
Some additional reordering of the code is needed.

Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

Move code which defines the compiler settings into separate CMake file.

Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

Move code which defines additional targets into separate CMake file.
Move code which generates the config files into separate CMake file.

Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

Check if environmnet variables SIMPATH and FAIRROOTPATH are set or if the paths
are passed on the commandline as -DSIMPATH=<path to FairSoft installation>
-DFAIRROOTPATH=<path to FairRoot installation> early during the cmake run.
After the initial check only the CMake variables are used.

Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

Remove obsolete, unneeded and unused code.

The external flesnet requires at least CMake 3.14.0.
Check already in our CMakeLists.txt if this version is available, otherwise
the test is done when building flesnet and the error is hidden in some log
file.
Use CMake from the FairSoft installation on the CI runner (lxir116 )where the
CMake version is to old.

VMC isn't any longer part of ROOT but an external package, so we have to find
it differently. The same problem also appears for NicaFemto such that we fix
the issue with a patch.

Administrator authored 2 years ago and Volker Friese committed 2 years ago

Use C++17 as C++ standard for the compilation of CbmRoot.
Check if compiler supports the C++17 standard.
Check if also FairSoft was compiled with C++17.
Check the required versions of ROOT (at least Root6), FairSoft( at least apr21)
and FairRoot (at leass v18.6.7).
If any of the checks fail stop the configuration with a meaningful error
message.
Remove some switches which are not needed any longer.
Remove checks for C++11 features. No need to check them any longer, since the
new standard is noe C++17.

Administrator authored 3 years ago and Florian Uhlig committed 3 years ago

Patch the AnalysisTree and AnalysisTreeQA buils systems to create also
correct dictionaries for ROOT versions below v6.18.0.
Add the directories where the header files of both packages are installed
to ROOT_INCLUDE_PATH which are paths used by rootcint to find needed
header files.

Before the change it was necessary to define an environment variable to switch
between different test suites like for example the Experimental or Nightly
tests. Now the test suites are chosen by the CMake command line parameter
CBM_TEST_MODEL. These are Experimental, Nightly and Weekly which differ by the
number of executed tests.

Administrator authored 3 years ago and Florian Uhlig committed 3 years ago

Build the algo subdirectory only if C++17 is the chosen c++ standard. The
value depends on the C++ standard used to compile FairSoft.