Skip to content
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.

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

Broken links in the documentation #1627

Closed
jd41 opened this issue May 28, 2020 · 3 comments
Closed

Broken links in the documentation #1627

jd41 opened this issue May 28, 2020 · 3 comments
Assignees
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.

Comments

@jd41
Copy link
Contributor

jd41 commented May 28, 2020

I write about both of these problems at once because I suspect that they have, in part, the same cause.

I used the tool linkchecker to check the NEST documentation of a ~1-week-old source (commit #4e9a69fab) for broken links:
jd41@pc:~$ sudo apt install linkchecker.

In <install-directory>/share (generated by help_generator?):
jd41@pc:~/nestnew/share$ find .|grep html$|xargs -d '\n' linkchecker --check-extern>linkchecker_installed.txt
Result: linkchecker_installed.txt (66 errors)

In <source-directory>/doc/_build (Sphinx build - doesn't currently get installed?)
jd41@pc:~/nest-simulator/doc/_build$ find .|grep html$|xargs -d '\n' linkchecker --check-extern>linkchecker_sphinx.txt
Result: linkchecker_sphinx.txt (38 errors)

In the help_generator documentation, the majority of cases are links to assumed HTML files in the directory /home/jd41/nestnew/share/doc/nest/help/cc/, which doesn't exist for me (e.g. /home/jd41/nestnew/share/doc/nest/help/cc/spike_generator.html). For the two examples I checked (spike_generator and iaf_psc_alpha), I can't access the corresponding help pages in the NEST SLI interface either. I was able to access them in my last build (from February). Might there be a relation to #1530?

This clearly checks only for links in pre-existing or built HTML files. I know there are some links in ipynb files as well (which I didn't check); I don't know if there are links anywhere else. I can confirm that no additional errors are generated by any HTML files in the whole source directory.

@jd41 jd41 changed the title broken links/nest.help("iaf_psc_exp") doesn't work Broken links/nest.help("iaf_psc_exp") doesn't work May 28, 2020
@jd41
Copy link
Contributor Author

jd41 commented May 29, 2020

@steffengraber

@jd41 jd41 changed the title Broken links/nest.help("iaf_psc_exp") doesn't work Broken links/nest.help("iaf_psc_alpha") doesn't work Jun 2, 2020
@terhorstd terhorstd added I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. labels Jun 9, 2020
@terhorstd
Copy link
Contributor

terhorstd commented Jun 9, 2020

I think this is an important step to the test-ability of documentation. Referential integrity is one point of a list @jessica-mitchell started some time ago. Others were for example: doctests, language linting, doc-style tests (a la PEP8, but for RST), ...
We would have to discuss where/how this fits into a general solution though, and where/when this could best run. There are many aspects to this, therefore I put it into the 'Doc' and 'Infrastructure' teams.

@steffengraber steffengraber self-assigned this Jun 10, 2020
@terhorstd terhorstd changed the title Broken links/nest.help("iaf_psc_alpha") doesn't work Broken links in the documentation Oct 21, 2020
@sarakonradi
Copy link
Contributor

At the NEST Hackathon, we agreed to close this issue and decouple help_generator from link checking. The former will be discussed offline. The latter is addressed in #1907.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.
Projects
Status: Done
Development

No branches or pull requests

4 participants