Project

General

Profile

Support #24898

Help update protoduneana to use larsoft v09

Added by Tingjun Yang about 2 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
09/01/2020
Due date:
% Done:

100%

Estimated time:
Spent time:
Experiment:
-
Co-Assignees:
Duration:

Description

Dear LArSoft experts,

We have a package protoduneana for ProtoDUNE analysis:
https://cdcvs.fnal.gov/redmine/projects/protoduneana
Currently protoduneana depends on larsoft/dunetpc v08_55_01. When I tried to upgrade the dependence to use dunetpc v09_00_01, I got a lot of errors related to service interfaces. Would you help us upgrade protoduneana to be compatible with larsoft v09?

Thanks,
Tingjun

History

#1 Updated by Kyle Knoepfel about 2 months ago

  • Assignee set to Kyle Knoepfel
  • Status changed from New to Assigned

#2 Updated by Jacob Calcutt about 2 months ago

To note: I updated protoduneana to v08_62_00.

#3 Updated by Kyle Knoepfel about 2 months ago

Understood. This is taking a while since there is a lot of code that requires updating--but I hope to have it ready by sometime early next week.

#4 Updated by Kyle Knoepfel about 2 months ago

  • % Done changed from 0 to 100
  • Status changed from Assigned to Resolved

I have pushed the feature/team_for_larsoft_v9 branch which should have all of the relevant changes. Please let me know if you run into any problems.

#5 Updated by Tingjun Yang about 2 months ago

Thanks Kyle for the great help. I tested it with e19 and everything works. There seems to be a problem with c7:

https://buildmaster.fnal.gov/buildmaster/view/DUNE/job/build-protoduneana/BUILDTYPE=debug,QUAL=c7,label1=swarm,label2=SLF7/15/console
CMakeFiles/PDSPmatchmc_module.dir/PDSPmatchmc_module.cc.o: In function `geo::GeometryCore::NSiblingElements(readout::TPCsetID const&) const':
/cvmfs/larsoft.opensciencegrid.org/products/larcorealg/v09_00_00/include/larcorealg/Geometry/GeometryCore.h:4853: undefined reference to `geo::GeometryCore::NTPCsets(geo::CryostatID const&) const'
CMakeFiles/PDSPmatchmc_module.dir/PDSPmatchmc_module.cc.o: In function `geo::GeometryCore::NSiblingElements(readout::ROPID const&) const':
/cvmfs/larsoft.opensciencegrid.org/products/larcorealg/v09_00_00/include/larcorealg/Geometry/GeometryCore.h:5053: undefined reference to `geo::GeometryCore::NROPs(readout::TPCsetID const&) const'
CMakeFiles/PDSPmatchmc_module.dir/PDSPmatchmc_module.cc.o: In function `geo::GeometryCore::IncrementID(readout::TPCsetID&) const':
/cvmfs/larsoft.opensciencegrid.org/products/larcorealg/v09_00_00/include/larcorealg/Geometry/GeometryCore.h:5640: undefined reference to `geo::GeometryCore::NTPCsets(geo::CryostatID const&) const'
clang-7: error: linker command failed with exit code 1 (use -v to see invocation)

#6 Updated by Lynn Garren about 2 months ago

A library is missing from the cmake link list. Not sure which one. It's curious that you only see the problem with the c7 build.

#7 Updated by Kyle Knoepfel about 2 months ago

Looks like CMakeLists.txt file for the PDSPmatchmc_module needs to include larcorealg_Geometry as a library dependency.

#8 Updated by Tingjun Yang about 2 months ago

Kyle Knoepfel wrote:

Looks like CMakeLists.txt file for the PDSPmatchmc_module needs to include larcorealg_Geometry as a library dependency.

Thanks Kyle. I added the missing library.

#9 Updated by Kyle Knoepfel about 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF