LoopStructural 1.6.6__py3-none-any.whl → 1.6.8__py3-none-any.whl

LoopStructural/__init__.py

@@ -20,6 +20,7 @@ ch.setLevel(logging.WARNING)
 loggers = {}
 from .modelling.core.geological_model import GeologicalModel
 from .interpolators._api import LoopInterpolator
+from .interpolators import InterpolatorBuilder
 from .datatypes import BoundingBox
 from .utils import log_to_console, log_to_file, getLogger, rng, get_levels
 

LoopStructural/datatypes/_bounding_box.py

@@ -43,7 +43,7 @@ class BoundingBox:
         if maximum is None and nsteps is not None and step_vector is not None:
             maximum = origin + nsteps * step_vector
         if origin is not None and global_origin is None:
-            global_origin = origin
+            global_origin = np.zeros(3)
         self._origin = np.array(origin)
         self._maximum = np.array(maximum)
         self.dimensions = dimensions
@@ -90,7 +90,7 @@ class BoundingBox:
 
     @property
     def global_maximum(self):
-        return self.maximum - self.origin + self._global_origin
+        return self.maximum + self.global_origin
 
     @property
     def valid(self):
@@ -242,6 +242,8 @@ class BoundingBox:
             )
         origin = locations.min(axis=0)
         maximum = locations.max(axis=0)
+        origin = np.array(origin)
+        maximum = np.array(maximum)
         if local_coordinate:
             self.global_origin = origin
             self.origin = np.zeros(3)
@@ -273,15 +275,50 @@ class BoundingBox:
         if self.origin is None or self.maximum is None:
             raise LoopValueError("Cannot create bounding box with buffer, no origin or maximum")
         # local coordinates, rescale into the original bounding boxes global coordinates
-        origin = self.origin - buffer * (self.maximum - self.origin)
-        maximum = self.maximum + buffer * (self.maximum - self.origin)
+        origin = self.origin - buffer * np.max(self.maximum - self.origin)
+        maximum = self.maximum + buffer * np.max(self.maximum - self.origin)
         return BoundingBox(
             origin=origin,
             maximum=maximum,
-            global_origin=self.global_origin + origin,
+            global_origin=self.global_origin,
             dimensions=self.dimensions,
         )
 
+    # def __call__(self, xyz):
+    #     xyz = np.array(xyz)
+    #     if len(xyz.shape) == 1:
+    #         xyz = xyz.reshape((1, -1))
+
+    #     distances = np.maximum(0,
+    #                         np.maximum(self.global_origin+self.origin - xyz,
+    #                                 xyz - self.global_maximum))
+    #     distance = np.linalg.norm(distances, axis=1)
+    #     distance[self.is_inside(xyz)] = -1
+    #     return distance
+
+    def __call__(self, xyz):
+        # Calculate center and half-extents of the box
+        center = (self.maximum + self.global_origin + self.origin) / 2
+        half_extents = (self.maximum - self.global_origin + self.origin) / 2
+
+        # Calculate the distance from point to center
+        offset = np.abs(xyz - center) - half_extents
+
+        # Inside distance: negative value based on the smallest penetration
+        inside_distance = np.min(half_extents - np.abs(xyz - center), axis=1)
+
+        # Outside distance: length of the positive components of offset
+        outside_distance = np.linalg.norm(np.maximum(offset, 0))
+
+        # If any component of offset is positive, we're outside
+        # Otherwise, we're inside and return the negative penetration distance
+        distance = np.zeros(xyz.shape[0])
+        mask = np.any(offset > 0, axis=1)
+        distance[mask] = outside_distance
+        distance[~mask] = -inside_distance[~mask]
+        return distance
+        # return outside_distance if np.any(offset > 0) else -inside_distance
+
     def get_value(self, name):
         ix, iy = self.name_map.get(name, (-1, -1))
         if ix == -1 and iy == -1:
@@ -319,7 +356,7 @@ class BoundingBox:
         self,
         nsteps: Optional[Union[list, np.ndarray]] = None,
         shuffle: bool = False,
-        order: str = "C",
+        order: str = "F",
         local: bool = True,
     ) -> np.ndarray:
         """Get the grid of points from the bounding box
@@ -361,8 +398,8 @@ class BoundingBox:
             rng.shuffle(locs)
         return locs
 
-    def cell_centers(self, order: str = "F") -> np.ndarray:
-        """Get the cell centers of a regular grid
+    def cell_centres(self, order: str = "F") -> np.ndarray:
+        """Get the cell centres of a regular grid
 
         Parameters
         ----------
@@ -372,7 +409,7 @@ class BoundingBox:
         Returns
         -------
         np.ndarray
-            array of cell centers
+            array of cell centres
         """
         locs = self.regular_grid(order=order, nsteps=self.nsteps - 1)
 
@@ -434,7 +471,7 @@ class BoundingBox:
         _cell_data = copy.deepcopy(cell_data)
         _vertex_data = copy.deepcopy(vertex_data)
         return StructuredGrid(
-            origin=self.global_origin,
+            origin=self.global_origin + self.origin,
             step_vector=self.step_vector,
             nsteps=self.nsteps,
             cell_properties=_cell_data,
@@ -460,6 +497,9 @@ class BoundingBox:
             (self.global_maximum - self.global_origin)
         )  # np.clip(xyz, self.origin, self.maximum)
 
+    def scale_by_projection_factor(self, value):
+        return value / np.max((self.global_maximum - self.global_origin))
+
     def reproject(self, xyz):
         """Reproject a point from the bounding box to the global space
 

LoopStructural/datatypes/_point.py

@@ -1,4 +1,4 @@
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 import numpy as np
 
 from typing import Optional, Union
@@ -10,9 +10,9 @@ logger = getLogger(__name__)
 
 @dataclass
 class ValuePoints:
-    locations: np.ndarray
-    values: np.ndarray
-    name: str
+    locations: np.ndarray = field(default_factory=lambda: np.array([[0, 0, 0]]))
+    values: np.ndarray = field(default_factory=lambda: np.array([0]))
+    name: str = "unnamed"
     properties: Optional[dict] = None
 
     def to_dict(self):
@@ -108,9 +108,9 @@ class ValuePoints:
 
 @dataclass
 class VectorPoints:
-    locations: np.ndarray
-    vectors: np.ndarray
-    name: str
+    locations: np.ndarray = field(default_factory=lambda: np.array([[0, 0, 0]]))
+    vectors: np.ndarray = field(default_factory=lambda: np.array([[0, 0, 0]]))
+    name: str = "unnamed"
     properties: Optional[dict] = None
 
     def to_dict(self):
@@ -129,9 +129,9 @@ class VectorPoints:
     def vtk(
         self,
         geom='arrow',
-        scale=0.10,
+        scale=1.0,
         scale_function=None,
-        normalise=True,
+        normalise=False,
         tolerance=0.05,
         bb=None,
         scalars=None,
@@ -140,9 +140,15 @@ class VectorPoints:
 
         _projected = False
         vectors = np.copy(self.vectors)
+
         if normalise:
             norm = np.linalg.norm(vectors, axis=1)
             vectors[norm > 0, :] /= norm[norm > 0][:, None]
+        else:
+            norm = np.linalg.norm(vectors, axis=1)
+            vectors[norm > 0, :] /= norm[norm > 0][:, None]
+            norm = norm[norm > 0] / norm[norm > 0].max()
+            vectors *= norm[:, None]
         if scale_function is not None:
             # vectors /= np.linalg.norm(vectors, axis=1)[:, None]
             vectors *= scale_function(self.locations)[:, None]
@@ -151,6 +157,7 @@ class VectorPoints:
             try:
                 locations = bb.project(locations)
                 _projected = True
+                scale = bb.scale_by_projection_factor(scale)
             except Exception as e:
                 logger.error(f'Failed to project points to bounding box: {e}')
                 logger.error('Using unprojected points, this may cause issues with the glyphing')
@@ -161,10 +168,10 @@ class VectorPoints:
         if geom == 'arrow':
             geom = pv.Arrow(scale=scale)
         elif geom == 'disc':
-            geom = pv.Disc(inner=0, outer=scale).rotate_y(90)
+            geom = pv.Disc(inner=0, outer=scale * 0.5, c_res=50).rotate_y(90)
 
         # Perform the glyph
-        glyphed = points.glyph(orient="vectors", geom=geom, tolerance=tolerance, scale=False)
+        glyphed = points.glyph(orient="vectors", geom=geom, tolerance=tolerance)
         if _projected:
             glyphed.points = bb.reproject(glyphed.points)
         return glyphed

LoopStructural/datatypes/_structured_grid.py

@@ -1,6 +1,6 @@
 from typing import Dict
 import numpy as np
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 from LoopStructural.utils import getLogger
 
 logger = getLogger(__name__)
@@ -8,12 +8,12 @@ logger = getLogger(__name__)
 
 @dataclass
 class StructuredGrid:
-    origin: np.ndarray
-    step_vector: np.ndarray
-    nsteps: np.ndarray
-    cell_properties: Dict[str, np.ndarray]
-    properties: Dict[str, np.ndarray]
-    name: str
+    origin: np.ndarray = field(default_factory=lambda: np.array([0, 0, 0]))
+    step_vector: np.ndarray = field(default_factory=lambda: np.array([1, 1, 1]))
+    nsteps: np.ndarray = field(default_factory=lambda: np.array([10, 10, 10]))
+    cell_properties: Dict[str, np.ndarray] = field(default_factory=dict)
+    properties: Dict[str, np.ndarray] = field(default_factory=dict)
+    name: str = "default_grid"
 
     def to_dict(self):
         return {
@@ -44,9 +44,9 @@ class StructuredGrid:
             z,
         )
         for name, data in self.properties.items():
-            grid[name] = data.flatten(order="F")
+            grid[name] = data.reshape((grid.n_points, -1), order="F")
         for name, data in self.cell_properties.items():
-            grid.cell_data[name] = data.flatten(order="F")
+            grid.cell_data[name] = data.reshape((grid.n_cells, -1), order="F")
         return grid
 
     def plot(self, pyvista_kwargs={}):
@@ -63,6 +63,34 @@ class StructuredGrid:
         except ImportError:
             logger.error("pyvista is required for vtk")
 
+    @property
+    def cell_centres(self):
+        x = np.linspace(
+            self.origin[0] + self.step_vector[0] * 0.5,
+            self.maximum[0] + self.step_vector[0] * 0.5,
+            self.nsteps[0] - 1,
+        )
+        y = np.linspace(
+            self.origin[1] + self.step_vector[1] * 0.5,
+            self.maximum[1] - self.step_vector[1] * 0.5,
+            self.nsteps[1] - 1,
+        )
+        z = np.linspace(
+            self.origin[2] + self.step_vector[2] * 0.5,
+            self.maximum[2] - self.step_vector[2] * 0.5,
+            self.nsteps[2] - 1,
+        )
+        x, y, z = np.meshgrid(x, y, z, indexing="ij")
+        return np.vstack([x.flatten(order='f'), y.flatten(order='f'), z.flatten(order='f')]).T
+
+    @property
+    def nodes(self):
+        x = np.linspace(self.origin[0], self.maximum[0], self.nsteps[0])
+        y = np.linspace(self.origin[1], self.maximum[1], self.nsteps[1])
+        z = np.linspace(self.origin[2], self.maximum[2], self.nsteps[2])
+        x, y, z = np.meshgrid(x, y, z, indexing="ij")
+        return np.vstack([x.flatten(order='f'), y.flatten(order='f'), z.flatten(order='f')]).T
+
     def merge(self, other):
         if not np.all(np.isclose(self.origin, other.origin)):
             raise ValueError("Origin of grids must be the same")

LoopStructural/datatypes/_surface.py

@@ -1,4 +1,4 @@
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 from typing import Optional
 import numpy as np
 import io
@@ -9,8 +9,8 @@ logger = getLogger(__name__)
 
 @dataclass
 class Surface:
-    vertices: np.ndarray
-    triangles: np.ndarray
+    vertices: np.ndarray = field(default_factory=lambda: np.array([[0, 0, 0]]))
+    triangles: np.ndarray = field(default_factory=lambda: np.array([[0, 0, 0]]))
     normals: Optional[np.ndarray] = None
     name: str = 'surface'
     values: Optional[np.ndarray] = None

LoopStructural/export/geoh5.py

@@ -78,8 +78,10 @@ def add_structured_grid_to_geoh5(filename, structured_grid, overwrite=True, grou
         if structured_grid.cell_properties is not None:
             for k, v in structured_grid.cell_properties.items():
                 data[k] = {
-                    'association': "CELL",
-                    "values": np.rot90(v.reshape(structured_grid.nsteps - 1, order="F")).flatten(),
+                    "association": "CELL",
+                    "values": np.flipud(
+                        np.rot90(v.reshape(structured_grid.nsteps - 1, order="F"), 1)
+                    ).flatten(),
                 }
         block = geoh5py.objects.BlockModel.create(
             workspace,

LoopStructural/interpolators/__init__.py

@@ -115,4 +115,5 @@ support_interpolator_map = {
 
 from ._interpolator_factory import InterpolatorFactory
 from ._interpolator_builder import InterpolatorBuilder
+
 # from ._api import LoopInterpolator

LoopStructural/interpolators/_discrete_interpolator.py

@@ -63,6 +63,20 @@ class DiscreteInterpolator(GeologicalInterpolator):
         logger.info("Creating discrete interpolator with {} degrees of freedom".format(self.nx))
         self.type = InterpolatorType.BASE_DISCRETE
 
+    def set_nelements(self, nelements: int) -> int:
+        return self.support.set_nelements(nelements)
+
+    @property
+    def n_elements(self) -> int:
+        """Number of elements in the interpolator
+
+        Returns
+        -------
+        int
+            number of elements, positive
+        """
+        return self.support.n_elements
+
     @property
     def nx(self) -> int:
         """Number of degrees of freedom for the interpolator
