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

ERASER scripts for hatexplain in /scripts folder #67

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

SuperMX
Copy link

@SuperMX SuperMX commented Jul 20, 2022

This PR contains my changes in the /scripts folder together with eraserbenchmark and the error inspector as well as the graphical penman editor (streamlit app).

/scripts/evaluate_hatexplain.py : now able to calculate the ERASER metrics! predicts rationals from the rules
/scripts/inspect_hatexplain.py : inspector for FP/FP/TN/TP samples, random+seed and paging mode
/scripts/eraserbenchmark/ : needed by those two

/frontend/graphedit.py : generates UD penman from text on the fly, as well as graph images from UD penman

TODO:
Edit eraserbenchmark so that data input files can be named other than val, test & train
Own output folder for different classification projects (not only /hatexplain)

Copy link
Owner

@adaamko adaamko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it is a good idea to add the whole eraserbenchmark repository here. A better practice would be to download the repository with the script if it is needed.

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.

None yet

2 participants