In RAiDER.aria.calcGUNW.py, TROPO_GROUP is defined as follows:
https://github.com/dbekaert/RAiDER/blob/87efdce2246419878983d7606338f8fb0f04d54b/tools/RAiDER/aria/calcGUNW.py#L112-L18
Later, it is used like this, where it is split into a list by spaces:
|
for k in TROPO_GROUP.split(): |
|
h5 = h5[k] |
However,
TROPO_GROUP has no spaces so this always produces a list with just one element. Is this meant to split by slashes instead? Or is the value of
TROPO_GROUP wrong?
To add to the confusion, the basename is also taken of this variable as if it is a path, although nothing else seems to indicate it is a path.
|
logger.info('Updated %s group in: %s', os.path.basename(TROPO_GROUP), path_gunw) |
What is TROPO_GROUP supposed to be? Is all this code behaving as expected?
Desktop (please complete the following information):
In
RAiDER.aria.calcGUNW.py,TROPO_GROUPis defined as follows:https://github.com/dbekaert/RAiDER/blob/87efdce2246419878983d7606338f8fb0f04d54b/tools/RAiDER/aria/calcGUNW.py#L112-L18
Later, it is used like this, where it is split into a list by spaces:
RAiDER/tools/RAiDER/aria/calcGUNW.py
Lines 112 to 113 in 87efdce
However,
TROPO_GROUPhas no spaces so this always produces a list with just one element. Is this meant to split by slashes instead? Or is the value ofTROPO_GROUPwrong?To add to the confusion, the basename is also taken of this variable as if it is a path, although nothing else seems to indicate it is a path.
RAiDER/tools/RAiDER/aria/calcGUNW.py
Line 174 in 87efdce
What is
TROPO_GROUPsupposed to be? Is all this code behaving as expected?Desktop (please complete the following information):
commit fe38b70 (HEAD -> linting, origin/linting)
Author: Nate Kean 14845347+garlic-os@users.noreply.github.com
Date: Wed Jul 17 13:27:30 2024 -0500
Add Linting RAiDER codebase #672 to Unreleased