in inferpymc_postprocessouts, the order of the flux dimensions is inferred by assuming that time is the smallest dimension, then the axes of the flux array are re-ordered so that time is the last axis. This is a hack to try to ensure that the flux dims are in the expected order, without using xarray
ultimately, we want to use xarray for postprocessing (issue #126) but that change will take longer
a test was added to run a full inversion with a flux whose dimensions are ordered: time, lat,lon`. This test failed initially, reproducing the error @hdelongueville found. After the fix, the test now passes.
Please check if the PR fulfills these requirements
Summary of changes
in
inferpymc_postprocessouts, the order of the flux dimensions is inferred by assuming thattimeis the smallest dimension, then the axes of the flux array are re-ordered so that time is the last axis. This is a hack to try to ensure that the flux dims are in the expected order, without using xarrayultimately, we want to use xarray for postprocessing (issue #126) but that change will take longer
a test was added to run a full inversion with a flux whose dimensions are ordered:
time,lat,lon`. This test failed initially, reproducing the error @hdelongueville found. After the fix, the test now passes.Please check if the PR fulfills these requirements
[x] Temporary work around for #126
[x] Tests added and passing
[x] Added an entry to the
CHANGELOG.mdfile