Recent changes to bugs

get rid of "xxx" ESPECIALLY in function names!

Craig Reynolds — Wed, 05 Nov 2008 01:06:14 -0000

Following a habit I picked up at a former job, I tend to mark code that is unfinished (or experimental or needs to be revisited) with "xxx". That some of these tags have been sitting in OpenSteer source for years is embarrassing. Worse is the fact that some of the basic API of OpenSteer has functions with "xxx" in their name (sinxxx, cosxxx, xxxsteerForSeek, etc) is unacceptable.

Thanks to kidwei for reminding me of this.

for doc: OpenSteer ports, wrappers, etc.

Craig Reynolds — Tue, 09 Sep 2008 21:33:54 -0000

OpenSteer ports, wrappers, etc.
http://sourceforge.net/forum/message.php?msg_id=5228592

Missing includement in PlugIn.h

Sun, 08 Jun 2008 09:59:12 -0000

The includement of <cstring> is missing in file include/PlugIn.h

The patch-file, if needed, is attached as a file.

minor optimization

Aidin Abedi — Wed, 21 May 2008 01:26:07 -0000

The following and similar-but-not-mentioned behaviors can be algorithmically optimized to use mult instead of div. In large simulations this is good.

Edit SteerLibrary.h at line 877 to:
if (neighbors > 0) steering = (steering - (forward() * (float)neighbors)).normalize();

Edit SteerLibrary.h at line 914 to:
if (neighbors > 0) steering = (steering - (position() * (float)neighbors)).normalize();

line no. refer to rev 188

/fooguru

note for doc about SVN and Mac OSX Xcode

Craig Reynolds — Fri, 16 May 2008 22:02:04 -0000

For users trying to work on OpenSteer on Mac OSX 10.4 (Tiger) with Xcode. This is old news since the current version of OSX is 10.5 (Leopard) but some of us are still living in the past:

SVN revision r187:

Changed OpenSteer project for Xcode (Mac OSX) to "Enable SCM" and select Subversion as "SCM System". (If you are running a version of OS X before 10.5 (Leopard) you may need to install Subversion from http://homepage.mac.com/martinott/ then copy the resulting files from /usr/local/bin to /usr/bin)

Compile error on gcc 4.1.2

Mathieu Roger — Sun, 02 Mar 2008 16:10:38 -0000

when invoking make for compiling opensteer 0.8.2 on debian/linux I got the following errors :

Compiling ../src/SimpleVehicle.cpp...
../include/OpenSteer/LocalSpace.h:77: warning: ‘class OpenSteer::AbstractLocalSpace’ has virtual functions but non-virtual destructor
../include/OpenSteer/LocalSpace.h: In instantiation of ‘OpenSteer::LocalSpaceMixin<OpenSteer::AbstractLocalSpace>’:
../include/OpenSteer/LocalSpace.h:357: instantiated from here
../include/OpenSteer/LocalSpace.h:135: warning: ‘class OpenSteer::LocalSpaceMixin<OpenSteer::AbstractLocalSpace>’ has virtual functions but non-virtual destructor
../include/OpenSteer/AbstractVehicle.h:56: warning: ‘class OpenSteer::AbstractVehicle’ has virtual functions but non-virtual destructor
../include/OpenSteer/Pathway.h:55: warning: ‘class OpenSteer::Pathway’ has virtual functions but non-virtual destructor
../include/OpenSteer/Pathway.h:97: warning: ‘class OpenSteer::PolylinePathway’ has virtual functions but non-virtual destructor
../include/OpenSteer/Obstacle.h:70: warning: ‘class OpenSteer::Obstacle’ has virtual functions but non-virtual destructor
../include/OpenSteer/Obstacle.h:93: warning: ‘class OpenSteer::SphericalObstacle’ has virtual functions but non-virtual destructor
../include/OpenSteer/SteerLibrary.h: In member function ‘bool OpenSteer::SteerLibraryMixin<Super>::isAhead(const OpenSteer::Vec3&, float) const’:
../include/OpenSteer/SteerLibrary.h:247: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:247: error: (if you use ‘-fpermissive’, G++ will accept your code, but allowing the use of an undeclared name is deprecated)
../include/OpenSteer/SteerLibrary.h:248: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘bool OpenSteer::SteerLibraryMixin<Super>::isAside(const OpenSteer::Vec3&, float) const’:
../include/OpenSteer/SteerLibrary.h:252: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:253: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘bool OpenSteer::SteerLibraryMixin<Super>::isBehind(const OpenSteer::Vec3&, float) const’:
../include/OpenSteer/SteerLibrary.h:258: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:259: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForWander(float)’:
../include/OpenSteer/SteerLibrary.h:330: error: there are no arguments to ‘side’ that depend on a template parameter, so a declaration of ‘side’ must be available
../include/OpenSteer/SteerLibrary.h:330: error: there are no arguments to ‘up’ that depend on a template parameter, so a declaration of ‘up’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForSeek(const OpenSteer::Vec3&)’:
../include/OpenSteer/SteerLibrary.h:343: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:344: error: there are no arguments to ‘velocity’ that depend on a template parameter, so a declaration of ‘velocity’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForFlee(const OpenSteer::Vec3&)’:
../include/OpenSteer/SteerLibrary.h:357: error: ‘position’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:358: error: there are no arguments to ‘velocity’ that depend on a template parameter, so a declaration of ‘velocity’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::xxxsteerForFlee(const OpenSteer::Vec3&)’:
../include/OpenSteer/SteerLibrary.h:372: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:373: error: there are no arguments to ‘maxSpeed’ that depend on a template parameter, so a declaration of ‘maxSpeed’ must be available
../include/OpenSteer/SteerLibrary.h:374: error: there are no arguments to ‘velocity’ that depend on a template parameter, so a declaration of ‘velocity’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::xxxsteerForSeek(const OpenSteer::Vec3&)’:
../include/OpenSteer/SteerLibrary.h:384: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:385: error: there are no arguments to ‘maxSpeed’ that depend on a template parameter, so a declaration of ‘maxSpeed’ must be available
../include/OpenSteer/SteerLibrary.h:386: error: there are no arguments to ‘velocity’ that depend on a template parameter, so a declaration of ‘velocity’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerToStayOnPath(float, OpenSteer::Pathway&)’:
../include/OpenSteer/SteerLibrary.h:400: error: there are no arguments to ‘predictFuturePosition’ that depend on a template parameter, so a declaration of ‘predictFuturePosition’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerToFollowPath(int, float, OpenSteer::Pathway&)’:
../include/OpenSteer/SteerLibrary.h:434: error: there are no arguments to ‘speed’ that depend on a template parameter, so a declaration of ‘speed’ must be available
../include/OpenSteer/SteerLibrary.h:437: error: there are no arguments to ‘predictFuturePosition’ that depend on a template parameter, so a declaration of ‘predictFuturePosition’ must be available
../include/OpenSteer/SteerLibrary.h:441: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerToAvoidObstacle(float, const OpenSteer::Obstacle&)’:
../include/OpenSteer/SteerLibrary.h:515: error: there are no arguments to ‘speed’ that depend on a template parameter, so a declaration of ‘speed’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerToAvoidObstacles(float, const OpenSteer::ObstacleGroup&)’:
../include/OpenSteer/SteerLibrary.h:534: error: there are no arguments to ‘speed’ that depend on a template parameter, so a declaration of ‘speed’ must be available
../include/OpenSteer/SteerLibrary.h:564: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:565: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h:567: error: there are no arguments to ‘maxForce’ that depend on a template parameter, so a declaration of ‘maxForce’ must be available
../include/OpenSteer/SteerLibrary.h:568: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h:568: error: there are no arguments to ‘maxForce’ that depend on a template parameter, so a declaration of ‘maxForce’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerToAvoidNeighbors(float, const OpenSteer::AVGroup&)’:
../include/OpenSteer/SteerLibrary.h:614: error: there are no arguments to ‘radius’ that depend on a template parameter, so a declaration of ‘radius’ must be available
../include/OpenSteer/SteerLibrary.h:643: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h:650: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:651: error: there are no arguments to ‘side’ that depend on a template parameter, so a declaration of ‘side’ must be available
../include/OpenSteer/SteerLibrary.h:659: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:660: error: there are no arguments to ‘side’ that depend on a template parameter, so a declaration of ‘side’ must be available
../include/OpenSteer/SteerLibrary.h:667: error: there are no arguments to ‘speed’ that depend on a template parameter, so a declaration of ‘speed’ must be available
../include/OpenSteer/SteerLibrary.h:669: error: there are no arguments to ‘side’ that depend on a template parameter, so a declaration of ‘side’ must be available
../include/OpenSteer/SteerLibrary.h:681: error: there are no arguments to ‘side’ that depend on a template parameter, so a declaration of ‘side’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘float OpenSteer::SteerLibraryMixin<Super>::predictNearestApproachTime(OpenSteer::AbstractVehicle&)’:
../include/OpenSteer/SteerLibrary.h:698: error: there are no arguments to ‘velocity’ that depend on a template parameter, so a declaration of ‘velocity’ must be available
../include/OpenSteer/SteerLibrary.h:717: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘float OpenSteer::SteerLibraryMixin<Super>::computeNearestApproachPositions(OpenSteer::AbstractVehicle&, float)’:
../include/OpenSteer/SteerLibrary.h:735: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h:735: error: there are no arguments to ‘speed’ that depend on a template parameter, so a declaration of ‘speed’ must be available
../include/OpenSteer/SteerLibrary.h:738: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerToAvoidCloseNeighbors(float, const OpenSteer::AVGroup&)’:
../include/OpenSteer/SteerLibrary.h:770: error: there are no arguments to ‘radius’ that depend on a template parameter, so a declaration of ‘radius’ must be available
../include/OpenSteer/SteerLibrary.h:772: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:778: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘bool OpenSteer::SteerLibraryMixin<Super>::inBoidNeighborhood(const OpenSteer::AbstractVehicle&, float, float, float)’:
../include/OpenSteer/SteerLibrary.h:806: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:825: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForSeparation(float, float, const OpenSteer::AVGroup&)’:
../include/OpenSteer/SteerLibrary.h:851: error: there are no arguments to ‘radius’ that depend on a template parameter, so a declaration of ‘radius’ must be available
../include/OpenSteer/SteerLibrary.h:856: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForAlignment(float, float, const OpenSteer::AVGroup&)’:
../include/OpenSteer/SteerLibrary.h:890: error: there are no arguments to ‘radius’ that depend on a template parameter, so a declaration of ‘radius’ must be available
../include/OpenSteer/SteerLibrary.h:902: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForCohesion(float, float, const OpenSteer::AVGroup&)’:
../include/OpenSteer/SteerLibrary.h:927: error: there are no arguments to ‘radius’ that depend on a template parameter, so a declaration of ‘radius’ must be available
../include/OpenSteer/SteerLibrary.h:939: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForPursuit(const OpenSteer::AbstractVehicle&, float)’:
../include/OpenSteer/SteerLibrary.h:965: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:971: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h:975: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h:977: error: there are no arguments to ‘speed’ that depend on a template parameter, so a declaration of ‘speed’ must be available
../include/OpenSteer/SteerLibrary.h:994: error: ‘gBlack’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:998: error: ‘gGray50’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1002: error: ‘gWhite’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1011: error: ‘gRed’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1015: error: ‘gYellow’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1019: error: ‘gGreen’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1028: error: ‘gCyan’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1032: error: ‘gBlue’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1036: error: ‘gMagenta’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h:1052: error: there are no arguments to ‘position’ that depend on a template parameter, so a declaration of ‘position’ must be available
../include/OpenSteer/SteerLibrary.h:1054: error: ‘gGray40’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForEvasion(const OpenSteer::AbstractVehicle&, float)’:
../include/OpenSteer/SteerLibrary.h:1070: error: ‘position’ was not declared in this scope
../include/OpenSteer/SteerLibrary.h: In member function ‘OpenSteer::Vec3 OpenSteer::SteerLibraryMixin<Super>::steerForTargetSpeed(float)’:
../include/OpenSteer/SteerLibrary.h:1094: error: there are no arguments to ‘maxForce’ that depend on a template parameter, so a declaration of ‘maxForce’ must be available
../include/OpenSteer/SteerLibrary.h:1095: error: there are no arguments to ‘speed’ that depend on a template parameter, so a declaration of ‘speed’ must be available
../include/OpenSteer/SteerLibrary.h:1096: error: there are no arguments to ‘forward’ that depend on a template parameter, so a declaration of ‘forward’ must be available
../include/OpenSteer/SteerLibrary.h: In member function ‘void OpenSteer::SteerLibraryMixin<Super>::findNextIntersectionWithSphere(OpenSteer::SphericalObstacle&, OpenSteer::SteerLibraryMixin<Super>::PathIntersection&)’:
../include/OpenSteer/SteerLibrary.h:1126: error: there are no arguments to ‘localizePosition’ that depend on a template parameter, so a declaration of ‘localizePosition’ must be available
../include/OpenSteer/SteerLibrary.h:1131: error: there are no arguments to ‘radius’ that depend on a template parameter, so a declaration of ‘radius’ must be available
../include/OpenSteer/PlugIn.h: At global scope:
../include/OpenSteer/PlugIn.h:85: warning: ‘class OpenSteer::AbstractPlugIn’ has virtual functions but non-virtual destructor
../include/OpenSteer/Camera.h:57: warning: ‘class OpenSteer::Camera’ has virtual functions but non-virtual destructor
../include/OpenSteer/LocalSpace.h: In instantiation of ‘OpenSteer::LocalSpaceMixin<OpenSteer::AbstractVehicle>’:
../include/OpenSteer/Annotation.h:60: instantiated from ‘OpenSteer::AnnotationMixin<OpenSteer::LocalSpaceMixin<OpenSteer::AbstractVehicle> >’
../include/OpenSteer/SteerLibrary.h:64: instantiated from ‘OpenSteer::SteerLibraryMixin<OpenSteer::AnnotationMixin<OpenSteer::LocalSpaceMixin<OpenSteer::AbstractVehicle> > >’
../include/OpenSteer/SimpleVehicle.h:91: instantiated from here
../include/OpenSteer/LocalSpace.h:135: warning: ‘class OpenSteer::LocalSpaceMixin<OpenSteer::AbstractVehicle>’ has virtual functions but non-virtual destructor
make: *** [SimpleVehicle.o] Erreur 1

It seems like SteerLibraryMixin is calling method position(), speed() ... that should probably be inherited from Super.

LQ (ProximityDatabase) bug for "outside" points

Craig Reynolds — Mon, 11 Jun 2007 17:27:08 -0000

Nicolas Brodu reported a bug in LQ (ProximityDatabase) and provided the fix, see: https://sourceforge.net/forum/message.php?msg_id=4343867

Mention PandaSteer in doc

Craig Reynolds — Wed, 14 Mar 2007 22:10:34 -0000

Mention PandaSteer (http://sourceforge.net/forum/message.php?msg_id=4191760 http://panda3d.org/phpbb2/viewtopic.php?p=11725\) in the Related Links section of the OpenSteer doc (http://opensteer.sourceforge.net/doc.html#RelatedLinks).

unimplemented basic obstacle shapes, unify shape code

Craig Reynolds — Mon, 12 Feb 2007 23:53:20 -0000

Several obvious basic obstacle shapes are missing and should be implemented, see details below. More generally, procedural shapes are used independently in several parts of PSCrowd. An effort should be made to unify this "shape describing" code and then use this shared shape library for describing the shape of obstacles, paths, surfaces and vehicle body shapes for non-penetration constraints.

It would be nice if we supported importing shape geometry in some interchange format (possibly Collada?: http://www.collada.org/ ) and then being able to use it as a standard shape for purposes of obstacle avoidance, or defining a surface on which vehicles can move.

(see also related discussion at: https://sourceforge.net/forum/message.php?msg_id=4154544 )

----

The current menagerie of obstacle shapes includes:

SphereObstacle
BoxObstacle
PlaneObstacle
RectangleObstacle

there is a To Do list here: http://sourceforge.net/forum/message.php?msg_id=2839263
which I have slightly updated below:

3D/Volumetric:

SphereObstacle [done]
BoxObstacle [done]
CylinderObstacle
CapsuleObstacle
ConeSphereObstacle
PrismObstacle
MeshObstacle
FrustumObstacle

Planar (but embedded in 3d space)

RectangleObstacle [done]
PlaneObstacle [done]
TriangleObstacle
DiskObstacle
PolygonObstacle

add comments for subtle feature in BoxObstacle

Craig Reynolds — Mon, 22 Jan 2007 18:59:39 -0000

see the reference to BoxObstacle::findIntersectionWithVehiclePath in https://sourceforge.net/forum/message.php?msg_id=4116493

The steerHint for BoxObstacle is significantly different than for a collection of six RectangleObstacles but I had to look at the code twice to find the relevant bit. It should be highlighted with a comment