Skip to content

added missing impl for multi-dim Mat::ones, Mat::zeros ( issue #3756)#2883

Merged
opencv-pushbot merged 1 commit intoopencv:2.4from
berak:b_3756_24
Jul 9, 2014
Merged

added missing impl for multi-dim Mat::ones, Mat::zeros ( issue #3756)#2883
opencv-pushbot merged 1 commit intoopencv:2.4from
berak:b_3756_24

Conversation

@berak
Copy link
Copy Markdown
Contributor

@berak berak commented Jun 20, 2014

No description provided.

@ilya-lavrenov ilya-lavrenov self-assigned this Jun 20, 2014
@ilya-lavrenov
Copy link
Copy Markdown
Contributor

👍

@SpecLad
Copy link
Copy Markdown

SpecLad commented Jun 25, 2014

I'd prefer to see some tests for this.

@berak
Copy link
Copy Markdown
Contributor Author

berak commented Jun 25, 2014

appended testcase to modules/core/test/test_mat.cpp

@ilya-lavrenov
Copy link
Copy Markdown
Contributor

@SpecLad, the test was added, any new questions or desires?

@SpecLad
Copy link
Copy Markdown

SpecLad commented Jul 3, 2014

The test itself looks fine, but I want to take another look at the main change (slightly later, though).

@SpecLad
Copy link
Copy Markdown

SpecLad commented Jul 8, 2014

Yeah, looks good.

@SpecLad
Copy link
Copy Markdown

SpecLad commented Jul 8, 2014

Except... I just merged another pull request, which makes this one fail to compile. Rebase, please.

@berak
Copy link
Copy Markdown
Contributor Author

berak commented Jul 8, 2014

no problem, already seen this coming.

last 2 buildbots did not seem to see my latest change, though

@opencv-pushbot opencv-pushbot merged commit 7b160fa into opencv:2.4 Jul 9, 2014
SpecLad pushed a commit that referenced this pull request Jul 9, 2014
@SpecLad
Copy link
Copy Markdown

SpecLad commented Jul 9, 2014

Those two don't rebuild automatically ATM.

@berak
Copy link
Copy Markdown
Contributor Author

berak commented Jul 9, 2014

ok. thanks for your patience with this !

@berak berak deleted the b_3756_24 branch July 9, 2014 07:52
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.

4 participants