njzjz / deepmd-gnn

DeePMD-kit plugin for various graph neural network models
GNU Lesser General Public License v3.0
21 stars 3 forks source link

support serialize and deserialize #10

Closed njzjz closed 2 months ago

pep8speaks commented 2 months ago

Hello @njzjz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 727:80: E501 line too long (84 > 79 characters) Line 740:80: E501 line too long (80 > 79 characters)

Comment last updated at 2024-09-05 20:44:52 UTC
codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Project coverage is 90.56%. Comparing base (c9811f8) to head (cee9bdc). Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
deepmd_mace/mace.py 85.71% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #10 +/- ## ========================================== + Coverage 89.45% 90.56% +1.11% ========================================== Files 4 4 Lines 256 265 +9 ========================================== + Hits 229 240 +11 + Misses 27 25 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.