{"action":"created","issue":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/3248","repository_url":"https://api.github.com/repos/mne-tools/mne-python","labels_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3248/labels{/name}","comments_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3248/comments","events_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3248/events","html_url":"https://github.com/mne-tools/mne-python/issues/3248","id":156870929,"number":3248,"title":"Correct freq bandwidth in FilterEstimator?","user":{"login":"kingjr","id":4881164,"avatar_url":"https://avatars.githubusercontent.com/u/4881164?v=3","gravatar_id":"","url":"https://api.github.com/users/kingjr","html_url":"https://github.com/kingjr","followers_url":"https://api.github.com/users/kingjr/followers","following_url":"https://api.github.com/users/kingjr/following{/other_user}","gists_url":"https://api.github.com/users/kingjr/gists{/gist_id}","starred_url":"https://api.github.com/users/kingjr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kingjr/subscriptions","organizations_url":"https://api.github.com/users/kingjr/orgs","repos_url":"https://api.github.com/users/kingjr/repos","events_url":"https://api.github.com/users/kingjr/events{/privacy}","received_events_url":"https://api.github.com/users/kingjr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-05-25T23:15:19Z","updated_at":"2016-05-26T16:26:02Z","closed_at":null,"body":"@trachelr I'm not entirely sure but I think there is a problem with the transition bandwidths in your [`FilterEstimator`](https://github.com/mne-tools/mne-python/pull/2292).\r\n\r\nCan you confirm or infirm that the low and high bandwidth are adequate?\r\nhttps://github.com/mne-tools/mne-python/blob/master/mne/decoding/transformer.py#L484\r\n\r\nIf not, I'll mark the issue as a bug, but don't worry about submitting a PR soon (although you may), we're about to refactor all decoding transformers and estimators.\r\n\r\nIf it's working as intended, well, my bad.."},"comment":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/comments/221922521","html_url":"https://github.com/mne-tools/mne-python/issues/3248#issuecomment-221922521","issue_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3248","id":221922521,"user":{"login":"Eric89GXL","id":2365790,"avatar_url":"https://avatars.githubusercontent.com/u/2365790?v=3","gravatar_id":"","url":"https://api.github.com/users/Eric89GXL","html_url":"https://github.com/Eric89GXL","followers_url":"https://api.github.com/users/Eric89GXL/followers","following_url":"https://api.github.com/users/Eric89GXL/following{/other_user}","gists_url":"https://api.github.com/users/Eric89GXL/gists{/gist_id}","starred_url":"https://api.github.com/users/Eric89GXL/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Eric89GXL/subscriptions","organizations_url":"https://api.github.com/users/Eric89GXL/orgs","repos_url":"https://api.github.com/users/Eric89GXL/repos","events_url":"https://api.github.com/users/Eric89GXL/events{/privacy}","received_events_url":"https://api.github.com/users/Eric89GXL/received_events","type":"User","site_admin":false},"created_at":"2016-05-26T16:26:01Z","updated_at":"2016-05-26T16:26:01Z","body":"It should match how it is done in `raw.filter`, so it looks like the low-pass and high-pass conditions are wrong"}}
{"actor":{"display_login":"Eric89GXL"}}