{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/opm-simulators/issues/772","repository_url":"https://api.github.com/repos/OPM/opm-simulators","labels_url":"https://api.github.com/repos/OPM/opm-simulators/issues/772/labels{/name}","comments_url":"https://api.github.com/repos/OPM/opm-simulators/issues/772/comments","events_url":"https://api.github.com/repos/OPM/opm-simulators/issues/772/events","html_url":"https://github.com/OPM/opm-simulators/issues/772","id":167378060,"number":772,"title":"Parallel Output relies on order of an unordered_map.","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},"labels":[{"url":"https://api.github.com/repos/OPM/opm-simulators/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-07-25T14:21:27Z","updated_at":"2016-07-25T20:23:44Z","closed_at":null,"body":"I experienced serious differences in the output of data of serial and parallel runs. It turns out that this even occurs for the initial data.\r\nIt turns out that when gathering data on one process the code in `PackUnPackSimulationDataContainer` holds a `state`, and `local_state` object of Type `SimulationDataContainer` and assumes that the keys are traversed in the same order when iterating over the `cellData()` members of both object. Apparently, this assumption does not hold and the `GASOILRATIO` and `SURFACEVOL` data get mixed up."},"comment":{"url":"https://api.github.com/repos/OPM/opm-simulators/issues/comments/235073036","html_url":"https://github.com/OPM/opm-simulators/issues/772#issuecomment-235073036","issue_url":"https://api.github.com/repos/OPM/opm-simulators/issues/772","id":235073036,"user":{"login":"joakim-hove","id":1321665,"avatar_url":"https://avatars.githubusercontent.com/u/1321665?v=3","gravatar_id":"","url":"https://api.github.com/users/joakim-hove","html_url":"https://github.com/joakim-hove","followers_url":"https://api.github.com/users/joakim-hove/followers","following_url":"https://api.github.com/users/joakim-hove/following{/other_user}","gists_url":"https://api.github.com/users/joakim-hove/gists{/gist_id}","starred_url":"https://api.github.com/users/joakim-hove/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joakim-hove/subscriptions","organizations_url":"https://api.github.com/users/joakim-hove/orgs","repos_url":"https://api.github.com/users/joakim-hove/repos","events_url":"https://api.github.com/users/joakim-hove/events{/privacy}","received_events_url":"https://api.github.com/users/joakim-hove/received_events","type":"User","site_admin":false},"created_at":"2016-07-25T20:23:44Z","updated_at":"2016-07-25T20:23:44Z","body":"The `SimulationDataContainer` class was an attempt to make  a container class to hold and exchange data which should work well both for the simulator code and the output layer, that is also the reason it is located in opm-common.\r\n\r\nTo satisfy both output and simulators many compromises were, and probably the approach was  doomed from the outset due to the strong induced coupling. Anyway - the end result was that the output layer does not employ this datastructure at all, and constraints and requirements are only those induced by the simulator - not really my call but I guess moving it to simulators would be a natural step."}}
{"actor":{"display_login":"joakim-hove"}}