Skip to content
Snippets Groups Projects

Benchmarking initial

Benchmarking initial
benchmarking-initial into development
All threads resolved!
Merged
Tim van der Horstrequested to merge
benchmarking-initial into development
All threads resolved!

Description

First version of the benchmarking. Does SIG style benchmarking, and is able to generate a ranking using two tables: one which assigns metric scores into risk categories, and another which assigns relative code weight in each risk category to a total system ranking. Closes #163 (closed)

Changes

None

Additions

  • Added a new endpoint /api/v1/assingment/aid/submission/sid/benchmark
  • Added benchmarking classes

Test and Review

To be filled in by the reviewers

  • All of the methods are commented to expectation

  • The methods are tested to satisfaction

  • There are no unnecessary files present in the MR

  • The continuous integration has no problems with the MR

  • The MR is filled in as requested (including labels, milestones, and reviewers)

  • The documentation is up-to-date

  • All nullable parameters are marked as such

Edited by Luc Everse

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Erik Oudsen
  • Erik Oudsen
  • Erik Oudsen
  • Erik Oudsen
  • Erik Oudsen
    Erik Oudsen @eoudsen
    Contributor

    Check @Nullable fields and parameters.

  • Tim van der Horst added 1 commit

    added 1 commit

    • 5b837d40 - Add the tests for the controller and update api proposal document

    Compare with previous version

  • Tim van der Horst added 1 commit

    added 1 commit

    • d32adb3b - Remove redundant else-blocks and add nullable annotation for project

    Compare with previous version

  • Erik Oudsen marked the checklist item All of the methods are commented to expectation as completed

    marked the checklist item All of the methods are commented to expectation as completed

  • Erik Oudsen marked the checklist item The methods are tested to satisfaction as completed

    marked the checklist item The methods are tested to satisfaction as completed

  • Erik Oudsen marked the checklist item There are no unnecessary files present in the MR as completed

    marked the checklist item There are no unnecessary files present in the MR as completed

  • Erik Oudsen marked the checklist item The continuous integration has no problems with the MR as completed

    marked the checklist item The continuous integration has no problems with the MR as completed

  • Erik Oudsen marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

    marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

  • Erik Oudsen marked the checklist item All nullable parameters are marked as such as completed

    marked the checklist item All nullable parameters are marked as such as completed

  • Erik Oudsen approved this merge request

    approved this merge request

  • Tim van der Horst resolved all discussions

    resolved all discussions

  • Luc Everse
  • Luc Everse
  • Luc Everse
  • Tim van der Horst added 1 commit

    added 1 commit

    Compare with previous version

  • Tim van der Horst resolved all discussions

    resolved all discussions

  • Luc Everse approved this merge request

    approved this merge request

  • Luc Everse marked the checklist item The documentation is up-to-date as completed

    marked the checklist item The documentation is up-to-date as completed

  • Erik Oudsen approved this merge request

    approved this merge request

  • Erik Oudsen mentioned in commit 4cee6c80

    mentioned in commit 4cee6c80

  • Erik Oudsen merged

    merged

    • Please register or sign in to reply