No subject
Sun Apr 3 09:54:08 EDT 2011
add our own "handlers" (whatever it means).
I'm not sure what it implies though.
Julien.
On Fri, Aug 19, 2011 at 12:14 PM, Daniel Haehn <haehn at bwh.harvard.edu>wrote:
> Hi Julien,
>
> I tried it but it did not change anything.. :/
>
> Daniel
>
> On Fri, Aug 19, 2011 at 12:03 PM, Julien Finet <julien.finet at kitware.com>
> wrote:
> > Maybe QVector<double> needs to be entered in the meta object system.
> > Can you try adding
> >
> > qRegisterMetaType<QVector<double> > ("QVector<double>");
> >
> > in the constructor function of ctkMatrixWidget (better: init() of
> > ctkMatrixWidgetPrivate if it exists)
> > j.
> > On Fri, Aug 19, 2011 at 11:55 AM, Daniel Haehn <haehn at bwh.harvard.edu>
> > wrote:
> >>
> >> Hi again,
> >>
> >> so it seems that PythonQt has problems to wrap a QVector<double>. The
> >> setValues method is not accesible from python and the values() method
> >> returns:
> >>
> >> QVector<double> (C++ Object 0x14c479690)
> >>
> >> rather than a python list. Maybe PythonQt only supports QVector<double*>
> ?
> >>
> >> Maybe it makes more sense to return a QVariantList which maps to
> >> python tuples in PythonQt?
> >>
> >> What do you guys think?
> >>
> >> Thanks,
> >> Daniel
> >>
> >> On Wed, Aug 17, 2011 at 7:56 PM, Daniel Haehn <haehn at bwh.harvard.edu>
> >> wrote:
> >> > Great, thank you!
> >> >
> >> > Daniel
> >> >
> >> > On Wed, Aug 17, 2011 at 7:48 PM, Julien Finet <
> julien.finet at kitware.com>
> >> > wrote:
> >> >> Added in CTK:
> >> >>
> >> >>
> https://github.com/commontk/CTK/commit/5ebd60003b230f961bfb5d2c89c17f0ddcb4a151
> >> >> Added in Slicer: r17743
> >> >>
> >> >>
> http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=17743
> >> >> Thanks,
> >> >> Julien.
> >> >>
> >> >> On Wed, Aug 17, 2011 at 4:55 PM, Daniel Haehn <haehn at bwh.harvard.edu
> >
> >> >> wrote:
> >> >>>
> >> >>> Thanks for the feedback,
> >> >>>
> >> >>> I just commited
> >> >>>
> >> >>>
> >> >>>
> https://github.com/haehn/CTK/commit/583a229fce5d99f06b78bab1c3a38cfe4255a328
> >> >>>
> >> >>> Would be great if we could encounter these changes to upstream and
> >> >>> then to Slicer CTK - the EMSegmenter needs it :)
> >> >>>
> >> >>> Thx,
> >> >>> Daniel
> >> >>>
> >> >>> On Wed, Aug 17, 2011 at 4:48 PM, Julien Finet
> >> >>> <julien.finet at kitware.com>
> >> >>> wrote:
> >> >>> > Looks good, except that when populating the values vector
> >> >>> > (ctkMatrixWidget::values()), I would call
> >> >>> > ctkMatrixWidget::value(i,j)
> >> >>> > instead of manually querying the data in order to factorize code.
> >> >>> > Thanks,
> >> >>> > Julien.
> >> >>> > p.s.I agree that setValues() would also beneficiate from some
> >> >>> > refactorization as well, but it's another story...
> >> >>> >
> >> >>> > On Wed, Aug 17, 2011 at 4:15 PM, Daniel Haehn
> >> >>> > <haehn at bwh.harvard.edu>
> >> >>> > wrote:
> >> >>> >>
> >> >>> >> Hi guys,
> >> >>> >>
> >> >>> >> could you please review
> >> >>> >>
> >> >>> >>
> >> >>> >>
> >> >>> >>
> >> >>> >>
> https://github.com/haehn/CTK/tree/add-SetValuesQPropertyMatrixWidget-feature
> >> >>> >>
> >> >>> >> Thx,
> >> >>> >> Daniel
> >> >>> >>
> >> >>> >> On Tue, Aug 16, 2011 at 3:57 PM, Jean-Christophe Fillion-Robin
> >> >>> >> <jchris.fillionr at kitware.com> wrote:
> >> >>> >> > +1 for setValues
> >> >>> >> > Jc
> >> >>> >> >
> >> >>> >> > On Tue, Aug 16, 2011 at 3:11 PM, Julien Finet
> >> >>> >> > <julien.finet at kitware.com>
> >> >>> >> > wrote:
> >> >>> >> >>
> >> >>> >> >> Because a "Q_PROPERTY adds features through the meta-object
> >> >>> >> >> system".
> >> >>> >> >> Practically in this case, it allows the property to be wrapped
> >> >>> >> >> with
> >> >>> >> >> python, and initial values can also be set directly via the
> >> >>> >> >> Designer.
> >> >>> >> >> As a more philosophical point, the matrix "values" are a
> >> >>> >> >> "property"
> >> >>> >> >> of
> >> >>> >> >> the
> >> >>> >> >> matrix.
> >> >>> >> >> Julien.
> >> >>> >> >> On Tue, Aug 16, 2011 at 3:04 PM, Daniel Haehn
> >> >>> >> >> <haehn at bwh.harvard.edu>
> >> >>> >> >> wrote:
> >> >>> >> >>>
> >> >>> >> >>> Hi Julien,
> >> >>> >> >>>
> >> >>> >> >>> why do you think a Q_PROPERTY is better? Anyway, I will give
> it
> >> >>> >> >>> a
> >> >>> >> >>> shot
> >> >>> >> >>> and let you know.
> >> >>> >> >>>
> >> >>> >> >>> Cheers,
> >> >>> >> >>> Daniel
> >> >>> >> >>>
> >> >>> >> >>> On Tue, Aug 16, 2011 at 1:48 PM, Julien Finet
> >> >>> >> >>> <julien.finet at kitware.com>
> >> >>> >> >>> wrote:
> >> >>> >> >>> > Either that or we make a Q_PROPERTY named "values":
> >> >>> >> >>> > Q_PROPERTY( QVector<double> values READ values WRITE
> >> >>> >> >>> > setValues)
> >> >>> >> >>> > There is already:
> >> >>> >> >>> > ctkMatrixWidget::setVector(QVector<double>),
> >> >>> >> >>> > maybe
> >> >>> >> >>> > it
> >> >>> >> >>> > could be renamed into setValues and values() should then
> also
> >> >>> >> >>> > be
> >> >>> >> >>> > written.
> >> >>> >> >>> > Julien.
> >> >>> >> >>> >
> >> >>> >> >>> > On Tue, Aug 16, 2011 at 12:37 PM, Daniel Haehn
> >> >>> >> >>> > <haehn at bwh.harvard.edu>
> >> >>> >> >>> > wrote:
> >> >>> >> >>> >>
> >> >>> >> >>> >> Hi guys,
> >> >>> >> >>> >>
> >> >>> >> >>> >> it seems that it is not possible to set a value of a
> >> >>> >> >>> >> ctkMatrixWidget
> >> >>> >> >>> >> from Python?
> >> >>> >> >>> >>
> >> >>> >> >>> >> Shall we just enable Q_INVOKABLE on the
> >> >>> >> >>> >> ctkMatrixWidget::setValue
> >> >>> >> >>> >> method?
> >> >>> >> >>> >>
> >> >>> >> >>> >> Thanks,
> >> >>> >> >>> >> Daniel
> >> >>> >> >>> >> _______________________________________________
> >> >>> >> >>> >> Ctk-developers mailing list
> >> >>> >> >>> >> Ctk-developers at commontk.org
> >> >>> >> >>> >>
> >> >>> >> >>> >>
> >> >>> >> >>> >>
> http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers
> >> >>> >> >>> >
> >> >>> >> >>> >
> >> >>> >> >>
> >> >>> >> >>
> >> >>> >> >> _______________________________________________
> >> >>> >> >> Ctk-developers mailing list
> >> >>> >> >> Ctk-developers at commontk.org
> >> >>> >> >>
> >> >>> >> >>
> http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers
> >> >>> >> >>
> >> >>> >> >
> >> >>> >> >
> >> >>> >> >
> >> >>> >> > --
> >> >>> >> > +1 919 869 8849
> >> >>> >> >
> >> >>> >> >
> >> >>> >
> >> >>> >
> >> >>
> >> >>
> >> >
> >
> >
>
--20cf305b0cb25b357904aade247f
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
I'm a bit reluctant to make ctkMatrixWidget's code less readable ju=
st to please scripting.<div><br></div><div>From=A0<a href=3D"http://pythonq=
t.sourceforge.net/Features.html">http://pythonqt.sourceforge.net/Features.h=
tml</a>: it seems that we need to add our own "handlers" (whateve=
r it means).<div>
I'm not sure what it implies though.</div><div>Julien.<br><br><div clas=
s=3D"gmail_quote">On Fri, Aug 19, 2011 at 12:14 PM, Daniel Haehn <span dir=
=3D"ltr"><<a href=3D"mailto:haehn at bwh.harvard.edu">haehn at bwh.harvard.edu=
</a>></span> wrote:<br>
<blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1p=
x #ccc solid;padding-left:1ex;">Hi Julien,<br>
<br>
I tried it but it did not change anything.. :/<br>
<font color=3D"#888888"><br>
Daniel<br>
</font><div><div></div><div class=3D"h5"><br>
On Fri, Aug 19, 2011 at 12:03 PM, Julien Finet <<a href=3D"mailto:julien=
.finet at kitware.com">julien.finet at kitware.com</a>> wrote:<br>
> Maybe=A0QVector<double> needs to be entered in the meta object s=
ystem.<br>
> Can you try adding<br>
><br>
> qRegisterMetaType<QVector<double> > ("QVector<doub=
le>");<br>
><br>
> in the constructor function of ctkMatrixWidget (better: init() of<br>
> ctkMatrixWidgetPrivate if it exists)<br>
> j.<br>
> On Fri, Aug 19, 2011 at 11:55 AM, Daniel Haehn <<a href=3D"mailto:h=
aehn at bwh.harvard.edu">haehn at bwh.harvard.edu</a>><br>
> wrote:<br>
>><br>
>> Hi again,<br>
>><br>
>> so it seems that PythonQt has problems to wrap a QVector<double=
>. The<br>
>> setValues method is not accesible from python and the values() met=
hod<br>
>> returns:<br>
>><br>
>> =A0QVector<double> (C++ Object 0x14c479690)<br>
>><br>
>> rather than a python list. Maybe PythonQt only supports QVector<=
;double*> ?<br>
>><br>
>> Maybe it makes more sense to return a QVariantList which maps to<b=
r>
>> python tuples in PythonQt?<br>
>><br>
>> What do you guys think?<br>
>><br>
>> Thanks,<br>
>> Daniel<br>
>><br>
>> On Wed, Aug 17, 2011 at 7:56 PM, Daniel Haehn <<a href=3D"mailt=
o:haehn at bwh.harvard.edu">haehn at bwh.harvard.edu</a>><br>
>> wrote:<br>
>> > Great, thank you!<br>
>> ><br>
>> > Daniel<br>
>> ><br>
>> > On Wed, Aug 17, 2011 at 7:48 PM, Julien Finet <<a href=3D"=
mailto:julien.finet at kitware.com">julien.finet at kitware.com</a>><br>
>> > wrote:<br>
>> >> Added in CTK:<br>
>> >><br>
>> >> <a href=3D"https://github.com/commontk/CTK/commit/5ebd600=
03b230f961bfb5d2c89c17f0ddcb4a151" target=3D"_blank">https://github.com/com=
montk/CTK/commit/5ebd60003b230f961bfb5d2c89c17f0ddcb4a151</a><br>
>> >> Added in Slicer: r17743<br>
>> >><br>
>> >> <a href=3D"http://viewvc.slicer.org/viewvc.cgi/Slicer4?vi=
ew=3Drevision&revision=3D17743" target=3D"_blank">http://viewvc.slicer.=
org/viewvc.cgi/Slicer4?view=3Drevision&revision=3D17743</a><br>
>> >> Thanks,<br>
>> >> Julien.<br>
>> >><br>
>> >> On Wed, Aug 17, 2011 at 4:55 PM, Daniel Haehn <<a href=
=3D"mailto:haehn at bwh.harvard.edu">haehn at bwh.harvard.edu</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> Thanks for the feedback,<br>
>> >>><br>
>> >>> I just commited<br>
>> >>><br>
>> >>><br>
>> >>> <a href=3D"https://github.com/haehn/CTK/commit/583a22=
9fce5d99f06b78bab1c3a38cfe4255a328" target=3D"_blank">https://github.com/ha=
ehn/CTK/commit/583a229fce5d99f06b78bab1c3a38cfe4255a328</a><br>
>> >>><br>
>> >>> Would be great if we could encounter these changes to=
upstream and<br>
>> >>> then to Slicer CTK - the EMSegmenter needs it :)<br>
>> >>><br>
>> >>> Thx,<br>
>> >>> Daniel<br>
>> >>><br>
>> >>> On Wed, Aug 17, 2011 at 4:48 PM, Julien Finet<br>
>> >>> <<a href=3D"mailto:julien.finet at kitware.com">julie=
n.finet at kitware.com</a>><br>
>> >>> wrote:<br>
>> >>> > Looks good, except that when populating the valu=
es vector<br>
>> >>> > (ctkMatrixWidget::values()), I would call<br>
>> >>> > ctkMatrixWidget::value(i,j)<br>
>> >>> > instead of manually querying the data in order t=
o factorize code.<br>
>> >>> > Thanks,<br>
>> >>> > Julien.<br>
>> >>> > p.s.I agree that setValues() would also benefici=
ate from some<br>
>> >>> > refactorization as well, but it's another st=
ory...<br>
>> >>> ><br>
>> >>> > On Wed, Aug 17, 2011 at 4:15 PM, Daniel Haehn<br=
>
>> >>> > <<a href=3D"mailto:haehn at bwh.harvard.edu">hae=
hn at bwh.harvard.edu</a>><br>
>> >>> > wrote:<br>
>> >>> >><br>
>> >>> >> Hi guys,<br>
>> >>> >><br>
>> >>> >> could you please review<br>
>> >>> >><br>
>> >>> >><br>
>> >>> >><br>
>> >>> >><br>
>> >>> >> <a href=3D"https://github.com/haehn/CTK/tree=
/add-SetValuesQPropertyMatrixWidget-feature" target=3D"_blank">https://gith=
ub.com/haehn/CTK/tree/add-SetValuesQPropertyMatrixWidget-feature</a><br>
>> >>> >><br>
>> >>> >> Thx,<br>
>> >>> >> Daniel<br>
>> >>> >><br>
>> >>> >> On Tue, Aug 16, 2011 at 3:57 PM, Jean-Christ=
ophe Fillion-Robin<br>
>> >>> >> <<a href=3D"mailto:jchris.fillionr at kitwar=
e.com">jchris.fillionr at kitware.com</a>> wrote:<br>
>> >>> >> > +1 for setValues<br>
>> >>> >> > Jc<br>
>> >>> >> ><br>
>> >>> >> > On Tue, Aug 16, 2011 at 3:11 PM, Julien=
Finet<br>
>> >>> >> > <<a href=3D"mailto:julien.finet at kitw=
are.com">julien.finet at kitware.com</a>><br>
>> >>> >> > wrote:<br>
>> >>> >> >><br>
>> >>> >> >> Because a "Q_PROPERTY adds fea=
tures through the meta-object<br>
>> >>> >> >> system".<br>
>> >>> >> >> Practically in this case, it allows=
the property to be wrapped<br>
>> >>> >> >> with<br>
>> >>> >> >> python, and initial values can also=
be set directly via the<br>
>> >>> >> >> Designer.<br>
>> >>> >> >> As a more=A0philosophical=A0point, =
the=A0matrix=A0"values" are a<br>
>> >>> >> >> "property"<br>
>> >>> >> >> of<br>
>> >>> >> >> the<br>
>> >>> >> >> matrix.<br>
>> >>> >> >> Julien.<br>
>> >>> >> >> On Tue, Aug 16, 2011 at 3:04 PM, Da=
niel Haehn<br>
>> >>> >> >> <<a href=3D"mailto:haehn at bwh.har=
vard.edu">haehn at bwh.harvard.edu</a>><br>
>> >>> >> >> wrote:<br>
>> >>> >> >>><br>
>> >>> >> >>> Hi Julien,<br>
>> >>> >> >>><br>
>> >>> >> >>> why do you think a Q_PROPERTY i=
s better? Anyway, I will give it<br>
>> >>> >> >>> a<br>
>> >>> >> >>> shot<br>
>> >>> >> >>> and let you know.<br>
>> >>> >> >>><br>
>> >>> >> >>> Cheers,<br>
>> >>> >> >>> Daniel<br>
>> >>> >> >>><br>
>> >>> >> >>> On Tue, Aug 16, 2011 at 1:48 PM=
, Julien Finet<br>
>> >>> >> >>> <<a href=3D"mailto:julien.fi=
net at kitware.com">julien.finet at kitware.com</a>><br>
>> >>> >> >>> wrote:<br>
>> >>> >> >>> > Either that or we make a=
=A0Q_PROPERTY named=A0"values":<br>
>> >>> >> >>> > Q_PROPERTY( QVector<dou=
ble> values READ values WRITE<br>
>> >>> >> >>> > setValues)<br>
>> >>> >> >>> > There is already:<br>
>> >>> >> >>> > ctkMatrixWidget::setVector=
(QVector<double>),<br>
>> >>> >> >>> > maybe<br>
>> >>> >> >>> > it<br>
>> >>> >> >>> > could be renamed into setV=
alues and values() should then also<br>
>> >>> >> >>> > be<br>
>> >>> >> >>> > written.<br>
>> >>> >> >>> > Julien.<br>
>> >>> >> >>> ><br>
>> >>> >> >>> > On Tue, Aug 16, 2011 at 12=
:37 PM, Daniel Haehn<br>
>> >>> >> >>> > <<a href=3D"mailto:haeh=
n at bwh.harvard.edu">haehn at bwh.harvard.edu</a>><br>
>> >>> >> >>> > wrote:<br>
>> >>> >> >>> >><br>
>> >>> >> >>> >> Hi guys,<br>
>> >>> >> >>> >><br>
>> >>> >> >>> >> it seems that it is no=
t possible to set a value of a<br>
>> >>> >> >>> >> ctkMatrixWidget<br>
>> >>> >> >>> >> from Python?<br>
>> >>> >> >>> >><br>
>> >>> >> >>> >> Shall we just enable Q=
_INVOKABLE on the<br>
>> >>> >> >>> >> ctkMatrixWidget::setVa=
lue<br>
>> >>> >> >>> >> method?<br>
>> >>> >> >>> >><br>
>> >>> >> >>> >> Thanks,<br>
>> >>> >> >>> >> Daniel<br>
>> >>> >> >>> >> ______________________=
_________________________<br>
>> >>> >> >>> >> Ctk-developers mailing=
list<br>
>> >>> >> >>> >> <a href=3D"mailto:Ctk-=
developers at commontk.org">Ctk-developers at commontk.org</a><br>
>> >>> >> >>> >><br>
>> >>> >> >>> >><br>
>> >>> >> >>> >> <a href=3D"http://publ=
ic.kitware.com/cgi-bin/mailman/listinfo/ctk-developers" target=3D"_blank">h=
ttp://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers</a><br>
>> >>> >> >>> ><br>
>> >>> >> >>> ><br>
>> >>> >> >><br>
>> >>> >> >><br>
>> >>> >> >> ___________________________________=
____________<br>
>> >>> >> >> Ctk-developers mailing list<br>
>> >>> >> >> <a href=3D"mailto:Ctk-developers at co=
mmontk.org">Ctk-developers at commontk.org</a><br>
>> >>> >> >><br>
>> >>> >> >> <a href=3D"http://public.kitware.co=
m/cgi-bin/mailman/listinfo/ctk-developers" target=3D"_blank">http://public.=
kitware.com/cgi-bin/mailman/listinfo/ctk-developers</a><br>
>> >>> >> >><br>
>> >>> >> ><br>
>> >>> >> ><br>
>> >>> >> ><br>
>> >>> >> > --<br>
>> >>> >> > <a href=3D"tel:%2B1%20919%20869%208849"=
value=3D"+19198698849">+1 919 869 8849</a><br>
>> >>> >> ><br>
>> >>> >> ><br>
>> >>> ><br>
>> >>> ><br>
>> >><br>
>> >><br>
>> ><br>
><br>
><br>
</div></div></blockquote></div><br></div></div>
--20cf305b0cb25b357904aade247f--
More information about the Ctk-developers
mailing list