Skip to content

Add structural engineering unit relations#1674

Merged
angularsen merged 2 commits into
angularsen:masterfrom
actbuildingsystems:feature/structural-engineering-relations
Jul 20, 2026
Merged

Add structural engineering unit relations#1674
angularsen merged 2 commits into
angularsen:masterfrom
actbuildingsystems:feature/structural-engineering-relations

Conversation

@JustASquid

@JustASquid JustASquid commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Motivation

Some generated unit relations that are useful in structural engineering were missing. Adding them makes it possible to express common derived relationships directly with UnitsNet quantities instead of manually converting through raw numeric values.

Changes

  • Add relation definitions for structural-engineering workflows, including density/length to area density, area/area to area moment of inertia, specific weight/volume to force, and acceleration/area density to pressure.
  • Regenerate the affected relation operators and tests.

Validation

  • GitHub PR Build: passing.
  • Azure UnitsNet PR: passing.
  • Local maintainer check: merged with current master, reran code generation with no extra diff, and ran the affected relation test classes on net9 successfully.
  • The only red check is Claude code review, which should not block this PR.

@angularsen angularsen changed the title Added unit relations relevant to structural engineering Add structural engineering unit relations Jul 20, 2026

@angularsen angularsen left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. The added relations are dimensionally consistent, useful for structural engineering, and fit the existing relation-generation pattern.

From my side this is ready to merge.

@angularsen
angularsen merged commit 022e1f3 into angularsen:master Jul 20, 2026
3 of 4 checks passed
@JustASquid
JustASquid deleted the feature/structural-engineering-relations branch July 20, 2026 22:36
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.

2 participants