Skip to content

feat: implement dimensional quantities and generated arithmetic - #1

Merged
ScottishTapWater merged 25 commits into
mainfrom
feat/dimensional-quantities
Oct 22, 2026
Merged

ScottishTapWater merged 25 commits into
mainfrom
feat/dimensional-quantities

Conversation

@ScottishTapWater

Copy link
Copy Markdown
Contributor

Summary

  • Implement exact rational dimensional maths, type-level dimension encodings, quantity storage, and SI unit naming/conversions.
  • Generate SI base and engineering catalogue aliases, prefix helpers, and multiplication/division operators discovered from consumer expressions, including scalar arithmetic.
  • Add focused core, discovery, emitter, generator and packaged-consumer tests, including incremental edits and removal of obsolete generated output.
  • Configure NuGet trusted publishing alongside GitHub Packages, use action version tags, and add GoSim/MIT package metadata and a concise README.

This brings the accumulated local implementation onto the initial repository scaffold. Discovery currently handles known quantity operands and nested multiplication/division; roots, powers, dimensionless results and unresolved intermediate-local inference remain follow-up work.

Verification

  • Release solution build passed.
  • 306 core tests, 73 Roslyn tests and 1 packaged-consumer integration test passed.
  • Independent core, generator and test reviews completed.
  • Package metadata, bundled licence/README, workflow YAML and publishing script checks passed.
  • Hosted NuGet OIDC authentication and actual publication have not yet been exercised.

The build reports the known RS1041 warning for deliberately targeting the compiler tooling at .NET 10, plus cached NU1900 audit-connectivity warnings from local restore.

@ScottishTapWater

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-22T20:46:20.642938Z 4ae5b34 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 4ae5b34e0c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ScottishTapWater
ScottishTapWater merged commit a3533fc into main Oct 22, 2026
1 check passed
@ScottishTapWater
ScottishTapWater deleted the feat/dimensional-quantities branch October 22, 2026 21:01
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.

1 participant