AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

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

Aperta
#1,824 2 commenti 1 reazione 0 assegnatari Rivendicata da @lijenicol Vedi su GitHub
good first issue help wanted
Lingua principale
C++
Stelle
2.1k
Fork
503
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia leggendo Config::getViews, getNumViews e getView, quindi confronta il loro comportamento relativo alle Views attive con le API analoghe getNumDisplaysAll e getDisplayAll. Determina l'API combinata prevista per le Views attive e inattive e verifica che i chiamanti possano recuperare tutte le Views di un Display senza iterare separatamente le Views condivise e quelle specifiche del Display.

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

Valutazione

Stack tecnologico
cpp
Ambito
api
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.