@emanuel-schmid Thanks a lot for that perspective! This is exactly the kind of perspective I was hoping for in the discussion of #807. I would be fine with going the "string-only" route. However, there are some inconsistencies we then have to fix. In the tests for Impact, there are two instances where Impacts are instantiated with non-string event names:

• event_name=[0, 1, "two", "three", 30, 31],

• imp.event_name = [0, 1, 2, 3, 4, 5, 6, 7, 8, 15]

  
  There might be more such instances throughout the code base.