-
Notifications
You must be signed in to change notification settings - Fork 17
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
Lidar azimuth fix #206
Lidar azimuth fix #206
Conversation
This reverts commit e377116.
… conform to CF/Radial 2
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #206 +/- ##
==========================================
- Coverage 91.97% 91.95% -0.03%
==========================================
Files 23 23
Lines 4538 4562 +24
==========================================
+ Hits 4174 4195 +21
- Misses 364 367 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Thanks @rcjackson! |
I've fixed the naming of the "time" dimension to be "azimuth" to be consistent with the CF/Radial 2 standard for the Halo Photonics reader. In addition, a few fixes were made to the sweep variables.
history.md