perf: reduce memory usage in build_model (sparse coefficients + per-effect share constraints)#595

Merged

FBumann merged 16 commits intofeature/element-data-classesfrom

feature/element-data-classes+memmory-issues

Feb 1, 2026

Commits on Feb 1, 2026

Switch to per effect shares
FBumann
committed
Firs succesfull drop to 10 GB
FBumann
committed
Make more readable
FBumann
committed
Go back to one variable for all shares
FBumann
committed
⏺ Instead of adding zero-constraints for uncovered combos, we should just set lower=0, upper=0 on those entries (fix the bounds), or better yet — use a mask on the per-effect
FBumann
committed
Only create variables needed
FBumann
committed
_create_share_var went from 1,674ms → 116ms — a 14x speedup! The reindex + + approach is much faster than per-contributor sel + merge
FBumann
committed
Revert
FBumann
committed
Revert
FBumann
committed
1. effects.py: add_temporal_contribution and add_periodic_contribution now raise ValueError if a DataArray has no effect dimension and no effect= argument is provided.
FBumann
committed