AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Add an easier way to get both active & inactive views for a display

Offen
#1,824 2 Kommentare 1 Reaktion 0 zugewiesene Personen Beansprucht von @lijenicol Auf GitHub ansehen
good first issue help wanted
Vorherrschende Sprache
C++
Sterne
2.1k
Forks
503
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

In OCIO v1, the Config getViews(display) method returned all views. This was considered a bug (per issue #67) and a fix was attempted in closed PR #324 and then finally fixed in PR #875 (as part of OCIO v2).

As a result, in OCIO 2.x, getViews only returns the active views. This is usually what an application would want.

However in some situations it is useful to get all the views (active & inactive). Currently there is this API to do that:
```
int getNumViews(ViewType type, const char * display) const;
const char * getView(ViewType type, const char * display, int index) const;
```
but it requires iterating over both the shared and per-display views for a display. It would be nice to have the equivalent of getNumDisplaysAll and getDisplayAll for views.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie mit dem Lesen von Config::getViews, getNumViews und getView. Vergleichen Sie anschließend deren Verhalten bei aktiven Views mit den entsprechenden getNumDisplaysAll- und getDisplayAll-APIs. Ermitteln Sie die vorgesehene kombinierte API für aktive und inaktive Views und überprüfen Sie, dass Aufrufer alle Views für ein Display abrufen können, ohne Shared- und Pro-Display-Views separat zu durchlaufen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
api
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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