Project

General

Profile

Necessary Maintenance #17926

Implement a better name for the track proxy accessor to the unfitted trajectory.

Added by Gianluca Petrillo over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
High
Assignee:
Gianluca Petrillo
Category:
Data products
Target version:
-
Start date:
10/16/2017
Due date:
% Done:

100%

Estimated time:
Spent time:
0.50 h
Experiment:
-
Duration:

Description

It so happens now that a track proxy object can access the fitted trajectory of the track via track->Trajectory() and the associated, unfitted trajectory as track.trajectory() (and its pointer with trajectoryPtr()).
This accidental ambiguity needs to be removed.

Associated revisions

Revision c79518dd (diff)
Added by Gianluca Petrillo over 3 years ago

Track proxy metod name trajectory() (and related) renamed.

The functions hasTrajectory(), trajectory() and trajectoryPtr() have
been renamed.
This solves issue #17926.

History

#1 Updated by Lynn Garren over 3 years ago

  • Status changed from New to Assigned
  • Assignee set to Gianluca Petrillo

We need the fix for this weeks larsoft release.

#2 Updated by Gianluca Petrillo over 3 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100
Names have been changed:
Old name New name
hasTrajectory() hasOriginalTrajectory()
trajectory() originalTrajectory()
trajectoryPtr() originalTrajectoryPtr()

The change has been pushed into lardata develop branch directly, despite it being a breaking change.
No committed code has shown any dependency on these methods (tried the whole LArSoft suite plus six experiment repositories).

#3 Updated by Gianluca Petrillo over 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF