seafloor-geodesy / gnatss

Community Seafloor Global Navigation Satellite Systems - Acoustic (GNSS-A) Transponder Surveying Software
https://gnatss.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
9 stars 12 forks source link

fix: Perform style updates and fixes #284

Closed lsetiawan closed 1 month ago

lsetiawan commented 1 month ago

This PR perfoms a huge style config updates with ruff, adding a check for scientific python repository review to ensure codebase quality. There are some rules that we are currently skipping however, such as mypy and a few others, but overall, we are following best practices put forth. In this PR lots of the changes from rule violations has been implemented.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 74.77876% with 57 lines in your changes missing coverage. Please review.

Project coverage is 76.19%. Comparing base (2f00e30) to head (84ca8bb). Report is 35 commits behind head on main.

Files Patch % Lines
src/gnatss/loaders.py 63.33% 11 Missing :warning:
src/gnatss/ops/io.py 54.54% 10 Missing :warning:
src/gnatss/ops/validate.py 46.15% 7 Missing :warning:
src/gnatss/ops/harmonic_mean.py 45.45% 6 Missing :warning:
src/gnatss/ops/kalman.py 20.00% 4 Missing :warning:
src/gnatss/solver/utilities.py 71.42% 4 Missing :warning:
src/gnatss/utilities/io.py 72.72% 3 Missing :warning:
src/gnatss/cli.py 66.66% 2 Missing :warning:
src/gnatss/dataspec/__init__.py 60.00% 2 Missing :warning:
src/gnatss/ops/data.py 75.00% 2 Missing :warning:
... and 3 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #284 +/- ## =========================================== + Coverage 49.67% 76.19% +26.51% =========================================== Files 16 29 +13 Lines 930 1785 +855 =========================================== + Hits 462 1360 +898 + Misses 468 425 -43 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.