AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Make the environment section of the config mandatory

Offen
#1,083 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Feature Request Needs Discussion
Vorherrschende Sprache
C++
Sterne
2.1k
Forks
503
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Der Issue schlägt vor, den Konfigurationsabschnitt für die Umgebung für OCIO v2 verpflichtend zu machen, stellt dies jedoch als Diskussionsanstoß dar und berichtet von Uneinigkeit bei einer Sitzung der Arbeitsgruppe. Lies den Vorschlag und seine Verweise auf PR #322 und Issue #149; im Issue werden keine Dateien oder Tests genannt. Mit der Arbeit kann erst begonnen werden, wenn eine Entscheidung über den Vorschlag und ein konkreter Implementierungsumfang vorliegen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
20/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.