You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'd like to get rid of unyt.physical_constants.Msun and unyt.physical_constants.msun and add a unyt.unit_symbols.msun but I think that might break code...
The text was updated successfully, but these errors were encountered:
There's also Solar_Mass which in some ways addresses this issue. It would be nice to define unyt.msun as 1 * unyt.Solar_Mass (or as an alias for it, actually).
P.S. There is also solar_mass which is inconsistent with Solar_Mass:
In [1]: unyt.solar_mass
Out[1]: unyt_quantity(1.98841586e+30, 'kg')
In [2]: unyt.Solar_Mass
Out[2]: Msun
This is weird:
It was also confusing in the real world.
I'd like to get rid of
unyt.physical_constants.Msun
andunyt.physical_constants.msun
and add aunyt.unit_symbols.msun
but I think that might break code...The text was updated successfully, but these errors were encountered: