search

DFHack / dfhack

Memory hacking library for Dwarf Fortress and a set of tools that use it
Other
1.88k stars 478 forks source link

"seedwatch all" doesn't watch all seed types #1941

Closed ab9rf closed 2 years ago

ab9rf commented 3 years ago

seedwatch all only covers the seed types that are in seedwatch's hardcoded abbreviation list (whether or not they're in the game). Arguably, this command should match all plants in the game that have seeds, whether or not seedwatch has an abbreviation for them.

In addition, the automatic enabling/disabling of cooking plants based on seed supply only works for plants on the hardcoded abbreviation list.

Krougal commented 3 years ago

These are the "legacy" plants is why. It was never updated to include the new ones. +1

myk002 commented 2 years ago

@ab9rf this seems very related to the autofarm code. Could I possibly assign this one to you?

ab9rf commented 2 years ago

I'd be likely to rewrite it entirely if assigned to me, but that might not be a bad idea anyway. I'll grab it for now.

myk002 commented 2 years ago

Thanks!