-
Notifications
You must be signed in to change notification settings - Fork 145
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
Bug Feat request : MOM6 temperature interpolation #773
Comments
The differences between in situ and potential temperature should be pretty small in a lot of the ocean, especially in the upper ocean where most of our obs are, see e.g. the profile in figure 1 here. Can you plot the model-data misfit to make sure it's not a K vs C issue? That said I think we'll still want to do the calculation to potential temp (maybe MITgcm folks figured it didn't matter for their obs?). I think we do have depth in MOM6 (we need this for the FOs anyway, right?) so maybe it's a matter of checking that we're using the right var and then testing out the POP code as a first step. |
Following POP conversion method for QTY_POTENTIAL_TEMPERATURE to QTY_TEMPERATURE: depth to pressure |
fixes #773 untested, also need to check with fortran standard has elemental.
May move this to a separate issue(s)
|
fixes #773 Converts potential_temp (model) to in-situ temp (obs) Following method in POP model_mod: #773 (comment) Uses element function - need to check with fortran standard has elemental.
elemental functions are f95 so no problem standard wise (a612e81) |
Salinity Depth |
Use case
The MOM6 model_mod was made super naively (by yours truly).
Currently it only interpolates a QTY if the QTY is in the state vector.
This is not sufficient for observations.
Filter output
This is world obs, with MOM6 only for the North Atlantic, so load of fails. But all temperature obs fail.
Side note: I need to trim the obs by region for CROCODILE.
The temp obs are all fails (because not QTY_TEMPERATURE)
There are some salinity fails, not sure if this is bugs or run-of-the-mill fails?
Required interpolations
Required interpolation QTY - note not exhaustive, this is just looking at what is in POP. There may be way more. Also I don't know the difference between the various temperatures, so watch out.
Is your feature request related to a problem?
Failing temperature forward operators.
Describe your preferred solution
step one: learn how temperature works in models.
step two(?):
In MOM6 We don't have depth, we have layer thickness so I'm not sure if I can just rip out the POP calculations and use in both MOM6 and POP (even assuming the units conversion is not a problem).
MITgcm_ocean has PTMP as QTY_TEMPERATURE
'PTMP', 'QTY_TEMPERATURE',
DART/models/MITgcm_ocean/trans_mitdart_mod.f90
Lines 382 to 383 in 55eaf62
Describe any alternatives you have considered
Why doesn't obs_def_mod take care of the mapping between observation and qty for things like:
potential temp, salinity, local pressure -> sensible temp?
Is this a unit disaster (salinity, pressure, etc) waiting to happen when you swap ocean models?The text was updated successfully, but these errors were encountered: