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

Tilt with Surpad(app) doesn’t work well #395

Open
APOS80 opened this issue Jun 28, 2024 · 13 comments
Open

Tilt with Surpad(app) doesn’t work well #395

APOS80 opened this issue Jun 28, 2024 · 13 comments

Comments

@APOS80
Copy link

APOS80 commented Jun 28, 2024

Subject of the issue

Describe your issue here. Additionally, screenshots are easy to paste into github.

Everything works well with SW Maps but not using tilt with Surpad.

Without tilt it works well but when enabling tilt Surpad thinks it’s thousands of kilometres away.

It makes me think that something changes with the NMEA message when enabling tilt.

Your workbench

  • What version of RTK firmware are you running? This can be found when the serial menu is opened (also in the settings.txt file, and in the serial output at power up).
    Torch
    June 27 binary
  • What radios are you using: Bluetooth, WiFi, and/or ESP-Now? What app are you using to connect over Bluetooth? Are you transmitting NTRIP back to the device?
    Bluetooth with Ntrip via phone
  • Are there any additional details that may help us help you?

Steps to reproduce

Tell us how to reproduce this issue. Please post any log files from serial output that may have been generated.

Expected behavior

Tell us what should happen

Actual behavior

Tell us what happens instead

@APOS80
Copy link
Author

APOS80 commented Jun 29, 2024

Without tilt
Screenshot_20240629-102917
With tilt
Screenshot_20240629-103021

Hope that helps

@nseidle
Copy link
Member

nseidle commented Jun 30, 2024

Thanks for reporting! I've got a demo version of Surpad working.

0.11m to 6555559m is a bit of a problem!

It makes me think that something changes with the NMEA message when enabling tilt.

Agreed. I suspect Surpad is parsing the NMEA in a different way. We are outputting (we believe) fully formed NMEA but there's always small changes. And different parsers can withstand varying levels of difference. AFAIK the lat/lon/alt NMEA sentences we output during tilt compensation is the same length and indistinguishable from non-tilt NMEA but obviously something is amiss.

Unfortunately I am out on vacation for a bit. I hope this doesn't block you from work for too long. I know you are capable of compiling the source so please poke at the problem if you have the desire.

@APOS80
Copy link
Author

APOS80 commented Jun 30, 2024

I’m also going away for a while.

Sadly I haven’t solved the compiling part at this moment.

off topic:
I’ve never done big stuff, only playing around with small stuff. As of now I’m trying to learn more SQL and Python for QGis work.

@APOS80
Copy link
Author

APOS80 commented Jul 11, 2024

I wonder if it’s as simple as LF instead of a CR character or the other way around.

@nseidle
Copy link
Member

nseidle commented Jul 21, 2024

I've tried to replicate this a few times and can't yet. What version of Surpad are you using? What view are you in?

I'm using Surpad v20240612, with Everywhere firmware v1.3, using the Point Survey mode.

image

image

Data in and out of tilt compensation mode seem to work just fine.

I know there is a newer version of Surpad but my trial version is expired so I haven't been able to test it yet.

Also, in the release candidate we've removed an extra null terminator character from the output. But I can't say it fixed anything until I can replicate the problem.

@APOS80
Copy link
Author

APOS80 commented Jul 22, 2024

I can try the release candidate and see if it works.

@nseidle
Copy link
Member

nseidle commented Jul 22, 2024

I was testing with v1.3 and I did not detect a problem.

What version of Surpad are you using? What view are you in?

@APOS80
Copy link
Author

APOS80 commented Jul 22, 2024

I was in the measuring view

@APOS80
Copy link
Author

APOS80 commented Jul 23, 2024

Surpad v4.2.220626.232644

@nseidle
Copy link
Member

nseidle commented Jul 23, 2024

I'm on v4.2.240104.164515. You might try upgrading. I'll see if I can find an older version of Surpad but after a quick search, it doesn't look possible.

@APOS80
Copy link
Author

APOS80 commented Jul 24, 2024

I’m unable to upgrade my version as of now.

But are you sure the NMEA messages is correct after going into tilt?

@APOS80
Copy link
Author

APOS80 commented Sep 1, 2024

Problem persists in v1.4

@nseidle
Copy link
Member

nseidle commented Sep 16, 2024

Everything is working for us. What version of SurPad are you using?

But are you sure the NMEA messages is correct after going into tilt?

Yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants