Skip to content

Fix to website #2583

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Fix to website #2583

wants to merge 2 commits into from

Conversation

jaeilepp
Copy link
Contributor

@jaeilepp jaeilepp commented Nov 2, 2015

Fixed the links for API.

@larsoner
Copy link
Member

larsoner commented Nov 2, 2015

You shouldn't need to do this to build the docs. I don't need to. It's also a bit misleading, because a lot of these things exist in the root module mne. but they're here referenced from e.g. mne.io.fiff. or so.

@larsoner larsoner mentioned this pull request Nov 2, 2015
Merged
agramfort
agramfort reviewed Nov 2, 2015
View reviewed changes
doc/python_reference.rst
write_bem_surface
write_trans
_make_forward.make_forward_solution
_field_interpolation.make_field_map
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't document private code

@agramfort
Copy link
Member

indeed it should not be necessary. I don't need this either.

@jaeilepp
Copy link
Contributor Author

jaeilepp commented Nov 2, 2015

Well the links from __init__ just don't seem to work for me no matter what I do.

@larsoner
Copy link
Member

larsoner commented Nov 2, 2015

Try upgrading Sphinx maybe? @agramfort are there other packages that might need to be upgraded?

You running Anaconda, or something else?

@agramfort
Copy link
Member

agramfort commented Nov 2, 2015 via email

@jaeilepp
Copy link
Contributor Author

jaeilepp commented Nov 3, 2015

Version 1.3.1

@jaeilepp
Copy link
Contributor Author

jaeilepp commented Nov 3, 2015

And I'm using anaconda.

@agramfort
Copy link
Member

I do have the same problem with anaconda + sphinx 1.3.1

not sure what's wrong but it used to work fine without exposing the internals of full imports.

@jaeilepp
Copy link
Contributor Author

jaeilepp commented Nov 3, 2015

Yeah, it seems to generate the rst files with full path and then tries to link them with the condensed path and fails.

@jaeilepp
Copy link
Contributor Author

jaeilepp commented Nov 4, 2015

After hours of debugging I found that it's Sphinx bug related to sphinx-doc/sphinx#1892.

Building the html now...

@jaeilepp jaeilepp closed this Nov 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jaeilepp @larsoner @agramfort