[Ns-developers] api breakage: bug 482
raj bhattacharjea
rbhattacharjea at gmail.com
Wed Feb 25 11:13:50 PST 2009
On Wed, Feb 25, 2009 at 6:47 AM, Mathieu Lacage
<mathieu.lacage at sophia.inria.fr> wrote:
> hi,
>
> This is an email asking for input on how to fix bug 482. Two options:
>
> 1) rename existing classes and add a new one:
> - StaticMobilityModel -> ConstantPositionMobilityModel
> - StaticSpeedMobilityModel -> ConstantSpeedMobilityModel
> - ConstantAccelerationMobilityModel
>
> 2) add a new class:
> - StaticAccelerationMobilityModel
>
> I don't care at all which option should be chosen but gustavo seems to
> like 1) better which will break API while 2) won't break any API. Unless
> there are dissenting voices, I would tend to follow him. Opinions ? Dear
> RM ?
I like Gustavo's option, with Tom's suggestion of s/Speed/Velocity . That is:
ConstantPositionMobilityModel
ConstantVelocityMobilityModel
ConstantAccelerationMobilityModel
This uses "Constant" in all cases instead of mixing Static and
Constant, so it is more self consistent. Furthermore this is
consistent with how every major physics text I've seen covers
kinematics; a point has a position; the time derivative of position is
velocity; the time derivative of velocity is acceleration.
I don't care much, and I hate lengthy discussions about naming; I only
reply because I was asked specifically :-)
--
Raj Bhattacharjea
Georgia Institute of Technology
School of Electrical and Computer Engineering
Ph.D. Candidate
Systems Analyst
404.894.2955
More information about the Ns-developers
mailing list