satisfactory-factories / application

A tool to visualise groupings of factories to create a true Modular Factory setup for the game Satisfactory.
https://satisfactory-factories.app
GNU Affero General Public License v3.0
25 stars 5 forks source link

Various fluid recipes are incorrect #2

Closed Maelstromeous closed 3 days ago

Maelstromeous commented 1 week ago

Getting reports of various fluid recipes being incorrect. Need to do a full comparison of them in game and see the issues.

Maelstromeous commented 1 week ago

Provisional fix in https://github.com/Maelstromeous/satisfactory-factories/commit/63c083b2c91be73066bf3e700a90f464cd4254f5, needs fix confirm

Maelstromeous commented 1 week ago

Can confirm there's still bugs:

Screenshot 2024-11-17 at 15 34 15

Screenshot 2024-11-17 at 15 34 26

Requests for water is too high.

Maelstromeous commented 1 week ago

Game data is correct, it's a calculation issue.

Screenshot 2024-11-17 at 15 37 18

        {
            "id": "AluminaSolution",
            "displayName": "Alumina Solution",
            "ingredients": [
                {
                    "part": "OreBauxite",
                    "amount": 12,
                    "perMin": 120
                },
                {
                    "part": "Water",
                    "amount": 18,
                    "perMin": 180
                }
            ],
            "products": [
                {
                    "part": "AluminaSolution",
                    "amount": 12,
                    "perMin": 120,
                    "isByProduct": false
                },
                {
                    "part": "Silica",
                    "amount": 5,
                    "perMin": 50,
                    "isByProduct": true
                }
            ],
            "building": {
                "name": "oilrefinery",
                "power": "8.5"
            },
            "isAlternate": false,
            "isFicsmas": false
        },
Maelstromeous commented 1 week ago

Or not... apparently I can't read the fact that water is a byproduct and was used internally... that's a visualisation issue akin to #18

Maelstromeous commented 1 week ago

Wrote a test to confirm this, and can confirm they are heavily broken.

samsmithnz commented 3 days ago

I don't see where this is broken. The AL example above appears to be working - with the exception of the byproducts - which is well documented in other issues.

Maelstromeous commented 3 days ago

If you're happy with the recent parser fixes and the template tests and there's no "exotic" fluid stuff we need to handle then happy to close it off. I would like to see some resting around e.g. Excited matter.

Also gases?

samsmithnz commented 3 days ago

gases are simpler than they sound, they are liquids with no head-lift requirements.

Maelstromeous commented 3 days ago

Alright, I'll close this off for now then since our confidence has increased.