-
Notifications
You must be signed in to change notification settings - Fork 2
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
[#611] refactoring toolchain entity #647
base: develop
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #647 +/- ##
===========================================
+ Coverage 62.44% 70.40% +7.96%
===========================================
Files 241 140 -101
Lines 13398 6487 -6911
Branches 370 370
===========================================
- Hits 8366 4567 -3799
+ Misses 4440 1790 -2650
+ Partials 592 130 -462
Flags with carried forward coverage won't be shown. Click here to find out more.
|
4c4ee21
to
85074d5
Compare
0a9c263
to
e6a8b64
Compare
@@ -305,11 +305,11 @@ function local_setup() { | |||
'.spec.uri=$uri | .spec.username=$username | .spec.password=$password' "${LOCAL_TEST_DATA}/odahuflow/dir/docker-pull-target.json" | jq "." | sponge "${LOCAL_TEST_DATA}/odahuflow/dir/docker-pull-target.json" | |||
|
|||
## docker image tags | |||
local ti_version="$(jq -r .mlflow_toolchain_version "${CLUSTER_PROFILE}")" | |||
local ti_version="$(jq -r .mlflow_training_integration "${CLUSTER_PROFILE}")" |
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.
This should be changed in cluster profiles too. Just a reminder if it's already done.
toolchain: | ||
description: IntegrationName of toolchain | ||
trainingIntegration: | ||
description: IntegrationName of trainer |
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.
description: IntegrationName of trainer | |
description: Training integration id |
@@ -64,8 +64,8 @@ type AlgorithmSource struct { | |||
type ModelTrainingSpec struct { | |||
// Model Identity | |||
Model ModelIdentity `json:"model"` | |||
// IntegrationName of toolchain | |||
Toolchain string `json:"toolchain"` | |||
// IntegrationName of trainer |
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.
// IntegrationName of trainer | |
// Training integration id |
|
||
IntegrationName of toolchain # noqa: E501 | ||
IntegrationName of trainer # noqa: E501 |
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.
IntegrationName of trainer # noqa: E501 | |
trainingIntegration id # noqa: E501 |
|
||
IntegrationName of toolchain # noqa: E501 | ||
IntegrationName of trainer # noqa: E501 |
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.
IntegrationName of trainer # noqa: E501 | |
trainingIntegration id # noqa: E501 |
issue #611