-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
List categorization questions for agribalyse #902
Comments
@raphael0202 maybe it's the wrong way of thinking about it. I see more than one way to do it:
In case 2 refresh might be slightly more difficult to handle (but it's a remove / recreate). I don't really know if there will be a big difference and if it's worth going for option "2". Or maybe someone sees a different idea ? (@raphael0202, @alexfauquette) |
A third possibility: have a hand made list of categories in a simple file and put it in a table akin to solution 2. |
Not supper familiar with robotoff code base, but would it be possible to add two fields:
that would be false by default and true it the prediciton is helping to compute the corresponding score |
During insight creation we may also set a flag in data->is_agribalyse_category, I think it would be the simplest way of doing it. |
Another very flexible way to implement this would be to add a And we would display only insights from this campaign with: This method is flexible enough for other campaigns where we want to focus on a subset of products. |
@raphael0202 later on, we could even add an API to add a list of codes to a campaign :-) and maybe integrate it with mass edit or something like that. So that any advanced user could create a campaign. |
Implemented, closing the issue. |
Since:
we now have an API to list categorizations opportunities by type of questions.
But our goal is
To do that, we would like to have the same request but only for agribalyse categories
An flexible way to do that is to add a parameter with_property=agribalyse_food_code
That will:
Implementation details to be discussed
The text was updated successfully, but these errors were encountered: