Closed Zatyrus closed 2 years ago
Hey, I just saw you already closed this.
Just in case you were wondering why the tests are failing, I think this is because you're not assigning values to self._msd_error
.
Also, apparently, the results of the new MSD calculation do not seem to closely match the ones of the old calculation. It would probably be a good idea to compare the results of both variants manually. If the new method looks good enough I can just update the tests.
Yeah, I saw too late that I totally forgot about the msd_error. Will add it tomorrow.
Added the new MSD calculation including garbage collection in
trait2d/analysis/__init__.py.
Also renamed
Diffusion.display_trajectory
toDiffusion.plot_trajectory
for easier naming convention. For this please see that the analysis scripty already use the name'plot_trajectory'.