algolia / algolia/color-extractor

opencv example in README leads to incorrect color identification with default settings

Open
#6 1 comment 3 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
284
Forks
69
PR merge metrics
No merged PRs in 30d

Description

In the README example, an image file is opened using opencv, which will open the file in BGR. However, the `color_names.npz` dataset that comes with the repo is stored as RGB. Therefore if you use the example verbatim, you will get incorrect color identifications. Since `from_file.py` uses skimage.io instead of opencv, the only corrections needed are on the README, which are:

- replacing opencv calls with skimage.io calls
- removing any mention of needing color name files stored as BGR.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the README example and compare it with from_file.py, which uses skimage.io. Check how color_names.npz is represented, then update the README so its image-loading example matches the dataset's RGB format and no longer mentions BGR color-name files.

Written by the indexing model from the issue text.

Assessment

Tech stack
opencv, python
Domain
documentation
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.