Skip to content

Remove unicode superscripts #30813

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

lthall
Copy link
Contributor

@lthall lthall commented Aug 3, 2025

I would like to discuss the possibility of still using it for comments in the code as it is much easier to read. This has been used in the EKF and Sub to great effect.

@tpwrules
Copy link
Contributor

tpwrules commented Aug 4, 2025

I think the American way would be "m/s^2" (caret instead of Unicode). "m/s/s" certainly appears strange to my eyes.

@timtuxworth
Copy link
Contributor

It seems like a very antiquated position to be against unicode.

@IamPete1
Copy link
Member

IamPete1 commented Aug 4, 2025

I think the American way would be "m/s^2" (caret instead of Unicode). "m/s/s" certainly appears strange to my eyes.

m/s^2 would be my preference too.

@peterbarker
Copy link
Contributor

m/s/s is closer to what we use for variable names.

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.

6 participants