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 metric op #10400

Closed
wants to merge 1 commit into from
Closed

add metric op #10400

wants to merge 1 commit into from

Conversation

astorfi
Copy link

@astorfi astorfi commented Jun 2, 2017

This pull request is aimed to create a metric_op for TF-Slim learning to be able to show the streaming metrics(accuracy,...) for the training phase.
The op reset the total and count local variables defined by streaming_metric(if it is streaming_accuracy ,...) per each epoch of training so the accumulative accuracy per epoch can be represented.

@tensorflow-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@sguada
Copy link
Member

sguada commented Jun 11, 2017

Please revert all the indentation changes, use 2 spaces per indentation level.

Copy link
Member

@sguada sguada left a comment

Choose a reason for hiding this comment

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

Revert all the indentation changes, currently it is hard to see what you actually change.
Use 2 spaces indentation.

@@ -250,6 +250,8 @@ def InitAssignFn(sess):
import os
import sys
import time
import tensorflow as tf
Copy link
Member

Choose a reason for hiding this comment

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

Don't import tensorflow directly only what is needed.

@martinwicke martinwicke added the stat:awaiting response Status - Awaiting response from author label Jun 14, 2017
@drpngx
Copy link
Contributor

drpngx commented Jun 26, 2017

@astorfi could you rebase and push again?

@astorfi
Copy link
Author

astorfi commented Jun 26, 2017

@drpngx There were some issues regarding my pool request. I am trying to solve them yet.

@vrv
Copy link

vrv commented Jul 24, 2017

Ping for @astorfi ...

@astorfi astorfi closed this Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes stat:awaiting response Status - Awaiting response from author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants