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.

Sign up for GitHub

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

Jump to bottom

Fix MLCube configure silently failing #494

Merged
merged 53 commits into from
Oct 11, 2023
Merged

Fix MLCube configure silently failing #494

merged 53 commits into from
Oct 11, 2023

Conversation

aristizabal95
Copy link
Contributor

This PR ensures errors while configuring an MLCube are not ignored, and instead they raise an exception.

@aristizabal95
Add Data Preparator cookiecutter template
28f7e7f
@aristizabal95
Rename cookiecutter folder
6f9e19e
@aristizabal95
Temporarily remove possibly offending files
df6e6a2
@aristizabal95
Remove cookicutter conditionals
a7db0cf
@aristizabal95
Inclube back missing pieces of template
a7a6d15
@aristizabal95
remove cookiecutter typo
e2f7108
@aristizabal95
Use project_name attribute
581b5bb
@aristizabal95
Change cookiecutter fields order
fd77804
@aristizabal95
Create empty directories on hook
6eebd59
@aristizabal95
Fix empty folders paths
5ef86a2
@aristizabal95
Create evaluator mlcube cookiecutter template
d04baf8
@aristizabal95
Fix JSON Syntax Error
02cec01
@aristizabal95
Update template default values
b3d7a1d
@aristizabal95
Remove reference to undefined template variable
7338236
@aristizabal95
Implement model mlcube cookiecutter template
d1cec5e
@aristizabal95
Update cookiecutter variable default values
7338755
@aristizabal95
Create medperf CLI command for creating MLCubes
3ae9226
@aristizabal95
Provide additional options for mlcube create
e07cde2
@aristizabal95
Start working on tests
68e136a
@aristizabal95
Add tests for cube create
b8e03ac
@aristizabal95
Ignore invalid syntax on cookiecutter conditionals
7896b25
@aristizabal95
Ignore more flake8 errors
4f78981
@aristizabal95
Remove unused import
f5dab5e
@aristizabal95
Empty commit for cloudbuild
a03d7f6
@aristizabal95
Fix inconsistency with labels paths
6bb60d0
@aristizabal95
Update mlcube.yaml so it can be commented on docs
43b6cab
@aristizabal95
Don't render noqa comments on template
55b5d22
@aristizabal95
Remove flake8 specific ignores
135c598
@aristizabal95
Exclude templates from lint checks
e9e2c32
@aristizabal95
Remove specific flake8 ignores
e95dab8
@aristizabal95
Fix labels_paht being passed in he wrong situation
d059b7a
@aristizabal95
Add requirements to cookiecutters
fcdaa7b
@aristizabal95
Set separate labels as true by default
37f3f3c
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
b45fdb9
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
fbf02b4
@aristizabal95
Remove duplicate templates
7a33c23
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
e9a1190
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
f2ff354
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
a682021
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
50592ee
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
31c6bbf
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
69ae2ed
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
8a47ad9
@aristizabal95
Abstract field-error dict formatting
5f32e50
@aristizabal95
Reformat errors dictionary for printing
3951af8
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
09b8d69
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
e7a8ae4
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
091b69c
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
c69bf4c
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
d0c2b77
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
c70bb9b
@aristizabal95
Merge branch 'main' of https://github.com/mlcommons/medperf
90b5cb3
@aristizabal95
Raise an error if mlcube configure fails
27b57d6
@aristizabal95 aristizabal95 requested a review from a team as a code owner October 10, 2023 15:50
@aristizabal95 aristizabal95 temporarily deployed to testing-external-code October 10, 2023 15:50 — with GitHub Actions Inactive
@github-actions
Copy link
Contributor

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

msheller
msheller approved these changes Oct 10, 2023
View reviewed changes
Copy link

@msheller msheller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and simple. Looks good.

@aristizabal95 aristizabal95 merged commit f69feeb into mlcommons:main Oct 11, 2023
8 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@msheller msheller msheller approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aristizabal95 @msheller