-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
top-level tuple variable causes "Unable to handle assignment for node..." #17
Comments
Good chance this bug is related to this CadQuery PR. |
Thanks for the pointer, that does look like my exact issue. I'll follow along that thread for updates. In the meantime, I'll just use a list instead of a tuple. |
@justbuchanan I tested your code above with the changes in the CQGI tuple PR and it worked fine. The code coverage and tests are green on that PR now, and after the GLTF/STL binary PR is merged I'll start moving it forward again. I'll have to figure out how to specify a git install in pyproject.toml because we'll need the bleeding edge CadQuery until its next release in January. |
@justbuchanan The CQGI tuple PR has been merged into CQ. Could you verify that fixes this issue for you? |
Works great, thank you! |
cq-cli version: latest from
main
(649587e)cadquery version: latest from conda (
mamba install -c conda-forge cadquery
)cq-cli command:
cq-cli --codec stl --infile model.py --outfile model.stl
Error message:
Changing
size
in the model from a tuple to a list fixes things:The text was updated successfully, but these errors were encountered: