Skip to content

Correct default values of model filenames in demo scripts#11

Merged
fengyuentau merged 1 commit intoopencv:masterfrom
fengyuentau:correct_demo_default_names
Nov 4, 2021
Merged

Correct default values of model filenames in demo scripts#11
fengyuentau merged 1 commit intoopencv:masterfrom
fengyuentau:correct_demo_default_names

Conversation

@fengyuentau
Copy link
Copy Markdown
Member

@fengyuentau fengyuentau commented Nov 2, 2021

related issue #10

@fengyuentau
Copy link
Copy Markdown
Member Author

fengyuentau commented Nov 2, 2021

cc @kaingwade @zihaomu

@fengyuentau fengyuentau mentioned this pull request Nov 2, 2021
Closed
@fengyuentau fengyuentau linked an issue Nov 2, 2021 that may be closed by this pull request
face_detection_yunet failed to load pretrained model as default #10
Closed
@atinfinity
Copy link
Copy Markdown

atinfinity commented Nov 2, 2021

@fengyuentau Thank you for your mention. #10 (comment)
I think that there is a concern. Because, the filename of pretrained model has date(e.g. face_detection_yunet_2021sep.onnx).
So, you will need to modify demo scripts when you update model.

@fengyuentau
Copy link
Copy Markdown
Member Author

fengyuentau commented Nov 3, 2021

@atinfinity I think that's not a big problem since models won't be updated quite frequently.

@fengyuentau fengyuentau merged commit 7e8654b into opencv:master Nov 4, 2021
@fengyuentau fengyuentau deleted the correct_demo_default_names branch November 4, 2021 07:45
@Sidd1609 Sidd1609 mentioned this pull request Sep 11, 2022
Closed
fengyuentau added a commit that referenced this pull request Jun 8, 2023
@fengyuentau
correct default values of model filenames in demo scripts (#11)
25e3347
@fengyuentau fengyuentau added this to the 4.9.0 (first release) milestone Dec 28, 2023
@fengyuentau fengyuentau self-assigned this Dec 28, 2023
@fengyuentau fengyuentau added bug Something isn't working demo anything related to demo in Python / C++ labels Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@fengyuentau fengyuentau

Labels

bug Something isn't working demo anything related to demo in Python / C++

Projects

None yet

Milestone

4.9.0 (first release)

Development

Successfully merging this pull request may close these issues.

face_detection_yunet failed to load pretrained model as default

2 participants

@fengyuentau @atinfinity
⚡