Project

General

Profile

Bug #18925

BranchDescription issue where wrappedName_ is not populated

Added by Eric Flumerfelt over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
02/08/2018
Due date:
% Done:

100%

Estimated time:
1.00 h
Spent time:
Scope:
Internal
Experiment:
-
SSI Package:
art
Duration:

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 over 1 year 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 over 1 year ago

  • Description updated (diff)

#3 Updated by Kyle Knoepfel over 1 year 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 over 1 year 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 over 1 year ago

  • Target version set to 2.10.02

#6 Updated by Kyle Knoepfel over 1 year ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF