No module named "Detectron2" Bug Fix #1
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes made
In
setup.py
added CustomInstall classAutodistillDetic
for installing dependencies when building. Some minor typo error. Install_detic() function is not removed or modified in thedetic_model.py
Tests
Ran in google colab with
pip install .
and in conda environment in Mac withpip install .
and compared the result withpip3 install autodistill-detic
in different conda environment.pip3 install autodistill-detic
returns error such as "no module named supervision, autodistill, or detectron2"With new modified setup.py using
pip install .
, the errors are fixed.Please test for further potential errors.