itinero / routing

The routing core of itinero.
Apache License 2.0
221 stars 70 forks source link

Bug when using proseccRestrictions with LoadOsmData #254

Closed TomQv closed 5 years ago

TomQv commented 5 years ago

void LoadOsmData(this RouterDb db, Stream data, bool allCore = false, bool processRestrictions = true, params Vehicle[] vehicles);

Using this overload with car, pedestrian, bicycle and processRestrictions =true is resulting in a routerdb with only car inluded. Pedestrian and bicycle are gone.

Same call with processRestrictions =false, routerdb is fine.

xivk commented 5 years ago

Trying to reproduce this with:

            var routerDb = new RouterDb();
            using (var stream = File.OpenRead(Download.LuxembourgLocal))
            {
                routerDb.LoadOsmData(stream, false, true, Itinero.Osm.Vehicles.Vehicle.Car, Itinero.Osm.Vehicles.Vehicle.Pedestrian,
                    Itinero.Osm.Vehicles.Vehicle.Bicycle);
            }

I get this in the routerdb:

image

TomQv commented 5 years ago

Strange, I also can't reproduce it anymore. Did you fix something in the meantime? Its possible, that I had the issue with an older version. Yesterday I upgraded to the latest Itinero to test this #253

xivk commented 5 years ago

This gets tested in almost the same way in the functional tests every build so this should be fine but I'll keep an eye on this.

Also, make sure to manually select the correct package. We've gotten to the bad situation again where the prerelease # exceeds 100, nuget handles this as < 99

TomQv commented 5 years ago

oups... maybe you should go to 1.4.1-pre001 or so....

grafik

xivk commented 5 years ago

Yes, it's a mess, the thing is that it should never have gotten to 100 in the first place. Working hard to finally release 1.4.0 stable as you may have noticed.

TomQv commented 5 years ago

Working hard to finally release

I know exactly what you mean...