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

house mapgen occasionally overlaps table and wall #11632

Closed
sparr opened this issue Mar 17, 2015 · 9 comments · Fixed by #36075
Closed

house mapgen occasionally overlaps table and wall #11632

sparr opened this issue Mar 17, 2015 · 9 comments · Fixed by #36075
Labels
<Bug> This needs to be fixed Map / Mapgen Overmap, Mapgen, Map extras, Map display (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@sparr
Copy link
Member

sparr commented Mar 17, 2015

While working on #11631 I found one glitch that I couldn't fix. The generic "house" sometimes spawns a table and some other furniture in a room that's too small for them, so the table intersects a wall.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@narc0tiq
Copy link
Contributor

Would forcing the room to be bigger help?

@sparr
Copy link
Member Author

sparr commented Mar 17, 2015

maybe. the house generation code is the most twisted of any of the dynamic-room functions I've touched yet.

@OzoneH3 OzoneH3 added the <Bug> This needs to be fixed label Nov 17, 2015
@codemime codemime added the Map / Mapgen Overmap, Mapgen, Map extras, Map display label Jan 15, 2017
@Leland Leland added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label May 31, 2017
@Leland
Copy link
Contributor

Leland commented Aug 17, 2017

Have never seen this – can anyone confirm that this still happens?

@Leland
Copy link
Contributor

Leland commented Aug 30, 2017

Closing due to lack of reproducibility; feel free to either ping me here to reopen or just create a new issue.

@Leland Leland closed this as completed Aug 30, 2017
@FulcrumA
Copy link
Contributor

@Leland
Pinging as this thing happened just now to me:
table

  • OS: Win7
  • Game Version: 0.D-4931-g7eb2ac0 ((build 9218)
  • Graphics version: Tiles
  • Mods loaded: "dda", "ew_pack", "blazemod", "Tanks", "No_Fungi", "no_flaming_weapons", "StatsThroughSkills"

It does seem that the very narrow room may be the cause or at least a circumstance conductive to the issue here.

@sparr
Copy link
Member Author

sparr commented Jun 25, 2019

I would reopen and flag as confirmed but someone revoked my dev/collaborator status :(

@ifreund ifreund reopened this Jun 25, 2019
@ifreund ifreund added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Jun 25, 2019
@Night-Pryanik
Copy link
Contributor

As we have lots of variations of manually-drawn houses in json, I wonder whether we still need these procedurally-generated full-of-bugs houses at all.

@sparr
Copy link
Member Author

sparr commented Jun 26, 2019

I would not be happy removing the procedurally generated houses before having at least 500 unique static houses drawn.

@kevingranade
Copy link
Member

kevingranade commented Jun 27, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Map / Mapgen Overmap, Mapgen, Map extras, Map display (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants