-
Notifications
You must be signed in to change notification settings - Fork 81
Multigrid documentation #171
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
base: master
Are you sure you want to change the base?
Conversation
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.
Other than a suggestion and a few typos, looks good to me.
evaluation functions offered by the `numint` module, the Multigrid algorithm | ||
can achieve an order of magnitude improvement in the computation of the Coulomb | ||
matrix and DFT XC matrix. | ||
|
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.
Maybe just add 1-2 sentences in the introduction to describe the theory/method underlying multigrid?
|
||
With this `Cell` object, we can initialize the DFT instance as we would for | ||
typical PBC DFT calculations. The following example demonstrates another | ||
way to utlize the Multigrid algorithm, whihc integrates |
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.
utlize -> utilize and whihc -> which
|
||
* The `MultiGridNumInt` class does not support the calculation of analytical nuclear gradients. | ||
|
||
* The `MultiGridNumInt2` class does not support non-orthogonal lattices, k-point |
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.
non-orthogonal cells are supported
No description provided.