{"action":"created","issue":{"url":"https://api.github.com/repos/santoshphilip/eppy/issues/62","repository_url":"https://api.github.com/repos/santoshphilip/eppy","labels_url":"https://api.github.com/repos/santoshphilip/eppy/issues/62/labels{/name}","comments_url":"https://api.github.com/repos/santoshphilip/eppy/issues/62/comments","events_url":"https://api.github.com/repos/santoshphilip/eppy/issues/62/events","html_url":"https://github.com/santoshphilip/eppy/issues/62","id":97561611,"number":62,"title":"function to run E+ from eppy","user":{"login":"santoshphilip","id":2014885,"avatar_url":"https://avatars.githubusercontent.com/u/2014885?v=3","gravatar_id":"","url":"https://api.github.com/users/santoshphilip","html_url":"https://github.com/santoshphilip","followers_url":"https://api.github.com/users/santoshphilip/followers","following_url":"https://api.github.com/users/santoshphilip/following{/other_user}","gists_url":"https://api.github.com/users/santoshphilip/gists{/gist_id}","starred_url":"https://api.github.com/users/santoshphilip/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/santoshphilip/subscriptions","organizations_url":"https://api.github.com/users/santoshphilip/orgs","repos_url":"https://api.github.com/users/santoshphilip/repos","events_url":"https://api.github.com/users/santoshphilip/events{/privacy}","received_events_url":"https://api.github.com/users/santoshphilip/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/santoshphilip/eppy/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"jamiebull1","id":3817160,"avatar_url":"https://avatars.githubusercontent.com/u/3817160?v=3","gravatar_id":"","url":"https://api.github.com/users/jamiebull1","html_url":"https://github.com/jamiebull1","followers_url":"https://api.github.com/users/jamiebull1/followers","following_url":"https://api.github.com/users/jamiebull1/following{/other_user}","gists_url":"https://api.github.com/users/jamiebull1/gists{/gist_id}","starred_url":"https://api.github.com/users/jamiebull1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamiebull1/subscriptions","organizations_url":"https://api.github.com/users/jamiebull1/orgs","repos_url":"https://api.github.com/users/jamiebull1/repos","events_url":"https://api.github.com/users/jamiebull1/events{/privacy}","received_events_url":"https://api.github.com/users/jamiebull1/received_events","type":"User","site_admin":false},"milestone":null,"comments":40,"created_at":"2015-07-27T21:49:21Z","updated_at":"2016-02-14T01:39:04Z","closed_at":null,"body":"#### function to run E+ from eppy\r\n\r\nSee implementation details in this [link](https://github.com/santoshphilip/eppy/issues/59#issuecomment-125295658)\r\n\r\nfull discussion of the issue is [here](https://github.com/santoshphilip/eppy/issues/59)\r\n\r\ndevelopment will be done in branch `i62_eplusrunner`\r\n\r\nJamie,\r\ndo `git checkout -b i62_eplusrunner origin/i62_eplusrunner ` to checkout this branch\r\n"},"comment":{"url":"https://api.github.com/repos/santoshphilip/eppy/issues/comments/183790728","html_url":"https://github.com/santoshphilip/eppy/issues/62#issuecomment-183790728","issue_url":"https://api.github.com/repos/santoshphilip/eppy/issues/62","id":183790728,"user":{"login":"santoshphilip","id":2014885,"avatar_url":"https://avatars.githubusercontent.com/u/2014885?v=3","gravatar_id":"","url":"https://api.github.com/users/santoshphilip","html_url":"https://github.com/santoshphilip","followers_url":"https://api.github.com/users/santoshphilip/followers","following_url":"https://api.github.com/users/santoshphilip/following{/other_user}","gists_url":"https://api.github.com/users/santoshphilip/gists{/gist_id}","starred_url":"https://api.github.com/users/santoshphilip/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/santoshphilip/subscriptions","organizations_url":"https://api.github.com/users/santoshphilip/orgs","repos_url":"https://api.github.com/users/santoshphilip/repos","events_url":"https://api.github.com/users/santoshphilip/events{/privacy}","received_events_url":"https://api.github.com/users/santoshphilip/received_events","type":"User","site_admin":false},"created_at":"2016-02-14T01:39:04Z","updated_at":"2016-02-14T01:39:04Z","body":"I stated earlier:\r\n>> In effect IDF0 -> IDF1 -> IDF2 -> follows the chronological development of the class.\r\n\r\nThis leads to the conclusion that the code would have greater clarity if it was a single class `IDF` that has all the functionality.\r\n\r\nAnother approach would be to start with and base class and to create a series of child classes, each of which has greater functionality that is obvious from the class name. The classic example that is usually waved around is shown below. It comes from computer graphics\r\n\r\n    #  much pseudocode here\r\n    class basicpoint(object) \r\n        x\r\n        y\r\n    \r\n    class coloredpoint(point)  \r\n        # rgb\r\n        red\r\n        green\r\n        blue\r\n\r\n    class drawpoint(coloredpoint)\r\n        def draw_thepoint()\r\n            # some code that draws the point on the screen in color\r\n        \r\nI think  IDF0 -> IDF1 -> IDF2 has such a logic to it. Especially if I give them more meaningful names and shuffle the code around a bit. In fact that was my original thinking behind it. I did not give them meaningful names at that time, because I did not know what I was doing until I did it :-)\r\n\r\nThe proof is in the pudding. So I need to look at the code to see if such a direction is viable and will actually add clarity. If the code does not look amenable to such a modification, we can just make a single class and be done with it.       "}}