AliceVision icon indicating copy to clipboard operation
AliceVision copied to clipboard

Fixes #1202: Added an available memory check to constrain the number of CPU c…

Open sanchayanghosh opened this issue 1 year ago • 0 comments

…ores used, and moved fetch.sh call inside a dockerfile, to allowe building inside docker without having to install cmake externally.

Description

A fix for #1202 - Here, I put a pre-condition check on the available memory on the system.

This will usually be the memory available to the individual docker. However, since we are using an overlay system, with no specific memory caps/ caps on CPU shares, we should be seeing the state of the entire system.

Features list

Fix #1202 - AliceVision would crash the linux environment because it exceeds the available memory when building.

Implementation remarks

During the actual build, I found that putting 2 GB per job was ideal. 1GB per job would crash the system.

In the worst case, we have an nproc/2 situation, which anyhow is the default behaviour of cmake.

sanchayanghosh avatar Aug 08 '22 15:08 sanchayanghosh