{"action":"created","issue":{"url":"https://api.github.com/repos/bigartm/bigartm/issues/489","repository_url":"https://api.github.com/repos/bigartm/bigartm","labels_url":"https://api.github.com/repos/bigartm/bigartm/issues/489/labels{/name}","comments_url":"https://api.github.com/repos/bigartm/bigartm/issues/489/comments","events_url":"https://api.github.com/repos/bigartm/bigartm/issues/489/events","html_url":"https://github.com/bigartm/bigartm/issues/489","id":141432807,"number":489,"title":"Review python API and consider changes for v0.8.0","user":{"login":"sashafrey","id":1252606,"avatar_url":"https://avatars.githubusercontent.com/u/1252606?v=3","gravatar_id":"","url":"https://api.github.com/users/sashafrey","html_url":"https://github.com/sashafrey","followers_url":"https://api.github.com/users/sashafrey/followers","following_url":"https://api.github.com/users/sashafrey/following{/other_user}","gists_url":"https://api.github.com/users/sashafrey/gists{/gist_id}","starred_url":"https://api.github.com/users/sashafrey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sashafrey/subscriptions","organizations_url":"https://api.github.com/users/sashafrey/orgs","repos_url":"https://api.github.com/users/sashafrey/repos","events_url":"https://api.github.com/users/sashafrey/events{/privacy}","received_events_url":"https://api.github.com/users/sashafrey/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/bigartm/bigartm/labels/api%20braking%20change","name":"api braking change","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/bigartm/bigartm/milestones/6","html_url":"https://github.com/bigartm/bigartm/milestones/v0.8.0","labels_url":"https://api.github.com/repos/bigartm/bigartm/milestones/6/labels","id":1568306,"number":6,"title":"v0.8.0","description":"","creator":{"login":"sashafrey","id":1252606,"avatar_url":"https://avatars.githubusercontent.com/u/1252606?v=3","gravatar_id":"","url":"https://api.github.com/users/sashafrey","html_url":"https://github.com/sashafrey","followers_url":"https://api.github.com/users/sashafrey/followers","following_url":"https://api.github.com/users/sashafrey/following{/other_user}","gists_url":"https://api.github.com/users/sashafrey/gists{/gist_id}","starred_url":"https://api.github.com/users/sashafrey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sashafrey/subscriptions","organizations_url":"https://api.github.com/users/sashafrey/orgs","repos_url":"https://api.github.com/users/sashafrey/repos","events_url":"https://api.github.com/users/sashafrey/events{/privacy}","received_events_url":"https://api.github.com/users/sashafrey/received_events","type":"User","site_admin":false},"open_issues":6,"closed_issues":5,"state":"open","created_at":"2016-02-06T23:59:13Z","updated_at":"2016-03-30T19:04:50Z","due_on":null,"closed_at":null},"comments":8,"created_at":"2016-03-16T23:36:13Z","updated_at":"2016-04-01T08:15:27Z","closed_at":null,"body":"We need to review python API and consider making breaking changes for v0.8.0. From that point we  should not introduce any breaking changes for v0.8.X. Next change to break API is v0.9.0.\r\n\r\nAs a first input we can review this pull request to see if we need to grab anything from it:\r\nhttps://github.com/bigartm/bigartm/pull/312/"},"comment":{"url":"https://api.github.com/repos/bigartm/bigartm/issues/comments/204302949","html_url":"https://github.com/bigartm/bigartm/issues/489#issuecomment-204302949","issue_url":"https://api.github.com/repos/bigartm/bigartm/issues/489","id":204302949,"user":{"login":"sashafrey","id":1252606,"avatar_url":"https://avatars.githubusercontent.com/u/1252606?v=3","gravatar_id":"","url":"https://api.github.com/users/sashafrey","html_url":"https://github.com/sashafrey","followers_url":"https://api.github.com/users/sashafrey/followers","following_url":"https://api.github.com/users/sashafrey/following{/other_user}","gists_url":"https://api.github.com/users/sashafrey/gists{/gist_id}","starred_url":"https://api.github.com/users/sashafrey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sashafrey/subscriptions","organizations_url":"https://api.github.com/users/sashafrey/orgs","repos_url":"https://api.github.com/users/sashafrey/repos","events_url":"https://api.github.com/users/sashafrey/events{/privacy}","received_events_url":"https://api.github.com/users/sashafrey/received_events","type":"User","site_admin":false},"created_at":"2016-04-01T08:15:27Z","updated_at":"2016-04-01T08:15:27Z","body":"@AnyaP Yes, this sounds reasonable (and possible :)). To clarify, how get_score(data) should behave for phi-scores (those that only depend on phi matrix itself, not on the data)? Should we just ignore the data, or throw an exception to warn the user that it is not needed?\r\n\r\nAlso, note that for theta scores `get_score(data=None)` will return score from the cache, while `get_score(data=batch_vectorizer)` will calculate the score. Could this be confusing? It sound like we need a separate method `calc_score` where `data` argument is required. "}}