Nabble has removed Mailing-list integration.
Posts created here DO NOT GET SENT TO THE MAILING LIST.
Mailing-list emails DO NOT GET POSTED TO THE FORUM.
So basically the Forum is now out of date, we are looking into migrating the history.

For now you should send emails, people will see them, discuss@lists.openscad.org.

Bug: cylinder() broken in today's svn

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

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
|

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
|

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.