Duplication of functions in persistency.py
Spotted duplication of functions in AllenConf/persistency.py, namely (new code / old code):
-
build_decision_ids: persistency.py#L18 and persistency.py#L57 -
register_decision_ids: persistency.py#L36 and persistency.py#L80 -
register_allen_encoding_table: persistency.py#L52 and persistency.py#L95
Tested locally and only old code (persistency.py#L57, persistency.py#L80, persistency.py#L95) is being executed when calling Allen from Moore (e.g. allen_mdf_write.qmt).
This looks like a bug where the proper set of functions is not used and one set should be removed, or if both sets are used, then functions should be renamed to clearly distinct between them.
New code: Better HltDecReports: !964 (merged) Old code: add encoding key to DecReport raw bank: !911 (merged)