Commit 03156ce7 authored by Ajinkya Deogade's avatar Ajinkya Deogade Committed by Facebook GitHub Bot
Browse files

Temp remove output typing for Base Runner create_shared_context

Summary:
Pull Request resolved: https://github.com/facebookresearch/d2go/pull/558

Presently, `D2GoSharedContext` is only imported in `mobile-vision/d2go/d2go/runner/default_runner.py` for type annotations. Unfortunately, this causes a circular dependency as the TARGET for `d2go.distributed` does not exists. Remove it temporarily and reintroduce back at the top of the stack once all the TARGETS have been introduced.

Reviewed By: wat3rBro

Differential Revision: D46096375

fbshipit-source-id: d8633ac755d39b807c18967f35a087178afc9787
parent 64c467e2
......@@ -24,7 +24,6 @@ from d2go.data.utils import (
maybe_subsample_n_images,
update_cfg_if_using_adhoc_dataset,
)
from d2go.distributed import D2GoSharedContext
from d2go.evaluation.evaluator import inference_on_dataset
from d2go.modeling import ema, kmeans_anchors
from d2go.modeling.api import build_d2go_model
......@@ -190,7 +189,7 @@ class BaseRunner(object):
pass
@classmethod
def create_shared_context(cls, cfg) -> D2GoSharedContext:
def create_shared_context(cls, cfg):
"""
Override `create_shared_context` in order to run customized code to create distributed shared context that can be accessed by all workers
"""
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment