Skip to content

physical_quantities: treat the angle as a dimension (radians) #22

Open

Description

@m-messer

Problem / use case

Radians are dimensionless in physical_quantities, so conversions such as 1 Hz ↔ 2π rad/s aren't recognised. compareExpressions' test_radians_to_frequency is xfail on every version.

Proposed solution

Treat the angle as a base dimension, as #1 did. #1 was closed without merging, so this issue tracks reopening that work or redoing it on the current code.

Alternatives considered

Special-casing rad in the conversion code.

🤖 Generated with Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions