SelectivePacker: Consistently check whether TES entries are explicitly requested
SelectivePacker only explicitly checked whether KeyedContainers and RecSummary it had access to were explicitly requested -- other TES entries were added to the packed output regardless of any request, as soon as they existed in the TES. This MR moves this check 'upstream' in the code so that this is done in the same place for every type of object.
As a result, it may be that objects other than KeyedContainer and RecSummary are no longer implicitly copied, and the configuration may need to be adapted to explicitly copy them instead. To help with this, a new property 'AlwaysPack' has been added which specifies locations that should be packed regardless of any Hlt/Spruce decisions.
Must be applied in conjunction with Moore!2281 (merged) -- which also depends on !4149 (merged)
Merge request reports
Activity
added bug fix label
added RTA label
added 1 commit
- 7e233780 - Consistently check whether TES entries are explicitly requested
mentioned in merge request Moore!2281 (merged)
added 6 commits
-
7e233780...fac15526 - 4 commits from branch
master
- a0aa1891 - Consistently check whether TES entries are explicitly requested
- 696e4a2e - add an AlwaysPack option to specify a set of TES location which will always be packed
-
7e233780...fac15526 - 4 commits from branch
added 1 commit
- a40a0b82 - add an AlwaysPack option to specify a set of TES location which will always be packed
- Resolved by Rosen Matev
Is the idea to set
AlwaysPack=[neutralPP2MCP, chargedPP2MCP]
option for sprucing and that's all?
added 14 commits
-
a40a0b82...b31754b7 - 12 commits from branch
master
- ed4b638d - Consistently check whether TES entries are explicitly requested
- add44fd9 - add an AlwaysPack option to specify a set of TES location which will always be packed
-
a40a0b82...b31754b7 - 12 commits from branch
added 12 commits
-
c8624373...376feb17 - 8 commits from branch
master
- 21bba43a - Consistently check whether TES entries are explicitly requested
- 3ee4afe1 - add an AlwaysPack option to specify a set of TES location which will always be packed
- 77aeb693 - add ExternalLocations property to identify location which are persisted...
- 5671a076 - better diagnostic in PackedProtoParticle
Toggle commit list-
c8624373...376feb17 - 8 commits from branch
added 1 commit
- 536e7051 - add option to make absent required containers a non-fatal error
added 1 commit
- 4510c512 - add option to make absent required containers a non-fatal error
added 10 commits
-
4510c512...6ff10929 - 5 commits from branch
master
- acc705b4 - Consistently check whether TES entries are explicitly requested
- b259ce7e - add an AlwaysPack option to specify a set of TES location which will always be packed
- 0a4fce80 - add ExternalLocations property to identify location which are persisted...
- 09420a40 - better diagnostic in PackedProtoParticle
- 390b5c87 - add option to make absent required containers a non-fatal error
Toggle commit list-
4510c512...6ff10929 - 5 commits from branch
added 21 commits
-
390b5c87...27b83152 - 16 commits from branch
master
- 500aae9b - Consistently check whether TES entries are explicitly requested
- b34149b9 - add an AlwaysPack option to specify a set of TES location which will always be packed
- 830070a3 - add ExternalLocations property to identify location which are persisted...
- e346db6c - better diagnostic in PackedProtoParticle
- 07dbdcc4 - add option to make absent required containers a non-fatal error
Toggle commit list-
390b5c87...27b83152 - 16 commits from branch
- [2023-06-17 21:40] Validation started with lhcb-master-mr#8276
- [2023-06-18 12:02] Validation started with lhcb-master-mr#8277
- [2023-06-19 14:34] Validation started with lhcb-master-mr#8283
- [2023-06-19 21:40] Validation started with lhcb-master-mr#8292
- [2023-06-28 22:26] Validation started with lhcb-master-mr#8401
Edited by Software for LHCb