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
nvnmd-v1 with 31-type chemical species #2676
Conversation
…ap input, check rang os switch function
for more information, see https://pre-commit.ci
…to devel nvnmd mapt flt ntype
Conflicting files "source/tests/test_nvnmd_se_a.py" and "source/tests/test_nvnmd_utils.py" have been removed. Because the same unit tests are included in "source/tests/test_nvnmd_entrypoints.py". |
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.
Is it possible to further reduce the size of the weight files weight_v?_cnn.npy
for testing? E.g. using smaller models for tests.
You need to resolve conflicts in your branch. |
for more information, see https://pre-commit.ci
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## devel #2676 +/- ##
==========================================
+ Coverage 78.36% 78.92% +0.55%
==========================================
Files 235 236 +1
Lines 24506 25157 +651
Branches 1469 1477 +8
==========================================
+ Hits 19204 19855 +651
- Misses 4910 4912 +2
+ Partials 392 390 -2
☔ View full report in Codecov by Sentry. |
@MoPinghui if your pr is ready for review, plz click the re-request bottoms besides the reviewers. |
se_atten
descriptor and 31-type chemical species. nvnmd-v0 is the old version withse_a
descriptor and 4-type chemical species.