Skip to content

Test against python 3.14#625

Open
bhazelton wants to merge 1 commit into
mainfrom
test_python_3.14
Open

Test against python 3.14#625
bhazelton wants to merge 1 commit into
mainfrom
test_python_3.14

Conversation

@bhazelton

Copy link
Copy Markdown
Member

Description

Add CI jobs with python 3.14 to ensure compatibility.

Motivation and Context

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Reference simulation update or replacement
  • Documentation change (documentation changes only)
  • Version change
  • Build or continuous integration change

Checklist:

For all pull requests:

Build or continuous integration change checklist:

  • If required or optional dependencies have changed (including version numbers), I have updated the readme to reflect this.
  • If this is a new CI setup, I have added the associated badge to the readme and to references/make_index.py (if appropriate).

@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (ef3bb28) to head (b538b01).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #625   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines         2455      2455           
=========================================
  Hits          2455      2455           

☔ View full report in Codecov by Harness.
📢 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 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 671fe53 Previous: a1f5f5f Ratio
tests/test_run_ref.py::test_run_sim[1.1_baseline_number] 0.29789669278930003 iter/sec (stddev: 0.10408227506529867) 0.4723100939736393 iter/sec (stddev: 0.08542779550019207) 1.59
tests/test_run_ref.py::test_run_sim[1.7_multi_beam] 0.011044262286891895 iter/sec (stddev: 0.024316630751507425) 0.013384082677412444 iter/sec (stddev: 0.3604262521317503) 1.21

This comment was automatically generated by workflow using github-action-benchmark.

@bhazelton

Copy link
Copy Markdown
Member Author

The warnings tests are failing due to IERS warnings. The astropy team are working on a fix that looks like it will be a real improvement in behavior: astropy/astropy#19995

@bhazelton bhazelton requested a review from steven-murray July 1, 2026 19:52

@steven-murray steven-murray left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to me. If we merge HERA-Team/hera_sim#393 then hopefully we get the herasim tests to work as well.

@bhazelton

Copy link
Copy Markdown
Member Author

This looks good to me. If we merge HERA-Team/hera_sim#393 then hopefully we get the herasim tests to work as well.

I just approved that one, sorry, didn't realize you'd updated it and were waiting on me.

@steven-murray

Copy link
Copy Markdown
Contributor

Yay it worked!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants