<div dir="ltr"><div class="gmail_extra"><div>Hi Kasun!<br><br></div>I'm lvh, the primary author of the ticket. <br><br><div class="gmail_quote">On Wed, Feb 12, 2014 at 3:19 AM, Kasun Thennakoon <span dir="ltr"><<a href="mailto:tmkasun@gmail.com" target="_blank">tmkasun@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-size:13px;font-family:arial,sans-serif">It took long time to go through the ticket[1] and understands, <br>
still i couldn't find any issue with it, instead its working perfectly for me. and thanks to the twisted positioning NMEA adapter, it become easy to me to write a adapter for the device(MVT380 device[2]) which i'm using in my project[3].</div>
</div></blockquote><div><br></div><div>Awesome! Glad to hear it's working so well for you.<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr">
<div style="font-size:13px;font-family:arial,sans-serif">IMHO, <br><br>1. isn't it good to use items() rather than iteritems() since python 3.x has deprecated iteritems(), <font face="arial, sans-serif"><br></font></div>
</div></blockquote><div><br></div><div>There are many more Python 3 issues, I suspect. Both of those would work; I'm personally a bigger fan of using things like six' iteritems(d); which does d.items() on 3.x and d.iteritems() on 2.x, since you guarantee that you have identical semantics (modulo all the fancy things you can do with views that you can't do with iterators). I think there's a few cases where I don't really want *all* the items (although, of course, premature optimization is the root of all evil :)).<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-size:13px;font-family:arial,sans-serif"><font face="arial, sans-serif">2. </font>In nmea._UNIT_CONVERTERS dictionary mapper, can't see any usage of key 'K' which convert Km/h unit to meters/s because it has a FIXER for speedInKnots but not for speedInKMh, i think in $GPVTG NMEA string type (it is not defined in _SENTENCE_CONTENTS) string they use text 'K' to indicates that speed over ground is in kilometers/hour.</div>
</div></blockquote><div><br></div><div>That's there for forward compatibility as well as compatibility with really bad GPSes that use illegal units. <br></div><div><br>The fix for that lives in:<br></div><div><pre class="">
def _fixUnits</pre>So it is independent from any particular unit. That just blindly indexes into that dict; so if a GPS says something is in kph, it will convert to m/s, even when according to the spec kph isn't a valid unit for that.<br>
<br></div><div>Also as you mentioned for some *other* sentences it *is* a valid unit, but that's okay, we can just leave it there and then later add GPVTG support and get units conversion for free!<br></div><div><br>
</div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">
<div style="font-size:13px;font-family:arial,sans-serif"></div><div style="font-size:13px;font-family:arial,sans-serif">other than those suggestions,its works perfectly well. i wonder why this positioning module didn't merged with the trunk yet,and i'll be happy if i can help to get it merged. </div>
</div></blockquote><div><br></div><div>The ticket is up for review, and has been for a long time. Since you have apparently
reviewed the ticket and used the code, would you mind giving it a review
so it can (finally!) get into Twisted itself? Review instructions are
here:<br><br><a href="https://twistedmatrix.com/trac/wiki/ReviewProcess" target="_blank">https://twistedmatrix.com/trac/wiki/ReviewProcess</a><br><br>Some of the review comments so far have asked for extensive changes. I am reminded of this e-mail by Glyph:<br>
<br><a href="https://twistedmatrix.com/pipermail/twisted-python/2014-January/027894.html">https://twistedmatrix.com/pipermail/twisted-python/2014-January/027894.html</a><br><br></div><div>... which is quite long, but I think the bottom line is that you should just trust the review process and do that :)<br>
</div><div><br></div><div>
hth<br>lvh<br></div></div></div></div>