{"action":"created","issue":{"url":"https://api.github.com/repos/mlpack/mlpack/issues/462","labels_url":"https://api.github.com/repos/mlpack/mlpack/issues/462/labels{/name}","comments_url":"https://api.github.com/repos/mlpack/mlpack/issues/462/comments","events_url":"https://api.github.com/repos/mlpack/mlpack/issues/462/events","html_url":"https://github.com/mlpack/mlpack/pull/462","id":111994379,"number":462,"title":"fix constructor bug(oivs_init.hpp)","user":{"login":"stereomatchingkiss","id":1801914,"avatar_url":"https://avatars.githubusercontent.com/u/1801914?v=3","gravatar_id":"","url":"https://api.github.com/users/stereomatchingkiss","html_url":"https://github.com/stereomatchingkiss","followers_url":"https://api.github.com/users/stereomatchingkiss/followers","following_url":"https://api.github.com/users/stereomatchingkiss/following{/other_user}","gists_url":"https://api.github.com/users/stereomatchingkiss/gists{/gist_id}","starred_url":"https://api.github.com/users/stereomatchingkiss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stereomatchingkiss/subscriptions","organizations_url":"https://api.github.com/users/stereomatchingkiss/orgs","repos_url":"https://api.github.com/users/stereomatchingkiss/repos","events_url":"https://api.github.com/users/stereomatchingkiss/events{/privacy}","received_events_url":"https://api.github.com/users/stereomatchingkiss/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-10-18T01:37:03Z","updated_at":"2015-10-18T17:58:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mlpack/mlpack/pulls/462","html_url":"https://github.com/mlpack/mlpack/pull/462","diff_url":"https://github.com/mlpack/mlpack/pull/462.diff","patch_url":"https://github.com/mlpack/mlpack/pull/462.patch"},"body":"fix bugs\r\n\r\n1 : Forgot to define the data member \"epsilon\"\r\n2 : The data member \"b\" should initialize at initialization list"},"comment":{"url":"https://api.github.com/repos/mlpack/mlpack/issues/comments/149035369","html_url":"https://github.com/mlpack/mlpack/pull/462#issuecomment-149035369","issue_url":"https://api.github.com/repos/mlpack/mlpack/issues/462","id":149035369,"user":{"login":"zoq","id":4209744,"avatar_url":"https://avatars.githubusercontent.com/u/4209744?v=3","gravatar_id":"","url":"https://api.github.com/users/zoq","html_url":"https://github.com/zoq","followers_url":"https://api.github.com/users/zoq/followers","following_url":"https://api.github.com/users/zoq/following{/other_user}","gists_url":"https://api.github.com/users/zoq/gists{/gist_id}","starred_url":"https://api.github.com/users/zoq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zoq/subscriptions","organizations_url":"https://api.github.com/users/zoq/orgs","repos_url":"https://api.github.com/users/zoq/repos","events_url":"https://api.github.com/users/zoq/events{/privacy}","received_events_url":"https://api.github.com/users/zoq/received_events","type":"User","site_admin":false},"created_at":"2015-10-18T17:58:57Z","updated_at":"2015-10-18T17:58:57Z","body":"Actually I didn't forget to define ``epsilon`` as data member. It's only needed in the constructor, so there is no need to define ``epsilon`` as data member. Regarding the data member ``b``, I avoided the initialization list in favor of readability, but maybe it doesn't make a huge difference?"}}