Skip to content
Snippets Groups Projects

Changed ROC and DET, CMC and EPC to use probabilities without percentages

All threads resolved!

Closes #68 (closed)

Edited by Manuel Günther

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
  • Amir MOHAMMADI
  • Amir MOHAMMADI
  • Amir MOHAMMADI added 1 commit

    added 1 commit

    • 25d56ae3 - removed hard coded style changes and just made the fig size small so that everything looks bigger

    Compare with previous version

  • Amir MOHAMMADI resolved all discussions

    resolved all discussions

  • @mguenther this looks great, thank you but I removed all hard coded style changes and added grid to the DET curves. Also, I made the figsize smaller which makes everything bigger! If you don't like the new style, try to make the fig really small (like it is used in publications most of the time) and you will see they look OK.

  • Amir MOHAMMADI added 1 commit

    added 1 commit

    • 72ee1de5 - change ylim on ROC and EPC plots to better show results of biometric systems…

    Compare with previous version

  • I had removed the set_tight_layout since you had added a better solution, namely bbox_inches='tight', which made the set_tight_layout superfluous. Also, I still got warning messages even with set_tight_layout, so I decided to remove it.

  • added 1 commit

    • 71472508 - Changed border space for all plots (except DET)

    Compare with previous version

  • I have also uniformized the border of the plots to contain little space (vertically at 0 and at 1), so that you can see the plots at the top and bottom a little better. Except for the DET plot, which does never reach the borders.

  • Manuel Günther resolved all discussions

    resolved all discussions

  • @amohammadi I am OK with your changes, the plots look good.

    Please merge after reviewing my changes -- given that you agree.

  • added 1 commit

    • 9338c38a - Made print-out for --criterion FAR better

    Compare with previous version

  • Amir MOHAMMADI mentioned in commit 2665bc5d

    mentioned in commit 2665bc5d

  • Please register or sign in to reply
    Loading