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

Extensions doc page contains errors #1291

Open
rly opened this issue Aug 28, 2020 · 1 comment · May be fixed by #1292
Open

Extensions doc page contains errors #1291

rly opened this issue Aug 28, 2020 · 1 comment · May be fixed by #1292
Assignees
Labels
category: bug errors in the code or code behavior priority: medium non-critical problem and/or affecting only a small set of NWB users

Comments

@rly
Copy link
Contributor

rly commented Aug 28, 2020

Description

Bug report from the AQuA developers.

The documentation for extensions contains code that does not run.

spec = NWBDatasetSpec('A custom NWB type',
    name='qux',
    attribute=[
        NWBAttributeSpec('baz', 'a value for baz', 'text'),
    ],
    shape=(None, None)
)

The keyword attribute should be attributes. This happens more than once on the page.

The page is also outdated and should point users to the NWB Extensions Catalog and associated tools to create extensions.

@rly rly added the category: bug errors in the code or code behavior label Aug 28, 2020
@rly rly self-assigned this Aug 28, 2020
@oruebel
Copy link
Contributor

oruebel commented Aug 28, 2020

I would also add a link to the YouTube video of your talk ;-)

@rly rly linked a pull request Aug 31, 2020 that will close this issue
5 tasks
@stephprince stephprince added the priority: medium non-critical problem and/or affecting only a small set of NWB users label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: bug errors in the code or code behavior priority: medium non-critical problem and/or affecting only a small set of NWB users
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants