Include material properties in checksums
Previously the checksum algorithm only checks the medium and material names. Through this merge request, we can also check their properties. Many thanks to @tlatham for giving a very great idea that rather than only checking the fraction and density quantities, the derived quantities, interaction length, and radiation length, can cover more things.
NOTE: The changes in checksum reference in all versions are not caused by the changes in geometry, just because we now include the material parts.
#11 (closed)
closeValidated by
-
Core Software -
RTA -
Simulation
Merge request reports
Activity
added enhancement label
assigned to @mexu
- Resolved by Gloria Corti
Ben provided some very helpful information about the property checks of materials in the DD4HEP project, https://github.com/AIDASoft/DD4hep/blob/master/DDCore/src/plugins/Compact2Objects.cpp#L420-L592. However, after checking these, I wonder if we really need such detailed/complicated checks here, maybe a simple check of density and fraction I have implemented is enough?
mentioned in issue Gauss#137
requested review from @gcorti
added update-working-checksum label
mentioned in issue Gauss#136
removed enhancement label
added enhancement label
mentioned in commit 4fec6762