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.

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

UBX_NAV_PVAT not supported #97

Closed
Cils opened this issue Jan 11, 2022 · 4 comments · Fixed by #99 or #111
Closed

UBX_NAV_PVAT not supported #97

Cils opened this issue Jan 11, 2022 · 4 comments · Fixed by #99 or #111

Comments

@Cils
Copy link

Cils commented Jan 11, 2022

Subject of the issue

The messages UBX_NAV_PVAT are not supported.

Your workbench

  • ZED-F9R, with newest firmware (v. 1.21)
  • Here the rest but probably not relevant
  • ESP32
  • Qwiic
  • USB

Steps to reproduce

Try to use the UBX_NAV_PVAT output messages.

Expected behavior

Use of messages of type UBX_NAV_PVAT. E.g.: message pull and log, ...

Actual behavior

The message type UBX_NAV_PVAT are not implemented in the library.

@PaulZC PaulZC linked a pull request Jan 14, 2022 that will close this issue
@PaulZC
Copy link
Collaborator

PaulZC commented Jan 14, 2022

Hi @Cils,

Thank you for raising this issue. We have added full support for PVAT in v2.2.0 of the library. It will be released in a few minutes.

Best wishes,
Paul

@PaulZC
Copy link
Collaborator

PaulZC commented Feb 5, 2022

Hi @Cils ,

Just to let you know: I found a bug in the "auto" support for PVAT. I had forgotten to include PVAT in getMaxPayloadSize. Polling with getNAVPVAT will work, but the full "auto" support (including data logging) will fail.

I will release a fix for this in a few days. I have corrected it - but I cannot release it yet as I am in the middle of making other changes to the library.

Best wishes,
Paul

@Cils
Copy link
Author

Cils commented Feb 10, 2022

Many thanks for adding the PVAT support and letting me know of the ongoing problem. I noticed that the "auto" PVAT was not working but didn't had time to look into it. You saved me some troubleshooting time.

best wishes,
Cils

@PaulZC
Copy link
Collaborator

PaulZC commented Feb 11, 2022

Reopening this, just in case any other users come across the same issue.
It will be resolved in the next release.
For now, please use the release_candidate branch if you need full "auto" PVAT.

@PaulZC PaulZC reopened this Feb 11, 2022
@PaulZC PaulZC linked a pull request Feb 20, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants