iterative / mlem

🐶 A tool to package, serve, and deploy any ML model on any platform. Archived to be resurrected one day🤞
https://mlem.ai
Apache License 2.0
717 stars 44 forks source link

Unclear bug message #441

Open aguschin opened 2 years ago

aguschin commented 2 years ago

From https://github.com/iterative/mlem/pull/397#discussion_r989115134

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 commented 2 years ago

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