DCMSeg: Patch for missing method getSegmentAlgorithmName

All other questions regarding DCMTK

Moderator: Moderator Team

Post Reply
Message
Author
ChristianHerz
Posts: 22
Joined: Mon, 2007-08-27, 10:37
Contact:

DCMSeg: Patch for missing method getSegmentAlgorithmName

#1 Post by ChristianHerz »

Hi developers,

I created a patch for adding a missing method "getSegmentAlgorithmName" which is needed for dcmqi(https://github.com/QIICR/dcmqi) from converting itk2dcm and dcm2itk including usage of jsoncpp for reading/wring a JSON metadata file.

https://github.com/che85/dcmtk/commit/6 ... f95c.patch

Thanks for patching.

P.S. If it's possible, please change my nickname here to ChristianHerz

Thanks,
Christian

Michael Onken
DCMTK Developer
Posts: 2048
Joined: Fri, 2004-11-05, 13:47
Location: Oldenburg, Germany
Contact:

Re: DCMSeg: Patch for missing method getSegmentAlgorithmName

#2 Post by Michael Onken »

Hi Christian,

thanks for the hint and patch, I'll integrate it next days and link to the commit here ;)

Best,
Michael

P.S: Colleagues from OFFIS might be able to change your user name, I'll see them later this week and ask (if they do not act before)

Jan Schlamelcher
OFFIS DICOM Team
OFFIS DICOM Team
Posts: 318
Joined: Mon, 2014-03-03, 09:51
Location: Oldenburg, Germany

Re: DCMSeg: Patch for missing method getSegmentAlgorithmName

#3 Post by Jan Schlamelcher »

Michael Onken wrote:P.S: Colleagues from OFFIS might be able to change your user name, I'll see them later this week and ask (if they do not act before)
I already did it...

Michael Onken
DCMTK Developer
Posts: 2048
Joined: Fri, 2004-11-05, 13:47
Location: Oldenburg, Germany
Contact:

Re: DCMSeg: Patch for missing method getSegmentAlgorithmName

#4 Post by Michael Onken »

:o

Michael Onken
DCMTK Developer
Posts: 2048
Joined: Fri, 2004-11-05, 13:47
Location: Oldenburg, Germany
Contact:

Re: DCMSeg: Patch for missing method getSegmentAlgorithmName

#5 Post by Michael Onken »

Hi Christian,

I finally added the patch to DCMTK. The related commit (641ea6f8cc59) will show up during the next days in the public OFFIS master branch.

Best,
Michael

Post Reply

Who is online

Users browsing this forum: Ahrefs [Bot], Bing [Bot], Google [Bot] and 1 guest