@QB3 can you check that the new way to do the stratified split behaves the same? Btw having mutable arguments as default (here, stratified shuffle split) is a bad design, as they are instanciated only once (stored in my_function.__defaults__) and then modifying them inside the functions modifies the default value)
QB3
commented
3 years ago
closes #112
It remains to add docstrings
@QB3 can you check that the new way to do the stratified split behaves the same? Btw having mutable arguments as default (here, stratified shuffle split) is a bad design, as they are instanciated only once (stored in
my_function.__defaults__) and then modifying them inside the functions modifies the default value)