Add PBR material support for planar texture alignment

[Aqil-Ahmad]

Description

Implements planar texture alignment for PBR materials, allowing checkbox to work with PBR texture transforms.

Related Issues

Fixes #2975

Issue Link:

---

Checklist

Please ensure the following before requesting review:

• I have provided a clear title and detailed description for this pull request.
• If useful, I have included media such as screenshots and video to show off my changes.
• The PR is linked to a relevant issue with sufficient context.
• I have tested the changes locally and verified they work as intended.
• All new and existing tests pass.
• Code follows the project's style guidelines.
• Documentation has been updated if needed.
• Any dependent changes have been merged and published in downstream modules
• I have reviewed the contributing guidelines.

---

Additional Notes

20251031-0933-17.2299779.mp4

[Aqil-Ahmad]

/claim #2975

[Copilot]

Pull Request Overview

This PR implements planar texture alignment support for PBR (Physically Based Rendering) materials, extending the existing alignment functionality that previously only worked with Blinn-Phong materials. The changes enable the planar alignment checkbox to properly function with PBR texture transforms.

Key Changes:

• Added a new overloaded calcAlignedPlanarTE() method that accepts explicit transform parameters for PBR materials
• Modified LLPanelFaceSetAlignedTEFunctor to handle both PBR and Blinn-Phong materials based on user selection
• Updated alignment logic to apply transforms to appropriate PBR texture channels (individual or all channels when "Complete material" is selected)

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
indra/newview/llpanelface.cpp	Core implementation of PBR-aware planar alignment, including functor modifications and logic to apply transforms to PBR material channels
indra/newview/llface.h	Declaration of new overloaded calcAlignedPlanarTE() method accepting explicit transform parameters
indra/newview/llface.cpp	Implementation of new overloaded method that calculates alignment using provided transform parameters instead of querying from texture entry

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Aqil-Ahmad]

@akleshchev thanks for review! ci build was failing, i think the errors are not related to what files i changed. should i resolve those in the other files?

[akleshchev]

ci build was failing, i think the errors are not related to what files i changed. should i resolve those in the other files?

Doesn't look like your fault, but please update from develop to try and resolve it as it runs fine in other PRs.

[Aqil-Ahmad]

but please update from develop to try and resolve it as it runs fine in other PRs.

done!

[akleshchev]

I think you need to provide screenshots or videos for this to proceed. But otherwise this should be up to bennettgoble.

[Aqil-Ahmad]

I think you need to provide screenshots or videos for this to proceed. But otherwise this should be up to bennettgoble.

Hi @akleshchev! video attached.

[akleshchev]

Not sure what's the expected behavior here. For all I know this might be correct or might be wrong. But either way, comparison of texture to PBR:

[bennettgoble]

@Aqil-Ahmad Thanks for your work so far. Unfortunately, the alignment is not working as expected. Here's an example:

Given: two planar mapped faces (A, B) with PBR materials.

1. Rotate face A by some amount (ex. 20).
2. Select face B, then A
3. Check "Align planar faces" box

Expected: Faces should align their repeats
Actual: Faces do not align

For an example of how the faces should align see the image above. The upper non-PBR faces have their offset and rotation correctly applied. Performing an identical operation on the PBR materials has a different result. You can see how the face guidelines are continuous across faces.

[Aqil-Ahmad]

Expected: Faces should align their repeats Actual: Faces do not align

the misalignment is likely due to the transform convention difference (pbr and bling pong texture).

working on it.