Add Layouts and renderFacade task - #200
Conversation
[Maven Build Status]📑 Commit: 📦 Download artifact: Generator.jar |
0614713 to
645c300
Compare
346456e to
d7ca222
Compare
renderFacade task
|
@pyrollo I focused mainly on removing dead code, resolving or completing TODOs, and improving the algorithm used by concatenate (on Also I have added multiple unit tests as I have discovered some bugs on For the algorithm, in a nutshell, I have used I reused your approach and added a second phase for starved builders. This fixes 2 issues where space where not distributed evenly. This work is now ready for a PR, and the documentation was done. It is no longer a work in progress. It is a bit unusual to open a PR at the end, hope you'll be able to take it from here (Although not a good time). |
Co-authored-by: Pierre-Yves Rollo <dev@pyrollo.com>
Co-authored-by: Pierre-Yves Rollo <dev@pyrollo.com>
d7ca222 to
ba339a2
Compare
Changes
This PR adds layouts and a task to render facades.
Feature description
Layouts are a kind of structure that can resize itself to fit a requested space. Contrary to Structure, its size aren't fixed, it can stretch, be repeated to fill the available space.
With that we can define layouts and have building facades rendered dynamically.
The type of Layouts are:
Showcase
Reason
This adds more expressiveness on building rendering. Moreover with models values, we can have layout that are rendered depending on metadata, randomly..
TODOs
Left one TODO on
full.yaml. Layout parameters can be very long, it could a good idea to define it elsewhere. (As it uses voxels defined on format, could becommon.yamlfile. But that is a detail.Self-checks
/docsfolder has been updatedexamples/work the same (or have been adapted if subject to changes in this PR)