Skip to content

Commit 607f315

Browse files
JustAlejandropixar-oss
authored andcommitted
Set the projection override even if the projection parameter list is empty.
The 'PxrOrthographic' projection plugin doesn't have any parameter and we still want to be able to set it. There is also the case for any other projection plugin where no parameter has been set and we rely on RenderMan default values. (Internal change: 2382547)
1 parent 55a32ff commit 607f315

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

third_party/renderman-26/plugin/hdPrman/renderPass.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -752,10 +752,8 @@ HdPrman_RenderPass::_Execute(
752752
projection,
753753
projectionParams);
754754

755-
if (projectionParams.GetNumParams() != 0) {
756-
cameraContext.SetProjectionOverride(
757-
RtUString(projection.c_str()), projectionParams);
758-
}
755+
cameraContext.SetProjectionOverride(
756+
RtUString(projection.c_str()), projectionParams);
759757
}
760758

761759
// Set Resolution, Crop Window, Pixel Aspect Ratio,

third_party/renderman-27/plugin/hdPrman/renderPass.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -761,10 +761,8 @@ HdPrman_RenderPass::_Execute(
761761
projection,
762762
projectionParams);
763763

764-
if (projectionParams.GetNumParams() != 0) {
765-
cameraContext.SetProjectionOverride(
766-
RtUString(projection.c_str()), projectionParams);
767-
}
764+
cameraContext.SetProjectionOverride(
765+
RtUString(projection.c_str()), projectionParams);
768766
}
769767

770768
// Set Resolution, Crop Window, Pixel Aspect Ratio,

0 commit comments

Comments
 (0)