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
10 stars 14 forks source link

(build): Need a config to ignore test data #221

Closed madhavmk closed 8 months ago

madhavmk commented 8 months ago
codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (73cbcac) 49.67% compared to head (470deee) 49.67%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #221 +/- ## ======================================= Coverage 49.67% 49.67% ======================================= Files 16 16 Lines 930 930 ======================================= Hits 462 462 Misses 468 468 ```

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

madhavmk commented 8 months ago

The point of concern is the large size of the ./tests directory, which causes the gnatss source distribution tar file to be unnecessarily large. In the future we are likely to increase the ./tests directory size significantly by adding additional data files. `

[SSEC/offshore-geodesy]$ du -sh * 4.0K LICENSE 0 MANIFEST.in 12K README.md 844K docs 4.0K noxfile.py 4.0K pyproject.toml 140K src 15M tests

[SSEC/offshore-geodesy]$ python -m build ... [SSEC/offshore-geodesy]$ du -sh dist/* 44K dist/gnatss-0.1.dev106-py3-none-any.whl 4.7M dist/gnatss-0.1.dev106.tar.gz `

After excluding the ./tests directory from the build step, the source distribution file reduced in size significantly. This speeds up gnatss pip installation speed due to faster downloading, and lesser unzipping. `

[SSEC/offshore-geodesy]$ du -sh 4.0K LICENSE 0 MANIFEST.in 12K README.md 844K docs 4.0K noxfile.py 4.0K pyproject.toml 140K src 15M tests [SSEC/offshore-geodesy]$ python -m build ... [SSEC/offshore-geodesy]$ du -sh dist/ 44K dist/gnatss-0.1.dev107-py3-none-any.whl 788K dist/gnatss-0.1.dev107.tar.gz `