Skip to content
This repository has been archived by the owner on Nov 21, 2023. It is now read-only.

FYI about upcoming changes: detectron package #414

Open
rbgirshick opened this issue May 4, 2018 · 0 comments
Open

FYI about upcoming changes: detectron package #414

rbgirshick opened this issue May 4, 2018 · 0 comments

Comments

@rbgirshick
Copy link
Contributor

Next Monday, we plan to merge a big diff that will place most of the Detectron code under a directory named detectron. This change will place Detectron's library code into a Python package named detectron and will help users avoid namespace collisions for common module names (like utils) when installing Detectron system-wide (addressing #228). Unfortunately this change will be somewhat disruptive for people who have built on top of or modified custom forks of the code. For the most part, if you rebase your code you will only need to change imports like:

from core.config import cfg

to

from detectron.core.config import cfg

reflecting the new directory structure:

Detectron
|_ cmake/
|_ CMakeLists.txt
|_ configs/
|_ CONTRIBUTING.md
|_ demo/
|_ detectron/
|   |_ core
|   |_ datasets
|   |_ modeling
|   |_ ops
|   |_ roi_data
|   |_ tests
|   |_ utils
|_ docker/
|_ FAQ.md
|_ GETTING_STARTED.md
|_ .github
|_ .gitignore
|_ INSTALL.md
|_ LICENSE
|_ Makefile
|_ MODEL_ZOO.md
|_ NOTICE
|_ README.md
|_ setup.py
|_ tools/

We may expect some bugs right after the merge, so please let us know if there are any issues. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant