[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2011-07-15 Thread Seif Lotfy
OK I am convinced we can do this in another way. Extension makes sense for me and thus I think as an extension it has nothing to do here. Thus I will make it as invalid ** Changed in: zeitgeist Status: Confirmed = Invalid -- You received this bug notification because you are a member of

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2011-02-10 Thread Seif Lotfy
** Changed in: zeitgeist Milestone: 0.8.0 = None -- 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/670358 Title: Add new aggregate API Status in Zeitgeist Framework:

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2011-01-19 Thread Seif Lotfy
** Changed in: zeitgeist Milestone: None = 0.8.0 -- 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/670358 Title: Add new aggregate API Status in Zeitgeist Framework:

Re: [Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-29 Thread Mikkel Kamstrup Erlandsen
Sorry to ruin the party, but I really don't like any of the proposed solutions. The use cases described in the wiki seems very academic and more intended on doing some theoretical counting exercises than solving actual user problems. Unless we have some crystal clear use cases (fx. a UI mockup

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-27 Thread Markus Korn
I updated my proposal at http://wiki.zeitgeist- project.com/index.php?title=Aggregation_API -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-27 Thread Michal Hruby
@Markus: I basically like your proposal, but since I was told that event ids are not unique, isn't there a huge flaw in that API? -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug notification because you are a member of Zeitgeist Framework Team, which is

Re: [Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-27 Thread Siegfried Gevatter
FindEventIdsStats(..., ResultType.MostRecentEvents) -- ([1, 2], [500.0, 250.0]) For `MostRecentEvents` the stats are returning the timestamps for each event. That's redundant, the events already include the last timestamp. -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-27 Thread Markus Korn
@Siegfried: yes, but it is only redundant for FindEventsStats as FindEventIdsStats only returns ids and not events. It can't be avoided in this particular case, as the stats field has to return *someting*. @Michal: as we already clearified on irc, the mapping is done based on index, so the first

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-27 Thread Michal Hruby
@Markus: I basically like your proposal, but since I was told that event ids are not unique, isn't there a huge flaw in that API? Nevermind, I misread the example... -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug notification because you are a member of

Re: [Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-27 Thread Seif Lotfy
I am very very pleased with the API proposal. GREAT GREAT WORK thekorn. You rock big time :) +1 from me On Sat, Nov 27, 2010 at 1:48 PM, Michal Hruby michal@gmail.com wrote: @Markus: I basically like your proposal, but since I was told that event ids are not unique, isn't there a huge

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-25 Thread Manish Sinha
I am really interested in this API since it would help a lot to the Music players like number of time a track is player. Music player is used by everyone, so it is a good use case which might reach out the maximum to the users. I just need some explanation def

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-25 Thread Manish Sinha
On day job I work on .NET where LINQ is a boon to all the programmers stuck up in Windows .NET programming Inspired from LINQ (which I use a lot) I would like to propose 5 basic aggregate operations as of now * Sum * Count * Max * Min * Average I know average/max/min sounds stupid at first

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-25 Thread Seif Lotfy
** Changed in: zeitgeist Milestone: 0.7.0 = None -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Zeitgeist Framework: Confirmed Bug

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-24 Thread Seif Lotfy
Again the use case is not visible to the user. It is all about sorting and ranking. I want to use it in Synapse. For that I need to have starred subjects which are all subject that have been ACCESSED or MODIFIED more than 10 times within the last 24h. These are usually things that are stamped

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-23 Thread Seif Lotfy
OK I think I need to spam this bug a bit. While I agree that use cases and user stories are the way to go. There is no such thing as a simple use case for some features. Developers want to rank their stuff so they need COUNTS. And currently we have no simple way of exposing COUNTS. The use

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-08 Thread Seif Lotfy
the problem we have with both methods is that we can only handle 1 event_template and the result types MostRecentEvents and LeastRecentEvent can not be used since it will always return a count of 1 -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-05 Thread Seif Lotfy
OK I gave it more thought and I think Markus's idea makes a lot of sense. This will allow us to even later work with more data_types, so if for example geo-location extension becomes officially supported we can shoot out the locations as a results. having a an array of dicts in data will allow

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-04 Thread Seif Lotfy
My current suggestion based on Michal Hruby's requirements would be a method called def get_events_count(timerange, event_templates): ... return dict where dict = {event_template_1: count1, event_template_2: count2, ... } --

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-04 Thread Michal Hruby
Why exactly are the event_templates duplicated to the result? -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Zeitgeist Framework: New

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-04 Thread Markus Korn
I suggest something like: def find_events_and_data(*find_event_arguments, datatype_const): ... return result result = [events, data] datatype_const: DATATYPE_COUNT - result of COUNT() from within the sql-query DATATYPE_RELATIVE_COUNT - result of COUNT() from within the

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-04 Thread Siegfried Gevatter
** Changed in: zeitgeist Assignee: (unassigned) = Siegfried Gevatter (rainct) -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in

[Zeitgeist] [Bug 670358] Re: Add new aggregate API

2010-11-04 Thread Seif Lotfy
** Changed in: zeitgeist Importance: Undecided = Wishlist ** Changed in: zeitgeist Status: New = Confirmed ** Changed in: zeitgeist Milestone: None = 0.7 -- Add new aggregate API https://bugs.launchpad.net/bugs/670358 You received this bug notification because you are a member of