Skip to content

convert tabulate data from np.ndarray to tf.Tensor#1657

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

convert tabulate data from np.ndarray to tf.Tensor#1657
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:tabulate_tensor

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Apr 25, 2022

A tensor can be reused in a graph but a numpy array can't. This avoids saving duplicated constants in the case #1423.

A tensor can be reused in a graph but a numpy array can't. This avoids saving duplicated constants in the case deepmodeling#1423.
@njzjz njzjz requested a review from denghuilu April 25, 2022 06:43
@codecov-commenter
Copy link

Codecov Report

Merging #1657 (95a7668) into devel (de7ba72) will decrease coverage by 11.86%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##            devel    #1657       +/-   ##
===========================================
- 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/utils/tabulate.py
deepmd/descriptor/se_t.py
source/op/_gelu.py
source/op/_soft_min_virial_grad.py
deepmd/utils/compat.py
deepmd/utils/data.py
deepmd/infer/deep_pot.py
deepmd/descriptor/se.py
deepmd/entrypoints/transfer.py
deepmd/loss/ener.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...95a7668. Read the comment docs.

@njzjz njzjz requested a review from wanghan-iapcm April 25, 2022 06:50
@wanghan-iapcm wanghan-iapcm merged commit 85a3a0e into deepmodeling:devel Apr 29, 2022
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