Great tool, really happy.
Once comment: can the value reporting be wrong? I have visited a system with Water World, that has already been discovered, but not mapped. Your plugin reported maximum value of 567k, while EDDiscovery reported over 1.379M... have a look...
Are you able to provide the line from the journal showing the Scan event for this body? Or, failing that, can you get a screenshot of the system map with this body highlighted please?
@Eahlstan my variable naming probably needs a lot to be desired but I don't believe it's the issue here (I reserve the right to edit this line later though

)
I calculate the value as:
Code:
GetPlanetValue(k + tk, Mass, IsFirstDiscoverer, IsMapped, IsFirstMapper, IsMappedEfficiently, IsOdyssey, false)
We don't need to dig into each value - we can tell all of them except IsFirstMapper are correct based on the current and potential max values shown in screenshot. For some reason IsFirstMapper gets sent through as false.
But IsFirstMapper is simple:
Code:
public bool IsFirstMapper { get { return !WasMapped && IsMapped; } }
We already know IsMapped is true. WasMapped is passed through from the Observatory Scan object. For the above to return false while IsMapped is true, WasMapped would have to be true.
The only other place it could be wrong is in GetPlanetValue, but that looks right to me:
Code:
if (isMapped)
{
if (isFirstDiscoverer && isFirstMapped)
{
mappingMultiplier = MAPPING_MULTIPLIER_FD_FM;
}
else if (isFirstMapped)
{
mappingMultiplier = MAPPING_MULTIPLIER_FM;
}
else
{
mappingMultiplier = MAPPING_MULTIPLIER;
}
}
mappingMultiplier ends up as MAPPING_MULTIPLIER not MAPPING_MULTIPLIER_FM meaning isFirstMapped (or IsFirstMapper as it's sent to the func) is false.
So my feeling is that body is already mapped - but FSS doesn't always show mapper and perhaps that's what happened here (hence asking for Scan or SysMap screenshot)