Closed MASisserson closed 3 months ago
[!IMPORTANT]
Auto Review Skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository.To trigger a single review, invoke the
@coderabbitai review
command.
The update enhances validation mechanisms in the MLStacks framework to ensure compatibility and correctness in machine learning stack and component configurations. It includes the introduction of new enums, validation functions, and error handling improvements across various modules to manage component types, flavors, and provider matching, aiming to enhance systematization, reliability, and user experience.
Files | Change Summary |
---|---|
src/mlstacks/constants.py |
Added ALLOWED_COMPONENT_TYPES , updated error messages, and MLStacks configuration constants. |
src/mlstacks/enums.py |
Introduced new enums for default flavors, spec types, and versioning. |
src/mlstacks/models/... |
Updated Component and Stack classes with enums and added validation logic. |
src/mlstacks/utils/... |
Expanded validation functions, added utility for testing, and improved YAML loading error handling. |
tests/unit/models/test_component.py |
Modified to use new enums and validation methods. |
tests/unit/utils/... |
Updated tests to use allowed providers and refactored for new validation logic. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@coderabbitai review
Also be sure to take a look at the failing CI. I think something around docstrings that should be easy to fix.
Thanks for the updates / fixes, @MASisserson! I think we're almost ready to merge. Just that one final query / comment above and we should be good to go.
@coderabbitai review
Thanks so much for this contribution, @MASisserson!
Describe changes
I implemented validations to check that stack and component spec files contained valid inputs for
spec_version
,spec_type
,component_type
, andcomponent_flavor
. Also added validation to check that theprovider
listed is the same between stack and components. This should provide more structure for users in the creation of spec documents.Pre-requisites
Please ensure you have done the following:
develop
and the open PR is targetingdevelop
. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.Types of changes
Summary by CodeRabbit
New Features
Refactor
Component
andStack
classes to use enums forspec_version
andspec_type
.Tests