Romain

Results 115 comments of Romain

If you want to keep the functionality, the proper line should read something like: ``` type=click.Choice(["none", UBF_CONTROL, UBF_TASK]), ``` I was under the impression that ubf may be used by...

@savingoyal -- if you are fine with this as well, we can merge. lgtm.

Waiting a tad -- poetry 1.8.0 seems to have some issues being worked our around their support for PEP 658 (lazy-wheel).

As a quick update -- I have it working (updated to 1.8.2) except for the hash override part. I am still working on this and also trying to support markers...

I added an initial PR. It's not done yet but hopefully gives you a scope of the work (it wasn't too bad).

@savingoyal -- could you comment on this one (you had some comments)

Mergeable anytime from my end -- no impact on core.

haven't looked yet in detail but I think we should name it `environment` to keep consistency with the extension.

I mean, it would be exposed to the user and there is similarity with this: https://github.com/Netflix/metaflow-nflx-extensions/blob/main/metaflow_extensions/netflix_ext/plugins/environment_cli.py so I wanted to try to minimize (more) divergence. There is also an argument...

> I remember asking @savingoyal if we wanna add this at the `metaflow conda` level or perhaps `python flow.py conda` level. The latter was probably chosen because we kind of...