grote / Transportr

Free Public Transport Assistant without Ads or Tracking
https://transportr.app
GNU General Public License v3.0
1.04k stars 187 forks source link

New data for Spain causing problems // Update description #264

Closed tenminutemail closed 7 years ago

tenminutemail commented 7 years ago

Hi there,

Navitia published a lot of new datasets for Spain. Navitia playground contains a complete list of all datasets now available.

I don't know any code, so help would be appreciated.

Cheers

patkan commented 7 years ago

Hi @tenmimutemail could you please add some directions on how to reproduce the observed exception? By entering arbitrary names I could successfully use Madrid. (See station names in screenshot) I did however test only this.

tmp_15343-screenshot_2016-12-02-11-44-25-195152541

tenminutemail commented 7 years ago

Hi @patkan, thanks a lot for you reply.

If I try to use Transportr in Madrid, Barcelona or Valencia it works for some connections, for others it doesn't work and I get the error message with the NullPointerException.

I tried the same route as you suggested for Madrid and it works just fine; now if I try to go, for example, from: "ALCALA-RETIRO, Madrid" to "ATOCHA, Madrid" the app gives me an error message (I get the same error in Barcelona: for example "Pg Bonanova, Barcelona" to "Vall d'Hebron, Barcelona").

If I try the same connection from Alcala-Retiro to Atocha in Navitia playground it works:

navitia

As you can see in the screenshot from Navitia, it isn't clear which type of transport (bus, train, metro, etc.) is used. So there is a question mark for line 1 and 2. Maybe the problem is related to this.

grote commented 7 years ago

The problem is that the API does not return a product which is unexpected and should be fixed on the API side. See journey 0 in your screenshot for an example.

I'll still prepare a fix for that in Transportr.

tenminutemail commented 7 years ago

Thanks a lot @grote

grote commented 7 years ago

@tenminutemail 1.1.6-beta2 got released over the weekend. Please test if it resolves all issues with the SpainProvider.

tenminutemail commented 7 years ago

@grote Yes, it works fine! Thanks a lot, that's really awesome.

There is just one minor design issue; as there is no bus (or train, metro etc.) symbol next to the number, the whole symbol looks a bit unproportional. See the S1 and S2 symbol in the screenshot. Is there a way to fix that?

Did you consider updating the network info (and cities) I proposed in my first post?

transportr

grote commented 7 years ago

I had updated the description, but forgot your nice list. Could you create a new ticket for that or even update it yourself? The convention we use so far is to first list all the cities and then the transport networks.

The problem with the missing icons is because there's no product returned from Navitia. Somebody should get in touch with them and see if they can fix this.

tenminutemail commented 7 years ago

267 Here is the PR, I hope that's fine.

I'll try to contact Navitia as well.

grote commented 7 years ago

Thanks! :)

When contacting Navitia you might also want to bring up the issue that many stations are upper-case (not only for Spain). It would be nice if they could normalize these names to proper capitalization when importing the datasets.

tenminutemail commented 7 years ago

Hi @grote

I've contacted Navitia and they have already fixed it. So now, a train, bus, tram symbol is displayed when looking up a connection.

They'll also try to normalize the station names and try to implement proper capitalization.

tenminutemail commented 7 years ago

I just realised that there is one more problem with the new data.

If I look up departures for one stop in Navitia, I get this result:

departure

In Transportr, the data misbehaves and I get this result, where one train lines is only going into one direction and the other train line only goes into the other direction:

dep2

grote commented 7 years ago

Thanks for getting them to fix the products issue. Regarding the line destination of departures, please open a dedicated ticket about that. I am not sure, but this might belong to PTE.