-
-
Notifications
You must be signed in to change notification settings - Fork 54
docs: add example using reproject to change wcs resolution #893
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
base: main
Are you sure you want to change the base?
Conversation
|
hmm sorry to be a stopper here, but should we really be encouraging people to upscale data? it may come across that you are getting "better" resolution which is not true? especially with an aia example? I like idea having some docs to show functionality - how about downscaling? |
|
Fair point, i've updated it to be simply "changing resolution"
the context is apparently imaging spectroscopists do this, but at least one
was doing it on the data array hence the motivation for how to do it while
preserving wcs
…On Tue, Nov 4, 2025 at 4:04 PM Laura Hayes ***@***.***> wrote:
*hayesla* left a comment (sunpy/ndcube#893)
<#893 (comment)>
hmm sorry to be a stopper here, but should we really be encouraging people
to upscale data? it may come across that you are getting "better"
resolution which is not true? especially with an aia example?
I like idea having some docs to show functionality - how about downscaling?
—
Reply to this email directly, view it on GitHub
<#893 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOMOKZTODNHPX7M6YEX2IF333DE7LAVCNFSM6AAAAACKVGA43WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIOBWG43TEMJSG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
| Changing the resolution of an NDCube | ||
| ===================================== | ||
|
|
||
| This example shows how to change the resolution of an NDCube by reprojecting to a finer grid. |
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.
I think to what @hayesla said we should talk about when to use this and when to use rebin here, and a big health warning etc.
| image_data = fits.getdata(AIA_171_IMAGE) | ||
| image_header = fits.getheader(AIA_171_IMAGE) |
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.
| image_data = fits.getdata(AIA_171_IMAGE) | |
| image_header = fits.getheader(AIA_171_IMAGE) | |
| image_data = fits.getdata(AIA_171_IMAGE, ext=1) | |
| image_header = fits.getheader(AIA_171_IMAGE, ext=1) |
Thoughts on this very simple upscaling example?
I also added a new piece of error handling to rebin as the current message when you use scale factors > 1 is misleading
(e.g. 2 is an integer factor)