New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sample: flow in pipeline #2620

Merged

elliotzh merged 32 commits into main from zhangxingzhi/flow-in-pipeline

Oct 19, 2023

Contributor

elliotzh commented Sep 5, 2023 •

edited

Loading

Description

Sample for using flow in pipeline job as anonymous component.

Checklist

I have read the contribution guidelines
I have coordinated with the docs team (mldocs@microsoft.com) if this PR deletes files or changes any file names or file extensions.
Pull request includes test coverage for the included changes.

wangchao1230 self-requested a review

September 7, 2023 06:25

guming-learning reviewed

View reviewed changes

cli/jobs/pipelines-with-components/flow_in_pipeline/1a_flow_in_pipeline/README.md Outdated Show resolved Hide resolved

elliotzh force-pushed the zhangxingzhi/flow-in-pipeline branch from ef5ab1e to a06b242 Compare

September 26, 2023 03:37

elliotzh force-pushed the zhangxingzhi/flow-in-pipeline branch from e91aa30 to d9d8c6e Compare

October 8, 2023 10:05

elliotzh changed the title ~~doc: flow in pipeline~~ sample: flow in pipeline

elliotzh force-pushed the zhangxingzhi/flow-in-pipeline branch 2 times, most recently from b66717b to 807aa61 Compare

October 8, 2023 10:18

wangchao1230 reviewed

View reviewed changes

cli/jobs/pipelines-with-components/flow_in_pipeline/1a_flow_in_pipeline/README.md Outdated Show resolved Hide resolved

wangchao1230 reviewed

View reviewed changes

cli/jobs/pipelines-with-components/flow_in_pipeline/1a_flow_in_pipeline/pipeline.yml Outdated Show resolved Hide resolved

wangchao1230 reviewed

View reviewed changes

cli/jobs/pipelines-with-components/flow_in_pipeline/1a_flow_in_pipeline/pipeline.yml Outdated Show resolved Hide resolved

wangchao1230 reviewed

View reviewed changes

cli/jobs/pipelines-with-components/flow_in_pipeline/1a_flow_in_pipeline/README.md Outdated Show resolved Hide resolved

wangchao1230 reviewed

View reviewed changes

sdk/python/jobs/pipelines/1l_flow_in_pipeline/additional_includes/convert_to_dict.py Outdated Show resolved Hide resolved

wangchao1230 reviewed

View reviewed changes

sdk/python/jobs/pipelines/1l_flow_in_pipeline/flow_in_pipeline.ipynb Outdated Show resolved Hide resolved

wangchao1230 reviewed

View reviewed changes

sdk/python/jobs/pipelines/1l_flow_in_pipeline/web_classification/flow.dag.yaml Outdated Show resolved Hide resolved

elliotzh force-pushed the zhangxingzhi/flow-in-pipeline branch 2 times, most recently from 9792639 to cc9e20b Compare

October 10, 2023 01:57

kdestin previously requested changes

View reviewed changes

sdk/python/setup.sh Outdated Show resolved Hide resolved

elliotzh and others added 13 commits

October 17, 2023 16:35


          doc: flow in pipeline for cli

ed6d4b5


          doc: flow in pipeline sdk experience

04b1e4d


          doc: remind to grant permission to compute cluster

29cd9a6


          Update README.md

01c7a67


          Update README.md

a85c895


          Update flow_in_pipeline.ipynb

5d00527


          update readme for cli sample


          move bs4 inside tool function to reduce dependency for compile

4e1575d


          doc: add information for office usage

5af8c1a


          doc: update component spec glob

bbc4e57


          feat: add requirements

f0bd710


          azure-ai-ml is released

b081efc


          fix: smoke

aec577a

elliotzh added 7 commits

October 17, 2023 16:35


          fix: resolve comments


          ci: add ci for flow in pipeline

e73955f


          doc: update readme

bd04b58


          fix: smoke

299cfb7


          create connection with wrong settings

8a367a8


          fix: smoke

6dd0e6a


          fix: highlight schema requirement

d59aa34

elliotzh force-pushed the zhangxingzhi/flow-in-pipeline branch from acd9ca6 to d59aa34 Compare

October 17, 2023 08:35

elliotzh added 12 commits

October 19, 2023 10:01


          feat: use a sample without connection

9786a2d


          fix: include flow.tools.json

7b60f7d


          fix: update text

12ba9c4


          fix: remove connection creation

162f239


          fix: update input setting

9bfba90


          fix: remove connection reference in python tool

ecfa0b8


          Merge branch 'main' into zhangxingzhi/flow-in-pipeline

20ae281


          fix: flow run setting

cfe7eb0


          fix: further clean connection reference

d9eacde


          feat: use the same pipeline in cli and sdk

4b6b813


          fix: black

86d4e15


          fix: make 2 example align

18ea175

brynn-code approved these changes

View reviewed changes

elliotzh dismissed kdestin’s stale review

October 19, 2023 08:24

request applied

zhengfeiwang approved these changes

View reviewed changes

elliotzh merged commit 5bb2997 into main

9 checks passed

elliotzh deleted the zhangxingzhi/flow-in-pipeline branch

October 19, 2023 08:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

wangchao1230 wangchao1230 left review comments

guming-learning guming-learning left review comments

zhengfeiwang zhengfeiwang approved these changes

brynn-code brynn-code approved these changes

kdestin kdestin left review comments

Labels

None yet