PecanProject / pecan

The Predictive Ecosystem Analyzer (PEcAn) is an integrated ecological bioinformatics toolbox.
www.pecanproject.org
Other
202 stars 230 forks source link

WIP: refactor model2netcdf.ED2 #3070

Closed Aariq closed 1 year ago

Aariq commented 1 year ago

Description

There are a few bugs and some design fragility in model2netcdf.ED2() that I think warrant a pretty substantial overhaul. I'm going to work on that in this PR.

Issues I hope to address:

Motivation and Context

Ran into a weird edge case with model2netcdf.ED2() and remembered what a quagmire it felt like to me last time I worked on it. I feel like I have a better handle on working with lists and ncdf4 that I can simplify this function a lot while making it easier to expand in the future.

Review Time Estimate

Types of changes

Checklist: