refactor layouter and layout optimization

[johannesdeselaers]

replaces #629 now includes

• optimization of layout via search for connected components
• refactoring of bricklayouter into brick- & plate- layouter and optimizer

[stuikomma]

For consistency use the term brick instead of 3L. Once in Layouter and in LegoPipeline.

• [x] done

[stuikomma]

sometimes I get this error when relayouting:

TypeError: this.plateLayouter.layout(...).then is not a function
    at LayoutOptimizer.splitBricksAndRelayoutLocally (http://localhost:3000/app.js:41778:55)
    at LayoutOptimizer.splitBricksAndRelayoutLocally (http://localhost:3000/app.js:41624:57)
    at Object.LegoPipeline.pipelineSteps.push.worker (http://localhost:3000/app.js:42259:40)
    at LegoPipeline.module.exports.LegoPipeline.runStep (http://localhost:3000/app.js:42322:19)
    at LegoPipeline.module.exports.LegoPipeline.runPromise (http://localhost:3000/app.js:42302:19)
    at LegoPipeline.runPromise (http://localhost:3000/app.js:42145:57)
    at http://localhost:3000/app.js:42309:24
    at lib$es6$promise$$internal$$tryCatch (http://localhost:3000/shared.js:6228:16)
    at lib$es6$promise$$internal$$invokeCallback (http://localhost:3000/shared.js:6240:17)
    at http://localhost:3000/shared.js:6785:13

[johannesdeselaers]

the error was caused by a missing Promise.resolve in the layout method of Layouter

• [x] fixed

[stuikomma]

And, of course, the numerous comments that don't start with a capital letter :wink: # [a-z] may help.

[stuikomma]

Some comments remain

• 4 in newBrickator.coffee
• 30+ in Brick.coffee
• 6 in DataHelper.coffee
• ~20 in Grid.coffee
• [x] done

[stuikomma]

:postbox: