Maya looks: support for native Redshift texture format

[Allan-I]

Brief description

Add support for native Redshift textures handling. Closes #2599

Description

Uses Redshift's Texture Processor executable to convert textures being used in renders to the Redshift ".rstexbin" format.

Additional info

Essentially emulating what maketx does by passing a number of arguments to a subprocess and returning a status code to indicate whether the operation was successful or not.

[ynbot]

Task linked: OP-2524 Maya looks: support for native Redshift texture format

[ghost]

Thank you Murphy, can you please recreate your environment through create_env in tools and try again?

[m-u-r-p-h-y]

run the command

but the publish ended with exactly the same error :(

[ghost]

Thank you Murphy, investigating now

[BigRoy]

look with textures extracted and published successfully

@m-u-r-p-h-y - Shouldn't that produce .rstexbin files with this PR instead of .tx files?

Did you enable to rstex attribute on the look instance?

[m-u-r-p-h-y]

look with textures extracted and published successfully

@m-u-r-p-h-y - Shouldn't that produce .rstexbin files with this PR instead of .tx files?

Did you enable to rstex attribute on the look instance?

Ohh you are completely right. I had old look instance in my scene.

Redid the test and it is correctly producing rstexbin files.

my question is, do we really need to specify the format in the instance? Isn't it clear that we should do

arnold -> tx redshift -> rxtexbin

this way it opens a space for human error as I already demonstrated . . .

[BigRoy]

my question is, do we really need to specify the format in the instance? Isn't it clear that we should do arnold -> tx redshift -> rxtexbin

Actually I'd say yes. I believe Redshift can actually render both .tx and .rstexbin files but they serve a different purpose. (I might be wrong however!) I recall having done a test on our infrastructure and it was usually faster to render without .rstexbin files and use regular textures or .tx files. But then again, I might be misremembering.

Check the Render Time Considerations

this way it opens a space for human error as I already demonstrated . . .

🙉 🙈 🙊

[antirotor]

I agree that we need to specify format, but we'll need to add in the future validators to deal with unsupported features enabled on existing render instance - for example arnold/vray/renderman won't understand .rstexbin, renderman will not understand .tx/.rstexbin but will read .tex and so on.

[m-u-r-p-h-y]

I agree that we need to specify format, but we'll need to add in the future validators to deal with unsupported features enabled on existing render instance - for example arnold/vray/renderman won't understand .rstexbin, renderman will not understand .tx/.rstexbin but will read .tex and so on.

the toggle in render instance could only state

• [ ] convert textures to native format

and we should know, depending on renderer, which format to use . . .

[ghost]

Would that be ideal? To have that toggle on the instance?

[m-u-r-p-h-y]

Would that be ideal? To have that toggle on the instance?

we have this currently

and it is not ideal definitely