Skip to content

[develop] fix: update AQM data paths to avoid user directory - #1349

Merged
MichaelLueken merged 2 commits into
ufs-community:developfrom
benkozi:fix/derecho-aqm-data
Jan 6, 2026
Merged

[develop] fix: update AQM data paths to avoid user directory#1349
MichaelLueken merged 2 commits into
ufs-community:developfrom
benkozi:fix/derecho-aqm-data

Conversation

@benkozi

@benkozi benkozi commented Dec 17, 2025

Copy link
Copy Markdown
Collaborator

DESCRIPTION OF CHANGES:

  • Remove dependency on user paths for AQM-related data files.
  • Moves location of fix_emis on Derecho.

Type of change

  • 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 not work as expected)
  • This change requires a documentation update

TESTS CONDUCTED:

Note: Derecho is unstable with the SRW, and I have been unable to run a full workflow.

  • derecho.intel
  • gaeac6.intel
  • hera.gnu
  • hera.intel
  • hercules.intel
  • orion.intel
  • ursa.gnu
  • ursa.intel
  • NOAA Cloud (indicate which platform)
  • Jenkins
  • fundamental test suite
  • comprehensive tests (specify which if a subset was used)

DEPENDENCIES:

DOCUMENTATION:

ISSUE:

CHECKLIST

  • My code follows the style guidelines in the Contributor's Guide
  • I have performed a self-review of my own code using the Code Reviewer's Guide
  • I have commented my code, particularly in hard-to-understand areas
  • My changes need updates to the documentation. I have made corresponding changes to the documentation
  • My changes do not require updates to the documentation (explain).
  • My changes generate no new warnings
  • New and existing tests pass with my changes
  • Any dependent changes have been merged and published

LABELS (optional):

A Code Manager needs to add the following labels to this PR:

  • Work In Progress
  • bug
  • enhancement
  • documentation
  • release
  • high priority
  • run_ci
  • run_we2e_fundamental_tests
  • run_we2e_comprehensive_tests
  • help wanted

CONTRIBUTORS (optional):

@selbigmtnwx23 (data staging)

@benkozi benkozi self-assigned this Dec 17, 2025
@benkozi benkozi added the bug Something isn't working label Dec 17, 2025
@MichaelLueken

Copy link
Copy Markdown
Collaborator

The AQM WE2E test successfully completed on Derecho:

----------------------------------------------------------------------------------------------------
Experiment name                                                  | Status    | Core hours used 
----------------------------------------------------------------------------------------------------
aqm_grid_AQM_NA13km_suite_GFS_v16_20260105121950                   COMPLETE            4078.39
----------------------------------------------------------------------------------------------------
Total                                                              COMPLETE            4078.39

Since this PR is related to just the AQM on Derecho, I will approve this PR and merge it now.

@MichaelLueken MichaelLueken left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@benkozi -

Given that the AQM WE2E test successfully ran on Derecho, I will approve this PR and merge it now.

@MichaelLueken
MichaelLueken merged commit 75337cc into ufs-community:develop Jan 6, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in SRW App's PR Reviewers Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants