Skip to content

Commit

Permalink
update katib laucher
Browse files Browse the repository at this point in the history
1. sync up since katib directory org changed
2. add option deleteafterdone
  • Loading branch information
hougangliu committed Apr 9, 2019
1 parent 52d7c7b commit 6797a48
Show file tree
Hide file tree
Showing 5 changed files with 24 additions and 7 deletions.
2 changes: 1 addition & 1 deletion components/kubeflow/katib-launcher/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ RUN apt-get update -y && \
pip install pyyaml==3.12 six==1.11.0 requests==2.18.4 grpcio gcloud google-api-python-client protobuf kubernetes && \
wget https://github.com/kubeflow/katib/archive/master.zip && unzip master.zip

ENV PYTHONPATH $PYTHONPATH:/katib-master/pkg/api/python:/katib-master/py
ENV PYTHONPATH $PYTHONPATH:/katib-master/pkg/api/v1alpha1/python:/katib-master/py

ADD build /ml

Expand Down
4 changes: 4 additions & 0 deletions components/kubeflow/katib-launcher/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
approvers:
- hougangliu
reviewers:
- hougangliu
6 changes: 4 additions & 2 deletions components/kubeflow/katib-launcher/component.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,16 @@ inputs:
- {name: Namespace, type: String, default: kubeflow, description: 'Namespace.'}
- {name: Optimization type, type: String, default: minimize, description: 'Direction of optimization. minimize or maximize.'}
- {name: Objective value name, type: String, description: 'Objective value name which trainer optimizes.'}
- {name: Optimization goal, type: Float, description: 'Stop studying once objectivevaluename value exceeds optimizationgoal'}
- {name: Optimization goal, type: Float, description: 'Stop studying once objectivevaluename value exceeds optimizationgoal.'}
- {name: Request count, type: Integer, default: 1, description: 'Number of requests to the suggestion service.'}
- {name: Metrics names, type: String, description: 'List of metric names (comma-delimited).'}
- {name: Parameter configs, type: YAML, default: '', description: 'Parameter configs (YAML/JSON format).'}
- {name: NAS config, type: YAML, default: '', description: 'NAS config (YAML/JSON format).'}
- {name: Worker template path, type: String, default: '', description: 'Worker spec.'}
- {name: Metrics collector template path, type: String, default: '', description: 'Metrics collector spec.'}
- {name: Suggestion spec, type: YAML, default: '', description: 'Suggestion spec (YAML/JSON format).'}
- {name: StudyJob timeout minutes, type: Integer, default: '10', description: 'Time in minutes to wait for the StudyJob to complete'}
- {name: StudyJob timeout minutes, type: Integer, default: '10', description: 'Time in minutes to wait for the StudyJob to complete.'}
- {name: Delete StudyJob flag, type: String, default: 'True', description: 'When StudyJob done, delete the StudyJob if it is True.'}
outputs:
- {name: Best parameter set, type: JSON, description: 'The parameter set of the best StudyJob trial.'}
implementation:
Expand All @@ -34,5 +35,6 @@ implementation:
--mcollectortemplatepath, {inputValue: Metrics collector template path},
--suggestionspec, {inputValue: Suggestion spec},
--studyjobtimeoutminutes, {inputValue: StudyJob timeout minutes},
--deleteafterdone, {inputValue: Delete StudyJob flag},
--outputfile, {outputPath: Best parameter set},
]
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

def kubeflow_studyjob_launcher_op(name, namespace, optimizationtype, objectivevaluename, optimizationgoal, requestcount, metricsnames,
parameterconfigs, nasConfig, workertemplatepath, mcollectortemplatepath, suggestionspec,
studyjob_timeout_minutes, output_file='/output.txt', step_name='StudyJob-Launcher'):
studyjob_timeout_minutes, delete=True, output_file='/output.txt', step_name='StudyJob-Launcher'):
return dsl.ContainerOp(
name = step_name,
image = 'liuhougangxa/ml-pipeline-kubeflow-studyjob:latest',
Expand All @@ -34,6 +34,7 @@ def kubeflow_studyjob_launcher_op(name, namespace, optimizationtype, objectiveva
"--mcollectortemplatepath", mcollectortemplatepath,
"--suggestionspec", suggestionspec,
"--outputfile", output_file,
"--deleteafterdone", delete,
'--studyjobtimeoutminutes', studyjob_timeout_minutes,
],
file_outputs = {'hyperparameter': output_file}
Expand Down
16 changes: 13 additions & 3 deletions components/kubeflow/katib-launcher/src/launch_study_job.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,14 @@ def yamlOrJsonStr(str):
def strToList(str):
return str.split(",")

def str2bool(v):
if v.lower() in ('yes', 'true', 't', 'y', '1'):
return True
elif v.lower() in ('no', 'false', 'f', 'n', '0'):
return False
else:
raise argparse.ArgumentTypeError('Unsupported value encountered.')

def _update_or_pop(spec, name, value):
if value:
spec[name] = value
Expand Down Expand Up @@ -121,6 +129,9 @@ def main(argv=None):
parser.add_argument('--outputfile', type=str,
default='/output.txt',
help='The file which stores the best trial of the studyJob.')
parser.add_argument('--deleteafterdone', type=str2bool,
default=True,
help='When studyjob done, delete the studyjob automatically if it is True.')
parser.add_argument('--studyjobtimeoutminutes', type=int,
default=10,
help='Time in minutes to wait for the StudyJob to complete')
Expand All @@ -129,7 +140,6 @@ def main(argv=None):

logging.getLogger().setLevel(logging.INFO)


logging.info('Generating studyjob template.')
template_file = os.path.join(os.path.dirname(os.path.realpath(__file__)), 'hp.template.yaml')
content_yaml = _generate_studyjob_yaml(template_file, args.name, args.namespace, args.optimizationtype, args.objectivevaluename,
Expand Down Expand Up @@ -157,8 +167,8 @@ def main(argv=None):
f.write(json.dumps(ps_dict))
if succ:
logging.info('Study success.')

study_job_client.delete_study_job(api_client, job_name, job_namespace)
if args.deleteafterdone:
study_job_client.delete_study_job(api_client, job_name, job_namespace)

if __name__== "__main__":
main()

0 comments on commit 6797a48

Please sign in to comment.