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

Add diagnostic: pbo #95

Closed
jkrasting opened this issue Mar 30, 2015 · 5 comments
Closed

Add diagnostic: pbo #95

jkrasting opened this issue Mar 30, 2015 · 5 comments
Assignees
@jkrasting
Milestone
CMOR-ization of d...

Comments

@jkrasting
Copy link
Contributor

Add the 'pbo' diagnostic, where:

pbo = col_mass + pso
@jkrasting jkrasting added this to the CMOR-ization of diagnostics milestone Mar 30, 2015
@jkrasting jkrasting changed the title Add the 'pbo' diagnostic Add diagnostic: pbo Mar 30, 2015
@adcroft
Copy link
Collaborator

adcroft commented Mar 31, 2015

The current post_data for col_mass uses a temporary "mass" variable which could be subsequently updated to include "pso".

@jkrasting
Copy link
Contributor Author

Pass in the fluxes type and multiply by 'g'

@jkrasting jkrasting self-assigned this Apr 2, 2015
@jkrasting
Copy link
Contributor Author

I've run into a problem with adding in the surface pressure to this diagnostic.

See this commit.

I modified the interfaces to pass in the fluxes forcing type, but adding fluxes%p_surf causes the model to SIGSEV. When I looked closer, there are several different varieties of p_surf in the fluxes type.

Any suggestions on how to fix this?

@adcroft
Copy link
Collaborator

adcroft commented Apr 6, 2015

@jkrasting I think you need to add an "if associated(fluxes%ps_urf)" around the pso contribution.

@jkrasting
Copy link
Contributor Author

Issuing a pull request.

Here is the output from the ocean_only/global_ALE tests. This diagnostic should also be tested out in the coupled configuration.

pbo

And the differences between the /layer and /z examples are less than 0.1%:

pbo_layer_minus_z

@jkrasting jkrasting mentioned this issue Apr 6, 2015
Merged
@adcroft adcroft closed this as completed in 00d93b8 Apr 7, 2015
nikizadehgfdl pushed a commit to nikizadehgfdl/MOM6 that referenced this issue Oct 9, 2017
@adcroft
Restored MOM6 URL and removed unneeded/empty files
2f6fb91 
- The URL for MOM6 in .gitmodules was pointing to a user account.
- CPU_stats was empty and changes from run to run and thus not version
  controlled.
- ocean.stats was empty and is normally renamed to reflect platform when
  used for regression testing. Regressions are only supported on the GFDL
  platform pending mom-ocean#95.
gustavo-marques added a commit to gustavo-marques/MOM6 that referenced this issue Apr 24, 2019
@gustavo-marques
Merge pull request mom-ocean#95 from alperaltuntas/fix_debug_mode
42582a1 
Fixes for debug mode
marshallward added a commit to marshallward/MOM6 that referenced this issue Mar 31, 2022
@raphaeldussin @marshallward
add residual loss term for internal tides (mom-ocean#95)
08582ee 
Co-authored-by: Raphael Dussin <[email protected]>
Co-authored-by: Marshall Ward <[email protected]>
SMoorthi-emc pushed a commit to SMoorthi-emc/MOM6 that referenced this issue Jul 22, 2022
@jiandewang
Merge pull request mom-ocean#95 from jiandewang/feature/update-to-mai…
ec1d17e 
…n-20220603

update to main branch 20220629 commit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jkrasting jkrasting

Labels
None yet
Projects
None yet
Milestone
CMOR-ization of diagnostics
Development

No branches or pull requests
2 participants
@adcroft @jkrasting