Re: Bug#793392: ITP: dcmtkpp -- Wrappers around DCMTK to have an easier API

2015-07-30 Thread Andreas Tille
Hi Julien,

On Thu, Jul 30, 2015 at 05:24:50PM +0200, Julien Lamy wrote:
 
 Hopefully also done. The short description will be meaningful to medical
 imaging people: in my experience most of them know what DICOM is, and
 those that develop DICOM applications (target audience of the package)
 will at least have heard about DCMTK. As you stated, other people will
 find nothing meaningful in DCMTK or DICOM and should quickly move away
 from this package.

Sounds good.  Just ping me if you consider the package ready for a
sponsored upload.

Kind regards

 Andreas.

-- 
http://fam-tille.de


-- 
To UNSUBSCRIBE, email to debian-med-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20150730152841.gs7...@an3as.eu



Re: Bug#793392: ITP: dcmtkpp -- Wrappers around DCMTK to have an easier API

2015-07-24 Thread Steve M. Robbins
On July 24, 2015 06:09:10 PM Julien Lamy wrote:

 Philip and Steve: thank you for your suggestions. I have written the ITP
 bug report way too quickly, and the description indeed needs a lot of
 clarifying. My apologies for this.
 
 Would you agree with the following modifications?

Suggested text works for me.  

I hope to find time very soon to actually try the code!
-Steve



signature.asc
Description: This is a digitally signed message part.


Re: Bug#793392: ITP: dcmtkpp -- Wrappers around DCMTK to have an easier API

2015-07-24 Thread Philip Hands
Julien Lamy l...@unistra.fr writes:
...
 Would you agree with the following modifications?

 Short description: C++ wrapper library for DCMTK (DICOM toolkit)

I think it ought to mention medical imaging or some similarly
understandable (to an outsider) phrase in the short description.

Then again, looking at the DCMTK packages, their descriptions strike me
as pretty useless, which I see is just a reflection of the web site for
DCMTK, which does not have the word medical anywhere on its extensive
main page, only mentioning the type of images on linked pages such as
the 'Introduction to DICOM' page.

The 'amide' package seems to have gone the other way:

  software for Medical Imaging

which strikes me as better, and has the benefit of turning up in
apt-cache searches for all of DICOM, DCMTK and medical, whereas the
dicom* packages don't come up for the 'medical' search. i.e.:

  apt-cache search medical

Also, I note that the extra info you included makes it plain that DCMTK
is also a C++ library, which is something that I'd not worked out from
the long description.  You might want to make sure that's clear.

That's perhaps completely obvious to the target audience though, in
which case the only important thing is to make sure that non medics will
get the message Not for you as quickly as possible, preferably in the
short description.

Strangely enough I think the dcmtk* packages achieve that, since:

  OFFIS DICOM toolkit command line utilities

tells me that I have no interest whatsoever in this, because I've no
idea what OFFIS or DICOM stand for (or at least that was the case last
week ;-) )

So, either make the short description meaningful to all, or make it
obviously uninteresting/incomprehensible to the uninterested.

