-
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
[AGENT-6081] Fix class label files not working for binary models #3
Conversation
.github/workflows/docker-image.yml
Outdated
@@ -3,6 +3,8 @@ name: Docker Image CI | |||
on: | |||
push: | |||
branches: [ "main" ] | |||
paths: | |||
- environments/java_codegen_monitoring/Makefile |
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.
Can you pull the version number out into a file with just the version in it. It's easy then to read it in from the Makefile. This way changes to the Makefile itself won't trigger a re-publish accidentally.
@@ -2,7 +2,7 @@ SHELL = /bin/sh | |||
|
|||
NS := datarobot-oss | |||
IMAGE_NAME := mlops-sap-monitoring-scoring-code | |||
VERSION := 1.0.0 | |||
VERSION := 1.0.1 |
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.
If this was split into a dedicated file, I'm pretty sure you can use a shell call with cat to slurp in the version.
Update start_server.sh to only include Class label files for multiclass deployments.
This was causing to fail for binary classification since it does not require this file.
*Update tests cases to cover this, and actually run the docker image with different target type
1.0
so no changes neede there.