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

Added doc for netsize setting, num_nodes specification, and sel setting in doc/troubleshooting/ #657

Merged
merged 78 commits into from
May 29, 2021

Conversation

tuoping
Copy link
Collaborator

@tuoping tuoping commented May 21, 2021

Created two subsections in troubleshooting/index.md:
"## Trouble shooting"
"## Parameter setting"

tuoping and others added 30 commits April 14, 2021 07:27
Renamed use-deepmd-kit as getting-started, and changed the contents a little.
…xtension sphinx-markdown-tables in conf.py
…her .md file. So I changed all the external links in getting-started.md, and deleted all the #... parts.
tuoping added 6 commits May 12, 2021 16:43
…ex.md; fixed typos in troubleshooting/rcut.md
Added doc about how to specify the num of nodes used by a job.
Added doc for sel setting
Moved rcut.md to howtoset_rcut.md.
Added classify_index_TS in conf.py to make two subsections in troubleshooting/index.md: "## Trouble shooting" and "## Parameter setting"
@amcadmus amcadmus requested a review from njzjz May 21, 2021 11:15
@codecov-commenter
Copy link

codecov-commenter commented May 23, 2021

Codecov Report

Merging #657 (6355903) into devel (363f9ba) will decrease coverage by 7.70%.
The diff coverage is n/a.

❗ Current head 6355903 differs from pull request most recent head e72c624. Consider uploading reports for the commit e72c624 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##            devel     #657      +/-   ##
==========================================
- Coverage   74.37%   66.66%   -7.71%     
==========================================
  Files          81        6      -75     
  Lines        6399       15    -6384     
==========================================
- Hits         4759       10    -4749     
+ Misses       1640        5    -1635     
Impacted Files Coverage Δ
source/op/_prod_force_se_r_grad.py
source/op/_gelu.py
deepmd/env.py
source/op/_soft_min_virial_grad.py
deepmd/op/__init__.py
deepmd/loggers/loggers.py
deepmd/utils/data_system.py
deepmd/entrypoints/train.py
deepmd/descriptor/se_t.py
deepmd/model/tensor.py
... and 65 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 363f9ba...e72c624. Read the comment docs.

@amcadmus amcadmus changed the base branch from devel to master May 25, 2021 00:07
@amcadmus amcadmus changed the base branch from master to devel May 25, 2021 00:07
tuoping and others added 4 commits May 25, 2021 08:21
tuoping and others added 2 commits May 26, 2021 15:22
@amcadmus amcadmus merged commit b3cfe45 into deepmodeling:devel May 29, 2021
This was referenced Jun 10, 2021
denghuilu added a commit to denghuilu/deepmd-kit that referenced this pull request Jun 15, 2021
add Important hint for getting-started.md

Update argcheck.py

add Important hint for variables

Update getting-started.md

check validity of data systems. print help message

add Important hint at getting-start.md (deepmodeling#622)

* add Important hint for getting-started.md

* add hint for some parameters

* add Important hint for variables

* Update argcheck.py

* Update getting-started.md

add doc of type embedding (deepmodeling#625)

Optimized mkindex function in doc/conf.py and added two files in troubleshooting. (deepmodeling#619)

support MPI and other atom_styles for LAMMPS atomic keyword (deepmodeling#628)

* support MPI and other atom_styles for LAMMPS atomic keyword

fix problems left in #44

* move out_each codes together

* indent the code

fix spell mistake (deepmodeling#638)

Atention -> Attention

Readme and Examples for Tensor mode (deepmodeling#632)

* Complete modification of tensor training, support combination of system with global/local label, and support polar label normalization to speed up training. Examples and documentation not added yet

* modify dipole json to pass ut test

* change json file (second time) to pass ut test

* modify test_data_modifier_shuffle.py file to fit new args rule

* modify data_modifier_shuffle: from dipole.npy to atomic_dipole.npy

* modify the name of pref_weight to pref and pref_atomic_weight to pref_atomic, plus some implementation mentioned by Han Wang in May 7th's email

* fix a bug occuring in ut test

* fix args of polar_se_a.json, to pass ut test

* change args: from loss_type to type, so that the args will be the same as ener mode, and will not cause conflict

* add examples and readme for tensor fitting mode in May 14

* change readmd content of tensor fit

* change the file name of readme file of tensor fitting

* Update train-fitting-tensor.md

* Update train-fitting-tensor.md

* Update train-fitting-tensor.md

* change the explanation of why some of lcurve.out is 0

* Update train-fitting-tensor.md

* Update train-fitting-tensor.md

append to out_file when LAMMPS restarts (deepmodeling#640)

This ensures the out file will not be override when LAMMPS restarts.
This commit may be conflicted with deepmodeling#392. Commit
@5597ea2b49f96e99a52a9779b04b6c12e5a79a04 should be dropped.

add an example of C++ inference to doc (deepmodeling#652)

* add an example of C++ inference to doc

* fix broken link

Add instructions for i-PI (deepmodeling#660)

* Add instructions for i-PI

* Update doc/getting-started.md

Co-authored-by: Jinzhe Zeng <[email protected]>

Co-authored-by: tuoping <[email protected]>
Co-authored-by: Jinzhe Zeng <[email protected]>

Added doc for netsize setting, num_nodes specification, and sel setting in doc/troubleshooting/ (deepmodeling#657)

fix issue 668 (deepmodeling#680)

* fix bug of issue 668
gzq942560379 pushed a commit to HPC-AI-Team/deepmd-kit that referenced this pull request Sep 1, 2021
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