Skip to content

Comments

Release RST 5.1.1#649

Merged
ksterne merged 19 commits intomainfrom
release/rst-5.1.1
Dec 11, 2025
Merged

Release RST 5.1.1#649
ksterne merged 19 commits intomainfrom
release/rst-5.1.1

Conversation

@ksterne
Copy link
Contributor

@ksterne ksterne commented Nov 3, 2025

Release RST 5.1.1

  • PATCH
  • MINOR
  • MAJOR

This is the start of the 5.1.1 release pull request which I'm going off the template used for the 5.1 release pull request. Also need to consult the release guidelines/checklist that we made up a few years ago (and update some of the guidelines as discussed during the workshop). Here's the key changes for this release:

  • New additions

  • New/Updated Radar Information

    • Corrected radar names for Siziwang radars (sze, szw)
    • Added Hokkaido East (hok) tdiff value after data analysis
    • Updated Hankasalmi (han) info for Borealis installation
    • Updated SANAE (san) info for Mabel installation
    • Updated Iceland East, West (ice, icw) info after electronics upgrades.
  • Updates to Current Binaries and Libraries

  • Documentation, Bug fixes and Miscellaneous

    • Following discussions around the 2025 SuperDARN Workshop and following Data Analysis Working Group meetings, the author list has been updated to include all known previous authors.

Deadline: 10 November 2025

Please comment in the Pull Request conversation if you need additional time to test the release.

Testing

  • Mac OSx
  • Linux deb - Debian, Ubuntu, Mint
  • Linux rpm - OpenSuse, Fedora, CentOS

Do any binaries/libraries require special attention during testing?

Checklist

  • Update radar.dat (if required) Check closer to the testing deadline
  • Update hardware files (if required) Check closer to the testing deadline
  • Update .rst.version
  • Update author list in .zenodo.json check again before merging
  • Update author list in AUTHORS.md

Extra Notes

A description of the release will appear soon.

Approving the release

After testing, please state which operating system you used and what you tested.

If you found no bugs during testing, please approve the PR.

@ksterne
Copy link
Contributor Author

ksterne commented Nov 5, 2025

Hello all, I've added two updates to the release branch. First is simple of updating the version number to 5.1.1. The second is based on the conversation at the workshop with regards to authorship where I've added all of the names listed in the AUTHORS.md file to the Zenodo json citation. I pulled ORCIDs and affliations where I could. It might be good to double check I didn't misplace a comma in this json file so that it works correctly before we make the release.

@ksterne
Copy link
Contributor Author

ksterne commented Nov 13, 2025

For those not on the e-mail thread, there's been some discussion about a value to use in updating the hdw.dat.hok file that arose late on Nov. 9th/early on the 10th. I'm waiting resolution on this before making an update and then making a release.

Copy link
Contributor Author

@ksterne ksterne left a comment

Choose a reason for hiding this comment

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

For future reference, does the spacing here matter for parsing the info or was this more for readability? I don't have a problem, but just want to make sure I don't cause a problem in the future.

(Sorry, I meant for this to reference 2ff5a09 but seems as though it didn't link)

@egthomas
Copy link
Member

For future reference, does the spacing here matter for parsing the info or was this more for readability? I don't have a problem, but just want to make sure I don't cause a problem in the future.

(Sorry, I meant for this to reference 2ff5a09 but seems as though it didn't link)

Yes, when I updated the geographic radar locations in the hdw.dat files several years ago the column spacing was made consistent across all of them.

@egthomas
Copy link
Member

Also, the Iceland radar hardware updates seem to be missing from both the release notes above and the DAWG report to the PIs.

@ksterne
Copy link
Contributor Author

ksterne commented Nov 14, 2025

OK, I was just hoping it wasn't a formatting issue for function, but agree that consistency and readability are good to have.

Apologies for forgetting the Iceland info and thanks for pointing it out. I really thought I had it in there the first time, so not sure what happened. I've added it to both places.

@egthomas
Copy link
Member

Should the SANAE radar's PI institution be updated from KwaZulu-Natal to SANSA? I think that change happened a few years ago.

@ksterne
Copy link
Contributor Author

ksterne commented Nov 25, 2025

Sorry for the delay, I've just e-mailed Judy Stephenson to confirm which institution should be used for the SANAE radar. I'm hoping to hear back in the next day or two and then get this release out.

@ksterne
Copy link
Contributor Author

ksterne commented Dec 2, 2025

I heard back from Judy late last week that both institutions (SANSA and UKZN) should be used for the SANAE radar as the PI has positions at both. I've updated the radar.dat file to reflect this change. I think we're good to go now. I'll leave this for a day unless someone else would like to merge it in. (Maybe do a double check that I updated the Zenodo authors correctly?)

@mtwalach
Copy link
Contributor

mtwalach commented Dec 2, 2025

FYI @ksterne I have been trying to install this clean on a Mac OS (working my way through #651) but got stuck and haven't had time to revisit. Don't want to let that stop you from merging but just thought I'd let you know where I was at.

@egthomas
Copy link
Member

egthomas commented Dec 2, 2025

There are also a few additional hardware file changes that we should discuss, so I would request that we hold off on merging this release until after this week's meeting.

@ksterne
Copy link
Contributor Author

ksterne commented Dec 2, 2025

@egthomas, is there something that you can preview or pass along that we should hold off on making this release?

@ksterne
Copy link
Contributor Author

ksterne commented Dec 11, 2025

Going for release now. I added a note at the end to reflect the change in authorship since this is the first release where all previous authors will be included in the Zenodo citation.

@ksterne ksterne merged commit 09bb159 into main Dec 11, 2025
3 of 5 checks passed
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.

5 participants