GPSBabel / gpsbabel

GPSBabel: convert, manipulate, and transfer data from GPS programs or GPS receivers. Open Source and supported on MacOS, Windows, Linux, and more. Pointy clicky GUI or a command line version...
https://www.gpsbabel.org
GNU General Public License v2.0
473 stars 126 forks source link

Make it clear that fields is not an option for writing in unicsv. #1319

Closed robertlipe closed 2 weeks ago

robertlipe commented 1 month ago
codacy-production[bot] commented 1 month ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +0.01% :white_check_mark:
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (5a08554b7b57802ed6f7870df6bb0ddc43c97e8f) | 27532 | 17243 | 62.63% | | | Head commit (c55a154b585796b67d54efe9bdce09f92952f1a9) | 55073 (+27541) | 34495 (+17252) | 62.64% (**+0.01%**) | **Coverage variation** is the difference between the coverage for the head and common ancestor commits of the pull request branch: ` - `
Diff coverage details | | Coverable lines | Covered lines | Diff coverage | | ------------- | ------------- | ------------- | ------------- | | Pull request (#1319) | 0 | 0 | **∅ (not applicable)** | **Diff coverage** is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: `/ * 100%`

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more

tsteven4 commented 1 month ago

The ideas I mentioned in a comment illustrated here (implicit conversion of positions between degrees and radians): https://github.com/GPSBabel/gpsbabel/compare/master...tsteven4:gpsbabel:implicitdegrad

The second usage in smplroute is a bit rough, that should clean up when linepart returns a PositionDeg.

tsteven4 commented 1 month ago

demo enhanced with linepart returning PositionDeg.

robertlipe commented 2 weeks ago

These changes have all landed in other PRs. Closing and nuking this fork.