Cheers, Phil.
-- 
|)|  Philip Hands  [+44 (0)20 8530 9560]  HANDS.COM Ltd.
|-|  http://www.hands.com/http://ftp.uk.debian.org/
|(|  Hugo-Klemm-Strasse 34,   21075 Hamburg,GERMANY


signature.asc
Description: PGP signature


Re: Bug#793392: ITP: dcmtkpp -- Wrappers around DCMTK to have an easier API

2015-07-24 Thread Steve M. Robbins
On July 23, 2015 08:44:04 PM Philip Hands wrote:
 Julien Lamy l...@unistra.fr writes:

  * Package name: dcmtkpp
  
Version : 0.2.1
Upstream Author : Julien Lamy l...@unistra.fr
  
  * URL : https://github.com/lamyj/dcmtkpp
  * License : CeCILL-B
  
Programming Lang: C++
Description : Wrappers around DCMTK to have an easier API
  
  DCMTK++ is a set of wrappers around DCMTK, leveraging C++ constructs to
  have an easier API, notably for the networking part. Included are
  exception-based error handling, generic access to datasets elements,
  standard JSON representation of datasets, explicit messages and generic
  implementation of SCU and SCP.
 
 Some indication of what field of endeavour this package might be of
 interest to would be helpful.
 
 It seems, after some searching, that DCMTK is an acronym encompassing the
 acronym DICOM, which is something to do with medical imaging.
 
 Medical imagining is a minority interest, so it would be good if the
 short description allowed the majority of people to quickly determine
 that they don't need this.
 
 Something like:  C++ library for dealing with DICOM medical imagery

Philip: I agree with essentially all your points.  

However, while acknowledging that medical imaging is indeed minority 
interest: within that community, DCMTK is a very well-known DICOM toolkit, so 
I'd advocate for retaining DCMTK in the short description.

Perhaps: C++ wrapper library for DCMTK (DICOM medical imaging toolkit)

-Steve


signature.asc
Description: This is a digitally signed message part.


Re: Bug#793392: ITP: dcmtkpp -- Wrappers around DCMTK to have an easier API

2015-07-24 Thread Julien Lamy
Le 24/07/2015 16:43, Steve M. Robbins a écrit :
 On July 23, 2015 08:44:04 PM Philip Hands wrote:
 Julien Lamy l...@unistra.fr writes:
 
 * Package name: dcmtkpp

   Version : 0.2.1
   Upstream Author : Julien Lamy l...@unistra.fr

 * URL : https://github.com/lamyj/dcmtkpp
 * License : CeCILL-B

   Programming Lang: C++
   Description : Wrappers around DCMTK to have an easier API

 DCMTK++ is a set of wrappers around DCMTK, leveraging C++ constructs to
 have an easier API, notably for the networking part. Included are
 exception-based error handling, generic access to datasets elements,
 standard JSON representation of datasets, explicit messages and generic
 implementation of SCU and SCP.

 Some indication of what field of endeavour this package might be of
 interest to would be helpful.

 It seems, after some searching, that DCMTK is an acronym encompassing the
 acronym DICOM, which is something to do with medical imaging.

 Medical imagining is a minority interest, so it would be good if the
 short description allowed the majority of people to quickly determine
 that they don't need this.

 Something like:  C++ library for dealing with DICOM medical imagery
 
 Philip: I agree with essentially all your points.  
 
 However, while acknowledging that medical imaging is indeed minority 
 interest: within that community, DCMTK is a very well-known DICOM toolkit, 
 so 
 I'd advocate for retaining DCMTK in the short description.
 
 Perhaps: C++ wrapper library for DCMTK (DICOM medical imaging toolkit)

Philip and Steve: thank you for your suggestions. I have written the ITP
bug report way too quickly, and the description indeed needs a lot of
clarifying. My apologies for this.

Would you agree with the following modifications?

Short description: C++ wrapper library for DCMTK (DICOM toolkit)

Long description:
DCMTK++ is a wrapper library for DCMTK, a toolkit handling the DICOM
medical imaging standard. DCMTK++ leverages C++ constructs to provide a
more user-friendly API, notably for the networking part. Included in
DCMTK++ are exception-based error handling, generic access to datasets
elements, standard JSON representation of datasets, and generic
implementation of messages, clients and servers for the various DICOM
protocols.

Other relevant information:
Compared to the two main free-software C++ DICOM toolkits (DCMTK and
GDCM), DCMTK++ provides either additional features (generic
implementation of DICOM servers, missing from GDCM) and a modern C++ API
(missing from DCMTK). We think that developers of DICOM application
would benefit from DCMTK++, in term of code readability and ease of
application maintenance.

The package will be maintained by the Debian Med team [1], and packaging
is underway on Alioth [2].

[1] https://lists.debian.org/debian-med/2015/07/msg00112.html
[2] https://anonscm.debian.org/cgit/debian-med/dcmtkpp.git/

I hope this clarifies your concerns; don't hesitate to come back to me
if you have more questions.

Best regards,
-- 
Julien Lamy



signature.asc
Description: OpenPGP digital signature


Re: Bug#793392: ITP: dcmtkpp -- Wrappers around DCMTK to have an easier API

2015-07-24 Thread Andreas Tille
Hi Phil,

On Fri, Jul 24, 2015 at 10:16:01PM +0100, Philip Hands wrote:
 Julien Lamy l...@unistra.fr writes:
 ...
  Would you agree with the following modifications?
 
  Short description: C++ wrapper library for DCMTK (DICOM toolkit)
 
 I think it ought to mention medical imaging or some similarly
 understandable (to an outsider) phrase in the short description.
 
 Then again, looking at the DCMTK packages, their descriptions strike me
 as pretty useless, which I see is just a reflection of the web site for
 DCMTK, which does not have the word medical anywhere on its extensive
 main page, only mentioning the type of images on linked pages such as
 the 'Introduction to DICOM' page.
 
 The 'amide' package seems to have gone the other way:
 
   software for Medical Imaging
 
 which strikes me as better, and has the benefit of turning up in
 apt-cache searches for all of DICOM, DCMTK and medical, whereas the
 dicom* packages don't come up for the 'medical' search. i.e.:
 
   apt-cache search medical
 
 Also, I note that the extra info you included makes it plain that DCMTK
 is also a C++ library, which is something that I'd not worked out from
 the long description.  You might want to make sure that's clear.
 
 That's perhaps completely obvious to the target audience though, in
 which case the only important thing is to make sure that non medics will
 get the message Not for you as quickly as possible, preferably in the
 short description.
 
 Strangely enough I think the dcmtk* packages achieve that, since:
 
   OFFIS DICOM toolkit command line utilities
 
 tells me that I have no interest whatsoever in this, because I've no
 idea what OFFIS or DICOM stand for (or at least that was the case last
 week ;-) )
 
 So, either make the short description meaningful to all, or make it
 obviously uninteresting/incomprehensible to the uninterested.

Fully ACK.  Any competent volunteer to fix the dcmtk desctiption (which
would be better as if I would try this).

Kind regards

   Andreas.

-- 
http://fam-tille.de


-- 
To UNSUBSCRIBE, email to debian-med-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20150724215851.gs26...@an3as.eu