-
Notifications
You must be signed in to change notification settings - Fork 76
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
Comments
@NikiDab : echopype/echopype/convert/api.py Line 472 in 2813a25
( 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 echopype/echopype/convert/parse_base.py Line 64 in 2813a25
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! |
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. |
Thank you. I will download the file.
…-Niki
On Tue, Nov 12, 2024 at 9:00 PM Gavin Macaulay ***@***.***> wrote:
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.
—
Reply to this email directly, view it on GitHub
<#1346 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BMY32VB4WWOB7NAFS27O5WD2AKXDRAVCNFSM6AAAAABJXZICA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZSGE3TANZVGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
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).
The text was updated successfully, but these errors were encountered: