-
Notifications
You must be signed in to change notification settings - Fork 103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate the NDCG@10 scorer from app.quepid.com to Docker version and Dev setup #91
Comments
@janhoy we are getting close on this! |
I have seen "a few" commits lately, yes :) Thanks for improving the Docker version. |
And we merged #100 |
Will there be a new release of quepid and docker images on hub for this improvement? I just re-installed my env (with Mysql) and after an upgrade I suppose I could just do |
Yes.. In place migration will be supported (joys of a hosted platform ;-) ). My plan is to actually release 6.2 tomorrow (after the presentation Berlin Buzzwords).. It has some pretty significant database migrations, so I need time to make sure that the hosted platform gets a good backup, we release carefully, and then test thoroughly. 6.2 has the NDCG@5 as a default, and the graph now properly works with ndcg compared to today. If you want to try out the 6.2 version, I have published a prelease image: |
I tested prerelease image, and ran https://gist.github.com/5ff8003acb2b548bcacbb57409c4c74e My MySql DB version is 5.7.26 while the one in docker-compose is 5.6.37, which could perhaps be the reason? |
I nuked the db |
Interesting! I did a practice migration on the quepid-staging.herokuapp.com version, and it worked, but this is definitly something I am concerned about. |
Describe the bug
We have created a NDCG@10 scorer that is the default scorer in app.quepid.com. However, we haven't migrated that scorer to either the Docker version of Quepid OR the dev version. This means if you run Quepid in either of those modes, then you must copy the NDCG@10 scorer from app.quepid.com to your setup.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
We expect a set of different scorers to be available regardless of if you are running Quepid as a developer, or on prem via Docker image, or on app.quepid.com. These should include P@, DCG@, CG@, MRR@ to start with.
Additional context
Right now we only have the old V1 scorer available to dev and docker, and our better newer scorers only in app.quepid.com. There isn't a well defined "seed" apprach.
Maybe we should think about having our scorers that we ship in a db/scorers, and that an initializer checks and updates them in the database if the
scorer.js
is newer that what is in the database?The text was updated successfully, but these errors were encountered: