search

valohai / valohai-utils

Python helper library for Valohai
MIT License
2 stars 2 forks source link

Duplicate code is needed to properly handle Execution parameters #1

Closed orasimus closed 3 years ago

orasimus commented 6 years ago

Currently when using Execution parameters, you need to define them in the valohai.yaml and then for example using @click or ArgumentParser in Python. This is basically duplicating the same information.

If this could be defined only once, it would improve the user experience and also remove possible errors caused by mismatching parameter handling.

This is based on feedback from Ari (Skillup).

akx commented 6 years ago

Sure. There's https://github.com/valohai/argparse-to-valohai-yaml already; I suppose we should also have the inverse.

It'd probably be a good idea to integrate this in valohai_yaml.

ruksi commented 6 years ago

just food for thought that the fundamental problem is that we don't know if the customer is running Python, C++ or something else, which is the reason it's not integrated to begin with

but yeah, these helper features are nice to have

akx commented 5 years ago

This will be addressed by https://github.com/valohai/valohai-utils.