View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0001436

VTK

(No Category)

public

2004-12-16 16:21

2011-01-13 17:00

Reporter

john platt

Assigned To

Mathieu Malaterre

Priority

low

Severity

major

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0001436: BandedContours - Contour edges are produced for cells which are completely in a contour band.

Description

vtkBandedPolyDataContourFilter.cxx, Revision: 1.42

At about line 560

Move

      //Produce contour edges if requested
      if ( this->GenerateContourEdges )
        {
        .....
        }

To after

      //Trivial output - completely in a contour band or a triangle
      if ( ! intersectionPoint || numFullPts <= 3 )
        {
        cellId = this->InsertCell(newPolys,npts,pts,cellId,s[idx],newScalars);
        continue;
        }

I have NOT tested this move until now! Originally, I put the condition if (

intersectionPoint ) around the code to be moved instead, so there is a difference.

Also, I have not tried with lines or strips. Hopefully it gives you an idea for a fix.

Tags

No tags attached.

Project

Type

Attached Files

Relationships

Notes
(0001833) Mathieu Malaterre (developer) 2004-12-16 16:58	Thanks again John ! $ cvs ci -m"BUG: Fix -partially- Bug 0001436 - BandedContours - Contour edges are produced for cells which are completely in a contour band. Thanks again to John Platt for patch" vtkBandedPolyDataContourFilter.cxx /cvsroot/VTK/VTK/Graphics/vtkBandedPolyDataContourFilter.cxx,v <-- vtkBandedPolyDataContourFilter.cxx new revision: 1.52; previous revision: 1.51 I am closing the bug until I (somebody) can reproduce it.

Issue History
Date Modified	Username	Field	Change
2010-11-29 17:59	Mathieu Malaterre	Source_changeset_attached	=> VTK master 525c5e2e
2011-01-13 17:00		Source_changeset_attached	=> VTK master a2bd8391
2011-01-13 17:00		Source_changeset_attached	=> VTK master 020ef709
2011-06-16 13:11	Zack Galbreath	Category	=> (No Category)