Skip to content

Commit

Permalink
CI: Remove caching of Cartopy maps
Browse files Browse the repository at this point in the history
The maps are now stored on S3, so caching isn't going to benefit much.
Also, caching had not really been working reliably since we seem to be
running up against storage limits due to our caching packages.
  • Loading branch information
dopplershift committed Sep 10, 2021
1 parent a3424de commit ff0a5a5
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 62 deletions.
15 changes: 0 additions & 15 deletions .github/workflows/docs-conda.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,21 +62,6 @@ jobs:
- name: Install dependencies
run: mamba install --quiet --yes --file ci/doc_requirements.txt --file ci/extra_requirements.txt --file ci/requirements.txt

# This imports CartoPy to find its map data cache directory
- name: Get CartoPy maps dir
id: cartopy-cache
run: echo "::set-output name=dir::$(python -c 'import cartopy;print(cartopy.config["data_dir"])')"

- name: Setup mapdata caching
uses: actions/[email protected]
env:
# Increase to reset cache of map data
CACHE_NUMBER: 0
with:
path: ${{ steps.cartopy-cache.outputs.dir }}
key: docs-cartopy-${{ env.CACHE_NUMBER }}
restore-keys: docs-cartopy-

- name: Install
# For some reason on Windows 3.7 building the wheel fails to properly include our extra
# stuff. Executing the egg_info beforehand for some reason fixes it. No idea why. We're
Expand Down
15 changes: 0 additions & 15 deletions .github/workflows/docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,21 +79,6 @@ jobs:
python -m pip install -c ci/${{ matrix.dep-versions }} numpy
python -m pip install -r ci/doc_requirements.txt -r ci/extra_requirements.txt -c ci/${{ matrix.dep-versions }}
# This imports CartoPy to find its map data cache directory
- name: Get CartoPy maps dir
id: cartopy-cache
run: echo "::set-output name=dir::$(python -c 'import cartopy;print(cartopy.config["data_dir"])')"

- name: Setup mapdata caching
uses: actions/[email protected]
env:
# Increase to reset cache of map data
CACHE_NUMBER: 0
with:
path: ${{ steps.cartopy-cache.outputs.dir }}
key: docs-cartopy-${{ env.CACHE_NUMBER }}
restore-keys: docs-cartopy-

- name: Install self
run: python -m pip install -c ci/${{ matrix.dep-versions }} .

Expand Down
15 changes: 0 additions & 15 deletions .github/workflows/tests-conda.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,21 +58,6 @@ jobs:
- name: Install dependencies
run: mamba install --quiet --yes --file ci/test_requirements.txt --file ci/extra_requirements.txt --file ci/requirements.txt

# This imports CartoPy to find its map data cache directory
- name: Get CartoPy maps dir
id: cartopy-cache
run: echo "::set-output name=dir::$(python -c 'import cartopy;print(cartopy.config["data_dir"])')"

- name: Setup mapdata caching
uses: actions/[email protected]
env:
# Increase to reset cache of map data
CACHE_NUMBER: 0
with:
path: ${{ steps.cartopy-cache.outputs.dir }}
key: cartopy-${{ env.CACHE_NUMBER }}
restore-keys: cartopy-

- name: Install
# For some reason on Windows 3.7 building the wheel fails to properly include our extra
# stuff. Executing the egg_info beforehand for some reason fixes it. No idea why. We're
Expand Down
17 changes: 0 additions & 17 deletions .github/workflows/tests-pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,23 +94,6 @@ jobs:
- name: Install test dependencies
run: python -m pip install -r ci/test_requirements.txt -c ci/${{ matrix.dep-versions }}

# This imports CartoPy to find its map data cache directory
- name: Get CartoPy maps dir
if: ${{ matrix.no-extras != 'No Extras' }}
id: cartopy-cache
run: echo "::set-output name=dir::$(python -c 'import cartopy;print(cartopy.config["data_dir"])')"

- name: Setup mapdata caching
if: ${{ steps.cartopy-cache.outputs.dir != '' }}
uses: actions/[email protected]
env:
# Increase to reset cache of map data
CACHE_NUMBER: 0
with:
path: ${{ steps.cartopy-cache.outputs.dir }}
key: cartopy-${{ env.CACHE_NUMBER }}
restore-keys: cartopy-

- name: Install
run: python -m pip install -c ci/${{ matrix.dep-versions }} .

Expand Down

0 comments on commit ff0a5a5

Please sign in to comment.