-
Notifications
You must be signed in to change notification settings - Fork 67
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
Prior: D2 dispersive correction #143
Conversation
raimis
commented
Oct 26, 2022
•
edited
edited
- Implement the D2 dispersive correction
- Tests
- Documentation
Conflicts: torchmdnet/priors/__init__.py
This will be a really useful feature to have. Can you use the same system as in #134, where the dataset provides |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't help much as I don't really know what this dispersive correction is. I leave it to Peter to review
@peastman any more comments? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Aside from the one last change to the docstring, I think it's ready to merge.