Patches for GenFSRJson to correct JSON output
Patch GenFSRJson
algorithm so that the output JSON file contains the Gaudi::Property
values and not their string representation. Furthemore correct the setting of default value of m_jsonOutputName
property to implement what original author intended.
Should go with Gauss!821 (merged)
Merge request reports
Activity
added 1 commit
- 8874722a - Patches for GenFSRJson to correct JSON output
added RTA label
requested review from @adavis
I'm too careful here. But if you, @adavis, could, please, check that everything is OK for merging, I could test this in
lhcb-head
by the end of the week. Already checked there are no compilation errors, though I'm a bit uncertain about theGaudi::Property::setValue
use ininitialize
.Edited by Alex Grecuremoved RTA label
added RTA label
will look this evening, thanks for the ping @agrecu
- Resolved by Alex Grecu
- Resolved by Alex Grecu
assigned to @agrecu
added 1 commit
- c04fb3bf - Set single property for JSON output file path
mentioned in issue Moore#382 (closed)
mentioned in issue Moore#390 (closed)
Hi @agrecu please assign to me when this is ready to be tested. Thanks
requested review from @peilian
assigned to @peilian