Discrepancies in README vs Protobuf variable names

[DenisCarriere]

Which ones are the correct ones?

• [ ] startIntersectionId or fromIntersectionId? (Geometry)
• [ ] endIntersectionId or toIntersectionId? (Geometry)
• [ ] inboundReferenceIds or inboundSegmentIds? (Intersection)
• [ ] outboundReferenceIds or outboundSegmentIds (Intersection)

My preference is the README's variable names.

README Geometry

{
    "type": "Feature",
    "properties": {
        "id": "NxPFkg4CrzHeFhwV7Uiq7K",
        "startIntersectionId": "5gRJyF2MT5BBErTyEesQLC",
        "endIntersectionId": "N38a21UGykpnqxwez7NGS3",
        "forwardReferenceId": "2Vw2XzW4cs7r32RLhQnqwA",
        "backReferenceId": "VXKSEokmvBJ81XHYhUronG",
        "roadClass": 3
    },
    "geometry": {
        "type": "LineString",
        "coordinates": [
            [-74.003388, 40.634538],
            [-74.003621, 40.634383],
            [-74.003621, 40.634383],
            [-74.004107, 40.63406]
        ]
    }
}

Proto Geometry message

message SharedStreetsGeometry {

    string id = 1;

    string fromIntersectionId = 2;
    string toIntersectionId = 3;
    string forwardReferenceId = 4;
    string backReferenceId = 5;

    RoadClass roadClass = 6;
    repeated float latlons = 7; // interleaved lat/lon pairs in sequence

}

[DenisCarriere]

Also noticed Intersection had a few discrepancies.

README

{
    "type": "Feature",
    "properties": {
        "id": "5gRJyF2MT5BBErTyEesQLC",
        "osmNodeId": 42460951,
        "outboundSegmentIds": ["6mjqqv7YNsp4541DmrrRbV", "jwwKcUvHuCw6GJJAT3mDQ", "2Vw2XzW4cs7r32RLhQnqwA"],
        "inboundSegmentIds": ["VmSkhzGKoEc767w98x35La", "VXKSEokmvBJ81XHYhUronG", "B7RPzs3hb1cSXqYcAKmUhE"]
    },
    "geometry": {
        "type": "Point",
        "coordinates": [-74.003388, 40.634538]
    }
}

Proto

message SharedStreetsIntersection {

  string id = 1;
  uint64 nodeId = 2;
  float lat = 3;
  float lon = 4;

  repeated string inboundReferenceIds = 5;
  repeated string outboundReferenceIds = 6;

}

[kpwebb]

Thanks for catching this! Just merged the commit to update docs (JSON examples now match protobuf names).