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] Support dp show for TF models #4019

Closed
njzjz opened this issue Jul 24, 2024 · 1 comment · Fixed by #4131
Closed

[Feature Request] Support dp show for TF models #4019

njzjz opened this issue Jul 24, 2024 · 1 comment · Fixed by #4131

Comments

@njzjz
Copy link
Member

njzjz commented Jul 24, 2024

Summary

Support dp show for TF models.

Detailed Description

It's now a PT-only subcommand.

Further Information, Files, and Links

https://docs.deepmodeling.com/projects/deepmd/en/latest/cli.html#show

@njzjz
Copy link
Member Author

njzjz commented Sep 6, 2024

Perhaps it should be more general - move everything to DeepEval.

njzjz added a commit to njzjz/deepmd-kit that referenced this issue Sep 16, 2024
@njzjz njzjz linked a pull request Sep 16, 2024 that will close this issue
github-merge-queue bot pushed a commit that referenced this issue Sep 18, 2024
Fix #4019.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

## Release Notes

- **New Features**
- Introduced a `show` command to display model attributes via the
command line.
- Added a method to retrieve the model definition script for enhanced
model access and usability.
- Enhanced model configuration options in YAML files, allowing for
detailed specifications of architecture and training parameters.

- **Bug Fixes**
- Simplified help messages for the `show` command, improving user
clarity.

- **Tests**
- Added new tests to validate the retrieval of model definition scripts,
ensuring accuracy in model configurations.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: Jinzhe Zeng <[email protected]>
@njzjz njzjz closed this as completed Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant