hohlraum / gdsCAD

A simple but powerful Python package for creating photolithography masks in the GDSII format.
GNU General Public License v3.0
88 stars 52 forks source link

Switch from layerdat to laydat #27

Closed tom-varga closed 9 years ago

hohlraum commented 9 years ago

Hi Tom,

looks good, but two comments:

1) Should objects_by_laydat include a recursion option to allow selection of referenced cell contents? Either way, docstring should be explicit that it does or does not recurse through subcells.

2) Isn't elements_by_laydat redundant since you can just do Elements(cell.objects_by_laydat(laydat))

hohlraum commented 9 years ago

Removed elements_by_laydat as redundant. But I'm willing to be convinced otherwise.

Merged in 8687bec7d.