View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0014460

VTK

(No Category)

public

2013-12-12 16:58

2014-10-01 20:22

Reporter

Ronald Römer

Assigned To

Berk Geveci

Priority

low

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0014460: vtkDataSet: GetCellTypes is useless in the current form

Description

With GetCellTypes its possible to get a list of cell-types that are defined in a dataset. It returns an object of type vtkCellTypes. But this object is not really the object thats internal used to store the recently added cells. It returns a light version of it, without the locations. Within GetCellTypes they are set to -1. But without that information you can not use it for a reasonable purpose. The same information is available via:

vtkIntArray *types = vtkIntArray::New();
for (int i = 0; i < pd->GetNumberOfCells(); i++) {
types->InsertNextValue(pd->GetCellType(i));
}

You see the method is totally useless.

The method should return a pointer to Cells or a deep copy of it.

Tags

No tags attached.

Project

TBD

Type

usability

Attached Files

Relationships

Notes
(0033406) Berk Geveci (administrator) 2014-10-01 20:22	This is not a bug. The author of the report misunderstood the intent of the method - it is to return the unique list of cell types in a dataset. The method has limited use for sure and maybe it is not worth forcing all subclasses to implement it but this is certainly not a bug.

Issue History
Date Modified	Username	Field	Change
2013-12-12 16:58	Ronald Römer	New Issue
2014-10-01 20:22	Berk Geveci	Note Added: 0033406
2014-10-01 20:22	Berk Geveci	Status	backlog => closed
2014-10-01 20:22	Berk Geveci	Assigned To	=> Berk Geveci
2014-10-01 20:22	Berk Geveci	Resolution	open => fixed