Add GeoJSON display class by gnestor · Pull Request #10253 · ipython/ipython (original) (raw)

On Tue, Feb 7, 2017 at 1:09 PM, Matthias Bussonnier < ***@***.***> wrote:@Carreau <https://github.com/Carreau> Can you clarify? Basically any update to the display mechanism need update to IPython itself that can take a year or so to get released. It would be better to provide a mechanism for other packages to provide this. In Python that would be entry-points.

I don't understand what would take a year. The GeoJSON class isn't an update to the display protocol. It is just a new class that a bunch of us are needing/using across different frontends. We already have common display classes for JSON, HTML, Image, etc. - this is yet another one of those. We are putting the more specialized ones (Vega, Plotly) in their respective libraries (altair, plotly, etc.). Then you can "just" update/publish an ipython-geosjon, ipython-vegalite,

ipython-mp3 , ipython-pdf , ipython-midi ... and have IPython auto discover these.

Ahh, OK I think I am understanding. This would allow third party developers to put things under the IPython.display namespace, while shipping things separately. Is this entry point already in place? For things like Vega/Plotly that (the entry point) makes sense, but for the GeoJSON case 1) it is a very common standard and 2) there isn't a python package for us to put it in - it is literally just that class. Grant, Kyle and I thought that putting it directly into IPython made the most sense. Then in case of updates needed, a new release of IPython is not necessary.

Got it!

…

-- Brian E. Granger Associate Professor of Physics and Data Science Cal Poly State University, San Luis Obispo@ellisonbg on Twitter and GitHub bgranger@calpoly.edu and ellisonbg@gmail.com