Skip to content

Conversation

proberts-cinesite
Copy link
Contributor

@proberts-cinesite proberts-cinesite commented Sep 20, 2025

IECoreScene.CurvesPrimitiveEvaluator.Result now provides accessors for interpolation coefficients and indices.

Generally describe what this PR will do, and why it is needed.

Adds public accessors for interpolation coefficients and indices to CurvesPrimitiveEvaluator.Result.

CurvesPrimitiveEvaluator.Result only supports interpolation of some data types. e.g. It does not support interpolation of Quatf data. Providing public accessors for the interpolation coefficients and indices allows client code to implement its own interpolation for such data types using the coefficients and indices computed by the CurvesPrimitiveEvaluator.Result class.

Checklist

  • I have read the contribution guidelines.
  • I have updated the documentation, if applicable.
  • I have tested my change(s) in the test suite, and added new test cases where necessary.
  • My code follows the Cortex project's prevailing coding style and conventions.

@proberts-cinesite proberts-cinesite force-pushed the CurvesPrimitiveEvaluatorResultAddInterpolationIndicesAndWeightsToApi branch from b9670b2 to 4b97bba Compare September 20, 2025 07:34
@proberts-cinesite proberts-cinesite changed the title IECoreScene CurvesPrimitiveEvaluator Result : Api additions IECoreScene CurvesPrimitiveEvaluator Result : Add coefficients/indices accessors Sep 20, 2025
@proberts-cinesite proberts-cinesite changed the base branch from main to RB-10.6 September 20, 2025 08:15
@proberts-cinesite proberts-cinesite force-pushed the CurvesPrimitiveEvaluatorResultAddInterpolationIndicesAndWeightsToApi branch 2 times, most recently from f6e8a5d to ba4c69d Compare September 20, 2025 08:54
IECoreScene.CurvesPrimitiveEvaluator.Result now provides accessors for interpolation coefficients and indices.
@proberts-cinesite proberts-cinesite force-pushed the CurvesPrimitiveEvaluatorResultAddInterpolationIndicesAndWeightsToApi branch from ba4c69d to 0cb7f76 Compare September 21, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant