Skip to content
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

add init-bulk in dpgen documents #907

Merged
merged 10 commits into from
Aug 28, 2022
Merged

add init-bulk in dpgen documents #907

merged 10 commits into from
Aug 28, 2022

Conversation

baihuyu12
Copy link
Contributor

No description provided.

Signed-off-by: baihuyu12 <[email protected]>
Signed-off-by: baihuyu12 <[email protected]>
Signed-off-by: baihuyu12 <[email protected]>
Signed-off-by: baihuyu12 <[email protected]>
Signed-off-by: baihuyu12 <[email protected]>
Signed-off-by: baihuyu12 <[email protected]>
This was referenced Aug 22, 2022
Signed-off-by: baihuyu12 <[email protected]>
Signed-off-by: baihuyu12 <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Aug 23, 2022

Codecov Report

Merging #907 (4771a0c) into devel (5dc7c1b) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            devel     #907      +/-   ##
==========================================
+ Coverage   38.14%   38.24%   +0.09%     
==========================================
  Files          99       99              
  Lines       17767    17795      +28     
==========================================
+ Hits         6778     6806      +28     
  Misses      10989    10989              
Impacted Files Coverage Δ
dpgen/generator/arginfo.py 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@wanghan-iapcm wanghan-iapcm requested a review from njzjz August 24, 2022 01:55
Comment on lines 58 to 79
Key | Type | Example | Description |
| :---------------- | :--------------------- | :-------------------------------------- | :-------------------------------------------------------------|
| **stages** | List of Integer | [1,2,3,4] | Stages for `init_bulk`
| **Elements** | List of String | ["Mg"] | Atom types
| cell_type | String | "hcp" | Specifying which typical structure to be generated. **Options** include fcc, hcp, bcc, sc, diamond.
| latt | Float | 4.479 | Lattice constant for single cell.
| from_poscar | Boolean | True | Deciding whether to use a given poscar as the beginning of relaxation. If it's true, keys (`cell_type`, `latt`) will be aborted. Otherwise, these two keys are **necessary**.
| from_poscar_path | String | "....../C_mp-47_conventional.POSCAR" | Path of POSCAR for VASP or STRU for ABACUS. **Necessary** if `from_poscar` is true.
| relax_incar | String | "....../INCAR" | Path of INCAR for VASP or INPUT for ABACUS for relaxation in VASP. **Necessary** if `stages` include 1.
| md_incar | String | "....../INCAR_md" | Path of INCAR for VASP or INPUT for ABACUS for MD in VASP. **Necessary** if `stages` include 3.|
| **scale** | List of float | [0.980, 1.000, 1.020] | Scales for transforming cells.
| **skip_relax** | Boolean | False | If it's true, you may directly run stage 2 (perturb and scale) using an unrelaxed POSCAR.
| **pert_numb** | Integer | 30 | Number of perturbations for each POSCAR.
| **pert_box** | Float | 0.03 | Percentage of Perturbation for cells.
| **pert_atom** | Float | 0.01 | Perturbation of each atoms (Angstrom).
| **md_nstep** | Integer | 10 | Steps of AIMD in stage 3. If it's not equal to settings via `NSW` in `md_incar`, DP-GEN will follow `NSW`.
| **coll_ndata** | Integer | 5000 | Maximal number of collected data.
| type_map | List | [ "Mg", "Al"] | The indices of elements in deepmd formats will be set in this order.
| init_fp_style | String | "ABACUS" or "VASP" | First-principle software. If this key is absent, the default value will be "VASP".
| relax_kpt | String | "....../KPT" | Path of `KPT` file for relaxation in stage 1. Only useful if `init_fp_style` is "ABACUS".
| md_kpt | String | "....../KPT" | Path of `KPT` file for MD simulations in stage 3. Only useful if `init_fp_style` is "ABACUS".
| atom_masses | List of float | [24] | List of atomic masses of elements. The order should be the same as `Elements`. Only useful if `init_fp_style` is "ABACUS".
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This table should be generated automatically by dargs. But I don't request it done by this PR.
cc @HuangJiameng

Signed-off-by: baihuyu12 <[email protected]>
"from_poscar": true,
"from_poscar_path": "....../C_mp-47_conventional.POSCAR",
```
`init_bulk` support both VASP and ABACUS for first-principle calculation. You can choose the software by specifying the key `init_fp_style`. If `init_fp_style` is not specified, the default software will be VASP.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

support -> supports

Signed-off-by: baihuyu12 <[email protected]>
@AnguseZhang
Copy link
Collaborator

Screen Shot 2022-08-28 at 11 30 56 PM
Han and I both confronted such strange problem. We don't know how to solve it currently. Does it works fine in your place? @njzjz If so, please merge the PR directly. Otherwise, we may create a new PR. Thanks.

@njzjz njzjz merged commit cdf6c68 into deepmodeling:devel Aug 28, 2022
@njzjz
Copy link
Member

njzjz commented Aug 28, 2022

I have the same issue but I can merge it on the app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants