Skip to content

Conversation

@FedericoNegri
Copy link
Contributor

@FedericoNegri FedericoNegri commented Aug 1, 2025

Prep in view of a new release targeting the upcoming HPS 1.3.

  • Adding v1.3.45 as HPS version we can run tests against

  • Updated the compatibility table (last row and last column are new)
    image

  • Added backward compatibility tests to make sure that the current PyHPS is still compatible with HPS 1.2 as documented. This is true apart for:

    • task definition template changes
    • debug field in the task definition
    • changes to output parameters

    Modified the tests to report explicit version dependent expected failures. Reasons for additional skipped/x-failed tests are reported in the pytest summary
    image

@codecov
Copy link

codecov bot commented Aug 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.64%. Comparing base (24d96a6) to head (d71debe).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #636      +/-   ##
==========================================
+ Coverage   87.62%   87.64%   +0.02%     
==========================================
  Files          65       65              
  Lines        2999     3004       +5     
==========================================
+ Hits         2628     2633       +5     
  Misses        371      371              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions bot added the maintenance Package and maintenance related label Aug 1, 2025
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Aug 6, 2025
@FedericoNegri FedericoNegri changed the title Add HPS 1.3 release option Prep for new release targeting the upcoming HPS 1.3 Aug 6, 2025
@FedericoNegri FedericoNegri marked this pull request as ready for review August 6, 2025 11:27
@FedericoNegri FedericoNegri requested a review from ojkoenig August 6, 2025 11:29
Copy link
Contributor

@ojkoenig ojkoenig left a comment

Choose a reason for hiding this comment

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

Reviewed the changes, very happy that you go all the efforts on version compatibility, even adding backward compatibility tests.

@FedericoNegri FedericoNegri merged commit c7840bc into main Aug 6, 2025
51 checks passed
@FedericoNegri FedericoNegri deleted the maint/add-1.3-release-version branch August 6, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation maintenance Package and maintenance related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants