Annanilsson-code / ProjectAppliedMolBiophys

Project in the course Applied Molecular Biophysics 2020
1 stars 1 forks source link

No need to double convert and store in list #1

Closed FilipeMaia closed 3 years ago

FilipeMaia commented 3 years ago

You are converting to an image in https://github.com/Annanilsson-code/ProjectAppliedMolBiophys/blob/4a0481cb27c25e07f177952885e9cbbe65aa52aa/cc_calculation.py#L52 and back to an array in https://github.com/Annanilsson-code/ProjectAppliedMolBiophys/blob/4a0481cb27c25e07f177952885e9cbbe65aa52aa/cc_calculation.py#L60-L61

There's no need to do that. You could just use the original noise_ellipse. You could also store things in a 3D array with shape (n, noise_ellipse.shape[0], noise_ellipse.shape[1]).

FilipeMaia commented 3 years ago

Fixed by d89b742.