Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@asmorkalov
Copy link
Contributor

@asmorkalov asmorkalov force-pushed the as/openvino-ci branch 2 times, most recently from 242e0b7 to 18b7e81 Compare July 24, 2023 15:43
@asmorkalov
Copy link
Contributor Author

Blocker: opencv/opencv#24051

@dkurt
Copy link
Member

dkurt commented Jul 28, 2023

[  FAILED  ] 3 tests, listed below:
[  FAILED  ] Test_Darknet_layers.region/0, where GetParam() = NGRAPH/CPU
[  FAILED  ] DNNTestHighLevelAPI.predict/0, where GetParam() = CPU
[  FAILED  ] Test_ONNX_conformance.Layer_Test/test_dropout_default_ratio_NGRAPH_CPU, where GetParam() = (test_dropout_default_ratio, NGRAPH/CPU)

Fixed tests on opencv/opencv#24072

@dkurt
Copy link
Member

dkurt commented Aug 2, 2023

@asmorkalov, remaining problem is about missed age-gender-recognition-retail-0013.xml file:

C++ exception with description "OpenCV(4.8.0-dev) /home/openvino/opencv/modules/ts/src/ts.cpp:1071: error: (-2:Unspecified error) OpenCV tests: Can't find required data file: intel/age-gender-recognition-retail-0013/FP32/age-gender-recognition-retail-0013.xml in function 'findData'

@asmorkalov
Copy link
Contributor Author

Yes, @TolyaTalamanov works on it.

@dkurt
Copy link
Member

dkurt commented Aug 2, 2023

There are two options: make file optional or download to CI system. Seems like it's just missed:
https://github.com/opencv/opencv/blob/0245c0cd10387f2de7c7bd076585a285d977e53c/modules/dnn/test/test_ie_models.cpp#L104-L106

@asmorkalov
Copy link
Contributor Author

I asked Anatoly to add the models to add the models to download_models.py in extra.

@dkurt
Copy link
Member

dkurt commented Aug 8, 2023

Any progress here? Just for reference, direct links are https://storage.openvinotoolkit.org/repositories/open_model_zoo/2023.0/models_bin/1/age-gender-recognition-retail-0013/FP32/. I can add them as a part of opencv/opencv_extra#1080.

@asmorkalov
Copy link
Contributor Author

@dkurt you are welcome. @TolyaTalamanov promised to do it, but he is on vacation now.

@dkurt
Copy link
Member

dkurt commented Aug 25, 2023

@asmorkalov, please re-run action to check fixes on latest 4.x branch.

@dkurt
Copy link
Member

dkurt commented Sep 1, 2023

@asmorkalov, reminder. Now tests expect to pass. I made some changes to 4.x branch.

@dkurt
Copy link
Member

dkurt commented Sep 27, 2023

@asmorkalov, checks passed 🎉

@asmorkalov
Copy link
Contributor Author

@dkurt @mshabunin please review and approve.

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