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

Bugfix for zlvs #28

Closed
DeniseWorthen opened this issue Jun 9, 2021 · 0 comments
Closed

Bugfix for zlvs #28

DeniseWorthen opened this issue Jun 9, 2021 · 0 comments

Comments

@DeniseWorthen
Copy link
Collaborator

An optional array was recently added to allow Icepack to calculate fluxes relative to different atmosphere levels for different variables (eg "2m" temperatures and "10m" winds). This change was added to Icepack but the corresponding change to allocate and initialize this array were not made to CICE repository at the same time.

The branch for the most recent update to emc/develop was made in the intervening time when Icepack included the zlvs feature but CICE itself did not. After I realized this, I tested a branch with the zlvs change added to CICE and found that all answers changed. The zlvs allocation and initialization should be added to CICE since they are optionally used in Icepack.

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

1 participant