AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

DisplayViewTransform should fall back to display color space aliases

Abierto
#2,337 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
C++
Estrellas
2.1k
Forks
505
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Currently, OCIO displays do not have aliases. This makes it difficult to change the name of a display in a config without breaking backwards compatibility (and using the inactive list is not ideal). Regular color spaces, including display color spaces already have this feature via the aliases attribute of the color space.

For displays that use views that define their display_colorspace as `` (which is most views in modern config files), I propose that if a DisplayViewTransform is unable to find the requested display that it fall back to looking at the aliases for all display color spaces.

One possible unintended consequence is that it would potentially allow some DisplayViewTransforms that legitimately should have failed to now succeed. I'm unable to think of a likely example of where that would be a problem, but we could potentially add a config-level attribute such as `allow_display_aliases: true` to require config authors to opt-in to the behavior.

Similar to the display scenario, I propose that if a DisplayViewTransform is unable to find the view, that it search in a new aliases attribute of the ViewTransforms.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza rastreando cómo DisplayViewTransform resuelve displays y views, y después compara ese comportamiento con el atributo aliases existente en los espacios de color normales. Revisa cómo se gestiona display_colorspace establecido en en los archivos de configuración modernos. Se considera terminado cuando exista un comportamiento acordado y probado para los alias de display y ViewTransform, incluida la cuestión de si quienes escriben archivos de configuración deben optar explícitamente por habilitarlos.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cpp
Área
computer-graphics
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Activo
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.