Hi metaflow team i tried adding local decorator su...
# dev-metaflow
n
Hi metaflow team i tried adding local decorator support in the following PR. I updated it after initial discussion, please review https://github.com/Netflix/metaflow/pull/1824 thanks
d
@square-wire-39606 — could you provide feedback here. I think we discussed a bit already.
s
@numerous-holiday-13767 can you help me with your use case for
@local
here?
i can also follow up on the GH PR
n
hi @square-wire-39606..i was learning metaflow and wanted to contribute to the project....so i went through good first issues....and found this issue interesting which i have mentioned in the PR... the use case defined in that PR which seemed interesting to me was following
Copy code
A possible use-case for this would be to annotate a GPU-heavy step that you want to run locally to save on AWS costs but take advantage of fast S3 access for other steps (from the EC2 machines batch uses).
so i tried implementing the solution. Not sure if my approach is right or not.
s
got it. before we merge this PR, we would need to align on the best UX we can offer. it is likely that offering a plain @local decorator might not be the best approach here and may likely lead to more confusion for the users. if you are interested in contributing, i would recommend starting out with PRs that don't introduce (or tweak) the current UX for quick progress - like this one or this one