[MNT] Replace assertions in networks module with raising Value errors

[hadifawaz1999]

Describe the issue

In all networks module, the parameters handled as input lists have an assertion for the length of the list, but assertions are not a good idea, they should be replaced by raise ValueError message

Suggest a potential alternative/fix

Example

instead of:

assert len(self.strides) == self.n_conv_per_residual_block

It should be :

if len(self.strides) != self.n_conv_per_residual_block:
    raise ValueError(
        f"Number of strides {len(self.strides)} should be"
        f" the same as number of convolution layers per block but is"
        f" not: {self.n_conv_per_residual_block}."
    )

Additional context

No response

[hadifawaz1999]

reserved issue

[hadifawaz1999]

Aeon-Assign bot assign @JjmWeber

[MatthewMiddlehurst]

@aeon-actions-bot assign @JjmWeber

[MatthewMiddlehurst]

@aeon-actions-bot assign @hadifawaz1999

[MatthewMiddlehurst]

Ok looks like it wont let you add unrelated people to issues. Probably been abused in the past 🙂

[hadifawaz1999]

yeah still requires a comment to be able to assign

[JjmWeber]

So here is my comment 😉