@@ -161,6 +175,7 @@ class DiscreteInterpolator(GeologicalInterpolator):
         """
         self.constraints = {}
         self.c_ = 0
+        self.regularisation_scale = np.ones(self.nx)
         logger.info("Resetting interpolation constraints")
 
     def add_constraints_to_least_squares(self, A, B, idc, w=1.0, name="undefined"):
@@ -737,3 +752,6 @@ class DiscreteInterpolator(GeologicalInterpolator):
             **super().to_dict(),
             # 'region_function':self.region_function,
         }
+
+    def vtk(self):
+        return self.support.vtk({'c': self.c})

LoopStructural/interpolators/_finite_difference_interpolator.py

@@ -7,12 +7,37 @@ import numpy as np
 from ..utils import get_vectors
 from ._discrete_interpolator import DiscreteInterpolator
 from ..interpolators import InterpolatorType
-
+from scipy.spatial import KDTree
 from LoopStructural.utils import getLogger
 
 logger = getLogger(__name__)
 
 
+def compute_weighting(grid_points, gradient_constraint_points, alpha=10.0, sigma=1.0):
+    """
+    Compute weights for second derivative regularization based on proximity to gradient constraints.
+
+    Parameters:
+        grid_points (ndarray): (N, 3) array of 3D coordinates for grid cells.
+        gradient_constraint_points (ndarray): (M, 3) array of 3D coordinates for gradient constraints.
+        alpha (float): Strength of weighting increase.
+        sigma (float): Decay parameter for Gaussian-like influence.
+
+    Returns:
+        weights (ndarray): (N,) array of weights for each grid point.
+    """
+    # Build a KDTree with the gradient constraint locations
+    tree = KDTree(gradient_constraint_points)
+
+    # Find the distance from each grid point to the nearest gradient constraint
+    distances, _ = tree.query(grid_points, k=1)
+
+    # Compute weighting function (higher weight for nearby points)
+    weights = 1 + alpha * np.exp(-(distances**2) / (2 * sigma**2))
+
+    return weights
+
+
 class FiniteDifferenceInterpolator(DiscreteInterpolator):
     def __init__(self, grid, data={}):
         """
