Remove sipm from PDVD detsim products

[YoannKermaidic]

No description provided.

[YoannKermaidic]

trigger build

[FNALbuild]

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

[FNALbuild]

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

[FNALbuild]

✔️ CI build for DUNE Succeeded on slf7 for c14:prof -- details available through the CI dashboard

parent CI build details are available through the CI dashboard

[FNALbuild]

❌ CI build for DUNE Failed at phase ci_tests DUNE on slf7 for e26:prof - ignored failure for unit_test -- details available through the CI dashboard

🚨 For more details about the failed phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

[YoannKermaidic]

trigger build

[FNALbuild]

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

[FNALbuild]

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

[FNALbuild]

✔️ CI build for DUNE Succeeded on slf7 for c14:prof -- details available through the CI dashboard

parent CI build details are available through the CI dashboard

[FNALbuild]

❌ CI build for DUNE Failed at phase ci_tests DUNE on slf7 for e26:prof -- details available through the CI dashboard

🚨 For more details about the failed phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

[YoannKermaidic]

trigger build

[FNALbuild]

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

[FNALbuild]

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

[FNALbuild]

✔️ CI build for DUNE Succeeded on slf7 for c14:prof -- details available through the CI dashboard

parent CI build details are available through the CI dashboard

[FNALbuild]

⚠️ CI build for DUNE Warning at phase ci_tests DUNE on slf7 for e26:prof -- details available through the CI dashboard

🚨 For more details about the warning phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

[YoannKermaidic]

Seems like the configuration is now fine.
Can we update the dunesw version as this PR solves a broken piece in the code (link)?

[YoannKermaidic]

Hi @tomjunk
Can this PR be merged such that the simulation can fully through?

[YoannKermaidic]

This is now all taken into account in the broader PR : #214

[lpaulucc]

Should this one be closed given this other PR? #214

[YoannKermaidic]

Should this one be closed given this other PR? #214

correct