decile-team / submodlib

Summarize Massive Datasets using Submodular Optimization
MIT License
84 stars 33 forks source link

Oom fix #9

Closed nab170130 closed 2 years ago

nab170130 commented 2 years ago

Added destructors for LogDetMI C++ code. There is currently an issue where instantiating LogDetMI on the C++ side is dynamically allocating new memory but is not freeing it once the lifetime of those objects end. Repeatedly calling LogDetMI eventually causes out-of-memory issues. Here, I've added a fix that simply adds the destructors that free this memory.