[Zeitgeist] [Bug 598666] Re: Invalid cache access (was: Error when trying to fetch items)

2011-01-12 Thread Markus Korn
Seif sent me an activity.log which he managed to create by [...] randomly clicking in GAJ and synapse[...]. This log is broken and results in the above mentioned KeyError. The db has a few broken entries: http://paste.ubuntu.com/553188/ -- You received this bug notification because you are a

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

2011-01-12 Thread मनीष सिन्हा
Review: Needs Information typecasting What should be done? If cast is done to string, then we end up with [[u'', u'1292500628312', u'', u'', u'application://foo.desktop'], [[u'', u'', u'', u'', u'', u'', u'']], dbus.Array([], signature=dbus.Signature('y'))] esp when the payload should be []

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

2011-01-12 Thread Seif Lotfy
It does not transform the actual event. but create a new popo event from the dbus event. The info is not needed and is not accessible by any1 outside the zeitgeist process. Thus I think its not an issue tbh. -- https://code.launchpad.net/~manishsinha/zeitgeist/fix-691167/+merge/43948 Your team

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

2011-01-12 Thread मनीष सिन्हा
Well, this means that the payload is next to useless. It is not an issue mostly as event template comparison does not involve payloads -- https://code.launchpad.net/~manishsinha/zeitgeist/fix-691167/+merge/43948 Your team Zeitgeist Framework Team is requested to review the proposed merge of

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

2011-01-12 Thread Markus Korn
While reading the python dbus docs [0] again I found out about the byte_array=True, this will solve all our problems immediately. By using this switch in the @dbus.service.method and @dbus.service.signal decorators, ay is not exposed as dbus.Array of dbus.Bytes anymore, but as dbus.ByteArray.

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

2011-01-12 Thread Seif Lotfy
Markus: Can you apply a fix for it and see if it somehow collides with https://code.launchpad.net/~seif/zeitgeist/fix-false-payloads, then requests for merge if all unit tests work. This could take off a big chunk of code off. Good catch --

[Zeitgeist] [Bug 598666] Re: Invalid cache access (was: Error when trying to fetch items)

2011-01-12 Thread Seif Lotfy
So I just added a little temporary fix that I would like to see with the next release. Instead of zeitgeist returning nothing when requesting a time-period with a broken event, zeitgeist now can ignore the broken event and return a list with healthy events. The broken events can be seen in the

[Zeitgeist] [Bug 702213] Re: zeitgeist-0.6 not datahub

2011-01-12 Thread Seif Lotfy
You need to get the zeitgeist-datahub. You have the following options: 1) look into the ppa 2) bzr branch lp:zeitgeist-datahub ** Changed in: zeitgeist Status: New = Invalid -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed