-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add point-and-click workflows for Helix operation #5059
Comments
Helix Case
Revolve Case
Helix with sketch case
|
well i kinda like its a very similar frunction to revolve only that its 3d versus 2d, so i think thats nice, ill tag in @jgomez720 for if its an issue that its not pinned to a plane other than that |
fwiw transform rotate around an axis is also local or global but different since its already operating on a solid |
Helix is not a sketch tool. It's a "path tool" so to speak. Helices will always be 3D. This does offer up a great question though, which is how do we define the start of the helix. The reason we can't do this is because sketches should always be 2D. Once you create a helix, you are turning this into a 3D object. // sketch plane example
sketch001 = startSketchOn('XY')
|> helix({
angleStart = 0,
ccw = true,
revolutions = 16,
length = 10,
radius = 5,
axis = 'Y'
}) However, I would be fine if helix took a plane (which can be a sketch) as a parameter just for initial start point. helix({
angleStart = 0,
ccw true,
revolutions = 16,
length = 10,
radius = 5,
axis = myDefinedAxis,
start = plane001 // or sketch001 |
Discussion with Pierre about point and click workflow for helix and what we can achieve. In the command bar workflow we can easily support floats as well as string option selects ( Questions:
Priority:
|
We have a helix function in KCL, and we should support point-and-click worflows for it:
toolbar.ts
(blocked by Create and add helix icon #5058)modelingCommandConfig.ts
, and corresponding actor tomodelingMachine.ts
#5072modifyAst.ts#deleteFromSelection
#5073The text was updated successfully, but these errors were encountered: