Skip to content

Conversation

@eedorenko
Copy link
Contributor

No description provided.

@eedorenko eedorenko requested a review from dtzar January 9, 2020 23:59
Copy link
Contributor

@dtzar dtzar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tested this using a registered diabetes dataset in blob storage? We should use it as default for the CI pipeline - I don't see it setup there...

@eedorenko
Copy link
Contributor Author

Have you tested this using a registered diabetes dataset in blob storage? We should use it as default for the CI pipeline - I don't see it setup there...

Yes, I have. With an ML WS in our subscription (EuUniqueML-AML-WS). I will setup that with the Eng subscription

@eedorenko eedorenko requested a review from jotaylo January 10, 2020 23:38
Copy link
Contributor

@jotaylo jotaylo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@jotaylo jotaylo merged commit da60ab1 into master Jan 13, 2020
@dtzar dtzar deleted the eedorenko/dataset branch January 13, 2020 17:42
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.

4 participants