{"action":"closed","issue":{"url":"https://api.github.com/repos/aowen87/BAR/issues/78","repository_url":"https://api.github.com/repos/aowen87/BAR","labels_url":"https://api.github.com/repos/aowen87/BAR/issues/78/labels{/name}","comments_url":"https://api.github.com/repos/aowen87/BAR/issues/78/comments","events_url":"https://api.github.com/repos/aowen87/BAR/issues/78/events","html_url":"https://github.com/aowen87/BAR/issues/78","id":388863339,"node_id":"MDU6SXNzdWUzODg4NjMzMzk=","number":78,"title":"Pick requiring pipeline re-excution unnecessarily","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":[{"id":1154293279,"node_id":"MDU6TGFiZWwxMTU0MjkzMjc5","url":"https://api.github.com/repos/aowen87/BAR/labels/bug","name":"bug","color":"d73a4a","default":true},{"id":1154294815,"node_id":"MDU6TGFiZWwxMTU0Mjk0ODE1","url":"https://api.github.com/repos/aowen87/BAR/labels/likelihood%20medium","name":"likelihood medium","color":"003152","default":false},{"id":1154294660,"node_id":"MDU6TGFiZWwxMTU0Mjk0NjYw","url":"https://api.github.com/repos/aowen87/BAR/labels/reviewed","name":"reviewed","color":"008ECC","default":false},{"id":1154294769,"node_id":"MDU6TGFiZWwxMTU0Mjk0NzY5","url":"https://api.github.com/repos/aowen87/BAR/labels/severity%20low","name":"severity low","color":"0F52BA","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/aowen87/BAR/milestones/11","html_url":"https://github.com/aowen87/BAR/milestone/11","labels_url":"https://api.github.com/repos/aowen87/BAR/milestones/11/labels","id":3881826,"node_id":"MDk6TWlsZXN0b25lMzg4MTgyNg==","number":11,"title":"2.9.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":0,"closed_issues":5,"state":"open","created_at":"2018-12-08T00:27:35Z","updated_at":"2018-12-08T00:38:34Z","due_on":null,"closed_at":null},"comments":0,"created_at":"2018-12-08T00:38:30Z","updated_at":"2018-12-08T00:38:33Z","closed_at":"2018-12-08T00:38:33Z","author_association":"OWNER","body":"Here's a script I created to reproduce the issue from #2169, it uses a 'screen pick' to duplicate mousepicking. The x and y are for a \"small\" window. \r\nThe second pick causes pipeline re-execution, though nothing has changed since the first pick.\r\nOne thing to note, each 'screen pick' via cli causes a StartPickMode and StopPickMode in the engine. This can be duplicated in gui by doing navigation in-between mouse picks.\r\n\r\n<pre>\r\nOpenDatabase(\"/usr/gapps/visit/data/multi_curv2d.silo\")\r\nDefineScalarExpression(\"dandp\", \"d+p\")\r\nAddPlot(\"Pseudocolor\", \"d\")\r\n# Coord swap forces re-execution of pipeline for pick\r\nAddOperator(\"CoordSwap\")\r\ncsa = CoordSwapAttributes()\r\ncsa.newCoord1 = csa.Coord2\r\ncsa.newCoord2 = csa.Coord1\r\nSetOperatorOptions(csa)\r\nDrawPlots()\r\nResetView()\r\nPick(vars=\"dandp\", x=228, y=346)\r\n# Second pick causes another pipeline re-execution, though nothing has changed!\r\nPick(vars=\"dandp\", x=228, y=346)\r\n</pre>\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: 2183\nStatus: Resolved\nProject: VisIt\nTracker: Bug\nPriority: Normal\nSubject: Pick requiring pipeline re-excution unnecessarily\nAssigned to: Kathleen Biagas\nCategory: \nTarget version: 2.9.1\nAuthor: Kathleen Biagas\nStart: 03/18/2015\nDue date: \n% Done: 100\nEstimated time: 2.0\nCreated: 03/18/2015 07:07 pm\nUpdated: 03/20/2015 05:33 pm\nLikelihood: 3 - Occasional\nSeverity: 2 - Minor Irritation\nFound in version: 2.8.2\nImpact: \nExpected Use: \nOS: All\nSupport Group: Any\nDescription:\n Here's a script I created to reproduce the issue from #2169, it uses a 'screen pick' to duplicate mouse-picking. The x and y are for a \"-small\" window. \r\nThe second pick causes pipeline re-execution, though nothing has changed since the first pick.\r\nOne thing to note, each 'screen pick' via cli causes a StartPickMode and StopPickMode in the engine. This can be duplicated in gui by doing navigation in-between mouse picks.\r\n\r\n<pre>\r\nOpenDatabase(\"/usr/gapps/visit/data/multi_curv2d.silo\")\r\nDefineScalarExpression(\"dandp\", \"d+p\")\r\nAddPlot(\"Pseudocolor\", \"d\")\r\n# Coord swap forces re-execution of pipeline for pick\r\nAddOperator(\"CoordSwap\")\r\ncsa = CoordSwapAttributes()\r\ncsa.newCoord1 = csa.Coord2\r\ncsa.newCoord2 = csa.Coord1\r\nSetOperatorOptions(csa)\r\nDrawPlots()\r\nResetView()\r\nPick(vars=\"dandp\", x=228, y=346)\r\n# Second pick causes another pipeline re-execution, though nothing has changed!\r\nPick(vars=\"dandp\", x=228, y=346)\r\n</pre>\n\nComments:\navtBehavior::RequiredReExecutionForQuery was returning true because PointsWereTransformed (and original nodes/zones weren't present).  For the CoordSwap operator, however, originalNodes/originalZone numbers aren't required for successful picking.  So I modified avtBehavior to consider whether Zone/Nodes are preserved and zones intact (not split) when PointsWereTransformed.Now CoordSwap operator will not by itself trigger a Re-Execute when being picked.M  /src/avt/Plotter/avtBehavior.C\n\n"}}
{"actor":{"display_login":"aowen87"}}