Skip to content

reset the graph before freezing the compressed model#1658

Merged
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:reset_graph
Apr 29, 2022
Merged

reset the graph before freezing the compressed model#1658
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:reset_graph

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Apr 25, 2022

A graph has a size limitation of 2 GB, so if we do not clean the graph, the limitation will be indeed 1GB...

This PR will increase the limitation from 1 GB to 2 GB.

A graph has a size limitation of 2 GB, so if we do not clean the graph, the limitation will be indeed 1GB...
@codecov-commenter
Copy link

Codecov Report

Merging #1658 (57dbfca) into devel (de7ba72) will decrease coverage by 11.86%.
The diff coverage is n/a.

❗ Current head 57dbfca differs from pull request most recent head 8e1895d. Consider uploading reports for the commit 8e1895d to get more accurate results

@@             Coverage Diff             @@
##            devel    #1658       +/-   ##
===========================================
- Coverage   76.15%   64.28%   -11.87%     
===========================================
  Files          94        5       -89     
  Lines        7850       14     -7836     
===========================================
- Hits         5978        9     -5969     
+ Misses       1872        5     -1867     
Impacted Files Coverage Δ
deepmd/entrypoints/compress.py
deepmd/loss/ener.py
deepmd/utils/data.py
deepmd/cluster/local.py
deepmd/utils/path.py
deepmd/infer/deep_polar.py
source/op/_prod_virial_se_a_grad.py
deepmd/env.py
deepmd/entrypoints/train.py
source/op/_prod_force_grad.py
... and 79 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 de7ba72...8e1895d. Read the comment docs.

@wanghan-iapcm wanghan-iapcm merged commit d1fa9e9 into deepmodeling:devel Apr 29, 2022
@njzjz njzjz deleted the reset_graph branch June 16, 2022 19:08
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