search

e0404 / matRad

An open source multi-modality radiation treatment planning sytem developed by e0404 @ DKFZ
http://www.matRad.org
Other
226 stars 176 forks source link

Possible bug in TOPAS engine with dij calculation #760

Closed remocristoforetti closed 2 months ago

remocristoforetti commented 2 months ago

strjoint requires a string not a char array. This is a possible fix, I only tested in one specific case, but it's a small fix.

wahln commented 2 months ago

Can you explain the issue?

A problem of this proposed fix is that "string" does not exist in Octave. We need to work with char arrays.

remocristoforetti commented 2 months ago

Ah I see, the error is triggered by strjoin at line 2078. Possibly the char array imageName is not accepted as a valid input. Avoiding the use of strjoin seams to work as well

wahln commented 2 months ago

But strjoin({'test','test2'},';') works without issues on cell arrays of chars, so I still don't get it.

wahln commented 2 months ago

Ah I guess that imageName already is a joined char array, so we don't need strjoin.