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

[param] rerequest don't count down retry counter. #276

Closed
vooon opened this issue Apr 18, 2015 · 3 comments
Closed

[param] rerequest don't count down retry counter. #276

vooon opened this issue Apr 18, 2015 · 3 comments
Labels
bug plugin PX4
Milestone
Version 0.12

Comments

@vooon
Copy link
Member

vooon commented Apr 18, 2015

Example mesages:

[ WARN] [1429353968.954751803]: TM: Clock skew detected (0.010165189 s). Hard syncing clocks.
[ WARN] [1429353969.569281850]: PR: request param #0 timeout, retries left 2, and 1 params still missing
[ WARN] [1429353970.593917810]: PR: request param #0 timeout, retries left 2, and 1 params still missing
[ WARN] [1429353971.603525702]: PR: request param #0 timeout, retries left 2, and 1 params still missing
[ WARN] [1429353972.612350979]: PR: request param #0 timeout, retries left 2, and 1 params still missing

Also i seen TM regular messages with USB connection.
@vooon
Copy link
Member Author

vooon commented Apr 18, 2015

Debugging:

[ WARN] [1429359741.420699527]: PR: request param #0 timeout, retries left 2, and 1 params still missing
[DEBUG] [1429359741.420806318]: PR:m: request '', idx 0
[DEBUG] [1429359741.420887088]: serial0: send: Message-Id: 20 [20 bytes] Sys-Id: 1 Comp-Id: 240 Seq: 132

Note that seq is out of order (prev 40, this 132 next 41).

@vooon
Copy link
Member Author

vooon commented Apr 18, 2015

Seems that seq error is corresponding #269.
For this issue here interesting log:

[ WARN] [1429361808.515206519]: PR: request param #0 timeout, retries left 2, and 1 params still missing
[DEBUG] [1429361808.515332821]: PR:m: request '', idx 0
[DEBUG] [1429361808.515431181]: serial0: send: Message-Id: 20 [20 bytes] Sys-Id: 1 Comp-Id: 240 Seq: 142
[DEBUG] [1429361808.518500555]: serial0: recv: Message-Id: 22 [25 bytes] Sys-Id: 1 Comp-Id: 50 Seq: 192
[DEBUG] [1429361808.518660580]: PR: Update param MKBLCTRL_TEST (1/365) value: 0 int

MR: request param_index = 0
PX: here your param_value with param_index == 1 (!)

@vooon vooon added this to the Version 0.12 milestone Apr 18, 2015
@vooon
Copy link
Member Author

vooon commented Apr 18, 2015

Also perhaps that this issue related to #274.

@vooon vooon closed this as completed in 86255ba Apr 18, 2015
vooon added a commit that referenced this issue Apr 18, 2015
@vooon
Merge branch 'master' into coverity_scan
1a2e421 
* master:
  plugin: param fix #276: add check before reset request downcounter.
  libmavconn #269: add seq number to debug
  gcs bridge fix #277: add link diagnostics
  vim: Add YouCompleteMe configuration file
  plugin: setpoint_position #273: add quirk for PX4.
  readme: fir glossary misprint
  readme: add notes about catkin tool
  0.11.1
  prepare release 0.11.1
  scripts #262: update mavwp
  scripts #262: mavsetp, new module mavros.setpoint
  mavftpfuse #129: done!
  mavftpfuse #129: cache file attrs
  mavftpfuse #129: initial import
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug plugin PX4
Projects
None yet
Milestone
Version 0.12
Development

No branches or pull requests
1 participant
@vooon