Fix path handling in SANS isis_reductions_steps #10715
Labels
High Priority
An issue or pull request that if not addressed is severe enough to postponse a release.
SANS
Issues and pull requests related to SANS
Milestone
The path handling in
scripts/SANS/isis_reduction_steps.py
is inconsistent. Some places useos.path.join
and others simply add/
separators. These all should harmonize on usingos.path.join
.Another related problem is path handling when loading the pixel correction file on line 1581 of
scripts/SANS/isis_reduction_steps.py
. The load is done via an ```eval}} call where all of the code is in a string. If the self._pixel_file is a windows path that happens to contain a directory name that starts with a character that can be interpreted as an escape character, e.g.where the
\r
would be interpreted as a carriage return then the path is invalid and the command fails. In this particular case I think line 1581 just needs to be changed toto make the
Filename
string a raw string so that escape characters are ignored.The text was updated successfully, but these errors were encountered: