Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
Gaudi
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Iterations
Requirements
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Locked files
Build
Pipelines
Jobs
Pipeline schedules
Test cases
Artifacts
Deploy
Releases
Container Registry
Model registry
Operate
Environments
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Code review analytics
Issue analytics
Insights
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Gaudi
Gaudi
Commits
93443e00
Commit
93443e00
authored
1 year ago
by
Marco Clemencic
Browse files
Options
Downloads
Plain Diff
Fix booking of n-tuple columns with and without ranges
Closes
#282
See merge request
!1522
parents
e1af807e
d4b57404
No related branches found
Branches containing commit
No related tags found
Tags containing commit
1 merge request
!1522
Fix booking of n-tuple columns with and without ranges
Pipeline
#6492040
passed
1 year ago
Stage: test
Changes
1
Pipelines
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
RootHistCnv/src/RCWNTupleCnv.cpp
+14
-3
14 additions, 3 deletions
RootHistCnv/src/RCWNTupleCnv.cpp
with
14 additions
and
3 deletions
RootHistCnv/src/RCWNTupleCnv.cpp
+
14
−
3
View file @
93443e00
/***********************************************************************************\
* (c) Copyright 1998-20
19
CERN for the benefit of the LHCb and ATLAS collaborations *
* (c) Copyright 1998-20
23
CERN for the benefit of the LHCb and ATLAS collaborations *
* *
* This software is distributed under the terms of the Apache version 2 licence, *
* copied verbatim in the file "LICENSE". *
...
...
@@ -19,6 +19,7 @@
#include
<cstdio>
#include
<cstring>
#include
<list>
#include
<type_traits>
#include
<utility>
#include
<vector>
...
...
@@ -121,8 +122,18 @@ namespace {
}
if
(
it
->
hasIndex
()
||
it
->
length
()
>
1
)
{
desc
+=
']'
;
}
lowerRange
=
it
->
range
().
lower
();
upperRange
=
it
->
range
().
upper
();
// 0 and -1 are used to mark that the range is not defined
lowerRange
=
0
;
upperRange
=
-
1
;
if
constexpr
(
std
::
is_integral_v
<
T
>
)
{
// An explicit range makes sense only for integral types so we check only in that case if it is defined.
// Note that later a range is only taken into account for int32_t.
if
(
it
->
range
().
lower
()
!=
it
->
range
().
min
()
&&
it
->
range
().
upper
()
!=
it
->
range
().
max
()
)
{
lowerRange
=
it
->
range
().
lower
();
upperRange
=
it
->
range
().
upper
();
}
}
desc
+=
typ
;
size
+=
item_size
*
dimension
;
}
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment