Added the Boberg extension for the T89c Magnetic Field Model and documentation - #78
Open
lkncl wants to merge 1 commit into
Open
Added the Boberg extension for the T89c Magnetic Field Model and documentation#78lkncl wants to merge 1 commit into
lkncl wants to merge 1 commit into
Conversation
Contributor
|
Thanks Lucas. It looks OK to me. To celebrate the rare occurence of the addition of a new magnetic field model, I suggest incrementing the version number to 5.1.0 . |
AntoineBrunet
self-requested a review
August 7, 2026 12:09
AntoineBrunet
approved these changes
Aug 7, 2026
| c*************************************************************************************************** | ||
| c | ||
| c Boberg extension refitted for T89c | ||
| c Authors : L. Nicolle, A. Brunet |
Contributor
There was a problem hiding this comment.
We should add @NLarsen15, as he did the fits for the coeffcients.
| c | ||
| c | ||
| if (kext .eq. 15) then | ||
| c |
Contributor
There was a problem hiding this comment.
Maybe add a comment mentionning this is for T89+Boberg
| - Based on T89a P. Boberg extension (https://doi.org/10.1029/95GL00948) | ||
| - Refitted on T89c by N. Larsen | ||
| - Behavior : | ||
| - if Kp_ < 57, same as model 4 (T89c) |
Contributor
There was a problem hiding this comment.
Maybe clearer if we say: Kp <= 5+ and Kp >= 6-
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow #11
Description
This PR adds a new external magnetic field model (
kext=15) implementing the Boberg extension of the Tsyganenko 89c model, allowing the ring current component of T89c to be extended using Dst.This work follows the discussion in PR #11. Instead of modifying the existing T89c implementation (
kext=4), a separate model is introduced in order to preserve the original T89c behaviour.This implementation follows the approach described by Boberg et al. https://doi.org/10.1029/95GL00948, which extends the T89 ring current coefficient C5 using a linear Dst dependence.$$Kp < 57$$ , but it uses Dst to drive the ring current above.
The model behave like T89C for
Since the original relation was derived for T89a, a new fit was performed for the T89c C5 coefficient (N. Larsen, unpublished), giving the following relationship :
For practical considerations, if the provided Dst is either the OMNI* or IRBEM
BADDATAvalue (respectively99999or-1e31), Dst is computed from Kp using an empirical relationship between the mean Dst value and Kp class derived from the OMNI dataset.OMNI* https://omniweb.gsfc.nasa.gov/html/ow_data.html
Implementation details
general_information.rstkextTests
Tests were performed to ensure consistency when switching between T89C and T89BOBERG models and to validate the Dst handling:
As expected, there is no difference when
Kp<57.When
Kp>=57, changinKpdoes not make any difference anymore when Dst is given but it does when Dst is set toBADDATA, as expected.