{"action":"created","issue":{"url":"https://api.github.com/repos/gwpy/gwpy/issues/699","repository_url":"https://api.github.com/repos/gwpy/gwpy","labels_url":"https://api.github.com/repos/gwpy/gwpy/issues/699/labels{/name}","comments_url":"https://api.github.com/repos/gwpy/gwpy/issues/699/comments","events_url":"https://api.github.com/repos/gwpy/gwpy/issues/699/events","html_url":"https://github.com/gwpy/gwpy/pull/699","id":302653505,"number":699,"title":"Added new SeriesAxes and SeriesPlot base classes","user":{"login":"duncanmmacleod","id":1618530,"avatar_url":"https://avatars3.githubusercontent.com/u/1618530?v=4","gravatar_id":"","url":"https://api.github.com/users/duncanmmacleod","html_url":"https://github.com/duncanmmacleod","followers_url":"https://api.github.com/users/duncanmmacleod/followers","following_url":"https://api.github.com/users/duncanmmacleod/following{/other_user}","gists_url":"https://api.github.com/users/duncanmmacleod/gists{/gist_id}","starred_url":"https://api.github.com/users/duncanmmacleod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/duncanmmacleod/subscriptions","organizations_url":"https://api.github.com/users/duncanmmacleod/orgs","repos_url":"https://api.github.com/users/duncanmmacleod/repos","events_url":"https://api.github.com/users/duncanmmacleod/events{/privacy}","received_events_url":"https://api.github.com/users/duncanmmacleod/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-03-06T10:54:29Z","updated_at":"2018-03-06T10:54:33Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/gwpy/gwpy/pulls/699","html_url":"https://github.com/gwpy/gwpy/pull/699","diff_url":"https://github.com/gwpy/gwpy/pull/699.diff","patch_url":"https://github.com/gwpy/gwpy/pull/699.patch"},"body":"This PR adds two new base classes, SeriesAxes and SeriesPlot to reduce duplication between the TimeSeries and FrequencySeries plotters.\r\n\r\nThis is a simplified version of #694, which included too many other changes."},"comment":{"url":"https://api.github.com/repos/gwpy/gwpy/issues/comments/370743319","html_url":"https://github.com/gwpy/gwpy/pull/699#issuecomment-370743319","issue_url":"https://api.github.com/repos/gwpy/gwpy/issues/699","id":370743319,"user":{"login":"pep8speaks","id":24736507,"avatar_url":"https://avatars1.githubusercontent.com/u/24736507?v=4","gravatar_id":"","url":"https://api.github.com/users/pep8speaks","html_url":"https://github.com/pep8speaks","followers_url":"https://api.github.com/users/pep8speaks/followers","following_url":"https://api.github.com/users/pep8speaks/following{/other_user}","gists_url":"https://api.github.com/users/pep8speaks/gists{/gist_id}","starred_url":"https://api.github.com/users/pep8speaks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pep8speaks/subscriptions","organizations_url":"https://api.github.com/users/pep8speaks/orgs","repos_url":"https://api.github.com/users/pep8speaks/repos","events_url":"https://api.github.com/users/pep8speaks/events{/privacy}","received_events_url":"https://api.github.com/users/pep8speaks/received_events","type":"User","site_admin":false},"created_at":"2018-03-06T10:54:33Z","updated_at":"2018-03-06T10:54:33Z","author_association":"NONE","body":"Hello @duncanmmacleod! Thanks for submitting the PR.\n\n - In the file [`examples/frequencyseries/percentiles.py`](https://github.com/gwpy/gwpy/blob/ceec64220fbc5b509ac5575fd80c88a5e5d32faf/examples/frequencyseries/percentiles.py), following are the PEP8 issues :\n\n> [Line 60:1](https://github.com/gwpy/gwpy/blob/ceec64220fbc5b509ac5575fd80c88a5e5d32faf/examples/frequencyseries/percentiles.py#L60): [E402](https://duckduckgo.com/?q=pep8%20E402) module level import not at top of file\n\n\n\n\n\n\n\n - In the file [`gwpy/plotter/series.py`](https://github.com/gwpy/gwpy/blob/ceec64220fbc5b509ac5575fd80c88a5e5d32faf/gwpy/plotter/series.py), following are the PEP8 issues :\n\n> [Line 267:1](https://github.com/gwpy/gwpy/blob/ceec64220fbc5b509ac5575fd80c88a5e5d32faf/gwpy/plotter/series.py#L267): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1\n\n - In the file [`gwpy/plotter/timeseries.py`](https://github.com/gwpy/gwpy/blob/ceec64220fbc5b509ac5575fd80c88a5e5d32faf/gwpy/plotter/timeseries.py), following are the PEP8 issues :\n\n> [Line 195:1](https://github.com/gwpy/gwpy/blob/ceec64220fbc5b509ac5575fd80c88a5e5d32faf/gwpy/plotter/timeseries.py#L195): [E305](https://duckduckgo.com/?q=pep8%20E305) expected 2 blank lines after class or function definition, found 1\n\n"}}
{"actor":{"display_login":"pep8speaks"}}