AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Move config checking code from the ociocheck app into the core library

Abierto
#1,949 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Feature Request help wanted
Lenguaje dominante
C++
Estrellas
2.1k
Forks
505
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

The ociocheck command-line app currently provides the most thorough validation of a config. Some of these checks would be quite useful for a variety of apps that use OCIO (and ocioview would be a prime example). But in order to make use of them, the checks should move from the ociocheck app itself into the core library where they are accessible from the API.

The current validate method on the Config class is very useful as a quick method of checking a config, but it does not try things such as resolving and loading FileTransforms (ociocheck loads all FileTransforms). It would be best to keep validate a quick check since that is already widely used by client apps. So any time-consuming tests or tests that require loading external files should be put in a separate function/method.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Empieza comparando el método Config::validate de la biblioteca principal con la validación que realiza la aplicación de línea de comandos ociocheck. Identifica qué comprobaciones requieren resolver o cargar FileTransforms y determina una API independiente para ellas, manteniendo validate rápido; al completar la implementación, esas comprobaciones deben estar disponibles para clientes como ocioview.

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

Evaluación

Stack tecnológico
cpp
Área
backend-api-design
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
30/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.