- Notifications
You must be signed in to change notification settings - Fork 3
Description
Hello,
I noticed the arguments for _maxTpermutation() from permutationTesting.py includes the tuple [diff_arr, nullmean, tail, seed] as our input.
Tuple unpacking is no longer compatible with python3, but the code still works in python2. I developed, however, a workaround for this fantastic script to run in python3:
Old code:
def _maxTpermutation((diff_arr,nullmean,tail,seed)): """ Helper function to perform a single permutation """ np.random.seed(seed) New code:
def _maxTpermutation(t_tuple: tuple): -> int: """ Helper function to perform a single permutation """ diff_arr = t_tuple[0] nullmean = t_tuple[1] tail = t_tuple[2] seed = t_tuple[3] np.random.seed(seed) I understand one solution could be to pass individual arguments, but our inputs is of variable length and incorporating type annotations allows your list of inputs to successfully pass through map_async(). Therefore, this solution should remain faithful to the logic of the original script.
I was able to successfully run pythonMaxT_Test.py with the new code. Hope this helps! I can submit a pull request for permutationTesting.py too, if you like.