Should we add a global seed for all transformations?
SirRob1997 opened this issue · comments
Almost all transformations such as, for example, butter_fingers_perturbation
or replace_numerical_values
use a seed
in their constructor that is set to some value. How are we going to handle the global seed? we could easily set one in initialize.py
that get's imported in each transformation and set that as the default, similar to what is currently done for spacy_nlp
. Otherwise, we can also set it during evaluation, as far as I could tell that is not currently done but I think having a global default is a little cleaner.
Happy to make the required changes if that's something we'd want.
It would be great to have such a facility. I think it's best it is set in initilialize.py
. Feel free to create a PR! :)
CC: @aadesh11