{"action":"created","issue":{"url":"https://api.github.com/repos/opensim-org/opensim-core/issues/927","repository_url":"https://api.github.com/repos/opensim-org/opensim-core","labels_url":"https://api.github.com/repos/opensim-org/opensim-core/issues/927/labels{/name}","comments_url":"https://api.github.com/repos/opensim-org/opensim-core/issues/927/comments","events_url":"https://api.github.com/repos/opensim-org/opensim-core/issues/927/events","html_url":"https://github.com/opensim-org/opensim-core/pull/927","id":149017055,"number":927,"title":"Wrap Reporters and TableSource; move Component wrapper back to Common","user":{"login":"chrisdembia","id":846001,"avatar_url":"https://avatars.githubusercontent.com/u/846001?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisdembia","html_url":"https://github.com/chrisdembia","followers_url":"https://api.github.com/users/chrisdembia/followers","following_url":"https://api.github.com/users/chrisdembia/following{/other_user}","gists_url":"https://api.github.com/users/chrisdembia/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisdembia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisdembia/subscriptions","organizations_url":"https://api.github.com/users/chrisdembia/orgs","repos_url":"https://api.github.com/users/chrisdembia/repos","events_url":"https://api.github.com/users/chrisdembia/events{/privacy}","received_events_url":"https://api.github.com/users/chrisdembia/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-04-18T01:30:38Z","updated_at":"2016-06-21T05:22:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/opensim-org/opensim-core/pulls/927","html_url":"https://github.com/opensim-org/opensim-core/pull/927","diff_url":"https://github.com/opensim-org/opensim-core/pull/927.diff","patch_url":"https://github.com/opensim-org/opensim-core/pull/927.patch"},"body":"This is an attempt to wrap the new reporter classes (TableReporter, ConsoleReporter, StatesTrajectoryReporter). I ran into a problem with the fact that `Component.h` is now wrapped in `simulation.i` instead of `common.i` (the Reporter.h is in Common): the Reporter classes do not have the methods they inherit from Component, Object, etc. I understand why the header was moved, but now that we have subclasses of Component in Common, it may be good to find a way to move Component.h back to `common.i`. \r\n\r\n@klshrinidhi, @aymanhab, any ideas? It is as if we need to \"forward declare\" Frame, etc. Perhaps the `getFrameList()`, etc. methods can be on Model instead of Component?"},"comment":{"url":"https://api.github.com/repos/opensim-org/opensim-core/issues/comments/227343428","html_url":"https://github.com/opensim-org/opensim-core/pull/927#issuecomment-227343428","issue_url":"https://api.github.com/repos/opensim-org/opensim-core/issues/927","id":227343428,"user":{"login":"chrisdembia","id":846001,"avatar_url":"https://avatars.githubusercontent.com/u/846001?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisdembia","html_url":"https://github.com/chrisdembia","followers_url":"https://api.github.com/users/chrisdembia/followers","following_url":"https://api.github.com/users/chrisdembia/following{/other_user}","gists_url":"https://api.github.com/users/chrisdembia/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisdembia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisdembia/subscriptions","organizations_url":"https://api.github.com/users/chrisdembia/orgs","repos_url":"https://api.github.com/users/chrisdembia/repos","events_url":"https://api.github.com/users/chrisdembia/events{/privacy}","received_events_url":"https://api.github.com/users/chrisdembia/received_events","type":"User","site_admin":false},"created_at":"2016-06-21T05:22:53Z","updated_at":"2016-06-21T05:22:53Z","body":"Builds now pass. Please review when you get the chance."}}
{"actor":{"display_login":"chrisdembia"}}