Cerys


Fulgora's moon of puzzles. Ancient wrecks embedded in thick ice can seemingly be repaired, including a colossal nuclear reactor that could transform the possibilities on the surface... if you can get it working. Cerys is a complete planet mod that does not modify the vanilla game, so is easy to include in existing saves.

Content
a day ago
2.0
16.6K
Factorio: Space Age Icon Space Age Mod
Planets Logistics Environment Mining Fluids Manufacturing Power

b Fill4Me not working with this mod

a month ago
(updated a month ago)

line in factorio-current.log:

Script @Fill4Me/stdlib/event/event.lua:154: core/lualib/production-score.lua:50: table index is nil

I don't know why Fill4Me calls production-score.lua (I'm not associated with this mod).

production-score.lua:50 (Factorio core) is within the body of function get_product_list, which appears to loop over all recipes in Factorio, and product.name being nil results in the logged error message.

I've hacked some logging into production-score.lua and observed this:

The loop works fine until it hits "cerys-repair-cryogenic-plant", which is the last one that works, processing the next raises the error.

This next "thing" in the loop is "cerys-discover-fulgoran-cryogenics" (at least, for me - only Cerys and Fill4Me activated with their dependencies, no other mods).

For which (cerys-discover-fulgoran-cryogenics), recipe_prototype.products seems to be:
{
{
amount = 1,
research_item = "fulgoran-cryogenics-progress",
type = "research-progress"
}
}

This structure doesn't contain a "name" attribute,
hence product.name is nil,
hence product_list[product.name] = {} is accessing an invalid index,
hence production-score.lua returns an error to Fill4Me. Which apparently breaks the logic further down the road in this mod (I haven't checked that, but probably some tables Fill4Me needs are not populated).

I have no clue about the workings of Factorio's products and recipes (and modding Factorio in general), but it seems either the product of cerys-discover-fulgoran-cryogenics should contain an attribute "name", or cerys-discover-fulgoran-cryogenics shouldn't have a product attribute at all - I don't know, at least from the name of it, it doesn't sound like it's something having a product...

Maybe this makes sense and it helps tracking down something. Would love that Fill4Me works again. Thank you for your effort!

a month ago

This crash is inside core/lualib, so is a base game bug and should be reported to Wube on the forums.

a month ago
(updated a month ago)

Not if the Cerys mod does something against the intended mechanics in Factorio. Not saying THAT it is, just IF it is. I know way too little to know which one is true. I find it hard to believe that it is a bug in the core libs with a gazillion mods working, but again, I don't know. I don't know about the significance of the attribute "product.name" and how it is intended to be populated. And I also don't know if it's ok for mods (Fill4Me) to call get_product_list...

a month ago
(updated a month ago)

Yes I should give you more context. This is not the first time that the existence of research-progress products, a new feature in 2.0, have crashed the core library. To my knowledge Cerys is the only mod which makes use of this feature which is intended by Wube to be supported. But on general principle — if the core library crashes, it’s a core library issue.

a month ago
(updated a month ago)

Ah it's new, ok I understand, sorry I didn't know that. Thank you for your explanation! I googled around and found e.g. this https://forums.factorio.com/viewtopic.php?t=125973 so it might very well be that it's a bug in core. Then I suppose you're using research-progress according to the specs, and indeed the core lib maybe doesn't handle some special case correctly... I'll think about posting a bug on the Factorio forums as you suggested.

Edit: LOL this link I pasted above is a bug YOU reported, I just saw the name of the reporter? Ok I guess we've gone full circle, the bug seems assigned and I'll think about adding the Fill4Me details, not 100% sure it will help or clutter the report.

Thanks!

a month ago
(updated a month ago)

I see, I should not have assumed my earlier report was fixed! Thanks for chasing this up, if it’s causing trouble you might want to say as such on the forum thread.

a month ago

from https://forums.factorio.com/viewtopic.php?p=666368#p666368:
"This should be fixed for the next release" => let's see how it behaves after next Factorio version is released. :-)

New response