Skip to content

Move SRTM to new plugin #71

Merged
agrenott merged 5 commits intomasterfrom
srtm_plugin
Dec 9, 2025
Merged

Move SRTM to new plugin #71
agrenott merged 5 commits intomasterfrom
srtm_plugin

Conversation

@agrenott
Copy link
Copy Markdown
Owner

@agrenott agrenott commented Dec 1, 2025

No description provided.

@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Dec 1, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for b8413371 97.68%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (b841337) Report Missing Report Missing Report Missing
Head commit (b97a51e) 1862 1730 92.91%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#71) 259 253 97.68%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

Massive clean-up of legacy code in NASASRTMUtil.py.
Removed support of SRTM v2.1 and associated CLI parameters.
Centralize default values in Configuration structure.
Remove some legacy options.
Rename some attributes.
Not maintained anyway.
Add unit tests for make_osm_filename.
Some code clean-up.
Some tiles are actually missing from SRTM dataset (still part of the index).
SRTM API returns a JSON error for those; ignore it as a non-existing tile.

Close #56
@agrenott agrenott merged commit 2e79933 into master Dec 9, 2025
22 of 23 checks passed
@agrenott agrenott deleted the srtm_plugin branch December 12, 2025 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants