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.

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

Who uses the code in driver/fvGFS/atmosphere.F90 #314

Closed
climbfuji opened this issue Jan 20, 2024 · 5 comments
Closed

Who uses the code in driver/fvGFS/atmosphere.F90 #314

climbfuji opened this issue Jan 20, 2024 · 5 comments
Labels
question Further information is requested

Comments

@climbfuji
Copy link

Is your question related to a problem? Please describe.
I am looking at reorganizing the data storage model for the GFS data types in the UFS / FV3ATM. Doing that requires making changes in driver/fvGFS/atmosphere.F90. I am wondering who else than the UFS or other models that inherited the GFS data types (e.g. because of IPD first, then CCPP) uses this code. In other words, if I am making changes there, do I need to wrap all of them in #ifdeg GFS_TYPES?

If nobody outside the UFS-CCPP world uses this part of the code (somewhat suggested by the subdirectory fvGFS), then my life will be a lot easier and we can also simplify the code quite a bit (less ifdef = better).

Describe what you have tried
n/a

@climbfuji climbfuji added the question Further information is requested label Jan 20, 2024
@wramstrom
Copy link

The HAFS moving nest code in the FV3/moving_nest directory uses the IPD structures, especially in moving_nest/fv_moving_nest_physics.F90. I am happy to discuss further with you to coordinate any needed changes here.

@bensonr
Copy link
Contributor

bensonr commented Jan 22, 2024

@climbfuji - each modeling system will have its own version of atmosphere.F90. If you look at the main branch, you'll see versions for SHiELD, GFDL climate, and a solo models. While the name is still driver/fvGFS, driver/UFS would probably now be more appropriate.

@climbfuji
Copy link
Author

Thanks for your quick feedback @bensonr and @wramstrom. So, if I understand you correctly, as long as I make sure that driver/UFS/* (I am going to rename that as part of the work) works for any of the UFS applications using the FV3 dynamical core, then we are ok; I do not need to worry about other modeling systems so long as I only touch the files in these directories.

@bensonr
Copy link
Contributor

bensonr commented Jan 30, 2024

@climbfuji - yes, that is correct.

@bensonr
Copy link
Contributor

bensonr commented Aug 8, 2024

PR #330 merged into dev/emc

@bensonr bensonr closed this as completed Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants