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

Add link for MLP. Had to Google it #54

Merged
merged 3 commits into from Nov 24, 2018
Merged

Conversation

marko-bast
Copy link
Contributor

No description provided.

@jachiam
Copy link
Contributor

jachiam commented Nov 24, 2018

Hi @marko-bast! Great to hear from you, it's been a while since the Hackathon. I hope Spinning Up is helping you study the ins and outs of modern AI, and makes it easier for you to transition into working in this space. :)

Could you separate out the MLP link from the commit to update the plotter? Will accept this PR immediately. I want to take a little bit to think about the plotter update.

@marko-bast
Copy link
Contributor Author

marko-bast commented Nov 24, 2018

I'm honoured you remember me and very grateful for the creation of Spinning Up :)

I've removed plotter change in this PR, it got here by error. The plotter change is in another PR already.

@jachiam jachiam merged commit 5eade94 into openai:master Nov 24, 2018
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.

None yet

2 participants