Retrieve object's width and height

[pablodiegoss]

Refactoring Proposal

Describe the design issue(s) you have identified in the code
Currently we do not retrieve the objects width and height after the user uploads a file, which creates problems with scaling on thumbnails and previews, its specifically worse for videos

We should retrieve an objects width and height, so that we can fix #473 and #319

[vjpixel]

Análise & pesquisa (#690)

Onde a info seria usada hoje:

• Thumbnails: core/models.py:20-23 (constants 50x50) + Object.as_html_thumbnail() linha 400, Marker.as_html_thumbnail() linha 262 — atualmente forçam 50×50.
• Renderização AR/MR: Object.as_html(height, width) linha 364, recebe height/width via parâmetro mas o caller não tem como inferir corretamente sem metadata — daí o stretch em vídeos (Broken "see all artworks page" and "create-exhibition" #473, Mp4 video proportions breaking create artwork #319).
• Artwork model: scale_x/scale_y (models.py:432-433) hoje refletem só o slider do usuário (forms.py:291-292).
• Vídeos sem aspect ratio nativo renderizam 1:1 — origem do problema.

Por tipo de arquivo:

Tipo	Lib	Tamanho	Dep runtime	Notas
gif (raster)	Pillow (já tem)	—	—	Image.open(file).size
mp4 / webm	ffprobe via subprocess	—	ffmpeg binário do sistema (~40 MB apt)	mais barato e confiável
mp4 / webm (alt)	av	~15 MB wheel	libav embutido	sem subprocess overhead, dep nova
glb	pygltflib (~0.5 MB) ou trimesh (~2 MB)	—	—	retorna bounding box 3D, precisa traduzir pra width/height 2D

Já no pyproject.toml: Pillow, boto3, django-storages. Tudo o que precisa pra GIF.

Onde plugar:

• UploadObjectForm.clean_source (src/core/forms.py:85-104) → extrair width/height após validar extensão.
• UploadObjectForm.save (linha 119-125) → persistir nos novos campos.
• Object model: adicionar width/height como IntegerField(null=True, blank=True). Migrations + ObjectEvent (pghistory) acompanham.

Backfill dos existentes:

• Management command extract_object_dimensions: percorre Object.objects.filter(width__isnull=True), abre via instance.source.open() (S3 ou local), extrai e salva.
• Bibliotecas runtime tem que estar disponíveis (ffprobe no host de produção). Bucket público; ffprobe aceita URL direto.
• Em produção provavelmente vale rodar via Celery / job assíncrono pra não bloquear a request loop em ~milhares de objetos.

Recomendação minha:

1. Fase 1 (pequena): Pillow para GIF. Migration adicionando width/height nullable. Cobre parte do problema de stretch e desbloqueia thumbnails proporcionais nos GIFs.
2. Fase 2: subprocess + ffprobe pra mp4/webm. Adicionar ffmpeg ao Dockerfile (apt-get install -y ffmpeg, ~40 MB). Sem dep Python nova.
3. Fase 3 (opcional): pygltflib para GLB se quiser bounding box. Pode ficar pra outro momento.
4. Backfill via management command, idempotente, processando uma fila por vez.

A fase 1 é mecânica (~50 linhas + migration). A fase 2 introduz dependência sistêmica (ffmpeg) — vale confirmar com você antes de empurrar pro Dockerfile.

@pablodiegoss me confirma se posso seguir com a fase 1 (só GIF via Pillow + adicionar campos) e abrir issues separadas para fases 2/3?

---

Generated by Claude Code

[vjpixel]

Review do PR #947 (Fase 1)

Verdict: ✅ Pronto pra merge — escopo intencionalmente pequeno.

Implementa apenas o que dá pra fazer com Pillow (já em deps). Fases 2/3 ficam pendentes da decisão sobre ffmpeg/glb (ver comentário anterior).

Mudanças:

• Object.width e Object.height (PositiveIntegerField(null=True, blank=True)).
• Migration core.0028 adiciona os campos. objectevent (pghistory) recebe os campos automaticamente.
• UploadObjectForm.save() extrai dimensões via Pillow só quando file_extension == "gif". Outros (mp4/webm/glb) ficam null até as fases seguintes.
• Management command backfill_object_dimensions com --dry-run, log de erros, skip de unreadable. Itera só GIFs faltando dimensões.

Verificado:

• Upload de GIF 320×240 → row salva com width=320, height=240.
• Upload de MP4 → fields ficam null (esperado).
• Comando dry-run lista sem persistir.

Backward compat: campos nullable, legacy rows ficam null até backfill rodar. Nenhum template lê width/height ainda — usar essa info na renderização AR/MR é PR separado depois que a backfill rodar.

CI verde. ~150 linhas. Risco zero por ser aditivo.

---

Generated by Claude Code

[pablodiegoss]

Artwork model: scale_x/scale_y (models.py:432-433) hoje refletem só o slider do usuário (forms.py:291-292).

This has nothing to do with the main issue