Skip to content
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

Add Veloway system #33

Merged
merged 3 commits into from
Jan 13, 2015
Merged

Add Veloway system #33

merged 3 commits into from
Jan 13, 2015

Conversation

bparmentier
Copy link
Contributor

I'm not sure how to handle the encoding of the wcom tag. Could you give it a try?

@eskerda
Copy link
Owner

eskerda commented Jan 11, 2015

Seriously, indeed, WTF at wcom, wouldn't touch that with a long poking stick.No wonder it takes a human life to render a map on their website. Thinking about adding a Hall of Shame of different gems we find on our journey.

Note that some feeds do not include wcom (well, as null), failing at fix_name and not appending it to the stations list. For instance, vel'in just reports back 1 station

$ virtualenv2 v
$ source v/bin/activate
(v) $ sh runtests.sh TestSystems.test_veloway
[...]
Vélo Bleu has 166 stations
↑ stations look ok ↑                          

DK'Vélo has 31 stations
↑ stations look ok ↑                          

Vélocéa has 24 stations
↑ stations look ok ↑                          

Vél'in has 1 stations
↑ stations look ok ↑  
.
----------------------------------------------------------------------
Ran 1 test in 4.778s

OK
  • My recommendation would be to just drop wcom, as what it does provide is rather funky, varying from a human-readable address to just 17 Station. There's an open issue (Real address #21) about adding a cohesive address field to all stations.
  • Should we drop stations that have neutral set to 1 (assuming that could happen)?
  • I can't grasp the difference between ac and tc.

@bparmentier
Copy link
Contributor Author

  • Hey, got it working (I think) in bparmentier@65b101c. We could have dropped the wcom tag but the problem remains in name anyway…
  • We could keep neutral, but I don't see any real difference with disp. Should be discussed.
  • As I understand it, tc is the number of slots physically present, while ac is the number of working slots (some may be deactivated/broken/malfunctionning).

@eskerda
Copy link
Owner

eskerda commented Jan 13, 2015

Good, thanks. Note that if foo is not None and foo != "" is equivalent in python to if foo

@bparmentier
Copy link
Contributor Author

Note that if foo is not None and foo != "" is equivalent in python to if foo

Fixed.

Maybe should I also drop extra['slots_available'] and just use ac for extra'[slots']. What do you think?

@eskerda
Copy link
Owner

eskerda commented Jan 13, 2015

If the info is there, I would use it. On another feed we've got faulty_slots, but I think slots_available makes sense in this case.

eskerda added a commit that referenced this pull request Jan 13, 2015
@eskerda eskerda merged commit 287b44a into eskerda:experimental Jan 13, 2015
@eskerda
Copy link
Owner

eskerda commented Jan 13, 2015

And off we go! 4 new bike sharing schemes. Many thanks!

@bparmentier bparmentier deleted the veloway branch January 15, 2015 13:12
eskerda added a commit that referenced this pull request Mar 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants