Skip to content

Conversation

bleqdyce
Copy link

@bleqdyce bleqdyce commented Aug 20, 2018

I think eval_interval_secs is originally designed to be passed as one of EvalSpec parameter - throttle_secs, which defines the frequency of saving checkpoint and evaluation.
Because setting eval_interval_secs is not working, there is no other way to change the frequency of evaluation.
Therefore, I simply parse eval_interval_secs from eval config and pass it to EvalSpec as a parameters to complete this functionality.

@ruifm
Copy link

ruifm commented Feb 7, 2019

This is a must have imo. Please merge it!

@pkulzc
Copy link
Contributor

pkulzc commented Feb 10, 2019

Could you please resolve the conflicts ?

@thusinh1969
Copy link

Gents,

Have this been fixed and applied fully in latest tensorflow object detection repo so we can re-download it ? This is killing us...

Thanks,
Steve

@kkulczak
Copy link

Any progress with the issue?

Fix conflicts with current master branch
@bleqdyce bleqdyce requested a review from dreamdragon as a code owner October 25, 2019 10:03
@googlebot
Copy link

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@bleqdyce
Copy link
Author

@googlebot I consent.

1 similar comment
@piercus
Copy link

piercus commented Oct 26, 2019

@googlebot I consent.

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@jaeyounkim
Copy link
Collaborator

@pkulzc Please review this pull request.

@jaeyounkim jaeyounkim self-assigned this Apr 25, 2020
@jaeyounkim jaeyounkim added the stat:awaiting review Waiting on review label Apr 25, 2020
@Mathieu4141
Copy link

Hi !
Is there a reason why this is not yet merged ?

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.