[BUG] Precusor Satellites gives +2 Sensor Power instead of +2 Range....

Precusor Satellites gives +2 Sensor Power instead of +2 Range like the description states.

      <Trigger>
        <OnEvent>OnEventChoice</OnEvent>
        <Target>
          <TargetType>Faction</TargetType>
        </Target>
        <Lifetime>Target</Lifetime>
        <Modifier>
          <EffectType>SensorPower</EffectType>
          <Scope>Global</Scope>
          <Target>
            <TargetType>Ship</TargetType>
          </Target>
          <BonusType>Flat</BonusType>
          <Value>2</Value> (This should be +12)
        </Modifier>
      </Trigger>

      <Trigger>
        <OnEvent>OnEventChoice</OnEvent>
        <Target>
          <TargetType>Faction</TargetType>
        </Target>
        <Lifetime>Target</Lifetime>
        <Modifier>
          <EffectType>SensorPower</EffectType>
          <Scope>Global</Scope>
          <Target>
            <TargetType>Colony</TargetType>
          </Target>
          <BonusType>Flat</BonusType>
          <Value>2</Value> (This should be +12)
        </Modifier>
      </Trigger>

70,525 views 12 replies
Reply #1 Top

Pretty sure I've reported this before.

Yep, right here:  https://forums.galciv3.com/477017/page/1/#3632698

Reply #2 Top

This has been fixed locally and should be in the next patch. 

Thanks for the report. 

Reply #3 Top

:thumbsup:

Reply #4 Top

TY Mormegil and Horemvore.

Reply #5 Top

Nope, this bug is still there in 1.8.1.8.

StarDock Devs: Check your merge status of the fix - remember, this fix is in a DLC, not the main game branch.

 

Also, for those who haven't figured out the formula for mapping Sensor Power to radius:

 

SP = (X+1)(X)/2

 

That is, the minimum SP necessary to have a radius X is the cumulative sum of X  ( 1 + 2 + 3 + ....  + (X-1) + X)

Solving the other way (i.e. given SP, find X):

 

X = (-1 + √(1+8*SP))/2

 

Changing the value from +2 to +12 seems reasonable, as it has the following effect:

  • For ships with a starting Radius of 3 (the minimum), it gives a +3 Radius bonus
  • For those with a starting Radius of 3 or 4, it gives a +2 Radius bonus.
  • For those with a starting Radius of 5 to 8, it gives either a +1 or +2 Radius bonus, depending on the exact SensorPower before the bonus
  • For those with a starting Radius of 9 to 11, it gives a +1 Radius bonus.
  • For those with a starting Radius of 12 or more, it gives either NO or +1 Radius bonus, depending on the exact SensorPower before the bonus

 

Also, we should change the wording of the Event itself:

$ diff -c data/English/Text/DLC4_ColonizeEventText.xml data/English/Text/DLC4_ColonizeEventText.xml.fixed
*** data/English/Text/DLC4_ColonizeEventText.xml        2016-07-29 09:28:34.795471900 -0700
--- data/English/Text/DLC4_ColonizeEventText.xml.fixed  2016-08-07 00:19:38.000812000 -0700
***************
*** 260,266 ****

    <StringTable>
      <Label>PrecursorSatellitesEvent_Benevolent_Bonus_Dec</Label>
!     <String>+10 Benevolent, +2 Sensor range to all ships, and colonies.</String>
    </StringTable>

    <StringTable>
--- 260,266 ----

    <StringTable>
      <Label>PrecursorSatellitesEvent_Benevolent_Bonus_Dec</Label>
!     <String>+10 Benevolent, +12 Sensor Power to all ships and colonies.</String>
    </StringTable>

    <StringTable>

Reply #6 Top

Quoting trims2u, reply 5

Nope, this bug is still there in 1.8.1.8.

Well yeah, its why i reported it, and Morg has fixed it locally so it will be in the NEXT patch.

Reply #7 Top

Next time, put the version number of what you're looking at in the Bug, either the Subject or the very first thing in the text.

You posted on 7/22, which was several days before the 1.8.1 patch was publicly available. So it was hard to tell what version you were reporting against, as well as what Morg's reply referred to - which, once again, was before the public release of 1.8.1, so it was really unclear if it was or wasn't applicable to the soon-to-be-released update.

 

Reply #8 Top

This is still an issue in v1.82.

+1 Loading…
Reply #9 Top

I was sure this was fixed, but i will double check. 

Reply #10 Top

OK, I see the problem, we had a slimier issue with the Precursor Satellite planet trait, that was fixed, I am fixing the event now and we will get it into the next patch. Sorry about that. 

+2 Loading…
Reply #11 Top

Thank you. 

Reply #12 Top

Don't forget to fix the Text, as well as the attribute value.  :-)