Skip to content

Conversation

@stognini
Copy link

This PR adds a new OpticalPropagation_module, which will replace the current PDFastSimPAR_module. The new module will take advantage of a new IOpticalPropagation abstract interface class, which will allow the implementation of (currently) 3 separate art::tools:

  • PDFastSimPAR: This will mirror the existing module.
  • Celeritas: This will allows the propagation of optical photons on CPU and GPU through Celeritas
  • Opticks: This will allow the propagation of optical photons on Nvidia GPUs through the Opticks package.

@stognini stognini marked this pull request as draft December 12, 2025 20:31
@stognini
Copy link
Author

stognini commented Dec 12, 2025

Marked PR as draft since I have a small list of todo's still, but will expedite discussion:

Todo's

  • Agree on nomenclature for new interface, tools, and module
  • Add fcl files for the module job execution and tools
  • Confirm/update code standards (from variable naming to class/function documentation)
  • Fix current build issue
In file included from /home/stognini/larsoft/srcs/larsim/larsim/PhotonPropagation/OpticalPropagationTools/OpticalPropPDFastSimPAR.h:8,
                 from /home/stognini/larsoft/srcs/larsim/larsim/PhotonPropagation/OpticalPropagationTools/OpticalPropPDFastSimPAR_tool.cc:6:
/home/stognini/larsoft/srcs/larsim/larsim/PhotonPropagation/OpticalPropagationTools/IOpticalPropagation.h:11:10: fatal error: lardataobj/Simulation/OpDetBacktrackerRecord.h: No such file or directory
   11 | #include "lardataobj/Simulation/OpDetBacktrackerRecord.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant