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.

Sign up for GitHub

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

Jump to bottom

UBX GPS Status never changes to 'Module Found' #49

Closed
dojomouse opened this issue Nov 12, 2012 · 1 comment
Closed

UBX GPS Status never changes to 'Module Found' #49

dojomouse opened this issue Nov 12, 2012 · 1 comment

Comments

@dojomouse
Copy link
Contributor

UBX GPS starts, connects to satellites, and displays accurate position. Status remains "Trying to connect to a ubx module" though, despite obviously being connected.

Bug seems to be in ubx.c; the conditional

if (!ubx_healthy && ubx_success_count == UBX_HEALTH_SUCCESS_COUNTER_LIMIT)

will never execute as 'ubx_healthy' is set to true two lines before the evaluation.
@LorenzMeier
Copy link
Member

Fixed in 66e8067

TSC21 pushed a commit to TSC21/Firmware that referenced this issue Feb 15, 2019
@priseborough
Merge pull request PX4#49 from PX4/pr-fixMagCovariance
96c0e18 
EKF: Fix magnetometer fusion and covariance handling
baumanta pushed a commit to baumanta/Firmware that referenced this issue May 27, 2019
@DanielePettenuzzo
Merge pull request PX4#49 from Auterion/devel-task-based-takeoff-trigger
8b56494 
mc_pos_control/FlightTasks: trigger takeoff based on task
PX4BuildBot added a commit that referenced this issue Jan 7, 2020
@PX4BuildBot
PX4/NuttX branch test: master
7b9ffea 
NuttX:
 - https://github.com/PX4-NuttX/nuttx/tree/master
 - PX4/NuttX@973938d

Changes from PX4/NuttX (PX4/NuttX@9331fda) in current PX4/master (fc9df31)
PX4/NuttX@9331fda...973938d

973938dc1a (HEAD, origin/master) Merge branch 'master' of bitbucket.org:nuttx/nuttx
c5e25dc259 Merge branch 'master' of github.com:apache/incubator-nuttx
9d5d60f Run all .c and .h files in last PR49 through nxstyle.
a8d63c0 various fixes (#49)
077ef70 cxd56xx improvements (#48)
c5090d3 Merge branch 'master' of github.com:apache/incubator-nuttx
1c53d2b Nxstyle checkpatch support (#47)
4ea49c5 Fix issues on fe310 including interrupt handling (#46)
926a41f bcm43xxx: correct typo error to fix build break in photon board (#45)
aad99a5 Fix typo in the file_read description (#44)

Apps:
 - https://github.com/PX4/NuttX-apps/tree/master
 - PX4/NuttX-apps@74ba8ff

Changes from PX4/NuttX-apps (PX4/NuttX-apps@91b6ad6) in current PX4/master (fc9df31)
PX4/NuttX-apps@91b6ad6...74ba8ff

74ba8ff2 (HEAD -> master, origin/master) Merge pull request #5 from xiaoxiang781216/fix-void-cast
f00b4e6f Merge pull request #4 from xiaoxiang781216/fix-suffix
85715845 Unify the void cast usage
6a65664c Application.mk remove _main suffix from REGLIST
73e1696e Merge pull request #3 from xiaoxiang781216/builtin
7a645baa Add .updated target to handle the no builtin app case correctly
8c8c4813 fix builtin_list.c:58:10: fatal error: builtin_proto.h: No such file or directory (#2)
64bc8f5b Run all .c and .h files modified in last PR through nxstyle.
49c99750 Wapi simplify (#1)
0536c5b5 apps/nshlib/nsh_parse.c:  Correct an error found in build testing.  Commit 2a462c78aa5f4ea6dc374eedd86bc85f9f79a0c4 was insufficient.  It was a mistake to revert Xiang's change of commit 9defae8af641752506d92b72ea68c8f94d24d580.  But we are on a different vector now.
PX4BuildBot added a commit that referenced this issue Jan 8, 2020
@PX4BuildBot
PX4/NuttX branch test: master
0891dd1 
NuttX:
 - https://github.com/PX4-NuttX/nuttx/tree/master
 - PX4/NuttX@f459d55

Changes from PX4/NuttX (PX4/NuttX@9331fda) in current PX4/master (f3cd5b1)
PX4/NuttX@9331fda...f459d55

f459d55808 (HEAD, origin/master) Merge branch 'master' of github.com:apache/incubator-nuttx
44dcda9 boards/arm/lpc43xx/bambino-200e/src/lpc43_appinit.c:  Fix typo noted in last PR by João Matos.
e298ccef36 Merge branch 'master' of github.com:apache/incubator-nuttx
208463e boards/arm/lpc43xx/bambino-200e/src/lpc43_appinit.c: Eliminate a warning found in build testing.
afc2bff net/icmpv6/icmpv6_netpoll.c: Fix two warnings found in build testing
d1bec5a6b5 Merge branch 'master' of github.com:apache/incubator-nuttx
15eddd2 cxd56xx improvements (#48) and various other fixes (#49)
973938dc1a Merge branch 'master' of bitbucket.org:nuttx/nuttx
c5e25dc259 Merge branch 'master' of github.com:apache/incubator-nuttx
9d5d60f Run all .c and .h files in last PR49 through nxstyle.

Apps:
 - https://github.com/PX4/NuttX-apps/tree/master
 - PX4/NuttX-apps@5c936ce

Changes from PX4/NuttX-apps (PX4/NuttX-apps@91b6ad6) in current PX4/master (f3cd5b1)
PX4/NuttX-apps@91b6ad6...5c936ce

5c936ce0 (HEAD -> master, origin/master) Various fixes (#6)
74ba8ff2 Merge pull request #5 from xiaoxiang781216/fix-void-cast
f00b4e6f Merge pull request #4 from xiaoxiang781216/fix-suffix
85715845 Unify the void cast usage
6a65664c Application.mk remove _main suffix from REGLIST
73e1696e Merge pull request #3 from xiaoxiang781216/builtin
7a645baa Add .updated target to handle the no builtin app case correctly
8c8c4813 fix builtin_list.c:58:10: fatal error: builtin_proto.h: No such file or directory (#2)
64bc8f5b Run all .c and .h files modified in last PR through nxstyle.
49c99750 Wapi simplify (#1)
dirksavage88 pushed a commit to dirksavage88/Firmware that referenced this issue Dec 28, 2023
@modaljc
Add 60k RPM support to voxl-esc driver (PX4#49)
d4f025b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@LorenzMeier @dojomouse