{"action":"created","issue":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5481","labels_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5481/labels{/name}","comments_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5481/comments","events_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5481/events","html_url":"https://github.com/scikit-learn/scikit-learn/issues/5481","id":112348245,"number":5481,"title":"Estimators should not try to modify X and y inplace in order to handle readonly memory maps","user":{"login":"arthurmensch","id":4078691,"avatar_url":"https://avatars.githubusercontent.com/u/4078691?v=3","gravatar_id":"","url":"https://api.github.com/users/arthurmensch","html_url":"https://github.com/arthurmensch","followers_url":"https://api.github.com/users/arthurmensch/followers","following_url":"https://api.github.com/users/arthurmensch/following{/other_user}","gists_url":"https://api.github.com/users/arthurmensch/gists{/gist_id}","starred_url":"https://api.github.com/users/arthurmensch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arthurmensch/subscriptions","organizations_url":"https://api.github.com/users/arthurmensch/orgs","repos_url":"https://api.github.com/users/arthurmensch/repos","events_url":"https://api.github.com/users/arthurmensch/events{/privacy}","received_events_url":"https://api.github.com/users/arthurmensch/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/labels/Bug","name":"Bug","color":"e10c02"},{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/labels/Easy","name":"Easy","color":"02d7e1"},{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/labels/Need%20Contributor","name":"Need Contributor","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-10-20T11:38:36Z","updated_at":"2015-10-20T12:42:25Z","closed_at":null,"body":"PR #4807 reveals a variety of estimators that fails on memory mapped input, once we allow check_array to process a memory map without copying its content in memory.\r\n\r\nEstimators failing on read only memory maps :\r\n- the whole PLS family\r\n- Factor analysis\r\n- Incremental PCA\r\n- NuSVC\r\n\r\nTransformers : \r\n- KernelCenterer\r\n- MaxAbsScaler\r\n- MinAbsScaler\r\n- RobustScaler\r\n- StandardScaler\r\n\r\nMost of these should be easy to fix (e.g X -= X.mean should be replaced etc.)\r\n"},"comment":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/comments/149552869","html_url":"https://github.com/scikit-learn/scikit-learn/issues/5481#issuecomment-149552869","issue_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5481","id":149552869,"user":{"login":"sygi","id":495380,"avatar_url":"https://avatars.githubusercontent.com/u/495380?v=3","gravatar_id":"","url":"https://api.github.com/users/sygi","html_url":"https://github.com/sygi","followers_url":"https://api.github.com/users/sygi/followers","following_url":"https://api.github.com/users/sygi/following{/other_user}","gists_url":"https://api.github.com/users/sygi/gists{/gist_id}","starred_url":"https://api.github.com/users/sygi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sygi/subscriptions","organizations_url":"https://api.github.com/users/sygi/orgs","repos_url":"https://api.github.com/users/sygi/repos","events_url":"https://api.github.com/users/sygi/events{/privacy}","received_events_url":"https://api.github.com/users/sygi/received_events","type":"User","site_admin":false},"created_at":"2015-10-20T12:42:25Z","updated_at":"2015-10-20T12:42:25Z","body":"As there's quite a few estimators/transformers to work on, I'll let myself to join. I'll start with FactorAnalysis, for instance ;)"}}