{"action":"created","issue":{"url":"https://api.github.com/repos/dcowden/cadquery/issues/81","labels_url":"https://api.github.com/repos/dcowden/cadquery/issues/81/labels{/name}","comments_url":"https://api.github.com/repos/dcowden/cadquery/issues/81/comments","events_url":"https://api.github.com/repos/dcowden/cadquery/issues/81/events","html_url":"https://github.com/dcowden/cadquery/issues/81","id":66508637,"number":81,"title":"Lego Example Questions","user":{"login":"jmwright","id":1015439,"avatar_url":"https://avatars.githubusercontent.com/u/1015439?v=3","gravatar_id":"","url":"https://api.github.com/users/jmwright","html_url":"https://github.com/jmwright","followers_url":"https://api.github.com/users/jmwright/followers","following_url":"https://api.github.com/users/jmwright/following{/other_user}","gists_url":"https://api.github.com/users/jmwright/gists{/gist_id}","starred_url":"https://api.github.com/users/jmwright/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmwright/subscriptions","organizations_url":"https://api.github.com/users/jmwright/orgs","repos_url":"https://api.github.com/users/jmwright/repos","events_url":"https://api.github.com/users/jmwright/events{/privacy}","received_events_url":"https://api.github.com/users/jmwright/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dcowden/cadquery/labels/question","name":"question","color":"cc317c"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-06T02:27:42Z","updated_at":"2015-04-06T10:29:27Z","closed_at":null,"body":"@dcowden I've started looking through the Lego brick example so that I can do a walk-through video, and have a few questions. \r\n\r\n1. The `thickness` variable isn't used. Was that intentional, or an oversight? It seems like this needs to be a boolean `thick` that switches the `height` between 3.2 and 9.6 mm. Here's what I would suggest switching it to:\r\n\r\n    ```python\r\n    thick = False     # Whether this is a thick or thin brick\r\n    ...\r\n    height = 9.6 if thick else 3.2  # Use our boolean to switch thickness\r\n    ```\r\n2. What does the `t` variable stand for in the following lines?\r\n\r\n    ```python\r\n    t = (pitch - (2 * clearance) - bumpDiam) / 2.0\r\n    postDiam = pitch - t  # works out to 6.5\r\n    ```\r\n3. Are there any concerns about Lego getting upset with us for modelling their product with measurements included?"},"comment":{"url":"https://api.github.com/repos/dcowden/cadquery/issues/comments/90006079","html_url":"https://github.com/dcowden/cadquery/issues/81#issuecomment-90006079","issue_url":"https://api.github.com/repos/dcowden/cadquery/issues/81","id":90006079,"user":{"login":"dcowden","id":1297923,"avatar_url":"https://avatars.githubusercontent.com/u/1297923?v=3","gravatar_id":"","url":"https://api.github.com/users/dcowden","html_url":"https://github.com/dcowden","followers_url":"https://api.github.com/users/dcowden/followers","following_url":"https://api.github.com/users/dcowden/following{/other_user}","gists_url":"https://api.github.com/users/dcowden/gists{/gist_id}","starred_url":"https://api.github.com/users/dcowden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcowden/subscriptions","organizations_url":"https://api.github.com/users/dcowden/orgs","repos_url":"https://api.github.com/users/dcowden/repos","events_url":"https://api.github.com/users/dcowden/events{/privacy}","received_events_url":"https://api.github.com/users/dcowden/received_events","type":"User","site_admin":false},"created_at":"2015-04-06T10:29:27Z","updated_at":"2015-04-06T10:29:27Z","body":"Are you looking at the version on pparts.com  or in the cadquery freecad\nmodule version?\n\nNo concerns with lego. Other companies make exact duplicates at this point,\nand the patents expired long ago\nOn Apr 5, 2015 10:27 PM, \"Jeremy Wright\" <notifications@github.com> wrote:\n\n> @dcowden <https://github.com/dcowden> I've started looking through the\n> Lego brick example so that I can do a walk-through video, and have a few\n> questions.\n>\n>    1.\n>\n>    The thickness variable isn't used. Was that intentional, or an\n>    oversight? It seems like this needs to be a boolean thick that\n>    switches the height between 3.2 and 9.6 mm. Here's what I would\n>    suggest switching it to:\n>\n>    thick = False     # Whether this is a thick or thin brick\n>    ...\n>    height = 9.6 if thick else 3.2  # Use our boolean to switch thickness\n>\n>     2.\n>\n>    What does the t variable stand for in the following lines?\n>\n>    t = (pitch - (2 * clearance) - bumpDiam) / 2.0\n>    postDiam = pitch - t  # works out to 6.5\n>\n>     3. Are there any concerns about Lego getting upset with us for\n>    modelling their product with measurements included?\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/dcowden/cadquery/issues/81>.\n>\n"}}