Skip to content

[Docs] Remove opencv-python-headless dependency by albumentations#5868

Merged
ZwwWayne merged 8 commits intoopen-mmlab:masterfrom
BIGWangYuDong:fix_albu_install
Sep 4, 2021
Merged

[Docs] Remove opencv-python-headless dependency by albumentations#5868
ZwwWayne merged 8 commits intoopen-mmlab:masterfrom
BIGWangYuDong:fix_albu_install

Conversation

@BIGWangYuDong
Copy link
Copy Markdown
Collaborator

Follow open-mmlab/mmpose#833,

By default, installing albumentations by pip has opencv-python-headless dependency even if opencv-python has already been installed. This could cause different opencv versions installed in the environment and lead to unexpected errors.

This PR updates the installation of albumentations to prevent using a wheel distribution of it with opencv-python-headless dependency. See its docs for details.

**Notice, ** now we can't use pip install -v -e . normally. Need to resolve the error

@BIGWangYuDong BIGWangYuDong changed the title [WIP] Remove opencv-python-headless dependency by albumentations [Docs] Remove opencv-python-headless dependency by albumentations Aug 25, 2021
@BIGWangYuDong
Copy link
Copy Markdown
Collaborator Author

Add a note in get_started docs, which users can select to install albumentations with or without installing opencv-python-headless

@BIGWangYuDong
Copy link
Copy Markdown
Collaborator Author

ping @ZwwWayne
I will update Chinese version after you review this PR

@BIGWangYuDong BIGWangYuDong requested a review from ZwwWayne August 25, 2021 07:53
@BIGWangYuDong
Copy link
Copy Markdown
Collaborator Author

BIGWangYuDong commented Aug 25, 2021

Done

@BIGWangYuDong BIGWangYuDong requested a review from ZwwWayne August 25, 2021 09:06
@BIGWangYuDong
Copy link
Copy Markdown
Collaborator Author

update Chinese version of get started

@BIGWangYuDong BIGWangYuDong requested a review from ZwwWayne August 31, 2021 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants