Skip to content

[ENH] Predictions: Output errors#6577

Merged
janezd merged 1 commit intobiolab:masterfrom
VesnaT:pred_output_err
Sep 22, 2023
Merged

[ENH] Predictions: Output errors#6577
janezd merged 1 commit intobiolab:masterfrom
VesnaT:pred_output_err

Conversation

@VesnaT
Copy link
Contributor

@VesnaT VesnaT commented Sep 15, 2023

Issue

Fixes #6572

Description of changes

Attach prediction errors to output when the errors are shown in the table.

Includes
  • Code changes
  • Tests
  • Documentation

@markotoplak
Copy link
Member

/rebase

@markotoplak markotoplak changed the title Prediction: Output error [ENH] Predictions: Output errors Sep 18, 2023
@markotoplak
Copy link
Member

@janezd, this PR seem like a cool addition. Initially I was against (should we really output things that people can easily compute in Formula?), but I changed my mind in discussion with Vesna. The single error column per predictor seems like a sensible idea (and users can also select error types in the widget, which was the thing that convinced me).

Maybe this could also go into 3.36.1, if you perhaps have to review it?

@janezd janezd self-assigned this Sep 22, 2023
@janezd janezd merged commit 88baad6 into biolab:master Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Predictions: include error in 'Predictions' output

3 participants