-
Notifications
You must be signed in to change notification settings - Fork 86
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
Update NWB Basics tutorial #1403
Update NWB Basics tutorial #1403
Conversation
Co-authored-by: Oliver Ruebel <[email protected]>
Co-authored-by: Oliver Ruebel <[email protected]>
…per parts similar to HDMF
…developer instructions
Just a heads up. I also made some changes to |
@oruebel Thanks for the heads-up! I'll close this PR and create a new tutorial page focusing on adding behavior data that is not covered yet in the tutorials (Subject, SpatialSeries, Position, etc.) or I can merge your changes here; What do you think? |
@weiglszonja I agree, adding a tutorial focused on behavioral data will be useful. Depending on the scope that tutorial may fit well in the "Domain-specific tutorials" section. BTW, if there are any changes from this PR that you think should remain part of the "NWB File basics" tutorial instead, then please feel free to make those changes in this or a new PR. I mainly wanted to give you a heads up about the other PR in case of merge conflicts. However, since the other PR mainly expands on the intro part of the "NWB File basics" tutorial, I don't think merging the changes should be too complicated. |
# Conflicts: # docs/gallery/general/file.py
@rly can we enable this feature? https://docs.readthedocs.io/en/stable/pull-requests.html I don't have the readthedocs credentials (or don't know them) |
correct an NWBHDF5IO.read link remove the "Adding More Data" section
@rly I made a few more adjustments. Good to go from my end! |
Extends the NWB File Basics tutorial with:
Checklist
flake8
from the source directory.