Viewing Issue Simple Details Jump to Notes ] Wiki ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0008699 [DCSS] Patches feature N/A 2014-06-18 01:55 2014-07-21 14:24
Reporter infiniplex View Status public  
Assigned To gammafunk
Priority normal Resolution done  
Status resolved   Product Branch 0.15 ancient branch
Summary 0008699: Improve LUA zonify functionality
Description This patch gives the zonify functions the ability to save multiple zones. The parameter was always there, but it used to do nothing. I also added an option for a minimum size zone (to avoid finding closets). You can now request e.g. keep up to 3 zones of at least 10 cells each.

This is not used anywhere yet but I have a layout in mind.
Additional Information
Tags No tags attached.
Attached Files ? file icon 0001-Improved-LUA-zonify-functionality.patch [^] (6,349 bytes) 2014-06-18 01:55 [Show Content]
png file icon layout_diamond_mine.png [^] (20,100 bytes) 2014-06-23 23:27


? file icon layout_diamond_mine.des [^] (7,721 bytes) 2014-06-23 23:28 [Show Content]

- Relationships

-  Notes
(0026586)
Reaver (developer)
2014-06-23 02:09

Hmm, adding another argument to several functions which already have 2-4 arguments isn't a great thing. If the layout which needs this gets implemented this is definitely worth pushing but I think this should wait until that happens.
(0026626)
infiniplex (reporter)
2014-06-23 23:27
edited on: 2014-06-23 23:29

And here is the layout that uses the patch: layout_diamond_mine. It is intended for the Orc Mines and probably should be added to an existing .des file.

The map is based on a number of interconnected diamond shapes. The diamonds are all the same size and placed on a grid, but most are missing, so the layout looks very irregular.

Convergence:
  -> deeper maps are based on larger diamonds
  -> the branch bottom has more open area (as in layout_caves)

(0026627)
infiniplex (reporter)
2014-06-23 23:36

I don't know if the Orc branch end always placing after the layout is intended. The end vaults that place others serial-vault-style may require it. I am currently having trouble with the wizmode "P" command (create a level based on a vault), so I can't easily test what happens the other way.
(0026805)
gammafunk (administrator)
2014-07-21 14:24

Pushed with only some minor cleanups to comments and a duplicate tag removal on the layout in 0.15-a0-2190-g04414b5, thanks!

- Issue History
Date Modified Username Field Change
2014-06-18 01:55 infiniplex New Issue
2014-06-18 01:55 infiniplex File Added: 0001-Improved-LUA-zonify-functionality.patch
2014-06-23 02:09 Reaver Note Added: 0026586
2014-06-23 23:27 infiniplex Note Added: 0026626
2014-06-23 23:27 infiniplex File Added: layout_diamond_mine.png
2014-06-23 23:28 infiniplex File Added: layout_diamond_mine.des
2014-06-23 23:29 infiniplex Note Edited: 0026626
2014-06-23 23:36 infiniplex Note Added: 0026627
2014-07-21 14:24 gammafunk Note Added: 0026805
2014-07-21 14:24 gammafunk Status new => resolved
2014-07-21 14:24 gammafunk Fixed in Branch => 0.15 development branch
2014-07-21 14:24 gammafunk Resolution open => done
2014-07-21 14:24 gammafunk Assigned To => gammafunk


Mantis 1.1.8[^]
Copyright © 2000 - 2009 Mantis Group
Powered by Mantis Bugtracker