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

Cross-slope heat transport using different methods to extract an isobath #12

Closed
wghuneke opened this issue Jun 22, 2023 · 6 comments
Closed

Comments

@wghuneke
Copy link
Collaborator

wghuneke commented Jun 22, 2023

Test how different the results are when calculating the heat transport using different isobath definitions (known as "Adele's" and "Wilma's"...).

Currently, we calculate the heat transport using "Adele's" definition. I propose to compare how different the integrated heat transport is using "Wilma's" definition. If they're not too different, we can use the same isobath and calculate point-by-point correlations.

@wghuneke
Copy link
Collaborator Author

wghuneke commented Aug 17, 2023

The main difference between the two contour definitions is that "Adele's" definition is step-wise, while "Wilma's" definition allows for diagonal connections between grid cells (also used in the cross-contour transport COSIMA-recipe).

So far, we have the heat transport calculated on the step-wise contour. I now calculated the heat transport (1 year average) on the diagonal contour and get quite different results: (compare figures below with figures in e.g. issue #11 which has values more than one order of magnitude smaller)

unbinned:
HT_1km_t_grid_orig_contour
binned:
HT_1km_t_grid_lon_bins

→ The total heat transport towards Antarctica in the model is highly dependent on the contour used for calculating it (assuming no errors in the code).

@willaguiar is currently calculating the ASC velocity on the step-wise contour. Maybe that is the way to to forward so we can make cell-by-cell* correlations of heat transport and ASC time series.
*Acknowledging that the result is likely noisy and will require some smoothing.

@wghuneke
Copy link
Collaborator Author

I just realised that the values in the seasonal plot in issue #4 also has heat transport values (calculated on the step-wise contour) which are an order of magnitude larger = similar to what I show above. @willaguiar will look into this.

@willaguiar
Copy link
Owner

I just realised that the values in the seasonal plot in issue #4 also has heat transport values (calculated on the step-wise contour) which are an order of magnitude larger = similar to what I show above. @willaguiar will look into this.

ok, I went to the code and I realized that I forgot to factor out the binned transport. Basically, because we use a bin width (3 degrees long) larger than a bin spacing (0.25 degrees long) when we cumsum the binned heat transport along the contour we sum things twice due to overlapping bins. We correct that by scaling the final total binned heat transport back to the total value of the unbinned one (a factor of ~0.08 generally).
I checked the code of the issue 4 and noticed that I forgot to apply that factor on the binned transport. After applying the factor it seems that the values for the total heat transport go back to the O[100] TW. (Fig below, correcting it in the issue). Thanks for noticing Wilma!

Screen Shot 2023-08-17 at 5 08 42 PM

@wghuneke
Copy link
Collaborator Author

Ok, (i) I found an error in my mistake, (ii) added the factor for the binned transport, and (iii) updated the code according to this issue. The new plot looks much more reasonable:

HT_1km_t_grid_lon_bins

@wghuneke
Copy link
Collaborator Author

Update: We decided to not use this method to extract the isobath but instead go with the step-wise contour definition. The main argument is that the zonal convergence calculation might not be accurate. Also, we have already started to calculate heat transports using the step-wise contour.

@wghuneke
Copy link
Collaborator Author

Closing this issue now.

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

No branches or pull requests

2 participants