Conversation
for more information, see https://pre-commit.ci
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## devel #511 +/- ##
==========================================
+ Coverage 82.77% 82.95% +0.17%
==========================================
Files 71 73 +2
Lines 6428 6493 +65
==========================================
+ Hits 5321 5386 +65
Misses 1107 1107
☔ View full report in Codecov by Sentry. |
dpdata/dftb_plus/inout.py
Outdated
| energy = float(s[2]) | ||
|
|
||
| symbols = np.array(symbols) | ||
| forces = -np.array(forces) |
There was a problem hiding this comment.
It seems that DFTB+ outputs forces instead of gradients, so forces should not multiply by -1.
(need check)
dpdata/dftb_plus/inout.py
Outdated
| import numpy as np | ||
|
|
||
|
|
||
| def read_dftb_plus(fn: str) -> Tuple[str, np.ndarray, float, np.ndarray]: |
There was a problem hiding this comment.
DFTB+ has input and output files separately, so I suggest not merging them into one file. Instead, pass a directory or two file arguments.
wanghan-iapcm
left a comment
wanghan-iapcm
left a comment
There was a problem hiding this comment.
Please provide doc for this format.
| Geometry = GenFormat { | ||
| 4 C | ||
| N H | ||
| 1 1 1.014150 0.112320 0.047370 | ||
| 2 2 3.909390 0.037985 -0.101159 | ||
| 3 2 0.702550 -0.851820 -0.060860 | ||
| 4 2 0.702550 0.603740 -0.789160 | ||
| } |
There was a problem hiding this comment.
From dftb+ documentation
The second line contains the chemical symbols of the elements present separated by one or more spaces. The following n lines contain a list of the atoms. The first number is the atom number in the structure (this is currently ignored by the program). The second number is the chemical type from the list of symbols on line 2. Then follow the coordinates. For S and C format, these are x, y, z in Å, but for F they are fractions of the three lattice vectors.
So the unit is Angstrom
… rename dftb_plus to dftbplus
|
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Please update dftb+ format to readme:
Lines 54 to 89 in 85a3b5e
3 participants
No description provided.