discuss@lists.openscad.org

OpenSCAD general discussion Mailing-list

View all threads

Union with polyhedron doesn't render with F6

N
NathanA
Sun, Dec 28, 2014 8:11 AM

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.

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.
MH
Miro Hrončok
Sun, Dec 28, 2014 10:43 AM

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

Telefon: +420777974800

2014-12-28 9:11 GMT+01:00 NathanA napter@gmail.com:

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
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

It does for me on almost latest git master. Miro Hrončok Telefon: +420777974800 2014-12-28 9:11 GMT+01:00 NathanA <napter@gmail.com>: > 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 > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org
M
MichaelAtOz
Sun, Dec 28, 2014 11:02 AM

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.

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.
NH
nop head
Sun, Dec 28, 2014 11:14 AM

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 oz.at.michael@gmail.com 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
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

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 <oz.at.michael@gmail.com> 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 > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >
PR
Peter Ragosch
Sun, Dec 28, 2014 11:16 AM

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 napter@gmail.com:

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
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

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 <napter@gmail.com>: > 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 > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >
B
Bananapeel
Sun, Dec 28, 2014 11:22 AM

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

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

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 -- View this message in context: http://forum.openscad.org/Union-with-polyhedron-doesn-t-render-with-F6-tp10769p10774.html Sent from the OpenSCAD mailing list archive at Nabble.com.
PR
Peter Ragosch
Sun, Dec 28, 2014 11:30 AM

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 nop.head@gmail.com:

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 oz.at.michael@gmail.com
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
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

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 <nop.head@gmail.com>: > 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 <oz.at.michael@gmail.com> > 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 > > Discuss@lists.openscad.org > > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org > >
N
NathanA
Sun, Dec 28, 2014 5:38 PM

@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.

@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.
NH
nop head
Sun, Dec 28, 2014 5:42 PM

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 napter@gmail.com 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
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

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 <napter@gmail.com> 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 > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >
MK
Marius Kintel
Sun, Dec 28, 2014 5:46 PM

On Dec 28, 2014, at 12:42 PM, nop head nop.head@gmail.com 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

On Dec 28, 2014, at 12:42 PM, nop head <nop.head@gmail.com> 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