Skip to content

Comments

Validation Metrics & UI Updates#547

Merged
s314cy merged 6 commits intodevelopfrom
367-validation-metrics-s314cy
Mar 20, 2023
Merged

Validation Metrics & UI Updates#547
s314cy merged 6 commits intodevelopfrom
367-validation-metrics-s314cy

Conversation

@s314cy
Copy link
Contributor

@s314cy s314cy commented Nov 18, 2022

List of changes

  • list of UI updates
    • changed a few core component styles (e.g. IconCard, CustomButton, ButtonCard)
    • regrouped buttons that are related, with a describing title
    • aligned several remaining style mismatches
    • moved the navigation button components to their corresponding navigation bar component for more intuitive style and logic control
    • fixed the validation accuracy graph updates
  • the validation panel
    • provides the full confusion matrix (for tasks with less than 8 classes)
    • provides sensitivity and f1-score for each label of binary classification tasks
    • provides the macro average f1-score for multiclass classification tasks

Related issues

  • contributes to Validation metrics #367
  • fixes label prediction for multi-class classification (replaced rounding by argmax)
  • fixes the validation graph not being updated

@s314cy s314cy added feature New feature or request web client Related to the browser environment discojs Related to Disco.js labels Nov 18, 2022
@s314cy s314cy self-assigned this Nov 18, 2022
@s314cy s314cy force-pushed the 367-validation-metrics-s314cy branch 2 times, most recently from 2280e8b to 8e3c573 Compare November 21, 2022 12:39
@s314cy s314cy changed the title Provide Validation Metrics Provide Validation Metrics & UI Updates Nov 22, 2022
@s314cy s314cy force-pushed the 367-validation-metrics-s314cy branch 4 times, most recently from af7bf0d to 4fdb2d8 Compare November 23, 2022 13:38
@s314cy s314cy changed the title Provide Validation Metrics & UI Updates Validation Metrics & UI Updates Nov 23, 2022
@s314cy s314cy marked this pull request as ready for review November 23, 2022 15:26
@s314cy s314cy force-pushed the 367-validation-metrics-s314cy branch from 90fb187 to 792f277 Compare March 20, 2023 09:29
@s314cy
Copy link
Contributor Author

s314cy commented Mar 20, 2023

I'm limiting the scope of this PR to the UI changes forementioned, thus keeping the validation matrix for later (most of the code has been made "inactive" but kept) and merging this PR now

this is because:

@s314cy s314cy merged commit 5049897 into develop Mar 20, 2023
@s314cy s314cy deleted the 367-validation-metrics-s314cy branch March 20, 2023 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

discojs Related to Disco.js feature New feature or request web client Related to the browser environment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant