Skip to content

Adding extra main dictionary items for BLS chains (ATR-24205)

Catrin Bernius requested to merge bernius/athena:BLSdict_ATR-24205 into master

Adding in two extra entries to the main dictionary for BLS chains as mentioned in ATR-24205 to accommodate the extra and variant topo algs for the BLS code. Tagging @tursom and @khoo to comment and @lyubushk, @abarton, @dzanzi for info.

For now, the entries are just for testing and do appear also in the AllowedTopos_Bphysics list but should be removed from there, this will be done by a BLS expert once the code is implemented in the signature domain. These lines are clearly marked in the SignatureDicts.py

What should be done as well is to make a check that no keyword appears in the allowed topos list more than once, this is addressed through changing the ifs into if...elif statements except for the comb topos following TJs comment in a thread below.

Also a question here is if we need to also remove the combo topos as well as the new BLS extra and variant topos from the cparts list in lines 208 - 212, this has been done.

Edited by Catrin Bernius

Merge request reports