vingerha / gtfs2

Support GTFS in Home Assistant GUI-only
https://github.com/vingerha/gtfs2
MIT License
84 stars 8 forks source link

This screen about intercity trains is confusing and unclear #25

Closed DeflateAwning closed 9 months ago

DeflateAwning commented 9 months ago

Problem:

I don't understand what this screen is asking: image

There are no inter-city trains in my city. Which options am I supposed to pick?

If you can explain what's going on here, maybe I can suggest some better wording?

vingerha commented 9 months ago

I see the issue...is a bit odd. This screen allows to selectr between using

vingerha commented 9 months ago

Suggestion... and do note that clicking on the 'doc' link will provide more details...it is impossible to provide all explanations in (size-limited) config screens image

DeflateAwning commented 9 months ago

You shouldn't have to visit documentation to understand what the options are.

I still don't understand, even after reading the docs.

What happens when you click each button? Which button did you click in your bus example in the documentation?

vingerha commented 9 months ago

You are the first and only that seems to have an issue with this, so if you think you can do better...please suggest a PR

DeflateAwning commented 9 months ago

I'm still trying to understand what the buttons do so that I can. Can you please try explaining what happens when you choose each option?

Sorry I'm not understanding. Please don't get angry, though.

vingerha commented 9 months ago

First : allows to select transport of those provided through the zip...but it excludes (!) selecting trains Second: allows to setup train connections, based on (manually) entered cities WHY? because train stations often have a lot (!) of stop_id and you'd need to configure each one of them for both start and end

EDIT: on the 2nd...it ONLY allows to deal with trains....nothing else

vingerha commented 9 months ago

Hey, are you OK with the above ? I alreayd brought a few things to production but I am not totally happy myself as well... 100% will never be achieved but maybe you have the right response to my blind eye(s) ? Else, please close the topic...if no response I will close in a few days myself

DeflateAwning commented 9 months ago

Ah sorry, thanks for the explanation.

May I propose this as the text?

Select a type of transport:

  1. Intracity Transport: exclude selecting intercity trains
  2. Intercity Transport: configure cities and stop IDs manually

Let me know if I'm still missing the point though.

vingerha commented 9 months ago

But that is not the situation

  1. can have long distance buses, can be for a region, country, etc.
  2. is ONLY trains and even trains have intra-city stops
DeflateAwning commented 9 months ago

I'll try again:

Select a display option:

  1. Default: Datasource includes many types of transit.
  2. Datasource is too large to enumerate; configure cities and stop IDs manually
DeflateAwning commented 9 months ago

Can you give an example of a city/GTFS source that I can view to better understand these two options?

vingerha commented 9 months ago

Thx for sticking / persevering :) but again....this is not what it is

  1. covers all transport, except trains
  2. covers only trains

Examples Netherlands ...there is only one zip covering the whole country, same for Switzerland Germany has various offerings, one for country/all, one for longdistance trains and multiple for regional trains

vingerha commented 9 months ago

I am closing this topic as it seems to not find a solution and no-one else joins or comments.

DeflateAwning commented 9 months ago

Hold on, your solution is to keep it as-is? I'm convinced that any of the options above are better than what's currently there.

vingerha commented 9 months ago

I changed it a bit in the latest release but you should also understand that I cannot uphold a discussion that has seemingly no interest to the larger group of users

vingerha commented 9 months ago

And donot get me wrong...if you have a matching good proposal...happy to implement, I donot have the single view on the truth :)

vingerha commented 9 months ago

New setup image

DeflateAwning commented 9 months ago

This looks great, way better, thanks!