Skip to content

Feature/cache transparency 121#188

Closed
intagliated wants to merge 18 commits into
drivendataorg:mainfrom
intagliated:feature/cache-transparency-121
Closed

Feature/cache transparency 121#188
intagliated wants to merge 18 commits into
drivendataorg:mainfrom
intagliated:feature/cache-transparency-121

Conversation

@intagliated
Copy link
Copy Markdown

Adds cache transparency features as requested in issue #121.

Changes Made

  • Added --show-cache-info CLI option to display cache location and total size in MB
  • Added --clear-cache CLI option to remove all cached satellite imagery
  • Removed exists=True validation from samples_path to allow cache options to run without a valid CSV
  • Cache size is calculated recursively and shown with warning when exceeding 500MB
  • Cache location is displayed during prediction operations

Testing

# Show cache information
cyfi predict --show-cache-info

# Clear all cached imagery  
cyfi predict --clear-cache

# Use custom cache directory
cyfi predict samples.csv --cache-dir ./my_cache

This passes --cache-dir through to pipeline loading for both predict commands and adds CLI coverage to verify cache path wiring.

Made-with: Cursor
- Add --show-cache-info to display cache location and total size in MB
- Add --clear-cache to remove all cached satellite imagery
- Remove exists=True from samples_path to allow cache options without file
- Display cache info during prediction operations
- Show warning when cache exceeds 500MB
- Fixes #121
@ejm714
Copy link
Copy Markdown
Collaborator

ejm714 commented May 3, 2026

This PR won't be reviewed until you remove all the extraneous files

@intagliated intagliated closed this by deleting the head repository May 5, 2026
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