On Wed, Jul 22, 2009 at 10:24 AM, Grant Ingersoll <[email protected]>wrote:

> Forcing instantiation of these objects is a
>> problem in several places. That change I'm still contemplating since
>> it's big... but, this change will actually make it somewhat easier.
>>
>
> I've often found the User/Item thing confusing in Taste.  Sometimes you use
> Ids, sometimes Strings, sometimes the objects.  I think dealing with Ids is
> a lot simpler, but you might consider some utility tools that make it easy
> for users to lookup the real things.


Recommendation engines should generally only traffic in id's, not things.

+1


>
> Soliciting thoughts before I pull the trigger.
>>
>>
> These all sound reasonable, but rather big, so I assume you will be adding
> patches and doing the work in JIRA so that others can try out before
> committing.    Sounds like a perfect fit for a lazy consensus model of:
>  "Here's a patch that addresses this issue.  I plan on committing in X days"
> (for X > 2 or 3, usually)


+1 as well.

-- 
Ted Dunning, CTO
DeepDyve

Reply via email to