[Ns-developers] removing Ns2CalendarScheduler

[Mathieu Lacage]

On 04/06/2012 09:59 PM, Tom Henderson wrote:
> I plan to remove src/core/model/ns2-calendar-scheduler.{cc,h} before 
> ns-3.14 release if there are no objections.  It is causing problems 
> with the threaded-simulator test (bug 1397) and it is not clear how 
> much effort to debug and cleanup this code would be required.  The 
> reason that we added it originally was that it has some improved 
> heuristics for dynamically adjusting the bins, based on work that 
> David Wei did for ns-2.  If someone would like to retain this 
> scheduler, all of the presently disabled asserts should be enabled, 
> and it should be documented and tested.
>
> I did some comparative runs of the manet-routing-compare program with 
> the different schedulers, and I didn't detect any significant 
> different between the Calendar scheduler and the Ns2Calendar scheduler 
> variants (or for any of these variants with that particular example 
> program).

My feeling is that we should aggressively remove the 
calendar-scheduler.cc/h and heap-scheduler.cc/h too since:
1) I am not aware of anyone ever trying to use them so I doubt anyone 
will feel a loss
2) we have already a great default scheduler
3) less code, less bugs, less time wasted removing the bugs.

Mathieu