update isort path

Summary: some files were skipped

Reviewed By: rbgirshick

Differential Revision: D20260481

fbshipit-source-id: 193c74cce8e6cdff68ebc20c3ab14d7076183e9b

Author: ppwwyyxx

INSTALL.md

@@ -18,24 +18,24 @@ also installs detectron2 with a few simple commands.
 
 After having the above dependencies and gcc & g++ ≥ 4.9, run:
 ```
-pip install 'git+https://github.com/facebookresearch/detectron2.git'
+python -m pip install 'git+https://github.com/facebookresearch/detectron2.git'
 # (add --user if you don't have permission)
 
 # Or, to install it from a local clone:
 git clone https://github.com/facebookresearch/detectron2.git
-cd detectron2 && pip install -e .
+cd detectron2 && python -m pip install -e .
 
 # Or if you are on macOS
-# CC=clang CXX=clang++ pip install -e .
+# CC=clang CXX=clang++ python -m pip install -e .
 ```
 
-To __rebuild__ detectron2 that's built from a local clone, `rm -rf build/ **/*.so` then `pip install -e .`.
-You often need to rebuild detectron2 after reinstalling PyTorch.
+To __rebuild__ detectron2 that's built from a local clone, use `rm -rf build/ **/*.so` to clean the
+old build first. You often need to rebuild detectron2 after reinstalling PyTorch.
 
 ### Install Pre-Built Detectron2
 ```
 # for CUDA 10.1:
-pip install detectron2 -f https://dl.fbaipublicfiles.com/detectron2/wheels/cu101/index.html
+python -m pip install detectron2 -f https://dl.fbaipublicfiles.com/detectron2/wheels/cu101/index.html
 ```
 You can replace cu101 with "cu{100,92}" or "cpu".
 

datasets/prepare_cocofied_lvis.py

@@ -1,12 +1,11 @@
 # -*- coding: utf-8 -*-
 # Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved
 
-import json
 import copy
+import json
 import os
 from collections import defaultdict
 
-
 # This mapping is extracted from the official LVIS mapping:
 # https://github.com/lvis-dataset/lvis-api/blob/master/data/coco_to_synset.json
 COCO_SYNSET_CATEGORIES = [

datasets/prepare_panoptic_fpn.py

@@ -1,16 +1,16 @@
 # -*- coding: utf-8 -*-
 # Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved
 
-import time
 import functools
 import json
 import multiprocessing as mp
 import numpy as np
 import os
+import time
+from fvcore.common.download import download
 from PIL import Image
 
 from detectron2.data.datasets.builtin_meta import COCO_CATEGORIES
-from fvcore.common.download import download
 
 from panopticapi.utils import rgb2id
 

detectron2/data/datasets/builtin.py

@@ -19,13 +19,13 @@
 
 import os
 
-from detectron2.data import MetadataCatalog, DatasetCatalog
-from .register_coco import register_coco_instances, register_coco_panoptic_separated
-from .lvis import register_lvis_instances, get_lvis_instances_meta
+from detectron2.data import DatasetCatalog, MetadataCatalog
+
+from .builtin_meta import _get_builtin_metadata
 from .cityscapes import load_cityscapes_instances, load_cityscapes_semantic
+from .lvis import get_lvis_instances_meta, register_lvis_instances
 from .pascal_voc import register_pascal_voc
-from .builtin_meta import _get_builtin_metadata
-
+from .register_coco import register_coco_instances, register_coco_panoptic_separated
 
 # ==== Predefined datasets and splits for COCO ==========
 

detectron2/data/datasets/cityscapes.py

@@ -8,12 +8,12 @@
 import os
 from itertools import chain
 import pycocotools.mask as mask_util
+from fvcore.common.file_io import PathManager
 from PIL import Image
 
 from detectron2.structures import BoxMode
-from detectron2.utils.logger import setup_logger
 from detectron2.utils.comm import get_world_size
-from fvcore.common.file_io import PathManager
+from detectron2.utils.logger import setup_logger
 
 try:
     import cv2  # noqa

detectron2/data/datasets/lvis.py

@@ -1,11 +1,11 @@
 # Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved
 import logging
 import os
-
-from fvcore.common.timer import Timer
-from detectron2.structures import BoxMode
 from fvcore.common.file_io import PathManager
+from fvcore.common.timer import Timer
+
 from detectron2.data import DatasetCatalog, MetadataCatalog
+from detectron2.structures import BoxMode
 
 from .builtin_meta import _get_coco_instances_meta
 from .lvis_v0_5_categories import LVIS_CATEGORIES

detectron2/data/datasets/pascal_voc.py

@@ -1,14 +1,13 @@
 # -*- coding: utf-8 -*-
 # Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved
 
-from fvcore.common.file_io import PathManager
-import os
 import numpy as np
+import os
 import xml.etree.ElementTree as ET
+from fvcore.common.file_io import PathManager
 
-from detectron2.structures import BoxMode
 from detectron2.data import DatasetCatalog, MetadataCatalog
-
+from detectron2.structures import BoxMode
 
 __all__ = ["register_pascal_voc"]
 

detectron2/data/datasets/register_coco.py

@@ -2,6 +2,7 @@
 import copy
 
 from detectron2.data import DatasetCatalog, MetadataCatalog
+
 from .coco import load_coco_json, load_sem_seg
 
 """

docs/tutorials/models.md

@@ -127,7 +127,8 @@ model = build_model(cfg)
 features = model.backbone(images.tensor)
 proposals, _ = model.proposal_generator(images, features)
 instances = model.roi_heads._forward_box(features, proposals)
-mask_features = model.roi_heads.mask_pooler(features, [x.pred_boxes for x in instances])
+mask_features = [features[f] for f in model.roi_heads.in_features]
+mask_features = model.roi_heads.mask_pooler(mask_features, [x.pred_boxes for x in instances])
 ```
 
 Note that both options require you to read the existing forward code to understand

projects/PointRend/point_rend/roi_heads.py

@@ -64,7 +64,7 @@ def _init_mask_head(self, cfg, input_shape):
             return
         self.mask_coarse_in_features = cfg.MODEL.ROI_MASK_HEAD.IN_FEATURES
         self.mask_coarse_side_size   = cfg.MODEL.ROI_MASK_HEAD.POOLER_RESOLUTION
-        self._feature_scales          = {k: 1.0 / v.stride for k, v in input_shape.items()}
+        self._feature_scales         = {k: 1.0 / v.stride for k, v in input_shape.items()}
         # fmt: on
 
         in_channels = np.sum([input_shape[f].channels for f in self.mask_coarse_in_features])

setup.cfg

@@ -3,7 +3,7 @@ line_length=100
 multi_line_output=3
 include_trailing_comma=True
 known_standard_library=numpy,setuptools,mock
-skip=datasets,docs
+skip=./datasets,docs
 skip_glob=*/__init__.py
 known_myself=detectron2
 known_third_party=fvcore,matplotlib,cv2,torch,torchvision,PIL,pycocotools,yacs,termcolor,cityscapesscripts,tabulate,tqdm,scipy,lvis,psutil,pkg_resources,caffe2,onnx