Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [sumo-user] Problem with implementing a new car following model

Jacob,

Thanks for the reply. I've changed the from.get call to from.getCFParam as you have suggested. 

        case SUMO_TAG_CF_GM:
            vtype->myCarFollowModel = new MSCFModel_GM(vtype,
                from.getCFParam(SUMO_ATTR_GM_Alpha,1.),
                from.getCFParam(SUMO_ATTR_GM_M, 1.),
                from.getCFParam(SUMO_ATTR_GM_L, 1.));
            break;
 
But I am still having rouble with the second error "no overloaded function takes 4 arguments". at MSVehicleType.cpp

Even though I've declared my parameters in both .h and .cpp file, I am still getting the same error. 

In cpp file, I have wrote the following

MSCFModel_GM::MSCFModel_GM(const MSVehicleType* vtype) :
    MSCFModel(vtype),
    myalpha(vtype->getParameter().getCFParam(SUMO_ATTR_GM_Alpha, 1.0)), // This is the sensitivity parameter, default value is set to 1
    myM(vtype->getParameter().getCFParam(SUMO_ATTR_GM_M, -2.0)), // This is the speed exponent, takes values between -2 to +2
    myL(vtype->getParameter().getCFParam(SUMO_ATTR_GM_L, 4.0)) { // This is the distance headway exponent, takes values between +4 to -1


}

MSCFModel_GM::~MSCFModel_GM() {}


and in .h file, I have declared the myalpha , myM and myL as the following

private:
    /// @name model parameter
    /// @{
    /// @brief General Motors sensitivity
    double myalpha;

    /// @brief General Motors speed exponent
    double myM;

    /// @brief General Motors Distance Headway Exponent
    double myL;
    /// @}


I would really appreciate the help.

Best Regards,
Sadullah.

On Sat, Oct 23, 2021 at 11:56 AM Jakob Erdmann <namdre.sumo@xxxxxxxxx> wrote:
Hello,

- the call from.get is obsolete and from.getCFParam is correct (documentation has been updated).
- the make the constructor call 'new MSCFModel_GM( ... )' with 4 parameters work, you have to define a corresponding constructor in MSCFModel_GM.h and .cpp

regards,
Jakob


Am Sa., 23. Okt. 2021 um 02:25 Uhr schrieb S G <sadullahgoncu@xxxxxxxxx>:
Greetings,

I am trying to implement a car-following model (General Motors model) to SUMO. I've successfully downloaded the developer version and compiled (built) the code on windows.

However, I am having some trouble with the implementation of a car-following model. I've followed and completed every step in the guide at the documentation ( https://sumo.dlr.de/docs/Developer/How_To/Car-Following_Model.html). My main issue occurs at the "MSVehicleType.cpp" when I try to compile again. According to the documentation, I need to add the following to the "MSVehicleType.cpp" ;

        case SUMO_TAG_CF_GM:
            model = new MSCFModel_GM(vtype,
                from.get(SUMO_ATTR_GM_Alpha,1.),
                from.get(SUMO_ATTR_GM_M, 1.),
                from.get(SUMO_ATTR_GM_L, 1.));
            break ;

However, when I try to compile the code again, the "from.get" command does not work and build fails around step 390-400 with the following errors;
  • 'get': is not a member of 'SUMOVTypeParameter'7
  • model : undeclared identifier
I have tried the following as well, which is the current format for the car-following models already existing inside the MSVehicleType.cpp

        case SUMO_TAG_CF_GM:
            vtype->myCarFollowModel = new MSCFModel_GM(vtype,
                from.getCFParam(SUMO_ATTR_GM_Alpha,1.),
                from.getCFParam(SUMO_ATTR_GM_M, 1.),
                from.getCFParam(SUMO_ATTR_GM_L, 1.));
            break ;

When I compile this I get the error that "no overloaded function takes 4 arguments.

Another version I've tried

        case SUMO_TAG_CF_GM:
            vtype->myCarFollowModel = new MSCFModel_GM(vtype)
             break ;

At this version, compile procedure successfully runs to the step 740 but fails with an error message as follows :

