Skip to content

Conversation

@prathyusha12345
Copy link
Contributor

Migrate Movie recommender to ML.Net 1.0.0-preview

Copy link
Contributor

@JRAlexander JRAlexander left a comment

Choose a reason for hiding this comment

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

Hi, it appears you are pulling in commits from an earlier PR. Can you check this? Thanks!

@prathyusha12345
Copy link
Contributor Author

prathyusha12345 commented Apr 24, 2019

@JRAlexander I see the commits are from TaxiFare. I don't remember that I have taken this commit. Anyhow I checked now by merging master from upstream into the current branch and only 2 files are changed that are specific to Movie Recommender only. There are no issues like merge conflicts or something.

Can you check once and let me know in case of changes

@JRAlexander
Copy link
Contributor

Thanks, @prathyusha12345!!

@JRAlexander JRAlexander merged commit c1c9b66 into dotnet:master Apr 25, 2019
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