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

Use new ensmallen logo #186

Merged
merged 3 commits into from
Mar 27, 2020
Merged

Use new ensmallen logo #186

merged 3 commits into from
Mar 27, 2020

Conversation

rcurtin
Copy link
Member

@rcurtin rcurtin commented Mar 27, 2020

This PR updates the README so that it uses the new ensmallen logo. NumFOCUS helped us design this by providing us with an excellent graphic designer, who came up with a logo that roughly represents the task of numerical optimization. She did way better than I could ever hope to do. :)

I've already put the logo on the website, and this puts it in the README. Maybe it could be cleaned up, but making things look nice is not my strongest ability. This is at least a start, though. :)

(Also, I've changed the text to use numerical optimization instead of mathematical optimization. That's the more standard term that is probably more recognizable for many.)

Example of what it will look like: https://github.com/rcurtin/ensmallen/tree/readme-new-logo

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.

cute!

Copy link
Member

@favre49 favre49 left a comment

Choose a reason for hiding this comment

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

Really nice :) The graphic designer did a great job

@favre49
Copy link
Member

favre49 commented Mar 27, 2020

I'll go ahead and merge this then.

@favre49 favre49 merged commit f948ea6 into mlpack:master Mar 27, 2020
@say4n
Copy link
Member

say4n commented Mar 27, 2020

The new logo is beautiful!
👍

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