Automatic pitch calculation in ScrewHole is broken

[lorf]

use <threads-scad/threads.scad>;

intersection() {
    union() {
        ScrewThread(5, 10);
        
        ScrewHole(5, 10)
            cylinder(d=10,h=10);
    }
    cube(10);
}

use <threads-scad/threads.scad>;

intersection() {
    union() {
        ScrewThread(5, 10);
        
        ScrewHole(5, 10, pitch=0.8)
            cylinder(d=10,h=10);
    }
    cube(10);
}

[jagerman]

I've love for #7 or #3 to get merged to fix this, as I just ran into it too.

Without the patch in place, a workaround is to always specify a pitch= value, which can be the (proper) default via:

ScrewHole(d, ...otherargs..., pitch=ThreadPitch(d))

[kevinmic]

I appreciate the findings here. I also ran into this problem. Apparently 2 years later and the fixes haven't been merged.