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

expanding tracing example to extract intensities #244

Merged
merged 2 commits into from
Nov 11, 2024

Conversation

Abinash-bit
Copy link
Contributor

PR Description

This pull request enhances the existing example for tracing coronal loops using ~sunkit_image.trace.occult2 by adding functionality to extract and plot the intensity profiles along the traced loops. The primary goal is to demonstrate how to use the SkyCoord objects obtained from the tracing process to extract and visualize intensity profiles along the traced loops.

Key Changes

  1. Tracing Coronal Loops: The first part of the code remains the same, where we trace coronal loops using the occult2 function.
  2. Plotting Detected Loops: The detected loops are plotted on the original image.
  3. Extracting Intensities: The new part of the code extracts the intensities along the first detected loop. This is done by converting the loop's pixel coordinates to SkyCoord objects and then sampling the intensity along these coordinates.
  4. Plotting Intensity Profile: The intensity profile along the loop is plotted in a separate subplot, showing the intensity vs. angular distance.

Related Issues

This fixes #198

Screenshot 2024-11-10 234904

@nabobalis
Copy link
Contributor

@nabobalis please review.

Please do not @ individuals unless it's important or if a PR is not reviewed after a week.

@Abinash-bit
Copy link
Contributor Author

@nabobalis please review.

Please do not @ individuals unless it's important or if a PR is not reviewed after a week.

ohh sorry.

@nabobalis nabobalis force-pushed the trace-example-expanding branch from 21e3717 to 39112c8 Compare November 11, 2024 00:10
@nabobalis nabobalis added the No Changelog Entry Needed Skip any changelog checks. label Nov 11, 2024
@nabobalis
Copy link
Contributor

Thanks again @Abinash-bit

@nabobalis nabobalis merged commit bd9ae41 into sunpy:main Nov 11, 2024
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Changelog Entry Needed Skip any changelog checks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use coord tracing code in the trace example to return the loop intensities
2 participants