[cgma-dev] Facet/Cholla engine

[Jane Hu]

Hi, David:

I debugged your program today, it seems that the code does have some bug
when trying to create individual surface and body through individual
 build_facet_surface() calls, here you are trying to create 4 surfaces and
4 bodies. However, I am not familiar with the code enough to make any
changes.

I did checked that if you pass in all the facets, with empty point list, it
can create a single FacetBody and  you can export it.

I attached the changed code for you, don't know if it's sufficient for your
purpose.

Jane


On Tue, Jan 14, 2014 at 1:35 PM, David Thompson
<david.thompson at kitware.com>wrote:

> Hi Jane,
>
> Thanks for your help. I've tried passing an empty point list into
> build_facet_surface() but it still crashes, and a plist containing just the
> points used by the triangle list. I've pared it down to a minimal
> reproducible, which is attached. If you understand what I am doing wrong,
> it would really help as I'm not sure I understand what CGM is doing where
> the crash occurs.
>
>         Thanks,
>         David
>
>
> On Jan 14, 2014, at 1:46 PM, Jane Hu <janejhu at gmail.com> wrote:
>
> > From what I read, facets sharing the same points and edges, if they are
> called from different build_facet_surface(), they should internally just
> create one set of points and edges, and should be automatically divided
> into two separate volumn/surface sets.
> >
> > There are functions to check existing points and use the old ones
> instead of create new ones.
> >
> > Jane
> >
> > Jane
> >
> >
> > On Tue, Jan 14, 2014 at 10:39 AM, Jane Hu <janejhu at gmail.com> wrote:
> > Hi, David
> >
> > I double checked the prototype as well as the code (I don't have a
> program suitable to debug), it seems that if your plist is not
> corresponding to the tflist and qflist, you may leave it empty. The code
> should fill the point list for you.
> > Unless the points are extra points which are not in the tflist nor in
> the qflist.
> >
> > Can you try it out and see if it works, I am now reading this part of
> code to see if coincident points are recognized if you call separately.
> >
> > Jane
> >
> >
> >
> > On Mon, Jan 13, 2014 at 10:03 PM, David Thompson <
> david.thompson at kitware.com> wrote:
> > Hi all,
> >
> > I have a question about the facet engine in CGM. Are multiple faceted
> surfaces per body supported? I see crashes if I call
> >
> >         DLIList <CubitPoint*> plist;
> >         DLIList<CubitFacet*> tflist;
> >         DLIList<CubitQuadFacet*> qflist;
> >         DLIList<Surface*> slist;
> >
> >         FacetModifyEngine::instance()->build_facet_surface(
> >           qflist, tflist, plist, -1, 0, false, false, slist);
> >
> > where tflist and qflist do not use every point in plist (because another
> call to build_facet_surface() will use those points later).
> >
> > If I create a separate plist for each call to build_facet_surface(), it
> does not crash... but will coincident points be recognized as identical and
> merged -- or will the geometry need some kind of stitching?
> >
> >         Thanks,
> >         David
> >
> >
> >
> > --
> > Jane Hu
> >
> > Asst. Researcher
> > Dept. of Engineering Physics
> > UW @ Madison
> >
> > "And we know that for those who love God, that is, for those who are
> called according to his purpose, all things are working together for good."
> (Romans 8:28)
> >
> >
> >
> > --
> > Jane Hu
> >
> > Asst. Researcher
> > Dept. of Engineering Physics
> > UW @ Madison
> >
> > "And we know that for those who love God, that is, for those who are
> called according to his purpose, all things are working together for good."
> (Romans 8:28)
>
>
>


-- 
Jane Hu

Asst. Researcher
Dept. of Engineering Physics
UW @ Madison

"And we know that for those who love God, that is, for those who are called
according to his purpose, all things are working together for good."
(Romans 8:28)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/cgma-dev/attachments/20140115/4df06c8a/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: multifaceted.cpp
Type: text/x-c++src
Size: 3710 bytes
Desc: not available
URL: <http://lists.mcs.anl.gov/pipermail/cgma-dev/attachments/20140115/4df06c8a/attachment.cpp>