search

FNNDSC / pypx

Python wrapper to interact with a PACS (based off DCMTK and PyDicom)
90 stars 25 forks source link

Fix construction of required parameters #11

Open aravindhp opened 3 years ago

aravindhp commented 3 years ago

serverPort in commandSuffix() is being treated as an integer when constructing a string. This commit fixes that by converting it into a string. Ref: https://github.com/FNNDSC/ChRIS_ultron_backEnd/issues/222#issuecomment-725556064

aravindhp commented 3 years ago

Looks like this did not fix the issue seen in https://github.com/FNNDSC/ChRIS_ultron_backEnd/issues/222#issuecomment-725556064

aravindhp commented 3 years ago

The problem was with how I was testing the fix. This PR fixed the issue. Please review @rudolphpienaar