Nabble removed Mailing-list integration from the Forum.
This killed the Forum. This is now an ARCHIVE.
It is likely Nabble will shutdown in the future.

So basically the Forum is now out of date, we are looking into migrating the history.

OpenSCAD support is now provided by the Mailing List.
See the first pinned post here for instructions on subscribing to the mailing list.

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.