Free flow and constrained speed

[Trietes]

Shouldn't the csv for historical traffic first contain the free flow speed and then the constrained speed? So
csv_writer.writerow([tile_id, FREE_FLOW_SPEED, CONSTRAINED_FLOW_SPEED, traffic_encoded_values])
instead of
csv_writer.writerow([tile_id, CONSTRAINED_FLOW_SPEED, FREE_FLOW_SPEED, traffic_encoded_values])
in the update_traffic.py?

https://github.com/alinmindroc/valhalla_traffic_poc/blob/7022ea363f2de800ad8ca3f334b581e5ad070d79/update_traffic.py#L37C9-L37C104

[Blagodarenko]

Yes, you right, free_flow before constrained speed

https://github.com/valhalla/valhalla/blob/89dfed0aa4e0d15d8a30fda19d44ff634a57734c/src/mjolnir/valhalla_add_predicted_traffic.cc#L109

[Trietes]

Thanks for checking! Found the same code in the valhalla repository. I'll create a PR to fix it, when I've some time.