You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The issue with m_bol is not new, e.g. #9040: m_bol and M_bol are conflicting on case-insensitive file systems, e.g. MacOS... And I guess tox is adding the option to treat warnings as error with Sphinx.
I got several of these warnings when I run it inside a conda env:
WARNING: Discarding $PYTHONPATH from environment, to override specify PYTHONPATH in 'passenv' in your configuration.
But other than that, docs built fine on Linux, so it is consistent with what @saimn said. I knew that astropy.units.function.logarithmic.m_bol looked familiar.
I vaguely some discussion between @bsipocz and @astrofrog about treating the warnings as errors recently.
make html does complete successfully, with the same warning that would turn into an error in tox. I think that tox -e build_docs should likewise complete and give a result even with errors. E.g. sometimes it is easier to understand a sphinx warning or error by looking at the output. One might add something like tox -e build_docs_strict to be strict.
/Users/aldcroft/git/astropy/docs/units/index.rst.rst:402: WARNING: autosummary: stub file not found 'astropy.units.function.logarithmic.m_bol'. Check your autosummary_generate setting.
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
[changelog_links] Adding changelog links to "changelog"
[changelog_links] Adding changelog links to "whatsnew/0.1"
[changelog_links] Adding changelog links to "whatsnew/0.2"
[changelog_links] Adding changelog links to "whatsnew/0.3"
[changelog_links] Adding changelog links to "whatsnew/0.4"
[changelog_links] Adding changelog links to "whatsnew/1.0"
[changelog_links] Adding changelog links to "whatsnew/1.1"
[changelog_links] Adding changelog links to "whatsnew/1.2"
[changelog_links] Adding changelog links to "whatsnew/1.3"
[changelog_links] Adding changelog links to "whatsnew/2.0"
[changelog_links] Adding changelog links to "whatsnew/3.0"
[changelog_links] Adding changelog links to "whatsnew/3.1"
[changelog_links] Adding changelog links to "whatsnew/3.2"
[changelog_links] Adding changelog links to "whatsnew/4.0"
[changelog_links] Adding changelog links to "whatsnew/4.1"
[changelog_links] Adding changelog links to "whatsnew/index"
WARNING: dot code 'digraph inheritancead6d126c6f {\nrankdir=LR;\nsize="8.0, 12.0";\n "AstropyWarning" [URL="../../api/astropy.utils.exceptions.AstropyWarning.html#astropy.utils.exceptions.AstropyWarning",fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)",target="_top",tooltip="The base warning class from which all Astropy warnings should inherit."];\n "ConfigItem" [URL="../../api/astropy.config.ConfigItem.html#astropy.config.ConfigItem",fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)",target="_top",tooltip="A setting and associated value stored in a configuration file."];\n "ConfigNamespace" [URL="../../api/astropy.config.ConfigNamespace.html#astropy.config.ConfigNamespace",fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)",target="_top",tooltip="A namespace of configuration items. Each subpackage with"];\n "ConfigurationMissingWarning" [URL="../../api/astropy.config.ConfigurationMissingWarning.html#astropy.config.ConfigurationMissingWarning",fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)",target="_top",tooltip="A Warning that is issued when the configuration directory cannot be"];\n "AstropyWarning" -> "ConfigurationMissingWarning" [arrowsize=0.5,style="setlinewidth(0.5)"];\n "InvalidConfigurationItemWarning" [URL="../../api/astropy.config.InvalidConfigurationItemWarning.html#astropy.config.InvalidConfigurationItemWarning",fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)",target="_top",tooltip="A Warning that is issued when the configuration value specified in the"];\n "AstropyWarning" -> "InvalidConfigurationItemWarning" [arrowsize=0.5,style="setlinewidth(0.5)"];\n "_SetTempPath" [fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)"];\n "set_temp_cache" [URL="../../api/astropy.config.set_temp_cache.html#astropy.config.set_temp_cache",fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)",target="_top",tooltip="Context manager to set a temporary path for the Astropy download cache,"];\n "_SetTempPath" -> "set_temp_cache" [arrowsize=0.5,style="setlinewidth(0.5)"];\n "set_temp_config" [URL="../../api/astropy.config.set_temp_config.html#astropy.config.set_temp_config",fontname="Vera Sans, DejaVu Sans, Liberation Sans, Arial, Helvetica, sans",fontsize=10,height=0.25,shape=box,style="setlinewidth(0.5)",target="_top",tooltip="Context manager to set a temporary path for the Astropy config, primarily"];\n "_SetTempPath" -> "set_temp_config" [arrowsize=0.5,style="setlinewidth(0.5)"];\n}\n': dot exited with error:
[stderr]
b''
[stdout]
b''
** LOTS MORE LIKE THAT **
generating indices... genindex py-modindexdone
highlighting module code... [100%] numpy.ma.core
writing additional pages... searchdone
copying images... [100%] wcs/../_build/plot_directive/wcs/index-1.png
copying downloadable files... [100%] timeseries/binned.csv
copying static files... ... done
copying extra files... done
dumping search index in English (code: en)... done
dumping object inventory... done
build succeeded, 13 warnings.
The HTML pages are in _build/html.
Sphinx-gallery successfully executed 0 out of 0 files subselected by:
gallery_conf["filename_pattern"] = '^((?!skip_).)*$'
gallery_conf["ignore_pattern"] = '__init__\\.py'
after excluding 9 files that had previously been run (based on MD5).
embedding documentation hyperlinks...
embedding documentation hyperlinks for generated/examples... [100%] fits-tables.html
Build finished. The HTML pages are in _build/html.
Activity
pllim commentedon Mar 10, 2020
Does a plain
make htmlwork?saimn commentedon Mar 10, 2020
The issue with
m_bolis not new, e.g. #9040:m_bolandM_bolare conflicting on case-insensitive file systems, e.g. MacOS... And I guess tox is adding the option to treat warnings as error with Sphinx.pllim commentedon Mar 10, 2020
I got several of these warnings when I run it inside a
condaenv:But other than that, docs built fine on Linux, so it is consistent with what @saimn said. I knew that
astropy.units.function.logarithmic.m_bollooked familiar.I vaguely some discussion between @bsipocz and @astrofrog about treating the warnings as errors recently.
taldcroft commentedon Mar 10, 2020
I don't think that ever in my life have I built astropy docs without getting warnings...
taldcroft commentedon Mar 10, 2020
make htmldoes complete successfully, with the same warning that would turn into an error in tox. I think thattox -e build_docsshould likewise complete and give a result even with errors. E.g. sometimes it is easier to understand a sphinx warning or error by looking at the output. One might add something liketox -e build_docs_strictto be strict.taldcroft commentedon Mar 10, 2020
Another nit is that I get this:
It needs to be
pip install -e '.[docs]', at least on zsh.bsipocz commentedon Mar 10, 2020
@pllim - Yes, we indeed talked about the exact same problem, but haven't yet followed up with a PR. I'll try to remember to make one later today.
pllim commentedon Mar 10, 2020
Re: zsh -- Maybe add a note to dev docs about it?
pllim commentedon Jul 22, 2021
Well, docs have been building since, but if there is an actionable item to follow up on, please open new issue. Thanks!