[Ns-bugs] [Bug 178] Randow Walk Sample Incorrect

bugzilla-daemon@nsnam-www.ece.gatech.edu bugzilla-daemon at nsnam-www.ece.gatech.edu
Wed May 14 15:23:26 PDT 2008


http://www.nsnam.org/bugzilla/show_bug.cgi?id=178





------- Comment #2 from tjkopena at cs.drexel.edu  2008-05-14 18:23 -------
So, yes, there were some issues:

- Somewhere along the line the syntax for rectangle strings changed from colons
to pipes.
- The bounds did have to change, nodes were being created outside of them and
an assert was failing.
- The callback signature was outdated, still using TraceContext instead of
std::string.

These lead to a couple of changes.  In [1] the first two simple issues have
been fixed.  More importantly though, it also:

- Removes some remaining references to TraceContext and TraceResolver
throughout the codebase.
- Removes MobilityModelNotifier.  The CourseChange trace source has now been
moved into MobilityModel.

The latter may break some people's simulation code, so it has not been pushed
directly.  In particular, mobility notifications are now done entirely via
traces rather than aggregating a notifier object that gets called.  Previously
you could in theory handle them either way.  For most people though, this will
hopefully simply involve changing the trace source used and eliminating calls
like MobilityHelper::EnableNotifier().

Comments welcome.

Thx

[1] http://code.nsnam.org/tjkopena/ns-3-dev-nonotifier/


-- 
Configure bugmail: http://www.nsnam.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


More information about the Ns-bugs mailing list