Quick bug fix in getSolution

[joanibal]

Purpose

When you call getSolution with multiple families the values calculated by the earlier families are zeroed on the last iteration.
I don't believe this was ever the intended behavior and frankly I'm surprised this was left on unfixed for so long.
When using evalFunctions 'getSolution` is called individually for each group which is why this isn't currently messing up everyone's cases

Expected time until merged

1 week

Type of change

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (non-backwards-compatible fix or feature)
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Documentation update
• Maintenance update
• Other (please describe)

Testing

Ask @lambe for his test script if you'd like

Checklist

• I have run flake8 and black to make sure the code adheres to PEP-8 and is consistently formatted
• I have run unit and regression tests which pass locally with my changes
• I have added new tests that prove my fix is effective or that my feature works
• I have added necessary documentation

[codecov]

Codecov Report

Merging #226 (20d2ac1) into main (ea3e48d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #226   +/-   ##
=======================================
  Coverage   41.04%   41.04%           
=======================================
  Files          13       13           
  Lines        3793     3793           
=======================================
  Hits         1557     1557           
  Misses       2236     2236           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[marcomangano]

Good catch. Ideally we should have a test to cover this, but I assume this is not going to happen in the short term.