Skip to content

Update PR template for expected changes#4278

Merged
DavidHuber-NOAA merged 5 commits intodevelopfrom
update_pr_template
Dec 5, 2025
Merged

Update PR template for expected changes#4278
DavidHuber-NOAA merged 5 commits intodevelopfrom
update_pr_template

Conversation

@DavidHuber-NOAA
Copy link
Contributor

@DavidHuber-NOAA DavidHuber-NOAA commented Nov 26, 2025

Description

This updates the PR template to include checklist items for expected changes.

Type of change

  • Maintenance (code refactor, clean-up, new CI test, etc.)

Change characteristics

  • Is this a breaking change (a change in existing functionality)? NO
  • Does this change require a documentation update? NO
  • Does this change require an update to any of the following submodules? NO

How has this been tested?

Rendered template

@DavidHuber-NOAA
Copy link
Contributor Author

@DavidHuber-NOAA
Copy link
Contributor Author

@JessicaMeixner-NOAA @CatherineThomas-NOAA @RuiyuSun could you provide guidance on what constitutes a 'change to baselines'? Would this include just changes to model/analysis answers or product contents? Or would it also include things like filename changes?

@RuiyuSun
Copy link

@JessicaMeixner-NOAA @CatherineThomas-NOAA @RuiyuSun could you provide guidance on what constitutes a 'change to baselines'? Would this include just changes to model/analysis answers or product contents? Or would it also include things like filename changes?

@DavidHuber-NOAA We need to watch for any changes from inputs to outputs and archives. So, I would say what you mentioned above should all be included.

@RussTreadon-NOAA
Copy link
Contributor

Question: Will baseline output be stored somewhere for developers to compare against? Otherwise, developers will need to run g-w CI twice (once for develop and a second time for the PR) and then compare the two sets of output to document if/how the PR alters the baseline?

@aerorahul
Copy link
Contributor

We should not update the PR template to enforce a check against a baseline.
The global-workflow does not do baseline testing at this moment, and that feature is far from being
executed. Baseline testing is extremely expensive and intensive to perform with a workflow that supports these many applications, configurations, and options. This feature needs to be carefully planned and executed, otherwise we risk falling in the same trap as some of the other components that do regression testing.

In addition, considerably more work is required to operationally harden the workflow for GFSv17 and that will include updates to filenames. We also have not concluded the archiving work as we don't yet have a full list of what files need to be archived to partition the tarballs into its appropriate groups.

FWIW, thats my feedback.

@RuiyuSun
Copy link

@DavidHuber-NOAA Thank you for starting the PR to fulfill the request we made to watch for the incoming changes. @aerorahul @DavidHuber-NOAA You are the experts. What do you think is the best way to monitor the changes that could affect the GFSv17 at current stage?

@DavidHuber-NOAA DavidHuber-NOAA changed the title Update PR template for baseline changes Update PR template for expected changes Dec 3, 2025
@DavidHuber-NOAA DavidHuber-NOAA merged commit 8ae8674 into develop Dec 5, 2025
8 checks passed
JessicaMeixner-NOAA pushed a commit to JessicaMeixner-NOAA/global-workflow that referenced this pull request Dec 9, 2025
This updates the PR template to include checklist items for expected
changes.
@DavidHuber-NOAA DavidHuber-NOAA deleted the update_pr_template branch December 29, 2025 18:51
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.

4 participants