Project

General

Profile

Support #21253

remove larg4 dependency on larsim

Added by Lynn Garren almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
10/29/2018
Due date:
% Done:

100%

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

Description

larg4 as initially deployed uses larsim/LArG4/ParticleFilters.h. This is
the only dependency on larsim. We expect that larsim will eventually
depend on larg4 rather than having larg4 depend on larsim.

It is agreed that ParticleFilters.h should move to lardataalg. It will go into a new directory, whose name is not yet determined. Suggestions so far are ParticleInfo and Utilities.

This work will be found on feature/lg_move_ParticleFilters in the appropriate directories.

This is a breaking change that is easily scriptable.

History

#1 Updated by Lynn Garren almost 2 years ago

Erica has suggested that ParticleFilters.h might fit better in larcorealg. The seems to make sense since this is where other geometry files reside. larcorealg has CoreUtils, GeoAlgo, Geometry, and TestUtils subdirectories. I would be inclined to put this header into CoreUtils.

#2 Updated by Lynn Garren almost 2 years ago

  • % Done changed from 0 to 100

The final change is in larsoft v07_10_00, and will soon be on the head of develop in larg4 and larsim.

#3 Updated by Lynn Garren almost 2 years ago

  • Status changed from Assigned to Resolved

#4 Updated by Kyle Knoepfel almost 2 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF