Skip to content

Commit 86c4ab3

Browse files
committed
[EMCAL-911] Improve documentation of RecoContainer
- Fix headers of classes (particularly ingroup) - Add class documentation
1 parent 7344154 commit 86c4ab3

File tree

3 files changed

+40
-7
lines changed

3 files changed

+40
-7
lines changed

Detectors/EMCAL/reconstruction/include/EMCALReconstruction/FastORTimeSeries.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ namespace o2::emcal
2121
/// \class FastORTimeSeries
2222
/// \brief Container for FastOR time series
2323
/// \author Markus Fasel <markus.fasel@cern.ch>, Oak Ridge National Laboratory
24-
/// \ingroup EMCALReconstruction
24+
/// \ingroup EMCALreconstruction
2525
/// \since April 19, 2024
2626
///
2727
/// Time series are encoded in bunches in the raw data, which are usually time-reversed.

Detectors/EMCAL/reconstruction/include/EMCALReconstruction/RecoContainer.h

Lines changed: 37 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,25 @@ struct RecCellInfo {
5454

5555
/// \class EventContainer
5656
/// \brief Containter of cells for a given event
57-
/// \ingroup EMCALReconstruction
57+
/// \author Markus Fasel <markus.fasel@cern.ch>, Oak Ridge National Laboratory
58+
/// \ingroup EMCALreconstruction
59+
/// \since May 30, 2023
60+
///
61+
/// The EventContainer is part of the reco container and keeps the cell / LEDMON / TRU
62+
/// data of a given event (trigger) defined by the BC and orbit in the raw data header.
63+
/// In case the gain type merging is activated the event container performs on-the-fly
64+
/// merging of high- and low-gain data for cells and LEDMONS keeping only high- or
65+
/// low-gain and preferring the high-gain due to better resolution if not saturated.
66+
///
67+
/// Error handling:
68+
/// The EventContainer can raise a TRUIndexException in case TRU information with an
69+
/// unexpected index (>= 52) is added / requested.
5870
class EventContainer
5971
{
6072
public:
6173
/// \class TRUIndexException
6274
/// \brief Handler for access of TRU data with invalid TRU index
63-
/// \ingroup EMCALReconstruction
75+
/// \ingroup EMCALreconstruction
6476
class TRUIndexException final : public std::exception
6577
{
6678
public:
@@ -222,7 +234,18 @@ class EventContainer
222234

223235
/// \class RecoContainer
224236
/// \brief Handler for cells/LEDMONS/Trigger data in timeframes
225-
/// \ingroup EMCALReconstruction
237+
/// \author Markus Fasel <markus.fasel@cern.ch>, Oak Ridge National Laboratory
238+
/// \ingroup EMCALreconstruction
239+
/// \since May 30, 2023
240+
///
241+
/// The RecoContainer handles the cell/LEDMON/trigger data of all events of a given
242+
/// timeframe during the reconstruction. Event data are handled internally via the
243+
/// EventContainer, where the RecoContainer provides access to.
244+
///
245+
/// Error handling:
246+
/// The RecoContainer can raise an InteractionNotFoundException in case read access
247+
/// is requested for an interaction based on the o2::InteractionRecord which is not
248+
/// for which no data was inserted into the container.
226249
class RecoContainer
227250
{
228251
public:
@@ -291,7 +314,17 @@ class RecoContainer
291314

292315
/// \class RecoContainerReader
293316
/// \brief Iterator over reco containers
294-
/// \ingroup EMCALReconstruction
317+
/// \author Markus Fasel <narkus.fasel@cern.ch>, Oak Ridge National Laboratory
318+
/// \ingroup EMCALreconstruction
319+
/// \since May 30, 2023
320+
///
321+
/// The RecoContainerReader iterates over the events stored in the RecoContainer in
322+
/// a time-ordered sequence. The function hasNext checks whether there are more
323+
/// events to iterate over, while nextEvent provides access to the next event.
324+
///
325+
/// Error handling:
326+
/// The RecoContainerReader can raise an InvalidAccessException in case access is tried to
327+
/// invalid data.
295328
class RecoContainerReader
296329
{
297330
public:

Detectors/EMCAL/reconstruction/include/EMCALReconstruction/TRUDataHandler.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ namespace o2::emcal
2828
/// \class TRUDataHandler
2929
/// \brief Helper class to handle decoded TRU data during the reconstruction
3030
/// \author Markus Fasel <markus.fasel@cern.ch>, Oak Ridge National Laboratory
31-
/// \ingroup EMCALReconstruction
31+
/// \ingroup EMCALreconstruction
3232
/// \since April 19, 2024
3333
///
3434
/// The decoded TRU data contains the following information
@@ -44,7 +44,7 @@ class TRUDataHandler
4444
public:
4545
/// \class PatchIndexException
4646
/// \brief Handler of errors related to invalid trigger patch IDs
47-
/// \ingroup EMCALReconstruction
47+
/// \ingroup EMCALreconstruction
4848
class PatchIndexException final : public std::exception
4949
{
5050
public:

0 commit comments

Comments
 (0)