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

Provide vessel speed over ground when available #1346

Open
gavinmacaulay opened this issue Jun 22, 2024 · 3 comments
Open

Provide vessel speed over ground when available #1346

gavinmacaulay opened this issue Jun 22, 2024 · 3 comments
Assignees
Milestone

Comments

@gavinmacaulay
Copy link
Contributor

I miss the speed_over_ground variable when reading in Simrad EK data files - is it somewhere? A search of the echopype code didn't show up any code dealing with extracting speed from NMEA data, so perhaps not?

The sonar-netcf4 convention has 'speed_over_ground' as a variable under Platform (v1) or under Platform/NMEA (v2).

@leewujung
Copy link
Member

leewujung commented Nov 13, 2024

@NikiDab :
For sound_over_ground, you can try set a breakpoint after

parser.parse_raw()

(parser.parse_raw() actually parses the data)
This way you can inspect what is in the parsed data object (some attributes are in a dict of dict of dict structure, so not the easiest to work with -- this came from PyEcholab and we may want to restructure down the line if there’s extra time).

For EK80 data, the NMEA data are parsed into the parser.nmea attribute

self.nmea = defaultdict(list) # Dictionary to store NMEA data(timestamp and string)

You can see if this variable is parsed in there in the test files we have.
If not, you may have to trace back to the parser code to see why that is not parsed properly.
It could also be something that Simrad added more recently (the .raw data format still gets updated regularly), and our test files may not include that.

@gavinmacaulay: could you provide a test file (ideally <5 MB, the smaller the better) so that @NikiDab could take a stab at this?

Thanks both!

@gavinmacaulay
Copy link
Contributor Author

I don't have example files that small. However, a 35MB one is a available here: https://aqualyd-my.sharepoint.com/:i:/g/personal/enquiries_aqualyd_nz/EdZq6IzuN8hNpIEpnh0xhv4BaJLTR0l4nLyxfTbIgSZ0Xw?e=exdtWN. The link will be valid for 1 week, so take a copy if you need.

@NikiDab
Copy link

NikiDab commented Nov 14, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

No branches or pull requests

3 participants