Skip to content

Fix image search path for Metal render test suite#2667

Merged
jstone-lucasfilm merged 2 commits intoAcademySoftwareFoundation:mainfrom
kwokcb:msl_image_path_fix
Nov 13, 2025
Merged

Fix image search path for Metal render test suite#2667
jstone-lucasfilm merged 2 commits intoAcademySoftwareFoundation:mainfrom
kwokcb:msl_image_path_fix

Conversation

@kwokcb
Copy link
Copy Markdown
Contributor

@kwokcb kwokcb commented Nov 12, 2025

Issue

Fixes: #2664.
where some tests would not find images in relative paths.

Change

Pass image search path to image handler search path for MSL test renderer.

Results

Example results. There are no image missing warnings anymore.

image

Notes

  • This is a subtle issue related to unused args as well as it is not caught due to the arg
    being commented out.
  • Recommend that an missing images should be tagged as render failures instead of warnings in test suite.

Passed in image search path was not being used
by image handler so some images would not be found.
@kwokcb kwokcb requested a review from ld-kerley November 12, 2025 03:48
Copy link
Copy Markdown
Member

@jstone-lucasfilm jstone-lucasfilm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks @kwokcb!

@jstone-lucasfilm jstone-lucasfilm merged commit 6029ef4 into AcademySoftwareFoundation:main Nov 13, 2025
32 checks passed
Copy link
Copy Markdown
Contributor

@ld-kerley ld-kerley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM too

@kwokcb kwokcb deleted the msl_image_path_fix branch November 22, 2025 02:54
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.

Metal test suite rendering : image files are not found

3 participants