Skip to content

ENH: set the parameters visible in Slicer's Ultrasound Remote Control#38

Merged
adamrankin merged 1 commit intoPlusToolkit:masterfrom
dzenanz:master
Feb 8, 2021
Merged

ENH: set the parameters visible in Slicer's Ultrasound Remote Control#38
adamrankin merged 1 commit intoPlusToolkit:masterfrom
dzenanz:master

Conversation

@dzenanz
Copy link
Copy Markdown
Contributor

@dzenanz dzenanz commented Feb 8, 2021

This avoids continuous stream of error messages:

|ERROR|300.652000| Command execution failed| in M:\Dev\Plus\SPc\PlusLib\src\PlusServer\vtkPlusCommandProcessor.cxx(183)

which are caused by a decision that absence of a setting is interpreted as error:
https://github.com/PlusToolkit/PlusLib/blob/50fa158100edb131f1e85ff9fff88cc7176c2409/src/PlusServer/Commands/vtkPlusGetUsParameterCommand.cxx#L223-L241

This avoids continuous stream of error messages:

|ERROR|300.652000| Command execution failed| in M:\Dev\Plus\SPc\PlusLib\src\PlusServer\vtkPlusCommandProcessor.cxx(183)

which are caused by a decision that absence of a setting is interpreted as error:
https://github.com/PlusToolkit/PlusLib/blob/50fa158100edb131f1e85ff9fff88cc7176c2409/src/PlusServer/Commands/vtkPlusGetUsParameterCommand.cxx#L223-L241
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants