View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0011426 | ITK | public | 2010-11-09 15:31 | 2010-11-18 21:09 | |||||
Reporter | David Doria | ||||||||
Assigned To | kentwilliams | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | ITK-4-A0 | ||||||||
Target Version | Fixed in Version | ITK-4-A2 | |||||||
Summary | 0011426: MetaImageWriter pixel type limitation | ||||||||
Description | I in itkImageIOBase.cxx, the Vector and CovariantVector pixel types are manually checked to be between length 2 and 7. I'm assuming this was arbitrarily chosen because of the way the pixel types are checked: ... else if ( ptype == typeid( Vector< T, 5 > ) ) { This->SetNumberOfComponents(5); This->SetPixelType(ImageIOBase::VECTOR); This->SetComponentType(ntype); return true; } ... Is there a way to do something more like "check the length of the vector, then set the number of components to the length" so we can allow images with arbitrary length vector pixels to be written? | ||||||||
Steps To Reproduce | Fixed by gerrit topic http://review.source.kitware.com/#change,353 [^] | ||||||||
Tags | No tags attached. | ||||||||
Resolution Date | 2010-11-15 | ||||||||
Sprint | |||||||||
Sprint Status | completed | ||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0023184) kentwilliams (developer) 2010-11-11 14:51 |
Short answer: No, not without a deeper revamping of the ImageIO and Pixel type system than we're likely to undertake. I'm submitting an ITKV4 patch that will allow Vector/CovariantVector/FixedArray pixel types up to a length of 12. The problem is that given the multiplicity of type, it's very difficult to form a useful template metaprogramming solution; I can't puzzle out how to discriminate between scalar types, Vector, RGB, RGBA, etc. |
(0023289) kentwilliams (developer) 2010-11-16 12:02 |
Fixed by gerrit topic http://review.source.kitware.com/#change,353 [^] |
(0023399) Luis Ibanez (manager) 2010-11-18 21:09 |
We have decided to skip the "resolved" status and move directly to "closed". |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2010-11-09 15:31 | David Doria | New Issue | |
2010-11-09 15:31 | David Doria | Sprint Status | => active |
2010-11-09 15:50 | Hans Johnson | Status | new => assigned |
2010-11-09 15:50 | Hans Johnson | Assigned To | => kentwilliams |
2010-11-11 14:51 | kentwilliams | Note Added: 0023184 | |
2010-11-15 11:36 | kentwilliams | Resolution | open => fixed |
2010-11-15 11:36 | kentwilliams | Steps to Reproduce Updated | |
2010-11-16 12:02 | kentwilliams | Note Added: 0023289 | |
2010-11-16 12:03 | kentwilliams | Resolution Date | => 2010-11-15 |
2010-11-16 12:03 | kentwilliams | Sprint Status | active => completed |
2010-11-16 12:03 | kentwilliams | Status | assigned => resolved |
2010-11-16 12:03 | kentwilliams | Fixed in Version | => ITK-4-A2 |
2010-11-18 21:09 | Luis Ibanez | Status | resolved => closed |
2010-11-18 21:09 | Luis Ibanez | Note Added: 0023399 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |