search

vcristiani / galaxy-chop

MIT License
5 stars 2 forks source link

test potential energy fixes #126

Closed ornemarioni closed 3 years ago

ornemarioni commented 3 years ago

This PR fixes the potential energy test.

RafaelPignata622 commented 3 years ago

En español es más facil.

Anda todo bien pero me saltan 6 warning, en test_core.py en el test test_type_energy por cada assert salta un warning:

DeprecationWarning: np.float is a deprecated alias for the builtin float. To silence this warning, use float by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use np.float64 here. Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations assert isinstance(p_g, (float, np.float, np.ndarray))

Basicamente que de np.float tiene que ser float directamente o np.float64.

Antes no me saltaba ese warning, a Vale le salta lo mismo, no se si serán los cambios que hiciste que hace que salte el Warning, no es grave pero hay que tenerlo en cuenta.

Por lo demás, todo luce excelente