[0.12.26][Twinsen] Rocket silo % not updated if productivity

This subforum contains all the issues which we already resolved.
Post Reply
Aardwolf
Long Handed Inserter
Long Handed Inserter
Posts: 95
Joined: Tue Feb 03, 2015 2:05 pm
Contact:

[0.12.26][Twinsen] Rocket silo % not updated if productivity

Post by Aardwolf »

I noticed a tiny cosmetic bug:

If the rocket silo has productivity modules, and the productivity bar is full, then one would expect the percentage to increase. But it doesn't. But when then the normal green bar is full, the percentage goes +2, so the productivity bonus at least got applied. I think it just doesn't graphically update the percentage.

Twinsen
Factorio Staff
Factorio Staff
Posts: 1330
Joined: Tue Sep 23, 2014 7:10 am
Contact:

Re: [0.12.26][Twinsen] Rocket silo % not updated if productivity

Post by Twinsen »

Well, this was interesting.
The problem is not what you described, but it was just that the GUI was skipping 53% (it does not matter if you use productivity or not).

Fixed in 0.12.27.

Aardwolf
Long Handed Inserter
Long Handed Inserter
Posts: 95
Joined: Tue Feb 03, 2015 2:05 pm
Contact:

Re: [0.12.26][Twinsen] Rocket silo % not updated if productivity

Post by Aardwolf »

Ha ha, it was indeed at 53%, awesome :D

Rwn
Inserter
Inserter
Posts: 32
Joined: Sun Mar 03, 2013 10:43 am
Contact:

Re: [0.12.26][Twinsen] Rocket silo % not updated if productivity

Post by Rwn »

Out of sheer curiosity, what was so special about the number 53 that caused it to be skipped ? :?

Twinsen
Factorio Staff
Factorio Staff
Posts: 1330
Joined: Tue Sep 23, 2014 7:10 am
Contact:

Re: [0.12.26][Twinsen] Rocket silo % not updated if productivity

Post by Twinsen »

Rwn wrote:Out of sheer curiosity, what was so special about the number 53 that caused it to be skipped ? :?

Code: Select all

 -     this->launchButton.setText(ssprintf("%u%%", uint32_t(rocketSilo->getRocketFinishedFraction() * 100)));
 +     this->launchButton.setText(ssprintf("%u%%", uint32_t(rocketSilo->getRocketFinishedFraction() * 100.0f + 0.5f)));
This was the line before and after the fix. It was a rounding error. Not exactly sure why 53 was the only one that was rounded badly, but it falls in the category of floating point magic.

Post Reply

Return to “Resolved Problems and Bugs”