Skip to content
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.

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

Always run the pipeline evaluation step #154

Merged
merged 2 commits into from
Jan 29, 2020
Merged

Conversation

jotaylo
Copy link
Contributor

@jotaylo jotaylo commented Jan 28, 2020

Set the ci-build-train pipeline to always run the evaluation step (even if the pipeline is cancelled). This will cause the Azure Pipeline to log the reason for cancellation rather than just "silently failing".
#151

@jotaylo jotaylo requested a review from sbaidachni January 28, 2020 22:00
@omartin2010
Copy link
Contributor

I would add a note in the documentation that says that the ML pipeline actually cancels itself. If you write the code yourself, that's obvious but if not, it's not particularly obvious.

@jotaylo
Copy link
Contributor Author

jotaylo commented Jan 29, 2020

@omartin2010 , @sudivate's change yesterday added a flag to disable the cancellation. The docs will need to be updated to mention both the default cancellation state and how to disable it.

@jotaylo jotaylo merged commit ecad47f into master Jan 29, 2020
@jotaylo jotaylo deleted the jotaylo/always_evaluate branch January 29, 2020 19:16
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.

3 participants