AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Move config checking code from the ociocheck app into the core library
- 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