search

Zabari / STEPS

4 stars 2 forks source link

Escalator Data Grabber fails for double digit escalator numbers #4

Closed afischer closed 9 years ago

afischer commented 9 years ago

e1,e2,v1,v2 = int(s[1]), int(s[7]), int(s[4]), int(s[10]) won't actually work for any of the double digit numbered escalators.

Check out if you can fix this?

amortenson commented 9 years ago

will do. Any idea why what was there before was breaking? Because I'm not sure how else to fix this...

On Thu, Jan 15, 2015 at 3:33 PM, Andrew Fischer notifications@github.com wrote:

Assigned #4 https://github.com/Zabari/STEPS/issues/4 to @trunkatedpig https://github.com/trunkatedpig.

— Reply to this email directly or view it on GitHub https://github.com/Zabari/STEPS/issues/4#event-219678854.

Zabari commented 9 years ago

What if you just put them on seperate lines

On Thu, Jan 15, 2015, 8:02 PM Aaron Mortenson notifications@github.com wrote:

will do. Any idea why what was there before was breaking? Because I'm not sure how else to fix this...

On Thu, Jan 15, 2015 at 3:33 PM, Andrew Fischer notifications@github.com

wrote:

Assigned #4 https://github.com/Zabari/STEPS/issues/4 to @trunkatedpig https://github.com/trunkatedpig.

— Reply to this email directly or view it on GitHub https://github.com/Zabari/STEPS/issues/4#event-219678854.

— Reply to this email directly or view it on GitHub https://github.com/Zabari/STEPS/issues/4#issuecomment-70192424.

amortenson commented 9 years ago

don't be ridiculous I don't even know what separate lines are

On Thu, Jan 15, 2015 at 8:24 PM, Daniel Zabari notifications@github.com wrote:

What if you just put them on seperate lines

On Thu, Jan 15, 2015, 8:02 PM Aaron Mortenson notifications@github.com wrote:

will do. Any idea why what was there before was breaking? Because I'm not sure how else to fix this...

On Thu, Jan 15, 2015 at 3:33 PM, Andrew Fischer < notifications@github.com>

wrote:

Assigned #4 https://github.com/Zabari/STEPS/issues/4 to @trunkatedpig https://github.com/trunkatedpig.

— Reply to this email directly or view it on GitHub https://github.com/Zabari/STEPS/issues/4#event-219678854.

— Reply to this email directly or view it on GitHub https://github.com/Zabari/STEPS/issues/4#issuecomment-70192424.

— Reply to this email directly or view it on GitHub https://github.com/Zabari/STEPS/issues/4#issuecomment-70194407.

afischer commented 9 years ago

reallytho

amortenson commented 9 years ago

​

On Thu, Jan 15, 2015 at 8:51 PM, Andrew Fischer notifications@github.com wrote:

[image: reallytho] https://camo.githubusercontent.com/ab3a49e50f23a0117b4b0f2ee2527e606379a025/687474703a2f2f692e74656c6567726170682e636f2e756b2f6d756c74696d656469612f617263686976652f30323533362f67656f7267652d627573682d335f32353336363032622e6a7067

— Reply to this email directly or view it on GitHub https://github.com/Zabari/STEPS/issues/4#issuecomment-70196709.

amortenson commented 9 years ago

Last commit should fix it; can't test without the raspberry pi but 95% sure it will work.