tvwenger / du-factory-generator

https://tvwenger.github.io/du-factory-generator/
MIT License
14 stars 24 forks source link

invalid required factory count #50

Closed kubanka-peter closed 3 years ago

kubanka-peter commented 3 years ago

First: it's awesome, but it has some calculation error and you have to fix it

I tested it with "Glass Panel M" 2 assemblers and maintain 1500,

Another test, what i dont built justs I used https://hq.hyperion-corporation.de to see the values: Basic Atmospheric Engine L - 2 assemblers and maintain 10,

lgfrbcsgo commented 3 years ago

First: it's awesome, but it has some calculation error and you have to fix it

I tested it with "Glass Panel M" 2 assemblers and maintain 1500,

  • your calculator says that I need 2x 3d printer (Basic casing xs) => 2/min
  • in reality I need 6x 3d printer to produce the required Basic Casing xs => 8/min

We need to investigate that. That seems to be a bug.

Another test, what i dont built justs I used https://hq.hyperion-corporation.de to see the values: Basic Atmospheric Engine L - 2 assemblers and maintain 10,

  • your calculator says that I need 3 steel smelter, but hyperion calculator says that 1 is enough.

That is by design. All industries should be able to run at 100% utilization 24/7. That allows us to easily add to an existing factory. We have some ideas on how to minimize the number of industries, but that's not a priority right now.

tvwenger commented 3 years ago

@kubanka-peter thanks for raising your concerns! Following up on what @lgfrbcsgo said:

First: it's awesome, but it has some calculation error and you have to fix it

I tested it with "Glass Panel M" 2 assemblers and maintain 1500,

* your calculator says that I need 2x 3d printer (Basic casing xs) => 2/min

* in reality I need 6x 3d printer to produce the required Basic Casing xs => 8/min

Thanks for finding this bug! This is a typo in the recipe for "Glass Panel M" (says it requires only 1 Basic Casing XS instead of 4).

Another test, what i dont built justs I used https://hq.hyperion-corporation.de to see the values: Basic Atmospheric Engine L - 2 assemblers and maintain 10,

* your calculator says that I need 3 steel smelter, but hyperion calculator says that 1 is enough.

As @lgfrbcsgo said, this is by design in our factory. We want all industries to be able to run at 100% at all times. That one Steel smelter is enough to feed the Reinforced Frame, Combustion Chamber, and Basic Screw assemblers for the Basic Atmo Engine, but that's it. If you try to add any other outputs or expand the factory later, then there is going to be a bottleneck in your factory. For example:

d6rks1lv3rz3r0 commented 3 years ago

I would just point out here for the future, and maybe in the readme that this is NOT a crafting calculator. It doesn't give you the laziest way to quickly achieve your end goal. It is a factory generator, so it will give you THE optimum way to create a factory that will produce things exactly to your specs.

For example, in the extreme case of a warp beacon, if you wanted to make 1 warp beacon as quick as possible, you would want to have maybe 64 electronics industries making Exotic Quantum Alignment Units. But to make 1 Warp Beacon every 36 days, and make exactly that (no over capacity, no wasted potential, no induced bottlenecks), you need only 8 electronics industries. However you will wait longer for the very first warp beacon, because the initial priming of the factory will take longer.

I am saying this because I have been using this approach since the beginning of industries (as opposed to the quickest way, or the intractably inefficient Factorio based approaches (over feeding lines) with thousands of transfer units). However I kept getting hounded to the point of abuse about why Dual.SH or Hyperion would say different things ("yeah you are the only smart person here" .. I guess I was lol where were you two @tvwenger @lgfrbcsgo ahahahaha).