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

snow, ice evaporation #97

Closed
eclare108213 opened this issue Feb 24, 2018 · 3 comments
Closed

snow, ice evaporation #97

eclare108213 opened this issue Feb 24, 2018 · 3 comments

Comments

@eclare108213
Copy link
Contributor

Check for potential bug. This might be easier (or already fixed) as part of the CMIP6/SIMIP history output (#93). Alex West's original email:

Hello Elizabeth,

Hope that you’re well. I’ve been having a go at adding two new diagnostics to CICE to report ice and snow evaporation separately – apart from being in the SIMIP data request, I think they may be useful for my PhD on the Arctic energy budget. However, I’ve been having some unexpected problems, and wondered if there are aspects of the evaporation I’m still not aware of (and if I’ve come across a possible reason why this hasn’t been done before!).

Basically, I’ve

  •      defined the new diagnostics, evap_ice_ai and evap_snow_ai, in the history routines in the normal way;
    
  •      defined new aggregate and category fields evap_ice, evapn_ice etc in ice_flux;
    
  •      added code to thickness_changes in ice_therm_vertical to calculate the fields evapn_ice, evapn_snow in the same way as evapn, and divide them by timestep length;
    
  •      added the new evaporation fields to the routine merge_fluxes in in_flux.
    

It looks to my naive eye that the new fields are then calculated in much the same way as the existing evapn field. However, when I look at the resulting diagnostic files, the new snow and ice evaporation fields are of a completely different order of magnitude to the total evaporation field (about 1.e3 bigger). Moreover, the sum of the two new fields actually has a different structure to the total evaporation field.

Do you have any idea what I might be missing here? I attach the copies of ice_flux, ice_step_mod, ice_therm_vertical, ice_history and ice_history_shared from my branch; unfortunately the output file with the three evaporation fields is too large to send by email.

Because this branch is a general ‘new diagnostics for SIMIP’ branch it also contains Dave Bailey’s new diagnostic changes, so a diff with the 5.1.2 trunk will show these differences also.

@dabail10
Copy link
Contributor

I have added two new variables in CESM with the SIMIP request to do this. So, I believe this will be resolved when I bring the CMIP variables in.

@dabail10 dabail10 mentioned this issue Sep 11, 2018
@dabail10
Copy link
Contributor

This is addressed in PR #182.

@dabail10 dabail10 reopened this Sep 11, 2018
This was referenced Sep 24, 2018
@dabail10
Copy link
Contributor

Resolved in PR#191.

JFLemieux73 pushed a commit to JFLemieux73/CICE that referenced this issue Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants