Skip to content

More complete hash for JetDefinition

@delsart, it seemed weird to me that we were not hashing & comparing equality with the algorithm parameter, and also VR jet parameters were not compared. Then I realised that basename neatly captures the full description of the jet clustering configuration, and as it's governed by the lproperties logic, it remains complete. It should only be the naming parameters like prefix, infix, suffix that can be ignored in the hash. I added the byVertex option as well, as it seems relevant.

Do you think I missed anything?

Added __repr__ = __str__ for a couple of the classes that were missing it as well, as I was seeing the pointer addresses show up in some printouts.

Merge request reports

Loading