Increased the generate_schedule optimiser run count to 160, and tweaked some output messages.

 - Legacy-Id: 18133

Author: levkowetz

ietf/meeting/management/commands/generate_schedule.py

@@ -23,7 +23,13 @@
 from ietf.person.models import Person
 from ietf.meeting import models
 
-OPTIMISER_MAX_CYCLES = 80
+# 40 runs of the optimiser for IETF 106 with cycles=160 resulted in 16
+# zero-violation invocations, with a mean number of runs of 91 and 
+# std-dev 41.  On the machine used, 160 cycles had a runtime of about
+# 30 minutes.  Setting default to 160, as 80 or 100 seems too low to have
+# a reasonable rate of success in generating zero-cost schedules.
+
+OPTIMISER_MAX_CYCLES = 160
 
 
 class Command(BaseCommand):
@@ -62,6 +68,7 @@ def __init__(self, stdout, meeting_number, name=None, max_cycles=OPTIMISER_MAX_C
     def run(self):
         """Schedule all sessions"""
 
+
         beg_time = time.time()
         self.schedule.fill_initial_schedule()
         violations, cost = self.schedule.total_schedule_cost()
@@ -309,8 +316,8 @@ def optimise_schedule(self):
             random.shuffle(items)
 
             if self.verbosity >= 2:
-                self.stdout.write('== Optimiser starting run {}, dynamic cost after last run {} =='
-                                  .format(run_count,  intcomma(last_run_cost)))
+                self.stdout.write('== Optimiser starting run {}, dynamic cost after last run {:,} =='
+                                  .format(run_count,  last_run_cost))
                 self.stdout.write('Dynamic violations in last optimiser run: {}'
                                   .format(last_run_violations))
             if shuffle_next_run:
@@ -321,8 +328,11 @@ def optimise_schedule(self):
             for original_timeslot, session in items:
                 best_cost = self.calculate_dynamic_cost()[1]
                 if best_cost == 0:
+                    if self.verbosity >= 1 and self.stdout.isatty():
+                        sys.stderr.write('\n')
                     if self.verbosity >= 2:
                         self.stdout.write('Optimiser found an optimal schedule')
+
                     return run_count
                 best_timeslot = None
 
@@ -351,7 +361,7 @@ def optimise_schedule(self):
         if self.verbosity >= 1 and self.stdout.isatty():
             sys.stderr.write('\n')
         if self.verbosity >= 2:
-            self.stdout.write('Optimiser did not find perfect schedule, using best schedule at dynamic cost {}'
+            self.stdout.write('Optimiser did not find perfect schedule, using best schedule at dynamic cost {:,}'
                               .format(self.best_cost))
         self.schedule = self.best_schedule