matsim-vsp / parallel_qsim_rust

Parallel and distributed implementation of the Qsim in Rust
8 stars 0 forks source link

Move VehicleType into Vehicle #76

Closed Janekdererste closed 11 months ago

Janekdererste commented 1 year ago

Currently, the vehicle type is pushed to the link as a separate argument. This should be wrapped into the vehicle. I.e. each vehicle should contain the information which vehicle type it is.

This conflicts with the current vehicle type, which determines whether the vehicle is teleported or on the network. This is used in the messaging logic and should then be done differently

Janekdererste commented 11 months ago

This is solved in #99

The vehicle carries a vehicle type ID, as well as important properties for the simulation, such as max_v or pce. This way we avoid look-ups during the simulation, while keeping the memory footprint relatively small.

This also solves the original issue, that we don't have to pass vehicle_definition structs around anymore