Remove non-init fields in set_default

Signed-off-by: Fabrice Normandin <fabrice.normandin@gmail.com>

Author: lebrice

simple_parsing/wrappers/dataclass_wrapper.py

@@ -3,7 +3,6 @@
 import argparse
 import dataclasses
 import functools
-import inspect
 import sys
 import textwrap
 from dataclasses import MISSING
@@ -21,10 +20,11 @@
 logger = getLogger(__name__)
 
 MAX_DOCSTRING_DESC_LINES_HEIGHT: int = 50
-"""
-Maximum number of lines of the class docstring to include in the autogenerated argument group
-description. If fields don't have docstrings or help text, then this is not used, and the entire
-docstring is used as the description of the argument group.
+"""Maximum number of lines of the class docstring to include in the autogenerated argument group
+description.
+
+If fields don't have docstrings or help text, then this is not used, and the entire docstring is
+used as the description of the argument group.
 """
 
 DataclassWrapperType = TypeVar("DataclassWrapperType", bound="DataclassWrapper")
@@ -172,7 +172,8 @@ def __init__(
                 # a "normal" attribute
                 field_wrapper = self.field_wrapper_class(field, parent=self, prefix=self.prefix)
                 logger.debug(
-                    f"wrapped field at {field_wrapper.dest} has a default value of {field_wrapper.default}"
+                    f"wrapped field at {field_wrapper.dest} has a default value of "
+                    f"{field_wrapper.default}"
                 )
                 if field_default is not dataclasses.MISSING:
                     field_wrapper.set_default(field_default)
@@ -217,9 +218,12 @@ def add_arguments(self, parser: argparse.ArgumentParser):
     def equivalent_argparse_code(self, leading="group") -> str:
         code = ""
         code += textwrap.dedent(
-            f"""
-        group = parser.add_argument_group(title="{self.title.strip()}", description="{self.description.strip()}")
-        """
+            f"""\
+            group = parser.add_argument_group(
+                title="{self.title.strip()}",
+                description="{self.description.strip()}",
+            )
+            """
         )
         for wrapped_field in self.fields:
             if wrapped_field.is_subparser:
@@ -295,6 +299,11 @@ def set_default(self, value: DataclassT | dict | None):
         self._default = value
         if field_default_values is None:
             return
+        # Ignore default values for fields that have init=False.
+        for field in dataclasses.fields(self.dataclass):
+            if not field.init and field.name in field_default_values:
+                field_default_values.pop(field.name)
+
         unknown_names = set(field_default_values)
         for field_wrapper in self.fields:
             if field_wrapper.name not in field_default_values:
@@ -314,7 +323,10 @@ def set_default(self, value: DataclassT | dict | None):
             unknown_names.remove(nested_dataclass_wrapper.name)
         unknown_names.discard("_type_")
         if unknown_names:
-            raise RuntimeError(f"{sorted(unknown_names)} are not fields of {self.dataclass} at path {self.dest!r}!")
+            raise RuntimeError(
+                f"{sorted(unknown_names)} are not fields of {self.dataclass} at path "
+                f"{self.dest!r}!"
+            )
 
     @property
     def title(self) -> str:
@@ -423,13 +435,16 @@ def destinations(self, value: list[str]):
 
     def merge(self, other: DataclassWrapper):
         """Absorb all the relevant attributes from another wrapper.
+
         Args:
             other (DataclassWrapper): Another instance to absorb into this one.
         """
         # logger.debug(f"merging \n{self}\n with \n{other}")
         logger.debug(f"self destinations: {self.destinations}")
         logger.debug(f"other destinations: {other.destinations}")
-        # assert not set(self.destinations).intersection(set(other.destinations)), "shouldn't have overlap in destinations"
+        # assert not set(self.destinations).intersection(set(other.destinations)), (
+        #     "shouldn't have overlap in destinations"
+        # )
         # self.destinations.extend(other.destinations)
         for dest in other.destinations:
             if dest not in self.destinations: