Removing dataclass default values

Author: Pushkar-Bhuse

forte/data/ontology/top.py

@@ -231,10 +231,10 @@ class Link(BaseLink):
     """
     # this type Any is needed since subclasses of this class will have new types
 
-    parent_type: Any = Entry
-    child_type: Any = Entry
-    parent: Optional[int] = None
-    child: Optional[int] = None
+    parent_type: Any
+    child_type: Any
+    parent: Optional[int]
+    child: Optional[int]
 
     ParentType = Entry
     ChildType = Entry
@@ -385,10 +385,10 @@ class MultiPackLink(MultiEntry, BaseLink):
     integers, one additional index on which pack it comes from.
     """
 
-    parent_type: Any = Entry
-    child_type: Any = Entry
-    parent: Tuple = (None, None)
-    child: Tuple = (None, None)
+    parent_type: Any
+    child_type: Any
+    parent: Tuple
+    child: Tuple
 
     ParentType = Entry
     ChildType = Entry
@@ -519,8 +519,8 @@ class MultiPackGroup(MultiEntry, BaseGroup[Entry]):
     r"""Group type entries, such as "coreference group". Each group has a set
     of members.
     """
-    member_type: Type[Entry] = Entry
-    members: Optional[FList[Entry]] = None
+    member_type: Type[Entry]
+    members: Optional[FList[Entry]]
 
     MemberType = Entry
 
@@ -1018,7 +1018,7 @@ class Payload(Entry):
 
     payload_idx: int
     modality_name: str
