Skip to content

Conversation

sebastienblor
Copy link
Collaborator

Changes proposed in this pull request
The first implementation for primvars on instanceable prims didn't fully work depending on how the primvars are defined.
We were not providing a proper "primvars stack" to the context used to read the primvars.

I'm adding a test scene (provided in ticket #944) to test this use case

Issues fixed in this pull request
Fixes #944

@sebastienblor sebastienblor self-assigned this Dec 13, 2021
@sebastienblor sebastienblor added bug Something isn't working procedural Related to the Arnold Procedural labels Dec 13, 2021
@sebastienblor sebastienblor merged commit c1a1a5a into Autodesk:fix-7.0.0 Dec 14, 2021
@sebastienblor sebastienblor deleted the pr/944_b branch December 14, 2021 10:20
sebastienblor added a commit that referenced this pull request Dec 29, 2021
* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions. (#881)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions.

Fixes #880
Fixes #772

* Only call AiBegin/AiEnd in the delegate if no arnold session is active #884 (#885)

* Support render tags in the render delegate. (#883)

* Checking for render tags.
* Tracking render tags for shapes.
* Improving render tag tracking.
* Interrupt before disabling nodes.
* Renaming render tag handling functions and moving the core logic to a separate function.
* Tracking the point instancer's render tag.

Fixes #843

* Improving conversions between integer types. (#889)

* Setting unsigned int parameters from int. (#887)

* Adding more tests and conversions. (#887)

* Passing render session to AiDeviceAutoSelect. (#903)

Fixes #902

* Avoid calling AiRenderBegin when render is already running. (#901)

Fixes #900

* Fixing the use of AI_TYPE_INT AOVs and removing a warning when setting blend_opacity. (#908)

Fixes #905

* Setting the returned version to default in the Sdr plugin. (#907)

Fixes #906

* Using GetVolumeShader if native rprim is volume. (#920)

Fixes #915

* Fix pointInstancer and prototypes visibilities (#916)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions. (#881) (#882)

Fixes #880
Fixes #772

* Updating master with changes from the fix branch (#886)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions. (#881)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions.

Fixes #880
Fixes #772

* Only call AiBegin/AiEnd in the delegate if no arnold session is active #884 (#885)

* Support render tags in the render delegate. (#883)

* Checking for render tags.
* Tracking render tags for shapes.
* Improving render tag tracking.
* Interrupt before disabling nodes.
* Renaming render tag handling functions and moving the core logic to a separate function.
* Tracking the point instancer's render tag.

Fixes #843

Co-authored-by: Sebastien Blaineau-Ortega <sebastien.blaineau.ortega@autodesk.com>

* Merging fix branch to master. (#894)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions. (#881)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions.

Fixes #880
Fixes #772

* Only call AiBegin/AiEnd in the delegate if no arnold session is active #884 (#885)

* Support render tags in the render delegate. (#883)

* Checking for render tags.
* Tracking render tags for shapes.
* Improving render tag tracking.
* Interrupt before disabling nodes.
* Renaming render tag handling functions and moving the core logic to a separate function.
* Tracking the point instancer's render tag.

Fixes #843

* Improving conversions between integer types. (#889)

* Setting unsigned int parameters from int. (#887)

* Adding more tests and conversions. (#887)

Co-authored-by: Sebastien Blaineau-Ortega <sebastien.blaineau.ortega@autodesk.com>

* Merging fix-7.0.0 to master (#909)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions. (#881)

* Fixing testsuite issues using 21.02+ and supporting new and old light parameter naming conventions.

Fixes #880
Fixes #772

* Only call AiBegin/AiEnd in the delegate if no arnold session is active #884 (#885)

* Support render tags in the render delegate. (#883)

* Checking for render tags.
* Tracking render tags for shapes.
* Improving render tag tracking.
* Interrupt before disabling nodes.
* Renaming render tag handling functions and moving the core logic to a separate function.
* Tracking the point instancer's render tag.

Fixes #843

* Improving conversions between integer types. (#889)

* Setting unsigned int parameters from int. (#887)

* Adding more tests and conversions. (#887)

* Passing render session to AiDeviceAutoSelect. (#903)

Fixes #902

* Avoid calling AiRenderBegin when render is already running. (#901)

Fixes #900

* Fixing the use of AI_TYPE_INT AOVs and removing a warning when setting blend_opacity. (#908)

Fixes #905

* Setting the returned version to default in the Sdr plugin. (#907)

Fixes #906

Co-authored-by: Sebastien Blaineau-Ortega <sebastien.blaineau.ortega@autodesk.com>

* Updating changelog and readme for Arnold 7. (#898)

Fixes #897

* Fix pointInstancer and prototypes visibilities #458

* Updating changelog and readme for Arnold 7. (#898)

Fixes #897

* Fix pointInstancer and prototypes visibilities #458

Co-authored-by: Pal Mezei <sirpalee@users.noreply.github.com>

* Update tests for Arnold 7. (#922) (#923)

* Don't write the default color manager node to usd #924 (#925)

* Remap curves primvars (#926)

* Remap curves primvars values #921

* Simplify calls to RemapCurvesVertexPrimvars #921

* Changing the default descriptor returned for AOVs. (#919)

Fixes #918

* Apply point instancer matrix to all instances #928 (#929)

Fixes #928

* Set product metadata. (#933)

Fixes #932

* Add support for primvars on instances #944 (#948)

* Add render session to AiMsg calls #953 (#954)

* Call AiResolveFilePath with procedural type #942 (#952)

* Support USD 21.11 (#958)

* Support USD 21.11

* Port schemas to USD 21.11 #957

* Fix build of usd_imaging with USD 21.11 #957 (#961)

* Fix build of usd_imaging with USD 21.11 #957

* Mising ifdefs for older versions of USD #957

* Fix primvars on instanceable prims #944 (#965)

* Extend the skinning baking interval to the surrounding integer frames #951 (#979)

* Support curves UVs #975 (#977)

* Light and shadow links default ignoreRoots to false #987 (#988)

* Read USD builtin normals on meshes #962 (#986)

* Read UsdLuxShadowAPI parameters #945 (#990)

Co-authored-by: Pal Mezei <sirpalee@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working procedural Related to the Arnold Procedural
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants