Skip to content
This repository has been archived by the owner on Feb 10, 2025. It is now read-only.

Rel 02 09 #7

Merged
merged 71 commits into from
Jul 5, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
71 commits
Select commit Hold shift + click to select a range
55cbc02
Remove C API
jmcarcell Feb 2, 2023
5eacdc9
Remove more files and fortran tests that do not work anymore
jmcarcell Feb 3, 2023
546401e
Remove examples that depended on the deleted API and fortran bindings
jmcarcell Feb 3, 2023
7b9be13
Remove BUILD_F77_TESTJOBS CMake option
tmadlener Feb 10, 2023
de4594c
Remove more leftover files from C-API and F77 bindings
tmadlener Feb 10, 2023
5329563
Remove no longer available CMake option from CI workflow
tmadlener Feb 10, 2023
e8cccbd
Remove f77 examples
tmadlener Feb 10, 2023
9da23ac
Add test dependencies
jmcarcell Feb 3, 2023
5955018
Use ubuntu-latest for pylint github action
tmadlener May 3, 2023
71ea217
Make proper patching of LCRelations possible (#165)
FinnJohannsen May 3, 2023
b2f82fb
Add Key4hep based CI workflow (#164)
tmadlener May 3, 2023
45d96ec
Fix small bug in collection name checking for patching collections
tmadlener May 12, 2023
8adef3f
PreGenerated Headers: remove self include (breaks include-what-you-use)
andresailer May 22, 2023
7897f36
Fix template syntax that is no longer valid for gcc13/c++20
andresailer May 22, 2023
a5770a6
SIO::RunEvent: add explicit default copy constructor and assignment o…
andresailer May 22, 2023
50f1a5c
TrackStateImple: explicitly add copy ctor, assignment operator
andresailer May 22, 2023
c54b929
LCObject: explicitly add copy ctor, assignment operator
andresailer May 22, 2023
7449748
Tests: ignore warnings from CLHEP headers
andresailer May 22, 2023
2374ca6
TrackState: use implicit dtor, so we can have the copy ctor, assignme…
andresailer May 22, 2023
da3bd4a
TrackStateImple, RunEvent, LCObject: add move constructor
andresailer May 25, 2023
7fca692
Release Notes for v02-20
tmadlener Jun 2, 2023
ed1a200
Updating version to v02-20
tmadlener Jun 2, 2023
11c0999
cleanup trailing whitespace
Zehvogel Jun 12, 2023
8688a7d
LCRelationNavigator: add getRelated{To,From}MaxWeightAndObject
Zehvogel Jun 12, 2023
947b6d1
remove redundant code
Zehvogel Jun 12, 2023
723f3c2
order return values to conform to method name
Zehvogel Jun 12, 2023
40af6b8
Drop FindCLHEP.cmake
jmcarcell Jul 15, 2023
6aef732
Don't make the uninstall target if it already exists
jmcarcell Jul 14, 2023
eace86d
Make cmake respect CMAKE_INSTALL_<dir> (#174)
jmcarcell Aug 15, 2023
93aff55
Make an alias to SIO::sio (#176)
jmcarcell Sep 7, 2023
65a9367
Export targets and make sure they can be used by other projects (#177)
jmcarcell Nov 14, 2023
a740f47
Release Notes for v02-20-01
tmadlener Nov 16, 2023
eb19f93
Updating version to v02-20-01
tmadlener Nov 16, 2023
189b332
Make sure to populat the include_dirs of the lcio target (#180)
tmadlener Dec 1, 2023
2520350
Make sure to also put pre-generated headers into include directories …
tmadlener Dec 6, 2023
ce87854
Release Notes for v02-20-02
tmadlener Dec 6, 2023
0eb805e
Updating version to v02-20-02
tmadlener Dec 6, 2023
3687a34
Replacing in-built sio with FetchContent mechanism (#181)
arummler Dec 6, 2023
fafe8f6
Fix two compiler warnings (#178)
jmcarcell Dec 13, 2023
1ff4e36
Bump the minimum CMake version (#175)
jmcarcell Dec 15, 2023
bbb9c91
Make sure that LCIO installations are cleanly consumable with builtin…
tmadlener Jan 11, 2024
0de7654
SIOReader and LCReader accessMode docstring corrected to UPDATE. (#185)
alaudrain Jan 11, 2024
15f42ed
Explicit PUBLIC/PRIVATE in LINK_LIBRARY. (#186)
alaudrain Jan 15, 2024
4c66607
Use c++20 on key4hep nightlies in CI (#187)
tmadlener Jan 15, 2024
dd05a25
Imported changes from ILCSoft
pandreetto Jan 31, 2024
011b885
Release Notes for v02-21
tmadlener Feb 6, 2024
53c37ea
Updating version to v02-21
tmadlener Feb 6, 2024
9f9c23b
Make PIDHandler usable as a const object
tmadlener Apr 12, 2024
416d7b6
Build all key4hep based workflows with c++20
tmadlener Apr 12, 2024
e8c5032
Add dumpmctree tool (#170)
dudarboh May 8, 2024
373dd13
Python3 compatibility of Reader class
bartosik-hep Nov 13, 2020
eb1e41f
adding number of holes to track class
madbaron May 3, 2022
74b29a1
missing nholes bits
May 12, 2022
ca450c9
Add subdetectorHoleNumbers to count holes by subdetector.
kkrizka Jun 30, 2022
9f099e1
Add missing Nholes entry in xml description.
kkrizka Jun 30, 2022
737984f
Add missing variable.
kkrizka Jun 30, 2022
f241bb3
version check in SIOTrackHandler.cc
madbaron May 28, 2024
ce8d1fd
Version check in src/java/hep/lcio/implementation/sio/SIOTrack.java
madbaron May 28, 2024
35155c1
Version checks for SIOTrackHandler.cc
madbaron May 28, 2024
10a1c3e
Version checks in SIOTrack.java
madbaron May 28, 2024
51bdf3e
Version check with > for SIOTrackHandler.cc
madbaron May 28, 2024
9d88fd8
version bump
madbaron May 29, 2024
f27db67
Merged branch track_holes
pandreetto May 31, 2024
b2f26e3
Changed version
pandreetto May 31, 2024
c76dea7
Changed version
pandreetto Jun 4, 2024
0bf2a45
Adding track nholes to EDM (#147)
madbaron Jun 6, 2024
f554bf5
Remove unused 32bit compatibility settings (#190)
tmadlener Jun 6, 2024
c0d5ce6
Remove mentions of the removed F77 API from the documentation (#189)
tmadlener Jun 7, 2024
d589acf
Release Notes for v02-22
tmadlener Jun 7, 2024
384e6dc
Updating version to v02-22
tmadlener Jun 7, 2024
3a7c50d
Merged master from ilcsoft
pandreetto Jun 12, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 37 additions & 0 deletions .github/workflows/key4hep.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
name: key4hep

on: [push, pull_request]

jobs:
linux:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
release: ["sw.hsf.org/key4hep",
"sw-nightlies.hsf.org/key4hep"]
steps:
- uses: actions/checkout@v3
- uses: cvmfs-contrib/github-action-cvmfs@v3
- uses: aidasoft/run-lcg-view@v4
with:
container: centos7
view-path: /cvmfs/${{ matrix.release }}
run: |
mkdir build install
cd build
cmake -DCMAKE_CXX_STANDARD=20 \
-DBUILD_ROOTDICT=ON \
-DCMAKE_CXX_FLAGS="-fdiagnostics-color=always -Werror " \
-DCMAKE_INSTALL_PREFIX=../install \
..
make -k
make install
ctest --output-on-failure
echo "::group::Test downstream build"
cd -
export CMAKE_PREFIX_PATH=$PWD/install:$CMAKE_PREFIX_PATH
cd tests/downstream-project-cmake-test
mkdir build && cd build
cmake .. -DCMAKE_CXX_STANDARD=20
make -k
12 changes: 9 additions & 3 deletions .github/workflows/linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
"LCG_102/x86_64-centos7-clang12-opt",
"LCG_102/x86_64-ubuntu2004-gcc9-opt"]
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- uses: cvmfs-contrib/github-action-cvmfs@v3
- uses: aidasoft/run-lcg-view@v4
with:
Expand All @@ -24,10 +24,16 @@ jobs:
cd build
cmake -DCMAKE_CXX_STANDARD=17 \
-DBUILD_ROOTDICT=ON \
-DCMAKE_CXX_FLAGS="-fdiagnostics-color=always" \
-DCMAKE_CXX_FLAGS="-fdiagnostics-color=always -Werror" \
-DCMAKE_INSTALL_PREFIX=../install \
-DBUILD_F77_TESTJOBS=ON \
..
make -k
make install
ctest --output-on-failure
echo "::group::Test downstream build"
cd -
export CMAKE_PREFIX_PATH=$PWD/install:$CMAKE_PREFIX_PATH
cd tests/downstream-project-cmake-test
mkdir build && cd build
cmake .. -DCMAKE_CXX_STANDARD=17
make -k
4 changes: 2 additions & 2 deletions .github/workflows/python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@ on: [push, pull_request]

jobs:
python-lint:
runs-on: ubuntu-18.04
runs-on: ubuntu-latest
strategy:
matrix:
LCG: ["LCG_98/x86_64-centos7-gcc10-opt"]
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Install CVMFS
run: |
wget --no-check-certificate https://ecsft.cern.ch/dist/cvmfs/cvmfs-release/cvmfs-release-latest_all.deb
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/python_bindings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
fail-fast: false

steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- uses: cvmfs-contrib/github-action-cvmfs@v3
- uses: aidasoft/run-lcg-view@v4
with:
Expand All @@ -19,7 +19,7 @@ jobs:
cd build
cmake -DCMAKE_CXX_STANDARD=17 \
-DBUILD_ROOTDICT=ON \
-DCMAKE_CXX_FLAGS="-fdiagnostics-color=always" \
-DCMAKE_CXX_FLAGS="-fdiagnostics-color=always -Werror " \
-DCMAKE_INSTALL_PREFIX=../install \
-G Ninja \
..
Expand Down
37 changes: 23 additions & 14 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,18 +2,17 @@
# cmake file for building LCIO
# @author Frank Gaede, DESY
# @author Jan Engels, DESY
CMAKE_MINIMUM_REQUIRED( VERSION 2.8 FATAL_ERROR )
CMAKE_MINIMUM_REQUIRED( VERSION 3.14 FATAL_ERROR )
########################################################



# project name
PROJECT( LCIO )
include(GNUInstallDirs)

# project version
SET( LCIO_VERSION_MAJOR 2 )
SET( LCIO_VERSION_MINOR 17 )
SET( LCIO_VERSION_MINOR 22 )
SET( LCIO_VERSION_PATCH 0 )

### set correct LCIO version in relevant files ############################
Expand Down Expand Up @@ -166,12 +165,23 @@ eval $*

ENDIF()

# Deal with SIO
FIND_PACKAGE( SIO QUIET )
# Deal with SIO (we need at least 0.1 since there are no targets before)
FIND_PACKAGE( SIO 0.1 QUIET )

IF( NOT SIO_FOUND )
MESSAGE( STATUS "SIO not found on your system. Using builtin sio" )
ADD_SUBDIRECTORY( sio )
include(FetchContent)
FetchContent_Declare(sio_extern
GIT_REPOSITORY https://github.com/iLCSoft/SIO.git
GIT_TAG v00-02
GIT_SHALLOW 1
# FIND_PACKAGE_ARGS
)

FetchContent_MakeAvailable(sio_extern)

get_target_property(SIO_VERSION SIO::sio VERSION)
get_target_property(SIO_INCLUDE_DIRS SIO::sio INCLUDE_DIRECTORIES)
ENDIF()

MESSAGE( STATUS "Using SIO (${SIO_VERSION})" )
Expand Down Expand Up @@ -287,14 +297,6 @@ OPTION( BUILD_LCIO_EXAMPLES "Set to ON to build LCIO examples" ON )
ADD_SUBDIRECTORY( ./src/cpp lcio )



# fortran examples
OPTION( BUILD_F77_TESTJOBS "Set to ON to build LCIO F77 testjobs" OFF )
IF( BUILD_F77_TESTJOBS )
ADD_SUBDIRECTORY( ./src/f77 f77 )
ENDIF()


# directory containing lcio tests
ADD_SUBDIRECTORY( ./tests )

Expand Down Expand Up @@ -325,3 +327,10 @@ MESSAGE( STATUS "" )
GENERATE_PACKAGE_CONFIGURATION_FILES( LCIOConfig.cmake LCIOConfigVersion.cmake )

INSTALL( FILES cmake/MacroCheckPackageLibs.cmake cmake/MacroCheckPackageVersion.cmake DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake)

install(EXPORT ${PROJECT_NAME}Targets
NAMESPACE ${PROJECT_NAME}::
FILE "${PROJECT_NAME}Targets.cmake"
DESTINATION "${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}/"
)

3 changes: 1 addition & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,7 @@ LCIO ( **L**inear **C**ollider **I/O** ) is a persistency framework and event da
## Introduction

LCIO is intended to be used in both simulation studies and analysis frameworks. Its light weight and portability makes it also suitable for use in detector R&D
testbeam applications. It provides a C++ and optionally a Java implementation with a common interface (API) - a Fortran interface to the C++ implementation also exists,
if built with ROOT, Python bindings are available .
testbeam applications. It provides a C++ and optionally a Java implementation with a common interface (API) - if built with ROOT, Python bindings are available .

Using a common persistency format and event data model allows to easily share results and compare reconstruction algorithms.
LCIO is used by almost all groups involved in linear collider detector studies and thus has become a de facto standard.
Expand Down
86 changes: 0 additions & 86 deletions cmake/FindCLHEP.cmake

This file was deleted.

6 changes: 6 additions & 0 deletions cmake/LCIOConfig.cmake.in
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,12 @@ CHECK_PACKAGE_LIBS( LCIO lcio @CHECK_PACKAGE_SIO_LIBRARY@ )
## ---------------------------------------------------------------


include(CMakeFindDependencyMacro)
find_dependency(SIO @SIO_VERSION@)

# Include the targets file to create the imported targets that a client can link
# to or execute
include("${CMAKE_CURRENT_LIST_DIR}/LCIOTargets.cmake")


# ---------- final checking ---------------------------------------------------
Expand Down
4 changes: 2 additions & 2 deletions cmake/MacroGeneratePackageConfigFiles.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ MACRO( GENERATE_PACKAGE_CONFIGURATION_FILES )
CONFIGURE_FILE( "${PROJECT_SOURCE_DIR}/cmake/${arg}.in"
"${PROJECT_BINARY_DIR}/${arg}" @ONLY
)
INSTALL( FILES "${PROJECT_BINARY_DIR}/${arg}" DESTINATION . )
INSTALL( FILES "${PROJECT_BINARY_DIR}/${arg}" DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME} )
#IF( EXISTS "${_current_dir}/MacroCheckPackageLibs.cmake" )
# INSTALL( FILES "${_current_dir}/MacroCheckPackageLibs.cmake" DESTINATION cmake )
#ENDIF()
Expand All @@ -26,7 +26,7 @@ MACRO( GENERATE_PACKAGE_CONFIGURATION_FILES )
CONFIGURE_FILE( "${PROJECT_SOURCE_DIR}/cmake/${arg}.in"
"${PROJECT_BINARY_DIR}/${arg}" @ONLY
)
INSTALL( FILES "${PROJECT_BINARY_DIR}/${arg}" DESTINATION . )
INSTALL( FILES "${PROJECT_BINARY_DIR}/${arg}" DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME} )
#IF( EXISTS "${_current_dir}/MacroCheckPackageVersion.cmake" )
# INSTALL( FILES "${_current_dir}/MacroCheckPackageVersion.cmake" DESTINATION cmake )
#ENDIF()
Expand Down
26 changes: 0 additions & 26 deletions cmake/ilcsoft_build_32bit_compatible.cmake

This file was deleted.

1 change: 0 additions & 1 deletion cmake/ilcsoft_default_settings.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ INCLUDE( ilcsoft_default_library_versioning )
INCLUDE( ilcsoft_default_build_output_directories )
INCLUDE( ilcsoft_default_rpath_settings )
INCLUDE( ilcsoft_default_cxx_flags )
#INCLUDE( ilcsoft_build_32bit_compatible )


# uninstall target may only be created once per project
Expand Down
4 changes: 3 additions & 1 deletion cmake/ilcsoft_default_uninstall_target.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,9 @@ IF( EXISTS "${_current_dir}/cmake_uninstall.cmake.in" )
"${PROJECT_BINARY_DIR}/cmake_uninstall.cmake" @ONLY )

# add uninstall target
ADD_CUSTOM_TARGET( uninstall "${CMAKE_COMMAND}" -P "${PROJECT_BINARY_DIR}/cmake_uninstall.cmake" )
if (NOT TARGET uninstall)
ADD_CUSTOM_TARGET( uninstall "${CMAKE_COMMAND}" -P "${PROJECT_BINARY_DIR}/cmake_uninstall.cmake" )
endif()

ENDIF( EXISTS "${_current_dir}/cmake_uninstall.cmake.in" )

Loading
Loading