Skip to content

Conversation

@snnn
Copy link
Contributor

@snnn snnn commented Dec 6, 2017

Allow users to export this model in TF's SavedModel format.

@k-w-w k-w-w self-requested a review December 16, 2017 00:19
@snnn
Copy link
Contributor Author

snnn commented Dec 18, 2017

I'll resolve the conflict.

@snnn
Copy link
Contributor Author

snnn commented Dec 18, 2017

How can I format the python code to meet your coding style? (sorry, I'm new to Python)

@snnn snnn requested a review from nealwu as a code owner December 18, 2017 21:42
Copy link
Contributor

@k-w-w k-w-w left a comment

Choose a reason for hiding this comment

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

Thanks for adding this code! I made a few style changes, and added an example so that users can easily use the SavedModel.

@k-w-w k-w-w merged commit 5a1faff into tensorflow:master Dec 18, 2017
k-w-w added a commit that referenced this pull request Dec 18, 2017
Dr-Champ added a commit to Dr-Champ/models that referenced this pull request Dec 31, 2017
# By Asim Shankar (7) and others
# Via GitHub
* 'master' of https://github.com/Dr-Champ/models: (30 commits)
  Update README.md
  Make the redirect to slim notice more prominent
  Update README.md
  Redirect users from cifar10 to cifar10_estimator
  [mnist]: Make flags function scoped (not module scoped).
  [mnist]: Address Neal's comment
  [mnist]: Fix typo
  compatible with tensorflow 1.4.1
  Update blog_custom_estimators.py
  [mnist]: Address another comment
  [mnist]: Address comments from PR
  Update README.md in adv_imagenet_models.
  lower case file name
  Don't put an activation on the logit layer
  use six for urlopen
  slice fix for 1.5
  [mnist]: Updates
  Add files via upload
  official/mnist: support savedmodel (tensorflow#2967)
  Fix the CODEOWNERS file syntax
  ...
Adrrei pushed a commit to Adrrei/models that referenced this pull request Dec 16, 2018
With examples, and updates to the README
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants