Commit 6566387a authored by Jeffrey Morgan's avatar Jeffrey Morgan
Browse files

add `TODO` for cuda overhead

parent 37708931
...@@ -131,8 +131,9 @@ func getCPUMem() (memInfo, error) { ...@@ -131,8 +131,9 @@ func getCPUMem() (memInfo, error) {
func CheckVRAM() (int64, error) { func CheckVRAM() (int64, error) {
gpuInfo := GetGPUInfo() gpuInfo := GetGPUInfo()
if gpuInfo.FreeMemory > 0 && (gpuInfo.Library == "cuda" || gpuInfo.Library == "rocm") { if gpuInfo.FreeMemory > 0 && (gpuInfo.Library == "cuda" || gpuInfo.Library == "rocm") {
// leave 20% of VRAM free for overhead // leave 25% of VRAM free for overhead
return int64(gpuInfo.FreeMemory * 4 / 5), nil // TODO: improve cuda memory allocation to avoid needing this large of an amount
return int64(gpuInfo.FreeMemory * 3 / 4), nil
} }
return 0, fmt.Errorf("no GPU detected") // TODO - better handling of CPU based memory determiniation return 0, fmt.Errorf("no GPU detected") // TODO - better handling of CPU based memory determiniation
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment