-
Notifications
You must be signed in to change notification settings - Fork 192
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
Adding short note on multiple users' usage #4109
Adding short note on multiple users' usage #4109
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @giovannipizzi . The way you describe it here, we are essentially saying that we don't support multi-user configurations. Having each user install their own instance does not count. If that is what we want to do, I think we should make this more explicit. Something like the following as a first sentence. "Setups with multiple users for a single AiiDA instance is currently not supported".
That being said, even though we don't have a formal multi-user system in place, there are certain ways to have some kind of multiple user setups. I feel it might still be useful to note these here, albeit with a fat disclaimer of the relevant caveats. Would you agree?
680abc0
to
f6afb51
Compare
This will be extended in the future when better multiuser support will be available
Co-authored-by: Sebastiaan Huber <mail@sphuber.net>
f6afb51
to
b775895
Compare
I have accepted the suggestion, and removed the note on UUIDs vs PKs. This is ready to be reviewed again (and merged - I have rebased). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @giovannipizzi some minor changes and then I will merge this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine and I agree with @sphuber 's comments. I'll approve once those comments are addressed and the merge conflict is resolved.
Co-authored-by: Sebastiaan Huber <mail@sphuber.net>
Should be done now |
@sphuber Please merge at your discretion. |
Currently, we state that multiple user setups for single AiiDA instances are not officially supported, and until we do, we do not describe the ways that people have used multiple users.
This will be extended in the future when better multiuser support will be available
Fixes #4005