@@ -44,6 +69,7 @@ class FiniteDifferenceInterpolator(DiscreteInterpolator):
         )
 
         self.type = InterpolatorType.FINITE_DIFFERENCE
+        self.use_regularisation_weight_scale = False
 
     def setup_interpolator(self, **kwargs):
         """
@@ -76,20 +102,19 @@ class FiniteDifferenceInterpolator(DiscreteInterpolator):
         for key in kwargs:
             self.up_to_date = False
             if "regularisation" in kwargs:
-                self.interpolation_weights["dxy"] = 0.1 * kwargs["regularisation"]
-                self.interpolation_weights["dyz"] = 0.1 * kwargs["regularisation"]
-                self.interpolation_weights["dxz"] = 0.1 * kwargs["regularisation"]
-                self.interpolation_weights["dxx"] = 0.1 * kwargs["regularisation"]
-                self.interpolation_weights["dyy"] = 0.1 * kwargs["regularisation"]
-                self.interpolation_weights["dzz"] = 0.1 * kwargs["regularisation"]
+                self.interpolation_weights["dxy"] = kwargs["regularisation"]
+                self.interpolation_weights["dyz"] = kwargs["regularisation"]
+                self.interpolation_weights["dxz"] = kwargs["regularisation"]
+                self.interpolation_weights["dxx"] = kwargs["regularisation"]
+                self.interpolation_weights["dyy"] = kwargs["regularisation"]
+                self.interpolation_weights["dzz"] = kwargs["regularisation"]
             self.interpolation_weights[key] = kwargs[key]
         # either use the default operators or the ones passed to the function
         operators = kwargs.get(
             "operators", self.support.get_operators(weights=self.interpolation_weights)
         )
-        for k, o in operators.items():
-            self.assemble_inner(o[0], o[1], name=k)
 
+        self.use_regularisation_weight_scale = kwargs.get('use_regularisation_weight_scale', False)
         self.add_norm_constraints(self.interpolation_weights["npw"])
         self.add_gradient_constraints(self.interpolation_weights["gpw"])
         self.add_value_constraints(self.interpolation_weights["cpw"])
@@ -101,6 +126,8 @@ class FiniteDifferenceInterpolator(DiscreteInterpolator):
             upper_bound=kwargs.get('inequality_pair_upper_bound', np.finfo(float).eps),
             lower_bound=kwargs.get('inequality_pair_lower_bound', -np.inf),
         )
+        for k, o in operators.items():
+            self.assemble_inner(o[0], o[1], name=k)
 
     def copy(self):
         """
@@ -271,6 +298,11 @@ class FiniteDifferenceInterpolator(DiscreteInterpolator):
             self.add_constraints_to_least_squares(A, B, idc[inside, :], w=w, name="gradient")
             A = np.einsum("ij,ijk->ik", dip_vector.T, T)
             self.add_constraints_to_least_squares(A, B, idc[inside, :], w=w, name="gradient")
+            # self.regularisation_scale += compute_weighting(
+            #     self.support.nodes,
+            #     points[inside, : self.support.dimension],
+            #     sigma=self.support.nsteps[0] * 10,
+            # )
             if np.sum(inside) <= 0:
                 logger.warning(
                     f" {np.sum(~inside)} \
@@ -318,7 +350,24 @@ class FiniteDifferenceInterpolator(DiscreteInterpolator):
             )
             # T*=np.product(self.support.step_vector)
             # T/=self.support.step_vector[0]
-
+            # indexes, inside2 = self.support.position_to_nearby_cell_indexes(
+            # points[inside, : self.support.dimension]
+            # )
+            # indexes = indexes[inside2, :]
+
+            # corners = self.support.cell_corner_indexes(indexes)
+            # node_indexes = corners.reshape(-1, 3)
+            # indexes = self.support.global_node_indices(indexes)
+            # self.regularisation_scale[indexes]  =10
+
+            self.regularisation_scale += compute_weighting(
+                self.support.nodes,
+                points[inside, : self.support.dimension],
+                sigma=self.support.nsteps[0] * 10,
+            )
+            # global_indexes = self.support.neighbour_global_indexes().T.astype(int)
+            # close_indexes =
+            # self.regularisation_scale[global_indexes[idc[inside,:].astype(int),]]=10
             w /= 3
             for d in range(self.support.dimension):
 
@@ -454,7 +503,11 @@ class FiniteDifferenceInterpolator(DiscreteInterpolator):
             a[inside, :],
             B[inside],
             idc[inside, :],
-            w=w,
+            w=(
+                self.regularisation_scale[idc[inside, 13].astype(int)] * w
+                if self.use_regularisation_weight_scale
+                else w
+            ),
             name=name,
         )
         return

LoopStructural/interpolators/_geological_interpolator.py

@@ -46,6 +46,15 @@ class GeologicalInterpolator(metaclass=ABCMeta):
         self.dimensions = 3  # default to 3d
         self.support = None
 
+    @abstractmethod
+    def set_nelements(self, nelements: int) -> int:
+        pass
+
+    @property
+    @abstractmethod
+    def n_elements(self) -> int:
+        pass
+
     @property
     def data(self):
         return self._data

LoopStructural/interpolators/_interpolator_builder.py

@@ -1,55 +1,134 @@
 from LoopStructural.interpolators import (
-    GeologicalInterpolator,
     InterpolatorFactory,
     InterpolatorType,
 )
 from LoopStructural.datatypes import BoundingBox
-from typing import Optional, Union
+from typing import  Union, Optional
 import numpy as np
 
+from LoopStructural.interpolators._geological_interpolator import GeologicalInterpolator
+
 
 class InterpolatorBuilder:
     def __init__(
         self,
         interpolatortype: Union[str, InterpolatorType],
         bounding_box: BoundingBox,
-        nelements: int = 1000,
-        buffer: float = 0.2,
+        nelements: Optional[int] = None,
+        buffer: Optional[float] = None,
         **kwargs,
     ):
