[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