{"comment":{"url":"https://api.github.com/repos/flashnotes/django-import-export/comments/9267611","html_url":"https://github.com/flashnotes/django-import-export/commit/781c2808bcbc4445a8731f15683065da4fc9ca55#commitcomment-9267611","id":9267611,"user":{"login":"jmyatt","id":4203517,"avatar_url":"https://avatars.githubusercontent.com/u/4203517?v=3","gravatar_id":"","url":"https://api.github.com/users/jmyatt","html_url":"https://github.com/jmyatt","followers_url":"https://api.github.com/users/jmyatt/followers","following_url":"https://api.github.com/users/jmyatt/following{/other_user}","gists_url":"https://api.github.com/users/jmyatt/gists{/gist_id}","starred_url":"https://api.github.com/users/jmyatt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmyatt/subscriptions","organizations_url":"https://api.github.com/users/jmyatt/orgs","repos_url":"https://api.github.com/users/jmyatt/repos","events_url":"https://api.github.com/users/jmyatt/events{/privacy}","received_events_url":"https://api.github.com/users/jmyatt/received_events","type":"User","site_admin":false},"position":5,"line":126,"path":"import_export/formats/base_formats.py","commit_id":"781c2808bcbc4445a8731f15683065da4fc9ca55","created_at":"2015-01-13T21:01:07Z","updated_at":"2015-01-13T21:01:07Z","body":"2 questions:\r\n- reading CSV in binary mode for Python 2.x seems to be what's documented [here] (https://docs.python.org/2/library/csv.html#csv.reader) (\"If csvfile is a file object, it must be opened with the ‘b’ flag on platforms where that makes a difference\") - is that information outdated or not relevant to these particular CSV reads?\r\n- if we change get_read_mode should we also change is_binary() (the other method in this class)?"}}