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

print MAE for dp test #2310

Merged
merged 2 commits into from
Feb 12, 2023
Merged

print MAE for dp test #2310

merged 2 commits into from
Feb 12, 2023

Conversation

njzjz
Copy link
Member

@njzjz njzjz commented Feb 10, 2023

No description provided.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@codecov-commenter
Copy link

codecov-commenter commented Feb 10, 2023

Codecov Report

Base: 74.89% // Head: 74.59% // Decreases project coverage by -0.31% ⚠️

Coverage data is based on head (309eaa6) compared to base (96d3a1d).
Patch coverage: 6.97% of modified lines in pull request are covered.

❗ Current head 309eaa6 differs from pull request most recent head d548c1d. Consider uploading reports for the commit d548c1d to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2310      +/-   ##
==========================================
- Coverage   74.89%   74.59%   -0.31%     
==========================================
  Files         217      217              
  Lines       21613    21578      -35     
  Branches     1586     1586              
==========================================
- Hits        16187    16096      -91     
- Misses       4401     4458      +57     
+ Partials     1025     1024       -1     
Impacted Files Coverage Δ
deepmd/entrypoints/test.py 10.20% <6.66%> (-0.71%) ⬇️
deepmd/utils/weight_avg.py 19.04% <7.69%> (-0.96%) ⬇️
source/op/add_flt_nvnmd.cc 60.46% <0.00%> (-9.31%) ⬇️
source/op/mul_flt_nvnmd.cc 56.86% <0.00%> (-7.85%) ⬇️
source/op/dotmul_flt_nvnmd.cc 73.52% <0.00%> (-5.89%) ⬇️
source/op/unaggregated_grad.cc 74.22% <0.00%> (-5.61%) ⬇️
source/op/map_flt_nvnmd.cc 83.82% <0.00%> (-4.42%) ⬇️
source/op/matmul_fitnet_nvnmd.cc 81.66% <0.00%> (-3.34%) ⬇️
source/op/flt_nvnmd.cc 85.71% <0.00%> (-2.86%) ⬇️
source/op/matmul_flt_nvnmd.cc 84.41% <0.00%> (-2.60%) ⬇️
... and 18 more

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wanghan-iapcm wanghan-iapcm merged commit 045010f into deepmodeling:devel Feb 12, 2023
dingye18 added a commit to dingye18/deepmd-kit that referenced this pull request Feb 17, 2023
* print MAE for `dp test` (deepmodeling#2310)

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* Fix bug deepmodeling#2311 when using FP32 in se_atten (deepmodeling#2312)

Fix bug deepmodeling#2311 when using FP32 in se_atten

* support multiple frames DeepPot in C/hpp API (deepmodeling#2309)

* Add DeepPot C API v2 to support multiple frames. Also, preserve the
arguments for fparam and aparam for future use (planned in deepmodeling#2236). V1 is
kept for API and ABI compatibility.
* Support multiple frames for DeepPot hpp API. The API compatibility is
kept. Add tests for new situation.

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>

* Implementation of se_a_mask op (deepmodeling#2313)

* Implement se_a_mask op. Training and inference (c++ interface) are
tested. Unit test files are required.

* Fix bug in atom pref setting for forces.

* Add the unit test for descrpt se_a_mask

* Remove the unit test for dp mask model

* Update dp mask system path for unit test

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* support exclude_types for se_atten (deepmodeling#2315)

Fix deepmodeling#2232.

Also, clean old useless codes for `exclude_types`.

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>

* fix restarting from the original model (deepmodeling#2317)

deepmodeling#2253 supports restarting from the compressed model but breaks the
original model. This PR fixes the error by detecting the model type
first.

```
FAILED_PRECONDITION: Attempting to use uninitialized value
```

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>

* package dp_ipi in the PyPI wheels (deepmodeling#2320)

Fix deepmodeling#2282.

In addition, change the return code of `dp_ipi` (without any arguments)
from 1 to 0.

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* build Docker images for each push (deepmodeling#2324)

and push the image to the GitHub container.

Also, provide the pypi CUDA support for pypi LAMMPS.

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>

* Fix typo in install docs (deepmodeling#2325)

Signed-off-by: Chun Cai <amoycaic@gmail.com>

* Add a script to detect model version (deepmodeling#2318)

This PR adds a script in deepmd/utils to detect the version of a Deep
Potential model.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Chun Cai <amoycaic@gmail.com>
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Duo <50307526+iProzd@users.noreply.github.com>
Co-authored-by: Chun Cai <amoycaic@gmail.com>
Co-authored-by: Yifan Li李一帆 <yifanl0716@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants