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
DCMSeg: Patch for missing method getSegmentAlgorithmName
Moderator: Moderator Team
-
- Posts: 22
- Joined: Mon, 2007-08-27, 10:37
- Contact:
-
- DCMTK Developer
- Posts: 2048
- Joined: Fri, 2004-11-05, 13:47
- Location: Oldenburg, Germany
- Contact:
Re: DCMSeg: Patch for missing method getSegmentAlgorithmName
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)
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)
-
- OFFIS DICOM Team
- Posts: 318
- Joined: Mon, 2014-03-03, 09:51
- Location: Oldenburg, Germany
Re: DCMSeg: Patch for missing method getSegmentAlgorithmName
I already did it...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)
-
- DCMTK Developer
- Posts: 2048
- Joined: Fri, 2004-11-05, 13:47
- Location: Oldenburg, Germany
- Contact:
-
- DCMTK Developer
- Posts: 2048
- Joined: Fri, 2004-11-05, 13:47
- Location: Oldenburg, Germany
- Contact:
Re: DCMSeg: Patch for missing method getSegmentAlgorithmName
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
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
Who is online
Users browsing this forum: Ahrefs [Bot], Bing [Bot], Google [Bot] and 1 guest