Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Field table yaml updates #52

Merged
merged 14 commits into from
Jul 30, 2024
Merged

Field table yaml updates #52

merged 14 commits into from
Jul 30, 2024

Conversation

uramirez8707
Copy link
Contributor

@uramirez8707 uramirez8707 commented Jul 16, 2024 •
edited
Loading

  1. Update the field table to yaml conversion to the new field_table.yaml format: Bettereliminatesubparams #45
  2. Update the field table converter and the combiner to be flake8 compliant
  3. Update the field table converter and the combiner to use click
  4. Add a debug option to the field table converter and the combiner

Eric Stofferahn and others added 11 commits May 6, 2024 12:58
Eliminated subparams reference in .yaml file. This will require rewor…
a8eafe3 
…ks to fm_yaml and potentially field_manager.F90
Eliminated subparams reference in .yaml file. This will require rewor…
ec97713 
…ks to fm_yaml and potentially field_manager.F90
Eliminated subparams reference in .yaml file. This will require rewor…
c32e3d6 
…ks to fm_yaml and potentially field_manager.F90
@uramirez8707
Merge branch 'bettereliminatesubparams' of github.com:GFDL-Eric/fms_y…
6307bd3 
…aml_tools into HEAD
@uramirez8707
make script flake8 compliant
3145078
@uramirez8707
move field_table_to_yaml converter to use click
e5c91ea
@uramirez8707
Make combine_field_table_yamls flake8 compliant
298097e
@uramirez8707
make the field table combiner flake8 compliant and move to click
4f7e499
@uramirez8707
implement the debug option to the combine field table yaml
6f9a481
@uramirez8707
Add a missing debug statement
5f8a5b6
@uramirez8707
remove unused function
4ff244d
This was referenced Jul 29, 2024
Closed
Merged
J-Lentz
J-Lentz requested changes Jul 30, 2024
View reviewed changes
setup.cfg Outdated Show resolved Hide resolved
@uramirez8707 uramirez8707 merged commit c4a6be7 into NOAA-GFDL:main Jul 30, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@J-Lentz J-Lentz J-Lentz approved these changes

@ceblanton ceblanton Awaiting requested review from ceblanton ceblanton is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@uramirez8707 @J-Lentz @rem1776