Skip to content

Conversation

@chdoc
Copy link
Member

@chdoc chdoc commented Jan 16, 2025

No description provided.

@chdoc
Copy link
Member Author

chdoc commented Jan 16, 2025

pre-commit.ci autofix

## New Tools
- `fix/stuck-squad`: allow squads and messengers returning from missions to rescue squads that have gotten stuck on the world map
- `gui/rename`: (reinstated) give new in-game language-based names to anything that can be named (e.g. units, governments, fortresses, or the world)
- `autocheese`: automatically make cheese using barrels that have accumulated sufficient milk
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs to be moved up to the current version

autocheese.lua Outdated
for _,workshop in ipairs(df.global.world.buildings.other.WORKSHOP_FARMER) do
if
not workshop.profile.blocked_labors[df.unit_labor.MAKE_CHEESE] and
#workshop.jobs == 0 and #workshop.profile.permitted_workers == 0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if the permitted worker is free to take the cheese making job?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is a good point. While having a dedicated cheese maker sounds strange to me, the tool should probably not fail if somebody does this, or otherwise assigns dedicated workers to all farmer's workshops. However, unless the dedicated cheese maker is kept on standby, there will be cycles missed; the approach taken inherently requires the unit to be assigned at the same moment as the job is created. But then, I suppose there is no harm in missing a barrel at 50 units and only creating cheese once you have 60 or 100.

chdoc and others added 3 commits January 21, 2025 19:43
Co-authored-by: Myk <myk.taylor@gmail.com>
- check that barrel can be brought to workshop
- handle workshops with assigned masters
@chdoc
Copy link
Member Author

chdoc commented Jan 21, 2025

Didn't have a whole lot of time to actually play/test tonight, so only did some minimal testing before pushing.

@chdoc
Copy link
Member Author

chdoc commented Jan 25, 2025

I'm not sure the merge was necessary, but I merged locally anyway. I also quickly checked that it still works with v51.02

@myk002 myk002 merged commit ce8efb4 into DFHack:master Jan 25, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants