-
Notifications
You must be signed in to change notification settings - Fork 39
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
[DIP] Add initial documents for image equivalence testing. #65
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work on this!
Just some minor comments and then this is LGTM :)
utils/README.md
Outdated
``` | ||
$ pip install pillow | ||
$ pip install numpy | ||
$ pip install sys |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you put this in a requirements.txt
? Perhaps name it requirements-optional.txt
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correted with pip install -r requirements.txt
utils/compareImg.py
Outdated
img1_dim = len(np.array(img1).shape) | ||
img2_dim = len(np.array(img2).shape) | ||
if img1_dim != img2_dim: | ||
print("Please provide images of the same type. Currently, one is a grayscale image and the other is a color image.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might not always be the case when the mismatched images are grayscale and color (one could be RGB
while the other could be RGBA
or any other no. of channels?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only the first sentence remained now. The RGB
and RGBA
or other channels is a problem, but im not sure what to write here.
6c1d11b
to
cad1d1e
Compare
Add README.md for comfigure virtual environment.
After configure venv, please run the code below: