Open-EO / openeo-geopyspark-driver

OpenEO driver for GeoPySpark (Geotrellis)
Apache License 2.0
26 stars 4 forks source link

Check memory size after taking into account python_max. #851

Closed EmileSonneveld closed 2 months ago

EmileSonneveld commented 2 months ago

Moved python_max calculations higher up in the function where test code can still reach. Throw error when asking for too much memory.

EDIT: No changes to memory calculation

EmileSonneveld commented 2 months ago

@soxofaan , looks fine? That memOverheadBytes was not taken into account was curious for me. We can also wait till Jeroen Dries is back

EmileSonneveld commented 2 months ago

This passed CI, but the github UI is not updated yet