jpasqua / VisibleTesla

Java App to monitor and Control the Tesla Model S
125 stars 41 forks source link

Wrong distance when in Europe #45

Closed sunejak closed 10 years ago

sunejak commented 10 years ago

When looking at trips the distance is not in km, so there is some missing conversion. In real life the distance is 53.8 km, but the trips tab shows 34 km screenshot from 2014-01-02 20 48 50 screenshot from 2014-01-02 20 48 21 screenshot from 2014-01-02 20 48 34

jpasqua commented 10 years ago

Thanks for the report. This should be fixed in 0.25.02 (released 2013-01-01). Please let me know if you still experience this problem. As mentioned in the release notes, I know there are still missing conversions lurking in the code and I will try to find and exterminate them.

sunejak commented 10 years ago

screenshot from 2014-01-02 21 51 36

Nice, I also like the new graph on the map :)

screenshot from 2014-01-02 21 54 17

sunejak commented 10 years ago

But the speed in the graph is not right, we are running on cruise control at 70 km/hr from about 20km to 50km

jpasqua commented 10 years ago

If you hover your mouse over a marker on the map you’ll get a popup that shows the time and speed (see attached image).

If you hover over a marker that corresponds to the time you were on cruise control at 70 km/hr, what does it say? If possible, please save the html file with the incorrect graph and mail it to me at joe@NoRoomAtTheInn.org.

Thanks for your help, Joe

On Jan 2, 2014, at 12:58 PM, Sune Jakobsson notifications@github.com wrote:

But the speed in the graph is not right, we are running on cruise control at 70 km/hr from about 20km to 50km

— Reply to this email directly or view it on GitHub.

sunejak commented 10 years ago

Sorry, but hoovering shows distance in miles... You have the files in your mailbox

jpasqua commented 10 years ago

These are both fixed in 0.25.03. I will close this bug after the release. I'm leaving this in VerifyFix mode until the release.

If anyone finds other unit conversion problems (I suspect there are more), please open a separate bug report for them. It makes it much easier for me to track individual problems even if they are related. Thanks.

jpasqua commented 10 years ago

Just released.