Skip to content

Update doc/source/_templates/indexcontent.html #2698

Closed
wants to merge 1 commit into from

4 participants

@charris
NumPy member
charris commented Mar 2, 2013

Ping travisbot.

@charris charris closed this Mar 2, 2013
@charris charris reopened this Mar 2, 2013
@charris
NumPy member
charris commented Apr 2, 2013

Looks like this might need a rebase. @pv What do you think of this fix?

@charris charris was assigned Apr 3, 2013
@rgommers
NumPy member
rgommers commented Jun 3, 2013

The correct fix is using py-modindex instead of np-modindex. However, the list of modules is still incomplete. linalg and random are missing for example.

@rgommers
NumPy member
rgommers commented Jun 3, 2013

Tried also to change to html_domain_indices, with value either True or a list of module names, but that doesn't add the missing modules.

@rgommers
NumPy member
rgommers commented Jun 3, 2013

Why do we need the module index anyway? There's a complete list in the reference guide that's easy to access, and a good number of the modules picked up in py-modindex are useless (numpy.lib.scimath anyone?). If it's easy to remove from the front page and the template (there's a link in the top right corner) then I propose to do that.

@pv
NumPy member
pv commented Jun 4, 2013

Yes, the module index doesn't seem to be very useful here, as the namespace is very flat. So we could just drop it.

@rgommers
NumPy member
rgommers commented Jun 4, 2013

OK, will send a new PR for that. Closing this one then. @iedwards thanks for your effort on this!

@rgommers rgommers closed this Jun 4, 2013
@rgommers
NumPy member
rgommers commented Jun 4, 2013

Trac ticket 2231 fell through the cracks during the Github move by the way, so there's no issue to close.

@rgommers
NumPy member
rgommers commented Jun 4, 2013

New PR: gh-3401

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.