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

DOC sphinx fairlearn logo link is broken #899

Closed
hildeweerts opened this issue Jul 14, 2021 · 2 comments · Fixed by #955
Closed

DOC sphinx fairlearn logo link is broken #899

hildeweerts opened this issue Jul 14, 2021 · 2 comments · Fixed by #955

Comments

@hildeweerts
Copy link
Contributor

hildeweerts commented Jul 14, 2021

Describe the issue linked to the documentation

Currently, the Fairlearn logo in the docs generated by sphinx points to "../_static/_static/images/fairlearn_full_color.png" which doesn't exist.

It seems that '_static' is currently prepended in html_logo = "_static/images/fairlearn_full_color.png" in conf.py even though we also define the html_static_path = ['_static']. I expect that this used to be a bug in sphinx and that it is now fixed there, breaking our link.

Suggest a potential alternative/fix

In conf.py replace line 144:

html_logo = "_static/images/fairlearn_full_color.png"

with:

html_logo = "/images/fairlearn_full_color.png"

UPDATE: As shown in #901 this apparently wasn't the right solution. When we change the url like this the image cannot be found at all while building the docs (WARNING: logo file '/images/fairlearn_full_color.png' does not exist).

UPDATE 2: It turns out we are not the only one with this issue: see this issue. So we need to make sure to update to sphinx 4.1.1 and then all will be fine.

@hildeweerts hildeweerts added documentation easy Relatively simple issues, harder than "good first issue" help wanted good first issue Good for newcomers and removed easy Relatively simple issues, harder than "good first issue" labels Jul 14, 2021
@svenkat19
Copy link

I have replaced html_logo = "_static/images/fairlearn_full_color.png" in conf.py. Is there anything else I must look into? Pls guide me

@hildeweerts
Copy link
Contributor Author

Hi @svenkat19! Did you open a pull request for the change? You can find more information on how to contribute to fairlearn in this helpful guide by @romanlutz: https://github.com/romanlutz/fairlearn-scipy-sprint/blob/main/README.md

And specifics on how to build the documentation locally can be found here: https://fairlearn.org/main/contributor_guide/contributing_documentation.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants