🔧 fix wrongful instance variable assignments (#198)

Author: sebastianMindee

mindee/parsing/custom/list.py

@@ -9,7 +9,7 @@ class ListFieldValueV1(FieldPositionMixin):
 
     content: str
     """The content text"""
-    confidence: float = 0.0
+    confidence: float
     """Confidence score"""
 
     def __init__(self, raw_prediction: StringDict) -> None:
@@ -24,7 +24,7 @@ def __str__(self) -> str:
 class ListFieldV1:
     """A list of values or words."""
 
-    confidence: float = 0.0
+    confidence: float
     """Confidence score"""
     reconstructed: bool
     """Whether the field was reconstructed from other fields."""

mindee/parsing/standard/amount.py

@@ -7,7 +7,7 @@
 class AmountField(FieldPositionMixin, BaseField):
     """A field containing an amount value."""
 
-    value: Optional[float] = None
+    value: Optional[float]
     """The amount value as a float."""
 
     def __init__(
@@ -23,6 +23,7 @@ def __init__(
         :param reconstructed: Bool for reconstructed object (not extracted in the API)
         :param page_id: Page number for multi-page document
         """
+        self.value = None
         super().__init__(
             raw_prediction,
             value_key="value",

mindee/parsing/standard/base.py

@@ -25,29 +25,31 @@ def _set_position(self, raw_prediction: StringDict):
             pass
         if self.polygon:
             self.bounding_box = get_bounding_box(self.polygon)
+        else:
+            self.bounding_box = None
 
 
 class FieldConfidenceMixin:
     """Mixin class to add a confidence score."""
 
-    confidence: float = 0.0
+    confidence: float
     """The confidence score."""
 
     def _set_confidence(self, raw_prediction: StringDict):
         try:
             self.confidence = float(raw_prediction["confidence"])
         except (KeyError, TypeError):
-            pass
+            self.confidence = 0.0
 
 
 class BaseField(FieldConfidenceMixin):
     """Base class for most fields."""
 
-    value: Optional[Any] = None
+    value: Optional[Any]
     """Raw field value"""
     reconstructed: bool
     """Whether the field was reconstructed from other fields."""
-    page_id: Optional[int] = None
+    page_id: Optional[int]
     """The document page on which the information was found."""
 
     def __init__(
@@ -57,6 +59,8 @@ def __init__(
         reconstructed: bool = False,
         page_id: Optional[int] = None,
     ):
+        self.value = None
+        self.confidence = 0.0
         """
         Base field object.
 

mindee/parsing/standard/date.py

@@ -13,9 +13,9 @@
 class DateField(FieldPositionMixin, BaseField):
     """A field containing a date value."""
 
-    date_object: Optional[date] = None
+    date_object: Optional[date]
     """Date as a standard Python ``datetime.date`` object."""
-    value: Optional[str] = None
+    value: Optional[str]
     """The raw field value."""
 
     def __init__(
@@ -49,3 +49,7 @@ def __init__(
             except (TypeError, ValueError):
                 self.date_object = None
                 self.confidence = 0.0
+        else:
+            self.date_object = None
+            self.confidence = 0.0
+            self.value = None

mindee/parsing/standard/locale.py

@@ -7,11 +7,11 @@
 class LocaleField(BaseField):
     """The locale detected on the document."""
 
-    language: Optional[str] = None
+    language: Optional[str]
     """The ISO 639-1 code of the language."""
-    country: Optional[str] = None
+    country: Optional[str]
     """The ISO 3166-1 alpha-2 code of the country."""
-    currency: Optional[str] = None
+    currency: Optional[str]
     """The ISO 4217 code of the currency."""
 
     def __init__(

mindee/parsing/standard/payment_details.py

@@ -7,13 +7,13 @@
 class PaymentDetailsField(FieldPositionMixin, BaseField):
     """Information on a single payment."""
 
-    account_number: Optional[str] = None
+    account_number: Optional[str]
     """Account number"""
-    iban: Optional[str] = None
+    iban: Optional[str]
     """Account IBAN"""
-    routing_number: Optional[str] = None
+    routing_number: Optional[str]
     """Account routing number"""
-    swift: Optional[str] = None
+    swift: Optional[str]
     """Bank's SWIFT code"""
 
     def __init__(

mindee/parsing/standard/position.py

@@ -10,15 +10,15 @@
 class PositionField(BaseField):
     """A field indicating a position or area on the document."""
 
-    value: Optional[Polygon] = None
+    value: Optional[Polygon]
     """Polygon of cropped area, identical to the ``polygon`` property."""
-    polygon: Optional[Polygon] = None
+    polygon: Optional[Polygon]
     """Polygon of cropped area"""
-    quadrangle: Optional[Quadrilateral] = None
+    quadrangle: Optional[Quadrilateral]
     """Quadrangle of cropped area (does not exceed the canvas)"""
-    rectangle: Optional[Quadrilateral] = None
+    rectangle: Optional[Quadrilateral]
     """Oriented rectangle of cropped area (may exceed the canvas)"""
-    bounding_box: Optional[Quadrilateral] = None
+    bounding_box: Optional[Quadrilateral]
     """Straight rectangle of cropped area (does not exceed the canvas)"""
 
     def __init__(

mindee/parsing/standard/text.py

@@ -7,7 +7,7 @@
 class StringField(FieldPositionMixin, BaseField):
     """A field containing a text value."""
 
-    value: Optional[str] = None
+    value: Optional[str]
 
     def __init__(
         self,
@@ -24,6 +24,7 @@ def __init__(
         :param reconstructed: Bool for reconstructed object (not extracted in the API)
         :param page_id: Page number for multi-page document
         """
+        self.value = None
         super().__init__(
             raw_prediction,
             value_key=value_key,