AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Make the environment section of the config mandatory

Abierto
#1,083 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Feature Request Needs Discussion
Lenguaje dominante
C++
Estrellas
2.1k
Forks
503
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

In PR #322 (and discussed in Issue #149), Malcolm Humphreys implemented a nice feature that adds an optional environment section at the top of a config file to declare the context variables used in the config. This serves several purposes:

1. It allows the config author to specify default values
2. It makes the config easier to understand since one may see at a glance which context vars are used
3. It makes it easy for OCIO to only load the necessary environment variables into the Context object
4. It allows additional validation options.

Point 3 has performance implications. In sophisticated studio setups, there may be 100 or more variables in the environment (with the vast majority not being used by the config). Nevertheless, if the environment section is not present, OCIO currently loads all of these into the Context object and this slows down the generation of the cacheIDs. The slowdown may be significant for timeline-based applications that need to update the context rapidly.

Given the many benefits, we propose making the environment section mandatory for OCIO v2.

When this was proposed at a recent OCIO v2 working group meeting, there were people both for and against the proposal, so I'm creating this issue to get feedback from a wider audience.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

El issue propone hacer obligatoria la sección de entorno de configuración para OCIO v2, pero lo plantea como un tema de discusión e informa de que hubo desacuerdo en una reunión del grupo de trabajo. Lee la propuesta y sus referencias a PR #322 y Issue #149; el issue no menciona archivos ni pruebas. El trabajo no está listo para comenzar hasta que haya una decisión sobre la propuesta y un alcance de implementación concreto.

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

Evaluación

Stack tecnológico
cpp
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
20/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.