Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

misses first of each group of four #1

Merged
merged 1 commit into from

2 participants

@fphillips

with v=1
N3011.8281 W09747.9048
30.1971, -97.7984
satsInView: 11
Sats:51,32,14,
Sats:01,30,25,
Sats:20,16,

with v=0
N3011.8268 W09747.9037
30.1971, -97.7984
satsInView: 11
Sats:31,51,32,14,
Sats:22,01,30,25,
Sats:11,20,16,

I used my test program here: https://gist.github.com/1861056

@jamesp
Owner

Good catch, no idea how I missed that!

@jamesp jamesp merged commit d97d74a into from
@fphillips

Could you push out an updated npm package, too? Thanks.

@jamesp
Owner

Sure, done now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 24, 2012
  1. @fphillips
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  nmea.js
View
2  nmea.js
@@ -91,7 +91,7 @@ exports.parsers = {
// $GPGSV,3,1,12, 05,58,322,36, 02,55,032,, 26,50,173,, 04,31,085,
var numRecords = (fields.length - 4) / 4,
sats = [];
- for (var i=1; i < numRecords; i++) {
+ for (var i=0; i < numRecords; i++) {
var offset = i * 4 + 4;
sats.push({id: fields[offset],
elevationDeg: +fields[offset+1],
Something went wrong with that request. Please try again.