{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/opm-core/issues/898","labels_url":"https://api.github.com/repos/OPM/opm-core/issues/898/labels{/name}","comments_url":"https://api.github.com/repos/OPM/opm-core/issues/898/comments","events_url":"https://api.github.com/repos/OPM/opm-core/issues/898/events","html_url":"https://github.com/OPM/opm-core/issues/898","id":110400827,"number":898,"title":"ACTNUM with cartesian grid ","user":{"login":"qilicun","id":2907423,"avatar_url":"https://avatars.githubusercontent.com/u/2907423?v=3","gravatar_id":"","url":"https://api.github.com/users/qilicun","html_url":"https://github.com/qilicun","followers_url":"https://api.github.com/users/qilicun/followers","following_url":"https://api.github.com/users/qilicun/following{/other_user}","gists_url":"https://api.github.com/users/qilicun/gists{/gist_id}","starred_url":"https://api.github.com/users/qilicun/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/qilicun/subscriptions","organizations_url":"https://api.github.com/users/qilicun/orgs","repos_url":"https://api.github.com/users/qilicun/repos","events_url":"https://api.github.com/users/qilicun/events{/privacy}","received_events_url":"https://api.github.com/users/qilicun/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-08T08:52:25Z","updated_at":"2015-10-08T09:33:20Z","closed_at":null,"body":"Hi,\r\n\r\nI'm making the tests for pinch processor, I suddenly found that iopm will not creat ACTNUM for cartesian grid, that means MinvpProcessor will not work also. Since this array will be empty, the following error will happen when we specify MINPV keyword together cartesian grid.\r\n```\r\nproject/eor/user/liuming/opm/opm-core/opm/core/grid/MinpvProcessor.hpp:96] Wrong size of ACTNUM input, must have one element per logical cartesian cel\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/OPM/opm-core/issues/comments/146471981","html_url":"https://github.com/OPM/opm-core/issues/898#issuecomment-146471981","issue_url":"https://api.github.com/repos/OPM/opm-core/issues/898","id":146471981,"user":{"login":"qilicun","id":2907423,"avatar_url":"https://avatars.githubusercontent.com/u/2907423?v=3","gravatar_id":"","url":"https://api.github.com/users/qilicun","html_url":"https://github.com/qilicun","followers_url":"https://api.github.com/users/qilicun/followers","following_url":"https://api.github.com/users/qilicun/following{/other_user}","gists_url":"https://api.github.com/users/qilicun/gists{/gist_id}","starred_url":"https://api.github.com/users/qilicun/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/qilicun/subscriptions","organizations_url":"https://api.github.com/users/qilicun/orgs","repos_url":"https://api.github.com/users/qilicun/repos","events_url":"https://api.github.com/users/qilicun/events{/privacy}","received_events_url":"https://api.github.com/users/qilicun/received_events","type":"User","site_admin":false},"created_at":"2015-10-08T09:33:19Z","updated_at":"2015-10-08T09:33:19Z","body":"@bska \r\nI think your modification will work, will test later. I'm thinking the same modification needs to be made for PinchProcessor.\r\n\r\n>We used to treat an empty ACTNUM array as \"all cells active\".\r\n\r\nThis hidden assumption will cause confusion for some people like me. Is there any possibility to add warnings when some codes needs to access to this array with cartesian grid? "}}