-
Notifications
You must be signed in to change notification settings - Fork 9
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
Least Squares Gaussian Fit + Gaussian Grid centroiding algos #89
Open
zeddie888
wants to merge
36
commits into
master
Choose a base branch
from
gaussian-fit
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 6 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
ae4c90d
Start implementing Least Squares Gaussian Fit 1D
zeddie888 c3cf0fa
Now using LM correctly...I think
zeddie888 4791623
Enable calculation for both x and y marginals
zeddie888 92f9478
merge
zeddie888 7dc98a7
Figured out how to use LM, finally
zeddie888 c812f3f
Cleanup some bugs
zeddie888 10c1b23
Test window sizes
zeddie888 ecc2147
Allow floating-point values for coordinates, my bad
zeddie888 df77c31
Floodfill as naive preprocessing
zeddie888 08d1d15
Unit test Gaussian helper functions
zeddie888 11b48bb
First try of 2D gaussian fit - still not good enough
zeddie888 7d308c6
smol syntax change
zeddie888 14e9dc2
more debugging LM
zeddie888 e725368
merge master
zeddie888 fa29b4b
sus Jacobian math
zeddie888 78cba87
LM working for 2D fit, implemented calculation of Jacobian
zeddie888 6ae68d2
Calculate Jacobian for 1D fit
zeddie888 bbcc314
make 1D and 2D fit use same preprocessing
zeddie888 c814be8
code cleanup
zeddie888 60ed8b2
replace double with float
zeddie888 82da274
label stars with guess of radius/magnitude
zeddie888 9cd89cf
Allow user to specify Gaussian fit window size through CLI
zeddie888 05f1fc3
Implement 5x5 Gaussian Grid
zeddie888 05fdcdc
Merge branch 'master' of https://github.com/UWCubeSat/lost into gauss…
zeddie888 5ee93ce
smol
zeddie888 59255ac
test weight for ggrid
zeddie888 c538ca5
Merge branch 'master' of https://github.com/UWCubeSat/lost into gauss…
zeddie888 c7908f0
Change magnitude of Gaussian methods to be floodfill size
zeddie888 420637f
Subtract noise from image BEFORE any other preprocessing
zeddie888 e0256f2
edge case temporary fix for cutoff=0
zeddie888 276eeb2
merge
zeddie888 3d29744
Add dynamic window resizing option for fitting algos
zeddie888 890efb9
disable tests since I got rid of function
zeddie888 9251622
subtract noise after computing cutoff
zeddie888 48b430d
don't modify original image
zeddie888 0bff6f5
Make point type a struct instead of a vector of int
markasoftware 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 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 |
---|---|---|
|
@@ -6,6 +6,9 @@ | |
*.png | ||
*.dat | ||
*.txt | ||
*.bin | ||
|
||
BSC5 | ||
|
||
/documentation/man-*.h | ||
|
||
|
This file contains 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
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 stuff shouldn't be changed