{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/opm-autodiff/issues/542","labels_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/542/labels{/name}","comments_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/542/comments","events_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/542/events","html_url":"https://github.com/OPM/opm-autodiff/pull/542","id":118423021,"number":542,"title":"implement abililty to change the geology during the simulation run","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},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-11-23T16:51:03Z","updated_at":"2015-11-27T11:16:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/OPM/opm-autodiff/pulls/542","html_url":"https://github.com/OPM/opm-autodiff/pull/542","diff_url":"https://github.com/OPM/opm-autodiff/pull/542.diff","patch_url":"https://github.com/OPM/opm-autodiff/pull/542.patch"},"body":"i.e. it now supports stuff like MULTFLT in the schedule\r\nsection. Possibly, the MPI-parallel code paths need some fixes. (but\r\nif the geology is not changed during the simulation, the parallel code\r\nwill do the same as before.)\r\n\r\nthe most fundamental change of this patch is that the\r\nreference/pointer to the DerivedGeology object is made non-constant. IMO\r\nthat's okay, though, because the geology can no longer assumed to be\r\nconstant over the whole simulation run."},"comment":{"url":"https://api.github.com/repos/OPM/opm-autodiff/issues/comments/160117594","html_url":"https://github.com/OPM/opm-autodiff/pull/542#issuecomment-160117594","issue_url":"https://api.github.com/repos/OPM/opm-autodiff/issues/542","id":160117594,"user":{"login":"blattms","id":1944640,"avatar_url":"https://avatars.githubusercontent.com/u/1944640?v=3","gravatar_id":"","url":"https://api.github.com/users/blattms","html_url":"https://github.com/blattms","followers_url":"https://api.github.com/users/blattms/followers","following_url":"https://api.github.com/users/blattms/following{/other_user}","gists_url":"https://api.github.com/users/blattms/gists{/gist_id}","starred_url":"https://api.github.com/users/blattms/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blattms/subscriptions","organizations_url":"https://api.github.com/users/blattms/orgs","repos_url":"https://api.github.com/users/blattms/repos","events_url":"https://api.github.com/users/blattms/events{/privacy}","received_events_url":"https://api.github.com/users/blattms/received_events","type":"User","site_admin":false},"created_at":"2015-11-27T11:16:07Z","updated_at":"2015-11-27T11:16:07Z","body":"I have checked these tpfa_*_compute stuff in opm-core and to me it seems that this all uses just local information. Which means these should work in parallel.\n\nThe only thing that might not scale is [SimulatorBase_impl.hpp lines180-182 ](https://github.com/andlaus/opm-autodiff/blob/976d38bf3730ec23c374bb334ef6673355bc3b99/opm/autodiff/SimulatorBase_impl.hpp#L180-L182). Maybe @joakim_hove can tell us what happens here?\n"}}