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

Skip to content

Fix GONGMagnetogramMap for correct DATE-OBS#8347

Merged
Cadair merged 2 commits into
sunpy:mainfrom
Cadair:fix_gong_mag_date
Sep 30, 2025
Merged

Fix GONGMagnetogramMap for correct DATE-OBS#8347
Cadair merged 2 commits into
sunpy:mainfrom
Cadair:fix_gong_mag_date

Conversation

@Cadair
Copy link
Copy Markdown
Member

@Cadair Cadair commented Sep 17, 2025

No description provided.

@Cadair Cadair added the backport 7.0 on-merge: backport to 7.0 label Sep 17, 2025
Comment thread sunpy/map/sources/gong.py Outdated
@Cadair Cadair requested a review from ayshih September 18, 2025 10:14
This allows permuting the header before making the map
@Cadair
Copy link
Copy Markdown
Member Author

Cadair commented Sep 29, 2025

@ayshih ping

Copy link
Copy Markdown
Member

@samaloney samaloney left a comment

Choose a reason for hiding this comment

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

LGTM

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.

Approved conditioned on a fix

Comment thread changelog/8347.bugfix.rst Outdated
@Cadair Cadair merged commit ec3a936 into sunpy:main Sep 30, 2025
27 of 29 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/sunpy that referenced this pull request Sep 30, 2025
@Cadair

This comment was marked as outdated.

@lumberbot-app

This comment was marked as outdated.

@lumberbot-app lumberbot-app Bot added the Still Needs Manual Backport This PR needs manually backporting. label Sep 30, 2025
@Cadair Cadair deleted the fix_gong_mag_date branch September 30, 2025 09:30
@Cadair Cadair removed the Still Needs Manual Backport This PR needs manually backporting. label Sep 30, 2025
Cadair added a commit that referenced this pull request Sep 30, 2025
…7-on-7.0

Backport PR #8347 on branch 7.0 (Fix GONGMagnetogramMap for correct DATE-OBS)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 7.0 on-merge: backport to 7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants