Skip to content

CI model cache - #969

Merged
Josef-Haupt merged 3 commits into
mainfrom
ci-model-cache
Jul 29, 2026
Merged

CI model cache#969
Josef-Haupt merged 3 commits into
mainfrom
ci-model-cache

Conversation

@Josef-Haupt

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 27, 2026 19:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves CI efficiency by introducing a persistent cache location for BirdNET model/label downloads and caching that directory across workflow runs, reducing repeated downloads during test jobs.

Changes:

  • Define a fixed BIRDNET_APP_DATA directory under the workspace for model/label downloads.
  • Add an actions/cache step to persist the BirdNET model directory across runs.
  • Ignore the local CI cache directory in .gitignore to avoid accidental commits.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
.gitignore Ignores the new CI model cache directory.
.github/workflows/ci.yml Sets BIRDNET_APP_DATA and caches the directory with actions/cache.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml
Comment on lines +48 to +51
# Cache the downloaded models across runs. Keyed on the birdnet dependency
# (via pyproject.toml) and the OS only - the model files are identical across
# Python versions, so all matrix jobs on an OS share one cache. birdnet
# re-downloads only missing/updated models, so restoring an older cache is safe.
@Josef-Haupt
Josef-Haupt merged commit 1d75285 into main Jul 29, 2026
9 checks passed
@Josef-Haupt
Josef-Haupt deleted the ci-model-cache branch July 29, 2026 11:14
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