Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add example to show how to overlay off-disk contours from one map onto another map#8012

Merged
nabobalis merged 44 commits into
sunpy:mainfrom
tanishy7777:new_example
Feb 20, 2025
Merged

Add example to show how to overlay off-disk contours from one map onto another map#8012
nabobalis merged 44 commits into
sunpy:mainfrom
tanishy7777:new_example

Conversation

@tanishy7777
Copy link
Copy Markdown
Contributor

@tanishy7777 tanishy7777 commented Jan 14, 2025

Fixes: #6506

Adds example to show that SphericalScreen can be used even without reprojection.

assume_spherical_screen seems to be deprecated and even the current examples in the example gallery seem to be using
sunpy.coordinates.screens.SphericalScreen and they do seem to be used with reprojection only. (#6506 (comment))

@tanishy7777 tanishy7777 requested a review from a team as a code owner January 14, 2025 15:28
Comment thread changelog/8012.doc.rst Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
@nabobalis nabobalis marked this pull request as draft January 14, 2025 17:06
@nabobalis nabobalis added Examples Related to the Example Gallery. Documentation Affects the documentation. backport 6.0 and removed backport 6.0 labels Jan 14, 2025
Comment thread examples/plotting/add_countour_with_Spherical_screen.py Outdated
@nabobalis nabobalis requested a review from wtbarnes January 16, 2025 18:11
Comment thread changelog/8012.doc.rst Outdated
@ayshih ayshih changed the title Add example to show that SphericalScreen can be used even without reprojection Add example to show how to overlay off-disk contours from one map onto another map Feb 18, 2025
@tanishy7777
Copy link
Copy Markdown
Contributor Author

I've overhauled the example, including splitting the plot into three plots. Please also rename the example file to offdisk_contours.py.

I have renamed it to offdisk_contours.py

@nabobalis nabobalis requested review from ayshih and removed request for wtbarnes February 18, 2025 22:18
Comment thread examples/plotting/offdisk_contours.py Outdated
Comment thread examples/plotting/offdisk_contours.py Outdated
Comment thread examples/plotting/offdisk_contours.py Outdated
Comment thread examples/plotting/offdisk_contours.py Outdated
Comment thread examples/plotting/offdisk_contours.py Outdated
Comment thread examples/plotting/offdisk_contours.py Outdated
Comment thread examples/plotting/offdisk_contours.py Outdated
Comment thread examples/plotting/offdisk_contours.py Outdated
Copy link
Copy Markdown
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.

Looks good to me!

Note to merger: please squash

@nabobalis nabobalis merged commit 266a324 into sunpy:main Feb 20, 2025
@nabobalis
Copy link
Copy Markdown
Member

Thanks again @tanishy7777 and @ayshih

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Affects the documentation. Examples Related to the Example Gallery. No Backport A PR that isn't to be backported to any release branch. (To be used as a flag to other maintainers)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add draw_contours + assume_spherical_screen example

3 participants