Pb/add relative stochastic perturbators

pablobd commented 1 year ago

In this PR, I am adding:

RelativePositivePerturbator: a refactored version of a multiplicative perturbator originally developed by @DaniGate in a dev branch
NormalPerturbator: additive stochastic perturbator, it perturbs by sampling from a Normal distribution
BetaRelativePositivePerturbator: a fancy perturbator, multiplicative and stochastic, sampling from a Beta distribution between (ranged between 0 and 1). The beta is conveniently reparametrized with the mean and scale.

100% of diff lines are tested having an effect of increasing coverage above 98%.

pablobd commented 1 year ago

LGTM. I'm wondering if it would be worth adding a parameter so that every perturbator could be used both as "additive" and as "relative", without the need to define two separate classes.

It's better to keep classes small and focused on what they should do

david26694 commented 1 year ago

Some minor changes, looks great

david26694 / cluster-experiments

Pb/add relative stochastic perturbators #110