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

01_Insert_Data.ipynb assumes lab member name will be inserted via nwb #964

Closed
shijiegu opened this issue May 8, 2024 · 1 comment
Closed

Comments

@shijiegu
Copy link

shijiegu commented May 8, 2024

Describe the bug
Just a couple suggestions for the 01 turorial.
In the tutorial, the 01 Insert Data tutorial. LabMember table insertion was not split out clearly. One would assume this is something admin would make for us. But it is not. Also not everyone will insert their own nwb in the first place.

The Team table requires no middle name. Yet the prior/upstream table Lab member Info allows it. It is inconsistent. Remove this requirement if possible in the Team table.

@CBroz1 CBroz1 changed the title 01 Insert Tutorial Notebook Is Still Error-Prone. 01_Insert_Data.ipynb assumes lab member name will be inserted via nwb May 9, 2024
@CBroz1
Copy link
Member

CBroz1 commented May 9, 2024

As an introduction to the package, I think 01 needs to make some assumptions to be digestible for a new user.

So far, this is the first I'm hearing of an assumption that an admin would edit the LabMember table on account creation across the last ~6 accounts I've created. I think doing so goes beyond the scope of our expectations of admin outlined here

@edeno edeno closed this as completed May 16, 2024
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

No branches or pull requests

3 participants