Skip to content

ci: fix test runs that broke out of nowhere#151

Merged
fmalatino merged 4 commits into
NOAA-GFDL:developfrom
romanc:romanc/ci-debug
Oct 16, 2025
Merged

ci: fix test runs that broke out of nowhere#151
fmalatino merged 4 commits into
NOAA-GFDL:developfrom
romanc:romanc/ci-debug

Conversation

@romanc
Copy link
Copy Markdown
Collaborator

@romanc romanc commented Oct 16, 2025

Description

CI broke again out of nowhere today. This seems to stabilize CI for now.

Front-loading numpy itself didn't make a difference, but is good practice. The radiation scheme from pySHiELD will pick it up if it's installed already in the environment.

Replacing the miniconda setup with a plain python GHA setup seems to do the trick. I have to admit that I don't fully understand the breakage. My hunch is that the miniconda setup somehow interfered with how pip resolves dependencies and then either two incompatible packages were installed or there were somehow two different python environments floating around.

How Has This Been Tested?

CI failed before (see this run that added just an empty file) and is running green again with these changes.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
    N/A
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
    N/A
  • New check tests, if applicable, are included
    N/A

@romanc romanc marked this pull request as ready for review October 16, 2025 12:27
@romanc romanc changed the title testing - let's see if any PR will break CI ci: fix test runs that broke out of nowhere Oct 16, 2025
@fmalatino fmalatino added this pull request to the merge queue Oct 16, 2025
Merged via the queue into NOAA-GFDL:develop with commit c584ef0 Oct 16, 2025
2 checks passed
@romanc romanc deleted the romanc/ci-debug branch October 16, 2025 13:17
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.

3 participants