[740/1025] C:\PROGRA~2\MICROS~3\2019\COMMUN~1\VC\Tools\MSVC\1429~1.301\bin\Hostx64\x64\cl.exe  /nologo /TP -DFLOAT_MATH_FUNCTIONS -IC:\SUMOLibraries\googletest-1.10.0\include -Isrc -I..\..\..\src -IC:\SUMOLibraries\xerces-c-3.2.3\include -IC:\SUMOLibraries\proj-7.2.0\include -IC:\SUMOLibraries\fox-1.6.57\include -IC:\SUMOLibraries\freetype-2.10.4\include\freetype2 -IC:\SUMOLibraries\3rdPartyLibs\zlib-1.2.11\include -IC:\SUMOLibraries\gdal-3.2.1\include -IC:\SUMOLibraries\FFMPEG-4.2.0\include -IC:\SUMOLibraries\OSG-3.6.5\include -IC:\SUMOLibraries\gl2ps-1.4.0\include -IC:\SUMOLibraries\eigen-3.3.9 /DWIN32 /D_WINDOWS /W3 /GR /EHsc /MP /bigobj /MD /O2 /Ob2 /DNDEBUG /showIncludes /Fosrc\utils\gui\images\CMakeFiles\utils_gui_images.dir\GUIIconSubSys.cpp.obj /Fdsrc\utils\gui\images\CMakeFiles\utils_gui_images.dir\utils_gui_images.pdb /FS -c ..\..\..\src\utils\gui\images\GUIIconSubSys.cpp
  [741/1025] cmd.exe /C "cd /D C:\Users\l\source\repos\sumo\out\build\x64-Release\src\libsumo && "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe" -E make_directory C:/Users/source/repos/sumo/tools/libsumo C:/Users//source/repos/sumo/out/build/x64-Release/src/libsumo && "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe" -E env SWIG_LIB=C:/SUMOLibraries/swigwin-4.0.2/Lib C:/SUMOLibraries/swigwin-4.0.2/swig.exe -python -keyword -outdir C:/Users/source/repos/sumo/tools/libsumo -c++ -interface _libsumo -IC:/SUMOLibraries/googletest-1.10.0/include -IC:/SUMOLibraries/xerces-c-3.2.3/include -IC:/SUMOLibraries/proj-7.2.0/include -IC:/SUMOLibraries/fox-1.6.57/include -IC:/SUMOLibraries/freetype-2.10.4/include/freetype2 -IC:/SUMOLibraries/3rdPartyLibs/zlib-1.2.11/include -IC:/SUMOLibraries/gdal-3.2.1/include -IC:/SUMOLibraries/FFMPEG-4.2.0/include -IC:/SUMOLibraries/OSG-3.6.5/include -IC:/SUMOLibraries/gl2ps-1.4.0/include -IC:/SUMOLibraries/eigen-3.3.9 -IC:/Users/source/repos/sumo/out/build/x64-Release/src -IC:/Users/source/repos/sumo/src -IC:/Users/anaconda3/include -o C:/Users/source/repos/sumo/out/build/x64-Release/src/libsumo/libsumoPYTHON_wrap.cxx C:/Users/source/repos/sumo/src/libsumo/libsumo.i"
  ninja: build stopped: subcommand failed.


Another common error I get at is the following:

Severity Code Description Project File Line Suppression State Detail Description
Error (active) E0289 no instance of constructor "MSCFModel_GM::MSCFModel_GM" matches the argument list microsim.lib (src\microsim\microsim.lib) - x64-Release C:\Users\sadul\source\repos\sumo\src\microsim\MSVehicleType.cpp 356            argument types are: (MSVehicleType *, double, double, double)


Sorry for the long e-mail but I would really appreciate any help. You can find the code files in the attachment as well.

