prmr / Creco

Recommendation System for Consumer Products
Apache License 2.0
6 stars 2 forks source link

Ranking Explanations #79

Closed prmr closed 10 years ago

prmr commented 10 years ago

We need to produce some sort of explanation for why a product is recommended, and display it in the product ranking.

asutcl commented 10 years ago

The checked boxes should probably stay shecked after clicking submit, to make it cleared what the ranking was based on. Or, will this be handled by a dynamic update of the sorted products?

prmr commented 10 years ago

the current idea is to get rid of the submit box altogether and refresh the ranking through ajax calls. Indeed the boxes should stay checked.

asutcl commented 10 years ago

@forgues and I thought that having the ScoredAttribute keep track of a ranking of values in it's range could be a neat feature. This way by sending the category ID, the Attribute ID and the the value of the attribute, SocredAttribute would return the ranking of that value amongst the category. This would simplify the ranking explanation. @MariamN do you think this would be helpful? I can implement this on Saturday.

MariamN commented 10 years ago

@asutcl yes, this will be a very useful feature. This way we can show the value of the attribute for each product, the rank of the attribute, and the rank of the value.

mangalagb commented 10 years ago

In the latest version of master, when the products are displayed, there is some text too. Is there where you are planning to put the ranking explanations or is for the product one liner that we discussed in class yesterday?

enewe101 commented 10 years ago

@mangalagb It's for the one-liner -- just figured I'd stake out a spot while I was moving things around. The explanation can probably go right under.

mangalagb commented 10 years ago

The one-liners for different products will end up looking very similar to each other. I don't know if its a good idea to keep them . We could just display images of these products. I assume the space on the left is for the image . In case of products that do not have images and the space remains blank, it might look a bit odd.

enewe101 commented 10 years ago

Oh -- I forgot. ok.

asutcl commented 10 years ago

Ranking explanations should be much easier now since you can get the Ranking based on the Value straight from the ScoredAttribute. Let me know if you have any questions on how to do so, it's been merged to master.

enewe101 commented 10 years ago

Could we get an update for this issue? @MariamN, are you still working on this?

MariamN commented 10 years ago

@enewe101 I'm working on it. I will push to master soon.

MariamN commented 10 years ago

@enewe101 Since there is a bug in the master branch for the product ranking. I wont be able to push to master until the bug is resolved. I created an issue for that #96, I tried to debug it but I couldn't pinpoint the exact location that is causing the problem. It might be related to the format of the ajax response, though.

nishanthtgwda commented 10 years ago

I pushed the display code of explanation to tmp-master . I am not sure if I should be pushing to master since there exists a bug. @enewe101 : please check out branch tmp-master for ranking explanations display

enewe101 commented 10 years ago

Hey, I pulled, but I'm not seeing anything changed. When I pull git gives me an error, saying that it doesn't know what to do with tmp-master. That's weird because it was working for pushing and pulling earlier. I don't have time to look into it now, so we'll have to discuss in class.

forgues commented 10 years ago

I pulled from tmp-master, but I'm not sure I understand the ranking explanations. I searched for humidifiers and sorted by output. All the top products have Output: 5.0, and then the value 19.0 in the green section on the bar. What does the value 19.0 represent? Some products have value 6.0 in orange and others have 1.0 in red.

I also tried sorting by nominal attributes (e.g. color, or antimicrobial claims), but this seems to cause the application to hang.

asutcl commented 10 years ago

A lot of improvements were done here. Improving the display can be part of Milestone 1.0. The changes are still in tmp-master and will remain until we successfully merge with #91.

The milestone will then be complete.