Agregate soil results with the rest of the scene

[mwoussen]

An issue is encounter with sorted function in _agregate function, l.35 of CaribuScene.py while activating soil_mesh.

The activation of soil_mesh leads to extend the groups list (l.540 CaribuScene.run(...) in CaribuScene.py) with string labels "soil" . This list is given afterward in l.655 raw[band][k] = _agregate(output[k], groups, list) . This leads to a TypeError given by the sorted function, because the list groups blends int and str elements.

I didn't want to replace the soil label in all the package, so only in the function _agregate we replace "soil" by -9999 (I supposed all the other labels were >= 0), and then put it back to "soil".

This bug was encountered with the package alinea.caribu on fredboudon

[pradal]

Thank you for your contribution.
Is it possible to provide a test?

[mwoussen]

The bug appears with Caribu 8.0.7 from https://anaconda.org/fredboudon/alinea.caribu and not with the Github version.
It seems like the commit hotfix soil_mesh solves this issue.

[pradal]

Could you test the caribu version on OpenAlea?

conda install -c conda-forge -c openalea3 alinea.caribu
This is the channel where we push the latest versions.

[mwoussen]

The test ends correctly !
(Tested on Windows 10 and Linux (WSL 2), python 3.9.15, caribu 8.0.10)