Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Could not find color with width 100px #1

Open
codinguncut opened this issue Sep 7, 2022 · 4 comments
Open

Could not find color with width 100px #1

codinguncut opened this issue Sep 7, 2022 · 4 comments

Comments

@codinguncut
Copy link

codinguncut commented Sep 7, 2022

  • cloned your repo
  • installed requirements (had to change opencv-contrib-python==4.6.0 to opencv-contrib-python==4.6.0.66), using python3.8.13
  • python color_correction.py --reference raw.jpg --input test.jpg --out output.jpg --width 1280
  • output:
[INFO] loading images...
[INFO] Finding color matching cards width 100px
[INFO] Could not find color with width 100px. Try width:500px
[WARNING] Could not find color matching cards in both images. Try a highter/better Resolution
  • same output/ error given with no width specified
@codinguncut
Copy link
Author

Interestingly with my own images the script ran through successfully.
I wonder if there is an issue with your ref/ test image, or if I am running into some kind of version/ platform issue.

@dazzafact
Copy link
Owner

dazzafact commented Sep 7, 2022

Please use your own Reference Images and Cards to test. Theses are only small examples. The Images has to be in a good Quality to detect the Color-Cards well

@codinguncut
Copy link
Author

Of course, thank you for making this available.
I only wanted to make you aware that the repo as checked out with your sample images failed on my machine.
Sorry to bother you.

@dazzafact
Copy link
Owner

dazzafact commented Sep 7, 2022

No problem
you can also print out and place the color cards yourself. That should work too.
https://github.com/dazzafact/image_color_correction/blob/main/color-card.png

Regards

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

No branches or pull requests

2 participants