{"action":"created","issue":{"url":"https://api.github.com/repos/EdwardRaff/JSAT/issues/32","labels_url":"https://api.github.com/repos/EdwardRaff/JSAT/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/EdwardRaff/JSAT/issues/32/comments","events_url":"https://api.github.com/repos/EdwardRaff/JSAT/issues/32/events","html_url":"https://github.com/EdwardRaff/JSAT/pull/32","id":129877381,"number":32,"title":"IndexOutOfBoundsException in PorterStemmer for single character words","user":{"login":"fkleon","id":448110,"avatar_url":"https://avatars.githubusercontent.com/u/448110?v=3","gravatar_id":"","url":"https://api.github.com/users/fkleon","html_url":"https://github.com/fkleon","followers_url":"https://api.github.com/users/fkleon/followers","following_url":"https://api.github.com/users/fkleon/following{/other_user}","gists_url":"https://api.github.com/users/fkleon/gists{/gist_id}","starred_url":"https://api.github.com/users/fkleon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fkleon/subscriptions","organizations_url":"https://api.github.com/users/fkleon/orgs","repos_url":"https://api.github.com/users/fkleon/repos","events_url":"https://api.github.com/users/fkleon/events{/privacy}","received_events_url":"https://api.github.com/users/fkleon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-29T20:43:13Z","updated_at":"2016-01-29T20:50:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/EdwardRaff/JSAT/pulls/32","html_url":"https://github.com/EdwardRaff/JSAT/pull/32","diff_url":"https://github.com/EdwardRaff/JSAT/pull/32.diff","patch_url":"https://github.com/EdwardRaff/JSAT/pull/32.patch"},"body":"Small bugfix to the PorterStemmer implementation.\r\n\r\nUsually short words would be filtered by an appropriate tokenizer before reaching the stemmer, but their use should not be mandatory and so it is possible to end up with single character words in the PorterStemmer. The original implementation handles this gracefully, as would JSAT's implementation with this patch.\r\n\r\nContribution by Catalyst.Net Ltd, submitted under the GPLv3."},"comment":{"url":"https://api.github.com/repos/EdwardRaff/JSAT/issues/comments/176964429","html_url":"https://github.com/EdwardRaff/JSAT/pull/32#issuecomment-176964429","issue_url":"https://api.github.com/repos/EdwardRaff/JSAT/issues/32","id":176964429,"user":{"login":"EdwardRaff","id":2008786,"avatar_url":"https://avatars.githubusercontent.com/u/2008786?v=3","gravatar_id":"","url":"https://api.github.com/users/EdwardRaff","html_url":"https://github.com/EdwardRaff","followers_url":"https://api.github.com/users/EdwardRaff/followers","following_url":"https://api.github.com/users/EdwardRaff/following{/other_user}","gists_url":"https://api.github.com/users/EdwardRaff/gists{/gist_id}","starred_url":"https://api.github.com/users/EdwardRaff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EdwardRaff/subscriptions","organizations_url":"https://api.github.com/users/EdwardRaff/orgs","repos_url":"https://api.github.com/users/EdwardRaff/repos","events_url":"https://api.github.com/users/EdwardRaff/events{/privacy}","received_events_url":"https://api.github.com/users/EdwardRaff/received_events","type":"User","site_admin":false},"created_at":"2016-01-29T20:50:36Z","updated_at":"2016-01-29T20:50:36Z","body":"Thanks for the bug fix! \n\nI can't accept contributions under he GPL. Changes like this are so small they are not actually covered under copyright, but since you've made it explicit - I'm not going to merge this. If you want to contribute it under public domain I'll merge, otherwise I'll just fix this when I get home tonight. \n\nSent from my iPhone\n\n> On Jan 29, 2016, at 3:43 PM, Frederik Leonhardt <notifications@github.com> wrote:\n> \n> Small bugfix to the PorterStemmer implementation.\n> \n> Usually short words would be filtered by an appropriate tokenizer before reaching the stemmer, but their use should not be mandatory and so it is possible to end up with single character words in the PorterStemmer. The original implementation handles this gracefully, as would JSAT's implementation with this patch.\n> \n> Contribution by Catalyst.Net Ltd, submitted under the GPLv3.\n> \n> You can view, comment on, or merge this pull request online at:\n> \n>   https://github.com/EdwardRaff/JSAT/pull/32\n> \n> Commit Summary\n> \n> Fix IndexOutOfBoundsException in PorterStemmer for single character words.\n> File Changes\n> \n> M JSAT/src/jsat/text/stemming/PorterStemmer.java (2)\n> M JSAT/test/jsat/text/stemming/PorterStemmerTest.java (1)\n> Patch Links:\n> \n> https://github.com/EdwardRaff/JSAT/pull/32.patch\n> https://github.com/EdwardRaff/JSAT/pull/32.diff\n> —\n> Reply to this email directly or view it on GitHub.\n> \n"}}