Recommenders: Update README

Created on 31 Jan 2019  路  7Comments  路  Source: microsoft/recommenders

We have to update the README for the new algos that are in the repo. We also have to update the benchmark results.

@gramhagen could you do VW?

I will do FastAI and NCF

The list of the new algos that we have is:

  • [x] VW
  • [x] NCF
  • [x] FastAI
  • [x] xDeepFM
  • [x] DKN
  • [x] RBM
documentation

Most helpful comment

Agreed. Maybe let's add an Algorithms section which includes a table that gives a 1 sentence description of each algorithm and which env it supports? Alternatively this info could be another column of the benchmark table.

We may also want to make the table look more qualitative instead of quantitative since we want the repo to be non-opinionated. In addition to a one-sentence description, pros and cons, use case scenarios, etc. can also be good things to have.

All 7 comments

@nikhilrj I was looking at the main readme and we don't list in the front page all the algos that we have. I think this is something that the user will like to see quickly.

What are your thoughts on this? what would be a good way to show the algos more clearly?

I already did that in a new PR #476

The 02_models/README is update with VW, are you looking for results that can be read in from the comparison notebook and put into the main /README?

@nikhilrj I was looking at the main readme and we don't list in the front page all the algos that we have. I think this is something that the user will like to see quickly.

What are your thoughts on this? what would be a good way to show the algos more clearly?

Agreed. Maybe let's add an Algorithms section which includes a table that gives a 1 sentence description of each algorithm and which env it supports? Alternatively this info could be another column of the benchmark table.

Agreed. Maybe let's add an Algorithms section which includes a table that gives a 1 sentence description of each algorithm and which env it supports? Alternatively this info could be another column of the benchmark table.

We may also want to make the table look more qualitative instead of quantitative since we want the repo to be non-opinionated. In addition to a one-sentence description, pros and cons, use case scenarios, etc. can also be good things to have.

good suggestion, I remember an ask for @gcampanella and other was to give a guidance on what recommendation algorithm to use. A recommendation about recommendation :-)

Agreed. Maybe let's add an Algorithms section which includes a table that gives a 1 sentence description of each algorithm and which env it supports? Alternatively this info could be another column of the benchmark table.

We may also want to make the table look more qualitative instead of quantitative since we want the repo to be non-opinionated. In addition to a one-sentence description, pros and cons, use case scenarios, etc. can also be good things to have.

This is a good point. When updating the Readme, I wrote something like "xDeepFM, good for CTR prediction, works on python + GPU + TF". We could structure this info a bit better e.g. by using a few columns for use case, platform etc.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

miguelgfierro picture miguelgfierro  路  4Comments

anargyri picture anargyri  路  3Comments

miguelgfierro picture miguelgfierro  路  3Comments

miguelgfierro picture miguelgfierro  路  4Comments

roalexan picture roalexan  路  6Comments