Clean up pylint warnings and fix doc issues

Fixes include:
- Copying config files from tests to docs
- Refactor sparse matrix aggregation code
- Add docstrings
- Refactor surface writing functions
- Fix minor issue causing numpy to complain

setup.py

@@ -27,10 +27,18 @@
 REQUIREMENTS_DOCS = setupx.parse_requirements("requirements/requirements_docs.txt")
 REQUIREMENTS_EXTRAS = {"tests": REQUIREMENTS_TESTS, "docs": REQUIREMENTS_DOCS}
 
-HC_FUNCTION = "grid3d_hc_thickness=xtgeoapp_grd3dmaps.avghc.grid3d_hc_thickness:main"
-AVG_FUNCTION = "grid3d_average_map=xtgeoapp_grd3dmaps.avghc.grid3d_average_map:main"
-AGG_FUNCTION = "grid3d_aggregate_map=xtgeoapp_grd3dmaps.avghc.grid3d_aggregate_map:main"
-MIG_FUNCTION = "grid3d_migration_time=xtgeoapp_grd3dmaps.avghc.grid3d_migration_time:main"
+HC_FUNCTION = (
+    "grid3d_hc_thickness=xtgeoapp_grd3dmaps.avghc.grid3d_hc_thickness:main"
+)
+AVG_FUNCTION = (
+    "grid3d_average_map=xtgeoapp_grd3dmaps.avghc.grid3d_average_map:main"
+)
+AGG_FUNCTION = (
+    "grid3d_aggregate_map=xtgeoapp_grd3dmaps.aggregate.grid3d_aggregate_map:main"
+)
+MIG_FUNCTION = (
+    "grid3d_migration_time=xtgeoapp_grd3dmaps.aggregate.grid3d_migration_time:main"
+)
 
 
 def src(anypath):

src/xtgeoapp_grd3dmaps/aggregate/_config.py

@@ -1,3 +1,4 @@
+# pylint: disable=missing-class-docstring
 """
 Configuration for the `aggregate` module. Starting from `RootConfig`, it is possible to
 deduce mandatory and optional parameters, as well as default values for whatever is not
@@ -9,6 +10,9 @@
 
 
 class AggregationMethod(Enum):
+    """
+    Enum representing the available aggregation methods for `grid3d_aggregate_map`
+    """
     MAX = "max"
     MIN = "min"
     MEAN = "mean"
@@ -65,6 +69,7 @@ class ComputeSettings:
 
     def __post_init__(self):
         if isinstance(self.aggregation, str):
+            # pylint: disable=no-member
             self.aggregation = AggregationMethod(self.aggregation.lower())
         if self.all is False and self.zone is False:
             raise ValueError(
@@ -74,6 +79,7 @@ def __post_init__(self):
 
 @dataclass
 class MapSettings:
+    # pylint: disable=too-many-instance-attributes
     xori: Optional[float] = None
     xinc: Optional[float] = None
     yori: Optional[float] = None