Skip to content

Commit 0c28c3d

Browse files
ppwwyyxxfacebook-github-bot
authored andcommitted
fix precise BN
Summary: It actually did not run Reviewed By: rbgirshick Differential Revision: D17900284 fbshipit-source-id: cf9ae95581f0948b3d2ff13e113ea2146d284f93
1 parent d250fcc commit 0c28c3d

File tree

2 files changed

+13
-11
lines changed

2 files changed

+13
-11
lines changed

detectron2/engine/hooks.py

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -388,13 +388,14 @@ def update_stats(self):
388388

389389
def data_loader():
390390
nonlocal num_iter
391-
num_iter += 1
392-
if num_iter % 100 == 0:
393-
self._logger.info(
394-
"Running precise-BN ... {}/{} iterations.".format(num_iter, self._num_iter)
395-
)
396-
# This way we can reuse the same iterator
397-
yield next(self._data_iter)
391+
while True:
392+
num_iter += 1
393+
if num_iter % 100 == 0:
394+
self._logger.info(
395+
"Running precise-BN ... {}/{} iterations.".format(num_iter, self._num_iter)
396+
)
397+
# This way we can reuse the same iterator
398+
yield next(self._data_iter)
398399

399400
with EventStorage(): # capture events in a new storage to discard them
400401
self._logger.info(

docs/conf.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -255,14 +255,14 @@ def autodoc_skip_member(app, what, name, obj, skip, options):
255255

256256

257257
def url_resolver(url):
258-
if '.html' not in url:
259-
url = url.replace('../', '')
258+
if ".html" not in url:
259+
url = url.replace("../", "")
260260
return "https://github.com/facebookresearch/detectron2/blob/master/" + url
261261
else:
262262
if DEPLOY:
263263
return "http://detectron2.readthedocs.io/" + url
264264
else:
265-
return '/' + url
265+
return "/" + url
266266

267267

268268
def setup(app):
@@ -272,7 +272,8 @@ def setup(app):
272272
# app.connect('autodoc-skip-member', autodoc_skip_member)
273273
app.add_config_value(
274274
"recommonmark_config",
275-
{ "url_resolver": url_resolver,
275+
{
276+
"url_resolver": url_resolver,
276277
"auto_toc_tree_section": "Contents",
277278
"enable_math": True,
278279
"enable_inline_math": True,

0 commit comments

Comments
 (0)