OpenAssetIO / usdOpenAssetIOResolver

An AR2 plugin that hosts OpenAssetIO
Apache License 2.0
24 stars 3 forks source link

UsdOpenAssetIOResolver Skeleton #3

Closed elliotcmorris closed 1 year ago

elliotcmorris commented 1 year ago

Closes #1

Adds the foundational project structure for a USD resolver projects, along with a single test, CI integration, and the formatting configuration from the main openassetIO repository.

USD has actually gone to some effort to make itself easy to build against once installed, initially I had assumed there would need to be a FindUSD.cmake file, but you can go completely without it, and any other real attempts to locate USD, as USD sets itself up on the system path to be findable by cmake out of the box.

Currently, this resolver merely logs the methods that are called, before forwarding through to DefaultResolver funcitonality.

The CI integration used the asfw/ci-vfxall:2022 image, which comes preinstalled with USD. Downloading the image takes some time, but no where near as much as building USD would take.

The test opens an empty stage, and verifies that some debug output has been logged, more to come on this.

elliotcmorris commented 1 year ago

New actions from forks don't seem to be running on this repo, (perhaps there's a permission I don't see, i'm a bit befuddled by it.)

Putting the PR up anyway for review, but won't merge til' we figure it out and get the test running.

elliotcmorris commented 1 year ago

Seems that initial commits dont run CI, so here's it running on my fork: https://github.com/elliotcmorris/usdOpenAssetIOResolver/pull/2

elliotcmorris commented 1 year ago

Slightly redid the debug logging to be formatted more clearly, after discovering it was a pain when we were together, and added the result outputs (plus missing inputs) when appropriate. Looks like this now :

image

e48a3c8

elliotcmorris commented 1 year ago

Fork branch tests pass, merging : https://github.com/elliotcmorris/usdOpenAssetIOResolver/actions/runs/3987780390/jobs/6838053253