[cgma-dev] r4884 - in cgm/trunk: geom/OCC test

[Jane Hu]

Thanks, James. I've just checked the two files in.

Jane

On Mon, Jun 6, 2011 at 2:08 PM, James Porter <jvporter at wisc.edu> wrote:

> So I finally figured out (part of?) what's going on with the distcheck
> errors. There are some missing files that never got added to this
> commit: Line_1.brep and Line_2.brep.
>
> There may be other errors of course, so adding those might not fix
> everything...
>
> On Thu, 2011-05-26 at 13:29 -0500, jiangtao_ma at yahoo.com wrote:
> > Author: janehu
> > Date: 2011-05-26 13:29:34 -0500 (Thu, 26 May 2011)
> > New Revision: 4884
> >
> > Modified:
> >    cgm/trunk/geom/OCC/OCCModifyEngine.cpp
> >    cgm/trunk/geom/OCC/OCCSurface.cpp
> >    cgm/trunk/test/Makefile.am
> >    cgm/trunk/test/modify.cpp
> > Log:
> > Made changes regarding webcut using sweep curve method that is a possible
> bug that Yu Wenhui asked. Passed make distcheck. Testcase is in modify.cpp
> >
> > Modified: cgm/trunk/geom/OCC/OCCModifyEngine.cpp
> > ===================================================================
> > --- cgm/trunk/geom/OCC/OCCModifyEngine.cpp    2011-05-26 13:49:07 UTC
> (rev 4883)
> > +++ cgm/trunk/geom/OCC/OCCModifyEngine.cpp    2011-05-26 18:29:34 UTC
> (rev 4884)
> > @@ -4442,19 +4442,17 @@
> >            }
> >          }
> >        }
> > -      else //sheet body
> > -      {
> > -        TopoDS_Face* topoface = occ_surface->get_TopoDS_Face();
> > -        TopAbs_Orientation ori = topoface->Orientation();
> > -        topoface->Orientation(ori == TopAbs_FORWARD ? TopAbs_REVERSED :
> > +    }
> > +    if(!occ_shell || occ_shell->is_sheet()) //sheet body
> > +    {
> > +      TopoDS_Face* topoface = occ_surface->get_TopoDS_Face();
> > +      TopAbs_Orientation ori = topoface->Orientation();
> > +      topoface->Orientation(ori == TopAbs_FORWARD ? TopAbs_REVERSED :
> >                                                      TopAbs_FORWARD);
> > -        occ_surface->set_TopoDS_Face(*topoface);
> > -        surface_list.append(occ_surface);
> > -      }
> > +      occ_surface->set_TopoDS_Face(*topoface);
> > +      surface_list.append(occ_surface);
> >        PRINT_INFO( "Modified volume\n" );
> >      }
> > -    else
> > -      PRINT_WARNING( "Volume was not modified\n" );
> >    }
> >    face_list = surface_list;
> >    return CUBIT_SUCCESS;
> > @@ -4639,22 +4637,12 @@
> >  CubitStatus OCCModifyEngine::get_sweepable_toposhape(OCCCurve*& curve,
> >                                                TopoDS_Shape&
> toposhape)const
> >  {
> > -  DLIList<OCCLoop*> loops;
> > -  loops =  curve->loops();
> > -  if( loops.size()) //not a free curve
> > -  {
> > -    //copy the curve
> > -    Curve* c_curve = make_Curve(curve);
> > -    if(c_curve)
> > -     curve = CAST_TO(c_curve, OCCCurve);
> > -    else
> > -    {
> > -      PRINT_ERROR("Can't copy the curve for sweep.\n");
> > -      return CUBIT_FAILURE;
> > -    }
> > -  }
> >    TopoDS_Edge *edge = curve->get_TopoDS_Edge( );
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/cgma-dev/attachments/20110607/db90491b/attachment.htm>