mosaicworld / vb

TEAM BVB app
0 stars 0 forks source link

Add logic to get more combos of team types #2

Open mosaicworld opened 11 years ago

mosaicworld commented 11 years ago

Currently logic is limited b/c cycles through: one 4's team, one 3's team, one 2's team. Might be better to cycle through one 3's team and one 4's team (then apply that logic recursively). Then assign remaining players to a 2's team. Then count the extra's.

mosaicworld commented 11 years ago

There are different flavors of results. There could be all 3's teams. There could be all 4's teams. There could be a variety of 4's, 3's, 2's teams. Generally want to optimize the results for the least # of extra players possible.

mosaicworld commented 11 years ago

The count of remaining players is not getting tracked correctly. As a result, the count of extra players is not identified correctly. This issue fixed but don't know if it was done in the most kosher way (use of global vars).

mosaicworld commented 11 years ago

Checked in an updated version in vb_main.js. But there is currently an issue where doubles players = 0; group players >= 22. Should only show court results for one 4's court, two 3's courts, and the rest should be extra players. But it is showing court results for two 4's courts, two 3's courts (where the last 3's court doesn't have full teams assigned).

mosaicworld commented 11 years ago

Also an issue (sim to above) where 8 <= group players <= 13. Automatically designating an addition 3's court even though there are not enough players to fill the teams.

mosaicworld commented 11 years ago

Checked in one fix (no longer getting 3's teams with blank or missing players). Currently issues with scenarios: group players 8 or 9 (assigns 3's courts where it would make more sense to assign 4's court first) group players 16 or 17 (same issue as above) group players 22 or 23 (same issue as above)

mosaicworld commented 10 years ago

Deprioritized. Would like to make this logic more flexible but it's not critical.