{"action":"created","issue":{"url":"https://api.github.com/repos/numenta/nupic.research/issues/272","labels_url":"https://api.github.com/repos/numenta/nupic.research/issues/272/labels{/name}","comments_url":"https://api.github.com/repos/numenta/nupic.research/issues/272/comments","events_url":"https://api.github.com/repos/numenta/nupic.research/issues/272/events","html_url":"https://github.com/numenta/nupic.research/issues/272","id":101269602,"number":272,"title":"classification_network.py assumes classifier has HTM-specific inputs/outputs","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},"labels":[{"url":"https://api.github.com/repos/numenta/nupic.research/labels/algorithms","name":"algorithms","color":"207de5"},{"url":"https://api.github.com/repos/numenta/nupic.research/labels/P2","name":"P2","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-16T14:33:45Z","updated_at":"2015-08-16T18:28:43Z","closed_at":null,"body":"The sequence classification network factory assumes the classifier region has inputs/outputs for `predictedActiveCells`, which is not always true -- e.g. KNNClassifier. This is a problem [here](https://github.com/numenta/nupic.research/blob/master/classification/classification_network.py#L302-305). An if statement checking for the region type would work, but I suggest we move region-linking logic to the specific createXRegion() methods. Thoughts @marionleborgne @scottpurdy?\r\n\r\nThis is currently breaking the nupic.fluent HTM classification model -- test [here](https://github.com/numenta/nupic.fluent/blob/master/tests/integration/classification_models_validation_test.py#L174)."},"comment":{"url":"https://api.github.com/repos/numenta/nupic.research/issues/comments/131598944","html_url":"https://github.com/numenta/nupic.research/issues/272#issuecomment-131598944","issue_url":"https://api.github.com/repos/numenta/nupic.research/issues/272","id":131598944,"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},"created_at":"2015-08-16T18:28:43Z","updated_at":"2015-08-16T18:28:43Z","body":"@marionleborgne yes this plan works for me; I made a followup issue (#274). The next step in the nupic.fluent experiments is to implement a Sensor -> UP -> classifier network, so this should help."}}