{"comment":{"url":"https://api.github.com/repos/akhilaananthram/nupic.fluent/comments/12680420","html_url":"https://github.com/akhilaananthram/nupic.fluent/commit/7941a8328de20c5a5d4b8633cc3f611b24e72ddd#commitcomment-12680420","id":12680420,"user":{"login":"BoltzmannBrain","id":8323700,"avatar_url":"https://avatars.githubusercontent.com/u/8323700?v=3","gravatar_id":"","url":"https://api.github.com/users/BoltzmannBrain","html_url":"https://github.com/BoltzmannBrain","followers_url":"https://api.github.com/users/BoltzmannBrain/followers","following_url":"https://api.github.com/users/BoltzmannBrain/following{/other_user}","gists_url":"https://api.github.com/users/BoltzmannBrain/gists{/gist_id}","starred_url":"https://api.github.com/users/BoltzmannBrain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BoltzmannBrain/subscriptions","organizations_url":"https://api.github.com/users/BoltzmannBrain/orgs","repos_url":"https://api.github.com/users/BoltzmannBrain/repos","events_url":"https://api.github.com/users/BoltzmannBrain/events{/privacy}","received_events_url":"https://api.github.com/users/BoltzmannBrain/received_events","type":"User","site_admin":false},"position":6,"line":36,"path":"fluent/models/classify_htm.py","commit_id":"7941a8328de20c5a5d4b8633cc3f611b24e72ddd","created_at":"2015-08-13T03:30:36Z","updated_at":"2015-08-13T03:30:36Z","body":"It would be more intuitive to have `spTrainingSize` and `tmTrainingSize` define the start of SP and TM training respectively, don't you think? Then we would also want a `classifierTrainingSize`. This may seem overkill in this model, but we shouldn't assume all htm network models will follow sp to tm to classifier.\r\n\r\nAnd update the docstring please."}}