Skip to content

Add test data for Convolution3D layer#594

Merged
alalek merged 2 commits intoopencv:3.4from
l-bat:conv3d
Apr 30, 2019
Merged

Add test data for Convolution3D layer#594
alalek merged 2 commits intoopencv:3.4from
l-bat:conv3d

Conversation

@l-bat
Copy link
Copy Markdown
Contributor

@l-bat l-bat commented Apr 11, 2019

No description provided.

@alalek
Copy link
Copy Markdown
Member

alalek commented Apr 11, 2019

Don't re-open PRs. There are almost no problems which can't be fixed without re-opening PR.
CI doesn't look on PRs from opencv_extra at all. Only branches are taken into account for testing.
Builds are triggered by pushing commits into the branch of the main PR only.

opencv_extra's PR is necessary for reviewing and merging.


Merge branch '3.4'

These commits should not be here. You need to rebase your branch.

model.eval()
save_data_and_model("unsqueeze", input, model)

input = Variable(torch.randn(1, 3, 8, 12, 16))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can do it smaller?

1*3*8*12*16*4 ~ 18.5KB

but, in exampe

input = Variable(torch.randn(1, 2, 3, 4, 5))
1*2*3*4*5*4 ~ 0.48KB (x38 smaller but also has vary dimensions).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


################################################################################

inp = tf.placeholder(tf.float32, [1, 8, 12, 16, 3], 'input') # NDHWC format
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we reduce test data here as well?

@dkurt
Copy link
Copy Markdown
Member

dkurt commented Apr 25, 2019

@l-bat, please resolve merge conflicts.

@alalek alalek merged commit 7313fbb into opencv:3.4 Apr 30, 2019
@alalek alalek mentioned this pull request Apr 30, 2019
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