Best Regards,
Sadullah.
_______________________________________________
sumo-user mailing list
sumo-user@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/sumo-user
_______________________________________________
sumo-user mailing list
sumo-user@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/sumo-user
/****************************************************************************/
// Eclipse SUMO, Simulation of Urban MObility; see https://eclipse.org/sumo
// Copyright (C) 2001-2021 German Aerospace Center (DLR) and others.
// This program and the accompanying materials are made available under the
// terms of the Eclipse Public License 2.0 which is available at
// https://www.eclipse.org/legal/epl-2.0/
// This Source Code may also be made available under the following Secondary
// Licenses when the conditions for such availability set forth in the Eclipse
// Public License 2.0 are satisfied: GNU General Public License, version 2
// or later which is available at
// https://www.gnu.org/licenses/old-licenses/gpl-2.0-standalone.html
// SPDX-License-Identifier: EPL-2.0 OR GPL-2.0-or-later
/****************************************************************************/
/// @file    MSCFModel_GM.h
/// @author  Sadullah Göncü
/// 
/// 
/// 
/// @date    Friday, 22 October 2021
///
// The original General Motors (1958) car-following model and parameter
/****************************************************************************/
#pragma once
#include <config.h>

#include "MSCFModel.h"
#include <utils/xml/SUMOXMLDefinitions.h>


// ===========================================================================
// class definitions
// ===========================================================================
/** @class MSCFModel_GM
 * @brief The General Motors (1958) car-following model and parameter
 * @see MSCFModel
 */
class MSCFModel_GM : public MSCFModel {
public:
    /** @brief Constructor
     *  @param[in] vtype the type for which this model is built and also the parameter object to configure this model
     */
    MSCFModel_GM(const MSVehicleType* vtype);
 

    /// @brief Destructor
    ~MSCFModel_GM();


    /// @name Implementations of the MSCFModel interface
    /// @{
    /// @brief apply dawdling
    double finalizeSpeed(MSVehicle* const veh, double vPos) const;

    /** @brief Computes the vehicle's safe speed (no dawdling)
     * @param[in] veh The vehicle (EGO)
     * @param[in] speed The vehicle's speed
     * @param[in] gap2pred The (netto) distance to the LEADER
     * @param[in] predSpeed The speed of LEADER
     * @return EGO's safe speed
     * @see MSCFModel::ffeV
     */
    double followSpeed(const MSVehicle* const veh, double speed, double gap2pred, double predSpeed, double predMaxDecel, const MSVehicle* const pred = 0) const;


    /** @brief Computes the vehicle's safe speed for approaching a non-moving obstacle (no dawdling)
     * @param[in] veh The vehicle (EGO)
     * @param[in] gap2pred The (netto) distance to the the obstacle
     * @return EGO's safe speed for approaching a non-moving obstacle
     * @see MSCFModel::ffeS
     * @todo generic Interface, models can call for the values they need
     */
    virtual double stopSpeed(const MSVehicle* const veh, const double speed, double gap2pred, double decel) const;


    /** @brief Returns the model's name
     * @return The model's name
     * @see MSCFModel::getModelName
     */
    virtual int getModelID() const {
        return SUMO_TAG_CF_GM ;
    }



private:
    class VehicleVariables : public MSCFModel::VehicleVariables {
    public:
        double rand;
    };

    /** @brief Returns the "safe" velocity
     * @param[in] gap2pred The (netto) distance to the LEADER
     * @param[in] predSpeed The LEADER's speed
     * @return the safe velocity
     */
    double _v(const MSVehicle* const veh, double speed, double vfree, double gap, double predSpeed) const;

private:
    /// @name model parameter
    /// @{
    /// @brief General Motors sensitivity
    double myalpha;

    /// @brief General Motors speed exponent
    double myM;

    /// @brief General Motors Distance Headway Exponent
    double myL;
    /// @}


    /// @name Setter methods
    /// @{
    /** @brief Sets a new value for maximum deceleration [m/s^2]
     * @param[in] accel The new deceleration in m/s^2
     */
    void setMaxDecel(double decel) {
        myDecel = decel;
        myTauDecel = myDecel * myHeadwayTime;
    }



    /** @brief Sets a new value for desired headway [s]
     * @param[in] headwayTime The new desired headway (in s)
     */
    void setHeadwayTime(double headwayTime) {
        myHeadwayTime = headwayTime;
        myTauDecel = myDecel * headwayTime;
    }
    /// @}


    /** @brief Duplicates the car-following model
     * @param[in] vtype The vehicle type this model belongs to (1:1)
     * @return A duplicate of this car-following model
     */
    virtual MSCFModel* duplicate(const MSVehicleType* vtype) const;


    MSCFModel::VehicleVariables* createVehicleVariables() const;


protected:
    /// @brief The precomputed value for myDecel*myTau
    double myTauDecel;


};


/****************************************************************************/
// Eclipse SUMO, Simulation of Urban MObility; see https://eclipse.org/sumo
// Copyright (C) 2001-2021 German Aerospace Center (DLR) and others.
// This program and the accompanying materials are made available under the
// terms of the Eclipse Public License 2.0 which is available at
// https://www.eclipse.org/legal/epl-2.0/
// This Source Code may also be made available under the following Secondary
// Licenses when the conditions for such availability set forth in the Eclipse
// Public License 2.0 are satisfied: GNU General Public License, version 2
// or later which is available at
// https://www.gnu.org/licenses/old-licenses/gpl-2.0-standalone.html
// SPDX-License-Identifier: EPL-2.0 OR GPL-2.0-or-later
/****************************************************************************/
/// @file    MSCFModel_GM.cpp
/// @author  Sadullah Göncü
/// @author  
/// @author  
/// @author  
/// @author  
/// @date    Friday, 22 October 2021
///
// The original General Motors (1958) car-following model and parameter
/****************************************************************************/
#include <config.h>

#include <microsim/MSVehicle.h>
#include <microsim/MSLane.h>
#include "MSCFModel_GM.h"
#include <microsim/lcmodels/MSAbstractLaneChangeModel.h>
#include <utils/common/RandHelper.h>
#include <microsim/MSGlobals.h>

// ===========================================================================
// DEBUG constants
// ===========================================================================
//#define DEBUG_COND (veh->getID()=="disabled")

// ===========================================================================
// method definitions
// ===========================================================================
MSCFModel_GM::MSCFModel_GM(const MSVehicleType* vtype) :
    MSCFModel(vtype),
    myalpha(vtype->getParameter().getCFParam(SUMO_ATTR_GM_Alpha, 1.0)), // This is the sensitivity parameter, default value is set to 1
    myM(vtype->getParameter().getCFParam(SUMO_ATTR_GM_M, -2.0)), // This is the speed exponent, takes values between -2 to +2
    myL(vtype->getParameter().getCFParam(SUMO_ATTR_GM_L, 4.0)) { // This is the distance headway exponent, takes values between +4 to -1 


}

MSCFModel_GM::~MSCFModel_GM() {}


double
MSCFModel_GM::finalizeSpeed(MSVehicle* const veh, double vPos) const {
    const double vNext = MSCFModel::finalizeSpeed(veh, vPos);
    VehicleVariables* vars = (VehicleVariables*)veh->getCarFollowVariables();
    vars->rand = RandHelper::rand(veh->getRNG());
    return vNext;
}


double
MSCFModel_GM::followSpeed(const MSVehicle* const veh, double speed, double gap, double predSpeed, double /*predMaxDecel*/, const MSVehicle* const /*pred*/) const {
    return MIN2(_v(veh, speed, maxNextSpeed(speed, veh), gap, predSpeed), maxNextSpeed(speed, veh));
}


double
MSCFModel_GM::stopSpeed(const MSVehicle* const veh, const double speed, double gap, double /*decel*/) const {
    return MIN2(_v(veh, speed, maxNextSpeed(speed, veh), gap, 0), maxNextSpeed(speed, veh));
}


MSCFModel::VehicleVariables*
MSCFModel_GM::createVehicleVariables() const {
    VehicleVariables* ret = new VehicleVariables();
    /// XXX should use egoVehicle->getRNG()
    ret->rand = RandHelper::rand();
    return ret;
}

double
MSCFModel_GM::_v(const MSVehicle* const veh, double speed, double vfree, double gap, double predSpeed) const {
    if (predSpeed == 0 && gap < 0.01) {
        return 0;
    }
    // !!! in the following, the prior step is not considered!!!
    double GM_acc = MAX2((double)(myalpha * pow(predSpeed, myM) / pow(gap, myL)) * (speed - predSpeed), (double)0);
    double vsafe = (double)(speed + ACCEL2SPEED(GM_acc));

    double v = MAX2((double)0, MIN2(vfree, vsafe));
    return v;
}


MSCFModel*
MSCFModel_GM::duplicate(const MSVehicleType* vtype) const {
    return new MSCFModel_GM(vtype);
}

/****************************************************************************/

Back to the top