Project

General

Profile

Feature #10888

Complete first round of services re-architecture

Added by Ruth Pordes almost 5 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Gianluca Petrillo
Category:
-
Target version:
v05_00_00
Start date:
11/09/2015
Due date:
12/31/2015
% Done:

100%

Estimated time:
Spent time:
136.00 h
Experiment:
-
Co-Assignees:
Duration: 53

Description

Complete first round of services re-architecture as described at https://cdcvs.fnal.gov/redmine/projects/larsoft/wiki/Architecture_revision

DatabaseUtil,LArProperties, DetectorProperties, TimeService, LArFFT, SpaceCharge

History

#1 Updated by Gianluca Petrillo almost 5 years ago

  • % Done changed from 50 to 90

The current state of the work is in feature branches:

  • feature/jpaley_LArPropertiesBreakup for all LArSoft repositories except larexamples and larsoft
  • feature/gp_ServiceCoreReview for all experiment code except DUNE: uboonecode, lariatsoft, argoneutcode and lar1ndcode
  • feature/jpaley_LArPropertiesBreakup (to be checked!) for DUNE (dunetpc)

Jonathan Paley has done great part of the work of splitting.
DUNE's code might need minor adjustments to date; the rest of the repositories is ready for test.

#2 Updated by Gianluca Petrillo almost 5 years ago

  • Target version set to v05_00_00

Pending: FHiCL configuration validation of LArProperties and DetectorProperties.
This is optional, but I would like to make it happen.

#3 Updated by Gianluca Petrillo almost 5 years ago

  • Status changed from Assigned to Resolved

FHiCL validation has been added to LArPropertiesServiceStandard, DetectorPropertiesServiceStandard, and their providers.

Target is delivered and expected to be merged into v05_00_00_rc branches.

The final list is:

  • LArProperties, DetectorProperties and TimeService heavily reworked
  • Geometry slightly changed to conform to new patterns
  • IChannelStatusService and IDetPedestalService renamed
  • DatabaseUtil deprecated and dropped from standard services
  • LArFFT and SpaceCharge not touched and still in need for factorization (not scheduled)

#4 Updated by Gianluca Petrillo almost 5 years ago

  • % Done changed from 90 to 100

A summary of the work has been presented to the Coordinators.

#5 Updated by Lynn Garren over 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF