{"action":"created","issue":{"url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/issues/3","repository_url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator","labels_url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/issues/3/comments","events_url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/issues/3/events","html_url":"https://github.com/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/pull/3","id":197334971,"number":3,"title":"Well Index Calculation for segmented wells","user":{"login":"agchitu","id":9414100,"avatar_url":"https://avatars.githubusercontent.com/u/9414100?v=3","gravatar_id":"","url":"https://api.github.com/users/agchitu","html_url":"https://github.com/agchitu","followers_url":"https://api.github.com/users/agchitu/followers","following_url":"https://api.github.com/users/agchitu/following{/other_user}","gists_url":"https://api.github.com/users/agchitu/gists{/gist_id}","starred_url":"https://api.github.com/users/agchitu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agchitu/subscriptions","organizations_url":"https://api.github.com/users/agchitu/orgs","repos_url":"https://api.github.com/users/agchitu/repos","events_url":"https://api.github.com/users/agchitu/events{/privacy}","received_events_url":"https://api.github.com/users/agchitu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"einar90","id":3439573,"avatar_url":"https://avatars.githubusercontent.com/u/3439573?v=3","gravatar_id":"","url":"https://api.github.com/users/einar90","html_url":"https://github.com/einar90","followers_url":"https://api.github.com/users/einar90/followers","following_url":"https://api.github.com/users/einar90/following{/other_user}","gists_url":"https://api.github.com/users/einar90/gists{/gist_id}","starred_url":"https://api.github.com/users/einar90/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/einar90/subscriptions","organizations_url":"https://api.github.com/users/einar90/orgs","repos_url":"https://api.github.com/users/einar90/repos","events_url":"https://api.github.com/users/einar90/events{/privacy}","received_events_url":"https://api.github.com/users/einar90/received_events","type":"User","site_admin":false},"assignees":[{"login":"einar90","id":3439573,"avatar_url":"https://avatars.githubusercontent.com/u/3439573?v=3","gravatar_id":"","url":"https://api.github.com/users/einar90","html_url":"https://github.com/einar90","followers_url":"https://api.github.com/users/einar90/followers","following_url":"https://api.github.com/users/einar90/following{/other_user}","gists_url":"https://api.github.com/users/einar90/gists{/gist_id}","starred_url":"https://api.github.com/users/einar90/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/einar90/subscriptions","organizations_url":"https://api.github.com/users/einar90/orgs","repos_url":"https://api.github.com/users/einar90/repos","events_url":"https://api.github.com/users/einar90/events{/privacy}","received_events_url":"https://api.github.com/users/einar90/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2016-12-23T09:09:41Z","updated_at":"2016-12-28T11:58:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/pulls/3","html_url":"https://github.com/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/pull/3","diff_url":"https://github.com/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/pull/3.diff","patch_url":"https://github.com/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/pull/3.patch"},"body":"This version extends calculation of well index to segmented wells. \r\nA well is defined as a list of well segments each segment being defined by its heel, toe and radius.\r\n\r\nThe main application will accept the argument --well-filedef <path_to_file> and load the well definition from the file. This is an ASCII file with one segment defined  per line: well_name hx hy hz tx ty tz radius\r\nWell_name\t995.00\t995.00\t2500.00\t995.00\t995.00\t2600.00\t0.076200\r\nWell_name\t994.92\t995.00\t2533.00\t895.00\t995.00\t2533.00\t0.025400\r\nIt is possible to define multiple wells in the same file.\r\n\r\nA new argument --debug is allowed. The tool will print in the file debug.dat all projection related data (lx, ly, lz, permx, .....)\r\n\r\nThe path to the grid file is required.\r\n \r\nSearching for intersected grid block will be restricted to a bounding box defined around the heel  and toe. This is done as a simple first try to speed up this process.\r\n\r\n(disclaimer: some family related issues made working this rather discontinuous - this is also the reason I am dumping so many commits in one pull request)"},"comment":{"url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/issues/comments/269467724","html_url":"https://github.com/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/pull/3#issuecomment-269467724","issue_url":"https://api.github.com/repos/PetroleumCyberneticsGroup/FieldOpt-WellIndexCalculator/issues/3","id":269467724,"user":{"login":"einar90","id":3439573,"avatar_url":"https://avatars.githubusercontent.com/u/3439573?v=3","gravatar_id":"","url":"https://api.github.com/users/einar90","html_url":"https://github.com/einar90","followers_url":"https://api.github.com/users/einar90/followers","following_url":"https://api.github.com/users/einar90/following{/other_user}","gists_url":"https://api.github.com/users/einar90/gists{/gist_id}","starred_url":"https://api.github.com/users/einar90/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/einar90/subscriptions","organizations_url":"https://api.github.com/users/einar90/orgs","repos_url":"https://api.github.com/users/einar90/repos","events_url":"https://api.github.com/users/einar90/events{/privacy}","received_events_url":"https://api.github.com/users/einar90/received_events","type":"User","site_admin":false},"created_at":"2016-12-28T11:58:54Z","updated_at":"2016-12-28T11:58:54Z","body":"This looks great, good work! I wont have time to look thoroughly at this (or merge it) until early January because of a course I have to finish.\r\n\r\nBut I would like some unit tests (in the `./tests` directory) for the new non-trivial methods in the `IntersectedCell` class. If you're not familiar with the Google C++ testing framework, there's a primer [here](https://github.com/google/googletest/blob/master/googletest/docs/Primer.md). If you don't want to spend time doing this that's okay too, just add some comments in the header file (or list it in a comment here), stating the expected input and output for the functions, and I'll take care of writing the tests."}}
{"actor":{"display_login":"einar90"}}