It was noticed that absolute Windows paths weren't getting mapped at all on the workers. This is because the pathmapping rules were being changed to using double backslashes \\ as part of a dictionary to string conversion which when set in HOUDINI_PATHMAP Houdini converts them internally into double forward slashes //. This meant that the paths would not be mapped as Houdini uses a single forward slash / as the separator when selecting files and folders on Windows.
What was the solution? (How)
All rules now have a find/replace done on them for \\->/
What is the impact of this change?
Windows absolute paths are now mapped correctly by the adaptor
How was this change tested?
Created a scene which referenced an absolute texture path and had an absolute path on the output of the Mantra node
Ran a submission from a Windows machine to a Linux worker with the current adaptor and saw that the texture was not mapped and the output not found since it was also not mapped
Tested again after these changes and saw that the texture was mapped correctly and the output file was found and uploaded as output correctly.
Was this change documented?
No
Is this a breaking change?
No
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
What was the problem/requirement? (What/Why)
It was noticed that absolute Windows paths weren't getting mapped at all on the workers. This is because the pathmapping rules were being changed to using double backslashes
\\
as part of a dictionary to string conversion which when set in HOUDINI_PATHMAP Houdini converts them internally into double forward slashes//
. This meant that the paths would not be mapped as Houdini uses a single forward slash/
as the separator when selecting files and folders on Windows.What was the solution? (How)
All rules now have a find/replace done on them for
\\
->/
What is the impact of this change?
Windows absolute paths are now mapped correctly by the adaptor
How was this change tested?
Was this change documented?
No
Is this a breaking change?
No
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.