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

2015-07-30 Thread Julien Lamy
Le 24/07/2015 23:16, Philip Hands a écrit :
> 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.

Done
(https://anonscm.debian.org/cgit/debian-med/dcmtkpp.git/commit/?id=d53d4e8be0f25b6787cdf126f3d968e3e9e6fb61)

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

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.

Thanks for your help!
-- 
Julien



signature.asc
Description: OpenPGP digital signature


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

2015-07-24 Thread Philip Hands
Julien Lamy  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


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.


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  writes:
> 
>>> * Package name: dcmtkpp
>>>
>>>   Version : 0.2.1
>>>   Upstream Author : Julien Lamy 
>>>
>>> * 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


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  writes:

> > * Package name: dcmtkpp
> > 
> >   Version : 0.2.1
> >   Upstream Author : Julien Lamy 
> > 
> > * 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.


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

2015-07-23 Thread Philip Hands
Julien Lamy  writes:

> Package: wnpp
> Severity: wishlist
> Owner: Debian Med team 
>
> * Package name: dcmtkpp
>   Version : 0.2.1
>   Upstream Author : Julien Lamy 
> * 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

You should also make sure that you either expand acronyms (if that
helps, or just make it obvious by context.

In this case I don't think 'Service Class User' and 'Service Class Provider'
add much in the way of clarity, so I'd suggest something along the lines
of:

  ... implementation of both client (SCU) and server (SCP) elements of
  the DICOM protocol.

I'm sure you can do better that that, being familiar with the meanings of
these things.

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


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

2015-07-23 Thread Julien Lamy
Package: wnpp
Severity: wishlist
Owner: Debian Med team 

* Package name: dcmtkpp
  Version : 0.2.1
  Upstream Author : Julien Lamy 
* 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.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org