-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: refactor metrics and switch to ruff #64
Merged
Merged
Conversation
This file contains 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
Coverage Report •
|
This was
linked to
issues
Jan 22, 2025
Closed
- Added new dependencies: matplotlib, colorama, and ipython for improved visualization and interactivity. - Updated cloud training documentation to reflect changes in training job initiation and monitoring. - Refactored training status retrieval to use a new `train_info` method, replacing the previous `train_status`. - Introduced a `metrics` method in the RemoteModel class to fetch training metrics. - Enhanced the `notebook_monitor_train` method to support real-time monitoring and plotting of training metrics. - Fixed typos in class names and improved code consistency across files.
This was
unlinked from
issues
Jan 24, 2025
Cosa cambia lato users? Solo la parte di metriche, giusto? Abbiamo aggiornato la doc in modo che sia allineata? |
fcdl94
approved these changes
Jan 24, 2025
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.
Enhance Model Training Monitoring and Integrate Ruff
CONTEXT
This PR was created to modernize the development toolchain and improve the model training monitoring capabilities. The existing linting tools were fragmented, and the model training lacked comprehensive monitoring and visualization features.
KEY CHANGES
Development Tools
Model Training & Monitoring
MetricsVisualizer
class for enhanced training metrics visualization.RemoteModel
class to improve training monitoring.notebook_monitor_train
method for real-time training monitoring in Jupyter notebooks.anyma_version
parameter from model training configuration.Code Quality & Structure
MetricsVisualizer
.IMPACT
TECHNICAL DETAILS
Development Tools Configuration
Updated
.pre-commit-config.yaml
to use RuffNew MetricsVisualizer Implementation
The
MetricsVisualizer
class provides methods for logging and visualizing training metrics:RemoteModel Enhancements
Refactored
RemoteModel
class to improve monitoring:Key Dependencies Updates
opencv-python
,matplotlib
,colorama
,ipython
isort
,black
,flake8
These changes aim to streamline the development process and enhance the model training experience with better monitoring and visualization tools.