Skip to content

Revamp#6

Merged
kylebarron merged 14 commits intomainfrom
kyle/revamp
Feb 10, 2026
Merged

Revamp#6
kylebarron merged 14 commits intomainfrom
kyle/revamp

Conversation

@kylebarron
Copy link
Member

@kylebarron kylebarron commented Feb 9, 2026

Change list

  • Remove dependency on aiocache. User should be able to define caching themselves via the store input.
  • Remove ctx. We shouldn't manage any "file context" inside of this class
  • Update typing
  • Expose header as public attribute in .header
  • Remove io.py module with implementations for connecting to S3/GCS/Azure. Those connections should all happen via obstore.
  • Rename Reader to PMTilesReader
  • Add py.typed to declare as typed.
  • Simplify pyproject.toml to use uv
  • Update tests

Related upstream PRs

Questions

  • I named the backend "store" to be consistent with obstore etc. We could call it client if preferred.
  • Instead of having an async context manager, we can just have an async class constructor, like open. This matches how async-geotiff has await GeoTIFF.open(input). We don't have any resources to destroy so we don't need to have it be an async context manager.

Closes #5

@kylebarron kylebarron merged commit 9e57c1f into main Feb 10, 2026
8 of 12 checks passed
@kylebarron kylebarron mentioned this pull request Feb 10, 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.

Switch to Obstore

2 participants