-
Notifications
You must be signed in to change notification settings - Fork 20
Add Hough transform #32
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
Merged
Merged
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
0178e61
Ignore IntelliJ metadata files
gselzer c1952b1
Add Hough line transform
panovr 20eabaf
HoughLineTransformTest: Optimize performance
gselzer c67a7c2
HoughLineTransformTest: add missing javadoc
gselzer 32ed03b
HoughLineTransform: Clone peak storage array
gselzer d36739f
HoughLineTransform: fix rawType warning
gselzer 8462174
HoughTransform: remove Benchmark import
gselzer a1c8843
HoughLineTransform: Refactor into static methods.
gselzer 6f673ab
PickPeaks: Allow user input for min peak value.
gselzer d95ef41
PickPeaks: Translate votespace for correct output
gselzer 9885dd1
LineTest: Add PickPeaks regression test
gselzer ca8b4f5
HoughTransforms: Add slope/intercept converters
gselzer 45b2909
Cleanup / format
gselzer f232cc6
Improve line transform documentation
gselzer ecb8a4b
computeLength: allow larger return values
gselzer 1518e4f
defaultRho: simplify instructions to one-liner
gselzer 694205d
pickPeaks: loosen typing of parameters
gselzer af244cb
voteLines: Restrict voteSpace to only IntegerType
gselzer 71b4800
pickPeaks: add alternative (convenient) option
gselzer beb1144
voteLines: Relax typing of input RAI
gselzer 69eac4e
fixup! pickPeaks: add alternative (convenient) option
gselzer 867e1c7
voteLines: Remove unnecessary variables
gselzer 0532b1b
HoughLineTest: use small delta for slope and yint
gselzer 9d61af7
HoughTransforms: Loosen helper method parameters
gselzer 28a13da
HoughLine: Remove unnecessary array
gselzer f3309eb
PickPeaks: Fix translation bug
gselzer 195f163
PickPeaks: Rename to PickLinePeaks
gselzer 10dd6cd
VoteLines: Refactor parameter order
gselzer 31e63e1
VoteLines: Move check outside for-loop
gselzer 66dfa48
HoughTransforms: Remove unused import
gselzer 3908da7
HoughLineTest: Refactor to better input image
gselzer 3a62b4f
getVotespaceSize: Allow for non-default thetas
gselzer 11995a6
HoughTransforms: Formatting
gselzer 66998bc
VoteLines: Translate image for zeroed min
gselzer 3ee8755
VoteLines: Refactor LUTs
gselzer 8c79996
VoteLines: fix predicate bug
gselzer 32f685c
VoteLines: Refactor fRho/r declaration
gselzer 29df455
VoteLines: Remove cursor positions from for loop
gselzer abc6829
VoteLines: remove unnecessary float cast.
gselzer 8f274a6
HoughLineTest: use ArrayImgs to make votespace
gselzer f1188a9
HoughLineTest: Formatting
gselzer 2e8f6b8
VoteLines: Fix javadoc
gselzer File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,3 +2,7 @@ | |
| /.project | ||
| /.settings/ | ||
| /target/ | ||
|
|
||
| # IntelliJ | ||
| /.idea/ | ||
| *.iml | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be a separate commit, with message like "Ignore IntelliJ metadata files"