Remove IECoreHoudini, IECoreMaya and IECoreNuke #1473
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
It hasn't been maintained for a long time. We can resurrect if someone wants to step up and maintain it full time.
This contains interesting contributions that never made it into the main codebase, and which are now out of date.
This will continue life as an internal Image Engine project.
This will continue life as an internal Image Engine project.
This will continue life as an internal project at Image Engine.
ivanimanishi
approved these changes
Jul 3, 2025
|
@murraystevenson, anything to add? I'd like to move on to the next round of refactoring if possible. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The open source landscape has changed considerably since Cortex was first released way back in 2007. Cortex itself no longer has any real traction outside of Image Engine, and this has left Cortex leading two lives. Firstly as the core of Gaffer, which is seeing increasingly widespread adoption, and secondly as the core of the Image Engine pipeline, which uses a much broader set of functionality than Gaffer, and to great effect.
This dual life has made maintenance difficult, and we've been making less progress than we'd like as a result. "Team Gaffer" has been burdened by the volume of code with no relevance to Gaffer, and Image Engine has been burdened by maintaining much of the code in public when it is used only internally. This PR starts the process of separating these two concerns by removing significant modules not needed by Gaffer, allowing them to return home to Image Engine.
We don't anticipate this affecting anyone, because we believe that all public usage of Cortex is mediated via Gaffer. If you have any concerns, please speak now (but be prepared to help out)!