Skip to content
This repository has been archived by the owner on Feb 9, 2021. It is now read-only.

Add another parameter centerOption='CenterOfMass' to CQ.workplane([offset, invert]) #122

Merged
merged 1 commit into from
Nov 22, 2015

Conversation

huskier
Copy link
Contributor

@huskier huskier commented Nov 21, 2015

Following Dave's suggestion for the issue " The relative position is inaccurate #60 " in jmwright/cadquery-freecad-module, I made some changes to the CADQuery code:

  1. Add CenterOfBoundBox() and CombinedCenterOfBoundBox() functions into Shape class;
  2. Add another parameter centerOption into CQ.workplane([offset, invert]), and default the parameter to CenterOfMass which is the current behavior.

Thanks for your consideration.

…to Shape class; 2. Add another parameter centerOption into CQ.workplane([offset, invert]), and default the parameter to CenterOfMass which is the current behavior.
@jmwright
Copy link
Collaborator

@huskier This is great. Thanks for contributing!

jmwright added a commit that referenced this pull request Nov 22, 2015
Add another parameter centerOption='CenterOfMass'  to CQ.workplane([offset, invert])
@jmwright jmwright merged commit c2e7b00 into dcowden:master Nov 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants