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

Add image coalignment module #78

Merged
merged 170 commits into from
Mar 10, 2022
Merged

Conversation

wtbarnes
Copy link
Member

This PR adds a coalignment module originally in sunpy/image/coalignment.py. This addresses sunpy/sunpy#5425.

I've done my best to include the commit history of the coalignment code from sunpy here. Even if we squash merge, we should still include the text of these commits.

wafels and others added 30 commits March 15, 2014 10:55
…ed data. This should be more efficient in memory.
mapcube alignment is now a separate function
100% test coverage for coalignment code
@pep8speaks
Copy link

pep8speaks commented Mar 10, 2022

Hello @wtbarnes! Thanks for updating this PR.

Line 7:101: E501 line too long (107 > 100 characters)
Line 112:29: E203 whitespace before ':'
Line 112:79: E203 whitespace before ':'
Line 112:101: E501 line too long (106 > 100 characters)
Line 218:35: E203 whitespace before ':'
Line 219:35: E203 whitespace before ':'
Line 432:55: E203 whitespace before ':'
Line 432:86: E203 whitespace before ':'
Line 432:101: E501 line too long (104 > 100 characters)
Line 556:101: E501 line too long (106 > 100 characters)

Line 110:101: E501 line too long (104 > 100 characters)
Line 160:101: E501 line too long (103 > 100 characters)
Line 211:101: E501 line too long (102 > 100 characters)
Line 212:9: W503 line break before binary operator
Line 213:101: E501 line too long (102 > 100 characters)
Line 214:9: W503 line break before binary operator
Line 244:53: E203 whitespace before ':'
Line 244:76: E203 whitespace before ':'
Line 250:101: E501 line too long (105 > 100 characters)
Line 288:101: E501 line too long (107 > 100 characters)
Line 306:101: E501 line too long (107 > 100 characters)
Line 386:101: E501 line too long (103 > 100 characters)

Comment last updated at 2022-03-10 20:03:54 UTC

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.