PhotonVisibilityService should be factorised
PhotonVisibilityService is currently not factorised.
Factorisation is needed for use in external algorithms like LArLite's flash matching.
#1 Updated by Gianluca Petrillo over 4 years ago
- Status changed from New to Assigned
- Assignee set to Gianluca Petrillo
- Estimated time set to 16.00 h
- Experiment LArSoft added
- Experiment deleted (
A preliminary reading confirms that the factorisation is both needed and feasible.There are two options:
- breaking change: the art service interface will lose the ability to provide the functionalities and will only deliver the service provider (as
- non-breaking change: the art service interface will retain the ability to provide the functionalities and will also expose the service provider interface (as
My preference is decidedly for the breaking change.
I expect the breakage to be fixable in single line changes, with some automation possible (although not foolproof), similarly with what happened for
LArProperties and the other core services.
#2 Updated by Gianluca Petrillo over 4 years ago
- % Done changed from 0 to 80
I have split the code so that framework dependencies are confined into a art service and the new service provider is shareable.
I have also added a unit test that should move with the service provider.
There may be additional work to do, though.
The branch is
lardata. It is updated to LArSoft
#3 Updated by Katherine Lato about 4 years ago
LArSoft will finish the integration of the new factorized code to replace the old code (which was in a feature branch). This requires running a few scripts to update the branch, test it, etc.
There was another issue to move the code so that it doesn't depend on art so LArLite can use it. That piece is the responsibility of LArLite and isn't being tracked here.
#5 Updated by Katherine Lato almost 3 years ago
- Status changed from Assigned to Accepted
- Assignee deleted (
Note, we are putting this back to the accepted state because we have no staff to work on this. We asked Jason Stock, who has ideas, but not the time since his emphasis is on analysis for the DUNE calibration effort and his thesis.