{"comment":{"url":"https://api.github.com/repos/jahabrewer/mlpack/comments/11242227","html_url":"https://github.com/jahabrewer/mlpack/commit/a63ddcdd77182244e19d9f11bcb0f6b43e403c32#commitcomment-11242227","id":11242227,"user":{"login":"rcurtin","id":1845039,"avatar_url":"https://avatars.githubusercontent.com/u/1845039?v=3","gravatar_id":"","url":"https://api.github.com/users/rcurtin","html_url":"https://github.com/rcurtin","followers_url":"https://api.github.com/users/rcurtin/followers","following_url":"https://api.github.com/users/rcurtin/following{/other_user}","gists_url":"https://api.github.com/users/rcurtin/gists{/gist_id}","starred_url":"https://api.github.com/users/rcurtin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rcurtin/subscriptions","organizations_url":"https://api.github.com/users/rcurtin/orgs","repos_url":"https://api.github.com/users/rcurtin/repos","events_url":"https://api.github.com/users/rcurtin/events{/privacy}","received_events_url":"https://api.github.com/users/rcurtin/received_events","type":"User","site_admin":false},"position":25,"line":233,"path":"src/mlpack/methods/det/dt_utils.cpp","commit_id":"a63ddcdd77182244e19d9f11bcb0f6b43e403c32","created_at":"2015-05-18T14:40:29Z","updated_at":"2015-05-18T14:40:29Z","body":"Did this change give a big runtime improvement?  I'd imagine it did.  I was trying to think of some way to remove the atomic here, but this looks like the right solution."}}