openreview-py icon indicating copy to clipboard operation
openreview-py copied to clipboard

Builder: Set readers of authors group to '~'

Open zbialecki opened this issue 4 years ago • 2 comments

Currently the builder creates the authors group to be public (readers are everyone). It would be preferable if the readers were set to '~' so that only logged in users could access the group.

This would entail changing the __create_group in builder.py to accept another flag and changing the logic that builds the readers array.

zbialecki avatar Apr 07 '20 17:04 zbialecki

@zbialecki from a refactor that Melisa did a while back, all the paper groups, such as paper authors, reviewers, etc are now created by a builder method called create_paper_groups. This method only creates non-public paper author groups as it does not pass any values for the param public when it calls the method __create_group.

So, is this issue still valid or am I missing something?

mohituniyal avatar Apr 14 '20 08:04 mohituniyal

I think the groups you're referring are different. I'm talking about the Authors console group, for example https://dev.openreview.net/group?id=thecvf.com/ECCV/2020/Conference/Authors. This authors group is always set to public even though it should only be accessible by logged in users.

zbialecki avatar Apr 14 '20 13:04 zbialecki