-    uri: Optional[str] = None
+    uri: Optional[str]
 
     def __init__(
         self,

ft/onto/base_ontology.py

@@ -317,17 +317,12 @@ class PredicateLink(Link):
         arg_type (Optional[str]):	The predicate link type.
     """
 
-    arg_type: Optional[str] = None
+    arg_type: Optional[str]
 
     ParentType = PredicateMention
     ChildType = PredicateArgument
 
-    def __init__(
-        self,
-        pack: DataPack,
-        parent: Optional[Entry] = None,
-        child: Optional[Entry] = None,
-    ):
+    def __init__(self, pack: DataPack, parent: Optional[Entry] = None, child: Optional[Entry] = None):
         super().__init__(pack, parent, child)
         self.arg_type: Optional[str] = None
 
@@ -341,18 +336,13 @@ class Dependency(Link):
         rel_type (Optional[str]):
     """
 
-    dep_label: Optional[str] = None
-    rel_type: Optional[str] = None
+    dep_label: Optional[str]
+    rel_type: Optional[str]
 
     ParentType = Token
     ChildType = Token
 
-    def __init__(
-        self,
-        pack: DataPack,
-        parent: Optional[Entry] = None,
-        child: Optional[Entry] = None,
-    ):
+    def __init__(self, pack: DataPack, parent: Optional[Entry] = None, child: Optional[Entry] = None):
         super().__init__(pack, parent, child)
         self.dep_label: Optional[str] = None
         self.rel_type: Optional[str] = None
@@ -361,23 +351,18 @@ def __init__(
 @dataclass
 class EnhancedDependency(Link):
     """
-    A `Link` type entry which represent a enhanced dependency:
+    A `Link` type entry which represent a enhanced dependency: 
      https://universaldependencies.org/u/overview/enhanced-syntax.html
     Attributes:
         dep_label (Optional[str]):	The enhanced dependency label in Universal Dependency.
     """
 
-    dep_label: Optional[str] = None
+    dep_label: Optional[str]
 
     ParentType = Token
     ChildType = Token
 
-    def __init__(
-        self,
-        pack: DataPack,
-        parent: Optional[Entry] = None,
-        child: Optional[Entry] = None,
-    ):
+    def __init__(self, pack: DataPack, parent: Optional[Entry] = None, child: Optional[Entry] = None):
         super().__init__(pack, parent, child)
         self.dep_label: Optional[str] = None
 
@@ -390,17 +375,12 @@ class RelationLink(Link):
         rel_type (Optional[str]):	The type of the relation.
     """
 
-    rel_type: Optional[str] = None
+    rel_type: Optional[str]
 
     ParentType = EntityMention
     ChildType = EntityMention
 
-    def __init__(
-        self,
-        pack: DataPack,
-        parent: Optional[Entry] = None,
-        child: Optional[Entry] = None,
-    ):
+    def __init__(self, pack: DataPack, parent: Optional[Entry] = None, child: Optional[Entry] = None):
         super().__init__(pack, parent, child)
         self.rel_type: Optional[str] = None
 
@@ -413,17 +393,12 @@ class CrossDocEntityRelation(MultiPackLink):
         rel_type (Optional[str]):	The type of the relation.
     """
 
-    rel_type: Optional[str] = None
+    rel_type: Optional[str]
 
     ParentType = EntityMention
     ChildType = EntityMention
 
-    def __init__(
-        self,
-        pack: MultiPack,
-        parent: Optional[Entry] = None,
-        child: Optional[Entry] = None,
-    ):
+    def __init__(self, pack: MultiPack, parent: Optional[Entry] = None, child: Optional[Entry] = None):
         super().__init__(pack, parent, child)
         self.rel_type: Optional[str] = None
 
@@ -436,9 +411,7 @@ class CoreferenceGroup(Group):
 
     MemberType = EntityMention
 
-    def __init__(
-        self, pack: DataPack, members: Optional[Iterable[Entry]] = None
-    ):
+    def __init__(self, pack: DataPack, members: Optional[Iterable[Entry]] = None):
         super().__init__(pack, members)
 
 
@@ -450,17 +423,12 @@ class EventRelation(Link):
         rel_type (Optional[str]):	The type of the relation.
     """
 
-    rel_type: Optional[str] = None
+    rel_type: Optional[str]
 
     ParentType = EventMention
     ChildType = EventMention
 
-    def __init__(
-        self,
-        pack: DataPack,
-        parent: Optional[Entry] = None,
-        child: Optional[Entry] = None,
-    ):
+    def __init__(self, pack: DataPack, parent: Optional[Entry] = None, child: Optional[Entry] = None):
         super().__init__(pack, parent, child)
         self.rel_type: Optional[str] = None
 
@@ -473,17 +441,12 @@ class CrossDocEventRelation(MultiPackLink):
         rel_type (Optional[str]):	The type of the relation.
     """
 
-    rel_type: Optional[str] = None
+    rel_type: Optional[str]
 
     ParentType = EventMention
     ChildType = EventMention
 
-    def __init__(
-        self,
-        pack: MultiPack,
-        parent: Optional[Entry] = None,
-        child: Optional[Entry] = None,
-    ):
+    def __init__(self, pack: MultiPack, parent: Optional[Entry] = None, child: Optional[Entry] = None):
         super().__init__(pack, parent, child)
         self.rel_type: Optional[str] = None
 
@@ -505,17 +468,17 @@ class ConstituentNode(Annotation):
     sentiment: Dict[str, float]
     is_root: Optional[bool]
     is_leaf: Optional[bool]
-    parent_node: Optional["ConstituentNode"]
-    children_nodes: FList["ConstituentNode"]
+    parent_node: Optional['ConstituentNode']
+    children_nodes: FList['ConstituentNode']
 
     def __init__(self, pack: DataPack, begin: int, end: int):
         super().__init__(pack, begin, end)
         self.label: Optional[str] = None
         self.sentiment: Dict[str, float] = dict()
         self.is_root: Optional[bool] = None
         self.is_leaf: Optional[bool] = None
-        self.parent_node: Optional["ConstituentNode"] = None
-        self.children_nodes: FList["ConstituentNode"] = FList(self)
+        self.parent_node: Optional['ConstituentNode'] = None
+        self.children_nodes: FList['ConstituentNode'] = FList(self)
 
 
 @dataclass
@@ -540,6 +503,7 @@ def __init__(self, pack: DataPack, begin: int, end: int):
 
 @dataclass
 class MCOption(Annotation):
+
     def __init__(self, pack: DataPack, begin: int, end: int):
         super().__init__(pack, begin, end)
 
@@ -617,11 +581,9 @@ class AudioPayload(Payload):
         sample_rate (Optional[int]):
     """
 
-    sample_rate: Optional[int] = None
+    sample_rate: Optional[int]
 
-    def __init__(
-        self, pack: DataPack, payload_idx: int = 0, uri: Optional[str] = None
-    ):
+    def __init__(self, pack: DataPack, payload_idx: int = 0, uri: Optional[str] = None):
         super().__init__(pack, payload_idx, uri)
         self.sample_rate: Optional[int] = None
 
@@ -632,9 +594,7 @@ class TextPayload(Payload):
     A payload that caches text data
     """
 
-    def __init__(
-        self, pack: DataPack, payload_idx: int = 0, uri: Optional[str] = None
-    ):
+    def __init__(self, pack: DataPack, payload_idx: int = 0, uri: Optional[str] = None):
         super().__init__(pack, payload_idx, uri)
 
 
@@ -644,7 +604,5 @@ class ImagePayload(Payload):
     A payload that caches image data
     """
 
-    def __init__(
-        self, pack: DataPack, payload_idx: int = 0, uri: Optional[str] = None
-    ):
+    def __init__(self, pack: DataPack, payload_idx: int = 0, uri: Optional[str] = None):
         super().__init__(pack, payload_idx, uri)