create metric invitation and compute stats for the venue #2321
Conversation
openreview/venue/venue.py
Outdated
| self.client.delete_edges(invitation=f'{reviewers_id}/-/Review_Assignment_Count', wait_to_finish=True, soft_delete=True) | ||
| openreview.tools.post_bulk_edges(self.client, review_assignment_count_edges) | ||
|
|
||
| self.client.delete_edges(invitation=f'{reviewers_id}/-/Review_Count', wait_to_finish=True, soft_delete=True) | ||
| openreview.tools.post_bulk_edges(self.client, review_count_edges) | ||
|
|
||
| self.client.delete_edges(invitation=f'{reviewers_id}/-/Discussion_Reply_Sum', wait_to_finish=True, soft_delete=True) | ||
| openreview.tools.post_bulk_edges(self.client, comment_count_edges) | ||
|
|
||
| self.client.delete_edges(invitation=f'{reviewers_id}/-/Review_Days_Late_Sum', wait_to_finish=True, soft_delete=True) | ||
| openreview.tools.post_bulk_edges(self.client, review_days_late_edges) |
There was a problem hiding this comment.
These can be moved outside the assignments_by_reviewers loop right? So we aren't deleting and posting all the edges every time we loop through a reviewer.
openreview/venue/venue.py
Outdated
|
|
||
| all_anon_reviewer_groups = [g for g in all_submission_groups if '/Reviewer_' in g.id ] | ||
| all_anon_reviewer_group_members = [] |
There was a problem hiding this comment.
/Reviewer_ can be replaced with the Reviewers group name.
There was a problem hiding this comment.
can you change it to use "venue.anon_reviewers_name"?
openreview/venue/venue.py
Outdated
| for g in all_anon_reviewer_groups: | ||
| profile = profile_by_id.get(g.members[0]) | ||
| if profile: |
There was a problem hiding this comment.
Sometimes there are no users in the anon group and g.members[0] will error. I'm not sure when this happens, maybe when there's an undeployment or unassignment?
There was a problem hiding this comment.
can you fix it and push the change here?
|
I have posted some changes so that it uses tags instead of edges. However, it requires the following PR to be merged for the tests to pass: https://github.com/openreview/openreview-api/pull/754 |
No description provided.