search

openreview / openreview-py

Official Python client library for the OpenReview API
https://openreview-py.readthedocs.io/en/latest/
MIT License
148 stars 22 forks source link

ARR: AC and Reviewer checklist changes #2197

Closed jkkummerfeld closed 3 months ago

jkkummerfeld commented 3 months ago

The ARR editors have requested a few changes for the reviewer checklists in June (so needed once matching is complete - June 26th).

  1. In the reviewer checklist, under "Need Ethics Review", please change the wording to:

"Should this paper be sent for an in-depth ethics review? Before you answer this question, please refer to https://aclrollingreview.org/ethics-flagging-guidelines/ for guidelines on what papers should and shouldn't be flagged. If your answer is yes, then explain why below, and we will ry to ensure that it receives a separate ethics review."

  1. The questions in the AC/AE checklist should have their wording modified to matching the reviewer checklist.
jkkummerfeld commented 3 months ago

@haroldrubio Following up on this as these will be used in the June cycle very soon.

melisabok commented 3 months ago

@jkkummerfeld for bullet 1 it says explain why below, which field should they explain the paper needs ethics review? ethical_concerns? that field is before needs_ethics_review.

I updated the description for the field in the review and metareview forms: https://github.com/openreview/openreview-py/pull/2212/commits/28fbe1acefc45cc0d95f4126009105a55061b79a

jkkummerfeld commented 3 months ago

Ah, in that case, let's make the text:

"Should this paper be sent for an in-depth ethics review? Before you answer this question, please refer to https://aclrollingreview.org/ethics-flagging-guidelines/ for guidelines on what papers should and shouldn't be flagged. If your answer is yes, then ensure you have explained why in the question above, and we will try to ensure that it receives a separate ethics review."

This is a small edit so I don't think I need to run it by the editors for confirmation.