Closed niksirbi closed 3 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 99.28%. Comparing base (
cf312d2
) to head (56c9905
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
No data about Duplication
Description
What is this PR
Why is this PR needed?
All the CI tests (including Build Sphinx Docs) were passing on PR 116, but Build Sphinx Docs failed after merging to main.
I suspected that this was due to PR 133 which was merged just ahead of PR 116. Here's what I think happened:
Because we switched to requiring
pandas
frompandas[hdf5]
, there was no longer a declared pip dependency ontables
. This is not a problem for users or during tests, because of theconda install pytables
which is run ahead of the pip install. However, only pip install steps are run during the docs build, so the GitHub runner that executes Build Sphinx Docs doesn't installtables
.What does this PR do?
It adds
tables
as a pip dependency to thedocs/requirements.txt
. Alternatively, we could have added it as a "normal" dependency inpyproject.toml
or switched back topandas[hdf5]
. Not sure which one is best. All of them would fix this problem (assuming I've diagnosed it correctly).No
conda install pytables
is required here, because the Build Sphinx Docs action runs in an Ubuntu runner, which should come with HDF5 libraries pre-installed.References
116
133
How has this PR been tested?
Yes, locally and in the CI of this PR.
Is this a breaking change?
No.
Does this PR require an update to the documentation?
No.
Checklist: