Request features from Review section with one call #233
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
required by #198 and https://github.com/src-d/backlog/issues/1202
fixes #229
As defined by #198 (comment), the ML Feature Extractor API will take a couple of UAST, and it will return a bunch of features per each UAST, and a
score
as a pair-featureThis PR prepares the frontend to do one call per FilePair being reviewed to obtain the features of each file, and the score itself.
This PR prepares the backend to answer following the expected schema.
No UI changes were done.
It was also added a
3445eb0 commit to fix #229 because otherwise the FilePairs beyond the first one cannot be reviewed.
Pending work
To be done once the ML Feature Extractor API is ready, but that does not block this PR:
model.Features
must be removed,features
db table must be removed → runningfeatures-drop-table
migration from DB Migrations #221