-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Computing Center of Mass #305
Comments
I agree. It's important for (multipole) moments too. We can put the functionality from grid here. Note that it is also a good fit for |
Makes sense. Because it is a relatively light computation, it can be implemented as a read-only property using What would you do when no atomic masses are set? We have tried to avoid including extensive atomic data in |
I think we can include the atomic masses (isotopic average) and the masses of the most common isotope. The alternative is to add a dependency on Note that this touches (indirectly and directly) on several issue we've had with |
It can be useful to have a method in
IOData
class for computing the center of mass. For example, this is useful when computing dipole moment with Grid. What do you think? @tovrstra, @PaulWAyers, and @marco-2023.The text was updated successfully, but these errors were encountered: