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.

Reconstruction/Jet/JetRecConfig/python/JetDefinition.py

@@ -123,7 +123,7 @@ class JetDefinition(object):
 
             
     def __hash__(self):
-        return hash((self._radius,self._inputdef,self.ptmin,str(self.ghostdefs),str(self.modifiers),str(self.extrainputs)))
+        return hash((self.basename,self._inputdef,self.ptmin,str(self.ghostdefs),str(self.modifiers),str(self.extrainputs),self.byVertex))
 
     def __eq__(self,rhs):
         return self.__hash__() == rhs.__hash__()
@@ -380,7 +380,9 @@ class JetInputExternal(object):
     # Define a string conversion for printing
     def __str__(self):
         return f"JetInputExternal({self.name},type={str(self.basetype)})"
-    
+    # Need to override __repr__ for printing in lists etc
+    __repr__ = __str__
+
     def __eq__(self,other):
         return all([
             self.name == other.name,
@@ -523,6 +525,8 @@ class JetInputConstit(object):
     # Define a string conversion for printing
     def __str__(self):
         return f"JetInputConstit({self.name},type={str(self.basetype)})"
+    # Need to override __repr__ for printing in lists etc
+    __repr__ = __str__