Skip to content

Conversation

silverjam
Copy link
Contributor

@silverjam silverjam commented Jun 9, 2022

Description

  • numpy is technically needed for the fftmonitor, but since that's the only thing that uses it I think we've intentionally omitted it from our dependency list... this breaks the documentation generator though, since it tries import the package-- previously this was installed via the generator (somehow) -- now we're explicitly installing the dependency in the tox.ini command that's used to run the documentation generator

  • Don't run autoremove in the dockerfile, this seems to be uninstalling enchant or something else that the pyenchant package needs

  • Fix Sphinx config error seen in recent release:

 sphinx-build -b spelling -d build/doctrees  -W source build/spelling
Running Sphinx v5.0.1
Warning, treated as error:
Invalid configuration value found: 'language = None'. Update your configuration to a valid langauge code. Falling back to 'en' (English).
Creating file ./sbp.rst.
Creating file ./sbp.acquisition.rst.
Creating file ./sbp.bootload.rst.

API compatibility

N/A

@silverjam silverjam requested a review from a team as a code owner June 9, 2022 03:24
@silverjam silverjam requested a review from a team as a code owner June 9, 2022 03:36
@silverjam silverjam requested review from adrian-kong and joelynch June 9, 2022 03:39
@silverjam silverjam force-pushed the silverjam/docker-and-doc-fixups branch from 69ef788 to e2fe861 Compare June 9, 2022 03:39
@fpezzinosn
Copy link
Contributor

I had the same error where one of the constructs test kept failing. This is the fix I found #1156 (comment)

@silverjam
Copy link
Contributor Author

I had the same error where one of the constructs test kept failing. This is the fix I found #1156 (comment)

I haven't had issues with the parallel tests fairly so frequently until now... I may have to undo the work I did to attempt to run them in parallel...

@silverjam
Copy link
Contributor Author

Force merging since this has to do with stability of the build

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

Successfully merging this pull request may close these issues.

2 participants