{"action":"created","issue":{"url":"https://api.github.com/repos/pysal/pysal/issues/844","repository_url":"https://api.github.com/repos/pysal/pysal","labels_url":"https://api.github.com/repos/pysal/pysal/issues/844/labels{/name}","comments_url":"https://api.github.com/repos/pysal/pysal/issues/844/comments","events_url":"https://api.github.com/repos/pysal/pysal/issues/844/events","html_url":"https://github.com/pysal/pysal/pull/844","id":166139510,"number":844,"title":"Geotable plot","user":{"login":"darribas","id":417363,"avatar_url":"https://avatars.githubusercontent.com/u/417363?v=3","gravatar_id":"","url":"https://api.github.com/users/darribas","html_url":"https://github.com/darribas","followers_url":"https://api.github.com/users/darribas/followers","following_url":"https://api.github.com/users/darribas/following{/other_user}","gists_url":"https://api.github.com/users/darribas/gists{/gist_id}","starred_url":"https://api.github.com/users/darribas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darribas/subscriptions","organizations_url":"https://api.github.com/users/darribas/orgs","repos_url":"https://api.github.com/users/darribas/repos","events_url":"https://api.github.com/users/darribas/events{/privacy}","received_events_url":"https://api.github.com/users/darribas/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/pysal/pysal/labels/Discussion","name":"Discussion","color":"fef2c0"},{"url":"https://api.github.com/repos/pysal/pysal/labels/Visualization","name":"Visualization","color":"5319e7"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":13,"created_at":"2016-07-18T16:46:12Z","updated_at":"2016-07-22T19:20:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/pysal/pysal/pulls/844","html_url":"https://github.com/pysal/pysal/pull/844","diff_url":"https://github.com/pysal/pysal/pull/844.diff","patch_url":"https://github.com/pysal/pysal/pull/844.patch"},"body":"This is a PR to bring into `pysal/dev` work to implement easy plotting for `geotables`. Not to be merged yet but rather to create a place to centralized the discussion around design details and so on. My idea is that, once we're happy with it we merge and, ideally, we have it in by the time @ljwolf's GSoC project gets merged as well so we can have a big splash on simplifying workflows with spatial data.\r\n\r\nSo far, this implements basic machinery to plot `geotables` with both a `matplotlib` and a `bokeh` backend and contains a `color.py` module with palette support that depends on `bewer2mpl`. There's a notebook (`geotable_plot`) explaining the new functionality."},"comment":{"url":"https://api.github.com/repos/pysal/pysal/issues/comments/234632565","html_url":"https://github.com/pysal/pysal/pull/844#issuecomment-234632565","issue_url":"https://api.github.com/repos/pysal/pysal/issues/844","id":234632565,"user":{"login":"sjsrey","id":118042,"avatar_url":"https://avatars.githubusercontent.com/u/118042?v=3","gravatar_id":"","url":"https://api.github.com/users/sjsrey","html_url":"https://github.com/sjsrey","followers_url":"https://api.github.com/users/sjsrey/followers","following_url":"https://api.github.com/users/sjsrey/following{/other_user}","gists_url":"https://api.github.com/users/sjsrey/gists{/gist_id}","starred_url":"https://api.github.com/users/sjsrey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sjsrey/subscriptions","organizations_url":"https://api.github.com/users/sjsrey/orgs","repos_url":"https://api.github.com/users/sjsrey/repos","events_url":"https://api.github.com/users/sjsrey/events{/privacy}","received_events_url":"https://api.github.com/users/sjsrey/received_events","type":"User","site_admin":false},"created_at":"2016-07-22T19:20:27Z","updated_at":"2016-07-22T19:20:27Z","body":"This change breaks [choroBrewer.ipynb](https://github.com/darribas/pysal/blob/geotable_plot/pysal/contrib/viz/choroBrewer.ipynb)\r\n```\r\n/home/serge/Dropbox/p/pysal/src/pysal/pysal/contrib/viz/mapping.py in geoplot(db, col, palette, classi, backend, color, facecolor, edgecolor, alpha, linewidth, marker, marker_size, ax, hover, p, tips, figsize, **kwargs)\r\n    611     '''\r\n    612     if col:\r\n--> 613         lbl,c = value_classifier(db[col], scheme=classi, **kwargs)\r\n    614         if type(palette) is not str:\r\n    615             palette = get_color_map(palette=palette, k=c.k)\r\n\r\n/home/serge/Dropbox/p/pysal/src/pysal/pysal/contrib/viz/mapping.py in value_classifier(y, scheme, **kwargs)\r\n     58     classification   : Map_Classifier instance\r\n     59     \"\"\"\r\n---> 60     c = classifier[scheme.lower()](y, **kwargs)\r\n     61     return (pd.Series(c.yb, index=y.index), c)\r\n     62 \r\n\r\nTypeError: __init__() got an unexpected keyword argument 'dtype'\r\n```"}}
{"actor":{"display_login":"sjsrey"}}