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

[Bug]: Experimenter form edge case #409

Open
3 tasks done
CodyCBakerPhD opened this issue Oct 2, 2023 · 0 comments
Open
3 tasks done

[Bug]: Experimenter form edge case #409

CodyCBakerPhD opened this issue Oct 2, 2023 · 0 comments
Labels
category: bug errors in the code or code behavior priority: low alternative solution already working and/or relevant to only specific user(s)

Comments

@CodyCBakerPhD
Copy link
Collaborator

What happened?

Discovered during the first official alpha test of NWB GUIDE

from nwbinspector import check_experimenter_form
from pynwb.testing.mock.file import mock_NWBFile

nwbfile = mock_NWBFile(experimenter="Doe, John.")
check_experimenter_form(nwbfile=nwbfile)
> None

But this should have been simply Doe, John

Need to check why the regex doesn't flag this; I had thought it would look only for a space before a letter then the period which is allowed for middle initials

Operating System

None

Python Version

None

Were you streaming with ROS3?

None

Package Versions

Copy and paste your output here

Code of Conduct

  • I agree to follow this project's Code of Conduct
  • Have you ensured this bug was not already reported?
  • To the best of your ability, have you ensured this is a bug within the code that checks the NWBFile, rather than a bug in the NWBFile reader (e.g., PyNWB or MatNWB)?
@CodyCBakerPhD CodyCBakerPhD added the category: bug errors in the code or code behavior label Oct 2, 2023
@stephprince stephprince added the priority: low alternative solution already working and/or relevant to only specific user(s) label Nov 6, 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: low alternative solution already working and/or relevant to only specific user(s)
Projects
None yet
Development

No branches or pull requests

2 participants