+        """This class helps initialise and setup a geological interpolator.
+
+        Parameters
+        ----------
+        interpolatortype : Union[str, InterpolatorType]
+            type of interpolator
+        bounding_box : BoundingBox
+            bounding box of the area to interpolate
+        nelements : int, optional
+            degrees of freedom of the interpolator, by default 1000
+        buffer : float, optional
+            how much of a buffer around the bounding box should be used, by default 0.2
+        """
         self.interpolatortype = interpolatortype
         self.bounding_box = bounding_box
         self.nelements = nelements
         self.buffer = buffer
         self.kwargs = kwargs
-        self.interpolator : Optional[GeologicalInterpolator]= None
-
-    def create_interpolator(self) -> 'InterpolatorBuilder':
         self.interpolator = InterpolatorFactory.create_interpolator(
-            interpolatortype=self.interpolatortype,
-            boundingbox=self.bounding_box,
-            nelements=self.nelements,
-            buffer=self.buffer,
-            **self.kwargs,
-        )
-        return self
+                    interpolatortype=self.interpolatortype,
+                    boundingbox=self.bounding_box,
+                    nelements=self.nelements,
+                    buffer=self.buffer,
+                    **self.kwargs,
+                )
 
-    def set_value_constraints(self, value_constraints: np.ndarray) -> 'InterpolatorBuilder':
+    def add_value_constraints(self, value_constraints: np.ndarray) -> 'InterpolatorBuilder':
+        """Add value constraints to the interpolator
+
+        Parameters
+        ----------
+        value_constraints : np.ndarray
+            x,y,z,value of the constraints
+
+        Returns
+        -------
+        InterpolatorBuilder
+            reference to the builder 
+        """
         if self.interpolator:
             self.interpolator.set_value_constraints(value_constraints)
         return self
 
-    def set_gradient_constraints(self, gradient_constraints: np.ndarray) -> 'InterpolatorBuilder':
+    def add_gradient_constraints(self, gradient_constraints: np.ndarray) -> 'InterpolatorBuilder':
+        """Add gradient constraints to the interpolator
+        Where g1 and g2 are two vectors that are orthogonal to the gradient
+        $'(X)\cdot g1 = 0$ and $'(X)\cdot g2 = 0$
+
+        Parameters
+        ----------
+        gradient_constraints : np.ndarray
+            x,y,z,gradient_x,gradient_y,gradient_z of the constraints
+
+        Returns
+        -------
+        InterpolatorBuilder
+            reference to the builder
+        """
+
         if self.interpolator:
             self.interpolator.set_gradient_constraints(gradient_constraints)
         return self
 
-    def set_normal_constraints(self, normal_constraints: np.ndarray) -> 'InterpolatorBuilder':
+    def add_normal_constraints(self, normal_constraints: np.ndarray) -> 'InterpolatorBuilder':
+        """Add normal constraints to the interpolator
+        Where n is the normal vector to the surface
+        $f'(X).dx = nx$
+        $f'(X).dy = ny$
+        $f'(X).dz = nz$
+        Parameters
+        ----------
+        normal_constraints : np.ndarray
+            x,y,z,nx,ny,nz of the constraints
+
+        Returns
+        -------
+        InterpolatorBuilder
+            reference to the builder
+        """
         if self.interpolator:
             self.interpolator.set_normal_constraints(normal_constraints)
         return self
+    #TODO implement/check inequalities
+    # def add_inequality_constraints(self, inequality_constraints: np.ndarray) -> 'InterpolatorBuilder':
+    #     if self.interpolator:
+    #         self.interpolator.set_value_inequality_constraints(inequality_constraints)
+    #     return self
+    # def add_inequality_pair_constraints(self, inequality_pair_constraints: np.ndarray) -> 'InterpolatorBuilder':
+    #     if self.interpolator:
+    #         self.interpolator.set_inequality_pairs_constraints(inequality_pair_constraints)
+    #     return self
+
+    def setup_interpolator(self, **kwargs) -> 'InterpolatorBuilder':
+        """This adds all of the constraints to the interpolator and
+        sets the regularisation constraints
 
-    def setup_interpolator(self, **kwargs) -> Optional[GeologicalInterpolator]:
+        Returns
+        -------
+        InterpolatorBuilder
+            reference to the builder
+        """
         if self.interpolator:
             self.interpolator.setup(**kwargs)
-            return self.interpolator
+        return self
+
+    def build(self)->GeologicalInterpolator:
+        """Builds the interpolator and returns it
+
+        Returns
+        -------
+        GeologicalInterpolator
+            The interpolator fitting all of the constraints provided
+        """
+        return self.interpolator