Remove SceneObject::getBinding() functions which don't fetch output for the binding

Open Beginner friendly
#320 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
64/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Active
Tech stack
cmake, cpp
Domain
backend

Research direction

Start by locating SceneObject::getBinding() declarations and implementations, then inspect TestSceneObject.cc and search the repository for calls to the legacy overloads. Update the test to use the new functions and remove remaining legacy calls and definitions. Done when no legacy calls remain and the project builds with the new API.

Written by the indexing model from the issue text.

Description

Remove support for the legacy SceneObject::getBinding() functions which don't know about output and update TestSceneObject.cc to use the new getBinding() functions. This should also include a thorough search for any remaining calls to the legacy getBinding() functions.

Dominant language
CMake
Stars
4.7k
Forks
321
Avg merge
13m
Merged PRs (30d)
3

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from OpenMoonRay/openmoonray

All issues in OpenMoonRay/openmoonray

Similar issues

More Backend & API Design issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.