AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Make the environment section of the config mandatory

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

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

L’issue propone di rendere obbligatoria la sezione dell’ambiente di configurazione per OCIO v2, ma presenta la questione come uno spunto di discussione e riferisce di un disaccordo durante una riunione del gruppo di lavoro. Leggi la proposta e i riferimenti a PR #322 e Issue #149; l’issue non indica file né test. Il lavoro non è pronto per iniziare finché non ci sarà una decisione sulla proposta e un ambito di implementazione concreto.

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

Valutazione

Stack tecnologico
cpp
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
20/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.