ADR Suggestion, Input check in setters and flag for skipping checks for performance

[damskii9992]

General

Adding checks for user inputs, and raising appropriate errors with useful error messages on wrong inputs, makes our libraries more user friendly for new users. Checks can slow down performance-sensitive code, so in order to avoid that issue we can have a global flag that skips all checks when set.

Current implementation

Currently, user input checks are made sporadically in our code and with no capability to skip them

Proposed implementation

An example of a setter with checks and a flag to skip them:

@variance.setter
@property_stack_deco
def variance(self, variance_float: float) -> None:
    """
    Set the variance.

    :param variance_float: Variance as a float
    """
    if not UNSAFE:
        if not isinstance(variance_float, numbers.Number):
            raise TypeError(f'{variance_float=} must be a number')
        if variance_float < 0:
            raise ValueError(f'{variance_float=} must be positive')
    self._scalar.variance = variance_float

where UNSAFE is a global boolean which can, for example, be set by our calculators and minimizers.

[rozyczko]

if not UNSAFE: is quite unintuitive.
Wouldn't it be better to just have if SAFE:

[AndrewSazonov]

I would vote for if SAFE. It's way easier to read.