Michael Bell

Results 95 comments of Michael Bell

Let's keep it open until we've completely removed it. It's still being used to install CI worker dependencies.

I did actually make more progress since the last update, to the point where I made a [hype video thread](https://twitter.com/mjjbell/status/1543701753186615301) showing what it could do. I was getting stuck on...

Yes I believe the AWS SDK is no longer needed after #6030

Sorry for the late response. I'm still keen on contributing, but like others I'm not sure how much time I can commit. On a practical note, it looks like the...

It does seem odd to code for a case that we then assert is not true. https://github.com/Project-OSRM/osrm-backend/blob/3bb82ce1e2fd2299712b96176e651e6f7e999aae/include/extractor/raster_source.hpp#L66-L72 IMO it would make sense to have the assert first and then have...

It would be beneficial to add minimal test cases to the Cucumber suite that capture the two problems you have identified.

@GitBenjamin I'm going to add the fix to this PR for problem 1. It's unclear if problem 2 would also be fixed, but we can address that in a separate...

Does this work correctly if you remove the `exclude=toll` parameter?

Without that parameter, it does the right thing on the demo server: http://map.project-osrm.org/?z=18&center=10.728346%2C106.695517&loc=10.7288943%2C106.6963952&loc=10.7287811%2C106.6937228&hl=en&alt=0&srv=1 So I'm interested in what you're seeing in this case.

Thanks, I'm able to reproduce the problem. Will investigate further.