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.

Sign up for GitHub

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

Jump to bottom

[Feature] Export the evaluation summary table in csv format to the outputs folder #223

Closed
wants to merge 5 commits into from
Closed

[Feature] Export the evaluation summary table in csv format to the outputs folder #223

wants to merge 5 commits into from

Conversation

tushar-badlani
Copy link

  • added export_explanation_csv in options, this flag can be set to true to save explanations in csv in output folder
  • changed evaluation.json in sample 4 to give an example for export_explanation_csv

fixes #221

@Udayraj123
Copy link
Owner

Udayraj123 commented Oct 2, 2024 •
edited
Loading

@tushar-badlani Thanks for the quick PR! Will go through it soon.

Can you please rebase over the latest dev branch? I just saw there's some additional diff and now there's some conflicts introduced after that.

p.s. Please make sure to get the issue assigned to yourself first. There are other folks who may end up simultaneously working on this.

@tushar-badlani
Copy link
Author

@Udayraj123 i merged my commits over latest dev branch

@Udayraj123
Copy link
Owner

@tushar-badlani the diff still seems high. Can you check again?

@Udayraj123
Copy link
Owner

crosslinking #228

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tushar-badlani @Udayraj123