{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/opm-autodiff/issues/320","labels_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/320/labels{/name}","comments_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/320/comments","events_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/320/events","html_url":"https://github.com/OPM/opm-autodiff/pull/320","id":59066575,"number":320,"title":"Fix pvt regions","user":{"login":"atgeirr","id":1876621,"avatar_url":"https://avatars.githubusercontent.com/u/1876621?v=3","gravatar_id":"","url":"https://api.github.com/users/atgeirr","html_url":"https://github.com/atgeirr","followers_url":"https://api.github.com/users/atgeirr/followers","following_url":"https://api.github.com/users/atgeirr/following{/other_user}","gists_url":"https://api.github.com/users/atgeirr/gists{/gist_id}","starred_url":"https://api.github.com/users/atgeirr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atgeirr/subscriptions","organizations_url":"https://api.github.com/users/atgeirr/orgs","repos_url":"https://api.github.com/users/atgeirr/repos","events_url":"https://api.github.com/users/atgeirr/events{/privacy}","received_events_url":"https://api.github.com/users/atgeirr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-26T12:39:23Z","updated_at":"2015-02-26T14:11:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/OPM/opm-autodiff/pulls/320","html_url":"https://github.com/OPM/opm-autodiff/pull/320","diff_url":"https://github.com/OPM/opm-autodiff/pull/320.diff","patch_url":"https://github.com/OPM/opm-autodiff/pull/320.patch"},"body":"This PR is based on #318 and therefore at the moment it is hard to tell apart the additional content in this, I may rebase it if #318 is merged to improve readability.\r\n\r\nIt contains two basic fixes:\r\n - The `pvtTableIdx_` member is removed from `BlackoilPropsAdFromDeck`, and a bug fixed (initialisation of `cellPvtRegionIdx_`, the `pvtTableIdx_` received the data instead).\r\n - Make the `test_rateconverter` test succeed again.\r\n\r\nEdit: I forgot to list the most important fix in this PR:\r\n - Fix pvt region behaviour. The existing code did not pass an array of regions (with the same size as the requested number of data points) to underlying classes, but the array of cell's region indices."},"comment":{"url":"https://api.github.com/repos/OPM/opm-autodiff/issues/comments/76183924","html_url":"https://github.com/OPM/opm-autodiff/pull/320#issuecomment-76183924","issue_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/320","id":76183924,"user":{"login":"andlaus","id":1837735,"avatar_url":"https://avatars.githubusercontent.com/u/1837735?v=3","gravatar_id":"","url":"https://api.github.com/users/andlaus","html_url":"https://github.com/andlaus","followers_url":"https://api.github.com/users/andlaus/followers","following_url":"https://api.github.com/users/andlaus/following{/other_user}","gists_url":"https://api.github.com/users/andlaus/gists{/gist_id}","starred_url":"https://api.github.com/users/andlaus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andlaus/subscriptions","organizations_url":"https://api.github.com/users/andlaus/orgs","repos_url":"https://api.github.com/users/andlaus/repos","events_url":"https://api.github.com/users/andlaus/events{/privacy}","received_events_url":"https://api.github.com/users/andlaus/received_events","type":"User","site_admin":false},"created_at":"2015-02-26T14:11:39Z","updated_at":"2015-02-26T14:11:39Z","body":"forget my last comment. I should have read your description before starting to type. sorry for the noise..."}}