View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0007583ParaView(No Category)public2008-08-29 09:422008-09-16 17:11
ReporterTakuya OSHIMA 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0007583: Server->client downlink of StringVectorProperty with ArraySelectionDomain broken
DescriptionWhen 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 InformationThe 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.
TagsNo tags attached.
Project
Topic Name
Type
Attached Filesgz file icon pseudoReader.tar.gz [^] (1,979 bytes) 2008-08-29 09:42

 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.

 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)


Copyright © 2000 - 2018 MantisBT Team