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

allow setup matching between SACs and submissions #2059

Merged
merged 26 commits into from
Apr 8, 2024

Conversation

celestemartinez
Copy link
Member

@celestemartinez celestemartinez commented Mar 26, 2024

We should allow SACs to be assigned to submissions as well, instead of only to ACs.

This are the changes to be made:

  • add this as an option in the request form. This should allow the PCs to setup the paper matching between SACs and submissions (compute conflicts and affinity scores)
  • add a property in the group.content about assigning ACs or submissions
  • change the SAC console to load the AC tab when the property says they are assigning ACs.
  • Create bid invitation for SACs to bid on papers (make this optional in bid stage form)
  • Fix the deploy when SACs should be assigned to papers
  • Fix any process functions that expect SACs to be assigned to ACs

@melisabok anything else missing?

Requires openreview/openreview-web#1876

@melisabok
Copy link
Member

Create bid invitation for SACs to bid on papers

I'm not sure if we should make this optional in the BidStage form.

Missing:

  1. add a property in the group.content about assigning ACs or submissions
  2. change the SAC console to load the AC tab when the property says they are assigning ACs.

@celestemartinez
Copy link
Member Author

Create bid invitation for SACs to bid on papers

I'm not sure if we should make this optional in the BidStage form.

I was planning on asking PCs in the bid stage form if they would like to enable it for SACs as well. I also don't think this should be the default.

Missing:

  1. add a property in the group.content about assigning ACs or submissions
  2. change the SAC console to load the AC tab when the property says they are assigning ACs.

I've added this two to the description. The first one is already done.

@celestemartinez celestemartinez changed the base branch from master to fix/rebuttal-readers April 1, 2024 16:06
@celestemartinez celestemartinez changed the base branch from fix/rebuttal-readers to master April 1, 2024 16:06
@celestemartinez celestemartinez force-pushed the feature/allow-sac-paper-matching branch from 3f19dcc to fda299c Compare April 1, 2024 16:13
@celestemartinez celestemartinez marked this pull request as ready for review April 1, 2024 20:56
@haroldrubio haroldrubio merged commit 71ad033 into master Apr 8, 2024
1 check passed
@haroldrubio haroldrubio deleted the feature/allow-sac-paper-matching branch April 8, 2024 16:32
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

3 participants