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.