<div dir="ltr">So, I will add RefVolume to the return list of iGeom_get_adjacent_entities() as the result of the discussion.<div><br></div><div>CubitEntity only has derived classes of RefEntity and CubitCoordinateSystem, iGeom_CGMA.cc only uses RefEntity as instance of the object. </div>
<div><br></div><div>Jane</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 16, 2014 at 11:35 AM, Jane Hu <span dir="ltr"><<a href="mailto:janejhu@gmail.com" target="_blank">janejhu@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">See comments below...<br><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="">On Mon, Jun 16, 2014 at 10:22 AM, Paul Wilson <span dir="ltr"><<a href="mailto:wilsonp@engr.wisc.edu" target="_blank">wilsonp@engr.wisc.edu</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    Hi Jane,<br>
    <br>
    I think this needs a thorough review!  Patrick and I hunted through
    iGeom_CGMA and found some inconsistencies in a few places.<br>
    <br>
    First, there is an inconsistency between returning pointers to
    RefEntity and pointers to CubitEntity.  This is probably less
    important, but should probably be resolved in any change.<br></div></blockquote><div> </div></div><div>I see RefEntity is inherited from CubitEntity, and CubitEntity is a pure virtual class. So even you got pointers to CubitEntity, you have to instantiate it to RefEntity objects.</div>
<div class="">
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    <br>
    Second, there are places where the geometry is queried for entities
    of dimension=3 (returning RefVolumes) and places where the geometry
    is queried for entities of type=iGeom_REGION (== Body).<br></div></blockquote><div><br></div></div><div>In this case, I can return both bodies and refvolumes, so after user gets the refentity list back, he can cast to see if they are bodies or refvolumes, and choose to keep what ever he needs.</div>
<div><div class="h5">
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    <br>
    This means that 2 calls into the interface that should resolve to
    the same entity may not do so for both reasons.<br>
    <br>
    All of this is related to the fundamental difference between the
    iGeom types (equivalent to vertex, edge, face, volume) and the
    CGM/ACIS types (equiavlent to vertex, edge, face, volume, body).  I
    think Tim's suggestion is the best, but will require some
    substantial effort.<span><font color="#888888"><br>
    <br>
    Paul</font></span><div><div><br>
    <br>
    <div>On 06/16/2014 10:14 AM, Jane Hu wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">I see in iGeom_CGMA.cc file,
        function iGeom_get_adjacent_entities, if asks for 3-D
        refentities, it returns body list. In CGM, bodies are not
        necessary to be 3-D entities, it may contain single surface or
        shell. Should we change the return to be only refvolumes which
        are 3-D entities (I prefer this, makes more sense)? Or I can
        return both bodies and refvolumes in the returned refentity
        list. Depending on your user case, I can return one of the two
        choices.
        <div>
          <br>
        </div>
        <div>Jane</div>
      </div>
      <div class="gmail_extra"><br>
        <br>
        <div class="gmail_quote">On Mon, Jun 16, 2014 at 9:32 AM, Jane
          Hu <span dir="ltr"><<a href="mailto:janejhu@gmail.com" target="_blank">janejhu@gmail.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
            <div dir="ltr">Yes, it returns a list of refVolume's which
              are RefEntities.<span><font color="#888888">
                  <div>
                    <br>
                  </div>
                  <div>Jane</div>
                </font></span></div>
            <div>
              <div>
                <div class="gmail_extra"><br>
                  <br>
                  <div class="gmail_quote">On Sun, Jun 15, 2014 at 11:55
                    PM, Patrick Shriwise <span dir="ltr"><<a href="mailto:shriwise@wisc.edu" target="_blank">shriwise@wisc.edu</a>></span>
                    wrote:<br>
                    <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
                      <div dir="auto">
                        <div>Hi Jane, </div>
                        <div><br>
                        </div>
                        <div>Will the ref_volumes() method return
                          RefEntities? </div>
                        <span><font color="#888888">
                            <div>
                              <br>
                            </div>
                            <div>Patrick</div>
                          </font></span>
                        <div>
                          <div>
                            <div><br>
                              On Jun 15, 2014, at 3:56 PM, Jane Hu <<a href="mailto:janejhu@gmail.com" target="_blank">janejhu@gmail.com</a>>
                              wrote:<br>
                              <br>
                            </div>
                            <blockquote type="cite">
                              <div>
                                <div dir="ltr">Look at line 6454, if you
                                  have body's pointer, directly call
                                  ref_volumes() will work. It's not an
                                  iGeom interface function, but you can
                                  access it in iGeom.
                                  <div><br>
                                  </div>
                                  <div>Good luck!</div>
                                  <div>
                                    <br>
                                  </div>
                                  <div>Jane</div>
                                </div>
                                <div class="gmail_extra"><br>
                                  <br>
                                  <div class="gmail_quote">On Sat, Jun
                                    14, 2014 at 8:55 AM, Patrick
                                    Shriwise <span dir="ltr"><<a href="mailto:shriwise@wisc.edu" target="_blank">shriwise@wisc.edu</a>></span>
                                    wrote:<br>
                                    <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
                                      <div text="#000000" bgcolor="#FFFFFF"> Hi Jane, <br>
                                        <br>
                                        I looked around for this but
                                        didn't see it, is there a
                                        specific line/function you were
                                        thinking of? Good to know the
                                        ref_volume information is so
                                        easy to get to, but it doesn't
                                        seem that its part of the iGeom
                                        interface. The issue being that
                                        there's no way to access a
                                        body's volumes via iGeom. <br>
                                        <br>
                                        Cheers, <br>
                                        <br>
                                        Patrick <br>
                                        <div>
                                          <div>
                                            <div>On 06/13/2014 11:26 PM,
                                              Jane Hu wrote:<br>
                                            </div>
                                            <blockquote type="cite">
                                              <div dir="ltr"><br>
                                                <div class="gmail_extra"><br>
                                                  <div class="gmail_quote">
                                                    <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
                                                      <div bgcolor="#FFFFFF" text="#000000"><br>
                                                        <br>
                                                         Is there a way
                                                        to get volumes
                                                        within bodies
                                                        via iGeom?</div>
                                                    </blockquote>
                                                    <div><br>
                                                    </div>
                                                    <div>See example in
                                                       iGeom_CGMA.cc,
                                                      body->ref_volumes(volumes).</div>
                                                    <div><br>
                                                    </div>
                                                    <div>Jane</div>
                                                    <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
                                                      <div bgcolor="#FFFFFF" text="#000000"><span><font color="#888888"><br>
                                                          <pre cols="72">-- 
