AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

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

Aperta
#1,949 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Feature Request help wanted
Lingua principale
C++
Stelle
2.1k
Fork
503
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia confrontando il metodo Config::validate nella libreria principale con la validazione eseguita dall’applicazione a riga di comando ociocheck. Identifica quali controlli richiedono la risoluzione o il caricamento di FileTransforms e determina un’API separata per questi, mantenendo validate veloce; al completamento dell’implementazione, questi controlli dovranno essere disponibili per client come ocioview.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
backend-api-design
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
30/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.