{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/opm-simulators/issues/737","repository_url":"https://api.github.com/repos/OPM/opm-simulators","labels_url":"https://api.github.com/repos/OPM/opm-simulators/issues/737/labels{/name}","comments_url":"https://api.github.com/repos/OPM/opm-simulators/issues/737/comments","events_url":"https://api.github.com/repos/OPM/opm-simulators/issues/737/events","html_url":"https://github.com/OPM/opm-simulators/pull/737","id":161703738,"number":737,"title":"Experimental sequential solvers","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,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-06-22T14:50:19Z","updated_at":"2016-06-24T13:05:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/OPM/opm-simulators/pulls/737","html_url":"https://github.com/OPM/opm-simulators/pull/737","diff_url":"https://github.com/OPM/opm-simulators/pull/737.diff","patch_url":"https://github.com/OPM/opm-simulators/pull/737.patch"},"body":"This PR adds sequential solvers, including a simulator variant using them (flow_sequential.cpp) with an integration test (running SPE1, same as for fully implicit).\r\n\r\nThe sequential code is capable of running several (but not all) test cases without tuning or special parameters, but reducing ds_max a bit (from default 0.2 to say 0.1) helps with transport solver convergence. The Norne model runs fine (esp. with a little tuning). A parameter iterate_to_fully_implicit (defaults to false) is available, when set the simulator will iterate with alternating pressure and transport solves towards the fully implicit solution. Although that takes a lot extra time it serves as a correctness check.\r\n\r\nPerformance is not competitive with fully implicit at this point: essentially both the pressure and transport models inherit the fully implicit model and do a lot of double (or triple) work. The point has been to establish a proof of concept and baseline for further experiments, without disturbing the base model too much (or at all, if possible).\r\n\r\nChanges to existing code has been minimized by merging most such changes as smaller PRs already, the only remaining such change is to NewtonIterationBlackoilInterleaved. Admittedly, that code (to solve the pressure system with AMG) is not ideal because it duplicates similar code in CPRPreconditioner.hpp and is not parallel. I propose to address this later by refactoring the \"solve elliptic system\" code from CPRPreconditioner into a separate class that can be used also from here.\r\n"},"comment":{"url":"https://api.github.com/repos/OPM/opm-simulators/issues/comments/228339943","html_url":"https://github.com/OPM/opm-simulators/pull/737#issuecomment-228339943","issue_url":"https://api.github.com/repos/OPM/opm-simulators/issues/737","id":228339943,"user":{"login":"bska","id":1871438,"avatar_url":"https://avatars.githubusercontent.com/u/1871438?v=3","gravatar_id":"","url":"https://api.github.com/users/bska","html_url":"https://github.com/bska","followers_url":"https://api.github.com/users/bska/followers","following_url":"https://api.github.com/users/bska/following{/other_user}","gists_url":"https://api.github.com/users/bska/gists{/gist_id}","starred_url":"https://api.github.com/users/bska/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bska/subscriptions","organizations_url":"https://api.github.com/users/bska/orgs","repos_url":"https://api.github.com/users/bska/repos","events_url":"https://api.github.com/users/bska/events{/privacy}","received_events_url":"https://api.github.com/users/bska/received_events","type":"User","site_admin":false},"created_at":"2016-06-24T13:05:06Z","updated_at":"2016-06-24T13:05:06Z","body":"```\r\n/home/travis/build/OPM/opm-simulators/opm/autodiff/BlackoilPressureModel.hpp:79:24: error: invalid initializer for array member ‘Opm::ADB Opm::BlackoilPressureModel<UnstructuredGrid, Opm::StandardWells>::scaling_ [3]’\r\n              state0_(3)\r\n                       ^\r\n```"}}
{"actor":{"display_login":"bska"}}