-
-
Notifications
You must be signed in to change notification settings - Fork 573
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
Fix bug when converting results from the HEKClient #7627
Conversation
See #5006 (comment) |
Thanks for the PR @bmampaey, can you add a changelog entry marking this as a bugfix? We have some details on the readme here: https://github.com/sunpy/sunpy/blob/main/changelog/README.rst |
Thanks for the changelog, can you tell me what query failed for you? |
I specified it in the issue #5006 (comment) It was from sunpy.net.hek import HEKClient
from sunpy.net import attrs as a
results = HEKClient().search(a.Time('2024-05-10', '2024-05-12'), a.hek.AR.NOAANum == 13664) |
…ime failed when some values where empty or missing
d74edc0
to
3d19583
Compare
Thanks for the PR @bmampaey |
…7-on-5.0 Backport PR #7627 on branch 5.0 (Fix bug when converting results from the HEKClient )
…7-on-5.1 Backport PR #7627 on branch 5.1 (Fix bug when converting results from the HEKClient )
PR Description
Fixes bug where convertion of results from the HEKClient to Astropy Time failed when some values where empty or missing