Merge pull request #72 from lsst-sitcom/tickets/DM-41885

DM-41885: Make TMAEvents hash so they work with sets

python/lsst/summit/utils/tmaUtils.py

@@ -470,6 +470,22 @@ def __repr__(self):
             f" end={self.end!r}"
         )
 
+    def __hash__(self):
+        # deliberately don't hash the blockInfos here, as they are not
+        # a core part of the event itself, and are listy and cause problems
+        return hash((self.dayObs,
+                     self.seqNum,
+                     self.type,
+                     self.endReason,
+                     self.duration,
+                     self.begin,
+                     self.end,
+                     self.version,
+                     self._startRow,
+                     self._endRow
+                     )
+                    )
+
     def _ipython_display_(self):
         print(self.__str__())
 

tests/test_tmaUtils.py

@@ -317,6 +317,10 @@ def test_endToEnd(self):
             self.assertEqual(event.type.name, types[eventNum])
             self.assertEqual(event.endReason.name, endReasons[eventNum])
 
+        eventSet = set(slews)  # check we can hash
+        eventSet.update(slews)  # check it ignores duplicates
+        self.assertEqual(len(eventSet), len(slews))
+
     @vcr.use_cassette()
     def test_noDataBehaviour(self):
         eventMaker = self.tmaEventMaker