Bug: cylinder() broken in today's svn

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Bug: cylinder() broken in today's svn

clothbot
Hi Clifford,

The cylinder() object broke for me between yesterday and today's build:

--snip--

// cylinder test cases

// works
translate([0,0,-100]) cylinder(r=200,h=50,center=false);
// looks trapezoidal
translate([0,0,-50]) cylinder(r=300,h=50,center=false);

mfgGrid=100.0;
scale([1/mfgGrid,1/mfgGrid,1/mfgGrid]) {
// works but z-axis wrong
cylinder(r=2.0*mfgGrid,h=3.0*mfgGrid,center=false);
// completely broken
translate([5.0*mfgGrid,0,0])
cylinder(r=3.0*mfgGrid,h=3.0*mfgGrid,center=false);
}

// the following two work but are too low-res at STL export 1unit =1mm  
scale required:
translate([0,5.0,0]) cylinder(r=2.0,h=3.0,center=false);
translate([5.0,5.0,0]) cylinder(r=3.0,h=3.0,center=false);

--end-snip--

Andrew.

--

"The future is already here.  It's just not very evenly distributed"  
-- William Gibson




Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Bug: cylinder() broken in today's svn

clothbot
That openscad.h tweak in r299 appears to have fixed the bug.

Thanks!

Andrew.

On 14-Jan-10, at 2:30 PM, Andrew Plumb wrote:

> Hi Clifford,
>
> The cylinder() object broke for me between yesterday and today's  
> build:
>
> --snip--
>
> // cylinder test cases
>
> // works
> translate([0,0,-100]) cylinder(r=200,h=50,center=false);
> // looks trapezoidal
> translate([0,0,-50]) cylinder(r=300,h=50,center=false);
>
> mfgGrid=100.0;
> scale([1/mfgGrid,1/mfgGrid,1/mfgGrid]) {
> // works but z-axis wrong
> cylinder(r=2.0*mfgGrid,h=3.0*mfgGrid,center=false);
> // completely broken
> translate([5.0*mfgGrid,0,0])
> cylinder(r=3.0*mfgGrid,h=3.0*mfgGrid,center=false);
> }
>
> // the following two work but are too low-res at STL export 1unit =1mm
> scale required:
> translate([0,5.0,0]) cylinder(r=2.0,h=3.0,center=false);
> translate([5.0,5.0,0]) cylinder(r=3.0,h=3.0,center=false);
>
> --end-snip--
>
> Andrew.
>
> --
>
> "The future is already here.  It's just not very evenly distributed"
> -- William Gibson
>
>
>
> _______________________________________________
> OpenSCAD mailing list
> [hidden email]
> http://rocklinux.net/mailman/listinfo/openscad

--

"The future is already here.  It's just not very evenly distributed"  
-- William Gibson




Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Bug: cylinder() broken in today's svn

Clifford Wolf
Hi,

On Thu, Jan 14, 2010 at 05:23:03PM -0500, Andrew Plumb wrote:
> That openscad.h tweak in r299 appears to have fixed the bug.

actually that was just a quick hack. I needed the night to think about the
problem and it is really fixed in r306 now..

yours,
 - clifford

--
Caffeine for mind, pizza for body, sushi for soul.

Loading...