-
-
Notifications
You must be signed in to change notification settings - Fork 187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switzerland: SBB Not work. #768
Comments
Thanks for your report, I was able to reproduce the problem. Most probably, SBB changed something on their end so that PTE (the library Transportr uses under the hodd) is unable to access the information. You might consider opening a bug report over there as well. |
Hej @Mannshoch (and others) I've implemented timetable.search.ch as a SBB replacement. Are you interested to participate in a beta-test? If yes, please message me at tobias[dot]bossert[at]fastpath[dot]ch, I will then send you an apk of Transportr with the new Provider included |
Hej testers! Thank you for your feedback. A new release is available with the following fixes:
You can download the new beta (unofficial!) release here: https://sun.fastpath.ch/index.php/s/PLs3xc8j7nzGm4i |
@sirtoobii I tried your unofficial beta, and it works like a charm. Thanks. |
Note that PTE upstream has SBB removed. If you want it back in Transportr, one would need to reach out to PTE for the reasons and depending on the answer add it back to the PTE staging fork Transportr is using. |
The reason was the shutdown of SBB's hafas endpoint (schildbach/public-transport-enabler#381). A merge of schildbach/public-transport-enabler#383 would resolve the issue. |
Cool, then you could consider submitting it to https://gitlab.com/opentransitmap/public-transport-enabler/ to get it into Transportr quicker. |
This seems to be the same source GNOME Maps uses, which is https://transport.opendata.ch/. By all means that should be merged then as it works fine with GNOME Maps. |
I know there's probably nothing you can do here, but I commented on the linked merge request again, because it should be merged. I hope once it is we'll get support here too |
https://gitlab.com/opentransitmap/public-transport-enabler/-/merge_requests/7 has been merged. Does this mean we can get this into Transportr now? |
I would say yes. I don't fully know that that, but it's what GNOME Maps uses and I'm pretty sure it works fine. |
Yes, this covers whole Switzerland. Regarding the logo and strings: I think its best if we keep the SBB logo (to avoid user confusion) but maybe use "search.ch" as name. Moreover, I found nothing in this regard in their ToS |
The current implementation is able to cover all public transports in switzerland. |
Personally I think we should also keep the name. As otherwise the logo and name would be mismatched. And the Swiss won't recognise search.ch |
Okay, so lets keep everything as is for now, that's also easiest :) |
I've tried to add the new provider, but at least on my Android emulator with Android 5.1 (API 22), Transportr crashes when selecting a station with
Similar things happen when searching for trips. It seems to be connected to the usage of anonymous functions in this provider, for example on the referenced line 166: https://gitlab.com/opentransitmap/public-transport-enabler/-/commit/9fb27a523c5e96a40261a12899aecca439918fbc#a6238f309ec31059e54599ca63e583410d7b57d8_0_166 I was suspecting Proguard to optimize this class away, but looking into the APK within IntelliJ shows the class as present. The same problem happens on both debug and release builds. I don't have ideas how to solve this issue for now. If someone feels like looking into it, the current version can be found on branch Btw, Android 12 (API 31) works as expected. |
seems fixed with 092b6ab |
@sirtoobii actually after discussing with @grote we would prefer not enabling library desugaring in general since it might lead to other, difficult to diagnose, problems in the long run. Could you perhaps instead try to rewrite some lines of your PTE provider to make it java 6 compatible, basically removing lambda expressions / replacing them by usual for-loops? |
Done: https://gitlab.com/opentransitmap/public-transport-enabler/-/merge_requests/10 |
For the Record, I found https://github.com/OpendataCH/Transport do jou use this API in Transportr? |
This implementation uses the same source as http://transport.opendata.ch/ (in german: https://opendata.ch/news/search-ch-rettet-transport-opendata-ch/#pll_switcher) |
Looking forward to seeing this in Transportr |
There hasn't been any activity on the GitLab issue for a bit now. What's going on? |
Describe the bug
If I use SBB as provider Transportr do not react on entry any citys.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Dropdown wit trainstations in Zürich appearr.
Versions (please complete the following information):
SBB API:
https://data.sbb.ch/api/v2/console
https://data.sbb.ch/api/v1/console
The text was updated successfully, but these errors were encountered: