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

ARR: Fix assignment quotas #2407

Merged
merged 9 commits into from
Dec 11, 2024
Merged

Conversation

haroldrubio
Copy link
Member

@haroldrubio haroldrubio commented Oct 30, 2024

This PR resolves an issue where, if the 3rd edge (out of the Assignment + Invite_Assignment edges) posted to a paper is an Invite_Assignment edge, the process function fails because the pre-process saw it as posting a 4th edge.

The pre-process functions are changed to exclude the posted edge from the edge count if there is already an edge with that ID.

To allow the Assignment_Recruitment process function to complete, only non-accepted and non-declined Invite_Assignment edges are counted. The logic is still preserved because if an Invite_Assignment edge is accepted, then there must be an Assignment edge which are counted

@haroldrubio haroldrubio added the arr Issues related to the ARR workflow label Oct 30, 2024
@haroldrubio haroldrubio self-assigned this Oct 30, 2024
@haroldrubio
Copy link
Member Author

There's a request to be able to increase the quota for some papers where the reviews may have been AI generated and need extra reviewers - I'm thinking of a way to properly implement this

@haroldrubio
Copy link
Member Author

I'll work on a separate request for the new issue I made here #2450, for now we can start with just the fix.

@celestemartinez celestemartinez merged commit d5d540f into master Dec 11, 2024
1 check passed
@celestemartinez celestemartinez deleted the fix/arr-assignment-quotas branch December 11, 2024 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arr Issues related to the ARR workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants