Project

General

Profile

Bug #18003

lariov::SIOVChannelStatusProvider depends on art

Added by Gianluca Petrillo about 2 years ago. Updated over 1 year ago.

Status:
Assigned
Priority:
Normal
Category:
Other
Target version:
-
Start date:
10/21/2017
Due date:
% Done:

0%

Estimated time:
Occurs In:
Experiment:
-
Co-Assignees:
Duration:

Description

The service provider lariov::SIOVChannelStatusProvider (larevt:source:larevt/CalibrationDBI/Providers/SIOVChannelStatusProvider.h) is required to be independent of the framework, but it uses art classes (I spot art::ServiceHandle).

These dependencies must be moved out to the art service.

History

#1 Updated by Gianluca Petrillo about 2 years ago

  • Description updated (diff)

#2 Updated by Lynn Garren about 2 years ago

  • Status changed from New to Assigned
  • Assignee set to Brandon Eberly

Brandon, are you able to fix this?

#3 Updated by Brandon Eberly over 1 year ago

This cannot be done without changing the behavior of the code or making a breaking change. There is a GetChannelsWithStatus function that returns all channels with a particular status. For this to work in the case of initializing from a default status (rather than database or a file), then the total number of channels needs to be known. The code currently uses the geometry service to get this information.

#4 Updated by Kyle Knoepfel over 1 year ago

We'd like to have a discussion with you regarding this.



Also available in: Atom PDF