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

Gsoc opencv project : The Fast Bilateral Filter #1317

Closed
wants to merge 42 commits into from

Conversation

kuan-wang
Copy link

@kuan-wang kuan-wang commented Aug 10, 2017

Add the source file for FastBilateralSolverFilter(located in modules/ximgproc/src/fbs_filter.cpp), and add the class declaration to the file modules/ximgproc/include/opencv2/ximgproc/edge_filter.hpp.

the test file and some sample(depth and colorization) for FastBilteralSolverFilter will be add next week.

TODO list:

  • performance test

@sovrasov sovrasov added the GSoC label Aug 10, 2017
@kuan-wang kuan-wang closed this Aug 12, 2017
@kuan-wang kuan-wang deleted the gsoc_opencv branch August 12, 2017 06:42
@kuan-wang kuan-wang restored the gsoc_opencv branch August 12, 2017 06:43
@kuan-wang kuan-wang reopened this Aug 12, 2017
@alalek
Copy link
Member

alalek commented Aug 12, 2017

@THUKey No need to re-open PRs. Just push updates on your branch.

typedef std::map<long long /* hash */, int /* vert id */> mapId;
#else
#include <boost/unordered_map.hpp>
typedef boost::unordered_map<long long /* hash */, int /* vert id */> mapId;
Copy link
Member

Choose a reason for hiding this comment

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

OpenCV should not depend on "boost" library



// int debugn = blurs.nonZeros(); //FIXME: if don't call nonZeros(), the result will be destroy
// debugn = 0;
Copy link
Member

Choose a reason for hiding this comment

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

dead code?

*colorPix = *gray_pix;
colorPix++;
}
gray_pix++;
Copy link
Member

Choose a reason for hiding this comment

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

cvtColor(gray, color, COLOR_GRAY2RGB)

{
Size sz(rnd.uniform(512, 1024), rnd.uniform(512, 1024));

int guideCn = rnd.uniform(1, 2);
Copy link
Member

Choose a reason for hiding this comment

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

result is always 1 here: "a <= x < b"

Ptr<FastBilateralSolverFilter> createFastBilateralSolverFilter(InputArray, double, double, double, int, double)
{
std::cout << "ERROR createFastBilateralSolverFilter : don't have eigen" << '\n';
exit(0);
Copy link
Member

Choose a reason for hiding this comment

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

CV_Error(Error::StsNotImplemented, ...)

@savuor savuor self-assigned this Feb 16, 2018
@berak berak mentioned this pull request Oct 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants