bjlittle / geovista

Cartographic rendering and mesh analytics powered by PyVista
https://geovista.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
159 stars 26 forks source link

use locks yml on rtd #688

Closed bjlittle closed 8 months ago

bjlittle commented 8 months ago

🚀 Pull Request

Description

This pull-request uses the explicit YAML file equivalent of the conda-lock environment file on readthedocs to build the conda environment.

Roughly, the major costs of building the documentation on readthedocs are:

This pull-request reduces the mamba env create task by ~40 seconds.

There can be large differences in runtime when building the documentation as we don't have any dedicated resource on the readthedocs server. How quickly our requests are serviced is affected by community demand. So the above values are speculative at best, and based on a small sample of job runs.


codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1765802) 94.65% compared to head (91371f7) 94.65%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #688 +/- ## ======================================= Coverage 94.65% 94.65% ======================================= Files 129 129 Lines 4900 4900 ======================================= Hits 4638 4638 Misses 262 262 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

trexfeathers commented 8 months ago

Thanks so much for giving this a go, @bjlittle!