search

acl-org / reviewer-paper-matching

Reviewer-paper matching for ACL
BSD 3-Clause "New" or "Revised" License
24 stars 3 forks source link

Major fixes (NAACL 2021) #24

Closed arumshisky closed 3 years ago

arumshisky commented 3 years ago

fix names -> name in evaluate_suggestions.py add python-slugify to requirements.txt

softconf_extract.py: move reviewer/AC/SAC detection into separate functions is_area_chair_from(role), is_senior_area_chair_from(role), is_reviewer_from(role) rename csv_lines to profiles_csv and submissions_csv to improve readability.

suggest_ac_reviewers.py: rm hardcoded moving_to add --moving-to argument and document it

suggest_reviewers.py: fix calc_reviewer_db_mapping call by removing a non-existent author_col warn if bid_file is not specified

suggest_utils.py: fix calc_reviewer_db_mapping documentation reduce verbosity of calc_reviewer_db_mapping

neubig commented 3 years ago

Thanks, these are great!

trevorcohn commented 3 years ago

Thanks Anna. Sorry to leave you with fragile code, but I hope this has been helpful for running NAACL. I see some of the issues are with softconf changing their data (column names), we had this too after ACL, which was particularly annoying.

Best,

-- Trevor Cohn, Professor School of Computing and Information Systems, University of Melbourne P: +61383441732 E: trevor.cohn@unimelb.edu.au W: people.eng.unimelb.edu.au/tcohn

On Fri, 15 Jan 2021 at 02:49, Graham Neubig notifications@github.com wrote:

Thanks, these are great!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/acl-org/reviewer-paper-matching/pull/24#issuecomment-760282791, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRGHIYCREMSJDW4LDYAHKDSZ4HA5ANCNFSM4V73RLCQ .