Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

minor: when using --csv option, chart-review should still print summary info #55

Open
comorbidity opened this issue Sep 19, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@comorbidity
Copy link
Contributor

For example, when not using the --csv flag we see the additional metadata

Comparing 50 charts (3529–3564, 3713–3715, 3717–3722, 3724–3728)
Truth: rena
Annotator: andy

F1     Sens   Spec   PPV    NPV    Kappa  TP  FN  TN  FP  Label           
0.918  0.933  0.933  0.903  0.955  0.862  56  4   84  6   *               
0.895  0.895  0.935  0.895  0.935  0.83   17  2   29  2   action-past     
0.815  0.917  0.895  0.733  0.971  0.747  11  1   34  4   action-present  
0.982  0.966  1.0    1.0    0.955  0.959  28  1   21  0   ideation-present

@mikix
Copy link
Contributor

mikix commented Sep 19, 2024

That would mean you couldn't pipe the output to a csv file.

We could try to detect a real tty vs a file output... Or we could print that content out to stderr, but that feels like a misuse of the error stream. But could be a way to get that behavior.

@comorbidity
Copy link
Contributor Author

@mikix OK , I appreciate your point. This is minor point , and the workaround of not using --csv flag is trivial, so we can close this ticket if you like.

@mikix mikix added the enhancement New feature or request label Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants