{"action":"opened","issue":{"url":"https://api.github.com/repos/aowen87/BAR/issues/1890","repository_url":"https://api.github.com/repos/aowen87/BAR","labels_url":"https://api.github.com/repos/aowen87/BAR/issues/1890/labels{/name}","comments_url":"https://api.github.com/repos/aowen87/BAR/issues/1890/comments","events_url":"https://api.github.com/repos/aowen87/BAR/issues/1890/events","html_url":"https://github.com/aowen87/BAR/issues/1890","id":388905869,"node_id":"MDU6SXNzdWUzODg5MDU4Njk=","number":1890,"title":"Large operatorcreatedvariable leak in the viewer.","user":{"login":"aowen87","id":11658578,"node_id":"MDQ6VXNlcjExNjU4NTc4","avatar_url":"https://avatars0.githubusercontent.com/u/11658578?v=4","gravatar_id":"","url":"https://api.github.com/users/aowen87","html_url":"https://github.com/aowen87","followers_url":"https://api.github.com/users/aowen87/followers","following_url":"https://api.github.com/users/aowen87/following{/other_user}","gists_url":"https://api.github.com/users/aowen87/gists{/gist_id}","starred_url":"https://api.github.com/users/aowen87/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aowen87/subscriptions","organizations_url":"https://api.github.com/users/aowen87/orgs","repos_url":"https://api.github.com/users/aowen87/repos","events_url":"https://api.github.com/users/aowen87/events{/privacy}","received_events_url":"https://api.github.com/users/aowen87/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/aowen87/BAR/milestones/18","html_url":"https://github.com/aowen87/BAR/milestone/18","labels_url":"https://api.github.com/repos/aowen87/BAR/milestones/18/labels","id":3881833,"node_id":"MDk6TWlsZXN0b25lMzg4MTgzMw==","number":18,"title":"2.7.1","description":null,"creator":{"login":"aowen87","id":11658578,"node_id":"MDQ6VXNlcjExNjU4NTc4","avatar_url":"https://avatars0.githubusercontent.com/u/11658578?v=4","gravatar_id":"","url":"https://api.github.com/users/aowen87","html_url":"https://github.com/aowen87","followers_url":"https://api.github.com/users/aowen87/followers","following_url":"https://api.github.com/users/aowen87/following{/other_user}","gists_url":"https://api.github.com/users/aowen87/gists{/gist_id}","starred_url":"https://api.github.com/users/aowen87/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aowen87/subscriptions","organizations_url":"https://api.github.com/users/aowen87/orgs","repos_url":"https://api.github.com/users/aowen87/repos","events_url":"https://api.github.com/users/aowen87/events{/privacy}","received_events_url":"https://api.github.com/users/aowen87/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":21,"state":"open","created_at":"2018-12-08T00:27:58Z","updated_at":"2018-12-08T09:43:51Z","due_on":null,"closed_at":null},"comments":0,"created_at":"2018-12-08T09:43:51Z","updated_at":"2018-12-08T09:43:51Z","closed_at":null,"author_association":"OWNER","body":"I ran valgrind on the viewer and found a large leak with the ExpressionList objects created by operator plugins (look for GetCreatedExpressions in the viewer). Nearly all of the cases where that method is called in the viewer results in memory leaks. A bunch of operators create expressions now and for large variable lists this leaks several megabytes. This would probably eventually take down the viewer when playing through animations of datasets that have time-varying metadata.\n\n\n\n\n\n-----------------------REDMINE MIGRATION-----------------------\nThis ticket was migrated from Redmine. As such, not all \ninformation was able to be captured in the transition. Below is\na complete record of the original redmine ticket.\n\nTicket number: 1657\nStatus: Resolved\nProject: VisIt\nTracker: Bug\nPriority: Normal\nSubject: Large operator-created-variable leak in the viewer.\nAssigned to: Kathleen Biagas\nCategory: \nTarget version: 2.7.1\nAuthor: Brad Whitlock\nStart: 11/14/2013\nDue date: \n% Done: 0\nEstimated time: \nCreated: 11/14/2013 04:58 pm\nUpdated: 11/20/2013 05:05 pm\nLikelihood: 3 - Occasional\nSeverity: 2 - Minor Irritation\nFound in version: 2.7.0\nImpact: \nExpected Use: \nOS: All\nSupport Group: Any\nDescription:\n I ran valgrind on the viewer and found a large leak with the ExpressionList objects created by operator plugins (look for GetCreatedExpressions in the viewer). Nearly all of the cases where that method is called in the viewer results in memory leaks. A bunch of operators create expressions now and for large variable lists this leaks several megabytes. This would probably eventually take down the viewer when playing through animations of datasets that have time-varying metadata.\n\nComments:\nThe operator's info GetCreatedExpressions method creates a 'new' expression list that gets returned, making it the Caller's responsibility to delete the expression list, but this was not occurring.  Fixed now.Also fixed some logic auto-generated code in the GetCreatedExpressions that could lead to code that did nothing.SVN Revision 22392 (2.7RC) 22394 (trunk)M       /src/tools/xml/GenerateInfo.hM       /src/viewer/main/ViewerPlotList.CM       /src/viewer/main/ViewerPlot.CM       /src/operators/CracksClipper/CracksClipper.codeM       /src/operators/CracksClipper/CracksClipperCommonPluginInfo.CM       /src/operators/SurfaceNormal/SurfaceNormalCommonPluginInfo.CM       /src/operators/ConnectedComponents/ConnectedComponentsCommonPluginInfo.CM       /src/operators/IntegralCurve/IntegralCurve.codeM       /src/operators/IntegralCurve/IntegralCurveViewerPluginInfo.CM       /src/operators/LCS/LCSViewerPluginInfo.CM       /src/operators/LCS/LCSAttributes.codeM       /src/operators/ModelFit/ModelFitAttributes.codeM       /src/operators/ModelFit/ModelFitCommonPluginInfo.CM       /src/operators/ChannelComm/ChannelCommCommonPluginInfo.CM       /src/operators/Flux/FluxCommonPluginInfo.C\n\n"}}
{"actor":{"display_login":"aowen87"}}