Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

buildingplanner suspending stairs #4609

Closed
Myos-tis opened this issue May 18, 2024 · 3 comments · Fixed by #4611
Closed

buildingplanner suspending stairs #4609

Myos-tis opened this issue May 18, 2024 · 3 comments · Fixed by #4611

Comments

@Myos-tis
Copy link

image

DFHack version 50.13-r2 (git: 7efaca9) on x86_64

Stairs, like the one to the right of the screenshot, seem to get unreasonably suspended despite being walkable.

@chdoc
Copy link
Contributor

chdoc commented May 18, 2024

@Myos-tis If you click on the stairs, what reason does suspendmanager give for keeping the stairs suspended? Also, could you provide screenshots of the z-levels above and below the stair that is being wrongly suspended?

@chdoc
Copy link
Contributor

chdoc commented May 18, 2024

I have localized at least one issue with suspending dead ends:

               if (isImpassable(building)) {
                    // suspend the current construction job to leave the entire plan suspended
                    suspensions[job->id] = Reason::DEADEND;

and

           // ...and continue the exploration from its position
           building = exit;

Basically, the job being suspended is not the construction job for the building that is checked for being impassible.

I have to run now, but I should be able to propose a fix tonight. I also want to do some minor refactoring while I am at it.

@myk002 can you assign me to this?

@Myos-tis
Copy link
Author

It's suspended due to blocking another build job.
image

The area pictured is 11x16x5, from (74,47) in the top left to (84,62) in the bottom right

z=54 (64, embark lowest is -10)
image

z=53 (63)
image
Despite being symmetrical, the stairs on the west pillar are not suspended

z=52 (62)
image
Again, the stairs on the west pillar aren't suspended, and in the middle, the stairs to the left of the east pillar aren't suspended either
image
Removing the middle stairs or allowing the walls below to be built doesn't change anything

z=51 (61)
image

z=50 (60)
image

All of the planned walls are bronze, all of the planned stairs are obsidian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants