Todo List

Member ACarlaGameModeBase::BeginPlay () override

Recorder should not tick here, FCarlaEngine should do it.

Member ACarlaGameModeBase::Tick (float DeltaSeconds) override

Recorder should not tick here, FCarlaEngine should do it.

Member ACarlaWheeledVehicle::FlushVehicleControl ()

This function should be private to AWheeledVehicleAIController.

Member ACarlaWheeledVehicle::SetAIVehicleState (ECarlaWheeledVehicleState InState)

This function should be private to AWheeledVehicleAIController.

Member ACollisionSensor::SetOwner (AActor *NewOwner) override

Deregister previous owner if there was one.

Namespace carla

This shouldn't be exposed in this namespace.

Class carla::client::BlueprintLibrary

Works as a list but its actually a map. We should assess the use cases and reconsider this implementation.

Class carla::client::detail::CachedActorList

Dead actors are never removed from the list.

Member carla::FileSystem::ListFolder (const std::string &folder_path, const std::string &wildcard_pattern)

Do permission check.

Member carla::geom::Math::DistanceArcToPoint (Vector3D p, Vector3D start_pos, float length, float heading, float curvature)

: Because Unreal's coordinates, hacky way to correct the -y, this must be changed in the future

Member carla::geom::Vector3D::msgpack_pack (Packer &pk) const

The following is copy-pasted from MSGPACK_DEFINE_ARRAY. This is a workaround for an issue in msgpack library. The MSGPACK_DEFINE_ARRAY macro is shadowing our z variable. https://github.com/msgpack/msgpack-c/issues/709

Member carla::geom::Vector3DInt::msgpack_pack (Packer &pk) const

The following is copy-pasted from MSGPACK_DEFINE_ARRAY. This is a workaround for an issue in msgpack library. The MSGPACK_DEFINE_ARRAY macro is shadowing our z variable. https://github.com/msgpack/msgpack-c/issues/709

Member carla::opendrive::parser::ProfilesParser::Parse (const pugi::xml_document &xml, carla::road::MapBuilder &map_builder)

superelevation and crossfall still need RoadInfo classes.

Member carla::road::element::DirectedPoint::ApplyLateralOffset (float lateral_offset)

Z axis??

Member carla::road::element::FLAGS

Temporary flags to search lanes where we can find road marks. This needs to be expanded searching in shoulders too, but since shouders have a small width, they can cause problems while finding the nearest center of a lane given a location that are in a road, but very close to a shoulder.

Member carla::road::element::GeometrySpiral::DistanceTo (const geom::Location &) const override

Member carla::road::element::LaneCrossingCalculator::Calculate (const Map &map, const geom::Location &origin, const geom::Location &destination)

This case should also be handled.

Member carla::road::element::RoadInfoElevation::Evaluate (const double dist, double &out_tan) const

unused? you can use the polynomial directly.

Member carla::road::MapBuilder::GetLaneNext (RoadId road_id, SectionId section_id, LaneId lane_id)

Is it correct to use a road id as section id? (NS: I just added this cast to avoid compiler warnings).

: Find a better way to change from 'const Lane*' to 'Lane*' and use opt.second

Class carla::rpc::EpisodeInfo

Rename, does not represent the episode info anymore.

Member carla::rpc::ResponseError::_what

Needs initialization, empty strings end up calling memcpy on a nullptr. Possibly a bug in MsgPack but could also be our specialization for variants

Member carla::sensor::SensorData::client::detail::Simulator

This shouldn't be exposed in this namespace.

Class carla::streaming::detail::MultiStreamState

Lacking some optimization.

Member clmdep_msgpack::MSGPACK_API_VERSION_NAMESPACE (MSGPACK_DEFAULT_API_NS)

Workaround for finding the type.

Member FActorDefinitionValidator::AreTagsValid (const FString &Tags)

Do more checks.

Member FActorDefinitionValidator::IsIdValid (const FString &Id)

Do more checks.

Member FActorDefinitionValidator::IsValid (const EActorAttributeType Type)

Do more checks.

Member FActorDefinitionValidator::IsValid (const FActorDefinition &ActorDefinition)

Validate Class and make sure IDs are not repeated.

Member FActorDefinitionValidator::ValueIsValid (const EActorAttributeType Type, const FString &Value)

Do more checks.

Member FActorInfo::Velocity

To be used solely by the FWorldObserver.

Member FCarlaServer::AsyncRun (uint32 NumberOfWorkerThreads)

Define better the number of threads each server gets.

Class FPixelReader

This class only supports PF_R8G8B8A8 format.

Member FPixelReader::SendPixelsInRenderThread (TSensor &Sensor, bool use16BitFormat=false, std::function< TArray< TPixel >(void *, uint32)> Conversor={})

Can we make sure the sensor is not going to be destroyed?

Member MapGen::fixGraphData (Graph &graph)

This can probably be done at graph creation.

Member MapGen::getRotation (float angle0, float angle1, float angle2)

There has to be a better way.

Member MapGen::randomize (Graph &graph, const int32 seed)

We skip first face because is the surrounding face. But this won't be always the case, if the graph is generated differently it might be a different one.

Member MapGen::RoadSegmentDescription::Add (const GraphHalfEdge &Edge)

Use a scale.

Member UActorBlueprintFunctionLibrary::MakePedestrianDefinition (const FPedestrianParameters &Parameters, bool &Success, FActorDefinition &Definition)

We need to validate here the params.

Member UActorBlueprintFunctionLibrary::MakePropDefinition (const FPropParameters &Parameters, bool &Success, FActorDefinition &Definition)

We need to validate here the params.

Member UActorBlueprintFunctionLibrary::MakeVehicleDefinition (const FVehicleParameters &Parameters, bool &Success, FActorDefinition &Definition)

We need to validate here the params.

Member UCarlaEpisode::LoadNewOpendriveEpisode (const FString &OpenDriveString, const carla::rpc::OpendriveGenerationParameters &Params)

this can take too long to finish, clients need a method to know if the navigation is available or not.