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]: Clarify form of CompassDirection units #342

Open
3 tasks done
CodyCBakerPhD opened this issue Feb 11, 2023 · 0 comments
Open
3 tasks done

[Bug]: Clarify form of CompassDirection units #342

CodyCBakerPhD opened this issue Feb 11, 2023 · 0 comments
Labels
category: bug errors in the code or code behavior

Comments

@CodyCBakerPhD
Copy link
Collaborator

CodyCBakerPhD commented Feb 11, 2023

What happened?

I made a CompassDirection with units rads instead of radians and the inspector issued a message - note the SI abbreviation is 'rad'

I could also see someone wanting to write "°" in place of "degrees" (and neither Python nor UTF-8 seem to prevent it)

Just wanted to confirm if we wanted to strictly enforce the form of these strings

Thoughts? @bendichter @rly @oruebel

Note: specific instance of more general discussion #208

Steps to Reproduce

No response

Traceback

No response

Operating System

Windows

Python Executable

Conda

Python Version

3.7

Usage

Command Line Interface

Were you streaming with ROS3?

No

Package Versions

No response

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 Feb 11, 2023
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
Projects
None yet
Development

No branches or pull requests

1 participant