Fixed #28

[ian-young]

Feature Name

Description

Fixed the max recursion depth. Found that the program was struggling making a split decision between prioritizing lower channel count or lower storage count which was causing the recursion error; to help nudge the program out of a indecision loop like this, or (surplus_channels == 0 and surplus_storage < min_surplus_channels) was added to give it more weight on filling out our surplus counts first.

[!NOTE] May need to add or (surplus_channels == 0 and surplus_storage < min_surplus_channels) at some point in the future to give it weight on deciding how to dish out storage once channels are complete in the future. This has not yet been observed behavior

Fixes #28

Type of change

• [x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

• [x] customer_sheet_1.csv
• [x] customer_sheet_2.csv
• [x] customer_sheet_3.csv
• [x] customer_sheet_4.csv
• [x] customer_sheet_5.csv
• [x] customer_sheet_6.csv
• [x] customer_sheet_7.csv
• [x] customer_sheet_8.csv
• [x] customer_sheet_9.csv
• [x] customer_sheet_10.csv
• [x] customer_sheet_11.csv

Checklist

• [x] My code follows the PEP8style guidelines.
• [x] I have performed a self-review of my code
• [x] I have commented my code, particularly in hard-to-understand areas
• [x] I have made corresponding changes to the documentation
• [x] My changes generate no new warnings
• [x] I have added tests that prove my fix is effective or that my feature works
• [x] New and existing unit tests pass locally with my changes
• [x] Any dependent changes have been merged and published in downstream modules