Skip to content

adding preptask sample - #266

Open
Manish Sharma (manishshar) wants to merge 22 commits into
Azure-Samples:masterfrom
manishshar:master
Open

adding preptask sample#266
Manish Sharma (manishshar) wants to merge 22 commits into
Azure-Samples:masterfrom
manishshar:master

Conversation

@manishshar

Copy link
Copy Markdown

Adding a sample preparation task for batch samples.

Thanks
Manish

@bgklein

Copy link
Copy Markdown
Contributor

Code LGTM.

Could you add some details on the sample to the README

Thematically I think it would make sense to swap this to sample3 and move that current sample 3 to 4 mainly due to complexity but I'll leave that discretion to Alina Park (@alfpark) and Matthew Christopher (@matthchr)

@manishshar

Copy link
Copy Markdown
Author

Hi Brandon Klein (@bgklein)

Thanks for your response.
I have done changes in document as well. Please check.

Thanks
Manish

@manishshar

Copy link
Copy Markdown
Author

Hi
Could you please review? Pls let me know if anything is need.

-Manish

Comment thread Python/Batch/README.md Outdated
Comment thread Python/Batch/README.md Outdated
Comment thread Python/Batch/README.md Outdated
Comment thread Python/Batch/README.md Outdated
Comment thread Python/Batch/sample4_preparationtask.py Outdated
Comment thread Python/Batch/sample4_preparationtask.py Outdated
Comment thread Python/Batch/sample4_preparationtask.cfg Outdated
@bgklein

Copy link
Copy Markdown
Contributor

Alina Park (@alfpark) / Matthew Christopher (@matthchr) / darylmsft (@darylmsft) I don't have ownership of this repo. If one of you could take a final look and merge/make any last suggestions that would be great

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants