There's some odd code in subprocess_utils.py relating to the use of six, a Python2/3 compatibility lib. This currently prevents subprocess utils from importing correctly, which breaks other modules.
There's no known need to support Python2 for the InSAR workflow, so it's not clear why Passang is using it. Possibly cruft from cut and paste code? @sixy6e do you have any insights here?
truth-quark
commented
3 years ago
There's some odd code in
subprocess_utils.pyrelating to the use ofsix, a Python2/3 compatibility lib. This currently prevents subprocess utils from importing correctly, which breaks other modules.There's no known need to support Python2 for the InSAR workflow, so it's not clear why Passang is using it. Possibly cruft from cut and paste code? @sixy6e do you have any insights here?
The required fixes are:
sixis not imported fromluigi): https://github.com/GeoscienceAustralia/gamma_insar/blob/pygamma_workflow/insar/subprocess_utils.py#L10dictiteritems()methods here: https://github.com/GeoscienceAustralia/gamma_insar/blob/pygamma_workflow/insar/subprocess_utils.py#L60-L62