StrangeLoopGames / EcoIssues

131 stars 21 forks source link

[0.8.2.0 staging 1001] rounding / multiplier error #12540

Closed Sobaton closed 4 years ago

Sobaton commented 5 years ago

Step to reproduce: (mortaring skill ist level 8) At a Masonry Table, increase the number of items to be crafted to be 100. (normaly u need 1.2 Sandstone to grind 1 Sand) but the game round it to 121 not 120 grafik

if you increase the number of items to be crafted to 1000 it is correct grafik

SlayksWood commented 5 years ago

Were fixes rounding, but the recipes with wrong rounding still remained. I will add all encountered errors with rounding here.

Sobaton commented 5 years ago

i test now all Tables whit max skill and found some more rounding / multiplier errors: Masnory Table: Grind Granite To Sand Grind Sandstone

Backery Oven: Baked Roast (Raw Roast) Bread (Yeast and Flour) Camas Bread (Yeast)

Kilm: Glass Tallow Lamp (Clay)

Anvil: Iron Pipe

Sawmill: Saw Boards

Screw Press: Iron Plate

Lathe: Iron Axle Screws

Shaper: Gear

Electric Machinist Table: Servo (Fiberglass & Circuit)

Electronics Assembly: Circuit (Substrate) Electric Wall Lamp (Steel) Fiberglass (Glass) Robotic Assembly Line (Electric Motor)

Electric Planer: Steel Gear (Steel & Epoxy)

Robotic Assembly Line: Advanced Combustion Engine (Radiator) Framed Glass Door (Framed Glass)

Rolling Mill: Flat Steel (Steel & Epoxy)

Bloomery: ok Workbench: ok Mill: ok Cast Iron Stove: ok Stove: ok Machinist Table: ok Assemnbly Line: ok Cement Kiln: ok Electric Stamping Press: ok Electric Lathe: ok Oil Refinery: ok Pump Jack: ok Laboratory: ok

Mint/Bank/Currency Exchange/Registrar/Treasury/Computer Lab out of controll

mshocklee commented 5 years ago

V 0.8.2.4 Masonry Level 7 Grind Sandstone tooltip says it uses 1.08 Sandstone At Masonry Table: Queue up 025 Sand, uses 28 Sandstone instead of 27 Queue up 050 Sand, uses 55 Sandstone instead of 54 Queue up 075 Sand, APPROPRIATELY uses 81 Sandstone Queue up 100 Sand, uses 109 Sandstone instead of 108 Queue up 125 Sand, APPROPRIATELY uses 135 Sandstone Queue up 150 Sand, APPROPRIATELY uses 162 Sandstone Queue up 175 Sand, APPROPRIATELY uses 189 Sandstone Queue up 200 Sand, uses 217 Sandstone instead of 216

mshocklee commented 5 years ago

Is this the same rounding error as the currency?

Valares commented 4 years ago

This bug still appears in the Version 0.8.3.1. Not sure if it's fixed in the dev version, I'll probably check it on the weekend.

Also, be aware that

Queue up 075 Sand, APPROPRIATELY uses 81 Sandstone

is not true (at least not for Version 0.8.3.0 and 0.8.3.1), as it states the correct amount, but actually consumes 1 more. Bug Eco The same is true for smelting Sand at the Kiln into Glass, and I guess for all the other ones listed too.

SlayksWood commented 4 years ago

Need check after changing in craft

mirasrael commented 4 years ago

@SlayksWood can you check this and said if it is actual?

SlayksWood commented 4 years ago

Still an issue:

SlayksWood commented 4 years ago

Seems fixed