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

Reorganize root directory #646

Draft
wants to merge 32 commits into
base: master
Choose a base branch
from
Draft

Conversation

ehennestad
Copy link
Collaborator

Motivation

Simplify the root folder and group files logically.

How to test the behavior?

N/A

Todo

  • Update relative paths in workflows
  • Add installation notes: addpath(path/to/matnwb/code)
  • Update relative links in documentation
  • Verify misc.getMatNwbDir works a expected wherever it is used
  • Verify that root directory is always resolved using misc.getMatNwbDir
  • Check that documentation builds as expected
  • Check that tests run as expected

Checklist

  • Have you ensured the PR description clearly describes the problem and solutions?
  • Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
  • If this PR fixes an issue, is the first line of the PR description fix #XX where XX is the issue number?

ehennestad and others added 30 commits December 6, 2024 12:32
…ting

Add "sections"
Specify input and output arguments
Reformat examples
(files are now located in docs/source/_static/html/tutorials)
Fix: Make sure neurodata types specfied with "short names" in the documentation for properties of generated classes are properly linked to their corresponding class pages in the docs i.e (TimeSeries)
Add links to the nwb format specification for each type
- Add docstring processors for adding role to class properties and change formatting for sphinx-autodetected class constructor from reference to constructor method to reference for class
add section from matnwb readme
add pages from nwb-overview
minor rearrangement
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

Successfully merging this pull request may close these issues.

2 participants