Skip to content

Add DetectorDataService::initialize(nlohmann::json) and remove other versions

Marco Clemencic requested to merge use-json-for-dds-init into master

With !254 (merged) I introduced the possibility of initializing DetectorDataService from a YAML object.

With the introduction of JSON as in memory condition format (!286 (merged)) we could use JSON instead of YAML for DetectorDataService initialization too.

There is not performance penalty in using YAML, but JSON is slightly more compact to use.

This MR adds DetectorDataService::initialize(nlohmann::json), removes the other signatures and adapts the tests to use the new method.

Requires LHCb!3851 (merged)

Edited by Sebastien Ponce

Merge request reports