Giter Site home page Giter Site logo

Comments (5)

yakir12 avatar yakir12 commented on June 15, 2024 2

I totally agree.
I'll give it a shot ASAP.

from imagefeatures.jl.

timholy avatar timholy commented on June 15, 2024

The Hough transform doesn't require that a line be continuous; take a few pixels out of the middle, and you get almost exactly the same transform. That's actually part of its point, it can identify lines in noisy images. So what does "per line" actually mean?

from imagefeatures.jl.

yakir12 avatar yakir12 commented on June 15, 2024

I mixed the terminologies, where "lines" should have been "pairs of r and θ". I've now edited my post, and hope it's clearer. My main point was that having access to the votes helps analyzing which pairs of r and θ belong to which line in the image and what their contribution to that line is. This knowledge can help improve the line estimates.

from imagefeatures.jl.

timholy avatar timholy commented on June 15, 2024

Maybe more generally, I think the current implementation mixes two things: I think there's a strong case to be made that accumulator_matrix is the Hough transform, and that what's currently being returned is the result of some analysis of the transform. Want to take a crack at a better API? Breaking changes are totally fine, we'll just bump the minor version number.

from imagefeatures.jl.

yakir12 avatar yakir12 commented on June 15, 2024

So I took a look, and while it should be straight forward to separate between generating the accumulator_matrix and the specific method of detecting line-candidates, I can't find a ρ range. There is a range for the angles, but not for the distances. Is this an implementation detail, and it's simply range(0, step = stepsize, length = numrho + 2), or is it because the distances are different per angle?

It's worth mentioning that the current implementation is pretty smart about avoiding saving a large matrix of useless r and θ pairs. So in that respect it seems like a shame to move away from that efficiency... With that in mind, maybe it would be good to simply add the vote value to each pair, opening up the possibility to the kind of post-analysis I mentioned above. I'll give that a look.

from imagefeatures.jl.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.