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 TensorFlow Snippets. #206

Merged
merged 3 commits into from
Jan 3, 2019
Merged

Add TensorFlow Snippets. #206

merged 3 commits into from
Jan 3, 2019

Conversation

mr-ubik
Copy link
Contributor

@mr-ubik mr-ubik commented Oct 14, 2018

Name of an extension you are adding

TensorFlow Snippets

Why do you think this extension is awesome?

Add a plethora of snippets that speed up TensorFlow models development.

Make sure that:

[x] Screenshot/GIF included (to demonstrate the plugin functionality)

[x] ToC updated

@lannonbr
Copy link
Collaborator

@mr-ubik it seems there was various changes you made to the TOC for the list, could you revert them given it is not related to the PR of adding the tensorflow snippets extension.

Otherwise it looking good. 😄

@lannonbr lannonbr added the New Extension PR / Issue for new extension / theme / etc. to the list label Oct 14, 2018
Fixed the errors introduced in the README.md
@mr-ubik
Copy link
Contributor Author

mr-ubik commented Oct 31, 2018

Sould be fixed now

@lannonbr
Copy link
Collaborator

Looks like there still are changes that are not related to this tensorflow snippets extension. Make sure the only changes in the README are the link to the extension in the TOC as well as the actual list entry.

These two additions are the ones I am mentioning. Otherwise revert everything else.

screen shot 2018-10-31 at 7 03 05 pm
screen shot 2018-10-31 at 7 03 09 pm

@viatsko viatsko merged commit ac79837 into viatsko:master Jan 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Extension PR / Issue for new extension / theme / etc. to the list
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants