Wrong string formatting causing image upload to crash in 3D Slicer

[che85]

commit 055fdc2 introduces a bug

When using 3D Slicer and trying to upload an image to the server

Traceback (most recent call last):
  File "/Users/herzc/sources/py/MONAILabel/plugins/slicer/MONAILabel/MONAILabel.py", line 1376, in onUploadImage
    if not self.getPermissionForImageDataUpload():
  File "/Users/herzc/sources/py/MONAILabel/plugins/slicer/MONAILabel/MONAILabel.py", line 1363, in getPermissionForImageDataUpload
    _(
KeyError: 'server_url'

MONAILabel/plugins/slicer/MONAILabel/MONAILabel.py

Lines 1363 to 1368 in ac0af6d

	_(
	"Source volume - without any additional patient information -"
	" will be sent to remote data processing server: {server_url}.\n\n"
	"Click 'OK' to proceed with the segmentation.\n"
	"Click 'Cancel' to not upload any data and cancel segmentation.\n"
	).format(self.serverUrl()),

if using string formatting, it should be

            _(
                "Source volume - without any additional patient information -"
                " will be sent to remote data processing server: {}.\n\n"
                "Click 'OK' to proceed with the segmentation.\n"
                "Click 'Cancel' to not upload any data and cancel segmentation.\n"
            ).format(self.serverUrl()),

I don't know if any other functionality is affected by that commit.

[lassoan]

It should be

_( 
     "Source volume - without any additional patient information -" 
     " will be sent to remote data processing server: {server_url}.\n\n" 
     "Click 'OK' to proceed with the segmentation.\n" 
     "Click 'Cancel' to not upload any data and cancel segmentation.\n" 
 ).format(server_url=self.serverUrl()), 

_() is for internationalization (to be able to translate the message to different languages). The named placeholder is not strictly needed if there is only a single placeholder, but useful for translators.

Could you test if the fix above works and maybe submit a pull request to MONAILabel?

[wien2020]

I had the same problem “Failed to run inference in MONAl Label Server.”

Traceback (most recent call last):
File "/Applications/Slicer.app/Contents/Extensions-32390/MONAILabel/lib/Slicer-5.6/qt-scripted-modules/MONAILabel.py", line 1545, in onClickSegmentation
result_file, params = self.logic.infer(model, image_file, params, session_id=self.getSessionId())
File "/Applications/Slicer.app/Contents/Extensions-32390/MONAILabel/lib/Slicer-5.6/qt-scripted-modules/MONAILabel.py", line 1526, in getSessionId
self.onUploadImage(init_sample=False, session=True)
File "/Applications/Slicer.app/Contents/Extensions-32390/MONAILabel/lib/Slicer-5.6/qt-scripted-modules/MONAILabel.py", line 1376, in onUploadImage
if not self.getPermissionForImageDataUpload():
File "/Applications/Slicer.app/Contents/Extensions-32390/MONAILabel/lib/Slicer-5.6/qt-scripted-modules/MONAILabel.py", line 1363, in getPermissionForImageDataUpload
_(
IndexError: Replacement index 0 out of range for positional args tuple