Remove noindex option under geopandas section so that permalinks work

[weiji14]

Hi there, I was trying to add an intersphinx link using "py:func:`pyogrio.read_dataframe`" at weiji14/zen3geo#19 (comment) but it seems like there wasn't a permalink available. The closest link I could find was https://pyogrio.readthedocs.io/en/latest/api.html#geopandas-integration, but ideally it would be possible to link directly to pyorgio.read_dataframe.

This Pull Request removes the :noindex: option added in #18 (specifically at commit 679b81b) so that intersphinx will work and the permalink shows up, like so:

And you can then click on https://pyogrio--130.org.readthedocs.build/en/130/api.html#pyogrio.read_dataframe!

[brendan-ward]

@weiji14 this passed all tests; is it ready for review?

[weiji14]

Yep, just another one-line commit. Noticed that the "Geopandas integration" subsection had a permalink like https://pyogrio--130.org.readthedocs.build/en/130/api.html#module-0 (due to duplicate use of .. automodule:: pyogrio), but it should be https://pyogrio--130.org.readthedocs.build/en/130/api.html#module-pyogrio.geopandas instead.

[weiji14]

Oh wait, now the docs say pyogrio.geopandas.read_dataframe which is probably not what we want. Hold on, give me a few more minutes to look at https://www.sphinx-doc.org/en/master/usage/extensions/autodoc.html#directive-automodule

[jorisvandenbossche]

Yes, we would prefer to use the public "pyogrio.read_dataframe" path being shown

[weiji14]

Ok, things look better now, just needed to use .. autofunction::. Links are:

• https://pyogrio--130.org.readthedocs.build/en/130/api.html#module-pyogrio.geopandas
• https://pyogrio--130.org.readthedocs.build/en/130/api.html#pyogrio.read_dataframe
• https://pyogrio--130.org.readthedocs.build/en/130/api.html#pyogrio.write_dataframe

[martinfleis]

Thanks!