From: Nick H. <nic...@ho...> - 2012-09-19 13:25:23
|
I am getting a runtime warning when using list views in trunk. The warning refers to the following bug: https://bugzilla.gnome.org/show_bug.cgi?id=683599 Apparently assigning values of any type other than an integer to a user_data field of a TreeIter is deprecated. We assign a handle (string) to this field. Nick. |
From: Benny M. <ben...@gm...> - 2012-09-19 13:36:20
|
I replied blindly to the bug ticket in light of what you mention. You think we should give a more technical reasoning for the need? If so, do comment. the more users of pygobject that react, the better. I can't think about an easy workaround? An int 2 key dict seems big overhead. Benny 2012/9/19 Nick Hall <nic...@ho...> > I am getting a runtime warning when using list views in trunk. The > warning refers to the following bug: > > https://bugzilla.gnome.org/show_bug.cgi?id=683599 > > Apparently assigning values of any type other than an integer to a > user_data field of a TreeIter is deprecated. > > We assign a handle (string) to this field. > > Nick. > > > > ------------------------------------------------------------------------------ > Live Security Virtual Conference > Exclusive live event will cover all the ways today's security and > threat landscape has changed and how IT managers can respond. Discussions > will include endpoint security, mobile security and the latest in malware > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ > _______________________________________________ > Gramps-devel mailing list > Gra...@li... > https://lists.sourceforge.net/lists/listinfo/gramps-devel > |
From: Nick H. <nic...@ho...> - 2012-09-19 21:55:23
|
I see that they have responded quickly to your post. It is interesting that they have written an implementation of GenericTreeModel. Perhaps there is now a way to re-implement methods in the TreeModel interface. I tried, as an experiment, to sub-class ListStore, but it didn't work. Nick. On 19/09/12 14:36, Benny Malengier wrote: > I replied blindly to the bug ticket in light of what you mention. You > think we should give a more technical reasoning for the need? If so, > do comment. the more users of pygobject that react, the better. I > can't think about an easy workaround? An int 2 key dict seems big > overhead. > > Benny > > 2012/9/19 Nick Hall <nic...@ho... > <mailto:nic...@ho...>> > > I am getting a runtime warning when using list views in trunk. The > warning refers to the following bug: > > https://bugzilla.gnome.org/show_bug.cgi?id=683599 > > Apparently assigning values of any type other than an integer to a > user_data field of a TreeIter is deprecated. > > We assign a handle (string) to this field. > > Nick. > > > ------------------------------------------------------------------------------ > Live Security Virtual Conference > Exclusive live event will cover all the ways today's security and > threat landscape has changed and how IT managers can respond. > Discussions > will include endpoint security, mobile security and the latest in > malware > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ > _______________________________________________ > Gramps-devel mailing list > Gra...@li... > <mailto:Gra...@li...> > https://lists.sourceforge.net/lists/listinfo/gramps-devel > > |
From: Benny M. <ben...@gm...> - 2012-09-19 22:15:32
|
2012/9/19 Nick Hall <nic...@ho...> > I see that they have responded quickly to your post. > > It is interesting that they have written an implementation of > GenericTreeModel. Perhaps there is now a way to re-implement methods in > the TreeModel interface. I tried, as an experiment, to sub-class > ListStore, but it didn't work. > We have a handle to path map, so we can use that to store integers and pass that instead of the handles. Annoying. The positive thing is I found another error in the port of treemodel while testing :-) Benny > > Nick. > > > > On 19/09/12 14:36, Benny Malengier wrote: > > I replied blindly to the bug ticket in light of what you mention. You > think we should give a more technical reasoning for the need? If so, do > comment. the more users of pygobject that react, the better. I can't think > about an easy workaround? An int 2 key dict seems big overhead. > > Benny > > 2012/9/19 Nick Hall <nic...@ho...> > >> I am getting a runtime warning when using list views in trunk. The >> warning refers to the following bug: >> >> https://bugzilla.gnome.org/show_bug.cgi?id=683599 >> >> Apparently assigning values of any type other than an integer to a >> user_data field of a TreeIter is deprecated. >> >> We assign a handle (string) to this field. >> >> Nick. >> >> >> >> ------------------------------------------------------------------------------ >> Live Security Virtual Conference >> Exclusive live event will cover all the ways today's security and >> threat landscape has changed and how IT managers can respond. Discussions >> will include endpoint security, mobile security and the latest in malware >> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ >> _______________________________________________ >> Gramps-devel mailing list >> Gra...@li... >> https://lists.sourceforge.net/lists/listinfo/gramps-devel >> > > > |
From: Benny M. <ben...@gm...> - 2012-09-21 19:25:04
|
2012/9/19 Nick Hall <nic...@ho...> > I am getting a runtime warning when using list views in trunk. The > warning refers to the following bug: > > https://bugzilla.gnome.org/show_bug.cgi?id=683599 > > Apparently assigning values of any type other than an integer to a > user_data field of a TreeIter is deprecated. > > Ok, I converted the flatbasemodel to integer for user_data. I'll investigate the treemodel next. Benny > We assign a handle (string) to this field. > > Nick. > > > > ------------------------------------------------------------------------------ > Live Security Virtual Conference > Exclusive live event will cover all the ways today's security and > threat landscape has changed and how IT managers can respond. Discussions > will include endpoint security, mobile security and the latest in malware > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ > _______________________________________________ > Gramps-devel mailing list > Gra...@li... > https://lists.sourceforge.net/lists/listinfo/gramps-devel > |
From: Benny M. <ben...@gm...> - 2012-09-21 19:41:30
|
2012/9/21 Benny Malengier <ben...@gm...> > > > 2012/9/19 Nick Hall <nic...@ho...> > >> I am getting a runtime warning when using list views in trunk. The >> warning refers to the following bug: >> >> https://bugzilla.gnome.org/show_bug.cgi?id=683599 >> >> Apparently assigning values of any type other than an integer to a >> user_data field of a TreeIter is deprecated. >> >> > Ok, I converted the flatbasemodel to integer for user_data. > I'll investigate the treemodel next. > Rev 20430, also treebasemodel, as we already converted to str a long anyway this was easy. So if all is ok, this warning for you should now be gone (I'm on older pygobject, so don't see this). I do obtain a seg fault when I do an expand all nodes in a treeview. Do you also have that with a newer pygobject? Benny > > Benny > > >> We assign a handle (string) to this field. >> >> Nick. >> >> >> >> ------------------------------------------------------------------------------ >> Live Security Virtual Conference >> Exclusive live event will cover all the ways today's security and >> threat landscape has changed and how IT managers can respond. Discussions >> will include endpoint security, mobile security and the latest in malware >> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ >> _______________________________________________ >> Gramps-devel mailing list >> Gra...@li... >> https://lists.sourceforge.net/lists/listinfo/gramps-devel >> > > |
From: Nick H. <nic...@ho...> - 2012-09-21 22:27:35
|
On 21/09/12 20:41, Benny Malengier wrote: > Rev 20430, also treebasemodel, as we already converted to str a long > anyway this was easy. > So if all is ok, this warning for you should now be gone (I'm on older > pygobject, so don't see this). > The warnings have now gone. > I do obtain a seg fault when I do an expand all nodes in a treeview. > Do you also have that with a newer pygobject? I don't get a seg fault on expand or collapse, but I do get seg faults when changing views. I get the following warning: src/gui/widgets/styledtexteditor.py:208: Warning: invalid cast from `GdkWindowImplX11' to `DbusmenuMenuitem' settings.set_property('gtk-show-unicode-menu', False) After the seg faults the backtrace contains a libdbusmenu entry and a line for the invalid cast. I also get the following warning in the place tree view: /usr/lib/python2.7/dist-packages/gi/types.py:47: Warning: unable to set property `markup' of type `gchararray' from value of type `PyObject' return info.invoke(*args, **kwargs) This doesn't cause a seg fault though. Nick. |
From: Benny M. <ben...@gm...> - 2012-09-22 08:39:49
|
2012/9/22 Nick Hall <nic...@ho...> > On 21/09/12 20:41, Benny Malengier wrote: > >> Rev 20430, also treebasemodel, as we already converted to str a long >> anyway this was easy. >> So if all is ok, this warning for you should now be gone (I'm on older >> pygobject, so don't see this). >> >> > The warnings have now gone. > > > > I do obtain a seg fault when I do an expand all nodes in a treeview. Do >> you also have that with a newer pygobject? >> > > > I don't get a seg fault on expand or collapse, but I do get seg faults > when changing views. > Ok, that is strange, I don't have problems with that. Python coding sure became more difficult with these bindings. Could be > > I get the following warning: > > src/gui/widgets/**styledtexteditor.py:208: Warning: invalid cast from > `GdkWindowImplX11' to `DbusmenuMenuitem' > settings.set_property('gtk-**show-unicode-menu', False) > > After the seg faults the backtrace contains a libdbusmenu entry and a line > for the invalid cast. > But does the note editor work, or does it fail? I think Unity of Ubuntu uses dbus for their global menu. Then it would be an Ubuntu hack issue, it would not be the first time Ubuntu global menu fails for gramps. If so, that is something they should fix or someone who wants to support Unity. > I also get the following warning in the place tree view: > > /usr/lib/python2.7/dist-**packages/gi/types.py:47: Warning: unable to set > property `markup' of type `gchararray' from value of type `PyObject' > return info.invoke(*args, **kwargs) > Warnings are annoying, but if we don't see problems in the GUI, not of immediate concern. Markup should be things like column with bold or italic text or color text that no longer shows. Don't know in place treeview if that is used Benny > > This doesn't cause a seg fault though. > > Nick. > > |
From: Nick H. <nic...@ho...> - 2012-09-22 17:42:28
|
On 22/09/12 09:39, Benny Malengier wrote: > But does the note editor work, or does it fail? > I think Unity of Ubuntu uses dbus for their global menu. Then it would > be an Ubuntu hack issue, it would not be the first time Ubuntu global > menu fails for gramps. If so, that is something they should fix or > someone who wants to support Unity. The note editor still works. I have now tried a few desktops. I get the same problem with Unity, Classic Gnome and KDE. Nick. |
From: Benny M. <ben...@gm...> - 2012-09-22 21:02:26
|
2012/9/22 Nick Hall <nic...@ho...> > On 22/09/12 09:39, Benny Malengier wrote: > >> But does the note editor work, or does it fail? >> I think Unity of Ubuntu uses dbus for their global menu. Then it would be >> an Ubuntu hack issue, it would not be the first time Ubuntu global menu >> fails for gramps. If so, that is something they should fix or someone who >> wants to support Unity. >> > > The note editor still works. > > I have now tried a few desktops. I get the same problem with Unity, > Classic Gnome and KDE. > Ok, but did you try all on ubuntu 12.04? Not that it must be an ubuntu problem for me :-) Benny > > Nick. > > |
From: Nick H. <nic...@ho...> - 2012-09-22 21:07:18
|
On 22/09/12 22:02, Benny Malengier wrote: > > > 2012/9/22 Nick Hall <nic...@ho... > <mailto:nic...@ho...>> > > On 22/09/12 09:39, Benny Malengier wrote: > > But does the note editor work, or does it fail? > I think Unity of Ubuntu uses dbus for their global menu. Then > it would be an Ubuntu hack issue, it would not be the first > time Ubuntu global menu fails for gramps. If so, that is > something they should fix or someone who wants to support Unity. > > > The note editor still works. > > I have now tried a few desktops. I get the same problem with > Unity, Classic Gnome and KDE. > > > Ok, but did you try all on ubuntu 12.04? Not that it must be an ubuntu > problem for me :-) No. All my Ubuntu testing is on 12.10 beta. I'm not too concerned with a beta version though, it may well be an Ubuntu issue. If no-one else is having problems, then it most likely is not an issue. The seg fault with pyexiv2 was just an Ubuntu issue that has now been fixed. In future I'll wait to see if anyone running a production OS can reproduce the problem. Nick. > > Benny > > > Nick. > > |
From: Benny M. <ben...@gm...> - 2012-09-23 08:00:44
|
2012/9/22 Nick Hall <nic...@ho...> > On 22/09/12 22:02, Benny Malengier wrote: > > > > 2012/9/22 Nick Hall <nic...@ho...> > >> On 22/09/12 09:39, Benny Malengier wrote: >> >>> But does the note editor work, or does it fail? >>> I think Unity of Ubuntu uses dbus for their global menu. Then it would >>> be an Ubuntu hack issue, it would not be the first time Ubuntu global menu >>> fails for gramps. If so, that is something they should fix or someone who >>> wants to support Unity. >>> >> >> The note editor still works. >> >> I have now tried a few desktops. I get the same problem with Unity, >> Classic Gnome and KDE. >> > > Ok, but did you try all on ubuntu 12.04? Not that it must be an ubuntu > problem for me :-) > > > No. All my Ubuntu testing is on 12.10 beta. I'm not too concerned with a > beta version though, it may well be an Ubuntu issue. If no-one else is > having problems, then it most likely is not an issue. > > The seg fault with pyexiv2 was just an Ubuntu issue that has now been > fixed. > > In future I'll wait to see if anyone running a production OS can reproduce > the problem. > Better if things are fixed before a release if we can make that happen. Doing a google, I only find that class in ubuntu due to unity: http://developer.ubuntu.com/api/ubuntu-12.04/python/Dbusmenu-0.4.html#Dbusmenu.Menuitem https://code.launchpad.net/~dbusmenu-team/dbusmenu/trunk.12.10 So if you post a bug ticket for this in launchpad, they can look at it. Benny > > Nick. > > > > Benny > >> >> Nick. >> >> > > |
From: Nick H. <nic...@ho...> - 2012-09-23 14:25:06
|
On 23/09/12 09:00, Benny Malengier wrote: > Doing a google, I only find that class in ubuntu due to unity: > > http://developer.ubuntu.com/api/ubuntu-12.04/python/Dbusmenu-0.4.html#Dbusmenu.Menuitem > > https://code.launchpad.net/~dbusmenu-team/dbusmenu/trunk.12.10 > <https://code.launchpad.net/%7Edbusmenu-team/dbusmenu/trunk.12.10> > > So if you post a bug ticket for this in launchpad, they can look at it. This doesn't explain why it is a problem with KDE though. I'll see if I can produce a test case. If I can I'll report the bug, if not I'm happy just to comment out the code that uses Gtk.Settings, for now. Nick. |
From: Benny M. <ben...@gm...> - 2012-09-23 14:57:29
|
2012/9/23 Nick Hall <nic...@ho...> > On 23/09/12 09:00, Benny Malengier wrote: > > Doing a google, I only find that class in ubuntu due to unity: > > > http://developer.ubuntu.com/api/ubuntu-12.04/python/Dbusmenu-0.4.html#Dbusmenu.Menuitem > https://code.launchpad.net/~dbusmenu-team/dbusmenu/trunk.12.10 > > So if you post a bug ticket for this in launchpad, they can look at it. > > > This doesn't explain why it is a problem with KDE though. > If they install this dbusmenu lib, then I think it hooks into GTK, so as to overrule certain things to allow the global menu. So this would then always be called, also in KDE. The solution would be to uninstall dbusmenu, but in Ubuntu, I would think it is a GTK dependency (if my guess is correct). Benny > > I'll see if I can produce a test case. If I can I'll report the bug, if > not I'm happy just to comment out the code that uses Gtk.Settings, for now. > > Nick. > |