MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015537ParaView(No Category)public2015-06-11 20:552015-09-06 12:17
ReporterAlan Scott 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Versiongit-master 
Target Version4.4Fixed in Version4.4 
ProjectSandia
Topic Name
Typeincorrect functionality
Summary0015537: Incorrect warnings in ProgressHandler
DescriptionThere is a vtkWarningMacro in vtkPVProgressHandler.cxx that should either be commented out for release, commented out totally or deleted. Since I am not sure the best way to go, I will just write up this bug report.

Issue is in ParaViewCore/ClientServerCore/Core/vtkPVProgressHandler.cxx line 260. It ls line:
    vtkWarningMacro(<< caller->GetClassName() << " reported invalid progress ("
      << progress << "). Value must be between [0, 1]. Clamping to this range.");

Please make sure it never prints for release. There is a mistake elsewhere (in the CTH code) that is putting out a huge number of these calls and prints.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-06-11 20:55Alan ScottNew Issue
2015-06-11 20:56Alan ScottTarget Version => 4.4
2015-07-08 16:36Utkarsh AyachitStatusbacklog => todo
2015-07-09 11:00Utkarsh AyachitNote Added: 0034717
2015-07-09 11:00Utkarsh AyachitStatustodo => customer review
2015-07-09 11:00Utkarsh AyachitFixed in Version => git-next
2015-07-09 11:00Utkarsh AyachitResolutionopen => fixed
2015-07-09 11:00Utkarsh AyachitAssigned To => Utkarsh Ayachit
2015-07-10 20:43Alan ScottNote Added: 0034753
2015-07-10 20:43Alan ScottStatuscustomer review => closed
2015-07-12 20:48Utkarsh AyachitFixed in Versiongit-next => git-master
2015-09-06 12:17Utkarsh AyachitFixed in Versiongit-master => 4.4

Notes
(0034717)
Utkarsh Ayachit   
2015-07-09 11:00   
Merge request:

https://gitlab.kitware.com/paraview/paraview/merge_requests/202 [^]
(0034753)
Alan Scott   
2015-07-10 20:43   
Closing untested. I will be testing the code end to end, optimized, before release. Thus, this will also get tested then.