-
Notifications
You must be signed in to change notification settings - Fork 1.1k
feat(#369): Implement Comprehensive Email Error Reporting System for Training Process #376
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
Open
PStarH
wants to merge
16
commits into
mindverse:master
Choose a base branch
from
PStarH:error_reminder
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rse#353) * Join AI Network -> Export your Second Me * Default Synthesis Mode -> high Default Epoch -> 3 * Set Thinking-Mode Default Value * Better Display Of ReadMe * default value of thinking mode * Set Default value of enableL0Retrival to false
* optimize doc * better code --------- Co-authored-by: kevinaimonster <kevinaimonster@gmail.com>
* feat: file display and deletion related * fix: file saving logic
Co-authored-by: kevinaimonster <kevinaimonster@gmail.com>
…erse#360) * Provide a selection of Chinese mirror websites for Huggingface * Remove default values
* feat:add current file to model download progress * git ignore yarn * feat: show current downloading --------- Co-authored-by: Ye Xiangle <yexiangle@mail.mindverse.ai> Co-authored-by: kevinaimonster <kevinaimonster@gmail.com>
* feature: add China image configuration options * feature: update Python and Poetry mirror configurations to use Tsinghua and Aliyun * feature: improve Poetry mirror configuration to check for existing sources * feature: rename setup-china to setup-cn and update pip mirror to Aliyun * Add MD file * Modify GitHub download * Change to Chinese Mainland --------- Co-authored-by: yanmuyuan <2216646664@qq.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Email Error Reporting System Implementation
Overview
This PR implements a comprehensive email error reporting system for the training process. The system is designed to monitor various stages of model training and send detailed error notifications when issues occur, enabling quick problem detection and resolution.
System Architecture
Email Service Implementation
EmailServiceclass to handle all email notificationsMajor Features
1. Model Download Stage
_download_modelmethod:2. Weight Merging Stage
merge_weightsmethod:3. Model Conversion Stage
convert_modelmethod:Error Notification Features
Content Structure
Each error notification email includes:
Smart Log Handling
Testing
Automated Tests
Implemented unit tests for:
Scenario Testing
The following scenarios have been tested:
All scenarios correctly send error notification emails with complete information.
Impact
This new email notification system will:
Future Enhancements
Potential future improvements: