Skip to content

Fix fetching of hdw files.#416

Merged
RemingtonRohel merged 1 commit into
developfrom
hotfix/hdw_dir
May 13, 2025
Merged

Fix fetching of hdw files.#416
RemingtonRohel merged 1 commit into
developfrom
hotfix/hdw_dir

Conversation

@RemingtonRohel

Copy link
Copy Markdown
Collaborator

Scope

Was failing if hdw/ directory not present - simply added a line to make the directory if not present.

issue: #410

Test

Pull this branch, delete the hdw/ dir, then try importing pydarn from a Python session and confirm that it works.

* Was failing if hdw/ directory not present - simply added a line to make the directory if not present.
@RemingtonRohel

Copy link
Copy Markdown
Collaborator Author

I've tested it on my computer, with no hdw/ dir, an empty hdw/ dir, and a non-empty hdw/ dir. All work as expected, I can import pydarn without trouble and those that need hdw files execute a download as expected.

@RemingtonRohel RemingtonRohel merged commit 0002af5 into develop May 13, 2025
2 checks passed
@RemingtonRohel RemingtonRohel deleted the hotfix/hdw_dir branch May 13, 2025 20:55
RemingtonRohel added a commit that referenced this pull request May 16, 2025
* Fix fetching of hdw files. (#416)
* Was failing if hdw/ directory not present - simply added a line to make the directory if not present.
* Update version and README for new release.
* Updating institution (#415) (#418)
Changed UAF to Penn State for the applicable radars.
Co-authored-by: Kevin Sterne <kevintyler@vt.edu>

* Updated KSR ownership to Penn State
* Updated pydarnio version dependency to avoid incompatibility at next pydarnio release

* Updated README to include Penn State radar updates in release description.

---------

Co-authored-by: Carley <60905856+carleyjmartin@users.noreply.github.com>
Co-authored-by: carleyjmartin <carleymartin@hotmail.com>
Co-authored-by: Daniel Billett <ddb524@usask.ca>
Co-authored-by: Kevin Sterne <kevintyler@vt.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant