Skip to content

HLTPrescale JSON: sort chains by name and add express prescale

Mark Stockton requested to merge mark/athena:master-hltprescale into master

Sort chains in HLTPrescale JSON by name rather than counter - will be easier to compare prescales and matches the L1prescale JSON. Also means counter can be removed (if needed still accessible from HLTMenu, but irrelevant and not used when reading the prescale).

Add express stream details - extra prescale/enabled information added to chains that are in the express stream. Hitting issues in retrieving the data from a map on the TrigConf part (so not included here), but would be good to have this menu change in the TR release to make uploaded keys contain this new detail. Then can work on testing patches if ready in time and also start adding into the TTweb display. Have checked uploading and running from the DB is unaffected by this addition.

As discussed with @rbielski, the thoughts on stream prescaling are simplified here to be the one use case we know we need. Anything more flexible can be considered later if needed, but while simple to add to the json would over complicate many areas downstream.

cc @dzanzi @khoo @astruebi @palacino

Old format

        "HLT_noalg_L1EM8VH": {
            "name": "HLT_noalg_L1EM8VH",
            "counter": 403,
            "hash": 4065285611,
            "prescale": 1,
            "enabled": true
        },
        "HLT_noalg_L1EM7_EMPTY": {
            "name": "HLT_noalg_L1EM7_EMPTY",
            "counter": 429,
            "hash": 271418477,
            "prescale": 1,
            "enabled": true
        },

New format

        "HLT_noalg_L1EM7_EMPTY": {
            "name": "HLT_noalg_L1EM7_EMPTY",
            "hash": 271418477,
            "prescale": 1,
            "enabled": true,
            "prescale_express": 1,
            "enabled_express": true
        },
        "HLT_noalg_L1EM8VH": {
            "name": "HLT_noalg_L1EM8VH",
            "hash": 4065285611,
            "prescale": 1,
            "enabled": true
        },

Wanted to also remove the name parameter, but found it is currently being used by TTweb so will get removed in a future iteration of the HLTPrescale JSON

Merge request reports