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

Added data for Tf LeakyReLU Test #844

Merged
merged 2 commits into from
Feb 4, 2021
Merged

Added data for Tf LeakyReLU Test #844

merged 2 commits into from
Feb 4, 2021

Conversation

jebastin-nadar
Copy link
Contributor

Test data for OpenCV PR : opencv/opencv#19449

@sl-sergei
Copy link
Contributor

@SamFC10 If you modified testdata/dnn/tensorflow/generate_tf_models.py, please, add these changes to this PR too

@jebastin-nadar
Copy link
Contributor Author

@sl-sergei I had no idea I had to change generate_tf_models.py as well. Sorry about that!

Also, should I wait for this PR to be merged before pushing an empty commit in opencv/opencv#19449? As you mentioned, the build fails as the scripts can't find the new data present in this PR.

@sl-sergei
Copy link
Contributor

@sl-sergei I had no idea I had to change generate_tf_models.py as well. Sorry about that!

Also, should I wait for this PR to be merged before pushing an empty commit in opencv/opencv#19449? As you mentioned, the build fails as the scripts can't find the new data present in this PR.

No, you don't need to wait, buildbot will use this branch, see opencv_extra=tf-leakyrelu in your first comment in opencv/opencv#19449. Please, push any commit there to restart tests

@alalek alalek merged commit 49adb8d into opencv:3.4 Feb 4, 2021
@jebastin-nadar jebastin-nadar deleted the tf-leakyrelu branch February 5, 2021 04:38
@alalek alalek mentioned this pull request Feb 5, 2021
@alalek alalek mentioned this pull request Apr 9, 2021
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