Cumulative Elevation Calculation about communityfirnmodel HOT 4 OPEN

Hi @Jez-Carter, thanks for digging into this. Sorry, I have not had time in the last month to dig into your previous issue #9. I think you are not missing anything; that dh code was actually developed prior to the CFM getting melt physics (i.e., it is pretty old!). If you want, you can dive into the code and try to fix the issue and make a pull request, or hopefully I can find a moment later this week or next week to take a look.

from communityfirnmodel.

Hi @maximusjstevens, thanks for offering to take a look, I think that would be best and to get your further thoughts. I'll also get in contact with Vincent as I believe he's worked with cumulative elevation before. I won't be able to spend much time digging into the code myself over the next couple of weeks as I have another deadline approaching for a different piece of work but will keep you updated if I am able to.

from communityfirnmodel.

Hello everyone,
Jezz, I hard-coded my own variable for computing elevation changes. The CFM structure has changed since then, so I cannot simply push my (very) old code. However, I am happy to look into that in the coming days. I can probably work on this aspect in the coming days (or early next week). Jezz, I will email you to get all your input files.

from communityfirnmodel.

I think that I fixed the problem. See the email I just sent to Max and you.

from communityfirnmodel.