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

Added test_resample.py, tests for resample and reshape_image_to_4d_superpixel. #512

Merged
merged 5 commits into from
Aug 13, 2013
Merged

Added test_resample.py, tests for resample and reshape_image_to_4d_superpixel. #512

merged 5 commits into from
Aug 13, 2013

Conversation

examon
Copy link
Contributor

@examon examon commented Jul 29, 2013

Here are tests for resample and reshape_image_to_4d_superpixel.

from sunpy.image.rescale import reshape_image_to_4d_superpixel


AIA_MAP = make_map(AIA_171_IMAGE)
Copy link
Member

Choose a reason for hiding this comment

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

this should use Map not make_map, so it should be AIA_MAP = sunpy.map.Map(AIA_171_IMAGE)

@Cadair
Copy link
Member

Cadair commented Jul 30, 2013

Looks good to me!!

@ghost ghost assigned wafels Jul 30, 2013
@Cadair
Copy link
Member

Cadair commented Jul 30, 2013

This now fixes #432 and #213.

@wafels
Copy link
Member

wafels commented Aug 12, 2013

Tests pass. Thanks!

@Cadair
Copy link
Member

Cadair commented Aug 12, 2013

@wafels Good to go?!

@wafels
Copy link
Member

wafels commented Aug 13, 2013

good to go.

On Mon, Aug 12, 2013 at 5:17 PM, Stuart Mumford notifications@github.comwrote:

@wafels https://github.com/wafels Good to go?!


Reply to this email directly or view it on GitHubhttps://github.com//pull/512#issuecomment-22526155
.

Cadair added a commit that referenced this pull request Aug 13, 2013
Added test_resample.py, tests for resample and reshape_image_to_4d_superpixel.
@Cadair Cadair merged commit 2c7eaa6 into sunpy:0.3 Aug 13, 2013
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.

None yet

4 participants