noi-techpark / odh-mentor-otp

4 stars 8 forks source link

new time travel layer #135

Closed stefanocudini closed 2 years ago

stefanocudini commented 2 years ago

require a rebuild of journey and run a new container traffic having a subdomain

rcavaliere commented 2 years ago

@RudiThoeni please integrate this PR, but only in the development environment. Thanks @stefanocudini for the update!

stefanocudini commented 2 years ago

compared to analytics I tried adding popups with travel times, it could be useful traffic_traveltime

rcavaliere commented 2 years ago

@stefanocudini we also tried this in the past, but from a mobile device this was a mess. Did you try to test this? I am curious to test how you implemented this!

stefanocudini commented 2 years ago

hi @rcavaliere I thought of another proposal, see screencast below: 1) show travel times only in case the traffic level is yellow or red on each linkstation 2) no need for popup clicks, but small tooltips are shown instead 3) this continues to give detailed information but without user interaction, I think is useful to know the travel time in a red street

if you think it's ok I'll send a PR soon that we can try in development traffic

rcavaliere commented 2 years ago

@stefanocudini that's really cool! yes, let's go in this direction. Is this implemented in this PR or do you still have to implement it? If its already the content of this PR @RudiThoeni let's put this in development and if everything works well let's then go into production

stefanocudini commented 2 years ago

just sent a new PR https://github.com/noi-techpark/odh-mentor-otp/pull/137 for this and other little fiexs