You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.
Error with --current_env in mlem build venv
The docs indicate the option is boolean: --current_env: Whether to install in the current virtual env, must be active [default: False]
but:
$ mlem build venv --model lyrics2emoji --current_env
╭─ Error ──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ Option '--current_env' requires an argument. │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
$ mlem build venv --model lyrics2emoji --current_env True
⏳️ Loading model from lyrics2emoji.mlem
💼 Detected the virtual env /Users/aguschin/Git/iterative/mlem/emoji/.venv
💼 Installing the required packages...
......
The text was updated successfully, but these errors were encountered:
mike0sv
changed the title
Error with --current_env in mlem build venv
Support for boolean options as flags (instead of string True/False values)
Jan 19, 2023
For this to work we need to check that field that is the source of cli option has boolean type and add is_flag when we generate the option somewhere here
Need to be careful though because we also might need to alter option name. Eg if is_on has default value False, it makes sense that --is_on will make it True. But if is_on default value is True, we should have something like --not_is_on as an option name
Error with
--current_env
inmlem build venv
The docs indicate the option is boolean:
--current_env: Whether to install in the current virtual env, must be active [default: False]
but:
The text was updated successfully, but these errors were encountered: