Skip to content

Conversation

@iamshnoo
Copy link
Member

@iamshnoo iamshnoo commented Jul 2, 2020

Relevant issue : #2200
PyTorch docs : here

To compare bare-bones armadillo implementation with PyTorch implementation, see this Google Colab notebook.

@zoq
Copy link
Member

zoq commented Jul 2, 2020

Looks like the notebook is private?

@iamshnoo
Copy link
Member Author

iamshnoo commented Jul 2, 2020

Looks like the notebook is private?

Updated the link. Thanks!

Copy link
Member

@birm birm left a comment

Choose a reason for hiding this comment

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

I don't know what's caused the style build to suddenly care about another file.
That said, this is good work. I like the new standard(?) of having a colab notebook to show correctness. Great work!

Copy link

@mlpack-bot mlpack-bot bot left a comment

Choose a reason for hiding this comment

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

Second approval provided automatically after 24 hours. 👍

@birm birm merged commit 9e70601 into mlpack:master Aug 5, 2020
@iamshnoo
Copy link
Member Author

iamshnoo commented Aug 5, 2020

Thanks for reviewing and merging this!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants