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

Adds suggestion in the example autoalign/reproject to use assume_spherical_screen() #6855

Merged
merged 11 commits into from Mar 22, 2023

Conversation

davnish
Copy link
Contributor

@davnish davnish commented Mar 17, 2023

This PR adds a suggestion in these two examples:

This suggestion tells the user that Off-disks HMI data is not retained here, and to retain the data they can use Helioprojective.assume_spherical_screen() to retain the data.

Merging this PR:
Closes #5988

@davnish davnish requested a review from a team as a code owner March 17, 2023 10:03
@davnish
Copy link
Contributor Author

davnish commented Mar 17, 2023

pre-commit.ci autofix

@nabobalis nabobalis requested a review from ayshih March 17, 2023 15:38
@nabobalis nabobalis added backport 4.0 Minor Change PR only needs one approval to merge Examples Affects the Example Gallery labels Mar 17, 2023
Copy link
Member

@ayshih ayshih left a comment

Choose a reason for hiding this comment

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

Some minor edits, otherwise good to go!

examples/map_transformations/autoalign_aia_hmi.py Outdated Show resolved Hide resolved
examples/map_transformations/autoalign_aia_hmi.py Outdated Show resolved Hide resolved
examples/map_transformations/autoalign_aia_hmi.py Outdated Show resolved Hide resolved
examples/map_transformations/reprojection_align_aia_hmi.py Outdated Show resolved Hide resolved
examples/map_transformations/autoalign_aia_hmi.py Outdated Show resolved Hide resolved
examples/map_transformations/reprojection_align_aia_hmi.py Outdated Show resolved Hide resolved
davnish and others added 6 commits March 22, 2023 20:58
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
Co-authored-by: Albert Y. Shih <ayshih@gmail.com>
@nabobalis nabobalis merged commit 8f5533c into sunpy:main Mar 22, 2023
3 of 4 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/sunpy that referenced this pull request Mar 22, 2023
@nabobalis
Copy link
Contributor

Thanks for the PR @davnish

meeseeksmachine pushed a commit to meeseeksmachine/sunpy that referenced this pull request Mar 22, 2023
nabobalis pushed a commit that referenced this pull request Mar 22, 2023
…lign/reproject to use `assume_spherical_screen()`) (#6878)

Co-authored-by: Nischal Singh <32027279+davnish@users.noreply.github.com>
nabobalis pushed a commit that referenced this pull request Mar 22, 2023
…lign/reproject to use `assume_spherical_screen()`) (#6877)

Co-authored-by: Nischal Singh <32027279+davnish@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Examples Affects the Example Gallery Minor Change PR only needs one approval to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suggest the use of assume_spherical_screen() in the autoalign/reproject examples
3 participants