Skip to content

Added missing default initial values for PV classes

Patrick Spradlin requested to merge spradlin_202305May_pvinit into master

Related

Follow-up to merge of !794 (merged) and Moore!2278 (merged).

Related to #422, but does not close it.

Description

Following the merge of !794 (merged) and Moore!2278 (merged), the RTA Software Manager noted that the new test introduced by Moore!2278 (merged), RecoConf.allen_gaudi_pv_with_mcchecking, produced unstable stdout for some architectures (see the discussion on the MR).

This instability was traced to unitialized date members in Allen's PV class. This MR provides default initialization for those data members, which should stabilize the test.

In local testing, this MR successfully stabilized the output of RecoConf.allen_gaudi_pv_with_mcchecking on the x86_64_v2-centos7-clang12+detdesc-opt platform without the need for an updated reference. A ci-test is required to check whether other platforms may require a reference update.

Merge request reports