Skip to content

Common - Add setting for compass magnetic declination#9761

Merged
PabstMirror merged 10 commits intomasterfrom
magneticDeclation
Dec 15, 2024
Merged

Common - Add setting for compass magnetic declination#9761
PabstMirror merged 10 commits intomasterfrom
magneticDeclation

Conversation

@PabstMirror
Copy link
Contributor

@PabstMirror PabstMirror commented Feb 1, 2024

@PabstMirror PabstMirror added kind/feature Release Notes: **ADDED:** 2.16 labels Feb 1, 2024
@PabstMirror PabstMirror added this to the Ongoing milestone Feb 1, 2024
@johnb432 johnb432 added 2.18 and removed 2.16 labels Feb 10, 2024
@PabstMirror PabstMirror changed the title Common - Add setting for compass magnetic declination [2.18] Common - Add setting for compass magnetic declination Mar 5, 2024
@PabstMirror PabstMirror marked this pull request as ready for review March 5, 2024 19:57
@jonpas jonpas changed the title [2.18] Common - Add setting for compass magnetic declination Common - Add setting for compass magnetic declination Mar 28, 2024
@johnb432 johnb432 removed the 2.18 label Oct 9, 2024
@johnb432 johnb432 modified the milestones: Ongoing, 3.18.2 Oct 9, 2024
Copy link
Contributor

@johnb432 johnb432 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity, which year's dataset did you use? I saw 2 PRs trying to update the values to 2020 (cmweiss/geomag#15), would be neat to use those if you don't already use them.

Co-authored-by: johnb432 <58661205+johnb432@users.noreply.github.com>
@PabstMirror
Copy link
Contributor Author

IIRC I used that python lib with an updated WMM.COF
I can try to rerun it, and we should see slight changes just going from feb->october

@PabstMirror PabstMirror modified the milestones: 3.18.2, Ongoing Oct 14, 2024
@johnb432 johnb432 modified the milestones: Ongoing, 3.18.2 Dec 15, 2024
@johnb432
Copy link
Contributor

Imo push this out for 3.18.2 and we can update the data later if deemed necessary.

@PabstMirror PabstMirror merged commit 472f919 into master Dec 15, 2024
3 checks passed
@PabstMirror PabstMirror deleted the magneticDeclation branch December 15, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature Release Notes: **ADDED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants