Skip to content

Commit

Permalink
chore: remove type embedding TODO from se_r serialize
Browse files Browse the repository at this point in the history
Closes deepmodeling#3554. I just find `se_r` never supports type embedding. It's a mistake in deepmodeling#3338.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
  • Loading branch information
njzjz authored May 31, 2024
1 parent 3a7fbcf commit 3f2f1a2
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions deepmd/tf/descriptor/se_r.py
Original file line number Diff line number Diff line change
Expand Up @@ -765,9 +765,6 @@ def serialize(self, suffix: str = "") -> dict:
raise NotImplementedError("spin is unsupported")
assert self.davg is not None
assert self.dstd is not None
# TODO: tf: handle type embedding in DescrptSeR.serialize
# not sure how to handle type embedding - type embedding is not a model parameter,
# but instead a part of the input data. Maybe the interface should be refactored...
return {
"@class": "Descriptor",
"type": "se_r",
Expand Down

0 comments on commit 3f2f1a2

Please sign in to comment.