Bug #18925
BranchDescription issue where wrappedName_ is not populated
Scope:
Internal
Experiment:
-
SSI Package:
art
Description
While debugging issues migrating artdaq to art v2_09_06, we found the following diff to be essential to get art to successfully output data:
diff --git a/canvas/Persistency/Provenance/BranchDescription.cc b/canvas/Persistency/Provenance/BranchDescription.cc
index 8234c7a..32ffe40 100644
--- a/canvas/Persistency/Provenance/BranchDescription.cc
+++ b/canvas/Persistency/Provenance/BranchDescription.cc
@@ -67,6 +67,7 @@ namespace art {
}
transients_.get().branchName_ = canonicalProductName(
friendlyClassName(), moduleLabel(), productInstanceName(), processName());
+ transients_.get().wrappedName_ = wrappedClassName(producedClassName());
}
ParameterSetID const&
History
#1 Updated by Eric Flumerfelt about 3 years ago
Sorry
diff --git a/canvas/Persistency/Provenance/BranchDescription.cc b/canvas/Persistency/Provenance/BranchDescription.cc index 8234c7a..32ffe40 100644 --- a/canvas/Persistency/Provenance/BranchDescription.cc +++ b/canvas/Persistency/Provenance/BranchDescription.cc @@ -67,6 +67,7 @@ namespace art { } transients_.get().branchName_ = canonicalProductName( friendlyClassName(), moduleLabel(), productInstanceName(), processName()); + transients_.get().wrappedName_ = wrappedClassName(producedClassName()); } ParameterSetID const&
#2 Updated by Kyle Knoepfel about 3 years ago
- Description updated (diff)
#3 Updated by Kyle Knoepfel about 3 years ago
- Status changed from New to Assigned
- Assignee set to Kyle Knoepfel
We will make this change. In the meantime, you should call:
#include "art/Framework/IO/Root/checkDictionaries.h"
...
for (auto const& bd : branch_descriptions) {
// Fluffs all relevant transients
art::checkDictionaries(bd);
}
#4 Updated by Kyle Knoepfel about 3 years ago
- Status changed from Assigned to Resolved
- % Done changed from 0 to 100
- Estimated time set to 1.00 h
- SSI Package art added
Implemented with commits canvas:56789f6 and canvas_root_io:a8b478b.
#5 Updated by Kyle Knoepfel about 3 years ago
- Target version set to 2.10.02
#6 Updated by Kyle Knoepfel about 3 years ago
- Status changed from Resolved to Closed