Union with polyhedron doesn't render with F6

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

Union with polyhedron doesn't render with F6

NathanA
Why won't this polyhedron and cylinder union render with F6?

p = [[0, 0, 0], [10, 0, 0], [10, 0, 10], [0, 0, 10], [0, 10, 0], [0, 10, 10], [10, 10, 10], [10, 10, 0], [0, 0, 10], [0, 10, 10], [10, 10, 10], [10, 0, 10]];
f = [[0, 3, 2, 1, 0], [0, 4, 5, 3], [5, 4, 7, 6, 5], [1, 2, 6, 7, 1], [0, 1, 7, 4, 0], [8, 9, 10, 11, 8]];

union() {
    polyhedron(points=p, faces=f);
    cylinder([1,1,1]);
}
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

Miro Hrončok
It does for me on almost latest git master.
Miro Hrončok

Telefon: +420777974800


2014-12-28 9:11 GMT+01:00 NathanA <[hidden email]>:

> Why won't this polyhedron and cylinder union render with F6?
>
> p = [[0, 0, 0], [10, 0, 0], [10, 0, 10], [0, 0, 10], [0, 10, 0], [0, 10,
> 10], [10, 10, 10], [10, 10, 0], [0, 0, 10], [0, 10, 10], [10, 10, 10], [10,
> 0, 10]];
> f = [[0, 3, 2, 1, 0], [0, 4, 5, 3], [5, 4, 7, 6, 5], [1, 2, 6, 7, 1], [0, 1,
> 7, 4, 0], [8, 9, 10, 11, 8]];
>
> union() {
>     polyhedron(points=p, faces=f);
>     cylinder([1,1,1]);
> }
>
>
>
> --
> View this message in context: http://forum.openscad.org/Union-with-polyhedron-doesn-t-render-with-F6-tp10769.html
> Sent from the OpenSCAD mailing list archive at Nabble.com.
>
> _______________________________________________
> OpenSCAD mailing list
> [hidden email]
> http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

MichaelAtOz
Administrator
F5 works on 2014.03, F6 does not,

Both F5 & F6 work on latest snapshop.
Admin - email* me if you need anything,
or if I've done something stupid...
* click on my MichaelAtOz label, there is a link to email me.

Unless specifically shown otherwise above, my contribution is in the Public Domain; to the extent possible under law, I have waived all copyright and related or neighbouring rights to this work.
Obviously inclusion of works of previous authors is not included in the above.


The TPP is no simple “trade agreement.” Fight it! http://www.ourfairdeal.org/ time is running out!
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

nophead
I think the problem is you don't need to repeat the first vertex in the faces list. If I remove all the last numbers it works fine in F5 and F6 in the old versions I use.


On 28 December 2014 at 11:02, MichaelAtOz <[hidden email]> wrote:
F5 works on 2014.03, F6 does not,

Both F5 & F6 work on latest snapshop.



-----
Unless specifically shown otherwise above, my contribution is in the Public Domain; To the extent possible under law, I have waived all copyright and related or neighbouring rights to this work. This work is published globally via the internet. :) Inclusion of works of previous authors is not included in the above.

The TPP is no simple “trade agreement.”   Fight it! http://www.ourfairdeal.org/
--
View this message in context: http://forum.openscad.org/Union-with-polyhedron-doesn-t-render-with-F6-tp10769p10771.html
Sent from the OpenSCAD mailing list archive at Nabble.com.

_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org


_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

Peter Ragosch
In reply to this post by NathanA
It doesn't on OpenSCAD version 2014.11.28 / OpenSUSE 31.1
Peter Ragosch


Am Sun, 28 Dec 2014 01:11:27 -0700 (MST)
schrieb NathanA <[hidden email]>:

> Why won't this polyhedron and cylinder union render with F6?
>
> p = [[0, 0, 0], [10, 0, 0], [10, 0, 10], [0, 0, 10], [0, 10, 0], [0,
> 10, 10], [10, 10, 10], [10, 10, 0], [0, 0, 10], [0, 10, 10], [10, 10,
> 10], [10, 0, 10]];
> f = [[0, 3, 2, 1, 0], [0, 4, 5, 3], [5, 4, 7, 6, 5], [1, 2, 6, 7, 1],
> [0, 1, 7, 4, 0], [8, 9, 10, 11, 8]];
>
> union() {
>     polyhedron(points=p, faces=f);
>     cylinder([1,1,1]);
> }
>
>
>
> --
> View this message in context:
> http://forum.openscad.org/Union-with-polyhedron-doesn-t-render-with-F6-tp10769.html
> Sent from the OpenSCAD mailing list archive at Nabble.com.
>
> _______________________________________________
> OpenSCAD mailing list
> [hidden email]
> http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
>



_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

Bananapeel
In reply to this post by NathanA
It gives me an CGAL error in 2014.12.04, so there is probably something wrong with your polyhedron. nophead pointed out the problem.

Fix:
f = [[0, 3, 2, 1], [0, 4, 5, 3], [5, 4, 7, 6], [1, 2, 6, 7], [0, 1, 7, 4], [8, 9, 10, 11]];

Rendering Polygon Mesh using CGAL...
CGAL error in CGAL_Nef_polyhedron3(): CGAL ERROR: assertion violation!
Expr: pe != 0
File: /data/OpenSCAD/libraries-mingw64-master/mxe-w64/usr/x86_64-w64-mingw32.static/include/CGAL/Nef_3/polyhedron_3_to_nef_3.h
Line: 214
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

Peter Ragosch
In reply to this post by nophead
Same with OpenSCAD version 2014.11.28 / OpenSUSE 13.1

and

have you noticed that the second face in the face list consists of 4
values but all others of 5 values ? The mismatch of numbers may cause
the problem - I didn't check.

Peter Ragosch


Am Sun, 28 Dec 2014 11:14:40 +0000
schrieb nop head <[hidden email]>:

> I think the problem is you don't need to repeat the first vertex in
> the faces list. If I remove all the last numbers it works fine in F5
> and F6 in the old versions I use.
>
>
> On 28 December 2014 at 11:02, MichaelAtOz <[hidden email]>
> wrote:
>
> > F5 works on 2014.03, F6 does not,
> >
> > Both F5 & F6 work on latest snapshop.
> >
> >
> >
> > -----
> > Unless specifically shown otherwise above, my contribution is in the
> > Public Domain; To the extent possible under law, I have waived all
> > copyright and related or neighbouring rights to this work. This
> > work is published globally via the internet. :) Inclusion of works
> > of previous authors is not included in the above.
> >
> > The TPP is no simple "trade agreement."   Fight it!
> > http://www.ourfairdeal.org/
> > --
> > View this message in context:
> > http://forum.openscad.org/Union-with-polyhedron-doesn-t-render-with-F6-tp10769p10771.html
> > Sent from the OpenSCAD mailing list archive at Nabble.com.
> >
> > _______________________________________________
> > OpenSCAD mailing list
> > [hidden email]
> > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
> >



_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

NathanA
In reply to this post by nophead
@nophead I removed the extra vertices and made all the faces five points. It is now a basic cube, 8 vertices and 6 faces. It still doesn't F6 on 2014.11.09. How did you get it to work?

p = [[0, 0, 0], [10, 0, 0], [10, 0, 10], [0, 0, 10], [0, 10, 0],
     [0, 10, 10], [10, 10, 10], [10, 10, 0]];
f = [[0, 3, 2, 1, 0], [0, 4, 5, 3, 0], [5, 4, 7, 6, 5], [1, 2, 6, 7, 1], [0, 1, 7, 4, 0], [3, 5, 6, 2, 3]];

union() {
    polyhedron(points=p, faces=f);
    cylinder([1,1,1]);
}
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

nophead
I removed the repeated vertex indices in faces. To define a square you need four indices. You don't repeat the first one to close it.

On 28 December 2014 at 17:38, NathanA <[hidden email]> wrote:
@nophead I removed the extra vertices and made all the faces five points. It
is now a basic cube, 8 vertices and 6 faces. It still doesn't F6 on
2014.11.09. How did you get it to work?

p = [[0, 0, 0], [10, 0, 0], [10, 0, 10], [0, 0, 10], [0, 10, 0],
     [0, 10, 10], [10, 10, 10], [10, 10, 0]];
f = [[0, 3, 2, 1, 0], [0, 4, 5, 3, 0], [5, 4, 7, 6, 5], [1, 2, 6, 7, 1], [0,
1, 7, 4, 0], [3, 5, 6, 2, 3]];

union() {
    polyhedron(points=p, faces=f);
    cylinder([1,1,1]);
}




--
View this message in context: http://forum.openscad.org/Union-with-polyhedron-doesn-t-render-with-F6-tp10769p10776.html
Sent from the OpenSCAD mailing list archive at Nabble.com.

_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org


_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

kintel
Administrator
On Dec 28, 2014, at 12:42 PM, nop head <[hidden email]> wrote:

> I removed the repeated vertex indices in faces. To define a square you need four indices. You don't repeat the first one to close it.
>
nophead is right.
We made face tessellation more robust ca. Dec 1st., so any snapshot after that should work.

 -Marius


_______________________________________________
OpenSCAD mailing list
[hidden email]
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
Reply | Threaded
Open this post in threaded view
|

Re: Union with polyhedron doesn't render with F6

Bananapeel
In reply to this post by NathanA
Did you read my previous message? Replace the line with f=... in your code with mine.