Re: [Zeitgeist] [Merge] lp:~manishsinha/zeitgeist/fix-691167 into lp:zeitgeist

2010-12-20 Thread Mikkel Kamstrup Erlandsen
Whether or not a chunk of data is "binary" or "text" is 100% in the eye of the beholder, not a property of how you opened the file. If you load bonary image data you can str() it without problems in Python (not before you try and print() it of course :-)). Ie. this will not error: binary = lo

[Gnome-zeitgeist] [Merge] lp:~tinchester/gnome-activity-journal/today-button-tweaks into lp:gnome-activity-journal

2010-12-20 Thread Tin Tvrtkovic
Tin Tvrtkovic has proposed merging lp:~tinchester/gnome-activity-journal/today-button-tweaks into lp:gnome-activity-journal. Requested reviews: GNOME Zeitgeist Team (gnome-zeitgeist) This branch contains the today button tweak proposed on the mailing list. -- https://code.launchpad.net/~tin

[Zeitgeist] [Bug 691167] Re: The payload is sometimes mentioned as string and sometimes as array of bytes

2010-12-20 Thread Manish Sinha
MIkkel asked this question in the merge request https://code.launchpad.net/~manishsinha/zeitgeist/fix-691167/+merge/43948/comments/97974/ and my reply is that as a array of bytes, it is easy to have binary information in it e..g tarball, image, db-dump, but I have no clue how string can be represe

[Zeitgeist] [Bug 691167] Re: The payload is sometimes mentioned as string and sometimes as array of bytes

2010-12-20 Thread Seif Lotfy
OK I just summed up the situation. A payload is an array of bytes. An Array of bytes should not be translated into a string (encoding could be an issue, especially if a binary). Thus I prefer having a list of int -- You received this bug notification because you are a member of Zeitgeist Framewo

[Zeitgeist] [Bug 691167] Re: The payload is sometimes mentioned as string and sometimes as array of bytes

2010-12-20 Thread Seif Lotfy
** Changed in: zeitgeist Importance: Undecided => High -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/691167 Title: The payload is sometimes mentioned as string and somet

[Zeitgeist] [Bug 692645] [NEW] sending payload from engine to client does not work

2010-12-20 Thread Seif Lotfy
Public bug reported: I wrote a test case in the below diff where I insert an event with a payload and try to retrieve it again. This does not work on trunk guys -.- --- === modified file 'test/remote-test.py' --- test/remote-test.py 2010-09-22 18:44:16 + +++ test/remote-test.py 2010-12-20 15

[Zeitgeist] [Merge] lp:~seif/zeitgeist/fixing-payloads into lp:zeitgeist

2010-12-20 Thread Seif Lotfy
Seif Lotfy has proposed merging lp:~seif/zeitgeist/fixing-payloads into lp:zeitgeist. Requested reviews: Zeitgeist Framework Team (zeitgeist) Related bugs: #692645 sending payload from engine to client does not work https://bugs.launchpad.net/bugs/692645 This branch fixes https://bugs.lau

[Zeitgeist] [Bug 691167] Re: The payload is sometimes mentioned as string and sometimes as array of bytes

2010-12-20 Thread Seif Lotfy
I beg to differ. But seems to me that they are not working at all. I wrote a test case in the below diff where I insert an event with a payload and try to retrieve it again. This does not work on trunk guys -.- --- === modified file 'test/remote-test.py' --- test/remote-test.py 2010-09-22 18:44

[Zeitgeist] [Bug 691660] Re: Allow Extensions without PUBLIC_METHODS

2010-12-20 Thread Seif Lotfy
** Changed in: zeitgeist Status: In Progress => Fix Committed -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/691660 Title: Allow Extensions without PUBLIC_METHODS St

[Zeitgeist] [Bug 691660] Re: Allow Extensions without PUBLIC_METHODS

2010-12-20 Thread Launchpad Bug Tracker
** Branch linked: lp:zeitgeist -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/691660 Title: Allow Extensions without PUBLIC_METHODS Status in Zeitgeist Framework: Fix Com

Re: [Zeitgeist] [Merge] lp:~manishsinha/zeitgeist/fix-691167 into lp:zeitgeist

2010-12-20 Thread Manish Sinha
Till date I have always thought that payload was binary information. I have always thought that the third case would not be a problem as image is binary data. It is as equal as opening an image file in binary mode, reading it till end and putting it in the payload of the event. I am still not v

Re: [Zeitgeist] [Merge] lp:~manishsinha/zeitgeist/fix-691167 into lp:zeitgeist

2010-12-20 Thread Mikkel Kamstrup Erlandsen
So how do I add a string payload with this api? I can see 3 cases that will be the most likely payloads: * xml * json * raw binary image data (fx png) So the two cases we need to have a clear line of support for is strings and binary data. Both of these will work with event.payload = str(dat