transitland / transitland-processing-animation

Animating scheduled transit trips using the Transitland API and Processing
MIT License
279 stars 55 forks source link

Improve template #7

Closed Danie closed 6 years ago

Danie commented 6 years ago

Hello,

I created two changes regarding the template:

  1. change easily date, day, time format in main input section
  2. dynamic vehicle type label so that only available vehicles shown

Regards,

Daniel

willgeary commented 6 years ago

This looks amazing! Will review, thank you @Danie !

willgeary commented 6 years ago

I love this @Danie . However it can be a bit distracting and hard to follow at times when there are many quick changes to the labels. I think I will include this as a second template option in templates/ called dynamic_labels.pde so that users can request dynamic labels with a--dynamic_labels command line argument. Thoughts?

Danie commented 6 years ago

Hello @willgeary - I know what you mean. It could be very "nervous" to watch on it. I changed some more things and now only vehicles are showing that are now or were in the past in flow.

But your idea with the second template is also an option.

willgeary commented 6 years ago

Very cool @Danie , thank you! Will review it.

willgeary commented 6 years ago

Merged, thanks @Danie ! It's a great improvement