Patrick C. Shriwise
Research Assistant
University of Wisconsin - Madison
Engineering Research Building - Rm. 428
1500 Engineering Drive
Madison, WI 53706
<a href="tel:%28608%29%20446-8173" value="+16084468173" target="_blank">(608) 446-8173</a>
</pre>
                                                          </font></span></div>
                                                    </blockquote>
                                                  </div>
                                                  <br>
                                                  <br clear="all">
                                                  <div><br>
                                                  </div>
                                                  -- <br>
                                                  Jane Hu<br>
                                                  <br>
                                                  Asst. Researcher<br>
                                                  Dept. of Engineering
                                                  Physics<br>
                                                  UW @ Madison<br>
                                                  <br>
                                                  "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) </div>
                                              </div>
                                            </blockquote>
                                            <br>
                                          </div>
                                        </div>
                                      </div>
                                    </blockquote>
                                  </div>
                                  <br>
                                  <br clear="all">
                                  <div><br>
                                  </div>
                                  -- <br>
                                  Jane Hu<br>
                                  <br>
                                  Asst. Researcher<br>
                                  Dept. of Engineering Physics<br>
                                  UW @ Madison<br>
                                  <br>
                                  "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)
                                </div>
                              </div>
                            </blockquote>
                          </div>
                        </div>
                      </div>
                    </blockquote>
                  </div>
                  <br>
                  <br clear="all">
                  <div><br>
                  </div>
                  -- <br>
                  Jane Hu<br>
                  <br>
                  Asst. Researcher<br>
                  Dept. of Engineering Physics<br>
                  UW @ Madison<br>
                  <br>
                  "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)
                </div>
              </div>
            </div>
          </blockquote>
        </div>
        <br>
        <br clear="all">
        <div><br>
        </div>
        -- <br>
        Jane Hu<br>
        <br>
        Asst. Researcher<br>
        Dept. of Engineering Physics<br>
        UW @ Madison<br>
        <br>
        "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)
      </div>
    </blockquote>
    <br>
    </div></div><div><pre cols="72">-- 
-- ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ --
Paul Wilson ~ UW-Madison ~ <a href="tel:608-263-0807" value="+16082630807" target="_blank">608-263-0807</a> ~ cal: <a href="http://bit.ly/pphw-cal" target="_blank">http://bit.ly/pphw-cal</a>
Professor, Engineering Physics. ~ <a href="http://cnerg.engr.wisc.edu" target="_blank">http://cnerg.engr.wisc.edu</a>
Faculty Director, Advanced Computing Infrastructure</pre>
  </div></div>

</blockquote></div></div></div><div><div class="h5"><br><br clear="all"><div><br></div>-- <br>Jane Hu<br>
<br>
Asst. Researcher<br>
Dept. of Engineering Physics<br>
UW @ Madison<br>
<br>
"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)
</div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Jane Hu<br>
<br>
Asst. Researcher<br>
Dept. of Engineering Physics<br>
UW @ Madison<br>
<br>
"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)
</div>