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

Add layer model blend test pictures #766

Open
wants to merge 3 commits into
base: 3.4
Choose a base branch
from

Conversation

jsxyhelu
Copy link

No description provided.

@jsxyhelu
Copy link
Author

jsxyhelu commented Feb 9, 2021

from opencv/opencv#17520
Because my test here needs to call related image resources, according to the current practice, they are placed in opencv_extra (https://github.com/opencv/opencv_extra/pull/766). In this case, you must first pass in extra, and the build-bot here can succeed. Please help me to promote This question.
@mshabunin

@jsxyhelu
Copy link
Author

jsxyhelu commented Feb 9, 2021

What should I do? First of all, upload the image to extra, and then opencv test can pass, but the extra here can't pass.

@alalek
Copy link
Member

alalek commented Feb 9, 2021

For related patches you should create branches with the same name:

add_layer_model_blend
add_layer_model_blending

Otherwise there is no reliable logic how to find that.


(currently it is fixed manually, added opencv_extra= instruction to the main PR)

P.S. In case of updating extra's PR, you should re-trigger builds of the main PR. So you need to push updated/amended commits to the main PR after the pushed update to opencv_extra.

@jsxyhelu
Copy link
Author

@alalek thanks,I will pay attention to keeping the names of both sides consistent next time

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.

2 participants