Skip to content

Conversation

@meesfrensel
Copy link
Collaborator

Description

Implement tiling. Currently for panoramas only, but hopefully written in a way that it could be adapted easily to normal images too.

Fixes # (issue)

How Has This Been Tested?

Needs tests.

API Changes

Adds /api/assets/<uuid>/tiles/<zoom level>/<column>/<row>.
Should still add the tiling information to some endpoint. Maybe to /api/assets/<uuid>'s DTO. AssetResponseDto is it called?

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

Please describe to which degree, if any, an LLM was used in creating this pull request.

none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants