View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0007583 | ParaView | (No Category) | public | 2008-08-29 09:42 | 2008-09-16 17:11 | ||||
Reporter | Takuya OSHIMA | ||||||||
Assigned To | Utkarsh Ayachit | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0007583: Server->client downlink of StringVectorProperty with ArraySelectionDomain broken | ||||||||
Description | When a reader tries to add an array name to the selection list in RequestData() or RequestInformation() after initializing the list at the first RequestInformation() call, a message "had to make up a value for selection" appears and the selection status is always shown as unselected in the UI selection list. | ||||||||
Additional Information | The problem is closely related to a fix to another problem done at revision 1.7 of pqSignalAdaptorSelectionTreeWidget.cxx, which removed a this->Internal->Property->ResetToDefault() call at line 190. Before the removal the ResetToDefault() call subsequently executed vtkSMArraySelectionDomain::SetDefaultValues() which copies the contents of associated information property to the StringVectorProperty. However with the removal now there's nothing which links the information property and the StringVector property, despite until the ArraySelectionDomain which interfaces the properties is updated properly. Hence the StringDomain of the newly added array name is found in pqSMAdaptor::getSelectionProperty(vtkSMProperty*, unsigned int) while the corresponding StringVectorProperty is not, which led to the "had to make up a value for selection" text output. I think the fix intended by the removal of ResetToDefault() must be done elsewhere. I attach a pseudo reader plugin to demonstrate the problem. Just build as a usual plugin, load into ParaView, and try reading any stub file with the *.pseudo extension. After clicking Apply, you'll see the "had to make up..." message every time you move the timestep back and forth. | ||||||||
Tags | No tags attached. | ||||||||
Project | |||||||||
Topic Name | |||||||||
Type | |||||||||
Attached Files | pseudoReader.tar.gz [^] (1,979 bytes) 2008-08-29 09:42 | ||||||||
Relationships | |
Relationships |
Notes | |
(0013307) Utkarsh Ayachit (administrator) 2008-09-05 14:44 |
For array status check the information properties before making up values. /cvsroot/ParaView3/ParaView3/Qt/Core/pqSMAdaptor.cxx,v <-- Qt/Core/pqSMAdaptor.cxx new revision: 1.42; previous revision: 1.41 |
(0013483) Alan Scott (manager) 2008-09-16 17:11 |
Closed without testing. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2008-08-29 09:42 | Takuya OSHIMA | New Issue | |
2008-08-29 09:42 | Takuya OSHIMA | File Added: pseudoReader.tar.gz | |
2008-09-05 14:44 | Utkarsh Ayachit | Status | backlog => @80@ |
2008-09-05 14:44 | Utkarsh Ayachit | Resolution | open => fixed |
2008-09-05 14:44 | Utkarsh Ayachit | Assigned To | => Utkarsh Ayachit |
2008-09-05 14:44 | Utkarsh Ayachit | Note Added: 0013307 | |
2008-09-16 17:11 | Alan Scott | Status | @80@ => closed |
2008-09-16 17:11 | Alan Scott | Note Added: 0013483 | |
2011-06-16 13:10 | Zack Galbreath | Category | => (No Category) |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |