-
Notifications
You must be signed in to change notification settings - Fork 540
feat(pt/tf): init-(frz)-model use pretrain script #3926
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
Conversation
WalkthroughWalkthroughThe recent updates involve extensive enhancements to the Changes
Sequence Diagram(s)sequenceDiagram
participant Tester
participant TempFileHandler
participant DeepPotInstance
participant ModelEvaluator
Tester ->> TempFileHandler: Create temporary files
Tester ->>+ DeepPotInstance: Initialize DeepPot instances
DeepPotInstance -->> Tester: DeepPot instances created
Tester ->> ModelEvaluator: Evaluate input data with DeepPot instances
ModelEvaluator -->> Tester: Evaluation results
Tester ->> DeepPotInstance: Validate assertions
Tester ->> TempFileHandler: Clean up temporary files and directories
Note over Tester, TempFileHandler: Temporary files and directories are managed before and after the tests.
Note over Tester, DeepPotInstance, ModelEvaluator: Multiple DeepPot instances are created and evaluated to ensure consistency and correctness.
Tip AI model upgrade
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## devel #3926 +/- ##
==========================================
- Coverage 82.89% 82.83% -0.06%
==========================================
Files 520 520
Lines 50696 50828 +132
Branches 3015 3011 -4
==========================================
+ Hits 42023 42103 +80
- Misses 7738 7787 +49
- Partials 935 938 +3 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shall we design a test case for this feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Support `--use-pretrain-script` for pt&tf when doing init-(frz)-model. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Tests** - Enhanced and added new test cases for deep learning model initialization and evaluation. - Improved setup and cleanup processes for temporary files and directories in tests to ensure a cleaner test environment. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Support
--use-pretrain-script
for pt&tf when doing init-(frz)-model.Summary by CodeRabbit