Skip to content

Conversation

@fengyuentau
Copy link
Member

@asmorkalov asmorkalov merged commit de5f678 into opencv:4.x Sep 19, 2022
@alalek
Copy link
Member

alalek commented Sep 19, 2022

Why do we need .jpg image in git history?
1 (ONE) commit must be merged only.

opencv_extra repo is already 500Mb+. We don't need more unused garbage here.

@fengyuentau
Copy link
Member Author

Apology for the useless git commit history. Originally .jpg was used, but then I found the inference results were different on different platforms if the image is jpg. Using a PNG image solved this issue. I will squash commits into one for every single pull request in opencv.

@fengyuentau fengyuentau deleted the naive_yolov7 branch September 20, 2022 02:12
@alalek alalek mentioned this pull request Jan 8, 2023
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.

3 participants