Skip to content
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

[Feature Request] Move backend-independent utility codes to deepmd_utils #3118

Closed
23 tasks done
Tracked by #3122
njzjz opened this issue Jan 9, 2024 · 0 comments
Closed
23 tasks done
Tracked by #3122
Milestone

Comments

@njzjz
Copy link
Member

njzjz commented Jan 9, 2024

Summary

Move backend-independent utility codes in the deepmd package to the deepmd_utils package so other backends can use them.

Detailed Description

Compatibility should be kept.

Move without modification

  • move utility to deepmd_utils (without modifaction) #3140
    • deepmd.common.add_data_requirement
    • deepmd.common.select_idx_map
    • deepmd.common.j_must_have
    • deepmd.common.j_loader
    • deepmd.common.expand_sys_str
    • deepmd.common.get_np_precision
    • deepmd.utils.argcheck
    • deepmd.utils.compat
    • deepmd.utils.data
    • deepmd.utils.data_system
    • deepmd.utils.pair_tab
    • deepmd.utils.path
    • deepmd.utils.plugin
    • deepmd.utils.random
    • deepmd.utils.weight_avg
    • deepmd.model.model_stat
    • deepmd.loggers

Move with minor modification

Further Information, Files, and Links

No response

@njzjz njzjz added this to the v3.0.0 milestone Jan 9, 2024
wanghan-iapcm pushed a commit that referenced this issue Jan 13, 2024
Move framework-independent codes to the `deepmd_utils` module without
modification, as a step of
#3118.

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
njzjz added a commit to njzjz/deepmd-kit that referenced this issue Jan 13, 2024
See deepmodeling#3118 and dptech-corp/deepmd-pytorch#137.
Subclass needs to implement `is_gpu_available` and `is_oom_error`.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
wanghan-iapcm pushed a commit that referenced this issue Jan 14, 2024
See #3118 and
dptech-corp/deepmd-pytorch#137. Subclass needs
to implement `is_gpu_available` and `is_oom_error`.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
wanghan-iapcm pushed a commit that referenced this issue Jan 14, 2024
…i} (#3144)

See #3118

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
wanghan-iapcm pushed a commit that referenced this issue Feb 9, 2024
Fix #3118.

Note: head is not supported yet in DeepEval

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz closed this as completed Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

1 participant