AU

Results 481 comments of AU

Looks OK, but the local coordinate system of the cylinder is ill-defined with only Z vector. Is that the point you are trying to make? ![afbeelding](https://github.com/CadQuery/cadquery/assets/13981538/1da182ce-1d60-44c9-ac07-0663fa04d30f)

Hey @kmarchais any updates on this? Do you intend to finalize it?

@lorenzncode it seems that your suggestions were applied. Are you OK with merging?

I think like so, but please test and report back: ```python import OCP pool = OCP.OSD.OSD_ThreadPool.DefaultPool_s() pool.SetNbDefaultThreadsToLaunch(N_THREADS) ```

Other ideas: ```python import OCP pool = OCP.OSD.OSD_ThreadPool.DefaultPool_s() pool.Init(N_THREADS) ``` or try setting the `TBB_NUM_THREADS` env var, if your OCCT is built with TBB.

Based on the OCCT code, maybe even like so: ```python import OCP pool = OCP.OSD.OSD_ThreadPool.DefaultPool_s(N_THREADS) ``` https://github.com/Open-Cascade-SAS/OCCT/blob/4c8faa5e2dd0d3e4bd7039c908016d8204889c09/src/OSD/OSD_ThreadPool.hxx#L57

Yup, confirmed. ![afbeelding](https://github.com/CadQuery/cadquery/assets/13981538/db8332b8-5adc-4778-a890-3294987b2174)

Good point, I think we should add `cq.Sketch.val()` to be consistent with `cq.Workplane()`. Also I think `sweep` could be extended to lower faces to wires (and check if they are...