Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Unclear bug message #441

Open
aguschin opened this issue Oct 17, 2022 · 1 comment
Open

Unclear bug message #441

aguschin opened this issue Oct 17, 2022 · 1 comment
Labels
cli MLEM command-line interface p2-medium Medium priority ux Things that matter for user experience

Comments

@aguschin
Copy link
Contributor

aguschin commented Oct 17, 2022

From #397 (comment)

Unclear bug message in release/0.3.0:

$ mlem declare env docker --registry local --daemon.host localhost docker-builder.mlem                         124ms
💾 Saving env to docker-builder.mlem
$ mlem build docker --file_conf docker-builder --model rf.mlem --image.name=rf-docker                         6.809s
❌ Unexpected error: not enough values to unpack (expected 2, got 1)
Please report it here: <https://github.com/iterative/mlem/issues>

Should be something like

File provided to `file_conf` option must have lines of form "arg=value"
@aguschin aguschin added ux Things that matter for user experience cli MLEM command-line interface labels Oct 17, 2022
@aguschin
Copy link
Contributor Author

Btw, this should not be --file_conf. It should be --load. Let's fix that as well.

@aguschin aguschin mentioned this issue Oct 27, 2022
18 tasks
@aguschin aguschin added the p1-high High priority label Nov 8, 2022
@omesser omesser added p2-medium Medium priority and removed p1-high High priority labels Mar 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cli MLEM command-line interface p2-medium Medium priority ux Things that matter for user experience
Projects
None yet
Development

No branches or pull requests

2 participants