Fixed issue with name attribute in CallableEventWithFilter

Fixes #2904

- Replaced DynamicClassAttribute by property
- other minor updates

Author: vfdev-5

ignite/engine/events.py

@@ -3,7 +3,6 @@
 import weakref
 from collections.abc import Sequence
 from enum import Enum
-from types import DynamicClassAttribute
 from typing import Any, Callable, Dict, Iterable, Iterator, List, Optional, TYPE_CHECKING, Union
 
 from torch.utils.data import DataLoader
@@ -34,16 +33,15 @@ def __init__(self, value: str, event_filter: Optional[Callable] = None, name: Op
         if not hasattr(self, "_value_"):
             self._value_ = value
 
-        if not hasattr(self, "_name_") and name is not None:
+        if not hasattr(self, "_name_"):
             self._name_ = name
 
-    # copied to be compatible to enum
-    @DynamicClassAttribute
+    @property
     def name(self) -> str:
         """The name of the Enum member."""
         return self._name_
 
-    @DynamicClassAttribute
+    @property
     def value(self) -> str:
         """The value of the Enum member."""
         return self._value_
@@ -197,15 +195,15 @@ def __eq__(self, other: Any) -> bool:
             return NotImplemented
 
     def __hash__(self) -> int:
-        return hash(self._name_)
+        return hash(self.name)
 
     def __or__(self, other: Any) -> "EventsList":
         return EventsList() | self | other
 
 
 class EventEnum(CallableEventWithFilter, Enum):
-    """Base class for all :class:`~ignite.engine.events.Events`. User defined custom events should also inherit
-    this class.
+    """Base class for all :class:`~ignite.engine.events.Events`. User defined custom events should inherit
+    from this class.
 
      Examples:
          Custom events based on the loss calculation and backward pass can be created as follows: